Leak in TextureSystem::get_imagespec

One of my co-workers claims that TextureSystem::get_imagespec has a leak ("due to a std::vector copy related to ParamValue").  Apparently switching to TextureSystem::imagespec() works around the leak.  No repro case, sorry.

Anybody want to point me to the function or line where the leak is suspected to occur?
I'll try to mock up a test.

https://github.com/OpenImageIO/oiio/pull/212
This allows one to test, via:
testtex --getimagespec --iters 10
testtex --getimagespec --iters 1000000

No difference in memory stats.  (Definitely takes longer, so I do think it's working as intended.)
Comments?

I'll try to get more info on this end.  Sorry if it's a red herring.

Mark

On Wed, Jan 11, 2012 at 8:34 PM, Larry Gritz <
<EMAIL_ADDRESS>
wrote:

https://github.com/OpenImageIO/oiio/pull/212
This allows one to test, via:
testtex --getimagespec --iters 10
testtex --getimagespec --iters 1000000
No difference in memory stats.  (Definitely takes longer, so I do think
it's working as intended.)
Comments?

Reply to this email directly or view it on GitHub:
https://github.com/OpenImageIO/oiio/issues/211#issuecomment-3442955

This might be user error.  I can't replicate the problem.  My co-worker was
using the Google perftools heap analyzer, and there's some confusion about
how it tracks STL container allocations.  Sorry for wasting your time!

Mark

On Thu, Jan 12, 2012 at 8:56 AM, Mark Leone<EMAIL_ADDRESS>wrote:

I'll try to get more info on this end.  Sorry if it's a red herring.

Mark

On Wed, Jan 11, 2012 at 8:34 PM, Larry Gritz <
<EMAIL_ADDRESS>
wrote:

https://github.com/OpenImageIO/oiio/pull/212
This allows one to test, via:
testtex --getimagespec --iters 10
testtex --getimagespec --iters 1000000
No difference in memory stats.  (Definitely takes longer, so I do think
it's working as intended.)
Comments?

Reply to this email directly or view it on GitHub:
https://github.com/OpenImageIO/oiio/issues/211#issuecomment-3442955

No prob.
On Jan 11, 2012, at 1:51 PM, Mark Leone wrote:

This might be user error.  I can't replicate the problem.  My co-worker was
using the Google perftools heap analyzer, and there's some confusion about
how it tracks STL container allocations.  Sorry for wasting your time!

Mark

On Thu, Jan 12, 2012 at 8:56 AM, Mark Leone<EMAIL_ADDRESS>wrote:

I'll try to get more info on this end.  Sorry if it's a red herring.

Mark

On Wed, Jan 11, 2012 at 8:34 PM, Larry Gritz <
<EMAIL_ADDRESS>
wrote:

https://github.com/OpenImageIO/oiio/pull/212
This allows one to test, via:
testtex --getimagespec --iters 10
testtex --getimagespec --iters 1000000
No difference in memory stats.  (Definitely takes longer, so I do think
it's working as intended.)
Comments?

Reply to this email directly or view it on GitHub:
https://github.com/OpenImageIO/oiio/issues/211#issuecomment-3442955

Reply to this email directly or view it on GitHub:
https://github.com/OpenImageIO/oiio/issues/211#issuecomment-3454330

--
Larry Gritz
<EMAIL_ADDRESS>


Remove version: BBC.iPlayer version 2.13.18

installers return following HTTP status code: 404
Microsoft Reviewers: Open in CodeFlow

URL: https://static.files.bbci.co.uk/iplayer-pc-download-app/releases/win32-x64/BBCiPlayerDownloadsSetup-2.13.18.exe
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 920)

BBC iPlayer has shutdown, the services and downloads are no longer available.
see: https://www.bbc.co.uk/iplayer/help/questions/recent-changes-to-iplayer/ipd-closure

[Policy] Needs-Manual-Merge


The reason for #28 is that we never released #20, not that there was some latent error. It's still good to merge these CI updates, but after that I'll cut a release featuring GHC 9.6 support.


[Bug]: Python3 package doesn't support setting install location

Brief description of your issue
The --location parameter is ignored when installing python.python.3 package, even with --scope machine.
Steps to reproduce
winget install --scope machine --location c:\apps\python3 python.python.3

As of today, that's using the 3.10.2150.0 version of the package.
Expected behavior
Python should install to my specified location (c:\apps\python3).
Actual behavior
This will still install to the default C:\Program Files\Python... location.
I believe this is due to the package-level setting of the following in Python.Python.3.installer.yaml:
InstallerSwitches:
  InstallLocation: DefaultCustomTargetDir=<INSTALLPATH>

According to Python docs, this only sets the path shown in the installer.
Since there is a different parameter depending on whether you're doing a per-user or per-machine install, I tested with the following in each of the Installer entries under Installers:
Where InstallAllUsers=0:
InstallLocation: DefaultJustForMeTargetDir=<INSTALLPATH>

Where InstallAllUsers=1:
InstallLocation: DefaultAllUsersTargetDir=<INSTALLPATH>

For my use-case (x64 per-machine install) this seems to do the trick.
Environment
Windows Package Manager v1.1.13405
Copyright (c) Microsoft Corporation. All rights reserved.

Windows: Windows.Desktop v10.0.19044.1466
Package: Microsoft.DesktopAppInstaller v1.16.13405.0

Tagging submitters of some of the more recent Python 3 packages for comment:
@nrudakov @jedieaston @remyjette @vedantmgoyal2009

Thanks for bringing this up. Once all of the linked PRs have merged, you should be good to go!

Awesome, thanks for the quick turnaround!

@gmckeown It appears they are all merged. Can you confirm this is sufficient to close this issue?

@denelon From a purely selfish perspective, I'm only really interested in the system-wide 64-bit 3.10.2 install and that is now working. So if that's enough then please do close the issue :)

@kristjanvalur FYI, I just made a PR for Python 3.11.4 here.

@kristjanvalur FYI, I just made a PR for Python 3.11.4 here.

Thanks.  Does this ensure that this gets included in future releases too?  It seems previous fixes only affect specific releases and new releases revert back to the old behaviour.

Thanks. Does this ensure that this gets included in future releases too? It seems previous fixes only affect specific releases and new releases revert back to the old behaviour.

Depends on how the manifests are created. If the new manifests are upgraded with Komac / Winget Release (or even copied and pasted) from 3.11.4, the fixes will be included. However, there are Python 3.12 manifests (albeit in beta version) created before my fix is created. I believe in order to incorporate the fixes in 3.12, the fixes need to be included in those manifests as well.


Documentation Site: sort assertion types and assertions in menu by name

That would mean that we can get rid of the assertion-menu.json file.

Sorry Gustav for side stepping you, but I just needed it done.

No worries :-) have not got much done lately. I'm sorry if you have been
blocked by my slacking.
Den mandag den 16. februar 2015 skrev Sune Simonsen <
<EMAIL_ADDRESS>
Sorry Gustav for side stepping you, but I just needed it done.
—
Reply to this email directly or view it on GitHub
https://github.com/sunesimonsen/unexpected/issues/121#issuecomment-74577648
.

You haven't been blocking anything, contrary you have been a great help putting this site together. I just want this thing out, so I'm trying to kill it :-)


Madlibs: Put the entire thing in a try, let Control + C work

Makes sense, right?

Fixed. I'm on a roll!


upsert-one - support $addToSet array update operator

Issue

Mongo update operators for arrays are specified here
Among the ones specified above, the CRUD service at the moment supports only the $push
Using $addToSet in the body of an upsert-one HTTP POST request thus results in a 400 bad request response

This feature has been merged into main an it will be soon available in the next service release


Move the torch based ice emulator to soca

Description

Moved some of the AI code from the daml repo to soca. I'm only moving the sea-ice balance, the other balances are not implemented or don't work yet.
Added a training ctest to generate the weights of the emulator instead of storing pre-computed weights
switched to using libtorch from the spack-stack, but it's unfortunately not installed as a cmake package (can't use find_package)

Issue(s) addressed

fixes #1014

Testing
How were these changes tested?
Are the changes covered by ctests? (If not, tests should be added first.)
3 ctests: training, dirac and 3dvar
It's missing a ctest for the adjoint ...
What compilers / HPCs was it tested with?
Only works with gnu, I have no idea how to build torch with intel, and it's also not in the intel spack-stack
Tested on hercules only

I broke the cmake logic to skip the build of MLBalance, oops ... I'll look into it this morning.

I pushed a small change: removed linking daml and renamed variables (removed _category from the name); ml tests pass on orion for me when passing -DTorch_ROOT during ecbuild step 🎉

I have one scientific question. Application of just this balance operator (K I K^T) to a dirac on ice concentration leads to reasonably looking increments in other variables, but also an increased ice concentration increment (it's no longer 1, but a value larger than 1). With a simple 2x2 matrix math I see why this happens, but it still goes against my naive intuition of what a balance operator should do. Does anyone understand whether that's desired behavior (and why if so)?

Following up on my question to close this thread. Here, balance operator transforms ice concentration into the balanced and unbalanced parts. The Identity covariance used in the simplified dirac test corresponds to a diagonal matrix with variance of the temperature error and variance of the unbalanced ice concentration error on the diagonal (both equal to 1). Variance of the total ice concentration error (total here == balanced + unbalanced) is higher and not equal to one, so this result is expected. Good explanation of balance operators [for ocean] in Weaver et al's ECMWF tech memo 491.


Internationalization of cancel button makes it impossible to get a clean and predictable cancellation response

I am developing an app that is supposed to be used across people of multiple languages, and their phones are set as so.
I can't afford to make the translation of the app to those languages and fix them everytime it is needed and using a translation service is kinda unacceptable as I want to cut on my app's web requests to the absolutely necessary.
So the app is written with english in mind, but the cancel button of the text dialog was set to be internationalized some time ago, and the "choice" variable that is returned by it is "cancel" in the language of the phone of the user. This made me take drastic measures like disabling the button and asking the user to leave the text field empty in case of cancellation, which is not clean and overall unacceptable in terms of design.
This could be solved in 3 ways: either give the developer the power of selecting the cancel button text or the notifier component's language(not sure if the latter is actually feasible), the event could be fired with a boolean too
Or is there something I am missing?
Note:ChoiceCancelled events are not applicable here, as the main event is still fired, which still ruins everything

I think extending the event with a separate parameter cancelled that is a boolean to indicate whether the Cancel button is pressed, regardless of the user's language, would be the best option.


GVim (with GTK+ 3.0) fails starting (Vim: Caught deadly signal SEGV) and falls back to terminal

Steps to reproduce

I runned vim with "vim -g" or gvim
Vim's output was:

Vim: Caught deadly signal SEGV
Vim: Finished.
E852: The child process failed to start the GUI

Press ENTER or type command to continue

Vim showed me the normal terminal window

Expected behaviour
I expected that the typical GVim UI starts.
Version of Vim
9.0.1255
VIM - Vi IMproved 9.0 (2022 Jun 28, compiled Jan 28 2023 18:45:24)
Included patches: 1-1255
Compiled by sfiedler@sfiedler-mbp
Huge version with GTK3 GUI.  Features included (+) or not (-):
+acl               +file_in_path      +mouse_urxvt       -tag_any_white
+arabic            +find_in_path      +mouse_xterm       -tcl
+autocmd           +float             +multi_byte        +termguicolors
+autochdir         +folding           +multi_lang        +terminal
-autoservername    -footer            -mzscheme          +terminfo
+balloon_eval      +fork()            +netbeans_intg     +termresponse
+balloon_eval_term +gettext           +num64             +textobjects
+browse            -hangul_input      +packages          +textprop
++builtin_terms    +iconv             +path_extra        +timers
+byte_offset       +insert_expand     -perl              +title
+channel           +ipv6              +persistent_undo   +toolbar
+cindent           +job               +popupwin          +user_commands
+clientserver      +jumplist          +postscript        +vartabs
+clipboard         +keymap            +printer           +vertsplit
+cmdline_compl     +lambda            +profile           +vim9script
+cmdline_hist      +langmap           -python            +viminfo
+cmdline_info      +libcall           -python3           +virtualedit
+comments          +linebreak         +quickfix          +visual
+conceal           +lispindent        +reltime           +visualextra
+cryptv            +listcmds          +rightleft         +vreplace
+cscope            +localmap          -ruby              +wildignore
+cursorbind        -lua               +scrollbind        +wildmenu
+cursorshape       +menu              +signs             +windows
+dialog_con_gui    +mksession         +smartindent       +writebackup
+diff              +modify_fname      -sodium            +X11
+digraphs          +mouse             -sound             -xfontset
+dnd               +mouseshape        +spell             +xim
-ebcdic            +mouse_dec         +startuptime       -xpm
+emacs_tags        -mouse_gpm         +statusline        +xsmp_interact
+eval              -mouse_jsbterm     -sun_workshop      +xterm_clipboard
+ex_extra          +mouse_netterm     +syntax            -xterm_save
+extra_search      +mouse_sgr         +tag_binary
-farsi             -mouse_sysmouse    -tag_old_static
   system vimrc file: "$VIM/vimrc"
     user vimrc file: "$HOME/.vimrc"
 2nd user vimrc file: "~/.vim/vimrc"
      user exrc file: "$HOME/.exrc"
  system gvimrc file: "$VIM/gvimrc"
    user gvimrc file: "$HOME/.gvimrc"
2nd user gvimrc file: "~/.vim/gvimrc"
       defaults file: "$VIMRUNTIME/defaults.vim"
    system menu file: "$VIMRUNTIME/menu.vim"
  fall-back for $VIM: "/usr/local/share/vim"
Compilation: /bin/clang-15 -c -I. -Iproto -DHAVE_CONFIG_H -DFEAT_GUI_GTK -pthread -I/home/linuxbrew/.linuxbrew/Cellar/libffi/3.4.4/include -I/usr/local/include/harfbuzz -I/home/linuxbrew/.linuxbrew/Cellar/dbus/1.14.4/include/dbus-1.0 -I/home/linuxbrew/.linuxbrew/Cellar/dbus/1.14.4/lib/dbus-1.0/include -I/home/linuxbrew/.linuxbrew/Cellar/glib/2.74.5/include -I/home/linuxbrew/.linuxbrew/Cellar/glib/2.74.5/include/gio-unix-2.0 -I/home/linuxbrew/.linuxbrew/Cellar/libxinerama/1.1.5/include -I/home/linuxbrew/.linuxbrew/Cellar/libxi/1.8/include -I/home/linuxbrew/.linuxbrew/Cellar/libxrandr/1.5.3/include -I/home/linuxbrew/.linuxbrew/Cellar/libxcursor/1.2.1/include -I/home/linuxbrew/.linuxbrew/Cellar/libxfixes/6.0.0/include -I/home/linuxbrew/.linuxbrew/Cellar/cairo/1.16.0_5/include/cairo -I/home/linuxbrew/.linuxbrew/Cellar/pango/1.50.12/include/pango-1.0 -I/home/linuxbrew/.linuxbrew/Cellar/harfbuzz/6.0.0_1/include/harfbuzz -I/home/linuxbrew/.linuxbrew/Cellar/pango/1.50.12/include/pango-1.0 -I/home/linuxbrew/.linuxbrew/Cellar/fribidi/1.0.12/include/fribidi -I/home/linuxbrew/.linuxbrew/Cellar/cairo/1.16.0_5/include/cairo -I/home/linuxbrew/.linuxbrew/Cellar/pixman/0.42.2/include/pixman-1 -I/home/linuxbrew/.linuxbrew/Cellar/fontconfig/2.14.1/include -I/home/linuxbrew/.linuxbrew/Cellar/expat/2.5.0/include -I/home/linuxbrew/.linuxbrew/opt/freetype/include/freetype2 -I/home/linuxbrew/.linuxbrew/Cellar/libxcb/1.15/include -I/home/linuxbrew/.linuxbrew/Cellar/libxrender/0.9.11/include -I/home/linuxbrew/.linuxbrew/Cellar/libxext/1.3.5/include -I/home/linuxbrew/.linuxbrew/Cellar/libx11/1.8.3/include -I/home/linuxbrew/.linuxbrew/Cellar/libxcb/1.15/include -I/home/linuxbrew/.linuxbrew/Cellar/libxau/1.0.11/include -I/home/linuxbrew/.linuxbrew/Cellar/libxdmcp/1.1.4/include -I/home/linuxbrew/.linuxbrew/Cellar/libpng/1.6.39/include/libpng16 -I/home/linuxbrew/.linuxbrew/Cellar/jpeg-turbo/2.1.4/include -I/home/linuxbrew/.linuxbrew/Cellar/libtiff/4.4.0_1/include -I/home/linuxbrew/.linuxbrew/Cellar/glib/2.74.5/include -I/home/linuxbrew/.linuxbrew/Cellar/zlib/1.2.13/include -I/home/linuxbrew/.linuxbrew/Cellar/util-linux/2.38.1/include/libmount -I/home/linuxbrew/.linuxbrew/Cellar/util-linux/2.38.1/include/blkid -I/home/linuxbrew/.linuxbrew/Cellar/glib/2.74.5/include -I/home/linuxbrew/.linuxbrew/Cellar/glib/2.74.5/include/glib-2.0 -I/home/linuxbrew/.linuxbrew/Cellar/glib/2.74.5/lib/glib-2.0/include -I/home/linuxbrew/.linuxbrew/Cellar/pcre2/10.42/include -I/usr/include/gtk-3.0 -I/usr/include/at-spi2-atk/2.0 -I/usr/include/at-spi-2.0 -I/usr/include/gtk-3.0 -I/usr/include/atk-1.0 -I/usr/include/gdk-pixbuf-2.0 -g -O2 -D_REENTRANT -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=1
Linking: /bin/clang-15 -L/usr/local/lib -L/home/linuxbrew/.linuxbrew/Cellar/pango/1.50.12/lib -L/home/linuxbrew/.linuxbrew/Cellar/cairo/1.16.0_5/lib -L/home/linuxbrew/.linuxbrew/Cellar/glib/2.74.5/lib -L/usr/lib/x86_64-linux-gnu -fuse-ld=lld -lstdc++ -lm -L/usr/local/lib -Wl,--as-needed -o vim -L/usr/local/lib -L/home/linuxbrew/.linuxbrew/Cellar/pango/1.50.12/lib -L/home/linuxbrew/.linuxbrew/Cellar/cairo/1.16.0_5/lib -L/home/linuxbrew/.linuxbrew/Cellar/glib/2.74.5/lib -L/usr/lib/x86_64-linux-gnu -lgtk-3 -lgdk-3 -lpangocairo-1.0 -lpango-1.0 -lharfbuzz -latk-1.0 -lcairo-gobject -lcairo -lgdk_pixbuf-2.0 -lgio-2.0 -lgobject-2.0 -lglib-2.0 -lSM -lICE -lXt -lX11 -lXdmcp -lSM -lICE -lm -ltinfo -lselinux

Environment
OS: Ubuntu 22.04
Terminal: GNOME Terminal 3.44.0
$TERM: xterm-256color
Shell: GNU Bash 5.1.16
Logs and stack traces
No response

Today I've tried to start QEMU (self-compiled) with GTK+ 3.0 and it also failed.
It seems to be a problem with GTK, not with VIM.

closing then

For me running gvim build with GTK 3 works just fine...


Unable to find Bundle in package target snapshot tests when package depends on another package containing resources

Versions
SnapshotTesting: 1.8.1
Xcode: 12.2/12.3
iOS: 13+
Problem
I'm using local Swift packages to organise code within my workspace. This has been working fine until I tried adding snapshot tests to my feature packages, which depend on another package containing assets.
My workspace is currently structured like this (where -> means depends on):
MyApp (Xcode Project) -> Login (Package) -> UILibrary (Package)

When adding snapshot tests to my UILibrary, the snapshot tests work fine, and everything loads correctly. However, when I add snapshot tests to my Login package (which depends on UILibrary) the tests reach a fatal error when the UILibrary is trying to load the Color/Image assets (generated by SwiftGen) which uses the generated Bundle.module extension generated by Xcode.
fatalError("unable to find bundle named UILibrary_UILibrary")

Does anyone have any ideas on how to fix/workaround this? I'm currently facing re-organising my code as Xcode frameworks instead of Swift packages, as I've not had this issue on previous projects.
Further details
Things I have tried:

Adding UILibrary as a dependency to my LoginTests – still can't load.
Forcing SwiftGen to use Bundle(for: BundleToken.self) instead of Bundle.module when loading resources – still can't load.

My Login Package.swift file looks like this:
let package = Package(
	name: "Login",
	platforms: [.iOS(.v13)],
	products: [
		.library(name: "Login", type: .dynamic, targets: ["Login"])
	],
	dependencies: [
		.package(path: "../Shared/UILibrary"),
		.package(name: "SnapshotTesting", url: "https://github.com/pointfreeco/swift-snapshot-testing.git", from: "1.8.1"),
	],
	targets: [
		.target(name: "Login", dependencies: ["UILibrary"]),
		.testTarget(name: "LoginTests", dependencies: ["Login", "SnapshotTesting"])
	]
)

My UILibrary Package.swift file looks like this:
let package = Package(
	name: "UILibrary",
	platforms: [.iOS(.v13)],
	products: [
		.library(name: "UILibrary", type: .dynamic, targets: ["UILibrary"])
	],
	dependencies: [
		.package(name: "SnapshotTesting", url: "https://github.com/pointfreeco/swift-snapshot-testing.git", from: "1.8.1"),
	],
	targets: [
		.target(name: "UILibrary", dependencies: [], resources: [.process("Resources")]),
		.testTarget(name: "UILibrary Tests", dependencies: ["UILibrary", "SnapshotTesting"])
	]
)

Note: Resources contains my colors & image asset catalogs so they are processed as part of the UILibrary target.

I have raised an issue on the Swift forums here where a temporary workaround has been provided. I'll also file a bug report with Apple for this.

I have raised an issue on the Swift forums here where a temporary workaround has been provided. I'll also file a bug report with Apple for this.


ContextNet: 66.1% accuracy, 0.8 million params and 41.9 fps

Congratulations for nice work.
Have you also seen BMVC 2018 paper, ContextNet with 0.8 million parameter yield 66.1% accuracy and runs in 41.9 fps in the oldest Titan X gpu?

Hi, rudrapoudel
Thank you for your recommendation, I am going to understand the ContextNet.

@rudrapoudel Is it your work?

I see an early implementation at https://github.com/klickmal/ContextNet but I don't think it is related to @rudrapoudel or co-authors.

Hi, we will release the code in the first half of the Jan. Which I will announce in my twitter account- http://twitter.com/RudraPoudel

Finally: Tensorflow code is here: https://www.toshiba.eu/eu/Cambridge-Research-Laboratory/Computer-Vision/Resources/ContextNet/


Add upgrade test for Epinio

Fix #1380
This PR adds a test to make sure we can upgrade from latest Epinio release to the latest (main branch) Epinio.
Before upgrading, the test deploys a very simple app to be sure the app will still work after the upgrade.
Later, to improve the test, we can think about a more complete application ( with env vars, configurations etc) but for now, this PR is the first step.
Steps

Deploy k3d cluster
Install latest release of Epinio via the helm chart repo (https://epinio.github.io/helm-charts)
Create a simple app and check if it's reachable via its route
Build docker image (epinio-server) with latest epinio code and import it in k3d
Upgrade epinio by using the helm chart from the submodule
Test if the app is still reachable

Notes:

domain has to be set again when we upgrade, I think that's because we deploy from the helm chart folder whereas we deploy from the official helm repo at the beginning

Local tests: :heavy_check_mark:
make acceptance-cluster-setup; make install-cert-manager; make prepare_environment_k3d
Env vars:
EPINIO_SYSTEM_DOMAIN=<IP_ADDRESS>.omg.howdoi.website
EPINIO_RELEASED=true

make test-acceptance-upgrade
...
Ran 1 of 1 Specs in 101.026 seconds
SUCCESS! -- 1 Passed | 0 Failed | 0 Pending | 0 Skipped

I installed 1.0.0 and execute all the steps, then I got latest epinio deployed:

CI: :heavy_check_mark:
https://github.com/epinio/epinio/runs/7125423165?check_suite_focus=true

CI fails with:
  Normal   BackOff    25s (x2 over 57s)  kubelet            Back-off pulling image "appscode/kubed:v0.13.2"
  Normal   Pulling    12s (x3 over 63s)  kubelet            Pulling image "appscode/kubed:v0.13.2"
  Warning  Failed     6s (x3 over 58s)   kubelet            Failed to pull image "appscode/kubed:v0.13.2": rpc error: code = Unknown desc = failed to pull and unpack image "docker.io/appscode/kubed:v0.13.2": failed to copy: httpReaderSeeker: failed open: unexpected status code http://epinio-acceptance-registry-mirror:5000/v2/appscode/kubed/manifests/sha256:40ca90967d4a43d03c590a1c4b233cadf8fed747efcf34e64ff6a1871468a746?ns=docker.io: 500 Internal Server Error - Server message: unknown: unknown error

  Warning  Failed       52s (x2 over 73s)  kubelet            Failed to pull image "nginx:1.21.6": rpc error: code = Unknown desc = failed to pull and unpack image "docker.io/library/nginx:1.21.6": failed to copy: httpReaderSeeker: failed open: unexpected status code http://epinio-acceptance-registry-mirror:5000/v2/library/nginx/manifests/sha256:2bcabc23b45489fb0885d69a06ba1d648aeda973fae7bb981bafbb884165e514?ns=docker.io: 500 Internal Server Error - Server message: unknown: unknown error
  Warning  Failed       52s (x2 over 73s)  kubelet            Error: ErrImagePull
  Normal   Pulling      52s (x2 over 73s)  kubelet            Pulling image "registry:2.7.1"
  Warning  Failed       47s (x2 over 68s)  kubelet            Failed to pull image "registry:2.7.1": rpc error: code = Unknown desc = failed to pull and unpack image "docker.io/library/registry:2.7.1": failed to copy: httpReaderSeeker: failed open: unexpected status code http://epinio-acceptance-registry-mirror:5000/v2/library/registry/manifests/sha256:169211e20e2f2d5d115674681eb79d21a217b296b43374b8e39f97fcf866b375?ns=docker.io: 500 Internal Server Error - Server message: unknown: unknown error
  Warning  Failed       47s (x2 over 68s)  kubelet            Error: ErrImagePull
  Normal   BackOff      33s (x2 over 68s)  kubelet            Back-off pulling image "nginx:1.21.6"
  Warning  Failed       33s (x2 over 68s)  kubelet            Error: ImagePullBackOff
  Normal   BackOff      33s (x2 over 68s)  kubelet            Back-off pulling image "registry:2.7.1"
  Warning  Failed       33s (x2 over 68s)  kubelet            Error: ImagePullBackOff
  Normal   Pulling      19s (x3 over 78s)  kubelet            Pulling image "nginx:1.21.6"

Maybe this can be used:
helm upgrade --help
...
      --reuse-values                 when upgrading, reuse the last release's values and merge in any overrides from the command line via --set and -f. If '--reset-values' is specified, this is ignored
...

Nice, I was not aware of this one and I missed it! Let me try it.


Support timezones when reading and writing datetime columns

Timezone information is ignored at the moment, both for reading and for writing.
See https://github.com/Toblerity/Fiona/blob/791785b813fce97e5c3d60bd725fa2190645d2e3/fiona/ogrext.pyx#L401-L404 and https://github.com/Toblerity/Fiona/blob/791785b813fce97e5c3d60bd725fa2190645d2e3/fiona/ogrext.pyx#L431-L435 how fiona handles this (and https://gdal.org/development/rfc/rfc56_millisecond_precision.html?highlight=TZFlag#core-changes with very brief details)
The PR that introduced this in fiona: https://github.com/Toblerity/Fiona/pull/915

Hey @theroggy have you done any work on this? If not I might have a look if that's okay.

Hey @theroggy have you done any work on this? If not I might have a look if that's okay.

@m-richards No I haven't... feel free!

So having a look at this in a bit more detail there's more complexity than I originally realised (unless I'm missing something), numpy datetime64 doesn't support utc offsets- which seems to make numpy arrays as the intermediate between dataframes and ogr a little tricky to make work.
Some thoughts:

Create a custom representation of a utc offset field as a datetime field and a second field encoding offset - bad for interop with other tools so not ideal
Convert timezone aware fields to UTC on write, read them back as utc - simpler, but not a fully faithful round-trip, still need to handle naive datetimes and distinguishing them from utc ones.
The last thought I have seems rather complex, but would it make sense to use the pandas datetime64[ns, tz] extension array dtype and feed that down (or the underlying representation) to ogr_write

(apologies if I'm missing something obvious, I thought this might be alright point to get familiar with some the pyogrio internals, but still verying much learning at this point)

Hmm... indeed... when writing it seems the datetimes at the moment will be implicitly converted to UTC already during the conversion of pandas series to numpy arrays:
https://pandas.pydata.org/docs/reference/api/pandas.Series.values.html

At the time of adding write support I noticed that the conversion to python datetime took quite some time... and so I wondered if it would be faster to convert the datetime columns to string and using (OGRParseDate)[https://gdal.org/api/vector_c_api.html#_CPPv412OGRParseDatePKcP8OGRFieldi] to let gdal parse them back to datetimes.
Not a super elegant solution, but it might be an option here as well...

and so I wondered if it would be faster to convert the datetime columns to string and using OGRParseDate to let gdal parse them back to datetimes.

That could indeed be something to test and time.
Unfortunately, I don't see any API for the reverse? (so we could also read the datetimes through a string representation, and then parse those on our side) Because that could also have been a solution for carrying the timezone information in the string.
It seems that the OGRParseDate ignores milliseconds though, so if we do this it would have to be conditional on the presence of that, making it less attractive option ..

Convert timezone aware fields to UTC on write, read them back as utc - simpler, but not a fully faithful round-trip, still need to handle naive datetimes and distinguishing them from utc ones.

I think getting back the datetimes in UTC would already be an improvement, and maybe the best we can do on the short term.

I was looking into this a bit, some observations:

For reading, we could use OGR_F_GetFieldAsString on a datetime field (instead of OGR_F_GetFieldAsDateTimeEx), to get a string formatted as "%Y/%m/%d %H:%M:%S(%z)" (so ISO formatting except the use of / instead of - for the date part). That way we could delay the parsing to datetime64 dtype, and for read_dataframe use pandas' to_datetime parser, which can handle strings with tz offsets to result in a UTC datetime64 dtype. This is also faster than the current implementation (data[i] = datetime.datetime(..).isoformat() in the reading loop).
One possible way to do this would be a datetime_as_string=False/True keyword to be added to read(), which can be propagated to get_fields/process_fields in _io.pyx. Alternatively, we could also always do this for datetime fields in _io.pyx, and only after reading all features convert the strings to datetime64[ms] dtyped array.

An alternative solution for reading would be to account for the tz offset when creating the datetime.datetime object (so we get the UTC values), but then we still need a way to signal to read_dataframe / pandas that the the datetime64[ms] array is tz-aware (as UTC) and not tz-naive, so we can use the correct dtype for the column in the resulting geodataframe.
We could somehow keep track of this and then returns those flags from ogr_read (in an extra key in the meta dict?), but that also sounds quite ugly to do this separately from returning the data (in the current way this is implemented, we are limited in returning just numpy arrays, and not more information about the original data type)

For writing, we have a similar problem that we are passing in a numpy datetime64 array as UTC values (which we already do), but so inside ogr_write we just see the numpy array, without knowing whether it was originally tz-aware (and we need to indicate that it is UTC when setting the field) or tz-naive. So also here we would pass down some kind of flag per column to indicate this extra information about the dtype. Or maybe, now I am writing this, we could also pass down the original pandas dtype per column, in addition to the numpy arrays of field_data. That's maybe more generic, and that information could be used then to check whether it's tz-aware or not in _io.pyx.

Also for writing, we could check if going to a string repr instead of creating a datetime.datetime object (to access all the datetime fields) is faster. As mentioned above, OGRParseDate (which GDAL will use under the hood when calling OGR_F_SetFieldString on a datetime field) documents to ignore milliseconds, but that's something we should verify, as I am not sure that is actually the case based on a quick look at the implementation.

I've been looking at this again too and have opened a very rough proof of concept at #253, to try and get us moving forward on this (joris, you already do have detailed notes, I don't know if you have a prototype as well, more than happy for that to be superceded). Some things I've learnt as part of doing that:

I opted to read datetimes as strings, rather than look at accounting for the tz offset when creating datetime.datetime objects inside ogr_read. Basically the string parsing just seemed simpler as a first draft
(Separately, the PR currently overlooks the case of the numpy api trying to read a spatial file with timezones without converting to strings, this probably should be added)

For writing,

inside ogr_write we just see the numpy array, without knowing whether it was originally tz-aware

we could also pass down the original pandas dtype per column, in addition to the numpy arrays of field_data

On both of these GDAL is expecting values in a local time, not utc. That's fine, we can convert a pandas datetime column to naïve, and then pass this down in the numpy array. However, we then need to recover the timezone offset for each row. Originally I thought we could pass down the timezone information per column, but this doesn't work if the original pandas data was e.g in timzone Australia/Sydney which has summer daylight saving. In these cases, the offset can be row dependent, so we'd need to track this for each row. If you pass the dtypes down, to actually use these to recover a timezone offset, you have to re-localize the the datetime objects, something like;
datetime = field_value.astype("datetime64[ms]").item()
gdal_tz = datetime.astimezone(tz).utcoffset() / datetime.timedelta(minutes=15)) + 100

at least as far as I can tell (I might be missing something). At this point I decided to park this and look at writing strings, I can't imagine this is very performant (but if we decide it's worth benchmarking explicitly it I can go back and have a look).

Thanks Matt!

[For writing] On both of these GDAL is expecting values in a local time, not utc. That's fine, we can convert a pandas datetime column to naïve, and then pass this down in the numpy array. However, we then need to recover the timezone offset for each row.

I think going through strings is a good alternative to explore, but in case we want to get back to using OGR_F_SetFieldDateTimeEx, I think it could also be a sufficient option to just store tz-aware data as UTC (yes, that doesn't preserve actual offset / local time information, but at least it preserves the time point information). For that, we can pass the numpy array with UTC values down to ogr_write, and then only need a single True/False flag for that column if it's local or GMT.

At this point I decided to park this and look at writing strings, I can't imagine this is very performant (but if we decide it's worth benchmarking explicitly it I can go back and have a look).

I think it might be worth checking performance, but I don't necessarily expect it to be too bad. Currently, the code path for writing datetimes is also not great, because we need to convert each timestamp value (stored as a single integer number by numpy/pandas) to a python datetime.datetime object (which stores components, and get the year, month, day, hour etc from it).


Correct a typo in configs/README.md

Issues associated with this PR
Your PR should present related issues you want to solve.
Sign the CLA
Make sure you have signed the CLA
Solutions
You should show your solutions about the issues in your PR, including the overall solutions,
details and the changes. At this time, Chinese is allowed to describe these.
UT result
Unit Test is needed if the code is changed, your unit test should cover boundary cases, corner cases, and some exceptional cases.
And you need to show the ut result.
Benchmark
If your code involves the processing of every request, you should give the Benchmark Result
Code Style

Make sure Goimports has run
Show Golint result

Codecov Report

Merging #793 into master will decrease coverage by 0.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #793      +/-   ##
==========================================
- Coverage   51.64%   51.57%   -0.08%
==========================================
  Files         205      205
  Lines       17052    17052
==========================================
- Hits         8807     8794      -13
- Misses       7352     7360       +8
- Partials      893      898       +5

Impacted Files
Coverage Δ

pkg/stream/stream.go
46.66% <0%> (-13.34%)
:arrow_down:

pkg/buffer/iobuffer.go
75.31% <0%> (-2.52%)
:arrow_down:

pkg/log/logger.go
55.05% <0%> (-1.69%)
:arrow_down:

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 3152073...3b44205. Read the comment docs.

Please sign CLA

I already signed, not sure why the bot didn't update here.

I already signed, not sure why the bot didn't update here.

you can close this pr, and start a new one

I already signed, not sure why the bot didn't update here.

do not reopen this one,  make a new pull request

this pr still have cla bug. close it .


Requestify AbstractStorageDecl::hasStorage().

The hasStorage() computation is used in many places to determine the signatures of other declarations. It currently needs to expand accessor macros, which causes a number of cyclic references. Provide a simplified request to determine hasStorage without expanding or resolving macros, breaking a common pattern of cycles when using macros.
Fixes rdar://109668383.

@swift-ci please smoke test

@swift-ci please smoke test

@swift-ci please test source compatibility

@swift-ci please clean smoke test

@swift-ci please smoke test


Is there cubie for DecisionTreeClassifier ?

Hey guys! I'm new to Factories.
Is there cubie for DecisionTreeClassifier so that I can serialize/deserialize it?

Looks like there is TreeNodeCubbie. I tried using it but deserialization doesn't seem to be working. I get key not found: left exception on fetch().
@lvilnis @strubell Guys, could you please take a look at it ?

And looks like problem is not in fetching cubie but in storing it. Leaves are not serialized.


Landing Page - Front End

Overview
Front End development, I made a fork of the project where I will be implementing  features and making improvements based on the wireframe.
Fork Information

Repository Fork URL: https://github.com/MatiSalinas/computer-laboratory-web-page

Setup Instructions

Clone the Repository
Open your terminal and run the following command:
git clone https://github.com/MatiSalinas/computer-laboratory-web-page.git

Change the branch
git checkout ramaFrontEnd
3 Navigate to the Project Directory
cd web

Install Dependencies
Ensure you have Node.js installed, then run:
npm install

Start the Development Server
npm run dev
The app will be available at http://localhost:5173 .

Additional Notes

Make sure to use the latest version of Node.js and npm.
The first commits could not be verified due to an issue with my GPG key, which I resolved in the subsequent ones.
So far I have made the components for : NavBar, Carousel,AboutUs and News.

If you have any questions or need further clarification, feel free to point out changes to be made.

Update on Recent Commits
In the latest commits of my fork, I have made the following improvements:

Made the Carousel and AboutUs components responsive, ensuring a better user experience across devices.
Disabled the animations on the News component for mobile view. However, I still need to apply the necessary styles to ensure it looks good on mobile.

Update

Review done


HTML and IMAGE printing support

Please provide html and image printing types also

Which code to kick cashdrawer using recta?

hello, is there any update for image or printing style?


Fix UI for setting keymaps

The Key Map Settings UI was not being displayed. This PR changes names of all three commands, so that "atom-" is prepended to each command, so that the Settings UI recognizes them.

Can anyone confirm if this is still an issue for them with the latest version of the package?


Support Compose Multiplatform

The library doesn't use any android only depdencies?
Then it can be updated to use Compose Multiplatform as to suppport all platforms.

It should have dependency to android Color. I'm not available to do it for the time being. Any contribution is more than welcome.


Trying to start Jasper on Raspberry Pi with Google TTS

For a school project we are installing Jasper to a Raspberry Pi using Google TTS as a text to speech engine.  As far as we know we have only the Google TTS installed and have never tried doing anything with the Sphinx engine.  We are having some problems

When starting Jasper we get an error "No handlers could be found for logger "cmuclmtk"".  cmuclmtk seems to be only used if using the Sphinx engine.  We cannot figure out were to go from here.

Check for the Sphinx dependencies. This may help you: http://jasperproject.github.io/documentation/installation/#installing-dependencies


Latest release - ErrorException  : Undefined index: groupName

apidoc:generate failing after updating to latest release:
/vendor/mpociot/laravel-apidoc-generator/src/Commands/GenerateDocumentation.php:88
    84|         $parsedRoutes = $this->processRoutes($generator, $routes);
    85|         $parsedRoutes = collect($parsedRoutes)->groupBy('groupName')
    86|             ->sortBy(static function ($group) {
    87|                 /* @var $group Collection */
  > 88|                 return $group->first()['groupName'];
    89|             }, SORT_NATURAL);
    90|
    91|         $this->writeMarkdown($parsedRoutes);
    92|     }

Can you dd($parsedRoutes) before line 85?

Thanks @shalvah, narrowed down my routes to just one and still running into issue.
Below is the dump you requested. Note in this case I have removed the @group annotation - and my default group Legacy is being applied, as configured.
Processed route: [POST] file/upload
array:1 [
  0 => array:15 [
    "id" => "8545ee0a306d7cad2682a39face4402f"
    "group" => "Legacy"
    "title" => "Upload a file."
    "description" => "asdfafd"
    "methods" => array:1 [
      0 => "POST"
    ]
    "uri" => "file/upload"
    "boundUri" => "file/upload"
    "queryParameters" => []
    "bodyParameters" => []
    "cleanBodyParameters" => []
    "cleanQueryParameters" => []
    "authenticated" => false
    "response" => null
    "showresponse" => false
    "headers" => array:1 [
      "Authorization" => "Bearer {token}"
    ]
  ]
]
'''

Adding the `@group` annotation back to the class docblock results in same error - group is including the description but perhaps this is expected:

Processed route: [POST] file/upload
array:1 [
0 => array:15 [
"id" => "8545ee0a306d7cad2682a39face4402f"
"group" => """
Uploads\n
\n
APIS for uploads
"""
"title" => "Upload a file."
"description" => "asdfafd"
"methods" => array:1 [
0 => "POST"
]
"uri" => "file/upload"
"boundUri" => "file/upload"
"queryParameters" => []
"bodyParameters" => []
"cleanBodyParameters" => []
"cleanQueryParameters" => []
"authenticated" => false
"response" => null
"showresponse" => false
"headers" => array:1 [
"Authorization" => "Bearer {token}"
]
]
]

Error is as described: `Undefined index: groupName` - can see no `groupName` in any of the tests I've run.

Please see PR 565 - has resolved the problem for me.

What release are you on? 3.12.0?

I can't accept your PR because it reverts a recently added feature. I've released version 3.14.0. Please take it for a spin and let me know if your issues still persist. (You may need to republish any assets you had published one more time.)

I've upgraded to latest v3.15.0
I've looked at the feature briefly - ability to set a group without a description?
I'm confused - should I update my documentation in some way to get this working? Still getting the same problem. Published apidoc views, ran rebuild, force published, even republished the config.
Thanks for any assistance!

Can I see some typical routes you're documenting?
What it looks like here is one part of the generation process is calling an old version of the code (group in parsedRoutes), while the other part is calling the current code (looking for groupName) . Extremely weird. Can you paste the content of the getRouteGroup method in the Generator.php file?

Sure - note that I rolled back to 3.14.0 to see if that helped things.

In 3.15.0 changing groupName to group did not allow doc generation - which is what I did in 3.12.0
Rolling back to 3.14.0 did allow me to generate docs by doing the above : groupName --> group

    /**
     * @param ReflectionClass $controller
     * @param array $methodDocBlock
     *
     * @return array The route group name, the group description, ad the route title
     */
    protected function getRouteGroup(ReflectionClass $controller, array $methodDocBlock)
    {
        // @group tag on the method overrides that on the controller
        if (! empty($methodDocBlock['tags'])) {
            foreach ($methodDocBlock['tags'] as $tag) {
                if ($tag->getName() === 'group') {
                    $routeGroupParts = explode("\n", trim($tag->getContent()));
                    $routeGroupName = array_shift($routeGroupParts);
                    $routeGroupDescription = trim(implode("\n", $routeGroupParts));

                    // If the route has no title (aka "short"),
                    // we'll assume the routeGroupDescription is actually the title
                    // Something like this:
                    // /**
                    //   * Fetch cars. <-- This is route title.
                    //   * @group Cars <-- This is group name.
                    //   * APIs for cars. <-- This is group description (not required).
                    //   **/
                    // VS
                    // /**
                    //   * @group Cars <-- This is group name.
                    //   * Fetch cars. <-- This is route title, NOT group description.
                    //   **/

                    // BTW, this is a spaghetti way of doing this.
                    // It shall be refactored soon. Deus vult!💪
                    if (empty($methodDocBlock['short'])) {
                        return [$routeGroupName, '', $routeGroupDescription];
                    }

                    return [$routeGroupName, $routeGroupDescription, $methodDocBlock['short']];
                }
            }
        }

        $docBlockComment = $controller->getDocComment();
        if ($docBlockComment) {
            $phpdoc = new DocBlock($docBlockComment);
            foreach ($phpdoc->getTags() as $tag) {
                if ($tag->getName() === 'group') {
                    $routeGroupParts = explode("\n", trim($tag->getContent()));
                    $routeGroupName = array_shift($routeGroupParts);
                    $routeGroupDescription = implode("\n", $routeGroupParts);

                    return [$routeGroupName, $routeGroupDescription, $methodDocBlock['short']];
                }
            }
        }

        return [$this->config->get(('default_group')), '', $methodDocBlock['short']];
    }

With regard to routes, making use of apiResource:
        Route::apiResource('comments', 'CommentController');
        Route::apiResource('projects', 'ProjectController');

and some other things. That's why I ignored all routes but one to see if something in my routes was contributing to the problem

An example of your doc block?
On Sun, 1 Sep 2019, 02:22 Nicholas Catanchin<EMAIL_ADDRESS>wrote:

With regard to routes, making use of apiResource:
    Route::apiResource('comments', 'CommentController');        Route::apiResource('projects', 'ProjectController');

and some other things. That's why I ignored all routes but one to see if
something in my routes was contributing to the problem
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mpociot/laravel-apidoc-generator/issues/561?email_source=notifications&email_token=ADNSD4NY4RVDSKU6R3J2AITQHMKLXA5CNFSM4IQTEME2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD5TYH5I#issuecomment-526877685,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADNSD4NAGCSIEQNBWKRNYELQHMKLXANCNFSM4IQTEMEQ
.

How about the contents of the processRoute method?

How about the contents of the processRoute method?

I've rolled back to v3.11.0 - outside of the changes from groupName to group that were in the PR etc, I have not made any modifications to the package source.
I'll do some more testing in the hopes of getting things to work but would expect that the codebase would match what you are seeing. Depending on version - what version are you requesting above

An example of your doc block?

Regarding doc block - nothing too unusual going on:
/**
 * Projects API.
 *
 * @group Projects
 */

Some controllers are not documented yet:
    /**
     * Store a newly created resource in storage.
     *
     * @param  \Illuminate\Http\Request  $request
     * @return \Illuminate\Http\Response
     */

Others quite straightforward:
    /**
     * Get many.

     *
     * @authenticated
     * @bodyParam client_id int required Client ID of the projects.
     *
     * @return \Illuminate\Http\Response
     */

To try to eliminate issues on my side, I reduced the parsed routes down to just 1.
I've seen mentioned elsewhere that Laravel ApiResources are not yet supported. Have not had an issue with those before.
Cheers

This is an extremely weird case. I can't reproduce it, no matter what I try. It doesn't even make sense logically.

To try to eliminate issues on my side, I reduced the parsed routes down to just 1.

Do you still get the error with one route?
Does 3.11.0 work for you?

@shalvah I have determined the cause of this issue - it was due to an Override put in several months ago in order to enable additional parameter types. The Generator class was therefore outdated and causing numerous issues.
Sincere apologies for wasting your time here. After removal of said override, I am happily generating docs with R3.16.0.


[2pt] Create a Game class in a separate .rb file. #3

All Game class properties visible in the diagram should be defined and set up in the constructor method.

Hello @sebalp100.Here you will find the link to the pull request relating to this issue.


Maybe isn't working for nighlys of iterm2?

I had some issues that docker-machine didn't support my latest iterm2 because the api had changed. Maybe this is a similar issue:
https://github.com/docker/toolbox/issues/81#issuecomment-166917347
~ ❯❯❯ itermocil test                                                                                ⏎
Traceback (most recent call last):
  File "/usr/local/Cellar/itermocil/0.1.8/libexec/bin/itermocil", line 6, in <module>
    main()
  File "/usr/local/Cellar/itermocil/0.1.8/libexec/bin/itermocil.py", line 687, in main
    instance = Itermocil(filepath, here=args.here, cwd=cwd)
  File "/usr/local/Cellar/itermocil/0.1.8/libexec/bin/itermocil.py", line 69, in __init__
    self.process_file()
  File "/usr/local/Cellar/itermocil/0.1.8/libexec/bin/itermocil.py", line 528, in process_file
    self.arrange_panes(len(window['panes']), layout)
KeyError: 'panes'

I'm on Build 2.9.20151006-nightly.
The config is as simple as this:
~ ❯❯❯ cat .teamocil/test.yml                                                                                ⏎
windows:
  - name: test
    root: ~/
    layout: tiled
    commands:
      - pwd
~ ❯❯❯

Oh, sorry, that was not it. Get the same in iterm 2.1.4.

Oh, sorry, my bad. Incorrect yml I see now :)

This might be the most confusing ticket ever, but I re-tried iterm2 beta after realizing the above was just bad .yml by myself and now I get:
~ ❯❯❯ itermocil test
26:33: execution error: An error of type -10665 has occurred. (-10665)
Traceback (most recent call last):
  File "/usr/local/Cellar/itermocil/0.1.8/libexec/bin/itermocil", line 6, in <module>
    main()
  File "/usr/local/Cellar/itermocil/0.1.8/libexec/bin/itermocil.py", line 687, in main
    instance = Itermocil(filepath, here=args.here, cwd=cwd)
  File "/usr/local/Cellar/itermocil/0.1.8/libexec/bin/itermocil.py", line 28, in __init__
    major_version = self.get_major_version()
  File "/usr/local/Cellar/itermocil/0.1.8/libexec/bin/itermocil.py", line 95, in get_major_version
    return float(v[:3])
ValueError: could not convert string to float:

That certainly seems like API issues against iterm2, right?

It actually worked with beta second time though…

Well, this was an emotional roller-coaster...
I actually have a fix for that convert string to float issue (that happens when Applescript can't find iTerm, so even though iTermocil will not fail, in theory the Applescript shouldn't run correctly).

Yeah, weird. It only happened once and then everything worked just fine. Sorry for all the previous confusion. And thanks for an excellent tool. Will save me tons of time.

I'm still experiencing this issue running iTerm2 v2.9.20160206.


Set up Krypto module for use in KPlutus

fixes: #87

It looks great to me!

Looks good to me too! I guess we should change the syntax of G2Point to be something like G2Point(Int, Int, Int, Int) instead, because that is confusing, but the syntax module change would probably have been needed later anyway!


Change to version 1.5 of traefik

This depends on the pull request of jenkins-utils before it can be verified and merged.


settings_display: Add spacing to the Save and Add Emoji button.

The save button in user groups of organization settings has zero spacing and its top padding needs to be increased.
To reproduce the bug:

Open the Settings menu
Open User groups in Organization section.

Note: The top padding needs to be increased only in certain media queries.

@zulipbot abandon

@zulipbot claim

@zulipbot claim

I'm still working on it!

@zulipbot claim

@zulipbot claim

@kashyaparka I'm already working on this issue and has raised the PR for the same for a while, it is already under review and needs some changes.

Commenting to inform that I am working on this issue!

@KarishmaVanwari please choose an issue with a "help wanted" label. This issue requires a design discussion before it's ready to be worked on. Thanks!

Started a discussion on chat.zulip.org here.


feat: support osv schema v1.5.0

https://github.com/ossf/osv-schema/releases/tag/v1.5.0
Closes #323

@Churro I can't request a review via the GitHub UI, but I would appreciate your input.

OSV schema changes LGTM.
Back then, I intentionally didn't add pub as there were only 6 advisories (and still not more) and IIRC none of those had a purl identifier, i.e., they could have never been found anyway. It seems this has changed meanwhile, e.g. GHSA-9324-jv53-9cc8 includes "purl": "pkg:pub/dio"`.


Convert to Java

Java conversion for the kata
Should be a line-by-line equivalent as far as possible.

Thankyou! looks good.


Improve README for project and generated app

@mshima We might want to move the Okta and Auth0 documentation to the project's README and then just link to it from the generated one. I'm not sure what the best approach is.

What about adding a section indicating the benefits of OAuth2 to mobile applications and add Okta and Auth0 sections to it?

What about adding a section indicating the benefits of OAuth2 to mobile applications and add Okta and Auth0 sections to it?

Done in https://github.com/mshima/generator-jhipster-ionic/pull/3.


Fix typo in cargoNextestExtraArgs docs

Motivation
The API docs seem wrong here, copy-pasta probably.
Checklist

[x] updated docs/API.md (or general documentation) with changes
[ ] updated CHANGELOG.md

Thanks for fixing this!

Thank you for Crane, it's proving really useful :pray:


Memory Insufficient Warning with docker

On a VM wich initially ran with only 1 GB of RAM, I increased that amount to 4 GB, installed docker and other requirements, then ran docker-composer up, as indicated on the Installation & Configuration page. But after downloading  5.2 GB of stuff, I get this message
Step 13/37 : RUN /frontend-mem-nag.sh         && cd /app/superset-frontend         && npm ci
 ---> Running in e18f62b44119
    ===============================================
    ========  Memory Insufficient Warning =========
    ===============================================

    It looks like you only have 0.284832 GB of
    memory free. Please increase your Docker
    resources to at least 3GB

    ===============================================
    ========  Memory Insufficient Warning =========
    ===============================================
npm ERR! network timeout at<EMAIL_ADDRESS>
npm ERR! A complete log of this run can be found in:
npm ERR!     /root/.npm/_logs/2020-05-01T20_06_18_860Z-debug.log
ERROR: Service 'superset' failed to build: The command '/bin/sh -c /frontend-mem-nag.sh         && cd /app/superset-frontend         && npm ci' returned a non-zero code: 1

Expected results
The process should use the available memory
$ free
              total        used        free      shared  buff/cache   available
Mem:        4041492      401264      513704        6940     3126524     3344720
Swap:       1046524        6412     1040112

Actual results
Not enough memory (for some reason)
Environment

superset version: as of May 1st 2020 (seriously, where can I find the version number?)
python version: 3.7.3
node.js version: 12.10.0
npm version: 6.14.4

Checklist
Make sure these boxes are checked before submitting your issue - thank you!

[x] I have checked the superset logs for python stacktraces and included it here as text if there are any.
[x] I have reproduced the issue with at least the latest released version of superset.
[x] I have checked the issue tracker for the same issue and I haven't found one similar.

Closing this issue because it can be resolved by boosting the RAM until it works. I didn't bother to try the bare minimum, 8 GB and this error goes away.
But I still have an error, so will open another issue.

@yanickrochon - what command did you use to increase the memory allocation for docker ?

I do not recall. We scrapped the idea of using docker and installed Superset as a Python module instead. Not as pretty, but we could spare the extra VM in this case. I would've hoped that someone more experienced with Docker would have answered by now.


[ENHANCEMENT] - Support for Microsoft Windows?

Environment

fastapi version is: 0.79.0
fastapi-responseschema version is: 1.2.2
python version is: 3.9.12

Describe the enhancement
Hello, I'm a big fan of your work. I was just trying to run this library on a MS Windows environment in Conda. However I get the error:
ModuleNotFoundError: No module named 'termios'
There are some references that the module termios is only available on Linux:
https://stackoverflow.com/questions/52062977/unable-to-import-termios-package
By any chance do you expect to support Windows too? Maybe by using a cross-platform module, don't know.
However I guess this is a low priority, just wanted to let you know.
Expected behavior
Work on Windows.
Snippets
Just used the example code in README.MD
Additional context
I'm a linux user usually, just happened to use Windows :(

Hello! Sorry for the delay, the problem was solved in version 1.3.0 there was an unused import causing the bug for non unix OS.
I removed the import in this commit.
Thank you so much for your appreciation! :slightly_smiling_face:


NIFI-13076: reduce enum array allocation

Summary
NIFI-13076
Motivation
reduce enum array allocation
Modifications
cache enum .values() in a static variable
Additional context
https://www.gamlor.info/wordpress/2017/08/javas-enum-values-hidden-allocations/
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[x] Apache NiFi Jira issue created

Pull Request Tracking

[x] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[x] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[x] Pull Request based on current revision of the main branch
[x] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[ ] Build completed using mvn clean install -P contrib-check

[ ] JDK 21

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

Similar work in Apache Pinot
https://github.com/apache/pinot/pull/12579
Similar work in Apache Iceberg
https://github.com/apache/iceberg/pull/10126

Thanks for the fix @sullis looks good to me. +1 merged to main


stdout should be provided on deployment and validation

When using arm-deploy @v1, there is no output of the validate or deployment steps - only these messages.

To validate the deployment or validation steps, it would be nice to know the general deployment output by just writing the output from the -o json to standard output.
This will also help with history keeping when looking back at the logs.

Closing this Issues due to inactivity.


Can't select specific point on downsampled data

When perform a general query
curl -G "http://example.com:8086/db/influxdb/series?u=root&p=root&pretty=true" --data-urlencode "q=select * from 1d.percentile.top_tenth.example"

I get the expected result
[
    {
        "name": "1d.percentile.top_tenth.example",
        "columns": [
            "time",
            "sequence_number",
            "percentile"
        ],
        "points": [
            [
<PHONE_NUMBER>000,
                1,
                1262
            ],
            ...
        ]
    }
]

However, if I try to select a specific point, as in the documented pattern
curl -G "http://example.com:8086/db/influxdb/series?u=root&p=root&pretty=true" --data-urlencode "q=select * from 1d.percentile.top_tenth.example where time =<PHONE_NUMBER>000 and sequence_number = 1"

I get an empty result
[]

I'm not sure if this is an API issue or if it's because I'm querying on a downsampled series with aggregate data and that isn't supported (the continuous query used for the series is select percentile(clicks, 99.9) from example group by time(1d) into 1d.percentile.top_tenth.example)

I ran into the same issue and it has nothing to do with downsampling as it can be reproduced from scratch like this:
# Create database
curl -X POST 'http://localhost:8086/db?u=root&p=root' -d '{"name": "test"}'

# Write point
curl -X POST 'http://localhost:8086/db/test/series?u=root&p=root&time_precision=u' -d '[{"columns":["time","value"],"name":"ts","points":[[1419770941003000,1234]]}]'

# List points
curl -G 'http://localhost:8086/db/test/series?u=root&p=root&time_precision=u' --data-urlencode 'q=select * from ts'
[{"name":"ts","columns":["time","sequence_number","value"],"points":[[1419770941003000,10001,1234]]}]

The following query returns an empty array in v0.8.5 and v0.8.8 on Mac OS X 10.10.1, whereas in v0.8.3 on Ubuntu 14.04.1 LTS a Couldn't look up columns error is returned:
curl -G 'http://localhost:8086/db/test/series?u=root&p=root&time_precision=u' --data-urlencode 'q=select * from ts where time =<PHONE_NUMBER>003000 and sequence_number = 10001'
[]

Furthermore when trying to delete a single point an error is returned too (v0.8.3, v0.8.3 and v0.8.8):
curl -G 'http://localhost:8086/db/test/series?u=root&p=root&time_precision=u' --data-urlencode 'q=delete from ts where time =<PHONE_NUMBER>003000 and sequence_number = 10001'
Delete queries can't have where clause that don't reference time

@vforgione
In order to select a specific point, you have to give the timestamp at microsecond precision.  Influfxdb queries return the timestamp at millisecond precision by default.  (I'm in 0.8.8)  Change your general query to this:
curl -G "http://example.com:8086/db/influxdb/series?u=root&p=root&pretty=true&time_precision=u" --data-urlencode "q=select * from 1d.percentile.top_tenth.example"
to get the timestamp with the right precision.
Then, perform your specific query like so:
curl -G "http://example.com:8086/db/influxdb/series?u=root&p=root&pretty=true" --data-urlencode "q=select * from 1d.percentile.top_tenth.example where time =<PHONE_NUMBER>000###u and sequence_number = 1"
where '###' is the last three digits of your microsecond-precision timestamp.  Don't forget the 'u'!


pre-commit autoupdate

https://pre-commit.com/#pre-commit-autoupdate
Updates our hooks to the latest versions
Description
This PR updates our git hooks.

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)
[X] build/CI

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[ ] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

Ran locally:
pre-commit run --all-files
How did you try to break this feature and the system with this change?

@blueorangutan package

@jbampton a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✔ el7 ✔ el8 ✔ el9 ✔ debian ✔ suse15. SL-JID 9114

@blueorangutan package

@DaanHoogland a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✔ el7 ✔ el8 ✔ el9 ✔ debian ✔ suse15. SL-JID 9372


Adding zwave components causes entity_registry to lose disabled_by property

**Home Assistant release with the issue: 0.66.0

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

**Component/platform: zwave

Description of problem:
I have recently been adding numerous zwave devices. When new devices are added to the entity_registry, all existing "disabled_by" properties are discarded.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
zwave:
  usb_path: /zwaveusbstick
  network_key: !secret zwave_secure

Traceback (if applicable):

Additional information:
Log file doesn't go far enough back, but is attached if required.
home-assistant.log

Hi. 0.75.2 (or may have been .1) and something is still corrupting my entity_registry. It happened when I added some Xiaomi sensors.
What I can't seem to see is where this breaks. I would hope the entity_register is just appended to, rather than rewriting the whole json file. Perhaps there is a syntax error in my file, which is causing it to be regenerated, but I don't see anything in the logs.

https://github.com/home-assistant/home-assistant/pull/16699 should fix this issue.


Cannot click Continue (+ a small thing)

if you can see the Continue button it is unclickable when i use this extension, i have to refresh the page and then click next to directly go to the next topic, if i disable the extension, the continue button works...
if you also see 25XP, it is in high contrast with the existing dark blue colorscheme
also see this image where the 100% is very dark as compared to the main colorscheme both the hexes are: #100B22

I'm using FloorP (Firefox), on Linux (Arch), lemme know if you want any more details, but this is happening everytime i reach the button stage (like after completing some project or quiz or lesson - and its pretty tedious to wait for the next time continue comes...)

if you can see the Continue button it is unclickable when i use this extension...

Thank you for bringing this to my attention, @daUnknownCoder !
I have submitted a new version to Firefox Addons, so you can manually check for the new version if it doesn't update automatically.
For the poor contrast text, I will fix this at my earliest opportunity.

Yep Continue works perfectly 🎉
Btw, can we get colors in this anti-human white mode boxes?

Found another occurrence (All Codebyte Examples given on resources page) of the color which is destined to destroy midnight coders:
Extension:

Official:

Webpage: https://www.codecademy.com/resources/docs/javascript/strings/split
Also, check this Course button:

so the Text color is too light or the bg is too white 😭

lol @Jamesllllllllll, ig the pricing page is also contrasted...


Sync the storage pool stats in DB with the actual stats from stats collector

Description
This PR syncs the pool stats in DB with the actual stats from stats collector if there are any discrepancies with in-memory or DB stats - both  capacity (disksizetotal) and used (disksizeused) bytes.

Types of changes

[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] New feature (non-breaking change which adds functionality)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] Enhancement (improves an existing feature and functionality)
[ ] Cleanup (Code refactoring and cleanup, that may add test cases)
[ ] build/CI

Feature/Enhancement Scale or Bug Severity
Feature/Enhancement Scale

[ ] Major
[x] Minor

Bug Severity

[ ] BLOCKER
[ ] Critical
[ ] Major
[ ] Minor
[ ] Trivial

Screenshots (if appropriate):
How Has This Been Tested?

Checked the pool stats after updating capacitybytes for the pool - DB sync-ed with the actual capacitybytes from stats collector.
How did you try to break this feature and the system with this change?

@blueorangutan package

@sureshanaparti a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

@blueorangutan package

@weizhouapache a [SL] Jenkins job has been kicked to build packages. It will be bundled with  KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

Packaging result [SF]: ✔ el7 ✔ el8 ✔ el9 ✔ debian ✔ suse15. SL-JID 8731

@blueorangutan test

[SF] Trillian test result (tid-9315)
Environment: kvm-centos7 (x2), Advanced Networking with Mgmt server 7
Total time taken: 44620 seconds
Marvin logs: https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr8686-t9315-kvm-centos7.zip
Smoke tests completed. 129 look OK, 0 have errors, 0 did not run
Only failed and skipped tests results shown below:

Test
Result
Time (s)
Test File


Add Icon for Twitter / GitHub / Linked etc.. icons on Introduction page.

Issue/Feature Description:
The links for the social media channels of SODA Foundation are present on the introduction page.
 - [SODA Website](https://sodafoundation.io/the-foundation/join/)
 - [SODA Slack](https://sodafoundation.io/slack)
 - [SODA Twitter](https://twitter.com/sodafoundation)
 - [SODA Mailing List](https://lists.sodafoundation.io)
 - [SODA LinkedIn](https://www.linkedin.com/company/sodafoundation/)

Add Icons for Twitter / GitHub / Linked etc.. icons on Introduction page.
Why this issue to fixed / feature is needed(give scenarios or use cases):
Gives a visual navigation to user rather than just textual.
How to reproduce, in case of a bug:
N/A
Other Notes /  Environment Information: (Please give the env information, log link or any useful information for this issue)
N/A

@anvithks : Please have a look.

I would love to fix this

@ikabir21 Please go ahead and raise the PR.

@ArjunGK22 Raise a PR and tag me for review.

thanks for the approval....I have created the PR kindly have a look at it sir.

Hi @anvithks @rajat-soda
I noticed that this issue regarding adding icons for social media channels to the introduction page was closed earlier. However, I'm interested in working on it and believe it could enhance the user experience by providing visual navigation.
If there were specific reasons for closing the issue initially, could you please provide some context? I'd like to understand if there were any concerns or issues that need to be addressed before reopening or working on it.
I'm eager to contribute to this project and would appreciate any guidance on how to proceed.
Thank you..
Kunal Gavhane

Hey @Kgkunal .
The issue is still open. It is the PR that was closed. There was an attempt by @ArjunGK22 .
You can give it a go and raise a PR.

Thank you for the clarification, @anvithks .
I appreciate the update that the issue remains open, and it was the related PR that was closed previously. I'll certainly take a look at the work done by @ArjunGK22 and see if I can pick up from there to contribute and raise a new PR.
If there are any specific guidelines or considerations I should keep in mind while working on this, please do let me know. I'm excited to collaborate and make a positive contribution to the project.
Thank you for your help..........

Hi @rajat-soda and @anvithks,
I'd like to kindly request your review of this PR. It enhances the Introduction page of our documentation by adding social media icons for improved user experience. Your feedback and insights would be greatly appreciated.
Thank you in advance for taking the time to review.....


Adding install script to simplify and allow automation of linux installation

@int19h can you take a look at this one?


Add python and shell CI/CD linters

Is your feature request related to a problem? Please describe.
Add automated linters for shell and python to the repo.
Describe the solution you'd like
Every time there's commit to master or any PR, run shell and Python linters.
Describe alternatives you've considered
The user and reviewer could run linters manually, but it's hard to enforce that behavior.
Additional context
We'll need to identify which linter checks to exclude, based on @BenWestgate's preference. For example, shellcheck enforces ${variable_name} instead of $variable_name which is nice but optional in many cases.

Seems like a good solution that'll make higher quality PRs.
@epiccurious I don't care if it puts {} around variable names or not, you can decide. Check the code formatting both ways and see what is easier to read, if drawn use the shellcheck defaults. I obviously typed it without curly braces to save time because I know where they are not needed.

Ok. I'll exclude the nit rule for double-quoting variables. For reference, here's the documentation for it https://www.shellcheck.net/wiki/SC2086

Ok. I'll exclude the nit rule for double-quoting variables. For reference, here's the documentation for it https://www.shellcheck.net/wiki/SC2086

I don't mind if it puts double quotes around variables either. Less custom rules is better, but less work complying with the rules is best.
When I worked on Tails, my IDE reformatted a lot of the python automatically and it was a pain to work on. Almost that same week after they saw my struggled they added a python code formatter to their linter/ci/cd/xkwjuhaishdflp! thing.

@epiccurious: Was the robot supposed to close your open issue? Explain if these are still an issue, it seems to run a shell and python linter when I make pull requests to master.


[BUG] Pylance strict mode: type of "delete_all" is partially unknown

When using pylance in strict mode a lot of static document methods have a typing issue
Type of "delete_all" is partially unknown
  Type of "delete_all" is "(session: ClientSession | None = None, bulk_writer: BulkWriter | None = None, **pymongo_kwargs: Unknown) -> Coroutine[Any, Any, DeleteResult | None]"Pylance[reportUnknownMemberType](https://github.com/microsoft/pyright/blob/main/docs/configuration.md#reportUnknownMemberType)

This is not only for delete_all but also for:

delete
insert
insert_many
...

And probably more
To Reproduce
In VSCode, set pylance type checking mode to strict
class Sample(Document):
    name: str
    age: int
    is_active: bool
    _id: Optional[PydanticObjectId] = None

Sample.delete_all() # pylance issue

Expected behavior
I would like to be able to use beanie document functions while using pylance strict type checking mode.
Is there a way to fix this apart from ignoring typing on the whole line with # type: ignore?

Hi, this problem hasn't been fixed yet. Can you reopen it?

Hi, we plan on tracking all of these remaining typing issues seen through Pylance with one issue ticket.
For updates, please see #696. Thank you


KubernetesManifestV0 support deployment to multiple namespaces

Question, Bug, or Feature?
Feature
Enter Task Name: KubernetesManifest
Environment

Server - Azure Pipelines or TFS on-premises?

If using TFS on-premises, provide the version:

If using Azure Pipelines, provide the account name, team project name, build definition name/build number:

Agent - Hosted or Private:

If using Hosted agent, provide agent queue name:

If using private agent, provide the OS of the machine running the agent and the agent version:

Issue Description
It would be great to add a support of multiple namespaces in the yaml files. F.e if namespace is not set in the task configuration, it uses the one from yaml files and default as a default namespace if it is not set in task or yaml.
Task logs
error: the namespace from the provided object "kube-system" does not match the namespace "default".
Error logs
error: the namespace from the provided object "kube-system" does not match the namespace "default".

The Azure option of K8s service connection is designed in such a way that it'll create ServiceAccount and RoleBinding objects to only allow the service account to operate on a specific namespace. The task interface is designed accordingly to align with this mapping between namespace and service connections. Do not want to support multiple namespaces within the task as a result.
To deploy the same manifests to to multiple namespaces, the recommended approach is having multiple Kubernetes Manifest task based steps in the job - where each step deploys to one of the many namespaces

Consider a helm chart such as the jetstack's cert-manager, which creates k8s objects both in a target namespace, as well as kube-system when rbac/webhooks are enabled. This means that it's impossible to deploy that chart using a combination of helm template and KubernetesManifest@0 - do you have a better solution for that case?

@shashankbarsin k8s deployments via a manifest are often quite elaborate, they involve namespaces for cert-manager like @mmisztal1980 mentioned, or API gateways or roles in the kube-system namespace
Splitting up the task and the yaml files wrt to namespace to support this use-case of azure devops task, doesn't make sense and renders the task quite unusable
You mentioned that the service connection the task uses is mapped only to a particular namespace, and that's the limitation faced here

Can we not create Service accounts and role binding to multiple namespace (ClusterRole?)
Will the cluster admin credentials not suffice for access into all namespaces?


Column preferences

Each column in the multiple spectra analysis could have the following preferences

variable name
value is the integral
value is the maximum intensity
value is the minimum intensity

This means that for each range we should calculate an object containing:

integral, min and max

@lpatiny
we can change the variable name as you suggested here so I think it's related to #666, instead of changing the column name in the table header we can do it from preferences I think it's better.

And this case you can complete: https://github.com/cheminfo/nmr-displayer/blob/master/.ncurc
Maybe you could also check the other module we fixed the version to know if it should still be the case.


Class documentation links are 404

The ResourceManagementClient and ContainerInstanceManagementClient links on this page are dead and should likely instead link to ResourceManagementClient and ContainerInstanceManagementClient.

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

ID: e732e187-7ff4-bd4f-9aad-44953af44a86
Version Independent ID: d4baa2d6-8f2f-ed76-e890-3240065f485a
Content: Azure Container Instances libraries for Python
Content Source: docs-ref-services/latest/containerinstance.md
Service: container-instances
Product: azure
Technology: azure
GitHub Login: @dlepow
Microsoft Alias: danlep

@smena, thanks for the feedback!
adding @msyyc to have a look on the doc issue.

@nickomang - Could you please take a look at the broken links on this page? I think the customer's suggestion to fix is correct. Thanks a lot!

Hi @scbedd I don't have access to update docs. Could you help to update it with right link?

The suggestion looks good to me. @scbedd , I opened a PR with the proposed changes, as well as fixes to the other API links in the article. Feel free to merge if it looks good to you, or reach out if you have any questions.

@Nickomang merged! Next deployment is tomorrow. I will close this issue once it verifies on live. Thanks muchly for the PR!

Confirming fixed on live. Closing this issue.


Replace the Slick2D code with custom code.

Code to Deprecate
All of Slick2D except for slick-utils.
The code what have to be replaced and would allow for better performance and use of newer OpenGL features or LWJGL 3

Added render system


Potential bug in trainer.py

I tried to deploy coach on my server without any changes, and it turns out that something is wrong in train.py when I tried to reproduce the reported results by running python slu_main.py --exp_name coach_lstmenc --exp_id atp_50 --bidirection --freeze_emb --tgt_dm PlayMusic --n_samples 50.

By careful debugging, I find that the loss update procedure in the function train_step is somewhat problematic. The loss forwarding of three constituents of the model are done with the same model snapshot, while their gradient computation are done separately.
Somehow the code works now with some unknown reasons, maybe because I have added several lines of comments : ). But I am still very entangled about the error prompt that happened before.

Hi,
Which PyTorch version did you use? This code is runnable under python 3.6 and PyTorch 1.4.0.
I think it is okay to separate the gradient computation (however, it could be slower in the training). It would be great if you merge the loss and do gradient computation only once.

I have tried merging them and updating at the end for once, though it is frustrating with F1 being around 55.1587, far below the reported 62.41 in Table 1. As I see it, the reported results are more Binary Slot-F1 rather than Final Slot-F1. However, in the unchanged version, the reproduced Final-F1 is around 59.7138. Therefore, I can conclude that the two implementations are different.
So, why they are different? When updated separately, the gradients we used are not the same thing.  Take the gradient of LSTM encoder w.r.t. the loss over the slotname_predictor for example. When updating for only once, the gradient is \frac {\partial {old loss}}{\partial {old LSTM}}; while updating separately, the gradient is \frac {\partial {old loss}}{\partial {new LSTM}}.

And by the way, the version of Pytorch I use is 1.5.0. And it seems that the aforementioned problem have no relation with that, because it is now running happily on my server.

Thanks for the information. The results should have some variances, and you can also try to download our models to reproduce the results.


Parse feature(crate_in_paths)

This test case has some examples of https://github.com/rust-lang/rfcs/pull/2126.

Fixed in dfc6df727476038d51bb313b9f32409a685d72dd.


[Machine Learning] Handpose Detection using MediaPipe and React.js

Proposed title of article
Your title should be descriptive of the article/tutorial. Be Specific.
Handpose Detection using MediaPipe and React.js
Proposed article introduction
We're looking for the first 2-3 paragraphs of your article that appropriately summarize what your article will be about.
We will use this writing snippet to help us assess the overall quality before approval so ensure it has been proofread, spellchecked and grammarchecked.
Handpose recognition is a deep learning technique that allows you to detect different points on your hand. These points on your hand are commonly referred to as landmarks. These landmarks consist of joints, tips, and bases of your fingers. The handpose model is provided by tensorflow.js and is able to detect 21 different distinct points in your hand. The model is a lightweight ML pipeline consisting of two models: A palm detector and a hand-skeleton finger tracking model. It predicts 21 3D hand key-points per detected hand.
This tutorial will show our readers how to build our very own Handpose detector using pre-trained model from tensorflow.js, react.js and mediapipe.
Key takeaways
What are the 3-5 most important things the reader should understand or be able to do after reading this article?
Use this area to get your ideas down on the bulk of your article or tutorial.
By the end of this tutorial, a reader will be able to:

Learn how to install and setup the pre-trained handpose model
Learn how to build the handpose application using a video camera/webcam
Learn how to detect these landmarks on the hand in real-time using the camera

References
Please list links to any published content/research that you intend to use to support/guide this article.

MediaPipe HandPose
Create a New React App

Article quality
How is your article unique? Tell us what makes your approach different from similar articles that have published on the same topic?
Is yours more in-depth? Does it cover additional topics? Do you provide handy tips or anecdotal advice?
Tensorflow.js has many pre-trained models. The handpose model is one of their latest releases. Not many articles exist on the subject. Those that are present are either too complex to understand. We need to democratize AI and the use of such amazing pre-trained models should be understood with only a few lines of code. My tutorial aims to do just that. Keeping it simple but still unique.

Great topic 🚀 , make sure it matches the following:

Adds value beyond what is in the official docs and what is covered in other blog sites.
To avoid potential article closure, it should not overlap with existing EngEd articles or incoming topic suggestions.

Please reference any relevant EngEd articles in yours and build a unique project - Approved. @smkinyan


how to calculate difference between series value and series time ?

I have a need to calculate the difference between series value and series time.
I've tried to use an expression like this
diffSeries(demo.water.line.metricId=expose,time(demo.water.line.metricId=expose))

which just produced an error:
sequence item 1: expected string or Unicode, list found

what can I do to make it work?  Thanks in advance!

the time() function returns a list of timeseries instead of a single one, diff doesn't know what to do with it. What are you trying to achieve?

@piotr1212
for example, I push a metric named abc with value 123 at time 124
echo "abc 123 124" | nc ${SERVER} ${PORT}

I want to know the difference between time and value, which should be 124 minus 123
I know that's a weird requirement, well I do need it

Thanks, clear.
Could you please post the full error, or stacktrace? Reading through the code I see that this should actually work.

@piotr1212
here the full stacktrace:

Environment:

Request Method: GET
Request URL: http://<IP_ADDRESS>:8086/render?from=15:30_20190513&until=now&format=json&maxDataPoints=10000&cacheTimeout=100&target=diffSeries(app.ba.realtime.10_23_4_172%23537776.water.line.metricId%3dexpose,time(app.ba.realtime.10_23_4_172%23537776.water.line.metricId%3dexpose))

Django Version: 1.8
Python Version: 2.7.5
Installed Applications:
('graphite.account',
 'graphite.browser',
 'graphite.composer',
 'graphite.dashboard',
 'graphite.events',
 'graphite.functions',
 'graphite.metrics',
 'graphite.render',
 'graphite.tags',
 'graphite.url_shortener',
 'graphite.whitelist',
 'django.contrib.auth',
 'django.contrib.sessions',
 'django.contrib.admin',
 'django.contrib.contenttypes',
 'django.contrib.staticfiles',
 'tagging')
Installed Middleware:
('graphite.middleware.LogExceptionsMiddleware',
 'django.middleware.common.CommonMiddleware',
 'django.middleware.gzip.GZipMiddleware',
 'django.contrib.sessions.middleware.SessionMiddleware',
 'django.contrib.auth.middleware.AuthenticationMiddleware',
 'django.contrib.messages.middleware.MessageMiddleware',
 'django.contrib.auth.middleware.SessionAuthenticationMiddleware')

Traceback:
File "/usr/lib/python2.7/site-packages/django/core/handlers/base.py" in get_response
  132.                     response = wrapped_callback(request, *callback_args, **callback_kwargs)
File "/opt/graphite-debug/webapp/graphite/render/views.py" in renderView
  118.       data.extend(evaluateTarget(requestContext, targets))
File "/opt/graphite-debug/webapp/graphite/render/evaluator.py" in evaluateTarget
  28.     result = evaluateTokens(requestContext, target)
File "/opt/graphite-debug/webapp/graphite/render/evaluator.py" in evaluateTokens
  57.     return evaluateTokens(requestContext, tokens.expression, replacements)
File "/opt/graphite-debug/webapp/graphite/render/evaluator.py" in evaluateTokens
  93.       return func(requestContext, *args, **kwargs)
File "/opt/graphite-debug/webapp/graphite/render/functions.py" in diffSeries
  542.   return aggregate(requestContext, seriesLists, 'diff')
File "/opt/graphite-debug/webapp/graphite/render/functions.py" in aggregate
  338.   name = "%sSeries(%s)" % (func, formatPathExpressions(seriesList))
File "/opt/graphite-debug/webapp/graphite/render/functions.py" in formatPathExpressions
  261.   return ','.join(pathExpressions)

Exception Type: TypeError at /render
Exception Value: sequence item 1: expected string or Unicode, list found

You can use the identity function to generate a synthetic series where the value of each point is equal to the timestamp.

@DanCech
That works like a charm. Thanks a lot!

@DanCech
That works like a charm. Thanks a lot!


Add some basic examples for WPF application

Hi! It will be great to add some examples for WPF application

@dora-g unfortunately, puppeteer is only about browser automation. It won't automate any other kind of app.

@dora-g unfortunately, puppeteer is only about browser automation. It won't automate any other kind of app.

I see, thanks for response.


Feature Request: Show debugging information for SQL queries

A very useful feature Kentico 13 had was SQL debugging - the ability to review a page load and see what specific SQL queries were run, and how long they took (including highlighting long-running queries).
This helped with determining if our caching was working (if the queries don't run and the data still shows up, the cache succeeded) - it also helped with performance-tuning queries and detecting when devs didn't apply proper filters/paging to queries (by viewing the result count).
Miniprofiler appears to be able to do that, judging by https://miniprofiler.com/dotnet/ and  https://docs.servicestack.net/built-in-profiling#sql-profiling (The latter is a third-party service and is using the MVC (not core) version of Miniprofiler, but both links show screenshots of Miniprofiler being used in the way I'm imagining)
Currently I don't believe the Kentico Miniprofiler implementation (Or XbyK internally) gives any insight into SQL queries at all, so anything would be an improvement.

Thank you for highlighting this request. We will review it internally and keep you informed of any developments.

@ChristopherBass are you sure you tried out this integration?
Here's what it looks like when enabled in the Kentico Community Portal

If you click on the "sql" column for the one of the results, it will show all the SQL queries run within that part of the call stack.

I also reproduced this in 29.1.3 on Dancing Goat (updated to .net 8)
Program.zip

Comparing projects with Sean Wright, I think this is just a documentation fix.
The documentation just says to put the 'AddKenticoMiniProfiler' call in your program.cs
and gives this as guidance:
var env = builder.Environment;

if (env.IsDevelopment())
{
    builder.Services.AddKenticoMiniProfiler();
}

// Other services ...

But if you put it before the builder.Services.AddKentico(), then you get the non-SQL version. It's possible AddKentico is touching the sql in a way that's kicking the profiler off of the SqlConnection, but in either case, as long as you add it after AddKentico() it seems to work as designed.

I think this was resolved by #11. We can open this issue back up if there are any additional problems related to the dependency registration/docs.


Request for code review

Project link
We have three members in our team.
Tasks for team member 1 is assigned to KDlamini.
Tasks for team member 2 is assigned to mbdesigns1989.
Tasks for team member 3 is assigned to hamid-murambiwa.

Hi @KDlamini , @mbdesigns1989 , @hamid-murambiwa
Your project is complete!
Everything is correct according to requirements.
Good luck! 🎉
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.


feat: determine available "environments" via directory names

Would be great to be able to do something like this:
helmfile --environments-dir /path/to/environments -e dog myhelmfile.yaml apply

where /path/to/environments has
dog/
   1.yaml
   2.yaml
cat/
   1.yaml
   2.yaml

And then I can just drop in new environments and not have to maintain some index of them in an environments: element in the helmfile itself.

Interesting idea! I agree we'd want something like this.
Let's split this into two different problems to discuss tho:

A convention to treat each sub-directory a helmfile environment
How to specify the parent directory for 1. - the proposal includes a flag --environments-dir. do we need it in config files as well?

Regarding item 2, do you mean if a command line argument like --environments-dir was passed, would helmfile also merge traditionally declared environments like:
environments:
   hog:
      ...

I would say yes, if both are present, the total set of environments is the union of the two. However I don't think they would need to be merged (for example having both a environment directory named "dog" and a separately defined "environments.dog" in a helmfile probobly would not need to be supported.

@bitsofinfo Good point. I don't think the merge functionality isn't a must-have, and yeah, the implementation would be much simpler if it isn't need to be supported!

agreed, my use case doesn't need that. I really just want to be able to drop new "environment" directories with arbitrary values files within them under a --environments-dir and have them immediately available. For me an "environment" would be either/or. (i.e. living under --environments-dir OR traditionally declared via environments.[envname].... but not across both.


[Enhancement] Use Streaming Lists when available

Description
This PR Addresses an issue of potential runaway usage of memory due to chucking of reported statuses while watching for Kubernetes resources. Kubernetes 1.27 introduces a WatchList feature still at the alpha stage that allows one to get statuses of objects being watched as a streaming list as opposed to current implementations that uses a list request first before a watch request. This is done by setting sendInitialEvents=true param in a watch request.
This PR fixes #221
Notes for Reviewers
This is still WIP and is allow continuous conversation to allow seamless integration with existing infrastructure
Signed commits

[ x] Yes, I signed my commits.

Using streaming lists boils down to adding the flag sendInitialEvents=true  param in a watch request without a List request for the desired Kubernetes object. DynamicInformers also use ListWatch functionality under the hood, and may eventually come to support use of streaming lists.

@leecalcote

@KiptoonKipkurui, great presentation today, sir.

@MUzairS15 this isn't ready for a master merge, it will break functionality


Insertion Sort of Algorithms in Javascript and C++

Proposed title of article
Insertion Sort of Algorithms in Javascript and C++
Proposed article introduction

Insertion sort is a sorting algorithm that creates a sorted array of items from an unsorted array, one item at a time.
In this article, we will see how the algorithm works and how to apply it in our code. Note if the current number from the unsorted array is less than any number it is being compared with in the sorted array, the one in the sorted array will move to the right and create an empty space on its previous position to insert the current number.
This will continue until the current number gets to a position where it is greater than the number it is being compared with. At this point, you insert the current number into the last created space.

Key takeaways

By the end of this article, the following is expected;

How to Use Insertion Sort
Insertion Sort in Java
Insertion Sort in C++

Article quality
This article will be on point, simple to understand, well formatted and I'm sure users will be able to learn how to sort Algorithm in both Javascript and C++ programming languages. The code snippets in this article will be simple to understand and implement. I hope we will have a good ride all along.
Reference
N/A

Good afternoon and thank you for submitting your topic to the EngEd program.
After some careful consideration it struck us that this topic may be a bit over saturated throughout other blog sites and official documentations. We typically refrain from publishing content that is covered widely on the net or other blogs. We're more interested in original, practitioner-focused content that takes a deeper dive into programming-centric concepts.
But in order to approve the topic it has to serve value to the larger developer community at large. Please feel free to suggest an alternate topic to explore. 🚀
An option and a great way to write this as an in-depth article and make it more add value to the greater developer community at large would be to walk the reader through the USE of methods and functions by building a unique, different, useful project.
https://www.google.com/search?q=Insertion+Sort+of+Algorithms+in+Javascript+and+C%2B%2B&rlz=1C1CHBF_enUS891US891&sourceid=chrome&ie=UTF-8