[BUG] Some themes fail to build

What did you expect to happen?
I have reinstalled emacs and doom fresh from the git HEAD. When I launch the GUI emacs, I get the following error:
condition-case: Error in a Doom startup hook: after-init-hook, doom-init-theme-h, (error "Unable to find theme file for ‘doom-one-light’")

This happens for all Doom themes, but not for themes I have installed myself in packages.el. However, when using other themes, this error pops up, but only on the GUI (and not on the TUI):
condition-case: Error in a Doom startup hook: after-init-hook, doom-init-theme-h, (doom-hook-error doom-load-theme-hook doom-themes-org-config (error "Autoloading file /Users/evar/.emacs.d.doom/.local/straight/build-28.0.60/doom-themes/doom-themes.el failed to define function doom-themes-org-config"))

There is some weird build failure going on that is probably related; E.g.,
(package! soft-stone-theme
  :recipe (:build (:not native-compile compile)))

will fail doom sync with:
> Executing 'doom sync' with Emacs 28.0.60 at 2021-10-13 16:24:17
  > Synchronizing your config with Doom Emacs...
    > Installing packages...
      > Updating recipe repos...
      > Cloning soft-stone-theme...smirror-mirror...
      > Building soft-stone-theme...
      x There was an unexpected error
        Message: Error with packages
        Error: (doom-package-error "soft-stone-theme" (error "soft-stone-theme.el:0:0: error: cl-assertion-failed: ((> (point) output-start))"))
        Backtrace:
          (signal doom-package-error ("soft-stone-theme" (error "soft-stone-theme.el...
          (condition-case e (let ((straight-use-package-pre-build-functions (cons #'...
          (progn (if (file-directory-p (straight--repos-dir local-repo)) nil (doom--...
          (let* ((--cl-rest-- (append (list :recipe recipe) recipe)) (recipe (car (c...
          (while --dolist-tail-- (setq recipe (car --dolist-tail--)) (let* ((--cl-re...
          (let ((--dolist-tail-- recipes) recipe) (while --dolist-tail-- (setq recip...
          (let* ((recipes (doom-package-recipe-list)) (built nil) (straight-use-pack...
          (and t (let* ((recipes (doom-package-recipe-list)) (built nil) (straight-u...
          (let* ((built (and t (let* ((recipes (doom-package-recipe-list)) (built ni...
          (let ((doom-output-indent (+ doom-output-indent-increment doom-output-inde...
      ! Extended backtrace logged to doom.error.log

Also happens with:
(package! jetbrains-darcula-theme)

      > Cloning jetbrains-darcula-emacs-theme (for jetbrains-darcula-theme)...
      > Building jetbrains-darcula-theme...
      x There was an unexpected error
        Message: Error with packages
        Error: (doom-package-error "jetbrains-darcula-theme" (error "jetbrains-darcula-theme.el:0:0: error: cl-assertion-failed: ((> (point) output-start))"))
        Backtrace:
          (signal doom-package-error ("jetbrains-darcula-theme" (error "jetbrains-da...
          (condition-case e (let ((straight-use-package-pre-build-functions (cons #'...
          (progn (if (file-directory-p (straight--repos-dir local-repo)) nil (doom--...
          (let* ((--cl-rest-- (append (list :recipe recipe) recipe)) (recipe (car (c...
          (while --dolist-tail-- (setq recipe (car --dolist-tail--)) (let* ((--cl-re...
          (let ((--dolist-tail-- recipes) recipe) (while --dolist-tail-- (setq recip...
          (let* ((recipes (doom-package-recipe-list)) (built nil) (straight-use-pack...
          (and t (let* ((recipes (doom-package-recipe-list)) (built nil) (straight-u...
          (let* ((built (and t (let* ((recipes (doom-package-recipe-list)) (built ni...
          (let ((doom-output-indent (+ doom-output-indent-increment doom-output-inde...
      ! Extended backtrace logged to doom.error.log

This same error also happens for doom-themes, but not as deterministically; Removing .local/straight/build-28.0.60/doom-themes (do keep .local/traight/repos/emacs-doom-themes or the error won't be triggered) and then running doom sync triggers it:
> Executing 'doom sync' with Emacs 28.0.60 at 2021-10-13 16:37:43
  > Synchronizing your config with Doom Emacs...
    > Installing packages...
      > Building doom-themes...
      x There was an unexpected error
        Message: Error with packages
        Error: (doom-package-error "doom-themes" (error "doom-themes-ext-visual-bell.el:0:0: error: cl-assertion-failed: ((> (point) output-start))"))
        Backtrace:
          (signal doom-package-error ("doom-themes" (error "doom-themes-ext-visual-b...
          (condition-case e (let ((straight-use-package-pre-build-functions (cons #'...
          (progn (if (file-directory-p (straight--repos-dir local-repo)) nil (doom--...
          (let* ((--cl-rest-- (append (list :recipe recipe) recipe)) (recipe (car (c...
          (while --dolist-tail-- (setq recipe (car --dolist-tail--)) (let* ((--cl-re...
          (let ((--dolist-tail-- recipes) recipe) (while --dolist-tail-- (setq recip...
          (let* ((recipes (doom-package-recipe-list)) (built nil) (straight-use-pack...
          (and t (let* ((recipes (doom-package-recipe-list)) (built nil) (straight-u...
          (let* ((built (and t (let* ((recipes (doom-package-recipe-list)) (built ni...
          (let ((doom-output-indent (+ doom-output-indent-increment doom-output-inde...
      ! Extended backtrace logged to doom.error.log

doom build will also trigger this doom-themes build failure error, though I have not tested it after my reinstall, as it takes ages to fail.
Update: Adding
(package! doom-themes
  :recipe (:build (:not native-compile compile)))

to packages.el can make the build error more deterministic. (The non-compile parts are irrelevant, I just added them to make sure the issue is not with native comp.)
What actually happened?
_
Describe your attempts to resolve the issue
No response
Steps to reproduce
_
System Information
https://paste.gnugen.ch/paste/Mf7M

@hlissner I do have an emacs 27 on my system (the mac port)(on /Applications/Emacs_mac.app/Contents/MacOS/Emacs), but I don't know how to make doom build packages for it.
BTW, here is the build directory of doom-themes:
❯ ls ~/.emacs.d.doom/.local/straight/build-28.0.60/doom-themes
.                                doom-nord-light-theme.el
..                               doom-nord-theme.el
doom-1337-theme.el               doom-nova-theme.el
doom-Iosvkem-theme.el            doom-oceanic-next-theme.el
doom-acario-dark-theme.el        doom-old-hope-theme.el
doom-acario-light-theme.el       doom-one-light-theme.el
doom-ayu-light-theme.el          doom-one-theme.el
doom-ayu-mirage-theme.el         doom-opera-light-theme.el
doom-badger-theme.el             doom-opera-theme.el
doom-challenger-deep-theme.el    doom-outrun-electric-theme.el
doom-city-lights-theme.el        doom-palenight-theme.el
doom-dark+-theme.el              doom-peacock-theme.el
doom-dracula-theme.el            doom-plain-dark-theme.el
doom-ephemeral-theme.el          doom-plain-theme.el
doom-fairy-floss-theme.el        doom-rouge-theme.el
doom-flatwhite-theme.el          doom-shades-of-purple-theme.el
doom-gruvbox-light-theme.el      doom-snazzy-theme.el
doom-gruvbox-theme.el            doom-solarized-dark-high-contrast-theme.el
doom-henna-theme.el              doom-solarized-dark-theme.el
doom-homage-black-theme.el       doom-solarized-light-theme.el
doom-homage-white-theme.el       doom-sourcerer-theme.el
doom-horizon-theme.el            doom-spacegrey-theme.el
doom-ir-black-theme.el           doom-themes-base.el
doom-laserwave-theme.el          doom-themes-ext-neotree.el
doom-manegarm-theme.el           doom-themes-ext-org.el
doom-material-theme.el           doom-themes-ext-treemacs.el
doom-miramare-theme.el           doom-themes-ext-visual-bell.el
doom-molokai-theme.el            doom-themes.el
doom-monokai-classic-theme.el    doom-tomorrow-day-theme.el
doom-monokai-machine-theme.el    doom-tomorrow-night-theme.el
doom-monokai-octagon-theme.el    doom-vibrant-theme.el
doom-monokai-pro-theme.el        doom-wilmersdorf-theme.el
doom-monokai-ristretto-theme.el  doom-xcode-theme.el
doom-monokai-spectrum-theme.el   doom-zenburn-theme.el
doom-moonlight-theme.el

I wonder if I can just copy the missing files here as a stopgap workaround?
Here is my doom info:
> Executing 'doom info' with Emacs 28.0.60 at 2021-10-13 21:25:28
    generated    Oct 13, 2021 21:25:28
      system       MacOS 11.2.1 Darwin 20.3.0 x86_64
      emacs        28.0.60 ~/.emacs.d.doom/ -> ~/.emacs.d.doom/
      doom         21.12.0-alpha HEAD -> develop a33865569 2021-10-11 20:25:18 +0200 ~/doom.d/ -> ~/doom.d/
      shell        /bin/dash
      features     ACL GIF GLIB GMP GNUTLS IMAGEMAGICK JPEG JSON LCMS2 LIBXML2 MODULES NATIVE_COMP NOTIFY KQUEUE NS PDUMPER PNG RSVG THREADS TIFF TOOLKIT_SCROLL_BARS XIM XWIDGETS ZLIB
      traits       batch server-running custom-file
      modules      :completion (company +tng) (ivy +fuzzy +icons) :ui doom doom-dashboard hl-todo hydra indent-guides modeline nav-flash ophints (popup +defaults) unicode vc-gutter vi-tilde-fringe :editor (evil +everywhere) file-templates fold snippets word-wrap :emacs dired electric undo vc :term vterm :checkers syntax :tools ein (eval +overlay) gist (lookup +dictionary +docsets +offline) (lsp +eglot +peek) magit pdf :lang clojure common-lisp coq crystal csharp data (dart +flutter) elixir emacs-lisp fsharp (go +lsp) idris json julia kotlin latex lua markdown nim nix ocaml (org +dragndrop +hugo +pandoc +gnuplot +jupyter +noter +present +roam) raku php (python +conda) rst (ruby +rails) rust scala sh swift (javascript +lsp) (web +html +css +lsp) yaml :app irc :os macos :config (default +bindings)
      packages     (solarized-theme) (humanoid-themes) (kaolin-themes) (spacemacs-theme) (apropospriate-theme) (darktooth-theme) (rebecca-theme) (solo-jazz-theme) (github-theme) (night-owl-theme) (colorless-themes) (leuven-theme) (base16-theme) (anti-zenburn-theme) (moe-theme) (poet-theme) (zaiste-theme :recipe (:host github :repo zaiste/zaiste-emacs-theme)) (tramp-theme) (load-theme-buffer-local :recipe (:build (:not native-compile compile))) (dired-quick-sort) (magit-section) (prism :recipe (:host github :repo alphapapa/prism.el)) (orgmdb :recipe (:host github :repo isamert/orgmdb.el)) (explain-pause-mode :recipe (:host github :repo lastquestion/explain-pause-mode)) (teco-screenshot :recipe (:host github :repo tecosaur/screenshot)) (plz :recipe (:host github :repo alphapapa/plz.el)) (ement :recipe (:host github :repo alphapapa/ement.el)) (gnus) (ediprolog) (helm-company) (vundo :recipe (:host github :repo casouri/vundo)) (ov) (rainbow-mode :recipe (:host github :repo NightMachinary/rainbow-mode) :pin nil) (org-super-links :recipe (:repo toshism/org-super-links :type git :host github :branch develop)) (targets :recipe (:repo noctuid/targets.el :type git :host github)) (exato) (evil-terminal-cursor-changer) (mips-mode) (async-await) (org-sticky-header) (applescript-mode) (magit-vcsh) (smeargle) (org-web-tools) (helm-org-rifle) (zoom) (embark) (sudoku) (blacken) (elpy) (pyvenv) (lispyville) (google-translate) (move-text) (rg) (deadgrep) (git-link) (neuron-mode) (nov) (hungry-delete) (ob-tmux) (iscroll) (evil-tutor) (orderless) (consult) (julia-mode) (julia-repl) (eglot) (eglot-jl) (org-drill) (org-ql) (org-ref) (org-vcard) (org-books) (helm-org-ql) (toc-org) (org-fragtog) (nndiscourse) (company-try-hard) (vlf) (bug-hunter) (telega) (disk-usage) (dired+) (poporg) (memoize) (vimrc-mode) (eredis) (xterm-color) (company-quickhelp) (company-quickhelp-terminal) (company-box) (eldoc-box)
      unpin        hl-todo company-mode helm-company rainbow-mode julia-mode julia-repl eglot-jl company-box

I don't know how to make doom build packages for it.

Set the EMACS envvar:
EMACS=/path/to/old/emacs doom sync

cl-assertion-failed: ((> (point) output-start))

As it turns out, this is an error caused by a recent breaking change on Emacs 28. Either downgrade Emacs or upgrade it (it's been fixed upstream).

Thanks. I will close the issue then.


Confusing and verbose README.

This looks like a great project for Typst, and I would love to help out. As discussed in #30 , the gifs in the README are largely error messages, but in general I think that plain text is superior to gifts, for ease of copying and file size. This project would greatly benefit from a cleaned up README with fewer gifs and more coherent writing. I'm a native English speaker and can help out with that if you would like!

First of all, thank you!
Yes, on #30 they mentioned gifs are wrong because I don't have prioritize the gifs.
But I think you right, after all, plain text can be less of a pain to use and to read.
As a non-native English speaker, it takes a bit of time to make sure that my english is easily understood by non-developers and is accurate enough. If you want to translate every gif and make a proper README, I'll glad to have you on it!
If you need any explanation, don't hesitate to ask questions, it will helps to make this project a better tool!
Thank you for your time, any help is always a pleasure <3

Sick I'll hop on this ASAP!

Since #32 is merged, I'll close it for now.
If there is something to add, don't hesitate!


Is there a Way to "Auto-Hide" a keyboard?

I was wondering if there is a way to auto hide the keyboard when the user press a key, because I can't hide it until the next message and I think that autohide would seem more practical for some simple selection keyboards that I'm using.

I don't think you can "autohide" a custom keyboard the way as you described. You can only do it "in the next message", as you said. If you really want this feature, you may talk to Telegram about it.

Well, I asked this because I used this bot, and it does that.
Here's a gif showing.

Bot: Language is set up
Bot: Hello, Alejandro!

And your question is: The custom keyboard is still there! How do I hide it?
Is my understanding correct?

Ahmm I don't get it, But, what I was trying to show is that it doesn't need other command to hide the keyboard, when I press a key it hides automatically.

Good. Then your problem is solved!

No, my question isnt answered, this bot is just an example of bot that hides the custom keyboard when the user press a key.
I want to do that in my bot without sending another message to hide the keyboard, and your answer was "Telegram doesn't have that implemented yet", but this bot does exactly what I want, don't know how.

Sorry, I was wrong and I misunderstood your question. There is a way to do it. You supply the one_time_keyboard parameter as True, as detailed in Bot API docs.
There is a lot more to be known besides what is in my examples. They are only simple usages that get you up to speed. You should read Bot API's documentations to fully exploit its power.

I see, so it was there all the time, I should have started there, thanks, works flawlessly.


Add too responsibility

Due to inactivity surrounding this topic I am closing it. If this is not solved please reopen the issue.


[GitHub Actions] "invalid response data" for pending jobs

Are you experiencing an issue with...
shields.io
🐞 Description
After switching to the new route for GitHub Actions workflows, as described in https://github.com/badges/shields/issues/8671 , the badge shows "invalid response data" when the workflow is running, rather than "running" or "pending" or a more informative status.
🔗 Link to the badge
https://img.shields.io/github/actions/workflow/status/kevinoid/example-slow-workflow/ci.yml?branch=main
💡 Possible Solution
No response

PR up for review from another maintainer https://github.com/badges/shields/pull/8717


Is deepspeed-mii meant to run in a notebook?

I saw this open issue here - https://github.com/microsoft/DeepSpeed-MII/issues/100
I can confirm I'm seeing the same behavior when running inside a notebook. In its current state, MII wasn't designed with the intention of running it in a notebook
However, I became aware of this project because of suggestion on an issue here on deepspeed main - https://github.com/microsoft/DeepSpeed/issues/4294
As I suggested before, [DeepSpeed-MII](https://github.com/microsoft/DeepSpeed-mii) may be exactly what you are looking for. With DeepSpeed-MII, you can create models sharded across multiple GPUs and do inference all from a single process. This library takes care of the hard bits involved with managing multiple processes. Here's a sample of what you would run from a single process (e.g., a jupyter notebook):
?

I ran the example on a kaggle.com (p100) notebook from that issue but I get this error:

RuntimeError                              Traceback (most recent call last)
Cell In[4], line 10
      8 # Run inference
      9 generator = mii.mii_query_handle("my_mii_deployment")
---> 10 result = generator.query({"query": ["The meaning of life is"]})
     11 print(result.response)
     13 # Shut down inference server

File /opt/conda/lib/python3.10/site-packages/mii/client.py:74, in MIIClient.query(self, request_dict, **query_kwargs)
     73 def query(self, request_dict, **query_kwargs):
---> 74     return self.asyncio_loop.run_until_complete(
     75         self._request_async_response(request_dict,
     76                                      **query_kwargs))

File /opt/conda/lib/python3.10/asyncio/base_events.py:625, in BaseEventLoop.run_until_complete(self, future)
    614 """Run until the Future is done.
    615
    616 If the argument is a coroutine, it is wrapped in a Task.
   (...)
    622 Return the Future's result, or raise its exception.
    623 """
    624 self._check_closed()
--> 625 self._check_running()
    627 new_task = not futures.isfuture(future)
    628 future = tasks.ensure_future(future, loop=self)

File /opt/conda/lib/python3.10/asyncio/base_events.py:584, in BaseEventLoop._check_running(self)
    582 def _check_running(self):
    583     if self.is_running():
--> 584         raise RuntimeError('This event loop is already running')
    585     if events._get_running_loop() is not None:
    586         raise RuntimeError(
    587             'Cannot run the event loop while another loop is running')

RuntimeError: This event loop is already running

Hi @qrdlgit, the suggested solution in #100 still works for me. Can you try again after restarting the Jupyter kernel and making sure that no background mii processes are still alive from a previous deployment?
Unfortunately, we do not currently have the bandwidth to further investigate a fix for running in a Jupyter Notebook, but we are happy to review contributions if you would like to create a PR!


404 on readme link - How to upgrade a Couchbase cluster

Wow, thanks for reporting. Somehow, we screwed up all the links in the root README. https://github.com/gruntwork-io/terraform-aws-couchbase/pull/80


"drop up" style dropdown menus

I didn't see any examples of this at http://twitter.github.com/bootstrap/ but I have a need for a "drop up" style drop down menu. I have a fixed footer that needs a drop down menu. Is this possible with the current version of twitter bootstrap? If not, can this be added? Or can you lead me to the place where I can do this myself and submit a pull request?

Instead of top: 100% you can use bottom: 100% to make the dropdowns, or in this case "dropups", appear above the element.

While that's true, I'd like to see Bootstrap adopt a auto-repositioning system like qTip that would dynamically move the overlay when its default position would leave it partially off the page.
This would fix this issue and provide additional functionality.

I've added the CSS to handle this with the .bottom-up class (added to .dropdown-menu) and at some point @fat will hook us up with a javascript plugin.

so is this available in version 2.0, or should I pull from master?

@jessor small fix to apply on -only bottom-up menus not all of them-
.navbar .dropdown-menu.bottom-up::after {
    border-bottom: 0px solid transparent !important;
    border-top: 6px solid white;
    top: auto !important;
    bottom: -6px;
}


need a pagination component

we need to extract out the pagination logic in frontend to an components.

working on sorting


Fix custom mysql.user creation

This PR fixes issue #181.
Due to recent changes (commit d8524b8), there exist two entries for root.
One for host %  (root@'%')
One for host localhost. (root@localhost)
As a result, the prior UPDATE statement to change any host "%" to "localhost" fails, as it conflicts with the already existing entry root@localhost. To 'block remote access' for root - the intended behavior according to the comment - we simply now delete the second entry for any host root@'%'.
Unrelated, the "IDENTIFIED BY" had to be removed from the "GRANT ALL" because it is invalid syntax for the recently updated Mysql version.

Hi thank you for this
ill merge it and also double check again, but good solution :)


Thoughts on Audio2Video

Hi there :)
I'm sure you're familiar with the recent Emo demo which takes image+audio and turns it to a consistent video (head) amazing quality
I've just heard a demo of Udio - Braking Bad The Musical
https://youtu.be/13M-FFkXVvs?t=5m53s
If you close your eyes you can practically see the Disney movie in your head
So in the future I see a pipeline of audio+text to video, where the vid is consistent with with audio and the text describe the scene, characters, and their actions and motions
After that there will be a Boximator style editing or rerendering parts of the video using text prompt or maybe even a video reference.

Thanks for your suggestion. Currently, audio support is not one of our priorities. We hope to reach Sora-level video generation quality first.

Hey yo ! 👋🏾
How's it coming along ?
All the best!


feat: weekday implemented

This PR closes #131

@all-contributors please add @OokanNumber1  for code


fix: added weekly goal to the goals model. This closes issue #336.

The branch requires Admin privilege to resolve conflict

Hey @bryanwaine please pull from the staging branch, resolve the conflicts, and then make another PR.

Hey man. I'm still getting conflicts here. Can you please try and resolve them on your terminal? Please make sure the code on your local machine is up-to-date with the code on the staging branch.
And please change "This closes issue #336" to "Closes #336"


Add option to only load a node instead of the whole body

This adds the "selector" option. When set, Snap will not replace the
whole document body with the new loaded body, but rather only the first
node that matches the selector (with document.querySelector) with the
matching node of the loaded page.
This makes it possible to easily persist some elements (i.e. chat
widgets that are added to the DOM by external libraries) across
navigation.
(I haven't tried to run the tests and haven't updated the doc, but it seems to work quite well for me so far, so I thought I'd share 😉)

Thanks a lot for the pull request! I would consider to rename selector to a more specific name like rootSelector, root or a similar one. What’s your opinion on this?

Yep, it couldn't hurt to be a little more explicit. IMO rootSelector (or maybe containerSelector?) would be the easiest to understand. I'll let you decide :)

rootSelector sounds good to me.

Commit updated!


Cannot read properties of undefined (reading 'attributes')

After upgrading both, integration and the card, enabling events I've got this error.
Here's yaml
type: custom:octopus-energy-rates-card
cols: 2
currentEntity: event.octopus_energy_electricity_(xxxx)current_day_rates
futureEntity: event.octopus_energy_electricity(xxx)_next_day_rates
showday: true
showpast: false
hour12: false
highlimit: 28.1
unitstr: p
roundUnits: 1
title: 🔌 Rates (normal = 28.3p)
Github slightly formatted the underline in the entity names, in original it's correct.

same
type: custom:octopus-energy-rates-card
currentEntity: sensor.octopus_energy_electricity_22j0006288_1591022098690_current_rate
cols: 2
showday: true
showpast: false

same
type: custom:octopus-energy-rates-card
currentEntity: sensor.octopus_energy_electricity_xxxxxx_xxxxx_current_rate
cols: 2
showday: true
showpast: false

Switch entity to event.xxx as per readme (activate them if not active)

don't have such entities

do not have such entities,

Did you upgrade the octopus integration (not card) to version 0.9.x? Please read the readme, there is clear instruction.
My case is different as I have these events and card is configured. Still throws an error

Did you upgrade the octopus integration (not card) to version 0.9.x? Please read the readme, there is clear instruction.
My case is different as I have these events and card is configured. Still throws an error
yes been running 9.0.x for a while now.

Have you enabled those event entities? That message is usually shown when they're not enabled.
…
On Sat, 25 Nov 2023 at 13:15, Andrew Blackburn @.> wrote: Did you upgrade the octopus integration (not card) to version 0.9.x? Please read the readme, there is clear instruction. My case is different as I have these events and card is configured. Still throws an error yes been running 9.0.x for a while now. — Reply to this email directly, view it on GitHub <#36 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AQ3KT4IJ3IJQEBRUORUABLDYGHVOTAVCNFSM6AAAAAA72CFPTKVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMRWGMYDKMRXGU . You are receiving this because you are subscribed to this thread.Message ID: @.>

I suspect the issue is the current version of the card expects past/future entities to be provided even though it doesn't require them. I've just submitted a fix https://github.com/lozzd/octopus-energy-rates-card/pull/37 to ensure they are optional. To fix in the mean time you can just provide them in the config.

I suspect the issue is the current version of the card expects past/future entities to be provided even though it doesn't require them. I've just submitted a fix #37 to ensure they are optional. To fix in the mean time you can just provide them in the config.

 Didn't cross my mind I have to provide all three events as the documentation table says optional for previous and next.
Now working. Thank you very much!
So, in order to have it working you need to specify all three, currentEntity, pastEntity and futureEntity - info for people struggling with it.

#37 merged, will cut a release


feat: no sqlexecutor split on semicolon

https://stackoverflow.com/questions/36938399/how-to-pass-multiple-statements-into-spark-sql-hivecontext
https://thomasdecaux.medium.com/multi-statement-with-spark-sql-fc9996724576
http://mark.thegrovers.ca/tech-blog/spark-sql-gotchas
this will probably not work.


[lua] cmake link library is absolute path

[ ] Changes comply with the maintainer guide
[ ] SHA512s are updated for each updated download
[ ] The "supports" clause reflects platforms that may be fixed by this new version
[ ] Any fixed CI baseline entries are removed from that file.
[ ] Any patches that are no longer applied are deleted from the port's directory.
[ ] The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
[ ] Only one version is added to each modified port's versions file.

Call the cmake_minimum_required() command at the beginning of the top-level CMakeLists.txt file even before calling the project() command.
see https://cmake.org/cmake/help/latest/command/cmake_minimum_required.html

The usage and feature has been tested successfully locally.
find_package(Lua REQUIRED)
target_include_directories(main PRIVATE ${LUA_INCLUDE_DIR})
target_link_libraries(main PRIVATE ${LUA_LIBRARIES})

Thanks!


incorrect monitoring data

why the data may be incorrect, and how can this be corrected?
Rancher Server Setup

Rancher version: v2.6.3-patch1
Оnly service pods installed

Information in UI rancher

Actually on the server

kubectl describe node <>

it turns out, no matter what the load in the cluster is, it definitely cannot even reach 8GB, not to mention large amounts of RAM

added monitoring, now the picture is like this

I think this is a duplicate of https://github.com/rancher/rancher/issues/35986

@WolfspiritM
no, it's a different situation.


fix (facebook): obtain profile picture from user profile

(these pull requests are done to maintain ally 4.1... someone is still using this probably)

Proposed changes
Profile picture via Facebook graph API is no longer possible. Profile picture is actually already returned when we get user profile from Facebook, under picture.data.url
Types of changes
What types of changes does your code introduce?
Put an x in the boxes that apply

[x] Bugfix (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 not work as expected)

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING doc
[x] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] I have added necessary documentation (if appropriate)

Further comments
If this is a relatively large or complex change, kick off the discussion by explaining why you chose the solution you did and what alternatives you considered, etc...

Thanks 👍


Wrong link to document

Link to export log file  document in [Multi-Geo}(https://learn.microsoft.com/en-us/viva/learning/configure-sharepoint-content-source#multi-geo) section leads to  wrong documentation

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

ID: 3055d301-6a49-ab29-5fd0-2bdf1b4076d6
Version Independent ID: 3055d301-6a49-ab29-5fd0-2bdf1b4076d6
Content: Add SharePoint as a learning content source for Microsoft Viva Learning
Content Source: Viva/learning/configure-sharepoint-content-source.md
Service: viva
Sub-service: viva-learning
GitHub Login: @bhaswatic
Microsoft Alias: bhaswatic

Hello @urban-karolis thanks, I'm working on this


DST-439 - [Design bug] Give a summary of the breach

Before:

After:

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 80.32%. Comparing base (d296037) to head (f8792bc).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #119      +/-   ##
==========================================
- Coverage   80.34%   80.32%   -0.03%
==========================================
  Files          63       63
  Lines        1954     1952       -2
==========================================
- Hits         1570     1568       -2
  Misses        384      384

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


Fix lint.

Not sure why go vet is passing on travis, but it complains about unkeyed fields and fails locally for me.

Current coverage is 68.89% (diff: 100%)

Merging #250 into master will not change coverage

@@             master       #250   diff @@
==========================================
  Files             4          4
  Lines          1135       1135
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits            782        782
  Misses          263        263
  Partials         90         90

Powered by Codecov. Last update b60ada7...352f58d

@jmcarp Thanks for the PR!
go vet is passing on my machine, too. Excuse me but would you mind telling me how you're running go vet?
FYI how it's running on travis can be seen at https://github.com/coreos/kube-aws/blob/master/make/test#L19

Sure, I'm running it using make test-with-cover, like travis. Here's what I get without this patch:
config/config.go:655: github.com/coreos/kube-aws/coreos/userdatavalidation.Entry composite literal uses unkeyed fields
config/config.go:656: github.com/coreos/kube-aws/coreos/userdatavalidation.Entry composite literal uses unkeyed fields
config/config.go:657: github.com/coreos/kube-aws/coreos/userdatavalidation.Entry composite literal uses unkeyed fields
exit status 1
nodepool/config/config.go:79: github.com/coreos/kube-aws/coreos/userdatavalidation.Entry composite literal uses unkeyed fields
exit status 1

I'm running go1.7.4. I think @icereval ran into the same issue locally.

FWIW, the error does not appear when running on the docker golang container (debian jessie), thus Travis' success.

I've also tried both the official golang 1.7.4 build and the brew install on OSX with the same result.

@jmcarp @icereval I could reproduce the issue on my mac after brew update && brew upgrade golang with golang 1.7.4 but still having no luck on travis even after upgrading it to use 1.7.x:
https://github.com/mumoshu/kube-aws/commit/de6560112b3027f573841dbca2b7b4bb24db4824
https://travis-ci.org/mumoshu/kube-aws/builds/192210197

Ah, never mind. What I was seeing were errors which are not from the current kube-aws itself but from my poc for kube-aws inside the same directory. I'm still unable to reproduce the issue:
$ go version
go version go1.7.4 darwin/amd64
$ go vet -v $(go list ./... | grep -v '/vendor/')
$ echo $?
0

Agreed and its strange, I too built specifically 1.7.4 on travis earlier and narrowed it down just to go vet and it results in 0 errors in the test environment.
I can work around this issue for now so its not a big deal.
Out of curiosity whats your 1.7.4 go sha256? mine is:
$ go version
go version go1.7.4 darwin/amd64
$ shasum -a 256 -b `which go`
e19967912c02675d44806c8bbdd681a6218844fcb7f6977a72dffdaf710c2e71 */usr/local/bin/go

Regarding the sha256, mine is:
$ go version
go version go1.7.4 darwin/amd64
$ shasum -a 256 -b `which go`
9118739c7497ffe86a4ba73867937030d011004f1f804ce260823b129241eabd */usr/local/bin/go

@jmcarp @icereval I'd rather like to merge this once the go vet errors are confirmed to be valid(which seems true to me).
Honestly I'm not very familiar with go vet but the composites check of go vet which seems to be emitting the errors should be enabled by default in go vet. Am I correct?

I'm merging this anyways! Thanks.


micronaut: 1.3.2->1.3.4

Motivation for this change
Updated to newer version and fixed issue when run with java 11 on PATH.
Things done

[ ] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

I updated this to 1.3.4 instead of 1.3.3. I also added the correct dependencies the script uses.

1.3.6 released: https://github.com/micronaut-projects/micronaut-core/releases/tag/v1.3.6


Generating docs when source and notebooks are in different folders

My project setup looks like this:
docs/
notebooks/
  - index.ipynb
  - train.ipynb
  - model.ipynb
src/
  project/
    - model.py

settings.ini
nbs_path = notebooks
doc_path = docs
doc_baseurl = /project/
lib_path = src/project

when executing nbdev_build_docs I keep getting the error for each notebook:
converting: /...../project_root/notebooks/train.ipynb
'/...../project_root/notebooks/train.ipynb' does not start with `/..../project_root/src'

while nbdev_build_lib works as expected.
What is wrong with my setup?

Solved by adding symlink to source folder as such ln -s src/project project.
Maybe there is another way to avoid this?

Not working on windows using mklink /D project src\project, where syntax is mklink /D  LINK TARGET. Any idea?

Best to use the forums to ask for help. forums.fast.ai


[Question] Can I run a transaction for multiple databases or multiple indexes in one database?

First of all, thank you for creating and sharing a great software!
I think it would be nice if I could run a transaction for multiple databases or I could have multiple in one database.
Is that possible?
I have one workaround in my mind. That is to use a composite key.

The first element: the key type.
The second or later elements: key values.

An example key type :

1: The data for the "tableA"
2: An index of certain columns on "tableA"
3: Another index of other columns on "tableA"

And then I encode the key type and the key value to a byte array.
Also I define a custom BasicComparer struct which decodes a byte array to
a key type and key value and compare two set of them appropriately.
Could you tell me what do you think?
Thanks!

I found CoackroachDB uses a naming rule for keys to save multiple kind of data in one key namespace.
https://github.com/cockroachdb/cockroach/blob/master/docs/design.md#logical-map-content
So one key namespace should be enough in one database.
I'm closing this now.


{} behaves like 1, END{} doesn't exhaust input

See:
$ seq 3 | goawk '{}'
1
2
3
$ seq 3 | { goawk 'END{}'; wc -l;}
3

According to POSIX the first command shouldn't print anything, and the second should print 0.

Ah, that's an interesting bug -- I'm surprised I hadn't noticed it before. Thanks for the report! I'll take a look in the next couple of weeks.

Fixed -- thanks for the report.


antlr4-python3-runtime is being installed using the legacy 'setup.py install'

Describe the bug
Installing azure-cli using pip outputs Warning about using outdated installation method for one of the components - antlr4-python3-runtime.
And it is going to completely fail with the future pip versions.
Python 3.10.12
# default pip version installed in the virtualenv (pyenv+direnv)
pip 23.0.1

pip install -U azure-cli
...
DEPRECATION: antlr4-python3-runtime is being installed using the legacy 'setup.py install' method, because it does not have a 'pyproject.toml' and the 'wheel' package is not installed. pip 23.1 will enforce this behaviour change. A possible replacement is to enable the '--use-pep517' option. Discussion can be found at https://github.com/pypa/pip/issues/8559
Running setup.py install for antlr4-python3-runtime ... done
...
[notice] A new release of pip is available: 23.0.1 -> 23.2.1
[notice] To update, run: pip install --upgrade pip

Related command
pip install -U azure-cli==2.50.0
Errors
DEPRECATION: antlr4-python3-runtime is being installed using the legacy 'setup.py install' method, because it does not have a 'pyproject.toml' and the 'wheel' package is not installed. pip 23.1 will enforce this behaviour change. A possible replacement is to enable the '--use-pep517' option. Discussion can be found at https://github.com/pypa/pip/issues/8559
Issue script & Debug output
N/A
Expected behavior
Clean installation without using deprecated ancient methods and techniques.
Environment Summary
azure-cli                         2.50.0
core                              2.50.0
telemetry                          1.0.8
Dependencies:
msal                              1.22.0
azure-mgmt-resource             23.1.0b2
Python location '/Users/den/projects/azure/.direnv/python-3.10/bin/python'
Extensions directory '/Users/den/.azure/cliextensions'
Python (Darwin) 3.10.12 (main, Jul 22 2023, 21:22:22) [Clang 14.0.0 (clang-14<IP_ADDRESS>)]
Additional context
No response

Thank you for opening this issue, we will look into it.

I believe bump antlr4 is the right way to solve it.

Currently, it does not affect CLI.
When pip is 23.2.1, here is the installation log:
pip install antlr4-python3-runtime==4.9.3 --no-cache-dir
Collecting antlr4-python3-runtime==4.9.3
  Downloading antlr4-python3-runtime-4.9.3.tar.gz (117 kB)
     ━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━ 117.0/117.0 kB 1.3 MB/s eta 0:00:00
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
  Preparing metadata (pyproject.toml) ... done
Building wheels for collected packages: antlr4-python3-runtime
  Building wheel for antlr4-python3-runtime (pyproject.toml) ... done
  Created wheel for antlr4-python3-runtime: filename=antlr4_python3_runtime-4.9.3-py3-none-any.whl size=144552 sha256=026a0c30aaef6e3957942e670b1bc201a92fe96fcb89d22216bccd190c0ec732
  Stored in directory: /tmp/pip-ephem-wheel-cache-24cvxjof/wheels/12/93/dd/1f6a127edc45659556564c5730f6d4e300888f4bca2d4c5a88
Successfully built antlr4-python3-runtime
Installing collected packages: antlr4-python3-runtime
Successfully installed antlr4-python3-runtime-4.9.3

pip list
Package                Version
---------------------- -------
antlr4-python3-runtime 4.9.3
pip                    23.2.1
setuptools             59.6.0


TS2339: Property 'div' does not exist on type '(tag: any) => any'.

Version
styled-components: 3.0.2
typescript: 2.6.2
ts-loader: 3.3.0
I am gradually changing the JavaScript based code to TypeScript based.
I refer to this repository.
using  both babel-loader and ts-loader
- webpack config
module.exports = config = {
  entry: ['babel-polyfill', './src/index'],
  module: {
    rules: [
      {
        test: /\.(t|j)s(x?)$/,
        exclude: /node_modules/,
        use: [
          {
            loader: 'babel-loader',
          },
          {
            loader: 'ts-loader',
          },
        ],
      },
      {
        enforce: 'pre',
        test: /\.js$/,
        loader: 'source-map-loader',
      },
    ],
  },
  plugins: [],
};

interface RootProps {
  className?: string;
  isFixed?: boolean;
  isOpen?: boolean;
}

does not work

const StyledRoot = styled.div`
  position: ${(props: RootProps) => (props.isFixed ? 'fixed' : 'static')};
  top: 0;
  right: 0;
  bottom: 0;
  left: 0;
  z-index: 9999;
  ${(props: RootProps) => (props.isOpen ? '' : 'pointer-events: none')};
`;

it worked

const StyledRoot = styled('div')`
  position: ${(props: RootProps) => (props.isFixed ? 'fixed' : 'static')};
  top: 0;
  right: 0;
  bottom: 0;
  left: 0;
  z-index: 9999;
  ${(props: RootProps) => (props.isOpen ? '' : 'pointer-events: none')};
`;

i assume also import seems to weired it says
[ts] cannot find module 'styled-components';

tsconfig.json

{
  "compilerOptions": {
    "target": "es5",
    "module": "es6",
    "allowJs": true,
    "jsx": "react",
    "sourceMap": true,
    "outDir": "./dist/",
    "strictNullChecks": true
  },
  "include": ["./src/"]
}

ADDITIONAL QUESTION
when i do import { StyledFunction } from 'styled-components';

it has [tsl] error
TS2305: Module '".../node_modules/styled-components/dist/styled-components.es"' has no exported member 'StyledFunction'.
i assuem ts-loader cause this error but i can't sure

resolved by yarn add @types/styled-components

for people with npm:
npm i @types/styled-components

Hi. I had @types/styled-components installed but for me a was a wrong tsconfig.json setup.. I had
"module": "NodeNext", "moduleResolution": "NodeNext",
configured in my compilerOptions. Changing it to
  "module": "es2020",     "moduleResolution": "node",
But I don't know why that is. Could anyone explain?
THX!


Can't import classes from com.android.internal

Hello, I added your jar for 23 API, added dependecies and repository, tried to rebuild the project and get error:
Error:Execution failed for task ':app:mockableAndroidJar'.

java.lang.NullPointerException (no error message)
Also classes from com.android.internal AS can't see, can you help to fix this?

I am also encountering this issue on Android Studio 2.2.2.  Whenever I try any clean or build operation this same error occurs.  I am trying to use the 19 API.

Please use Hidden API v24 to fix this problem.


Deduce default currency based on location

At the moment the default currency is hard-coded to EUR. Instead, the currency should be deduced from the users rough location (via IP), browser locale setting etc.

Get user's country from http://ipinfo.io/:
$.getJSON('http://ipinfo.io', function(data){
  console.log(data)
})

Then look up currency here: http://country.io/currency.json


ImportError: No module named backend_dist.manage when using Flask-Script

Hi Everyone,
Thanks for contributing this project. It is extremely useful.
I'm struggling to get it working with Flask-Script however. Running

python manage.py rq

lists all the commands correctly.
However, as soon as I attempt to start a worker with:

python manage.py rq worker

I am hit with:
Traceback (most recent call last):
  File "manage.py", line 23, in <module>
    manager.run()
  File "/Users/name/Documents/Development/venv/lib/python2.7/site-packages/flask_script/__init__.py", line 412, in run
    result = self.handle(sys.argv[0], sys.argv[1:])
  File "/Users/name/Documents/Development/venv/lib/python2.7/site-packages/flask_script/__init__.py", line 383, in handle
    res = handle(*args, **config)
  File "/Users/name/Documents/Development/venv/lib/python2.7/site-packages/flask_rq2/script.py", line 42, in __call__
    return self.run(*args, **kwargs)
  File "/Users/name/Documents/Development/venv/lib/python2.7/site-packages/flask_rq2/script.py", line 179, in run
    queues=queues or self.rq.queues,
  File "/Users/name/Documents/Development/venv/lib/python2.7/site-packages/rq/cli/cli.py", line 175, in worker
    worker_class = import_attribute(worker_class)
  File "/Users/name/Documents/Development/venv/lib/python2.7/site-packages/rq/utils.py", line 151, in import_attribute
    module = importlib.import_module(module_name)
  File "/usr/local/Cellar/python/2.7.8_2/Frameworks/Python.framework/Versions/2.7/lib/python2.7/importlib/__init__.py", line 37, in import_module
    __import__(name)
ImportError: No module named backend_dist.manage

Looking through flask_rq2.app.py I noticed a module_path is being set on line 189.

self.module_path = 'flask_rq2.backend_%s' % app.name

I do have an application in dist.manage but I suspect flask_rq2 may be looking for it in the wrong place. My directory structure is:
manage.py
dist
- manage
-- __init__.py
-- *.py

Any help much appreciated.
(I know this probably isn't a bug, so this may not be the best way of asking for help, but thanks anyway!)

Ah I see, there are two things going on here:

your app probably uses __name__ as the first parameter when it's instantiated so it's internally dist.manage. This isn't ideal since it means Flask and some extensions won't be able to use the name of the application package as a prefix to find things, e.g. in a traceback. I encourage you to read the "About the First Parameter" section on http://flask.pocoo.org/docs/0.11/api/#application-object for some background.

Flask-RQ2 stupidly assumes that app names don't contain dots, which is a bug you've seen. I'll fix it here, but as a work around you should be able to set the app's name (in your dist.manage package) to just dist for example.

Thanks for the report!

Thank you. I've read up on the __name__ parameter and can now see what was happening. Thanks for the help. Really appreciate you taking the time to write this module and respond.


fix(usrObj): Fixing typo of usbObj to usrObj

:tada: This PR is included in version 4.15.6 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Release 5.8.0 to Main

This is the release from 5.8.0 to Main

Since Mubarak is on vacation, this is a non reviewed merge.


[IMP] Add several remote converters

@rosell-dk this is a low-priority request feature.
Maybe it would be nice to add several remote webp converters to have high availability in case something goes wrong with one convert cloud (or website) and you don't have the posibility to use imagemagick or any other type of conversion because of a cheap shared web hosting.
Thank you for the amazing work.

Yes, that makes sense.

I've noticed that the issue goes on webp-express repo. Sorry about that.


nodesConnectable prop doesn't seem to be reactive to value changes

What platform were you using when you found the bug?

Svelte Flow version: 0.0.40
Browser and version: Version 1.68.128 Chromium: 127.0.6533.73 (Official Build) (64-bit)
OS and version: Ubuntu 24.04

Live code example
https://stackblitz.com/edit/svelte-flow-ts-ppskg3?file=src%2FFlow%2Findex.svelte
Describe the Bug
In the Stackblitz above, elementsSelectable={!isLocked} updates reactively, meaning that nodes are correctly not selectable if canvas is locked.
nodesConnectable={!isLocked} on the other hand does not seem to update reactively and nodes are still connectable even when isLocked is true
Steps to reproduce the bug or issue
See Stackblitz for reproduction
Expected behavior
I would expect not to be able to draw any edges between nodes whenever nodesConnectable evaluates to false.
Screenshots or Videos
No response
Additional context
No response

This should be fixed in the newest update

Thanks, already using 0.1.15 which now works perfectly for our use case!


Pattern: logo grid: design specs

Wonil-Suh1 created the following on Sep 08:
User story
As an IBM.com user, I need to be able to see what other companies are using the product/solution/offering presented on the page so that I can make a better decision.
Notes
Here is an example of the band used in the past:

Additional Information

Logo Grid Design Study # 405
Logo Grid Visual Design # 404

Acceptance criteria

[ ] Design specs are available

Original issue: https://github.ibm.com/webstandards/digital-design/issues/1621

We have this coming in from the IBM.com reboot team. Let's close this story.


New test Class for Label Component

This PR adds new test cases class for Label component.
Label in App Inventor is a TextView in actual android.
Instead of testing the label class setter method with the label class getter method, we can test it with Android's getter method, as the label class setter method also sets the instance variable of the label class which can retrieved by getter methods. In such circumstances almost all the test cases are passed and we do not get the correct result.
When testing, we also have to check whether the value actually changes or whether the value is returned by the instance variable of the labeled class.
Thank you
Preet Vadaliya

Can one of the admins verify this patch?

@AppInventorWorkerBee ok to test

@SusanRatiLane thank you for your compliment.
As per your suggestion I added 2 new test cases for setHeightPercent and setWidthPercent.
But I am still exploring in HTMLFormat.

@SusanRatiLane
I am a bit confused as to what you are pointing me for changes, but let me clarify what I have done,
whenever the test case started executing the SetUp() method will be called and it will create a new Label into Form and textView is cast from the newly created Label so I think we not required to test whether the textview is correct or not.

@SusanRatiLane Thanks for your Suggestions.
I just made some changes according to your suggestion if these test cases are okay then I will update my other PRs according to this.


A11y_VSCode_Output panel_KeyBoard:Tooltip not visible when keyboard focus moves to the clear output, toggle scroll lock, close button

Environment Details:
VSCode Version :
OS Version : Mac OS
Additional Details:
MAS Violated : MAS 1.1.1, MAS 2.1.1
Repro Steps:
Open Visual studio code editor
Press ctrl+shift+u
Navigate to the clear output, toggle scroll lock, close button using keyboard
Observe the tool tip is visible or not when the focus is on the button.
Actual:
Tool tip is not visible when focus moves to the clear output, toggle scroll lock, close button.
Expected:
Tool tip should be visible when focus moves to the keyboard
Recommendations:
Refer below link which is repository of bug fixes code snippets:
https://microsoft.sharepoint.com/teams/msenable/mas/pages/browse-fixes.aspx
User Impact:
Keyboard users will face dificulty to understand the control.
MAS Reference:
MAS 1.1.1 - https://microsoft.sharepoint.com/teams/msenable/_layouts/15/WopiFrame.aspx?sourcedoc={d2d2051f-bdc8-4af7-8e18-38aae867e216}
MAS 2.1.1 - https://microsoft.sharepoint.com/teams/msenable/_layouts/15/WopiFrame.aspx?sourcedoc={8492c4eb-c179-40ae-8777-cd044ed725a2}
Attachment For Reference:
Mac_A11y_VSCode_Output panel_ScreenReader_ Tooltip not visible.pptx
Does this issue occur when all extensions are disabled?: Yes

Duplicate of https://github.com/microsoft/vscode/issues/52016


Improve color/style of h4 headings

Hi there,
@proddata suggested at https://github.com/crate/crate-docs-theme/pull/248#issuecomment-820234534:

I think making the h4-headings a bit darker would improve the readability quite a bit.

With kind regards,
Andreas.

@belringer responded at https://github.com/crate/crate-docs-theme/pull/248#issuecomment-822394865:

I suggest color #000000 and additionally font-weight: bold;

I believe this has been implemented by 26a25ef6a already.


List of products

@JoaoMoraesJr Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

Looks like a test to me. Please use your own fork for that.


'children' is missing in props validation(react/prop-types) when using React.PropsWithChildren as of 7.25.0

When you have a typescript component like:
const MyComponent = (props: React.PropsWithChildren<{}>): React.ReactElement => {
	return <>{props.children}</>;
};

As of 7.25 you get the warning:

'children' is missing in props validation(react/prop-types)

React.PropsWithChildren is defined as:
type PropsWithChildren<P> = P & { children?: ReactNode | undefined };

If you change the Component props type to:
const MyComponent = (props: React.PropsWithChildren<{ children?: ReactNode | undefined }>): React.ReactElement => {
	return <>{props.children}</>;
};

The warning disappears.
I believe this is a mistake. The warning should not trigger in both cases.
It did not trigger in 7.24. It does as of 7.25.

cc @Wesitos

Not sure if this is a bug, children is indeed not defined right? @ljharb @FritsvanCampen

Will look into it

I'm seeing the same with React.FC when upgrading from 7.24.0 to 7.25.0

@johnhunter the PR fixes React.FC also
this should work
          import React from 'react';
          const Person: React.FC<{}> = (props): React.ReactElement => (
              <>{props.children}</>
          );

Fixed in #3064, which will be released soon.

Thanks for the quick fix!

I still see 'children' is missing in props validation  react/prop-types using React.FC with v7.25.1

@johnhunter you’ll see that for eternity, since that version doesn’t contain the fix.


⚠ Bofrost.fr has degraded performance

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

HTTP code: 200
Response time: 1547 ms

Resolved: Bofrost.fr performance has improved in 6bfec10 after 6 minutes.


small delete register behaviour different when using ctrl-R

Steps to reproduce
Consider below text
foo bar foobar

With cursor one the first character, I type ciw'<Ctrl-R>"', I get 'foo'. The value in register @. is 'foo'.
But when I type ciw'<Ctrl-R>-', I get the same 'foo'. But the value in register @. is '^R-'.
This different behavior allows to use the . command to repeat the change.
Is this intentional or a bug? I don't see this behavior documented. (I couldn't find it)
https://vimhelp.org/change.txt.html#quote-
https://vimhelp.org/insert.txt.html#i_CTRL-R
Expected behaviour
Documentation for the different behavior
Version of Vim
9.0.1736
Environment
arch linux
Logs and stack traces
No response

That is intentionally there since https://github.com/vim/vim/releases/tag/v8.2.2189
How about this doc patch then:
diff --git a/runtime/doc/insert.txt b/runtime/doc/insert.txt
index ccf5178a8..8696e912f 100644
--- a/runtime/doc/insert.txt
+++ b/runtime/doc/insert.txt
@@ -127,7 +127,11 @@ CTRL-R {register}                          *i_CTRL-R*
                        '/'     the last search pattern
                        ':'     the last command-line
                        '.'     the last inserted text
+                                                       *i_CTRL-R_-*
                        '-'     the last small (less than a line) delete
+                               register. This is repeatable using |.| since
+                               it remembers the register to put instead of
+                               the literal text to insert.
                                                        *i_CTRL-R_=*
                        '='     the expression register: you are prompted to
                                enter an expression (see |expression|)

Thank you for clarifying. doc patch looks good.


Add  functions and testing to Decimal64

Closes: #235
Closes: #236

@ckormanyos this adds testing of almost all the cmath functions for decimal64. The 32-bit platforms are slow as expected but it looks like we're in a good spot. Also brings coverage back above 99%.


nixos/opengl: fix wrong function application

No idea how that one slipped through. Thanks for catching @maxeaubrey!
Description of changes

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Oh god.

Thanks, much appreciated!


Automatic expand/collapse when mouse hovers on the menu bar.

What's this PR do?
Previously, hidden bar automatically collapse even when the user is using the menu bar, which is interruptive. This PR solves this issue by only collapsing the menu bar when no status bar windows are detected.
What are the relevant tickets?
#66 [Request]Don't hide icon when using
Any background context you want to provide? (if appropriate)
For some reason, status bar window usually has kCGWindowLayer bigger than 100. I have only tested this on Big Sur, so I'm not sure wether or not it works on previous (or future) versions of macOS.
There are several issues with this trick:

Status bar menus (top right) and app menu (top left) both have kCGWindowLayer > 100, so the app also disables auto collapse when the user is using the app menu.
Some apps have custom menu UI window, such as VMWare Fusion, which cannot be identified using this trick.

@p-z-l I'm not sure is this the correct way to detect Menu is open or not. I hope some one have experience on this give a feedback.

After reading your comment on #99, I think a better solution is to:

Expand when the user hovers the mouse onto the menu bar
Collapse if the user:

moves the mouse outside the menu bar without interacting with it
interacted with the menu bar and moves the mouse outside for a certain amount of time

I'm think there is no way to detect hover because when mouse enter we collapse the long button then first response will be other app menu items which is out of our control.

NSEvent.addGlobalMonitorForEvents allows calling a function when the user moved the mouse, even if the app is not the first responder
You can get the menu bar size by CGWindowListCopyWindowInfo, similar to what I did in Util.menuBarIsInUse;
You can also get the mouse location by NSEvent.mouseLocation;
Hence, there is a way to check mouse hover on menu bar, I'm just not sure about the performance impact.

If we can read mouse position anytime we need and the Screen mouse current on then the check only every time we prepare to collapse. This won't impact performance.

If we get screen mouse currently on (user may use many monitor) we can compare mouse position to top left of screen. The status bar height if constant, don't need to lookup it in List of windows.

Keep in mine, this check only using to prevented collapse trigger, not using to trigger expand (unhide). The even using to trigger expand is a view embedded in the long button using to detect mouse enter event.

Added automatic expand when hover on menu bar, the performance impact seems to be very minimal (I played some heavy 3D games for testing, didn't notice any lag).
Also, if the user disables the feature, it won't impact performance at all.

Do whatever convenient for you my friend.


Is it possible to create a 'step line chart' with echarts?

问题简述 (One-line summary)
Is it possible to create a 'step line chart' with echarts?
版本及环境 (Version & Environment)

ECharts 版本 (ECharts version):
浏览器类型和版本 (Browser version):
操作系统类型和版本 (OS Version):

重现步骤 (Steps to reproduce)
期望结果 (Expected behaviour)
I can't find any documentation or examples for a step line area chart that would look something like this:

可能哪里有问题 (What went wrong)
ECharts配置项 (ECharts option)

option = {

}

其他信息 (Other comments)

FYI http://echarts.baidu.com/demo.html#line-step

that linked example solves for if your x-axis is of type "category", what about if your x-axis is of type "value" and your series data looks something like this?
xAxis : [
        {
            type : 'value',
            scale: true
        }
    ],
    yAxis : [
        {
            type : 'value',
            scale: true
        }
    ],
series: {
            type:'line',
            itemStyle: {normal: {areaStyle: {type: 'default'}}},
            data: [
                [90,45],
                [92,40],
                [94,25],
                [97,20],
                [98,17],
                [98.5,13],
                [99,4],
                [99.5, 1],
                [100,0],
                [100.5, 1],
                [101, 3],
                [102,5],
                [102.5, 8],
                [103,18],
                [104,23],
                [105,29],
                [107,34],
                [108,39]
            ]
        }


Issue #422: Double instance of subnet manager running in IB cluster

Signed-off-by: sakshiarora13<EMAIL_ADDRESS>

@sakshiarora13 do we want to always disable SM on the switch, or would it make more sense to let the switch handle SM, and only use the control plane to run open_sm when the switch can't (i.e., unmanaged switch).

@lwilson the intention of running subnet manager on server is so that there would be enough CPU power to cope with things such as major topology changes. Running SM on switch would work fine in a small cluster when we have few switches only. So, to generalize the architecture we prefer running SM only on server.
Also, we are taking a boolean input: ib_switch_support in base_vars.yml Subnet manager is configured only when this variable is set to true. If user has unmanaged IB switch, we expect user to set this variable to false and omnia won't configure anything for IB in order to honor default behavior.


fix: Use DB credentials for Redis Streams MesssageBus connection

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-examples/blob/master/.github/CONTRIBUTING.md.
What is the current behavior?
Can not use Redis Streams MesssageBus in secure mode due to lack of password on connection.
Issue Number: #2777
What is the new behavior?
Now the DB credentials are used for the Redis Streams MessageBus connection enabling it to work in secure mode.
Does this PR introduce a breaking change?

[ ] Yes
[x] No

New Imports
Are there any new imports or modules? If so, what are they used for and why?
no
Specific Instructions
Are there any specific instructions or things that should be known prior to reviewing?
Other information

Codecov Report

Merging #2792 into master will increase coverage by 0.02%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##           master    #2792      +/-   ##
==========================================
+ Coverage   39.16%   39.19%   +0.02%
==========================================
  Files         164      164
  Lines       12730    12737       +7
==========================================
+ Hits         4986     4992       +6
- Misses       7436     7438       +2
+ Partials      308      307       -1

Impacted Files
Coverage Δ

internal/core/data/init.go
0.00% <0.00%> (ø)

internal/core/data/v2/application/event.go
81.03% <0.00%> (+10.34%)
:arrow_up:

Continue to review full report at Codecov.

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


Publish 0.16.0

I believe this should be done ASAP so as not to hinder users keeping up with latest nightlies.
cc @tikue

Thanks for tracking this! I'll do it today.

Done; give it a whirl!


Weights for multiscale consistency

Hi, Thanks for your excellent work. I noticed you use learnable weights for three scales in multi-scale consistency.
https://github.com/Kangningthu/ADELE/blob/7195bd0af39be79c533d67dd7eab7f9bfd6a4285/train.py#L323
https://github.com/Kangningthu/ADELE/blob/7195bd0af39be79c533d67dd7eab7f9bfd6a4285/train.py#L547
https://github.com/Kangningthu/ADELE/blob/7195bd0af39be79c533d67dd7eab7f9bfd6a4285/lib/utils/JSD_loss.py#L40
Have you ever tried fixed weights? Are the learnable weights outperforming fixed weights like 1/3?

Hi, Thanks for your interest in our work. We didn't notice a significant difference in making the weights fixed or not.


cmake-js added as external library without any types

What PR on Definitely Typed is this for?
Also, cmake-js doesn't appear to have types in its package, so I don't know of a reason to allow it as a package.json dependency on Definitely Typed. (But seeing a real PR might make it clear to me.)

Sorry. That was a mistake from my side. I didn't correctly read a exception message. Closing this pr.


Github API deprecation

The current auth method has been deprecated as of Nov 2020 by GitHub; authentication would need to be revised to the new oauth specifications.

I would be happy to receive a PR.


Mapping the Relationship field type onto relational DBs

KS 5 models it's Relationship field type after the KS 4 type of the same name. In the Mongoose adapter, fields of this type store either a single ID field or an array of IDs when many: true is set, thereby supporting one-to-many and many-to-many relationships.
In a relational DB, many-to-many relationships are usually modelled using a associative/junction/link table. How then, should we implement this type of relationship for the Knex adapter (#297)?
I believe our options are...

Follow the current strategy and use array based relationships in the relational/Knex adapter
Postgres has excellent array support but using them for relationships might be counter intuitive a relational context and probably (?) introduce a performance penalty. Other DB platforms require serialisation to support anything resembeling an array (eg. MySQL, SQLite) which either breaks many-to-many relationships or results in terrible performance.

Refactor the current field type to always use an associative table/collection
Really not ideal for Mongo due to lack of (performant) joins. Arrays of IDs actually make sense in the "key-value-like" document store land of Mongo.

Use arrays in Mongo and an associative table in relational DBs within the current field type
This is probably workable but supporting both methods will add significant complexity to the field type implementation, especially given our desired relationship workflows (#251).

Implement a different relationship type (or types) for relational DBs
These two types of data persistence represent fundamentally different paradigms which reasonable and intuitively model relationships in different ways. Trying to fit one-to-many and many-to-many for both relational and document store data layers into a single field type is unnecessary and possibly undesirable.

For reference, KS Next didn't follow the Relationship field type of KS 4, opting instead for a solution that more closely reflected the underlying structure with the ForeignKey and RelatedChildren field types (with the intention to add a RelatedItem type for one-to-one relationships).

This has been implemented.


[Bug/issue] Isssue ) RAC ; How to get the same evaluation result at the paper

Hi
Thank you for sharing nice work.
Some advanced utilities (e.g visualization) are great!
I want to get the same evaluation result at RAC paper
Specifically, Samba scene and Swing scene (like BANMo)
But I found that there is not enough explanation about evaluation of RAC at RAC & Lab4D github.
I tried to evaluate the result of RAC following BANMo code (render_vis.py) and met some issues.
I will describe how I trained RAC model before details of issues.

I followed the code at https://lab4d-org.github.io/lab4d/tutorials/category_model.html
There's no any changes except inst_id at export,
from
python lab4d/export.py --flagfile=logdir/$logname/opts.log --load_suffix latest --inst_id 0
to
python lab4d/export.py --flagfile=logdir/$logname/opts.log --load_suffix latest --inst_id 16 or
python lab4d/export.py --flagfile=logdir/$logname/opts.log --load_suffix latest --inst_id 20
because the 16th video in human-48 is same as the 1st video in T_samba && the 20th video in human-48 is same as the 1st video of  T_swing.

Initially, I used preprocessed data from lab4d , and later, tried to use dataset without skipping frames by following https://lab4d-org.github.io/lab4d/tutorials/preprocessing.html
(Indeed, later was not helpful for solving issues)

And then, I met two issues
1. No root pose of RAC
Could you provide guidance on how to extract the root pose from RAC? At my experience at BANMo, root pose was necessary to align GT mesh and predicted mesh for evaluation. But the result of python lab4d/export.py doesn't contain root pose information (e.g. cam-00000.txt)
2. Strange mesh result
This is my result ; Mesh at last frame of T_samba1 video. (visualized by meshlab)

I expected the result like below.

As you can see, wrinkles at leg appears and the coarse shape does not match the figure in the paper.
I would appreciate your help in resolving these issues.
Best regards.

Hi @minsu1206, what you tried looks correct. If you want to compare to RAC without re-training, we provided model checkpoints that produced the numbers in the paper here.
To answer your questions, the root pose can be found here:
logdir/human-48-category-comp/export_0000/fg/motion.json
It stores the root to camera transformation as data["field2cam"].
The wrinkle issue is more complicated. It has something to do with frequency in NeRF positional encoding, and how is eikonal loss enforced. In short, either a high frequency or a low eikonal weight can cause this issue. The recent lab4d-PPR branch has it fixed, but we haven't verified the fix with lab4d-RAC.
Moving forward in terms of evaluation, we have an evaluation script in PPR branch, which is compatible with lab4d and will be merged.
Let me know if you have more questions.

Thank you for your answer.
Now I try to use evaluation script in PPR branch, but still get strange results

I can't extract meshes using pretrained checkpoints at RAC github , due to some options. I guess that they are for BANMo flags, but not compatible with Lab4d

I also try to use the evaluation scripts at PPR branch. But the result is quites strange.

To reproduce this issue, I write what I've done below.
(1) data['field2cam'] only contains extrinsic matrix. Although evaluation script requires intrisic matrix, this matrix is not involved in evaluation process. Therefore, I ignore this matrix
(2) I set
pred_mesh_paths = List[path of extracted $seqname-mesh-* .obj]  and
pred_camera_paths = List[path of extracted $seqname-cam-*.txt]
I saved $seqname-cam-{frame number:05d}.txt from data['field2cam'] in advance.
I also saved $seqname-mesh-{frame number:05d}.obj from extracted meshes in advance.
Below codes reflect them.
# lab4d/projects/ppr/eval/compute_metrics.py ln61 ~
if args.pred_prefix == "": ...
else:
    pred_mesh_paths = glob.glob("%s/%s-mesh-*.obj" % (args.testdir, args.pred_prefix)) # What I changed
    pred_camera_paths = sorted(glob.glob("%s/%s-cam-*.txt" % (args.testdir, args.pred_prefix)))  # What I changed
    ...

And this is the command which I used.
python projects/ppr/eval/compute_metrics.py --testdir ../rac_result/human-48-skel-soft/export_0020 --gt_seq T_swing-1 --pred_prefix T_swing1 --fps 30
And this is the result
Jitting Chamfer 3D
Loaded JIT 3D CUDA chamfer distance
found 150 groune-truth meshes
/workspace/lab4d/projects/ppr/eval/eval_utils.py:83: UserWarning: Creating a tensor from a list of numpy.ndarrays is extremely slow. Please consider converting the list to a single numpy.ndarray with numpy.array() before converting to a tensor.
Evaluating::   0%|                                                                                | 0/150 [00:00<?, ?it/s]Frame 0: CD=51.90cm<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Frame 1: CD=51.52cm<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Frame 2: CD=51.33cm<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Evaluating::   2%|███▎                                                                                | 3/150 [00:00<00:05, 28.01it/s]Frame 3: CD=51.40cm<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Frame 4: CD=51.70cm<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Frame 5: CD=51.92cm<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Evaluating::   4%|██████▋                                                                                | 6/150 [00:00<00:05, 26.61it/s]Frame 6: CD=52.10cm<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Frame 7: CD=52.30cm<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Frame 8: CD=52.42cm<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Evaluating::   6%|██████████                                                                                | 9/150 [00:00<00:05, 25.69it/s]Frame 9: CD=52.33cm<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Frame 10: CD=52.25cm<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Frame 11: CD=52.21cm<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Evaluating::   8%|█████████████▎                                                                                | 12/150 [00:00<00:05, 25.30it/s]Frame 12: CD=52.14cm<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Frame 13: CD=52.00cm<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Frame 14: CD=51.61cm<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>...
Frame 132: CD=58.63cm<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Frame 133: CD=58.71cm<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Frame 134: CD=59.02cm<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Frame 135: CD=59.16cm<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Evaluating::  91%|██████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████▌               | 136/150 [00:02<00:00, 71.71it/s]
Frame 136: CD=59.16cm<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Frame 137: CD=58.60cm<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Frame 138: CD=57.93cm<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Frame 139: CD=57.82cm<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Frame 140: CD=57.75cm<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Frame 141: CD=57.31cm<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Frame 142: CD=56.68cm<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Frame 143: CD=55.92cm<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Frame 144: CD=55.21cm<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Frame 145: CD=54.74cm<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Frame 146: CD=54.74cm<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Frame 147: CD=54.83cm<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Evaluating::  99%|███████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████▊  | 148/150 [00:02<00:00, 81.33it/s]Frame 148: CD=54.82cm<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Frame 149: CD=54.85cm<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>Evaluating:: 100%|██████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████████| 150/150 [00:02<00:00, 51.15it/s]
Finished evaluation
Avg chamfer dist: 53.71cm
Avg f-score at d=10cm: 10.2%
Avg f-score at d=5cm:  3.7%
Avg f-score at d=2cm:  1.2%
I'm not sure what I did wrong. I want to know the right way to do it.
Could you share your thoughts on those with me
Thanks.

I think the issue is following

lab4d main branch saves deformed meshes in the camera space.
ppr saves deformed meshes in the object space

The simplest fix is to set the camera extrinsics to identity in the evaluation code. You can also visualize whether the reconstruction and GT are aligned, in the rendered .mp4 file generated by the eval script.
Let me know how it works.

Thank you for your guidance.
I tried 2 methods.

(1) Follow your instruction
It would be easier to fine the cause If I visualized mesh. By the way, setting predicted camera matrix to identity matrix seems the solution.

(T_swing 149th frame)
(yellow : raw prediction / pink : prediction aligned by ICP (and others))

(T_swing 149th frame)
(green : GT mesh / pink : same as above)
And the result is
... # swing scene
Finished evaluation
Avg chamfer dist: 11.37cm
Avg f-score at d=10cm: 83.3%
Avg f-score at d=5cm:  56.3%
Avg f-score at d=2cm:  25.2%

This result seems reasonable in that PPR's evaluation result at PPR main page shows chamfer distance around 10 and F-score around 60
(2) Fix some lines : enforce better alignment
I found that the evaluation protocol at PPR branch is different from BANMo's one. (some changes look trivial, but lead to large gap, I think) and was still concerned about the difference in the number presented in the paper.
The size of predicted mesh at (1) doens't match the GT well
My idea : How about moving GT & predicted mesh into origin and scaling the predicted mesh with coarse bbox size, (not median value (at BANMo) and not overall coarse bbox (at PPR branch))?
I changed some lines like below. (I used BANMo:scripts/render_vis.py at here. sorry for confusing)
# original
...
bbox_max = float((verts_gt.max(1)[0]-verts_gt.min(1)[0]).max().cpu())
verts_gt = obj_to_cam(verts_gt, Rmat_gt, Tmat_gt)

import chamfer3D.dist_chamfer_3D
import fscore
chamLoss = chamfer3D.dist_chamfer_3D.chamfer_3DDist()

## use ICP for ours improve resutls
fitted_scale = verts_gt[...,-1].median() / verts[...,-1].median()
verts = verts*fitted_scale

frts = pytorch3d.ops.iterative_closest_point(verts,verts_gt, \
                    estimate_scale=False,max_iterations=100)
...

# changed for RAC
...
bbox_max = float((verts_gt.max(1)[0]-verts_gt.min(1)[0]).max().cpu())
verts_gt = obj_to_cam(verts_gt, Rmat_gt, Tmat_gt)
verts_gt -= torch.mean(verts_gt, axis=1)
import chamfer3D.dist_chamfer_3D
import fscore
chamLoss = chamfer3D.dist_chamfer_3D.chamfer_3DDist()

## use ICP for ours improve resutls
verts -= torch.mean(verts, axis=1)
bbox_max_gt = float((verts_gt.max(1)[0] - verts_gt.min(1)[0]).max().cpu())
bbox_max_pred = float((verts_ours.max(1)[0] - verts_ours.min(1)[0]).max().cpu())
verts /= bbox_max_pred
verts *= bbox_max_gt
# fitted_scale = verts_gt[...,-1].median() / verts[...,-1].median()
# verts = verts*fitted_scale

frts = pytorch3d.ops.iterative_closest_point(verts,verts_gt, \
                    estimate_scale=False,max_iterations=100)
...

And the result is

(also T_swing 149th frame ; green : aligned GT mesh ; orange: aligned predicted mesh)
(swing scene)
average CD : 6.773
average F-score 2% : 65.139
(samba scene)
average CD : 6.875
average F-score 2% : 65.419
This evaluation result is quite close the numbers in the paper.
And there were no severe problems when visualizing the meshes.

So this is the last question (Sorry for making it longer ... )
Because both BANMo and RAC estimates root pose (=camera extrinsic matrix) and camera intrinsic matrix,
the estimated scale also be a subject of evaluation.
In this context, I wonder whether (2) method may be a kind of cheating or not.
In conclusion,
How do you think about method (2) at above ? Is it okay to use ? (w.r.t fair comparison) I would appreciate if you share your insight.
Thanks.
p.s
I found that the meshes at human-48-soft-skel/export0016 have different vertices size (48007 or 48008).
This makes an error at here
"all_verts_pred = torch.tensor(all_verts_pred, device=device, dtype=torch.float32)"

Sure.
At compute_metrics.py
for fidx, mesh_path in enumerate(sorted(pred_mesh_paths)):
        fidx = int(mesh_path.split("/")[-1].split("-")[-1].split(".")[0])
        pred_mesh_dict[args.skip * fidx] = trimesh.load(mesh_path, process=False)
        pred_mesh_dict[args.skip * fidx].apply_transform(extrinsics[fidx])
        # pred_mesh_dict[args.skip * fidx].apply_transform(np.linalg.inv(Gmat_gt))

I just comment the code that applies transformation into predicted mesh. (just leave it in camera space)
Since transformation with identity matrix means no chanage, so I thought this makes sense.

@minsu1206
I am currently using PPR code for eval on the Eagle dataset, and I have some problem . I see that the code needs to use groundtrue Camera.Pmat.cal data. May I ask if he is the root pose? If so, why does its weight seem so huge?
At compute_metrics.py：
        gt_name, gt_cam_id = args.gt_seq.split("-")     gt_cam_path = "%s/%s/calibration/Camera%s.Pmat.cal" % (ama_path, gt_name, gt_cam_id)     intrinsics_gt, Gmat_gt = load_ama_intrinsics(gt_cam_path)
Thanks a lot!

@minsu1206 What you did looks good. The metrics are designed for surface reconstruction quality (in the sense it is not entangled with camera extrinsics, or intrinsics), and we don't expect the method to be able to estimate scale correctly due to the fundamental scale ambiguity. Therefore, we fit the scale and SE(3) before eval.
It should be adopted if (2) generally aligns prediction with GT better than (1). In RAC eval, I recall what we did was to use height to align instead of median depth as follows. We also use the first frame to find scale and SE(3) and use them for the rest of the frames to make sure they are consistent over time.
            verts[...,2] -= verts[...,2].mean() * (1-1./focal_correction_ratio)
            if i==0: # TODO extend to multiple frames
                fitted_scale = ((verts_gt[...,-1].max()+verts_gt[...,-1].min())/2) /\
                               ((verts[...,-1].max()+verts[...,-1].min())/2) # this is more accurate
            verts = verts*fitted_scale

            if i==0:
                frts = pytorch3d.ops.iterative_closest_point(verts,verts_gt, \
                        estimate_scale=False,max_iterations=100)
            verts = ((frts.RTs.s*verts).matmul(frts.RTs.R)+frts.RTs.T[:,None])

@JAMESYJL Pmat is defined as GT object to camera transformations. If you are using the eagle data in BANMo, probably want to set it to identity for camera view 0 as here.

@gengshan-y  Sorry for the late reply, thank you for your detailed explanation. I am currently encountering a new problem and would like to consult with you. I am using lab4d_ The PPR code evaluated the Eagle and Hands datasets, but yielded strange results. Regarding the rootpose, do both groundtrue and pred extractions set to identity? In Banmo's preprocessing, there is a cam.txt file in the folder that seems to store groundtrue extractions data. Can it be directly loaded as groundtrue extractions? Sorry for asking so much, thank you very much!

I believe only GT needs to be set to identity for eagle and hands sequences.

I believe only GT needs to be set to identity for eagle and hands sequences.
thank for reply！When I tested the Eagle dataset, I only set GT to identity and the CD value I ran was very abnormal, with a value of 57. When I set both GT and pred to identity, the result I ran was 11. Comparing the gt mesh and the pred mesh I output, it seems that 11 is a more accurate result, but I am not sure if this is correct. Thank you for your reply!

Sorry for the late reply.
Thanks you for sharing your insight.
The key point is to use the scale consistent over time or not ; consider temporal effect or not.
I hope my method has no logical error and believe the intention of evaluation would decide whether the scale contain temporal context or not.
All conversations at here helped me a lot.
I'll close the isssue.


Request: Flexible toolbar space

feature request:
What is the expected behavior?
Flexible toolbar space:
https://material.io/guidelines/patterns/scrolling-techniques.html
What is the current behavior?
md-toolbar does not have the scrolling technique mentioned on the material guideline.

You can use flex-layout https://github.com/angular/flex-layout or css.

@tinayuangao I'm sorry, I don't understand how the flex-layout effects the flexible space of toolbar on scroll. Can you give me an example?

@DevVersion I believe this is requesting a feature to expand/collapse the toolbar on scroll, is that something we plan to do?

@mmalerba, @DevVersion
I think, visually, this is one of the nice things about Material design.
Material design toolbar with image or 'expanded' look really gives a nice visual queue for users and the way Material design changes the toolbar on scroll minimizes the initial visuals so that the content can take over on priority.
I believe this feature is especially helpful on mobile view.

@mmalerba I didn't plan to do that when I initially created the design doc.
We had something similar in Material 1.x and it required us to have a different directive called md-content.
Still I think it's a useful feature and maybe we should consider creating another design doc just for this.

+1

Closing this since we don't have any plans to add this feature.


Updated README with simple Transform Shapes App

A simple app that demonstrates the uses of the Transform widget of Flutter. There are 4 rounded squares with transform in accordance with the slider at the top. The first square rotates in clockwise direction , the second square grows in size, the third square translates towards its right while the fourth one us for skew. It demonstrates hoe the transform widget together with other widgets can be used to get beautiful motions and animations in our app. This app is for GSoC 2020 as per the instructions of Andrew Brogdon<EMAIL_ADDRESS>Dartpad:  https://dartpad.dev/3dc4b782c794d95a807e614704b11ac3

Hey @aksayushx!! Thanks a lot for this PR but this sample cannot be added to the list as it's quite basic. Feel free to work on it and come up with something new and awesome.


fix for recent nightlies' TestOpts change

See https://github.com/rust-lang/rust/pull/81356. The filter: Option<String> field was changed to filters: Vec<String>.

Clippy failure wise... yeah, it will fail on stable. It's using unstable nightly APIs that are different between nightly and stable, it literally cannot pass clippy stable. The other two problems are fixed in the other PR.

Ok, so a few updates:

I made it compile on stable as well by using version_check.
I think datatest is, by about rust 1.52, maybe going to stop working on stable. See build.rs for a note on that.
The last commit making test_case_registration the default is optional, I'm not sure if it is a breaking change or not. Probably not? The whole test suite passes with or without it. Maybe you didn't use it by default previously because that API might be a bit less stable than the ctor method. I've missed a bit but feel free to merge without it.

Used this PR in my project and it solved the compile issue and runs the data-dependent tests successfully.

I'll be sure to keep my main branch clean so that keeps working until this is merged.

Hi! Any estimation on when this could get merged?
Thanks!

Thanks for merging


What is the recommended way for caching ajax requests or using push notifications w/ service workers in a CRA based PWA?

I'm using the newest versions of create-react-app and react-scripts.
For a fully functional offline PWA, I would need to cache network requests to show some data while the user is offline, Most tutorials I saw were listening to the fetch event in the service-worker itself using self.addEventListener, but since CRA uses workbox-plugin to autogenerate the service worker as a developer I don't have access to "self" so I can't freely listen to the fetch event.
As a workaround I did something like this to cache my get requests..
function cachedGet (url, config) {
  return Promise.race([
    getFromCacheAfter4Seconds(url, config),
    getFromNetworkAndSaveToCache(url, config)
  ])
    .catch(error => {
      console.error('error', error)
      return getFromCacheAfter4Seconds(url, config)
    })
}

function getFromCacheAfter4Seconds (url, config) {
  const cacheKey = url.split('?')[0]

  return new Promise((resolve, reject) => {
    let timeout = setTimeout(() => {
      clearTimeout(timeout)
      const cachedResponse = window.localStorage.getItem(cacheKey)
      if (cachedResponse) {
        const response = JSON.parse(cachedResponse)
        console.log('getting from cache', url)
        console.log('cachedResponse', response)
        resolve(response)
      } else {
        console.log('no cached response found falling back to network', url)
        getFromNetworkAndSaveToCache(url, config)
          .then(resolve)
      }
    }, 4000)
  })
}

function getFromNetworkAndSaveToCache (url, config) {
  const cacheKey = url.split('?')[0] // For requests that use a startDate & endDate parameters
  return axios.get(url, config)
    .then(res => res.data)
    .then(data => {
      window.localStorage.setItem(cacheKey, JSON.stringify(data))
      return data
    })
}

or to avoid calling the setTimeout anyway whenever Promise.race() is called I did something like this
function cachedGet (url, config) {
  return getFromNetworkAndSaveToCache(url, config)
    .catch(error => {
      if (!error.status) {
        // Network error
        return getFromCacheAfter4Seconds(url, config)
      } else {
        throw error
      }
    })
}

Is there a better/ more recommended way? perhaps a configuration file I can use to specify cache-first vs offline-first vs whichever is faster? Same thing if I were to use other service worker features like background sync and/or push notifications.

You can do this if you set runtimeCaching option in webpack config, or use InjectManifest plugin instead of GenerateSW which CRA currently uses. Both of those don't seem possible without forking react-scripts or ejecting. I think the optimal solution here is not using CRA's service worker and just using workbox directly. It's always loaded from CDN anyway.

You can append an epilog to the auto-generated SW in a build step and, there, add a message listener which can access SW self object. For example, I use this technique to upgrade SW in https://github.com/emibcn/Rac1.js/blob/master/app/src/sw-epilog.js :
// Add a listener to receive messages from clients
self.addEventListener('message', function(event) {
  // Force SW upgrade (activation of new installed SW version)
  if ( event.data === 'skipWaiting' ) {
    self.skipWaiting();
  }
});

And then, from your app https://github.com/emibcn/Rac1.js/blob/master/app/src/index.js :
registration.waiting.postMessage('skipWaiting');

@emibcn that's exactly what I needed, thanks!


Custom color support

Resolves #10
Found some docs that specify how to add custom colors for use by extensions
Added 6 custom colors to the extension contribution points. They have placeholder values for light and dark themes, and can be customized under workbench.colorCustomizations. Added some more notes about it in the README!
This is my first time working on a VSCode extension but everything worked as expected in testing!

@emb-isaac Hello, sorry for missing this. I've been pretty caught up at work. Will take a look at this PR and merge it in this week :) Thank you so much!!

@jacob-j Any chance you'd be able to review this? Totally understand being busy with work, not trying to bug, but I just found this extension and would looooove to have custom color capabilities since the default colors don't have amazing contrast against my theme's background

@emb-isaac Thank you for the pull request, I have merged it and publishing a new version right now :)
Thank you for the reminder @DaltonSW 😊


Changed AddDaprClient return type to IServiceCollection to allow methods chaining

Description
Changed AddDaprClient() extension method return type to IServiceCollection to allow methods chaining.
Issue reference
https://github.com/dapr/dotnet-sdk/issues/817
Checklist
Please make sure you've completed the relevant tasks for this PR, out of the following list:

[x] Code compiles correctly
[ ] Created/updated tests
[x] Extended the documentation

Since the issue for this is now in the 2.0 future milestone, I'm going to close this PR for now to keep the dashboard clear.


How to theme/style tree base

Can someone please post an example on how to theme or override the style on the tree base and other components.
I can update the header render for the tree items but not the actual tree base and things like the arrow animation.

Perfect thanks, would be nice if the documentation was a little clearer but this works perfectly.

I agree

Do you know if it's possible to do alternative line styles like
li:nth-child(even) {
background-color: black;
color: white;
}
li:nth-child(odd) {
    background-color: #0000335f;
}

I don't think so, because the style isn't in the node. Maybe you should set the style in the state of the component and change it dynamically

Hi @maximilianoforlenza , I just came across this question and saw your example code, I see that you are setting the color for an active link to brown. But the color does not change when it is active, which is the exact problem I have and  why I came across this question.
Do you know if there is a way to set the color of an active link?


Add missing with clause to release gh action

Fix missing with statement. Not sure why this missing passed an act release -n.

We need this for finding all the tags to find the latest release
On Tue, Oct 10, 2023, 5:14 PM French Ben @.***> wrote:

@.**** commented on this pull request.
LGTM - By default this is set to 1, do we actually need to fetch all?
0 indicates all history for all branches and tags
IMHO we can probably remove it in favor of the default
—
Reply to this email directly, view it on GitHub
https://github.com/helm/helm/pull/12437#pullrequestreview-1669167431,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABFCX4U3Q4XENH6DC3TLQ3LX6XCGFAVCNFSM6AAAAAA5MVUJCOVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTMNRZGE3DONBTGE
.
You are receiving this because you authored the thread.Message ID:
@.***>


Update institutional partners

Rather than keeping a list of people up-to-date in two places, I'd suggest linking to the website from here
My current employer (Gousto) runs a "tech ten percent" scheme, and they've said they're happy for me to spend part of that time working on pandas - this would be one Friday every other week. Per the governance.md#institutional-partners-and-funding document, my understanding is that I could/should list them here

@jorisvandenbossche tagging you as it doesn't look like I can request reviewers here

Rather than keeping a list of people up-to-date in two places, I'd suggest linking to the website from here

I would personally prefer to keep this repo as the "source of truth". But, linking the other way around (in some way generating the list that is now in  pandas/web/pandas/config.yml based on the content in this repo) doesn't seem that easy ..
Maybe for this PR, could you keep it to the addition of Gousto? (which I think is fine, as discussed on the mailing list, and I am happy to directly merge that)

Maybe for this PR, could you keep it to the addition of Gousto? (which I think is fine, as discussed on the mailing list, and I am happy to directly merge that)

Sure, done, thanks 👍

But, linking the other way around (in some way generating the list that is now in pandas/web/pandas/config.yml based on the content in this repo) doesn't seem that easy

True, but I'll look into it

This was discussed on the internal mailing list, so merging.


Compilation on aarch64 (arm) fails: expected i32, found i64

error[E0308]: mismatched types
  --> /home/user/.cargo/registry/src/github.com-1ecc6299db9ec823/heim-host-0.0.10/src/sys/linux/users/other.rs:79:25
   |
79 |             session_id: entry.ut_session,
   |                         ^^^^^^^^^^^^^^^^ expected `i32`, found `i64`

error: aborting due to previous error

For more information about this error, try `rustc --explain E0308`.
error: could not compile `heim-host`.

To learn more, run the command again with --verbose.
warning: build failed, waiting for other jobs to finish...
error: failed to compile `bottom v0.4.7`, intermediate artifacts can be found at `/tmp/cargo-install1nVP67`

Hi, @mrusme!
i686 and x86_64 are the only architectures supported right now (see platforms support page for details), but if anyone wants to fix this issue and ensure that CI correctly compiles heim for your arch, I'll be happy to review the PR.
Some ARM targets are already tested in CI, can you elaborate on what target do you use?
https://github.com/heim-rs/heim/blob/b8491b68a4cf493a799b660fe800b981093f997e/.github/workflows/ci.yml#L65-L73

I get this issue on aarch64-unknown-linux-gnu debian-buster

I have the same issue on aarch64-unknown-linux-gnu on rpi4 running archlinuxarm

Fails with aarch64-unknown-linux-gnu on NixOS unstable using binfmt+qemu-user-static

This is a duplicate of https://github.com/heim-rs/heim/issues/207

I successfully compiled bottom for aarch64-linux by cherry-picking https://github.com/heim-rs/heim/commit/98a347029bf5fffe73acb8719e21a1bfd1667f22 on top of 0.10.0 and changing bottom to use that.