Can not preview note in vscode.dev via tunnel

Please select the area your bug applies to. (Multiple selections are Possible. You can leave blank if you're not sure.)

[ ] Workspace.  VSCode workspace, vaults, Intellisense/autocomplete, Dendron settings
[ ] Lookup.  Dendron's Lookup Command
[x] Views.  Dendron Preview, Tree View, Side Panels in the UI
[ ] Schema.  Dendron Schemas
[ ] Pod.  Data import from / export to Dendron
[ ] Publish.  External Site Publish
[ ] Markdown.  Markdown features, such as syntax support and features

Describe the bug
When using vscode.dev via a tunnel, notes can not be previewed.
To Reproduce
Steps to reproduce the behavior:

Run code tunnel
Authenticate your tunnel
Open your tunnel in vscode.dev
Open Network via F12
Try to preview a note

Expected behavior
A clear and concise description of what you expected to happen.
The expected behavior is to see a preview of the note.
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information)
Version: 1.84.2
Commit: 1a5daa3a0231a0fbba4f14db7ec463cf99d7768e
Datum: 2023-11-09T10:53:00.082Z (Vor 2 Woche(n))
Browser: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:109.0) Gecko/20100101 Firefox/115.0
dendron v0.124.0
Dendron Paste Image v1.1.1
Dendron Markdown Shortcuts v0.12.1
Dendron Snippet Maker v0.1.6
Additional context
Add any other context about the problem here.
The Plugin Dendron Markdown Preview Enhanced does not work at all.
Dendron Log file
Please attach the output of > Dendron:Dev: Open Logs here
{"level":30,"time":1700821351898,"pid":18593,"hostname":"ctvscode","name":"dendron","ctx":"WindowWatcher:onDidChangeVisibleTextEditors","editorPaths":[]}
{"level":30,"time":1700821351899,"pid":18593,"hostname":"ctvscode","name":"dendron","ctx":"refreshBacklinksChangeActiveTextEditor"}
{"level":30,"time":1700821351918,"pid":18593,"hostname":"ctvscode","name":"dendron","ctx":"WindowWatcher:onDidChangeVisibleTextEditors","editorPaths":["/root/projects/athena/notes/test.md"]}
{"level":30,"time":1700821351920,"pid":18593,"hostname":"ctvscode","name":"dendron","ctx":"refreshBacklinksChangeActiveTextEditor"}
{"level":30,"time":1700821351980,"pid":18593,"hostname":"ctvscode","name":"dendron","ctx":"WindowWatcher:onDidChangeActiveTextEditor","editor":"/root/projects/athena/notes/test.md"}
{"level":30,"time":1700821352078,"pid":18593,"hostname":"ctvscode","name":"dendron","ctx":"updateDecorations","msg":"decorating...","payload":{"decorationsLength":2,"diagnosticsLength":0}}
{"level":30,"time":1700821353477,"pid":18593,"hostname":"ctvscode","name":"dendron","ctx":"dendron.togglePreview:run","msg":"pre-execute"}
{"level":30,"time":1700821353642,"pid":18593,"hostname":"ctvscode","name":"dendron","ctx":"dendron.togglePreview:run","msg":"post-execute"}
{"level":30,"time":1700821356774,"pid":18593,"hostname":"ctvscode","name":"dendron","ctx":"WindowWatcher:onDidChangeVisibleTextEditors","editorPaths":[]}
{"level":30,"time":1700821356776,"pid":18593,"hostname":"ctvscode","name":"dendron","ctx":"refreshBacklinksChangeActiveTextEditor"}
{"level":30,"time":1700821356778,"pid":18593,"hostname":"ctvscode","name":"dendron","ctx":"WindowWatcher:onDidChangeVisibleTextEditors","editorPaths":["/root/.vscode-server/data/logs/20231124T090215/exthost10/dendron.dendron/dendron.log"]}
{"level":30,"time":1700821356780,"pid":18593,"hostname":"ctvscode","name":"dendron","ctx":"refreshBacklinksChangeActiveTextEditor"}

I have a similar issue that is probably related.  When installing the dendron extension in a github codespace and attempting to go through the tutorial, the rendered markdown preview fail to display and instead shows this error:
{"fileName":"https://vscode-remote+codespaces-002bturbo-002dspace-002dwinner-002d4qvjp5qxrxcpp.vscode-resource.vscode-cdn.net/home/codespace/.vscode-remote/extensions/dendron.dendron-0.124.0/dist/static/js/index.bundle.js","lineNumber":2,"columnNumber":1489453,"message":"Network Error","config":{}}


Represent gaps in sequence annotations with bare backbone

When there are two SequenceAnnotations where one ends and the next starts that are not contiguous, but they are found to be ordered with none in-between then a glyph should be added with a bare backbone.  Clicking on it should allow you to edit the sequence at this location only.  Note that no Component or SequenceAnnotation need be generated for this region of the DNA.

Not doing this one.  Closing.


Rebuild for ICU70 (Qt-main 5.15.3)

Build status:

[x] linux-64. https://anaconda.org/mark.harfouche/qt-webengine/ linux_64_log.zip
[x] linux-aarch64. User hmaarrfk. conda-forge aarch64 server. In progress
linux_aarch_64-log.zip
[x] windows -- uploaded above win_64_log.zip
[x] osx-64 log: https://www.dropbox.com/s/qrs9ski61fai63o/qt-webengine-osx-64-py2.log?dl=0
[x] osx-arm64 log: https://www.dropbox.com/s/s4ghi80ls1m9ouj/qt-webengine-osx-arm64-py2.log?dl=0

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin please rerender

Looks like we can get rid of the 0004 patch for arm64 because a similar patch was upstreamed: https://github.com/qt/qtwebengine-chromium/commit/019a6b9282e7e162ca21832a4724a7e27dfea0b1

Do I need to pull that patch in?

No - because we specify a git_rev to that branch it is automatically being pulled in.

if people could help build the mac builds that would be very helpful.
I can try windows later.

My build has been stuck at
( test -e Makefile.gn_run || $PREFIX/bin/qmake -o Makefile.gn_run $SRC_DIR/src/core/gn_run.pro QMAKE_LIBDIR=$PREFIX/lib 'QMAKE_LFLAGS+=-Wl,-rpath,$PREFIX/lib -Wl,-rpath-link,$PREFIX/lib -L$PREFIX/lib' INCLUDEPATH+=$PREFIX/include PKG_CONFIG_EXECUTABLE=$BUILD_PREFIX/bin/pkg-config ) && make -f Makefile.gn_run
Project MESSAGE: Running: /home/conda/feedstock_root/build_artifacts/qt-webengine_1651027265996/work/qtwebengine-build/src/3rdparty/gn/out/Release/gn gen /home/conda/feedstock_root/build_artifacts/qt-webengine_1651027265996/work/qtwebengine-build/src/core/release --script-executable=/usr/bin/python2 --args='use_qt=true init_stack_vars=false is_component_build=false is_shared=true enable_debugallocation=false enable_media_remoting=false enable_message_center=false enable_nacl=false enable_remoting=false enable_reporting=false enable_resource_allowlist_generation=false enable_swiftshader=false enable_swiftshader_vulkan=false angle_enable_swiftshader=false enable_web_speech=false enable_widevine=true forbid_non_component_debug_builds=false has_native_accessibility=false safe_browsing_mode=0 skia_use_dawn=false toolkit_views=false treat_warnings_as_errors=false use_allocator_shim=false use_allocator="none" use_custom_libcxx=false chrome_pgo_phase=0 enable_hangout_services_extension=true optimize_webui=false enable_js_type_check=false v8_use_external_startup_data=false strip_absolute_paths_from_debug_symbols=false use_jumbo_build=true jumbo_file_merge_limit=8 jumbo_build_excluded=["browser"] enable_precompiled_headers=true is_official_build=true is_debug=false symbol_level=0 blink_symbol_level=0 remove_v8base_debug_symbols=true optimize_for_size=true use_cups=false use_gio=false use_gnome_keyring=false use_udev=true use_bundled_fontconfig=false use_sysroot=false enable_session_service=false is_cfi=false use_ozone=true use_x11=false ozone_auto_platforms=false ozone_platform_headless=false ozone_platform_external=true ozone_platform="qt" ozone_extra_path="/home/conda/feedstock_root/build_artifacts/qt-webengine_1651027265996/work/src/core/ozone/ozone_extra.gni" use_gold=false use_lld=false is_clang=false custom_toolchain="/home/conda/feedstock_root/build_artifacts/qt-webengine_1651027265996/work/qtwebengine-build/src/toolchain:target" host_toolchain="/home/conda/feedstock_root/build_artifacts/qt-webengine_1651027265996/work/qtwebengine-build/src/toolchain:host" host_cpu="arm64" pkg_config="pkg-config" host_pkg_config="/home/conda/feedstock_root/build_artifacts/qt-webengine_1651027265996/_build_env/bin/pkg-config" pdfium_use_system_zlib=true use_system_libpng=true pdfium_use_system_libpng=true use_system_libjpeg=false use_system_freetype=true use_system_harfbuzz=false use_glib=false enable_basic_printing=true enable_print_preview=true enable_pdf=true enable_plugins=true enable_spellcheck=true enable_webrtc=true proprietary_codecs=false enable_extensions=true use_kerberos=false use_pulseaudio=false use_alsa=true ozone_platform_x11=true use_xkbcommon=true use_xscrnsaver=true rtc_use_x11=true use_system_libevent=true use_system_libwebp=true use_system_opus=true qtwebengine_target="/home/conda/feedstock_root/build_artifacts/qt-webengine_1651027265996/work/qtwebengine-build/src/core/release:QtWebEngineCore"' --root=/home/conda/feedstock_root/build_artifacts/qt-webengine_1651027265996/work/src/3rdparty/chromium

for the last 12 hours. Not sure if it is a hot spot. Looking at htop i see the job taking up CPU every once in a while.

I'm running with qemu emulation on an x86-64 machine

@conda-forge/qt-webengine @conda-forge/qt-main is anybody able to build OSX? I got the rest of the builds.

I am happy to build osx-arm64 on my M1. If I can build osx-64 without issues on my M1 machine, too, I can do that as well.

Hmm using build-locally.py for osx-arm64 I am running into this problem:
ld: library not found for -lc++
clang-13: error: linker command failed with exit code 1 (use -v to see invocation)

Simply doing conda mambabuild . -m ../.ci_support/osx_arm64_.yaml leads to an empty package (same for osx_64_.yaml by the way) - I think it is because "QtWebEngineCore module will not be built. Python version 2 (2.7.5 or later) is required to build QtWebEngine." (there used to be a hack with some python2 stuff but it does not seem to be the case anymore here).

On my Intel Mac, the SDK is not found:
WARNING: QtWebEngine requires a macOS SDK version of 10.13 or newer. Current version is .
WARNING: QtPdf requires a macOS SDK version of 10.13 or newer. Current version is .
WARNING: QtWebEngine will not be built.
WARNING: QtPdf will not be built.

Not sure what's going on. Any tips?

can you try using python build-locally.py on osx x86?

So looking at the Azure build logs, it is clear that /usr/bin/python2 is being used - but that's not installed on my machine. I think we need this hack again:
https://github.com/conda-forge/qt-feedstock/blob/a01eb4467f57ccdcb468ff36f50d4ac9c4ca301d/recipe/build.sh#L67-L68

I have no idea - all I can tell is that the build is now running fine when reinstantiating the hack.

Are you able to push to my branch?

I successfully built osx-64 with this change (it would be great if you could push that to your branch @hmaarrfk). Currently building osx-arm64.
diff --git a/recipe/build.sh b/recipe/build.sh
index aef8bc6..029be30 100644
--- a/recipe/build.sh
+++ b/recipe/build.sh
@@ -32,6 +32,10 @@ pushd qtwebengine-build
 USED_BUILD_PREFIX=${BUILD_PREFIX:-${PREFIX}}
 echo USED_BUILD_PREFIX=${BUILD_PREFIX}

+# qtwebengine needs python 2
+mamba create --yes --prefix "${SRC_DIR}/python2_hack" --channel conda-forge --no-deps python=2
+export PATH=${SRC_DIR}/python2_hack/bin:${PATH}
+
 if [[ $(uname) == "Linux" ]]; then
     ln -s ${GXX} g++ || true
     ln -s ${GCC} gcc || true

Both packages are now in my channel: https://anaconda.org/tobiasrobotics/qt-webengine
osx-64 log: https://www.dropbox.com/s/qrs9ski61fai63o/qt-webengine-osx-64-py2.log?dl=0
osx-arm64 log: https://www.dropbox.com/s/s4ghi80ls1m9ouj/qt-webengine-osx-arm64-py2.log?dl=0

how does the patch work for osx-arm64 where is python2 downloaded from?

ah ok you cross compiled osx-arm64

Thank you ^_^

Looks like we are having a problem @hmaarrfk: the osx-arm64 builds "succeed" in CI and the package is uploaded although none of the qt-webengine stuff was built (see also #6 where this can be observed too).
I think we 1) need to remove (mark as broken) the spurious package, and 2) need to have a look at the build script to see why it not fails despite an error being thrown.

Ah damn it. I just realised the same is the case with my manually built package :(.

yea.... I swear I thought I looked at the package size. I can make a PR to pull it.
I think I want to keep the other packages though. They are rather hard to build, and I think they are good.

Can you make a PR to mark:
osx-arm64/qt-webengine-5.15.4-hcae7d30_2.tar.bz2
osx-arm64/qt-webengine-5.15.4-h214f810_2.tar.bz2

as broken
(Being part of core has made this harder since I have to do the whole local branching dance)

It seems like with this additional line in the build.sh I can get things to compile. Does that look okay?
cp ${SRC_DIR}/python2_hack/lib/python2.7/_sysconfigdata_x86_64_apple_darwin13_4_0.py ${SRC_DIR}/python2_hack/lib/python2.7/_sysconfigdata_arm64_apple_darwin20_0_0.py

Do you have an M1? Can you test the qt-webengine somehow?

sorry, the package you "compiled" I know it might take a day to finish compilation.

So I am compiling on my M1 within an osx-64 environment. Hence compilation is quick and has already finished.
The new package is now on https://anaconda.org/tobiasrobotics/qt-webengine - however, the hash has not changed compared to the previous (broken) version. Shall I bump the build number and built again? Then I can also provide logs.
I have tested the package and using it I can successfully compile the test project in the test folder.
I also suggest adding some basic file check tests to check if lib/libQt5WebEngine${SHLIB_EXT} etc. exist, so an empty package leads to an error even if cross-compiling where the tests are not run.

I think bumping the build number is appropriate. The bot opened a new branch. maybe you can build off that? I'm not sure if we do test.
Yo ucan try playing with:
https://conda-forge.org/docs/maintainer/conda_forge_yml.html?highlight=native#test

you can bump the build number, I just don't think we need to rebuild the others.


adjust changelog layout and update contributing details.

closes #1029
Signed-off-by: Wayne Witzel III<EMAIL_ADDRESS>

close/reopen to kick signoff check

@nrb @carlisia @ncdc pls review - let's get this in soon so we can start including changelog entries with each PR

one other note - we'll want to make sure we have release notes under Development release for any PRs that have gone in since v0.10.  @wwitzel3 can you do that in a separate PR once this has gone in?

@skriss Yeah, I can do a PR after this lands for our current development features.

LGTM, ready to merge


Arm TTK : URIs Should Be Properly Constructed

Here, is the variable in use
`"variables" : {
"LogAnaltyicsUri":"[replace(environment().portal, 'https://portal', concat('https://', toLower(parameters('WorkspaceId')), '.ods.opinsights'))]"
}`
While running arm-ttk tests locally, I noticed an error
 [-] URIs Should Be Properly Constructed (17 ms)                                                                         Function 'concat' found within 'LogAnaltyicsUri' Line: 95, Column: 6                                                Function 'concat' found within 'LogAnaltyicsUri' Line: 101, Column: 6
But looking into the branch, lots of other users have contributed with the above format itself, I dont know why this fails.
I tried using a different approach,
"LogAnaltyicsUri": "[format('https://%s.ods.opinsights.azure.com', parameters('AzureSentinelWorkspaceId'))]",
This too led to the following error,
Function 'format' found within 'LogAnaltyicsUri' Line: 95, Column: 6
I am creating this as an issue based on my initial question over here https://learn.microsoft.com/en-us/answers/questions/1660160/arm-ttk-validation-help?comment=question#newest-question-comment

@ashwinvenkatesha, the problem is manual construction of variables that have 'url' or 'uri' contained in the name must have been formed used uri() which would not work for your use case as you create the URI yourself; one workaround for you simply remove uri from your variable name.
A real problem here that does need to be solved is the following Bicep:
resource storage 'Microsoft.Storage/storageAccounts@2022-09-01' existing = {
  name: storageName
}

resource vm 'Microsoft.Compute/virtualMachines@2024-07-01' = {
  ...
  diagnosticsProfile: {
    bootDiagnostics: {
      enabled: true
      storageUri: storage.properties.primaryEndpoints.blob
    }
  }
}

Results in the following ARM output for the virtual machine resource:
{
   ...
   "diagnosticsProfile": {
     "bootDiagnostics": {
       "enabled": true,
       "storageUri": "[reference(resourceId('Microsoft.Storage/storageAccounts', format('dev{0}', uniqueString(resourceGroup().id, 'dev'))), '2023-01-01').primaryEndpoints.blob]"
     }
   }
}

Though primaryEndpoints.blob is a URI, it is converted by bicep into something TTK deems unsafe. The problem is uri() is used to construction URI's with path components and so I am unable to see how to fix this problem.


USHIFT-1737: fix: add wait for microshift serice to make sure health check succeeds

Seems like we might have a timing problem between when the teardown of kustomize happens and when we validate microshift is up

/retest-required

/retest-required

/lgtm

/cherry-pick release-4.14


Display compiler versions (and, optionally, to switch between them)

Observed behaviour
At the moment, one has to do a set of heuristics to determine which compiler version of FunC or Tact the project is using.
For example, to check if the compiler supports Tact v1.2.0, you may try to use += operator. And if that doesn't work, the version must be lower than 1.2.0.
Repeat this process to find the actual version used at the moment (or look into Nujan source code to find out faster, though still a bit impractical :)
Expected behavior or possible solutions

Display the currently used compiler version — it may be hidden under the cog icon of settings on the lower left.
Optionally, make it possible to switch between a pre-defined set of FunC/Tact versions.

@rahulyadav-57 We should definitely have the functionality to switch between Tact versions: both released ones (should support at least two latest minor releases) and the version from GitHub.

Absolutely, adding the ability to switch between Tact versions would provide great flexibility.


Memory leak / no garbage collection (?)

It seems the app doesn't deallocate memory after deleting datasets. Adding, deleting and re-adding data keeps making the allocated memory number climb up.
In the image below,

Snapshot 1 (529 MB) is when two datasets are loaded
Snapshot 2 (527 MB) is when one of the datasets is deleted
Snapshot 3 (678 MB) is when the dataset is re-loaded

I've identified two sources of leaks:

Scene graph leaks (should be fixed in https://github.com/Kitware/vtk-js/pull/2613)
PiecewiseGaussianWidget touch event handler leaks


API Gateway Infrastructure

Adding the necessary AWS infrastructure to support an API via API Gateway. This includes a helper Python file to enrich a provided OpenAPI spec for upload.

We might need to test this more or break it down so that the Python script is tackled separately. It's raising havoc during testing.

I think you are right and it would be better to tackle the Python piece separately since that is going to require some more thought and changes than I originally expected. I will work on getting the PR updated and re-request review when ready


add Fixed Navbar and Scroll to top

web we reach the bottom of the website we can add a bottom by which we can reach top of the website.
please assign this issue to me under hacktomberfest 2022

Keep it very small and responsive for all mobile screen also place it above chat bot so that both doesn't collide

can you please assign this to me under hactoberfest

@pranjay-poddar hey I tried but was not able to add you can assign this to any other person,
thank you.

Can i try to fix this issue this till this no had fixed it

Can I fix this issue , assign to me


Support for Photon OS 4.0 GA

The current repository doesn't appear to support Photon OS 4.0.
You might be able to leverage my new Packer template (https://github.com/timothybrush/packer-templates) under photon4 that uses Photon OS 4.0 GA to build out Vagrant boxes for each of the 4 different installation types (developer, minimal, ostree host, real time).  However, it currently only supports the vmware_desktop provider since I don't use or have Virtualbox installed.
You might also find my Vagrant Boxes useful.
Notes:

Photon OS packages and OSTree show errors when upgrading to the latest versions.
OSTree Host installs but don't run rpm-ostree update to upgrade it.  The system will fail to restart.  This is an issue that the Photon OS team should investigate.

Hi @timothybrush,
i tried on rpm-ostree update command on 4.0 Ostree Host VM. It works without any issue.  Tried with 4.0 GA ISO
root@photon-6c5d42ddb1f2 [ ~ ]# rpm-ostree update
⠠ Receiving objects: 99% (2950/2952) 3.9 MB/s 120.0 MB
Receiving objects: 99% (2950/2952) 3.9 MB/s 120.0 MB... done
Staging deployment... done
Upgraded:
 audit 2.8.5-6.ph4 -> 2.8.5-7.ph4
  cpio 2.13-3.ph4 -> 2.13-4.ph4
 curl 7.74.0-1.ph4 -> 7.75.0-1.ph4
 curl-libs 7.74.0-1.ph4 -> 7.75.0-1.ph4
 glib 2.66.1-1.ph4 -> 2.68.0-1.ph4
 glib-networking 2.66.0-1.ph4 -> 2.66.0-2.ph4
 glibc 2.32-2.ph4 -> 2.32-5.ph4
 glibc-iconv 2.32-2.ph4 -> 2.32-5.ph4
 gnutls 3.6.15-3.ph4 -> 3.6.15-4.ph4
 gobject-introspection 1.66.0-3.ph4 -> 1.66.0-4.ph4
 grub2 2.04-2.ph4 -> 2.06~rc1-1.ph4
 grub2-efi 2.04-2.ph4 -> 2.06~rc1-1.ph4
 grub2-efi-image 2.04-2.ph4 -> 2.06~rc1-1.ph4
 grub2-pc 2.04-2.ph4 -> 2.06~rc1-1.ph4
 libxslt 1.1.34-2.ph4 -> 1.1.34-3.ph4
 linux 5.10.4-16.ph4 -> 5.10.25-3.ph4
 lua 5.3.5-1.ph4 -> 5.3.6-1.ph4
 mozjs 78.3.1-2.ph4 -> 78.3.1-3.ph4
 nettle 3.6-1.ph4 -> 3.7.2-1.ph4
 openldap 2.4.53-3.ph4 -> 2.4.57-1.ph4
 openssl 1.1.1i-2.ph4 -> 1.1.1k-1.ph4
 python3 3.9.1-2.ph4 -> 3.9.1-4.ph4
 python3-gobject-introspection 1.66.0-3.ph4 -> 1.66.0-4.ph4
 python3-libs 3.9.1-2.ph4 -> 3.9.1-4.ph4
 python3-setuptools 3.9.1-2.ph4 -> 3.9.1-4.ph4
 python3-xml 3.9.1-2.ph4 -> 3.9.1-4.ph4
 rpm <IP_ADDRESS>-1.ph4 -> <IP_ADDRESS>-2.ph4
 rpm-libs <IP_ADDRESS>-1.ph4 -> <IP_ADDRESS>-2.ph4
 systemd 247.3-1.ph4 -> 247.6-1.ph4
 systemd-libs 247.3-1.ph4 -> 247.6-1.ph4
 systemd-pam 247.3-1.ph4 -> 247.6-1.ph4
 systemd-rpm-macros 247.3-1.ph4 -> 247.6-1.ph4
 systemd-udev 247.3-1.ph4 -> 247.6-1.ph4
Run "systemctl reboot" to start a reboot
root@photon-6c5d42ddb1f2 [ ~ ]# systemctl reboot
After reboot:
root@photon-6c5d42ddb1f2 [ ~ ]# uname -a
Linux photon-6c5d42ddb1f2 5.10.25-3.ph4 #1-photon SMP Thu Apr 22 03:01:20 UTC 2021 x86_64 GNU/Linux
Could you please point out the exact error which you have faced along with error logs if possible?
Also , it would be great if you could share the steps which you have followed to reproduce the issue if there is any.
Thanks,
Ankit

I found the issue.  It was with my template and has not been fixed.  Apologies for any inconvenience.


move extensions and some plugin tasks to java

Only ApolloPlugin is left. This will be in a separate PR, because it's a bigger change.
I'll move the tests to use Spec for Kotlin, otherwise the test methods names would be messy. (also in a separate PR)

oh TIL. I'll update it then.


GET /appointments endpoint

Description

[x] getAll appointmentsModel function
[x] GET /appointments route

Type

[ ] This is a bugfix.
[x] This is a new feature.
[x]  This is part of a feature.
Organization
[ ]  Have you linked this pull request to a Trello card?
[ ]  Have you named the branch in the format of feature/describe_feature_or_change OR bug/describe_bug_this_is_fixing
Review
[ ]  Is this a work in progress?
[x] Is this ready to be reviewed?
[ ]  Have you added two reviewers to this pull request?


chore: make Create and Update Event screens more similar to EventBrite

Fixes #1251
Checklist:

[x] I have checked for PMD and check-style issues
[x] I have read the Contribution & Best practices Guide and my PR follows them.
[x] My branch is up-to-date with the Upstream development branch.

Changes: Adds Organizer details and event description to the top, hides latitude and longitude fields, some style and minor changes
Screenshots for the change:

@Masquerade0097 @srv-twry Please review


🖌 Workorder : (Sidesheet) -  Gap in Cutoff compared to PCS

Again, cant seem to find any diff. Cutoff contains all ref from procosys with current = N.


Add warning for plugin configs with unknown fields

Adds a warning when an unknown field is found in the plugin configuration. Uses the interface for go-toml v2.

/retest


support rails 4

Everything seems to work fine in my tests. The test suite passes on my local machine, though we'll see what Travis says =)

:+1:  Please merge this


KsponSpeech recipe sample Colab notebook does not work.

This issue is defferent from #1402.
I am trying to run this Collab notebook for using the pre-training model, but I get a size mismatch error.
(https://colab.research.google.com/drive/10N98aGoeLGfh6Hu6xOCH5BbjVTVYgCyB?usp=sharing#scrollTo=ND6pglfXnnER)
In original code, it installs speechbrain version 0.5.10. with below command.
!pip install git+https://github.com/speechbrain/speechbrain.git@c762107
But this version can not download pretrained model files.
After installing sb version 0.5.13 with below command, it downloaded model files successfully.
!pip install git+https://github.com/speechbrain/speechbrain.git
But size mismatch errors show and I can not infer with pretrained model.
I think model files are defferent from hyperparams.yaml.
Parts of error messages :
.....
size mismatch for 1.convblock_0.convs.norm_0.norm.weight: copying a param with shape torch.Size([64]) from checkpoint, the shape in current model is torch.Size([40, 64]).
size mismatch for 1.convblock_0.convs.norm_0.norm.bias: copying a param with shape torch.Size([64]) from checkpoint, the shape in current model is torch.Size([40, 64]).
size mismatch for 1.convblock_1.convs.norm_0.norm.weight: copying a param with shape torch.Size([32]) from checkpoint, the shape in current model is torch.Size([20, 32]).
size mismatch for 1.convblock_1.convs.norm_0.norm.bias: copying a param with shape torch.Size([32]) from checkpoint, the shape in current model is torch.Size([20, 32]).
size mismatch for 2.encoder.layers.0.convolution_module.after_conv.2.weight: copying a param with shape torch.Size([256, 256, 1]) from checkpoint, the shape in current model is torch.Size([256, 256]).
size mismatch for 2.encoder.layers.1.convolution_module.after_conv.2.weight: copying a param with shape torch.Size([256, 256, 1]) from checkpoint, the shape in current model is torch.Size([256, 256]).
size mismatch for 2.encoder.layers.2.convolution_module.after_conv.2.weight: copying a param with shape torch.Size([256, 256, 1]) from checkpoint, the shape in current model is torch.Size([256, 256]).
size mismatch for 2.encoder.layers.3.convolution_module.after_conv.2.weight: copying a param with shape torch.Size([256, 256, 1]) from checkpoint, the shape in current model is torch.Size([256, 256]).
.....

Hi @starcell it's reproducible (and I don't know how to handle this propely).
I get the same error logs in another task I am working on. Let me reformat it in a way that helps our discussion
RuntimeError: Error(s) in loading state_dict for ModuleList:
	size mismatch for 1.convblock_0.convs.norm_0.norm.weight: copying a param with shape torch.Size([64]) from checkpoint, the shape in current model is torch.Size([40, 64]).

as:   1.convblock_0.convs.norm_0.norm.weight: torch.Size([64]) from checkpoint => current model is torch.Size([40, 64])

	1.convblock_0.convs.norm_0.norm.bias: torch.Size([64]) => torch.Size([40, 64]).
	1.convblock_1.convs.norm_0.norm.weight: torch.Size([32]) => torch.Size([20, 32]).
	1.convblock_1.convs.norm_0.norm.bias: torch.Size([32]) => torch.Size([20, 32]).
	2.encoder.layers.0.convolution_module.after_conv.2.weight: torch.Size([256, 256, 1]) => torch.Size([256, 256]).
	2.encoder.layers.1.convolution_module.after_conv.2.weight: torch.Size([256, 256, 1]) => torch.Size([256, 256]).
	2.encoder.layers.2.convolution_module.after_conv.2.weight: torch.Size([256, 256, 1]) => torch.Size([256, 256]).
	2.encoder.layers.3.convolution_module.after_conv.2.weight: torch.Size([256, 256, 1]) => torch.Size([256, 256]).
	2.encoder.layers.4.convolution_module.after_conv.2.weight: torch.Size([256, 256, 1]) => torch.Size([256, 256]).
	2.encoder.layers.5.convolution_module.after_conv.2.weight: torch.Size([256, 256, 1]) => torch.Size([256, 256]).
	2.encoder.layers.6.convolution_module.after_conv.2.weight: torch.Size([256, 256, 1]) => torch.Size([256, 256]).
	2.encoder.layers.7.convolution_module.after_conv.2.weight: torch.Size([256, 256, 1]) => torch.Size([256, 256]).
	2.encoder.layers.8.convolution_module.after_conv.2.weight: torch.Size([256, 256, 1]) => torch.Size([256, 256]).
	2.encoder.layers.9.convolution_module.after_conv.2.weight: torch.Size([256, 256, 1]) => torch.Size([256, 256]).
	2.encoder.layers.10.convolution_module.after_conv.2.weight: torch.Size([256, 256, 1]) => torch.Size([256, 256]).
	2.encoder.layers.11.convolution_module.after_conv.2.weight: torch.Size([256, 256, 1]) => torch.Size([256, 256]).

The module list is quitte clear:
https://github.com/speechbrain/speechbrain/blob/aef146861cc89482102f7ce63cda1e8d3366a229/recipes/KsponSpeech/ASR/transformer/hparams/conformer_medium.yaml#L155
For the 1st module, the interface is currently defined here:
https://github.com/speechbrain/speechbrain/blob/aef146861cc89482102f7ce63cda1e8d3366a229/speechbrain/lobes/models/convolution.py#L12
and for the 2nd module:
https://github.com/speechbrain/speechbrain/blob/aef146861cc89482102f7ce63cda1e8d3366a229/speechbrain/lobes/models/transformer/TransformerASR.py#L22
That's for understanding the error stack better.

Pragmatically, the following might help you.
Regarding loading HF repo based on a YAML, try out this snippet:
import sys
from hyperpyyaml import load_hyperpyyaml
from speechbrain.pretrained.fetching import fetch
from speechbrain.utils.distributed import run_on_main

def from_hparams(
        cls,
        source,
        hparams_file="hyperparams.yaml",
        pymodule_file="custom.py",
        overrides={},
        savedir=None,
        use_auth_token=False,
        revision=None,
        **kwargs,
    ):
        if savedir is None:
            clsname = cls.__name__
            savedir = f"./pretrained_models/{clsname}-{hashlib.md5(source.encode('UTF-8', errors='replace')).hexdigest()}"
        hparams_local_path = fetch(
            hparams_file, source, savedir, use_auth_token, revision
        )
        try:
            pymodule_local_path = fetch(
                pymodule_file, source, savedir, use_auth_token, revision
            )
            sys.path.append(str(pymodule_local_path.parent))
        except ValueError:
            if pymodule_file == "custom.py":
                # The optional custom Python module file did not exist
                # and had the default name
                pass
            else:
                # Custom Python module file not found, but some other
                # filename than the default was given.
                raise
        # Load the modules:
        with open(hparams_local_path) as fin:
            hparams = load_hyperpyyaml(fin, overrides)
        # Pretraining:
        pretrainer = hparams["pretrainer"]
        pretrainer.set_collect_in(savedir)
        # For distributed setups, have this here:
        run_on_main(pretrainer.collect_files, kwargs={"default_source": source})
        # skip the rest :D

from speechbrain.pretrained import EncoderDecoderASR

from_hparams(EncoderDecoderASR, source="ddwkim/asr-conformer-transformerlm-ksponspeech", savedir="pretrained_models/asr-conformer-transformerlm-ksponspeech")

It's a copy of the downloading code - but it skips the class construction.
Does it work to continue then with this version?
!pip install git+https://github.com/speechbrain/speechbrain.git@c762107

The next step would be to get the HuggingFace repo working again. Which includes the YAML file and the stored data.
Maybe @dave-rtzr @ddwkim or @kimdwkimdw know more.
Btw - the database URL from the readme is giving a 404.

@anautsch
Thank you for your detail explanation.
Running this code :
from_hparams(EncoderDecoderASR, source="ddwkim/asr-conformer-transformerlm-ksponspeech", savedir="pretrained_models/asr-conformer-transformerlm-ksponspeech")
after installing below version :
!pip install git+https://github.com/speechbrain/speechbrain.git@c762107
The result shows this message :
TypeError: cached_download() takes 1 positional argument but 2 were given
I would like to know about hyperparams.yaml for inference.
Can I get documents or tutorials about hyperparams.yaml for inference?

Hi @starcell hm, sorry I didn't check with that particular version (the loading snippet is what works for me on the current develop branch); so I see your problem a bit more. I do not understand why only this model has problems across SpeechBrain versions but others work just fine.
We provide some tutorials, for example here:
https://speechbrain.github.io/tutorial_basics.html
Here's one on the YAML files:
https://colab.research.google.com/drive/1Pg9by4b6-8QD2iC0U7Ic3Vxq4GEwEdDz?usp=sharing
The hyperparameters are for loading a model into SpeechBrain (pyTorch function are wrapped) and the nested parameters correspond to input arguments of the init functions for SpeechBrain classes (default values are assumed for unspecified arguments).
It could very well be that the checkpoint model is alright - but that one or two lines in the YAML file need to be changed, so that the loading is handled properly.
If you find a solution, please share (opening PRs is also possible on the HuggingFace infrastructure).

This yaml was used for training:
https://github.com/speechbrain/speechbrain/blob/develop/recipes/KsponSpeech/ASR/transformer/hparams/conformer_medium.yaml
and here's the one for inference/loading the final checkpoint:
https://huggingface.co/speechbrain/asr-conformer-transformerlm-ksponspeech/blob/main/hyperparams.yaml
The ConvolutionFrontEnd module is also used in these recipes:
$ grep -r ConvolutionFrontEnd recipes
recipes/LibriSpeech/ASR/transformer/hparams/transformer.yaml:CNN: !new:speechbrain.lobes.models.convolution.ConvolutionFrontEnd
recipes/LibriSpeech/ASR/transformer/hparams/conformer_small.yaml:CNN: !new:speechbrain.lobes.models.convolution.ConvolutionFrontEnd
recipes/LibriSpeech/ASR/transformer/hparams/conformer.yaml:CNN: !new:speechbrain.lobes.models.convolution.ConvolutionFrontEnd
recipes/CommonVoice/ASR/transformer/hparams/train_fr.yaml:CNN: !new:speechbrain.lobes.models.convolution.ConvolutionFrontEnd
recipes/AISHELL-1/ASR/transformer/hparams/train_ASR_transformer.yaml:CNN: !new:speechbrain.lobes.models.convolution.ConvolutionFrontEnd
recipes/Fisher-Callhome-Spanish/ST/transformer/hparams/transformer.yaml:CNN: !new:speechbrain.lobes.models.convolution.ConvolutionFrontEnd
recipes/Fisher-Callhome-Spanish/ST/transformer/hparams/conformer.yaml:CNN: !new:speechbrain.lobes.models.convolution.ConvolutionFrontEnd
recipes/KsponSpeech/ASR/transformer/hparams/conformer_medium.yaml:CNN: !new:speechbrain.lobes.models.convolution.ConvolutionFrontEnd

The depending training recipe also looks normal (at a first glance):
https://github.com/speechbrain/speechbrain/blob/develop/recipes/KsponSpeech/ASR/transformer/train.py

Hi,
Thanks for sharing this issue.
The issue seems to be related with the version of huggingface-hub, which you can temporarily fix by
pip install git+https://github.com/speechbrain/speechbrain.git@c762107
pip install --upgrade huggingface-hub==0.7.0

The installation of the incompatible version of huggingface-hub is probably due to the loose restriction of its version as indicated by requirements.txt.
Since issues are constantly raised by KsponSpeech model users, we currently finished updating the model with the latest version of Speechbrain, and will soon request PR.

@ddwkim
Thank you very much for your answer.
It does work.
I think the tutorial work with speechbrain version 0.5.10(@c762107) and hugginface version 0.7.0.
That's O.K.
And now I want to infer on my local computer with my own model files.
I am trying with recipe/KsponSpeech.
Can I use the hyperparams.yaml from huggingface with my own model files after model training with the recipe and dataset in  KsponSpeech recipe?
I saw the pad_index is -1 in Tokenizer training, but the values is 0 in the hyperparams.yaml.
Does not this matter?
If the hyperparams.yaml file is not fit for the recipe, should I make hyperparams.yaml manually to infer with my own pretrained model?
Thank you again.

Hi @ddwkim please let us know if there are other issues you found with the huggingface-hub - we will work on the HuggingFace integration (refactoring) over the next weeks: any hints on what are possible issues -> it would help.

@starcell
pad_index has no effect on the result since the relative wave lengths will automatically ignore the padded regions.
In other words, the result will be the same regardless of the value of pad_index in the yaml files.
It must have been clearer if pad_index was not specified at all. I will look into that if not specifying pad_index also works.
Sure, but I remember I had to be careful with matching the EncoderDecoderASR parameters with the parameters trained from the train.py script.
We will also fix this issue soon, so that you can seamlessly transfer trained parameters to the EncoderDecoder interface without a hassle.

@ddwkim
Thank you very much.
I am expecting your fixing.

@ddwkim
I came to another problem, so I need your help.
I trained as the recipe KsponSpeech with sb version 0.5.12. And I used yaml file.
The result of inference with trained files on local is "'쉑u-ouner-eo-eri핼 할핼 수가읎어읎어쯧'.
It must be '반말을 할 수가 없어'
My pretrained model files(I trained 26 epochs) :
asr.ckpt : rename from recipes/KsponSpeech/ASR/transformer/results/transformer/7774/save/CKPT+2022-09-30+08-19-25+00/model.ckpt
normalizer.ckpt : recipes/KsponSpeech/ASR/transformer/results/transformer/7774/save/CKPT+2022-09-30+08-19-25+00/normalizer.ckpt
lm.ckpt : recipes/KsponSpeech/LM/results/Transformer/2222/save/CKPT+2022-09-28+19-11-23+00/model.ckpt
tokenizer.ckpt : recipes/KsponSpeech/LM/results/Transformer/2222/save/tokenizer.ckpt
hyperparams.yaml : https://huggingface.co/speechbrain/asr-conformer-transformerlm-ksponspeech/blob/main/hyperparams.yaml
I think the result of asr_model.transcribe_file() is wrong.
Where did I wrong?
Could you help me?
Thank you.

I encountered a similar problem with a newly trained model.
It is likely the wrapper is not running the model as intended.
I suspect that the problem is associated with the configuration in the yaml file.
I am looking into it, and I will get back to you if I find a solution.

@ddwkim
Thank you for your fast answer.
If you tell me the good place to try to look into, I will try.

@ddwkim
Do you find a solution?
I didn't.
I run ASR model training until epoch 100, and I can know the ASR trained well to see wer_eval_clean.txt file.
KsponSpeech_E00099, %WER 0.00 [ 0 / 4, 0 ins, 0 del, 0 sub ]
반말을 ; 할 ; 수가 ; 없어
=  ; = ; =  ; =
반말을 ; 할 ; 수가 ; 없어
It transcribed well, but asr_model.transcribe_file('KsponSpeech_E00099.wav') returns
'짯 반말을 할 수가 없어핼 수가 없어읎어읎어쯧'
May there be some bug in EncoderDecoderASR.from_hparams()?


Update to Sonoma version 14

after update the system to Sonoma v14, the jump between windows not working any more !!!
my skhdrc file bellow, and need to understand this issue !!!
skhd version 0.3.5
yabai-v5.0.3
# change window focus within space
alt - j : yabai -m window --focus south
alt - k : yabai -m window --focus north
alt - h : yabai -m window --focus west
alt - l : yabai -m window --focus east

#change focus between external displays (left and right)
alt - s: yabai -m display --focus west
alt - g: yabai -m display --focus east

# rotate layout clockwise
shift + alt - r : yabai -m space --rotate 270

# flip along y-axis
shift + alt - y : yabai -m space --mirror y-axis

# flip along x-axis
shift + alt - x : yabai -m space --mirror x-axis

# toggle window float
shift + alt - t : yabai -m window --toggle float --grid 4:4:1:1:2:2

# maximize a window
shift + alt - m : yabai -m window --toggle zoom-fullscreen

# balance out tree of windows (resize to occupy same area)
shift + alt - e : yabai -m space --balance

# swap windows
shift + alt - j : yabai -m window --swap south
shift + alt - k : yabai -m window --swap north
shift + alt - h : yabai -m window --swap west
shift + alt - l : yabai -m window --swap east

# move window and split
ctrl + alt - j : yabai -m window --warp south
ctrl + alt - k : yabai -m window --warp north
ctrl + alt - h : yabai -m window --warp west
ctrl + alt - l : yabai -m window --warp east

# move window to display left and right
shift + alt - s : yabai -m window --display west; yabai -m display --focus west;
shift + alt - g : yabai -m window --display east; yabai -m display --focus east;

#move window to prev and next space
shift + alt - p : yabai -m window --space prev;
shift + alt - n : yabai -m window --space next;

# move window to space #
shift + alt - 1 : yabai -m window --space 1;
shift + alt - 2 : yabai -m window --space 2;
shift + alt - 3 : yabai -m window --space 3;
shift + alt - 4 : yabai -m window --space 4;
shift + alt - 5 : yabai -m window --space 5;
shift + alt - 6 : yabai -m window --space 6;
shift + alt - 7 : yabai -m window --space 7;

# stop/start/restart yabai
ctrl + alt - q : brew services stop yabai
ctrl + alt - s : brew services start yabai
ctrl + alt - r : brew services restart yabai

Jumping from window to window is working within the same space for me. The real issue is to move from space to space. It requires to disable System Integrity Protection and use yabai --load-sa as I understand. I am not willing to make those types of change to my computer. (I believe that's the only way to get space switching and window switching across spaces to work).

@hugodemenez  it is currently broken as of MacOS Sonoma 14.1. This happens every new Apple update it seems.

just update your yabai to 6.0 and all works fine again.

@philipphoertnagl  yabai 6.0 was released 2 weeks ago,
but is it working for you on MacOS Sonoma 14.1? Yabai window-space switching with Kernel-integrity protection disabled?

oh I did not even notice that there is already 14.1. I had problems with yabai 5.x when upgrading from MacOS 13.x to 14.0 and yabai 6.0 fixed it, but I did not upgrade yet to 14.1 and probably won't for the next weeks.

I feel ya.

after any update should restart yabai service

Did it work even with the System Integrity Protection enabled?


Basic console & integration testing

Codecov Report

Merging #31 (c93fb9e) into develop (1a846db) will decrease coverage by 0.42%.
The diff coverage is 48.83%.

:exclamation: Current head c93fb9e differs from pull request most recent head 48a443d. Consider uploading reports for the commit 48a443d to get more accurate results

@@             Coverage Diff             @@
##           develop      #31      +/-   ##
===========================================
- Coverage    57.82%   57.40%   -0.43%
===========================================
  Files           20       21       +1
  Lines         2210     2296      +86
===========================================
+ Hits          1278     1318      +40
- Misses         744      784      +40
- Partials       188      194       +6

Impacted Files
Coverage Δ

api/rest/service/atom/atom.go
48.83% <48.83%> (ø)

db/db.go
65.30% <0.00%> (-0.55%)
: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 1a846db...48a443d. Read the comment docs.


Jira link class based views

Description
This patch updates the JIRA-related views to be class-based and refactors common logic in JIRA-related forms.
Test results
Testing done manually to create (via Express and 'normal' methods), view, edit, and delete Jira links.

Thanks for these cleanup changes, much appreciated :+1:


Company permissions

Added permissions to views and templates.
Note the change on the InvenTreeRoleMixin class to allow permission_required declaration in views.

@eeintech thanks again for the contribution 👍


Origin/vic2 to ho i4 release 0.1

Added support-artillery regiments if there are artillery brigades left over. Some code cleanup.

I just was working on some features of my own, and noticed that the force_multiplier setting wasn't in the configuration file. Could you add it to the master copy in data_files, and also work it into the frontend's data file (Vic2toHoI4DefaultConfiguration.xml)? Thanks!

Not quite with you there; the frontend does not, so far as I can discover, have a VictoHoI4DefaultConfiguration XML file. It does have one for each of the other converters, so perhaps this is an oversight? At any rate I added the line to configuration.txt.

Sorry, I should have been more clear. Vic2toHoI4DefaultConfiguration.xml is in Vic2ToHoI4/Data_Files/. It gets copied to the frontend's directory as part of the build process.

Oh, right. Didn't realise I hadn't gotten that change into the release branch. Worked fine on my dev branch. :)


different tensor dimensions from original paper in generator

Hello, I noticed that the generator and the sampler have different dimensions than in the original paper.
For example, in sampler, h0 = tf.reshape(linear(z, self.gf_dim*8*s_h16*s_w16, 'g_h0_lin'), [-1, s_h16, s_w16, self.gf_dim * 8]), I think self.gf_dim should be multiplied by 16 in order to reproduce the dimensions in the original paper. Is there any reason behind multiplying it only by 8 and thus having only half of that dimension?
Thank you very much!
P.S. also referencing #214

@andreistirb You are right, the highest depth dimension that the networks reach, excluding the flattening layer, is 512. I don't know if it brings a great improvement. Have you tried adding another layer?

@MihaiDogariu I haven't tried adding another layer, I just modified the dimensions of the current ones. The  default values of df_dim and gf_dim in the constructor of DCGAN class are 64 and I tried with 128 in order to have the same dimensions as in the original paprer.

@andreistirb And did that produce any significant change in the result?

@MihaiDogariu I can not respond to your question because I do not know how to quantify the changes. :D


I cannot select a Python 3.8 virtual environment for Jupyter notebook that is running remotely via SSH plugin

Environment data

VS Code version: XXX

Jupyter Extension version (available under the Extensions sidebar):

Python Extension version (available under the Extensions sidebar):

OS (Windows | Mac | Linux distro) and version: XXX
local OS: Mac OSX / remote OS:

$ lsb_release -a
LSB Version:	core-9.20170808ubuntu1-noarch:security-9.20170808ubuntu1-noarch
Distributor ID:	Ubuntu
Description:	Ubuntu 18.04.6 LTS
Release:	18.04
Codename:	bionic

$ uname -a Linux ayan 4.15.0-162-generic #170-Ubuntu SMP Mon Oct 18 11:38:05 UTC 2021 x86_64 x86_64 x86_64 GNU/Linux

Python and/or Anaconda version: XXX

lrwxrwxrwx 1 root root 9 Oct  1 19:43 /SeaExp/mona/venv/dpcc/bin/python -> python3.8

$ /SeaExp/mona/venv/dpcc/bin/python
Python 3.8.0 (default, Feb 25 2021, 22:10:10)
[GCC 8.4.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>>

Type of virtual environment used (N/A | venv | virtualenv | conda | ...): XXX
venv (but it doesn't even allow to select it via pressing enter)
Jupyter server running: Local | Remote | N/A
remote via the SSH extension

^ When I press enter here in above image, nothing happens for change kernel. It keeps using Python 2.

^ I tried activating the virt env inside the Jupyter but no luck obviously.
Expected behaviour
To be able to use my own virt env /SeaExp/mona/venv/dpcc/bin/python here for running Jupyter ( I have no problem doing this for normal Python scripts inside VSCode remote)
Actual behaviour
Keeps using Python 2 for Jupyter ipynb notebooks.
Steps to reproduce:
[NOTE: Self-contained, minimal reproducing code samples are extremely helpful and will expedite addressing your issue]

XXX

Logs

Output for Jupyter in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Jupyter)

XXX

as you can see, I have no problem activating or using this Python Virtual Environment inside VSCode for running normal .py scripts.

@monajalal Thanks for the detailed report. If you could, could you please pull the logs from the "Jupyter" output panel when you open the ipynb when connected via SSH. Those logs would be the best first step that I could help with debugging here.

I did just notice as well that you seem to be on older versions of VSCode and our extension. If you could please update to the newest versions before you repro that would be very helpful.

Thank you. Now, it works :)

Awesome, glad that it's working for you now.

Hi @IanMatthewHuff I have the same exact problem in VSCode in a new machine we recently purchased that I am connecting remotely to.
So, I basically installed the latest version of VSCode and similarly, the latest version of all the related extensions.
Yet, when I click on the "gear" icon, it is not responsive, hence I cannot select my own native Python virtual environment. Also, when I paste the path of my virt env and press "Enter" nothing happens.
Could you please advise what would be a solution in this case?

@monajalal That control actually can't accept just a path input. When working with python the dropdown selection should show all the possible options, which would be all the Jupyter kernels and Python environments that we detect on that machine.
When you mention your own native python environments do you mean on your machine? If you connect remotely you should just be seeing the python environments installed on that remote box that you are connected to.

@IanMatthewHuff  no this /home/jalal/research/venv/dpcc/bin/python is installed in the remote machine echo.bumc.bu.edu

solved, had to initially set the interpreter from the command pallet and then select it in the notebook.


Update girder for upstream deployment fixes

Ping.  I think I finally have a working version here.

I thought we had removed the dependence on Romanesco?
I assume that this failure you are fixing was for the girder-worker core plugin in girder.  Could we also fix it by adding the girder-worker to the
 - export IGNORE_PLUGINS=

line in .travis.yml?
I'm fine with whatever you do, asking to understand it better.

LGTM

You're right we could remove girder_worker, but this update was necessary to bring the fontello fix from girder.  https://github.com/girder/girder/pull/1550


PyTorch version training issues

After training the PyTorch version for a while, the metrics will not change. I trained twice and this problem started at 454 epochs.

By the way, can we also add SSIM evaluation metrics to the PyTorch version? Thank you

Hello. Will add SSIM evaluation today.
About the training issues. I still need to fix the model forward pass a bit. Moreover, training on PyTorch has different behaviour to TensorFlow apparently, so I will still be checking to see how to bring it as close as possible to the TensorFlow initial implementation.

Hello. Will add SSIM evaluation today. About the training issues. I still need to fix the model forward pass a bit. Moreover, training on PyTorch has different behaviour to TensorFlow apparently, so I will still be checking to see how to bring it as close as possible to the TensorFlow initial implementation.

Okay, thank you for your reply. Do you still need to add files related to testing in the future? Looking forward to the complete PyTorch version

感觉是在那个轮次训练已经达到饱和了，哈哈哈

@longlong161 Don't know exactly what the issue with training is for you. I didn't have that. Looks like a gradient problem. Please retry with the last code and let me know if it still persists.
@stuhao251 It might be the case, unfortunately. I still get better metrics when training with TF for some reason. However, the baseline PyTorch implementation of the model should be exactly the same as the TF version, as both the TF and Torch versions of the model have 44923 parameters. So please feel free to develop on it if you think there's space for improvement.

@albrateanu I used the latest code you uploaded and encountered a new error that appeared on both machines.

@albrateanu I used the latest code you uploaded and encountered a new error that appeared on both machines.

Hello, has pytorch version come out yet? Why is my result so much worse than the results of tensorflow in the paper?

@longlong161 I'm not sure how I can help with that. Please make sure the environment is ok and running PyTorch 2. And perhaps just attempt retraining more? Or try setting either the Smooth L1 loss or the MS-SSIM loss weight to 0? It seems like a gradient explosion problem and it's not very clear what causes it given that I didn't have that.
@GZY2000 I keep saying this, I am not sure. I am able to obtain at least around 0.8-1dB more on TensorFlow than on PyTorch. But it's a bit out of my knowledge.

@GZY2000 I have just posted PyTorch weights. On LOLv2 Real, PyTorch implementations has 0.61dB more than TensorFlow.

@GZY2000 I have just posted PyTorch weights. On LOLv2 Real, PyTorch implementations has 0.61dB more than TensorFlow.

That's true, but synthetic is more than 2db off on this dataset, which is the best result you can get after a lot of training!

Hello,May I ask which journal your paper has been submitted to?


Use application name as unique identifier for an application

Should:

check whether there are application name duplication.
use the application name when storing offest to kafka.

For the second part, now property appname is not available to the TaskActor, is it OK to put this into TaskContext?

we may also need a unique task identifier for visualization purposes (DAG)


Update build-s3-dist.sh

Using curl with multiple retries to because during my tests while I was running this script at the ec2 boot process, the download failed because the instance didn't have internet connection yet. After implementing this workaround, I didn't have the issue again.

Thanks for opening the pull request. We have created a backlog ticket to track this request.

updated with v1.3.1


Add ability to hide ActionBar icons for switching between commits.

The ActionBar buttons for the extension (the top right buttons) take a lot of space that some users, including myself, would rather have to see more tabs at once. An option to hide these buttons and free up this space while having the extension enabled would be fantastic.

Duplicate of #900


numpy tests research

understand in numpy what the tests are and how they work.

[ ] test__all__.py
[ ] test_ctypeslib.py
[x] test_matlib.py
[x] test_numpy_version.py
[ ] test_public_api.py
[ ] test_reloading.py
[ ] test_scripts.py
[ ] test_warnings.py

test_numpy_version.py:
take the version format as string value and compares it using regular expressions.

  test_matlib.py:
made several tests using matrices.
best reference to our usage so far

Fechando para reformulação dos testes.


NEW: New extension point for has one and many fields.

New extension point for has one and many field

this provides better configuration options as existing extension point updateField is not called via handleRequest
new extension points are also more specific so it's easier to target customisation

Example:

I don't want to have auto complete component for has one field as links can't be shared between objects in my project

Related issues

https://github.com/gorriecoe/silverstripe-linkfield/issues/18

Looks good to me


I found that the input prompts were different, such as "building" and "house", and the detection results were very different.

When I run the target detection task, I find that the input prompts are different, such as "building" and "house", and the detection results are very different. Are there any good suggestions to help me with the detection? Do I need to know your open vocabulary glossary?

Hello, I think it depends on your needs and the model's actual performance under your scenarios. You may try different prompts in a set of images and compute the performance to determine the final prompt usage.


Split ydoc server into separate module

Fixes #10156
Split GUI2 package into smaller packages suitable for monorepo. Migrated relevant build scripts to use proper pnpm script hierarchy.
The ydoc server standalone module can be built by invoking pnpm compile. Then, a dependency-free single file nodejs script is created in app/ydoc-server-nodejs/dist/main.mjs. @hubertp This file is all that needs to be packaged into a docker image.
The server script accepts 3 environment variables: PORT, HOSTNAME and LANGUAGE_SERVER_URL.
When language server url is specified, the server will IGNORE URLs provided by Gui, and always link to specified language server. This is the expected configuration for cloud deployments, as it doesn't allow clients to create arbitrary outgoing connections from our servers. Also that allows the ydoc-server to connect to the language server using a private (or loopback) network address.
Important Notes
The command to run the gui dev environment has been changed. Invoking the old command will print a message about that.
From now on, use pnpm dev:gui2 in repository root.
Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:

[x] The documentation has been updated, if necessary.
[x] Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
[x] All code follows the
Scala,
Java,
TypeScript,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.
[x] Unit tests have been written where possible.

This is likely going to be problematic for backend when it generates JS bundle from existing nodeJS ydoc. Added him to the review.
Re dockerization - this looks sufficient.


nav container implementation as nested view

hi @oblomov-dev
im trying to implement nav container as nested view of the recent pr control toolpage,
it seems not to work as expected..
i figured it will work with other layout type control besides flexible column layout..
here is my demo of it..
demo_88.txt
when pressing the Application nav button it supposed to destory all pages and insert new page, the nested view.
Edit:
maybe it worth a shot to try using the method “to”.
most of the navigation control have it and it also gives us transition animations when moving between views..
https://sapui5.hana.ondemand.com/sdk/#/api/sap.f.FlexibleColumnLayout%23methods/to
https://sapui5.hana.ondemand.com/sdk/#/api/sap.m.NavContainer%23methods/to

hi @oblomov-dev
this demo is working for me,
but when i try to make it work on the icon tab header, its not working.. does it suppose to work like that only ?
here is my demo of what im trying to do..
demo_89.txt

great ill take a look, sorry for all the questions btw..

im getting this in demo 89

fixed!
Harmonized parameter naming of _event and _event_client

still getting the same error..

https://github.com/abap2UI5/abap2UI5-samples/blob/353c253e24f6a2aabef60dce2b9595532a1b9ec9/src/z2ui5_cl_app_demo_89.clas.abap#L131
this time i am sure it works

demo still not working.. i think something changed,, it seems like my old snippet.. not the one with the event on the icon tab header..

89 there was a still a bug, updated again but now 88/89 both working on my side.

yes. now both are working... thanks..
should combine them i think btw.. they are pretty much the same..


CIF-2154 - Improve UI tests for commerce content fragment component

Add waitUntil block for flaky check.
Description
Improve stability of UI test.
Related Issue
CIF-2154
Types of changes

[X] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have signed the Adobe Open Source CLA.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[X] I have read the CONTRIBUTING document.
[X] I have added tests to cover my changes and the overall coverage did not decrease.
[X] All unit tests pass on CircleCi.
[X] I ran all tests locally and they pass.

Three consecutive build passed: https://app.circleci.com/pipelines/github/adobe/aem-cif-guides-venia?branch=CIF-2154


Daemonsets that would not be provisioned on a provisioner are included in the scheduler calculation

Description
Observed Behavior:
Daemonsets that would not be provisioned on a provisioner due to pod template affinity rules ( not selector labels ) are also included in the scheduler calculation in getDaemonOverhead
This issue occurs due to multiple reasons:

Daemonset pods are not generated from template but instead retrieved from cache of existing pods due to #3199
Daemonset admission controller would replace the Pod nodeAffinity on creation to one that match to only 1 node:

spec:
affinity:
nodeAffinity:
requiredDuringSchedulingIgnoredDuringExecution:
nodeSelectorTerms:
- matchFields:
- key: metadata.name
operator: In
values:
- ip-10-110-217-205.ap-southeast-1.compute.internal

Multiple daemonsets targeting different provisioners using node affinities and not selectors are running in the same namespace.

Thus, since no node selectors are defined, and all Pod affinity rules are rewritten by the admission controller, losing the information of nodeAffinity matchExpressions, all the daemonsets resources are considered in the overhead calculation process.
We can get around this with using proper Node Selectors, but we also wanted to utilise the NotIn operator for matchExpressions in nodeAffinity, and could not do the same with Node Selectors
Expected Behavior:
NodeAffinity defined in daemonset Pod templates should be respected even after admission controllers change them.
Reproduction Steps (Please include YAML):

Create 1 provisioner(A) with instance types for 4cpu and labels foo : bar
Create another provisioner (B) with instance types for 2cpu and labels foo: voo
Create Daemonset(A) with requirements for 3cpu and select ProvisionerA using nodeAffinity rules within the Pod template:

apiVersion: apps/v1
kind: DaemonSet
spec:
template:
spec:
affinity:
nodeAffinity:
requiredDuringSchedulingIgnoredDuringExecution:
nodeSelectorTerms:
- matchExpressions:
- key: foo
operator: In
values:
- bar

Create Pod which targets provisioner B using nodeAffinity rules.
Pod cannot be scheduled on provisioner B even though Daemonset A does not target provisioner B.

Versions:

Chart Version: v0.30.0
Kubernetes Version (kubectl version): 1.23

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

Attempted PR #533 but do suggest any other fix to this problem too

Ah, yep. Confirmed that this is an issue. You can see the required nodeAffinity logic that's done here: https://github.com/kubernetes/kubernetes/blob/c5cf0ac1889f55ab51749798bec684aed876709d/pkg/controller/daemon/util/daemonset_util.go#L176. They've opted to completely override the node affinity rather than just append it to the existing set.
Seems like the easy fix for this would be to just use the DaemonSet template nodeAffinity in all cases since we expect it to get overridden when pods are created.


Users can interactively debug cells

I assume this feature will put a "Debug Cell" code lens next to the "Run cell" code lens, just like unit tests have a "Run test" and "Debug test" code lens, right?
Why aren't more people giving it a thumbs up? This feature is super useful

@amittleider Yes, a "Debug Cell" code lens is the current thinking.

eThis would be great. At the moment, I'm defining unit tests and using the 'Debug Test' button that the extension inserts above them to emulate this functionality.


Issue 795: Add subcommand to install GNAT Studio plugin

Closes #795

Squashed FIXUP commits and rebased to main.


Try disabled gc in the wild

Just gonna merge this bad boy, to see if this (SUUUUUPER hacky fix) works on full datasets.


Get a video downloadable details:

@MichaelJamesParsons , this PR contains the API call that gets a video's download details:

Download URL
The expiration time (timestamp)
Status: available, unavailable or pending

Please rebase and fix conflicts

Ok got it

I just did that into #30.


Specifying the language doesn't seem to work with custom HTML

Describe the issue/behavior that seems buggy
I'm trying to use custom HTML with highlight.js and want to specify the language of the code block (because highlight.js seems to guess wrong). This doesn't seem to work, as highlight.js add a seconds language-... class.
Sample Code or Instructions to Reproduce

I'm activating `highlight.js` with the following code:

The HTML for the code block looks like this:

<p>
	<a class="la-look-dangerbutton" href="#">
		<i class="far fa-check la-lpad"></i>Save
	</a>
	&nbsp;
	<a class="la-look-button" href="#">
		<i class="far fa-undo la-lpad"></i>Cancel
	</a>
</p>

Expected behavior
The DOM after highlight.js has done the highlighting looks like this:

<p>
	<a class="la-look-dangerbutton" href="#">
		<i class="far fa-check la-lpad"></i>Speichern
	</a>
	&nbsp;
	<a class="la-look-button" href="#">
		<i class="far fa-undo la-lpad"></i>Abbrechen
	</a>
</p>

i.e. highlight.js has added an additional language-xml CSS class. Does this mean that highlight.js hasn't recognized my language-html class?
I would have expected that the result would start with:

	...

There is no 'html" language.  HTML is an alias for xml.  When highlighting the canonical language name is always added, so in this case you have both - which causes no harm.  If you only want one switch your code to use the xml instead of html.

This is almost entirely true, but not totally:

HTML has predefined entity references for characters (like — etc.), XML doesn't.
Some HTML elements are predefined to have a CDATA content model (for example


bug: Harper lints inline math

Right now Harper throws a hissy fit every time inline math is used. Inline math should be ignored by Harper.
I've been meaning to fix this issue for quite some time now. When inspecting Markdown, Harper currently uses pulldown_cmark, which doesn't presently support math.
However, they recently merged support into the branch for the upcoming release. Before we can include the changes, we need that version of pulldown_cmark to be on crates.io.
The merged request

This has been fixed as of harper-ls v0.8.5 with the introduction of the feature into pulldown_cmark.


.Net 7 support

Could you add .net7.0 to the targets and release the new version?

While we don't mind newer versions, what does just bumping the TFM in a library get you?

So I can build my .net 7 project

You can target Markdig from a .Net 7 project.
You'll end up using the 6.0 build of the library, which will work just as well as a 7.0 one would,since there are no platform specific changes made.

Agreed with @MihaZupan, Markdig is supporting net6.0 which is compatible with net7.0. There is no need to provide a net7.0 version today as we are not using any new features introduced in net7.0.


Increase line height on body text

Completely re-wrote typography styles.  Better responsive headings, body text, etc.


How to ease development/preview in emulator

The bootloader currently defines 'drawWidgets' and this means that if you just do a direct upload to the watch (or run something in the emulator) it's not there and causes an error. There may be more things in the future where this is an issue.
Maybe Bangle.js should just define those variables internally at boot time? That does obviously make stuff a bit less flexible though

Maybe Bangle.js should just define those variables internally at boot time?

This might be a good option to avoid having to encapsulate the drawWidgets function into a try block try { drawWidgets(); } catch(err) {}

Done now


How can we  implement push model(through azure app configuration) in azure functions

I have implemented dynamic configuration(through azure app configuration) in an Azure Functions app using this link(https://docs.microsoft.com/en-us/azure/azure-app-configuration/enable-dynamic-configuration-azure-functions-csharp),
This is poll model, Next I want to implement dynamic configuration using push model in azure functions.

Great, you already have the poll model implemented because in general, the push model is implemented with two modifications on top of the poll model.

A much longer config-refresh cache expiration time, for example, an hour or a day. As config refresh will normally be triggered by push events, this serves as a safety net just in case the push events were missed.
A callback function that will respond to the push notification event and you call SetDirty to signal the immediate need of config refresh.

App Configuration sends push notification events via Event Grid. Azure Functions is one of the handler services Event Grid supports.

If your Function app has only one instance, you can add step 2 above responding to the Event Grid notification directly.
If your Function app has more than one instance, you can add Service Bus to broadcast the Event Grid notification to all instances of your Function app as suggested in this document. Please make sure to create separate Service Bus subscription for each instance. An example can be found in #549.


Feature request: mute std out/std err

A class could be added in the utilities to silence any message that goes through std out and/or std err (both by default).
This can for instance be called using a with statement, encapsulating the simulation itself, so that any print message can be directly muted.

Implemented this in tudatpy.util.redirect_std, pushed in commit 7a14a0b.

Nice feature, thanks for proposing and adding it! Have you also added an API entry?

Yes I have, you can find the associated documentation here.


fix imports dependencies

This allows for the tool not to depend on opencv and pyshark for normal operations.

For simplicity's sake, I have now decided not to "allow" the software to be used without installing it. I also tried to improve the installation of the dependencies via the bash script. And I explained in the readme how to install the package without going through AUR which is not up to date.

Those changes are made to avoid the need to have opencv installed on the system when no command using opencv will be used. Same for pyshark.
python-pyshark:
Total Installed Size:  111.62 MiB
python-opencv
Total Installed Size: 202.77 MiB
In #24 I made those optional dependencies which is a common thing to do in those cases, but currently even basic command will crash on import error as those imports are made first and not on the command depending on it.

Oh right, the two PR are linked, I understand better.
However, without opencv, no command can work (unless it was uninstalled after the quick configuration) and in general, opencv is already installed (Howdy uses it). But ok, why not for pyshark it makes it possible to save some MB. Even though nowadays 100Mib is not much.
I will push in another commit with the changes for pyshark and an update for the package.
Merci beaucoup d'avoir pris le temps de me partager ton expérience :smile:.

That is true, I use howdy so opencv is installed anyways. I think python-opencv isn't even listed as dependency to howdy even though it is required...
The main concern about having dependencies is not only the space but the headache of solving an issue if any arise. pyshark pulls wireshark from dependencies in the AUR and that comes with many others that I don't want to deal with :)
De rien :wink:


Understand getChatMember

Im traying to check if a user is in a chat, i just found this fucntion, but i dont know how to work with it.
I try to open in with a log but there is no usefull info, also i try an If/else, but when a user was not the chat it throws an error.
Can anyone help me? Or maybe help me with a diferent function to do that?.
Thanks

Can you please elaborate the problem and copy the snippet please.

You should use this:
bot.getChatMember(chatId,userId).then(() => {     console.log("Exist");     }).catch((e) => {         if(e.response.body.error_code == 400){             console.log("Not exist");         }     } );

Use this:
bot.getChatMember(chatId,userId).then(() => { console.log("Exist"); }).catch((e) => { if(e.response.body.error_code == 400){ console.log("Not exist"); } } );

Thanks man, it works.

Closing since question is answered. The method returns a promise which has to be resolved.


Added certificate provider,  peer client pool, hash ring and client factory

Description

Added HashRing class which maps identification keys to Data Prepper hosts.
Added certificate provider factory which provides acm, s3 and file certificates
Added peer client pool which creates a WebClient
Added client factory which creates HashRing and sets PeerClientPool attributes.

New additions other than existing logic from peer forwarder plugin include

getServerIp() methods in HashRing class which takes set of identification keys instead of trace id.
getHttpClient in PeerClientPool creates a WebClient instead of TraceServiceGrpc
PeerForwarderAppConfig
PeerForwarderConfiguration added more ACM related config options
PeerForwarderClientFactory
CertificateProviderFactory is same as the one in otel trace source

Issues Resolved
This PR helps with resolving #1606 which will be a separate PR
Check List

[ ] New functionality includes testing.
[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed with a real name per the DCO

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #1663 (ac5995b) into main (98ab3bb) will decrease coverage by 0.18%.
The diff coverage is 84.17%.

@@             Coverage Diff              @@
##               main    #1663      +/-   ##
============================================
- Coverage     93.73%   93.55%   -0.19%
- Complexity     1267     1298      +31
============================================
  Files           166      170       +4
  Lines          3638     3784     +146
  Branches        291      298       +7
============================================
+ Hits           3410     3540     +130
- Misses          159      174      +15
- Partials         69       70       +1

Impacted Files
Coverage Δ

...prepper/peerforwarder/discovery/DiscoveryMode.java
100.00% <ø> (+11.11%)
:arrow_up:

...pper/peerforwarder/PeerForwarderConfiguration.java
80.13% <44.44%> (-3.70%)
:arrow_down:

...pper/peerforwarder/PeerForwarderClientFactory.java
76.47% <76.47%> (ø)

...com/amazon/dataprepper/peerforwarder/HashRing.java
91.48% <91.48%> (ø)

...azon/dataprepper/peerforwarder/PeerClientPool.java
92.30% <92.30%> (ø)

...aprepper/peerforwarder/PeerForwarderAppConfig.java
100.00% <100.00%> (ø)

...warder/certificate/CertificateProviderFactory.java
100.00% <100.00%> (ø)

...repper/peerforwarder/discovery/DiscoveryUtils.java
50.00% <0.00%> (+50.00%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@asifsmohammed , Thanks for moving this code over and generalizing it for core peer-forwarder.
The main comment I'd like resolved is the one about createPeerClientPool. Also, there is a possible memory leak. We can create an issue to ensure we fix this.
The other two comments are optional as they mostly clean up internal design.


Removed warning in benchmarks

One value was not unwrapped, and the compiler has been warning
about unused result while checking target benchmarks.
Pre-review checklist

[x] I have split my patch into logically separate commits.
[x]  All commit messages clearly explain what they change and why.
[ ] I added relevant tests for new features and bug fixes.
[x] All commits compile, pass static checks and pass test.
[x] PR description sums up the changes and reasons why they should be introduced.
[ ] I added appropriate Fixes: annotations to PR description.

The CI fails because of nonmerged clippy "deprecated use" fix

@wprzytula please rebase, the CI check should pass now but the PR needs to be based on current main.


QA with SLES15-SP4 VMWare image

Please test vSphere provider provisioning with the SLES15-SP4 image for VMWare.

This needs to go through PM for prioritization before being assigned to the team


fix: zig 0.13.0-dev.46+3648d7df1

Minor change to work with zig v13

Thanks.
It would be easier to review without all the reformatting. Can you try and remove those?

sure, oops sorry for that

sure, also tests were passing for me. i'll the debugging to see what caused the workflow to fail

that should be good, lmk! 🙂

that should be good, lmk! 🙂

sorry for all this mess, I'm familiarizing myself with this codebase.
this commit has most of the changes which is very minor (switching from std.ComptimeStringMap).
tests should pass now.

No worries I'll squash all that :)

that should be good?

oh oops! haha sorry

that should be it! :D

The error must also exists on the buzz side. You can add it under here:
https://github.com/buzz-language/buzz/blob/main/src/lib/errors.buzz#L3
The following means we're going to push on the stack the error errors.FileSystemError.InvalidBatchScriptArg.
ctx.vm.pushErrorEnum("errors.FileSystemError", @errorName(err))

Right now zig errors have their counterpart on the buzz side but I plan on simplifying those a little on the buzz side (https://github.com/buzz-language/buzz/issues/157)

The error must also exists on the buzz side. You can add it under here:
https://github.com/buzz-language/buzz/blob/main/src/lib/errors.buzz#L3
The following means we're going to push on the stack the error errors.FileSystemError.InvalidBatchScriptArg.
ctx.vm.pushErrorEnum("errors.FileSystemError", @errorName(err))

Right now zig errors have their counterpart on the buzz side but I plan on simplifying those a little on the buzz side (#157)

sorry I missed that, added now. Yes that is true. but I think the access to know what the error exactly is on buzz is also great.

Huh. I'm not reproducing the CI error on my mac. Seems like the mir building step is doing it on a different cpu from the following steps.

Merging anyway, i'll temporarily disable the macos test if this persists


Cleanup proto

What this PR does / why we need it?

Remove unnecessary client information from the protobuf response
Change clientID type from bytes to string in the API

Any background context you want to provide?
What are the relevant tickets?

Related https://github.com/yorkie-team/yorkie/issues/498
Checklist

[x] Added relevant tests or not required
[x] Didn't break anything

@chacha912 Thanks for your contribution. 👍

@chacha912 Do we have to merge this to the main branch too?
Related to https://github.com/yorkie-team/yorkie-js-sdk/issues/623

I apologize for the confusion. As I mentioned in the previous #616, due to the proto change in #611 causing test failures, I had set the target branch to concurrent-case-handling. Should I reopen PRs for #616 and #617 to the main branch?


Tensorflow 2.0

Does this work with tensorflow 2.0 keras, because I am trying to use within tf2.0 but failed.

faced the same issue.

It seems to be resolved now. Try in colab or Kaggle
Try with Tensorflow > 2.1.0

remember modify
from classification_models.keras import Classifiers
to
from classification_models.tfkeras import Classifiers

Tried it on colab after installing it with pip.
It seems it can find the module classification_models, but cant find classification_models.tfkeras

I installed using pip install git+https://github.com/qubvel/classification_models.git
But I am getting this error when importing: cannot import name 'get_submodules_from_kwargs' from 'classification_models' (/opt/conda/lib/python3.7/site-packages/classification_models/__init__.py)

Hmm, this is still an issue. Got the same error with these versions:
Keras                     2.4.3
Keras-Applications        1.0.8
Keras-Preprocessing       1.1.2
tensorflow                2.3.0

Tried also with Keras-Application==1.0.7 (as this was mentioned somewhere else), but still the same error.

The following worked for me:  ( I had to install keras-applications .  Then all worked )
!pip install image-classifiers
!pip install keras-applications
from classification_models.tfkeras import Classifiers
Classifiers.models_names()


Update sample.config for config changes

This change is

r? @vinipsmaker
(maidsafe_highfive has picked a reviewer for you, use r? to override)


Automatically detect google app id in iOS

Resolves https://github.com/fastlane/fastlane-plugin-firebase_app_distribution/issues/32

@googlebot I signed it!

@snatchev, hello! I doesn't have merge button.

@Cy-4AH thank you for the contribution (and ping)!


Prepare for Gradle plugin release 4.0.1

#278  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @rbro112 and the rest of your teammates on  Graphite


Visual Studio Code never close my files

Type: Feature Request
Visual Studio Code never closes my files even when I save and close the files
VS Code version: Code 1.73.1<PHONE_NUMBER>f055b99068d3688932416f2346dd3b, 2022-11-09T04:27:29.066Z)
OS version: Windows_NT x64 10.0.19044
Modes:
Sandboxed: No

What happens instead?

Extension bisect?

Well this happen when i open a folder, vs code keeps open an unsaved few files even after saved and close them. I try disabling all my extentions but nothing happen, there is any configuration for that ?.
I made a GIF trying to be more graphic, sorry i'm newbie.
https://media.giphy.com/media/r71QTcgsFweL7LAO7W/giphy.gif

Please see if you can find any logged output that could help us find the source of the issue:

run code --verbose from the command line and attach the output
in Code select Help | Toggle Developer Tools and select the Console
attach any output you see in the Console you can find

Thanks!

Both outputs are to big, i put it in my onedrive. https://1drv.ms/u/s!AolNQinqyFwSty9Z2A39JpLUhuNT?e=Iz58ce

I wonder if this issue is being caused by an installed extension. Can you try to run VS Code without extensions? From the command line (NOT the integrated terminal in Code), execute: code --disable-extensions and try your steps again to see if it reproduces. If you see it is an issue with the extension, please file it against the extension repository itself.

I'm using a older version now (1.54), and the problem is gone. The reason why i update my vscode is because the Errorlens extension doesn't work (and still is not working), but i'm okay. Thanks for the help.

@eduardodiaz92 when you quit VSCode, can you see if there are any contents in C:\Users\yourname\AppData\Roaming\Code\Backups

This issue has been closed automatically because it needs more information and has not had recent activity. See also our issue reporting guidelines.
Happy Coding!


Github doesn't have comments

Github doesn't have comments, so I'll write it here.
This is simply the best portfolio site I've ever seen! :clinking_glasses:

Thank you @YashKumarVerma . I'm glad you liked it :smile:


CLOUDP-76488: update go client

Proposed changes
Jira ticket: CLOUDP-76488
Checklist

[x] I have signed the MongoDB CLA
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added any necessary documentation (if appropriate)
[x] I have run make fmt and formatted my code

Further comments
This needs mor etesting

closed by #520


shields: add generic shield to use CDC ACM UART as backend

Add  generic shields that provide devicetree and configuration
overlays, and configure USB device stack so that CDC ACM UART
can be used as backend for console, logging, or shell.
This shield is not for a real hardware device but for an emulated UART controller,
analogous to how we have emulated boards.
This approach allows us to avoid many identical overlays in
samples and tests directories, and cumbersome board configurations.
These shields use Kconfig option CONFIG_USB_DEVICE_INITIALIZE_AT_BOOT
and is mainly intended to be used with boards like
nrf52840dongle_nrf52840 or bl654_usb.
Draft for discussion (derived from #40220).

Hi @jfischer-no I'm thankful for this work but I have a big problem with this being "manually opt in by specifying a shield" as per
 Set ``-DSHIELD=cdc_acm_shell`` when you invoke ``west build``. For example:

This is just a non-starter for our customers, they need an easy way of getting zephyr and being able to test things as they do with any other board, not have to jump through hoops trying to get the most basic sample applications working. For our BL654 USB dongle it's basically a given that almost everyone will be using the CDC port either for Bluetooth or as a general UART and this should cater for those majority of users.

This is just a non-starter for our customers, they need an easy way of getting zephyr and being able to test things as they do with any other board, not have to jump through hoops trying to get the most basic sample applications working. For our BL654 USB dongle it's basically a given that almost everyone will be using the CDC port either for Bluetooth or as a general UART and this should cater for those majority of users.

I cannot take this seriously.

BL654_USB, https://docs.zephyrproject.org/latest/boards/arm/bl654_usb/doc/bl654_usb.html, looks like an end product, it is not a development board, even if it would (like nrf52840dongle) it is still very limited in terms of capabilities.
Zephyr's main tree is not a BSP for BL654_USB, you have to compromise and understand that not everything here is about your board.
It is obvious that we cannot enable USB device support by default at boot time. And there will always be a conflict between configuration at board level and configuration in the samples. So we need additional option for the boards like BL645_USB if it should have best possible coverage of the samples. There is no way around something like:  west build -b nrf52840dongle_nrf52840 samplel -- -DADDITIONAL_OPTION=y, or a board variant like bl654_usb_usb_on, which is not better.

Taking (1) and (2) into account, what are most basic applications for BL654_USB dongle? Probably none. But useful applications, such as hci_usb or ot-coprocessor, work fine even with minimal board configuration, https://docs.zephyrproject.org/latest/guides/porting/board_porting.html#general-recommendations.
Finally, considering the steps required to get the firmware onto the board, is the additional option (3), like -DSHIELD=cdc_acm_shell, to build the firmware is the least evil of all.

This PR addresses a problem which one way or the other has to be solved, and this seems like a rather elegant solution, getting rid of the need to create overlays for each new program/tailored configs for each board enabling usb at boot.
I myself am working on a port of Wio Terminal board, which doesn't have a native UART, so needs to use CDC ACM to open a console accessible over USB.

This shield is not for a real hardware device but for an emulated UART controller,
analogous to how we have emulated boards.

I remain opposed to the idea of using SHIELD for htis.

Dev-review (Dec 16, 2021):

Clarify in board porting guide about USB being ok to enable if that is being used for serial console for the board
Feeling from @mbolivar-nordic, @galak, @tejlmand felt that using SHIELD here didn't make sense as this was misusing the concept of a SHIELD.  We need to look how we solve this  -- Take a look at https://github.com/zephyrproject-rtos/zephyr/pull/40669

Dev-review (Dec 16, 2021):

@galak if I recall correctly, it was also desired at the dev-review that this commit https://github.com/zephyrproject-rtos/zephyr/pull/40645/commits/9294f232b307d138a935979a3ef41ca6e4f062df was taken into a dedicated PR, cause the ability to initialize USB device at boot time is a useful feature regardless of the opposition to use shield mechanism for cdc acm uart.


Fix overpass speed being detected

We dont want to abs() because sometimes the speed limit is only 5 and apperently thats the default for tesla, so it'll ignore most speed limits when it gets to that point.
That said, I think a better solution like @bitshop said is the right approach. Unfortunately, I don't think the map data gives a direction for the lane you are in. So we can't go off that. we need some memory of previous speed limits at different ticks, and only act after a certain number?

@SippieCup if I'm not mistaken the var naming is misleading. It looks like pedal_set_speed_ms is the current pid speed and speed_limit_ms is max pedal speed.
I didn't test it but in that case this change would not work as expected. Also I don't know how 46.3 m/s matches with the comment stating 20 mph.
And lastly this commit only affects fleet speed and not speed limit detection. Was this intentionally?
Keep in mind that there are countries like Germany with no speed limit (currently capped in code at 170 kph), which may also cause corner cases.


Marketplace Users & Units menu cannot be accessed by customer admin

[Version] OSCM 17.4.0 2017/10/09
Docker image with TomEE
Chrome
[How to reproduce]

log in at marketplace as customer administrator
press users & units menu

[Expected result]
users & units menu can be accessed
[Observed result]
error message "You do not have sufficient authorities to view the requested page."

[build] 2017/10/19 verified


TestEnv.ClientRes.Status is not behaving as expected when ExitStatus is called.

Steps to reproduce:
lab.go:
package golab45

import (
	"net/http"

	"github.com/Kong/go-pdk"
)

type (
	Config struct {
	}
)

func (c *Config) Access(kong *pdk.PDK) {
	kong.Response.ExitStatus(http.StatusBadRequest)
}

lab_test.go:
package golab45_test

import (
	"golab45"
	"net/http"
	"testing"

	"github.com/Kong/go-pdk/test"
)

func Test(t *testing.T) {
	env, _ := test.New(t, test.Request{Method: http.MethodGet, Url: "http://example.com"})
	env.DoHttp(&golab45.Config{})
	if env.ClientRes.Status != http.StatusBadRequest {
		t.Errorf("expected %d got %d", http.StatusBadRequest, env.ClientRes.Status)
	}
}

test output:
Running tool: /usr/local/go/bin/go test -timeout 30s -run ^Test$ golab45

--- FAIL: Test (0.00s)
    ./test.go:579: Access
    ./lab_test.go:15: expected 400 got 200
FAIL
FAIL	golab45	0.003s
FAIL

Possibly related to https://github.com/Kong/go-pdk/issues/119

Thank you for the repro sample.

Internally tracked: KAG-3813
I agree it should be related to #119

This issue is fixed as of version v0.10.2; please give it a try! Closing the issue for now. Thank you for taking the time to report!


 I suspect most of this is leftovers from splitting the SNAT code into ingress/egress paths.
Also clean up two small inconsistencies in the ICMPv6 NS / NA handling.

/test

/test

Add two more small patches, and rebase to pick up https://github.com/cilium/cilium/pull/26641 for instant gratification.

/test


repeating genes in pl.rank_genes_groups_heatmap

[x] I have checked that this issue has not already been reported.
[ ] I have confirmed this bug exists on the latest version of scanpy.
[ ] (optional) I have confirmed this bug exists on the master branch of scanpy.

Hello
It seems the heatmap generated by pl.rank_genes_groups_heatmap is showing repeated genes, as seem in the picture below

This occured in 1.7.1, which I had to use because of #1941
I assume this is not working as intended.
The command I have used is sc.pl.rank_genes_groups_heatmap(adata, n_genes=5, standard_scale='var', swap_axes=True)

Hi @dm8000,
I think it might just be that these genes are markers of multiple populations in your sc.tl.rank_genes_groups() results. You could check whether they are by looking at the adata.uns where you stored your results. There is a ['names'] column where you can get the names of the top genes per cluster.


[System Tests] RestartKafkaBrokers test is reporting an error many times

Describe the bug
The RestartKafkaBrokers test is reporting an error because the broker is not restarted as expected, but it is out of scope the test.
To Reproduce
Run the RestartKafkaBrokers test
Expected behavior
The broker should be restarted successfully. If this is something that is happening a lot I'd suggest to be less restricted about checking if it is restarted successfully or not, we just want a broker to be removed and check that the messages are still being received.
YAML files and logs
io.fabric8.kubernetes.client.KubernetesClientTimeoutException: Timed out waiting for [300000] milliseconds for [Pod] with name:[my-cluster-kafka-0] in namespace [kafka].

	at io.fabric8.kubernetes.client.dsl.internal.BaseOperation.waitUntilCondition(BaseOperation.java:946)
	at io.fabric8.kubernetes.client.dsl.internal.BaseOperation.waitUntilCondition(BaseOperation.java:98)
	at io.kroxylicious.systemtests.utils.KafkaUtils.restartBroker(KafkaUtils.java:126)
	at io.kroxylicious.systemtests.steps.KafkaSteps.restartKafkaBroker(KafkaSteps.java:85)
	at io.kroxylicious.systemtests.KroxyliciousST.restartKafkaBrokers(KroxyliciousST.java:96)
	at java.base/java.lang.reflect.Method.invoke(Method.java:580)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1596)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1596)

KroxyliciousST_restartKafkaBrokers_cluster_dump_2024_09_20_08_39_08.zip
Additional context
Add any other context about the problem here.

The broker should be restarted successfully. If this is something that is happening a lot I'd suggest to be less restricted

Surely, the broker should come back after it is restarted. If it doesn't something is wrong, no?

io.fabric8.kubernetes.client.KubernetesClientTimeoutException: Timed out waiting for [300000] milliseconds for [Pod] with name:[my-cluster-kafka-0] in namespace [kafka].

what's the state of the pod according to the kube client?  Looking at the logs you attach, the broker is running:
2024-09-20 08:34:24,385 INFO [BrokerLifecycleManager id=0] The broker has been unfenced. Transitioning from RECOVERY to RUNNING. (kafka.server.BrokerLifecycleManager) [broker-0-lifecycle-manager-event-handler]


Fix broken UTF8 when truncating with ellipsis

The regular expression used to remove trailing punctuation before adding ellipsis cannot handle UTF-8 strings and sometimes  creates invalid UTF-8 sequences, causing incorrect results and DOMDocument errors:
DOMDocumentFragment::appendXML(): Entity: line 1: parser error : internal error: detected an error in element content

DOMDocumentFragment::appendXML(): Entity: line 1: parser error : Input is not proper UTF-8, indicate encoding !
Bytes: 0xC3 0xE2 0x80 0xA6

To solve this, this MR just adds the 'u' (PCRE_UTF8) flag to that regular expression. Test case is included.

please, can you merge this?, we could stop using our own fork if you do.

Thanks


[release-1.9] fix revision missing for reverse translation.

This is an automated cherry-pick of #30206

@googlebot I consent.

@googlebot I consent.


Refactor the navbar's social icons

Description
Refactor the navbar's social icons code. Removed the repetition part, and instead of that used the mapping to display all icons.
Fixes #1448
Type of change

Code Refactoring

How Has This Been Tested?

Tested this change over Chrome Browser and Microsoft Edge

Checklist:

[x] I have performed a self-review of my own code
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works

Screenshots or example output

@priyankarpal, let me know if any changes are required in it.

@priyankarpal, I renamed all links to NavLinks and also, Rename the icon to Icon in NavLink mapping to stick to the PascalCase convention for JSX components

@priyankarpal, I renamed all links to NavLinks and also, Rename the icon to Icon in NavLink mapping to stick to the PascalCase convention for JSX components

thank you


Leftover tasks for nonroot user support

Bug description

All of these changes will break the new installer I think, since it will still try to do things like bind to 443?

update document for galley validation server and sidecar webhook server.

Affected product area (please put an X in all that apply)
[ ] Configuration Infrastructure
[x] Docs
[x] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

2019-08-01T16:09:00.999634Z     fatal   Invalid validationArgs: 1 error occurred:
        * port number 443 must be in the range 1024..65535

Getting this when installing using the new installer
We shouldn't reject lower ports, users can run as root if they want to.


[keyvault] Verify challenge response matches request domain

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-keyvault-certificates


Octane plugin for TFS 2015 -> PAT

Hi,
During the configuration the PAT is required, however this feature was introduced in TFS 2017?
What should be the option/solution when using TFS 2015?
Thanks!

We supports from 2015 update 4
For this case - our configuration allows to to authentication with user/password


Add a collection to the index

Collection name
abc
Maintainer name
a
Maintainer contact link (i.e. repo issues link, email)
b
Repository URL
c
OCI Reference
d
I acknowledge that this collection provides new functionality, distinct from the existing dev container collections part of this index

[x] Confirm

#sign-off