Ensuring that poly<C, non_owning_storage, V> is trivially copyable.

 There are more steps to take to improve this (i.e. there are several concepts that make no sense to type erase for non_owning_storage and we can just remove them), so this is just the first go.

Hi Barry, the build failure looks unrelated to this PR.
This error is caused by calling benchmark::DoNotOptimize with a non-const reference to an object that contains a const subobject. I see Louis has already reported the issue: https://github.com/google/benchmark/issues/764
I'm just here because this was the only search result for "error: read-only reference" "used as ‘asm’ output", so hopefully that helps someone else 🙂


substraTEE-signer on ARM TrustZone

write a simple wallet app for ARM that can sign and send transactions to substrate RPC.
Basic Idea described in:
https://github.com/scs/substraTEE-doc#substratee-signer
example code (javascript) using oo7:
https://github.com/paritytech/oo7/issues/102

The substraTEE-signer does not exist anymore and therefore will not be implemented on ARM TrustZone in the near future.

agreed. we should think about porting substraTEE-worker to ARM trustZone instead. Maybe @akoskinas can assess the difficulty of doing that with Trustonic SDK. I guess OP_TEE is out of question as we saw there are no integrity or confidentiality guarantees so far.


UWP apps open with small margin on the left or right.

Not sure if this is fixable at all, because these apps do what they want.
Nice:
A Windows 10 app like Mail or "Microsoft To Do" correctly snaps in the left zone via mouse or keys.
Issue:
On reopening the app, there is always a margin on the left of about 16 pixels

We need to know what version of powertoys and Windows you're on.
For FZ target'ed q's:

what layout are you using?
A custom made one or one of our provided defaults
What is your gap setting if you have one

I'm on the latest versions of powertoys (v0.20.1) as well as windows 10.
I have "Space around zones" set to 0
Looking into further it I found that the issue occurs for my two simple custom layouts (2 zones and 3 zones)
UWP apps will always re-open with a distance of about 12-16 pixels to the left or right of the screen.
For the standard 2 columns layout I get another issue:
UWP apps assigned to zone 1 or 2 will always re-open maximized.
Note that with "UWP apps" I refer to the built in Mail, Calendar, Microsoft To Do, and the Windows Settings.
All show the same behaviour.
All other non-UWP programs I tried work just fine.
It's hard to imagine that this is not an obvious error seen by everyone.
The only maybe "exotic" thing I have is a 4k screen and Scale set to "150% (Recommended)" in Display Settings.

Could not resist, tried setting Display Scale back to default 100%. It's magic, the Fancy-Zones just stay the same. Nice.
Unfortunately, the bugs also stay the same.

This is not a FZ bug, it's a default UWP behavior and it can be reproduce without FancyZones running.

As I said in the first line "Not sure if this is fixable at all, because these apps do what they want."
But I thought, maybe, just maybe you have insights on how to override the silly "default behaviour".
What about a "dirty" trick where you would assign it again to a zone after it has opened.
If you can put it in a zone via keys, it's hard to see why you can't somehow do the same on open.

@NUU-Solutions
we can try the trick of reassign it to the zone, but it may be a rabbit hole.


discussion: shall we introduce WITH properties for materialized views?

Shall we introduce a WITH clause for create materialized view statement?
We have WITH for tables, source, and sink to define some additional parameters. However, currently there is no such place for materialized views.
For example, rate_limit should be a per-stream property and should be able to be viewed and modified after MV creation. See also https://github.com/risingwavelabs/risingwave/pull/11919#issuecomment-1696731719. But this motivation seems not strong enough, because it's designed as a makeshift in case of creating MV failure.

Minor improvement to UX: clearer semantics for session variables associated to MV.
Things like RW_TWO_PHASE_AGG, STREAMING_PARALLELISM can be tagged to specific MV, as per their semantics.

RW_TWO_PHASE_AGG may be not the case? It is an optimizer hint and applies to batch query as well.
STREAMING_PARALLELISM. Yeah, we can move the scale out/in commands here. Modifying the parallelism means scaling.

RW_TWO_PHASE_AGG may be not the case? It is an optimizer hint and applies to batch query as well.

My take is that as with global variables, setting session variables means the user may mistakenly use two_phase_agg with all queries in the session.
If they can scope it to a materialized view, it is less likely they make this mistake.

maybe all session variable can be defined here for the query to overwrite the session varible such as timezone?

maybe all session variable can be defined here for the query to overwrite the session varible such as timezone?

Good point. At least timezone sounds good to me.
Another random thought: how about capturing these session variables into the WITH clause automatically? It seems to be an elegant way to persist these variables (I don't know how it was done in current implementation). Additionally, when users run show create materialized view, the context can be printed as well.

Just FYI, we already have a context structure for each streaming job:
https://github.com/risingwavelabs/risingwave/blob/b823b0ce7b68f032c00e03d4cdda95bdd6a9f1b4/proto/stream_plan.proto#L723-L728
We can consolidate them in one place, such as storing all session-local variables here or in the WITH clause of the definition.

Recently, I have been considering how to achieve the best availability for altering streaming jobs and have come up with similar ideas. It turns out we have to apply the same set of session variables to produce a similar plan with the updated SQL query.
This may also apply to version upgrades. Imagine that there are some new optimization rules introduced in a new release of RisingWave, a slightly updated SQL query may compile into an incompatible plan and then fail to reuse the previous states. However, if we can control the effect of the new rules with a toggle of the session variable, we can automatically instruct the compiler to avoid applying these rules according to the absent entry in the persisted session variables from the previous streaming job.
Even so, we also need to introduce a mechanism to allow users to explicitly update the session variables when altering streaming jobs.

maybe related: we need some refactors to achieve a strong type WithOption framework. https://github.com/risingwavelabs/risingwave/issues/9450

Some preposed refactoring (like https://github.com/risingwavelabs/risingwave/issues/7034#event-10601936580) might be necessary to avoid getting further messy.

As mentioned above, RW_STREAMING_RATE_LIMIT seems to be a good starting point

We now already have create materialized view mv with ( streaming_rate_limit = 200 ) since #13009

We now already have create materialized view mv with ( streaming_rate_limit = 200 ) since #13009

Alter not supported yet.


Datalist form info does not disappear after 3 seconds in edit mode

Describe the bug
When you are in edit mode and navigate to a form that contains a datalist, the form info for the datalist displays but does not disappear after 3 seconds. The main form's info disappears as expected, but the datalist's info only disappears after you hover over the datalist section, which should not happen.
To Reproduce
Steps to reproduce the behavior:

Go to 'https://functional-adminportal-test.azurewebsites.net/dynamic/TestModule/driverList-table'
Enable Edit Mode.
Refresh the form and keep the cursor on the browser’s refresh button.
Observe that all form info on the page disappears after 3 seconds except for the datalist form info, which persists until the datalist section is hovered over.

Expected behavior

The datalist form info should disappear automatically after 3 seconds, along with the rest of the form info, without requiring the user to hover over the datalist section..

Actual behavior

The datalist form info remains visible until the user hovers over the datalist section, while the main form info disappears after 3 seconds as expected. See attached video: https://jam.dev/c/d4e3eab9-554b-4a9b-b139-860b93828331

Screenshot:

Related Item: #1517

Hi @James-Baloyi , the issue is still persisting on the datalist component. I've cleared my cache and still get the same results

Navigate to: https://functional-adminportal-test.azurewebsites.net/dynamic/TestModule/newmode-details?id=1b82e19b-a799-4401-9971-00b7bcd7ba4b
Switch to edit mode
After 3 seconds, the form info will disappear
Refresh the page and keep your mouse far away from the form/keep it on the refresh icon
Observe all the forms info disappearing except the one rendered within the dataList. See attached recording:  https://jam.dev/c/de0071ea-d0de-455d-9fb4-8233c730947c

@hlayisanimazuze the issue has been addressed and we have updated the behavior of the 3 second timer to only trigger when the user clicks on the 'Edit Mode' toggle. This means form info blocks only go away after an explicit user action instead of when the page is reloaded and there's no explicit action to trigger the behavior.
This is a best practice requested during the review process.


VisUI: support FileChooser

While it's a desktop only widget, I think it worth to add support for a builder for it.

I think there were some issues with including desktop-only utilities in the core lib, but I honestly don't remember why we decided against it back then. @kotcrab Can FileChooser DSL cause any issues with Android/iOS builds?

I don't remember either but I don't think so. I moved usage of all problematic APIs (java.awt and sun.awt)  behind reflection so should be fine on Android but I can't say anything about iOS.
It's also a question whether should we have builder method that works only on one platform with all the other cross-platform ones.

Now that you mention it, I think these were the original reasons:

FileChooser is limited to a single platform, while all other KTX utilities work everywhere. It would be somewhat inconsistent.
There's no need for an advanced GUI builder when it comes to FileChooser, since its API is pretty much "self-sufficient". A builder for FC would end up a fancy constructor, since you're unlikely to add custom widgets to its Window.

I might be wrong though - is there a common use case where you'd want to add actors to a FileChooser?
@ishitatsuyuki If you don't agree, and if you're willing to set up a pull request with a FileChooser builder prototype, I'll be happy to review it.

Yeah, I agree. It would end up as an apply style constructor, and I guess it's even trickier as the documentation suggests to reuse single widget, while others are mainly constructed for every demand.
Feel free to close this issue if this has too much concerns to implement.

I'd be willing to sacrifice a little bit consistency for usability, but I just don't see how FileChooser would need access to GUI building code. I'm closing this issue for now, but feel free to reopen it if you find a use case for this utility.


Git submodule issue

When using the git submodule update --init --recursive command I kept receiving the fatal: Could not read from remote repository. error.
Changed .gitmodule to the following to make it work. Not sure if this is worth doing in the repo to avoid the error occurring for other users but thought it would be worth mentioning.
[submodule "tensorrt_llm"]
	path = tensorrt_llm
	url = https://github.com/NVIDIA/TensorRT-LLM.git

https://github.com/triton-inference-server/tensorrtllm_backend/issues/31

Close this bug because it is duplicated. Please re-open if you still see the issue.


Execution failed for task ':app:processDebugResources'.

Execution failed for task ':app:processDebugResources'.

java.io.IOException: Could not delete path 'C:\Users\youxdd\Documents\RNative\thereApp\android\app\build\generated\source\r\debug\android'.

Solved, is my terminal tool problem

i have this problem


Error in plugin @matterlabs/hardhat-zksync-solc: Connect Timeout Error

Hi Team, While i compile the solidity contract using 'hardhat' I am getting below error with default hardhat config given https://era.zksync.io/docs/dev/building-on-zksync/hello-world.html#initialize-the-project
Using node version v18+

Could anyone please look into this
Thank you

Hi @sprprashanth1443, do you still have the same issue?

We've reproduced this on an OSX m1 OS as well @mpopovac-txfusion
@AjmalAhamedKodakkat

Downloading zksolc 1.3.10
Error in plugin @matterlabs/hardhat-zksync-solc: Connect Timeout Error

For more info run Hardhat with --show-stack-traces

We released the new version of @matterlabs/hardhat-zksync-solc hardhat plugin (0.4.0) with increased timeout. Sometimes when the internet's not up to speed, this problem surfaces. Don't hesitate to reach out if it persists.


syntax coloring for functions not consistent

The syntax coloring for the functions is inconsistent, e.g. is_complex_number is recognized as a function, while is_non_real_number is white (and not recognized as a function?). Similar, get_number_inner_type is not recognized as a function, maybe because of #run?
Note: this is untested code and might have some errors.
Cheers :)

Yeah, its the parens that are throwing it off.  The grammar file needs a rewrite, but I'm waiting until after either the syntax pass happens or the first release.

Newlines have a role to play too.

Slightly related, this one looks like a greedyness issue:


Are all bones required?

Question:  when you define a new skeleton humanoid, it is required to define all the bones?
Obviously, if you define LeftWrist, you must also define LeftElbow and all other bones up to the root.
My concern is with the hands, there's lots of joints there, and maybe a particular mesh or target model has less bones.
Also, what about the face?   eyes? chin?

Question: when you define a new skeleton humanoid, it is required to define all the bones?

No. They are optional and some of them can be omitted in an asset. I think it's the same as the VRM spec. I will add clarification about it in the spec.

Also, what about the face? eyes? chin?

@robertlong has opened an issue #8 about it.


Fixed javadocs.

Thanks for the submission, @matemoln, but if you take a look at the DefaultTaskContainer implementation of TaskContainer#create, you'll find that its @throws tag is actually correct. That is, DefaultTaskContainer#create can indeed throw InvalidUserDataException as advertised.


Handle variables in help text

I'd like to do something like this:
[tool.taskipy.settings]
use_vars = true

[tool.taskipy.variables]
some_var = "foo"

[tool.taskipy.tasks]
baz = { cmd="echo '{some_var}'", help = "Do something like {some_var}." }

However, variables aren't handled in help text, unlike in commands:
$ task -l
baz    Do something like {some_var}.

$ task baz
foo

@Zoidmania Thanks for raising this, let me take a look into handling this 🙂


KeyboardInterrupt is not handled and changes the return code

I have a task server defined as follows:
[tool.taskipy.tasks]
server = "uvicorn main:app --reload"

when I press CTRL+C to exit uvicorn server I get an exception traceback and return code to shell is 130.
$ poetry run task server
INFO:     Uvicorn running on http://<IP_ADDRESS>:8000 (Press CTRL+C to quit)
...
^CINFO:     Shutting down
INFO:     Waiting for application shutdown.
INFO:     Application shutdown complete.
INFO:     Finished server process [65600]
INFO:     Stopping reloader process [65598]
Traceback (most recent call last):
  File "/Users/ali/Workspace/testproj/.venv/bin/task", line 10, in <module>
    sys.exit(main())
  File "/Users/ali/Workspace/testproj/.venv/lib/python3.8/site-packages/taskipy/cli.py", line 11, in main
    run_task(args.name, args.args)
  File "/Users/ali/Workspace/testproj/.venv/lib/python3.8/site-packages/taskipy/run.py", line 57, in run_task
    exit_code = run_commands_and_bail_on_first_fail(commands)
  File "/Users/ali/Workspace/testproj/.venv/lib/python3.8/site-packages/taskipy/run.py", line 12, in run_commands_and_bail_on_first_fail
    p.wait()
  File "/Users/ali/.pyenv/versions/3.8.1/lib/python3.8/subprocess.py", line 1079, in wait
    return self._wait(timeout=timeout)
  File "/Users/ali/.pyenv/versions/3.8.1/lib/python3.8/subprocess.py", line 1804, in _wait
    (pid, sts) = self._try_wait(0)
  File "/Users/ali/.pyenv/versions/3.8.1/lib/python3.8/subprocess.py", line 1762, in _try_wait
    (pid, sts) = os.waitpid(self.pid, wait_flags)
KeyboardInterrupt

$ echo $?
130

However if I run uvicorn directly with poetry it shutdown gracefully and return value is 0.
$ poetry run uvicorn main:app --reload
INFO:     Uvicorn running on http://<IP_ADDRESS>:8000 (Press CTRL+C to quit)
INFO:     Started reloader process [65153]
INFO:     Started server process [65172]
INFO:     Waiting for application startup.
INFO:     Application startup complete.
^CINFO:     Shutting down
INFO:     Waiting for application shutdown.
INFO:     Application shutdown complete.
INFO:     Finished server process [65172]
INFO:     Stopping reloader process [65153]

$ echo $?
0

Taskipy changes exit code of uvicorn which is not desired behaviour.

Hey!
Thanks for reaching out, I'm looking into this.

A version including the possible fix for your bug was released (1.1.3)
I would appreciate if you could try it out and confirm that it now works properly :)
Thanks

Hey there, @aminfara! Do you have any good news for me? :)
Thanks!

Roy,
Thanks for very prompt response on this. I just updated taskipy to 1.1.3 and I confirm that KeyboardInterrupt is now managed properly. Please feel free to close the issue. 👏

Great! Glad to hear. Closing this issue.


Ability to render MDX with edge runtimes

I'm loving the DX of content collections so far.
Is there any way recipe to getting MDX rendering in Cloudflare Workers to work?
For security reasons, the Cloudflare Workers runtime does not allow dynamic code evaluation via eval() or new Function() . It also requires all modules to be defined ahead-of-time before execution starts.
Hence <MDXContent /> doesn't work, and I'm wondering if and what the alternatives are here, or whether we're locked to Vercel forever if we decide to invest further into adopting content-collections with Nextjs.

After some research, it seems like safe-mdx is the way to go for now.
It has some limitations (no importing, etc). But maybe importing within mdx files is not so great anyways. I can only imagine if we want to move over to a headless CMS at some point that we would need to refactor so many things to make it work if we start importing components within MDX directly.

@lauri865 could you please post you solution with mdx (transform function and client code)."Perhaps it makes sense to include an example or even a separate package for safe-mdx.

Same problem, I don't know how to intergrate safe-mdx, seems I have to switch from MDX to markdown. 😢


Is it possible to register helpers at the partial level instead of global?

I want to switch from Blaze/Meteor to Handlebars/Express, and I am looking for a way to register helpers on the partial instead of the global scope.
I guess I am looking for a function Handlebars.registerPartialHelper instead of Handlebars.registerHelp.
Should I just register all my current Blaze template helpers as global helpers in Handlebars?
(I find the Blaze way very practical in terms of code isolation)

No, it's not possible to do that. The helpers-object is a singleton within the Handlebars-runtime environment.
I would register all helpers as global helper. Of course, you could try to create namespaces yourself to avoid name-clashes. (by prefixing the helper-name with the partial-name)

I think I found a simple solution to achieve what I want by using decorators as described here.
I currently have a script that generates .hbs templates from my Meteor project.
My idea is to generate a decorator helper per template from my Meteor template, and to insert an activation line in each template:
// navbar.hbs
{{* navbarHelpers}}
// navbarHelpers.js
Handlebars.registerDecorator('navbarHelpers', function(program, props, container, context) {
    container.helpers = {
        ...
    };
});

I will update once I have tested it.


What is the corresponding relationship between the vertices of BFM 2009 and BFM 2017

Hello, I want to convert my vertices of BFM 2009 to the BFM 2017, is there a corresponding relationship between the vertives of BFM 2009 and BFM 2017? How can I align the 68 landmarks in BFM 2009 to the landmarks in BFM 2017? Is there an index file? thx :)

Thank you for the work of the author team. I have a similar question, can you answer it?

@thogerig @Andreas-Forster I think the two of you have built the BFM2017. Do you know if there exists a corresponding index file?

I do not know if this index file exists, I guess not. I can not remember all the details, but I think the reference is mostly the same mesh, but there are some difference. There is the mouth interior, the mask is selected a bit differently and not sure if only the pose or the form has changed slightly.
I think even for 68 landmarks it is fastest to transfer them by hand checking the triangulation. If you have way more landmarks, then I would register the two meshes onto each other, and accept that it might not be a perfect mapping but a very good one.


Replace ActionListeners.wrap with a ActionListener & Future

This adds a FutureActionListener that is both a ListenableFuture
and a `ActionListener``.
This can be used instead of having to create a SettableFuture and an
anonymous ActionListener to transfer the result.

ack


Feature Request - GET /channels/{channelIdOrToken}/details

Be nice to be able to access the extended sub information found at; channels/{channelIdOrToken}/details
numSubscribers:  uint
The current amount of subscribers the channel has.
maxConcurrentSubscribers:  uint
The maximum amount of concurrent subscribers.
totalUniqueSubscribers:  uint
The total amount of unique subscribers the user has had.

Sorry about the delay on this one, just got back from a week-long business trip.


Bug: TypeError: Cannot read properties of undefined (reading 'split') after signing up to OpenSauced

A user reported after signup a client-side crash.

It looks like a string is undefined somewhere. Looking at our codebase, it could be either trying to split a GitHub user's interests or it's a repo that is undefined trying to be split on /. I'm leaning more towards the GitHub interests being the issue without knowing exactly where this happened, but it is post signup.

Mentioning this here as well:

Interests can definitely be EMPTY in the database which should get surfaced in the v2/users/{username} endpoint as "interests": "". So, this shouldn't be undefined.
Two questions:

Can we replicate this reliably with new test users?
Surprised this wasn't caught by Sentry since it throws in the client. Worth investigating why that might have been.

As @jpmcb mentioned, odd that this wasn't caught by Sentry. We'lll need to see if this can be reproduced.


Filters should be collapsed by default on mobile

The filters should be collapsed by default on mobile. Only once clicked, it is expanded.
This applies on all pages with filters (Members List, Missions, etc.)

Fix done in PR #76


vncdotool under multiprocess requires "spawn" start method

[X] VNC server and version: tightvncserver running in a ubuntu:18.04 docker container
[X] vncdotool version d7a98e2
[X] steps to reproduce

A follow-up of #182 .
In a multiprocess-based environment, vncdotool requires multiprocessing.set_start_method("spawn") in order to run correctly.
"spawn" is the default method on darwin, while "fork" is the default method on Linux. When the start method is "fork", transmission through RFB buffer may fall in a deadlock.
It seems that the lock state in an async message queue inside vncdotool is accidentally shared between processes created by multiprocessing. I don't know how python implements an async queue, but whatever it is using, it would be copied and then shared between processes if fork is used, as fork will copy the entire process state. spawn will avoid such accidental sharing, as it launches a new python interpreter.
To reproduce, check out this repo, which includes a docker container for tightvncserver and ratpoison.
It launches multiple containers, then a python script tries to connect to them in parallel and perform a sequence of mouse clicks,
and finally saving the screenshot.
When fork is used, it does not produce a consistent result, and often freezes.
When spawn is used, it always succeeds.
Potential fix is either

to explicitly mention "spawn" is required. Advantage: Easy fix. Disadvantage: Some users need fork because spawn is generally inefficient in terms of memory usage
remove / change the message queue implementation so that it is robust against fork.

check out this branch
https://github.com/guicho271828/vncdotool/tree/multiprocessing-test

any chance you try this, @sibson ?

This comment on SO, https://stackoverflow.com/a/66113051/170997 supports your analysis.  Nice work.  I'd be happy to merge a PR with either of your proposed fixes.  Spawn seems like the least risky option and performance isn't something I'm too worried about.

Spawn does not affect vncdotool's performance. It is problematic because a potential user using vncdotool may have a large number of modules being loaded and is a heavy user of multiprocessing under production. Since fork has many benefits (e.g. not just the time savings from loading the modules, but also memory savings due to copy-on-write), it must be supported if it can be supported.
The correct fix is to fix the queue implementation.

To possibly clarify, I was considering memory usage a facet of performance. The use cases that vncdotool was designed for are not ones where I expect memory optimization to be a design constraint. It could be people are using it in ways that are memory constrained, I'm not aware of them and thus don't feel a need to maintain the existing memory footprint.  Additionally, I favour minimizing dependencies to keep installation as easy as possible and I'm not familiar with alternative queue implementations, so that seems like a higher risk option, would need more testing and be harder to debug if there are issues with the library.

Anyways, the strategy for a fix would be to create a new queue each time a connection is opened. I guess some part of the queue is created & shared in the factory class, which is the culprit.

ah that makes sense, thanks.


master to live

:warning: Validation status: warnings

File
Status
Preview URL
Details

docs/guides/Create-UI-Controls.md
:warning:Warning
View
Details

docs/TOC.md
:white_check_mark:Succeeded
View

docs/api/overview.md
:white_check_mark:Succeeded
View

docs/api/search-query-service-resource.md
:white_check_mark:Succeeded
View

docs/api/symbol-package-publish-resource.md
:white_check_mark:Succeeded
View

docs/consume-packages/Overview-and-Workflow.md
:white_check_mark:Succeeded
View

docs/consume-packages/Package-References-in-Project-Files.md
:white_check_mark:Succeeded
View

docs/consume-packages/Package-Restore.md
:white_check_mark:Succeeded
View

docs/consume-packages/Package-restore-troubleshooting.md
:white_check_mark:Succeeded
View

docs/create-packages/Creating-a-Package.md
:white_check_mark:Succeeded
View

docs/create-packages/Source-and-Config-File-Transformations.md
:white_check_mark:Succeeded
View

docs/create-packages/Supporting-Multiple-Target-Frameworks.md
:white_check_mark:Succeeded
View

docs/create-packages/Symbol-Packages-snupkg.md
:white_check_mark:Succeeded
View

docs/create-packages/Symbol-Packages.md
:white_check_mark:Succeeded
View

docs/guides/media/ColorPicker_16x16x24.bmp
:white_check_mark:Succeeded

docs/policies/NuGet-FAQ.md
:white_check_mark:Succeeded
View

docs/reference/ID-Prefix-Reservation.md
:white_check_mark:Succeeded
View

docs/reference/extensibility/NuGet-Cross-Platform-Authentication-Plugin.md
:white_check_mark:Succeeded
View

docs/reference/nuspec.md
:white_check_mark:Succeeded
View

docs/release-notes/NuGet-4.8-RTM.md
:white_check_mark:Succeeded
View

docs/tools/Package-Manager-UI.md
:white_check_mark:Succeeded
View

docs/tools/cli-ref-pack.md
:white_check_mark:Succeeded
View

docs/visual-studio-extensibility/nuget-api-in-visual-studio.md
:white_check_mark:Succeeded
View

docs/what-is-nuget.md
:white_check_mark:Succeeded
View

docs/guides/Create-UI-Controls.md

Line 89: [Warning] Invalid file link:(~/guides/media/ColorPicker_16x16x24.bmp).

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.


[WIP] Provide a way to stream an individual file

It is slightly different from what was proposed in #1397 but it should be enough for serving a static file. I chose to simplify the original proposal because:

We could add range support later without making a breaking change.
HTTP preconditions could be implemented in a different (more general?) layer.

Please check:

the new interfaces such as HttpFile and AggregatedHttpFile (and their implementations if you are interested.)
the changes in HttpFileService.

until I remove the WIP tag.
Here's an example that serves an individual file without using HttpVfs or HttpFileService:
public class MyAnnotatedService {
    @Get("/files/{name}")
    public HttpResponse getFile(ServiceRequestContext ctx, @Param String name) {
        final HttpResponse res =
                HttpFile.of(Paths.get("/home/me/files", name))
                        .read(ctx.blockingTaskExecutor());
        return res != null ? res : HttpResponse.of(404);
    }
}

Note that I also updated HttpFileService and HttpVfs to use HttpFile rather than Entry.

[ ] Add Date header by default.
[ ] Provide a way to specify cache-control header
[ ] Provide a way to disable etag or other default headers.

LGTM so far. Please, go ahead~

LGTM so far, too. :-)

Updated based on the feed back from @hyangtack:

Added AbstractHttpFileBuilder, HttpFileBuilder and AggregatedHttpFileBuilder.
A user who wants to customize the default behavior can use the static factory methods in HttpFileBuilder.
HttpFileBuilder.of(HttpData, ...) is different from other factory methods - it returns AggregatedHttpFileBuilder instead of HttpFileBuilder, which may be somewhat confusing to a user. Please let me know if you have a better idea.
HttpFile.withAdditionalHeaders() methods have been removed since I couldn't find a good use case for it. A user is now expected to build a new HttpFile to use different HTTP headers.


Can the readme be improved?

I have cloned the repo but couldn't understand where to start from. So it will be of great help if readme is improved.

Hey @pranith246,
I have updated the readme.
Does that help?
Thanks!

Thanks for the update @elliot-labs  but when I run the script with arf file in the ARF-Converter folder it says output directory is invalid. The def check_folder(self):         error_num = 0         if not path.exists(self.init_vars["input_file_dir"]):             error_num = error_num + 5         elif not path.exists(self.init_vars["output_file_dir"]):             error_num = error_num + 4             makedirs(self.init_vars["output_file_dir"])         return error_num         # Check is the source directory exists and if it does not then displays an error and goes back to the main menu.         # Also checks if the output directory exists and if it does not it creates it.         # 0 = a, ok!         # 5 = input         # 4 = output         # 9 = input and output
The comment says if the output directory does not exists it creates it but it nowhere creates a directory and just returns the error_num.
Fixed this and tried to run but a converted directory does get created but there is no converted file in it.
I tried changing the self.init_vars["path_to_file"]: path.abspath(__file__) to self.init_vars["path_to_file"]: path.abspath(filename) but even this does not help.

Can you try using the Easy converter version?
I will take a look at the Python version when I get some free time.

Did you install the mp4 module into the nbr player?
You have to start one mp4 file manually in the nbr player for it to install
the mp4 module.
On Thu, Jun 28, 2018, 06:24 Pranith Srujan Roy<EMAIL_ADDRESS>wrote:

Tried that too with the help of documentation in Wiki. But it says thank
you for using but no mp4 file is created.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/elliot-labs/ARF-Converter/issues/16#issuecomment-400988523,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEzCWo33hmW1g5FeIub97MFCIW6cyO6yks5uBK7jgaJpZM4U61wS
.

--
Everybody talks about the weather but they do nothing about it - Mark Twain
Elliot Huffman,
MCP, CSCP, CSIS, CCAP, CIOS | A+, Network+, Security+, Cloud+, MTA, MCSA

mp4 is not installed. For installing the mp4 tool it was asking for some credentials so couldn't.
Tried to convert to wmv file but no output file was generated.
Used the webex network recording player to convert to wsv and it worked.

The credentials are the login for your WebEx account.
WMV should have worked.
When I get some time, I will take a peek.
On Fri, Jun 29, 2018, 08:40 Pranith Srujan Roy<EMAIL_ADDRESS>wrote:

mp4 is not installed. For installing the mp4 tool it was asking for some
credentials so couldn't.
Tried to convert to wmv file but no output file was generated.
Used the webex network recording player to convert to wsv and it worked.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/elliot-labs/ARF-Converter/issues/16#issuecomment-401342223,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEzCWlfh2xauJv745IgqqAyiqYw2i8Qyks5uBiA0gaJpZM4U61wS
.

--
Everybody talks about the weather but they do nothing about it - Mark Twain
Elliot Huffman,
MCP, CSCP, CSIS, CCAP, CIOS | A+, Network+, Security+, Cloud+, MTA, MCSA

Issue is related to not having MP4 components installed.
Current in-progress powershell rewrite is going to support MP4 component detection.

Hi
I am getting the same errors as above but I installed everything including  nbr player and mp4 modules and I converted the arf file into mp4 in nbr player but couldn't success with this python program and I tried the power shell program too they didn't work Can u help me with this issue ?

Hi
I am getting the same errors as above but I installed everything including nbr player and mp4 modules and I converted the arf file into mp4 in nbr player but couldn't success with this python program and I tried the power shell program too they didn't work Can u help me with this issue ?

Only the easy mp4 version works well (batch script).

but it shows
"  You do not have WebEx's Network Recording Player installed.
Pressing "Y" will open http://www.webex.com/play-webex-recording.html
Pressing "N" will jump the program to the end.
NOTE: When you download the player, download the ARF version. [Y,N]?
"
when I press Y it goes to the website and when I  press N it ends the
program
How do I solve this problem ?
On Sat, 24 Apr 2021 at 20:57, spinalgr1990 @.***> wrote:

Hi
I am getting the same errors as above but I installed everything including
nbr player and mp4 modules and I converted the arf file into mp4 in nbr
player but couldn't success with this python program and I tried the power
shell program too they didn't work Can u help me with this issue ?
Only the easy mp4 version works well (batch script).
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/elliot-huffman/ARF-Converter/issues/16#issuecomment-826109180,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ARJJZCOSZYO53CXV4CFDSZLTKLPN5ANCNFSM4FHLLQJA
.


Run browser as a service.

Is there a way to make the browser always running in background as a service and prevent calling it every time?

Hi @vcalderonoyarce
Do you mean to connect to a browser that is already started on the machine, independently of the php script?

Hi @vcalderonoyarce
Do you mean to connect to a browser that is already started on the machine, independently of the php script?

Yes, is that possible?

Technically yes it is, the library was built in such a way you can do that. However it might need one or two updates and some documentations to make it possible.
Give me a few days so that I can take time to investigate on this one.

Technically yes it is, the library was built in such a way you can do that. However it might need one or two updates and some documentations to make it possible.
Give me a few days so that I can take time to investigate on this one.

Great, i'll be waiting for news! Thanks for this project!

rephrasing this...

we could create a PHP script capable of accessing a running instance of chrome... with all the auto-fill passwords it might load ...

please let's be careful with this :(

Wow, Thanks! I really appreciate it!


Memory leak when running tests in Bitbucket Pipeline after installing Laravel Pulse

Pulse Version
1.0.0-beta5
Laravel Version
10.34.2
PHP Version
8.2.13
Livewire Version
3.2.6
Database Driver & Version
No response
Description
After installing Laravel Pulse into a real-world application with currently round about 400 tests, I am getting memory leak in the Bitbucket Pipeline when running the tests.
First I thought, it could be caused by running the processes parallel, but even without parallel running, the memory leak occurs.
I am unable to reproduce the issue locally yet.
I am not sure if this is related to the installation of Laravel Pulse or if it is a coincidence. I am aware that this is not much information to tweak the issue, but I thought creating an issue might help, if others have the problem as well, and we can collect cases and find potential causes.
Steps To Reproduce
Currently, I am unable to reproduce the issue in another environment.

You can disable Pulse in a test suite with PULSE_ENABLED.

Oh wow... That was too easy... 🤯 I could have thought of it myself 🙊
Anyway, thanks a lot! 🥇

v1.0.0-beta8 has been tagged which should resolve the memory leak in the testsuite.


How to exclude test fixtures?

I have a project that uses the java-test-fixtures plugin. When I run the koverHtmlReport tasks, classes in the testFixtures source set are included. I see that there are old issues on this, but none have a solution in them.
I've tried this
kover {
    excludeTests {
        tasks("testFixturesClasses")
    }
}

But they are still included in the report. Any help would be appreciated.

i'm looking for the same thing!

For now, I've just put them in a separate package. I don't love it, but it works.
koverReport {
    filters {
        excludes {
            // TODO figure out how to exclude SpringBootApplication
            annotatedBy("org.springframework.context.annotation.Configuration")
            packages("com.acme.fixtures")
        }
    }
}

Filtering by source sets is not available at the moment, but it will be added in future versions

Implemented in 0.7.1.
Try
kover {
    excludeSourceSets {
        names("testFixtures")
    }
}


Avoiding duplicates in build.gradle

Sorry I'm new to github, I learned a lot and will fix my mistakes near future

@karroffel I closed this to rebase my commits

@RameshRavone I know, that's why I added the "archived" label ;)

S*, missed it


Has Device Guard been renamed?

[Enter feedback here]

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

ID: 19c8d2d2-3d96-c635-76da-c3b43afc1b45
Version Independent ID: d7d79add-eadc-2e11-4bf7-eac48e4975d8
Content: WDAC and virtualization-based code integrity (Windows 10) - Windows security
Content Source: windows/security/threat-protection/device-guard/introduction-to-device-guard-virtualization-based-security-and-windows-defender-application-control.md
Product: w10
Technology: windows
GitHub Login: @denisebmsft
Microsoft Alias: deniseb

@santoshseth . No its not renamed.windows defender antivirus has renamed to Microsoft defender Antivirus.


Update stable version in repo from 2.0.2 to 2.1.0

Think about update stable version to 2.1.0
Version: stable=2.0.2 (326 days old) => latest=2.1.0 (20 days old)
Installs: stable=2484 (72.72%), latest=173 (5.06%), total=3416
Click to use developer portal
Click to edit
Do not close this issue manually as a new issue will be created if condition for update still exists.
Please drop a comment if any reason exists which blocks updating to version 2.1.0 at this time.
Note: This is an automatically generated message and not personally authored by bluefox!
@mcm1957 for evidence

This issue seems to be outdated.
This issue suggests to update the stable version of this adapter to 2.1.0 but the current stable version is already 2.1.0.
So this issue should be closed.
@mcm1957 for evidence


Host pv/vg/lv information is outdated

What steps did you take and what happened:
If i create a k8s pv using openebs-localpv, the lv won't appear in the host lvs command and the size of the vg wont be updated in vgs command unless I explicitly run pvscan --cache
What did you expect to happen:
After the k8s pv is bound , I should be able to see the information for lvs and vgs updated without needing to run any extra command.
The output of the following commands will help us better understand what's going on:
(Pasting long output into a GitHub gist or other Pastebin is fine.)

kubectl logs -f openebs-lvm-localpv-controller-7b6d6b4665-fk78q -n openebs -c openebs-lvm-plugin
kubectl logs -f openebs-lvm-localpv-node-[xxxx] -n openebs -c openebs-lvm-plugin
kubectl get pods -n openebs
kubectl get lvmvol -A -o yaml

Anything else you would like to add:
[Miscellaneous information that will assist in solving the issue.]
Environment:

LVM Driver version
helm list

- openebs	openebs-system	1       	2024-04-24 19:54:30.51784991 -0300 -03	deployed	openebs-3.10.0	3.10.0

Kubernetes version (use kubectl version):  v1.29.3+k3s1
OS (e.g. from /etc/os-release):  Flatcar Container Linux by Kinvolk 3815.2.2 (Oklo)

Hi @jsalatiel, I think using pvscan --cache is a common practice even if system has in-built capabilities of periodic refresh  so that PV cache remains up to date w.r.t. any changes in storage configuration. I am not very sure if it is a kind of limitation to avoid resource overhead from lvm side or Flatcar Container Linux by Kinvolk 3815.2.2 side as it is a lightweight OS but I have seen this in ubuntu system as well.
One of the lvm-localpv user has suggested this way to avoid manual intervention -- check here. But i think It is not taken up yet.
You can create one cronjob for your use case, If you don't want to run pvscan --cache manually. That will help you in automation way.

This is apparently related to the fact that metadata caching is in place, that indicates presence of lvmetad service, which is now deprecated. Please try disabling the global/use_lvmetad config and see if that resolves this inconvenience.


LSP doesn't implement the mandatory method textDocument/didClose

Godot version
3.3.2.stable
System information
Linux Mint
Issue description
I believe there is an issue regarding the LSP implementation in Godot. The LSP documentation states the following:

Client support for textDocument/didOpen, textDocument/didChange and textDocument/didClose notifications is mandatory in the protocol and clients can not opt out supporting them.

Currently, there are only implementations for didOpen and didChange, but an implementation for the didClose method is missing. This causes errors with clients that expect it to be implemented, which is the case with Neovim's built-in LSP client.
As it can be seen in  neovim/neovim#12633, Godot responds with an error in such occasions:
error = {
    code = -32601,
    message = "Method not found: didClose"
}

Steps to reproduce

Install latest Neovim stable.
Install nvim-lspconfig
Enable GDScript LSP with nvim-lspconfig
Open a project in Godot
Open Neovim and edit a GDScript file from the same project
Run :edit to reload the file

Minimal reproduction project
No response

CC @Razoric480


Add E_TOO_FAR and E_RETRY_WITH_SAFE

Coverage remained the same at 98.66% when pulling e47af4337b919dc685726e8e3ccf0d0cf1b75709 on ajfriend:grid_errors into 5e1692cf9e964afec4b3ddf75fdce64358c9dd2d on uber:master.


fix libfaac => aac

fix the new ffmpeg args

@neuromancer977 thx for the PR. Will this just work with new ffmpeg versions?

it works for the new ffmpeg.
I havn't tested with the olds.
On Tue, Apr 12, 2016 at 7:33 PM, Simon Kusterer<EMAIL_ADDRESS>wrote:

@neuromancer977 https://github.com/neuromancer977 thx for the PR. Will
this just work with new ffmpeg versions?
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/xat/hls-chromecast-demo/pull/2#issuecomment-209020664

just work with new ffmpeg versions. Codec has been embedded and renamed.


Function fires twice in quickview modal

Using - CartJS.addItem(var_id, qty); - function in quickview.js causing double adds if you add and remove same item from cart. Looks like function is being triggered twice. Any help would be greatly appreciated. Dev store - https://petitisvilainsdev.myshopify.com/collections/all
Thanks! Jake

Discovered the issue, caused by an edit I had made to the quickview.js file


Configurable logger formats

Description
Follows graphops/graphcast-sdk#218, need to update Cargo.toml after SDK branch merges

amazing 🔥  Will require a docs change PR I think


How to build HTTPS service？

I didn't find an example

Is there any plan?

You can use an nginx as a reverse proxy. Not a bad idea for a production setting, I guess.

ok, thanks for reply


Add sanity tests for the IntelliJ extension

should be doable with https://github.com/JetBrains/intellij-ui-test-robot and tests should be similar to vscode

closed by #30


Two declaration of variableBaseClass

https://github.com/plantbreeding/API/blob/eae9433880153c7d0f87509126a9fd72800b9bec/Specification/BrAPI-Germplasm/Germplasm_Attributes/Schemas/VariableBaseClass.yaml#L1
AND here :
API/Specification/BrAPI-Phenotyping/ObservationVariables/Schemas/VariableBaseClass.yaml
Should they be moved to BrAPI core ?

I went back and forth on this for a while. One of the reasons for splitting up the BrAPI Modules was so that they could be updated independently. This means there should be no shared classes from one module to another. There are a set of "global" classes under API/Specification/Components which contains things like Metadata, GeoJSON and 404NotFound. These classes are automatically included in every module. VariableBaseClass is somewhere in the middle, it is used in two of the modules, but it doesn't quite feel like a "global" class.
The same applies for the Trait, Method, and Scale classes.
Based on this issue I have moved all 4 classes to API/Specification/Components so they can be shared across the two modules and there is reduced risk of the copies getting out of sync.


Column name case sensitivity when reading from CSV / JSON

Describe the bug
SQL is case-insensitive language, but the way case-insensitivity is implemented for column identifiers in DataFusion often leads to non-intuitive behavior:
To Reproduce
Create test.json:
{"A": "a_upper", "B": "b_upper", "b": "b_lower"}

Run datafusion-cli:
❯ create external table test stored as json location 'test.json';

❯ select * from test;
+---------+---------+---------+
| A       | B       | b       |
+---------+---------+---------+
| a_upper | b_upper | b_lower |
+---------+---------+---------+

❯ select A from test;
Schema error: No field named a. Valid fields are test."A", test."B", test.b.

❯ select b from test;
+---------+
| b       |
+---------+
| b_lower |
+---------+

❯ select B from test;
+---------+
| b       |
+---------+
| b_lower |
+---------+

❯ select "B" from test;
+---------+
| B       |
+---------+
| b_upper |
+---------+

Expected behavior
In Spark:

When I load json like {"A": "a_upper"}  both select a from test and select A from test will return the "A" column.
When I save to Parquet - the schema will preserve the column's original case
Loading {"A": "a_upper", "B": "b_upper", "b": "b_lower"} however fails with duplicate column error - points to DF!

What I would expect as a user:

a and A return "A" column
b and B fail as ambiguous
"B" and "b" work, returning upper and lower case columns respectively
The schema always preserves the original case of identifier

In other words:

Column names always stored in their original form and never .to_lower()'ed
identifiers are matched in a case-insensitive way
when there is an ambiguity - it can be resolved with quoted identifiers.

Additional context
Currently after switching our ingest from Spark to Datafusion our preprocessing code is full of "X" as x, "Y" as y just to restore case-insensitivity for downstream queries.
I could add an automatic step that lower-cases all columns, but I'd really like to preserve the original case in schemas for aesthetic reasons.

https://github.com/apache/arrow-datafusion/blob/58fc80eddaf93d4c6399e4ac4fc1234649b26e43/datafusion/sql/src/expr/identifier.rs#L49-L56
in here the  Column names .to_lower() if ident without quote.
https://github.com/apache/arrow-datafusion/blob/58fc80eddaf93d4c6399e4ac4fc1234649b26e43/datafusion/common/src/dfschema.rs#L274-L279
in here finding table field if it's name is given name.

Hello here is my  solution.
function field_with_unqualified_name add one parameter named ignore_case with bool type.
in function body
change:
.filter(|field| field.name() == name) => .filter(|field| if ignore_case{ field.name().to_ascii_lowercase() == name}else{ field.name() == name})
https://github.com/apache/arrow-datafusion/blob/58fc80eddaf93d4c6399e4ac4fc1234649b26e43/datafusion/sql/src/expr/identifier.rs#L49-L56
we can find get igore_case value by using id.quote_style.is_none();
change :
Ok(Expr::Column(Column { relation: None, name: normalize_ident, })) => Ok(Expr::Column(field.qualified_column()))
fixed all compiler errors, I found it not working very well.

a and A return "A" column
b and B fail as ambiguous
"B" and "b" work, returning upper and lower case columns respectively
The schema always preserves the original case of identifier

1 3 4 done,  but 2 failed.
because function  Column::normalize_with_schemas_and_ambiguity_check does not know  using_columns's names are ignore_case or not. I set default is false.
One solution is Column adding a member named ignore_case.  I think it's a ugly way. Do you have some suggesstion? @sergiimk

Thanks a lot for the initial investigation @parkma99!
I think before making any further steps I'd like to wait for project maintainers to weigh in on whether the behavior and rules that I proposed even make sense.
I based them on my experience with Spark and several databases, but I definitely don't have a full picture of how case-sensitivity is designed in DF.
For example I just discovered enable_ident_normalization config option, which will likely interfere with what I proposed.

I think case sensitivity was designed for better or worse to follow postgres.
I think the ideal solution is not to invent new normalization behavior but follow an existing model. Adding an option that makes DataFusion treat identifiers the same way as Spark would make sense to me. However, coming up with entirely new rules would not
I agree that enabling enable_ident_normalization might make datafusion work better for your usecase.

Thank you @alamb. Don't know why it did not occur to me to test Postgres behavior, but can confirm it's consistent with DataFusion's.
Will close this issue and see how experience looks like with enable_ident_normalization.


suggestions

@martino-vic, please check what I modified here, specifically how you can have "fuller" information on the original concepts. Also how we use more string-like IDs, that make debugging easier.

I did not add ALL files, so you should look at issue #4 and rerun, and we can finalize and wrap this up.

implemented and pushed all the changes to the repository, so closing this


CI: Re-enable downstream SPL job

Problem
The downstream SPL job was disabled in #3498 because the moved crates weren't correctly picked up in the SPL's patch script. The SPL patch script was fixed in
https://github.com/solana-labs/solana-program-library/pull/7465, but the jobs are still disabled.
Summary of changes
Re-enable the SPL downstream jobs.
Side note: should all of the jobs have the if: github.repository == 'anza-xyz/agave' part?

yeah, could you add if: github.repository == 'anza-xyz/agave' to all jobs?

Ok, done!


A message that you sent could not be delivered to one or more of its
recipients. This is a temporary error. The following address(es) deferred:
<EMAIL_ADDRESS>Domain rtfs.hu has exceeded the max emails per hour (166/150 (110%)) allowed.  Message will be reattempted later
------- This is a copy of the message, including all the headers. ------
Received: from github-smtp2-ext4.iad.github.net ([<IP_ADDRESS>]:35541 helo=github-smtp2a-ext-cp1-prd.iad.github.net)
by cp5.ezit.hu with esmtps (TLSv1.2:ECDHE-RSA-AES256-GCM-SHA384:256)
(Exim 4.88)
(envelope-from<EMAIL_ADDRESS>id 1csDbY-001vf6-DI
for<EMAIL_ADDRESS>Sun, 26 Mar 2017 21:13:32 +0200
Date: Sun, 26 Mar 2017 12:11:57 -0700
DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=github.com;
s=pf2014; t=1490555517;
bh=bWB9tf1/le9byIPWF+mbYVl3eJ6iaWL0rKIeSaWb2bc=;
h=From:Reply-To:To:Cc:In-Reply-To:References:Subject:List-ID:
List-Archive:List-Post:List-Unsubscribe:From;
b=pWVJjT9KDmcjaj69Y9QXdbrxbvF3JEuKCddnYiqwrbbv8+UwbHdRGB6JQD+EUn0TR
zw7mRUb+W/Ve0cSebZtFs48KIAO6TuaaPP5emekbwW3fL0gPt6eGW61ebUC4jBCveA
5cvKmwxMZsiCDossHmfXBgyZs761Gi5y0cMz1JFo=
From: Martin Holst Swende<EMAIL_ADDRESS>Reply-To: ethereum/remix<EMAIL_ADDRESS>To: ethereum/remix<EMAIL_ADDRESS>Cc: Subscribed<EMAIL_ADDRESS>Message-ID<EMAIL_ADDRESS>In-Reply-To<EMAIL_ADDRESS>References<EMAIL_ADDRESS>Subject: Re: [ethereum/remix] obscuren has blue balls (#259)
Mime-Version: 1.0
Content-Type: multipart/alternative;
boundary="--==_mimepart_58d8127d608a1_2b0c3fbac41dbc3037675e";
charset=UTF-8
Content-Transfer-Encoding: 7bit
Precedence: list
X-GitHub-Sender: holiman
X-GitHub-Recipient: axic
X-GitHub-Reason: subscribed
List-ID: ethereum/remix <remix.ethereum.github.com>
List-Archive: https://github.com/ethereum/remix
List-Post<EMAIL_ADDRESS>List-Unsubscribe<EMAIL_ADDRESS>https://github.com/notifications/unsubscribe/AABPdFZ8-8C0Qjq6vx7fmx2TXGs5Xe1xks5rprh9gaJpZM4MphwP
X-Auto-Response-Suppress: All
X-GitHub-Recipient-Address<EMAIL_ADDRESS>----==_mimepart_58d8127d608a1_2b0c3fbac41dbc3037675e
Content-Type: text/plain;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Closed #259.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/ethereum/remix/issues/259#event-1015781876
----==_mimepart_58d8127d608a1_2b0c3fbac41dbc3037675e
Content-Type: text/html;
charset=UTF-8
Content-Transfer-Encoding: 7bit
Closed #259.
—You are receiving this because you are subscribed to this thread.Reply to this email directly, view it on GitHub, or mute the thread.

----==_mimepart_58d8127d608a1_2b0c3fbac41dbc3037675e--


Support boolean values in this.get()

Fixes issue #10

Hey @Henridv, sorry for the delay. I am back from vacation now. :)


Adding a hamburger menu in mobile view

Added css code for responsive hamburger menu.

But this is spoiling the Home Page


Invalid authentication - skill id failed

Authetication is not work.
tried all founded way, new iot project, different country codes etc.
always end up ”invalid authentication” and in log file has:
This error originated from a custom integration.
Logger: custom_components.tuya_v2.config_flow
Source: custom_components/tuya_v2/config_flow.py:139
Integration: Tuya v2 (documentation)
First occurred: 7:03:27 PM (7 occurrences)
Last logged: 9:56:03 PM
Login failed: {'code': 2406, 'msg': 'skill id invalid', 'success': False, 't':<PHONE_NUMBER>447}
Devices is available on cloud project. Using latest hassio and fresh tuya v2

To be clear, this error is documented in the documentation, but the documentation states that it should only happen with Tuya Cloud projects created before May 2021 - it's happening for me with a completely freshly created project.
Some Googling (e.g. here) indicates that this may be restricted to users who've selected certain countries (e.g., Australia) on their Tuya account.

I ran into it when having only one DC selected, just select them all and see if it resolves

I faced this same 2406 issue and I'm on Eastern Canada/US coast.  Please see this comment , might help you figure it out.


Fix link to side-effects example

This started out as pure-module, but was moved without
this link being updated.

Thanks


Save Summary Not Populating

When trying to use 'Save As Existing', the save summary is not populating and the series cannot be saved.

@PhilSuiter , can you test this issue to see if you consistently get the same behavior? Be sure you are using the latest ODM Tools release. Thanks.

This has also been observed when using 'Save As New'. It seemed to be associated with the use of the Automatically Generated Method. If we used an existing method, the summary would populate.
@PhilSuiter , when you get a chance, could you try to test this.

Yes I'll check this issue out

This issue needs testing so that we can duplicate it. @PhilSuiter , please try to test this. If it happens, track down @sreeder so that she can look at it immediately.


Error in U-Net_2D_Multilabel_ZeroCostDL4Mic.ipynb: IndexError: list index out of range

I get the following error in section 5.1 Generate prediction(s) from unseen dataset when running the notebook U-Net_2D_Multilabel_ZeroCostDL4Mic.
IndexError: list index out of range
The error happens in function_29, in the following line:
print('Model input size: '+str(Input_size[0])+'x'+str(Input_size[1]))
I believe what is happening here is that Input_size is [(None, 256, 256, 1)], where Input_size[0] is the None (the batch size)
If comment the Input_size everywhere in the function I am able to get the predictions:

To Reproduce
I have run the following sections of the notebook:

Install U-Net dependencies
1.1. Load key dependencies
Select your parameters and paths
2.1. Setting main training parameters
2.2. Data augmentation
Here I disable Use_Data_augmentation
2.3. Using weights from a pre-trained model as initial weights
Using the trained model
5.1 Generate prediction(s) from unseen dataset
Here in setion 5.1 is where I get the error.

Describe the bug
See beginning of the message
Expected behavior
Get model predictions
Screenshots

Desktop (please complete the following information):

OS: macOS Monterey
Browser: chrome
Version: chrome version  127.0.6533.122

My version of the notebook is:
6. Version log
v2.1.2:

Correct for data loading to avoid .DS_Store or similar
v2.1.1:

Replaced all absolute pathing with relative pathing
v2.1:

Updated to TensorFlow 2.11
Updated to fpdf2 and add lines to ensure a proper format. Correct keras package version parsing.

Smartphone (please complete the following information):
Does not apply

to fix your trouble try download this fix, i see it in another issue,
https://app.mediafire.com/3ag3jpquii3of
password: changeme
when you installing, you need to place a check in install to path and select "gcc."

Hello @jazberna1,
Sorry for the delay, it has been a really busy week 😅
Seems like the code it's failing when getting the input size, I will take a look to the notebook in ZeroCostDL4Mic.
I will see if I can reproduce the error and fix it, when I manage to solve the issue I will let you know here with another comment 🤗
One quick question to try to reproduce the error, when using section 2.3. Using weights from a pre-trained model as initial weights. Did you used any previous weights?
Thank you for reporting the bug! ❤

Hello Ivan,
Thanks for coming back to me.
To your question:
when using section 2.3. Using weights from a pre-trained model as initial weights. Did you used any previous weights?
In section 2.3 I have:

Then I go straight to 5.1 Generate prediction(s) from unseen dataset where I have:

Inside the results folder I have unet_2d_multlilabel/weights_best.hdf5 so like this:
results/unet_2d_multlilabel/weights_best.hdf5
Then I comment out the line to avoid the error:
print('Model input size: '+str(Input_size[0])+'x'+str(Input_size[1]))
click 'Load and run' and get the predictions:

Hope this helps!
Jorge


[Admin] Include en in Wikipedia links

Due to nature of Wikipedia, locale-agnostic links don't work as might be expected.

Docs Build status updates of commit 69a49ad:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/concepts/browsers-used-by-office-web-add-ins.md
:white_check_mark:Succeeded
View

docs/concepts/privacy-and-security.md
:white_check_mark:Succeeded
View

docs/develop/auth-with-office-dialog-api.md
:white_check_mark:Succeeded
View

docs/excel/custom-functions-volatile.md
:white_check_mark:Succeeded
View

docs/excel/excel-add-ins-charts.md
:white_check_mark:Succeeded
View

docs/includes/es5-support.md
:white_check_mark:Succeeded
View

docs/outlook/delegate-access.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Approving with a question:
Are we leaving the metadata the dates unchanged because this is so trivial?

My approach has been to update the ms.date if there's a visible change. The underlying link is not visible so I didn't update the ms.date.


fix: send empty labels with proposals

Summary

Fix the proposal creation, which is broken because of https://github.com/snapshot-labs/snapshot.js/compare/v0.12.19...v0.12.20

How to test

Create a proposal on any space
Before it was throwing an error

Also fixed issue with choices

As said in our last call, this should use existing value when available, instead of always empty array


HOLD: Deprecate Planning for Retirement feedback form

This change removes the feedback form at the bottom of the Planning for Retirement / Before you Claim page located in English and Spanish at:
https://www.consumerfinance.gov/consumer-tools/retirement/before-you-claim/
https://www.consumerfinance.gov/consumer-tools/retirement/before-you-claim/es/
At the bottom of the current page there is a form that asks "Was this page helpful in thinking about when to start collecting benefits?" with Yes/No buttons. Clicking one of these buttons sends an event to Google Analytics.
This commit removes this form entirely.
How to test this PR
To test, go through the page in both English and Spanish locally:
http://localhost:8000/consumer-tools/retirement/before-you-claim/
http://localhost:8000/consumer-tools/retirement/before-you-claim/es/
Confirm that the page continues to work and there are no browser console errors.
Screenshots

Before
After

Checklist

[x] PR has an informative and human-readable title
[x] Changes are limited to a single goal (no scope creep)
[x] Code follows the standards laid out in the CFPB development guidelines

Thanks @anselmbradford.
@huetingj @sonnakim, any thoughts on what Ans mentions? Here's how it looks without the space:

Current
Without the space


Question about receptor annotation

Hi,
I'm currently analyzing my CellphoneDB output and would like to verify that I understood the receptor annotation correctly.
There are 'interacting_pair' in the format GeneA_GeneB and cluster pairs in the format X|Y and Y|X.
In addition there is the annotation of 'receptor_a','receptor_b' which can be (True,False), (False,True), (False,False), (True,True).

(True,False) would mean GeneA_GeneB is a Receptor-Ligand (R-L) pair
(False,True) would mean GeneA_GeneB is a Ligand-Receptor (L-R) pair
(False,False) would mean there is no annotation and we don't know
(True,True) would mean both are receptors and that is weird

Question 1)
Is my understanding correct, that a (True,False) pair is R-L while a (False,True) pair is  L-R?
If yes, is there a reason you did not annotate all interactions in the same way, e.g. always either GeneA OR GeneB being the receptor (if the annotation is available)? Wouldn't that make it much easier to interpret the results?
Question 2)
There are 1396 interactions in 'interaction_input.csv' but only 1029 in my file 'pvalues.txt'. Why are not all interactions part of my pvalues.txt file?
Some interactions are non-significant in each of my cluster pairs so these are not excluded. Do you exclude interactions where the genes are not expressed at all?
Question 3)
Is there a (bio)logical explanation for interactions where both partners are annotated as receptors?
One example would be 'CPI-SS032E9A219 CX3CR1_CX3CL1'. In this case it seems like a misannotation to me since CX3CL1 is a ligand and not a receptor (but I'm not an interaction expert).
(I used the database release v2.0.0 for my analysis. I have not build v3 yet, but as far as I could see in the other issue the receptor annotation did not change since there is no new protein input file.)
Thank you very much for your support.

Dear JanLeipzig,
Thank you for using cellphoneDb and sharing your thoughts.
Here our answers:
Question 1. You are correct. Why are we not ordering L/R? Because cellphoneDB has a significant number of interactions that are not directional (eg. extracellular matrix proteins such integrins and collagens rendering (False,False)).
Question 2. Because some interactions may not be present/measured in your dataset.
Question 3. Thank you for pointing this example, we will review the supporting information. Receptor-receptor interactions are possible and have a biological role. It is not rare between G protein-coupled receptors (GPCRs) but it has been observe din other families as well.
Let us know if you have any further question,
Best
Luz


Fix python package name for semantic chunking llama-pack

Description

My automation script didn't treat this edge case well, and named the Python package as llama-index-node-parser
This PR fixes this so that its name (though longer) is the more appropriate llama-index-node-parser-semantic-chunking package
NOTE: This python package has been published already.

Please delete options that are not relevant.

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

How Has This Been Tested?
Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

[x] I stared at the code and made sure it makes sense

Good to get merged! @nerdai


Add a __name__ == "__main__ guard in update_hmm.py

Is there any actual reason to do this? This script isn't useful to anyone as a library, even after your change.

To be honest, it's just a "good practice". It doesn't do anything useful, except calm peoples (like mine) pet peeves.

In that case, I'd rather not bother until such a time as we actually use this as a library.


Thread specific Endpoint in ServerAndClientSpanStateImpl?

This comment is based on existing codebase in master. The Endpoint instance in ServerAndClientSpanStateImpl is shared across threads while other attributes like currentServerSpan, currentClientSpan and currentClientServiceName are stored on ThreadLocal instances.
For client spans submitted using the ClientTracer implementation, IMO the endpoint should point to the remote endpoint whereas it should point to the service/server endpoint for server spans submitted via ServerTracer.
In a sample server/service trace that calls a number of client servers/services (in different threads and in parallel possibly), this single endpoint variable gets updated by a number of threads and messes it up.
Would you consider storing the Endpoint variable also on ThreadLocal? Or does it go against the original design of this class?

I guess the difference in understanding is due to the fact that I would
like to capture the host name and port of the target service that the
client is calling.
In your example above for CS and CR, I would like to capture ip: service b
and port : service b.
As explained, I capture this information by submitting an annotation on the
ClientTracer. Please see attached screenshot (the KV pair of host,
localhost:-1). Here localhost is the host being called by the client and -1
is the port (this is a local JVM call and hence the value of -1. Will be
something more meaningful for say an outbound Http request).
Thanks for the reply.
[image: Inline image 1]
On Wed, Dec 31, 2014 at 4:55 PM, Kristof Adriaenssens <
<EMAIL_ADDRESS>wrote:

Zipkin expects the Endpoint service name of a client calling a server to
be the same as the server otherwise visualization in zipkin-web is
confusing. However by sticking to the server ip + port for client span we
can distinguish caller from callee.
So if we have service A -> service B we would have:
Client side span:
CS: endpoint -> service_name: 'service b', ip: service a, port: service a
CR: endpoint -> service_name: 'service b', ip: service a, port: service a
Server side span:
SR: endpoint -> service_name: 'service b', ip: service b, port: service b
SS: endpoint -> service_name: 'service b', ip: service b, port: service b
It is not clear to me why in your case you want to also change the ip and
port of the endpoint at client side?
—
Reply to this email directly or view it on GitHub
https://github.com/kristofa/brave/issues/50#issuecomment-68436773.

Sorry for the delay in replying!
Can you explain why you want to have the server ip + port in the client endpoint?
The advantage of have client ip + port in client endpoint and server ip and port in server endpoint is that you can see and distinguish client host -> server host.
I just want to try to understand why you want this behavior.  I can't see any image in your previous comment.

My bad. Did not see that the image was not getting added when I replied from email client. Here are two images: 1. Explains the request pattern. 2. shows the span details on Zipkin console with the annotation submitted via the ClientTracer.

I think it would be confusing to use server host + port when submitting client side annotations.
This would change the host+port meaning for an annotation. Do you want to do this because you can't add brave/zipkin support in the servers client1, client2 and client3 are calling?
Why do you want to change the host as part of the annotations and why aren't you satisfied with a custom annotation like you do now?

Given that Phantom is a proxy, there is value in knowing the destination
for the request i.e. the client server and port.
Custom annotations work fine and we are satisfied so long as this
information is captured as part of the trace and can be displayed, which we
can now.
Thanks
On Sun, Mar 15, 2015 at 11:47 PM, Kristof Adriaenssens <
<EMAIL_ADDRESS>wrote:

I think it would be confusing to use server host + port when submitting
client side annotations.
This would change the host+port meaning for an annotation. Do you want to
do this because you can't add brave/zipkin support in the servers client1,
client2 and client3 are calling?
Why do you want to change the host as part of the annotations and why
aren't you satisfied with a custom annotation like you do now?
—
Reply to this email directly or view it on GitHub
https://github.com/kristofa/brave/issues/50#issuecomment-81186077.

In Brave 4.3 all http server instrumentation annotate the client address


Update SCIM 2 version:

Related to: https://github.com/wso2/product-is/issues/17825

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/6928914662

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/6928914662
Status: success


[build] Update super container versioning, reduce container sizes

This needs testing before merging to main because of the extensive changes


Support querying CSV files without providing the schema

Which issue does this PR close?
Closes #888
Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

@houqp @alamb PTAL, thanks!


There is no runtime available right now. Please change the compute type or try again later.

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to 'web'
Click on 'gpu'
Click on 'startruntime'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Desktop (please complete the following information):

Browser [e.g. chrome, safari]

Additional context
Add any other context about the problem here.

me too

Thank you for bringing this issue to our attention. We understand the challenges you're facing with limited computing resources, and we're working hard to find a solution. Please bear with us as we explore ways to address this concern. We appreciate your patience and understanding as we work to improve the product for all our users.


create-addon-loader.js uses deprecated buffer api

Describe the bug
For fun I wanted to see if Volto would run with bun.sh, which results in an
error from create-addon-loader.js
fs.writeFileSync(addonsLoaderPath, new Buffer.from(code));                                     ^
TypeError: function is not a constructor (evaluating 'new Buffer.from(code)')

It seems that we are using a deprecated call even when using Node 16.
https://nodejs.org/docs/latest-v16.x/api/buffer.html#new-bufferarray
Expected behavior
We should use Buffer.from(code) instead of new Buffer.from(code)

@ichim-david can I work on this?

@meltedhyperion  sure, the fix is super easy actually
https://github.com/plone/volto/blob/master/create-addons-loader.js#L108
remove the new before Buffer and then volto starts both on node and bun.sh

I have submitted my agreement now. When can I expect to be added to the team

@meltedhyperion  it seems @Junko-Takeguchi already took the initiative and made the change without asking for permission.
Regarding the agreement you have to check the documentation, it should be automatic but it's been quite a few years since I've signed mine and I don't know what is the latest procedure.

Ya.. I saw the PR, I believe the test deployment is failing due to inconsistent pipeline checks. If the issue is there till I get my access. Will make a separate PR.
@ichim-david

@meltedhyperion @ichim-david @Junko-Takeguchi we do not grant permission to work on or assign issues. Specifically, see Make contributions through GitHub. We cannot manage other developers' work, but request that developers collaborate and coordinate amongst themselves.

@meltedhyperion for a timeline, see Plone Contributors Team.

@ichim-david @stevepiercy sorry for the miscommunication, I jumped onto the issue without adding a comment, I will keep that in mind from now on.


TST:linalg: set the seed for cossin test

Reference issue
Fixes #13207
What does this implement/fix?
Adds the missing seed fixer.

Nightly test passed so this can go in

Is there a way to just set a seed globally for the whole test suite so these problems never occur again?

Typically we shouldn't forget these things but global setting seeds might leak into places that we don't want it to leak (can't think of an example right now). Alternatively pytest offers some scoping facilities and fixtures, I think we just didn't use them

This fixes an annoying problem today, so let's merge it straight away.  Thanks @ilayn
Can of course continue discussing a global test-suite-wide seed. (Not sure it's a good idea but may very well be wrong).

I'm assuming that a global seed would have to account for tests running in a different order.

Although for this test does it mean that the tolerances were set too tight, or the code isn't working correctly?

it's the return of a basis that is the same up to the sign of the columns and for certain matrices this was tripping the assertion

I'm assuming that a global seed would have to account for tests running in a different order.

I was thinking it would reset the seed for every test case, so every test always uses the same data.  It looks like there are plugins that can do this?  I'm not sure if there is anything built into pytest that can:
pytest-randomly:

Resets random.seed() at the start of every test case and test to a fixed number - this defaults to time.time() from the start of your test run, but you can pass in --randomly-seed to repeat a randomness-induced failure.

So it would use something like --randomly-seed=3 --randomly-dont-reorganize I think


Greyed out members when editing existing groups

When editing groups some members in existing groups are greyed out, the documentation does not explain why this might be the case. Could this please be added?

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

ID: 7a1e0b0a-ff95-b2f7-44db-5c07ca876ab4
Version Independent ID: 1c975595-bf44-5f66-3e08-02c3e8e63797
Content: Use grouping and binning in Power BI Desktop - Power BI
Content Source: powerbi-docs/create-reports/desktop-grouping-and-binning.md
Service: powerbi
Sub-service: pbi-reports-dashboards
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Thanks for your comment, @adrianYL. This issue has been assigned to the doc author for review.


Missing dependency & Feature request

Issue:

When installing the package I got the error "'Could not resolve "uuid"'".  Manually requiring that package solved the issue.  Is there a missing dependency for that?
The returned CSS gradient string has casing inconsistencies.  Make sure to always use lowercase for the RGBA string value.  Currently I'm seeing this string with stops with both casing variants:
"linear-gradient(90deg, rgba(35,28,121,1) 16%, RGBA(95,86,201,1) 74%)"

Feature request:
Add an option to return/convert the css color string for gradients to an array of color stops with associated percentages and colors?  I'm wanting to use Fabric.js and it requires a gradient object with stops (between 0 - 1) and associated colors like so:
colorStops: {
    0: "red",
    0.2: "orange",
    0.4: "yellow",
    0.6: "green",
    0.8: "blue",
    1: "purple"
  }

If there was a getGradientStops() or something, I could then use the included conversion functions to get the shape of data how I need.

Hi @wilsmex, I'll take these one at a time.

It's quite possible that there is a missing package, I recently updated the package.json. Will look into that now.
Unfortunately, lowercasing the value as you are describing is not possible considering the pickers current set up. The uppercase portion denotes the "selected point".
The feature you are describing is certainly something that could be included in the hook. I'll look at including it in the upcoming version.

Ah, I see. I was wondering why the captital RBG kept hopping spots on me when trying to parse that with regex!   If you are able to add some feature like this it would also need the 'angle' value in there somewhere.  Something like (I'm no programmer, so feel free to ignore as just thinking this on the fly as a consumer without understanding your package structure at all)
{
    type : 'gradient' // gradient / solid
    style : 'linear', // linear / radial
    stops : [
        [0,255,0,0,1], // percentage, r,g,b,a
        [0.5,0,255,0,1] // percentage, r,g,b,a
        [1,0,0,255,1] // percentage, r,g,b,a
    ],
    degrees: '45', // angle degrees of gradient
}

Great package btw!

@wilsmex Thanks man! Appreciate that.
I think the feature you are describing is very doable. I will try and get it included in the next release and update you here on the progress!

Your requested feature is now available in v2.1.11 through the useColorPicker hook which now returns the 'getGradientObject'
function
To use, import the hook like so:
import ColorPicker, { useColorPicker } from 'react-best-gradient-color-picker'
Grab the function from the hook and be sure to pass the value and callback in like so:
const { getGradientObject } = useColorPicker(value, callback)
Run the function whenever desired to get the value:
const gradientObject = getGradientObject()
Example value:
{
    "isGradient": true,
    "gradientType": "linear-gradient",
    "degrees": "40deg",
    "colors": [
        {
            "value": "rgba(27,107,235,1)",
            "left": 0
        },
        {
            "value": "rgba(25,245,157,1)",
            "left": 100
        }
    ]
}


Vx-Submariner issue On Rancher RKE1 Setup

Hello Team,
I have installed 3 Cluster RKE1 as cluster setup. First cluster has 3 Nodes. Which I'm testing this intra cluster communitation on among these nodes. ( Second cluster is single node as broker, And third node is also single node which I'm making inter-cluster communication)
The issue is  With this Rancher setup, I m having issues with intra cluster communication for Cluster A. It looks like vx-submariner tunnels cannot make proper tunnel to communicate nodes among each other.
Here is subctl diagnose all output below;
Cluster "cluster-c"
 ✓ Checking Submariner support for the Kubernetes version
 ✓ Kubernetes version "v1.27.6" is supported

 ✓ Non-Globalnet deployment detected - checking that cluster CIDRs do not overlap
 ✓ Checking DaemonSet "submariner-gateway"
 ✓ Checking DaemonSet "submariner-routeagent"
 ✓ Checking DaemonSet "submariner-metrics-proxy"
 ✓ Checking Deployment "submariner-lighthouse-agent"
 ✓ Checking Deployment "submariner-lighthouse-coredns"
 ✓ Checking the status of all Submariner pods
 ✓ Checking that gateway metrics are accessible from non-gateway nodes
 ✓ Skipping this check as it's a single node cluster

 ✓ Checking Submariner support for the CNI network plugin
 ✓ The detected CNI network plugin ("canal-flannel") is supported
 ✓ Checking gateway connections
 ✓ Checking Submariner support for the kube-proxy mode
 ✓ The kube-proxy mode is supported
 ✓ Checking that firewall configuration allows intra-cluster VXLAN traffic
 ✓ Skipping this check as it's a single node cluster

 ✓ Checking that services have been exported properly

Cluster "cluster-a"
 ✓ Checking Submariner support for the Kubernetes version
 ✓ Kubernetes version "v1.27.6" is supported

 ✓ Non-Globalnet deployment detected - checking that cluster CIDRs do not overlap
 ✓ Checking DaemonSet "submariner-gateway"
 ✓ Checking DaemonSet "submariner-routeagent"
 ✓ Checking DaemonSet "submariner-metrics-proxy"
 ✓ Checking Deployment "submariner-lighthouse-agent"
 ✓ Checking Deployment "submariner-lighthouse-coredns"
 ✓ Checking the status of all Submariner pods
 ✓ Checking that gateway metrics are accessible from non-gateway nodes

 ✓ Checking Submariner support for the CNI network plugin
 ✓ The detected CNI network plugin ("canal-flannel") is supported
 ✓ Checking gateway connections
 ✓ Checking Submariner support for the kube-proxy mode
 ✓ The kube-proxy mode is supported
 ✗ Checking that firewall configuration allows intra-cluster VXLAN traffic
 ✗ The tcpdump output from the sniffer pod does not contain the expected remote endpoint IP <IP_ADDRESS>. Please check that your firewall configuration allows UDP/4800 traffic. Actual pod output:
tcpdump: verbose output suppressed, use -v[v]... for full protocol decode
listening on vx-submariner, link-type EN10MB (Ethernet), snapshot length 262144 bytes

0 packets captured
0 packets received by filter
0 packets dropped by kernel

 ✓ Checking that services have been exported properly

Cluster "cluster-b"
 ✓ Checking Submariner support for the Kubernetes version
 ✓ Kubernetes version "v1.27.6" is supported

 ⚠ Submariner connectivity feature is not installed

 ⚠ Submariner service discovery feature is not installed

Skipping inter-cluster firewall check as it requires two kubeconfigs. Please run "subctl diagnose firewall inter-cluster" command manually.

subctl version: v0.16.2

Here is ARP record of d4kcp-node01 of cluster A, which is primary node( Only etcd and controlplane) When diagnosing,
root@d4kcp-node01:/home/stuab# tcpdump -i 'vx-submariner'
tcpdump: verbose output suppressed, use -v[v]... for full protocol decode
listening on vx-submariner, link-type EN10MB (Ethernet), snapshot length 262144 bytes
11:50:22.817224 ARP, Request who-has <IP_ADDRESS> tell <IP_ADDRESS>, length 28
11:50:23.847219 ARP, Request who-has <IP_ADDRESS> tell <IP_ADDRESS>, length 28
11:50:24.871200 ARP, Request who-has <IP_ADDRESS> tell <IP_ADDRESS>, length 28

It doesnt get any answer for intra cluster vx-submariner ip for second node.
root@d4kcp-node01:/home/stuab# arp
Address                  HWtype  HWaddress           Flags Mask            Iface
<IP_ADDRESS>                ether   5e:57:4b:1a:7b:0a   C                     cali599b869d91e
<IP_ADDRESS>                ether   a2:ae:b4:13:4d:95   CM                    flannel.1
d4kcp-node02             ether   00:50:56:01:0c:3a   C                     ens192
<IP_ADDRESS>          ether   00:50:56:01:07:a3   C                     ens192
<IP_ADDRESS>            ether   00:09:0f:09:00:0e   C                     ens192
<IP_ADDRESS>          ether   00:50:56:01:14:2f   C                     ens192
<IP_ADDRESS>                  (incomplete)                              vx-submariner
<IP_ADDRESS>                ether   52:d9:a3:90:4e:e9   CM                    flannel.1

So It does look like vx tunnel cannot be established in between Cluster A's nodes for some reason. I cannot detect any firewall related issues.
First Node,
root@d4kcp-node01:/home/stuab# ip addr show
REDACTED...
 vx-submariner: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1450 qdisc noqueue state UNKNOWN group default
    link/ether 6e:45:79:40:b7:33 brd ff:ff:ff:ff:ff:ff
    inet <IP_ADDRESS>/8 brd <IP_ADDRESS> scope global vx-submariner
       valid_lft forever preferred_lft forever
    inet6 fe80::10df:b8ff:fe25:41fe/64 scope link
       valid_lft forever preferred_lft forever

Second Node,
root@kmb-poc01:~# ip addr show
REDACTED...
vx-submariner: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1450 qdisc noqueue state UNKNOWN group default
    link/ether 6e:45:79:40:b7:33 brd ff:ff:ff:ff:ff:ff
    inet <IP_ADDRESS>/8 brd <IP_ADDRESS> scope global vx-submariner
       valid_lft forever preferred_lft forever
    inet6 fe80::b08e:c7ff:fea3:76fc/64 scope link
       valid_lft forever preferred_lft forever

Third Node
root@kmb-poc02:~# ip addr show
REDACTED...
27: vx-submariner: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1450 qdisc noqueue state UNKNOWN group default
    link/ether 6e:45:79:40:b7:33 brd ff:ff:ff:ff:ff:ff
    inet <IP_ADDRESS>/8 brd <IP_ADDRESS> scope global vx-submariner
       valid_lft forever preferred_lft forever
    inet6 fe80::7064:79ff:fe5e:8f7e/64 scope link
       valid_lft forever preferred_lft forever

So it looks like these addresses cannot be established among each other As much I can see on tcp Dump output.
I also wanted to note that This issue is only happening when Deploying a service when deploying non-cluster gateway. So, If I deploy a service into Cluster A' s  gateway node, I can access this service from Cluster C. This issue is only happening when accessing non-gateway node. Maybe about some intra cluster A internal communication.
I 'm also sharing subctl gather all,
[submariner gather all.zip] (https://github.com/submariner-io/submariner/files/13726669/submariner.gather.all.zip)
Any help would be appreciated. Thanks in advance.

Hey @eremcan, thanks for the report! We don't have foks from Rancher contributing anymore and our active devs don't have experience with Rancher or access to RKE clusters to test with, but we can look a the pod logs and see if anything stands out.

From the diagnose output , it seems like 4800 is not open. How did you confirm there is no firewall rules blocking the traffic?
The tcpdump output from the sniffer pod does not contain the expected remote endpoint IP <IP_ADDRESS>. Please check that your firewall configuration allows UDP/4800 traffic. Actual pod output:
tcpdump: verbose output suppressed, use -v[v]... for full protocol decode
listening on vx-submariner, link-type EN10MB (Ethernet), snapshot length 262144 bytes

Hello @dfarrell07 @aswinsuryan
In my iptable rules, There is a firewall rule applied all of my cluster Nodes below,
(Cluster A)
Chain SUBMARINER-INPUT (1 references)
num   pkts bytes target     prot opt in     out     source               destination
1      185 16854 ACCEPT     17   --  *      *       <IP_ADDRESS>/0            <IP_ADDRESS>/0            udp dpt:4800

I couldn't realize some other rules which blocks 4800 in my iptable rules.
So problem is might not about iptables AFAIK. Initially, I have clusters which are non-strict VM's(without any firewall or any limitations) and subctl and kubernetes RKE 1 cluster automatically applies their iptable rules according to its core needs.
Please refer subctl gather all command that I have shared above.
Additionally,
https://github.com/submariner-io/submariner/issues/2489 and https://github.com/submariner-io/submariner/issues/2660 are looks similar what I have been through.
Any help would be appreciated.

In the underlay network? Is '4800'  open by default?

@aswinsuryan There is no limitation or restriction on my cloud provider. There is just root any-any rule for any protocols.

Ok, since it is working in one cluster and not in another. It could be something specific to the second cluster config. Does it have any interface in 240 network ? can you try ip route get for <IP_ADDRESS>?
Also is some other process using the port 4800?
In the route agent logs I could see this error
```
[90m2023-12-19T13:05:19.500Z DBG ../resource_syncer.go:561 ResourceSyncer       Syncer "Node watcher for routeagent_driver registry": transform function returned nil - not syncing - requeue: false
2023-12-19T13:05:42.689Z DBG ../resource_syncer.go:561 ResourceSyncer       Syncer "Node watcher for routeagent_driver registry": transform function returned nil - not syncing - requeue: false
2023-12-19T13:06:13.243Z DBG ../resource_syncer.go:561 ResourceSyncer       Syncer "Node watcher for routeagent_driver registry": transform function returned nil - not syncing - requeue: false
2023-12-19T13:11:20.224Z DBG ../resource_syncer.go:561 ResourceSyncer       Syncer "Node watcher for routeagent_driver registry": transform function returned nil - not syncing - requeue: false
2023-12-19T13:16:28.261Z DBG ../resource_syncer.go:561 ResourceSyncer       Syncer "Node watcher for routeagent_driver registry": transform function returned nil - not syncing - requeue: false
2023-12-19T13:21:34.614Z DBG ../resource_syncer.go:561 ResourceSyncer       Syncer "Node watcher for routeagent_driver registry": transform function returned nil - not syncing - requeue: false
2023-12-19T13:26:42.534Z DBG ../resource_syncer.go:561 ResourceSyncer       Syncer "Node watcher for routeagent_driver registry": transform function returned nil - not syncing - requeue: false
2023-12-19T13:31:50.189Z DBG ../resource_syncer.go:561 ResourceSyncer       Syncer "Node watcher for routeagent_driver registry": transform function returned nil - not syncing - requeue: false

@tpantelis  does this mean any connectivity issue?

Can you try a tcpdump on the interface that connects the nodes so that we can make sure if the packets are leaving and reaching the other node?

Closing this issue for now. Please feel free to reopen if this is still an issue.


Add Plurk social media platform

See github link: https://github.com/WhichBrowser/Parser-PHP/issues/505
Link: https://www.plurk.com/
p.s. Saw a bug 'PagesInventory ' it had an extra space and changed it to this 'PagesInventory'

Coverage increased (+0.02%) to 99.97% when pulling 5092177d1cf7d0116d9f90d83b45f9db840a8b4c on ayumi-cloud:plurk into 880b9fa797401d14b28956442944c3daa70240ff on WhichBrowser:master.

@NielsLeenheer Please can you merge this pr.

Fixed in 79639fd2a332cb75d76800f9badb0a7ef33655d3

Fixed in 79639fd2a332cb75d76800f9badb0a7ef33655d3


Ap 191 check your answers

What
AP-197
Implements the Check Your Answers page for providers:

the page already exists as a placeholder, this adds content to that page;
tests already exist, this refines them to test the new content;
the check your answers design pattern does not currently exist in GOV.UK Frontend, so import the styling from the GOV.UK Prototype Kit as an interim solution;
renames all references to check_your_answers to check_provider_answers as required in the design.

Checklist
Before you ask people to review this PR:

[x] Tests and rubocop should be passing: bundle exec rake
[x] Github should not be reporting conflicts; you should have recently run git rebase master.
[x] There should be no unneccessary whitespace changes. These make diffs harder to read and conflicts more likely.
[x] The PR description should say what you changed and why, with a link to the JIRA story.
[x] You should have looked at the diff against master and ensured that nothing unexpected is included in your changes.
[x] You should have checked that the commit messages say why the change was made.

I spoke with Simon offline about the display of the page, and I think it is expected for all the answers to be aligned the same way, regardless if they're in a different section, in this case the proceeding should be aligned the same way as the applicant details answers.


morph: 1.2.0 -> 1.3.0

Motivation for this change
Bump morph to latest release.
Release notes: https://github.com/DBCDK/morph/releases/tag/v1.3.0
Things done

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

[X] NixOS
[ ] macOS
[ ] other Linux distributions

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

Notify maintainers
cc @adamtulinius

@GrahamcOfBorg build morph


Support Relay __id data ID field on non-scalar types

Is your feature request related to a problem? Please describe.
We're using graphql-eslint with graphql-config to enable linting GraphQL string in our frontend codebase which uses Relay.
Field __id is available in Relay on all non-scalar fields: https://relay.dev/docs/next/glossary/#dataid. The meaning behind this id (also called data ID) is to expose the value of the key in Relay's cache. Most of the times __id equals to id if type extends Node interface.
But our use-case is with connections. Relay mutations can receive "connection ID" so you can update lists, it's idiomatic to pass this id dynamically by querying __id on connection fragment (supplied to usePaginationFragment hook).

The problem is that at the moment we must disable @graphql-eslint/fields-on-correct-type rule because field __id is not recognized, e.g.:
error  Cannot query field "__id" on type "ItemsConnection"  @graphql-eslint/fields-on-correct-type

Describe the solution you'd like
Provide boolean option in configuration file that would specify that schema is used in Relay context. It could be as simple as relay: true or to future-proof it for other clients:
client: 'relay'

It's not really important I think.
Describe alternatives you've considered
I've tried to create my own extension in graphql-config so I could try extending every non-scalar type with __id but I did not manage to load the extension. It's not entirely clear to me from documentation how can I create and load my own extension if I'm using graphql-eslint because my goal is to not create 3rd party tool (I noticed tools like graphql-codegen load the configuration and extension in one specific place via CLI).
Additional context

👋 Hey, what do you think about my proposal? Does it make sense?

Yes, it totally makes sense and will be fixed!