Feature/ddw 271 Ada redemption options update - FORK

This PR is a FORK of https://github.com/input-output-hk/daedalus/pull/924
This PR adds two new tabs to the "Ada redemption" screen:

Recovery - regular,
Recovery - force vended.

Todo:

[x] Inject missing Japanese translations
[x] Find and inject real decrpytion-key into the intro text of both of new tabs
[x] Test new features on macOS installer
[x] Test new features on Windows installer
[x] Test new features on Linux installer
[x] Cover new tabs with acceptance tests
[x] Check if storybook works correctly

Screenshots:
Recovery - regular

Recovery - force vended

Review Checklist:
Basics

[ ] PR is updated to the most recent version of target branch (and there are no conflicts)
[ ] PR has good description that summarizes all changes and shows some screenshots or animated GIFs of important UI changes
[ ] CHANGELOG entry has been added and is linked to the correct PR on GitHub
[ ] Automated tests: All acceptance tests are passing (npm run test)
[ ] Manual tests (minimum tests should cover newly added feature/fix): App works correctly in development build (npm run dev)
[ ] Manual tests (minimum tests should cover newly added feature/fix): App works correctly in production build (npm run package / CI builds)
[ ] There are no flow errors or warnings (npm run flow-test)
[ ] There are no lint errors or warnings (npm run lint)
[ ] Text changes are proofread and approved (Jane Wild)
[ ] There are no missing translations (running npm run manage-translations produces no changes)
[ ] UI changes look good in all themes (Alexander Rukin)
[ ] Storybook works and no stories are broken (npm run storybook)
[ ] In case of npm dependency changes both package-lock.json and yarn.lock files are updated

Code Quality

[ ] Important parts of the code are properly documented and commented
[ ] Code is properly typed with flow
[ ] React components are split-up enough to avoid unnecessary re-rendering
[ ] Any code that only works in Electron is neatly separated from components

Testing

[ ] New feature / change is covered by acceptance tests
[ ] All existing acceptance tests are still up-to-date
[ ] New feature / change is covered by Daedalus Testing scenario
[ ] All existing Daedalus Testing scenarios are still up-to-date

After Review:

[ ] Merge PR
[ ] Delete source branch
[ ] Move ticket to done on the Youtrack board

All Ada redemption tabs have been manually tested in macOS and Windows mainnet installers and Linux staging installer. Everything works as expected 👍

Closing this PR unmerged in favour of: https://github.com/input-output-hk/daedalus/pull/933


Broken basic configuration without sentinel/replication in v4.4.0

Hello, while updating the snc-redis library in my project which uses basic predis configuration for this bundle, the configuration I'm using fails with InvalidArgumentException:
Values evaluating to FALSE are not accepted for `replication`

My configuration looks like this:
snc_redis:
    clients:
        default:
            type: predis
            alias: default
            dsn: '%env(REDIS_DSN)%'

I believe the issue is caused by the change in config options for the newly added sentinel support in the Configuration.php file https://github.com/snc/SncRedisBundle/commit/6f654e43ed9042e4ca53496e7a2b109fc6749ae0
We're not using replication so I believe it's impossible for me to adjust the config so the value can be passes. Neither the above config nor including the following works.
options:
  replication: false

duplicate of https://github.com/snc/SncRedisBundle/issues/683


file.content is a stream and toString does not work

Is it a mistake that it tries to do a file.content.toString() when the file.content is a stream.
https://github.com/ipfs/interface-ipfs-core/blame/master/SPEC/FILES.md#L333

I can easily fix it if we agree :)

@kevinsimper the line you pointed to is doing file.path.toString(), not content.

@diasdavid Yes, that is probrably another mistake as the comment says above  // write the file's path and contents to standard out
https://github.com/ipfs/interface-ipfs-core/blame/master/SPEC/FILES.md#L331

@kevinsimper if you want to get a buffer, use ipfs.files.get instead

Thanks for fixing with #237 <3


take in the timeout as a parameter

added the option to take in the timeout for the initial post as a parameter

Hey David,
I sent you a couple of emails that I haven't gotten a response to yet.
The CrossBrowserTesting.com plugin doesn't work for SeBuilder 3 because the initial handshake connection between SeBuilder and the CrossBrowserTesting.com Selenium Hub times out. On line 162 of "seleniumbuilder/content/html/builder/selenium2/rcPlayback.js" the timeout is hardcoded to 10000ms (10 seconds). I added the option to take in the timeout as a parameter or default to 10 seconds.
I have forked SeBuilder 3. If you would, can you pull in these changes to keep the source together.
We have customers waiting to use SeBuilder 3 with our service, so we need to get this change in a timely manner. If you feel it would be for CrossBrowserTesting.com to release our own version of SeBuilder ourselves, let us know.
Thanks for the help.


Added stop/restart functionality in sources/sinks

Motivation
Currently we have create/delete source/sinks but no stop/restart/getstatus. This pr fills that gap.
Modifications
Describe the modifications you've done.
Result
After your change, what will change.

rerun java8 tests


Implementing Streaming Output with Autogen and Chainlit Integration

I have a question regarding an example on https://github.com/Chainlit/cookbook/blob/main/pyautogen/app.py (or async version: https://github.com/Chainlit/cookbook/blob/main/pyautogen/async_app.py) that combines autogen and chainlit. I've been trying to modify the code to enable streaming output, but so far, no results have appeared on the screen. My code is as follows:
class ChainlitAgent(AssistantAgent):
    def __init__(self, *args, **kwargs):
        super().__init__(
            system_message=SYSTEM_PROMPT,
            name="assistant",
            llm_config={
                "stream": True
            }
        )

    async def a_send(
        self,
        message: Union[Dict, str],
        recipient: Agent,
        request_reply: Optional[bool] = None,
        silent: Optional[bool] = False,
    ) -> bool:

        msg = cl.run_sync(
            cl.Message(content="", author="assistant")
        )

        async for part in message:
            if token := part.choices[0].delta.content or "":
                print(token)
                await msg.stream_token(token)

        await msg.send()

        await super(ChainlitAgent, self).a_send(
            message=message,
            recipient=recipient,
            request_reply=request_reply,
            silent=silent,
        )

Is there a way to implement streaming with autogen in a chainlit environment?

Does autogen support streaming in the first place? If so can you link the docs?

Ref here: https://github.com/microsoft/autogen/issues/217#issuecomment-1783895515

Interesting! I see a code snippet to enable streaming but I don't see an example to consume the stream?

Based on my understanding, right now Autogen supports streaming the response from llm, but not the completed answer (the message one agent sent to another).

I recently experimented with using monkey patching to implement streaming functionality. This method enables the dynamic and flexible addition of new Agents without the need to alter the original autogen source code. Note: You can isolate the monkey patching method into its own separate library. Here's an example of the successful code I developed. If this approach seems viable, I'm willing to propose a pull request.
class ChainlitAssistantStreamableAgent(AssistantAgent):
    def __init__(self, *args, **kwargs):
        super().__init__(
            system_message=SYSTEM_PROMPT,
            name="assistant",
            llm_config=llm_config,
        )
        self.patch_openai_wrapper()

    def patch_openai_wrapper(self):
        from openai import __version__ as OPENAIVERSION
        from openai.types.chat import ChatCompletion
        from openai.types.chat.chat_completion import ChatCompletionMessage, Choice
        from openai.types.completion_usage import CompletionUsage
        from autogen.token_count_utils import count_token

        def new_completions_create(client, params):
            completions = client.chat.completions if "messages" in params else client.completions
            if params.get("stream", False) and "messages" in params and "functions" not in params: # note: you should enable stream at first
            # if params.get("stream", False) and "messages" in params:
                response_contents = [""] * params.get("n", 1)
                finish_reasons = [""] * params.get("n", 1)
                completion_tokens = 0

                # Set the terminal text color to green
                print("\033[32m", end="")

                # Add chianlit
                msg = cl.Message(content=f"")
                cl.run_sync(msg.send())

                for chunk in completions.create(**params):
                    if chunk.choices:
                        for choice in chunk.choices:
                            content = choice.delta.content
                            finish_reasons[choice.index] = choice.finish_reason

                            if content is not None:
                                print(content, end="", flush=True)
                                cl.run_sync(msg.stream_token(cc.convert(content)))
                                response_contents[choice.index] += content
                                completion_tokens += 1
                            else:
                                print()
                                cl.run_sync(msg.update())

                # Reset the terminal text color
                print("\033[0m\n")

                # Prepare the final ChatCompletion object based on the accumulated data
                model = chunk.model.replace("gpt-35", "gpt-3.5")  # hack for Azure API
                prompt_tokens = count_token(params["messages"], model)
                response = ChatCompletion(
                    id=chunk.id,
                    model=chunk.model,
                    created=chunk.created,
                    object="chat.completion",
                    choices=[],
                    usage=CompletionUsage(
                        prompt_tokens=prompt_tokens,
                        completion_tokens=completion_tokens,
                        total_tokens=prompt_tokens + completion_tokens,
                    ),
                )
                for i in range(len(response_contents)):
                    if OPENAIVERSION >= "1.5":  # pragma: no cover
                        # OpenAI versions 1.5.0 and above
                        choice = Choice(
                            index=i,
                            finish_reason=finish_reasons[i],
                            message=ChatCompletionMessage(
                                role="assistant", content=response_contents[i], function_call=None
                            ),
                            logprobs=None,
                        )
                    else:
                        # OpenAI versions below 1.5.0
                        choice = Choice(
                            index=i,
                            finish_reason=finish_reasons[i],
                            message=ChatCompletionMessage(
                                role="assistant", content=response_contents[i], function_call=None
                            ),
                        )

                    response.choices.append(choice)
            else:
                # If streaming is not enabled or using functions, send a regular chat completion request
                # Functions are not supported, so ensure streaming is disabled
                params = params.copy()
                params["stream"] = False
                response = completions.create(**params)
            return response

        self.client._completions_create = new_completions_create

Coming back to this: since release 0.2.21 supports IoStream, could this be achieved by creating a custom IoStream? Something like this (proto-code):
import chainlit as cl
from typing import Any
from autogen.io.base import IOStream

class ChainlitIOStream(IOStream):
    def __init__(self, author):
        self.author = author
        self.message = cl.Message(author=author)

    def print(self, *args: Any, **kwargs) -> None:
        return self.message.stream_token(args)

    def input(self, prompt: str = "", *, password: bool = False) -> str:
        return cl.AskUserMessage(prompt).send()

default_io_stream = ChainlitIOStream(author="MyBot")

def set_custom_IO_overrides():
    IOStream.set_global_default(default_io_stream)

Then when the OpenAIClient.create gets called it does the printing of the stream here:
                        # If content is present, print it to the terminal and update response variables
                        if content is not None:
                            iostream.print(content, end="", flush=True)
                            response_contents[choice.index] += content
                            completion_tokens += 1
                        else:
                            # iostream.print()
                            pass

Ps. @willydouhard yes, Chainlit supports streaming


Feature/error detail implements i error detail

Kindly review the implementation  of IErrorDetail so that we can do a final release.

v5.26.0 [release-type final]


ERROR - xml - TUFIN_PS_XML - Line 91 - Could not find XML element using XPath expression 'owner' under XML node '<Element 'application'>

I'm working on Tufin automation integration and I have the following error when trying to simple list Secure Application:
ERROR - xml - TUFIN_PS_XML - Line 91 - Could not find XML element using XPath expression 'owner' under XML node '<Element 'application' at 0x7f57805e1b88>'
ERROR - test1 - TUFIN_PS_COMMON - Line 29 - Could not find XML element using XPath expression 'owner' under XML node '<Element 'application' at 0x7f57805e1b88>'

The code:
(...)
sa = Secure_App_Helper(HOST, (USER, PASSWD))
datalist = sa.get_application_list()
(...)

Tufin version I'm working on is: SecureChange Version: 19.2 HF3

Works now. Thanks. I'll close it now.


Add ! as alias for /run

It would be a nice affordance to be able to type ! like I'm able to in python and various debuggers. The functionality can be identical to /run but is just shorter and consistent with other shell-like tools.

Thanks for trying aider and filing this issue. I like this idea.

The change is available in the main branch. You can get it by installing the latest version from github:
python -m pip install --upgrade git+https://github.com/paul-gauthier/aider.git

If you have a chance to try it, let me know if it works better for you.


"make test-interactive" gives "exec format error" on MacOS

"make test-interactive" gives "exec format error" on MacOS.
Almost certainly the problem is that the Makefile builds observatorium_api for the OS, but then packages the OS-specific binary in the Linux-specific container image.
Work around:
rm ./observatorium-api
OS=linux GOARCH=amd64 make test-interactive

Output without the work-around
16:13:17 Starting observatorium_api
16:13:18 observatorium_api: standard_init_linux.go:228: exec user process caused: exec format error
16:13:52
Error: No such object: e2e_interactive-observatorium_api

    interactive_test.go:33: interactive_test.go:33:

         unexpected error: docker container observatorium_api failed to start: exit status 1

Thanks for the report @esnible! https://github.com/observatorium/api/pull/190 should fix this.


Parse uppercase headers in get_object and get_object_metadata

There is no reason to assume server to always return HTTP headers in lowercase, so we had better handle both.
Also I fixed put_bucket_policy to use correct content-type(json).

Did you see an issue with header case? It looks to me like lhttpc already is making all headers lowercase. lhttpc_client:read_response calls lhttpc_lib:canonical_header.

We got bitten by this too - erlcloud's rebar.config refers to an lhttpc from talko's repository - that version calls canonical_header, but the version from esl (which we were using) doesn't.
Thankfully, as of right now, the talko repo is the same as 1.3.0 at esl with e17 fixes and that header fix, so was safe for us to switch; and that makes this change redundant (except for the extra header you added for bucket policy)


Can't access the background edit mode when covered by shape

Bug Description
When using background media with shape covering the full background area, the background media edit mode is not accessible.

Expected Behaviour

Perhaps that's expected but would be good to clarify.

Steps to Reproduce

Add background media
Add a shape, e.g. triangle
Resize the shape to cover the whole area
Try to enter edit mode of background media (e.g. by double-clicking to the area free from the visible part of the shape.
See the shape getting selected instead, even though the click is on the area which is not directly on the visible shape.

Screenshots

Additional Context

Plugin Version:
Operating System:
Browser:

Do not alter or remove anything below. The following sections will be managed by moderators only.
Acceptance Criteria

Implementation Brief

@o-fernandez Created this issue since I wanted to zoom the background after creating this shape on top, however, apparently, it was not possible. Perhaps it's the expected behavior but I'm not certain, would be great if you could confirm.

@miina This shouldn't be an issue. If you placed another image on top of this, you wouldn't be expected to be able to select the bottom image. Nothing should be done for this use case and I think users will know that they occluded the bottom image.
Can we close this ticket.

In this case it's a triangle shape on top of the background though, not just an image. Didn't we somewhere define that, when clicking on the space not covered by the shape, we'd select the visible element underneath?

Just went back to check the editor and I had initially thought it was not possible to select the area side the shape to get into edit mode. But it seems that you could so if it's technically possible to be able to select the area within the selection box but outside of the shape that would be great.

Figma

@barklund @bmattb FYI Pulling this into the Sprint

@barklund @bmattb FYI Pulling this into the Sprint

Verified in QA

Verified in QA


[No Date] .cz domains

Problem
There's a whois server registered to the IANA Route Database Zone for all .cz domains. But the script can't get the date of expiration.
Conclusion
That's make all .cz domains fails with WHOIS.

Fixed with https://github.com/funilrys/funceble/commit/94a4266884d4e5fccc6569ef448dae4963629ce0


Revert "Added regSquare to partial valve models for improved robustness "

Reverts DLR-SR/ThermofluidStream#111

After internal discussions it was decided to remove the regSquare-function from the valves because it didn't lead to the desired improvements. Further tests will be done, details can be seen in issue #118


Have a Generic placeholders definition section in API.md

Fix #5567, Referring to configuration.md, to have a Generic placeholders definition section in API.md, so that users can easily find the definition of unix_timestamp or rfc3399 timestamp.
Before:

After:

Signed-off-by: Luke Chen<EMAIL_ADDRESS>

Thanks!


Convert into an Nx workspace for easier building/developing

Thoughts on converting this into an Nx workspace to easily support typescript, building and packaging?
I am currently working on switching this package to using TypeScript and it would make it easier to dev for sure!

Hi @yharaskrik! Thanks for the suggestion. As Nx is a monorepo tool, and this package is just a single package with a few code files, it's is not appropriate here IMO.
We can discuss the TypeScript migration at #3!

Hi @yharaskrik! Thanks for the suggestion. As Nx is a monorepo tool, and this package is just a single package with a few code files, it's is not appropriate here IMO.
We can discuss the TypeScript migration at #3!


[5.3] Let setData handles data transformation

It does not make any sense to restrict the type of data at the instantiation, but not when setting data with \Illuminate\Http\JsonResponse::setData.
At the very end, setData is used, so no need to transform data and pass an array.

Hesitating on which branch to target tho

This probably needs doing on 5.4 instead.

This probably needs doing on 5.4 instead.

This is also what I thought @GrahamCampbell , this also reminds me https://github.com/laravel/framework/pull/11988


Docker Desktop - Version packs

Tell us about your request
Provide the ability to swap the version of Docker engine and K8s that is running in Docker Desktop
Which service(s) is this request for?
Docker Desktop
Tell us about the problem you're trying to solve. What are you trying to do, and why is it hard?
Desktop currently only has one version of Engine and Kube, this means it is hard for me to match production locally as often they are at different versions.
Are you currently working around the issue?
Accepting the existing versions of K8s or Docker or using a different non-prod machine to test first.
Additional context
Currently an Enterprise feature

Personally, I don't use Docker Desktop because I'm on Linux, but on a related note I do something like this that I think might be valuable. I swap out my version of kubectl based on the server version. The idea is based off of https://github.com/jakepearson/k.
The reason I bring this up is that we are trying to allow people to match their production environments, and as a user, I don't really care what version it is and I personally don't want to have to manage versions as a developer, if I can help it. I want to just say "match <insert_environment_name>". With docker contexts, shouldn't we be able to query the server to see what the version is being run?
As an Enterprise feature, it was just a list of Docker Enterprise versions, which meant nothing to me. I want to see a list of environments, so that my docker context and the local versions of the engine/Kubernetes/whatever else match up.

Personally, I did not faced any requirement of running a specific version of Docker engine (no compatibility issues at this stage).
However, running a specific version of Kubernetes is, imo, a lot more important, due to relatively quick deprecation of some K8S APIS.

Personally, I did not faced any requirement of running a specific version of Docker engine (no compatibility issues at this stage).
However, running a specific version of Kubernetes is, imo, a lot more important, due to relatively quick deprecation of some K8S APIS.


IndexOutOfBoundsException when processing javadocs

codegen will crash while processing javadocs that are:
/**
 * @see #handler(RoutingContext ctx)
 * @param ctx  the context
 */

The problem seems to be on the @see #handler due to some regular expression not capturing a variable name


refactor: change the way action filter checks custom attribute in controller

I've changed the way of mvc action filter checking if controller has FluentValidationAutoValidation attribute.
Tested it in WeatherForecast (dotnet webapi template app) and it seems to work fine.

Looking back at the code I think the annotations on methods will break since it only retrieves annotations on the type. Did you test putting an annotation on a controller method?

Looking back at the code I think the annotations on methods will break since it only retrieves annotations on the type. Did you test putting an annotation on a controller method?

thank you fot noticing that, it works now for both controller and action annotation.

@alshuriga I'm still debating whether this change will have a detrimental impact on performance since it uses reflection. Where possible I would like to avoid using reflection. I'm going to look into the EndpointMetadata some more. Maybe .NET offers some public API's that expose the applied attributes.

@mvdgun Yes, GetCustomAttribute actually does impact performance if called every request. I found another way to retrieve attributes, it actually seems to work very similar to the original logic, but there's no restrictions in the documentation for using it in application code. Please take a look.

Closes #4

Merged, thanks for you contribution @alshuriga!


Docker version tag doesn't work with arm64 images

Are you experiencing an issue with...
shields.io
🐞 Description
I'm trying to use the "Docker Image Version (tag latest semver)" from shields.io on an arm64  architecture image. The resulting badge gives a "digest not found for given tag", even though there is digest for it on Docker hub.
Just in case my image would be broken, the same thing happens for MongoDB's arm64 badge: https://img.shields.io/docker/v/arm64v8/mongo/latest
🔗 Link to the badge
https://img.shields.io/docker/v/dolittle/runtime/6.1.0-arm64
💡 Possible Solution
Looking into this, one old issue mentioned something regarding this https://github.com/badges/shields/issues/4904#issuecomment-613742586 and how it used to default to amd64 architecture. Looking at the current code, it supports multiple architectures but it still defaults to amd64 if nothing else is defined. So this could be about how shields.io handles setting that particular value.

You need to use the arch parameter in cases where you either want to use a different architecture or where there aren't any amd64 images
https://img.shields.io/docker/v/dolittle/runtime/6.1.0-arm64?arch=arm64

We have to default to something, and the x86 64 bit target was the most sane default. However, we subsequently added the arch parameter because as noted in the prior issues you referenced as well as the case at hand, it's definitely not always the desired behavior


Web configuration

I've tried two different ways to skin this cat in this branch, and slightly favor the one here.
Both are suboptimal, but I am not sure what a more complete settings/configuration setup looks like at this point.  Though I have some vague ideas.

RUN sed -i -E 's/ws\:\/\/localhost\:5000/wss\:\/\/auth.po.et/g' /web/src/configuration.ts
That command can't be run in the current dockerfile, because as it is, the source code to be executed is loaded at runtime.
That should be fixed.

Ok.  This "works".
We're moving toward something that I understand better.  Merging this would be helpful for our production deployment if we want to continue on this path.

Alright, this should be good to go.


[multistage] Run ExpandSearch Rule After All Filter Pushdown Rules

In #10409 I had added support for properly supporting predicate pushdown. However it introduced a bug where we could get "Range is not implemented yet" error because the filter pushdown rules could create a Sarg of their own.
This can be reproduced using the COLOCATED_JOIN Quickstart with the following query. It can also be reproduced using the unit-test case I have added.
select count(*) from userAttributes_OFFLINE
  WHERE deviceOS NOT IN ('macos')
  AND daysSinceFirstTrip > -4294967296 and daysSinceFirstTrip <= 10
  AND (userUUID NOT IN (select userUUID FROM userGroups_OFFLINE WHERE groupUUID = 'group-1')) AND (userUUID NOT IN (SELECT userUUID FROM userGroups_OFFLINE WHERE groupUUID = 'group-2'))

cc: @walterddr

Codecov Report

Merging #10627 (2a0fb5b) into master (27a8a26) will increase coverage by 0.01%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master   #10627      +/-   ##
============================================
+ Coverage     24.49%   24.51%   +0.01%
  Complexity       49       49
============================================
  Files          2090     2090
  Lines        112578   112578
  Branches      16972    16972
============================================
+ Hits          27578    27594      +16
- Misses        82089    82092       +3
+ Partials       2911     2892      -19

Flag
Coverage Δ

integration1
24.51% <ø> (+0.01%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 41 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


allow igmp in template

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

Can one of the admins verify this patch?

"ok to test" to accept this pull request for testing
"test this please" for a one time test run
"add to whitelist" to add the author to the whitelist

@chengtcli : Can you add description to the PR.
This pr is extending igmp to all envs and earlier it was only included in centos env?

ok to test

retest

tested on tardis
I can create VMs successfully based on heat_stack.yml in any env(ci-full-rhel, ci-full-ubuntu, ci-ceph-redhat .etc.)


Add check for restriction on hyphens in pipe names

Giving a pipe a name with hyphens causes a compilation error. A contributor suggested adding a check to codelyzer.
https://github.com/angular/angular/issues/31787

I'd suggest to implement this as part of the ESLint Angular effort in https://github.com/angular-eslint/angular-eslint. Let me enable the issues there.


Add new address Pool addr ix data type. Simplify SharedState n k.

Issue number
ADP-1308
Overview
Previous work in epic ADP-1043 introduced delta encodings, DBVars, and an embedding of the wallet state and its delta encodings into a database table. It's time to integrate these tools with the wallet code. To facilitate code review, the integration proceeds in a sequence of refactorings that do not change functionality and pass all unit tests.
In this step, we continue refactoring the address discovery state.

First, we implement a new abstract data type, Pool addr ix, which aids with BIP-44 style address discovery. Its main benefit is that it is polymorphic in the address type and thus focuses on the essence of address discovery with an address gap while avoiding the plethora of type classes related to specific address formats.
Then, we use this data type to simplify the implementation of the address discovery state for mult-signature wallets, SharedState n k.

Details

Pool is an abstract data type, its constructor is not to be exported. Its internal invariants are collected in prop_consistent. To bypass the invariants, use loadUnsafe; the database layer is allowed to do that.
Some of the unit tests pertaining to the address discovery aspects SharedState are partially superseded by the more general unit tests in Cardano.Wallet.Address.PoolSpec.

Comments

The SeqState n k states will be refactored in terms of the new Pool addr ix data type next.
Merge PR #3056 before this one, because this pull request is based on the branch of the former.

thanks @sevanspowell for this remark! Conal Elliot's "Denotational Design" looks very smart, indeed! I will have to think about in the next days. On my side, if we convert to this way of design, then it would be good to spread that throughout the rest of codebase. Also, is it not a perfect topic for the first Fri dev meeting in Jan?

@paweljakubas
Sandy Maguire's Algebra-Driven Design (https://leanpub.com/algebra-driven-design) gives a great introduction to Conal's ideas.
I'd love to explore the consequences of using those ideas in the code base more (and have been trying to here and there), but the risks and limitations are unknown to me. It sounds like @HeinrichApfelmus would know more about the limitations of such a design methodology.
So yeah, great topic for a dev meeting.

bors try

bors merge


Image.change_orientation() removes the field absolutepath of an image

Description
Image.change_orientation() removes the field absolutepath of an image.
Steps to Reproduce
SCT version: e7cfb597c24e32805ea974fac2ab264876cdfff2
dummy code:
img = Image.('image')
img.absolutepath
> awesomepath
img.change_orientation('LPI')
img.absolutepath
> # Nothing is output!

Took a look at the method and noticed that there is a keyword argument generate_path that modifies the path with an orientation-specific suffix when set to True (default value is False).
https://github.com/neuropoly/spinalcordtoolbox/blob/4fd149d7d381c834e79dfdf8c0116b97aaef2fb7/spinalcordtoolbox/image.py#L437-L460
I'm assuming that the reasoning is the following: If we change an image's orientation we don't necessarily want to maintain the absolutepath, because it may not make sense. The function provides the ability to "intelligently" modify the path via generate_path, otherwise it is voided and the responsibility is on the caller to provide the correct path (when saving for instance).
In my opinion this makes sense. It is safe, because without an absolutepath the caller can not save the image and risk accidentally overwriting it.
>>> a = Image("/work/exmakhina/neuropoly/spinalcordtoolbox/sct_testing_data/t2/t2.nii.gz")
>>> a.change_orientation('RPI')
<spinalcordtoolbox.image.Image object at 0x7f7428821080>
>>> a.save()
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/work/exmakhina/neuropoly/spinalcordtoolbox/spinalcordtoolbox/image.py", line 507, in save
    raise RuntimeError("Don't know where to save the image (no absolutepath or path parameter)")

What do you think @jcohenadad ?

hum... there is probably a very good reason why we came up with this in the past, but in the past, things were also differently handled.
To me, this method is a manipulation on an image, similarly to other methods that are in the Image class (eg: copy_qform_from_ref(), set_sform_to_qform()). So, from a coding perspective, I find it much more intuitive to leave the responsibility to the caller to make sure the file name should be changed upon saving the image. E.g., my way of doing things would be:

img = Image.load(FNAME_IMAGE)
img_new = img.copy()
img_new.SOME_MANIPULATION
img_new.save(FNAME_NEW)

The advantage being that we can use the existing field. E.g.:

img_new.save(ADD_SUFFIX(img_new.absolutepath, '_new'))

What do you think?
Note: going through the code I also realized that this concerns other methods:
https://github.com/neuropoly/spinalcordtoolbox/blob/4fd149d7d381c834e79dfdf8c0116b97aaef2fb7/spinalcordtoolbox/image.py#L413
https://github.com/neuropoly/spinalcordtoolbox/blob/4fd149d7d381c834e79dfdf8c0116b97aaef2fb7/spinalcordtoolbox/image.py#L462

I agree. Just to clarify: with the current approach we can still do everything that you pointed out. The only difference is that if we don't void the absolutepath in the manipulation method and then go on to save the image without specifying a new path it will overwrite the original image without complaining.
I think the cleanest way would be to refactor the manipulations method that do more than manipulating (for example changing paths via generate_path and move the path modification to the caller.

In #3320, there are a few spots where this issue had to be worked around: https://github.com/neuropoly/spinalcordtoolbox/pull/3320#discussion_r611218125, https://github.com/neuropoly/spinalcordtoolbox/pull/3320#discussion_r611218650


BSE-2 Two Poll:: Ready Calls could cause critical event loss

BSE-2 Two Poll:: Ready Calls could cause critical event loss
Auditor: Movebit
Code: Atomic Bridge
Level: Major
Code Location: protocol-units/bridge/shared/src/blockchain_service.rs#45

We now query the blockchain directly via RPC queries, not WS events which could be deemed unstable. Also this particular part of code has been refactored for the new event monitoring.


Fixes #35 Made a new about us page

https://github.com/user-attachments/assets/d946cde1-4876-44ff-bdd8-685d8f5ff2dd
Hey I have attached the video of the new about us page, please merge and label it

it seems like there is some issue with the image and its displaying alternate text instead ? Please look into it and improve UI for this page


Visualize distribution of studies accross samples

To what extent are the search-result samples bunched up in just a few big studies?

Histogram?
Sort studies by most-samples?  (This would be more useful if we decide only to show studies at first, and then make the user click something to drill down to samples.)

@cndewey @mbernste How well do you feel this problem is addressed by the updated design, with more compact view of the studies and studies sorted by greatest-number-of-samples?

I think what you propose here would address the "results summary" goal.  I don't know how useful more sophisticated visualizations would be. At the very least, I don't think this should be prioritized in the remaining month


sign missing from loss function definition

@googlebot I fixed it

@googlebot I fixed it

Thanks for the catch!


No filter button shown in the web page (visdom 0.1.7)

I could not find any filter button in the web page.
I have already updated visdom to latest 0.1.7.
I have tested in chrome and firefox.

Did you restart the visdom server after you updated? It looks like your
browser is still hitting an old version of the visdom server.
On Wed, May 9, 2018 at 5:56 PM yfzhang<EMAIL_ADDRESS>wrote:

I could not find any filter button in the web page.
I have already updated visdom to latest 0.1.7.
I have tested in chrome and firefox.
[image: image]
https://user-images.githubusercontent.com/3224485/39846735-59697d7a-53cb-11e8-9fae-097b137a0cc1.png
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/facebookresearch/visdom/issues/345, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ABHSrokm6FwUbBRIL1daOZ1hhLLug39Fks5tw5AvgaJpZM4T5NEM
.

You should be able to pip install version <IP_ADDRESS> which certainly has the filter working. Reopen this issue if you're still having issues.


Support docker images with targets containing .s.

Fix parsing of image targets for building docker images so it only uses subs containing only ASCII letters, so targets like thumbv8m.main-none-eabihf aren't mistaken for a target/sub combination.
Closes #1020.

seems fine, but I think there's a better way.
Instead make target.none be a substitute for target, and thus require troublesome target names to be named target.withdot.none
I'm happy with this either way

I tried to avoid that just because that would require cargo build-docker-image "$target".none, however, you are right that this may conflict if we have a triple where the environment might have a .. I don't ever see this occurring, however, since I've never seen a triple where this exists.

feel free to merge unless you want to implement the other solution

bors r=Emilgardis


Fixed #31779 -- Bumped MariaDB requirement for JSONField to 10.2.16.

Ticket #31779.

Closing per ticket.


Revert "Revert "Fold Channel and CompletionQueue from grpc_impl to grpc""

Reverts grpc/grpc#23756

Interop Cloud-to-Cloud Tests (internal CI): #23511


[REQUEST] build prebuilt wheels

I want you can release with prebuilt wheels.
And I have build a github actions to auto build wheels you can use, but i can't build with evoformer because it require real cuda device.

Yes, so i compiled with almost all ops, only evoformer can't compile. And i compiled with the matrix like this, maybe it's a good start for auto build wheels.
matrx:
python-version: ["3.7", "3.8", "3.9", "3.10", "3.11", "3.12"]
torch-version: ["1.12.1", "1.13.1", "2.0.1", "2.1.2", "2.2.1"]
cuda-version: ["11.8.0", "12.1.1"]
build options
TORCH_CUDA_ARCH_LIST="6.1;7.0;7.5;8.0;8.6;8.9;9.0"
DS_BUILD_OPS=1
DS_BUILD_SPARSE_ATTN=0           // not support torch 2.0
DS_BUILD_EVOFORMER_ATTN=0  // need real cuda device? may be a bug?

@AlongWY - for the evoformer attention op, you would need to install cutlass and deepspeed-kernels.
But I'm curious what the advantage of a pre-built wheel with the ops installed is, since if the user grabs the wrong one, or has other environment issues, they will hit them regardless of if they use a wheel with pre-built ops or use the wheel off PyPI and the ops are JIT compiled?

I installed cutlass and deepspeed-kernels but can't compile, it use the torch.cuda.device(0) to detect the real device so it not work.
Advantages of Pre-Built Wheel:

Convenience: No additional compilation steps required for the user.
Performance: Saves the time and resources needed for JIT compilation.
Plug-and-Play: Especially beneficial for users lacking the environment or resources for compilation.

Advantages of JIT-Compiled Wheel:

Flexibility: Adapts to different users' specific environments.
Compatibility: Reduces issues due to system mismatches.
Customization: Tailored for specific system environments, enhancing operational efficiency.

@AlongWY I mostly agree, though pre-built wheels are not helpful to the users if they do not have the underlying hardware to run on, and they are more likely to think they can run with certain features then get errors later if they cannot.  Though we've also observed little difference in speed with pre-built wheels vs JIT compilation as well.
However, this does require additional testing and building support, since even the matrix listed above is 50 different wheels.  At least for now, it is unlikely that we will have the bandwidth to support this.


Easy deployment question

I run
npm run build
but get a folder called 'compiled' rather than 'dist' which only has a .js file rather than any css and index.html like other guides online I read.
Docs don't have much about deploying to your own server, can anyone help?

npm run build creates two folders; /compiled and /public.

/compiled contains the file server.js which is the actual NodeJS / express server
/public contains all clinet-side asses, ie. styles, scripts, etc.

Regarding there being no index.html; the static html for the project is minimal, and to 99% generated from react components. You can find the static html here

Thanks Alan. Does this mean I can just point the server to server.js rather than making an index.html with .js imported?

Does this mean I can just point the server to server.js

I do not know wheat you mean by "point", but yes, running the dist/server.js does indeed run the app. This can be done by running the start script, ie. npm start.
As an example, if you look in the Procfile, that's how Heroku would run the application.


Add table of survey proportions in Canada

Prob best as a table below Table 12. Includes CV for each country so bit hard to squeeze into Table 12.

Closed in 04abee27.


PPPoE not working anymore

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

[x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x] I am convinced that my issue is new after having checked both open and closed issues at https://github.com/opnsense/core/issues?q=is%3Aissue

Describe the bug
You may not believe, but: When I deactivated the company VPN (Openconnect) at my notebook the PPPoE connection at OPNsense stopped working about 10-20 seconds later.
From there on the PPPoE is broken as can be seen on the screenshot. There's also no specific message as the logging message seems to be empty:
[opt5_link0] PPPoE: Connecting to ''
[opt5_link0] PPPoE connection timeout after 9 seconds
[opt5_link0] Link: DOWN event
[opt5_link0] LCP: Down event

I tested the DSL modem directly with my notebook and it's still working as expected.
I updated the OPNsense twice within the last weeks, but nothing seems to be changed.
To Reproduce
Steps to reproduce the behavior:
PPPoE has been configured via VLAN 5.
Expected behavior
There are more verbose logging messages and the connection to the modem will be established.
Describe alternatives you considered
None
Screenshots

Relevant log files
Here is the log from the web UI:
[opt5_link0] PPPoE: Connecting to ''
[opt5_link0] PPPoE connection timeout after 9 seconds
[opt5_link0] Link: DOWN event
[opt5_link0] LCP: Down event

Additional context
None
Environment
Software version used and hardware type if relevant, e.g.:
OPNsense 24.1.9_4-amd64
FreeBSD 13.2-RELEASE-p11
OpenSSL 3.0.14
AMD Ryzen 3 3200G with Radeon Vega Graphics (4 cores, 4 threads)

I tried several days several things and yesterday I found out that the modem has a bug (?) that leads to returning packets being on the wrong link. Without the management link the modem is working again with OPNsense! :joy:
So: It's not a bug in OPNsense! :smiley:
But: That leads me to another question: Is there any way to improve the logging entries for PPPoE? Is that even technically possible? Because:

The message PPPoE: Connecting to '' is empty. Why?
What can a message PPPoE: connection timed out after x seconds mean?

Has OPNsense found a modem on layer 2? Is there even a MAC address? In my case no MAC address was available as I know from my switch in between the devices. I'm also missing a MAC table overview in the web gui, but I also found out that the command ifconfig vtnet0 addr for showing the MAC addresses is not working with any of my interfaces. So I found no way to confirm if a communication on layer2 was even working
Has the PPPoE connection attempt with the provider timed out?

Could you tell me what you mean with "Without the management link" ? I am having a similar issue and would like to see if your fix might also resolve my problem.

@Zerwin My Draytek modem has two LAN links that can be used for management or PPPoE (you can choose which port you use for which purpose by simple connecting the cables, there's no configuration logic in the web gui).
My modem stopped working and i found out that I had to disconnect the management cable, reboot the modem and PPPoE is working again. ...but I can't connect to the web gui anymore due to the missing managament cable... but at least I'm able to use PPPoE again.


✨ Add support for v3 endpoint

This issue mentions the recent changes to the Minecraft Server Status API and that the package needs and update with the following changes:

[ ] Recognize map, plugins and protocol attributes as arrays with different properties.
[ ] Modify the handling procedure for the players attribute.

Updated endpoints:

https://api.mcsrvstat.us/3/
https://api.mcsrvstat.us/bedrock/3/

Update: As of October 2023 I'm currently a bit busy with ongoing studies and other tasks, so even though I've already delayed the release by quite some time, I'll still be rolling it out very soon.

Update #2: I'm currently working on the v3 endpoint implementation inside api.mcsrvstat.py (Python) and once done, I'll work on this one shortly after.


Automate continuous integration checks

**As a** Developer
**I need** automation to build and test every pull request
**So that** I do not have to rely on manual testing of each request, which is time-consuming

Assumptions
* GitHub Actions will be used for the automation workflow
* The workflow must include code linting and testing
* The Docker image should be postgres:alpine for the database
* A GitHub Actions badge should be added to the README.md to reflect the build status

#### Acceptance Criteria
```gherkin
Given code is ready to be merged
When a pull request is created
Then GitHub Actions should run linting and unit tests
And the badge should show that the build is passing
```

added task


[p0ng] Game settings not saved on production build

Only happens with production build. Cannot see game settings file in user:// Debug build works fine.

Resolved in #22

Explanation of what caused the issue:
ConfigFile.load() and ConfigFile.save() don't play well with the assert statement in production build: If you have code such as:
var config_file: ConfigFile = ConfigFile.new()
assert(config_file.load() == OK, "Cannot load file")

or
var config_file: ConfigFile = ConfigFile.new()
assert(config_file.save("res://config.ini") == OK, "Cannot save file")

Then in production build, these lines would just not get run (refer to Godot), and there will be no saving nor loading.


Is this library still being developed?

Is this library still being developed?

Hi,
See this thread https://github.com/CreateJS/EaselJS/issues/1040
And this Slack https://app.slack.com/client/TU5SGL79R/CUHBD3DM3

@owendwyer your slack's link is broken :D

Yes, it seems the Slack channel isn't working anymore.  I guess recent events have changed people's priorities.

So I have the same question, I don't understand the consortium because I don't speak english very well... I hope this project is still supported. So is it still supported or not ? Because there are not updates since 2 years. I use EaselJS for all my project, because it offers high levels functionnalities to manipulate shapes, like Java AWT.  I converted java programs to JS + EaselJS + HammerJS, so I hope I can keep this library

I have selected this project because I saw it is sponsored by Adobe and Microsoft.... So why is there a silent since 2 years ?

I see another project : FabricJS but I don' t like it too much but I see it is active. why this silent for EaselJS ???
Nobody needs high level functionalities for JS game ???

It is maintained with zimjs by Danzen et al. Because Zimjs is built on createjs, danzen takes care of createjs related issues. You can check this slack group.
https://zimjs.slack.com/


Break apart and improve the JS for rasterizing SVGs and downloading files

It now uses a blob URL instead of a data URL in the download process, which is cleaner and better performance.

I will trust you that this just works :tm:


step 5 loading url fails

App fails with message:
You have not configured and imported the Firebase SDK. Make sure you go through the codelab setup instructions.
Followed all instructions but can't see what I'm missing

Turned out I had edited the wrong index.html to include the config


Fixing changelog title for v0.5.0-rc1

What type of PR is this?
/kind cleanup
What this PR does / why we need it:
Removing "unreleased" text
Does this PR introduce a user-facing change?:
NONE

/lgtm


queue:listen (with force flag) doesn't process jobs in maintenance mode

Laravel Version: 9.38.0
PHP Version: 8.1.0
Database Driver & Version: Mysql 8.0

Description:
When maintenance mode is enabled, the command queue:listen --force doesn't process jobs but queue:work --force works.
The flag --force is well documented for the queue:listen command.
Steps To Reproduce:

Clone this repo: https://github.com/JamesHemery/laravel-queue-force-bug
Create and migrate database
Make sure that QUEUE_CONNECTION=database
Run php artisan down
Run App\Jobs\TestJob::dispatch(); in tinker
Run php artisan queue:listen --force (don't work)
Run php artisan queue:work --force (work well)

Seems like you sent in a PR to fix this so let's see how it goes. Thanks


@W-9021650 implement rev 2 of imperative extension

When applied this patch updates the current implementation of imperative extension to the revision 2 specified in the RFC.
More specifically, it:

prevent use ofexternal method.
deserialized imperative extension JSON objects in UtamUtilityMethodAction objects.
add validations to prevent PO authors to write mixed compose statements.
generate proper sub types of Operand and Operation for imperative extensions (getComposeAction method)
generate the appropriate code statement.
clean and delete all references to the previous implementation.
add unit tests for the new implementation.

Thank you Jim! I merged current dev branch and pushed the updated version of the branch. Feel free to merge it when it's appropriate

@jimevanssfdc I pushed a new commit that adds support for the UtamUtilitiesContext object, feel free to have a look when you have time


expose compileUntyped

I'm writing a routing layer that needs to know whether a query is a subscription or not. In order to do this I can call QueryParser.parseText and look at the resulting UntypedOperation, but there's no public way to compile this intermediate result down to an Operation. This PR just exposes that operation.

I'm still not quite seeing what it is you're trying to do ... could you write some pseudo code annotated with "this bit is impossible" flagged up and I'll see if I can come up with something. Or even add a test to this to give me a clearer idea of how you plan to use it.


Facing Issue in compiling with VS2022

When I am compiling in Visual Studio 2022 (in Windows 11) I am getting error as per attachment. I have not made any changes to source code. I have downloaded the files through a git clone as per https://openfast.readthedocs.io/en/main/source/install/index.html#get-the-code, section 3.2.2. I have given the snap of the solution explorer and FAST Property Page. Kindly help me resolve the issue.
I would like to mention here that I first tried to compile in 'Building OpenFAST on Windows with CMake and Cygwin 64-bit' option. It was working fine until I didn't change anything in the source code. But when I replace the 'SubDyn.f90' file with the file provided by Bentley SACS for their integration with OpenFAST, I was getting error. this process required to link a static library file as additional dependency which I was not able to do in CMake environment. It would be appreciated, if anybody can reflect this issue as well. Thanks & Regards.
snap from VS2022.pdf

I would like to give an update. the above issue has been resolved by adding system variable for git. But after correcting I am getting two error while linking as below:
FAST_Prog.f90
ifort: remark #10448: Intel(R) Fortran Compiler Classic (ifort) is now deprecated and will be discontinued late 2024. Intel recommends that customers transition now to using the LLVM-based Intel(R) Fortran Compiler (ifx) for continued Windows* and Linux* support, new language support, new language features, and optimizations. Use '/Qdiag-disable:10448' to disable this message.
Linking...
FASTlib_x64_Debug.lib(mapapi.obj) : warning LNK4075: ignoring '/EDITANDCONTINUE' due to '/INCREMENTAL:NO' specification
libucrt.lib(frexp.obj) : error LNK2005: frexp already defined in libmmt.lib(frexp_iface_c99.obj)
LINK : warning LNK4098: defaultlib 'LIBCMTD' conflicts with use of other libs; use /NODEFAULTLIB:library
....\build\bin\openfast_x64_Debug.exe : fatal error LNK1169: one or more multiply defined symbols found

I would not recommend using the /FORCE:MULTIPLE linking option. Instead, I would attempt to figure out what is causing the linking errors. Likely, the SACS library you are attempting to link with the executable has different libraries and/or build options than the configuration you are attempting to build.
Since you are building with a debug configuration of FAST, I would guess that there is a conflict with the Debug Multithreaded libraries it normally builds with:

Could you try building the Release configuration with your static library instead?

@bjonkman Thanks for your response. As per my understanding, /FORCE:MULTIPLE command Used has nothing to do with the external library that I am trying to link with. Actually, before replacing the Subdyn module and linking the external static library, I tried to compile the OpenFAST as it is. So, without changing anything I tried to compile it and I got LNK2005 error. Though it was in debug mode; I have changed it to Release mode and I got the same error (without using /FORCE:MULTIPLE COMMAND). Here I have attached the build log for your reference. Awaiting your feedback.
Build_Log_Release_Mode.txt

I agree that /FORCE:MULTIPLE is not related to your external library. I just don't think it is good practice to use that command since it could likely result in a problem when running the executable.
I would start with the unchanged OpenFAST code (original SubDyn and no linking to an additional external library). Then make sure you are starting from a fresh build (either do a "Rebuild Solution" or "Clean Solution" and then "Build"):

It looks like you are building the Fortran projects with Intel® Fortran Compiler Classic 2021.12.0 [Intel(R) 64]. What version of the C compiler is the MAP++ dll using? This is the Platform Toolset in the project properties box:

Dear @bjonkman, thanks for response. As you suggested, I started with unchanged OpenFAST code and clean and build the solution. I got the same LNK2005 error.
I am using IFORT Intel Fortarn Compiler Classic and regarding C++ compiler version, I can see that it is v143. kindly refer the below snaps from my visual studio FAST project:

Awaiting your feedback.

Some things to try:

Do you get the linking error with both Debug and Release configurations? Or just Debug? I have had issues with certain versions of the 2019 Math Kernel Library's static debug libraries, but I have not tried the 2021 version you seem to be using.
Can you build this using a different Platform Toolset in MAP? I don't know what C compilers you have available in Visual Studio, but you could try a different one and see if there is still a conflict.

Just FYI: I am using the 2024.1 version of the IFORT compiler with VS 2022 without issue. I installed the oneAPI base toolkit, HPC toolkit and 32-bit base toolkit.

Dear @bjonkman, downgrading Intel oneAPI Base Toolkit to 2024.1 successfully compiled OpenFAST. Thank you for your valuable support in resolving the issue.
I would appreciate your continued assistance on the issue which I explained in my third comment of this thread. to summarize the issue, I need to replace SubDyn.f90 and link a static library as an additional dependency to my FAST project in Visual Studio.

Since I don't really know what SubDyn changes you are implementing, I don't know how much help I will be. If you are still having issues with the build, I would recommend:

Verify that the version of SubDyn they are replacing is from the same version of OpenFAST, or contact them for help in updating their version of SubDyn. If they are missing the SubDyn routines for linearization, it may be much more difficult than just copying the new routines to your version of SubDyn.
Make sure that the static library you are linking with OpenFAST was created with compatible build options. There might be linking conflicts if it uses different runtime libraries than OpenFAST is using.

@bjonkman , thank you for your input. I'm working on gathering the requested information from Bentley.
In the meantime, I've attempted to use FASTv8 with the original, unmodified SubDyn file. Unfortunately, I'm encountering linking errors.
To provide more context, I've attached screenshots of my FAST project configuration, compilation output, and the SubDyn module. I've added the necessary directories for *mod files and the static library to the linker input.

Build_Log_ReleaseX64_FASTv8.txt
SubDyn_SACS.txt
I'm hoping you can provide guidance on the potential causes of the linking error and possible solutions. Please note that my programming experience is limited, and I have a civil engineering background.
Thank you for your assistance

I would first deal with this line from the build log: C:\Program Files (x86)\Bentley\Engineering\SACS 12 CONNECT Edition\SACS_FAST_Mod_DLL.lib : warning LNK4272: library machine type 'x86' conflicts with target machine type 'x64'
You can't mix libraries for 32-bit and 64-bit architectures, so you will have to build your OpenFAST (or FAST8) configuration with Platform set to Win32 (or x86 if that's not an option)
The second issue refers to unresolved symbols:
FAST_Solver.obj : error LNK2019: unresolved external symbol SD_CALCOUTPUT referenced in function FAST_SOLVER_mp_ED_SD_HD_BD_ORCA_INPUTOUTPUTSOLVE
FAST_Solver.obj : error LNK2019: unresolved external symbol SD_UPDATESTATES referenced in function FAST_SOLVER_mp_FAST_ADVANCESTATES
FAST_Subs.obj : error LNK2019: unresolved external symbol SD_INIT referenced in function FAST_SUBS_mp_FAST_INITIALIZEALL
FAST_Subs.obj : error LNK2019: unresolved external symbol SD_END referenced in function FAST_SUBS_mp_FAST_ENDMODS

The PRIVATE statement in your SubDyn module tells the compiler/linker that the subroutines aren't by default defined outside of the SubDyn module. This allows modules to have routines that potentially have the same names as routines in other modules without causing issues. In OpenFAST, there are PUBLIC statements for specific routines to allow the OpenFAST glue code to call them. You can try to add public statements for the routines that it needs, or you could remove the PRIVATE statement in SubDyn.


Status indication

When running NYC methods, say nyc.repoContributors, the user has no information on its status other than that something is going on which is bad for UX and also for the DX to the extent that someone using NYC for a project, he/she won't be able to tell at which stage NYC is in getting the result(s).
What would also be nice is a way to tell when the operation will finish or how far it is from finishing.

Agreed.


Amcache Windows 10

Get-Amcache does not seem to parse Windows 10 Amcache.hve files correctly

There was an issue with a key that had no subkeys. The Amcache class now checks for this.


Add esql getting started notebook

ℹ First commits are draft to test CI
TODO

[ ] Write introduction

@leemthompo are there any differences between cloud and docker image with regards to this? The tests run against a self-hosted minimal container, so the behavior might  be different than running against cloud.
Another issue you are going to hit is that the notebook is also going to run against 8.11 and 8.12. Sounds like I'll need to figure out a mechanism to specify which versions a notebook needs to run on.

The easy option is to add this notebook to the exempt list in this file. The hard option would be to figure out this failure and adapt the notebook to work under Docker as well. And I need to expand the file I linked above to be version aware and provide different lists of notebooks to skip for each given stack release, which I'm planning to anyway.

The hard option would be to figure out this failure and adapt the notebook to work under Docker as well.

I'm not sure if multi-node Docker clusters solves the issue, but unfortunately I ran out of time to confirm this today.

I don't think we can stand up a multi-node Docker instance within the resource constrained GitHub Actions. I think the best course of action for now is to skip this notebook when testing.


cursor.js Not found error in Plugin system example

Wavesurfer.js version(s):
4.0.1
Code needed to reproduce the issue:
Going to example section of plugin system, it not works, because 404 error in CursorCustomPlugin. Locally in localhost works instead.

is this related to #1992 or is it a different issue @marizuccara?

Different issue. The example at public wavesurfer.js address not works because it gives this error, but locally in localhost it find the file. So I don't know if there are necessary other configurations? Maybe it's necessary to import the cursor.js file in different way...


feat: metrics visualiser for mono vertex

config driven metrics framework to display existing metrics in the UI

Currently supports histogram metrics for MonoVertex

When there are many pods.

Mouse hover on the dotted line, the cursor does not always show it's clickable.

Move container info on top of pod info? The pod info remains unchanged when click different container icon but the container info, users need to scroll down to see the changes.
In general, the pod and container info take too much space, reducing it would be better.

What is the reason the MonoVertex can not be deleted/paused? we don't have the UI features?

also reduce the spacing between each entities (elements pod and container info)?

What is the reason the MonoVertex can not be deleted/paused? we don't have the UI features?

we did not implement it as mono vertex did not support pause / resume initially.

we did not implement it as mono vertex did not support pause / resume initially. also for
delete / edit the monovertex, we need to add backend support as well.

i think we can do that in follow up PR


Example SPA app Using ACK NestJs

Hello, I really like your project. Can you share a simple SPA project where we can test the backend project?

Is SPA means Single Page Application?
If yes, is SPA like swagger ui will help you?

Hi @yavuzgunay ,
The API Reference has been deployed. You can visit here


infisical: 0.28.6 -> 0.30.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for infisical is: Official Infisical CLI
meta.homepage for infisical is: https://infisical.com
meta.changelog for infisical is: https://github.com/infisical/infisical/releases/tag/infisical-cli%2Fv0.30.0
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.30.0 with grep in /nix/store/b9ys9d67zq7cyd6gp6wvzpm9ljcc6ijb-infisical-0.30.0
found 0.30.0 in filename of file in /nix/store/b9ys9d67zq7cyd6gp6wvzpm9ljcc6ijb-infisical-0.30.0

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

infisical

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/b7c509164f11f23b544f086d23f03a0dcefd987d#infisical

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

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

1 package built:

infisical

Maintainer pings
cc @akhilmhdh @hauskens for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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

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

1 package built:

infisical


University Project - Porting to a new technology

Hi, I'm a student at USC, and I'm working with a professor on a project where we were hoping to utilize OpenRAM. I see that you have guidelines for what might be required to port OpenRAM to a new technology, but I haven't found any examples of people who have successfully done it.
I was wondering if it is possible to port OpenRAM to a new PDK, and if yes, how would you define the complexity of such a project? Can you point me to any teams who have succesfully ported to a new technology not currently supported directly by the OpenRAM team? Thank you.

Hi Rushikesh,
This is a big project. I would say it is a "large" MS project or a good 3-4 person undergrad project. This partly depends on the technology and how different it is from our existing ones. There may be some features that need to be added to OpenRAM for FinFET, for examle, or other new things.
Hope that helps.
Matt

The documentation is here:
https://github.com/VLSIDA/OpenRAM/blob/stable/docs/source/index.md


Navigating between large, code folded files causes performance problems

OS: MacS 10.12.3 Sierra and Windows 10
Brackets Version: Release 1.9 build 1.9.0-17312 (release 189f6d39a)
Repro  Steps:

Make sure code-folding.enabled preference is set to true.
Make sure code-folding.saveFoldStates preference is set to true.
Set code-folding.maxFoldLevel preference to a high number like 100. Collapse all should really be collapsing all folds.
Create two decent sized xml test files. I'm using this file for both of my test files.
Open both files so they are in the Working Files section of the Project Sidebar.
Collapse All both files so there is plenty of fold state data.
Navigate back and forth between the two files continuously, showing one, then the other, in the editor.

Observed Results:
After three or four loops of navigating to each file, the time it takes for the contents of each file to show up in the editor will get longer and longer, roughly doubling in time for each loop. Eventually, the delay is so long the files can no longer be navigated in a reasonable amount of time.
Expected Results:
The time to navigate to each test file should be about the same regardless of how many time you loop between them.
Workaround:
Make sure code-folding.saveFoldStates preference is set to false. You will need PR https://github.com/adobe/brackets/pull/13269 before you can use this workaround.
Other Notes:
The problem occurs with all extensions disabled.
Since this problem goes away when fold states are turned off, and the time seems to roughly be doubling each time, maybe new fold states are being appended to old fold states instead of replacing them. Feels like there might be some redundant, duplicate data being saved somewhere, just a theory, haven't tested it.

Pinging @thehogfather, wanted to make sure you saw this issue, Patrick.

@lkcampbell got it 👍 .

@thehogfather let me know if you need help. I am curious if my theory is correct. Either way, let me know.

@lkcampbell your theory is in the right direction - but we could nail it down better. The problem is not actually with the routine for saving fold states. The issue occurs when we attempt to restore the saved fold states for the editor gaining focus.
Here is a quick overview of what I have found so far.
When the active editor changes, the fold states for the editor losing focus are saved while the fold states for the editor gaining focus are restored. Saving is not costly. Restoration however involves several calls to codeMirror functions for rendering fold marks on the gutter and creating folded marks in the document. We need to perform this operation at least once - although there is no need to restore the folds for a document already in the working set.
The reason why setting saveFoldStates: false gets rid of the problem is because there is nothing to restore when the saveFoldStates is set to false. Hence the restoreLineFold function has an early exit.
Mind you I am still not sure why calling the function to fold some code and generate the html markup becomes less and less performant. We might need to do some further profiling.
So a quick experiment to address this issue is to prevent the redundant call to restore line folds for files already in the working set. I am currently not aware of any API available in brackets to check if a file is in the working set, (but for now a workaround is to check if the editor object has editor._codeMirror._lineFolds). Do you know of an existing API that doe the same thing?
Here are a couple of charts from a profiling experiment
Buggy

Patched
Ignore the idle time at the start of this profile. It is clear that the first two calls to switching between the files are still slow - and we can try to figure out how to improve that still. But subsequent calls are close to instantaneous.

@thehogfather that's an impressive improvement with your patch. I will probably have more time to look into this Friday and this weekend. I will see if I can find an answer to your Brackets API question then as well.

@thehogfather, I think MainViewManager.findInWorkingSet() is the Brackets API you want to use.

oh awesome I'll have a play with that.

@lkcampbell After tinkering with MainViewManager.findInAllWorkingSets I decided it was not quite right for this use case.  For instance when Brackets is opened, it may already contain a list of files in the working set from a prior session (even before any user interaction). Hence presence in the working set does not necessarily mean the fold gutters and states have been initialised/restored for a given editor. See a16d58e1003fb433afd395cf8db2f52affb62f00 for details of my fix if you would like to test.

@thehogfather, okay, it looked like the right API from the documentation. There has to be a way to  tell if a file is in the current working set somewhere, but let's not let it block getting this bug fixed. I will take a look at your fix soon, thanks.

@lkcampbell the initial fault in framing the problem is mine as I assumed that the presence of a file in the working set implies that the Editor fold gutters would have been initialised. This is not always the case. Specifically when Brackets is closed and re-launched, files are added to the working set based on settings saved from the previous session. I believe this is largely because code folding is still loaded as an extension and not initialised as part of the Editor constructor. I think this is fine since initialising code folding could be expensive and it is best loaded on demand.
So to be clear, the API you suggested does exactly what it says on the tin - unfortunately that functionality is not what we need to solve the problem here.

@thehogfather, your change already got merged before I had a chance to look at it :). I will test it later today and update this issue appropriately.

Closing as fixed.


Editing all gif frames

Question about an existing feature
What are you trying to achieve?
I need to add a watermark to a gif

When you searched for similar issues, what did you find that might be related?
Nothing
Please provide a minimal, standalone code sample, without other dependencies, that demonstrates this question

I tried this code but it didn't work:
  await sharp(original, { animated: true })
  .composite([
    {
      input: watermark,
      gravity: 'center',
      blend: 'overlay'
    }
  ])
  .webp()
  .toFile(original + ".webp");

Please provide sample image(s) that help explain this question

Please see https://github.com/lovell/sharp/issues/2680


feat: add small tags

Changes description

Add medium and small tags
Generic tags now refers to medium tags
Fix typography for medium tags

Context

Closes #151
Checklist

[X] Make sure you are requesting to pull a topic/feature/bugfix branch. Please, don't request directly from your main!
[X] Check commits & PR names matches our requested structure. It must follow the https://www.conventionalcommits.org pattern.
[X] Check your code additions will fail neither code linting checks.
[X] I have reviewed the submitted code.
[X] I have tested on a phone device/emulator.
[X] I have tested on a tablet device/emulator.
[X] I have tested on a large screen device/emulator.
[X] If it includes design changes, please ask for a review design-system-core-team-design GitHub team.

Does this introduce a breaking change?

No

Screenshots
Phone

Tablet

Large screen

Other information

You can download the apk here :
sample-debug.apk :
https://app.bitrise.io/app/62ac2962b2dd627a/build/abab5d59-6c16-4ba7-8413-857015f12277/artifact/2fe94ab2352dbbf4/p/cc7d7f41375ea80734e362bcaf877e6e

You can download the apk here :
sample-debug.apk :
https://app.bitrise.io/app/62ac2962b2dd627a/build/126a831e-8a02-40ed-885c-bf9ef7602a11/artifact/315354d0ca161723/p/4371d83019c8a8203373c9247ea18175

You can download the apk here :
sample-debug.apk :
https://app.bitrise.io/app/62ac2962b2dd627a/build/fbb655cd-bb8a-4bba-ad65-6b6d91f7f9df/artifact/4371e5d171c97050/p/97a040cabf7853de87d6f933e7b9527d


require "ap" Leads to Conflicts

Hi, first thanks for taking this over.
Amazing and Awesome Print both allow one to say require "ap". In the case of existing  Awesome Print  code this mostly works. The exception being code that references AwesomePrint, e.g., AwesomePrint.irb!.
Suggestions:

In ap.rb: AwesomePrint = AmazingPrint (assume they're compatible interfaces)
1 + warnings about this being Amazing Print and AwesomePrint constant will be removed
Unless a goal is backward compatibility (seems like no; from README: ... and should be used in it's place to avoid conflicts) remove ap.rb from future release. Though ruby -rap -e'...' is nice option!
Do nothing!

I think 2 is best. Maybe there are other routes.

@paddor what do you think?

Yeah, I like option 2 using Kernel#warn.

Ok so we can print a warning when require 'ap' is called and AwesomePrint is defined. We could do the same when AmazingPrint.irb! is called.

Assuming you meant AwesomePrint.irb!, how would you implement that? A simple AwesomePrint = AmazingPrint wouldn't be possible anymore. Maybe a new object forwarding all calls to AmazingPrint by default could do the job.

I was thinking that we could update here
https://github.com/amazing-print/amazing_print/blob/7440a61cb80e57014d7ab45f91a5fd85b5df6aaf/lib/amazing_print/custom_defaults.rb#L38-L42
Something like

 def irb!
   return unless defined?(IRB)

   if defined?(AwesomePrint)
     warn 'Things may get a bit weird when AmazingPrint is used along side AwesomePrint'
   end

   IRB.version.include?('DietRB') ? diet_rb : usual_rb
 end

And/or here: https://github.com/amazing-print/amazing_print/blob/7440a61cb80e57014d7ab45f91a5fd85b5df6aaf/lib/amazing_print.rb
if defined?(AwesomePrint)
  warn 'Things may get a bit weird when AmazingPrint is used along side AwesomePrint'
end

I was thinking something like this (untested):

if defined? AwesomePrint
  warn "You've loaded AmazingPrint and AwesomePrint .. some useful info here ..."
else
  AwesomePrint = Class.new do
    AmazingPrint.methods(false).each do |name|
      define_singleton_method(name) do |*argz|
        warn "More useful info here ..."
        AmazingPrint.public_send(name, *argz)
      end
    end
  end
end

@sshaw what's the advantage to defining AwesomePrint within AmazingPrint?

This is the gist of solution 2 in my original comment.
If code requires ap and expects AwesomePrint but instead it gets AmazingPrint things break.
A warning implies continued execution and continued execution may depend on the existence of AwesomePrint. So why warn when seconds later a NameError (uninitialized constant AwesomePrint) will be raised. So the idea is to support Awesome Print code for a time, but warn the user about the situation and eventually remove it.
The other 2-like option is to just assign to the constant and support it without warning or until there's reason to break compatibility and alter the existing AwesomePrint interface.
Hope that clarifies things!

In this case:
def irb!
   return unless defined?(IRB)

   if defined?(AwesomePrint)
     warn 'Things may get a bit weird when AmazingPrint is used along side AwesomePrint'
   end

   IRB.version.include?('DietRB') ? diet_rb : usual_rb
 end

It assumes one has both loaded. That's possible but I think it's probable  unless two different paths are required  since require only loads once.

I guess I'm less concerned about situations like
require 'amazing_print'
AwesomePrint.irb!

because it would be strange for someone to add these lines together or it would be an incomplete change to update the require statement and not the next line.
In this case, why not keep the error so the user updates the next line?

This is quite stale now so I'm going to close unless there's still interest in this.


Update MARK-V to version 1.01

Includes:

Multi-sequence move-to-front
Coding by id descriptor
Statistical coding of non-id words
Joint coding of opcode and num_operands

Bots show assertion failures.

The AppVeyor failure might be spurious. I  plan to verify on my own Windows machine first.

Ready for review again.

Failing on Windows bots.  See https://ci.appveyor.com/project/Khronoswebmaster/spirv-tools/build/1548/job/w959v91c4iaiql4s
Please rebase and make sure it pases bots.

Waiting for the bots to go green.

I manually tested on OSX with Clang in a Debug build, and the tests passed.

Rebased, squashed, and pushed into master as  b36acbec0e


Add new unit tests to web app

Tasks

[ ] Add new unit tests to the web app (use a fork, see CONTRIBUTION.md first)

Bonus Task

[ ] Develop inside the dev container

Blockers

#24 (not completed yet but no longer a blocker to get started with adding tests)

Setting ticket status from Blocked to Open.
@rrchai The main branch now includes several pages and components for which you can add your tests. I recommend starting with a PR that only adds new tests to one existing component to get you familiarized with the new environment. After that, it is still a good idea to stick to PR of relatively small size, e.g. a PR that adds tests to a single page.


openapi: resolve the synthetic_size duplication

We had accidentally left two endpoints for tenant: /synthetic_size and /size. Size had the more extensive description but has returned 404 since renaming. Remove the /size in favor of the working one and describe the text/html output.

Running make gogenerate in the cloud repo with the updated swagger did the trick last time (see neondatabase/cloud#12340)

I'll do it (or have someone do it) once I get the #7650 haggled (this week).
@duskpoet could you actually give this a quick round of review since I think you added the original /synthetic_size description but I forgot to tag you in? If you are delayed, I might merge this in and we'll need to make sure the javascript rendering the SVG will not be broken by this or you can just reply if you know there's an automatic test against this openapi.yaml and where can I see it fail.

hey! nothing has changed in the structure, so I don't see how anything can break


Add note: download directory must be outside repo

See #104

We addressed this in https://github.com/deepmind/alphafold/commit/0be2b30b98f0da7aecb973bde04758fae67eb913.
Thanks a lot for suggesting this, we acknowledged you in the release notes for v2.1.0.


Does this way of writing chainer v2.1.0 fix some first layer parameter for fine tune correct?

I am sorry I put the question here rather than stackoverflow because I am too hurry for CVPR 2018, which deadline 11.08 I can't wait.
Does this way of writing chainer fix some first layer parameter for fine tune correct?
I am modifying your chainercv faster-RCNN to my research.
I want to fix some parameter of first serveral layers of chainer to fine tune an already pretrained model:

def __call__(self, x):

    if self.fix:
        with chainer.no_backprop_mode():
            h = self.bn1(self.conv1(x))
            h = F.max_pooling_2d(F.relu(h), 3, stride=2)
            h = self.res2(h)
    else:
        h = self.bn1(self.conv1(x))
        h = F.max_pooling_2d(F.relu(h), 3, stride=2)
        h = self.res2(h)
    h = self.res3(h)
    h = self.res4(h)
    return h

Does this using chainer.no_backprop_mode correct? I am using resnet101 to fix until res2 layer.

Yes.
You can confirm that the weighs are not updating by printing the value of weights.
Something like res2.disable_update() will also cancel updating weights.


Loot tracker shows graceful boots in seed pack

Type
Incorrect behavior
Operating System
Windows
OS Version
Windows 10
Bug description
The loot tracker shows the graceful boots as being received from a seed pack
Screenshots or videos

RuneLite version
No response
Logs
client.log
launcher.log

it tracks inventory changes when you open a pack, if you also have your boots (or any other item) appear in the inventory when this happens they also get tracked


Test

No significant performance differences

No significant performance differences

No significant performance differences

I tested a slight modification of this PR with python35_config on Kokoro (it doesn't have python3.6, but it has python3.5) and it has passed.
https://sponge.corp.google.com/invocation?id=6dc5fa71-87c4-400a-b446-29e2575d3cbe&searchFor=
To get tests passing on Kokoro, can we use python3.5 as the default compiler whenever we are running on Kokoro? (if env variable KOKORO_JOB_NAME is set, it's a good indication we are)?

I'm fine keeping it consistent for now with 3.5, and we can move to 3.6 once that is supported.

@kpayson64  nice, I didn't know we had Python35 on our Jenkins workers.

No significant performance differences

#11386

This breaks sanity - the Dockerfile needed to be updated in the template file.

Oops, fixed in https://github.com/grpc/grpc/pull/11395


fix wrong arg name

Problem
cli.js --url doesn't get pass to webhook.start cause webhook.start expects --webhook-url
Solution
Make webhook.start accepts url arg instead
Result
Make cli.js works when pass along --url

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


Set the document title  for  Bog page -- @Idrice Contact and Produits has been done

Test it:

Browser the app
See the title on the tab header : MNS 237
Select Actualites ----> you should see MNS 237 - Actualites

Same test for contact --> You seen   MNS 237 - Contact

this is a problem on the cloud
i can test the app on cloud for ( Actualites and MNS and Contact ). when i click on it it does not open .

Check you job email

done

what means Done when you check your email job?

https://idrice24.github.io/mns/contact --> on Clound working fine

i am checking it and i am seeing an respond from one company (Khalifa Building office No.105, same building of Al Raha Furniture Al Istaqlal Street beside Pan Gulf Furniture on Main Road Abu Shagara Sharjah) that i should come with my CV, PASSPORT copy, VISA copy at this company

you have to search information abut those compagnies --> i will ask you question through channel phone

oK i am doing it
On Tuesday, December 22, 2020, 12:23:21 PM GMT+1, Ghislain<EMAIL_ADDRESS>wrote:
Test it:

Browser the app
See the title on the tab header : MNS 237
Select Actualites ----> you should see MNS 237 - Actualites

Same test for contact --> You seen MNS 237 - Contact
—
You are receiving this because you were assigned.
Reply to this email directly, view it on GitHub, or unsubscribe.


Lesson not found in training session history

Your Vatsim CID
1560689
Description
When using training history to look up a students tickets, when I clicked on one to view it, the message Lesson not found was shown
Minimal Reproduction
Search for a student, select a ticket.
i.e. Akil Gunn, 2-2-1
Additional info
Tickets appear normally when looking in the training sessions tab and clicking
Validation

[X] I have searched open/closed issues and did not find any similar bugs

Known bug already fixed


Test solution E2E

In this PR:

Updated SDK version to 34.
Added Edge to Edge.
Removed unused variable, focusManager.
Corrected the vertical alignment for Round the tip amount Row.

Navigation bar is not translucent. It is not showing the bottom of the letter "p" that descends.
App is not going to the left edge when in landscape orientation.

This will be as is:

App is not going to the left edge when in landscape orientation.

API 33 e2e UI changed and no longer matches pre e2e API 33 UI.
Note - PR was approved (e2e UI was correct) but subsequent commits after approval changed the UI so it was un-approved.

I am not able to reproduce:


I have been trying to load images from xml/svg but it is not loading in iOS.

[Enter feedback here]
Hi,
I am creating a new MAUI app but it is facing some issues when building the iOS app. It is not loading any images from xml/svg . Is there any solution or workaround for this?

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

ID: cb4d16da-8c0f-5b7e-54ee-1fe36c549837
Version Independent ID: cb4d16da-8c0f-5b7e-54ee-1fe36c549837
Content: Image - .NET MAUI
Content Source: docs/user-interface/controls/image.md
Product: dotnet-mobile
Technology: dotnet-maui
GitHub Login: @davidbritch
Microsoft Alias: dabritch

Hi @Awesomer9561
Sorry to hear that you're having problems. This is an issue with the product rather than with the docs, so I suggest you open an issue in the engineering repo where the engineers will be able to advise whether there are issues on iOS. Thanks!


feature-32 Update Colors in styling/theme

Checklist ✅

[x] I have followed the contributor guideline
[x] Tests are included
[x] Documentation is changed or added
[x] Related issue has been created
[x] Commits messages follows commit guideline

resolves #32

I'll close this one, lets create a new one with all the develop changes.


06/12/2018 03:55 : Create Pull Request for branch master -> prTesting1_2018061203

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestCon1.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Add GovGen chain

Can you please changed the 8-space tabs tabs with just 2 spaces?

Done here ed8d777
Thanks for the review!

@JeremyParish69 all good for it, few wallets are using this repo as reference to add for custom chains support

@tbruyelle Hi. Added some fixes for Allnodes nodes and submitted PR to your github account.

@tbruyelle Hi. Added some fixes for Allnodes nodes and submitted PR to your github account.

Thank you, merged!


SVGGraphics getSVG() generated pattern cannot be displayed correctly

new pdfjsLib.SVGGraphics(page.commonObjs, page.objs).getSVG(...);
PDF that can reproduce this issue:
Testing.pdf
Result of generated SVG:

It works fine with PDF.js demo site, I checked code the implementation of SVG seems not the same as the Canvas version.

This is most likely a duplicate of one (or more) existing issue in https://github.com/mozilla/pdf.js/labels/svg
Furthermore, please note that as mentioned in the FAQ the SVG back-end is not officially supported; see https://github.com/mozilla/pdf.js/wiki/Frequently-Asked-Questions#backends


Refactor RequestData

This PR is meant as the first step to undo making RequestData the God Object it is today.

RequestData is now completely readonly
PostData is moved outside RequestData
HttpMethod and Path and RequestParameters now live outside RequestData, more info on this later.
OpenTelemetryData now lives outside RequestData.
Fixes several instances of boxing in HeaderList constructors.

This now also introduces 2 new records.
EndpointPath a readonly record struct that instructs what API we're calling.
Endpoint takes and EndpointPath and Node to take ownership of Uri creations.
This is hopefully a first step where we can cache these in calling libraries.

TransportConfiguration now implementents IRequestConfigation

The goal here is for RequestData to turn into BoundConfiguration and instance we can cache per TransportConfiguration. Unless a local per request configuration is provided in which case we'll need to new a new instance which is rare.

Without having looked at it:

Unless a local per request configuration is provided in which case we'll need to new a new instance which is rare

Maybe we can find a way to avoid the allocations. I need the UserAgent to be present in the IRequestConfiguration in order to override it in high level integrations that allow users to pass in a custom instance of the client. In these cases, we can't modify the user agent in the ITransportConfiguration without affecting the whole client instance.
This would cause the allocation to happen for every single request.

The goal although not yet implemented is that if TransportConfiguration is provided (one exist per client) we never have to allocate more then 1 RequestData for all requests. This is 90% of the time the case.

Sounds great 👍

Do you need to specify a different UserAgent per request/api?

That's not the case, no. But I still need a way to override the UserAgent for every single request. This is beacuse in some high level integrations like e.g. the Semantic Kernel connector, we allow the user to pass an already initialized ElasticsearchClient instance to the integration (this is a pattern that we can't easily change as it's essential in a lot of scenarios like e.g. in a dependency injection container).
This client instance is internally used to perform the requests. We want to use a custom UserAgent here for tracking purposes, but at the same time we do not want to modify the "global" TransportConfiguration, because that would mess with the UserAgent on all other places where the user uses the same client instance.
My idea was to add UserAgent to IRequestConfiguration for this purpose.
Your refactoring is a good improvement anyways for all other cases where we do not have to override on a per-request basis. Maybe we can be smart in some way to optimize the per-request-override case as well.

That's not the case, no. But I still need a way to override the UserAgent for every single request. This is beacuse in some high level integrations like e.g. the Semantic Kernel connector, we allow the user to pass an already initialized ElasticsearchClient instance to the integration (this is a pattern that we can't easily change as it's essential in a lot of scenarios like e.g. in a dependency injection container).

Would it make sense to consider the client passed to Semantic Kernel dedicated to that use, i.e. create an instance with the configured UA etc. that is only used by Semantic Kernel. If the app also needs to use a client for other ES-related code, they could have a second instance of the client configured for that use case. Although there is some extra overhead, this might be the rare use case where having more than one client in an app is reasonable?


Does OpenAttack2.0 support NER

Sorry that OpenAttack 2.0 doesn't support attacks against NER models, because we haven't found any studies on attacks on NER. In fact, adversarial attacks against NER are not well defined, e.g., is it OK to perturb the entity sequence? what is the aim of attacks (misclassify an entity or wrongly recognize a sequence)?


Tiny enhancements

I have changed these from my (as a user) experiences.
IMO,

Backup variables before the download links would be more convenient. There's a possibility to forget about this if it's under the long comments.
We can use the latest.zip URL on the download link so that we can avoid updating the URL every time.
Usually, we update to the latest version. So, the reverse order could be more convenient.

@biplobice Thx


Allow to set up argilla server settings through config file

Support for settings.json or settings.yaml file containing Argilla server configurations.
This can enhance the way users can set up specific settings for backend or database configuration. Also, can help users to know all available settings from a well-documented settings file.

I agree, and ideally, we should also align this with CLI support.

@gabrielmbmb is there any plan to maintain more than one settings.json or sort those by name? e.g.
{
    "default": {
        "host": ...,
        "key": ...,
    },
    "project-a": {
        "host": ...,
        "key": ...,
    },
    ...
}

And select one of those as argilla use project-a? I'm asking in case we want to allow logging in different existing Argilla server instances, while keeping always one with the identifier default to be used by default, and changing that based on the argilla use value or something similar.


Validate configuration files before starting the bot

Describe the bug
Based on a previous issue https://github.com/threefoldtech/tfgrid-sdk-go/issues/167#issuecomment-1575603963 where i didn't realize that something was wrong with my .env file and the bot reported that there's an error with the relay url. validation for the configuration files should be added to prevent errors like this in the future.
The monitoringbot should validate both the .env & wallets.json files before running the bot and in case of incorrect validation it should stop the bot and display a clear error message mentioning that the validation is correct for a certain field and if it's not possible to mention where the exact error is then it should mention that one or more input/s in this file is incorrect.
The main inputs that could be validated are:

Mnemonic (validate that it has the correct format and registered in the mentioned network)
tfchain address (Validate that it has the correct address and belongs to a twin on the mentioned network(if possible) )
Not sure what the validation of the other fields could look like but mainly it could be valid format or inputs (ex. Mins field should accept numeric values only, etc...)

To Reproduce

While configuring the .env or the wallets.json enter some incorrect input/s.
Run the bot.

Expected behavior
The bot shouldn't run in the first place if one the inputs is invalid.

Validating mnemonics is done, validating wallets is moved to a later milestone #181

Verified.
After building the latest version of the monitoring bot locally since it's not released yet. i was able to test the new validation by adding incorrect mnemonic for each network and now when the mnemonic is invalid it fails to start and mentions that the mnemonic for this specific network is invalid.


app zip caching

I'm not sure if this is a toast issue but I have a 2014 UN55H6203AF TV and it keeps caching my app zip file.  I end up having to unplug my tv and then re-sync to get the tv to accept the new app.
any thoughts?

Hello, bstegman.
I'm sorry but i can't understand the question.
what is the meaning of "app zip caching" & " ~ keeps caching my app zip file. ~" ?
plz explain more specifically.
Thank you.

When I prepare/build my app and then I deploy it to the TV it does not get updated on the TV.  The TV keeps using the previous built app.  Even if I manually delete it from the TV then re-sync it keeps using the previously built app.
Your help is appreciated.  Let me know if you need more.

Hi, bstegman
Thank you for your explanation. I got it your problem.
In conclusion, "14 device" for faster loading any application is saving in cache memory its source code.
So that issue happened.
There are few solution :

instead of synch apps, try to launch apps from USB, and before, every time delete an application.
...
...

If you want more details, plz go to under Link.
Link
Thank you.

I will try that.  thanks.