[docs] <feedback.html include> removal

Description
Related elements or patterns features have been merged and is automatically added to every elements page, so the {% include 'feedback.html' %} is redundant.
Suggested solution
Search elements/rh-* for {% include 'feedback.html' %} and remove it. (Related elements section is automattically added to every page.

docs/patterns/video-thumbnail.njk
213:{% include 'feedback.html' %}

docs/patterns/skip-navigation.njk
89:{% include 'feedback.html' %}

docs/patterns/sticky-card.njk
193:{% include 'feedback.html' %}

docs/patterns/search-bar.njk
295:{% include 'feedback.html' %}

docs/patterns/sticky-banner.njk
216:{% include 'feedback.html' %}

docs/patterns/index.md
54:{% include 'feedback.html' %}

docs/patterns/promo.njk
228:{% include 'feedback.html' %}

docs/patterns/link.njk
384:{% include 'feedback.html' %}

docs/patterns/link-with-icon.njk
135:{% include 'feedback.html' %}

docs/patterns/filter.njk
197:{% include 'feedback.html' %}

docs/patterns/form.njk
500:{% include 'feedback.html' %}

docs/foundations.njk
66:{% include 'feedback.html' %}

docs/patterns/disclosure.njk
227:{% include 'feedback.html' %}

docs/elements/index.md
52:{% include 'feedback.html' %}

docs/elements/elements.html
32:        {% include 'feedback.html' %}

docs/patterns/announcement.njk
162:{% include 'feedback.html' %}

elements/rh-tooltip/docs/30-code.md
25:{% include 'feedback.html' %}

elements/rh-tooltip/docs/20-guidelines.md
74:{% include 'feedback.html' %}

elements/rh-tooltip/docs/10-style.md
67:{% include 'feedback.html' %}


Can't get /perception/inner/PrefusedObjects

System information

OS Platform and Distribution (e.g., Linux Ubuntu 14.04):
Apollo installed from (source):
Apollo version (3.5):

Steps to reproduce the issue:
We successfully run the fusion_camera_detection_component.cc and see obstacle info is published in channel /perception/obstacles in cyber_monitor as well.
However, we are not able to see the channel /perception/inner/PrefusedObjects in cyber_monitor.
In the mean time, we check the log and see send out prefused msg, ts:<PHONE_NUMBER>.174979 ret: 1
I just wondering what go wrong!
Thank you so much.
Supporting materials (screenshots, command lines, code/script snippets):

Same here, we are able to see /perception/obstacles and /perception/camera_debug channels. But we cannot see /perception/inner/PrefusedObjects and /perception/inner/camera_viz_msg which are defined in modules/perception/production/conf/perception/camera/fusion_camera_detection_component.pb.txt.

Currently /perception/obstacles is an only visible topic outside of the perception module.

Closing this issue as it appears to be resolved. Feel free to reopen if the issue persists. Thanks!

Currently /perception/obstacles is an only visible topic outside of the perception module.

Hello I want to configure it to visible topic,could you tell me where I can configure

Currently /perception/obstacles is an only visible topic outside of the perception module.

Hello, I want to configure it to visible topic,could you tell me where I can configure

@techoe  Hello I want to configure it to visible topic,could you tell me where I can configure

@qwetqwe did you ever figure out how to do this? It seems there's a lot of useful information being passed around internally within the perception module. Did you need to create a new node to add to the perception dag which reads the inner channel and republishes it to an external channel?

Hello, I find that /perception/inner/PrefusedObjects is a class type, /perception/obstacles is a proto.
Proto topic can display in cyber_monitor.
I want to know how to display a class type topcic.
Thanks.


pkg_tar should provide includes / excludes glob filter options

The pkg_tar rule should provide glob filter options in order to perform file filtering transforms on existing tar files. My particular use case is stripping out files when creating a Debian package from an existing binary distribution package like test case files and some documentation in go1.12.9.linux-amd64.tar.gz as an example.
Besides being useful for the rules in this package, it should also be useful in combination with rules_docker.

I'm not sure it belongs in pkg_tar or in the rules used to structure packages.
Can you provide an example of the kind of rule set you want to lay out? I can sort of guess you have something like this
pkg_tar(
name = x
deps = [":go1.12.9.linux-amd64.tar.gz"]
exclude_patterns = [ "doc/.", "test/."],
)
I think I would  prefer doing
files_from_tar(
name = "stripped_go",
deps = [":go1.12.9.linux-amd64.tar.gz"]
exclude_patterns = [ "doc/.", "test/."],
strip_path = ....
new_path = ...
)
pkg_tar(
name = x
deps = [":stripped_go"]
)
The reason for my approach is that I would also make a FilesFromTar provider that would include the full transform of the excludsion and path transforms, then use it from pkg_tar, pkg_zip, pkg, deb, pkg_rpm.   I am always going to prefer generic solutions that mix into any package type.

Yes, agree that a more generic solution would be preferable. I could see something like files_from_tar being useful here and also other contexts.
One possible motivation to build filtering into the library that backs the all the rules is that it seems like it would skip putting files on disk. like the ones in :stripped_go above, that would not be referenced outside of building another package -- but I don't have any specific use cases where that matters for me.

@aiuto, I'm still interested in this, but I don't think my original request of transforming an existing tar input is as important as just providing includes and excludes on the items in srcs. Your proposal of a files_from_tar rule would still be useful of just extracting the archive into a filegroup for further processing.
For example given a @npm_prod repo which was generated from a yarn_install, I currently need to do something like the following just to filter out some undesired files when packaging up a NextJS app when using rules_docker that are unavoidable even when using --production arg for yarn_install:
# Package with destination directory to avoid possible dot dir flattening.
# https://github.com/bazelbuild/rules_docker/issues/1974
pkg_tar(
    name = "app-node-modules-unfiltered",
    srcs = [
        "@npm_prod//:node_modules",
    ],
    # Remap external repo files.
    # https://github.com/bazelbuild/rules_pkg/issues/251#issuecomment-499464418
    remap_paths = {
        "../npm_prod/": "/app/",
    },
    # Don't flatten directories.
    # https://github.com/bazelbuild/rules_docker/issues/317
    strip_prefix = "/",
)

# Package up node_modules ready to go in the container /app directory
genrule(
    name = "app-node-modules",
    srcs = [
        ":app-node-modules-unfiltered",
    ],
    outs = ["app-node-modules.tar"],
    cmd = """
    TMP=$$(mktemp -d || mktemp -d -t bazel-tmp)
    trap "rm -rf $$TMP" EXIT
    tar -C $$TMP -xf $(location :app-node-modules-unfiltered)
    tar -C $$TMP \\
        --exclude='app/node_modules/@next/swc-linux-x64-gnu' \\
        --exclude='app/node_modules/@next/swc-linux-x64-musl' \\
        -hcf $(location app-node-modules.tar) app
    """
)

# Use app-node-modules.tar in tars of a container_image rule...

Or alternatively:
genrule(
    name = "node-modules",
    srcs = [
        "@npm_prod//:node_modules",
    ],
    outs = ["node-modules.tar"],
    cmd = """
    TMP=$$(mktemp -d || mktemp -d -t bazel-tmp)
    trap "rm -rf $$TMP" EXIT
    tar -C external/npm_prod \\
        --exclude='node_modules/@next/swc-linux-x64-gnu' \\
        --exclude='node_modules/@next/swc-linux-x64-musl' \\
        -hcf $(location node-modules.tar) node_modules
    """
)

container_layer(
    name = "app-node-modules-layer",
    directory = "/app",
    tars = [
        ":node-modules"
    ],
)

It would be really nice to not use a genrule step since one needs to be aware of portability, e.g., macOS's tar does not have --transform to easily create something like an /app prefix. Having includes and excludes also matches up with the expectation of basic GNU tar functionality.
I'd like to see something like:
pkg_tar(
    name = "app-node-modules",
    srcs = [
        "@npm_prod//:node_modules",
    ],
    # Remap external repo files.
    # https://github.com/bazelbuild/rules_pkg/issues/251#issuecomment-499464418
    remap_paths = {
        "../npm_prod/": "/app/",
    },
    # Don't flatten directories.
    # https://github.com/bazelbuild/rules_docker/issues/317
    strip_prefix = "/",
    # Remove development files based on the remapped paths
    excludes = glob([
      "app/node_modules/@next/swc*/**",
    ]),
)

In terms of implementation, would filtering content_map before writing the manifest technically work? It looks like if an entry isn't in the manifest, then it isn't going to be added to the output.
But to make it work for all archive types, maybe just pass some includes and excludes filter parameters to methods like add_single_file in pkg/private/pkg_files.bzl?

Filtering before writing the manifest might be sufficient.  But I think your example with a genrule is more flexible. The idiom is that we would filter in a pkg_files (or expand_tar) rule between the input archive (.tar, .npm, ...) and the final packaging. That would allow for fine grained filtering and remapping from multiple input sources.
One thing to consider is that remap_paths from pkg_tar is going away. It is a confusing piece of technical debt in the code so are goal is to eliminate it. Instead, pkg_files can strip out existing paths and rebase them with new ones. I am presuming from your examples that you need to remap paths, the eventual way to write that is with an intermediate pkg_file target to do that remapping. Adding the filtering to that target is equivalent to doing it in pkg_tar.
I still have to think about how either solution will compose efficiently. The ideal state is that if you ask for
existing .tar, .npm, .jar, ... => rules that remap paths and filter files   =>  pkg_{zip, rpm, tar, deb, cab}

Then the series of actions we produce will not expand the .tar into individual files to send to the final packaging. Instead, we would ship the original .tar to the final packaging action and the build_tar helper would act on instructions from the manifest (or similar) to unarchive and repackage. I would also like to limit the number of times we have to scan the input archive.
I can make this fairly efficient for  deb X tar X zip, since I can unpack inputs from the same process as the one writing the final archive. rpm is different because we need to use an external writer (rpmbuild). The thing in my mind right now is that tar_expand could be a macro that produces two targets, one is just a provider of remap and filter information, the other is a tree artifact of the untarred (and filtered) tar file. A pkg_rpm rule would have to take the tree artifact as srcs. That would avoid the unfurling when not needed, but would be ugly because the pkg_rpm instance would have to use the name of the artifact target.

I've got it over here on a fork
https://github.com/bazelbuild/rules_pkg/compare/main...aspect-forks:rules_pkg:main

@matt-sm When the tar manifest is a public API, this is trivially done with any rule that modifies the manifest file. Since we have a plain BSD tar/mtree rule https://github.com/aspect-build/bazel-lib/blob/main/docs/tar.md I don't plan to spend time on that here in rules_pkg and am just migrating rules I bump into to use that instead.


Common backend interface

Good day,
I was thinking about common interface for all backends. This PR has an example of how it could be done, the main idea is to get dynamic backend loading without static lists in init.

Hi, thanks for your PR. I also think it's a better way to do that.
And this is related to #10 that I've just opened, maybe you can implement this feature in this PR too ?

Hi, I'll try to do it during weekend.

Added some code to cover #10. Added a small test for get_backend

Can one of the admins verify this patch?

@dmajere @philpep Is anyone working on this?
Though the code looks very promising to me, it has been several months since this PR created.
Today it has some conflicts but I guess I have some time to resolve them and recreate PR.

@dmajere @philpep Is anyone working on this?
Though the code looks very promising to me, it has been several months since this PR created.
Today it has some conflicts but I guess I have some time to resolve them and recreate PR.

@mumoshu note that this can currently be done by overriding the TestinfraBackend fixture in a conftest.py and implementing a testinfra.backend.base.BaseBackend like class.
I'm not sure yet about implementing this feature like this PR does, I must be more convinced. What custom backend would you like to use ? Maybe I could try to implement it outside of testinfra, this will give me an idea of required changes (or not) in testinfra.
(sorry for the late response)


set-akshcinodepool -vmsize not documented

Hello,
when reviewing the powershell doc for the "set-akshcinodepool" command, the -vmsize parameter is not documented.
The parameter is only mentioned in the upscaling doc:
https://learn.microsoft.com/en-us/azure/aks/hybrid/concepts-vertical-node-pool-scaling
Can the parameter be added to the command documentation? the command mentioned in the article about upscaling is working fine:
Set-AksHciNodePool -ClusterName mycluster -name mycluster-linux -vmsize Standard_A4_v2
Link: https://learn.microsoft.com/en-us/azure/aks/hybrid/reference/ps/set-akshcinodepool
or https://github.com/MicrosoftDocs/azure-stack-docs/blob/main/AKS-Hybrid/reference/ps/set-akshcinodepool.md
Thanks

@azvise
Thanks for your feedback! We will investigate and update as appropriate.

@azvise
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

#assign:sethmanheim

Hi @azvise, thank you for pointing this out. I have a PR filed to fix this and am just waiting for some further information from the product team. Should be updated in a day or two. #please-close


Typos

Q
A

content fix?
yes

New article?
no

Related issues?

What's in this Pull Request?
Simply fixes a couple of typos.

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

thx Korey, appreciated.


[en] Fixes typo in documentation

@DhritiShikhar, It seems this PR includes changes to files across multiple languages i.e. English and Chinese. The documentation process varies for each localization. So you have to make changes in the Primary website content which is English content with this PR. Once English content PR is merged, you can raise PR for other languages.

hold until this PR only covers one language (en)
/hold
remove the hold label once the PR is modified to one language

@DhritiShikhar, It seems this PR includes changes to files across multiple languages i.e. English and Chinese. The documentation process varies for each localization. So you have to make changes in the Primary website content (English content) with this PR. Once English content PR is merged, you can raise PR for other languages.

I have removed other changes. Now, this PR fixes typo in English documentation only. I will raise another one for Chinese documentation changes. Can you please take a look again @Shubham82 ?

Thanks, @DhritiShikhar for this fix
The preview looks good to me.
https://deploy-preview-47060--kubernetes-io-main-staging.netlify.app/docs/concepts/cluster-administration/flow-control/#recursive-server-scenarios
/lgtm

/unhold

/remove-language zh
/remove-area localization


Fix races in subscribe

by @sungshik

Closing this one, after some discussion, we solved it in a different way.


⚠ Bofrost.fr has degraded performance

In 5fc0ec9, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 870 ms

Resolved: Bofrost.fr performance has improved in e908bd1.


Pwsh crashing with Not enough quota is available to process this command

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
Pwsh crashes with this error:
Application: pwsh.exe
CoreCLR Version: <IP_ADDRESS>210
.NET Version: 6.0.0
Description: The process was terminated due to an unhandled exception.
Exception Info: System.ComponentModel.Win32Exception (1816): Not enough quota is available to process this command.
   at MS.Win32.UnsafeNativeMethods.PostMessage(HandleRef hwnd, WindowMessage msg, IntPtr wparam, IntPtr lparam)
   at System.Windows.Interop.HwndTarget.UpdateWindowSettings(Boolean enableRenderTarget, Nullable`1 channelSet)
   at System.Windows.Interop.HwndTarget.UpdateWindowPos(IntPtr lParam)
   at System.Windows.Interop.HwndTarget.HandleMessage(WindowMessage msg, IntPtr wparam, IntPtr lparam)
   at System.Windows.Interop.HwndSource.HwndTargetFilterMessage(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
   at MS.Win32.HwndWrapper.WndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam, Boolean& handled)
   at MS.Win32.HwndSubclass.DispatcherCallbackOperation(Object o)
   at System.Windows.Threading.ExceptionWrapper.InternalRealCall(Delegate callback, Object args, Int32 numArgs)
   at System.Windows.Threading.ExceptionWrapper.TryCatchWhen(Object source, Delegate callback, Object args, Int32 numArgs, Delegate catchHandler)
   at System.Windows.Threading.Dispatcher.LegacyInvokeImpl(DispatcherPriority priority, TimeSpan timeout, Delegate method, Object args, Int32 numArgs)
   at MS.Win32.HwndSubclass.SubclassWndProc(IntPtr hwnd, Int32 msg, IntPtr wParam, IntPtr lParam)

It happens mainly when I call some external native commands (Julia script), but I'm not sure whether it's related.
How it behaves:

In ConEmu: the tab simply disappears
In Windows terminal: the message about the quota is shown in the window and the promp disappears. I can only close the tab munually. Nothing else.
The pwsh in

Expected behavior
No crash..

Actual behavior
Crashing.

Error details
I experienced this problem in Windows Terminal, so I switched to ConEmu and it's still happening.

Environment data
$PSVersionTable

Name                           Value
----                           -----
PSVersion                      7.2.0
PSEdition                      Core
GitCommitId                    7.2.0
OS                             Microsoft Windows 10.0.19041
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals

/cc @DHowett @daxian-dbw for information.

@stej Can you please share the specific script/command you ran that triggered this crash?
The event and the call stacks suggest the exception was thrown from the .NET runtime.
If you are able to consistently reproduce the issue, can you please collect a crash dump by following the instructions from https://docs.microsoft.com/en-us/dotnet/core/diagnostics/dumps#collect-dumps-on-crash?
Please use MiniDumpWithFullMemory as the dump type, by setting DOTNET_DbgMiniDumpType to 4.

Sure, I'll try. Meanwhile the command line looks like this:
cls; $f='c:\temp\values.csv';$pre=[float](Import-Csv $f)[-1].Total; C:\util-setdateToCurrent.ps1 ; C:\temp\runDownloadsAndAnalysis.bat; $post=[float](Import-Csv $f)[-1].Total; $diff="Difference: $([int]($post-$pre))`$"; if ($diff -match ' -') { Show-Warning $diff } else { Show-Info $diff }

(one line).
Where

util-setdateToCurrent.ps1 calls Set-Date with some time
runDownloadsAndAnalysis.bat - calling pwsh (creates new process), so I don't think it's causing the problem
Show-Warning and Show-Info - this is my bet.

The body:
function Show-WpfMessage($title, $str, $headingBackColor, $messageBackColor, $messageColor, $height, $width)
{
  Add-Type -Assembly PresentationFramework
  Add-Type -Assembly PresentationCore
  $xaml = @"
<Window xmlns="http://schemas.microsoft.com/winfx/2006/xaml/presentation" xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml"
  Title="$title" Height="$height" Width="$width"
  Opacity="10" OpacityMask="Green"
  Topmost="True" WindowStyle="None" AllowsTransparency="True" Background="Transparent" Padding="20" Top="10" Left="10">
  <DockPanel>
    <Border DockPanel.Dock="Top" CornerRadius="10" BorderThickness="2" BorderBrush="Black" Background="$headingBackColor"
      Padding="8" Margin="10">
      <TextBlock FontSize="52" TextWrapping="Wrap" Margin="20" HorizontalAlignment="Center">$title</TextBlock>
    </Border>
    <Border DockPanel.Dock="Bottom" CornerRadius="10" BorderThickness="1" BorderBrush="Black"
      Padding="8" Margin="10" Background="$messageBackColor">
      <TextBlock Foreground="$messageColor" FontSize="42" Padding="60" TextWrapping="Wrap" HorizontalAlignment="Center">$str</TextBlock>
    </Border>
  </DockPanel>
</Window>
"@

  $global:window = [Windows.Markup.XamlReader]::Load((New-Object System.Xml.XmlNodeReader ([Xml]$xaml ) ) )
  $window.Add_MouseLeftButtonDown(  { $_.Handled = $true; $window.DragMove() } )
  $window.Add_MouseRightButtonDown( { $_.Handled = $true; $window.Close()   } )
  $window.ShowDialog()
}

function Show-Info($str) {
  Show-WpfMessage "Information" $str -headingBackColor LightBlue -messageBackColor White -messageColor Black 400 800
}
function Show-Warning($str) {
  Show-WpfMessage "Warning" $str -headingBackColor Yellow -messageBackColor Red -messageColor Black 600 1200
}


Add Chrome sync

I wanting Chrome Bookmark sync plss.
this browser will be best if added.
Thanks

Currently you can import Chrome html bookmark file to Ninja.

Yes, but need chrome sync so i take my bookmarks every where i go.
Pls do. Pls.

OK, let me thinking..

Plsss, I be your favor!!

Can add a option to open play store links in play store? twitter links in twitter? facebook links in facebook apps and other thing? Please do it. Now, it opens play store links in a webpage, but I want open it in play store. HELP PLS!!

OK I will try but currently mainly fix bugs.

where you? I not see update many days, give good update fast pls

take time but give good update with new feature, make nice

Sorry I feel tired and I have to prepare the final exam, so pause upgrade.

password not save? I try but it not save. Pls fix.
And how I open links in front? I dont want open in background, pls add too


how do you dump the bootrom

how do you dump the bootrom？ it can be read directly？
I try to read stm32 bootrom but failed！
please

Hi!
So basically you have to look where the ROM resides, from which address to which address, normally you can find this in the memory mapping documentation, and then flash a code that looks like this into the device:
for(Addr=(unsigned char *)0x1FFF0000;Addr<=(unsigned char *)0x1FFFBFFF;Addr++)	 {		 printf("%02X",*Addr);	 }
This repo has a code for dumping STM32F401RE bootloader:
https://github.com/qais744/EmbeddedSoftware

thanks , I got it .


WizardSelect disabled attribute only disables select field not toggle switch

Describe the bug
The component wizard-select does not have a properly working disabled attribute. At the moment this attribute is inherited from mwc-select used within wizard-select and is therefore only disabling the text field not the toggle switch.
To Reproduce
Open publisher plugin and have a look at any of the text fields there. Those should be disabled completely, but the toggle switch is still active.
Expected behavior
Ideally, wizard-select would have a disabled attributed that disable both the text field and the toggle button.

This needs a re-design.


The properties "info" and "default" do not work on fields inside a field type "set"

Using version 0.10.0
This is my options object for the "set" field:
{
  "fields": [
    {
      "name": "siteName",
      "label": "Site Name",
      "info": "some stuff",   <- does not show
      "default": "hello",    <- does not show
      "type": "text"
    },
    {
      "name": "image",
      "type": "image"
    }
  ]
}

Yes this is missing for sets! Thanks for the hint!


filter by message type in the client CORE-3915

@maxtaco @songgao @patrickxb r?
This filters by message type on the client

I'm going to merge this, since it is out of sync with Gregor right now, and I think the error case is probably fine, and will likely be fluid anyway as GUI development starts.


A better name?

rav1e is great and all, but it doesn't quite roll off the tongue. Let's comment on this issue with name ideas, and then vote on them using 👍.

Rave

The 'r' of rav1e made me think of the R of Racing version. If #1 goal of this project is SPEED.
So, I'd propose 'av1R'  or 'av1Rus'.
Btw, I guess you mean the 'Rave', the English word?

The 1 is silent, I'm ok with rav1e.

I still like ravioli (with or without the i as 1).  I would also support xav1 :)

ravioli

Is not a serious enough encoder name. I guess Android names are not serious either, ...

Yes, we wanna go sth like: 2F2F, Need For Speed.

So, nobody likes av1Rus (didn't mean 'toysRus' in the beginning :)

rav1e seems search friendly so far.
It can be pronounced as rave or ravìe for additional confusion points during the presentations.
I'm sure we can come up with enough shape of pasta for the release codenames. (I book the first pre release as plin)

TGV encoder, as in The Greatest Video encoder, but mostly as a reference to the Train à Grande Vitesse.

I like another suggestion of smarter's, "zeppelin":
<smarter> ltrudeau: we could call the encoder zeppelin

Or, I can even name it from the Street name where TDLinux's apartment is located.
What was it, TD?

Metropole
as an hommage to the best hotel in Brussels. When in Brussels, any self respecting person stays at the Metropole as it not only the best but it is not a family friendly hotel, and thus does not inherit all the faults of family friendly hotels.

ROFLCoder
         ________________
         \  [ROFLCoder]   \
1010101   \                \   L0L
           \ ________________]

Closing for now since it does not seem like a name change is expected, but feel free to suggest more very serious and professional names. For everyone's consideration there is also @smarter's latest suggestion Ramen.


Additional examples of running v2

This looks fantastic. However - I run the main script but how do I have it call out to the newer items for v2?
can you provide some example of running this with the new module. Thanks!

Cloned the entire repo. Ran without issues! Thanks


Detect from image not work

Describe the bug
Detect from image not work
To Reproduce
click Detect from image，select a picture，and then error
Screenshots

remove stable-diffusion-webui\models\body_pose_model.pth and relaunch will work for you


Fix #469 updating sidebar interaction model

As per issue here: https://github.com/duckdb/duckdb-web/issues/469
I think this should improve navigation, adding a reachable Sitemap top-level + having only leaves having content (this is more visible on mobile, whether a click means both open sub-menu AND go to page)
I personally think this makes sense, but would be nice to have someone (@Alex-Monahan, @Mause, ?) taking a proper look at this.

Ah, let me see if I understand what you mean. Each section header should be a link that takes you to the overview page directly? Or do we want to avoid that behavior and only expand the tree menu when clicking / tapping on a header?

I am not completely sure this makes sense, but the idea behind this is:
Each 'leaf' node is an article, and each 'internal' node only expand the menu. Moving eventual content from the 'internal' nodes to an added 'Overview' leaf node as first child of the internal node.
Plus moving the content currently reachable from here (https://duckdb.org/docs/), but that you can't go back from the docs itself while navigating, to a 'Sitemap' own category (that itself is a 'leaf' node of the tree).
I am not completely sure this works, since I have a hard time building this on work mac thanks to dependency hell, so would not trust this without a proper look from someone else.

I'm supportive of this - we should also fix the Guides heading though

Now I understand! This is much nicer. Thanks!
@Mause - I believe this fixes the guides heading already? Or is there more to fix?

@Alex-Monahan oops you're right 😅


Add a diagnostic service to allow clients to subscribe to server logging streams.

Essentially we want a way for clients to register interest in logging information from the server.  Potentially, we want some kind of level idiom (info or fine/finer).
In short order we want to do away with hacks like https://github.com/dart-lang/sdk/issues/24900 where for expedience we're using an error notification to carry what is really just information.

Would love to have this on a general purpose diagnostics domain :)

Would love to have this on a general purpose diagnostics domain :)

:+1:

Thoughts on this? Is it important to add this to the diagnostic domain, or does the existing server.error stream work ~well enough for this purpose?

does the existing server.error stream work ~well enough for this purpose?

Not as it's defined but maybe that's a better place to look.  What we really want is something like server.info.

In my mind, the question is what kind of information are we passing back and why?
In the case of #24900, this is a short term work-around. The longer term goal is to remove the need to run pub list-package-dirs entirely, at which point we won't need the notification. I'm sort of ok with using server.error for short-term solutions in order to avoid creating new API that won't be needed very long (well, depending on your definition of "long").
Is there other information that we think it's useful to pass back to clients?

@pq Are we running on this for 1.14?

@kevmoo : not for 1.14...

Closing as stale.


Fix newline parsing in pgpass

Previous behavior
When parsing pgpass files, the BufReader's read_line function is used which does not strip \n or \r. To handle this, a substring was created that strips out the last character (&line[..line.len() - 1])
Why is this an issue
This behavior fails in two cases:

If the file uses \r\n instead of just \n
If the file has no trailing newline

Case (2) is subtle because it means if you modify your pgpass to add a new entry and forget to add a trailing newline to your file, [..line.len() - 1] will cut off the last letter of your password (ex: hunter12 -> hunter1) giving you an authentication error.
How can we fix it?
Two options:

Similar to my PR, we manually strip the newline characters from the end of the string
We switch from the read_line to the lines method which strips the newline characters for you (I don't think this is a performance concern because pgpass files are presumably not large files)

Tests
I noticed there weren't any tests for load_password_from_file  -- only for load_password_from_line. I assume it's because setup for a test on file reading is tedious and any issue would get caught integration tests. However, this means I'm not sure what you suggest for a test for this specific case. Please let me know if you're okay without a test or if you have a specific way you'd like me to set it up (or push to my branch yourself)

If you want to generate a file and check it in to tests/postgres/, that sounds fine. The login credentials are hardcoded as postgres/password.

Just wrote a unit test combining the other test cases into a single "file", although it required a bit of modification.


The list of channels is not displayed

 static StreamWriter WTelegramLogs = new StreamWriter("WTelegram.log", true, Encoding.UTF8) { AutoFlush = true };
        static Client client;
        static void Main() => Initialize().GetAwaiter().GetResult();
        static async Task Initialize()
        {
            Helpers.Log = (lvl, str) => WTelegramLogs.WriteLine($"{DateTime.Now:yyyy-MM-dd HH:mm:ss} [{"TDIWE!"[lvl]}] {str}");
            Console.WriteLine("[API] Connecting...");
            client = new Client(Config);
            await client.LoginUserIfNeeded();
            Console.WriteLine("[API] Connected!");
            await displayAllChannels();

        }
        static async Task displayAllChannels()
        {
            var chats = await client.Messages_GetAllChats();
            foreach (var (id, chat) in chats.chats)
                Console.WriteLine($"{id}: {chat}");

            Console.ReadKey();
        }

Logs:
2023-01-09 11:27:57 [I] Loaded previous session
2023-01-09 11:27:57 [D] WTelegramClient 3.2.1 running under .NET 6.0.11
2023-01-09 11:27:57 [I] Connecting to <IP_ADDRESS>:443...
2023-01-09 11:27:58 [D] 5>Sending   InvokeWithLayer`1                        #C8A9
2023-01-09 11:27:58 [D] 5>Receiving MsgContainer                             2023-01-09 08:27:58Z (svc)
2023-01-09 11:27:59 [D]             → NewSessionCreated                      2023-01-09 08:27:58Z
2023-01-09 11:27:59 [D]             → MsgsAck                                2023-01-09 08:27:58Z (svc)
2023-01-09 11:27:59 [D] 5>Receiving RpcResult                                2023-01-09 08:27:58Z
2023-01-09 11:27:59 [D]              → Config                                #C8A9
2023-01-09 11:27:59 [I] Connected to DC 5... phonecalls_enabled, default_p2p_contacts, revoke_pm_inbox, has_reactions_default
2023-01-09 11:27:59 [D] 5>Sending   MsgContainer                             2023-01-09 08:27:58Z (svc)
2023-01-09 11:27:59 [D]             → Users_GetUsers                         #CDB6
2023-01-09 11:27:59 [D]             → MsgsAck
2023-01-09 11:27:59 [D] 5>Receiving RpcResult                                2023-01-09 08:27:59Z
2023-01-09 11:27:59 [D]              → UserBase[]                            #CDB6
2023-01-09 11:27:59 [D] 5>Sending   MsgContainer                             2023-01-09 08:27:59Z (svc)
2023-01-09 11:27:59 [D]             → Messages_GetAllChats                   #FD47
2023-01-09 11:27:59 [D]             → MsgsAck
2023-01-09 11:27:59 [D] 5>Receiving RpcResult                                2023-01-09 08:27:59Z
2023-01-09 11:27:59 [D]              → Messages_Chats                        #FD47

Console Output:

Just tried your exact code. Worked for me. 🤷🏻♂

(closing because I think the problem is on your side, not with the library)


Change graphviz font family to sans

[X] Tests passed
[X] Passes black dask / flake8 dask / isort dask

I was working on #7869 and realized that the Graphviz output would look even cleaner if we use a sans-serif font instead of the current serif font.
In this PR, I have changed the font of the Graphviz from Times-Roman to Roboto. I have also cleaned the code in the method to_graphviz() to be more consistent in dot.py (LOW LEVEL) and highlevelgraph.py (HIGH LEVEL). Now, they look similar and it might be easier to work with this code in the future.
It follows the https://marketing.dask.org/en/latest/colors.html

(Font mentioned in image: Roboto)
Even the HTML Representation uses a Sans font, so it makes sense to have the Graph representations also follow suit.
Demo
import dask.array as da

x = da.ones((10, 10), chunks=(5, 5))
x = x + 100
x = x * 100

x.visualize()

Before

After

import dask.array as da

XL = da.ones((67552, 365941), chunks=(652, 5792))
XL2 = XL.T.astype("f4")
XC = da.ones((365941, 26), chunks=(365941, 26))
LS = da.linalg.lstsq(XC, XL2)[0]
XC_LS = XC @ LS
XLP = XL2 - XC_LS

XLP.dask.visualize(\)

Before

After

/cc @GenevieveBuckley @martindurant

My personal opinion here is the same as described in https://github.com/dask/dask/pull/7869#issuecomment-879533398

I'm mildly supportive of the suggestion to use a sans serif font for consistency with other representations of the high level graph, but I wouldn't die on this hill if other people dislike it. Now that the font size is often a lot bigger, it certainly seems more like a bunch of headings (where sans serif is typically expected) rather than paragraph-like text.

I am also +1 here, so lets merge in a day if there is no dissent.

This is ready to merge, if no one has any issues.
/cc @martindurant @jrbourbeau

:yellow_heart:


New fill modes for lines

Allows the user to customize how the line fills.
A new fillMode property configures this. It defaults to 'zero' which matches the current behaviour. It can also be set to 'top' or 'bottom'.
Top Mode

Bottom Mode

Fixes #3176

Looks good but just thinking: do we really need a new option?
fill: true|false|'zero'|'top'|'bottom'

fill: true would an alias to fill: 'zero'

I can condense down to a single option

@simonbrunel updated to use a single option

#3176


Bug: Container registry having '/' in the name cannot be edited

📜 Description
If a container registry has '/' used in the registry name, the registry fails to expand once saved.
👟 Reproduction steps
Save a container registry with '/' in the name eg. abc/def
👍 Expected behavior
We should not allow saving registry with '/' in the name
👎 Actual Behavior
We allow saving registry with '/' in the name, as a result the registry fails to expand once saved.
☸ Kubernetes version
NA
Cloud provider
NA
🌍 Browser
Chrome
🧱 Your Environment
No response
✅ Proposed Solution
Add validation for registry name, we can use the same validation what we have for application name
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find any similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

AB#6157

ADO issue: https://dev.azure.com/DevtronLabs/Devtron/_workitems/edit/5768/


Don't Merge! PR real time test for source branch full build

Docs Build status updates of commit c0c8d8c:
:clock10: Pending: waiting for processors (20 builds ahead of you)
:warning: Docs Build is busy, currently there are 20 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit c0c8d8c:
:clock10: Full build: the average full build time is 7 min(s) 45 sec(s), this is based on the last 30 build(s) for this repository.

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

File
Status
Preview URL
Details

docfx.json
:white_check_mark:Succeeded

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


python: No matching distribution found for requirements.txt

Hey there,
the dependency kafka-python is not installed.
error: from install container
/kubeless/requirements.txt: OK
WARNING: The directory '/.cache/pip/http' or its parent directory is not owned by the current user and the cache has been disabled. Please check the permissions and owner of that directory. If executing pip with sudo, you may want sudo's -H flag.
WARNING: The directory '/.cache/pip' or its parent directory is not owned by the current user and caching wheels has been disabled. check
the permissions and owner of that directory. If executing pip with sudo, you may want sudo's -H flag.
ERROR: Could not find a version that satisfies the requirement requirements.txt (from -r /kubeless/requirements.txt (line 1)) (from versions: none)
ERROR: No matching distribution found for requirements.txt (from -r /kubeless/requirements.txt (line 1))

requirements.txt
kafka-python

function.yaml
apiVersion: kubeless.io/v1beta1
kind: Function
metadata:
  name: python-demo
  namespace: kubeless
  labels:
    created-by: kubeless
    function: python-demo
spec:
  runtime: python3.7
  timeout: "180"
  handler: function.app
  deps: "requirements.txt"
  checksum: sha256:FUNCTION-CHECKSUM
  function: "FUNCTION-URL"
  function-content-type: url+zip

any ideas how to debug this ?

if your function YAML, deps should be the content of the requirements file (not the path to the file). In your case, it should be enough if you change:
apiVersion: kubeless.io/v1beta1
kind: Function
metadata:
  name: python-demo
  namespace: kubeless
  labels:
    created-by: kubeless
    function: python-demo
spec:
  runtime: python3.7
  timeout: "180"
  handler: function.app
  deps: "kafka-python"
  checksum: sha256:FUNCTION-CHECKSUM
  function: "FUNCTION-URL"
  function-content-type: url+zip

Thats not how we do it for node js. I wish you could change this to use the requirements file. how do I add multiple deps then ?


Fix removed add-path command

Fixes https://github.com/deadsnakes/issues/issues/135
Before the fix: job
After the fix: job
See new usage.
See removal log.


Pyodbc-azure unique key error

Hi, I am facing the same issue as #45. Whenever i am trying to add new ticket, I am getting error ,Violation of UNIQUE KEY constraint 'UQ__river_tr_6fe987667dhdjsj008'. Cannot insert duplicate key in object 'dbo.river.transitionapproval'. The duplicate key value is (). I am using River 3.0.0,Pyodbc-azure <IP_ADDRESS>.
Whats the solution for this??

For transactioner_id, it is taking value as None instead of auth_user.id
Also, for transaction_date , it is taking None. I feel that is the cause of error. Can you also, point out ,location of  code where when I add data in my model, transitionapproval table population triggers.

So, i edited the columns it was giving error on from table. Now , i am getting new error, Incorrect syntax near 'cte'. Incorrect syntax near ','. What is the meaning of this? And how to resolve this?

Where can I find the code description for get_available_approvals method? It seems it is not able to get object from transitionapproval table for the user.

One more error i got with cte was, SQL Server Error, “Invalid usage of the option FIRST in the FETCH statement.Incorrect syntax near 'cte'. Incorrect syntax near ','. "

Hi,
@PS-CR7 , simple answer to all of those is that, I am afraid, django-river doesn't support Microsoft SQL Server. I am looking into this to customize the SQL queries when the driver is for Microsoft SQL Server but it doesn't seem to be able to go out soon.

I am not promising anything but at least we have a ticket ( #141 ) for this which you can follow;

Yeah, appreciate you updating about it. I was thinking the same that it doesn't support, because of errors. Thank you .

Hi, I just want your thoughts on this. My requirement is to use river for api only. So, If I can host my project data on sql server and Workflow data on sqlite db. Do you think, it will work?

Also, how much is river customizable. I am not using Django user instead my own user table. Also,  transition and transition approval table will be populated using code in views.py in my case,since I will be reading data from react ui and then populate it. Will that work?

The only dependency of djangor-river with your data model is that your model contains a state field that is a foreign key to django-river's State model. So I don't think it is possible to separate these two to separate databases since you can't have foreign keys cross databases ( cross-database-relations ). But it sounds like an interesting idea. I will think about this

Also, how much is river customizable. I am not using Django user instead my own user table. Also, transition and transition approval table will be populated using code in views.py in my case,since I will be reading data from react ui and then populate it. Will that work?

django-river uses the given user model that is specified with settings.AUTH_USER_MODEL. As long as you have that variable set to be a different user model, django-river should work just fine. But feel free to report a bug if it doesn't work. Because it is meant to be working.

Hi again @PS-CR7 ,
I think django-river can now support Microsoft SQL Servers after I put some love into it  😄  🎉 🎉 🎉
But it is not merged yet and it is definitely up for testing. It is on a branch named issue-141-support-microsoft-sql . Can you give it a try to see if you observe any problem so that we can improve it even? You can install it from this branch of the repo instead of pypi.

This problem should go away after version 3.2.0 which has already been published. Try it out and report if you observe any new problems


Incorrect documentation?

https://github.com/schteppe/cannon.js/tree/master/examples#threejs
rigidbody.position.copy(mesh.position);
rigidbody.quaternion.copy(mesh.quaternion);
Shouldn't it be the other way around?

Yes it should! Thanks.


Question: Beacon with no power

I'm using EID to my solution and sometimes a beacon gets their battery out of power. So i'm having problems with the sync with the Google Resolver, to resolve this beacon that stopped clocks it's clock. Anyone knows how much time it takes to the Google Solver solve this issue?
Here and here says respectively:

The Google Proximity Beacon API uses sightings of the beacon to infer each beacon's characteristic clock drift in order to ensure high reliability. The system is robust against power outages of several days, though re-synchronising the beacon clock's offset may take some time for longer outages.

The trusted resolver should implement resolution over a time window sufficient to allow for both reasonable clock drift and this type of power loss recovery.

So, i'm wondering if the google api is a trusted resolver, because i'm lossing lots of beacons and all the time that I loose a beacon I have to register the beacon again and set it up again.

If the beacon internally loses its registration clock offset and its Identity Key, you can consider it dead. Since the EID "resolving" is nothing more than a huge database, it's useless to try a "clock drift resync" (whatever that means) if the beacon is computing the new EIDs using an incorrect current timestamp (in hours, days, or years). To recover and restart EID beacon which lost power you'd need a few pieces of information:

"cold" data: the registration clock, Identity Key, and exponent
"warm" data: a synchronized method to get a correct relative time (for example the current correct time, which you could pass somehow into the beacon after a restart). You need this to compute the initial and continuous EID keys, because this is the same relative time base the EID "solver" will use to generate the future keys as well.

So my guess is that your problem is related to not being able to continue advertising of an existing EID beacon.


Need early bandage option for genesis start

e.g. summer genesis start with testnet-prod blockchain configs.

Fixed.


[SPARK-5101] Add common ML math functions

When x is positive and large, computing math.log(1 + math.exp(x)) will lead to arithmetic
overflow. This will happen when x > 709.78 which is not a very large number.
It can be addressed by rewriting the formula into x + math.log1p(math.exp(-x)) when x > 0.

Test build #25112 has started for   PR 3915 at commit 6b3ca72.

This patch merges cleanly.

Test build #25112 has finished for   PR 3915 at commit 6b3ca72.

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

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

Test build #25120 has finished for   PR 3915 at commit e4957d5.

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

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

For MLOR case, it's not as simple as binary case, but I managed to address it. Will be in another PR.
Actually, this solves instability issue when the initial weights is not properly set.
Since in this case, the margin will be very large, and it's just overflow.

Test build #25150 has started for   PR 3915 at commit 23144f3.

This patch merges cleanly.

Test build #25151 has started for   PR 3915 at commit 3239541.

This patch merges cleanly.

Test build #25152 has started for   PR 3915 at commit bec6a84.

This patch merges cleanly.

Test build #25150 has finished for   PR 3915 at commit 23144f3.

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

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

Test build #25151 has finished for   PR 3915 at commit 3239541.

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

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

Test build #25152 has finished for   PR 3915 at commit bec6a84.

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

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

LGTM. The method successfully avoids overflow in the intended way, there's a test, and from checking the old and new logic they should be identical.

Merged into master. Thanks!


Unable to download dictionaries

I just installed the application, then clicked on the download icon for the English dictionary. The download manager starts but does not seem to be doing anything. After a few seconds the application reports the error "Download failed: null".

The download manager isn't really specific to this app, so if it fails that probably means it's indeed not possible to download the file and not some app bug.
Just in case: Are you sure you installed the 5.1.x version and not the old one?
If yes, can you manually download the file at http://github.com/rdoeffinger/Dictionary/releases/tag/v0.2-dictionaries ?
If still yes, I am out of ideas, though I should probably make sure it always prints something more useful than "null".

Yup, version is 5.1.3 from F-Droid.
I can manually download files from that link. I have just tried with Firefox on the phone and it worked all right.
Whatever the download manager is doing, it is not downloading anything, judging by the network activity icons.
Any suggestions as to what I can do to collect some debug info? Cheers.

Btw, the notification from the download manager shows  as the name of the download. Not sure if that's expected.

Argh! Github eats angle brackets! The previous comment should read:
Btw, the notification from the download manager shows "Untitled" as the name of the download. Not sure if that's expected.
Except that the quotes should be replaced by angle brackets instead.

I can tell you that it's normal for it to show Untitled as name of the download until it manages to connect with the server.
I have not yet looked at the download manager code at all so I currently have no further ideas on how to debug it.
You could add some information about your device, what kind of internet connection (via WLAN, 3G,...) and/or try on another device or on a different network.
It's not overly likely to help until I get time to review the related code, but if you don't mind it would be nice to have any such information.

I know you already said you said you checked the app version, but just thought I'd mention this: "Download failed: null" is the exact error I was getting with the old app (which prompted me to switch to this one).

From commit 646486e09434e65b127bf900d86b3b70fb12158b it will now print the reason why it failed, which should help with this.
Will be part of the next v5.2.0 release (with lots of other changes).

Version with extended download error messages is now released, tell me if you get any additional information from it.

@jonathancross Yes I totally checked. You can see a screenshot of the about box in issue #13.
@rdoeffinger I have tried on someone else's phone, on the same WiFi network, and the other phone did download the dictionary while mine didn't. Main difference between the two is that my phone is Google-free. I will install 5.2 when released and report back with any extra info I can gather.

I read up a bit and it seems many implementations of DownloadManager are buggy to a degree you wouldn't believe.
I could actually reproduce some issues on my LG G4, but there it "only" ends up sending the "download done" notification multiple times, which ends up in spurious and incorrect error messages but a still working download.
I don't think "Google-free" is a likely issue (but who knows), but more likely just a question of how broken the DownloadManager in that ROM is...
Many developers seem to implement their own download solution, but I consider that a horrible way to fix it, plus I'd have to request internet permission just for that.
Btw. 5.2.1 is out, it is available as a download here or you can wait for F-Droid which hopefully will have it soon too (I accidentally pushed the tag but not the code/commit itself, so I just hope I didn't break F-Droid).

F-Droid 5.2.1 reports failure with error code 1004. I found the reason though: I have most pre-installed SSL certificates disabled, including DigiCert's, which are the CA for https://github.com/ which is where the dictionaries are hosted. Enabling DigiCert CA certificates gets the thing working.
I have no idea if the DownloadManager provides enough info that one can figure out what the error is, I have had issues with other applications also and they also fail to give helpful information as to what the error is, unlike say a desktop web browser which will clearly tell you, for example, that it does not trust a certificate in the chain.
Thanks a lot for looking into this. :-)

I'm afraid the DownloadManager is really bad/basic, it can basically report status codes and select few specific errors, but there is none for certificate errors.
I could probably suggest checking certificates for 1004 code, but not sure that's a good idea as it's a very generic error (ERROR_HTTP_DATA_ERROR). Well, actually there is no information whatsoever on what can trigger most of those error codes anyway unfortunately.
I'll keep my eyes open for good solutions, HTTPS errors might be kind of nice to handle as they will also happen when the date is set incorrectly.
(Then  again there is a question whether I should use HTTPS at all admittedly - I think I have plugged the deserialization security issues in the latest version so there should be limited risk from using plain HTTP and it would also work nicer with caching/proxies)

It does appear to be a problem with Android not being too explicit about certificate problems, as no Android application I know of offers any sort of helpful information in this situation. Perhaps it would be helpful to open an issue upstream with the Android / Cyanogenmod guys and leave it at that.
I don't think it is an application issue as such from what I've seen, so I wouldn't worry about it. :-)

The app doesn't recognize browser-downloaded dictionaries that are copied directly into the data dir, either. It actually deletes the file when downloads are attempted...Both local storage and sdcard storage have the same issue


😕 hmm?

How much it takes time to replace icons with some modern icons like MS office?

Not long, know any good graphic artists?

Me 🙂

Do you want application icon too ?
To be more MS like :/
Give me a zip includes all icons that you want to make change in them
And tell me if you want them in svg(in case u want convert them xml) or png or ico format

Oh, sweet :-) The icons live here: https://github.com/microsoft/XmlNotepad/tree/master/src/XmlNotepad/Resources.  I think the xmlnote.ico is just for the desktop icon and I think that's ok for now.  But I'm excited you want to take a crack at updating them, feel free to also make them bigger and tell me what size they need to be etc.  I also want to fix the "dark" theme so it includes the whole window including the menus and toolbars.  So if you can design 2 sets of icons perhaps, one for light and one for dark theme, or is there a way to "process" the bitmap to work as light or dark, I think that's what VisualStudio is doing.  Anyway, awesome.  I probably should also flatten some of the rounded looking resizer bars, so it will look consistent with flatter metro style icons.  Maybe some things don't need to be in the toolbar, for example cut/copy/paste was removed from office toolbars long ago because everyone knows the shortcuts now...

I think deleting copy-cut-paste buttons is a good idea which make app less confusing and more clear 😂
I was thinking about some colorish icon but some flat icons will be good two and doesn't take to much time
I guess icons are 16by16 ,so make a little of gap will satisfy high display expensive
I guess 20 or 22 is good enough
I like help icon more than a help menu, but hey ! Who even need it 😅
So as you said , some part of app nee new color and feel like
Tabs to be same size (action or non-active)
Divide lines to be just one pixel and more ligher or darker depend on which theme they are
BTW bitmap is not good, should you use png or maybe xml , i think png or ico is good enough
If you know how to configure applications for dark mode, may I ask you bring dark mode for one of my favorite applications (which is free and open source)
Almost 6 years with no update

Can you use aero glass effect or something in the app?

I prefer transparent .png images for the icons.  I hope to port the app to WPF one day and the icons will work nicely there as .png.  I like your other suggestions.  I'm not a huge fan of glass UI's.  I've noticed every time windows tries it they end up backing off.  The calculator is still slightly transparent, but only when it is the active app.  But I'd skip transparency for now.

I prefer transparent .png images for the icons.  I hope to port the app to WPF one day and the icons will work nicely there as .png.  I like your other suggestions.  I'm not a huge fan of glass UI's.  I've noticed every time windows tries it they end up backing off.  The calculator is still slightly transparent, but only when it is the active app.  But I'd skip transparency for now.

You are right, using glassy effects were a bad experience but now in windows 10 it kinda ok as long as they use noises in the effect
Soooo. I will make some stuff
png format in black and white colors
And some colorish ones maybe
in 20x20 pixel or you 16x16 and let you make the gaps fot it, think it's better
Let's finalize this issue🙃

Oh BTW can i see some results through using reshacker to apply icon and...?

Oh, let's go all the way to 32x32.  I think what's happening as screen resolutions and DPI continue to increase buttons are getting bigger and bigger so they can be easily hit.  So 32 gives me some UI options for big ribbon.  I can always provide a collapse ribbon option for folks who want them smaller again, and then I can just resize to the smaller size as needed.
Sorry don't know what reshacker is, looks like a .ico editor.  I use Axialis Icon Workshop, but do you need me to export some .ico files as png's ?  I think the app icon is the only .ico I'm using and that can remain as is for now I think.

I didn't mean that, reshacker is an icon resources editor which in something case u can edit icons and pngs in some executed apps

BTW You know how to configure a wpf app to have a dark mode, right?
Can you help me in some problems?

https://mega.nz/file/3chShYwa#QCY-S8M5O1xJldF3uq1b9QocnTBx-yFNfA7PP9Orm2Y
Here is some changes! Includes a beautiful app icon in svg and ico

yep, I usually start with this Wpf App Template.  Sure I can help with problems.  If they are unrelated to this issue, you can email me at microsoft.com.  My email is first letter of first name followed by my last name, all one word.

Oh thanks , I will mail you about that sort of problems

Closing for now, the WPF refresh will change everything, not just the icons.


styleguide initialize command: nodejs error

On OSX 10.11.3 while trying styleguide for the first time I encountered this issue with the styleguide initialize command.
env: node\r: No such file or directory
It has apparently to do with line endings on mac/unix systems.
I solved it for now by following this answer: http://stackoverflow.com/a/30349952
I had to run sudo dos2unix styleguide in my /usr/local/lib/node_modules/devbridge-styleguide/bin

Thanks for posting this! I was having the same issue.

Had the same issue, using node 5.10.1
This fixed it for me as well, thanks

Thanks for notice, we will make an update on our end as well.

I just tried this fix with the most recent release version, and I am still getting this error:
env: node\r: No such file or directory
I rand the dos2unix command, said it was successful, but I still get the error when running the styleguide initialize command.

@jpserrett it is not yet fixed.
There was no release after @gediminas-s comment.

@PizzaPete I see that. I just wanted to put it out there that I wasn't able to get the same success as posted above, in case that helped any.
Thanks for the update!

still says : no such file or directory, please help

env: node\r: No such file or directory

Thanks for building such an awesome tool.
FYI, getting same: env: node\r: No such file or directory

Agreed, thanks for the awesome tool, but I am also getting this error:
env: node\r: No such file or directory
I'm installing on a Mac OSX 10.11.4, using Gulp
Thanks!

As @mpons instructed, using dos2unix to convert the styleguide file inside of /bin did the trick.
I had to:

Install Homebrew
Use Homebrew to install dos2unix
run sudo dos2unix styleguide in my /usr/local/lib/node_modules/devbridge-styleguide/bin

still get the same error: node\r: No such file or directory

ditto

Any updates on this?
I am still getting this issue after using dos2unix.

same here, waiting for the update....... Does anyone know? Please help.  Thank you!

Same error here.

I resolved following the first comment of the thread by @mpons

Fix worked for me as well, thank you.

Any chance of releasing this fix to npm?

@edsu this fix should already work. Are you still experiencing this?

Yes, I'm still experiencing it with v0.4.9, which was released 3 weeks ago:
env: node\r: No such file or directory

Ok, so line endings are correct, so I'm assuming, that node installed in your system is only accessible with nodejs, so my suggestion is to make a symlink or alias named node to nodejs location.

node is available:
zen:x ed$ which node
/usr/local/bin/node

I think that the problem is that node_modules/devbridge-styleguide/bin/styleguide has DOS line endings. Did you see the '\r' in the error above? I realize that this is fixed here in GitHub. But it has not yet been released to npm, so someone following the instructions in the README.md will not get the fixed code.

If the command dos2unix is not available, you have to install it first.
Here's how to install dos2unix on Mac OSX: http://macappstore.org/dos2unix/ (if you already have Homebrew installed, jump to step 3).
Once both Homebrew and dos2unix are installed, you'll be able to run the command styleguide initialize. Hope this helps.

New version is already published


Add schema management to cerbosctl

Add schema management commands to cerbosctl

https://github.com/cerbos/cerbos/pull/536


fix _bc_rec_mul error

[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 update/change?

Description
fix bcmul error when length is very long.

Thanks!


Serval-side mixed source support

THIS IS A DRAFT
There are still tests to add, refactoring to do, and documentation to add as well as a relatively small change to the Machine repo to allow it to handle multiple target files appropriately, but I think you can see the idea. I'll set this aside until I have your feedback and either your OK to wrap things up or your instructions to jump ship and use a different API design. That being said, I may work on the change to machine since I believe that will be needed regardless.

This change is

Codecov Report
Attention: Patch coverage is 48.41270% with 455 lines in your changes missing coverage. Please review.

Project coverage is 53.18%. Comparing base (4382410) to head (a3e10ed).
Report is 19 commits behind head on main.

Files with missing lines
Patch %
Lines

...lation/Controllers/TranslationEnginesController.cs
30.20%
162 Missing and 9 partials :warning:

...l/src/Serval.Translation/Services/EngineService.cs
71.10%
103 Missing and 12 partials :warning:

.../Serval.DataFiles/Controllers/CorporaController.cs
0.00%
81 Missing :warning:

...on/Contracts/TranslationParallelCorpusUpdateDto.cs
0.00%
12 Missing :warning:

...val/src/Serval.DataFiles/Services/CorpusService.cs
52.94%
7 Missing and 1 partial :warning:

...nslation/Contracts/TranslationParallelCorpusDto.cs
0.00%
8 Missing :warning:

...Serval/src/Serval.DataFiles/Contracts/CorpusDto.cs
0.00%
7 Missing :warning:

...Serval/src/Serval.Shared/Contracts/CorpusResult.cs
0.00%
6 Missing :warning:

...on/Contracts/TranslationParallelCorpusCorpusDto.cs
0.00%
5 Missing :warning:

.../src/Serval.DataFiles/Contracts/CorpusConfigDto.cs
0.00%
4 Missing :warning:

... and 16 more

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #497      +/-   ##
==========================================
- Coverage   56.59%   53.18%   -3.42%
==========================================
  Files         275      297      +22
  Lines       14174    15704    +1530
  Branches     1908     2171     +263
==========================================
+ Hits         8022     8352     +330
- Misses       5563     6751    +1188
- Partials      589      601      +12

Flag
Coverage Δ

53.18% <48.41%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Note: I'm realizing I'm accidentally mixing across corpus pairs (easy to fix) but in case someone gets to looking at this before I make that change, I'm aware.

I took a quick look. This looks like a good direction. You should continue down this path.

Also, I realize more testing is needed. Would you like that to be a separate PR or should I go ahead and do that now?

src/Serval/src/Serval.Translation/Services/EngineService.cs line 425 at r4 (raw file):
Previously, ddaspit (Damien Daspit) wrote…
SF will need to migrate engines from the old API to the new API. During this transition, it might be simpler if engines can have both, so maybe we should just allow it for now.

Since this will only be temporary functionality, I don't see why extra checks are needed.

Reviewed 3 of 3 files at r7, all commit messages.
Reviewable status: all files reviewed, 1 unresolved discussion (waiting on @Enkidu93)

src/Serval/test/Serval.ApiServer.IntegrationTests/TranslationEngineTests.cs line 808 at r7 (raw file):

    [TestCase(new[] { Scopes.UpdateTranslationEngines }, 404, DOES_NOT_EXIST_ENGINE_ID)]
    [TestCase(new[] { Scopes.ReadFiles }, 403, ECHO_ENGINE1_ID)] //Arbitrary unrelated privilege
    public async Task AddParallelCorpusToEngineByIdAsync(

I find using test cases in this way makes the tests difficult to follow. Can you refactor these test cases into separate tests?

I know you do; I'm sorry. I only wrote them like this for consistency with the existing corpus tests - this way, they're totally parallel and we can be certain we're testing the same breadth of things. But sure, given that the non-parallel-corpus-style corpus tests will eventually be removed, I can refactor these.


[Request] Border on the resizeable part should not disappear

When I want to change the width of the sidebar, I can't see how far the line I'm touching is. So the border should not disappear.

@rokimiftah Thanks for the bug, PR#4 is merged. v1.0.7 has been rolled out. You can update the theme.


Improve input speed

Suggestion by mr X :
You should take a look at the method 'My Expenses' app employs for adding entries. In it, you press a '+' icon to open a page in which you can add the amount, choose category, make notes, etc., before finally pressing a √ sign to save and exit. If you want to just save and make another entry, you press '√+' that's next to the √ icon. Pressing '√+' automatically clears certain fields so the user can make a new entry quickly. I find this method to be very nifty and fast. Do check it out.

Needs to be studied, Action nbossard.


Table support

It would be nice add the Table support
https://plotly.com/javascript/table/

Table support has been added thanks to @baiguoname and @naijim in #207 .


chore: simplify assertions in test_gateway_register_with_federation

I'm currently ramping up on the devimint codebase and other testing code and noticed this assertion that doesn't appear to serve any purpose

Closing now that #5231 is merged


Add budges on README

General
Add the following budges at README:

Version
Build  (failing/pass)
Contributors
Chat

DoD

Aforementioned budges exist on README

Blocked By

#44
#43


Samples do not work

Hi, i am trying to test the serial communication along the pdf you provided, bute i got the follwoing error:
`>>> from aq import AQ

aq = AQ()
Traceback (most recent call last):
File "", line 1, in
File "/home/pi/pi_aq/pi_aq/aq.py", line 13, in init
self.ser = serial.Serial("/dev/ttyS0", 9600)
AttributeError: module 'serial' has no attribute 'Serial'`

Solved, had ti install pyserial.

Ah - you had me worried for a moment. Glad you sorted it.
Si.
On Fri, Dec 10, 2021 at 9:04 PM TepMan @.***> wrote:

Solved, had ti install pyserial.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/monkmakes/pi_aq/issues/2#issuecomment-991291777, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ALCJWNGDTL3LJAZ3CCYFXS3UQJTMNANCNFSM5J2AC2OQ
.
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.

--
Dr. Simon Monk.
MonkMakes Ltd.


Hi Can i somehow change the path where the grabber is looking for his infos?

so i have this as a windows service and its running as system so the user is system so it doesnt get the right uer directory.
can i just change the grabber.Passwordpath?
pls help

@Stanley-GF sry for ping but i need answer

WDYM ? Yes, ofc

public string ChromeCookiePath = @"C:\Users\" + Environment.UserName + @"\AppData\Local\Google\Chrome\User Data\Default\Cookies";
public string ChromeKeyPath = @"C:\Users\" + Environment.UserName + @"\AppData\Local\Google\Chrome\User Data\Local State";
public string ChromePasswordPath = @"C:\Users\" + Environment.UserName + @"\AppData\Local\Google\Chrome\User Data\Default\Login Data";

You can change all of them for example,
// [...]


content/ja/docs/concepts/storage/volume-pvc-datasource.md is outdated

This is a Feature Request

What would you like to be added
Update content/ja/docs/concepts/storage/volume-pvc-datasource.md to follow the en updates.
Why is this needed
content/ja/docs/concepts/storage/volume-pvc-datasource.md is outdated.
Comments
/language ja
/assign
diff
https://gist.github.com/oke-py/3832d2e94e9fa4ce2b315fcea1175201

/close


XSRF token missing

Lost...
When I use the ziggy route to generate the url I get a 419 (CSRF token missing), and when I use the fully qualified same url directly into the axios call, i do not this error
In the request headers in both situations I see the cookies and in the cookie the XSRF-TOKEN is set,
but in the case of the 419, the XSRF-token is not translated into x-xsrf-token: in the header, hence the 419
When I add the x-xsrf-token header manualy in the axios request interceptor then the call works fine, but I guess that should not have to be the case.
Any ideas ?
     const url = this.route('users.logs.delete');       this.axios         .delete(url) //  419
`
this.axios
.delete(https://example.com/users/log/delete) //  no 419
`

Update:
const url = this.route('users.logs.delete').url();
then it works again

This is because Ziggy v0.9 returns a JavaScript String object, which axios does not consider to be strictly equal to the actual URL for security reasons.
In Ziggy v0.9 and earlier, the fix is the solution you found: calling .url() explicitly.
Ziggy v1 returns a plain string, so updating would also fix this for you.

Thanks... just figured that out after the .url() and seeing the commits.
I can confirm that by upgrading to 1.05 everything works again
Thanks for the awesome package (and incredibly speedy response)


CEO-455 Use global color vars for the CSS

Purpose
Create global color vars instead of using hex values everywhere
Submission Checklist

[x] Included Jira issue in the PR title (e.g. CEO-### <title>)
[x] Code passes linter rules (npm run eslint/clj-kondo --lint src)

Great start!  There are other places in the rest of the code that reference the hex.  Can you update these as well?

src/css/custom.css
I had "hide white space changes" set in my last review.  I see now that you changes the indent from 4 to 2. We can talk about doing that at a later point, but it should not be a part of this PR.
src/js/project/CreateProjectWizard.js
469, I was expecting a change here from PR 1532.  It looks like your a few commits behind (I think you based off of 1533).  You can pull the latest development and rebase to see the changes from 1532.

src/js/project/CreateProjectWizard.js
475, should the yellow be --yellow?
475, Lets define a var for gray so we can modify it in the future if we need to.  Its also possible the default gray is different for each computer if we dont define it specifically.
475, I think it would be cleaner to set stepColor to the color only and combine with var where used.  const stepColor = isSelected ? "--lightgreen" : stepComplete ? "--yellow" : "--gray"; or just reduce some redundant code: const stepColor = 'var(${isSelected ? "--lightgreen" : stepComplete ? "--yellow" : "--gray"})'; (note that I cant type back ticks into github the single quote should be a back tick)

I don't
src/js/project/CreateProjectWizard.js
475, Yeah...I corrected it before but somehow slipped with cherry-picking commits and all.
475, Yes, I was debating that in my head. Makes sense.
475, What if we add a small utility function that decorates the color string to a css var that we can also use elsewhere?

This is a little simpler version of my comment above 'var(--${isSelected ? "lightgreen" : stepComplete ? "yellow" : "gray"})';`


if x is Type statement provides autocompletion for x in its scope but fails to infer types from properties and function calls

Godot version
4.0.3
System information
linux
Issue description
consider this:
extends Node

func do(n):
    if n is Camera3D:
        var f := n.get_frustum()

godot provides autocompletion for n in this scope:

but then immediately complains about the supposedly unknow return type:

Steps to reproduce
Type this script and notice how you get autocompletion prompts and an error.
extends Node

func do(n):
    if n is Camera3D:
        var f := n.get_frustum() # type this

Minimal reproduction project
see above

Likely related to #76414

I'm not sure if this is a bug. Yes, we're inferring the weak type for easier autocompletion in the editor, but the hard variable's type is still not fixed, so you can do something like:
func do(n):
    if n is Camera3D:
        n = "string"
        var f := n.get_frustum()

See also godotengine/godot-proposals#6941. This use of the is operator is more advanced static analysis. We haven't reached a consensus yet.


IME window follow the input cursor.

Abstruct set_ime_position to OS class.
Update ime position for LineEdit and TextEdit.

Just to make sure: the IME overlay doesn't leave the screen/window, right?

@bojidar-bg Yes.
Make sure the IME always under and near but never cover the input cursor.

Should there be some is_ime_enabled check before updating the position, or is the function simple enough that it doesn't matter if it's run on non-IME enabled systems?

I think it doesn't matter.
Just like my desktop environment cinnamon and on Windows they always allow IME windows popup when key pressed (If  IME is actived).
So the godot get same input experience with systems.

#9350 was reverted. Can you check what's wrong with it? Specifically, echo events are not being marked as echo.

Sorry I don't really understand what is the echo event ～
2017年6月26日 08:41，"George Marques"<EMAIL_ADDRESS>
#9350 https://github.com/godotengine/godot/pull/9350 was reverted. Can
you check what's wrong with it? Specifically, echo events are not being
marked as echo.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/pull/9366#issuecomment-310939465,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGpFTDyf7vYVJAtQaGcNz00waw1kOoT4ks5sHv5SgaJpZM4OEpJj
.

@Geequlim when the user holds a key, the OS sends new keydown events to the application at certain intervals, those repeated events are marked as "echo", so you can check with InputEven.is_echo() and avoid (or not) firing new actions in the game. Basically the same as when inputting text here, if keep a key down it'll keep repeating such key.

@vnen I will have a try off work today.

I have found the solution: check all events in the queue. I will update later.

 I hope you can join the QQ group 302924317 if you are.
As there are a lot of awesome guys who using godot.

Yes. Thanks. I will try when I have more time.

Rebased for #9396

Needs a small rebase to fix a merge conflict, then I'll merge :)

Rebased.

We should use XIM "on-the-spot" type.


Demographics org reference can use external identifier

https://www.pivotaltracker.com/story/show/143697363

modified Reference model, so that Organizations can be referenced from within other fhir api calls (e.g. User.update_from_fhir) via an external identifier

format: instead of {'reference': 'api/organization/<id>'} (or variants), expects {'reference': 'api/organization/<identifier_system>/<identifier_value>'} (or variants)
as with normal org reference, throws an error if no org is found; however, also throws an error if more than one org is found (reference can only be used with unique identifiers)

added test for org external identifier reference to test_demographics
added GET api endpoint at /api/organization/<identifier_system>/<identifier_value>
added test for GET endpoint

@pbugni gotcha. now includes the api get endpoint (+ tests), and uses a new MultipleReference exception for multiple results returned during the reference search.


x/sys/unix: TestIoctlGetRTCWkAlrm "failed to perform ioctl: invalid argument" on linux-ppc64le-power9osu since August

--- FAIL: TestIoctlGetRTCWkAlrm (0.00s)
    syscall_linux_test.go:121: failed to perform ioctl: invalid argument
FAIL
FAIL	golang.org/x/sys/unix	0.589s

Given the timing of the first recent failures, I suspect that CL 340369 may be related (CC @mdlayher , @tklauser).
If this syscall is not supported on linux-ppc64le, then the test should be skipped on that platform.
greplogs --dashboard -md -l -e 'FAIL: TestIoctlGetRTCWkAlrm'
2021-09-21T06:55:28-437939a/085c609/linux-ppc64le-power9osu
2021-09-21T06:55:28-437939a/3664950/linux-ppc64le-power9osu
2021-09-21T06:55:28-437939a/abc4f09/linux-ppc64le-power9osu
2021-09-21T06:55:28-437939a/e128749/linux-ppc64le-power9osu
2021-09-17T16:11:53-d61c044/1192135/linux-ppc64le-power9osu
2021-09-17T16:11:53-d61c044/abc4f09/linux-ppc64le-power9osu
2021-09-17T16:11:53-d61c044/b6dddac/linux-ppc64le-power9osu
2021-09-15T08:33:10-ed5796b/6d02ce8/linux-ppc64le-power9osu
2021-09-15T08:33:10-ed5796b/e79c297/linux-ppc64le-power9osu
2021-09-10T15:07:52-751e447/71adc65/linux-ppc64le-power9osu
2021-09-10T15:07:52-751e447/ee91bb8/linux-ppc64le-power9osu
2021-09-08T23:34:32-aa78b53/2481f6e/linux-ppc64le-power9osu
2021-09-08T23:34:32-aa78b53/a295b3c/linux-ppc64le-power9osu
2021-09-08T23:34:32-aa78b53/a53e3d5/linux-ppc64le-power9osu
2021-09-08T23:34:32-aa78b53/c84f3a4/linux-ppc64le-power9osu
2021-09-08T16:03:47-a851e7d/0406d3a/linux-ppc64le-power9osu
2021-09-08T16:03:47-a851e7d/c8d4fe2/linux-ppc64le-power9osu
2021-09-08T16:03:47-a851e7d/d419f9c/linux-ppc64le-power9osu
2021-09-06T17:05:28-6f6e228/9295723/linux-ppc64le-power9osu
2021-09-06T17:05:28-6f6e228/bab79dd/linux-ppc64le-power9osu
2021-09-06T17:05:28-6f6e228/da790cc/linux-ppc64le-power9osu
2021-09-03T07:17:46-97244b9/4957976/linux-ppc64le-power9osu
2021-09-03T07:17:12-2934436/ab7c904/linux-ppc64le-power9osu
2021-09-02T05:02:50-f475640/782aa42/linux-ppc64le-power9osu
2021-08-31T04:25:30-f4d4317/1bd35fa/linux-ppc64le-power9osu
2021-08-31T04:25:30-f4d4317/36c1717/linux-ppc64le-power9osu
2021-08-31T04:25:30-f4d4317/6c5f028/linux-ppc64le-power9osu
2021-08-31T04:25:30-f4d4317/711e1c8/linux-ppc64le-power9osu
2021-08-31T04:25:30-f4d4317/c35f8a3/linux-ppc64le-power9osu
2021-08-31T04:25:30-f4d4317/df4c625/linux-ppc64le-power9osu
2021-08-31T04:25:30-f4d4317/fcce86c/linux-ppc64le-power9osu
2021-08-23T07:06:55-63515b4/61120c6/linux-ppc64le-power9osu
2021-08-23T07:06:55-63515b4/8250141/linux-ppc64le-power9osu
2021-08-20T12:10:16-41cdb87/5d0c284/linux-ppc64le-power9osu
2021-08-20T12:10:16-41cdb87/6e50991/linux-ppc64le-power9osu
2021-08-20T12:10:16-41cdb87/c991278/linux-ppc64le-power9osu
2021-08-20T12:10:16-41cdb87/e17439e/linux-ppc64le-power9osu
2021-08-19T13:52:13-f52c844/3bdc179/linux-ppc64le-power9osu
2021-08-19T13:52:13-f52c844/9871726/linux-ppc64le-power9osu
2021-08-19T13:52:13-f52c844/e9e0d1e/linux-ppc64le-power9osu
2021-08-19T07:20:04-44f02d9/0d53084/linux-ppc64le-power9osu
2021-08-18T15:36:20-00dd8d7/0c83e01/linux-ppc64le-power9osu
2021-08-18T15:36:20-00dd8d7/687f2ac/linux-ppc64le-power9osu
2021-08-18T15:36:20-00dd8d7/8f0578e/linux-ppc64le-power9osu
2021-08-17T19:03:40-bfb29a6/0f85b0c/linux-ppc64le-power9osu
2021-08-17T19:03:40-bfb29a6/946e254/linux-ppc64le-power9osu
2021-08-17T19:03:40-bfb29a6/ec51703/linux-ppc64le-power9osu
2021-08-17T13:44:02-fefb4af/0d53084/linux-ppc64le-power9osu
2021-08-16T18:31:51-1e6c022/1951afc/linux-ppc64le-power9osu
2021-08-16T18:31:51-1e6c022/29ec74f/linux-ppc64le-power9osu
2021-08-16T18:31:51-1e6c022/6406227/linux-ppc64le-power9osu
2021-08-16T18:31:51-1e6c022/a05a7d4/linux-ppc64le-power9osu
2021-08-16T18:31:51-1e6c022/a8d39f1/linux-ppc64le-power9osu
2021-08-16T18:31:51-1e6c022/df9c5d8/linux-ppc64le-power9osu
2021-08-16T17:40:21-fcbd12d/5c7a460/linux-ppc64le-power9osu
2021-08-16T07:42:44-15123e1/3d679c6/linux-ppc64le-power9osu
2021-08-16T07:42:44-15123e1/acbe242/linux-ppc64le-power9osu
2021-08-16T07:42:44-15123e1/c88e3ff/linux-ppc64le-power9osu
2021-08-16T07:39:13-b9628f2/7aa57a9/linux-ppc64le-power9osu
2021-08-16T07:10:09-649d0fc/0d53084/linux-ppc64le-power9osu
2021-08-09T22:24:54-d867a43/48dfddb/linux-ppc64le-power9osu
2021-08-09T22:24:54-d867a43/89a4f99/linux-ppc64le-power9osu
2021-08-09T22:24:54-d867a43/acbe242/linux-ppc64le-power9osu
2021-08-09T20:39:39-8946682/acbe242/linux-ppc64le-power9osu
2021-08-09T20:39:39-8946682/f1dce31/linux-ppc64le-power9osu
2021-08-06T18:45:41-e5e7981/70546f6/linux-ppc64le-power9osu
2021-06-30T00:52:30-0f9fa26/37c117f/linux-ppc64le-power9osu
2021-06-30T00:52:30-0f9fa26/818c5f6/linux-ppc64le-power9osu
2021-06-30T00:52:30-0f9fa26/ba93baa/linux-ppc64le-power9osu

This is a release-blocker for Go 1.18 via #11811.

--- FAIL: TestIoctlGetRTCWkAlrm (0.00s)
    syscall_linux_test.go:121: failed to perform ioctl: invalid argument
FAIL
FAIL	golang.org/x/sys/unix	0.589s

Given the timing of the first recent failures, I suspect that CL 340369 may be related (CC @mdlayher , @tklauser).

That CL changed ioctls related to ifreq, but the failing test doesn't involved such an ioctl. I suspect it's rather CL 340915 which may be related.

If this syscall is not supported on linux-ppc64le, then the test should be skipped on that platform.

AFAIK that syscall should be supported on linux-ppc64le and it previously worked without problems.
CL 340915 changed the way pointer arguments are passed to Syscall and incidentially on linux/ppc64le (and linux/ppc64), Syscall` is not implemented in assembly as on other platforms, see
https://github.com/golang/sys/blob/437939a70204d143b964abe4ee5e6662d973a384/unix/syscall_unix_gc_ppc64x.go#L14-L16
I suspect this might lead to problems considering the following in unsafe.Pointer docs:

The compiler handles a Pointer converted to a uintptr in the argument list of a call to a function implemented in assembly by arranging that the referenced allocated object, if any, is retained and not moved until the call completes, even though from the types alone it would appear that the object is no longer needed during the call.

see https://pkg.go.dev/unsafe#Pointer, point (4).
This specific implementation of Syscall and related functions for linux/ppc64{,le} was added in CL 146518 for #16662. /cc @laboger

The first failure in the August logs was from CL 340369, which was just before CL 340915 was merged.

I'm afraid we are beyond my area of expertise here. I'm not sure how 340369 could have impacted an unrelated ioctl call which does not use ifreq under the hood.

We'll take a look.

I did a few experiments on some systems and I think this is kernel related. I didn't see any failures on power8, only power9 just
as on the dashboard. I have one power9 where I can make it fail consistently:
(4.18.0-305.3.1.el8_4.ppc64le #1 SMP Mon May 17 10:03:20 EDT 2021 ppc64le ppc64le ppc64le GNU/Linux)
and another where it passes consistently:
(4.15.0-91-generic #92-Ubuntu SMP Fri Feb 28 11:08:26 UTC 2020 ppc64le ppc64le ppc64le GNU/Linux)
Could there have been a kernel update on the power9 builder systems recently?

On the system where it fails the strace shows this:
ioctl(3, RTC_WKALM_RD, 0xc0001d4e90)    = -1 EINVAL (Invalid argument)
When it passes I see this:
ioctl(3, RTC_WKALM_RD, {enabled=0, pending=0, time={tm_sec=0, tm_min=0, tm_hour=0, tm_mday=1, tm_mon=0, tm_year=70, ...}}) = 0
Almost like the first one doesn't like the pointer argument.

I think the rtc alarm ioctls return EINVAL if the device does not support an alarm feature. I think the generic PPC rtc driver is one such case. Should the test be ignored in these cases?


Closes #25

:white_check_mark: Build mentorbot 1.0.114-dxxddcud completed (commit https://github.com/MentorSource/mentorbot/commit/0c5c6452a4 by @rosenkolev)

Codecov Report

Merging #38 into master will increase coverage by 1.1%.
The diff coverage is 93.56%.

@@            Coverage Diff            @@
##           master      #38     +/-   ##
=========================================
+ Coverage   94.89%   95.99%   +1.1%
=========================================
  Files          33       36      +3
  Lines        1411     1549    +138
=========================================
+ Hits         1339     1487    +148
+ Misses         72       62     -10

Impacted Files
Coverage Δ

...Bot.Functions/Connectors/OpenAir/OpenAirFactory.cs
100% <ø> (ø)
:arrow_up:

...ons/App/Extensions/AccessTokenServiceExtensions.cs
100% <100%> (ø)

...entorBot.Functions/Services/TableStorageService.cs
100% <100%> (+2.66%)
:arrow_up:

src/MentorBot.Functions/AzureFunctions/Queries.cs
100% <100%> (ø)
:arrow_up:

src/MentorBot.Functions/AzureFunctions/Commands.cs
100% <100%> (ø)
:arrow_up:

src/MentorBot.Functions/Services/StorageService.cs
92.5% <100%> (+0.39%)
:arrow_up:

.../MentorBot.Functions/Processors/RepeatProcessor.cs
96.96% <100%> (+15.15%)
:arrow_up:

...MentorBot.Functions/Connectors/OpenAirConnector.cs
99.25% <100%> (ø)
:arrow_up:

src/MentorBot.Functions/App/ServiceLocator.cs
100% <100%> (ø)
:arrow_up:

...rBot.Functions/Services/AzureStorage/AzureQuery.cs
83.87% <83.87%> (ø)

... and 12 more

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 fa21c8b...3dbb076. Read the comment docs.

:white_check_mark: Build mentorbot 1.0.115-tuvkjjuj completed (commit https://github.com/MentorSource/mentorbot/commit/a488060215 by @rosenkolev)


use kubeconfig from manager for worker pool.

We should use the same kubeconfig from the manager, instead of getting new one from in-cluster, because in some cases, we may want to override the by specify --kubeconfig or KUBECONFIG env, then the  in-cluster kubeconfig will be different.
Signed-off-by: morvencao<EMAIL_ADDRESS>

/lgtm


End to End tests for NodePortLocal

This commit introduces e2e tests for NodePortLocal. As part of the tests, Pod annotation,
IPTABLES rules and traffic are being tested. We focus on Pod addition, deletion and
Antrea Agent restart.
Additional tests for Pod selection based on Service annotation would be added later.

Codecov Report

Merging #1766 (ebb9fc7) into master (9d3d10b) will decrease coverage by 20.64%.
The diff coverage is 37.53%.

@@             Coverage Diff             @@
##           master    #1766       +/-   ##
===========================================
- Coverage   63.31%   42.67%   -20.65%
===========================================
  Files         170      108       -62
  Lines       14250    13570      -680
===========================================
- Hits         9023     5791     -3232
- Misses       4292     7292     +3000
+ Partials      935      487      -448

Flag
Coverage Δ

kind-e2e-tests
?

unit-tests
42.67% <37.53%> (+1.39%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/agent/agent.go
11.46% <0.00%> (-37.25%)
:arrow_down:

pkg/agent/agent_linux.go
0.00% <0.00%> (-100.00%)
:arrow_down:

...g/agent/cniserver/interface_configuration_linux.go
5.79% <0.00%> (-14.44%)
:arrow_down:

pkg/agent/cniserver/pod_configuration.go
34.52% <0.00%> (-19.24%)
:arrow_down:

pkg/agent/config/node_config.go
0.00% <0.00%> (-100.00%)
:arrow_down:

...gent/controller/noderoute/node_route_controller.go
32.57% <0.00%> (-13.90%)
:arrow_down:

pkg/agent/controller/traceflow/packetin.go
10.91% <0.00%> (-53.37%)
:arrow_down:

...agent/controller/traceflow/traceflow_controller.go
0.00% <0.00%> (-82.70%)
:arrow_down:

pkg/agent/nodeportlocal/npl_agent_init.go
65.62% <ø> (ø)

pkg/agent/openflow/client.go
35.67% <ø> (-24.20%)
:arrow_down:

... and 168 more

Codecov Report

Merging #1766 (ebb9fc7) into master (9d3d10b) will decrease coverage by 20.64%.
The diff coverage is 37.53%.

@@             Coverage Diff             @@
##           master    #1766       +/-   ##
===========================================
- Coverage   63.31%   42.67%   -20.65%
===========================================
  Files         170      108       -62
  Lines       14250    13570      -680
===========================================
- Hits         9023     5791     -3232
- Misses       4292     7292     +3000
+ Partials      935      487      -448

Flag
Coverage Δ

kind-e2e-tests
?

unit-tests
42.67% <37.53%> (+1.39%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/agent/agent.go
11.46% <0.00%> (-37.25%)
:arrow_down:

pkg/agent/agent_linux.go
0.00% <0.00%> (-100.00%)
:arrow_down:

...g/agent/cniserver/interface_configuration_linux.go
5.79% <0.00%> (-14.44%)
:arrow_down:

pkg/agent/cniserver/pod_configuration.go
34.52% <0.00%> (-19.24%)
:arrow_down:

pkg/agent/config/node_config.go
0.00% <0.00%> (-100.00%)
:arrow_down:

...gent/controller/noderoute/node_route_controller.go
32.57% <0.00%> (-13.90%)
:arrow_down:

pkg/agent/controller/traceflow/packetin.go
10.91% <0.00%> (-53.37%)
:arrow_down:

...agent/controller/traceflow/traceflow_controller.go
0.00% <0.00%> (-82.70%)
:arrow_down:

pkg/agent/nodeportlocal/npl_agent_init.go
65.62% <ø> (ø)

pkg/agent/openflow/client.go
35.67% <ø> (-24.20%)
:arrow_down:

... and 168 more

Currently when we run the e2e tests in the kind clusters, there is a timeout of 35 minutes. After adding the new tests for NodePortLocal (some of which restarts Antrea Agent) a few tests were not getting completed, so I have increased this timeout.

Currently when we run the e2e tests in the kind clusters, there is a timeout of 35 minutes. After adding the new tests for NodePortLocal (some of which restarts Antrea Agent) a few tests were not getting completed, so I have increased this timeout.

Currently when we run the e2e tests in the kind clusters, there is a timeout of 35 minutes. After adding the new tests for NodePortLocal (some of which restarts Antrea Agent) a few tests were not getting completed, so I have increased this timeout.

@antoninbas and @lzhecheng can provide guidance here.

Currently when we run the e2e tests in the kind clusters, there is a timeout of 35 minutes. After adding the new tests for NodePortLocal (some of which restarts Antrea Agent) a few tests were not getting completed, so I have increased this timeout.

@antoninbas and @lzhecheng can provide guidance here.

Besides Kind, please also update timeout here for Jenkins tests.
IPv4:
https://github.com/vmware-tanzu/antrea/blob/master/ci/jenkins/test-vmc.sh#L390
https://github.com/vmware-tanzu/antrea/blob/master/ci/jenkins/test-vmc.sh#L392
IPv6, dual-stack, Windows:
https://github.com/vmware-tanzu/antrea/blob/master/ci/jenkins/test.sh#L335

Besides Kind, please also update timeout here for Jenkins tests.
IPv4:
https://github.com/vmware-tanzu/antrea/blob/master/ci/jenkins/test-vmc.sh#L390
https://github.com/vmware-tanzu/antrea/blob/master/ci/jenkins/test-vmc.sh#L392
IPv6, dual-stack, Windows:
https://github.com/vmware-tanzu/antrea/blob/master/ci/jenkins/test.sh#L335

Besides Kind, please also update timeout here for Jenkins tests.
IPv4:
https://github.com/vmware-tanzu/antrea/blob/master/ci/jenkins/test-vmc.sh#L390
https://github.com/vmware-tanzu/antrea/blob/master/ci/jenkins/test-vmc.sh#L392
IPv6, dual-stack, Windows:
https://github.com/vmware-tanzu/antrea/blob/master/ci/jenkins/test.sh#L335

@lzhecheng Thanks. I have updated for IPv4, we are skipping for IPv6 for now.

Besides Kind, please also update timeout here for Jenkins tests.
IPv4:
https://github.com/vmware-tanzu/antrea/blob/master/ci/jenkins/test-vmc.sh#L390
https://github.com/vmware-tanzu/antrea/blob/master/ci/jenkins/test-vmc.sh#L392
IPv6, dual-stack, Windows:
https://github.com/vmware-tanzu/antrea/blob/master/ci/jenkins/test.sh#L335

@lzhecheng Thanks. I have updated for IPv4, we are skipping for IPv6 for now.

/test-e2e

@jianjuns do you want to take another look at this, it looks good on my side

/test-networkpolicy

/test-conformance


send_fleet issue

send_fleet function returns me 'False"
Do you know what's wrong ?

Ok i've found, the issue was that my fleet cannot transport more resources  than it can.
So the best would be to have the possibility  to say to the send_fleet fonction, to take as much resources than it can.

ogame server dosent accept invalid send fleet requests
You can solve your problem in 3 ways.

Create an resource function that calculates the needed space of resources you wanna send and compare it with your resources.
Build more Transporters than you have resources for on your celestial
create an standart fleet whit exact known resources and just use it as a template

There was a time ogame accepted this and it would do it. but they now check if it makes sense now before the fulfill it

yes i created a function that calculate resources depending on my fleet capacity, with deut prio 3, crystal prio 2 and metal 1

this lib is quite "low level" it is basicly just giving you one to one functions that the ogame browser client also does.
Because of its "low level" nature, programmers that create more complex things will need to do more work.
And they check before they send. There is a function in ogame where you can give it a fleet configuration and it will tell you if it is valid.
But send Fleet already does that.


TypeScript doesn't recognize the types and fail v7

Screenshot

Description
after upgrading to version 7, TypeScript compiler and all related tools to it seems to not be able to read the declaration files from the bundle anymore, I found the files in my node_modules but I think that TSC isn't going to read the exports field with that nested deep object to get the types,
My expo application is failing and I can be sure that other tools relaying on TSC will fail as well.
Reproduction Steps/Repo Link
just install v7 of this lib in a repo that uses TSC.

You need to setup TS to read from package.exports. Duplicate of https://github.com/jasonkuhrt/graphql-request/issues/841 and others before.


Issue with importing YDR prop in blender, exported with CodeWalker

Hey, i exported prop from rpf explorer by searching it in the search bar, (prop name - imp_prop_impexp_postlift_up)

added to Shared textures folder in the preferences folder with the textures from car lift

but it appears with different color and collision being broken? (placed in world origin)

and if i select tintPaletteSampler texture in blender, it still doesn't show correct colors/texture

The collision issue is fixed in https://github.com/Skylumz/Sollumz/pull/218. We will look into the tint issue

Fixed on yft branch (will be merged soon)