E2E tests using OCI image

Description
Having provided image reference (OCI reference) we fetch resources from the module and run e2e tests against those.
The testing is done by invoking

helm install btp-manager chart
testing existence of resources,
helm uninstall manager in a sequence.

Changes proposed in this pull request:

...

Related issue(s)

/test pull-btp-manager-module-build

/test pull-btp-manager-module-build

/test pull-btp-manager-module-build

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

/test pull-btp-manager-build

/retest


Cherry-pick Pick #34112

Proposed changes
Issue Number: close #xxx

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

run buildall

#34112


https://github.com/HECODAO/HECO123/blob/483a3463e36ba8d26871ba74b9f8b6d068d626b8/audit/HecoDAO-Audit-Report-112921.pdf

If you have a general question, please consider asking on Discord: https://httpie.io/chat

https://github.com/HECODAO/HECO123/blob/483a3463e36ba8d26871ba74b9f8b6d068d626b8/audit/HecoDAO-Audit-Report-112921.pdf


Adulltswim extractor broken

Hi,

[ x] I've verified and I assure that I'm running youtube-dl 2017.09.15

Before submitting an issue make sure you have:

[x ] At least skimmed through the README, most notably the FAQ and BUGS sections
[ x] Searched the bugtracker for similar issues including closed ones

What is the purpose of your issue?

[ x] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

Trying to get new Rick and Morty and this happens:
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--ap-mso=DTV', '--ap-username=PRIVATE', 'http://www.adultswim.com/videos/rick-and-morty/mortys-mind-blowers', '--verbose']
Type TV provider account password and press [Return]:
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2017.09.15
[debug] Python version 3.4.4 - Windows-10-10.0.14393
[debug] exe versions: ffmpeg 3.0.2
[debug] Proxy map: {}
[AdultSwim] mortys-mind-blowers: Downloading webpage
[AdultSwim] sUCOK3V2SNq46JBhJyzBcQ: Downloading XML
[AdultSwim] 1477814: Downloading Provider Redirect Page
[AdultSwim] 1477814: Downloading Provider Login Page
[AdultSwim] 1477814: Logging in
ERROR: Unable to extract post url; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpl1klaw4y\build\youtube_dl\YoutubeDL.py", line 776, in extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpl1klaw4y\build\youtube_dl\extractor\common.py", line 434, in extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpl1klaw4y\build\youtube_dl\extractor\adultswim.py", line 138, in _real_extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpl1klaw4y\build\youtube_dl\extractor\turner.py", line 79, in _extract_cvp_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpl1klaw4y\build\youtube_dl\extractor\adobepass.py", line 1511, in _extract_mvpd_auth
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpl1klaw4y\build\youtube_dl\extractor\adobepass.py", line 1360, in post_form
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpl1klaw4y\build\youtube_dl\extractor\common.py", line 806, in _html_search_regex
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpl1klaw4y\build\youtube_dl\extractor\common.py", line 797, in _search_regex
youtube_dl.utils.RegexNotFoundError: Unable to extract post url; please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

Ah jeez Rick, please help.

Duplicate of #12997.


Support bitcoincash as forkcoin from bitcoin

Support bitcoincash by using the existing 'bitcoin' library, since they share the same API, assuming BitcoinABC client in mind.

Hi, so the only viable change is a change of table name in CREATE TABLE statement? Honestly that seems ad-hoc and unnecessary addition to the code base. print-schema command is meant to help with generating initial SQL statements, which must be then reviewed and executed manually. As it is needed to be done just once it's easy to change the table name manually. Also you would still need to change output table name while exporting (with --output-postgres-table option of export command).
So, indeed the tool does work correctly with Bitcoin forks, in particular we run it with Bitcoin ABC, Bitcoin SV, Bitcoin Gold and Bitcoin Private. However I don't see a necessity to have some special support for forks - from the tool's point of view they all can be treated as Bitcoin.

so, let me step back - is the tool meant to be used for one database per forked coin?

Yes, with one output table per coin (fork or not), if that's what you mean.


[BUG][python] Validation for nullable property

Bug Report Checklist

[x] Have you provided a full/minimal spec to reproduce the issue?
[x] Have you validated the input using an OpenAPI validator (example)?
[x] Have you tested with the latest master to confirm the issue still exists?
[x] Have you searched for related issues/PRs?
[x] What's the actual output vs expected output?
[ ] [Optional] Sponsorship to speed up the bug fix or feature request

Description
When using nullable properties in the openAPI specification, the validation of this property fails with the error Invalid type. Required value type is immutabledict and passed type was NoneType at [...] .
Given the below specification, the created python schema for the AllowedClassification looks like this:
# coding: utf-8

"""
    the test title
    The test REST API  # noqa: E501
    The version of the OpenAPI document: 1
    Generated by: https://github.com/openapi-json-schema-tools/openapi-json-schema-generator
"""

from __future__ import annotations
from openapi_client.shared_imports.schema_imports import *  # pyright: ignore [reportWildcardImportFromLibrary]

AdditionalProperties: typing_extensions.TypeAlias = schemas.NotAnyTypeSchema

from openapi_client.components.schema import attribute_values
AllOf = typing.Tuple[
    typing.Type[attribute_values.AttributeValues],
]

@dataclasses.dataclass(frozen=True)
class AttributeValues(
    schemas.AnyTypeSchema[schemas.immutabledict[str, schemas.OUTPUT_BASE_TYPES], typing.Tuple[schemas.OUTPUT_BASE_TYPES, ...]],
):
    # any type
    all_of: AllOf = dataclasses.field(default_factory=lambda: schemas.tuple_to_instance(AllOf)) # type: ignore

Properties = typing.TypedDict(
    'Properties',
    {
        "attributeValues": typing.Type[AttributeValues],
    }
)

class AllowedClassificationDict(schemas.immutabledict[str, schemas.OUTPUT_BASE_TYPES]):

    __required_keys__: typing.FrozenSet[str] = frozenset({
    })
    __optional_keys__: typing.FrozenSet[str] = frozenset({
        "attributeValues",
    })

    def __new__(
        cls,
        *,
        attributeValues: typing.Union[
            schemas.INPUT_TYPES_ALL,
            schemas.OUTPUT_BASE_TYPES,
            schemas.Unset
        ] = schemas.unset,
        configuration_: typing.Optional[schema_configuration.SchemaConfiguration] = None,
    ):
        arg_: typing.Dict[str, typing.Any] = {}
        for key_, val in (
            ("attributeValues", attributeValues),
        ):
            if isinstance(val, schemas.Unset):
                continue
            arg_[key_] = val
        used_arg_ = typing.cast(AllowedClassificationDictInput, arg_)
        return AllowedClassification.validate(used_arg_, configuration=configuration_)

    @staticmethod
    def from_dict_(
        arg: typing.Union[
            AllowedClassificationDictInput,
            AllowedClassificationDict
        ],
        configuration: typing.Optional[schema_configuration.SchemaConfiguration] = None
    ) -> AllowedClassificationDict:
        return AllowedClassification.validate(arg, configuration=configuration)

    @property
    def attributeValues(self) -> typing.Union[schemas.OUTPUT_BASE_TYPES, schemas.Unset]:
        val = self.get("attributeValues", schemas.unset)
        if isinstance(val, schemas.Unset):
            return val
        return val
AllowedClassificationDictInput = typing.TypedDict(
    'AllowedClassificationDictInput',
    {
        "attributeValues": typing.Union[
            schemas.INPUT_TYPES_ALL,
            schemas.OUTPUT_BASE_TYPES
        ],
    },
    total=False
)

@dataclasses.dataclass(frozen=True)
class AllowedClassification(
    schemas.Schema[AllowedClassificationDict, tuple]
):
    """NOTE: This class is auto generated by OpenAPI JSON Schema Generator.
    Ref: https://github.com/openapi-json-schema-tools/openapi-json-schema-generator

    Do not edit the class manually.
    """
    types: typing.FrozenSet[typing.Type] = frozenset({schemas.immutabledict})
    properties: Properties = dataclasses.field(default_factory=lambda: schemas.typed_dict_to_instance(Properties)) # type: ignore
    additional_properties: typing.Type[AdditionalProperties] = dataclasses.field(default_factory=lambda: AdditionalProperties) # type: ignore
    type_to_output_cls: typing.Mapping[
        typing.Type,
        typing.Type
    ] = dataclasses.field(
        default_factory=lambda: {
            schemas.immutabledict: AllowedClassificationDict
        }
    )

    @classmethod
    def validate(
        cls,
        arg: typing.Union[
            AllowedClassificationDictInput,
            AllowedClassificationDict,
        ],
        configuration: typing.Optional[schema_configuration.SchemaConfiguration] = None
    ) -> AllowedClassificationDict:
        return super().validate_base(
            arg,
            configuration=configuration,
        )

Possible source
We can see that it correctly reference the AttributeValues schema in the class definition of AttributeValues. When checking the schema for AttributeValues  in the corresponding python schema file, we can see it specifies     types: typing.FrozenSet[typing.Type] = frozenset({schemas.immutabledict}).  However, it can also be null in context of the AllowedClassification schema.

openapi-json-schema-generator version
Version 4.1.2

OpenAPI declaration file content or url
openapi: 3.0.0
components:
    examples: {}
    headers: {}
    parameters: {}
    requestBodies: {}
    responses: {}
    schemas:
        AttributeValues:
            properties: {}
            type: object
            additionalProperties:
                items:
                    $ref: '#/components/schemas/SingleAttributeValue'
                type: array
        AllowedClassification:
            properties:
                attributeValues:
                    allOf:
                        -
                            $ref: '#/components/schemas/AttributeValues'
                    nullable: true
            type: object
            additionalProperties: false
        SingleAttributeValue:
            anyOf:
                - type: string
                - type: boolean
info:
    title: the test title
    version: '1'
    description: 'The test REST API'
    contact: {}
paths:
    '/example/get':
        get:
            operationId: GetExample
            responses:
                '200':
                    description: Ok

Generation Details

Steps to reproduce

Related issues/PRs

Suggest a fix

This is not a bug. Please read the definition of allOf in the json schema spec. When allOf is used, all allOf schemas must validate the payload. The payload nullfails validation for that schema. If you want null to be allowed also, my suggestion is to upgrade to 3.1.0 and use oneOf to allow type null or the refed definition AttributeValues.

You are right, my mistake.
We use TSOA to generate the openAPI specification and there is an open bug for this behavior. We have found a small workaround and it works now. Thanks a lot.

You are welcome :)


Install on a shared Linux hosting?

Hello,
Can this be installed in a shared Linux hosting?
Thanks,

Why not?

Hi,
I just don't know how to do this.
Basically, I have a shared linux web hosting which supports python. And basically, I want to be able to hot link youtube videos to players. For e.g.: http://yt.mydomain.com/ytdownloadscript.py?link=https://www.youtube.com/watch?v=1MEK0s7TkmU&res=720p & insert this URL on a media player like XBMC, WMplayer, VLC & play it straight away.
Thanks,

can, someone reopen this issue please?
It just got closed for some reason.

youtube-dl is not a web app. To run it as at web you'll need to write a web frontend for it or use any existing.


IT-2651: Infra takes over deployment of synapse-dev-cmk stack

The synapse-dev-cmk stack sets up the deployer role assumed by the Jenkins system in the dev account, it also sets up the root encryption key used to encrypt the secrets (in SecretMgr) used by the Synapse Stack builder. The key was managed by an IAM role (to be deleted), the policy has been modified to give access to users with admin SSO access, to root (as before), to the deployment role (as before) and to the OIDC role (so we can deploy using infra).
This template was run already from the IAM role to give permissions to SSO/OIDC.
This PR is just to setup the deployment in the infra (Cloudformation should not detect any change).

Replaced by #911


Replace big number title divs with h2 headings

Replace these divs with  headings:
‘Total number of lab-confirmed UK cases’
‘Daily number of lab-confirmed cases’
‘Total number of COVID-19 associated UK deaths in hospital’
‘Daily number of COVID-19 associated UK deaths in hospital’
ACL completed when divs are replaced with equivalent h2 headings

 Replace big number title divs with  headings


Simplify KubernetesAPI

Changes:

Remove AgentManagment because only KubernetesAPI implements it now and there is no need for this abstraction and moved the used functions straight onto KubernetesAPI.
Split off pod template to a separate ContainerVariant implementation used inside of JobVariant and JobTargetedVariant.

is the pod variant actually used or to be used?

is the pod variant actually used or to be used? also I think it might belong in the operator repo as it's only used by it?

JobVariant<PodVariant<'c>> It's currently used inside of JobVariant as well as why I needed to make it generic now


Add support for /api/events

Complementary PR for https://github.com/serlo/api.serlo.org/pull/88

Yeah, we will not implemented this in this repo any more...


asyncio support

#13
Couple of open issues:

Need to understand how to compute the status code as _state is undefined for the servicer_context.
The tests will need a different setup as we will need an aio grpc server which we can't start with pytest.

Hello, @aviklai !
I've been trying to use the async version you've provided in this fork and have been having some difficulties in making it work, would you be able to provide any ideas of what may be the cause?
I use a standard async grpc with reflection:
import asyncio
import grpc
from grpc_reflection.v1alpha import reflection
from concurrent import futures
from py_grpc_prometheus.prometheus_aio_server_interceptor import PromAioServerInterceptor

server = grpc.aio.server(
    futures.ThreadPoolExecutor(max_workers=1000),
    options=[
        ('grpc.max_send_message_length', 400000000),
        ('grpc.max_receive_message_length', 400000000)
    ],
    interceptors=(PromAioServerInterceptor(legacy=True),)
)
start_http_server(9090)
api_pb2_grpc.add_DWHApiServicer_to_server(dwhrc_servicer, server)
service_names = (
    api_pb2.DESCRIPTOR.services_by_name['DWHApi'].full_name,
    reflection.SERVICE_NAME,
)
reflection.enable_server_reflection(service_names, server)
server.add_insecure_port('[::]:50051')

However, when trying to execute a request (sent via grpcurl), the server fails with the following error:
2023-03-26 12:31:30.887 | ERROR    | asyncio.events:_run:81 - Unexpected [TypeError] raised by servicer method [/grpc.reflection.v1alpha.ServerReflection/ServerReflectionInfo]
Traceback (most recent call last):

  File "/app/server.py", line 139, in <module>
    asyncio.run(serve())
    │       │   └ <function serve at 0x7fc1981ff1f0>
    │       └ <function run at 0x7fc1a6c028b0>
    └ <module 'asyncio' from '/usr/local/lib/python3.8/asyncio/__init__.py'>

  File "/usr/local/lib/python3.8/asyncio/runners.py", line 44, in run
    return loop.run_until_complete(main)
           │    │                  └ <coroutine object serve at 0x7fc198491c40>
           │    └ <function BaseEventLoop.run_until_complete at 0x7fc1a6482b80>
           └ <_UnixSelectorEventLoop running=True closed=False debug=False>
  File "/usr/local/lib/python3.8/asyncio/base_events.py", line 603, in run_until_complete
    self.run_forever()
    │    └ <function BaseEventLoop.run_forever at 0x7fc1a6482af0>
    └ <_UnixSelectorEventLoop running=True closed=False debug=False>
  File "/usr/local/lib/python3.8/asyncio/base_events.py", line 570, in run_forever
    self._run_once()
    │    └ <function BaseEventLoop._run_once at 0x7fc1a6484670>
    └ <_UnixSelectorEventLoop running=True closed=False debug=False>
  File "/usr/local/lib/python3.8/asyncio/base_events.py", line 1859, in _run_once
    handle._run()
    │      └ <function Handle._run at 0x7fc1a6515430>
    └ <Handle <TaskStepMethWrapper object at 0x7fc1a4e9b220>()>
  File "/usr/local/lib/python3.8/asyncio/events.py", line 81, in _run
    self._context.run(self._callback, *self._args)
    │    │            │    │           │    └ <member '_args' of 'Handle' objects>
    │    │            │    │           └ <Handle <TaskStepMethWrapper object at 0x7fc1a4e9b220>()>
    │    │            │    └ <member '_callback' of 'Handle' objects>
    │    │            └ <Handle <TaskStepMethWrapper object at 0x7fc1a4e9b220>()>
    │    └ <member '_context' of 'Handle' objects>
    └ <Handle <TaskStepMethWrapper object at 0x7fc1a4e9b220>()>
> File "src/python/grpcio/grpc/_cython/_cygrpc/aio/server.pyx.pxi", line 682, in grpc._cython.cygrpc._handle_exceptions
  File "src/python/grpcio/grpc/_cython/_cygrpc/aio/server.pyx.pxi", line 817, in _handle_rpc
  File "src/python/grpcio/grpc/_cython/_cygrpc/aio/server.pyx.pxi", line 670, in _handle_stream_stream_rpc
  File "src/python/grpcio/grpc/_cython/_cygrpc/aio/server.pyx.pxi", line 472, in _finish_handler_with_stream_responses
  File "/usr/local/lib/python3.8/site-packages/py_grpc_prometheus/prometheus_aio_server_interceptor.py", line 113, in new_behavior
    raise e
  File "/usr/local/lib/python3.8/site-packages/py_grpc_prometheus/prometheus_aio_server_interceptor.py", line 63, in new_behavior
    response_or_iterator = await behavior(request_or_iterator, servicer_context)
                                 │        │                    └ <grpc._cython.cygrpc._ServicerContext object at 0x7fc174e4c940>
                                 │        └ <generator object wrap_iterator_inc_counter at 0x7fc17494d510>
                                 └ <bound method ReflectionServicer.ServerReflectionInfo of <grpc_reflection.v1alpha._async.ReflectionServicer object at 0x7fc19...

TypeError: object async_generator can't be used in 'await' expression

@tesserata Hi,
Never tried this with server reflection.
Can you try it without server reflection?

@aviklai
I wrote an implementation based on yours. After studying the implementation of metrics in other languages, I found differences in the calculation of metrics, please pay attention to my implementation and the difference with yours.
Now at the stage of writing tests.
https://gist.github.com/Roktober/61140e1d817e565d8521082d9a9bba22

@Roktober Hi,
Thanks but the metrics calculation is not my implementation.
I just applied the same implementation in this repo with the async capability support.
Maybe you can create a pull request to this repo with the relevant changes?


Allow windows tool to work from interval midpoints rather than edges.

While you are at it, why not allow negative slop values to narrow (rather than widen) the window?

You can use the slop tool for this, then pass to intersect.


[GHSA-gxg6-rc6c-v673] Improper Input Validation in BeanShell

Updates

Affected products

Comments
Several other components are also affected as a result of cloning or shading. Proof-of-Vulnerability projects with tests to verify the presence of the CVE can be found here: https://github.com/jensdietrich/xshady-release/.

Hey @jensdietrich, sorry for the delay. Same question as https://github.com/github/advisory-database/pull/2258
Can you elaborate on what the results are and how they were generated?

Please see my response for #2258 -- do let me know if more info is needed for this specific PR.


Legend issues

Is there a downside to bumping the font down by a point or two?

Is there an easy way to indicate that the legend layers are clickable? An arrow next to each layer name, or a plus sign? Barring that: small-font words next to/under "Legend" -- something like "Click to expand"

or have them automatically opened (this may be too ugly)

Dropping the font and space is no problem.
I'm attempting to add chevrons to indicate collapsibility, but that made me realize we still had a TON of external dependencies that needed to be brought in (so the tool doesn't suddenly stop working once a website stops hosting the specific version of a file we want).
Looking to do something like this: https://codepen.io/zabielski/pen/ZGLbav


Chapters in branches are not displayed

Steps to reproduce
open Власть книжного червя (Новелла)
Expected behavior
show chapter 1 of branch one when branch one is opened.
show chapter 1 of branch two when branch two is opened.
Actual behavior
Chapter 1 is only available in one of the two branches.
Chapter 1 will not be displayed in the other branch.
LNReader version
v2.0.0-beta.2
Android version
Android 13
Device
moto 20 pro
Other details
lnreader-plugins RanobeLib 2.0.1
lnreader-plugins/ranobelib.ts I don't practice TypeScript and haven't tested the code, but the code does work with branches.
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with a source, I should be opening an issue in the sources repository.
[X] I have updated the app to version 2.0.0.
[X] I will fill out all of the requested information in this form.

If this is an issue with a source, I should be opening an issue in the sources repository.


feat: add message type literal to usage stats

Please describe the purpose of this pull request.
Is it to add a new feature? Is it to fix a bug?
How to test
How can we test your PR during review? What commands should we run? What outcomes should we expect?
Have you tested this PR?
Have you tested the latest commit on the PR? If so please provide outputs from your tests.
Related issues or PRs
Please link any related GitHub issues or PRs.
Is your PR over 500 lines of code?
If so, please break up your PR into multiple smaller PRs so that we can review them quickly, or provide justification for its length.
Additional context
Add any other context or screenshots about the PR here.

previous:
data: {"id":"...","date":"...","message_type":"function_return","function_return":"None","status":"success","function_call_id":"call_xHoMu2jCexf8mqlmfaY2xG5e"}
data: [DONE]
data: {'usage': {'completion_tokens': 31, 'prompt_tokens': 2385
, 'total_tokens': 2416, 'step_count': 1}}

now:
data: {"id":"...","date":"...","message_type":"function_return","function_return":"None","status":"success","function_call_id":"call_xHoMu2jCexf8mqlmfaY2xG5e"}
data: [DONE]
data: {'message_type': 'usage_statistics', 'completion_tokens': 31, 'prompt_tokens': 2385
, 'total_tokens': 2416, 'step_count': 1}

type checker will be fixed by: https://github.com/letta-ai/letta-cloud/pull/415


[Addition]: Scrawny Lace Hare

Hotspot b58 Addresses
112wB94xSJDVkroWqsrQoD2cxDfoiN8FmK3KatGHvR9CUp69MmFD
Discord Handle
No response
Reason(s)
one of the packet forwarding hotspots

One of the devices that can do packet forward. please ban this cheater


Dev ln2

Focused on getting house footprints working, adding merged zooms, included bounds for wms tile requests

there seems to be some kind of bug with the backend and frontend. 1. Frontend is missing the empty map option 2. Backend datapipeline dies with error and database - still should probably merged because the bugs seem to have already existed beforehands


Ftr/add purchases to seed data

Resolves #1197
Description
I added Vendors and Purchases to the seed file. The issues only called for Purchases, but Purchases cannot be created without a valid vendor_id so I added Vendors to the seed file as well. I also added the Faker gem to make it easier to add data for vendors.
Faker is the only dependency.
Type of change

New feature (non-breaking change which adds functionality)

How Has This Been Tested?
I ran the seed file locally and it created all of the vendors and purchases

@armahillo I updated the pr with your suggestions, however,  rails db:seed kept failing because the block in the seeds file that creates StorageLocations only creates records where organization_id = 1. This meant that when organization = Organization.all.pluck(:id).sample returned "2" Purchase.create() would fail. To fix this I passed pdx_org as the first argument to random_record_for_org, ensuring storage_location.id always returns "1" and keeping with the patterns I see in other place in the seeds file.

@rymcmahon Awesome, looks great!


docs: Update documentation to recommend certbot.

This is a first pass at updating our production installer documentation to reflect Certbot support.
I anticipate that we will not be able to merge it without either (1) moving the installer link to point a hardcoded 1.7.0 version of the ReadTheDocs (which might be a good idea anyway) or (2) doing a 1.7.1 release with the new installer support included, which would also be a fine plan, once we've tested this a bit more.
This requires somewhat involved gymnastics in our installer docs, since having the certbot option available substantially changes the priority of some sections of the docs.
@rht @gnprice @adambirds can you take a look at this?
I wanted to do the first version of this update to make sure we move around content sufficiently aggressively, but it's possible that someone else with more time should take over for further editing on this (@gnprice, feel free to just make changes and push edits to my branch).
Some TODO notes:

We could probably make the ALLOWED_HOSTS stuff simpler by allowing the user to specify multiple hosts on the install command line.
We might want to change the way the certbot installer works to, by default, provide their ToS prompts, with an option to not display them, to make sure Certbot's lawyers are happy.  But it's worth looking at what other applications that have integrated certbot into their installer do.

I've just pushed a version that's rebased onto master (including the great docs reorg, and #7285 which added some documentation of this option), and has some additional revisions by me.
Before merging, I think I want to look into at least Tim's TODO above about the Certbot ToS.

OK, on reading up a bit, I think we should just delete the --agree-tos option in our script, assuming that certbot then does something reasonable to present the ToS. It isn't really appropriate for our script to pass that flag with no evidence of the user actually having any knowledge of, or intent to agree to, any such ToS.
I'm travelling starting tomorrow, and don't have right now the 20 minutes it'd require to set up an appropriate test machine and run the script with that change, to see how it behaves. If someone else wants to try it, and report what the interaction looks like, that'd be very helpful. Otherwise it'll be on my list when I'm back in a couple of weeks.


Update to OpenTelemetry 0.8.0-beta.1

I'll take a look at this this morning.  A question about how to work with PRs from forks: is there any way to pull down your branch to my local copy of our repo, or do I need to clone your fork and test from there?


Filter / Search Databases, Tables, Stored Procedures etc. in the Connections Side Bar

Is your feature request related to a problem? Please describe.
It takes time to scroll through hundreds of tables in the Connections Side Bar to find a table and look at column definitions, while I work on a query in the Query Editor.
Describe the solution or feature you'd like
I would like to be able to search or filter the databases, tables, stored procedures etc. within the Connections Side Bar, to quickly find the object I am looking for. An implementation similar to SSMS's (right-click on object) filter would suffice.
Describe alternatives you've considered
I have used the dashboard to search but it doesn't work well (speed wise) and doesn't show column or structure definitions from a drop down.
Additional context
The image below illustrates the view which I would like to filter in the Connections Side Bar.

The image below shows the existing SSMS feature that I would like to see implemented.

This is really a important missing feature, at least for me where govern objects by schema

How we can filter tables like SSMS by right click on Tables folder?

This is an invaluable and seemingly simple feature to implement. A free text field above everything that we can type into and just filters the resulting tree accordingly would be amazing. Please do not relegate it to a context menu like SSMS.

Would love to see some progress on this, a simply function but makes using ADS very difficult to use without.

Would love to see this get in there!

I also logged a ticket like this one,  hoping they add it in.

+1 for this feature. Trying to find a specific field in an unfamiliar database with hundreds of tables is a needle in a haystack. Going to try SSMS instead.

It's hard to believe Microsoft has ignored this for almost 2-1/2 years.  I agree with everything every member has mentioned above.  And having to use the "Manage" tab is painful.  Not having the same functionality from SSMS' object explorer in the Connections pane in AZD is...a pain.  Why would Microsoft remove functionality that so many users have relied on for years is something I don't understand.  But, it's also very typical of Microsoft to just do what they want and ignore what users want.

I use SMMS in combination with a datawarehouse with I don't know like 5000+ database objects. I use the filter functionality in SSMS extensively. Every time scrolling through the list in ADS makes me run back to SSMS.

I had thought I would like to see schema as a subgroup of tables in the tree, but a filter with the ability to select only specific schemas would work as well.

That is one of the reasons why I don't use Azure Data Studio. If an application is not capable of handling common daily tasks, it wont get accepted. And you cant keep installing an extension for everything, even for the most crucial functionality. Some things need to be built-in.

Any update on this enhancement request? It would improve usability massively

I think that maybe only a  connection with the Product group owning this
one will actually be able to provide feedback and prioritize.
But sometimes they seem really unreacheable.
A sábado, 18/03/2023, 14:07, Paul Griffiths @.***>
escreveu:

Any update on this enhancement request? It would improve usability
massively
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/azuredatastudio/issues/12932#issuecomment-1474861077,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AM6OPVYNUFQC3IVVCVTL5RDW4W6TFANCNFSM4SRQFICA
.
You are receiving this because you commented.Message ID:
@.***>

This.
 And one single guy behind HeidiSQL nailed it a decade ago or maybe even earlier.
And SSMS... Well, at least it has some way of filtering tables and columns, but HeidiSQL's implementation is superior IMO - it's right there, just text field above tables list, it works instantaneously and it supports wildcards, so I can type something like chat|user and immediately see tables that I need in the list: chatrooms, chatroom_to_users, users.
Would be so great if ADS finally got this feature...

Filtering was added in #22937 and will be available for the May release.

WHOOO HOOO!  Thank you!


Constructors and fields should explicitly dependes on their top-level decls.

Fixed assertion error when compiling Reflection.aya

bors merge


Demos: Include blender's built in HDRIs?

Blender uses these HDRIs in its default world presets. They are CC0 and all from from HDRI Haven. They are a really good selection and I think it would be super cool if you include them as selectable in the demos where appropriate.
Also, I don't think you should blur environment maps by default :D Blurring softens the shadows, which can be a cool effect, but it's not the expected result of the environment map.

I uploaded a set of HDRIs from Polyhaven to here which should be a better selection:
https://github.com/gkjohnson/3d-demo-data/tree/main/hdri
But I should probably download the 1k versions rather than the 2k versions for the sake of file size. If you want to make a PR into that repo and this one with HDRIs you like I'd be happy to get the demos upgraded!

Also, I don't think you should blur environment maps by default :D Blurring softens the shadows, which can be a cool effect, but it's not the expected result of the environment map.

Yeah the reason they're blurred is because with the random tiny env map highlights cause the models to take forever to resolve. Like hundreds of thousands or multiple millions of samples to get something that looks nice. Getting #4 working would help images resolve much more quickly with unblurred environment maps. Once that's in I'll change the demos to use unblurred environment lighting.
Here's a comparison of a blurred vs unblurred env map at 500 samples

0.35 blur
0.0 blur

A quick update on env map importance sampling. Here's how much more quickly it resolves with "sharp" environment maps now. The images show 20 samples.

Before
After

Whoa - that is great! I can't wait to see it on other demos!

At least on the demos linked from github, environment blur doesn't seem to work. Also: https://gkjohnson.github.io/three-gpu-pathtracer/example/bundle/materialBall.html#transmission the env map doesn't seem to be having an effect on the objects.

Correct that was one sacrifice that had to be made -- "backgroundBlur" now just blurs the background. If you want a blurry env map it should be blurred ahead of time. I'm thinking at some point a utility could be provided to help blur the environment map as needed (see #124)

It does -- I noticed that I forgot to revert a change to the demo this morning so I fixed it. I also added the ability to "blur" the environment map using mip maps but it's probably best to have a more robust solution like #124 in the long run.

Added more env maps to the main demo:


Add Unicode Grapheme Cluster Break Tests

Today we have a series of tests that we wrote. That's great, but as #88 showed we can easily miss things that should have been caught by a relatively simple test case.
Unicode publication UAX #29, "Text Segmentation", attests that their tests do not cover all possible cases. That's fine, all cases isn't the target, but including their recommendations for tests would at least help to ensure that we can pass the basics.
Add the referred to tests in that document for grapheme cluster break to rosed. If any fail, generate tasks to address this and assign them to milestone 1.1.0.

started work on this in branch

These are now added, in file internal/gem/unicode_test.go. however, several of them are currently failing. these must be addressed before this ticket can be closed.


chore(repo): remove snaefull-ui

Description of change
I do not believe we need this package anymore. The release-please job also takes way too long, I'm not sure why. There has been no activity on this subpackage for months. If it is needed, then I think we can revert this commit later or re-implement it nicely.

@d1onys1us The site is still live, but I suppose we can remove it from release-please.
I'd keep the workflows and code, in case we want to update/fix something and create a new release. But removing it from release-please-config.json should do the trick, right?

@d1onys1us The site is still live, but I suppose we can remove it from release-please.
I'd keep the workflows and code, in case we want to update/fix something and create a new release. But removing it from release-please-config.json should do the trick, right?

Yea that would help. But do we plan to keep the site up indefinitely? If not, it could be better to have a sunset plan and totally remove it. Maybe can notify the community and say it's coming down in X days/weeks?


pythonPackages.uamqp: Fix build on darwin

Motivation for this change
https://github.com/NixOS/nixpkgs/pull/109473
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109538 run on x86_64-darwin 1

1 package marked as broken and skipped:

python39Packages.azure-servicebus

5 packages built:

python37Packages.azure-servicebus
python37Packages.uamqp
python38Packages.azure-servicebus
python38Packages.uamqp
python39Packages.uamqp

This is a semi-automatic executed nixpkgs-review which does not build all packages (e.g. lumo, tensorflow or pytorch)
If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 109538 run on x86_64-darwin 1

1 package marked as broken and skipped:

python39Packages.azure-servicebus

5 packages built:

python37Packages.azure-servicebus
python37Packages.uamqp
python38Packages.azure-servicebus
python38Packages.uamqp
python39Packages.uamqp

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

5 packages built:

python37Packages.azure-servicebus
python37Packages.uamqp
python38Packages.azure-servicebus
python38Packages.uamqp
python39Packages.uamqp

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

5 packages built:

python37Packages.azure-servicebus
python37Packages.uamqp
python38Packages.azure-servicebus
python38Packages.uamqp
python39Packages.uamqp

wow, and a patch, thanks @SuperSandro2000 wasn't expecting it to be this much work. But also shame on microsoft on not have CI around darwin.

wow, and a patch, thanks @SuperSandro2000 wasn't expecting it to be this much work. But also shame on microsoft on not have CI around darwin.


Allow install_managed to use environment variables

Feature Type
I'm interested in functionality that solves a current problem or limitation in how CumulusCI works.
Does your feature request solve a current problem?
Currently, installed_managed doesn't support environment variables.
Describe the solution you'd like
install_managed should support environment variables
Describe alternatives you've considered
I am looking into using update_dependencies. This will require create a dependency on a package back to itself. I would prefer to use install_managed.
Additional context
Context: Thread with David Reed on Trailblazer Community: https://trailhead.salesforce.com/trailblazer-community/feed/0D54S00000JgBbw

Hi @davidmreed  - I created the issue. Thank you again for responding back to me on the trailblazer community :)


Unable to Create Student after scaffolding

Description
I've been following the tutorial step by step, but seem to be running into some weird issues.
I'll create separate issues for them, because they're about different steps, but I get similar behaviour.
First of all on this document:
https://learn.microsoft.com/en-us/aspnet/core/data/ef-rp/intro?view=aspnetcore-7.0&tabs=visual-studio#scaffold-student-pages
The scaffolding works, the changing of the references from _context.Student. to _context.Students.works. However, I find 7 references of_context.Student.and 14 references of_context.Student`, I need to change all of them to work.
Everything said and done, I get to step
https://learn.microsoft.com/en-us/aspnet/core/data/ef-rp/intro?view=aspnetcore-7.0&tabs=visual-studio#test-the-app
When going to the create page, I get the form correctly, but the Create button doesn't do anything.
Now since I'm still learning about C# , my colleague helped me track down the 'problem' to the Create.cshtml.cs file:
if (!ModelState.IsValid || _context.Students == null || Student == null)
            {
                return Page();
            }

The ModelState.IsValid always comes back as false, causing the button to return to return Page();, never getting to the _context.Students.Add(Student) block.
When changing the default block provided by scaffolding to
if ( _context.Students == null || Student == null)
            {
                return Page();
            }

Everything works as expected.
It would be great if this would either be fixed or documented on what to do there, because if not for my colleague I would've given up on the tutorial right there.

Extra info
I've also removed the Pages/Students folder complely and re-did the scaffolding again, which immediately "fixed" having to change everything from Student to Students, but still caused issues with the Create button.
I've also restarted the project on 2 different machines and run in to the exact same problems.
Page URL
https://learn.microsoft.com/en-us/aspnet/core/data/ef-rp/intro?view=aspnetcore-7.0&tabs=visual-studio
Content source URL
https://github.com/dotnet/AspNetCore.Docs/blob/main/aspnetcore/data/ef-rp/intro.md
Document ID
fa488196-1f98-08ea-adcc-17d9dac86727
Article author
tdykstra

tracked it down with my colleague:
is caused by the nullable entry for Enrollments property under the Students class.
It seems I had missed this little gem under the Courses documentation:

To eliminate the warnings from nullable reference types, remove the following line from the ContosoUniversity.csproj file:

<Nullable>enable</Nullable>

The scaffolding engine currently does not support nullable reference types, therefore the models used in scaffold can't either.


onBlur is not working for all input types

Describe the bug
It is common to have onBlur (when in form, you focus away from the input) event to fire, usually the payload is same as the onChange, but for form validation, where focusing away from an input generates an error if the field is required.
To Reproduce
Steps to reproduce the behavior:

Go to any component'
Add onBlur prop
See error: Warning: Failed prop type: DateRangePicker: unknown props found: onBlur

Expected behavior
A onBlur event is fired.

semantic-ui-react
semantic-ui-css (or any alternative)
semantic-ui-calendar-react

Is there onBlur prop on semantic-ui-react's Input element? As far as I know there isn't, correct me if I'm wrong.
This module is build on top of semantic-ui-react and deals with only date-picker related stuff. Handling input events is a low level task for this module and it relies on semantic-ui-react in things like that.

Hi
I would like to use semantic-ui-calendar-react component in forms like React Final Form, but I cannot since field validation never occurs.
Form Field validation occurs when onBlur event is fired.
It seems that InputView::onBlur() only calls its internal "closePopup()" function here but does not forward onBlur event.

solved in #171


[llbuild3] Add support for passing a TCASDatabase to TEngine.

Exposed the existing makeExtCASDatabase function to Swift via SwiftAdaptors.hpp. Added a casDB parameter to TEngine's main initializer, and used makeExtCASDatabase to make the necessary CASDatabaseRef when casDB is not nil.
This matches the behavior of the TActionCache? parameter to TEngine's main initializer.
This commit resolves #959.

@swift-ci please smoke test


restore otlp/proto submodule

Now this temporally uses kakkun61/git-hooks.nix. After a PR https://github.com/cachix/git-hooks.nix/pull/494 will be merged, I will switch to use the original.

Close because of #158


Remove some of the re-exports within the core crate

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
There has been a lot of refactoring recently in order to implement subquery support and also to move the SQL query planner and the logical optimization rules into their own crates.
As a result, we now have many re-exports in the core crate that were implemented to reduce the size of PRs. I would like to propose removing these re-exports prior to the 9.0.0 release as part of stabilizing the public API for DataFusion.
Describe the solution you'd like
Remove many re-exports and update imports to use the real locations.
Describe alternatives you've considered
None
Additional context
None

I now plan on working on this after the 9.0.0 release


Who are the maintainers?

Following this old issue:
https://github.com/indexzero/ps-tree/issues/10
Who are the current maintainers of this package?
For what I saw:
@indexzero is the author (can push on the repo and can publish on npm)
@nelsonic is one of the maintainers  (can only push on the repo)
Am I missing someone?

Can we add more contributors to this repo?
https://github.com/indexzero/ps-tree/issues/18#issuecomment-373294868

cc @zixia @soyuka

We maybe could release a community fork? I've a pending fix on @zixia 's fork also haha

Let's see what @nelsonic says about that.
@soyuka are you referring to this https://github.com/zixia/ps-tree/pull/1?
Can you create the PR on this repo too?

Can you create the PR on this repo too?

It's based on @zixia ´s work on the repository so I guess we need to merge his changes first :|

Oh right!
So let's make a single PR with both changes.

@soyuka Let me merge your change in my PR, then I believe my PR at here will include your PR....
:-/

@zixia can you open another PR removing the addition of appveyor?
We have already https://github.com/indexzero/ps-tree/pull/29 for that.

@simonepri Ok no problem.
I had just modified my PR and remove the README&Appveyor changes.
Please let me know if you think I should modify other parts as well.

Thank you! I've commentend under the PR.

@indexzero See also this
https://github.com/indexzero/ps-tree/pull/28#issuecomment-373663239

FTR merged #17 #24 #27 and #28 to https://github.com/soyuka/ps-tree

@soyuka Lets wait @nelsonic before proceed please.
I don't see any advantage of working on a community fork if we can push on npm from this one.

@simonepri  I don't want to bother anyone ;) I'm just using this in pidusage-tree, and therefore will use my fork for the time being :). Obviously if things gets handled here I'll use the original version.
https://twitter.com/andrestaltz/status/961600427832872961

@nelsonic @indexzero news?

@simonepri pinging them is useless just let them answer if they want to I'm sure they've been notified by now ;). Note that my last contributions to this were almost 1 year ago and there has been no answer yet.

Actually nelsonic have already merged some of my PRs some days ago:
https://github.com/indexzero/ps-tree/pull/26
https://github.com/indexzero/ps-tree/pull/25

But it seems that @nelsonic only merge Badges...

@simonepri I saw your pidtree npm, and thanks for adding me to the collaborators.
It seems we have to do a hard fork for this module finally...

@huan @simonepri if you'd like to discuss merging back into a<EMAIL_ADDRESS>API please let me know. My apologies for not responding sooner. This was on my to-do list, but kept getting pushed back.

Updating the title of this issue to reflect the current discussion.

@indexzero pidtree is a from scratch re-work of this package not 100% sure that having it merged back here is the right choice. @soyuka

@simonepri exactly! And it has an improved algorithm that is more correct.
This module is downloaded about 5,000,000 times a month. Wouldn't it be great if all those folks got a better quality module?
By releasing it as 2.0.0 it gives folks an upgrade path that is opt-in as well.
If you are interested I would add you as an owner and a maintainer on nom. Let me know! ^_^

@indexzero it's great to see you wake up and come back to us! :D

@indexzero Your point is right.
I'm still not 100% happy with the implementation of pidtree and I would like to do some rework before starting to think about merging it back here.
My idea is to create separate packages for each platform and then to have pidtree or ps-tree to be just a wrapper for the right implementation.
Also, I wold like to add a more sophisticated cross-platform testing though docker.
The ETA of this would be at least a month, since I'm super busy right now.

@simonepri that makes sense. Given your past excellent work on PR #24, #27, and #29 I've added you as a collaborator to this project. When you accept that invite I am happy to make you a full Administrator.
Once you've reached a place you're happy with from an implementation perspective please let me know and I will add you as an owner on npm.

So it turns out that for personal repositories they only the person they are associated with can be administrators. I am going to move this over to the foreverjs org since it's heavily depended on by that package –– unless there are other thoughts of course.


AKS Deploy helper - AKS Private Cluster

Describe the bug
Getting a deployment error when I use the deployment helper to deploy a private AKS cluster (without Sample APPs).
To Reproduce
Steps to reproduce the behavior:

Go to 'https://azure.github.io/AKS-Construction/...'
Click on '"I want a managed environment"'
Scroll down and select 'Private cluster with isolating networking controls'
Click on '"Networking Details"
Select  "Deploy Azure firewall for your cluster egress (Custom VNet Only)"
Select '"Use Azure Bastion to facilitate RDP/SSH public internet inbound access into your virtual network"'
Select "Uses a private IP address from your VNet to access your dependent Azure service, such as Azure KeyVault, Azure Container Registry etc"
Select "CNI" network plugin
Select "Custom Networkin"
All other options/selections remain default (no changes)

Expected behavior
A successful deployment of a private AKS cluster through Azure CLI
Screenshots

If applicable, add screenshots to help explain your problem.
Additional context
P.S. Everything works fine when I use the 0.4.5 template.

Duplicate of #210
PR is raised awaiting approval #211


Right align not showing correctly on Menu items

Since this update was published the register, login & logout menu items (pull-md-right) sits on the left hand side of the page.  This can also be seen on your live version.  What has changed?

@kiwiingenuity not sure if this is because of any code change or an environment related issue. When you run the website in dev, it looks ok, Only when you deploy the site to production or run locally in production mode (that includes minified version of packages) the right hand menu moves to the left.
Nothing obvious wrong at the moment, but will investigate this.

I do not have the same experience as yourself.  When I'm running in dev mode the menu is still not aligning correctly.

Oh yeah, you were right. Just fixed it and pushed the change, see the change in header.component.html. I think it was a change in newer version of bootstrap which broke the layout.

Thanks that resolved the reported issue!
However the extra  tag at the top of the page is confusing and the mobile menu does not appear to be working correctly?

ouch... forgot to save the file after few checks I guess. Pushed the change now.
I will have a look at the mobile menu. Would you mind raising another one please?
Thanks.


Audit for ceilf*

Description of changes:
Restores and completes the audit for the ceilf32 (ceilf) and ceilf64 (ceil) intrinsics.
Resolved issues:
Part of #1163
Part of #1025
Call-outs:

Testing:

How is this change tested? Adds 2 tests, removes 2 tests.

Is this a refactor change? No.

Checklist

[x] Each commit message has a non-empty body, explaining why the change was made
[x] Methods or procedures are documented
[x] Regression or unit tests are included, or existing tests cover the modified code
[x] My PR is restricted to a single feature or bugfix

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 and MIT licenses.

Updated the tests to follow a similar style as the ones in #1250 (some concrete cases and two properties). In this case, however, the property 0 <= |x - ceil(x)| < 1 does not hold for all cases (very small numbers cause it, and the behavior is reproducible in Rust), so we need to test 0 <= |x - ceil(x)| <= 1.


Disable test which ruins the test runner

Description:
Disabling this test restores our test runner, so it executes all the other tests.
Naturally we should fix this test too, but it is still better to skip this single test until then.

Are the failing browser tests on this branch another issue?


Support Merging Recommendations instead of "Last one wins"

According to the docs, when multiple recommendations files recommend a version on a single dependency, the "last one wins".
Please consider supporting the actual merging and conflict resolution of multiple recommendations such that the highest version wins. This new behavior can be governed by a property if needed.

Fixed in 4.5.0 https://github.com/nebula-plugins/nebula-dependency-recommender-plugin/commit/893c5bdb231b97a85718a346e4295f2bd4345650


Chunk to content on DITA Map fails when the DITA Map has spaces in file name

I have a DITA Map with spaces in its file name "flowers by season_map.ditamap" which has chunk=to-content set on the root element.
Publishing it to HTML5 fails with:
    /.../plugins/org.dita.base/build_preprocess.xml:260: java.lang.IllegalArgumentException: Illegal character in path at index 7: flowers by season_map.dita
at java.base/java.net.URI.create(URI.java:906)
at java.base/java.net.URI.resolve(URI.java:1089)
at org.dita.dost.reader.ChunkMapReader.chunkMap(ChunkMapReader.java:209)
at org.dita.dost.reader.ChunkMapReader.process(ChunkMapReader.java:136)
at org.dita.dost.writer.AbstractDomFilter.read(AbstractDomFilter.java:41)

flowers22.zip

@raducoravu, very interesting. Thank you for reporting this. I recently used flowers.ditamap as a test case for something and it failed. I hadn't investigated yet, but this might be the explanation.

@lief-erickson our original flowers.ditamap does not have submaps, I just created a submap with spaces inside its file name to reflect this problem and attach a sample project to the issue. So it's possible your test case failed because of something else.


fix: 🐛 call  onElement from componentDidMount

Build version: 2.13.0-pr-44.173 :crossed_fingers: fix-modal on Travis :tada:

:tada: This PR is included in version 2.13.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


feat: add _TZ3000_akqdg6g7 to TS0201 humidity sensor

Closes #1040

Hi and thanks for info @omBratteng, I have added the data to the upcoming release.


chore: update linting

[ ] Ensure that the stylelint configuration is still valid and that there is still a need for the stylelintignore file.
[ ] Determine whether there is a need for any additional linting
[ ] Figure out the stylelint-a11y debacle

Closed by #69


fix: watch update was comparing new object with new object so no new rows would be written

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

Needs approval from an approver in each of these files:

OWNERS

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

/approve

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

Needs approval from an approver in each of these files:

OWNERS [rawlingsj]

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


(AzureCXP) MCAS Update

Based on customer feedback from the below GitHub thread, removing MCAS from limitation of features available for Azure US Gov.
https://github.com/MicrosoftDocs/azure-docs/issues/51829
Currently MCAS is available for GCC High as per below documentation https://docs.microsoft.com/en-us/enterprise-mobility-security/solutions/ems-cloud-app-security-govt-service-description, not sure if that can be generalized for this document as Azure Gov.
Please review and revert back for any queries.

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

#sign-off


Incorrect output reuse

Hello, for a architecture with 3-levels of memory (L0, L1, L2) and 16x16 PE array, following is one of the mappings that I got.
{'W': [[(4, 13), (6, 2), (6, 2), (6, 2), (6, 2)], [(6, 12), (2, 5), (3, 13), (5, 3)], [(1, 5)], [(5, 64)]], 'I': [[(4, 13), (6, 2), (6, 2), (6, 2), (6, 2)], [(6, 12), (2, 5), (3, 13), (5, 3)], [(1, 5)], [(5, 64)]], 'O': [[(4, 13), (6, 2), (6, 2), (6, 2), (6, 2)], [(6, 12), (2, 5), (3, 13), (5, 3)], [(1, 5)], [(5, 64)]]}

The mapping has levels: [spatial, L0, L1, L2]
The statistics reported shows that the reuse of "output" operand at L0 is 4800. To me that does not seem quite right. Could you please help me out?

Sure. Could you upload or email me<EMAIL_ADDRESS>the corresponding output report file (the .xml file)?


Layer overlay - styling

Proper styling for the layer selection

Map preview + name? React to connection changes?


Webpack - Cannot resolve module 'marvelous-aurelia-core/*'

When the project is run via WebPack, there are a series of errors generated:
Cannot resolve module 'marvelous-aurelia-core/utlis'
Cannot resolve module 'marvelous-aurelia-core/aureliaUtils'
Cannot resolve module 'marvelous-aurelia-core/optionsReader'
Cannot resolve module 'marvelous-aurelia-core/pubsub'
Cannot resolve module 'marvelous-aurelia-core/compiler'
These appear to be down to the differences in module resolution and location between WebPack and SystemJS (As used by JSPM).
@PWKad - have you experience with this, and resolving it?
It's very easy to reproduce - if you open the main Aurelia WebPack skeleton project, and run the following commands, it'll show you the errors:
npm install marveloussoftware/marvelous-aurelia-core --save
npm install marveloussoftware/marvelous-aurelia-query-language --save
npm intsall marveloussoftware/marvelous-aurelia-grid --save

webpack -v

(The webpack -v shows verbose errors, so you can see why and where it failed - in this case, trying to find modules that don't exist).

Did you try to use any other library, preferably some which is already proven to be working on Webpack? Maybe it is just a WebPack configuration issue?

Yeah, we've got quite a complex set of build-tooling on this project.  Other libraries work.
I've managed to get those to load by aliasing them, however then the actual functionality fails... I'm going to put together a repro against the basic Aurelia Skeleton, get it as far as I can and hopefully someone can figure out where the actual issue lies.  Thanks for the help on this!

I'll try to fix it later today or tomorrow, but since I never used WebPack then I cannot make any promises.

I appreicate it - I've been on quite a learning curve with the WebPack environment too :D

I've created a fork. It's got two tags InstallErrors, which shows what happens after you just install and reference in main.ts, and then PostAliasErrors which shows what happens after adding the aliases to the webpack.config.js (Again, I'm definitely not a WebPack expert, so that may be the wrong thing to do, but it resolve the first batch of errors, so it seemed right).

This issue is similar to https://github.com/aurelia/dialog/issues/127. It looks like the imports for css are not compatible with WebPack.

I have a similar issue, after following the installation instructions:
webpack-dev-server.js:3 ./~/marvelous-aurelia-grid/dist/commonjs/grid/column.js Module not found: Error: Can't resolve 'marvelous-aurelia-core/utils' in '/Users/donpeterkofsky/Documents/gc-2-dev/node_modules/marvelous-aurelia-grid/dist/commonjs/grid' resolve 'marvelous-aurelia-core/utils' in '/Users/donpeterkofsky/Documents/gc-2-dev/node_modules/marvelous-aurelia-grid/dist/commonjs/grid'   Parsed request is a module   using description file: /Users/donpeterkofsky/Documents/gc-2-dev/node_modules/marvelous-aurelia-grid/package.json (relative path: ./dist/commonjs/grid)     Field 'browser' doesn't contain a valid alias configuration   after using description file: /Users/donpeterkofsky/Documents/gc-2-dev/node_modules/marvelous-aurelia-grid/package.json (relative path: ./dist/commonjs/grid)     resolve as module       /Users/donpeterkofsky/Documents/gc-2-dev/node_modules/node_modules doesn't exist or is not a directory       /Users/donpeterkofsky/Documents/node_modules doesn't exist or is not a directory       /Users/donpeterkofsky/node_modules doesn't exist or is not a directory       /Users/node_modules doesn't exist or is not a directory       /node_modules doesn't exist or is not a directory
Seems like there are some hard-coded bad paths to node_modules - it is looking everywhere except in the actual project directory.


SH-I-178: support using external JRE instead of the embedded one

Description
support using external JRE instead of the embedded one
Issues Resolved
https://github.com/chef/chef-server/issues/2219
Check List

[ ] New functionality includes tests
[ ] All buildkite tests pass
[ ] Full omnibus build and tests in buildkite pass
[x] All commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/master/CONTRIBUTING.md#developer-certification-of-origin-dco
[x] PR title is a worthy inclusion in the CHANGELOG

While I appreciate the effort you've put into this issue (and PR), it's not something that we'd like to include in the Chef Infra Server. Chef Infra Server is a collection of many individual services, which interact with each other in complex ways. We take great effort to ensure that each of those components are fully tested including the bundled release of Java. Before we ship a release with a new version of Java, the entire package undergoes an extensive end to end testing process to make sure we don't introduce potentially impacting regressions. Allowing versions of Java that are not fully validated would most likely introduce hard to detect and potentially problematic failures for users.
If there are particular issues with our Java configuration or version of Java that we ship I'd highly encourage you to raise those problems, but we do want to make sure that folks consuming the Chef Infra Server package are always running a validated release of Java.

While I appreciate the effort you've put into this issue (and PR), it's not something that we'd like to include in the Chef Infra Server. Chef Infra Server is a collection of many individual services, which interact with each other in complex ways. We take great effort to ensure that each of those components are fully tested including the bundled release of Java. Before we ship a release with a new version of Java, the entire package undergoes an extensive end to end testing process to make sure we don't introduce potentially impacting regressions. Allowing versions of Java that are not fully validated would most likely introduce hard to detect and potentially problematic failures for users.
If there are particular issues with our Java configuration or version of Java that we ship I'd highly encourage you to raise those problems, but we do want to make sure that folks consuming the Chef Infra Server package are always running a validated release of Java.

Thanks for your response @tas50, I ran into a situation that requires me to update JDK due to security vulnerabilities with the embedded JDK version. I'll create another issue to give details.

Thanks for your response @tas50, I ran into a situation that requires me to update JDK due to security vulnerabilities with the embedded JDK version. I'll create another issue to give details.


[turborepo] Error using node:18-alpine Docker image: thread 'main' panicked at 'Failed to execute turbo...'

What version of Turborepo are you using?
1.7.4
What package manager are you using / does the bug impact?
npm
What operating system are you using?
Linux
Describe the Bug
When installing and running Turborepo in a node:18-alpine Docker container, this error is raised upon running ./node_modules/.bin/turbo:
thread 'main' panicked at 'Failed to execute turbo.: Os { code: 2, kind: NotFound, message: "No such file or directory" }', crates/turborepo/src/main.rs:50:10

Expected Behavior
Executing ./node_modules/.bin/turbo should not exit with this 101 error.
To Reproduce
$ docker run -it --entrypoint /bin/sh node:18-alpine
/ # mkdir app
/ # cd app
/app # npm install<EMAIL_ADDRESS>
added 2 packages in 3s
npm notice
npm notice New minor version of npm available! 9.3.1 -> 9.5.0
npm notice Changelog: https://github.com/npm/cli/releases/tag/v9.5.0
npm notice Run npm install -g<EMAIL_ADDRESS>to update!
npm notice
/app # ./node_modules/.bin/turbo
thread 'main' panicked at 'Failed to execute turbo.: Os { code: 2, kind: NotFound, message: "No such file or directory" }', crates/turborepo/src/main.rs:50:10
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace
/app # echo $?
101

Reproduction Repo
No response

I was able to resolve this using the -slim image instead. This was a useful post: https://medium.com/swlh/alpine-slim-stretch-buster-jessie-bullseye-bookworm-what-are-the-differences-in-docker-62171ed4531d
Found this helpful comment: https://github.com/vercel/turbo/blob/dd85f00f8efc6bd2c78f6b0106a0e409992d6141/examples/with-docker/apps/api/Dockerfile#L5
Closing for now, but perhaps it would be helpful for Turborepo to produce a more helpful error message if it's installed on a system where libc6-compat is missing.

For future viewers: I just had this issue and adding RUN apk add --no-cache libc6-compat to the Dockerfile fixed it. So, no need to reach out for slim. (reference)


POM declaration

Hello,
Firstly, thank you for this library.
I have tried to add it to my project with my local repository maven, but when I try to use it in my project, but impossible to import all classes like OnvifDevice, etc.

However, the library is well present in the m2/repositoty/de/onvif/onfif/1.0/onvif.jar.

And I have declared it in my pom.xml

Do you have an idea?
Thank you for your support

I do not support Maven, please search for a resolution on your own.


[Bug]: Type for output_cls in BaseSynthesizer.init() should be Optional[Type[BaseModel]]

Bug Description
In llama_index/response_synthesizers/base.py, the argument output_cls is currently typed as BaseModel:
    def __init__(
        self,
        service_context: Optional[ServiceContext] = None,
        streaming: bool = False,
        output_cls: BaseModel = None,
    ) -> None:

I think it should be Optional[Type[BaseModel]] instead:
    def __init__(
        self,
        service_context: Optional[ServiceContext] = None,
        streaming: bool = False,
        output_cls: Optional[Type[BaseModel]] = None,
) -> None:

To make the type checker happy, line 123 should be changed from:
        if isinstance(response_str, self._output_cls):

to
        if self._output_cls is not None and isinstance(response_str, self._output_cls):

Version
0.8.48
Steps to Reproduce
I wrote this code:
class SynthesizedResponse(BaseModel):
    answer: str
    attributed_passages: List[str]
    answerability_probability: float

class MyTextSynthesizer(BaseSynthesizer):
    def __init__(self, **kwargs: Any):
        super().__init__(output_cls=SynthesizedResponse, **kwargs)

Relevant Logs/Tracbacks
No response

Good catch, I'm not sure why our linter didn't catch this.

It’s not a big issue. Just minor annoyance with the type checker. Since I discovered it and it’s not a big deal to fix, please assign it to me. I’ll be happy to provide a PR to fix it.
I tried to assign it to myself, but the github UI won’t let me.


window.location.reload()

It is not supported in IE9 and below:
https://social.msdn.microsoft.com/Forums/ie/en-US/de9c5845-5fac-4fda-9b2d-2b435e805abd/windowlocationreloadtrue-is-not-working-in-ie-9?forum=ieextensiondevelopment
Supported by all other modern browsers.

It is supported in IE9. https://developer.mozilla.org/en-US/docs/Web/API/Location/reload


runtime/trace: custom trace events

Wasn't there an issue for this? Did I imagine that?


[WIP] Bump API version to 1.9 and add some additional features.

As discussed in issue #1, I would like to add function for reading historical events.
Unfortunately I was unable to obtain mobile client using API 1.8 and in API 1.9 the format of messages/responses is little bit different. So I decided to bump API version to 1.9 and rewrite existing functions.
I did only basic tests, but for now it works well.
What is left untouched is login function, I was unable to migrate it to new endpoint userAuthorize.json.

@OttoHollmann thank you for your contribution! Shall I review and test it? I have a few comments, do you prefer to do a review or want me to adapt where needed?

I would like to fix the login function and also add/update some tests. So you can only review it and I will fix your objectinos.

Thank you for your contribution! I have merged it to my dev branch and will make changes there, will tag you in the PR.


Update site with new colors and logo

Update to the new colors and logo using the RevUC fall 2016 branding pack :raised_hands:
Also update heading/body fonts to Ubuntu to match marketing assets: https://fonts.google.com/specimen/Ubuntu

Have local updated copy, will push to master by this evening. It's almost done


ImportError:cannot import name 'current_app'

When I try to run this file. The error that comes up is: ImportError:cannot import name 'current_app'. Detials as follows:
Traceback (most recent call last):
File "G:\Anaconda\lib\site-packages\werkzeug\http.py", line 22, in
from email.utils import parsedate_tz
File "C:\Users\lenovo\Desktop\flasky-master\app\email.py", line 2, in
from flask import current_app, render_template
ImportError: cannot import name 'current_app'
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:/Users/lenovo/Desktop/flasky-master/app/init.py", line 1, in
from flask import Flask
File "G:\Anaconda\lib\site-packages\flask_init_.py", line 17, in
from werkzeug.exceptions import abort
File "G:\Anaconda\lib\site-packages\werkzeug_init_.py", line 152, in
import('werkzeug.exceptions')
File "G:\Anaconda\lib\site-packages\werkzeug\exceptions.py", line 71, in
from werkzeug.wrappers import Response
File "G:\Anaconda\lib\site-packages\werkzeug\wrappers.py", line 26, in
from werkzeug.http import HTTP_STATUS_CODES,
File "G:\Anaconda\lib\site-packages\werkzeug\http.py", line 24, in
from email.Utils import parsedate_tz
File "C:\Users\lenovo\Desktop\flasky-master\app\email.py", line 2, in
from flask import current_app, render_template
ImportError: cannot import name 'current_app'
Any idea ? Thanks !

This is an odd error to get. Do you have Flask installed in your environment? My first thought is that your installation is somehow corrupted and missing stuff.

Well,I do have Flask installed in my environment(ubuntu16), and I use virtualenv to install relative packages using the requirement.txt. And pip shows that except for six==1.4.1 is versionConflict(the default six i have is 1.10),everything else installed successfully.
However, it still doesn't work :(
Is there anything else may contribute to it? Really Thanks!

I think the best way to confirm that your virtualenv is correct is to trash it and make a brand new one. Also make sure that when you run your application, you either activate the virtualenv, or else use the python interpreter from the virtualenv, not the global one.

Got it to work.
Thanks again for the reply Miguel~


Error: spawn EINVAL

Recently I had noticed that sniffer had stopper working. Although fixer did work.
I started debugging the extension and found that I am getting this error:
rejected promise not handled within 1 second: Error: spawn EINVAL
extensionHostProcess.js:148
stack trace: Error: spawn EINVAL
	at ChildProcess.spawn (node:internal/child_process:421:11)
	at spawn (node:child_process:799:9)
	at validate2 ({DIR}\out\main.js:6648:55)

So when the sniffer.ts calls spawn the process fails because it seems that something changed in windows OS https://github.com/nodejs/node/issues/52554
The current solution is to add { shell: true } option in the  spawn function.

Same problem after the latest vscode update.

same here, roll back to VSCode version 1.91, it works fine

Thanks, everyone, for reporting; I will be looking into this in the coming week.


Fix Errors in Primitives

Summary
This PR changes QiskitError to ValueError.
Details and comments

@woodsp-ibm I think you reported this issue. Could you review this PR?

It seems that there are some fidelity tests for mismatched left and right circuits that are testing the sampler usage by fidelity to presumably ensure its catching the expected error when it calls run. https://github.com/Qiskit/qiskit-terra/blob/4ac8b24af1cf6f99a418b5f118c29661f89a2b2c/test/python/algorithms/state_fidelities/test_compute_uncompute.py#L141 While the job.result() call is in a try catch block where it re-raises the exception, the call run, like the gradients is not, so this change affected the test. It seems more like its testing the sampler than the fidelity as such with this error checking though - bit since the fidelity has none itself on the run and relies on this I guess it checks the overall behavior is as expected. @ElePT FYI

It seems that there are some fidelity tests for mismatched left and right circuits that are testing the sampler usage by fidelity to presumably ensure its catching the expected error when it calls run.
https://github.com/Qiskit/qiskit-terra/blob/4ac8b24af1cf6f99a418b5f118c29661f89a2b2c/test/python/algorithms/state_fidelities/test_compute_uncompute.py#L141
While the job.result() call is in a try catch block where it re-raises the exception, the call run, like the gradients is not, so this change affected the test. It seems more like its testing the sampler than the fidelity as such with this error checking though - bit since the fidelity has none itself on the run and relies on this I guess it checks the overall behavior is as expected. @ElePT FYI

I see. Given that as Steve mentioned, these errors are actually handled by the Sampler and not ComputeUncompute, I can remove this test from the fidelity module. I see 2 options:

I have an open PR addressing a few bug fixes in the fidelity classes #8755, it's a bit messy but I could include the fix there.
If there is no rush to include this PR in the next release, I could open a quick bug-fix PR for this purpose.

Whichever you prefer :)

TBH, I don't have a strong opinion to this issue. We don't have any major rules about Qiskit errors...
Since this is an API breaking, I would like to change it before next release.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

14 of 31 (45.16%)  changed or added relevant lines in 2 files are covered.
121 unchanged lines in 19 files lost coverage.
Overall coverage increased (+0.06%) to 84.394%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit/primitives/base_sampler.py
6
13
46.15%

qiskit/primitives/base_estimator.py
8
18
44.44%

Files with Coverage Reduction
New Missed Lines
%

qiskit/algorithms/evolvers/imaginary_evolver.py
1
83.33%

qiskit/compiler/transpiler.py
1
89.71%

qiskit/providers/fake_provider/fake_provider.py
1
70.91%

qiskit/transpiler/passes/basis/unroll_3q_or_more.py
1
97.14%

qiskit/transpiler/passes/optimization/collect_linear_functions.py
1
98.97%

qiskit/transpiler/passes/routing/commuting_2q_gate_routing/commuting_2q_gate_router.py
1
99.06%

qiskit/transpiler/passes/routing/sabre_swap.py
1
95.7%

qiskit/transpiler/passes/synthesis/unitary_synthesis.py
1
93.33%

qiskit/algorithms/aux_ops_evaluator.py
2
95.56%

qiskit/extensions/quantum_initializer/squ.py
2
79.78%

Totals

Change from base Build<PHONE_NUMBER>:
0.06%

Covered Lines:
59257

Relevant Lines:
70215

💛  - Coveralls

Well, you already fixed the CI but decided to remove the test anyway to prevent future situations like this.


Scrub all public API calls to ensure that all input parameters are properly sanitized before use

As a matter of policy, all CFE function calls that are available for use by external apps or libraries (i.e. all headers in the fsw/cfe-core/src/inc subdirectory) should sanity-check their input values before using them.
During recent code reviews it was noted that at least two of these functions were missing a required range check, as noted in bug #299.
This task is to perform an additional review on the public API calls and ensure that proper input value sanitization is being performed.

Imported from trac issue 269.  Created by jphickey on 2019-03-27T14:12:08, last modified: 2019-08-14T14:09:36

Trac comment by jhageman on 2019-07-03 12:48:08:
Moved unfinished 6.6.1 issues to next minor release

Trac comment by jhageman on 2019-04-01 16:19:55:
As part of this scrub also confirm failures are reported as errors, and unit testing adequately covers the failure cases.
Closing #273 as a duplicate of this effort (also somewhat a duplicate of #299).

I finished the API scrub. It is attached.
Also note that CFE_ES_SetAppState does not show up in the audit. That is because it is an internal function.
API_Audit_v2.xlsx

See #546 #547 #548 #549
The API scrub effort was completed, resolution tickets written so no further action required on this ticket.


[Streaming, Engineering] Reenable functional Direct Line Speech tests

Description
The Direct Line Speech functional tests that were added in 4.6 were disabled in master in https://github.com/microsoft/botbuilder-dotnet/pull/2861. This is due to the fact that Direct Line Speech support is not available in ARM, so the DLS tests will fail as the resources are recreated in the pipeline.
Steps to Solution
Azure CLI or REST support for Direct Line Speech and the configuration required to enable streaming.
See https://github.com/microsoft/botframework-sdk/issues/5655

Currently blocked on ARM Support, which is being worked by @DDEfromOR

Bumping this to after R7. It requires changes on the service side and then having a partner team pull those changes in to their product, which isn't likely to land before R7 due to blackouts.


Add SDK version to OE Environment preparation

This change is required in order for the Azure DCAP build tests added in microsoft/Azure-DCAP-Client#65

bors r+

bors r+

Yes, we need this SDK to build the DCAP Client for Windows. Thanks.


inversion train issue: weird results

When the global step < 15000, the losses are decreasing and images in logs/images/train are natural though they're a bit different from input. Then the losses rises to about 1.0 and images in logs/images/train get weird. I'm confused, hope to get your help.

I would need some more details to help you understand what could be causing this. Can you provide some details on your parameters and data?

I would need some more details to help you understand what could be causing this. Can you provide some details on your parameters and data?

Sorry, I'm busy in collect data these days so that I send this message today. The encoder is a pretrained pSp model trained with our own face data. Some parameters: output_size=8, batch_size=8, lr=1e-4, n_iters_per_batch=5, loss hyperparameters are same as the article.

The parameters you mention seem fine, but you say the images get weird. What do you mean by this? What kind of data are you working with? What's your domain? If you can provide some example results, this could help me understand what may be causing the issue.

The parameters you mention seem fine, but you say the images get weird. What do you mean by this? What kind of data are you working with? What's your domain? If you can provide some example results, this could help me understand what may be causing the issue.

The first image is the 1000 step's, the second is the 11000 step's.

Were any other adjustments made to the code?


XCTest universal framework target

I've duplicate the universal framework script target to create a XCTest compatible version.

Thanks @tibo - this inevitably would have come in with Carthage support, and so I'm closing this PR 👍


Listing modules and station in name order

Make the list of your own modules and the list of your own stations in name order A-Ö.
https://moduleregistry.azurewebsites.net/Modules/My
https://moduleregistry.azurewebsites.net/Stations/Index

Fixed in 1.7.11.


Support libgit2 error message report when assertion failed

Addresses #516
Current libgit2-sys implementation does not call git_error_last() to get git_error object and reporting its message member contents.
E.g. cargo just complains the following error when git_libgit2_init() is failing with the current implementation:
$ ./cargo --version
thread 'main' panicked at 'couldn't initialize the libgit2 library: -1', /usr/src/debug/cargo-native/1.37.0-r0/rustc-1.37.0-src/vendor/libgit2-sys/lib.rs:3356:9
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace.

Thanks!


cmd/compile: unnecessary full complex multiplication for x*c where x is complex and c a real constant

These two functions
const c = 2.71828

func mul1(z complex128) complex128 {
	return z*c
}

func mul2(z complex128) complex128 {
	return complex(real(z)*c, imag(z)*c)
}

both multiply a complex variable z with a scalar (real) constant c.
Since c is an untyped real constant, mul1 can be implemented as mul2. However, the compiler appears to generate code for a full complex multiplication x * complex(c, 0) for mul1, leading both to slower multiplication and possibly a less accurate result.
Assembly code for the two functions:
	0x0000 00000 (c.go:5)	TEXT	"".mul1(SB), NOSPLIT|ABIInternal, $0-32
	0x0000 00000 (c.go:5)	FUNCDATA	$0, gclocals·33cdeccccebe80329f1fdbee7f5874cb(SB)
	0x0000 00000 (c.go:5)	FUNCDATA	$1, gclocals·33cdeccccebe80329f1fdbee7f5874cb(SB)
	0x0000 00000 (c.go:6)	MOVSD	$f64.4005bf0995aaf790(SB), X0
	0x0008 00008 (c.go:6)	MOVSD	"".z+8(SP), X1
	0x000e 00014 (c.go:6)	MULSD	X1, X0
	0x0012 00018 (c.go:6)	XORPS	X2, X2
	0x0015 00021 (c.go:6)	MOVSD	"".z+16(SP), X3
	0x001b 00027 (c.go:6)	MULSD	X3, X2
	0x001f 00031 (c.go:6)	SUBSD	X2, X0
	0x0023 00035 (c.go:6)	MOVSD	X0, "".~r1+24(SP)
	0x0029 00041 (c.go:6)	XORPS	X0, X0
	0x002c 00044 (c.go:6)	MULSD	X0, X1
	0x0030 00048 (c.go:6)	MOVSD	$f64.4005bf0995aaf790(SB), X0
	0x0038 00056 (c.go:6)	MULSD	X3, X0
	0x003c 00060 (c.go:6)	ADDSD	X0, X1
	0x0040 00064 (c.go:6)	MOVSD	X1, "".~r1+32(SP)
	0x0046 00070 (c.go:6)	RET

and
	0x0000 00000 (c.go:9)	TEXT	"".mul2(SB), NOSPLIT|ABIInternal, $0-32
	0x0000 00000 (c.go:9)	FUNCDATA	$0, gclocals·33cdeccccebe80329f1fdbee7f5874cb(SB)
	0x0000 00000 (c.go:9)	FUNCDATA	$1, gclocals·33cdeccccebe80329f1fdbee7f5874cb(SB)
	0x0000 00000 (c.go:10)	MOVSD	$f64.4005bf0995aaf790(SB), X0
	0x0008 00008 (c.go:10)	MOVSD	"".z+8(SP), X1
	0x000e 00014 (c.go:10)	MULSD	X0, X1
	0x0012 00018 (c.go:10)	MOVSD	X1, "".~r1+24(SP)
	0x0018 00024 (c.go:10)	MOVSD	"".z+16(SP), X1
	0x001e 00030 (c.go:10)	MULSD	X1, X0
	0x0022 00034 (c.go:10)	MOVSD	X0, "".~r1+32(SP)
	0x0028 00040 (c.go:10)	RET

The compiler can easily statically determine that c has a 0 imaginary component and simplify the code generation.

cc: @randall77, @josharian

See src/math/cmplx/log.go, TODO in func Log10 and simplify that code once this is fixed.

I'm not sure if this is a correct optimization. I think it's the equivalent of the rule x * 0 == 0, which isn't actually true for float64.
(x+yi)*(c+0i) = (x*c-y*0) + (x*0+y*c) i

Say x == +inf, y == c == 1. Then the complex result is +inf + NaN i. Just multiplying both parts by 1 gives +inf + i.
Having said that, I think our story on inf/nan complex numbers is haphazard at best.

NaN's really are a mistake. Posits wouldn't have this problem... :-(
I suppose we could specify (in the spec) that a real scalar multiplication of a complex number really is translated into element-wise multiplication, etc. But that would probably lead to other troubles.
Sigh. Closing.


Simplify Gothic to use our session store instead of creating a different store

We have been using xormstore to provide a separate session store for our OAuth2 logins
however, this relies on using gorilla context and some doubling of our session storing.
We can however, simplify and simply use our own chi-based session store. Thus removing
a cookie and some of the weirdness with missing contexts.
This should fix several weird inconsistencies with openid and oauth2.
Signed-off-by: Andrew Thornton<EMAIL_ADDRESS>

Codecov Report

Merging #17507 (3c2e609) into main (4e8a817) will decrease coverage by 0.01%.
The diff coverage is 4.65%.

@@            Coverage Diff             @@
##             main   #17507      +/-   ##
==========================================
- Coverage   45.49%   45.48%   -0.02%
==========================================
  Files         791      791
  Lines       88717    88742      +25
==========================================
+ Hits        40364    40366       +2
- Misses      41842    41864      +22
- Partials     6511     6512       +1

Impacted Files
Coverage Δ

cmd/web.go
0.00% <0.00%> (ø)

cmd/web_letsencrypt.go
0.00% <0.00%> (ø)

services/auth/source/oauth2/store.go
0.00% <0.00%> (ø)

services/auth/source/oauth2/init.go
36.00% <100.00%> (+0.28%)
:arrow_up:

modules/git/repo_base_nogogit.go
82.85% <0.00%> (-2.86%)
:arrow_down:

modules/git/utils.go
68.05% <0.00%> (-2.78%)
:arrow_down:

modules/process/manager.go
72.83% <0.00%> (-2.47%)
:arrow_down:

models/repo_list.go
78.43% <0.00%> (-0.79%)
:arrow_down:

modules/queue/queue_bytefifo.go
59.28% <0.00%> (-0.60%)
:arrow_down:

services/pull/pull.go
41.78% <0.00%> (-0.41%)
:arrow_down:

... and 8 more

Continue to review full report at Codecov.

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

Don't merge just yet - there's an old setting that may need to be deleted.

OK I've added back in the maximum token length


Exposed API Key

https://github.com/UW-Madison-Lee-Lab/LanguageInterfacedFineTuning/blob/afed08d7544540e94249de33b844b1cf2bd2d22e/Others/ridge/ridge.py#L150
Howdy,
Do not expose your API key in your source code.
Instead, consider storing it an environment variable, or using a key management service, or even software such as Vault.
Then be sure to rotate your API key after it is safely stored, or else it may still be recoverable in your commit history.
Also, see Openai's best practices page for api keys for more details:
https://help.openai.com/en/articles/5112595-best-practices-for-api-key-safety

Dear Nahatx,
Thank you so much for bringing this to our attention. We have removed all OpenAI keys from our code and replaced them with placeholders instructing users to insert their own keys. Rest assured, all previous keys have been invalidated and are no longer in use.
We appreciate your vigilance and your guidance on best practices. We'll certainly bear these in mind moving forward.
Best regards,
Yuchen Zeng on behalf of all authors


created withQueryValues method

Fixes #196

Thanks @dnyg. Good job with the refactoring for performance reasons. That's exactly what I had in mind.


Some notes for Linux

OK, so I decided to use this library with linux. But, as with many other things it didn't work right away. So, I've decided to write this as a note for others who might want to do the same.

Install raylib (duh!)
The binaries for raylib in the release don't work on linux. Luckily, raylib is quite easy to get-just follow the instructions here.

2)Get rid of Windows dependencies in the binding
There are 4 Windows.IntBool s in Types/PhysicsBodyData.bf. I replaced them with int32 to maintain the size of the struct, but this might have broken something in the physics system-NOT TESTED.
3)Add linker flags to the BeefProj.toml file of YOUR project - not the library.
[Configs.Debug.Linux64]
OtherLinkFlags = "$(LinkFlags) -lraylib -lglfw -lGL -lopenal -lm -pthread -ldl"

Adding this should solve any linker errors, and the project should now compile and run.

Oh yeah i forgot to add some info about non windows OS support. I am gonna do that ASAP.

Yeah... About that-while the example on the readme works fine, the overall bindings are... problematic.
I transpiled the "2d camera" demo from https://www.raylib.com/examples, and it didn't work. Here's the transpiled code:
using raylib_beef;
using raylib_beef.Types;
using System;
using raylib_beef;

namespace raylibtest
{
	public static class Program : Raylib
	{
		const int MAX_COLUMNS = 20;
		const int MAX_BUILDINGS = 100;

		public static void Main()
		{
			// Initialization
			//--------------------------------------------------------------------------------------
			const int screenWidth = 800;
			const int screenHeight = 450;

			InitWindow(screenWidth, screenHeight, "raylib [core] example - 2d camera");

			Rectangle player = Rectangle(400, 280, 40, 40);
			Rectangle[] buildings= scope Rectangle[MAX_BUILDINGS];
			Color[] buildColors = scope Color[MAX_BUILDINGS];

			int spacing = 0;

			for (int i = 0; i < MAX_BUILDINGS; i++)
			{
			    buildings[i].width = GetRandomValue(50, 200);
			    buildings[i].height = GetRandomValue(100, 800);
			    buildings[i].y = screenHeight - 130 - buildings[i].height;
			    buildings[i].x = -6000 + spacing;

			    spacing += (int)buildings[i].width;

			    buildColors[i] = .((uint8)GetRandomValue(200, 240), (uint8)GetRandomValue(200, 240), (uint8)GetRandomValue(200, 250), 255);
			}

			Camera2D camera;
			camera.target = .(player.x + 20, player.y + 20);
			camera.offset = .(screenWidth/2, screenHeight/2);
			camera.rotation = 0.0f;
			camera.zoom = 1.0f;

			SetTargetFPS(60);                   // Set our game to run at 60 frames-per-second
			//--------------------------------------------------------------------------------------

			// Main game loop
			while (!WindowShouldClose())        // Detect window close button or ESC key
			{
			    // Update
			    //----------------------------------------------------------------------------------

			    // Player movement
			    if (IsKeyDown(.KEY_RIGHT)) player.x += 2;
			    else if (IsKeyDown(.KEY_LEFT)) player.x -= 2;

			    // Camera target follows player
			    camera.target = .(player.x + 20, player.y + 20);

			    // Camera rotation controls
			    if (IsKeyDown(.KEY_A)) camera.rotation--;
			    else if (IsKeyDown(.KEY_S)) camera.rotation++;

			    // Limit camera rotation to 80 degrees (-40 to 40)
			    if (camera.rotation > 40) camera.rotation = 40;
			    else if (camera.rotation < -40) camera.rotation = -40;

			    // Camera zoom controls
			    camera.zoom += ((float)GetMouseWheelMove()*0.05f);

				Console.WriteLine(GetMouseWheelMove());

			    if (camera.zoom > 3.0f) camera.zoom = 3.0f;
			    else if (camera.zoom < 0.1f) camera.zoom = 0.1f;

			    // Camera reset (zoom and rotation)
			    if (IsKeyPressed(.KEY_R))
			    {
			        camera.zoom = 1.0f;
			        camera.rotation = 0.0f;
			    }
			    //----------------------------------------------------------------------------------

			    // Draw
			    //----------------------------------------------------------------------------------
			    BeginDrawing();

			        ClearBackground(.RAYWHITE);
					Console.WriteLine(camera.offset.ToString());
					Console.WriteLine(camera.target.ToString());
					Console.WriteLine(camera.rotation);
					Console.WriteLine(camera.zoom);
			        BeginMode2D(camera);

			            DrawRectangle(-6000, 320, 13000, 8000, .DARKGRAY);

			            for (int i = 0; i < MAX_BUILDINGS; i++)
						{
							DrawRectangle((int)buildings[i].x, (int)buildings[i].y, (int)buildings[i].width, (int)buildings[i].height, buildColors[i]);
						}

			            DrawRectangleRec(player, .RED);

			            DrawLine((int)camera.target.x, -screenHeight*10, (int)camera.target.x, screenHeight*10, .GREEN);
			            DrawLine(-screenWidth*10, (int)camera.target.y, screenWidth*10, (int)camera.target.y, .GREEN);

			        EndMode2D();

			        DrawText("SCREEN AREA", 640, 10, 20, .RED);

			        DrawRectangle(0, 0, screenWidth, 5, .RED);
			        DrawRectangle(0, 5, 5, screenHeight - 10, .RED);
			        DrawRectangle(screenWidth - 5, 5, 5, screenHeight - 10, .RED);
			        DrawRectangle(0, screenHeight - 5, screenWidth, 5, .RED);

			        DrawRectangle( 10, 10, 250, 113, Fade(.SKYBLUE, 0.5f));
			        DrawRectangleLines( 10, 10, 250, 113, .BLUE);

			        DrawText("Free 2d camera controls:", 20, 20, 10, .BLACK);
			        DrawText("- Right/Left to move Offset", 40, 40, 10, .DARKGRAY);
			        DrawText("- Mouse Wheel to Zoom in-out", 40, 60, 10, .DARKGRAY);
			        DrawText("- A / S to Rotate", 40, 80, 10, .DARKGRAY);
			        DrawText("- R to reset Zoom and Rotation", 40, 100, 10, .DARKGRAY);

			    EndDrawing();

				//Console.WriteLine(buildings[50].width);
			    //----------------------------------------------------------------------------------
			}

			// De-Initialization
			//--------------------------------------------------------------------------------------
			CloseWindow();        // Close window and OpenGL context
			//--------------------------------------------------------------------------------------

		}
	}
}

it compiles fine, and the draw calls outside the BeginMode2D() block go through:

So, after some digging around with gdb, I think I found the issue - structs such as the Camera2D, or Vector2 get really messed up when they go through. Inspecting the camera struct right before
the BeginMode2D(), this is what gdb says is in it:
{<System::ValueType$part> = {<No data fields>}, offset = {<System::ValueType$part> = {<No data fields>}, x = 400, y = 225}, target = {<System::ValueType$part> = {<No data fields>}, x = 420, y = 300}, rotation = 0, zoom = 1}
and, if I inspect the struct right after its been passed across to a raylib function, here's whats in it now:
{offset = {x = 1.14273087e-40, y = 0}, target = {x = 1.56384909e-42, y = 0}, rotation = 1.56384909e-42, zoom = 0}
So, yeah. Also, one last thing I noticed was that when using the GetMouseWheelMove() function, an upwards scroll returned one, as it should, but when scrolling down it returned<PHONE_NUMBER>, which just so happens to be the largest possible value for uint32, meaning it returns an unsigned integer despite being marked as signed, making a scroll down snap the zoom value to 3, because of the check.
There's a good chance these bugs are in beef, or raylib in linux has issues with interop(natively compiling the original examples has no issues), but since I experienced these issues here, I've decided to put them here as well.

And.. Good news! I figured out why the zoom wasn't working! It was because when specifying int, Beef makes it the native size - 64 bits, while the C library is using 32 bit integers. so, using 32 bit integers solved the issue.

You can submit pull request if you want

I will, but I'll do it after I can get the camera to work.

Turns out the bug is in beef. Made an issue @beefytech/Beef#267


Kraken IDE not displaying in Mozilla Firefox

Describe the bug
After starting the containers, IDE is not getting properly displaying in Mozilla Firefox browser. Please check the screenshot.
To Reproduce
Steps to reproduce the behavior:

Launch Gatling IDE or Administrator UI in the latest version of Firefox

Expected behavior
IDE should launch properly.
Screenshots
Screenshot has been attached.

Hi.
Does it work in Chrome? Is there an error displayed in the developer console?
Thanks.

I get this error "SyntaxError: invalid regular expression flag s"

@QAInsights thanks for the feedback. I released a fix: https://kraken.octoperf.com/downloads/kraken-1.2.1.zip

Thanks @geraldpereira Let me validate.


✨ Version 0.4.0

Includes a lot of restructuring and the new CronSchedule struct for working with cron expressions.

Codecov Report

Merging #2 (84ba831) into main (431bc55) will not change coverage.
The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@             Coverage Diff             @@
##              main        #2     +/-   ##
===========================================
  Coverage   100.00%   100.00%
===========================================
  Files           13        19      +6
  Lines         2322      3851   +1529
===========================================
+ Hits          2322      3851   +1529

Impacted Files
Coverage Δ

src/shared.rs
100.00% <ø> (ø)

src/cron.rs
100.00% <100.00%> (ø)

src/date.rs
100.00% <100.00%> (ø)

src/datetime.rs
100.00% <100.00%> (ø)

src/errors/mod.rs
100.00% <100.00%> (ø)

src/lib.rs
100.00% <100.00%> (ø)

src/time.rs
100.00% <100.00%> (ø)

src/util/date/convert.rs
100.00% <100.00%> (ø)

src/util/date/manipulate.rs
100.00% <100.00%> (ø)

src/util/date/validate.rs
100.00% <100.00%> (ø)

... and 7 more