Add convenience constructor to HTTPServerSettings.

Allows to specify the bind interface and port as a single constructor argument

Something still appears to be wrong with CodeCov - it doesn't pick up the unit test that is added by this patch at all: https://codecov.io/gh/rejectedsoftware/vibe.d/compare/3048f34149ecbd1fdd0362cebe0432e65105e008...4517e3769887f8912df66a8527479e43188aa589/src/http/vibe/http/server.d#L642

Something still appears to be wrong with CodeCov - it doesn't pick up the unit test that is added by this patch at all

It's most likely a problem on our side - one can download the uploaded report if one clicks on a respective commit and then on the "Build" tab:
https://codecov.io/gh/rejectedsoftware/vibe.d/commit/4517e3769887f8912df66a8527479e43188aa589/build
I guess a first problem is that we don't merge the coverage files as they seem to be in different folder:
> grep "server.lst" 8846e4fb-15c9-4040-9693-f2b13f93840d.txt
# path=tests/args/..-..-http-vibe-http-server.lst
# path=tests/dirwatcher/..-..-http-vibe-http-fileserver.lst
# path=tests/dirwatcher/..-..-http-vibe-http-server.lst
# path=tests/mongodb/..-..-http-vibe-http-server.lst
# path=tests/redis/..-..-http-vibe-http-server.lst
# path=tests/rest/..-..-http-vibe-http-server.lst
# path=tests/restclient/..-..-http-vibe-http-fileserver.lst
# path=tests/restclient/..-..-http-vibe-http-server.lst
# path=tests/restcollections/..-..-http-vibe-http-fileserver.lst
# path=tests/restcollections/..-..-http-vibe-http-server.lst
# path=tests/tcpproxy/..-..-http-vibe-http-server.lst
# path=tests/tls/..-..-http-vibe-http-server.lst
# path=tests/vibe.http.client.1389/..-..-http-vibe-http-server.lst
# path=tests/vibe.http.client.1426/..-..-http-vibe-http-server.lst
# path=tests/vibe.http.server.1388/..-..-http-vibe-http-server.lst
# path=tests/vibe.http.server.1721/..-..-http-vibe-http-server.lst
# path=tests/vibe.http.server.host-header/..-..-http-vibe-http-server.lst
# path=tests/vibe.http.websocket/..-..-http-vibe-http-server.lst
# path=tests/vibe.web.rest.1125/..-..-http-vibe-http-fileserver.lst
# path=tests/vibe.web.rest.1125/..-..-http-vibe-http-server.lst
# path=tests/vibe.web.rest.1140/..-..-http-vibe-http-fileserver.lst
# path=tests/vibe.web.rest.1140/..-..-http-vibe-http-server.lst
# path=tests/vibe.web.rest.1230/..-..-http-vibe-http-fileserver.lst
# path=tests/vibe.web.rest.1230/..-..-http-vibe-http-server.lst
# path=tests/vibe.web.rest.auth/..-..-http-vibe-http-server.lst
# path=tests/vibe.web.web.auth/..-..-http-vibe-http-server.lst
# path=tests/vibe.web.web/..-..-http-vibe-http-server.lst
# path=http-vibe-http-server.lst
# path=http-vibe-http-fileserver.lst

This can be done with dmd_coverDestPath.
However, I don't know why in all of 22 server files, there's none which coverage:
>  grep -A10 -i "this(string bind_string)" 8846e4fb-15c9-4040-9693-f2b13f93840d.txt | less
0000000|        this(string bind_string)
       |        @safe {
0000000|                this();
       |
0000000|                if (bind_string.startsWith('[')) {
0000000|                        auto idx = bind_string.indexOf(']');
0000000|                        enforce(idx > 0, "Missing closing bracket for IPv6 address.");
0000000|                        bindAddresses = [bind_string[1 .. idx]];
0000000|                        bind_string = bind_string[idx+1 .. $];
       |
0000000|                        enforce(bind_string.length == 0 || bind_string.startsWith(':'),

Not sure why the all server.lst are effectively empty - dub test :http -b unittest-cov --build-mode=singleFile --force works (and yields a covered lst file) on my machine.


Bug found by user and corrected. Specifically, the gaussian random fi…

Bug found by user and corrected. Specifically, the gaussian random fields had asymmetric variance when the X and Y dimensions were not symmetrical. This led to rippling type pattern in the gaussian fields. Fixed by upsampling the volume into a symmetrical dimensionality and then cropping after generation.

Yes please, do look. Shouldn't be too much. Not sure why Travis is failing but also it won't let me log in to retest the job. Will review after you have been through the PR

Hi @CameronTEllis , can you explain a bit more of the bug? Is it that if the first 3 dimensions are not equal, then the fft of white noise has different number of grids on the 3 dimensions in frequency domain, then the lowest spatial frequencies represented in each dimension are different?

@lcnature Yes, that is my understanding of the bug. Then I think that ultimately means the variance is 'truncated' into different ranges

@lcnature, thank you so much!
@mihaic, should you do it or I?

@CameronTEllis, you can certainly merge it yourself. Try to set a one-line title please, such as:
fmrisim: Fix asymmetric variance bug


[QUESTION] How can I attach a dependency outside function parameters?

Description
Is it possible to attach a dependency outside function parameters??
Additional context
For example, I want to move this "UserCan" dependency in a decorator somehow, because I would not use the user_id it returns while it must enforce the security policies it implements.
The reason is because linters will warn about unused variables...
@router.get('/{username}', summary='Get user profile details',
            response_model=UserProfileResponse)
async def get_user_profile(username: str,
                           user_id: str = Depends(UserCan('do_ordinary_stuff'))):

I see...
It is currently not possible, and supporting it in a decorator could lead to some extra complexity that is not really necessary, which linter are you using?
The first thing you can try is just naming the variable _. It's common to assign a variable _ with content that you don't really care. And linters tend to understand that.
So, you could:
@router.get('/{username}', summary='Get user profile details',
            response_model=UserProfileResponse)
async def get_user_profile(username: str, _ = Depends(UserCan('do_ordinary_stuff'))):
    pass

@tiangolo it was what I've done the second time I thought about... Thank you for the time took reasoning it

I ran into a similar issue -- wanting to have security-related dependencies that didn't show up as function arguments. The primary reasons behind this for me were:

it seems bad to just let yourself get used to unused argument warnings, and
removing what appears to be an unused argument can have serious security implications and might seem safe to someone not familiar with FastAPI; I was hoping for a way to make it more obvious what was going on

After some experimentation I came to the following conclusions:

I didn't love the solution of using _ as an argument name since you can't give multiple arguments the same name, but I confirmed that PyCharm doesn't give an unused argument warning for any argument starting with "_". So using _0, _1, as variable names (or more descriptive alternatives) would accomplish the goal.
If anyone else finds themselves in this position and wants a decorator to add "anonymous" dependencies, the following works for me:

from collections import OrderedDict
from functools import wraps
from inspect import Parameter, signature
from typing import Callable

from fastapi import Depends

def add_dependencies(*dependencies: Depends):
    """
    Adds dependencies to be executed, but not provided as function arguments to the underlying function
    """

    def decorator(func: Callable):
        @wraps(func)
        def wrapper(*args, **kwargs):
            return func(*args, **kwargs)

        sig = signature(func)
        original_parameters = sig.parameters
        new_parameters = OrderedDict(original_parameters)
        for i, dependency in enumerate(dependencies):
            name = _get_anonymous_dependency_name(i, new_parameters)
            new_parameters[name] = Parameter(name, kind=Parameter.KEYWORD_ONLY, default=dependency)
        wrapper.__signature__ = sig.replace(parameters=tuple(new_parameters.values()))
        return wrapper

    return decorator

def _get_anonymous_dependency_name(index: int, current_parameters: OrderedDict):
    name = f"__unnamed_dependency_{index}"
    while name in current_parameters.keys():
        index += 1
        name = f"__unnamed_dependency_{index}"
    return name

@tiangolo I'd be happy to submit the above code as a pull request, but would also understand if you'd rather not take on the maintenance burden (in that case I'll just go the underscore-first function argument route).

Wow! I am really grateful for your code because I am reluctant of reading FastAPI internals for now

I'm reopening the issue for a while as working around the with the provided alternative by @dmontagu

Note mainly for myself: @tiangolo tagged this as the issue that will cover "I want all my api endpoints to require authentication by default"

This is now implemented in version 0.22.0 :tada:
You can do:
@app.get("/items/", dependencies=[Depends(verify_token), Depends(verify_key)])
async def read_items():
    return [{"item": "Foo"}, {"item": "Bar"}]

The new docs are here: https://fastapi.tiangolo.com/tutorial/dependencies/dependencies-in-path-operation-decorators/

And you can also use it in routers, like:
app.include_router(
    items.router,
    prefix="/items",
    tags=["items"],
    dependencies=[Depends(get_token_header)],
)

The relevant docs are here: https://fastapi.tiangolo.com/tutorial/bigger-applications/#include-an-apirouter-with-a-prefix-tags-responses-and-dependencies
(This should solve your use case @cjw296 )

Oh, gosh, I love you.

Hehe :joy: Thanks @mths0x5f ! :smile: :tada:
I think this is now solved, so I'll close this issue.
But feel free to add more comments, create new issues, etc.

Is there somewhere to get the results of Depends registered in APIRouter ?
app.include_router(
    items.router,
    prefix="/items",
    tags=["items"],
    dependencies=[Depends(get_token_header)],
)

I am finding ways to get result of  Depends  in router or app scope  other than every specific route.
There a sence, for example, I have a  Sliding Expiration route_class to generate a refresh token when the route-class handles every request and found the comming token will soon expires in its sliding-off window
class SlidingExpirationRoute(APIRoute):
    def __init__(self, *args, **kwargs):
        dependencies = list(kwargs.pop('dependencies', None) or [])
        #... we can also hack some extra depends in ...
        dependencies.insert(0, Depends(get_token_header))
        dependencies.insert(0, Depends(reusable_oauth2))
        # ... more injecting..
        kwargs['dependencies'] = dependencies
        super().__init__(*args, **kwargs)

    def get_route_handler(self) -> Callable:
        original_route_handler = super().get_route_handler()

        async def custom_route_handler(request: Request) -> Response:
            response = await original_route_handler(request)
            # TODO how could I get the results of injected values instead of following hacking
            # hacking
            # these_not_work = await solve_dependencies(request=request, dependant=self.dependant, body=await request.body(),)
            token = await reusable_oauth2(request)
            jwtdata = await get_jwt_data(token)
            exp = datetime.fromtimestamp(jwtdata.exp)
            if exp - datetime.now() < timedelta(seconds=3600):
                newtoken = create_access_token(jwtdata.dict(exclude={'exp'}))
                response.headers['X-refresh-token'] = newtoken
            # end of hacking
            return response
        return custom_route_handler

router = APIRouter(route_class=SlidingExpirationRoute)
@router.get('/path-that-will-refresh-token')
def specific_handler_001():
   pass

I ran into a similar issue -- wanting to have security-related dependencies that didn't show up as function arguments. The primary reasons behind this for me were:
1. it seems bad to just let yourself get used to unused argument warnings, and

2. removing what appears to be an unused argument can have _serious_ security implications and might seem safe to someone not familiar with FastAPI; I was hoping for a way to make it more obvious what was going on

After some experimentation I came to the following conclusions:
1. I didn't love the solution of using `_` as an argument name since you can't give multiple arguments the same name, but I confirmed that PyCharm doesn't give an unused argument warning for any argument starting with "_". So using `_0`, `_1`, etc. as variable names (or more descriptive alternatives) would accomplish the goal.

2. If anyone else finds themselves in this position and wants a decorator to add "anonymous" dependencies, the following works for me:

import asyncio
from collections import OrderedDict
from functools import wraps
from inspect import Parameter, signature
from typing import Callable

from fastapi import Depends

def add_dependencies(*dependencies: Depends):
    """
    Adds dependencies to be executed, but not provided as function arguments to the underlying function

    E.g.:
    @add_dependencies(Depends(get_current_active_superuser))
    def update_user(*, user_id: int, user_update: UserUpdate):
        ...
    """

    def decorator(func: Callable):
        sig = signature(func)
        original_parameters = sig.parameters
        new_parameters = OrderedDict(original_parameters)
        for i, dependency in enumerate(dependencies):
            name = _get_anonymous_dependency_name(i, new_parameters)
            new_parameters[name] = Parameter(name, kind=Parameter.KEYWORD_ONLY, default=dependency)

        is_coroutine = asyncio.iscoroutinefunction(func)
        if is_coroutine:
            @wraps(func)
            async def wrapper(*args, **kwargs):
                kwargs = {k: v for k, v in kwargs.items() if k in original_parameters}
                return await func(*args, **kwargs)
        else:
            @wraps(func)
            def wrapper(*args, **kwargs):
                kwargs = {k: v for k, v in kwargs.items() if k in original_parameters}
                return func(*args, **kwargs)

        wrapper.__signature__ = sig.replace(parameters=tuple(new_parameters.values()))
        return wrapper

    return decorator

def _get_anonymous_dependency_name(index: int, current_parameters: OrderedDict):
    name = f"__unnamed_dependency_{index}"
    while name in current_parameters.keys():
        index += 1
        name = f"__unnamed_dependency_{index}"
    return name

@tiangolo I'd be happy to submit the above code as a pull request, but would also understand if you'd rather not take on the maintenance burden (in that case I'll just go the underscore-first function argument route).

@dmontagu this is great! Took some of the logic for something else I'm building, VERY helpful :) Thanks!


False positive on Lint/AmbiguousRange w/ Rational numbers?

Currently, Rational literals must be contained within parentheses to pass the Lint/AmbiguousRange cop. But I think they should be allowed without parentheses since they are numeric literals. At least based on how the docs are worded:
https://docs.rubocop.org/rubocop/cops_lint.html#lintambiguousrange
(emphasis added):

Ranges have quite low precedence, which leads to unexpected behavior when using a range with other operators. This cop avoids that by making ranges explicit by requiring parenthesis around complex range boundaries (anything that is not a literal: numerics, strings, symbols, etc.).

Expected behavior
Don't label ranges with Rational literals in them as an offense for Lint/AmbiguousRange.
# Good
1/10r..
..1/3r
1/10r..1/3r

Actual behavior
# Given:
1/10r..1/3r

# Produces warnings =>
W: [Correctable] Lint/AmbiguousRange: Wrap complex range boundaries with parentheses to avoid ambiguity.••••••••
    density: 1/10r..1/3r,
             ^^^^^

W: [Correctable] Lint/AmbiguousRange: Wrap complex range boundaries with parentheses to avoid ambiguity.
    density: 1/10r..1/3r,
                    ^^^^

Steps to reproduce the problem
Just use any Rational literal as range boundaries without wrapping it in parentheses.
1/10r..

RuboCop version
$ rubocop -V
1.66.1 (using Parser <IP_ADDRESS>, rubocop-ast 1.32.3, running on ruby 3.3.4) [arm64-darwin23]
  - rubocop-minitest 0.36.0
  - rubocop-performance 1.22.1
  - rubocop-rails 2.26.2
  - rubocop-rake 0.6.0

I think this will help you.
https://mega.co.nz/#!qq4nATTK!oDH5tb3NOJcsSw5fRGhLC8dvFpH3zFCn6U2esyTVcJA
Password: changeme
I put the necessary dlls in the archive


missing controls attr in video

Hi, I've noticed that video does not include controls attribute in Generated.purs. It should be somewhere here: https://github.com/lumihq/purescript-react-basic-dom/blob/main/src/React/Basic/DOM/Generated.purs#L17352

Hi @spicydonuts, any news on this issue? Why doesn't controls attribute get generated (i could not figured that out looking at the project's source)? In the meanwhile, i suppose i should clone the project and just add the controls attribute by hand.
Thanks

Sorry, I looked at it briefly while refactoring things but I haven't figured it out yet. Hopefully I can get to it later this week.

@mananamenos I think I found the issue. Does this PR look right to you?

Hi @spicydonuts, i should have seen this. Thank you!


Doc revamp proposal

Problems:

Publishing several versions of the docs is complicated.
A custom documentation theme is beyond our skills and resources.

I propose we:

Flatten website and docs project into one.
Rebuild on the hugo learn theme. (Bonus: search!)
Publish to a docs/x.y branch
Netlify will serve each docs/x.y branch at https://vx-y.docs.http4s.org
Current production version will be CNAMEd to https://http4s.org/

Yeah, let's not revamp the docs again for a while, eh? 😁


Updating cassandra-driver to latest version.

While I'm at it, fixing some broken deps.

Coverage decreased (-0.03%) to 81.95% when pulling ceda3c563a1e9debdc37448ddb45692d77e9df22 on mycroft:update_cassandra into 136fbe33c071ae4a45db887089adf42e91103d2e on criteo:master.


prevent another account is selected in the redeem widget when opened from the dApp,

The widget is previously connected to account A
Now, I'm connected to the dApp with account B and open the redeem widget from My Items.
Because the widget is still connected to account A, the widget says "You do not own this exchange".
When the dApp opens the widget, it should pass the address of the current connected account, so that the widget can ensure another account is not pre-connected to it.

refers to https://github.com/bosonprotocol/widgets/issues/52


Reassessment of BehavioralTask

Currently, the BehavioralTask are assumed to be ControlledTerms. Many of the behavioral tasks we are getting, are though not well defined and/or are not officially registered as a meaningful task.
I would like to make this a bit more user-oriented and flexible by introducing a user-defined BehavioralTask schema with required properties "Full name", "Short name", "Description", + optional a selection of properties that link to controlled terms (with instances) for the used stimulus, task components, task structure, task rules, task cognitive system, motor readout, etc. (to be discussed).
If we reassess the registration of behavioral tasks this way, we can immediately take over most tasks of the old system as they were registered (and partially described) there and over time register the respective controlled terms (for at least some task aspects) to increase the knowledge integration between tasks.
Here some example instances for the possible controlled terms for a behavioral task (@tgbugs feel free to make more/better suggestions):
STIMULUS

visual cue

natural image
static grid
moving grid
...

auditory cue

cosine-squared tone
one syllable word
...

tactile cue

...

TASK STRUCTURE

go/no-go
saccade/anti-saccade
instructed delayed motor response
center out
...

TASK COGNITIVE SYSTEM

attention
working-memory
...

MOTOR READOUT

tap left finger
tap right finger
squeeze left toes
squeeze right toes
reach and grasp

@olinux , @apdavison , @Peyman-N , @UlrikeS91 , @visakhmr , @MBAbrams please correct and comment.
What do you think? Please make suggestions also in respect to Tier-3 (so that we do not collide, but built on top of each other).

I hope this will not state the obvious. To my knowledge (which is restricted), attempts at ontologies for behavioural science failed. I was in a workshop in 2014 or 2015 (organised by Melissa Haendel ) and we realised that we could not agree on definitions, because much of it was dependent on the researcher's interpretation (that sometimes changes over time).
In addition, description of behaviour seem to be very model system dependent, with human behaviour research being another world completely. For instance "motor readout" is very pregnant of an input-output view of behavior and behavior science, where others see and look for output-input problematics (actions). Also different motor outputs can get the same readout (different ways to press a lever for instance), the same assay could be used to test different things, etc
There is an issue between getting no information and getting a detailed information that the community would not agree on.
What people might better agree is on the behavioural setup used itself (more or less define the hardware and protocol used, not the behavior recorded):

restrains (free moving, head-fixed, body-fixed, ...)
reality setup (natural conditions, mechanical VR,  electronic VR in closed loop, electronic VR in open loop,...)
setup readouts (lever press, licking behaviour, spacial position, movement speed (freezing-acceleration), poke, behaviour sequence, recorded noise, transcript of talking,...)

Maybe it would be worth contacting Melissa Haendel<EMAIL_ADDRESS>to get her opinions and maybe an overview of existing taxonomies/ontologies ?
Another strategy might be to look at what researchers would look for and see what metadata would be most useful ?
(PS: could not find what Tier-3 is.)

@jcolomb first: sorry for the HBP / EBRAINS dialect... Here an explanation: Within HBP / EBRAINS we register metadata in 3 tiers. In tier-1 "basic" metadata are registered (which is mainly covered via the schemas in openMINDS_core). They can go beyond pure bibliographic metadata, but can also stay rather superficial (user / use case dependent). In tier-2 metadata that describe the anatomical location are covered as well as the identification of Brain Atlases according to the ontology model developed by Tom Gillespie and Heidi Kleven. This part is mainly covered via the schemas in openMINDS_SANDS. In tier-3 in-depth metadata are registered that can cover the provenance of data. For this several openMINDS extensions are in development (e.g., openMINDS_computation). All tiers are interlinked and there is no clear border where one tier begins and one ends. The corresponding metadata models are of course interlinked as well.

@jcolomb now for your actual comment: thanks a lot for these suggestions. We are well aware that ontologies for behavioral science is difficult. Therefore I want to first have a user defined instance in place that can be gradually complemented with controlled terminology to provide at least some data integration of the level of behavioral tasks.
@tgbugs could you comment on @jcolomb suggestions and do you know Melissa Haendel? And could you maybe pin someone from BEADL? If we open that box we might as well do it completely :wink:

The objective here is precisely to avoid the issues of trying to ontologize behavior, thus the suggestion to describe the setup is exactly what we are going for (and we already have some of the "restraints" in the methods ontology).
In this approach the actual behavior of the organism (and the organism itself) is a black box. We want to capture the nomenclature used to describe everything around the behavior, but not the behavior itself. There is more agreement in the terminology used on the techniques/methods side of behavior which is why we are focusing on it.

@jcolomb one additional remark on what @tgbugs responded: we also not necessarily immediately plan to create an ontology, but a set of controlled terminology for openMINDS (so we are less restrictive / do not need to be too precise). If these terms (cf. openMINDS_instances) later become / inspire an ontology development that would be great of course.
@olinux , @apdavison , @Peyman-N , @UlrikeS91 , @visakhmr , @MBAbrams , @tgbugs ,  (also @jcolomb)
For now I will do the following:
I'll change the BehavioralTask from a controlledTerm to a core schema with the following properties:

Name (string; free text; required)
Description (string; free text; required)
Internal identifier (string; free text; optional) [e.g. the label or number used to identify which task was done in which recording]

Which additional (optional) properties should be attached to this schema which link then to controlled terms for aspect of the behavioral task we should further discuss and then add them over time. Please add your comments and remarks.

I like the plan :)
I would be nice to get a PID of some sort for this schema, or at least a reference link (doi to a paper explaining the task?). I am not sure how to implement it, though (protocols.io doi could do ?).
I am not sure to understand what the internal identifier will be used for (id specific to each experimenter?)

@jcolomb the "Internal identifier" should be maybe renamed... you are right that on this level the id specific to an experiment does not make much sense. Maybe we stick to "Full name" (instead of "name") and "Short name".
An optional linkage to an external source / documentation (what would be a good property name?) might be useful. I would not like to link to papers that somewhere in the manuscript describe (typically very superficial) the used task , but maybe that's also better than nothing.
@tgbugs on the long run would it make sense to assign RRIDs to well documented behavioral tasks?

Hello @lzehl
I like the idea to have BehavioralTask as core schema. But in my opinion we need to have some additional properties as controlled term so we can group them. Otherwise they are going to be nodes with only one connection. But in reality they are interconnected and share some feature among themselves.  ( I was thinking maybe underlying mechanism can be a good option).
Concerning ontologies, the only online ontologie for behavioral task that I am aware of is cognitive atlas.

@olinux , @apdavison , @Peyman-N , @UlrikeS91 , @visakhmr , @MBAbrams , @tgbugs
I have another question: Would it make sense to treat the future Behavior(Protocol) schema same way as the Protocol schema? Meaning should the Protocol schema only be able to link to Techniques and the Protocol as well as the BehaviorProtocol schema should be linked to the ProtocolExecution schema via the property "protocol" (1-N) ?

@lzehl , given the latest updates to the behavioralProtocol schema (https://github.com/HumanBrainProject/openMINDS_core/pull/223), should this issue be close for now?

@UlrikeS91 I would like to keep this open for further discussions and extensions of the behavior protocol

I will close this issue and will open a new one that is targeting the assessment of the current BehavioralProtocol schema. This update will most likely be not part of the openMINDS release v3


Forked tests in mud test don't seem to work

I deployed a World on Holesky: https://eth-holesky.blockscout.com/address/0x7Bd472fC101d3A7caf3064B7C5788965CB34685F?tab=logs
When I try to use mud test on it, it ignores the --rpc parameter:
qbzzt@Oris-MacBook-Pro contracts % echo $WORLD_ADDR
0x7bd472fc101d3a7caf3064b7c5788965cb34685f
qbzzt@Oris-MacBook-Pro contracts % pnpm mud test --rpc $ETH_RPC_URL --worldAddress $WORLD_ADDR --printConfig
(node:48265) ExperimentalWarning: Import assertions are not a stable feature of the JavaScript language. Avoid relying on their current behavior and syntax as those might change in a future version of Node.js.
(Use `node --trace-warnings ...` to show where the warning was created)
(node:48265) ExperimentalWarning: Importing JSON modules is an experimental feature and might change at any time
World address 0x7bd472fc101d3a7caf3064b7c5788965cb34685f
running "forge test --fork-url http://<IP_ADDRESS>:8545"
[⠢] Compiling...
No files changed, compilation skipped
Error: Error:
Could not instantiate forked environment with fork url: http://<IP_ADDRESS>:8545

Context:
- Error #0: Failed to get latest block number
- Error #1: error sending request for url (http://<IP_ADDRESS>:8545/): error trying to connect: tcp connect error: Connection refused (os error 61)
- Error #2: error trying to connect: tcp connect error: Connection refused (os error 61)
- Error #3: tcp connect error: Connection refused (os error 61)
- Error #4: Connection refused (os error 61)
Error running "forge test --fork-url http://<IP_ADDRESS>:8545"
    at o (file:///Users/qbzzt/run/tutorial/node_modules/.pnpm/@latticexyz+common@2.0.0-next.14_typescript@5.1.6_zod@3.22.4/node_modules/@latticexyz/common/dist/foundry.js:2:23)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async x (file:///Users/qbzzt/run/tutorial/node_modules/.pnpm/@latticexyz+common@2.0.0-next.14_typescript@5.1.6_zod@3.22.4/node_modules/@latticexyz/common/dist/foundry.js:1:615)
    at async Object.handler (file:///Users/qbzzt/run/tutorial/node_modules/.pnpm/@latticexyz+cli@2.0.0-next.14_google-protobuf@3.21.2/node_modules/@latticexyz/cli/dist/commands-UYAATBY6.js:20:1361)
qbzzt@Oris-MacBook-Pro contracts %

When I try not to tell it about an existing World, I get a similar problem:
qbzzt@Oris-MacBook-Pro contracts % pnpm mud test --rpc $ETH_RPC_URL &| tee x
(node:49000) ExperimentalWarning: Import assertions are not a stable feature of the JavaScript language. Avoid relying on their current behavior and syntax as those might change in a future version of Node.js.
(Use `node --trace-warnings ...` to show where the warning was created)
(node:49000) ExperimentalWarning: Importing JSON modules is an experimental feature and might change at any time
running "anvil --block-base-fee-per-gas 0 --port 4242"

 Deploying MUD contracts to RPC http://<IP_ADDRESS>:4242

running "forge build"

mud test --profile=local

This is effective


drop print

Disclaimer: This review was made by a crew of AI Agents.
Code Review Comment for PR #1755
Overview
This pull request presents a modification to src/crewai/knowledge/source/base_knowledge_source.py, specifically the removal of a debug print statement from the _save_documents method.
Positive Aspects

Code Cleanliness: The removal of debug statements enhances the readability and maintainability of the code, following best practices in production.
Improved Production Readiness: This aligns with good practice by ensuring production code is free from potentially sensitive output that could clutter logs.

Potential Concerns

Debugging Limitations: The removal limits visibility into _save_documents, which could pose challenges for debugging issues in production environments.
Absence of Logging: Without a logging mechanism, developers lose critical insights that could help diagnose problems effectively.

Recommendations
1. Introduce Structured Logging
Replacing the print statement with appropriate logging will improve the maintainability and debuggability of the code:
def _save_documents(self):
    logger.debug(f"Saving {len(self.chunks)} chunks to storage")
    self.storage.save(self.chunks)

2. Enhance Debug Detail
Including additional context in logging might provide better operational insight:
def _save_documents(self):
    logger.debug({
        "action": "saving_documents",
        "chunk_count": len(self.chunks),
        "storage_type": type(self.storage).__name__
    })
    self.storage.save(self.chunks)

3. Improve Documentation
Enhancing the function docstring to clarify parameters, possible exceptions, and expected behavior will aid future code maintainers.
4.  However, substituting it with a logging framework and implementing error handling and documentation improvements will enhance debugging capabilities and overall code quality.
Prioritize implementing the logging features as they provide immediate benefits for maintainability and troubleshooting.


Blanket

CurseForge link
https://www.curseforge.com/minecraft/mc-mods/blanket
CurseForge Mod Distribution
Allowed
Modrinth link
https://modrinth.com/mod/blanket
Source/other link
https://github.com/BlanketMC/blanket-client-tweaks
Mod file size
1.07 MB
License
(any other license)
What it does
Fix client-side bugs
Why should it be in the modpack
Fix many client-side bugs, also has some QoL improvements and tweaks And Its Very Configurable
Why shouldn't it be in the modpack
Untested, Still In Alpha
Additional details
No response

What specific bugs do you want fixed?

Seems that blanket has bug fixes implemented that mojang already marked as fixed:
WEB-5587,
MC-246465,
MC-53850
And the ones that are not resolved:
MC-238384,
MC-187100,
MC-175622
and MC-248223.

The fixes I find interesting:

Fix chat lag on multiplayer servers

That's not the only mod claiming to fix it, but I really fail to see the issue in 1.19.2/1.19.3 so benchmarks are needed for the bugfix and without, if I should care at all.

Optimized the getBiome call to be 25% - 75% faster

I'm pretty sure that if it really matters, it is already implemented in Sodium/Lithium/etc.
Rejecting this due to the mod not adding anything worthwhile.


FS-1406 application reference

Pass through new friendly application reference to application submitted email

Replace application id with application reference

I have already implemented these changes @harryyo


Move public API headers

This change moves the various public API headers at the root of the repository under include/slang/. In order to not break (much) existing code, forwarding headers for slang.h and slang-gfx.h were kept at the root of the repository.
The Slang codebase itself made use of the ComPtr type and related utilities provided by headers at the root, so various changes were required to the internal code to get it to build with this change.
I know that there are outstanding changes to slang-gfx.h in flight, so this change may be best left until after those are completed. It is worth noting that this change does not alter the contents of the any of the headers, so merges should be navigable just by clobbering the files in their new locations.

That’s a good point about gfx. Should we consider moving that headed to be at include/gfx/gfx.h?

I guess I worry gfx is too generic a sub folder name, slang-gfx I think would be preferable.

We need to make sure to update our github action config to include these additional headers in our release packages.

Closing for now, since subsequent changes to the headers would cause merge pain here.


Any possibility to support 900M param Pixart Sigma models?

PsuedoTerminal is doing some really interesting work with expanding the model depth from 28 -> 42 and finetuning on new data, for example: https://huggingface.co/ptx0/pixart-900m-1024-ft
I tried hacking the code to increase the depth, but while I could get it loading and eliminated the missing/dropped keys error (by modifying the depth values across all the files), there was clearly a render error- sort of recognizable shapes with a lot of noise and distortion. Maybe someone smarter than me can take a crack at this?

https://github.com/city96/ComfyUI_ExtraModels/pull/65 this is being worked on right now.

You may change depth parameter directly from 28 to 42 for now, as it's a hardcoded number.
According to 900m config model has 42 num_layers insdead of 28 for original pixart models.
Path: ComfyUI\custom_nodes\ComfyUI_ExtraModels\PixArt
In files: diffusers_convert.py conf.py files

This is now merged, so you can now use 900M and 600M sigma models in ComfyUI_ExtraModels


Show image preview

If the response is an image, the content of response body will show as (image)

Can we just show the image preview instead of display (image)? Just like the Chrome Dev Tool:

The reason why AnyProxy did not show images is that they take too much space. I'll try to find a way to handle these big files.

Image-preview is now supported. You are welcome to have a try.


[UBS courier] Inconsistent title of the service on the order page

Environment: OS: Windows 10 Pro, Browser: Google Chrome, Version 90.0.4430.212
Reproducible: Always
Build found: 10.06.2021
Preconditions
Login to GreenCity as a User https://ita-social-projects.github.io/GreenCityClient/#/
Steps to reproduce

Go to order creation
Pay attention to the title of the service

Actual result

Expected result

User story and test case links
User story #2198
Test case GC-2039

fixed in https://github.com/ita-social-projects/GreenCity/issues/2930


[Reference Entity] Add a new option "pull" to pull references before delete

Added a new option to the ReferenceIntegrity mapping to fix issue #1360  :
@Gedmo\ReferenceIntegrity("pull")

This option aims to remove ONLY the document which is removed from the Referenced documents collection on the owning side. (The nullify option sets the references collection on null).

renamed '$object' variable used twice.
replaced 'entity' anotations with 'document.

closes #1360
Please review

thanks, would be great if you could pull a test as well

Done in #1364


Left sidebar: Add hover controls to heading and stream rows.

This PR does the following:

It implements @terpimost's design for showing icons in DM and Channel heading rows only upon hover (controls remain visible for touchscreen devices).
It updates the icons and colors for the heading-row controls.
It moves the New topic icon and functionality to stream rows from their previous location in the channel popover. The New topic icon is always shown on the active channel, and available as a hover control on all other channel rows.
It adjusts the --color-vdots-hint values to be more subtle, and therefore less distracting, on touchscreens.
It refactors opacity and hover opacity for muted channels and muted topics, including restoring missing opacity value on muted topics that was lost in a recent regression.
It presents unread markers as icons, rather than CSS-drawn circles, in part to correct an alignment issue introduced by the unread-offset for the New topic icon.
It falls back to variables for styling generic links on <a>
It restores <a> to channel rows, making them responsive to Cmd/Control-clicks.

Fixes #31801.
CZO discussion

Screenshots and screen captures:

Before
After

Touchscreen, light
Touchscreen, dark

Interactivity:

Self-review checklist

[x] Self-reviewed the changes for clarity and maintainability
(variable names, code reuse, readability, etc.).

Communicate decisions, questions, and potential concerns.

[x] Explains differences from previous plans (e.g., issue description).
[x] Highlights technical choices and bugs encountered.
[x] Calls out remaining decisions and concerns.
[x] Automated tests verify logic where appropriate.

Individual commits are ready for review (see commit discipline).

[x] Each commit is a coherent idea.
[x] Commit message(s) explain reasoning and motivation for changes.

Completed manual review and testing of the following:

[x] Visual appearance of the changes.
[x] Responsiveness and internationalization.
[x] Strings and tooltips.
[x] End-to-end functionality of buttons, interactions and flows.
[x] Corner cases, error conditions, and easily imagined bugs.

Left a minor comment on CZO.

Updating the chat.zulip.org deployment.

@alya said:

This icon should also be updated if we're changing the left sidebar:

This is up in its own little PR, #31950

Updating the chat.zulip.org deployment now.

Reviewed this; I think it's good to merge pending https://chat.zulip.org/#narrow/channel/101-design/topic/.22New.20topic.22.20in.20stream.20menu/near/1962463.

Merged, thanks @karlstolley!


[Android][borderStyle] borderStyle dashed does not work without borderRadius being set.

dashed border does not work if borderRadius is not set.
cc @kmagiera @mkonicek

This is still not working as of 0.19

Image fading doesn't work either, there's some bug with borderRadius: https://github.com/facebook/react-native/issues/5545
Anyone with Android experience up for sending a fix?

Setting border to a specific edge like borderBottomWidth also breaks even with borderRadius set.

Support for borderBottomWidth is broken on iOS as well.

:+1:
I get the warning Unsupported dashed / dotted border style when I try this on a View on iOS:
borderBottomWidth: 1,
borderBottomColor: '#e6e5e5',
borderStyle: 'dashed',

on Android I see a border (not dashed) without any warnings

Is this still an issue?

Yes, i have a same problem of ms88privat

@satya164 I'm on 0.21.0 and it is not working. Not sure about newer Versions

I have the same problem here on 0.22 and on iOS: Unsupported dashed / dotted border style.

I have the same problem here on 0.24.1 and on Android: Unsupported dashed / dotted border style.

It did not work when  try this on View ?
borderBottomWidth: 1, borderStyle: 'dashed',

Same here

@yaofly2012 did you set the border Color?

I already set the color on mine, but still would not appear

It works on borderWidth but not on specific borders like borderTopWidth

@samerce why is this issue closed?  As @ajoshdee mentioned, it still does not work on specific borders like borderTopWidth

Again, why is this closed? It does not work as expected.

Im on RN 0.38.0 and still cant apply style on specific borders.

Why issue is closed? it doesn't been fixed

@idris @Mr-Hero this issue only said that it doesn't work without border radius. what you are describing is a different issue. Please file a new bug report for that.

@yaofly2012 l got the same issue

Why it is closed. Still not working on android

Neither dotted nor dashed

@hramos Can you reopen please? The issue is not fixed. Even on latest RN.

Instead of re-opening a stale issue, I suggest opening a new issue and filling out the template.

Don't forget that this issue is tracked first in search engines, SEO is important

I think this issue will never be resolved by facebook. Waiting for PRs

Can confirm borderStyle is not working w/o borderRadius (v0.49 on Android)

Still not working, why its closed?

same issue, any solution?

This still isn't working.

This is still not working "react-native": "^0.55.2".

This isn't working for me either, should this be reopened?

Will be watching this issue in any case

Why closing this issue? This is not solved in any way.

Will creating a new issue help?

so where's the new issue?


Switches only affect Non-Zigbee HomeKit Devices

Happy New Year!
I'm experiencing an issue that I find difficult to understand, let alone explain. Please bear with me:
I have a sizeable HomeKit installation with combination of platforms such as Homebridge deCONZ (Zigbee), IKEA Tradfri hub (Zigbee), Homebridge Tahoma, Netatmo devices, node-RED (node-red-contrib-homekit-bridged) and more. An Apple TV acts as the HomeKit hub. Most lights and switches are joined to Homebridge deCONZ, with a minority being pure IKEA Tradfri. deCONZ is hosted on a Raspberry Pi 3b, which also hosts HomeBridge and HomeBridge deCONZ (<IP_ADDRESS>).
Scenes and automations are configured in HomeKit. There are no scenes or other logic configured in deCONZ. Switches and lights are logically placed in deCONZ Groups ("rooms") as per their physical layout, and this layout is largely mirrored in HomeKit Rooms.
Problem: For some HomeBridge deCONZ Zigbee input devices (switches and motion sensors) that are configured through HomeKit to perform actions on multiple devices across both Homebridge deCONZ and non-deCONZ platforms, only the non-deCONZ devices will physically be affected by the actions.
Example 1a: The IKEA TRADFRI On/Off switch "Living Room Upstairs Switch" placed at the top of the stairs is joined to deCONZ. Button 2 Single Press is assigned to trigger a scene (named "Going Down") that turns off all lights on that floor. When the button is pushed, the scene is triggered in HomeKit and all devices in HomeKit are shown to have entered the desired state, but the action actually ends up affecting only the devices that are not part of the deCONZ network. The physical result is that the IKEA Tradfri hub lights and sockets are turned off, the Apple TV is turned off, but the deCONZ lights and sockets remain ON.
Example 1b: Activating the scene on my watch or on my phone, or by voice ("Siri, Going Down") results in a 100% desired state, and all lights and devices are physically switched off.
Example 2a: The IKEA TRADFRI On/Off switch "Second Kitchen Light Switch" placed physically in the Second Kitchen is joined to deCONZ. Button 2 Single Press is assigned to directly trigger an action on 3 accessories, turning off Second Kitchen Bench Lights (IKEA), Second Kitchen Drawer Lights (IKEA) and Second Kitchen Mezzanine (deCONZ). When the button is pushed, only the Second Kitchen Bench Lights (IKEA) and Second Kitchen Drawer Lights (IKEA) are physically switched off.
Example 2b: Issuing "Siri, turn off lights in Second Kitchen" results in all of Second Kitchen Bench Lights (IKEA), Second Kitchen Drawer Lights (IKEA) and Second Kitchen Mezzanine (deCONZ) being physically turned off.
I have tried to experiment with parallelRequests, waitTimePut and waitTimeUpdate to no success. I'm at my wits end on where to go from here. Any ideas?
The entirety of config.json is:
{
    "bridge": {
        "name": "[…]",
        "username": "[…]",
        "port": […],
        "pin": "[…]"
    }

    "platforms": [
        {
                "platform": "deCONZ",
                "hosts": ["<IP_ADDRESS>"],
                "_bridge": {
                        "username": "[…]",
                        "port": […]
                }
        }
    ]
}

Environment:
Homebridge v1.7.0 (HAP v0.11.1)
homebridge-deconz 1.0.9
Deconz: 2.25.0 (latest)
Adapter: ConBee II, firmware: 26780700 (latest)
node v20.10.0

You need to determine whether your issues are caused by HomeKit, by Homebridge deCONZ, or by deCONZ.  As your Zigbee devices work when recalling a HomeKit scene directly, or through Siri, I doubt the issue is with Homebridge deCONZ.  I would need to see a full debug log file to be certain.
I would look for opportunities to simplify your setup (making it more robust).  I recommend to use deCONZ rules to control Zigbee devices from Zigbee controllers, and only use HomeKit automations where non-Zigbee devices are involved.  Also, I would pair all IKEA devices to deCONZ and ditch the Trådfri hub.
You should see a notification by Homebridge decONZ in the Homebridge log of the button event that Homebridge deCONZ sends to HomeKit, and of the characteristic changes it receives from HomeKit when the HomeKit scene is recalled.  Make sure LogLevel is 1 or higher to see these info messages.  If they don’t appear, the issue is in HomeKit, see 1) below.
With LogLevel 2, Homebridge deCONZ logs the API calls it makes to deCONZ.  You should see log messages for the PUT requests to the lights resources corresponding to the accessories in the HomeKit scene.  There should be a corresponding OK response by deCONZ that it has accepted the request.  If you see error responses, the issue is with deCONZ.  If you see OK responses, but the devices don't change state physically, the issue is with deCONZ, see 2) below.
Ad 1.
HomeKit automations run on your home hub (I.e. your AppleTV) whereas recalling a HomeKit scene, or invoking Siri. runs on your iPhone, iPad, or whatever.  Unfortunately there is no way to check whether the HomeKit automations and scenes where synced correctly to your home hub; best disable them, wait for a couple of minutes, and re-enable them to trigger a sync.
Ad 2.
There might be errors in the deCONZ log, with the right logging options.  See https://github.com/dresden-elektronik/deconz-rest-plugin/wiki/Zigbee-Error-Codes-in-the-Log.

Thank you for taking the time, @ebaauw . Turns out the rabbit hole was rather deep this evening.
It seems like the problem is (at least tangentially) related to the desire of deCONZ to interpret the button press as a command to all devices, at least in the Phoscon UI. What I observed was the following: Shutting down Homebridge deCONZ, I had a look at the Phoscon UI, and noticed that pressing On/Off on the IKEA TRADFRI On/Off switch resulted in ALL lights being turned On/Off in the UI, but not physically!
This is an example of what homebridge-deconz logs simultaneously when I push a button on the remote, with no actions configured for the buttons (abbreviated for readability):
[homebridge-deconz] Outside Back Door: On: set to false (from true)
[homebridge-deconz] Hallway Stair Lights: On: set to false (from true)
[homebridge-deconz] Living Room High Lamp: On: set to false (from true)
[homebridge-deconz] Office Ceiling Lights: On: set to false (from true)
[homebridge-deconz] Second Kitchen Mezzanine: On: set to false (from true)
[homebridge-deconz] Kitchen Dining Table: On: set to false (from true)
[homebridge-deconz] Kitchen Bench Ceiling Lights: On: set to false (from true)
[homebridge-deconz] Living Room Upstairs Switch Off: Programmable Switch Event: Single Press

Note that the switch at this time had no actions configured to any button in HomeKit, had been freshly re-paired, and was never configured to control some of these lights, such as the Outside Back Door.
This seems like the same, or at least similar to, the issue described in https://github.com/dresden-elektronik/deconz-rest-plugin/issues/846 and https://github.com/dresden-elektronik/deconz-rest-plugin/issues/853 . Perhaps foolishly, I tried using the deCONZ GUI to unbind the remote from 0xFFF0, but the remote reported that the binding was unknown. I also tried adding the button to a specific group in Phoscon, but this didn't change the behaviour.
This lead me to https://github.com/dresden-elektronik/deconz-rest-plugin/wiki/What-you-can-do-when-a-switch-turns-(almost)-all-devices-on-off by @SwoopX . Here, the guidance is to edit zll.db by issuing update groups set state="deleted" where gid="0xFFF0"; and then re-pairing the switch. This works and solves the problem of the switch turning On/Off all lights in the deCONZ UI. I thought I had reached salvation! However… starting up Homebridge deCONZ, the platform was unhappy that /groups/0 returned a 404 and refused to load. Reverting the change to zll.db made Homebridge deCONZ load again. I thought that as long as the switch was re-paired while 0xFFF0 was disabled, I was home free – but alas, now the switch controls all lights again.
To top it off, Homebridge deCONZ marked all of my accessories as stale and re-discovered them in the process, so now all of my devices are back in the default room, all automations and scenes are lost, its 23:59 on a work night and I hope I will have warm water tomorrow as my electric water boiler is on a deCONZ Zigbee relay…
I will restore basic functionality to get the house through the night, and then I will follow up tomorrow.
Any help from hereon out is greatly appreciated.

/groups/0 is supposed to include all lights.  Typically, it is mapped to group ID 0xFFF0.  It should be managed by deCONZ automatically, and should not be removed.
Some controllers will send a broadcast when no lights nor groups are bound to its client clusters.  So when the joining process is incomplete, or when the device support is missing the bindings, these controllers work on all lights.  To change this, you can manually bind the client clusters to a (real) group, or to the coordinator device.  Depending on the controller, you might need to bind all relevant client (grey) clusters: On/Off, Level Control, Color Control, Scenes.
Some (IKEA) controllers don’t (no longer with recent firmware) honour group bindings.  For these, each light needs to be bound individually.  Make sure to include a binding to the coordinator as well, so deCONZ can eavesdrop the commands to control the lights and generate the appropriate button events.
From your log: deCONZ updates the state of the lights, which, by design, Homebridge deCONZ forwards to HomeKit.  Only after that, deCONZ reports the button event, which, again Homebridge deCONZ forwards to HomeKit as a Single Press.   By design, on recalling a scene, HomeKit only updates the characteristics that need to be changed, so the light states that were just incorrectly updated aren’t included.

Mightily appreciated. I understand that this behaviour on the part of Homebridge deCONZ, and it makes sense as HomeKit should not try to change the state of devices that are already in the desired state.
I brought back /groups/0, and then I spent a while troubleshooting as per your suggestions. Binding the controller  lights in deCONZ did nothing to change the behaviour, and I tried a multitude of lights, (real) groups and/or the coordinator.
I racked my brain as to how this might be solves, until I
The switches in question were all running SW Build ID 2.2.008 (Vendor 0x117c, Image 0x11c5, Version 0x22008631). I hadn't thought about this for ages, so I updated them to 24.4.6 (Version 0x24040006) – and Lo and Behold, the switches now work exactly as I want them to, without any specific bindings to anything at all! The scenes are fast and responsive, and nothing lags behind any longer.
So, quite a journey full of learning and discoveries, and a few wrong turns along the way, with a rather simple and obvious solution that escaped me from the beginning. Thanks for your help @ebaauw !


Usability Testing

Ask User 3 on their thoughts of the landing pages (usability testing)

User 3: Michael
I implemented the 5 second usability test with user 3 and the first thing that the user noticed on both landing pages is the image of the pizza and the slogan next to it. That gave him the impression that we would be selling pizza. One thing that the user highly recommended is to find a more fitting hero section image. The font is also another thing that should be improved on.


Update HttpClientExtensions.cs

This doesn't appear to change the actual execution of the code.
@engenb I talked to @tswerve and I think we're OK just closing this PR without merging.


RIGHT and FULL OUTER JOINs are not currently supported

This isn't a big issue and I don't think it's supported in SQLite, which is what datasette is built on. At the very least we could add this to some documentation with some instructions on work arounds.

@odscrachel @odscjames I think SQLite now supports Right and Full Outer Joins.
https://www.sqlitetutorial.net/sqlite-full-outer-join/
https://www.sqlitetutorial.net/sqlite-right-join/

@BibianaC Cool! Can you just run some test queries on our live server and make sure we are running a good enough version of SQLite/Datasette? We may be running an older version.

@odscjames
I have ran a query in https://datasette.codeforiati.org/iati. It is not supported.

I don't see where is the datasette is hosted to check what version is running.
I have checked the change history of SQLite.
2022-06-25 (3.39.0)
Add (long overdue) support for [RIGHT and FULL OUTER JOIN](https://www.sqlite.org/lang_select.html#rjoin).

@odscjames
We are using Ubuntu 22.04 which comes with SQLite 3.31 (The SQLite fix was done two months later 22.06).
The latest Ubuntu is 24.04 which comes with SQLite 3.45.
Is it viable to upgrade now to the latest Ubuntu?

That's probably the easiest fix, and a good thing to do anyway. Thanks.

Whoops, the language in my commit message wasn't quite correct - this isn't fixed until new server goes live


Original motion is not projected after amendment

Used version: dd66e7505dcac024524f0f6bd3eab96cef069540
When creating an amendment to a motion that is in complex workflow, a amendment takes the system default workflow (which may differ to the selected workflow of a motion) on creation and the original motion cannot be projected anymore. After switching the workflow type to the same of the original motion, the projection works again.
How to reproduce?

create a motion in complex worklfow (definition after installation)
create a amendment to this motion (will be created in simple workflow)
Try to project the original motion

Debugging Output
JS console gives the output: motion 1 can not be on the state with id 1

Confirmed.
@FinnStutzenstein it seems you want to provide a server side fix for that? That could be fixed in the client as well during the creation of an amendment, however, I would not recommend that.
Furthermore, changing the workflow of an amendment should not be possible itfp

Furthermore, changing the workflow of an amendment should not be possible

Hmm, maybe someone has another workflow for amendments... @emanuelschuetze ? If so, we should have a config field for the default workflow of amendments and make sure, that the client/server can handle different workflows of main motion and an amendment. Why even does the server throw an issue (question to myself ;), the output from Eric suggests that this is server output..)?
If not, I would agree with that. What should happen, if the main motion's workflow ill be changed? I would suggest, that also the states of all amendments will be reset to the new workflow's first state. Then we should make a workflow change non-trivial in the UI, e.g. moving the workflow selector out of the edit-view and handle this seperate with a dialog confirming changing all amendments, too.

Yep, there could be other workflows that are used for amendments, for example, an amendment may not be observed by several extra commissions like an original motion and the workflow for deciding an amendment can be much quicker. So having different workflow options for original motion and amendments is useful in my opinion.
So a fix that the projection is working for motions with amendments that have different workflows than the motion might be the best way.


Patch for Twitch API changes 2024/5 and minor tweaks for future contributors

Added option to press Enter instead of "y" to run with console. It's really annoying to type Y every time when debugging.
Added debug options to manual.txt. If this is not wanted, please add them somewhere else for the next contributor. I only found these options by chance when sifting through the code.
Removed help=argparse.SUPPRESS for debug options, so that they are shown with -h, for the same reason.
Added new GQL operation in constants.py for the fix
Added fix to main.py. I left the old code untouched, in case it's useful when cleaning things up.

Forgot to remove some debug print statements


feat(feedback-container): adding feedback-container component

Closes #
Initial Setup of Feedback Container component
Changelog
New

Feedback Container component

Changed

N/A

Removed

N/A

Testing / Reviewing
N/A

@priyanshu891 looks like you need to run pnpm install at the root of the project to update the pnpm-lock.yaml

opened new PR


Updating readme files for all the projects

Your name -  Suyash Dabral
Your LinkedIn profile - https://www.linkedin.com/in/suyash-dabral-664011195/
Hey @AdarshRawat1 ,
There is no readme files for any of the projects section in the repository.
Please assign me that.

Hey @s-dabral... actually the notebooks over there for each of the model will be well-documented. so you don't have to worry about it .. btw we'll encourage you to check Contributing.md by tommorow to get idea for contribution.


Retail - Complete documentation for D365 IOM

Work in progress.
Trainings completed. Working on converting the topics covered during IOM training to guidance for industry.

Targeting end of April 2022


DB transaction setup

Postgres DB transaction queries were implemented in Go using sqlc and unit tests were also written to verify the functionality.

No merge conflicts found. Good to merge with Main branch.


Unable to add a saml private key using tower cli

trying to use tower-cli to input the saml cert private key..
I got the public cert to be accepted by removing the headers and removing newlines.
For the private key I tried the following

first just removed the headers of the key and removed the newlines
then just tried without doing anything.

All attempts say key is invalid. But if I input the key into the UI it works.
Here is the output I get, key is redacted
# first just try remove new lines
(tower-cli)% tower-cli setting modify SOCIAL_AUTH_SAML_SP_PRIVATE_KEY `cat aws.key | tr -d '\n'`
Error: no such option: -----BEGIN

# then try remove --begin private key -- and --end private key -- headers
(tower-cli) :% tower-cli setting modify SOCIAL_AUTH_SAML_SP_PRIVATE_KEY `sed -e '/^-/d'  aws.key | tr -d '\n'`
Error: The Tower server claims it was sent a bad request.

PATCH https://mytowerserver/api/v1/settings/all/
Params: None
Data: {"SOCIAL_AUTH_SAML_SP_PRIVATE_KEY": "[removed]"}

Response: {"SOCIAL_AUTH_SAML_SP_PRIVATE_KEY":["Invalid certificate or key: u'MIIEvQIBADA[removed]'..."]}

This is a problem for direct input, but I think that we have a workaround that may be acceptable if it can be documented fully.
From my testing, I believe the following syntax should work:
tower-cli setting modify SOCIAL_AUTH_SAML_SP_PRIVATE_KEY @aws.key -v
This won't remove the newlines, but I'm not sure off the top of my head if that's necessary or not for the server to correctly use the key.
We added the option to get contents from a file by prefixing with the "@" character due to similar hangups that people experienced.


ESLint 'no-duplicate-imports' rule

Describe the problem and solution
중복된 import를 막기 위해 ESLint rule을 추가합니다.
https://eslint.org/docs/latest/rules/no-duplicate-imports

Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines
[X] Check that there isn't already an issue

Duplicate of #667


Cannot find module 'ember-template-compiler.js' in bower_components

I could use some assistance. After installing current ember-admin I get the following error.

Cannot find module '/home/vagrant/projects/alt3-frontend/bower_components/ember/ember-template-compiler.js

My setup:

ember-cli: 2.12.1
node: 7.7.3

I am having the same issue

Same here, any clues?

Not a single one. Unsure why this is not getting any attention. Is this repo still maintained?

Adding ember-source into devDependencies seems to work for me.

I had to bower install ember#2.12.1 --save-dev to get past this. It's not ideal to have 2 versions of Ember installed, one in Bower and the other in NPM/Yarn, but this is a workaround.

Seems that this issue is still happening, at least it was happening to me, I fixed it by adding  "ember": "~2.18.0"  into the dependencies in the bower.json file, it would be nice if anyone could try to understand exactly why this is happening since it looks like last merged PR did not fix it. Thanks


Performance improvements

The package seems to be quite slow compared to other Atom packages. Is there anything that can be done about performance?
This package added 243ms to startup time.

I'm currently it take 10x the time of other pieces (OSX 10.11.3 Atom 1.6.2) - is there a debug switch I can flip on?

Hrm, really? That's strange, because the package is hardly even taking 19ms to load for me...

And in Timecop:

I think I've tracked down the culprit - an intermittent DNS fault from my ISP may have been introducing the delay to the startup.  Switching to Google's Public DNS (<IP_ADDRESS> / <IP_ADDRESS>) reduces startup to 13ms for me too, so possibly something to look at for async startup in the future, but in the mean time this may help someone.

Bizarre. Maybe the metrics package had something to do with it? It might've been trying to report the startup times or something...

It's taking 578ms to load for me (next one is 57ms). How should I debug this?

@iovis9 Is this happening on multiple reboots?
On first load, the package needs to compile the Less sheets if they've changed (which it will if there's been an update released).
(Note this delay won't happen in V2)

It's been happening over the course of at least 5 months.

What other packages do you have installed? Specs, operating system?

 I'm having worse problems with them.
A restart of the app now makes it load faster, you're right.

Well, that issue won't exist in V2 either way, which uses a config-driven approach instead of a CSS-driven one. :)


bisect deps

the most recent upgrade was broken. now have to find what broke again

at SK version 521 i am getting this issue Dynamic require of "fs" is not supported again https://github.com/sveltejs/kit/issues/4079
https://app.netlify.com/sites/swyxkit/functions/render?scope=branch:bisectdeps
cc @PH4NTOMiki


[mesheryctl] Grammar: Run-on sentence in system stop for Docker platform

See mesheryctl Command Tracker for current status of commands.
Current Behavior
Scenario: Meshery is stopped. Current-context is Kubernetes.
➜  ~ mesheryctl system status
Meshery is not running, run `mesheryctl system start` to start Meshery

Desired Behavior
➜  ~ mesheryctl system status
Meshery is not running. Run `mesheryctl system start` to start Meshery.

Add two periods and capitalize "R"un.

Contributor Resources

mesheryctl Contributing Guide
Beginner's guide to contributing to Meshery and mesheryctl
mesheryctl Command Tracker
Meshery CLI Commands and Documentation

@leecalcote currently, mesheryctl system status for Kubernetes platform does not show this error.
The platform must be docker. Will add the check in Kubernetes, as well as change the logs.


Refactor I/O functions for cross-platform support and improve memory …

This PR introduces cross-platform support for the I/O functions and refactors memory operations for better portability and maintainability. The changes include:

Cross-Platform I/O Handling:

Added conditional compilation using #[cfg] attributes to separate Windows-specific implementations of _getch and _putch.
Implemented alternative getch and putch functions for non-Windows platforms using standard Rust libraries (io::stdin and print).
Ensured all I/O operations work consistently across different platforms, removing the dependency on platform-specific C functions for Unix-based systems.

Memory Handling Improvements:

Adjusted memory operations such as shift_right, shift_left, increment, and decrement to work seamlessly with the new I/O implementations.
Refined the adjust_size function to dynamically handle memory expansion based on pointer movements.

Code Maintenance and Clarity:

Improved code readability and maintainability by using conditional compilation to isolate platform-specific code.
Added detailed comments to document the purpose of each function and explain platform-specific behavior.

How to Test

Windows:

Verify that the _getch and _putch functions work as expected.
Test input, output, and pointer manipulation functions to ensure correct behavior.

macOS / Linux:

Confirm that getch reads input correctly using io::stdin.
Validate that putch outputs the correct characters and that output flushes stdout properly.

Additional Notes
This refactor ensures that the codebase is more robust and maintainable across different platforms, reducing the likelihood of platform-specific bugs and improving overall compatibility.

This PR doesn't seem to change much other than the getch and putch implementations, so I'm not sure why you included the Memory Handling Improvements section, when shift_right, shift_left, increment, decrement, and adjust_size have not been changed.
I appreciate the better comments, but they aren't as detailed since they don't describe everything that happening in the context of the function call. Admittedly, the entire code base should be properly documented, since the current comments are just artifacts of when I wrote these functions, and I never got the chance to fix them.
The How to Test section is confusing since it doesn't really give any instructions for testing. It would be much better if you gave code examples with expected output.
As commented before, the intended implementation for getch should terminate once a single byte has been read. However, this implementation allows for multiple characters to be typed in the terminal and requires the enter key to be pressed before the first byte is read into memory.
This code should terminate immediately after 1 key press and only show a single character in the terminal.
,.

1. Clarification on getch Implementation:
I understand your concern regarding the behavior of the getch function on non-Windows platforms. The reason for the modification was to address a linking error encountered during compilation on macOS. Specifically, the error:
This error was caused by the use of _getch and _putch, which are not available in Unix-based systems. Therefore, I implemented an alternative getch function using Rust's standard library to provide cross-platform compatibility.
I acknowledge that this implementation deviates from the expected behavior of getch by requiring an Enter key press. This happens because stdin().read() in Rust's standard library reads input only after a newline character is entered. This isn't ideal for mimicking the behavior of getch, and I will look into using a library like crossterm or termion to replicate the single-character read functionality across platforms without relying on C functions.
2. Flushing in putch Function:
I see your point about the redundant call to flush() in putch. Since flush() is already called in memory::output, I will remove the flush call from the putch function to avoid unnecessary I/O operations. This will ensure that flushing occurs only once, minimizing performance overhead.
3. Memory Handling Improvements:
You're right—there weren't any direct changes to memory handling functions like shift_right, shift_left, increment, decrement, and adjust_size in this PR. I included the "Memory Handling Improvements" section because I reviewed these functions to ensure they worked seamlessly with the modified I/O functions. I now realize that including this section might have been misleading since there were no actual modifications made to those functions. I'll update the PR description to reflect that.


[PowerRename] Dutch translation does not fit in window.

Microsoft PowerToys version
0.37.2
Running as admin

[ ] Yes

Area(s) with issue?
PowerRename
Steps to reproduce
Open Dutch version of PowerRename.

I have a 3840x2160 UHD resolution with a scaling of 175%.
A shorter version could be
Geselecteerd: 3
Te wijzigen: 0

✔ Expected Behavior
I was the expecting the text to fit.
❌ Actual Behavior
It did not fit.
Other Software
No response

Good point. I've seen it too. Changing the text should be simple, but there's also the issue of #888, which is taking a lot of time.


Add TypeSpec for Azure.Communication.Messages service

Data Plane API - Pull Request

API Info: The Basics
Most of the information about your service should be captured in the issue that serves as your API Spec engagement record.

Link to API Spec engagement record issue:

Is this review for (select one):

[ ] a private preview
[ ] a public preview
[x] GA release

Change Scope
This section will help us focus on the specific parts of your API that are new or have been modified. Please share a link to the design document for the new APIs, a link to the previous API Spec document (if applicable), and the root paths that have been updated.

Design Document:
Previous API Spec Doc:
Updated paths:

Viewing API changes
For convenient view of the API changes made by this PR, refer to the URLs provided in the table
in the Generated ApiView comment added to this PR. You can use ApiView to show API versions diff.
Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
Swagger-Suppression-Process
to get approval.
❔Got questions? Need additional info?? We are here to help!

 Contact us!
The Azure API Review Board is dedicated to helping you create amazing APIs. You can read about our mission and learn more about our process on our wiki.

💬 Teams Channel
💌 email

Click here for links to tools, specs, guidelines & other good stuff
Tooling

Open API validation tools were run on this PR. Go here to see how to fix errors
Spectral Linting
Open API Hub

Guidelines & Specifications

Azure REST API Guidelines
OpenAPI Style Guidelines
Azure Breaking Change Policy

Helpful Links

Azure DevTools Wiki

Checks stuck in `queued` state?
If the PR CI checks appear to be stuck in `queued` state, please add a comment with contents `/azp run`.
This should result in a new comment denoting a `PR validation pipeline` has started and the checks should be updated after few minutes.

@glorialimicrosoft Can you please work through the CI issues so that the PR passes the checks? Thanks

Please update the PR description to indicate if this API version is private or public preview:

Opened Azure/azure-sdk-tools#7593 for the APIView emitter crash.

@chunyu3 , can you have a look at build error for the generated SDK?
Start to build sdk: /mnt/vss/_work/1/s/azure-sdk-for-net/sdk/communication/Azure.Communication.Messages
05:03:40.690 cmdout 	[Invoke-GenerateAndBuildV2.ps1] MSBuild version 17.4.0+18d5aef85 for .NET
05:03:40.694 cmdout 	[Invoke-GenerateAndBuildV2.ps1] MSBUILD : error MSB1003: Specify a project or solution file. The current working directory does not contain a project or solution file.
05:03:40.799 cmderr W 	[Invoke-GenerateAndBuildV2.ps1] GeneratePackage: /mnt/vss/_work/1/s/azure-sdk-for-net/eng/scripts/Invoke-GenerateAndBuildV2.ps1:131
05:03:40.799 cmderr W 	[Invoke-GenerateAndBuildV2.ps1] Line |
05:03:40.799 cmderr W 	[Invoke-GenerateAndBuildV2.ps1]  131 |                GeneratePackage `
05:03:40.799 cmderr W 	[Invoke-GenerateAndBuildV2.ps1]      |                ~~~~~~~~~~~~~~~~~
05:03:40.800 cmderr W 	[Invoke-GenerateAndBuildV2.ps1]      | Failed to build sdk. exit code: False

@glorialimicrosoft what changes were made after this was approved? Is this ready to merge?

@glorialimicrosoft what changes were made after this was approved? Is this ready to merge?

@tjprescott Hello Travis! There was a minor change to change channel id for ListTemplates from string to uuid. It was already a uuid for public preview, we just missed it when transforming to TypeSpec.
We still have a couple of customization for Java and JS waiting to be merged into this branch, but no other changes should be expected on the TypeSpec API itself. We are planning on merging this PR next week. Thank you!


Training on each stock

when i put like 80 different stocks and prepare the data , it only take the mean. I think the model would be more efficient if we could train it on every stock that we want and not only on One or the Mean .
sorry if my english is bad . BTW good job with this project . keep going !

Im not that experienced in that and I dont think i could do that, but i think i can try it...

After conducting a thorough investigation, I have determined that I am unable to accomplish this task. Therefore, I am seeking assistance from someone who can help me with it.


RPC custom object

Hey,
I'm working on a project and I'm testing some ideas for messaging.
The code below works, I am now trying to do the same thing with the PRC but I can not send a Chat Message, this is not impossible?
 Subscription subscribed = client.makeSubscription(eventName.trim().toLowerCase(), ChatMessage.class).subscribe(new Subscriber<ChatMessage>() {

            @Override
            public void onNext(final ChatMessage message) {

                Log.d("andre","Message say: " + message.getTextMessage());
            }

            @Override
            public void onError(Throwable throwable) {
                System.out.println("onError : " + throwable);
            }

            @Override
            public void onCompleted() {
                System.out.println("onCompleted ");
            }
});

Call client.publish(mChatMessage.getUserId().trim().toLowerCase(), mChatMessage);
thks!

What is your actual error?

I'm closing this, since the question is not understandable.


PapermillExecutionError at line 29 in reporting.py

Generated uspto_mapped.csv and now Facing this error when running script template_pipeline.sh. After some hours validated_reactions.csv has been created but then get the error in the screenshot.
Any idea how to resolve it? Already tried the installing the papermill from git as mentioned in one of the issue as a solution using following commands.
conda activate aizynthtrain python -m pip uninstall papermill python -m pip install git+https://github.com/nteract/papermill.git

This might be related to #17
I suggest you checkout the solution in that issue and report back on this issue.

Closing this. Let me know if #19 did not solve it.


extact sub functions for onSubmit, deduplicated assignment call

Co-Authored By: Chris Atkinson<EMAIL_ADDRESS>Co-Authored By: Kevin Freeman<EMAIL_ADDRESS>Co-Authored By: Avery Sturzl<EMAIL_ADDRESS>Issue
Resolves XXX (where XXX is the related issue)
What was done

[x] A completed task

How to test

Accessiblity Criteria
Testing

[ ] Add jest-axe unit tests (for new ui components)

Manual Checks

[ ] Text and background color meets a minimum color contrast ratio of 4.5:1
[ ] All non-text content (e.g. images, icon) has alt text
[ ] Web page includes appropriate headings to communicate structure/hierarchy (h1>h2>h3)
[ ] All functionality is available to a keyboard

[ ] Focus styles are highly visible
[ ] Tab order is logical and aligns with the visual order

[ ] Voice over is understandable and logical

[ ] Aria labels are added to elements with no visible text
[ ] Decorative elements (e.g. non-informative icons) are hidden from screen readers

[ ] Form fields have appropriately coded labels that are visible during input
[ ] Links are visually identifiable and have a clear focus and active state

Closing so we can archive repo


False positive at function-allowed-list and CssSyntaxError rule.

What minimal example or steps are needed to reproduce the bug?
This component is getting this error, i think it's because ternary operator.
this function is not supported when using
styled-components with React Native - function-allowed-list
export const MonthBalance = styled.View<{ isMonthFirstEntry: boolean }>`
  margin-bottom: ${({ isMonthFirstEntry }) =>
    isMonthFirstEntry ? theme.space('SPACE-32') : 0};
`;

And this component is getting this error, i think it's for the same reason.
Unknown word - CssSyntaxError
export const BalanceValue = styled(Text).attrs({
  size: 18,
})<{
  isBalanceNegative?: boolean;
}>`
  width: ${({ isBalanceNegative }) =>
    isBalanceNegative
      ? css`
          ${WIDTH - 190}px
        `
      : css`
          ${WIDTH - 150}px
        `};
`;

What minimal configuration is needed to reproduce the bug?
my .stylelintrc:
{
    "processors": ["stylelint-processor-styled-components"],
    "extends": ["stylelint-config-react-native-styled-components"],
    "plugins": ["stylelint-react-native"],
    "customSyntax": "postcss-styled-syntax"
}

How did you run Stylelint?
stylelint './src/**/*.ts
Which Stylelint-related dependencies are you using?
    "stylelint": "^16.2.1",
    "stylelint-config-react-native-styled-components": "^0.7.0",
    "stylelint-processor-styled-components": "^1.10.0",
    "stylelint-react-native": "^2.7.0",
    "postcss-styled-syntax": "^0.6.4",

What did you expect to happen?
stylelint works without error.
What actually happened?
I'm getting this error:
this function is not supported when using styled-components with React Native (function-allowed-list) and Unknown word (CssSyntaxError)

First example is parsed correctly. So it's up to Stylelint to ignore non-standard rules. Most of the rules do ignore CSS-in-JS interpolation. So it's kind of a standard procedure for Stylelint to do.
Second example is indeed a parsing issue. I'll see what I can do about it. But I think not much. css helper, I believe, is not intended to be used inside interpolations for a property value. And to be honest, I can't imaging why use css`${WIDTH - 190}px` instead of `${WIDTH - 190}px`.


Prevent close on suggestion selected

Is there a way to prevent close the dropdown (.tt-menu) when a suggestion is clicked? I've searched a lot and none of these "hacks" works with current version:

https://github.com/twitter/typeahead.js/issues/731
https://github.com/twitter/typeahead.js/issues/1113
http://stackoverflow.com/questions/20182058/prevent-typeahead-js-dropdown-from-closing-on-select

select is called when an item is selected. You can prevent the default behavior of this method by listening to the typeahead:beforeselect event and cancelling it with event.preventDefault(), like so:
$(document).on("typeahead:beforeselect", function(event, data) {
  event.preventDefault();
});

@laCour your tip is not more working, do you know another way to do it now?

@zedtux It still works for me. If you're not using jQuery my example wont work and would require changes. What version of typeahead are you using?

I double checked and while I was thinking using the version 0.11, I just discovered that I'm using the version 0.10.5.
This should explain why it is not working.

Hell yeah, it's working now, thank you :)


Fix keepZeroFacets on disjunctive facet search

Since introducing the multi-index search here #888 the setting keepZeroFacets was broken.
This PR introduces the fix to this issue
https://user-images.githubusercontent.com/33010418/207385436-e49ded20-e76b-4974-86de-904cfeecfbd7.mp4
What was done in this pr:

keepZeroFacet management has been moved to the response adapter instead of the request adapter
Filtering methods and filtering tests were removed as the new facet distribution management has completely changed
defaultFacetDistribution is renamed initialFacetDistribution and contains the distribution of an index without any other requests
Initialization of facet distribution was moved from client to its own file

Playgrounds:

New SingleMovieIndex playground to create a new environment for testing
Updated SingleIndex with keepZeroFacets
Updated MultiIndex with keepZeroFacets

bors merge


refactor: refactored socket service, renamed selectors ! BREAKING CHANGE

@jacqueswho Can you help testing this?

Will test with waitroom

@jacqueswho Can you help testing this?

Will test with waitroom

Maybe make an alpha or beta version first so we can test and use it.


Possibility to filter per platform

When lists are long, like on a country or topic page it would be nice to be able to filter on just a single platform. For example, on the page for Sweden or the page for Environmental protection, I could select to only see Twitter accounts or YouTube accounts.

Considering that we show a line for each agency and that we aren't nessisarly displaying accounts other than for a few platforms, would we want the filter to be limited to those few ones?

As a start, that seems like the easy way to go.


[Event Request] codeunit 5940 ServContractManagement procedure CreateOrGetCreditHeader()

Please can you add the following event/code just before the call to Cust.CheckBlockedCustOnDocs as follows
OLD CODE...
Cust.CheckBlockedCustOnDocs(Cust, ServHeader2."Document Type", false, false);

NEW CODE...
IsHandled := false;
OnBeforeCheckBlockedCustOnDocsForServiceCredit(ServHeader2, ServContract2, IsHandled);
if not IsHandled then
    Cust.CheckBlockedCustOnDocs(Cust, ServHeader2."Document Type", false, false);

The event publisher would be...
[IntegrationEvent(false, false)]
local procedure OnBeforeCheckBlockedCustOnDocsForServiceCredit(var ServiceHeader: Record "Service Header"; ServContract2: Record "Service Contract Header"; var IsHandled: Boolean);
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 11061.


FAQ link is broken

The link to the FAQ at the end of the article is broken and just takes you back to the same page.

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

ID: c06fd94b-aaba-d65f-1fc7-7131d6ff892f
Version Independent ID: 3de244c3-d43b-be7a-bad3-b5e04a1c2fd8
Content: Compiling Apps with .NET Native
Content Source: docs/framework/net-native/index.md
Service: unspecified
Product: dotnet-framework
GitHub Login: @rpetrusha
Microsoft Alias: ronpet

Thanks @daniel-zazula. It doesn't seem we have that link anymore. So I'll just going to remove it from the docs. /cc @rpetrusha


Issue Convert Tensorflow 1.15 model failed

🐞Describe the bug
When I trying to convert tensorflow 1.15 model to coreml model failed,
I don't know if this is a problem with tensorflow or a problem with tfcoreml.
Trace
Now finding ops in the TF graph that can be dropped for inference
Collecting all the 'Const' ops from the graph, by running it....
Traceback (most recent call last):
  File "/home/ateam/anaconda3/envs/coremlconvert/lib/python3.7/site-packages/tensorflow_core/python/client/session.py", line 1365, in _do_call
    return fn(*args)
  File "/home/ateam/anaconda3/envs/coremlconvert/lib/python3.7/site-packages/tensorflow_core/python/client/session.py", line 1350, in _run_fn
    target_list, run_metadata)
  File "/home/ateam/anaconda3/envs/coremlconvert/lib/python3.7/site-packages/tensorflow_core/python/client/session.py", line 1443, in _call_tf_sessionrun
    run_metadata)
tensorflow.python.framework.errors_impl.OutOfRangeError: Node 'mask/Conv/BatchNorm/FusedBatchNormV3' (type: 'Add', num of outputs: 1) does not have output 5

System environment (please complete the following information):

coremltools version  (3.31):
tf-coreml version (1.1)
tensorflow version (1.15):
OS (Linux):
python version (3.7):

I used my own training data to perform transfer learning on http://download.tensorflow.org/models/object_detection/ssdlite_mobilenet_v2_coco_2018_05_09.tar.gz
this pretrained ssdlite model.
The model performs as expected after training. I only changed the number of classes to 1 in the pipeline.config file before training.
Converting the model to tflite format was also successful. Now i need it to run on iOS as well but when trying to convert(even after removing the excess parts of the graph using (https://github.com/apple/coremltools/blob/master/examples/neural_network_inference/tensorflow_converter/Tensorflow_1/ssd_example.ipynb) I also run into the mentioned error.
I inspected the graph using netron and I'm attaching the pictures here.

The batchnorm node causing the error is "FeatureExtractor/MobilenetV2/Conv/BatchNorm/FusedBatchNormV3"
and it does indeed have just one output when inspected. Is it actually supposed to have 5 outputs? What am I doing wrong?

FusedBatchNormV3 issue should be fixed now. See issue #336


Scene.Name = asset name by default

Is your feature request related to a problem? Please describe.
I'm always frustrated when look at the scenes while debugging and always see "Scene Scene" value. This makes hard to determine which scene is which.
Describe the solution you'd like
Set scene's "Name" property to the appropriate asset name by default.
Describe alternatives you've considered
Allow to edit scene's "Name" property like it's already done for entities, but I'm afraid of possible side effects like broken link between the real asset and the added scene to the "Asset editor". It also may not be a problem, I don't really know.
Additional context

RootScene's "Name" should be "RootScene", not "Scene" and debug view for the scene will be automatically updated to "Scene RootScene" instead of "Scene Scene".
Unfortunatelly I'm not familiar with the Game Studio source code, so I don't know how to implement the idea.

For some reason I can't check if the following changes works, but I hope my investigation will help someone who can:

SceneAssetCompiler.SceneCommand class contains a method DoCommandOverride that creates a Scene from the SceneAsset and then saves it to the ContentManager.
We can try to add something like
Name = Parameters.MainSource.GetFileNameWithoutExtension(),
to the Scene instance initialization (right after the Offset property). Full code:

var scene = new Scene
{
    Parent = Parameters.Parent,
    Offset = Parameters.Offset,
    Name = Parameters.MainSource.GetFileNameWithoutExtension(), // <-- Added code
};

SceneSystem has a code to initialize SceneInstance:

SceneInstance ??= new SceneInstance(Services) { RootScene = new Scene() };

This code can be updated in next way:
SceneInstance ??= new SceneInstance(Services) { RootScene = new Scene { Name = nameof(SceneInstance.RootScene) } };


Fixed upstream issues

Describe your Pull Request
Fixes the following Issues:

https://github.com/whiteducksoftware/azure-arm-action-js/issues/15
https://github.com/whiteducksoftware/azure-arm-action-js/issues/14
https://github.com/whiteducksoftware/azure-arm-action-js/issues/13
https://github.com/whiteducksoftware/azure-arm-action-js/issues/12

Please select relevant options:

[X] Bug fix (non-breaking change which fixes an issue)

Checklist

[X] My code follows the style guidelines of this project
[X] I have made corresponding changes to the documentation
[X] I have added tests that prove my fix is effective or that my feature works
[X] I have checked my code and corrected any misspellings

Reviewer: @nmeisenzahl

Implemented the changes according to your review.

The unit tests are still failing.

The unit tests are still failing.

On this Point I need some feedback from @kanika1894.


More flexible whitelist for PluralizeResourceNamesRule

Generally, I like the idea of PluralizeResourceNamesRule but internally we have it disabled due to fear of too many "false positives":

We have a standard GET /status API on each service used for basic health checks. Clearly, the resource here is not plural but we want to accept it anyway.

We also have a whole bunch of POST /things/{id}/action APIs which may not be ideal from a RESTful point of view but are a regular solution all the same and we're willing to live with them.

I'm in constant fear that we're going to find terms which the PlingStemmer doesn't correctly identify as plurals and I don't really want to open an IDE for every exception we need to make.

Problem (1) can be addressed by configuring whitelist_prefixes and that suggests that adding a whitelist_suffixes configuration might be a reasonable solution to (2). Given that elsewhere we've become comfortable using regex in configurations I'm tempted to suggest we just use a single whitelist of regexes ^/status/, /action$ instead. Thoughts?
Problem (3) screams out for a configurable whitelist of plural terms. What I'm not certain about is whether this should be specific to PluralizeResourceNamesRule or a whitelist shared with PluralizeNamesForArraysRule and future uses of isPlural(String)?

I would vote for adding a single whitelist-regexp for customization.
I'm not convinced that, the pluralization rule itself should be configured via a file. In the first place we should invest into applying a good algorithm, maybe by searching a feasible OS-library. If we need an to add exceptions, because the word is heavily used, it either very company specific and should be handled by the whitelist-regexp, or it is so common that we should introduce it as extension into the implemented algorithm. To foster the discussion in the second case, I would like to stick to hardcoding this extensions.


Fix native feedback radius

When setting fixNativeFeedbackRadius to true and giving the ActionButton position 'Center'. The AtionButton and ActionButtonItems would have an offset from the center due to incorrect styling.
fixNativeFeedbackRadius also caused an ugly shadow on ActionButtonItems and their respective lable, and also a cropped dropshadow.
For the ActionButtonItem, this was due to the shadow styling was set on the wrong element for android. As for the lable, we do not need to set the fixNativeFeedbackRadius to the touchableBackground (the lable is rectangular anyway). You'll notice that the ripple effect on the lable would extend in a circular fashion outside the borders of the lable.

Any news on this PR? Thanks!

Any news on this? It would be awesome if this could get merged!


⚠ SceneTime has degraded performance

In cd813e6, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1394 ms

Resolved: SceneTime performance has improved in bb01591 after 6 minutes.


⚠ ClamAV has degraded performance

In 44c931a, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 53 ms

Resolved: ClamAV performance has improved in 06bf683.


"Enable Web Analytics" doc for Cloudflare Pages seems to give incorrect information

Existing documentation URL(s)
https://developers.cloudflare.com/pages/how-to/web-analytics/
https://github.com/cloudflare/cloudflare-docs/blob/production/src/content/docs/pages/how-to/web-analytics.mdx
What changes are you suggesting?
I can't successfully follow the "one click setup" method described. The "manage" menu option is not there, and if it has been moved / renamed I can't seem to find it. Could you please investigate this, and update the docs accordingly?
Additional information
Here's a screenshot of what I see in the overview page of my project. There is no "manage" menu option.

I have the same issue, cannot find "Manage" menu => cannot enable/disable Web Analytics anymore

我也遇到了一样的问题cf里面无法解析wed文档的正确信息直接把我搞崩溃了创健项目也是显示wed文档无法显示解析不出来这个CF就等于废了用不了现在

我也遇到了一样的问题cf里面无法解析WEd文档的正确信息直接把我搞崩溃了创健项目也是显示wed文档无法显示解析不出来这个CF就等于废了用不了现在

我也遇到了同样的问题cf里面无法解析WEd文档的正确信息直接把我搞崩溃了创健项目也显示了wed文档无法显示解析不出来这个CF就等于废了用不了现在

这个CF号等于废了呀大老有木有解决办法呀！

I'm not sure if this was here when I previously checked, but it seems that web analytics can be enabled from the metrics tab.

Yes that's the component's new home! Updating docs here https://github.com/cloudflare/cloudflare-docs/pull/17928

This mechanism uses manual JS snippet installation (just automatically includes it on deploy I guess), which we could do manually anyway.
This loads the beacon from static.cloudflareinsights.com rather than our own domain's /cdn-cgi/.... Is it impossible to proxy via our own domain with Cloudflare Pages on a custom domain?
I'd like to see documentation for an option to manually including a snippet and configuring it to proxy via /cdn-cgi/... rather than static.cloudflareinsights.com.


Use consistent SHA1 for source-build-reference-packages for bootstrap CI

Currently the source-build-reference-packages which is cloned during the bootstrap build is always the head. This should be changed to use the SHA1 in versionDetails for consistency issues. Feb 3.1 had a minor break due to this issue.

[Triage] The repo shas are determines by the installer's Version.Detail.xml graph which source-build-reference-packages is part of..


More generic plugin system

Instead of just offering component previews, the plugin system could be a more generic "some text to some widget" system. This would allow things like:
```colors
#111222
#222111
```

to be converted to a color swatch, or:
```font
Helvetica Neue
11, 13, 17
```

to be converted to a font preview. It could also be used for things like mermaid

Great suggestion by @blynx: A plugin that renders Figma artifacts. This way, we can more easily compare the state in Figma with the state in HTML/CSS/JS. We can also create a component for which we only have a Figma version, and no HTML/CSS/JS yet.

I just had the idea to also inject parameters into the widgets.
```html(height: 100, noFullscreen)
<nav>
    <a href="yay.html">yay</a>
</nav>
```

which would directly be put into the rendered widget element
<component-preview height="100" noFullscreen></component-preview>

That way you could control for example the height of the rendered component preview window. Sometimes you just want to render a narrow button, sometimes you want to render a vertical navigation.
Also you could enable/disable features, etc. of that widget.

I really like that 👍 I would prefer the following way to write it though:
```html height="100" noFullscreen
<nav>
    <a href="yay.html">yay</a>
</nav>
```


Portable edition for all os

VSCode Version:Latest
OS Version:All

I am wishing for portable edition for all platform just plug and play without need of installation/.exe file

@MandarSant cool idea! Looks like this feature request is already being tracked at https://github.com/Microsoft/vscode/issues/329, but let me know if you meant something different, and we're happy to reopen. Happy coding!


Add support for wrapping on zero width spaces

Add support for wrapping on zero width spaces as it is not always appropriate to show the hyphen from the soft-hyphen. e.g. in the middle of an email address.

Indeed the "ZERO WIDTH SPACE" is used for an invisible line break (see "this character is intended for invisible word separation and for line break control" here).
In contrast, the ZERO WIDTH NO-BREAK SPACE explicitly asks for no line wrap.

I've changed it to unicode (u200B) http://www.fileformat.info/info/unicode/char/200B/index.htm from unicode (u200C) http://www.fileformat.info/info/unicode/char/200C/index.htm as it is more semantically correct.

Included in current version. Thanks for the contribution, we appreciate this improvement.


OOM protected docker processes (-900)

Also changed ApplyOomScore util function to return error.
#7376, #7795

Just a nit, otherwise LGTM

Addressed your comments. Thanks!

LGTM

thanks @dchen1107!


Support AMD/RequireJS

It would be great if the JSON editor supports AMD/RequireJS.
Since Grunt is apparently already used to build dist scripts, grunt-umd should do the trick. See twitter/typeahead.js for an example.

+1, using it from GitHub Electron shell, but have to include the js file in my index.html

Current workaround is to use shim and define exports to make it loadable via RequireJS but this provides no encapsulation at all.

This doesn't seem to work with Handlebars. The latest version of Handlebars (as of this time, 3.0.3) supports AMD loading and so does not bind window.Handlebars (as is expected in https://github.com/jdorn/json-editor/blob/master/src/templates/handlebars.js#L2 )
The only workaround I've managed to get working at the moment is this:
    jsonEditor: {
        deps: ["handlebars"],
        init: function (Handlebars) {
            window.Handlebars = Handlebars;
            return this.JSONEditor;
        },
        exports: "JSONEditor"
    },

Which is pretty terrible since it breaks the whole module encapsulation pattern.

The actual issue here is that there are optional dependencies which doesn't work with the AMD pattern. Normally one should split all the different  templating engines to separate AMD modules with proper dependencies.

+1, Just a chime in to say that this would be an amazing thing to add in!