Calico 3.2 + flannel - bad routing table in containers

Expected Behavior
In 3.1.3:
$ kubectl -n kube-system exec -ti centos -- bash
bash-4.2# netstat -nr
Kernel IP routing table
Destination     Gateway         Genmask         Flags   MSS Window  irtt Iface
<IP_ADDRESS>         <IP_ADDRESS>     <IP_ADDRESS>         UG        0 0          0 eth0
<IP_ADDRESS>     <IP_ADDRESS>         <IP_ADDRESS> UH        0 0          0 eth0

Current Behavior
In 3.2.1:
$ kubectl -n kube-system exec -ti centos -- bash
bash-4.2# netstat -nr
Kernel IP routing table
Destination     Gateway         Genmask         Flags   MSS Window  irtt Iface
<IP_ADDRESS>    <IP_ADDRESS>     <IP_ADDRESS>   UG        0 0          0 eth0
<IP_ADDRESS>     <IP_ADDRESS>         <IP_ADDRESS> UH        0 0          0 eth0

So routing is set only for pod's subnet.
Possible Solution
No idea. I believe one of the PRs merged into 3.2 did something with routing. Could not find any documentation on this behavior change and how to retain old behavior.
Steps to Reproduce (for bugs)
Followed https://docs.projectcalico.org/v3.1/getting-started/kubernetes/installation/flannel and https://docs.projectcalico.org/v3.2/getting-started/kubernetes/installation/flannel. Behavior is shown above.
Context

The issue is breaking connectivity from pods to any subnet other than pod's subnet, including services created in cluster.
Your Environment

Calico version 3.2.1
Orchestrator version (e.g. kubernetes, mesos, rkt): Kubernetes 1.10.5
Operating System and version: CoreOS 1800.7.0 (Rhyolite)

Hm, I just installed v3.2.1 on a fresh cluster using the link above and it looks like the routes are correct to me:
root@bb-67b45b59d6-w76q6:/# ip route
default via <IP_ADDRESS> dev eth0
<IP_ADDRESS> dev eth0  scope link

root@bb-67b45b59d6-w76q6:/# netstat -nr
Kernel IP routing table
Destination     Gateway         Genmask         Flags   MSS Window  irtt Iface
<IP_ADDRESS>         <IP_ADDRESS>     <IP_ADDRESS>         UG        0 0          0 eth0
<IP_ADDRESS>     <IP_ADDRESS>         <IP_ADDRESS> UH        0 0          0 eth0

I wonder if there is another environmental difference?

@caseydavenport that's odd. What can affect routing table installed in containers? Any parameters/settings set somewhere? And why would behavior differ between calico 3.1 and 3.2?

What I have found so far. Increased log level verbosity in kubelet and set calico cni loglevel to debug.
3.2.1 produces the following line:
https://github.com/projectcalico/cni-plugin/blob/release-v3.2/k8s/k8s.go#L149
Sep 06 14:41:35 lvdkbw501 kubelet-wrapper[8633]: 2018-09-06 21:41:35.484 [INFO][10949] k8s.go 149: Using custom routes from CNI configuration. ContainerID="ed4e18eb87782994a4f50cddb16a3b919d414db4a142fc65f45a924969341433" Namespace="kube-system" Pod="centos" WorkloadEndpoint="lvdkbw501-k8s-centos-eth0" routes=[]*net.IPNet{(*net.IPNet)(0xc4203c8480)}

Reading the source code suggests that the https://github.com/projectcalico/cni-plugin/blob/release-v3.2/k8s/k8s.go#L71 condition is true. Is it supposed to be true using flannel for networking?
Seeing that the type is set to host-local from the calico perspective:
Sep 06 14:41:23 lvdkbw501 kubelet-wrapper[8633]: 2018-09-06 21:41:23.363 [DEBUG][10672] utils.go 135: Looking for IPAM plugin in paths ContainerID="d67ec769f4b2bb1fa1146d476d0f90299e08e8ac9069177384295b97df928c8f" paths="/opt/flannel/bin:/opt/cni/bin" type="host-local"

Investigated a bit.
I think this is because when using Flannel as a delegate / meta-plugin, it populates the "routes" section of the IPAM config.
See here: https://github.com/containernetworking/plugins/blob/master/plugins/meta/flannel/flannel.go#L236
In earlier versions of Calico, we ignored this. But, v3.2 now respects it. One option is to just now use flannel in delegate mode and to instead just run the Calico plugin + flanneld - this is what the manifest above does (and why I didn't see this in my repro attempts).

@fasaxc yeah, the etcd one still uses delegation, whereas kdd does not.
I'm not sure what the right fix is - either in flannel or in the Calico plugin. The same issue was fixed for flannel+bridge in the bridge plugin by adding a field which tells the bridge plugin it is the default gw.
IMO this is really something that should be fixed in flannel, so it can be configured to pass the right set of routes. Otherwise every delegated plugin will need to implement the same logic.

A fix for this went in there: https://github.com/projectcalico/cni-plugin/pull/592
It allows enabling a default route to be programmed. Not in a release yet, but I hope to cherry-pick it into the next v3.2 release. Barring that, it will definitely be in Calico v3.3.0.

Thank you!


Not downloading

It's stuck at 2 videos of 12. Please check again if it is working or not. Thank you very much for this project.
https://prnt.sc/12c9kvy

I met the same issue for organization user.

Same

My downloads stopped working, looks like some issue appeared and udemy changed something in their videos or files path to access....

Mine also not downloading any videos. Please check. Thanks


New chips

following issue #458
Enrich standard library with commonly used chips

Looks good! Can you please add a switch for "On Any Key Pressed", that activates its children if any key is down?
Thanks!


T.herman/cylinder tex coords

In the end i used original texturing, same as was in opengl part, except its mirrored along u-axis in cylinder part


Refactor unit test

I've break down our single Test class into smaller test cases, Each layout will be tested in a standalone class, this will ensure better readability and easier maintainability.
all layout test cases are assembled in a Junit test suite. I configured maven-surfire-plugin to run only the test suite.

Coverage decreased (-0.5%) to 92.711% when pulling acb65fbee196acba73b58a77559b16bafed23a36 on Hdebbech:master into d20bebde8636ab94579f4be8b05193bb25ba3bc9 on JsonSchema-JavaUI:master.

Coverage remained the same at 93.166% when pulling cc5bdf35af2d6c3b446c42618e920b1264dfec72 on Hdebbech:master into d20bebde8636ab94579f4be8b05193bb25ba3bc9 on JsonSchema-JavaUI:master.


Hide ignored files on the HTML report

Is there any way to hide the ignored files on the HTML report?
Currently they appear on the report having 0% coverage.

Hi @ArthurAlvarez 👋
Ignored files were supposed to appear greyed out, will take a look 👍

@ArthurAlvarez this is how ignored files appear on my HTML reports.

Are you talking about the ignored files on the report or the target?

I was talking about the targets but I was able to solve this by using the 'exclude_targets' flag.
Thanks for the support!


[gherkin-c] Fix UNIX build, fix install path

The official CMakeLists uses make to install the project which is incorrect.
And fix the header files installation:
-- Performing post-build validation
Restricted headers paths are present. These files can prevent the core C++ runtime and other packages from compiling correctly:

    include/error.h
    include/parser.h

Fixes #22574.

Thanks!


[Crane] Couldn't inline method call 'Row' into @androidx.compose.runtime.Composable

org.jetbrains.kotlin.codegen.CompilationException: Back-end (JVM) Internal error: Couldn't inline method call 'Row' into
@androidx.compose.runtime.Composable
private fun MonthHeader(modifier: androidx.compose.ui.Modifier = ..., month: kotlin.String, year: kotlin.String): kotlin.Unit defined in androidx.compose.samples.crane.calendar in file Calendar.kt
@Composable
private fun MonthHeader(modifier: Modifier = Modifier, month: String, year: String) {
Row(modifier = modifier) {
Text(
modifier = Modifier.weight(1f),
text = month,
style = MaterialTheme.typography.h6
)
Text(
modifier = Modifier.align(Alignment.CenterVertically),
text = year,
style = MaterialTheme.typography.caption
)
}
}
Cause: Not generated
File being compiled: (79,5) in
Crane/app/src/main/java/androidx/compose/samples/crane/calendar/Calendar.kt
The root cause java.lang.IllegalStateException was thrown at: org.jetbrains.kotlin.codegen.inline.InlineCodegen$Companion.getCompiledMethodNodeInner(InlineCodegen.kt:596)
at org.jetbrains.kotlin.codegen.inline.InlineCodegen.throwCompilationException(InlineCodegen.kt:105)

Thanks for the bug report!
Could you please provide more information:

Which sample app you are experiencing this crash on? Are you still experiencing this issue? We are going through cleanup of the issues here.
Android Version
Compose Version
Kotlin version
Steps to reproduce the issue
Thanks!

Closing as inactive and unable to reproduce


Ctrl+Click on a constant opens doc but doesn't scroll to definition (2)

Godot 3.1.1
Similar to https://github.com/godotengine/godot/issues/17396
In the following script, Ctrl+clicking UPDATE_ONCE opens Viewport docs, but doesn't scroll to the enum value.
extends Viewport

func update():
	render_target_update_mode = Viewport.UPDATE_ONCE

Seems to be fixed in the master. Cannot reproduce this (CTRL+Click goes to exact doc definition).

I can't reproduce this either on https://github.com/godotengine/godot/commit/3db1d400458853bfa33c4605c819d368c11e4a57, so I'll close this.


Support Dynamic component as offcanvas

Hello,
As the doc says, "Conceptually, they are quite similar to the Modal component". Unfortunately, unlike modals, they don't provide a way to show a dynamic component.
I would be great to have the same feature for Offcanvas.

Thx @gvreddy04 !


Some songs are missing?

I used to download a song named "What you know" mapped by BennyDabeast from this URL:
Now it has a 404 error
Is this a bug of new system? or this song is removed by someone?
I'm sure the URL is correct

Not all maps got migrated, old versions of maps were skipped due to technical constraints. Any maps that failed to convert were also skipped. If you know the original hash of the beatmap I can look into where it went. Without the map's hash there's not much I can do.

Is there a log of what maps have been skipped? Would be nice to inform the mappers so they can reupload their content.

@lolPants
I don't have the hash of the song.
So you are going to drop those songs completely? or it will convert in the future?
It would be nice to at least save old songs somewhere , and make a way to load legacy songs by same old method used before in SongCore plugin.
or at least i can download and remap some of my favorites again. we can't drop them!

@lolPants
I don't have the hash of the song.
So you are going to drop those songs completely? or it will convert in the future?
It would be nice to at least save old songs somewhere , and make a way to load legacy songs by same old method used before in SongCore plugin.
or at least i can download and remap some of my favorites again. we can't drop them!

@n3tman can you search in the dump data by name?
I doubt that anyone has hashmap of the songs. because that number was in the folder of the song. we don't have the song so we don't have the hashmap.

You'll have to ask the operators of BeatSaver for that info. As stated in the readme I do not operate BeatSaver and this is just the code they use.


fixed invalid carriage return coming from oracle ojdbc8 <IP_ADDRESS>

After upgrading to Java 11 and comparing SqlServer with Oracle DB (ojdbc8 <IP_ADDRESS>) we discovered invalid 'carriage return' %#13; at the line ending of the SQL definition of oracle views.
This change makes sure that the lineendings are standardized.

Thanks for your pull request!
We have assigned this into our internal sprint #7.3.

Hi @Trepky , just an update, this appears to be in "code review" internally.  Will update on change.

Moved this to Beta 2

This change has been brought in with a previous PR


Text seems not to be rendered in iPhone 14 Pro, iOS 16.

Text seems not to be rendered in iPhone 14 Pro, iOS 16.
Here are some details:

Works well in iPhone 14 and older devices, only appears in iPhone 14 Pro and later devices.
Input some text, turns out that it seems not to be render.
Switch the app to the background, and open it again, it renders well, but breaks again if you continue inputing.

What‘s in my mind?
Seems like DTCoreText did not render properly. If you long press the editorView and DTLoupeView pops out, the text will be rendered well again.
What I have tried?

setNeedsLayout and setNeedsDisplay when editorView did change

Any suggestions will be highly appreciated, thanks for your effort!

Turns out it didn't work well if you set UIViewEdgeAntialiasing to YES in your info.plist. Problem solved.


vector reduction when the element 0 is masked

Suppose teh first element is masked in a reduction instruction, I need to confirm:

the reduction result still stores in element 0 of destination register;
vs1[0] and vs2[0] don't participate reduction.

And another corner case, suppose all elements are masked. the element 0 of destination register is 0, or unchanged?

For these instructions, the mask only controls which elements participate in the reduction.  vd[0] is written with the result even if all elements are masked.  (In that case, vs1[0] is simply copied to vd[0].)
vs1[0] does participate in the reduction, irrespective of the mask.
vs2[0] participates in the reduction if element 0 is not masked.

Got it, thanks. And the spec needs to clarify it.

I will keep this issue open.

BTW, @aswaterman when vl=0, will vd[0] be updated?

I think these should follow the usual rule that no destination elements are updated when vstart >= vl (i.e. when vl=0, since vstart must be 0).


Use 1ES pool for pipeline "cosmos-ctl-image-build"

All pipelines should use 1ES pools for Windows and Ubuntu jobs.
If it's safe to do so, you can run the pipeline on this PR to verify it still runs correctly on the 1ES pool.  I'm not sure what the pipeline does, so I'm not sure what impact running it on this PR might have.

Keeping issue active.

@aayush3011 can you please take a look at this issue?
@mikeharder I believe this is blocked on us?

@kushagraThapar: Yes, this is waiting on the Cosmos team to verify the pipeline still works correctly.  It's just moving the agent used by this pipeline to a different pool (but the same OS image), so I wouldn't expect anything to break, but it should be verified before this is merged.

You should just need to run the pipeline against this PR, and if everything looks good this can be merged.

Got it @mikeharder , thanks!
@aayush3011  - let's run the pipeline for CTL against this branch and merge this PR, thanks!

@mikeharder What is the status on this PR?

@JonathanGiles: It would be good to merge this, but it needs to be approved by the Cosmos team.

@kushagraThapar ^^ thanks

@mikeharder / @JonathanGiles apologies for the delay on this, our CTL builds, and pipelines have been broken for quite some time now. I will follow up on this internally and will approve it soon, thanks for your patience!

@mikeharder  - this is the pipeline that needs to be run on this PR - https://dev.azure.com/azure-sdk/internal/_build?definitionId=1813&_a=summary
Can you please schedule a run for this if possible. I don't see an option to run it on this PR.

@mikeharder - this is the pipeline that needs to be run on this PR - https://dev.azure.com/azure-sdk/internal/_build?definitionId=1813&_a=summary Can you please schedule a run for this if possible. I don't see an option to run it on this PR.

Run: https://dev.azure.com/azure-sdk/internal/_build/results?buildId=2053374&view=results

@kushagraThapar: The build was successful.  OK to merge?
https://dev.azure.com/azure-sdk/internal/_build/results?buildId=2053374&view=results

@kushagraThapar: The build was successful. OK to merge?
https://dev.azure.com/azure-sdk/internal/_build/results?buildId=2053374&view=results

Yes please, approved it. Feel free to merge it or let me know if you need my help on merging it.

/check-enforcer override


Same config, works well on training/test, but fails in analyze_results

Thanks for your error report and we appreciate it a lot.
Checklist

I have searched related issues but cannot get the expected help.
I have read the FAQ documentation but cannot get the expected help.
The bug has not been fixed in the latest version.

Describe the bug
Same config, works well on training/test, but fails in analyze_results
Reproduction

What command or script did you run?

file_client_args = dict(backend='disk')
load_pipeline = [
    dict(type='LoadImageFromFile', file_client_args=dict(backend='disk')),
    dict(type='LoadAnnotations', with_bbox=True, with_mask=True),
    dict(type='Resize', img_scale=(xxxx, xxxx), keep_ratio=True),
    dict(type='FilterAnnotations', min_gt_bbox_wh=(0.01, 0.01)),
    dict(type='RandomFlip', flip_ratio=0.5),
    dict(type='Pad', size_divisor=32)
]
train_pipeline = [
    dict(type='CopyPaste', max_num_pasted=100),
    dict(
        type='Normalize',
        mean=[123.675, 116.28, 103.53],
        std=[58.395, 57.12, 57.375],
        to_rgb=True),
    dict(type='DefaultFormatBundle'),
    dict(type='Collect', keys=['img', 'gt_bboxes', 'gt_labels', 'gt_masks'])
]
test_pipeline = [
    dict(type='LoadImageFromFile', file_client_args=dict(backend='disk')),
    dict(
        type='MultiScaleFlipAug',
        img_scale=(xxxx, xxxx),
        flip=False,
        transforms=[
            dict(type='Resize', keep_ratio=True),
            dict(type='RandomFlip'),
            dict(
                type='Normalize',
                mean=[123.675, 116.28, 103.53],
                std=[58.395, 57.12, 57.375],
                to_rgb=True),
            dict(type='Pad', size_divisor=32),
            dict(type='ImageToTensor', keys=['img']),
            dict(type='Collect', keys=['img'])
        ])
]
data = dict(
    samples_per_gpu=2,
    workers_per_gpu=2,
    train=dict(
        type='MultiImageMixDataset',
        dataset=dict(
            type='CocoDataset',
            ann_file='data/train.json',
            img_prefix='data/train',
            pipeline=[
                dict(
                    type='LoadImageFromFile',
                    file_client_args=dict(backend='disk')),
                dict(type='LoadAnnotations', with_bbox=True, with_mask=True),
                dict(type='Resize', img_scale=(xxxx, xxxx), keep_ratio=True),
                dict(type='FilterAnnotations', min_gt_bbox_wh=(0.01, 0.01)),
                dict(type='RandomFlip', flip_ratio=0.5),
                dict(type='Pad', size_divisor=32)
            ]),
        pipeline=[
            dict(type='CopyPaste', max_num_pasted=100),
            dict(
                type='Normalize',
                mean=[123.675, 116.28, 103.53],
                std=[58.395, 57.12, 57.375],
                to_rgb=True),
            dict(type='DefaultFormatBundle'),
            dict(
                type='Collect',
                keys=['img', 'gt_bboxes', 'gt_labels', 'gt_masks'])
        ]),
    val=dict(
        type='CocoDataset',
        ann_file='data/val.json',
        img_prefix='data/val',
        pipeline=[
            dict(
                type='LoadImageFromFile',
                file_client_args=dict(backend='disk')),
            dict(
                type='MultiScaleFlipAug',
                img_scale=(xxxx, xxxx),
                flip=False,
                transforms=[
                    dict(type='Resize', keep_ratio=True),
                    dict(type='RandomFlip'),
                    dict(
                        type='Normalize',
                        mean=[123.675, 116.28, 103.53],
                        std=[58.395, 57.12, 57.375],
                        to_rgb=True),
                    dict(type='Pad', size_divisor=32),
                    dict(type='ImageToTensor', keys=['img']),
                    dict(type='Collect', keys=['img'])
                ])
        ]),
    test=dict(
        type='CocoDataset',
        ann_file='data/val.json',
        img_prefix='data/val',
        pipeline=[
            dict(
                type='LoadImageFromFile',
                file_client_args=dict(backend='disk')),
            dict(
                type='MultiScaleFlipAug',
                img_scale=(xxxx, xxxx),
                flip=False,
                transforms=[
                    dict(type='Resize', keep_ratio=True),
                    dict(type='RandomFlip'),
                    dict(
                        type='Normalize',
                        mean=[123.675, 116.28, 103.53],
                        std=[58.395, 57.12, 57.375],
                        to_rgb=True),
                    dict(type='Pad', size_divisor=32),
                    dict(type='ImageToTensor', keys=['img']),
                    dict(type='Collect', keys=['img'])
                ])
        ]))

Did you make any modifications on the code or config? Did you understand what you have modified?
Yes, I change the coco datasets into my own

What dataset did you use?
My own datasets with standard coco format.

Environment

Please run python mmdet/utils/collect_env.py to collect necessary environment information and paste it here.
You may add addition that may be helpful for locating the problem, such as

How you installed PyTorch [e.g., pip, conda, source]
Other environment variables that may be related (such as $PATH, $LD_LIBRARY_PATH, $PYTHONPATH, etc.)

Error traceback
If applicable, paste the error trackback here.
Traceback (most recent call last):
  File "tools/analysis_tools/analyze_results.py", line 210, in <module>
    main()
  File "tools/analysis_tools/analyze_results.py", line 199, in main
    cfg.data.test.pipeline = get_loading_pipeline(cfg.data.train.pipeline)
  File "/home/tiger/.local/lib/python3.7/site-packages/mmdet/datasets/utils.py", line 112, in get_loading_pipeline
    'The data pipeline in your config file must include ' \
AssertionError: The data pipeline in your config file must include loading image and annotations related pipeline.

Bug fix
If you have already identified the reason, you can provide the information here. If you are willing to create a PR to fix it, please also leave a comment here and that would be much appreciated!

@mitming This bug has been fixed in https://github.com/open-mmlab/mmdetection/pull/7922, please update the code

@mitming This bug has been fixed in #7922, please update the code

Thanks a lot


[mui v5 styled-components] - Typescript doesn't recognise "styleProps"

I'm following this guide for the Stepper Component: https://next.material-ui.com/components/steppers/#customized-horizontal-stepper
In my Stepper.styles.ts file I copied the code from there:
export const StyledStepIcon = styled('div')(({ theme, styleProps }) => ({
  backgroundColor: 'blue',
  ...(styleProps.active && {
    backgroundColor: 'red
  }),
}));

However, typescript is complaining:

Property 'styleProps' does not exist on type '{ theme?: Theme; as?: ElementType; sx?: SxProps; } & ClassAttributes & HTMLAttributes & { ...; }'.ts(2339)
(parameter) styleProps: any

Does anyone know how I can fix this? Thanks!

Was renamed to ownerState


Adds support for disabling access_log globally

This PR adds support for disabling the access_log globally, through a directive in ConfigMap.
It's related to issue #234

This change is

Coverage increased (+0.02%) to 44.686% when pulling d0c4e0d713306daa75b269c21fd1b8da752496d1 on rikatz:nginx-disable-access-log into 2119b23cb850c1bcd5c30a7529283eb745dfb3ee on kubernetes:master.

/lgtm

@rikatz thanks!


Pixart Homework

comfort_level: 2
completeness: 2
I've been struggling with making homework assignments work as intended lately. I feel like I am grasping the concepts in class and everything makes sense to me, but when it comes time to apply the concepts to homework assignments, I feel like I have no idea what I am doing.

:+1: What have you tried to fix this? pairing with other students? writing pseudo code? asking questions in slack? here and happy to help if you need

@jshawl I've tried pairing with students on just a couple of occasions and it seems to help just a little bit, but not too much. It seemed that the people who I had paired with were just as confused as I was about the questions I had. I have not written any pseudo code, aside from following along in class. I'm embarrassed to ask questions in Slack because in most cases, especially recently, I feel like I don't even know where to start. I am going to stay after today and try to get some guidance on tonight's homework assignment. Maybe if I make a habit of staying after to work on homework, something will eventually click. I'm honestly terrified that I'm just not getting it fast enough and that I'm going to end up being booted from the course.


Metrics: add snapshot transport to grafana

What is changed and how it works?

Issue Number: Ref #13409
What's Changed:

Related changes
Check List
Tests

Manual test (add detailed scripts or steps below)

Side effects
Release note
None

/cc @tonyxuqqi @pingyu

/merge

/merge

/rebuild


Pin filesystem type on Google Compute Engine image to ext4

Motivation for this change
Fix #61859.
This PR fixes the mentioned issue by pinning the filesystem type of GCE disk image to ext4.
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"
[ ] 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)
[ ] Assured whether relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Please format your commit message according to the Contribution Guidelines

@Infinisil Sorry. The commit message is updated now.


AbstractTaskDriver - Assert subtask instance has no parent.

Add an assertion to AddSubTaskDriver to make sure the subtask instance hasn't already been added to another Task Driver.
What is the current behaviour?
Though it will produce incorrect behaviour it is possible to add one subtask driver instance to multiple task drivers (or even the same task driver multiple times)
What is the new behaviour?
An assertion will be thrown when a subtask driver instance is added multiple times.
What issues does this resolve?

None

What PRs does this depend on?

None

Does this introduce a breaking change?

[ ] Yes
[x] No

Can open a new PR to adjust if we decide to change but to me this is a super cheap check that will only come up which you're in the process of adding subtasks and developing this feature. It's not like an edge case behaviour at runtime will suddenly trigger this condition.
So a message isn't required. It's pretty apparent what's gone wrong if this triggers.
In terms of ANVIL_DEBUG_SAFETY vs DEBUG, really if there was an editor specific assert I would have used that because this is never going to happen in a build if you've already tested in the editor.
Rationale

Cleaner.
Never going to happen in a build.
Obvious what has gone wrong.

I vote we leave it as is.

🚚  💥  🚲
💀


Call to undefined method Kreait\Firebase\Database::getMessaging()

Bug:
On accessing firebase cloud component I am getting the error which is
Call to undefined method Kreait\Firebase\Database::getMessaging()
before that, I was using createMessaging() function which is depreciated.

PHP code: (Laravel 6.0):
$serviceAccount = ServiceAccount::fromJsonFile($serviceAccountFile);
$factory = (new Factory)
->withServiceAccount($serviceAccount)
->withDatabaseUri($firebaseDbUrl)
->create()->getDatabase();
$messaging = $factory->getMessaging();
$factory->unsubscribeFromTopic($topic, $token);
Expected behavior
It should access the cloud component but unfortunately, it doesn't.
Screenshots

**Environment **

OS: [Ubuntu 18.04, ...]
PHP version: [7.1]
Firebase SDK Version: [4.368]

I am trying to install 4.8.0 version explicitly but it is installing 4.36
don't know why?
also, Can I ask you to give me a favor?
Please update the Documentation it was some depreciated function there.

Could you please make sure that the code examples are properly formatted? It's difficult to read, especially on mobile devices (I'm on the road right now) 😅
There's a Laravel 6 compatible bundle at github.com/kreait/laravel-firebase that I recommend using. With its configuration, you can use Dependency Injection or Facades to access the single components.
In your issue, you write that you are accessing Database::getMessaging() - the Database class doesn't have this method.
self::firebaseConnect() is a custom method of yours, perhaps you can find something in there.
If you want to explicitly install release 4.8.0, add "kreait/firebase": "4.8.0" to your composer.json - although I don't see a reason why you should stick with such an old version - any newer 4.x release is fully backward compatible.
The documentation only refers to non-deprecated methods - if you're using a deprecated method, your IDE is capable to show the recommended replacement. If it doesn't please let me know where so that I can add it.
Deprecation warnings are no errors - you can still use deprecated methods without issues, they only warn you that this method/class will not exist anymore in a future 5.x release.

I just was able to understand what you wrote in the code: $factory->create() returns an instance of the Firebase class, $factory->create()->getDatabase() returns a Database object. You‘re probably set when you remove the ->getDatabase() part.

I just was able to understand what you wrote in the code: $factory->create() returns an instance of the Firebase class, $factory->create()->getDatabase() returns a Database object. You‘re probably set when you remove the ->getDatabase() part.

@jeromegamez  Thanks a lot. It is working now

Just making sure everybody sees this with v5.x of the library:
require DIR . '/vendor/autoload.php';
use Kreait\Firebase\Factory;
use Kreait\Firebase\ServiceAccount;
use Kreait\Firebase\Messaging\CloudMessage;
use Kreait\Firebase\Messaging\Notification;
$factory = (new Factory)->withServiceAccount(DIR.'/YOURKEY.json');
$messaging = $factory->createMessaging();
Its create messaging not getMessaging

Thanks @aloncarmel! This is also mentioned in the troubleshooting docs: https://firebase-php.readthedocs.io/en/5.11.0/troubleshooting.html#call-to-private-undefined-method
I'll create a new GitHub issue label so that it's easier to see if an issue is related to a 4.x, 5.x or future release. 🤞

Thanks @aloncarmel! This is also mentioned in the troubleshooting docs: https://firebase-php.readthedocs.io/en/5.11.0/troubleshooting.html#call-to-private-undefined-method
I'll create a new GitHub issue label so that it's easier to see if an issue is related to a 4.x, 5.x or future release. 🤞

Thanks! I think it's worth noting that the naming changed also. After a lot of searching I discovered that it's createm... and not getm... it's not mentioned anywhere. Thanks for this.

Solution: https://github.com/kreait/firebase-php/issues/641


[usm] Refactor Kafka encoder

What does this PR do?
Refactor the Kafka encoder by leveraging changes introduced in https://github.com/DataDog/datadog-agent/pull/16844
Motivation

Additional Notes

Possible Drawbacks / Trade-offs

Describe how to test/QA your changes

Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[ ] If applicable, the config template has been updated.

Rebased on main

Before we merge this, is there a load testing cluster we can trial this on ?

I tested in a load test, the results appear in the PR description


New JSON Tools settings make it very difficult to use

Hi,
I've always really liked this plugin, as it can be used for linting, formatting, compressing, viewing etc.
But the newer versions have changed the settings (default settings?) whereby it's really tricky to use now.
If using this for linting, by default settings means it just deletes stuff it doesn't like (json format errors).
I wouldn't have thought there would be anyone using this plugin that would want that!
Used to show where the error was, so it could be edited and fixed.
Also now the new default seems to be to sort the keys. JSON isn't primarly meant for human reading.
There are certainly times where sorting keys could be handy, but this should be a separate shortcut key (more of an edge case e.g sorting a long array list of labels).
Auto-sorting when doing data structures often (usually?) makes the json less readable (e.g. putting a "meta" object in the middle instead of at the top where it was originally placed).
Thank you for making/maintaining this excellent plugin. If the defaults can be made less 'heavy handed' that would be awesome!.

@00155837364765867

Also now the new default seems to be to sort the keys.

You are not using the most recent version of JsonTools. JsonTools has had that setting off by default since v7. Rather than address your complaints one by one, I would suggest that you upgrade to v8.2 and see whether you are satisfied.

If using this for linting, by default settings means it just deletes stuff it doesn't like (json format errors).

More recent versions of JsonTools have an error form and a Check JSON syntax now command that refreshes the error form and shows any errors that were found without reformatting the file.

@00155837364765867
Since you have given me no reason to believe that you have any issues that haven't been fixed in my most recent release of JsonTools, I'm closing this issue. If you have an issue that's not related to the ones you described in your initial comment, you should probably open a new issue (or comment on an existing issue if yours is related), rather than commenting on this issue.

This documentation, should help you install a newer JsonTools release manually. I recommend v8.3.1.


Demand advice

Can you add a WalletConnect scan method?

Not sure what you want. Can you be a little more specific?

Not sure what you want. Can you be a little more specific?

Like this
https://community.trustwallet.com/t/how-to-use-walletconnect-with-trust-wallet/36247

Sorry, I just seem to have seen what I need


Add client side node balancing

Currently when I refresh my assets page, all assets are initially at 0 in default order. Then when the response is obtained from the server + wildly resorted based on the assets i have.
It would be ideal if we could at least have the assets I have, be kept in local storage during refreshes. Waiting for the numbers to fill in at least feels more natural, but the sort order vastly changing is odd.

This may add a lot of complexity to maintain. IMO, we should work on adding more/better query nodes so data loads sooner.
@pyramation and I have been in talks of some sort of FE js lib to load balance traffic to available and performant nodes, or, less preferably, sending requests to a proxy load balancer service.

Perhaps, we can go further and not fallback to a provider, but generate an ideal provider for the client. Perhaps based on proximity, load, etc.

Repurposing issue

tracking in telescope as well https://github.com/osmosis-labs/telescope/issues/167

maybe not relevant to this issue in context for FE, but for backend proxy, https://github.com/eco-stake/cosmos-directory is open source and @tombeynon mentioned we could deploy a server with cosmos-directory on it

There are pros and cons to both approaches. But IMO I think lean toward a client-side solution since it would be more decentralized and would also mean less infra to maintain.

Another option could be using an API like https://status.cosmos.directory/osmosis to enhance your client side load balancing, that's essentially the data the proxy uses to load balance. That API is public/open source, but it's pretty trivial to implement manually if preferred.
CORS was the other reason I moved to a proxy instead of client side load balancing, but I'm relying on chain registry nodes for REStake so have less control of that side of things.

a hybrid approach could be to do client-side loadbalancing with keplr endpoint as the osmosis-fe main endpoint, and then we can fallback to cosmos.directory. This gives us best of both frontend and backend load balancing.
If we did that, we'd likely want to test out how much traffic we'd anticipate sending to @tombeynon's server and if we should host our own.

Stale


feat: Add clickable links for Table

Proposed API
Set data_type to 'link' and use Markdown notation [label](url)
 ui.table(
        name='table',
        columns=[
            ui.table_column(name='link', label='Link', data_type='link')
        ],
        rows=[
            ui.table_row(name='row1', cells=['[Wave](https://wave.h2o.ai/)'])
        ]
    )

I tried to use ui.link as suggested by @mturoci but I didn't find a way to pass label to it because cells only accept strings. I wish I could do something like cells=[ui.link(label='my label', path='https://website.com')].
/** Create a table row. */
interface TableRow {
  /** An identifying name for this row. */
  name: Id
  /** The cells in this row (displayed left to right). */
  cells: S[]
}

It would be good to also think about supporting links that will be able to also open external pages in new tabs (https://github.com/h2oai/wave/issues/567) and allow download as well. Also use TableCellType instead of data_type as this is a specific type of cell, not a type of data, it's a string after all.

This is a good direction. Let's do the following changes:

URLTableCellType -> LinkTableCellType
new_tab -> target - for consistency with ui.link

Should it default to always open in a new tab?

No, the default should be the same as ui.link.

Should I remove the current behavior if the user doesn't provide a URL with http(s) prefix?

Can you provide a link? I was not able to find the string you quoted in table.md

I thought that maybe it could be used for routing to different apps in the same domain, so the user wouldn't have to pass the full domain.

It should work like a common web routing (relative links). I see your point, but that would prevent people who want to just append to the current URL (for some reason). Specifying a full domain for routing to a different app on the same server is fine.

If there is a need to provide row-specific actions, there's already the other PR with the contextual menu

That PR implementation has every row with the same actions. We can discuss further improvements there.

Is there a link to the specific feature request or issue?

I couldn't find any issue or discussion opened by @vopani. @mturoci by any chance, do you remember what was the usecase?

If the need is to support links on each row that open in new windows, then the request is too specific, and the implementation/API seem hacky (regex, markdown-subset, etc.)

@lo5  It seems hacky indeed. What do you suggest as an alternative?

What do you suggest as an alternative?

Hard to say, since I'm not sure about the origin of this requirement.
One option is to have a MarkdownCellType which interprets cells as markdown. This opens up the possibility of using arbitrary formatted content (including links, e.g. [this](/foo) or [that](bar)). Implementation-wise, this can use the same markdown processor as ui.text(), with maybe some niceties like removing margins if p:only-child.
The decision on whether to open links in _blank can be based on whether the <a href= is an absolute (open in _blank) or relative URL (open in same window).
I'm not sure if the table already supports markdown formatting.

The original issue is #428 (found via the branch name) and the requirement is to be able to open the links right from the table. I am fine with MarkdownCell, but make sure the spacing is correct and does not interfere with the rest of the table.

The original issue is https://github.com/h2oai/wave/issues/428 (found via the branch name)

I meant where this request came from. If I see "via " I assume it comes from somewhere else, GH discussion, another GH issue, or a private conversation. #428 doesn't describe a use case. Like @lo5 said, maybe we already support the use case since it's an old feature request.

I am fine with MarkdownCell, but make sure the spacing is correct and does not interfere with the rest of the table.

Ok

Markdown doesn't support setting the target to open in a new tab.

Convert markdown to html, insert into DOM, use querySelectorAll() on the ref to get all a elements, inspect the paths, add an onClick handler to the ones that need to open in _blank.

Convert markdown to html, insert into DOM, use querySelectorAll() on the ref to get all a elements, inspect the paths, add an onClick handler to the ones that need to open in _blank.

This could also work but I wonder if there is any benefit over using regex and XLink component if the provided string is a link.

I'd prefer not to encourage using HTML.

Ok.

Not sure what you mean by this. The markdown used in a table cell should be no different from that used elsewhere.

Open in a new tab when http(s):// is present is not part of the Markdown specification, so my question was whether we need to document only this behavior and also point the user to some online reference in case they are not familiar with Markdown in general.

This could also work but I wonder if there is any benefit over using regex and XLink component if the provided string is a link.

The main benefit is that the technique I suggested would support arbitrary markdown, not just links.

my question was whether we need to document only this behavior

Add an attribute to MarkdownCellType that specifies whether or not external links should be opened in _blank. Default to opening all links in the same window.

Could you please clarify this point? Don't we want to only extend the markdown link feature to support opening in a new tab? Maybe I'm missing something.

The goal is to support arbitrary links in table cells. Links can be opened in the same tab or in a new tab. Agreed?
You're suggesting using regex and XLink. I'm saying that's hacky and error-prone.
The way I see it, there's not much of a difference between supporting hyperlinks, and supporting arbitrary Markdown inside cells.
If we're only going to support hyperlinks, it raises more questions:

What if there are 1+ hyperlinks in a cell?
What if the hyperlinks are accompanied by text?
What if the text requires alternate formatting?

So I don't see a good reason to support only hyperlinks. Might as well support Markdown, since Markdown already supports hyperlinks. Hence, MarkdownCellType.
That leaves one question open: do we open links in the current tab or a new tab? To which I suggested that we leave that decision to the user by adding an attribute (flag) to MarkdownCellType that specifies whether or not external links should be opened in _blank. And if they are to be opened in _blank, I suggested using the ref/querySelectorAll() technique, since that is the most robust way to implement this.
Hope this makes it clear. If not, call me.

I  dynamically added the target attribute instead of a click handler like you suggested.

Good idea.

Should we wrap MarkdownCellType by default?

Yes

Do you know any other case Markdown would break the cell?

There might be, but should be enough to handle the p tag. Blockquotes, tables, headings, etc. don't make sense inside table cells anyway.


AKS cluster can't be restored after deletion, could I know if there can have an answer in our FAQ?

[Enter feedback here]
I know the AKS cluster can't be restored after deletion, could I know if there can have an answer in our FAQ?

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

ID: d501ff67-9208-8bcb-e8df-f09e9b16a802
Version Independent ID: e3fc3851-ddc4-d26c-251d-6ae53d714081
Content: Frequently asked questions for Azure Kubernetes Service (AKS) - Azure Kubernetes Service
Content Source: articles/aks/faq.md
Service: container-service
GitHub Login: @zr-msft
Microsoft Alias: zarhoads

@ZHIXIN-SUN
Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate.

@zr-msft
Can you please check and add your comments on this doc update request as applicable.

Hi @ZHIXIN-SUN - thanks for your feedback! I'm currently working on updates to the AKS FAQ and have included your feedback, adding the question and answer for whether you can restore your cluster after deletion (which, as you said, you cannot). The updates will be live soon. I will now close this GH issue. Thank you again.
#please-close


Merge to publish

:x: Validation status: errors

File
Status
Preview URL
Details

:x:Error

Details

Line 16: [Error] Restore nuget package W:\5poy-s.optemp\packages\opbuild.scripts.1.46.2\tools\packages.config in folder:W:\5poy-s.optemp\packages failed, possibly due to transient errors from nuget servers. Please retry building your content. If the issue still happens, open a ticket in http://SiteHelp. Detailed error: The request was aborted: Could not create SSL/TLS secure channel.

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.

:x: Validation status: errors

File
Status
Preview URL
Details

:x:Error

Details

[Error] Run script failed with no detailed information. Please retry building your content again. If the issue still happens, open a ticket in http://SiteHelp and include the error report for our team to troubleshoot

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.

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/add-ins/add-in-commands-for-outlook.md
:white_check_mark:Succeeded

docs/add-ins/addin-tutorial.md
:white_check_mark:Succeeded

docs/add-ins/contextual-outlook-add-ins.md
:white_check_mark:Succeeded

docs/add-ins/images/outlook-mobile-design-cell-donts-android.png
:white_check_mark:Succeeded

docs/add-ins/index.md
:white_check_mark:Succeeded

docs/add-ins/outlook-addin-design.md
:white_check_mark:Succeeded

docs/add-ins/outlook-mobile-addins.md
:white_check_mark:Succeeded

docs/add-ins/quick-start.md
:white_check_mark:Succeeded

docs/add-ins/use-rest-api.md
:white_check_mark:Succeeded

docs/add-ins/web-services.md
:white_check_mark:Succeeded

docs/index.md
:white_check_mark:Succeeded

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


Raylib and possible vendor improvements

I was investigating cross compilation and decided to try out the vendor raylib. I had a problem with linking glfw, so i looked at the this configuration:
"linux-x64" : {
  "link-args" : [],
  "dependencies" : [],
  "linked-libraries" : ["raylib", "GLESv2", "glfw", "c"]
},
"windows-x64" : {
  "linked-libraries" : ["raylib", "opengl32", "kernel32", "user32", "gdi32", "winmm", "winspool", "comdlg32", "advapi32", "shell32", "ole32", "oleaut32", "uuid", "odbc32", "odbccp32"],
  "wincrt": "none"
},

Considering the pre compiled library available at the vendor is compiled without any modifications on the options on the build or even obtained by the Releases in github from raylib itself, then linking against glfw is not needed as it's already compiled along the library. The same thing would apply to the windows target, with the exception of two dependencies which they specify need to be linked: GDI32 and WinMM. Check here.
Those two targets then would look like this:
"linux-x64" : {
  "link-args" : [],
  "dependencies" : [],
  "linked-libraries" : ["raylib"]
},
"windows-x64" : {
  "linked-libraries" : ["raylib", "gdi32", "winmm"],
  "wincrt": "none"
},

Specifying to link extra libraries doesn't yield an error given that you have those in the system, however it's best in my opinion to minimize the extra efforts if possible, specially because linking with GLESv2 doesn't seem to change anything either. I'm not sure what is "wincrt": "none", but it seems to be required.
The second error i observed was in the naming of the .libs for windows when compiling in linux, for instance:
lld-link: error: could not open 'gdi32.lib': No such file or directory
lld-link: error: could not open 'winmm.lib': No such file or directory
lld-link: error: could not open 'dbghelp.lib': No such file or directory
lld-link: error: could not open 'kernel32.lib': No such file or directory
lld-link: error: could not open 'user32.lib': No such file or directory
lld-link: error: could not open 'Shlwapi.lib': No such file or directory
lld-link: error: could not open 'Ws2_32.lib': No such file or directory
lld-link: error: could not open 'MSVCRT.lib': No such file or directory
lld-link: error: could not open 'OLDNAMES.lib': No such file or directory
lld-link: error: could not open 'uuid.lib': No such file or directory
Failed to create an executable: (null)

Those libs exist, considering we ran the python script to create the directory, but the problem are that the names are matching with case sensitive. Renaming the .libs in the directory made it work. So we either make the script convert the names for us, or we the compiler grabs those with case insensitive.
I'm not sure if it is as intended, but the vendor-fetch on linux yielded this error:
Error: vendor-fetch only available when compiled with cURL.

Would be great if there was a way to know the vendor options. It would also be good to allow us to specify different compiled library options besides the one packed with the vendor, for example if i compiled raylib to use a different opengl version and want to use that instead.
Compiler info:
C3 Compiler Version:       0.6.4 (Pre-release, Nov  5 2024 09:51:35)
Installed directory:       /home/ygor/c3_lang/
Git Hash:                  08d1b29301eb057494a5b6c59278200f0db33ad4
Backends:                  LLVM
LLVM version:              17.0.6
LLVM default target:       x86_64-pc-linux-gnu

I think the linked libraries are taken from the examples for raylib. If there is a smaller set then that is fine. Just make sure it's not because those libraries are linked from other sources anyway.
Re the windows-on-linux issue: fixing the case by doing it when copying in the python script seems to be the only solution from what I'm hearing on the discord. It's worth filing an issue for it.
As for vendor-fetch not being present: this is if it's compiled without curl being found on the system.

As far as i'm aware the deps are all bundled into the lib, with the exception of the two on windows i mentioned. Someone else could help verify this as well.
I'll create an issue for it.
I didn't compile it myself, i downloaded from the releases. Could it be a problem there?

On the topic of raylib improvements, could it be possible to ditch the Vector2 raylib structure, and just use builtin float[<2>]? it should? and would make code allot easier to write without having to always write a conversion function and call it etc.

No, it's important in order to get ABI compatibility. Although I think replacing the Vector2 structure with float[2] would be possible, and float[<2>] are implicitly convertible between each other. If you want some convenience right now though you can use splat:
float[<2>] vec = ...
call_with_vector2({ ...vec })


Make paranoid and use idiomatic bash

Minor tweaks of the launch.sh script to use idiomatic bash, and to make it more paranoid w.r.t. variables with spaces in them.

@benschumacher Please update your fork, then I'll review it for @stevendborrelli

@benschumacher @stevendborrelli This PR is out of date and some of the suggestions have been implemented in other PRs. I suggest that this PR should be closed.


Don't set startAtOperationTime if resumeToken is available

Avoid sending startAtOperationTime on cursor reconnects if resumeToken is available
What changed?
This is basically a duplicate of JAVA-2987 but in the node driver.  When the cursor tries to reconnect, it has an internal resumeToken value that it has acquired while iterating over earlier changes.  If a startAtOperationTime was set when the cursor was originally created, it will also be sent and the server will respond with "Only one type of resume option is allowed, but multiple were found"
Are there any files to ignore?
No

The node 10 tests are failing but I suspect it's a fluke as it seems totally unrelated to the proposed changes (I may be wrong).  It doesn't seem that I can retry/restart the build short of committing another arbitrary change...  Tests in the same environment pass locally.

For anyone that runs across this, I'm currently hacking around it by doing the following after calling collection.watch()
setTimeout(() => { changeStream.options.startAtOperationTime = null; } }, 500);

@timjnh Thank you for putting up this PR! You are correct, the test failures appear to be unrelated.
The reason that we were not originally preventing this is because there is a requirement that if a user specifies both values, we must pass both of them on and let the server return an error. See the specification here:

startAtOperationTime, resumeAfter, and startAfter are all mutually exclusive; if any two are set, the server will return an error. Drivers MUST NOT throw a custom error, and MUST defer to the server error.

I am curious if the following will fix your error: instead of using the if...else, you could add an extra line here after the resumeToken is cached to remove the startAtOperationTime:
changeStream.resumeToken = change._id;
changeStream.options.startAtOperationTime = undefined;

Does that change fix your issue?

@daprahamian That seems like a much better solution.  PR updated.

merged. Thank you for the pr @timjnh!

Thanks for the help @daprahamian !


Improve insert method used

Overview
As noted in #305 there has been a long-standing need to add the column names into the insert statements for each table to make upgrading easier and more robust. This adds those column names along with some improvements to the macros folder:

Addition of sub-folders to make identifying macros easier
Addition of macro documentation
Tidy up of upload_results script to make it more DRY

Additionally picked up this very related issue:

Fixing issue related to quoting in Snowflake (as noted in #360 )

Update type - breaking / non-breaking

[ ] Minor bug fix
[ ] Documentation improvements
[x] Quality of Life improvements
[ ] New features (non-breaking change)
[ ] New features (breaking change)
[ ] Other (non-breaking change)
[ ] Other (breaking change)
[ ] Release preparation

What does this solve?

Fixes #305
Fixes #360

Outstanding questions

What databases have you tested with?

[x] Snowflake
[x] Google BigQuery
[x] Databricks
[ ] Spark
[ ] N/A

Much appreciated


Don`t work vueMultiSelect field

Hello!
Try to use vueMultiSelect field. vueMultiSelect installed via npm correctly.
import VueFormGenerator from "vue-form-generator";
import Multiselect from 'vue-multiselect'

Vue.component("FieldVueMultiSelect", Multiselect);

Vue.use(VueFormGenerator);

Vue.use(Vuedals);

const app = new Vue({
  el: '#app',
  template: '<App/>',
  router,
  store,
  components: {
    App
  }
})

in schema:
 {
  type: "vueMultiSelect",
  label: 'books',
  model: "books",
  required: true
}

And in runtime i have errors:
'vue-multiselect' is missing. Please download from https://github.com/monterail/vue-multiselect and register the component globally!
[Vue warn]: Unknown custom element: <multiselect> - did you register the component correctly? For recursive components, make sure to provide the "name" option.

found in

---> <FieldVueMultiSelect>
       <VueFormGenerator>
         <CreateProject> at src/pages/projectManager/components/createProjectForm/createProjectForm.vue
           <Vuedals>
             <App> at src/App.vue
               <Root>

And multiselect dont show.
Please, help me! Where can be the issue?

I found the issue. I have to register global component as:
Vue.component("multiselect", Multiselect);


macos installation instructions fail

$ brew install riscv-gnu-toolchain gettext

fails with
Warning: No available formula with the name "riscv-gnu-toolchain".
==> Searching for similarly named formulae and casks...
Error: No formulae or casks found for riscv-gnu-toolchain.

@yorhodes Can you try
$ brew tap riscv-software-src/riscv
$ brew install riscv-tools

?


doodle: 0.7.2 -> 0.7.3

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/doodle/versions.
meta.description for doodle is: Tool to quickly index and search documents on a computer
meta.homepage for doodle is: https://grothoff.org/christian/doodle/
Updates performed

Version update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.7.3 with grep in /nix/store/kzqs60fk19gbpj8zcpjbn11y5ibmdg30-doodle-0.7.3
found 0.7.3 in filename of file in /nix/store/kzqs60fk19gbpj8zcpjbn11y5ibmdg30-doodle-0.7.3

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

doodle

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/405373238923351208e4278b71be9f3ffddfc874#doodle

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

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

1 package built:

doodle

Maintainer pings
cc @viric for testing.

lgtm


regen with the latest gir & bump versions

A new release is required for webkit2gtk-rs/crate future merge request

Thanks.

Do you mind pinging me once there's a new release? thanks!

@bilelmoussaoui Done.


Unit Test Failures

Mod source
Github
Which mod are you having an issue with?

[X] pyalienlife
[X] pyalternativeenergy
[X] pycoalprocessing
[X] pyfusionenergy
[X] pyhightech
[X] pyindustry
[X] pypetroleumhandling
[X] pypostprocessing
[X] pyrawores

Operating system

=Windows 10

What kind of issue is this?

[ ] Compatibility
[ ] Locale (names, descriptions, unknown keys)
[ ] Graphical
[ ] Crash
[ ] Progression
[ ] Balance
[X] Pypostprocessing failure
[X] Other

What is the problem?
I ran Angel's Unit Tests against the Py mod suite. I wanted to look for issues which might confuse Helmod as well as establish a base line so I can test if adding some of Bob's mods to a Py game breaks anything.
Steps to reproduce
Enable Py Alternative Energy + prerequisites.
Additional context
Vanilla recipes
Vanilla techs Nuclear fuel reprocessing and Koverex enrichment process have been hidden. However the recipe unlocked by those techs have not been marked as hidden.
Please mark the following techs as hidden:

nuclear-fuel-reprocessing
kovarex-enrichment-process

Possible missing science packs

Military 2 depends on Logistic Science Pack tech but only requires Automation and Py science pack 1
The following techs depend on Quantum computing tech but don't require Space science pack

Atomizer upgrade
Biotechnology machines - Stage 4
Botany - Stage 4
Dingrits upgrade
Land animals - Stage 4
Machines 5
Mycology - Stage 5
Oil industry machinery - Stage 4
Power armor MK2
Smelters 4
T.U.R.D. reset 1

Accessible techs unlocking hidden recipes

Tech nexelit-mk01 unlocks recipe digosaurus-hidden-recipe which is hidden.
Tech thermal-mk01 unlocks recipe lrf-steam-1 which is hidden.
Tech thermal-mk02 unlocks recipe lrf-steam-2 which is hidden.

Techs unlocking a recipe twice

Recipe thickener-mk01 is unlocked twice by Tech vanadium-processing
Recipe gas-separator-mk01 is unlocked twice by Tech sulfur-processing
Recipe medium-distillate-to-stripped-distillate is unlocked twice by Tech light-oil-mk02
Recipe "fish-to-tin" is unlocked twice by Tech molecular-decohesion-mk02

Log file
No response

202307172328.txt
Attached is the full unit test log file. I tried to pick out and explain the most relevant parts.


Dependabot run doesn't time out (hangs)

Is there an existing issue for this?

[X] I have searched the existing issues

Package ecosystem
maven
Package manager version
N/A
Language version
Java 21
Manifest location and content before the Dependabot update
https://github.com/arquillian/arquillian-extension-drone/network/updates
dependabot.yml content
N/A
Updated dependency
N/A
What you expected to see, versus what you actually saw
Dependabot job hangs and runs forever, cannot cancel
Native package manager behavior
No response
Images of the diff or a link to the PR, issue, or logs
No response
Smallest manifest that reproduces the issue
No response

timed out after an hour


How to add support for immutable js?

When using for example Map() object from immutable.js , Chrome throws an error:
Uncaught TypeError: Constructor Map requires 'new'

You probably for to import Map from Immutable because the built-in Maps in JavaScript need to be instantiated with 'new'
import { Map } from 'immutable'

You're probably not using a supported browser. I'd recommend using Babel

Don't this is related to this project.


replacing outdated haskell action

The depreciated haskell/actions/setup has now broken; all CI will need this change.
PR checklist:

[ ] Test coverage for the proposed changes
[ ] PR description contains example output from repl interaction or a snippet from unit test output
[ ] Documentation has been updated if new natives or FV properties have been added. To generate new documentation, issue cabal run tests. If they pass locally, docs are generated.
[ ] Any changes that could be relevant to users have been recorded in the changelog
[ ] In case of  changes to the Pact trace output (pact -t), make sure pact-lsp is in sync.

Additionally, please justify why you should or should not do the following:

[ ] Confirm replay/back compat
[ ] Benchmark regressions
[ ] (For Kadena engineers) Run integration-tests against a Chainweb built with this version of Pact

Looks like this is already fixed in a branch Lars and others are working on, that fixes more details as well.  Closing this and deleting branch.


[SPARK-32415][SQL][TESTS] Enable tests for JSON option: allowNonNumericNumbers

What changes were proposed in this pull request?
Enable two tests from JsonParsingOptionsSuite:

allowNonNumericNumbers off
allowNonNumericNumbers on

Why are the changes needed?
To improve test coverage.
Does this PR introduce any user-facing change?
No
How was this patch tested?
By running the enabled tests.

Test build #126422 has started for PR 29207 at commit c2a27f5.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/31036/
Test PASSed.

Test build #126422 has finished for PR 29207 at commit c2a27f5.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/126422/
Test PASSed.

Merged to master.


Return the original string if no modifications were made

Either do html.to_string() or return Cow to avoid allocations

Quite unlikely scenario, not worth changing the API


update chron so that the action runs on the 1st and 15th of every month

@TheJuanAndOnly99 merged, thank you!
Could you please set a reminder for the 2nd of June and make sure the action was triggered?
I'd also suggest to apply the same change on the legend juju integration repo.
Thanks!

@maoo PR sent https://github.com/finos/legend-integration-juju/pull/76 and remainder set.


pkg/osutil: fix arm build

pkg/osutil/osutil_linux.go:44:13: cannot use info.Totalram (type uint32) as type uint64 in return argument

Codecov Report

Merging #1399 into master will not change coverage.
The diff coverage is 0%.

@@           Coverage Diff           @@
##           master    #1399   +/-   ##
=======================================
  Coverage   57.82%   57.82%
=======================================
  Files         131      131
  Lines       24849    24849
=======================================
  Hits        14368    14368
  Misses       9812     9812
  Partials      669      669

Impacted Files
Coverage Δ

pkg/osutil/osutil_linux.go
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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

LGTM


Suggestion - Item Notes

It would be nice to have notes/descriptions/stories for items. Even if it's just a data scraper note, or a very short story, I think it adds alot to the immersion. Thanks :)

Hey, I would like to help with addition of notes. How should I contact you regarding this?

You can message me on discord at shirograhm!


AWS - EC2 (instances, EBS, load-balancing, NAT)

Supported resources:

[x] aws_instance
[x] aws_launch_configuration
[x] aws_launch_template
[x] aws_autoscaling_group
[x] aws_ec2_fleet
[x] aws_ebs_volume
[x] aws_ebs_snapshot
[x] aws_ebs_snapshot_copy
[x] aws_alb / aws_lb
[x] aws_elb
[x] aws_nat_gateway

All of these are supported. Some more testing would help.


CORS issue when using ollama

Please help me, thanks!
when using "Rebuild entire vault index", I got the error message from console "CORS"

I start Ollama from windows terminal using command " $env:OLLAMA_ORIGINS="app://obsidian.md*"; ollama serve", and the terminal output is

Also, I tried add environment variable "OLLAMA_ORIGINS" and "OLLAMA_HOST" to system and reboot windows

but still not working.
And I tried "LocalGPT plugin" and test it, it's OK.
How to fix this?

Please help me, thanks!
when using "Rebuild entire vault index", I got the error message from console "CORS"
I start Ollama from windows terminal using command " $env:OLLAMA_ORIGINS="app://obsidian.md*"; ollama serve", and the terminal output is
Also, I tried add environment variable "OLLAMA_ORIGINS" and "OLLAMA_HOST" to system and reboot windows
but still not working.
And I tried "LocalGPT plugin" and test it, it's OK.
How to fix this?

My computer is windows 11 home x64 edition
Obsidian: 1.77
Smart Composer:1.0.11
Ollama: 0.5.4

This is the smart composer config.

Thanks for the report. Normally, Ollama models should work fine without needing to set any environment variables at all.
Could you please:

Share a screenshot of your Obsidian console showing the exact CORS error message.
Let us know if the chat or apply models are working correctly, and only the embedding model is failing.

I’ll test this on Windows later, but so far I haven't been able to reproduce the problem.

This is the console output.
I noticed that url is "http: //localhost:11434//v1/embedding", There's an extra slash here。
so I config the base URL in smart composer to "http://<IP_ADDRESS>:11434", not appending '/'.
Rebuild vault index again,and console output is "POST http://<IP_ADDRESS>:11434/v1/embeddings 404 (Not Found)".
It seemed that ollama api is changed:

Generate Embeddings

I don't know why, but it works on my machine.
I updated ollama to latest and still works.
May I know your ollama version and which model do you use? You can check ollama version by ollama -v

I don't know why, but it works on my machine. I updated ollama to latest and still works. May I know your ollama version and which model do you use? You can check ollama version by ollama -v

My ollama version 0.5.4 and model is llama3.1:8b


feat(state_table): add upsert method for StateTable, and use it in managed value state

I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?
Managed value state view state table as a one-row table, it update the single row in flush previously with state_table.insert. However, two problems exist, 1) insertion is not correct in term of semantics, 2) insert disallows multiple insertion with the same pk within one epoch, which prevent us from insert the output into state table during apply_chunk.
In this PR, upsert method is added to solve these problems.
This is a step towards the removal of is_dirty and flush.
Checklist

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Refer to a related PR or issue link (optional)
#4035

Would you please give a simple benchmark of the performance degradation?

Would you please give a simple benchmark of the performance degradation?

Agreed. That will be better

Maybe we should also add a upsert RowOp in Memtable and do not do the SANITY_CHECK in StateTable::batch_write_rows

As the sanity check for aggregation is not enabled yet, I think we may do this later.

Benchmark
Running create materialized view mv as select count(*) as c from person group by city; on NexMark schema, gives the following results:
main branch

this branch

The throughput is not stable on main branch, but for average throughput, this PR faces a slight degradation.

Based on some offline discussions, we may hold this PR for a while as the strategy introducing in #5290 will guarantee that states updated in current epoch won't be evicted.

Can we close this PR as https://github.com/risingwavelabs/risingwave/pull/5565 merged?

As #5565, which simply adds upsert, has been merged, and we are doing a new refactoring on branch sts/refactor_proto_stream_agg that can remove is_dirty and flush without performance impacts, this PR can be closed.


expected range for gws incorrect

*Term to change: gsw
Submitter: Kim
Justification: value wrong order of magnitude
I suggest the following changes (leave blank whatever would not change):

Term name:
Definition of the term:
Format of the term:
Additional Instructions (recommendations regarding content, etc.): change max range from 100 to 1000
Examples:
Refines (name of the broader term this term refines, if applicable):
Replaces (name of the existing term that would be replaced by this term, if applicable):

Change made.


testing a theory

Docs Build status updates of commit b91deb5:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs-ref-conceptual/index.yml
:white_check_mark:Succeeded
View

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.


Breaking links and typos

Commit Message: to fix broken link and typos in docs
Additional Description:
Risk Level:Low
Testing: Unit and format check
Docs Changes: Yes
Release Notes:
[Optional Runtime guard:]
[Optional Fixes #11215
[Optional Deprecated:]

Thank you for finding and fixing small things like this. It's greatly appreciated!
I see you have a separate PR open for config fragments, and I think this PR contains some of the same change as that one. Did you mean for just the lastest commit in this PR to be included here?

@ggreenway Hi , Yes. those PRs for separate issues #10843  i am working on.

@ankatare it might be easier if you could specify which PR should be reviewed first, if they are overlapping.

@jmarantz really sorry for any confusion here, this particular PR is intended for fixing docs issues.

@jmarantz really sorry for any confusion here, this particular PR is intended for fixing docs issues.

If you look at the diff of this PR, it currently includes the v2->v3 config changes, as well as the doc fixes. To move forward, please separate the two, so this PR only contains the diffs for the doc fixes.

@ggreenway Should i close this PR and create fresh one so that it contains changes wrt doc fixes only. OR is there any other way to proceed with this PR.

@ggreenway Should i close this PR and create fresh one so that it contains changes wrt doc fixes only. OR is there any other way to proceed with this PR.

That is probably easiest.

@ggreenway Ok, Thanks. will do the same

closing this PR due to confusions from commits. will push changes in new PR.


Update init scripts to install openJDK 8

Jenkins 2.60 requires Java 8, so the agents should also have this version (it's backwards compatible, so older Jenkins master will still work)

I've tested and switched to Ubuntu 16.04 LTS as agent template. It includes java 8 and works well with old master. This could be another option.


Saving a recipe doesn't always show available cookbooks.

What happens:
When a user is signed in, and has a cookbook, the save recipe button doesn't always show their available cookbooks. Instead it shows the no cookbooks found warning
What should happen:
If a user has cookbooks, it should that it is loading or list the cookbooks.

Why the bug is happening:
When you launch the app and go to the profile page first, and then try save a recipe to a cookook -> then you get the warning message.


Test

Depends-On: https://github.com/openstack-k8s-operators/ci-framework/pull/528
Depends-On: https://github.com/openstack-k8s-operators/install_yamls/pull/522

recheck


[Bug] After clearing the data, mqttx cannot find the log file

what did I do
cleaned the user data. Then when I close the MQTTX, the error report.
what happened
get an error
environment

MacOS Big Sur 11.1
MQTT x v1.5.0

log/Screenshot/context

Fixed on https://github.com/emqx/MQTTX/releases/tag/v1.5.1

Fixed on https://github.com/emqx/MQTTX/releases/tag/v1.5.1


Missing encryption key when loading bucket SSE settings from DB

Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:

/kind bug

/kind feature

What happened:
The PR was closed and the code was re-written and a new PR raised.
In this, missed the code to load the encryption key from DB, to be used during a PUT to encrypt and to decrypt during a GET
What you expected to happen:
PUT object will upload encrypted object, GET object will allow download of decrypted object
How to reproduce it (as minimally and precisely as possible):
Anything else we need to know?:
Environment:

Gelato(release/branch) version:
OS (e.g. from /etc/os-release):
Kernel (e.g. uname -a):
Install tools:
Others:

fixed in #833


Architecture docs: Deployment options

Project "Let's write down what is only our heads":
I basically digged into some old internal project reports, when designing the existing kuksa.val security architecture and translated them/modernized the terms.
This is more a system-level guideline how you can use kuksa.val and what are intended/thought of use cases.
@lukasmittag Please read/check for typos and general "understandability" -> does it make sense generally
@erikbosch  Your well-trained VSS eye might help -> is this useful/logic for the VSS afficionado?

Remade all pictures according to the style and naming convention in ./terminology , updated the text with the suggestions and also current terminology


Update to 0.86.0

Based on changelog here I'd identify the following things to be done:

[ ] 0.81.1 - Addition of st.form() and st.form_submit_button()
[ ] 0.81.1 - Addition of st.caption()
[ ] 0.84.0 - Addition of st.session_state()
[ ] 0.86.0 - You can now use st.columns, st.container and st.expander without the beta_ prefix.

Thanks for the pull request @arnaudmiribel! Very useful. Now merged, so issue fixed.


iOS build support

This shouldn't be too difficult. Just need to target the build for iOS and then create an ipa instead of a .app (the structure seems pretty similar in some ways.

stackotter/delta-client#13

This is in the process of being implemented in the sponsor-only version of Swift Bundler (see https://swiftbundler.dev).

And it's mostly working, so I'll close this issue.


TEST: Enable and fix test_bn2padded() in test/bntest.c

This looks like old code, written when the padded variety of BN_bn2bin()
was developped, and disabled by default...  and forgotten.
A few simple changes to update it to the current API is all that was
needed to enable it.

I see this as a bug in test/bntest.c, we forgot to update and enable the test code when the BN_bn2binpad() function had matured.  Therefore, this should go to 3.0 as well as master.

CI failure is relevant.

CI failure is relevant.

Fixed

Approved for master. I am not sure about 3.0. Probably OTC needs to decide.

Question for OTC to decide: Is adding a new test or fixing and enabling a broken old one a bug fix eligible for stable releases?

Adding a new test is against policy.
Fixing a bug in an existing test ought to be okay -- but if it's not failing is it really a bug?
This feels like it's a new test even though the code was there, it's conditioned away.

For me, it's quite clear cut.  That test was supposed to do something, but did nothing, i.e. it fails to do what it was supposed to do.  Bug.  That it's graceful about its failure to do something doesn't matter in my mind.
But OK, OTC decision in the end.

24 hours has passed since 'approval: done' was set, but as this PR has been updated in that time the label 'approval: ready to merge' is not being automatically set.  Please review the updates and set the label manually.

Merged into master:
23750f677ef61b6bea4e81f23f335ad08fc49b51 TEST: Enable and fix test_bn2padded() in test/bntest.c

OTC: This is allowed for backport to 3.0 and 1.1.1

Backport merges:
3.0:
015e3f59434651c454c94888d0c6d57c2203cd42 TEST: Enable and fix test_bn2padded() in test/bntest.c
1.1.1:
162bd56e99b2e73cfdc6777acb3f1b3dafccc9ba TEST: Enable and fix test_bn2padded() in test/bntest.c


Framy Hyprland with external monitor

Hyprland Version
Hyprland, built from branch HEAD at commit d74607e414dcd16911089a6d4b6aeb661c880923  (props: bump ver to 0.33.1). Tag: v0.33.1
Bug or Regression?
Bug
Description
I'm using Hyprland 0.33 with Nvidia (545.29.06) and having this issue. My frame rate would dip to below 30Hz with external monitor connected and worsen with any movement on the screen. Without external monitor, 0.33 works flawless.
Version 0.32.3 works fine with external monitor connected.
How to reproduce
Install hyprland / hyprland-git
Crash reports, logs, images, videos
0.33

0.32.3

git bisect please

git bisect please

I'm sorry I'm bad at this. How to do bisect?

build from source
git bisect good tags/v0.32.3
git bisect bad main

keep building and use git bisect good/bad until it finds the commit that causes it

Still not sure how to do bisect so I cloned the repo and run bisect (is this correct?) @MightyPlaza
~/Downloads/Hyprland    git bisect good 'v0.32.3'                                                                                Wed 27 Dec 2023 11:55:20 AM +08
You need to start by "git bisect start"

Do you want me to do it for you [Y/n]?
status: waiting for both good and bad commits
status: waiting for bad commit, 1 good commit known
 ~/Downloads/Hyprland    git bisect bad 'v0.33.0'                                                                                1278ms  Wed 27 Dec 2023 11:55:37 AM +08
Bisecting: 57 revisions left to test after this (roughly 6 steps)
[334d0ae31b9b192b6d8c35a40bc83f9076d22b9d] monitor: fix transform matrix calculations for transformed
 ~/Downloads/Hyprland   334d0ae3    git bisect good 334d0ae31b9b192b6d8c35a40bc83f9076d22b9d                                                            Wed 27 Dec 2023 11:55:50 AM +08
Bisecting: 28 revisions left to test after this (roughly 5 steps)
[b80c72c7ddbf79147d64759c8659eb5939d0327e] groupbar: fix crash in renderGradientTo
 ~/Downloads/Hyprland   b80c72c7    git bisect good b80c72c7ddbf79147d64759c8659eb5939d0327e                                                            Wed 27 Dec 2023 11:56:07 AM +08
Bisecting: 14 revisions left to test after this (roughly 4 steps)
[8440a30231ea41a6b1dacc60a28837b265a6efec] input: fix overzealous mouse capture on resize_on_border (#4010)
 ~/Downloads/Hyprland   8440a302  git bisect good 8440a30231ea41a6b1dacc60a28837b265a6efec                                                               Wed 27 Dec 2023 11:56:14 AM +08
Bisecting: 7 revisions left to test after this (roughly 3 steps)
[e496b0f250139b572307d048f3189aa6dde72f18] screencopy: fix detecting gl shm formats
 ~/Downloads/Hyprland   e496b0f2  git bisect good e496b0f250139b572307d048f3189aa6dde72f18                                                               Wed 27 Dec 2023 11:56:21 AM +08
Bisecting: 3 revisions left to test after this (roughly 2 steps)
[68783d904d850df65887adb1bab7eff59943c1ac] screencopy: use buffer format for glReadPixels
 ~/Downloads/Hyprland   68783d90  git bisect good 68783d904d850df65887adb1bab7eff59943c1ac                                                               Wed 27 Dec 2023 11:56:29 AM +08
Bisecting: 1 revision left to test after this (roughly 1 step)
[c6804ccaab1cdd0de5026907e3537ad5362861a9] opengl: fixup blur dirty repaint conditions with solitary
 ~/Downloads/Hyprland   c6804cca  git bisect good c6804ccaab1cdd0de5026907e3537ad5362861a9                                                               Wed 27 Dec 2023 11:56:37 AM +08
Bisecting: 0 revisions left to test after this (roughly 0 steps)
[2d04cb1cc664b8f16a42cae6e4c5dd5848712bf5] input: make fallback layout us
 ~/Downloads/Hyprland   2d04cb1c  git bisect good 2d04cb1cc664b8f16a42cae6e4c5dd5848712bf5                                                               Wed 27 Dec 2023 11:56:42 AM +08
3bb9c7c5cf4f2ee30bf821501499f2308d616f94 is the first bad commit
commit 3bb9c7c5cf4f2ee30bf821501499f2308d616f94
Author: Vaxry<EMAIL_ADDRESS>Date:   Tue Dec 5 00:52:16 2023 +0000

    props: bump ver to 0.33.0

 props.json | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
 ~/Downloads/Hyprland   2d04cb1c 

yes but doesn't seem like you did it properly since you didn't find the bad commit
are you sure you built (again, make sure to git submodule update as well) and tested every commit?

yes but doesn't seem like you did it properly since you didn't find the bad commit
are you sure you built (again, make sure to git submodule update as well) and tested every commit?

Still cant figure out bisect. It failed to build at every commit it gave me.

git bisect gave me this. hopefully i get this right

6f733292bf2dae2adf1e7524546e140a93824033 is the first bad commit
commit 6f733292bf2dae2adf1e7524546e140a93824033
Author: Vaxry<EMAIL_ADDRESS>Date:   Sat Nov 25 17:45:08 2023 +0000

    renderer: nvidia checks and use glFinish on nvidia

    fixes #3952 #3946

 CMakeLists.txt          |  1 -
 src/render/Renderer.cpp | 49 ++++++++++++++++++++++++++++++++++++++++++++++++-
 src/render/Renderer.hpp |  3 +++
 3 files changed, 51 insertions(+), 2 deletions(-)

okay that means our nvidia anti-flicker is doing this. I'll make it toggleable.

try disabling opengl:nvidia_anti_flicker.

Yes that fixes the issue. Tq


Component imports only succeed if folder is named components?

Prerequisites
I have the following code in frontend/components/TestComponent.tsx
import { Component } from "uix/components/Component.ts";

@template(() =>
Hello World!
)
export default class TestComponent extends Component { }

I then create a file frontend/routes/example.tsx:
import TestComponent from "../components/TestComponent.tsx";

function Example() {
    return

        <TestComponent />

}

export default Example;

This works, and the component displays.
Expected
I now move the file:

frontend/components/TestComponent.tsx
-> frontend/foobar/TestComponent.tsx

and update the import:

import TestComponent from "../components/TestComponent.tsx";
->import TestComponent from "../foobar/TestComponent.tsx";

and the component should still display.
Actual
The component does not display.

I get no warning in the backend console.
The browser console says:Loading failed for the module with source<EMAIL_ADDRESS>example:55:39
Loading module from<EMAIL_ADDRESS>was blocked because of a disallowed MIME type (“text/plain”). example
Uncaught (in promise) TypeError: error loading dynamically imported module<EMAIL_ADDRESS>Uncaught TypeError: error loading dynamically imported module<EMAIL_ADDRESS>
and the link<EMAIL_ADDRESS>leads to a page saying Not Found.

Notes
This also happens if the file is in a subfolder of components, e.g. frontend/components/foobar.

@nwLSteam Just to clarify - this error still occurs after restarting UIX?
Can you provide your current UIX version and the command line arguments you used?

UIX v0.1.42 (https://cdn.unyt.org/uix/)

DATEX Core:  v0.0.46 (https://cdn.unyt.org/datex-core-js-legacy/)
Deno:        1.39.4
TypeScript:  5.3.3
V8:          12.0.267.8

I run UIX with uix -wblr.
Restarting UIX fixes the issue. However, after restarting UIX, renaming the foobar dir and changing the import to point to the new dir makes it break again. I have adjusted the issue title accordingly.


ZHA: 'dict' object has no attribute 'split'

Home Assistant release with the issue:

0.93.2
Last working Home Assistant release (if known):
Operating environment (Hass.io/Docker/Windows/etc.):

arch | x86_64
-- | --
dev | false
docker | false
hassio | false
os_name | Linux
python_version | 3.6.8
timezone | Europe/Amsterdam
version | 0.93.2
virtualenv | true

Component/platform:

https://www.home-assistant.io/components/zha
Description of problem:
When clicking Config > ZHA > Add Devices it's not finding anything but shows traceback in the logs (see below)
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
zha:
  usb_path: /dev/ttyUSB1
  database_path: /home/hass/.homeassistant/zigbee.db
  radio_type: deconz
  enable_quirks: true

Traceback (if applicable):
2019-05-25 18:44:03 ERROR (MainThread) [homeassistant.components.websocket_api.http.connection.139969276399120] Error handling message: Unknown error
Traceback (most recent call last):
  File "/srv/hass/lib/python3.6/site-packages/homeassistant/components/websocket_api/connection.py", line 76, in async_handle
    handler(self.hass, self, schema(msg))
  File "/srv/hass/lib/python3.6/site-packages/voluptuous/schema_builder.py", line 267, in __call__
    return self._compiled([], data)
  File "/srv/hass/lib/python3.6/site-packages/voluptuous/schema_builder.py", line 589, in validate_dict
    return base_validate(path, iteritems(data), out)
  File "/srv/hass/lib/python3.6/site-packages/voluptuous/schema_builder.py", line 381, in validate_mapping
    cval = cvalue(key_path, value)
  File "/srv/hass/lib/python3.6/site-packages/voluptuous/schema_builder.py", line 811, in validate_callable
    return schema(data)
  File "/srv/hass/lib/python3.6/site-packages/homeassistant/components/zha/core/helpers.py", line 153, in convert_ieee
    return EUI64([uint8_t(p, base=16) for p in ieee_str.split(':')])
AttributeError: 'dict' object has no attribute 'split'

Additional information:

Are you clicking on "add devices" on top of the ZHA page or are you picking a device from device picker and click add devices on devices page?

@dmulcahey  I can reproduce it in the following way:

Config panel -> ZHA -> Add Devices on top of page
wait till it finishes
click on "Search Again" and I get in the trace:

2019-05-28 22:38:48 DEBUG (MainThread) [homeassistant.components.websocket_api.http.connection.139865744173656] Received {'id': 40, 'type': 'zha/devices/permit', 'ieee': {'isTrusted': True}}
2019-05-28 22:38:48 ERROR (MainThread) [homeassistant.components.websocket_api.http.connection.139865744173656] Error handling message: Unknown error
Traceback (most recent call last):
  File "/home/lex/lex/src/ac-hass/homeassistant/components/websocket_api/connection.py", line 83, in async_handle
    handler(self.hass, self, schema(msg))
  File "/home/lex/.venv/dev/lib/python3.5/site-packages/voluptuous/schema_builder.py", line 267, in __call__
    return self._compiled([], data)
  File "/home/lex/.venv/dev/lib/python3.5/site-packages/voluptuous/schema_builder.py", line 589, in validate_dict
    return base_validate(path, iteritems(data), out)
  File "/home/lex/.venv/dev/lib/python3.5/site-packages/voluptuous/schema_builder.py", line 381, in validate_mapping
    cval = cvalue(key_path, value)
  File "/home/lex/.venv/dev/lib/python3.5/site-packages/voluptuous/schema_builder.py", line 811, in validate_callable
    return schema(data)
  File "/home/lex/lex/src/ac-hass/homeassistant/components/zha/core/helpers.py", line 153, in convert_ieee
    return EUI64([uint8_t(p, base=16) for p in ieee_str.split(':')])
AttributeError: 'dict' object has no attribute 'split'

Same behavior if I calling add devices from "Device Picker". 1st click on add devices works fine, but clicking on Search again produces the above trace.

I'm seeing the same behavior with an Xbee device. I also see warning messages on startup that look like the serial responses from the device are getting lost somehow leading to timeouts. Here's the full log of relevant messages from startup to poking the Config panel -> ZHA -> Add Devices twice. It never discovers any devices. The same setup works fine with XCTU and a node-red app that can directly talk to the Xbee so I'm not suspecting a hardware issue. It also worked properly a couple of weeks ago, but I'm not sure what the last working version was.
2019-06-04 11:48:11 DEBUG (MainThread) [zigpy_xbee.uart] Connection made
2019-06-04 11:48:12 DEBUG (MainThread) [zigpy.appdb] Loading application state from /home/homeassistant/.homeassistant/zigbee.db
2019-06-04 11:48:12 DEBUG (MainThread) [zigpy_xbee.api] at command: AP (2,)
2019-06-04 11:48:12 DEBUG (MainThread) [zigpy_xbee.api] Command at (b'AP', b'\x02')
2019-06-04 11:48:12 DEBUG (MainThread) [zigpy_xbee.uart] Sending: b'\x08\x01AP\x02'
2019-06-04 11:48:15 WARNING (MainThread) [zigpy_xbee.api] at: No response to AP command
2019-06-04 11:48:16 DEBUG (MainThread) [zigpy_xbee.zigbee.application] No response to API frame. Configure API mode
2019-06-04 11:48:17 DEBUG (MainThread) [zigpy_xbee.uart] Command mode sending b'+++' to uart
2019-06-04 11:48:20 DEBUG (MainThread) [zigpy_xbee.api] Command mode no response to AT '+++' command
2019-06-04 11:48:20 DEBUG (MainThread) [zigpy_xbee.api] Failed to enter AT command mode at 57600bps, trying 1200 next
2019-06-04 11:48:22 DEBUG (MainThread) [zigpy_xbee.uart] Command mode sending b'+++' to uart
2019-06-04 11:48:28 DEBUG (MainThread) [zigpy_xbee.api] Command mode no response to AT '+++' command
2019-06-04 11:48:28 DEBUG (MainThread) [zigpy_xbee.api] Failed to enter AT command mode at 1200bps, trying 2400 next
2019-06-04 11:48:29 DEBUG (MainThread) [zigpy_xbee.uart] Command mode sending b'+++' to uart
2019-06-04 11:48:32 DEBUG (MainThread) [zigpy_xbee.api] Command mode no response to AT '+++' command
2019-06-04 11:48:32 DEBUG (MainThread) [zigpy_xbee.api] Failed to enter AT command mode at 2400bps, trying 4800 next
2019-06-04 11:48:33 DEBUG (MainThread) [zigpy_xbee.uart] Command mode sending b'+++' to uart
2019-06-04 11:48:35 DEBUG (MainThread) [zigpy_xbee.api] Command mode no response to AT '+++' command
2019-06-04 11:48:35 DEBUG (MainThread) [zigpy_xbee.api] Failed to enter AT command mode at 4800bps, trying 9600 next
2019-06-04 11:48:36 DEBUG (MainThread) [zigpy_xbee.uart] Command mode sending b'+++' to uart
2019-06-04 11:48:38 DEBUG (MainThread) [zigpy_xbee.api] Command mode no response to AT '+++' command
2019-06-04 11:48:38 DEBUG (MainThread) [zigpy_xbee.api] Failed to enter AT command mode at 9600bps, trying 19200 next
2019-06-04 11:48:40 DEBUG (MainThread) [zigpy_xbee.uart] Command mode sending b'+++' to uart
2019-06-04 11:48:42 DEBUG (MainThread) [zigpy_xbee.api] Command mode no response to AT '+++' command
2019-06-04 11:48:42 DEBUG (MainThread) [zigpy_xbee.api] Failed to enter AT command mode at 19200bps, trying 38400 next
2019-06-04 11:48:43 DEBUG (MainThread) [zigpy_xbee.uart] Command mode sending b'+++' to uart
2019-06-04 11:48:45 DEBUG (MainThread) [zigpy_xbee.api] Command mode no response to AT '+++' command
2019-06-04 11:48:45 DEBUG (MainThread) [zigpy_xbee.api] Failed to enter AT command mode at 38400bps, trying 57600 next
2019-06-04 11:48:46 DEBUG (MainThread) [zigpy_xbee.uart] Command mode sending b'+++' to uart
2019-06-04 11:48:48 DEBUG (MainThread) [zigpy_xbee.api] Command mode no response to AT '+++' command
2019-06-04 11:48:48 DEBUG (MainThread) [zigpy_xbee.api] Failed to enter AT command mode at 57600bps, trying 115200 next
2019-06-04 11:48:49 DEBUG (MainThread) [zigpy_xbee.uart] Command mode sending b'+++' to uart
2019-06-04 11:48:51 DEBUG (MainThread) [zigpy_xbee.api] Command mode no response to AT '+++' command
2019-06-04 11:48:51 DEBUG (MainThread) [zigpy_xbee.api] Failed to enter AT command mode at 115200bps, trying 230400 next
2019-06-04 11:48:52 DEBUG (MainThread) [zigpy_xbee.uart] Command mode sending b'+++' to uart
2019-06-04 11:48:55 DEBUG (MainThread) [zigpy_xbee.api] Command mode no response to AT '+++' command
2019-06-04 11:48:55 DEBUG (MainThread) [zigpy_xbee.api] Couldn't enter AT command mode at any known baudrate.Configure XBee manually for escaped API mode ATAP2
2019-06-04 11:48:55 ERROR (MainThread) [zigpy_xbee.zigbee.application] Failed to configure XBee API mode.
2019-06-04 11:50:23 DEBUG (MainThread) [zigpy_xbee.zigbee.application] Broadcast request seq 1
2019-06-04 11:50:23 DEBUG (MainThread) [zigpy_xbee.api] Command tx_explicit (00:00:00:00:00:00:ff:fc, <BroadcastAddress.ALL_ROUTERS_AND_COORDINATOR: 65532>, 0, 0, 54, 0, 0, 32, b'\x01<\x00')
2019-06-04 11:50:23 DEBUG (MainThread) [zigpy_xbee.uart] Sending: b'\x11\x02\x00\x00\x00\x00\x00\x00\xff\xfc\xff\xfc\x00\x00\x006\x00\x00\x00 \x01<\x00'
2019-06-04 11:51:56 ERROR (MainThread) [homeassistant.components.websocket_api.http.connection.1449729968] Error handling message: Unknown error
Traceback (most recent call last):
  File "/srv/homeassistant/lib/python3.5/site-packages/homeassistant/components/websocket_api/connection.py", line 76, in async_handle
    handler(self.hass, self, schema(msg))
  File "/srv/homeassistant/lib/python3.5/site-packages/voluptuous/schema_builder.py", line 267, in __call__
    return self._compiled([], data)
  File "/srv/homeassistant/lib/python3.5/site-packages/voluptuous/schema_builder.py", line 589, in validate_dict
    return base_validate(path, iteritems(data), out)
  File "/srv/homeassistant/lib/python3.5/site-packages/voluptuous/schema_builder.py", line 381, in validate_mapping
    cval = cvalue(key_path, value)
  File "/srv/homeassistant/lib/python3.5/site-packages/voluptuous/schema_builder.py", line 811, in validate_callable
    return schema(data)
  File "/srv/homeassistant/lib/python3.5/site-packages/homeassistant/components/zha/core/helpers.py", line 153, in convert_ieee
    return EUI64([uint8_t(p, base=16) for p in ieee_str.split(':')])
AttributeError: 'dict' object has no attribute 'split'
2019-06-04 11:52:23 ERROR (MainThread) [homeassistant.components.websocket_api.http.connection.1449729968] Error handling message: Unknown error
Traceback (most recent call last):
  File "/srv/homeassistant/lib/python3.5/site-packages/homeassistant/components/websocket_api/decorators.py", line 17, in _handle_async_response
    await func(hass, connection, msg)
  File "/srv/homeassistant/lib/python3.5/site-packages/homeassistant/components/zha/api.py", line 111, in websocket_permit_devices
    node=ieee)
  File "/srv/homeassistant/lib/python3.5/site-packages/zigpy/application.py", line 141, in permit
    broadcast_address=t.BroadcastAddress.ALL_ROUTERS_AND_COORDINATOR
  File "/srv/homeassistant/lib/python3.5/site-packages/zigpy/device.py", line 198, in broadcast
    broadcast_address=broadcast_address
  File "/srv/homeassistant/lib/python3.5/site-packages/zigpy_xbee/zigbee/application.py", line 284, in broadcast
    return await asyncio.wait_for(request, timeout=TIMEOUT_TX_STATUS)
  File "/usr/lib/python3.5/asyncio/tasks.py", line 404, in wait_for
    raise futures.TimeoutError()
concurrent.futures._base.TimeoutError

My problem is solved now, not sure which of these actions did the trick but I:

Removed the zha: entries from my configuration.yaml
Deleted the ZHA integration using the trashcan button at the top right of Config->Integrations->ZHA
Manually deleted /zigbee.db
Manually deleted /.storage/zha.storage
Manually removed all zha entries from /.storage/core.device_registry
Re-added the ZHA integration using the Configuration->Integrations->
Searched for new devices using the Configuration->ZHA->Add Devices button

I think the problem started when my system went down due to power outage and the Xbee device got assigned to a different USB device when it rebooted. Cleaning up the old entries from the prior USB device assignment seems to be the cure.

I did notice the AttributeError: 'dict' object has no attribute 'split error message if I used the "Search Again" without going to another page, not sure if that was ever related to the other errors. Backing out to the top level and starting again with Configuration->ZHA->Add Devices seemed to work every time.

Search again is broken. I’ll be working on it this week.

thanks, same here. JFYI I have
{'isTrusted': True}
as dict structure here for the ieee_str