AssertionError in group_symbols

So I am getting an assertion error:
Python 3.6.0 (default, Dec 24 2016, 08:03:08)
[GCC 6.2.1 20160830] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> from xkbgroup import XKeyboard
>>> xkb = XKeyboard()
>>> xkb.groups_symbols
3 2
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/m45t3r/Source/xkbgroup/venv/lib/python3.6/site-packages/xkbgroup/core.py", line 261, in groups_symbols
    return _ListProxy(symdata.symbol for symdata in self._symboldata_list)
  File "/home/m45t3r/Source/xkbgroup/venv/lib/python3.6/site-packages/xkbgroup/core.py", line 428, in _symboldata_list
    return _parse_symbols(b_symbol_str.decode(), ["pc", "inet", "group"])
  File "/home/m45t3r/Source/xkbgroup/venv/lib/python3.6/site-packages/xkbgroup/core.py", line 487, in _parse_symbols
    assert len(indices) == len(set(indices))    # No doubles
AssertionError
>>>

From my debugging, the problem seems to be this:
print(symboldata_list)
[SymbolData(symbol='br', variant=None, index=0), SymbolData(symbol='us', variant='intl', index=1), SymbolData(symbol='terminate', variant='ctrl_alt_bksp', index=0)]

As you can see, there is two layouts reporting index=0. This comes from my /etc/X11/xorg.conf.d/00-keyboard, set by localectl:
# Read and parsed by systemd-localed. It's probably wise not to edit this file
# manually too freely.
Section "InputClass"
        Identifier "system-keyboard"
        MatchIsKeyboard "on"
        Option "XkbLayout" "br"
        Option "XkbModel" "abnt2"
        Option "XkbOptions" "terminate:ctrl_alt_bksp"
EndSection

So for some reason, xkbgroup is detecting XkbOptions as another layout. This shouldn't happen: terminate: ctrl_alt_bksp is not actually a layout, it is part from layout br.

@m45t3r, thank you for the detailed report!
Using setxkbmap with the same settings as in your /etc/X11/xorg.conf.d/00-keyboard
$ setxkbmap -layout "br" -model "abnt2" -option "terminate:ctrl_alt_bksp" -verbose
Trying to build keymap using the following components:
keycodes:   evdev+aliases(qwerty)
types:      complete
compat:     complete
symbols:    pc+br+inet(evdev)+terminate(ctrl_alt_bksp)
geometry:   pc(pc104)

reveals that there is another non-symbol name terminate apart from pc, inet and group (which are present under my settings):
$ setxkbmap -layout "us,ru,ua" -option "grp:alt_shift_toggle" -verbose
Trying to build keymap using the following components:
keycodes:   evdev+aliases(qwerty)
types:      complete
compat:     complete
symbols:    pc+us+ua:2+ru:3+inet(evdev)+group(alt_shift_toggle)
geometry:   pc(pc104)

So adding terminate to the set of names to ignore should solve this specific issue.

Regarding long-term solutions:
Currently, non-symbol names are filtered out manually and this part is hardcoded. This is based on the assumption that the set of non-symbol names is finite and immutable (but the set of symbol ones is possibly infinite).
It's hard to include every possible non-symbol name. So more gracefully this could be done by allowing users to specify their own set of names to filter out while still providing the default set.

@m45t3r, just made a commit and uploaded a new version to PyPI. Try it out!

Seems to work now, thanks!


Adding encryptionScheme Field Results In Application Crash On Some Platforms

Unfortunately we found a bug on Play Station 4 and SkyQ set top box. Versions 2.1.3 and above crashes application and results in video start failure.
More details in Slack thread: https://video-dev.slack.com/archives/C01QRAFHLQK/p1726265521763269
After careful inspection and some tests on these devices, I realized that this is caused by encryptionScheme field in audio and video capabilities.
Example from Play Station 4:

Closing this issue because filed another in shaka player
https://github.com/shaka-project/shaka-player/issues/7354


Invite link in the email not working

I've deployed the app to meteor.com and when I send an invite as an admin the email arrives to the destination mailbox all good. But clicking on the link in the email opens new tab in browser and points to url http://atsignup/
Any idea what might be going wrong here?

Same here

Closing as duplicate of https://github.com/TelescopeJS/Telescope/issues/1194


Does not handle thin windows well

Typing a message in the textarea draws focus to the right of the textarea, which expands beyond the right side of the screen on small windows.
I'd argue that chat programs are likely to be open alongside other windows, so a thin window is a use case worth considering.

i'm gonna consolidate all your textarea issues into one issue


Error response from daemon: No such image: xilinx/vitis-ai-gpu:<IP_ADDRESS>8

I have following this to install vitisAI and trying to install gpu docker,
below is mine configuration :
Linux C001886040 5.11.0-41-generic#45~20.04.1-Ubuntu SMP Wed Nov 10 10:20:10 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux
Nvidia RTX A3000,  NVIDIA-SMI 495.44       Driver Version: 495.44       CUDA Version: 11.5
Vitis unified platform 2021.1
Docker version 20.10.11, build dea9396
cmake version 3.20.2
When run ./docker_build_gpu.sh
it shows :
The following information may help to resolve the situation:
The following packages have unmet dependencies:
xrt : Depends: libboost-filesystem1.71.0 but it is not installable
Depends: libboost-program-options1.71.0 but it is not installable
Depends: libc6 (>= 2.29) but 2.27-3ubuntu1.4 is to be installed
Depends: libncurses6 (>= 6) but it is not installable
Depends: libprotobuf17 but it is not installable
Depends: libtinfo6 (>= 6) but it is not installable
E: Unable to correct problems, you have held broken packages.
The command '/bin/bash -c wget --progress=dot:mega -O xrt.deb ${XRT_URL}     && ls -lhd ./xrt.deb     && apt-get update -y     && apt-get install -y ./xrt.deb     && rm -fr /tmp/*' returned a non-zero code: 100
Error response from daemon: No such image: xilinx/vitis-ai-gpu:<IP_ADDRESS>8

full error msg:
Vitis-AI_docker_gpu_build_Error.txt
Thanks

I have found the same issue when creating cpu docker ./docker_build_cpu.sh

full error msg:
Vitis-AI_docker_cpu_build_Error.txt

Hi,  I am also encountering similar issue, but different error message, when I try to build the docker_build_gpu.sh . I am on Ubuntu 18.04 host PC.

@JasonTryharder vitis-ai hasn't supported Nvidia Ampere generation (yours is A3000). To be concrete, vitis-ai hasn't supported CUDA 11 + CUDA 10 don't support Nvidia Ampere gen
About building vitis-ai cpu, you can pull the pre-built docker image from here: https://hub.docker.com/r/xilinx/vitis-ai-cpu/tags
@schegde Try to switch to tag 'v1.4' or 'v1.4.1'. I'm usually avoid using master branch, it's inconsistent

@JasonTryharder @schegde ,
Are you guys still having error message building the Vitis AI GPU docker image?
Odd to see package conflict when installing XRT dependencies - boost, libc and so on. XRT uses the OS' default package version, and for Vitis AI GPU it is based on Ubuntu 18.04.

I have some problem:
make[2]: Leaving directory '/tmp/protobuf-3.4.0/src'
Makefile:1488: recipe for target 'all-recursive' failed
make[1]: Leaving directory '/tmp/protobuf-3.4.0'
make[1]: *** [all-recursive] Error 1
make: *** [all] Error 2
Makefile:1395: recipe for target 'all' failed
The command '/bin/bash -c cd /tmp; wget --progress=dot:mega https://codeload.github.com/google/protobuf/zip/v3.4.0     && unzip v3.4.0     && cd protobuf-3.4.0     && ./autogen.sh     && ./configure     && make -j     && make install     && ldconfig     && rm -fr /tmp/*' returned a non-zero code: 2
Error response from daemon: No such image: xilinx/vitis-ai-gpu:<IP_ADDRESS>3
Error response from daemon: No such image: xilinx/vitis-ai-gpu:<IP_ADDRESS>3
Error response from daemon: No such image: xilinx/vitis-ai-gpu:<IP_ADDRESS>3

@siniGAL Please raise a separate Github Issue. These are different build errors.
Are you building from the latest v2.0 release?
I see multiple lines of
g++: internal compiler error: Killed (program cc1plus)

This may happen due to lack of resources on the docker build host. How much memory is allocated for docker build, and is there sufficient free disk space - recommended 100GB?

@hanxue thanks.
I create a new issue. I solved it.

Getting error while executing

./docker_build_cpu.sh

returned a non-zero code: 137
Error response from daemon: No such image: xilinx/vitis-ai-cpu:<IP_ADDRESS>3
Error response from daemon: No such image: xilinx/vitis-ai-cpu:<IP_ADDRESS>3
Error response from daemon: No such image: xilinx/vitis-ai-cpu:<IP_ADDRESS>3


REPL: Rename of file without changing name clears the file contents

Describe the bug
Doing a rename without an actual name change (e.g. because you changed your mind or accidentally started to rename the wrong file) just deletes everything in the file.
This applies to both the Svelte 4 and 5 preview REPL.
Reproduction

Add a component
Add some content to the file
Click the file name to start rename
Click away to end rename without changing anything
Observe file contents

Logs
No response
System Info
Tested in Chrome and FF.

Severity
annoyance

The new playground does not have this issue.


Chrome: when nesting SVG, text becomes jagged and performance seems degraded

Not sure which versions of Chrome that this applies to but we see this in our project. Even for small amounts of SVG, the text becomes jagged and non-antialiased. For larger amounts of SVG, rendering sometimes stops in mid air and leaves blank areas on the screen.
We don't see it in Safari so it doesn't seem to be a webkit thing.

Doesn't this make it a Blink thing, since Chrome uses Blink, and Safari
uses Website?
Den 13/04/2015 14.13 skrev "Martin Vindahl Olsen"<EMAIL_ADDRESS>
:

Not sure which versions of Chrome that this applies to but we see this in
our project. Even for small amounts of SVG, the text becomes jagged and
non-antialiased. For larger amounts of SVG, rendering sometimes stops in
mid air and leaves blank areas on the screen.
We don't see it in Safari so it doesn't seem to be a webkit thing.
—
Reply to this email directly or view it on GitHub
https://github.com/mvindahl/angular-pan-zoom/issues/34.

Might very well be ...
Anyway, I found a fix. Seems like Chrome handles the zoom style (http://caniuse.com/#search=zoom) nicely but doesn't always handle the transform style (http://caniuse.com/#search=transform) very well. IE, on the other hand, does not support the zoom style at all.
Let me reveal that the fix involves browser vendor sniffing ... :-P


Docker image for integration testing

A docker image built with the pulsar distribution tarball. The same
image can be used to run zookeeper, bookkeeper, the broker or a
proxy. Scripts are provided to run each. There is also a script to
initialize the cluster, and another script to coordinate the
containers so that, for example, the broker doesn't start before the
cluster is initialized.
All services are run with supervisor, so that tests will be able to
start and stop the services.

@sijie @merlimat updated to use the standard image as the base image.

retest this please // flake


Insiders 'File' and 'Help' is missing in nightly build

VSCode Version: Cant tell you because the help option is missing (see below)
OS Version: Cant tell you because the help option is missing (see below)

Steps to Reproduce:

Open VSCode

One that works

Duplicate of #55624


ItemsPanelTemplate TemplateBinding causes AVLN2000 compile error after upgrading from 11.2.0 to 11.2.1

Describe the bug
A TemplateBinding in a StackPanel within a ItemsPanelTemplate was working in 11.1.x and 11.2.0, after 11.2.1 update it causes this error:
Error AVLN2000 Avalonia: Unable to resolve suitable regular or attached property Spacing on type Avalonia.Controls:Avalonia.Controls.ItemsControl Line 36, position 37.

Seems TemplateBinding is resolving to the ItemsControl instead of the actual template control with the Spacing property, is this an intended behavior change?
Possibly related to https://github.com/AvaloniaUI/Avalonia/pull/17483
To Reproduce
<Styles xmlns="https://github.com/avaloniaui"
        xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml">

    <Style Selector="controls|StackCard">
        <Setter Property="Template">
            <ControlTemplate>
                <ScrollViewer>
                    <ItemsControl>
                        <ItemsControl.ItemsPanel>
                            <ItemsPanelTemplate>
                                <StackPanel
                                    Spacing="{TemplateBinding Spacing}"/>
                            </ItemsPanelTemplate>
                        </ItemsControl.ItemsPanel>
                    </ItemsControl>
                </ScrollViewer>
            </ControlTemplate>
        </Setter>
    </Style>
</Styles>

public class StackCard : TemplatedControl
{
    public static readonly StyledProperty<int> SpacingProperty = AvaloniaProperty.Register<StackCard, int>(
        "Spacing",
        4
    );

    public int Spacing
    {
        get => GetValue(SpacingProperty);
        set => SetValue(SpacingProperty, value);
    }
}

Expected behavior
No response
Avalonia version
11.2.1
OS
Windows
Additional context
No response

I had the same problem 😂.

This one puzzled me for a moment, as this wasn't supposed to be a supported scenario. But if it worked in previous versions, that would indeed be a source breaking change that needs fixing...
It turns out this never really worked!
While the above XAML didn't produce a compilation error before 11.2.1, the binding wasn't working at runtime, because it was effectively targeting the ItemsControl, not the parent StackCard control.
This is easily verifiable. Adjust the bound Spacing: it simply doesn't work. Bind to a common property such as Background: the color comes from the ItemsControl, not the StackCard:
<Style Selector="local|StackCard">
  <Setter Property="Background" Value="OrangeRed" />
  <Setter Property="Spacing" Value="32" />
  <Setter Property="Template">
    <ControlTemplate>
      <ScrollViewer>
        <ItemsControl Background="DodgerBlue">
          <ItemsControl.ItemsSource>
            <generic:List x:TypeArguments="x:String">
              <x:String>Abc</x:String>
              <x:String>Def</x:String>
            </generic:List>
          </ItemsControl.ItemsSource>
          <ItemsControl.ItemsPanel>
            <ItemsPanelTemplate>
              <StackPanel
                Height="100"
                Background="{TemplateBinding Background}"
                Spacing="{TemplateBinding Spacing}" />
            </ItemsPanelTemplate>
          </ItemsControl.ItemsPanel>
        </ItemsControl>
      </ScrollViewer>
    </ControlTemplate>
  </Setter>
</Style>

I've tested 11.0.13, 11.1.4 and 11.2.0 and they all exhibit the same behavior here: the TemplatedParent is the ItemsControl at runtime, despite the compiler thinking otherwise.
In 11.2.1, the compiler now properly informs you that the binding isn't correct.
Plus, in 11.2.1 you can now use a TemplateBinding inside ItemsPanelTemplate when it's outside of a Style or ControlTemplate (for example as a local value or inside a resource).
ItemsPanelTemplate and ControlTemplate now have the exact same behavior as far as bindings are concerned.
Here's a summary table showing the resolved TemplatedParent type for a TemplateBinding inside ItemsPanelTemplate depending on its scope:

Scope
11.0.x
11.1.x
11.2.0
11.2.1

ControlTemplate (runtime)
✔ ItemsControl
✔ ItemsControl
✔ ItemsControl
✔ ItemsControl

ControlTemplate (compiler)
❗ Parent control
❗ Parent control
❗ Parent control
✔ ItemsControl

Style for ItemsControl
✔ ItemsControl
✔ ItemsControl
❌ Error
✔ ItemsControl

Local value
❌ Error
❌ Error
❌ Error
✔ ItemsControl

Resource
❌ Error
❌ Error
❌ Error
✔ ItemsControl

Thanks for the detailed investigation @MrJul, can confirm in our project it seems the Spacing property never bound effectively before either. I think this can just be closed then? Unless @huangliu sees some different behavior.
For the binding I intended to do, should it be using something like Spacing="{Binding $parent[controls:StackCard].Spacing}" instead?

Thanks for the detailed investigation @MrJul, can confirm in our project it seems the Spacing property never bound effectively before either.

Thanks for confirming! I'm closing this issue.

For the binding I intended to do, should it be using something like Spacing="{Binding $parent[controls:StackCard].Spacing}" instead?

Indeed, that would be the correct way to proceed.

I keep getting this error over and over again.

0>UserListView.axaml(47,37): Error AVLN2000 Avalonia: Unable to resolve property or method of name 'GoToDetailsView' on type 'System.Object'. Line 47, position 37.
0>------- Finished building project: MA_Presentation_MVVM_Avalonia. Succeeded: False. Errors: 1. Warnings: 0
Build completed in 00:00:01.019

I'm learning web development, and I daily drive Linux, so I can't really use WPF etc. so I went the Avalonia route, because I thought It was mature enough to use based on the applications and info I could find. There's been a lot of frustration this past week, because I've only used Avalonia this week, while the others are learning about WPF. I have no idea how to resolve this and ChatGPT only gave me answers that aligned with what I had already done.
Also there's been a bit of a problem with other stuff, ChatGPT keeps telling me that ListView & PasswordBox should exist and work, but It doesn't really seem like that since their responding classes doesn't show up as an alternative, so I have to make due with what shows up. This is the code that I have problem with, it should work, but it doesn't, so I must have done something wrong which is more likely since I'm new to this and is learning, OR Avalonia is borked. "/
`
<ListBox.ItemTemplate>

<Grid.ColumnDefinitions>

</Grid.ColumnDefinitions>
                    <StackPanel Grid.Column="0">
                        <TextBlock Text="{Binding DisplayName}" />
                        <TextBlock Text="{Binding Email}" FontSize="10" />
                    </StackPanel>

                    <StackPanel Grid.Column="1">
                        <Button Command="{Binding DataContext.GoToDetailsViewCommand, RelativeSource={RelativeSource AncestorType=UserControl}}"
                                CommandParameter="{Binding .}"
                                Content="…" Padding="8 5" />
                    </StackPanel>
                </Grid>
            </DataTemplate>
        </ListBox.ItemTemplate>
    </ListBox>`


Support of arguments

Some annotation processor may require to pass custom arguments. Is it possible to apt configuration section to the build.gradle file like the android plugin ?
https://bitbucket.org/hvisser/android-apt

Passing processor arguments
Some annotation processor may require to pass custom arguments, you can use apt.arguments for that purpose. For instance for AndroidAnnotations you can use the following configuration:

apt {
    arguments {
            resourcePackageName android.defaultConfig.applicationId
            androidManifestFile variant.outputs[0]?.processResources?.manifestFile
    }
}

I'm not a fan of this in android-apt.
Gradle has no similar mean for setting any compiler argument globally, so there's no reason to add one for annotation processor arguments. Plus, it's really easy to do it "manually":
tasks.withType(JavaCompile) {
  options.compilerArgs += [ "-Afoo=bar", "-Abaz=${quux}" ]
}

I know that the experimental experimental new Android Gradle plugin (2.2 alpha) has an annotationProcessorArguments similar to android-apt, but it is inside a javaCompileOptions (that, in addition, is per-flavor).
So I'm not keen in adding this to this plugin. I may add something at the JavaCompile task level so you don't have to properly "format" the key-values with -A prefix and = separator, but not globally like android-apt does it; that does not feel "Gradle-native" to me.

Fair enough, thanks for the explanation.


Data importing fails over and over again.

Note: for support questions, please join our Discord server

I'm submitting a ...
[x ] bug report
[ ] feature request
[ ] question about the decisions made in the repository

Action taken (what you did)
Trying to import BTC-LTC data from for a few different 4 month time periods. Repeated a dozen times with different time periods, and from all possible BTC-LTC exchanges.

Expected result (what you hoped would happen)
To get data for 4 months time period.

Actual result (unexpected outcome)
Sometimes importing does not start at all, sometimes I get data just for few days or weeks and then the importer stops. No errors on a screen.

Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. stackoverflow, etc)
Latest Gekko + Windows 10.

I have the same issue. Importing doesn't work from polonium for me.

There seems to be many users having problems with data importing currently, with all exchanges. Is there anything I could do to help solving this problem? Any logs, debug information somewhere etc.?

IMPORT ERROR: Child process has died.

Error: SQLITE_BUSY: database is locked
RECEIVED ERROR IN IMPORT<PHONE_NUMBER>1060076
Child process has died.

Could anyone help with this?? It's quite frustrating that I cannot do anything with Gekko since I cannot get the data to test with :-(
Rkostevka; where exactly did you find those messages?

Actually, now that I tried once again, all imports from Poloniex seem to work just fine! :-o  But all other exchanges fail every time with every pair.

I'm very new to this. I'm running bash on windows 10. The manual says you should change WAL to DEL in handle.js , however, I found the solution in changing to DEL as described in "add journalmode to sqlite plugin config #1185"  in sample-config.js with nano sample-config.js


Deploying OTEL collector on AKS Namespace

Describe the issue you're reporting
Hi, I am trying to deploy OTEL collector in Agent mode on AKS namespace, however when i install it it gets deployed to other namespaces as well.
below is the command i am using
helm -n devns --values splunk-otel-collector-chart/helm-chart/splunk-otel-collector/values.yaml   splunk-otel-collector/helm-chart.splunk-otel-collector
tried setting nodeSelector in Values.yaml file as below
nodeSelector:
component: "devns-node"
but it seems its not taking this configuration
how can i restrict to only devns node pools.  thanks in advance

its resolved after adding under image section


Add a full step-by-step guide to provision a Keylime dev environment

Hopefully more comprehensive and intuitive for newcomers/beginners.
Credit to @mpeters and @lkatalin - most of the guide was written by them.
Signed-off-by: Mark Bestavros<EMAIL_ADDRESS>

@mpeters I've made some small changes beyond your suggestions - the troubleshooting section is now at the very bottom, and I've linked to the guide from the main README.


Should assign a buffer to file.contents

Due to a change in vinyl-fs, I noticed this plugin was causing failures in build pipelines.  It seems that you assign a string to the vinyl object but you are supposed to wrap that in a buffer.  I've put a temporary fix in vinyl-fs but in 3.0, that will be removed.  Please update accordingly.  Thanks!

Ref https://github.com/gulpjs/vinyl-fs/commit/165149c6a676ac867933173db28300cf3bab2a5f

It looks like this is a different problem and that node-sass is generating something called a SlowBuffer that we are failing on.  I'm going to keep digging.

It looks like a problem with the is-buffer module on npm.  Please disregard.


initial execv fails unless -p exec is explicitly set (musl libc)

On a barebones musl (Linux 6.5.8)
$ pledge -p "stdio" /bin/date
Bad system call
$ pledge -p "stdio exec" /bin/date
Mon Apr 29 22:44:12 UTC 2024

The culprit seems to be __pledge_mode & PLEDGE_STDERR_LOGGING
https://github.com/jart/pledge/blob/8693ebe15a30bd4235165ad72a469da29ca067cf/cmd/pledge.c#L841
since if I pass -q to pledge, exec works as expected:
$ pledge -q -p "stdio" /bin/date
Mon Apr 29 22:45:30 UTC 2024

Additionally, passing -k to pledge (sets __pledge_mode = PLEDGE_PENALTY_KILL_PROCESS) also results in "Bad system call", even with -q:
$ pledge -k -p "stdio" /bin/date
Bad system call
$ pledge -k -q -p "stdio" /bin/date
Bad system call

If I disable "implicit exec" by commenting out these lines...
https://github.com/jart/pledge/blob/8693ebe15a30bd4235165ad72a469da29ca067cf/cmd/pledge.c#L838-L843
... execv fails with the expected "Operation not permitted" error message:
$ pledge -p "stdio" /bin/date
/bin/date: execve failed: Operation not permitted

~The least messy fix I've come up with is to always include a SECCOMP_RET_ERRNO filter:~
Edit: I realize now that this patch breaks stuff (as it returns early)
--- a/libc/calls/pledge-linux.c	2023-11-08 18:18:17.000000000 +0000
+++ b/libc/calls/pledge-linux.c	2024-04-29 23:10:44.057225745 +0000
@@ -2129,6 +2129,9 @@
     }
   }

+  sf[0].k = SECCOMP_RET_ERRNO;
+  AppendFilter(&f, PLEDGE(sf));
+
   // now determine what we'll do on sandbox violations
   if (mode & PLEDGE_STDERR_LOGGING) {
     // trapping mode

(Note: /bin/date in this example is a static executable, so sandbox.so is not involved)
Linux 6.5.8 is built with:
CONFIG_HAVE_ARCH_SECCOMP=y
CONFIG_HAVE_ARCH_SECCOMP_FILTER=y
CONFIG_SECCOMP=y
CONFIG_SECCOMP_FILTER=y

Pledging tty and rpath makes it work as well:
$ pledge -p "stdio" /bin/date
Bad system call
$ pledge -p "stdio rpath" /bin/date
Bad system call
$ pledge -p "stdio tty" /bin/date
Bad system call
$ pledge -p "stdio rpath tty" /bin/date
Mon Apr 29 23:36:15 UTC 2024

/bin/date is busybox — perhaps busybox reads the file system and calls e.g. SYS_ioctl TIOCGWINSZ on every invocation..?


Fix separator being off on session load

Without this event, the split border is off:

With the SessionLoadPost event, the border is placed in the correct place. I am not sure this is the optimal way to solve these kind of issues, but it works.

yes.it ok


Open callout when the map opens

I have my map zoom to a specific pin when they click a locate on map button from another view controller. I was curious if it is possible to open the callout for that specific annotation as well when I hit the locate on map button.

Pretty sure we do this kind of thing in Artsy/Eigen, would recommend reading the source code of it


Allow supplying of data values while building a package

Describe the problem/challenge you have
This might be needed if user wants to be conscious about required non-defaulted values supplied. Or if there are images that need to be relocated but are only referred to in the config conditionally.
Describe the solution you'd like
PackageBuild API should provide a high level API to supply values while building a Package.
The same should be usable via kctrl.

Vote on this request
This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.
👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"
We are also happy to receive and review Pull Requests if you want to help working on this issue.

Some thought needs to be given here around inline vs secretRef input
We might need to start loading secrets without requiring a server connection.


Wrong term

Issue in ./help/edge/fundamentals/executing-commands.md
function(result) {
// The command succeeded.
// "value" is whatever the command returned
I think in this example, the term "value" should be replaced with "result"

Thank you for submitting this issue. We will review the documentation and make the necessary updates as quickly as possible. Tracking internally in PLAT-178616.

We've updated the documentation with the correct term. Closing this issue.


Action to get network info

As an app developer, I want to get the current device connection details, including IP address
https://pub.dev/packages/network_info_plus
Proposed EDL
View:
  onLoad:
    getNetworkInfo:
      onError:
      onCompleted:
         # provide a map of result here

@amin-nas I think this is more similar to camera or contacts than notifications. With notifications, you just check the permissions but don't do anything with it. Here you need to get some info. It's like camera permissions.
We don't need checkPermission for this one as just calling the action would return whether the user gave the permission or not.
Also it's 3 possibilities -
onError - when there is an error getting the info (check if this is a valid condition @reubensjohn )
onSuccess
onDenied

@kmahmood74 makes sense - For contacts, we seem to only support getPhoneContacts if the docs are up-to-date.
Modeling this after contacts, we could do:
View:
  onLoad:
    getNetworkInfo:
      onSuccess:
         # provide a map of result here
       onDenied:
      onError:

Assumptions:

if this is the first time app is triggering this action, user will be prompted to give permission.
if user has previously granted the permission, onSuccess will automatically be triggered.
if user has previously denied the permission, I believe the OS will not trigger the pop up again. In this case, we should trigger the onDenied so that the app developer can show proper messaging to user.

@reubensjohn any update here?

@amin-nas
not sure why we need an action for networkinfo. Could just be -
if ( ensemble.networkInfo.checkPermissions != 'granted' ) {
  var status = ensemble.networkInfo.requestPermissions();
  if ( status == 'granted' ) {
    var wifiName = ensemble.networkInfo.wifiName;
    ....
   }
}

ipv6, ipv4, gateway, broadcast, submask - do not need any permissions so user can use them anytime with just ensemble.networkInfo.ipv6 etc.
None of this is available on the web, just a FYI.
Nice thing with action is that it is clean in EDL instead of js. The con of an action is that it breaks the logic as it is asynchronous
if the above is fine, we'll just go with it for now instead of action as wifi info is a developer thing anyway
let me know

here's the detailed documentation - https://docs.ensembleui.com/actions/get-network-info


[RFC] luci-base: Add various dummy nodes

Adds DummySection (like SimpleSection but can
be used with a Map that is associated with
an actual UCI config).  Also SimpleValue and
other Simple* deriviates of AbstractValue allow you
to have UI elements that have no effect on the uci,
nor have normal server-side write action unless one
overrides the write function.
Signed-off-by: Daniel Dickinson<EMAIL_ADDRESS>This is just an RFC for now so that I can link to the PR in dependant patches commit messages.

Closing as have taken a break from LEDE/OpenWrt development for various reasons, so will reopen when this is back on the radar.


Set Parameters crashes after enabling background subtraction

Hi @luigiferiani,
I installed Tierpsy from source yesterday on my Windows 11 laptop. I have a problem in the "Set Parameters" window in the GUI. To reproduce, I select a video file, then click "Background" and check "Subtract Background," setting background buffer size = 250 and gap between frames = 25 like in the tutorial; then if I click "Play" the window becomes mostly irresponsive to user input (for example, switching back to the "Mask" section can make the window stop responding).
Also, the window stops responding briefly (but then resumes working) if I click the "Update Background" button. It appears that this button applies the background subtraction (although I can't find any mention of it in the docs). Clicking "Play" after "Update Background" still makes the window nearly impossible to use.
Perhaps the issue is that running "Play" with background subtraction is computationally expensive, so the computer can only play the video at a rate of 1 frame per >3 seconds. This makes sense since sometimes I can see the frame count increasing.
Am I using background subtraction incorrectly?
As a side question, where can I find info about choosing appropriate parameter settings (background buffer size and gap between frames) for background subtraction?

Thank you! I successfully used background subtraction to ignore some detritus in a video. I also looked at the background in the viewer.
So Tierpsy collects N equally-spaced frames and uses the maximum brightness pixels from these N frames to construct the background, then subtracts this background from the video at each frame before further processing?
Is the ratio between buffer size and gap supposed to be approximately 10:1?


Move terminal print to the UI

Even without a terminal, e.g. desktop app via PyInstaller, UI could be adequately meaningful.

Stats still not in the UI, but I might make that later. Good dashboard is hard, but not worth opening an issue.

Not using console (terminal) might require migrating away from pywebview.api*, as the built-in calling Python from JavaScript doesn't support sending Chinese characters -- https://github.com/r0x0r/pywebview/issues/1470
This might be bypassed by converting to a Bottle server with REST API, without needing to install extra packages.

Even after avoiding pywebview.api.*, REST API doesn't work on Chinese payload, if it is packaged with pyinstaller --noconsole.
Also, there is no server-side error output in console while running the executable of  pyinstaller --noconsole.


Send Data to View and Render it in view

Hello, I need to send data taken from a model and show them in the view, this is my current code:
const Project = use('App/Models/Project')

class DashboardController {
    async index({ view }) {
        const projects = await Project.all()
        return view.render('dashboard.index', { projects })
    }
}

module.exports = DashboardController

How can I do this?

Hey @bryannbarbosa! 👋
Please, use the forum or the Discord Server to ask this kind of question.
Your code is correct, the only thing you miss is to convert your data to JSON.
You can do that by using the toJSON() method on your data.
return view.render('dashboard.index', { projects: projects.toJSON() })

Related doc: http://adonisjs.com/docs/4.0/serializers#_introduction

Hi @RomainLanz!

With .toJSON method, the page return this error, how can i solve this?

.toJSON is applicable to Lucid query results only not a basic array. You
can skip that on basic array of objects
On Thu, Mar 29, 2018, 6:01 PM Bryann Barbosa<EMAIL_ADDRESS>wrote:

Hi @RomainLanz https://github.com/RomainLanz!
[image: error print]
https://user-images.githubusercontent.com/19507090/38099459-065d96b0-3351-11e8-88e1-0945420f7f0c.PNG
With .toJSON method, the page return this error, how can i solve this?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/adonisjs/adonis-framework/issues/820#issuecomment-377284265,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAa-nUxbqncpjVYaYnOlBJYeKPbFfTlRks5tjQVngaJpZM4S_YVo
.

--
Regards,
Skrabak Lukas
Application Developer - IBM Slovakia
Website: http://skrabak.eu
Linked-in: http://www.linkedin.com/in/skrabakl
Email<EMAIL_ADDRESS><EMAIL_ADDRESS>Cellphone: +421 (0) 917 800 588


Support for OSCAL version 1.1.1?

Issue description / feature objectives
The current version of OSCAL json from https://github.com/usnistgov/oscal-content/tree/main/nist.gov/SP800-53/rev5/json is 1.1.1:
"metadata": {       "title": "NIST Special Publication 800-53 Revision 5.1.1 HIGH IMPACT BASELINE",       "last-modified": "2023-12-15T20:25:00.649143Z",       "version": "5.1.1+u2",       "oscal-version": "1.1.1",
However, when attempting to use with trestle, it errors with:
OSCAL version: 1.1.1 is not supported, use 1.0.4 instead. (type=value_error)
Is there a plan to support OSCAL 1.1.1?

Thanks @AleJo2995 I missed the roadmap issue. Maybe add a link to it from the readme?
I'll try and get along to a community call to say hello.

Hi @deenine . You're completely right, with the current code base OSCAL v.1.1.1 is not supported, instead v1.0.4 is the only one supported.
Currently as you can see in issue #1480, that's the actual work mapped for starting to work on next weeks. So keep up to date in here for new announcements.
Also, I'd really like to invite you our community calls happening on Tuesdays every 2 weeks where we make announcements and give details on our current development status. You can find more information about it here
Thanks

Hello, I just ran into this issue with the SSP I created. Do you know if there have been any updates to transition from 1.0.4 to 1.1.1?

The trestle develop branch supports the latest OSCAL (1.1.2). You can try
that until the next release of trestle.
On Tue, May 28, 2024 at 4:31 PM JoseGHdz @.***> wrote:

Hi @deenine https://github.com/deenine . You're completely right, with
the current code base OSCAL v.1.1.1 is not supported, instead v1.0.4 is the
only one supported.
Currently as you can see in issue #1480
https://github.com/oscal-compass/compliance-trestle/issues/1480, that's
the actual work mapped for starting to work on next weeks. So keep up to
date in here for new announcements.
Also, I'd really like to invite you our community calls happening on
Tuesdays every 2 weeks where we make announcements and give details on our
current development status. You can find more information about it here
https://github.com/oscal-compass/compliance-trestle?tab=readme-ov-file#community-call
Thanks
Hello, I just ran into this issue with the SSP I created. Do you know if
there have been any updates to transition from 1.0.4 to 1.1.1?
—
Reply to this email directly, view it on GitHub
https://github.com/oscal-compass/compliance-trestle/issues/1494#issuecomment-2136055824,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAD66XJICEECKTQI737TM5LZETSTFAVCNFSM6AAAAABBS6WG6WVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMZWGA2TKOBSGQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


docs: Add Cloud Asset API activation requirement

Description of changes

[ ] Added Cloud Asset API to gcp credential instructions

GitHub issues resolved by this PR

N/A
Quality Assurance

Once the changes in this PR are merged and deployed, success criteria is: a credential created following this article should have all the requirements for server provision enabled

More info
Server provision failed because this API wasn't enabled.

@cla-bot check

Thank you for your contribution @m4ycon!
And congrats for your first pull requests merged into our repository.
Hopefully the first of many! 🎉


(PUP-6797) update smf tests for PUP-6796

Prior to this PR, the puppet tests would check for puppet output indicating a previously nonexistent smf service by checking that it had transitioned from 'stopped' to 'running'. However, the fact that puppet would output ensure => stopped for non-existent services was in error, addressed as PUP-6797 - it should have been ensure => absent as it is with many other providers. This commit updates the various acceptance tests to instead look for the now-correct transition, which is the definition of ensure as running.
Second, prior to this PR, SMF testing was unnecessarily broken out into multiple files each with redundant setup and teardown steps, and in many cases actually testing the exact same thing. This PR merges these tests into the smf_basic_tests.rb file, with no impact to test value but cutting run time from ~30seconds to ~10seconds. Not a huge win, but this feels simpler/cleaner to me.

ping @kurtwall @LuvCurves

successful run: https://gist.github.com/MosesMendoza/0b6da348cc20fa328ec0671a2b9deff9

I definitely like the simplification into one file 👍

Thanks @MosesMendoza. We'll keep this in mind over the next few test cycles if there are new, interesting failures. :)

@MosesMendoza 💯  nice work. Should be easier to maintain.  Just need it in stable.


C++ code generation improvements

Having fun creating my first GUI program again, since 15 years or so. wxGlade makes life so much easier. Thanks for that.
It could make my life even easier. May I suggest the following features, concerning C++ code generation?

getting rid of the event table and bind events using the Bind() member functions of the widgets. I'd especially love defining event handlers as c++11 lambda functions.

a way to declare additional member functions in generated class headers, as well as an optional additional member function implementation block in the generated cpp file. I presumed the extracode block was meant for the purpose, but it is placed outside of the class declaration, just after the dependencies block. (is this a mistake?)

Thanks for considering it.
Best regards,
Philippe.

Did you try "Code to be inserted before" and "Code to be inserted after"?
You can also activate the application property "Keep user code" and wxGlade will keep the code that you enter into the generated files. Just do not touch the blocks that are marked begin wxGlade / end wxGlade.
But for adding member functions, it's better to import from your own files and derive a class there.
See also in the documentation "Create and Use Source Code" http://wxglade.sourceforge.net/docs/source_code.html
For this you may use "Instance class" to make the app import and use your derived class. I have not tested this with C++, though, as I'm only using Python.
Dynamic event binding has been on the list for some time already.
Would you contribute example / test code?
Just modify the files listed below and submit as PR.


Use "Fancier Author Bio" plugin for author bio on archive pages

The plugin author says that the plugin doesn't allow this functionality but says that there's another very similar plugin that does. Should we switch?

I'm okay with switching!
On Sat, Jan 10, 2015 at 1:42 PM, Dylan Kinnett<EMAIL_ADDRESS>wrote:

The plugin author says that the plugin doesn't allow this functionality
https://wordpress.org/support/topic/adding-fancier-author-box-using-template-tag?replies=2
but says that there's another very similar plugin that does. Should we
switch?
—
Reply to this email directly or view it on GitHub
https://github.com/real-pants/Real-Pants/issues/62#issuecomment-69466810
.

ok I think the one that will work best (of the free options) is startbox. I'll do a trial run of it now, but it may mean that part of the site could look funny for a little minute.


Pre-set state for in-game briefing

Is there a possibility to set a side to ready before mission start.
Mission will go live only when one team readies up.

simply set IGBRIEF_bluforIsReady or IGBRIEF_opforIsReady to true in the postInitGlobal customization file

So to clarify.
There no state for INDFOR?

no, only blufor/opfor. The sides are not directly used, you could use it for opfor vs indfor, the boards and their placement are the important thing

Is this still an issue? What's the ask, to make an IGBRIEF_indforIsReady and IGBRIEF_civforIsReady variable?

I was originally asking to allow the IGB module to allow a mission maker to set a side to "ready" at mission start.
An example of this is let's say you have an attack/defend mission, I once had a missions like this where I wanted the defending team to choose when they wanted to start the attack from blufor so to speak.

Once the defending team is happy with their defences, they personally can start the mission/make it go live.
But to also allow INDFOR to be a side that can use this.
I don't see why it just doesn't have the ability to use blufor, opfor, indfor, and civfor, for all intents and purposes.


go test doesn't pass on BungBungame Photon 2

🤔

That fix causes crashes on MacBook Pro. Reverted.


Removed assignment to variable before return

Description
Removed assignment to variable before return
Type of change

[x] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[ ] Configuration Update
[ ] Bump-up dependent library
[ ] Bump-up library or tool used for development (does not change the final image)

/retest

/lgtm

/approve

/retest

/override "Red Hat Konflux / lightspeed-service-on-pull-request"

/retest


Remove inactive user from github workflow [skip ci]

remove pxli from blossom-ci.yml

build

Should target to branch24.02, and auto-merge will update the change to branch-24.04


[4.X] [Editor] Correct tilemap editor typo

Corrected a typo.

These files should not be edited directly, you change them here


Allow notes on individual submissions

It would be cool if we could leave mod-notes on individual submissions as we can with users.  Maybe this has been suggested but would be a backend nightmare?

Dup of #557


Can't install apps with the newest version of aiidalab (22.07.0)

Recently, we update aiidalab to version 22.07.0, and after the update, some of our apps cannot be installed with the error message: "Failed to install dependencies". During the installation no dependencies were installed as they were all present.

Downgrading aiidalab to version 22.06.0 resolves the issue

@csadorf would you be able to have a look?

@csadorf would you be able to have a look?

Yes, I will check that.

Should be resolved in release https://github.com/aiidalab/aiidalab/releases/tag/v22.07.1 .


Increase cpp version

Increased a CPP version to support new Apple Silicon processors with arm64 architecture.

should resolve this issue https://github.com/felixrieseberg/macos-notification-state/issues/21


Refactor code base

Signed-off-by: hossainemruz<EMAIL_ADDRESS>

Any news on this @hossainemruz ?

This PR do not add any new features. It's just improves the code quality. It was part of my learning to clean code 😀. Can't remember why I didn't merge it. I will check again.

Ah okay!
I had a look on the source code and was wondering, how you get the linked issues.
Because I need to get access to the development section of a pull request on the top right corner.
Maybe we can have a chat somehwere.

Sure. I am on GMT+6 time zone.

Well, I prefer chatting on Discord.
But besides that, I am thinking to fork your repo and doing changes regarding #3 and adding an additional output like "error" or even better: "requirements": "There are no issues set on 'development'!"
If you prefer a different wording, let me know!
So before parsing and finding an issue on html source, there need to be a check, if there are one or more issues linked to this pull request.

Hi @Ismoh. I have merged the changes in main branch. You can do a PR against the main branch. If you need to discuss anything, feel free to open a discussion or provide a link where we can discuss.


Strong source subtraction giving intended results when Crab/Virgo A is too close to Sun

Now we are subtracting a 200x 200 arcminute box centered around the A-team sources. However when a source comes too close to Sun, this means, we are also subtracting a part of the Sun. This was first noted and understood in June 12, 2024 data (image attached). The sudden dark region, we believe, is coming because of this. During this time, Crab was less than 3 degrees away from the Sun.

Fixed in #97


Fix an example for Comparable

https://carc.in/#/r/u7d

Thanks! (and please don't git blame that line :-P)


Make error to upgrade to Typescript 4.1 google-able

Ryan ran into this today with Group by:
server.ts:6:25 - error TS2615: Type of property 'OR' circularly references itself in mapped type '{ [K in keyof { AND?: Enumerable<ProductScalarWhereWithAggregatesInput>; OR?: Enumerable<ProductScalarWhereWithAggregatesInput>; ... 4 more ...; category?: string | StringWithAggregatesFilter; }]: Or<...> extends 1 ? { ...; }[K] extends infer TK ? GetHavingFields<...> : never : {} extends FieldPaths<...> ? never : K...'.
6   const grouped = await prisma.product.groupBy({
                          ~~~~~~~~~~~~~~~~~~~~~~~~
7     by: ['category']
  ~~~~~~~~~~~~~~~~~~~~
8   });
  ~~~~
server.ts:6:48 - error TS2554: Expected 0 arguments, but got 1.
6   const grouped = await prisma.product.groupBy({
                                                 ~
7     by: ['category']
  ~~~~~~~~~~~~~~~~~~~~
8   });
  ~~~

He was using a globally-installed typescript 4.0.5. Upgrading to 4.1 fixed his problem.
It would be nice to be able to google Type of property 'OR' circularly references itself in mapped type and land on the System Requirements page or something that says hey you need to upgrade your Typescript implementation.

Could also be a stackoverflow answer :-D

Could also be a stackoverflow answer :-D

Added to system requirements page as its troubleshooting related to version used: https://github.com/prisma/docs/blob/master/content/400-reference/100-system-requirements.mdx

Added to system requirements page as its troubleshooting related to version used: https://github.com/prisma/docs/blob/master/content/400-reference/100-system-requirements.mdx


Feature: Ethereal transporter

Would be nice to get support for Ethereal transporter.
The key things are:
    return nodemailer.createTestAccount()
      .then((account) => {
        log.debug('Ethereal account created:', account)

        transporter = nodemailer.createTransport({
          host: 'smtp.ethereal.email',
          port: 587,
          secure: false,
          auth: {
            user: account.user,
            pass: account.pass
          }
        })

        return transporter
      })

Then later when email gets sent:
const url = nodemailer.getTestMessageUrl(info)
Logger.info('Email URL: %s', url)

Of course, optionally, instead of creating a new account, can let user set an account in config as well. But Ethereal has this feature so you can create accounts on the fly.
But maybe could be a helper that creates an account and updates the config.

I could work on PR for this, just need some guidance.
Specifically, is it possible to get access to log inside a transporter so I could log the message URL?

Why not use SMTP driver directly and use Ethereal credentials?

That is a possibility, but I think the reason a built in transporter would be more helpful is:

It would output a direct link to the email that was sent to log. Eases development. Just click the link and it opens the email.
Do not need to pre-configure any settings. A mailbox can be created on the fly.

Yes, it will be nice to have it. You can contribute with a PR by adding the transport. Otherwise I will do it in some free time

Hello @moltar
I was planning to start working on it and thought to get your ideas around it too. This is what I am planning for.

If pre-existing account details exists inside the config file, use them, otherwise create a new account behind the scenes. ( For development it will be nice to have one pre-configured account )
Mail.send method will return the email URL, so you can log it anywhere on the console or a file.

That is very reasonable. And this is what I do.
I think it should log the URL by default and this could be controlled in the config. E.g. have a boolean toggle for this feature, or set a log desired log level OR false. Otherwise the code would not work universally in development and production if you need to resolve the promise and then log the URL yourself at every mail call.

Yup, makes sense


Try to recover old Cypress fix

Old branch with a single commit for fixing Cypress tests.  Checking and testing.

Thanks for the PR!
Any successful deployments (not always required) will be available below.
Backend
Frontend
Oracle-API
Once merged, code will be promoted and handed off to following workflow run.
Main Merge Workflow

Cypress tests not fixed.  Closing.


growth rate in % over last n days

i have made some graphs in ArcGIS Pro, maybe intresting to generate on daily basis (and not in ArcGIS)?
last 3 days:

last 7 days:

last 14 days:

Nice plots. Do you have Python skills? The draft is already there (map gifs). Not that hard to adjust.

Some skills.. Just used arcgis pro for visuals. Can do a remake of the map gifs and add a 3 day growth rate animation.

@J535D165 still few todo's be there is some progress.
coronaWatchNLapi

allmost.....
todo:
splitmap
timeslider

update, hope i can do some more this weekend..
http://hazedd.pythonanywhere.com/coronaWatchNL/map

issue closed: folium abandoned.


Cannot train the module

I run the following command in my terminal and I get the following message:
$ set CUDA_VISIBLE_DEVICES=0 & python train_mtl.py --config config.json --dataset deepglobe --model_name "LinkNet34MTL" --exp dg_L34_mtl --multi_scale_pred false
Random Seed:  7
Single Cuda Node is avaiable
Training with dataset => DeepGlobeDataset
****************************************************************************************************
Trainable parameters for Model LinkNet34MTL : 22.004903 M
****************************************************************************************************
...
...
Some numba warnings and runtime warnings
...
...

[>....... 1/1 ............]S:9s492ms|T:0ms|Loss: -0.284238 | VecLoss: 4.207048 | road miou: 28.0047%(18.9634%) | angle miou: 0.2155%

Traceback (most recent call last):
  File "train_mtl.py", line 443, in <module>
    main()
  File "train_mtl.py", line 434, in main
    train(epoch)
  File "train_mtl.py", line 294, in train
    write=True,
  File "C:\Users\...\road_connectivity-master\utils\util.py", line 147, in performMetrics
    100 * fwavacc,
TypeError: a bytes-like object is required, not 'str'

Do you know how to solve this issue?

It is PyTorch Version issue and request you to use 0.3 version. I hope you might have solved it and if not then request you to please re-open it.


Please update hls.js to 0.8.1

Its has a lot of bugfixes and improvement.

Thanks. hls.js has been upgraded in the master branch. A release will come up by the end of this week. Please star the project to support it


CLOUD-87717 Allow users to specify the HDP repo when using the YARN Orchestrator

@doktoric - would you mind taking a look?


fix: delete associated works when bindings are deleted

Description of your changes
This PR fixes an issue where associated works are not deleted when the scheduler removes bindings.
I have:

[x] Run make reviewable to ensure this PR is ready for review.

How has this code been tested
N/A
Special notes for your reviewer

Closing this PR for now as the root cause seems to be elsewhere.

This PR is sent as, when writing the E2E tests, it occurs that the system will not delete works derived from deleted bindings; logically it should be handled by the work generator (which creates the works). Though, when refreshing the tests it seems that the controller expects some kind of GC controller to do the work instead, which does not seem to exist at this moment. If the GC controller does exist, please let me know, and I would fix the issue differently.


Search results should be wrapped in an unordered list

Link to the Page
No response
Describe the Issue
We are going against our own advice with this;  elements with links should have an unordered list.
Are you able to create a Pull Request with the fix?
Yes

Reference: https://github.com/skeletonlabs/skeleton/pull/575


Remove github key and add as argument and avoid decoding

https://github.ibm.com/Developer-Playground/work-items/issues/658


Feature Request: Test Result doesn't navigate to the test

When running tests via IntelliJ and you double click a test from the test result, it does not navigate to the test in the file. Junit tests do this. It's especially nice when a test fails and you want to quickly navigate to it. Instead you have to click the test, then find the line in the stacktrace to navigate to the test.

The classname should work right ?
The test name navigation is fixed in 5.0 currently (needs that intellij release that I rolled back mind)

Classname seems to collapse in intellij. Test name right now takes to the top of the describe when using DescribeSpec which isn't that helpful.

Can you try in 5.0 with latest plugin

Been looking at this. It should work if you run tests from the plugin. If you're using gradle, you're hitting a gradle bug which doesn't support nested tests that are not methods.

I upgrade the plugin this morning and the behavior is slightly better as if I have the following:
it("test 1") {
} <--- cursor goes here when test 2 failed

it("test 2"){
}

Ideally it'd go to the assert that failed in test 2 though.
Not sure what you mean by via the plugin, I'm running it via intellij's gutter. I did try changing  Run tests using: IntelliJ IDEA and as Gradle, both have the same behavior of going to the test above it that I described above.

The assertions capture a stack trace so you should be able to go directly
to the line via the stack. That information isn't carried through to the
test location hints. I suppose it could be in theory, but might prove
tricky trying to determine which frame of the stack to use.
On Mon, 29 Nov 2021 at 09:19, Scott Busche @.***> wrote:

I upgrade the plugin this morning and the behavior is slightly better as
if I have the following:
it("test 1") {
} <--- cursor goes here when test 2 failed
it("test 2"){
}
Ideally it'd go to the assert that failed in test 2 though.
Not sure what you mean by via the plugin, I'm running it via intellij's
gutter. I did try changing Run tests using: IntelliJ IDEA and as Gradle,
both have the same behavior of going to the test above it that I described
above.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/kotest/kotest-intellij-plugin/issues/154#issuecomment-981734514,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFVSGWQOUNFZJWNIGLMK5DUOOKXZANCNFSM5HJHKI2A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

That's what we do now, but the stacktraces are often quite long, so this is
quite a drawback.
On Mon, Nov 29, 2021, 10:28 AM Sam Sam @.***> wrote:

The assertions capture a stack trace so you should be able to go directly
to the line via the stack. That information isn't carried through to the
test location hints. I suppose it could be in theory, but might prove
tricky trying to determine which frame of the stack to use.
On Mon, 29 Nov 2021 at 09:19, Scott Busche @.***> wrote:

I upgrade the plugin this morning and the behavior is slightly better as
if I have the following:
it("test 1") {
} <--- cursor goes here when test 2 failed
it("test 2"){
}
Ideally it'd go to the assert that failed in test 2 though.
Not sure what you mean by via the plugin, I'm running it via intellij's
gutter. I did try changing Run tests using: IntelliJ IDEA and as Gradle,
both have the same behavior of going to the test above it that I
described
above.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<
https://github.com/kotest/kotest-intellij-plugin/issues/154#issuecomment-981734514
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AAFVSGWQOUNFZJWNIGLMK5DUOOKXZANCNFSM5HJHKI2A
.
Triage notifications on the go with GitHub Mobile for iOS
<
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
<
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub
.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/kotest/kotest-intellij-plugin/issues/154#issuecomment-981799558,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGRMZFWRCINKAGEFFYRX2LUOOSZTANCNFSM5HJHKI2A
.

We thought about adding some kind of configurable "stack trace cleaner".
So you could say like, only keep com.mycompany + next 5 lines, or something
like that.
Also intellij is supposed to collapse stack traces on certain package
names, like io.kotest.
https://github.com/kotest/kotest-intellij-plugin/blob/master/src/IC-213/resources/META-INF/plugin.xml#L82
On Mon, 29 Nov 2021 at 13:57, Scott Busche @.***> wrote:

That's what we do now, but the stacktraces are often quite long, so this is
quite a drawback.
On Mon, Nov 29, 2021, 10:28 AM Sam Sam @.***> wrote:

The assertions capture a stack trace so you should be able to go directly
to the line via the stack. That information isn't carried through to the
test location hints. I suppose it could be in theory, but might prove
tricky trying to determine which frame of the stack to use.
On Mon, 29 Nov 2021 at 09:19, Scott Busche @.***> wrote:

I upgrade the plugin this morning and the behavior is slightly better
as
if I have the following:
it("test 1") {
} <--- cursor goes here when test 2 failed
it("test 2"){
}
Ideally it'd go to the assert that failed in test 2 though.
Not sure what you mean by via the plugin, I'm running it via intellij's
gutter. I did try changing Run tests using: IntelliJ IDEA and as
Gradle,
both have the same behavior of going to the test above it that I
described
above.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<

https://github.com/kotest/kotest-intellij-plugin/issues/154#issuecomment-981734514

,
or unsubscribe
<

https://github.com/notifications/unsubscribe-auth/AAFVSGWQOUNFZJWNIGLMK5DUOOKXZANCNFSM5HJHKI2A

.
Triage notifications on the go with GitHub Mobile for iOS
<

https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675

or Android
<

https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub

.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<
https://github.com/kotest/kotest-intellij-plugin/issues/154#issuecomment-981799558
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AAGRMZFWRCINKAGEFFYRX2LUOOSZTANCNFSM5HJHKI2A
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/kotest/kotest-intellij-plugin/issues/154#issuecomment-981966953,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFVSGRKQEIBWXD5D7PCBH3UOPLK5ANCNFSM5HJHKI2A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

This still doesn't with and one of the main reasons I won't convert all projects over to kotest.

Remind me how you're running your tests.

Via intellij. Same behavior using gradle or intellij to run them.

It won't work via gradle because gradle is broken in respect to location reporting.
It should work via intellij though - if you're running with the plugin and not just running gradle inside intellij.

Running via the normal test runner inside intellij.
On Sun, Jan 30, 2022, 8:16 PM Sam Sam @.***> wrote:

It won't work via gradle because gradle is broken in respect to location
reporting.
It should work via intellij though - if you're running with the plugin and
not just running gradle inside intellij.
—
Reply to this email directly, view it on GitHub
https://github.com/kotest/kotest-intellij-plugin/issues/154#issuecomment-1025317525,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAGRMZG7H7DSNHK6ERSRBE3UYXWIVANCNFSM5HJHKI2A
.
You are receiving this because you authored the thread.Message ID:
@.***>

Do you have this set to gradle:


Make internal struct ShaderData non-pub

Objective
ShaderData is marked as public, but is an internal type only used by one other
internal type, so it should be made private.
Solution
ShaderData is only used in ShaderCache, and the latter is private,
so there is no need to make the former public. This change removes the
pub keyword from ShaderData, hidding it as the implementation detail
it is.
Split from #5600

Technically a breaking change, but I don't think we want or need a migration guide.
Can you add ShaderData to the PR title so the commit message is nicer.

bors r+

@alice-i-cecile we should have forced a CHANGELOG with a Migration Guide note section since this is a breaking change. Sorry I didn't think about it. Any idea if we could make that check automated?

@djeedai do you think you can come up with a quick migration guide for this? You can either update the PR text or you can PR it to https://github.com/bevyengine/bevy-website/pull/470

@IceSentry done, let me know if that's ok?

Yep, should be good, I'll add it to the guide.

Ok thanks for that!


Facelift + Tickets

still WIP, but should work

@hroncok Good thinking, but these don't have limit (just in sum of all types) and we don't show it anyway.

So returning None in that case makes sense. 🤔


Simple Pull Request

I made this pull request on my own fork of the Apache AGE repository just for the sake of doing a pull request.

@mhs42
Thank you for taking the time to submit a pull request, your interest in contributing to Apache AGE is much appreciated.
It appears that the PR submitted seems to be empty or lacks the necessary content to be reviewed.
To ensure the productivity and efficiency of the project's development, it is important for contributors to verify their submissions for completeness and relevance to the project and to open issues.
Should you need guidance on how to make a contribution, please refer to the project's contributing guidelines.


Alwaysshown attribute to prevent $hide with save and cancel

Added attribute alwaysshown to xeditable-form. Ensures that form is initially shown (as with shown attribute) and prevents $hide execution in all situations.

Hi @kimmowikman , could you also add an example how to use this new feature to the index.html file

Added documentation to reference table. Do you think there should be an example case too?

Yes please.

I've been busy with other things recently.
I need to improve the pull request a little bit. Pressing cancel should revert fields to original values.

I found a one-liner that effectively does what I originally wanted without change to xeditable itself.
Just call this in onaftersave(): $scope.myFormName.$show()
I need to rethink how can this be applied with alwaysshown attribute.

Should this request be closed?


Docker compose env vars

asana card

wrapped up env vars getting to avoid empty strings as a legal result.

Hi @myxrome 👋 Firstly, thanks for fixing up the linting issues in my TwilioClient code, that saves me a job 😆
Reading through your PR, your code looks good, though I think I'm missing something fundamental about why your new get_env_var method is required?
I understand you're trying to avoid accidentally passing around empty strings, though in my testing, when given a key that doesn't map to an environment variable os.environ.get returns None, rather than an empty string (""), hence I'm struggling to understand what the get_env_var method is giving us?
Testing in a local flask shell, it appears to me that os.environ.get and get_env_var behave in exactly the same way when given a key for an unset environment variable:
import os
from app.utils import get_env_var

os.environ.get('dave') == None
True

get_env_var('dave') == None
True

I strongly suspect I'm misunderstanding something, perhaps specific to Docker, though perhaps you could help to enlighten me? Thanks! 🙏

Hi @dvjones89,
It works perfectly while running on the host itself. But if we pass an undefined environment variable from a host to a docker container using the environment section in a docker-compose configuration like:
version: '3'

services:
  api:
    ...
    environment:
      ...
      DAVE: ${DAVE}

we get the following results:
import os
from app.utils import get_env_var

os.environ.get('DAVE') == None
False

get_env_var('DAVE') == None
True

I have found the issue for the docker-compose project, but it refers to docker behaviour. And it seems that no one refers to that as a docker's issue.


Update portable-ruby to verison 2.3.7 for ARM [Linux]

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew tests with your changes locally?

I've built and uploaded the portable-ruby bottle for ARM (Raspberry Pi).
https://bintray.com/homebrew/bottles-portable-ruby/portable-ruby/2.3.7#files
https://github.com/Homebrew/homebrew-portable-ruby/releases/tag/2.3.7
I never managed to successfully build the dbm module for Ruby 2.3.7 (it was successfully built for 2.3.3). Good riddance to dbm. Long live JSON!

Merged. Thanks for the review, Colin!

Sure. I can look into that.

Are the leopard_64 and x86_64_linux bottles built by CI?

I see that TravisCI tests building the Docker image, but it doesn't appear to run that image.
https://github.com/Homebrew/homebrew-portable-ruby/blob/77d3e18296c18fcc8524c367e36855f6d6fee02f/.travis.yml#L44


Add more text to the incomplete paragraph

Hello people,
This pull request fixes issue #1638.
I believe that this sentence must have been skipped out in the relevant paragraph during the refactoring of the existing documentation.
Please let me know if I've made any mistake so I could make changes accordingly.
Thanks,
Jafer

Thanks!!

You're most welcome!


bug: The dbz format is allowed to be created when using backhole sink.

Describe the bug
create table t (v1 int, v2 int);
create sink invalid_dbz_type_sink from t with (connector = 'blackhole', type = 'debezium');

Expected behavior
It's supposed to fail but actually succeeded.
Additional context
NOTE:
Once fixed, please add the above test case in e2e_test/sink/append_only_sink.slt.

I believe we should introduce a better framework to check sink configurations in the connector node. We should be able to define fields as optional or required and specify possible values for each field in an easy way. cc @wenym1

@tabVersion Can you please take a look?

@tabVersion Can you please take a look?

@neverchanje what type is accepted in blackhole sink? or blackhole sink should not work with type attr


Add HttpPoster

We need a command to make a POST to an URL, see e.g. https://gitlab.com/oersi/oersi-etl/-/issues/64.
This could also be used not to just make a POST request and get data but also to upload processed data to web service.

The usecase within OERSI needs pagination too:
https://gitlab.com/oersi/oersi-import-scripts/-/blob/master/python/EduSharingHelpers.py#L63


How to server static files from folder & Upload file save on this folder.

I am stuck at following problem

How to serve static file folder (Just like express.static('/files'))
How to save file from chunk with original mime types (tried example code)

My app have 4 api so I don't want to manage another server for this :)
Thank you.

use the Discussions  tab for questions, for basic folder serve example see https://github.com/uNetworking/uWebSockets.js/discussions/407#discussioncomment-218059

use the Discussions  tab for questions, for basic folder serve example see https://github.com/uNetworking/uWebSockets.js/discussions/407#discussioncomment-218059

That snippet works for small files only - if you reach GB of data you have to properly stream the data - and really, you kind of always should stream!

That snippet works for small files only - if you reach GB of data you have to properly stream the data - and really, you kind of always should stream!

Look at examples/VideoStreamer.js instead - it shows how you can properly stream huge files with minimal backpressure.

Look at examples/VideoStreamer.js instead - it shows how you can properly stream huge files with minimal backpressure.


Add mtu functional test

Add test for both netdevice and for vfio
This PR also update the OCP version to 4.16, this is needed to have the newer multus-cni that support the MTU network-status field

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.04%) to 39.964%

Totals

Change from base Build<PHONE_NUMBER>:
0.04%

Covered Lines:
5115

Relevant Lines:
12799

💛  - Coveralls


Consider adding synthwave boy

https://www.youtube.com/watch?v=MVPTGNGiI-4

18/Feb/2024 and we still without the syntwave boy, plz...


Missing null in value type

I have a small issue with this code:
<script lang="ts">
	import Flatpickr from 'svelte-flatpickr';

	export let selected_date_time: string | Date | Date[] | null | undefined = undefined;
</script>

<Flatpickr bind:value={selected_date_time}> <!-- here the error -->
  <!-- More code here -->
</Flatpickr>

error:
Type 'string | Date | Date[] | null | undefined' is not assignable to type 'string | Date | Date[] | undefined'.
  Type 'null' is not assignable to type 'string | Date | Date[] | undefined'.

I think because the type for value is only string | Date | Date[] | undefined, missing the null.
Can we add it too, please?
Or maybe I'm totally wrong.

null is a possible value for the first binding only. I'm handling undefined as well in my code.
Anyway I think I "fixed" it by using the below code which seems more correct. What do you think?
<script lang="ts">
	import Flatpickr from 'svelte-flatpickr';

	export let selected_date_time: string | Date | Date[] | null | undefined = undefined;

	let value: string | Date | Date[];

	if (selected_date_time) {
		value = selected_date_time;
	}

	$: selected_date_time = value || undefined;
</script>

<Flatpickr bind:value={selected_date_time}></Flatpickr>

Is there a reason why you need to introduce null at all?
<script lang="ts">
	import Flatpickr from 'svelte-flatpickr';

	export let selected_date_time: string | Date | Date[] | undefined = undefined;
</script>

<Flatpickr bind:value={selected_date_time}></Flatpickr>

I need null because some dates in my objects are possibly null.
This is the issue.

I see. I've widened the types in 3.3.0 which should be published now.


Better entities parsing

I tried to improve the entities list parsing, not least in reaction to #139. I hope it works and fails (!) now as expected.

Codecov Report

Merging #143 (500c151) into master (9368fd1) will increase coverage by 0.47%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #143      +/-   ##
==========================================
+ Coverage   64.31%   64.78%   +0.47%
==========================================
  Files          19       19
  Lines        2057     2059       +2
  Branches      135      138       +3
==========================================
+ Hits         1323     1334      +11
+ Misses        599      587      -12
- Partials      135      138       +3

Impacted Files
Coverage Δ

src/Poseidon/EntitiesList.hs
66.66% <100.00%> (+60.41%)
: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 9368fd1...500c151. Read the comment docs.

Alright - I added support for empty lines and comments. Could you please take another quick look, @stschiff? It was a great parsec learning opportunity (!!), but maybe my solution is not to your standards :sweat_smile:


Angular 9 compatibility?

Is this component compatible with angular 9?

Yep! https://github.com/mattlewis92/angular-resizable-element/blob/master/CHANGELOG.md#326-2019-10-11


Every page request generates new session entry in the 'sessions' table

Just imagine how soon my production web hosting will down from a lack of space.

@alex-valchuk can you be more specific and provide a reproduction for me please so I can hunt this one down?

I think that @alex-valchuk might be referring to requests not originating from a browser (e.g. curl)?
I observed this behaviour with our "health-check" endpoint: with each GET request Flask will create a new session object. Using flask-session this will result in a new row in the server db. If the TTL of the session is long enough, you'll quickly end up with a huge session table.
However, I am not sure this problem can be fixed from within flask-session, for I believe it is Flask that asks for a new session after each request. Unfortunately, when I looked into this issue, I wasn't able to find a way to tell Flask to exclude some endpoints from session creation, but this was a while ago, so things might have changed 🤷🏽

@alex-valchuk can you be more specific and provide a reproduction for me please so I can hunt this one down?

@Lxstr, I gave up storing a session in database and just use default mechanism with cookies.
I did nothing special except following the instructions described in the documentation.
Later I realized that I simply don't need the session in db for now.

Thanks @giuppep and @alex-valchuk. I think this is now solved on the development branch with the use of overriding the boolean evaluation that was being used to determine if a session is empty. It was previously seeing an empty session with only permanent set as not empty.
This could probably be something more specific like a __is_empty method but at least it's a start.
class ServerSideSession(CallbackDict, SessionMixin):
    """Baseclass for server-side based sessions."""

    def __bool__(self) -> bool:
        return bool(dict(self)) and self.keys() != {"_permanent"}


Discussion about further improvements for the colored tab

@zadjii-msft @DHowett As mentioned in #3789 I'm opening this issue to have a discussion about further improvements and adjustments of the tab's custom color selection. If this was not what you had in mind, please close the issue.
The idea about ability to pick a custom color for a tab is described in #2994. Currently (at least in my opinion) there are two, maybe three things, which need to be discussed:

Should the coloring of the split button be removed and exactly which pieces of code need to go away? Or should be this activated by a setting?
How to signify the selected tab? Currently a lot of alpha transparency is added to color of the deselected tabs, is this good enough?
There was a suggestion that the painting of the non-client is maybe provided as a follow-up PR. Is this really needed, in which from, when (after or before the implementation of the xaml themeing)?

(I tagged this one for discussion and put in on the backlog. Thanks for filing it!)

So, about the first two things:

I think that the coloring of the split button should go away in any form. It is awkward and looks bad.
Regarding the signifying of the selected tab - I did a thing

Now the selected tab has a colored bar inside of it (like the orange one). Firefox does something pretty similar with its container tabs. Why within the tab not above it? Because I need to calculate the color of the selection bar based on the tab's background. But this can collide with the color of the non-client area (even if they are different colors, a low contrast between them can make the indicator bar invisible).
Pros:

it is better visible than if slapped above the tab
hey, I can make the corner of the tabs round again
Cons:
the tab is a little bit fatter and the split button looks a bit rectangular

I think I can bring the indicator bar a little bit closer to the bottom.
Also I'm pretty sure this is the way to go in regards to the indicating the selected bar. What are your thoughts/feedback?

I think this is a related question or issue but I'm looking for a way to set the tab color as part of the profile json file.

@jdhitsolutions That's actually been discussed in a whole bunch of threads recently (search, #6878,#6707,#7073) -  please refer to #1337, #3327, and the spec #5772 for more discussion.

You know, we've let this discussion fester for a while. In the meantime though, we shipped colored tabs a while ago, and we really haven't had any complaints from people about which tab is selected. Honestly, people seem really happy with it as is (myself included!)
For housekeeping purposes, I'm gonna close this one out. Thanks again for working on this!


Improve documentation for folding range type

So FoldingRange.type can be a FoldingRangeType or a string. What is the string supposed to be?
If it's a FoldingRangeType what makes each type special? Are they displayed differently or does it serve a different purpose?

If it is supposed to be categorized by source element (presumably so an editor can determine if a region should be collapsed by default or something), can we get more?
Ex: Collapsing a declaration, heredoc string (like in PHP), or automatically implemented property with a getter/setter (like in C#; which could be collapsed by default).

Somewhat related, I'm surprised that (what I feel to be) "standard" types like class and function or method don't exist.

Assuming that's the case, I'd also consider renaming this to FoldingRangeSourceKind so that FoldingRangeType can be reserved for something else.


Radio Dashboard example doesn't run

Open 'Radio Dashboard' example
Observe block editor shows an error at the declaration of the clients variable in the 'on start' block. Simulator fails to run the project.
Switch to JavaScript editor
Add const (or let) before clients: Client[] = []
Observe Simulator now successfully runs the project
Switch back to block editor
Observe the same error is shown as in (1)
Switch back to JavaScript editor
Observe the edit from (3) is lost

@riknoll  is this fixed in beta?

This is fixed.


Skip missed builds

In case the daily pipeline skips a few builds, dont rerun them trying to catchup

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by:
To fully approve this pull request, please assign additional approvers.
We suggest the following additional approver: mattdelco
Assign the PR to them by writing /assign @mattdelco in a comment when ready.
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

release/pipeline/OWNERS

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

@hklai in case a paused daily dag is restarted more than a day later, multiple daily runs are started. This PR fixes that issue

@rkpagadala: The following test failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
90be856bbf23ade25c5b4e6a90d7214595e7afd1
link
/test istio-pilot-e2e

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@rkpagadala: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
90be856bbf23ade25c5b4e6a90d7214595e7afd1
link
/test istio-pilot-e2e

prow/e2e-bookInfoTests.sh
90be856bbf23ade25c5b4e6a90d7214595e7afd1
link
/test e2e-bookInfo

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@rkpagadala: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
90be856bbf23ade25c5b4e6a90d7214595e7afd1
link
/test istio-pilot-e2e

prow/e2e-bookInfoTests.sh
90be856bbf23ade25c5b4e6a90d7214595e7afd1
link
/test e2e-bookInfo

prow/e2e-simpleTests.sh
90be856bbf23ade25c5b4e6a90d7214595e7afd1
link
/test e2e-simple

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test e2e-bookInfo
/test e2e-simple

@rkpagadala: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
90be856bbf23ade25c5b4e6a90d7214595e7afd1
link
/test istio-pilot-e2e

prow/e2e-bookInfoTests.sh
90be856bbf23ade25c5b4e6a90d7214595e7afd1
link
/test e2e-bookInfo

prow/e2e-simpleTests.sh
90be856bbf23ade25c5b4e6a90d7214595e7afd1
link
/test e2e-simple

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@rkpagadala: The following tests failed, say /retest to rerun them all:

Test name
Commit
Details
Rerun command

prow/istio-pilot-e2e.sh
90be856bbf23ade25c5b4e6a90d7214595e7afd1
link
/test istio-pilot-e2e

prow/e2e-simpleTests.sh
90be856bbf23ade25c5b4e6a90d7214595e7afd1
link
/test e2e-simple

prow/e2e-bookInfoTests.sh
90be856bbf23ade25c5b4e6a90d7214595e7afd1
link
/test e2e-bookInfo

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@hklai tested this on old environment, it works and seems to fix the out-of-order runs issue also.

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

Needs approval from an approver in each of these files:

release/pipeline/OWNERS [hklai]

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


Add AMD Compressonator bindings

https://github.com/GPUOpen-Tools/Compressonator

Windows only?

Yes, assuming we have a good solution for #100.
Libraries like AMD Compress are useful during production. It wouldn't be a run-time dependency, so being limited to a single OS is not an issue.

If all we need to do is tell the generator not to generate those files the it should be decently easy. Currently the generator looks for system values and can choose not to build libraries off of that. We could probably use that system as if you look at the gradle build system I have mostly working on my fork all the system variables are set though gradle using easy to edit variables in a separate file.
Though it may not be that easy it seemed to work when I was testing it as long as you don't leave out a library needed by other libraries

The build system isn't the issue, it can be implemented in a dozen different ways. The issue is automatically building the artifacts with CI, hosting them somewhere, deploying to Maven Central, etc.
Deciding which libraries to invest our limited time in (it's always the bottleneck) is also a good question. The open issues are there because someone has expressed interest in those libraries, but that doesn't mean we have to support anything and everything, right now. There are priorities and the current top priority is releasing a stable 3.0.0.

Compressonator now supports Linux and macOS.


add jericho setup instructions

Description
Adds setup instructions for the jericho backend, under a Setup header in the project's README.

beautiful


Use kubectl instead of curl to load kube-system manifests

curl can silently fail, leaving the kube system only partially set up
ideally we would use the quay.io/coreos/hyperkube:v1.2.2_coreos.0 image and run /hyperkube kubectl, however this doesn't work due to
https://github.com/kubernetes/kubernetes/issues/24088
this PR uses a 3rd party image for kubectl, which means it shouldn't be merged.
i am hoping someone can help me add kubectl to the hyperkube image directly, or suggest other alternatives

@daveey Great PR!
I had also needed to depend on a 3rd party image in my PR #465
What I have done there was to make image/command passed to docker run fully configurable via cluster.yaml and noted that it may change in the future. See https://github.com/coreos/coreos-kubernetes/pull/465/files#diff-071e0ee4ae8c6adf4865f25d851f30c9R74 for code regarding it.
With that, I believe that there will be small chances we can get our PRs merged as is, because we won't break things in fragile/backward-incompatible ways.
If that is not enough, I guess we can contribute a Dockerfile for kubectl and ask CoreOS guys if they could docker build/push it in a Docker repo something like quay.io/coreos/kubectl.
@colhom @aaronlevy Excuse me for being obtrusive but would you mind looking into this? We like to contribute but the same issue seems to be blocking us.

Couple things to cover here:

As far as the underlying issue, can you give more detail on how curl is silently failing in a way that kubectl would surface more info? If curl fails, I would expect the apiserver response to be in the journal output for the unit - if not we can likely resolve this.

As far as kubectl in the hyperkube container, I would prefer that the flag issue be fixed upstream, rather than trying to work around the bug by including a separate binary or needing to package a separate image. We try to keep our images as close to upstream as possible (and others in the community could also benefit from this fix).

Once the flag bug is resolved, I wouldn't necessarily be opposed to running kubectl via a container (although I'd likely prefer that it was run via rkt). But it would be slightly slower on the installation process (vs curl).

@aaronlevy thanks for the review, I should have posted a link to the issue.
https://github.com/coreos/coreos-kubernetes/issues/447
Take a look at the discussion there which addresses the "apply" issue, as well as why curl is problematic.
I agree with the preference for the original hyperkube image, however the flag bug has been open for a while and not addressed so I don't know when we can expect it.

Commented on the linked issue.
Would you be interested in taking a stab at fixing the upstream issue? On a quick scan it looks like it would mean improving flag parsing here: https://github.com/kubernetes/kubernetes/blob/master/cmd/hyperkube/hyperkube.go#L117

@aaronlevy i won't have time to work on the upstream issue for a while

@daveey I'll be looking into the upstream hyperkube issue soon to see if we can get this resolved.

@daveey update on things. kubernetes/kubernetes#25512 is merged, fixing the hyperkube flag issue.
we're going to wait until v1.3 (with that PR in it) is released before going forward with this PR.

@daveey would this be something you're still interested in tackling? kubectl is now part of hyperkube, so we could now do something like rkt run quay.io/coreos/hyperkube --exec /hyperkube -- kubectl create -f foo.yaml

I'm going to close this due to inactivity - but please re-open if this is something you would like to see merged.


UnicodeDecodeError: 'charmap' codec can't decode byte 0x98 in position 16380: character maps to

Suddenly I'm getting this error when trying to update my host list by running custom profile:
python updateHostsFile.py --auto --backup --extensions porn gambling fakenews
Full output:
Updating source data\adaway.org from https://raw.githubusercontent.com/AdAway/adaway.github.io/master/hosts.txt
Updating source data\add.2o7Net from https://raw.githubusercontent.com/FadeMind/hosts.extras/master/add.2o7Net/hosts
Updating source data\add.Dead from https://raw.githubusercontent.com/FadeMind/hosts.extras/master/add.Dead/hosts
Updating source data\add.Risk from https://raw.githubusercontent.com/FadeMind/hosts.extras/master/add.Risk/hosts
Updating source data\add.Spam from https://raw.githubusercontent.com/FadeMind/hosts.extras/master/add.Spam/hosts
Updating source data\Badd-Boyz-Hosts from https://raw.githubusercontent.com/mitchellkrogza/Badd-Boyz-Hosts/master/hosts
Updating source data\CoinBlockerList from https://raw.githubusercontent.com/ZeroDot1/CoinBlockerLists/master/hosts_browser
Updating source data\KADhosts from https://raw.githubusercontent.com/azet12/KADhosts/master/KADhosts.txt
Updating source data\malwaredomainlist.com from http://www.malwaredomainlist.com/hostslist/hosts.txt
Updating source data\mvps.org from http://winhelp2002.mvps.org/hosts.txt
Updating source data\someonewhocares.org from http://someonewhocares.org/hosts/zero/hosts
Updating source data\StevenBlack from https://raw.githubusercontent.com/StevenBlack/hosts/master/data/StevenBlack/hosts
Updating source data\UncheckyAds from https://raw.githubusercontent.com/FadeMind/hosts.extras/master/UncheckyAds/hosts
Updating source data\yoyo.org from https://pgl.yoyo.org/adservers/serverlist.php?hostformat=hosts&mimetype=plaintext&useip=<IP_ADDRESS>
Updating source extensions\fakenews from https://raw.githubusercontent.com/marktron/fakenews/master/fakenews
Updating source extensions\gambling from https://raw.githubusercontent.com/Sinfonietta/hostfiles/master/gambling-hosts
Updating source extensions\porn\clefspeare13 from https://raw.githubusercontent.com/Clefspeare13/pornhosts/master/<IP_ADDRESS>/hosts
Updating source extensions\porn\sinfonietta from https://raw.githubusercontent.com/Sinfonietta/hostfiles/master/pornography-hosts
Updating source extensions\porn\sinfonietta-snuff from https://raw.githubusercontent.com/Sinfonietta/hostfiles/master/snuff-hosts
Updating source extensions\social from https://raw.githubusercontent.com/Sinfonietta/hostfiles/master/social-hosts
Traceback (most recent call last):
  File "updateHostsFile.py", line 1500, in <module>
    main()
  File "updateHostsFile.py", line 187, in main
    merge_file = create_initial_file()
  File "updateHostsFile.py", line 636, in create_initial_file
    write_data(merge_file, curFile.read())
  File "C:\Program Files (x86)\Python36-32\lib\encodings\cp1257.py", line 23, in decode
    return codecs.charmap_decode(input,self.errors,decoding_table)[0]
UnicodeDecodeError: 'charmap' codec can't decode byte 0x98 in position 16380: character maps to <undefined>

It was working fine few weeks ago. Updated the program to the latest version, but still this error.

Hi Tromi @tormit, what's your python --version?
Try it under python3?

Hi Steven @StevenBlack his log suggests that he is under windows + python 3.6.x :

C:\Program Files (x86)\Python36-32\lib\encodings\cp1257.py

But Tromi @tormit are you using the latest update script? Cause #520 and its children were supposed to fix every Unicode*Error :thinking:

Yes, it's Python 3.6.5 and I have the latest version of this software.
I discovered that data/StevenBlack is causing the problem. Deleted this folder and it generated hosts file successfully.

This time Steven @StevenBlack  it is because we do not touch/convert to idna commented lines ...
The issue is the following
# Office Telemetry Upload Reporting – ‘Heartbeart’ and error events that
                                   ---^

Happening in current HEAD (2246df0c0b0b80c6743d395f8d3c928d29457972)
$ python updateHostsFile.py
Do you want to update all data sources? [Y/n] y
Updating source ./data/adaway.org from https://raw.githubusercontent.com/AdAway/adaway.github.io/master/hosts.txt
Updating source ./data/add.2o7Net from https://raw.githubusercontent.com/FadeMind/hosts.extras/master/add.2o7Net/hosts
Updating source ./data/add.Dead from https://raw.githubusercontent.com/FadeMind/hosts.extras/master/add.Dead/hosts
Updating source ./data/add.Risk from https://raw.githubusercontent.com/FadeMind/hosts.extras/master/add.Risk/hosts
Updating source ./data/add.Spam from https://raw.githubusercontent.com/FadeMind/hosts.extras/master/add.Spam/hosts
Updating source ./data/Badd-Boyz-Hosts from https://raw.githubusercontent.com/mitchellkrogza/Badd-Boyz-Hosts/master/hosts
Updating source ./data/CoinBlockerList from https://zerodot1.gitlab.io/CoinBlockerLists/hosts_browser
Updating source ./data/KADhosts from https://raw.githubusercontent.com/azet12/KADhosts/master/KADhosts.txt
Updating source ./data/lightswitch05 from https://raw.githubusercontent.com/lightswitch05/hosts/master/ads-and-tracking-extended.txt
Updating source ./data/malwaredomainlist.com from http://www.malwaredomainlist.com/hostslist/hosts.txt
Updating source ./data/mvps.org from http://winhelp2002.mvps.org/hosts.txt
Updating source ./data/someonewhocares.org from http://someonewhocares.org/hosts/zero/hosts
Updating source ./data/StevenBlack from https://raw.githubusercontent.com/StevenBlack/hosts/master/data/StevenBlack/hosts
Updating source ./data/UncheckyAds from https://raw.githubusercontent.com/FadeMind/hosts.extras/master/UncheckyAds/hosts
Updating source ./data/yoyo.org from https://pgl.yoyo.org/adservers/serverlist.php?hostformat=hosts&mimetype=plaintext&useip=<IP_ADDRESS>
Updating source ./extensions/fakenews from https://raw.githubusercontent.com/marktron/fakenews/master/fakenews
Updating source ./extensions/gambling from https://raw.githubusercontent.com/Sinfonietta/hostfiles/master/gambling-hosts
Updating source ./extensions/porn/clefspeare13 from https://raw.githubusercontent.com/Clefspeare13/pornhosts/master/<IP_ADDRESS>/hosts
Updating source ./extensions/porn/sinfonietta from https://raw.githubusercontent.com/Sinfonietta/hostfiles/master/pornography-hosts
Updating source ./extensions/porn/sinfonietta-snuff from https://raw.githubusercontent.com/Sinfonietta/hostfiles/master/snuff-hosts
Updating source ./extensions/social from https://raw.githubusercontent.com/Sinfonietta/hostfiles/master/social-hosts
Do you want to exclude any domains?
For example, hulu.com video streaming must be able to access its tracking and ad servers in order to play video. [Y/n] n
OK, we'll only exclude domains in the whitelist.
Traceback (most recent call last):
  File "updateHostsFile.py", line 1513, in <module>
    main()
  File "updateHostsFile.py", line 187, in main
    merge_file = create_initial_file()
  File "updateHostsFile.py", line 640, in create_initial_file
    write_data(merge_file, start + curFile.read() + end)
UnicodeDecodeError: 'ascii' codec can't decode byte 0xc3 in position 394551: ordinal not in range(128)

Using Python 2.

That one is related to #700 @luipugs not this issue.

@StevenBlack : Judging from the conversation, I think this one can be closed?

@gfyoung yup!
Closing.