Updated README.md

Changed headerCreate to createHeaders

Travis failed because the JVM crashed. Don't know why. Certainly unrelated to this documentation change.
@phdoerfler thank you for the PR. In fact the documentation for excludes reflects the state of the upcoming 2.0.0 release, where we changed the task name to headerCreate. But you're write this does not belong into the current docs. I'll apply this PR and add the renaming of the task to #99.


CI test for RL examples: Drive and VehicleFollowing

Added CI test for RL Drive and VehicleFollowing examples, which tests the RL training process.
Removed make build-all-scenarios command and only build necessary scenarios in CI to reduce test time.
Note: The CI test for this PR will pass after #2061 is merged.

The test times appear to be ~18 minutes and 7 minutes. I think we should find a way to run these less frequently, maybe only against certain commit messages and master.


merging into ui-improvements

closing to try to fetch into ui-improvements branch

May have figured out the issue, committed and fetched into ui-improvements and it seems ready for review now.  Please let me know if it's in the proper state for review.

closing this PR to try to push into original ui-improvements branch.  sorry for the confusion while I figure out how to get the branch back up to date with new code.


Add context to translations

Hi,
I wanted to know if it would be possible to add a context flag. I know that for deepl there is such parameter.
Let me know if you have already something otherwise you can assign it to me without problem.

Hi @Telsho,
Thanks for the suggestion. I just added context and formality parameters, they both can be used only for deepl api translate. Deepl Docs
New parameters are available from v1.9.0


CMD.exe terminal no longer substitutes environment variables in its PATH

Does this issue occur when all extensions are disabled?: Yes

VS Code Version:
Version: 1.95.1 (user setup)
Commit: 65edc4939843c90c34d61f4ce11704f09d3e5cb6
Date: 2024-10-31T05:14:54.222Z
Electron: 32.2.1
ElectronBuildId: 10427718
Chromium: 128.0.6613.186
Node.js: 20.18.0
V8: 12.8.374.38-electron.0
OS Version:
OS: Windows_NT x64 10.0.26100

Steps to Reproduce:

Open a CMD.exe terminal in VS Code (on Windows).
Run the command "set PATH"
Observe:

C:\TEMP>set path
Path=C:\Program Files\...;<multiple paths>;...\bin;%USERPROFILE%\AppData\Local\Microsoft\WindowsApps;%USERPROFILE%\.dotnet\tools;...;<more paths>

Normal command windows, whether using the old terminal window or the new one, elevated or not, show those %USERPROFILE% variables in the PATH expanded and replaced with their values.
The result is that command-line tools installed into "%USERPROFILE%\AppData\Local\Microsoft\WindowsApps;" no longer run from within the VS Code terminal window. See for instance, I can no longer invoke the WinDbg debugger from the VSCode terminal:
C:\TEMP>windbgx
'windbgx' is not recognized as an internal or external command,
operable program or batch file.

This works fine in all other command windows in my machine, and used to work from within VSCode.
This seems to be a regression (I recently allowed VS Code to update itself).

nothing has changed recently from our side. if you install earlier versions, does it work?

I don't know what to say. I run "windbgx" regularly from the terminal window, and as soon as I got the update it stopped working. And it only happens to terminal windows in VSCode, AFACS. regular desktop console windows don't have the problem. I might not have updated to every intermediate version (I've been known to ignore the prompts :-)), and I don't know what I had before.
Is it possible to install old versions side-by-side? I can try to bisect for you.

This type of issue is typically related to environment differences between VS Code's terminal and the external terminal. You can try diffing the two environments by printing them (eg. env, Get-ChildItem env:) and investigating any differences.
Closing this off as it doesn't look actionable on our end, you can read more about our issue process here. Thanks for understanding.

If this is a confirmed regression, it would help to know which change caused it. We maintain a node.js based CLI tool  vscode-bisect that automatically downloads and runs previous VS Code insiders and asks for each build if the issue reproduces. It takes up to 8 steps to find exactly the build that caused the regression.
Steps:

install node.js (if not yet done)
npx --yes @vscode/vscode-bisect@latest
follow the instructions until you found the offending build (its fine to leave the commit empty)
report back the commit range in this issue

Thanks!

I can verify that it's not happening anymore. I seem to have been upgraded to 1.95.3 :-P. And I've been made to reboot my computer a couple of times since.
"try diffing the two environments" -- The issue, as I reported, is the difference in the environment. Inside of VSCode's terminal, variables like "%USERPROFILE%" in environment variables (especially PATH) were not expanded, but outside of VSCode they were.
I always open VSCode using the Windows UI (not from the command-line), and in fact, all instances I use are invariably re-opened as long as I click the VSCode icon that I have pinned in my taskbar. I had been seeing other subtly strange things on Windows, all of which seem to be resolved after the reboots, so I have to assume Windows was the culprit. Maybe all UI apps were getting bad environments.
Anyway, I agree this item should be closed now.
Thanx!


fixed kernel.json to new name, alt. way install kernelspec

This should fix the installation via jupyter kernelspec.
Also added alternative installscript to the pyproject.toml.
You can just run jupyterdartkernel after the pip install to install the kernelspec.
That's why i got rid of the hyphens in the first place :-)

Hi @jans-code .
Thanks again for the PR.
I had a couple of questions/issues I was running into.

When I try to pip install -e . from the cloned directory, I receive the following error:

File "setup.py" or "setup.cfg" not found. Directory cannot be installed in editable mode:

Do we need to add these files to be able to pip install?

I love that this can be used from within pip.  Should I go ahead and create a ticket, assuming we work out all the details, to be able to publish this to PyPi and do this as part of some CI/CD pipeline (github actions or something) on this repo?

As a fallback, I tried installing the kernel w/o pip: jupyter kernelspec install jupyterdartkernel, but I also get some errors.

AsyncIOLoopKernelRestarter: restarting kernel (4/5), new random ports
...
Traceback (most recent call last):
File "/root/miniconda3/lib/python3.9/site-packages/tornado/web.py", line 1713, in _execute
         result = await result
       File "/root/miniconda3/lib/python3.9/site-packages/jupyter_server/services/kernels/websocket.py", line 62, in get
        await self.pre_get()
       File "/root/miniconda3/lib/python3.9/site-packages/jupyter_server/services/kernels/websocket.py", line 58, in pre_get
         await self.connection.prepare()
       File "/root/miniconda3/lib/python3.9/site-packages/jupyter_server/services/kernels/connection/channels.py", line 314, in prepare
         raise TimeoutError("Kernel never reached an 'alive' state.")
     TimeoutError: Kernel never reached an 'alive' state.
 [E 2023-04-16 21:40:32.434 SingleUserLabApp log:178] {
       "X-Forwarded-Proto": "ws",
       "X-Forwarded-Port": "8000",
       "X-Forwarded-For": "<IP_ADDRESS>",
       "Sec-Websocket-Extensions": "permessage-deflate; client_max_window_bits",
      "Sec-Websocket-Key": "Rmp5xmzCtMAixVz19LXQGw==",
       "Cookie": "jupyterhub-user-ws850f096482484c49b839=[secret]; redirect=[secret]; testing=[secret]; sid=[secret]; PLAY_SESSION=[secret]; _xsrf=[secret]; username-localhost-8888=[secret]; JSESSIONID=[secret]",
       "Accept-Encoding": "gzip, deflate, br",
       "Sec-Websocket-Version": "13",
       "Origin": "http://localhost:8000",
       "Upgrade": "websocket",
       "Accept-Language": "en-US,en",
       "User-Agent": "Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36",
       "Cache-Control": "no-cache",
       "Pragma": "no-cache",
       "Connection": "Upgrade",
       "Host": "localhost:8000"
     }

Let me know what you think.  Thanks again for the PR.  Apologies for the tardiness of the response.  Will try to get this merged quickly.  :+1: :+1:

Also, this error above doesn't happen during installation of the kernelspec.  That seems to work fine.  The timeout error happens when I open up a new Dart notebook and attempt to run some code in it.

Do we need to add these files to be able to pip install?

The pyproject.toml should be sufficient.
What versions of python, pip and setuptools are running?
Maybe something is a bit dated (2021 or older)?

TimeoutError: Kernel never reached an 'alive' state.

This error should be a result of the installation problem.

Python 3.9.12
pip 21.2.4
setuptools 61.2.0

Tried upgrading pip and setuptools.  Still getting the kernel restart and timeout error.  Listing the kernelspec, it looks to be installed.
New Versions:
pip 23.1
setuptools    67.6.1
jupyter kernelspec list ->
...
jupyterdartkernel       /usr/local/share/jupyter/kernels/jupyterdartkernel
...

what happens when you pip install -e . from project folder now?

@jans-code   maybe this is just on my end.  I'm going to go ahead and merge the PR, and we can see if anyone else has any issues, but it LGTM.
Thank again.  I'll create a couple of issues related to putting this on PyPi and maybe creating a CI/CD action to do that.

LGTM.


Evaluation metric bits-per-byte

Hi there,
Megabyte paper uses bits-per-byte in Table 2 as their evaluation metric. It seems it has difference compared with byte level perplexity, since their number in arXiv and Code is < 1. So it should not be perplexity. This repo uses the cross-entropy loss and can easily calculate the byte level perplexity. May I ask how to compute bits-per-byte metric?
Thanks a lot.

BPB can be computed from cross entropy loss as:

def cc_to_bpb(cc_loss: float):
    return cc_loss * math.log2(math.e)


Adding an option to make the PageView call onPageChanged only when the drag ends.

This would be a pretty small change and I could do a PR for it.
if (notification.depth == 0 && widget.onPageChanged != null && notification is ScrollUpdateNotification)

Depending on a boolean ScrollUpdateNotification would be replaced with ScrollEndNotification.
Opinions?

/cc @HansMuller

Fixed by #27365
Please let us know if you're still seeing this with latest master

Closed per https://github.com/flutter/flutter/pull/27365#issuecomment-461981580


chore(tsconfig): Remove extranious peer deps from tsconfig

Actually, this is correct. We reference the jest and node types in these tsconfigs, so they should get pulled in.


Ramp Up/Down weight for new datasets

https://github.com/soo89/CSD-RFCN/blob/45497e9b69ac6b0d962e1efbe6150ed8ca4188b0/train_csd.py#L147
Hi @soo89 ,
Thanks for providing the code. I am curious about how to set up the ramp weight for consistency loss if I want to apply to a new dataset.
Q1: Does "data_len" written in the rampweight mean that the training iterations for each epoch?
Q2: If I want to apply this method to the new dataset (such as applying to other datasets), do you have any suggestions on setting this ramp weight?
Thank you!

And, I try to understand your ramp scheduler. Does the following interpretation correct?
1. Iteration:  0 to ramp_up_end (2/7 x max_training iterations):
ramp_weight = math.exp(-5 * math.pow((1 - iteration / ramp_up_end), 2)) * beta
2. Iteration:  ramp_up_end (2/7 x max_training iterations) to ramp_down_start (6/7 x max_training iterations):
ramp_weight = 1 * beta
3. Iteration:  ramp_down_start (6/7 x max_training iterations) to max_training_iterations:
ramp_weight = math.exp(-12.5 * math.pow((1 - (iter_max - iteration) / (iter_max - ramp_down_start)), 2)) * beta
Thank you!

Yes, you are right.
We tried to follow the previous semi-supervised learning schedule.
(https://arxiv.org/abs/1610.02242) (Appendix A)
And, Our scheduling description is in the supplementary material.
(https://papers.nips.cc/paper/9259-consistency-based-semi-supervised-learning-for-object-detection)

Got it. Thanks for replying, @soo89  !


Changing the datepicker popup position

I am having a “small” issue with the angularjs ui datepicker. I need to indicate somehow, if an input with a datepicker attach, should show the popup in the right side of the textbox
Thanks

https://github.com/angular-ui/bootstrap#support

As I am new to github I didn't understand the solution you have specified can you please tell in detail.
Thank you

He is suggesting that you look at the #Support section of the readme.  Since your "issue" is not a bug or feature request for this source code you need to look at other channels for getting support.


Add use_local_cluster flag to e2e test readme

Release note:

/assign @sebastienvas

/lgtm
Thanks

quick question: what does use_local_cluster actually means ?
(what's a local cluster, as opposed to... ?)

Something like minikube. If you re own the same network as your cluster.
On Aug 15, 2017 3:24 PM, "Laurent Demailly"<EMAIL_ADDRESS>wrote:

quick question: what does use_local_cluster actually means ?
(what's a local cluster, as opposed to... ?)
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/pull/559#issuecomment-322606374, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AQ80Zzz8qH3umELtR9GK9vDoX-r_8ii1ks5sYhq0gaJpZM4O2zbW
.

Ah thx, yeah so it's not really "use local cluster" as much as "(my) cluster is local/reachable"
(the "use" part is confusing)
https://github.com/istio/istio/blob/5ec8259fa89bd2431ecbd038a7acfb5299bd107c/tests/e2e/framework/kubernetes.go#L127

It is a little confusing... you actually should not need minikube or local cluster.   You could point to a cluster using node port as the istio ingress endpoint.

Yes, the main difference is, inside or outside of the cluster. If we want to use in-cluster IP, we need to be inside of the cluster, like minikube or when CI runs tests. While on the other hand, when we connect to a gke cluster and start tests on my machine (laptop, desktop), we need external IP which is hold for load balancer.

Reading the util.GetIngressPod() code, it can get the host IP and nodeport, which can be accessible if that host IP is on public internet.   So it doesn't have to be inside of the cluster.

should be rename to use_pod_ip or local_cluster maybe?

Sure local_cluster seems fine. but there are bunch of script to update with that.


I made a Trackpad-V2

This new version allows you to combine it with another program I made (Mouse2Keyboard) to turn your mouse into a keyboard. This means you can operate your keyboard as a mouse at the same time you operate your mouse as a keyboard.
Some changes had been done to Trackpad for this to work.
I wasn't sure If I should have made a Fork/Pull Request, so I ended up making my own Repository.
Keyboard2Mouse can be found https://github.com/M4rkFlor/Keyboard2Mouse
Mouse2Keyboard can be found  https://github.com/M4rkFlor/Mouse2Keyboard

Can you add it here as a branch, please?


fix OOM on iOS

Resolves #18
Tested on my iPhone SE 2023 (4GB or RAM in total)
FYI @felladrin , the getWasmMemory introduced in this PR decided that 1.6GB is usable on my iPhone (instead of 1.2GB)

Great solution! Thanks for the quick response, @ngxson!


Any plans for rclone?

Loved your mediabox, Are you planning on adding rclone or something similar to mount with Gdrive? You can keep this optional like your deluge+VPN

Hey! Thanks!. Yes, I'm very interested in replacing my current overGrive setup with a docker container. Would you recommend rclone for this?
If you feel like it, you could create a PR. I'd love for people to start collaborating on this project. What do you think?.


fix: support search hidden file

Types

[x] 🐛 Bug Fixes

Background or solution
before:

after:

Changelog
support search hidden file

LGTM


[!] SSL_connect returned=1 errno=0 state=error: certificate verify failed

New Issue Checklist

[ ] Updated fastlane to the latest version
[ ] I have read the Contribution Guidelines

Issue Description
Complete output when running fastlane, including the stack trace and command used

You can use: --capture_output as the last commandline argument to get that collected for you

[INSERT OUTPUT HERE]
Environment
Please run fastlane env and copy the output below. This will help us help you :+1:
If you used --capture_output option please remove this block - as it is already included there.
[INSERT OUTPUT HERE]

🚫 fastlane environment 🚫
Stack

Key
Value

OS
10.12.6

Ruby
2.2.4

Bundler?
false

Git
git version 2.12.2

Installation Source
~/.fastlane/bin/bundle/bin/fastlane

Host
Mac OS X 10.12.6 (16G29)

Ruby Lib Dir
~/.fastlane/bin/bundle/lib

OpenSSL Version
OpenSSL 1.0.2g  1 Mar 2016

Is contained
false

Is homebrew
false

Is installed via Fabric.app
false

Xcode Path
/Library/Developer/CommandLineTools/

System Locale

Error

No Locale with UTF8 found 🚫

fastlane files:
No Fastfile found
No Appfile found
fastlane gems

Gem
Version
Update-Status

fastlane
2.49.0
✅ Up-To-Date

Loaded fastlane plugins:
No plugins Loaded
Loaded gems

Gem
Version

slack-notifier
1.5.1

CFPropertyList
2.3.5

claide
1.0.2

colored2
3.1.2

nanaimo
0.2.3

xcodeproj
1.5.1

rouge
2.0.7

xcpretty
0.2.8

terminal-notifier
1.8.0

unicode-display_width
1.3.0

terminal-table
1.8.0

plist
3.3.0

public_suffix
2.0.5

addressable
2.5.1

multipart-post
2.0.0

word_wrap
1.0.0

tty-screen
0.5.0

babosa
1.0.2

colored
1.2

highline
1.7.8

commander-fastlane
4.4.5

excon
0.57.1

faraday
0.12.2

unf_ext
<IP_ADDRESS>

unf
0.1.4

domain_name
0.5.20170404

http-cookie
1.0.3

faraday-cookie_jar
0.0.6

fastimage
2.1.0

gh_inspector
1.0.3

uber
0.1.0

declarative
0.0.9

declarative-option
0.1.0

representable
3.0.4

retriable
3.0.2

mime-types-data
3.2016.0521

mime-types
3.1

little-plugger
1.1.4

multi_json
1.12.1

logging
2.2.2

jwt
1.5.6

memoist
0.16.0

os
0.9.6

signet
0.7.3

googleauth
0.5.3

httpclient
2.8.3

google-api-client
0.12.0

json
1.8.1

mini_magick
4.5.1

multi_xml
0.6.0

rubyzip
1.2.1

security
0.1.3

xcpretty-travis-formatter
0.0.4

dotenv
2.2.1

bundler
1.15.3

faraday_middleware
<IP_ADDRESS>

generated on: 2017-07-27

Hey @T-Jedsada - would you mind sharing a bit more of your terminal output so we can have a bit more context as to how you are calling the s3 action? Thank you!! 💯 👍

Hey @T-Jedsada, sorry I should have been more clear, could you copy and paste in this output, including the output from before the crash so we can see what sort of things are going on before it fails? Thank you!! 😄

@ohayon I went to upload file apk to s3 sorry i talk not clear.

Could you try the following
def suggest_ruby_reinstall(e)
      ui = FastlaneCore::UI
      ui.error "-----------------------------------------------------------------------"
      ui.error e.to_s
      ui.error ""
      ui.error "SSL errors can be caused by various components on your local machine."
      if Gem::Version.new(RUBY_VERSION) < Gem::Version.new('2.1')
        ui.error "Apple has recently changed their servers to require TLS 1.2, which may"
        ui.error "not be available to your system installed Ruby (#{RUBY_VERSION})"
      end
      ui.error ""
      ui.error "The best solution is to use the self-contained fastlane version."
      ui.error "Which ships with a bundled OpenSSL,ruby and all gems - so you don't depend on system libraries"
      ui.error " - Use Homebrew"
      ui.error "    - update brew with `brew update`"
      ui.error "    - install fastlane using:"
      ui.error "      - `brew cask install fastlane`"
      ui.error " - Use One-Click-Installer:"
      ui.error "    - download fastlane at https://download.fastlane.tools"
      ui.error "    - extract the archive and double click the `install`"
      ui.error "-----------------------------------------------------------"
      ui.error "for more details on ways to install fastlane please refer the documentation:"
      ui.error "-----------------------------------------------------------"
      ui.error "        🚀       https://docs.fastlane.tools          🚀   "
      ui.error "-----------------------------------------------------------"
      ui.error ""
      ui.error "You can also install a new version of Ruby"
      ui.error ""
      ui.error "- Make sure OpenSSL is installed with Homebrew: `brew update && brew upgrade openssl`"
      ui.error "- If you use system Ruby:"
      ui.error "  - Run `brew update && brew install ruby`"
      ui.error "- If you use rbenv with ruby-build:"
      ui.error "  - Run `brew update && brew upgrade ruby-build && rbenv install 2.3.1`"
      ui.error "  - Run `rbenv global 2.3.1` to make it the new global default Ruby version"
      ui.error "- If you use rvm:"
      ui.error "  - First run `rvm osx-ssl-certs update all`"
      ui.error "  - Then run `rvm reinstall ruby-2.3.1 --with-openssl-dir=/usr/local`"
      ui.error ""
      ui.error "If that doesn't fix your issue, please google for the following error message:"
      ui.error "  '#{e}'"
      ui.error "-----------------------------------------------------------------------"
    end

@KrauseFx thank

Glad to help 👍

@KrauseFx Sorry, It not work


new copy roles added

@scoen , @MingtaoSun , @yetti , @AaronCMuller , @ninhnguyen can you please look.
new copy roles added http://ourweb.nla.gov.au/apps/jira/browse/DSCS-1559

:+1:


Stache: Helper - stache not providing temporary element for element callback

In a stache template helper, using the element callback helper, the temporary element is not provided to the callback, but instead an empty string is provided:
With Mustache
http://jsfiddle.net/pt5cps21/2/

var rndr = can.mustache("
{{items}}
");

can.mustache.registerHelper("items", function(options){
  return function(li){
    console.log(li);
    li.innerHTML = "I am an item";
    return li;
  };
});

var frag = rndr();

document.querySelector('body').appendChild(frag);

output:
* I am an item

The temporary element (an li in this case) is created and passed as first argument to the callback.
With Stache
http://jsfiddle.net/pt5cps21/3/

var rndr = can.stache("
{{items}}
");

can.stache.registerHelper("items", function(options){
  return function(li){
    console.log(li);
    li.innerHTML = "I am an item";
    return li;
  };
});

var frag = rndr();

document.querySelector('body').appendChild(frag);

output:

The first argument ends up being an empty string. (and therefore not working)

I just checked this, it does provide a textnode element.  Put a debugger in your fiddle and you'll see it's a text node.
This is not a bug.  Closing.

The Documentation says it should (and Mustache does) return a  as the default element

And other matching elements (e.g. a
 when you are in a
)

This is not how stache works.  If the docs for stache says this, they are wrong.  Can you point to that in stache's docs?  I can update them.
Stache should not work this way ... it would undermine a big part of its performance benefits.

If you need this, you can always swap in the element you need.
function(){
  return function(textNode){
    var tagName = can.view.elements.tagMap[ textNode.parentNode.nodeName ] || "span"
    var el = document.createElement(tagName);
    $(textNode).replaceWith(el)
  }
}

Looks like the original link was linked to localhost, I've edited it.
http://canjs.com/docs/can.stache.helper.html#section_Returninganelementcallbackfunction
So I'll just add this as a documentation error then.


get_service -> update_service drops some parameters

Environment details

Python version: 3.11.2
pip version: 23.0.1
google-cloud-run version: 0.7.1

Steps to reproduce

Create Cloud Run instance with "Startup CPU boost" enabled.
Run the following code sample on such instance.

Code example
import google.cloud.run_v2
instance = 'projects/.../locations/.../services/...'
client = google.cloud.run_v2.ServicesClient()
service = client.get_service(name=instance)
service.template.revision = ''
client.update_service(service=service)

Expected result
Service is redeployed with identical settings.
Actual result
Service is missing at least the "Startup CPU boost" option.
I'd say it is due to ResourceRequirements class missing the startupCpuBoost parameter. In JSON it is represented as a boolean:
        "resources": {
          "limits": {
            "cpu": "1000m",
            "memory": "1Gi"
          },
          "startupCpuBoost": true
        },

Thanks for reporting this issue. I'm going to close this issue as a duplicate of https://issuetracker.google.com/issues/279052397. Please follow the discussion in https://issuetracker.google.com/issues/279052397. Please open a new issue if you have any additional questions.

The change has been merged last week:
https://github.com/googleapis/python-run/commit/c907927b59079867825954786192954293aa6184


vkGetRefreshCycleDurationGOOGLE() doesn't work on multi-display.

@billhollings @spnda I finally got a moment to test merged PR #1596 on a dual-display setup, and it doesn't work on non-main displays, e.g., an external display connected to a MBP - It always falls back to querying the NSMainScreen, regardless of which screen the window is on.
At least not with my code that manually creates a CAMetalLayer and assigns it to the contentView of my NSWindow, ie., assigned via:
[[window contentView] setWantsLayer:YES];
[[window contentView] setLayer: [CAMetalLayer layer]];

I think this is called a layer hosting view?
I haven't tested this with MTKView or a self-made layer backed NSView, so maybe this is just a failure specific to above method, but above method is recommended in various places as one of the valid ways to do it.
The while(layer.superlayer) - loop finds and selects a superlayer which does not match the view.layer of the windows contetnView, so no NSWindow is found -> No proper NSScreen is assigned and we always end with the NSMainScreen, which often is the wrong choice for a multi-display setup.
One simple solution is to remove that while loop and stick with _mtlLayer instead of layer. Iow, my original solution from my closed pull request #1358, with its associated limitations.
Another thing i tried, and which also worked for me, is the following replacement for the while(layer.superlayer) loop:
while (![layer.delegate isKindOfClass: [PLATFORM_VIEW_CLASS class]] && layer.superlayer) {
	layer = layer.superlayer;
}

if ([layer.delegate isKindOfClass: [PLATFORM_VIEW_CLASS class]]) {
    NSView* view = (NSView*) layer.delegate;
    NSWindow* window = view.window;
    if (window) {
        screen = [window screen];
    }
}

Iow. iterate upwards through the superlayers if needed, until a layer.delegate is found which is a NSView. Use that views associated NSWindow to get the screen.
This was inspired by the implementation of MVKSurface::initLayerObserver(), which uses the Metal layers delegate as a source of wisdom for the layer observer, assuming that delegate is an NSView or UIView. And some bits in Apple docs which suggests that if a view is created as a CAMetalLayer backed view, it will assign itself as the layers delegate.
Also I just realized that this is very similar to what @cdavis5e proposed in our original discussion at comment https://github.com/KhronosGroup/MoltenVK/pull/1358#discussion_r628809014
I don't know if the latter solution is better than simply going back to my original proposal, or something else, my understanding of all this layer stuff is very limited. Also, I'm out of time again for further testing or refining this. But adding this additional snippet would be a step into the right direction i guess, although short of attempting the more comprehensive solution attempt that @cdavis5e proposed in his comment.

I find it weird that the NSView backing a NSWindow can have a super layer... but so be it. Sorry about this though.
Your suggested solution that did work with manually setting the CAMetalLayer on the NSWindow would work in your case, but as previously discussed won't work with AppKit. So, I just thought of this other implementation, and while it really isn't the quickest (though most apps will only have a single NSWindow) it should actually be the best solution. This doesn't assume that the highest level view is in fact the window's contentView but will instead iterate as far as needed. Would you mind testing this snippet?
	CALayer* layer = _mtlLayer;
	NSScreen* screen = nil;
	NSApplication* app = [NSApplication sharedApplication];
	while (layer.superlayer) {
		layer = layer.superlayer;

		for (NSWindow* window in [app windows]) {
			NSView *view = [window contentView];
			if (view && ([view layer] == layer)) {
				screen = [window screen];
			}
		}

		if (screen != nil)
			break;
	}

What if it's a subview of the content view?

Well that's why we do a loop checking each super layer? We go up the view tree until we arrive at the window's contentView. Or does this not work?

Well that's why we do a loop checking each super layer? We go up the view tree until we arrive at the window's contentView. Or what exactly do you mean?

What if the subview is a layer-hosting view, but the content view doesn't use layers at all? (Is that even possible?)

Still surprised that the currently implemented code doesn't work. How is the topmost layer not the layer assigned to NSWindow directly... To quote Apple's docs:

The window’s content view, the highest accessible view object in the window’s view hierarchy.

What if the subview is a layer-hosting view, but the content view doesn't use layers at all? (Is that even possible?)

Do you know of any situation where this happens with normal Apple API? Though if you want to be really pedantic, you could also check all superviews in the NSView.
    NSScreen* screen = [NSScreen mainScreen];
    CALayer* layer = _mtlLayer;
    NSApplication* app = [NSApplication sharedApplication];
    while (layer.superlayer) {
        layer = layer.superlayer;

        for (NSWindow* window in [app windows]) {
            NSView *view = [window contentView];
            if (view == nil)
                continue;

            if ([view layer] == layer) {
                screen = [window screen];
                break;
            }

            // Use a C++ lambda instead?
            std::function<NSScreen*(NSView*)> checkSubviews = ^(NSView* subview) {
                for (NSView* kSubview in [subview subviews]) {
                    if ([kSubview layer] == layer) {
                        return [window screen];
                    } else {
                        checkSubviews(kSubview);
                    }
                }
                return (NSScreen*)nil;
            };
            screen = checkSubviews(view);
        }

        if (screen != nil)
            break;
    }

With many NSView's and NSWindow's this could get crazy expensive... but for most people this would work in pretty much all cases now. At this point, however, I ask myself why we don't ask the user for a NSWindow or the topmost NSView when creating the surface?

Well that's why we do a loop checking each super layer? We go up the view tree until we arrive at the window's contentView. Or what exactly do you mean?

What if the subview is a layer-hosting view, but the content view doesn't use layers at all? (Is that even possible?)

Fwiw: Docs for the NSView wantsLayer instance property say: "Setting the value of this property to YES turns the view into a layer-backed view—that is, the view uses a CALayer object to manage its rendered content. Creating a layer-backed view implicitly causes the entire view hierarchy under that view to become layer-backed. Thus, the view and all of its subviews (including subviews of subviews) become layer-backed."

PR #1669 should fix this across multiple display screens. Please test with latest.
That PR now uses both techniques...first attempting to retrieve the layer's delegate view, through which to retrieve the screen, and if this fails, attempts to retrieve the window the layer is on by iterating through the windows and then the layers of the window's contentView.
That second iterative approach is now modified to check first the CAMetalLayer, and each super layer against the contentView layer. It checks each level of the layer hierarchy, beginning at the bottom, because it appears that Apple can sometimes add synthetic super layers under the covers, even though the view remains attached to the lower-level view. So we need to accommodate both an app-created layer hierarchy within the view, and a synthetic CA-created layer hierarchy.

Just retested with a dual-display 60 Hz + 120 Hz setup and can confirm this works correctly with my setup, both when setting a suitable layer delegate, and with the slower fallback path. Tested on Catalina and Monterey.
Thanks.


OntologyInfoWidget does not load values if "https://www.ebi.ac.uk/ols/api/" is selected

The primary and only story of OntologyInfoWidget has the SemLookP api and ontologyId='atc' as args. As ebi's api does not feature an ontology with id atc, changing the api obviously runs into the widget not loading. This can be confusing to a storybook user, but is no bug, so this issue will be closed again.


check for message attribute

Describe your changes
Added check for message attribute when displaying Slack link. This is required for exceptions which have message as their first parameter and not args.
Issue number
Closes #X
Checklist before requesting a review

[ ] Performed a self-review of my code
[ ] Formatted my code with pkgmt format
[ ] Added tests (when necessary).
[ ] Added docstring documentation and update the changelog (when needed)

:books: Documentation preview :books:: https://ploomber-core--81.org.readthedocs.build/en/81/

@neelasha23 I'm guessing this is because of: https://github.com/ploomber/cli/issues/7 ?
is it because ClickException has message? https://github.com/ploomber/cli/blob/main/src/ploomber_cloud/exceptions.py

@neelasha23 I'm guessing this is because of: ploomber/cli#7 ?

yes

is it because ClickException has message? https://github.com/ploomber/cli/blob/main/src/ploomber_cloud/exceptions.py

Yes it doesn't seem to be accessing the args attribute while displaying the error. It has the message attribute : https://click.palletsprojects.com/en/7.x/api/#click.ClickException. Setting this is enabling the community link display
@edublancas

@edublancas Ready for review.

I've released this in 0.2.19: https://github.com/ploomber/core/commit/538e7e7a8bfc5a887aead8d04e7382d7de5e9ae5
should be on PyPI in a few minutes, please update your other PR and pin ploomber-core>=0.2.19 with a note explaining that we need this version bc of this change


Fixing Swarm Install with Machine doc

This fixes #1767 and partially addresses #1771.
Changes include mounting the certs directory into Swarm manager container and updating port numbers for swarm join.
Signed-off-by: Nishant Totla<EMAIL_ADDRESS>

cc @moxiegirl @dongluochen @vieux

LGTM

LGTM

LGTM

Thanks for these fixes @nishanttotla


Serving slides from S3 is very slow

Hi. I have been trying to host slides on S3 (through Girder's S3 assetstore) and it appears that HistomicsTK can load WSIs from S3. However, the speed is very low and loading tiles sometimes takes more than 10-15 seconds. Is this expected? Is there a particular setup that I need to follow to speedup S3 queries?

Some S3 storage is faster than others.
For instance, I can connect to Google storage via the S3 compatible api.  This is slow; the first tile often takes 10s and subsequent tiles takes from 40 ms to 4 s.  The initial slow speed is purely the time it takes to fetch bytes from the bucket.
However, the same exact file when stored in my own S3 bucket takes less than a second to get the first tile.
I'm not knowledgeable enough in S3 setup to know what configurations are best.

What regions are the s3 bucket and server in? If they are not the same..
that's the first issue
On Tue, Nov 19, 2019, 10:53 AM David Manthey<EMAIL_ADDRESS>wrote:

Some S3 storage is faster than others.
For instance, I can connect to Google storage via the S3 compatible api.
This is slow; the first tile often takes 10s and subsequent tiles takes
from 40 ms to 4 s. The initial slow speed is purely the time it takes to
fetch bytes from the bucket.
However, the same exact file when stored in my own S3 bucket takes less
than a second to get the first tile.
I'm not knowledgeable enough in S3 setup to know what configurations are
best.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/DigitalSlideArchive/HistomicsTK/issues/759?email_source=notifications&email_token=AAFODTV5PMX24JYOIDE3V5DQUQDWXA5CNFSM4JPDCO22YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEEOVO2A#issuecomment-555571048,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAFODTXWUMM5QZC6T6VJWUDQUQDWXANCNFSM4JPDCO2Q
.

Thank you both. In my case, some tiles take a few milliseconds while others can take more than 15 seconds. Both server and S3 bucket are in Singapore.

Is Girder using S3 Select for partial retrieval of objects from S3 or does it mount S3 bucket and then accesses the objects as files? I was wondering using S3 Select could potentially improve the performance significantly, particularly for WSI objects. Thanks!

Girder accesses S3 objects as files, but does range-requests for them as needed.  When accessing S3 objects as large_image tile sources, the request is sent through the girder-mount Fuse.  By default, linux will buffer reads -- rather than making one request per read, a request is made that reads a larger block of data, then that data is served from memory if the requested area is available.
I've looked at the network traffic for pyramidal tiff files as generated by vips; for those, the requests end up being fairly efficient in terms of a low number of requests and total bytes transferred.  Other formats are probably much less efficient.
It might be worth seeing if using fadvise would improve access times.

AWS recommended us to implement S3 Select as it appears to be the most efficient way of retrieving part of objects from S3. They also advised against FUSE. Are there any plans to extend Girder to use S3 Select? I guess that would cause some issues because other providers like GCP still don't have an equivalent.

There aren't currently plans to use S3 Select.  None of our funded projects currently require faster S3 access than what they have.  Targeted funding or PRs are always welcome -- it shouldn't be too much work to do it.

Thanks. We will try to implement it and make a PR. Would you please give us some pointers to start?

I would expect this code that needs to change is here: https://github.com/girder/girder/blob/master/girder/utility/s3_assetstore_adapter.py#L369-L408

Thanks for the pointer. I'm closing this issue.


mjml-font not working

I am trying to include Google Fonts into my mjml file, however, it seems as if only the "Raleway" font works.

Can you share your (complete) code?
Kind Regards,
:----------------------------:
Dale McConnell
On 15 February 2017 at 15:25, pmacswebteam<EMAIL_ADDRESS>wrote:

I am trying to include Google Fonts into my mjml file, however, it seems
as if only the "Raleway" font works.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mjmlio/mjml/issues/554, or mute the thread
https://github.com/notifications/unsubscribe-auth/AKxLOFKxCfX1wRSqWKZtN4cghAIUBJ8sks5rcwrEgaJpZM4MBxnN
.

`

Hello world

<!-- Header -->
<mj-section background-color="#036" border-right="15px solid #600">
  <mj-column>
    <mj-image src="http://www.med.upenn.edu/pmacsnewsletter/images/psom_logo_white.png" width="213"/>
  </mj-column>
  <mj-column width="100%">
    <mj-text align="center" font-size="18" color="#39C" font-family="Slabo, sans-serif">
      Penn Medicine Academic Computing Services
    </mj-text>
    <mj-text align="center" font-size="32" color="#FFF" font-family="Slabo, sans-serif">
      Newsletter | Winter 2017
    </mj-text>
  </mj-column>
</mj-section>

<!-- Banner Image -->
<mj-section>
  <mj-image src="http://www.med.upenn.edu/pmacsnewsletter/images/front_image.jpg" width="0"/>
</mj-section>

<!-- Main Content -->
<mj-section></mj-section>

`
Thanks!

Hi @pmacswebteam !
Your mj-font is inside mj-attributes that's why it doesn't work, fix : https://mjml.io/try-it-live/HksYGbftg

Hello, I moved it outside of attributes as you suggested, however, the correct google font still is not being pulled in. It should look like this: https://fonts.google.com/?query=slabo

It does not appear to be working in the example you provided here as well:  https://mjml.io/try-it-live/HksYGbftg
Thanks again for your help!

Hi @pmacswebteam
Looks like to work on first paragraph, and working on the 2nd when removing sans-serif

Hi, I've attached a screenshot of what I'm seeing

Ok, I have a Slabo font on this computer that explain it.
But  https://fonts.googleapis.com/css?family=Slabo throw a 400, so that's probably why it doesn't apply on your side

Ah, I see what I was missing. Thanks for your help with this!


get travis ci working

@EamonNerbonne FWIW now that we have CI working, this should hopefully help prevent any future recurrence of the truncation issue. :smile:

That's one long-running test there :-).  You managed to capture the race condition somewhat reliably in a test?  (incidentally, this race-condition occurred for me even for much shorter runs of a few seconds, though I'm not sure it did so reliably)

"Reliably" might be relative here, but of the ten or so times I ran the
long-running test, it always failed (hitting the output truncation) within
the first 10 seconds.
Since you have me curious now, I'll checkout that commit (the commit work
the failing test went in before the fix) and have it run to failure N times
and see how long it takes to fail.
Admittedly, this is very likely to be highly machine specific, and I'm not
sure how long it would take it to fail on the Travis CI docker containers.
However, I'll make a branch and pull request to see what Travis reports. :)
On Aug 18, 2016 3:20 AM, "Eamon Nerbonne"<EMAIL_ADDRESS>wrote:

That's one long-running test there :-). You managed to capture the race
condition somewhat reliably in a test? (incidentally, this race-condition
occurred for me even for much shorter runs of a few seconds, though I'm not
sure it did so reliably)
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/jamesmanning/RunProcessAsTask/pull/5#issuecomment-240643832,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AANxWqYXGd53eO6ge0VVQ2vjOLfjtHekks5qhAfBgaJpZM4Jmbb7
.

@EamonNerbonne I couldn't get it to reliably happen in a reasonable amount of time when running 1 at a time, but it always happened in 60 seconds or less once I had it run many in parallel
Parallel.ForEach(Enumerable.Range(1, 100), index => WhenProcessReturnsLotsOfOutput_AllOutputCapturedCorrectly());

Funny enough, once I did that, along with the original "sometimes the output is truncated", I now got crashes as the stdout.ToArray() call would fail during process exit, as it would hit a race condition where a new item got inserted after it had already allocated the target array to copy to.
https://ci.appveyor.com/project/jamesmanning/runprocessastask/build/1.0.18
System.ArgumentException: Destination array was not long enough. Check destIndex and length, and the array's lower bounds.
   at System.Array.Copy(Array sourceArray, Int32 sourceIndex, Array destinationArray, Int32 destinationIndex, Int32 length, Boolean reliable)
   at System.Collections.Generic.List`1.ToArray()
   at RunProcessAsTask.ProcessEx.<>c__DisplayClass1_0.<RunAsync>b__2(Object sender, EventArgs args) in C:\projects\runprocessastask\src\RunProcessAsTask\ProcessEx.cs:line 50
   at System.Diagnostics.Process.OnExited()
   at System.Diagnostics.Process.RaiseOnExited()
   at System.Diagnostics.Process.CompletionCallback(Object context, Boolean wasSignaled)

Since this was testing the known-broken original code (before any fix applied), I didn't bother going down the path of trying to catch the exception in the Exited event to continue to get more timing numbers, but thought you might find that amusing since you had correctly stated that the ToArray wasn't thread-safe (since the output hadn't closed and was still writing to the list).
In any case, I feel fairly (not highly) confident that with the parallelism enabled now, if it survives for 5 minutes, then that bug hasn't regressed. :smile:

This kind of stuff you'll never manage to test exhaustively anyhow - at least, not without some fairly invasive, hard to set up threading simulators.  Neat that you managed to get that far in the first place!


Add Command options' meaning

It is always better to explicitly mention the meaning of each option(s) provided with mender artifact command.
Added the meaning of provided options for update module example.
Changelog: None
Signed-off-by: Ajith P V<EMAIL_ADDRESS>(cherry picked from commit d838fe41cf110166f3f9a05ea121bdf3ff5fa7d8)

Cherry-picking #691


only run one instance 'cron.run()'

If call twice
func (c *Cron) Start() {
	c.running = true
	go c.run()
}

So is this a question or a statement?

I mean Start() method shoud be a singleton method, to protected call Start()  many times, or appliction must be protected self.

@winxxp thanks. That makes sense. Probably the Start should be like this:
func (c *Cron) Start() {
	if c.running {
		return
	}
	c.running = true
	go c.run()
}

thread-safe better.

Is there a good reason your application can't call Start once?
Or are you saying, in case a poorly behaved application calls Start more than once, then cron should handle that situation better?

I used zookeeper to elect master,  when network broken or other slave online,  all node recvived new nodes to elect new leader, if leader call start, or stop, sometime  the elected reslt is the leader  node also leader node, so it call twice or more.
Now I protected it out Start/Stop self.


Publication is not loading for https://www.pdxfinder.org/data/pdx/TRACE/EMC0049PM#patients

Publications section is not loading for the given url.
The problem:
The pubMedId contains the character   which prevents the PublicationService class to get the data from europepmc.
Proposed solution:
Remove special character from the id.

PR created: https://github.com/PDXFinder/pdxfinder/pull/531

@zperova fixed. See http://dev.pdxfinder.org/data/pdx/TRACE/EMC0049PM


RTC startup problem after login screen ( after 2.5.5)

Hi Cedric,
I am using version 2.5.5 without any problems. I am grateful
I checked the updates and wanted to install version 2.9.2 but. somehow i couldn't get it to work.
I am using Windows Server 2019 Datacenter. windows defender and firewall have been turned off. It doesn't even have a virus program.
Login screen comes up, I enter the information. After that, the blank screen remains. There is a menu on the top.
I have tried many things but have not been successful. And I decided to ask you for help :)
I'm sending the log files to help.
Myrtille.Services.log
Myrtille.Web.log
Thank you so much for your effort and support...

Hi, from the logs it looks like the Myrtille frontend and backend can't communicate, but there is nothing changed on that side since 2.5.5. The Myrtille services are started, listening on local port 8080 (tcp), but the Myrtille web application can't communicate with them. Do you have something also using the port 8080 besides Myrtille? You can change the default port into Myrtille.Services and Myrtille.Web config files; restart Myrtille.Services (Windows service) then.

Hi Cedric,
Thank you for the answer.
As I said before, the 2.5.5 installation works without any intervention.
I uninstall version 2.5.5 and install version 2.9.2 on the same machine, but it doesn't work.
Like you said I changed port 8080 to 9090. To avoid mistakes, I uninstalled and reinstalled the program (9090) but the result still didn't work. I am posting the attached log folder as it may help you.
log.zip

Could you check if the windows service "Myrtille.Services" is started? the log file Myrtille.Services.log doesn't show any entry, unlike the previous logs you posted, and this is not normal. This service must be set to "automatic" start type, started and running under the LocalSystem account.
You still have the same issue, the Myrtille frontend (Myrtille.Web) can't communicate with the backend (Myrtille.Services). This is normally done by the installer, but could you check if you enabled WCF HTTP activation? (https://github.com/cedrozor/myrtille/blob/master/DOCUMENTATION.md#installation):

Into the roles and features management, ensure you have enabled HTTP Activation under ".NET Framework 4.5+ Features" > "WCF Services" (required by the Myrtille services and REST API).

yes, both services are working correctly.
I read the installation documentation many times before notifying you a problem.
(https://github.com/cedrozor/myrtille/blob/master/DOCUMENTATION.md#installation):
Windows Server 2019 Datacenter,IIS 10 and .NET Framework 4.7 is installed on my server
windows defender disabled. There is no virus program. no firewall and no proxy
I'm doing the tests on the local network.
I don't know if it makes the difference, but the operating system is Turkish and the regional options are Turkey.

From the myrtille machine, could you try to access the urls of the myrtille services? for example, using a browser: http://localhost:8080/Myrtille/MFAAuthentication
What is the output you have?

I've tried.
HTTP Hatası 404.0 - Not Found

I've tried.
MFAAuthentication Hizmeti
Bir hizmet oluşturdunuz.
Bu hizmeti sınamak için bir istemci oluşturmanız ve hizmeti çağırmak için onu kullanmanız gerekir. Komut satırında aşağıdaki sözdizimiyle svcutil.exe aracını kullanarak bunu yapabilirsiniz:
svcutil.exe http://localhost:9090/Myrtille/MFAAuthentication?wsdl
Hizmet açıklamasına tek bir dosya olarak da erişebilirsiniz:
http://localhost:9090/Myrtille/MFAAuthentication?singleWsdl
Bu, istemci sınıfını içeren bir yapılandırma dosyası ve kod dosyası oluşturur. İki dosyayı istemci uygulamanıza ekleyin ve Service işlevini çağırmak için oluşturulan istemci sınıfını kullanın. Örneğin:
C#
class Test
{
static void Main()
{
MFAAuthenticationClient client = new MFAAuthenticationClient();
    // Hizmetteki işlemleri çağırmak için 'client' değişkenini kullanın.

    // Her zaman istemciyi kapatın.
    client.Close();
}

}
Visual Basic
Class Test
Shared Sub Main()
Dim client As MFAAuthenticationClient = New MFAAuthenticationClient()
' Hizmetteki işlemleri çağırmak için 'client' değişkenini kullanın.
    ' Her zaman istemciyi kapatın.
    client.Close()
End Sub

End Class

Myrtille services are running (Myrtille.Services.log is now ok), but you have the following into Myrtille.Web.log:

2021-05-14 18: 03: 17,482 [9] ERROR System.Diagnostics redirection [(null)] - Failed to get enterprise adapter mode (System.ServiceModel.EndpointNotFoundException: Listen at http://localhost:9090/Myrtille/EnterpriseService This is usually due to an incorrect address or SOAP action. For details, see InnerException if available. ---> System.Net.WebException: Cannot connect to remote server ---> System.Net.Sockets. SocketException: Connection failed because target machine actively refused <IP_ADDRESS>:9090

This means you have something preventing communication from the Myrtille web application (hosted by IIS) and the Myrtille services (self-hosted WCF). Do you have a specific IIS configuration on the machine? did you changed the application pool used by Myrtille?
and you have this later:

2021-05-14 23: 34: 51,133 [10] ERROR System.Diagnostics redirection [(null)] - Failed to get enterprise adapter mode (System.ServiceModel.EndpointNotFoundException: Listen at http://localhost:9090/Myrtille/EnterpriseService There was no endpoint to accept the message. This is usually due to an incorrect address or SOAP action. See InnerException for details if available. ---> System.Net.WebException: The remote server returned an error: (404) Not found.

It's because Myrtille.Services (Windows service) was stopped at this time.
Is http://localhost:9090/Myrtille/EnterpriseService working from a browser? (like it was working with http://localhost:9090/Myrtille/MFAAuthentication).

I am sending all settings visually so that I can help you and avoid any misunderstanding.
I did not make any settings in IIS in 2.5.5.
deneme yanılma yaparken bazı ayarları değiştirmiş olabilirim.
MySettings.docx

Yes, please try to install Myrtille on a fresh machine. It's probably a system/config issue after you uninstalled 2.5.5 and installed 2.9.2. If you didn't uninstalled 2.5.5 before installing 2.9.2, the installer tried to upgrade Myrtille but something might have gone wrong (the upgrade support was introduced into 2.8.0).

Did it worked with a fresh install? please keep us updated

Hi Cedric,
Unfortunately I couldn't try.
I will try and get back to you as soon as possible.


[G2M] Plotly - fix formatted value on hovertext for tooltip

fixes #202

Fixed decimals format value for tooltip on hover
Added percentage check for tooltip on hover text display ( when the total sum value is 100, text value won't be display instead only percentage will be visible)

BEFORE:

AFTER:

UPDATE: ( percentage check for tooltip )
https://user-images.githubusercontent.com/46459367/195700351-1f59ca46-cd93-4cab-bc98-2fb150acc351.mov

@Hamaad102 I added hovertext in for getObjectByType in typeConfig return so I have to add it into the plotly.test.js array for each chart? I am not passing the test check there

@kc-leung a bunch of tests are failing

@kc-leung you would have to do provide a value unfortunately, otherwise you could add a default value to the function?

@kc-leung a bunch of tests are failing

yea I am asking @Hamaad102 to confirm me a question regarding the fail cuz I added a new value in the object return. It seems that I have to add into the test as well because in the object value check doesn't have that newly added value

@kc-leung while we get the reviews in, could you please upgrade package to 0.8.1-alpha.1 and make a pre-release using this branch so I can test it through widget-studio & snoke. I am trying to get this in the Cox app today.

@kc-leung the percentage value logic that you implemented for Pie chart can also appear in the Bar chart.
Also, I mentioned yesterday, we should not show the unformatted value if we don't have a case when the values amount to 100%.
Like in this Bar chart, the value appears twice in tooltip, we should only keep the formatted one in this case.

@kc-leung are these changes only for the pie chart? I noticed on the plotly bar horizontal formatted that the decimal values are still unformatted.

Otherwise, it all looks good!

@kc-leung are these changes only for the pie chart? I noticed on the plotly bar horizontal formatted that the decimal values are still unformatted.

Otherwise, it all looks good!

I only added the feature for Pie. I just had a talk with Erika to add it for all the other charts as well


Add gitcode.net

Certainly, the site distributes mirrors of well-known repositories on GitHub.com, but it also seems to host a number of homegrown repositories in addition to that.
I do not know the position of this site in the Chinese developer community, but it is less malicious than the other sites listed in this project.


Reset button doesn't restart program when flashed with espflash

I am flashing an esp32-c6-WROOM-1 devkit using espflash. The program runs fine but when I click the reset button on the board the board freezes but does not restart the program. E.g. a blinky application will freeze the LED in the state at the time of the reset button push. The program does not start up again.

Hi! Few questions:

Can you open the monitor after flashing?

If so, have you tried resetting the board with Ctrl+R?

Have you tried reproducing this behavior with esptool

Not sure if this is related with espflash if the flashing/monitoring does not throw any errors.

Yes I can open the monitor after flashing. Ctrl + R does restart the board. I can flash the board see the serial out put use ctrl r to restart. When I click the reset button the board freezes (blinky will freeze)  and the serial monitor disconnects. I can still re-connect the monitor and use ctrl + R to restart the program but the reset button will never restart the program.
Will try with esptool

I should also note that when I click the reset button the board completely freezes in whatever state it is in. e.g. if the LED is on the board will freeze with the LED on and there is no way to get it to go off without unplugging. No combination of BOOT or RESET button pushes will turn off the light

I am unsure how to use esptool to flash the board, the documentation for write_flash references a bootloader.bin but my rust program does not output a file like that. I just have /target/riscv32imac-esp-espidf/release/CRATE_NAME after running cargo build --release --target riscv32imac-esp-espidf.json

I am unsure how to use esptool to flash the board, the documentation for write_flash references a bootloader.bin but my rust program does not output a file like that. I just have /target/riscv32imac-esp-espidf/release/CRATE_NAME after running cargo build --release --target riscv32imac-esp-espidf.json

See elf2image command and, in case you need the bootloader, you can download it from https://github.com/esp-rs/espflash/tree/main/espflash/resources/bootloaders

Will try tonight, thanks

Will try tonight, thanks

Hi! Do you have any updates?

Hello @seanaye, do you have any updates?

I will be closing the issue as it's inactive, feel free to reopen it if the issue is still present


Add UUID field for Problem

Just started reviewing :) :octocat: Sent from GH.

Hey @jtydhr88 check the failing tests, they are failling for me locally as well.

Please resend, thanks! :octocat: Sent from GH.


BUG: console is locked after bps.input_plan()

After using the plan stub that requests command-line input from a user, the terminal sessions is locked and no keyboard input is accepted.  (all typing is ignored, including ^C and [Enter])
Problem is 100% reproducible for us.
Expected Behavior
Keyboard should remain function after the plan, regardless of what is typed.
Current Behavior
keyboard input is ignored
Possible Solution
?
Steps to Reproduce (for bugs)
start ipython session in terminal window
import bluesky
import bluesky.plan_stubs as bps
RE = bluesky.RunEngine({})

def question():
    r = yield from bps.input_plan("Simple question: ")
    print(f"received: {r}")

RE(question())

After typing a response, the plan completes but all keyboard input from this time will be ignored.
Here's a session with that:

Context
User (APS mirror coating laboratory) wants interactive user confirmation before executing certain procedures.  bps.input_plan() looks like it is intended to do provide this interaction.
Your Environment
Observed in both RHEL8 (gnome) and Linux Mint (mate) terminals.
In [2]: import bluesky

In [3]: bluesky.__version__
Out[3]: '1.8.1'

Is this code executed?  https://github.com/bluesky/bluesky/blob/566dce9e7ada95e2346c9aa61206c6a2f0a189bf/bluesky/utils/init.py#L1038-L1053

Yes, per https://github.com/bluesky/bluesky/blob/dc8b36fb1030a9116b6fe908ccdd32b58e254804/bluesky/run_engine.py#L2407-L2417

I am very glad someone actually wants this :) I always thought this was a cool feature, but no one at NSLS-II seemed interested 🤷🏻
The core of the problem is that we are using readline https://github.com/bluesky/bluesky/blob/05238c2177c0d4b5e58905d65395b079454aabfb/bluesky/utils/init.py#L1048-L1049 but IPython now uses prompt toolkit.  That code needs to get a bit smarter and detect if we in a readline on prompt toolkit context.

Great report!  In case it helps:
(bluesky_2022_1) prjemian@zap:~$ conda list prompt
# packages in environment at /home/prjemian/.conda/envs/bluesky_2022_1:
#
# Name                    Version                   Build  Channel
prompt-toolkit            3.0.20             pyhd3eb1b0_0
prompt_toolkit            3.0.20               hd3eb1b0_0
(bluesky_2022_1) prjemian@zap:~$ conda list readline
# packages in environment at /home/prjemian/.conda/envs/bluesky_2022_1:
#
# Name                    Version                   Build  Channel
readline                  8.1                  h27cfd23_0

(venv) root ➜ /workspaces/bluesky (1783-drop-events-that-are-out-of-date) $ python
Python 3.12.4 (main, Aug  2 2024, 14:41:30) [GCC 12.2.0] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import bluesky
>>> import bluesky.plan_stubs as bps
>>> RE = bluesky.RunEngine({})
>>> print(RE)
<bluesky.run_engine.RunEngine object at 0x7f3a27437200>
>>> def question():
...     r = yield from bps.input_plan("SImple question: ")
...     print(f"received: {r}")
...
>>> RE(question())
SImple question: hello
received: hello
()
>>> b

verified that this happens on 08/06/2024 with
> bluesky.__version__'1.13.0a5.dev24+ga4978f22'
and prompt_toolkit==3.0.47
and no readline package.


s3fs: Update to 1.79

Please don't forget to rename commit's name following CONTRIBUTING.md

Done, tnx!


NPE in KubernetesDiscoveryClient when endpoints are not available

On 0.2.0-RELEASE
When KubernetesDiscoveryClient goes through services and discovers a service that has been scaled down or has no available endpoints it will throw a NPE.    The stack trace is below.    Essentially the endpoints API returns a 404 and thus has no subsets to map to the Endpoint object.
Caused by: java.lang.NullPointerException: null
at org.springframework.cloud.kubernetes.discovery.KubernetesDiscoveryClient.getInstances(KubernetesDiscoveryClient.java:85)
at org.springframework.cloud.client.discovery.composite.CompositeDiscoveryClient.getInstances(CompositeDiscoveryClient.java:47)

I worked around this by modifying it to deal with no endpoints as an EmptyList.  Not sure if this is the best place to address this issue or if the KubernetesClient should have different handling for a 404 on endpoints lookup.
return Optional.ofNullable(                 Optional.ofNullable(client.endpoints().withName(serviceId).get()).orElse(new Endpoints())                         .getSubsets()).orElse(Collections.emptyList())                 .stream()

If you would like us to look at this issue, please provide the requested information. If the information is not provided within the next 7 days this issue will be closed.

No, I was able to remove my workaround with 0.3.0-RELEASE.


Builtin Documentation

Second try, now without a singleton.
I created a system to have a built-in documentation.
How it looks in the master tree:

An example page. Pages are saved as markdown files:

Longer pages have a content menu. There is always a back to the top button:

It works okay with the light themes:

Here is the file system the documentation is made from:

Things the markdown parser/page viewer supports right now:

Bold text **
Lists (+-*)
Mono ``
Underlines __
Striketrhough ~~
Headings 1-5
images with github path, relative paths or godot paths (no sizes right now, all )
links (are opened in the browser or emit a signal that can be used to open another docs page in godot) can use www-paths, godot paths, relative paths or section paths (with a #)
The page viewer has a back to top button and a content menu that contains all ##heading2s
The page viewer has an edit mode that enables a refresh and edit button that are VERY usefull when editing the documentation.

There are some more things that come with this documentation:

A DocsTreeHelper that is used to create a documentation tree (or add it to an existing tree) from the documentations content folder.

The helper looks through the addons/dialogic/Documentation/Content folder and creates a tree from that.
Folders with only Images in them are ignored and not shown in the documentation tree.

Integration into dialogic
There is not a lot of changes to dialogic:

The DocsTreeHelper ist added as a child of the MasterTree
There is a new DocumenationViewer (that is like the other Editors, but editor was a bad word for it)

The DocumentationViewer mostly only consists of the PageViewer and a bit of code.

And of course there are changes to the MasterTree Script, adding a new docs root node. All the children of that item are created by the DocsTreeHelper

Notes:
This documentation was designed to work both in godot AND on github. The best way to work with it is to use only relative paths for images and links!
This documentation was also designed to be as little integrated into dialogic as possible for it to be easily reused for other purposes and plugins.

I would love to revisit this one!

Fixed the merge issues and it should up to date with main, still haven't tested if it actually works or not, but at least I resolved all the stuff

Well, it seems that I'm able to see the tree with the documentation, but the editor doesn't work. I'll leave this one here for now but at least it is closer to working again than before! :)

When opening an existing project with this branch I see the editor added below:

Error messages:

It would be nice to remove the white outline to the documentation reader and make it integrate better:

Make the max-width of the text not be 100% so when your screen is too wide the lines don't go on forever on the screen. The regular documentation already does this and adds a left margin:

I would also try to style it the same way as the regular Godot documentation is. That means link colors, title colors, font sizes, etc... We need to get the styles from the actual theme so that if the user modified things they would also reflect on Dialogic's documentation. You can use this plugin to get any reference to the internal theme variables there are: https://github.com/pycbouh/godot-editor-theme-explorer
And lastly I would modify the root item to be called "Help" instead of "Documentation".
After all those changes I think it is safe for us to start writing the actual content. It doesn't have to be much, but at least the basics.

the font is too small on retina displays (Currently using a macbook air)

I'm merging so I force myself to write the docs hahaha


Add support for making recordings external to the repository

Fixes https://github.com/Azure/azure-sdk-tools/issues/4257
This adds support to the recording package to work with test proxy asset sync, which will allow the test recordings to be moved out of the go repository into https://github.com/Azure/azure-sdk-assets.
The primary behavior change is: if an assets.json file is found in the current test service directory or any parent, the recording framework will request external assets via the test proxy during test setup. If there is no assets.json, then pre-existing recordings will be used instead. This change should support running tests via run_tests.ps1 or invoking go test directly.
I also added some minor improvements to the testing scripts.

Improve error and pipeline handling in run_tests.ps1
Add test proxy default assets directory to gitignore
Support test proxy external asset mode for test recordings

Looks like windows tests are failing due to some path handling differences with the OS. Investigating...

@jhendrixMSFT @richardpark-msft do you anticipate a future requirement to define configs for recording assets in directories below the service directory (e.g. deeper than sdk/keyvault/azkeys)? That is to say, will we need more granularity with recordings than the main package? I've skipped that functionality for now assuming it won't be needed.

In azblob there are tests in subdirectories. Does this mean the feature is required?

In azblob there are tests in subdirectories. Does this mean the feature is required?

It's a weird scenario I'm thinking of where you would have tests in a sub-directory, and an asset configuration for those test recordings in the sub-directory, but you invoke the tests from a parent directory (either with ./... or like ./container/...). Let me dig back into it.

We definitely need go test ./... from the azblob directory to work.

Alright, the latest update is much more flexible with the working directory of go test and the asset config path relative to the recording path, as well as fixing some windows file and root detection issues.

/azp run go - internal - ci


Support local custom elements

An initial prototype for supporting the definition of custom elements. Here's an example of how you might use it to build a Trix component:
class TrixComponent < Phlex::Component
  register_element :trix_editor

  def template
    input id: "content", type: "hidden"
    trix_editor input: "content", role: "textbox"
  end
end

Custom elements, for now, are local to the component where they're registered.

That looks great!
Just a small question, wouldn't it be possible to add this behavior directly on the Phlex module and add the custom element to the Phlex::Tag::StandardElement::ELEMENTS array?
Something like this:
module Phlex
  def self.register_element(tag_name)
    Phlex::Tag::StandardElement::ELEMENTS << tag_name
  end
end

The limitation is that once we load the Phlex::Component class, we can't add more custom elements as the class_eval already run. I'm not sure it would work though depending on the order the classes are read.

@alexandreruban we could definitely support global custom elements like that, but I wanted to first look at supporting custom elements specifically scoped to only the component that registered them. I think in practice, this will be sufficient. But still open to globals if we can think of a use-case.

It's worth noting that you can register custom elements in an abstract component class, e.g. ApplicationComponent and then subclass that with your other components.


[SPARK-37895][SQL] Filter push down column with quoted columns

What changes were proposed in this pull request?

The problem happens when you have a column that is quoted because it has special characters.
select view1.`Имя1` , view1.`Имя2`, view2.`Имя3`
from view1 left join  view2
on view1.`Имя2`=view2.`Имя4`

It result in the following JDBC push down
SELECT "Имя3","Имя4" FROM "public"."tab2"  WHERE ("`Имя4`" IS NOT NULL)

This PR fix this and in the push down query only remains the dialect quote
SELECT "Имя3","Имя4" FROM "public"."tab2"  WHERE ("Имя4" IS NOT NULL)

Why are the changes needed?

Fix this bug in the JDBC push down
Does this PR introduce any user-facing change?

No
How was this patch tested?

Unit testing

CC @cloud-fan

Can one of the admins verify this patch?

thanks, merging to master!


Multiple .travis*.yml files - What is the Interaction?

In this repo, there are three .travis*.yml files.

.travis.yml
.travis-repo.yml
.travis-rpm.yml

Not being an expert on Travis, it's not clear if all three of these are intended to be used together in this Repo as-is, and that Travis CI has special handling for multiple .travis*.yml files, or if they are simply provided as references for the tutorial.  I did look through some Travis documentation and do some google searching but could not find a clear reference explaining if/how this works.  I found lots of discussion about it, but nothing I could comprehend.
Can you please clarify what's happening there?

Those are just demo files, travis accepts only one file. feel free to ask when you have a question.

Is it possible to build both DEB and RPM in the same travis file?

long story short, yes, https://github.com/mh-cbon/emd/blob/master/.travis.yml#L43


Migrate ODBC docs from ibmi-oss-examples

The documentation at https://github.com/IBM/ibmi-oss-examples/blob/master/odbc/odbc.md would be more appropriate here, since it's not really an "example". Considering how much we link to that page, though, we should probably replace the page with a link to the new location instead of just deleting it.

This is done


Start trying out Jupyter Book

This is just an initial attempt to use Jupyter Book.

Here are some things I've figured out:

Markdown tables created by code cells don't render correctly. I'm currently looking for a workaround.
We might need to run a separate script to generate the glossary before the rest of the book is built. Maybe we could have a Makefile with a command to run a script that generates the glossary, followed by the jupyter-book build command?
I'd like to start building PRs to RTD, but I can never remember how to do that.

I'd like to start building PRs on RTD, but I can never remember how to do that.
* It seems like this might be complicated with Jupyter Book. See [here](https://jupyterbook.org/sphinx/index.html?highlight=readthedocs#convert-your-jupyter-book-into-a-sphinx-website).

Was looking for how to set up preview of PRs for the jupyter book of the starter kit and it seems the jupyter book mentioned circle-CI:
https://legacy.jupyterbook.org/advanced/circleci.html

If we don't use sphinx-hoverxref (which requires RTD), then I think that CircleCI could work. But I really want tooltips!

Going to merge this now to get the Action in.


Introducing weight country column

In order to facilitate item prioritization for UI/UX and computations the country/countries can be assigned an integer value useful for sorting records in dropdowns and other use cases.

Hi @moijafcor , Thank you for your contribution.
I think sorting by names alphabetically is better. Almost all of the apps I have used, sort countries by names.
However, I will think more if this needs to be merged.
Thanks Again!

Hello,
I took the seeders and integrated them into my own application. However, since you added the extra state "Khyber Pakhtunkhwa" after your the commit of "updatd pakistan states", the ids of all the cities after this state were shifted. When I placed this state at the bottom of the states array and ran it again, the data was validated.
If you check and rearrange this data, you will have an error-free project.
Thanks.


BatchWriter is missing functionality for limiting the queue size

As far as I can see, the BatchWriter feature of this library has no functionality for limiting the queue size. So if the remote database becomes unavailable, there is nothing to prevent the application from running out of memory if points keep being added to the BatchWriter.
My particular use case, for what it is worth: I'm logging a significant number of events/metrics to an InfluxDB database every second. If my InfluxDB database goes down, my application should keep running. But if what I wrote above is true, the BatchWriter would quickly kill my application by making it run out of memory because of the unbounded queue.
I thus suggest new functionality for putting an upper bound on the number of enqueued points. It would be nice with the following two options for when the limit is reached:

Block the add operation until there is room in the queue (not a good idea for my use case, but is likely useable in other use cases)

Discard new points (what I would like in my use case).

It should be possible to add an event handler that gets notified in either case. In my use case, I would log the issue to our error management system.

Hi,
I agree, this is a good idea. I would be in favour of the option 2). And perhaps also option 3) which would start dropping the oldest points but keep the latest. And then once the DB becomes ready, flush.
However, at the moment I unfortunately don't have enough free time to work on this. If someone makes a PR, I'll gladly accept it.


Consider setindex for assoc

FixedSizeArrays uses setindex to mean what assoc does in this package. Personally, I find setindex more consistent. I don't have any opinion as to whether assoc should be kept or deprecated (note that setindex has opposite argument order).
Perhaps a new package can be spun off with definitions for non-mutating versions of Base functions, and packages like FunctionalCollections can extend those. In particular, it would be useful to make push, append, and the like extensible by other packages without requiring a dependency on FunctionalCollections. But that's a different issue.

Seems quite reasonable to me.  This package doesn't receive much attention, so

please feel free to submit a PR
bump the PR if you don't get a quick response.

If you are interested in helping maintain/update the package, I suspect you could get commit access quite easily.


initial_recipe

Checklist before submitting

[X] Did you read the contributor guide?
[ ] Did you update any affected documentation?
[ ] Did you write any tests to validate this change?

Description
Fixes # (issue).
Review process to land

All tests and other checks must succeed.
At least one maintainer must review and approve.
If any  maintainer requests changes, they must be addressed.

We shouldn't remove LICENSE.txt from conda-forge and add our Apache LICENSE file. You can keep conda-forge's LICENSE.txt and just add IBM's copyright line in it.


Windows app on my Android phone

I want to install and run a Windows app on my Android phone. Is there a way?

This application can't run on Android.


possible to use in setup composition api

As title mentoined, how can I use this in setup?

Small fix for this is to provide the modal instance.
install(app: App): void {
    setCurrentAppContext(app._context);
    app.config.globalProperties.$vbsModal = modal;
    app.provide('vbsModal', modal)
  },

Then we are able to use inject('modal') in setup


[Extensions] GitHub Classic or PAT Support for Snapshot Service

Current Behavior
Currently, the MeshMap Snapshot GitHub Action is authorized under the "Meshery" GitHub App via OAuth. This can be an impedance depending upon administrative domains within an organization.
Desired Behavior
Support optional use a Classic or fine grained PAT on Github instead of OAuth-only.
Implementation

Authenticating with a GitHub App on behalf of a user

Acceptance Tests

Users may elect to use OAuth or a token as the authorization mechanism for Cloud integration with GitHub.

Contributor Guides and Handbook

📚 Meshery Build and Release Strategy
🎨 UI Design For Meshery Extensions Figma in Figma (open invite)
📚 Meshery Extensibility Providers
⌨ Meshery API Docs
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

I would like to take this up!

@Shiva953 I missed your comment.

@Shiva953 Golang is a proficiency of yours, right? Ping me in Slack and I'll give you some pointers on this.

This issue has been open for some time with no recent activity, unassigning to open it up for new contributors to give it a go.


Bump entity-naming version

Pull Request Functional Test Report for #1769 / ddcbf07533f076ed76c5607bf5ef3ec1366584fa
No tests identified for validation.
Help


Ctrl-z doesn't output correct Scenetrans setup code

Especially 0 values come out as 0f rather than 0.0f.
Solution might be to output float{val}

Example:
Scenetrans setup code:
    v.setSceneTrans (morph::vec<float,3>({0f, 0f, -3.5f}));
scene rotation is Quaternion[wxyz]=(1,0,0,0)
Writing scene trans/rotation into /tmp/Visual.json... Success.

Fixed with 64b7cc8


when the source data is a url, the context menu should say ‘open url in new tab’

Just to understand the use-case, what is behind this link? If it is a web-page, then using supplementary file is the wrong approach. Then I would rather cite a website (reference) or use a link (external web link).
We were assuming that a file location provided with a supplementary file, can always resolve to a file download.

Use cases:

link to a local file that needs to be included in the package;
link to a remotely hosted file for direct download;
(more tricky) link to a landing page from which the file can be directly downloaded.

In SmartFigures we render #2 and #3 next to each other for convenience (the file is here :

The directly downloadable file is probably cleaner and promotes the figures as the actual landing page. One issue is when the file is very large and people should be warned of the size beaforehand. But this is a UI issue mainly, right? So displaying the size of the remote file would be important.

Turning this into a feature-request.
Instead of blindly downloading the linked file, Texture should fetch info about the linked file first before (using HTTP headers), and present the user information about the file size, or if it is a landing page let the the user know that it will open the URL in a new tab/window.

This feature is not trivial but do-able, but needs some evaluation and estimation.


Add unit tests for tablify and matrixify

I couldn't see them. Can you point them out if they exist, or add some please?

Feel free to close when ready.


How does load balancing work for the event-gateway?

@brianneisler what kind of load balancing do you mean?

Hey @mthenw, sorry should have been more clear. The docs don't clearly call out how the event-gateway is meant to be load balanced. From our earlier discussions, the thought was that the event-gateway was meant to live at the edge meaning it IS the balancer.
First, I wanted to check if that was still the plan. If so, we should document that somewhere in the readme. Second, what would be the suggested DNS balancing strategy? How does the gateway operate if it has nodes running in every provider? Does an event get forwarded to the node running on that provider before making the call to invoke the function? Any other clarity around what the strategy will look like here would be good to know.
A side note, I'm also wondering what it would look like to support persistent client connections like websockets and how that would operate.

EG can be either at the edge or it can run behind LB (AWS ELB, etc) depending on the use case, traffic, perf requirements nd other factors. It must be capable of being edge proxy/LB which means it has to have enough security features (TLS, timeouts).

What would be the suggested DNS balancing strategy?
Depending on the use case. By default, I would always propose to run it behind LB. That's the easiest way. In other cases DNS round robin might be the case.

How does the gateway operate if it has nodes running in every provider?
Backing etcd needs to run somewhere, in one region. All EG instances connect to that etcd. EG is a stateless service.

Does an event get forwarded to the node running on that provider before making the call to invoke the function?

No need for that as all instances use the same state (stored in etcd). Routing this event would only add latency.

A side note, I'm also wondering what it would look like to support persistent client connections like websockets and how that would operate.

That's something to figure out. I don't have any solution for that right now. EG is a stateless, eventually consistent service. Supporting persistent connections means that we would have to add partitioning for users' sessions. That's a bit different problem to solve.

@mthenw So can I run three instances of EG today, and they form a cluster and sync on etcd? If that is the case, then should I be able to just hit any EG instance url (Event/Config API) and it will just work?
Just curious. Is that a future plan?

@rupakg it's possible right now. You just need to provide etcd address with -db-hosts flag.

@mthenw thanks for the replies. I've added a couple of issues around documenting some of your responses. #262 #263


[upstream][community] Add operator for Spinnaker

This operator is supports native Spinnaker deployment.

It seems to be conflict in updating existing operator and adding new operator. So cancelling this PR and will submit a fresh PR with resolution.


fix: port Luis intent2 properties from dotnet

Fixes #2398

Pull Request Test Coverage Report for Build 516281290

0 of 0   changed or added relevant lines in 0 files are covered.
20 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-0.03%) to 84.685%

Files with Coverage Reduction
New Missed Lines
%

libraries/botbuilder-lg/src/templateExtensions.ts
1
84.48%

libraries/botbuilder-ai/src/luisAdaptiveRecognizer.ts
4
84.96%

libraries/botbuilder-ai/src/luisRecognizer.ts
5
94.57%

libraries/adaptive-expressions/src/triggerTrees/node.ts
10
90.51%

Totals

Change from base Build 516156245:
-0.03%

Covered Lines:
18066

Relevant Lines:
20371

💛  - Coveralls

Pull Request Test Coverage Report for Build 516281290

0 of 0   changed or added relevant lines in 0 files are covered.
20 unchanged lines in 4 files lost coverage.
Overall coverage decreased (-0.03%) to 84.685%

Files with Coverage Reduction
New Missed Lines
%

libraries/botbuilder-lg/src/templateExtensions.ts
1
84.48%

libraries/botbuilder-ai/src/luisAdaptiveRecognizer.ts
4
84.96%

libraries/botbuilder-ai/src/luisRecognizer.ts
5
94.57%

libraries/adaptive-expressions/src/triggerTrees/node.ts
10
90.51%

Totals

Change from base Build 516156245:
-0.03%

Covered Lines:
18066

Relevant Lines:
20371

💛  - Coveralls


Curie icon ideas

It would be good to have a more distinctive icon for Currie. At first I have not noticed it is even running, because it looks like this for me:

Thank you @piotrwach, good point. Interesting, I wasn't aware of this behaviour. I always saw terminal-like icon, but I guess since the app does not have its own icon, NSApplication inherits the icon from the parent process. I will try to prepare and set an icon in the next few weeks.

Resolving. @piotrwach, hope you will like the new icon.

The new icon looks great! Thanks!


LYWSD03MMC always shows 109% on Battery value

Hi,
my tempsensors always show 109% of battery which in my eys can't be correct value here, or?
msg : Object object payload: object battery: 109 temperature: 20.9 humidity: 51 address: "a4:c1:xx:xx:xx:xx" _msgid: "ef91c9b2.cf4118"
This is now shown for some month so I even don't see any change in that value.
Can you check if your using the correct hex for the battery value here?
Kind regards
Ocean

For me, it's also 109% - I'm trying to debug it, but I'm not completely sure about the code.
msg.battery = data.toString().charCodeAt(0); Line 24, xiaomi-ble.js
This part gets the battery state - but data.toString() doesn't contain any battery information for me. It contains a domain name.
 const prep = typeof data === typeof 's' ? data : JSON.stringify(data.toString('hex')).replace(/\"/gi, ''); Line 58, xiaomi-ble.js gets the data which contains temperature, humidity and battery voltage in it.
Temperatue are the first two hex numbers in little endian, followed by one hex numer big endian humidity. the other two hex numbers are the voltage in mV little endian.
Example:
prep contains b8083f030c

b8 08  convertes to 2232 (be aware of little endian!) the display states 22.3°C - this fits the 2232
3f convertes to 63, the display shows also 63%
-03 0c_ convertes to 3075 (also little endian!), which means 3075 mV or 3.075 V. This could be plausible because I'm using a fresh coin cell. A CR2032 coin cell has a nominal voltage of 3V.

So the doesn't report a percentage, it reports the voltage. A percentage could be calculated, based on the coin-cell discharge curve. There is already some code at zigbee-herdsman-converters which could be used for the conversion from voltage to percent.
Also it could be possible to return two battery fields, on with the percentage, one with the voltage.
I could prepare a pull request for all of that, I changed the code for me already and tested it.
But I only have LYWSD03MMC, so I can't check if my changes would work with other devices too.


docs: remove stray comment

@bors r+

:pushpin: Commit 98a9435ab4690b3b22a07060c4c59ba3dd96ae6e has been approved by epage
It is now in the queue for this repository.

:hourglass: Testing commit 98a9435ab4690b3b22a07060c4c59ba3dd96ae6e with merge 7dcf764cbcff23b54fd061473f5ce66223cc0f86...

:sunny: Test successful - checks-actions
Approved by: epage
Pushing 7dcf764cbcff23b54fd061473f5ce66223cc0f86 to master...


OCPBUGS-14005: Remove Ross from OWNERS

Ross left the project some time ago already. Let's clean up the OWNERS file.

/jira refresh


This likely means that the library (ngx-countup) which declares CountUpModule is not compatible with Angular Ivy. Check if a newer version of the library is available, and update if so. Also consider checking with the library's authors to see if the library is expected to be compatible with Ivy.

[ ] Bug
[ ] Build issue
[ ] Feature request

Version info

Angular:
Angular CLI:
ngx-countup:

Description
Hello we are using latest version of this package
and angular 16 version
and we face error like
https://prnt.sc/hZMXtG401oCW

No response from issue creator, closing because unable to reproduce.

seems it working now after install again
On Tue, Jan 16, 2024 at 10:38 PM Jamie Perkins @.***>
wrote:

No response from issue creator, closing because unable to reproduce.
—
Reply to this email directly, view it on GitHub
https://github.com/inorganik/ngx-countUp/issues/88#issuecomment-1894161046,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGLK3BS4W2OHOAICIMB75NDYO2XYRAVCNFSM6AAAAABBFGNBCWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOJUGE3DCMBUGY
.
You are receiving this because you authored the thread.Message ID:
@.***>


ADD Q&A Suggestion Template

Related Issue
Fixes: #267

Description
This PR addresses the issue by implementing a Q&A Suggestion Template feature that allows users to easily document questions and answers in a structured format. It improves documentation clarity and enhances user understanding by providing a consistent structure for FAQs or support documents.

Type of PR

[x] Feature enhancement

Screenshots / Videos (if applicable)
N/A

Checklist:

[x] I have made this change from my own.
[ ] I have taken help from some online resources.
[x] My code follows the style guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have tested the changes thoroughly before submitting this pull request.
[x] I have provided relevant issue numbers and screenshots after making the changes.

@jahnvisahni31 the task has been completed. Kindly review the PR
Don't forget to add labels and assign the level

@jahnvisahni31 the task has been completed. Kindly review the PR Don't forget to add labels and assign the level

@jahnvisahni31 kindly review and merge, the session is going to get Over


What's the difference between A.all and A.every, A.any and A.some?

According to the documentation, there are four functions to check if the whole array or at least one element of it satisfies the condition.
However, I don't see the difference between them, because they act the same.
// A.all / A.every

console.log(A.all([2, 4, 6, 8, 10], (value) => value % 2 === 0)); // true
console.log(A.every([2, 4, 6, 8, 10], (value) => value % 2 === 0)); // true

console.log(A.all([1, 4, 5, 7, 9], (value) => value % 2 === 0)); // false
console.log(A.every([1, 4, 5, 7, 9], (value) => value % 2 === 0)); // false

console.log(A.all([1, 3, 5, 7, 9], (value) => value % 2 === 0)); // false
console.log(A.every([1, 3, 5, 7, 9], (value) => value % 2 === 0)); // false

// A.any / A.some

console.log(A.any([2, 4, 6, 8, 10], (value) => value % 2 === 0)); // true
console.log(A.some([2, 4, 6, 8, 10], (value) => value % 2 === 0)); // true

console.log(A.any([1, 4, 5, 7, 9], (value) => value % 2 === 0)); // true
console.log(A.some([1, 4, 5, 7, 9], (value) => value % 2 === 0)); // true

console.log(A.any([1, 3, 5, 7, 9], (value) => value % 2 === 0)); // false
console.log(A.some([1, 3, 5, 7, 9], (value) => value % 2 === 0)); // false

It's not said if the one function of them is an alias for another and their declarations are a bit only different. So I find it confusing for developers using this library.

let all = (xs, predicateFn) => Belt.Array.everyU(xs, (. value) => predicateFn(value))
let every = (xs, fn) => Belt.Array.everyU(xs, fn)
let any = (xs, predicateFn) => Belt.Array.someU(xs, (. value) => predicateFn(value))
let some = (xs, fn) => Belt.Array.someU(xs, fn)
they are exactly the same. both calls Belt's Array.everyU, Array.someU

There are many names for the same idea, such as for all, every, universal quantification, and every.
but I also agree that many names for the same idea, in a single library is not a good idea.


Add a way to hide the compose box

While having the compose box show up inline is useful, it'd be nice to have a way to hide it for when you just want to have more screen real estate for reading messages (especially relevant when using the app in landscape orientation).

This would really nice to have, I created two ways for this issue one is hiding chatBox and toolbar on scroll and the other one is implementing a FloatingActionButton to show the keyboard on clicking it displays the chatBox and hides it after some seconds (5 secs in the gif), in both these when coming down hide the views and when moving up the list it shows the views.
Preview of FloatingActionButton
Preview of ChatBox
I would go for the FAB method, as hiding the chatBox will have issues at the bottom and top it will overlap the list as shown in the GIF + FAB looks good, just need some suggestions for the background color of the FAB!

@kunall17 the FAB looks great!  Once you have the chat box open, how would you collapse back into the FAB again?

When chatbox is displayed after some time [5 seconds in the gif] it would hide the chat box and bring up the FAB, I will add some checks if the user is using the chatbox edittexts it would not hide.
And on scroll if the chatbox is displayed it would hide the chatbox.

@niftynei @timabbott need your final suggestions on this one! I have to implement this in #81
I'm with FAB method, looks more clean and no bugs which will occur in hiding chatBox as it will overlap the list!

That approach sounds reasonable to me!

(Though I'd consider implementing it as a new PR rather than part of #81, since #81 is already a very large PR, and it can often slow down actually merging changes to keep adding additional new features to an existing PR)

Well AFAIK for coordinator layout scroll flags listView is not supported, a recyclerView is needed for the scroll flags so I will have to add the commits in #81!

As of #81 can now swipe to dismiss the box.


How to load the model in another file?

I followed your Upgraded Model Tutorial (and the others of course) but I can't seem to get it working with flask.
What do I need to save (vocab, padding?) and how do I do that? Pickle?
How can I use the model in another py document? Copy the model class? I can't seem to get it working even though I looked at some tutorials online.

Assuming that you're trying to make this production-ready, you should've saved:

The model to some checkpoint (contains the trained weights, check the PyTorch documentation of saving and loading models)
The model hyperparameters (usually saved to YAML files)
The vocab of TEXT field (Refer to this previous issue

Coming to the second part, why do you need to copy the class? You can just import it, from that particular file.


Add USE inputs

Different transformation specs that will be created for related tasks will have a lot of common structure: e.g., they will have to define the same inline transformers in each spec. Please add the possibility of a new input kind, USE, allowing to factor out common parts into reusable libraries, and to include them into each transformation spec.

Because USE inputs do not need names, it is better to split them off the input array into a separate use array.


Change default sku to Basic for Linux

Specific to Linux Apps Only
We've seen issues related to the Linux Free SKU in Azure App Service wrt to deployment successes and general performance issues of their applications. To alleviate the impact of those issues on users, we believe it will be a better experience to default customers on Linux to the Basic (B1) SKU . Since the default is a paid SKU, I also suggest we output into the console that the SKU is paid, and link customers to the pricing page: https://azure.microsoft.com/en-us/pricing/details/app-service/linux/ and make them aware of the free sku.
One potential set of outputs:

The default SKU selected is Basic (B1). This is a paid App Service plan with a low monthly cost. For details on pricing and performance, please see the Azure pricing page: https://azure.microsoft.com/en-us/pricing/details/app-service/linux/.
You may also migrate the App Service plan to the free offering, or more performant offering in the Azure Portal.

Note: The portal and CLI default to the Premium SKU.

Closing in favor of https://github.com/microsoft/vscode-azureappservice/issues/1862


Implementing Sentiment Analyzer using RNN

💥 Proposal
Implementing a sentiment analyzer using RNN and NLP techniques to train the model on word2ve dataset.

@okaditya84 - you can go ahead! We are assigning you 21 days for this project, after which it will be assigned to someone else if not completed. All the best!
Name the file as: algorithm_dataset.ipynb and link it in the readme of the labeled directory as algorithm - dataset.

@okaditya84 - you can go ahead! We are assigning you 21 days for this project, after which it will be assigned to someone else if not completed. All the best! Name the file as: algorithm_dataset.ipynb and link it in the readme of the labeled directory as algorithm - dataset.

Where will I be storing the weights of the dataset ?


Option to add vertical scroll with an external button

Hey,
Right now the grid only adds scroll when you add an object on the bottom so it needs to grow.
What about an option where the grid doesn't need objects to grow so you can add scroll with a button (for example) and then fill this space with the objects that you want?
Regards,

Hey @pau-garrido ,
If I understood well, you want to set the height of the grid yourself manually?
Which is your use case? You are using a non vertical compact position? Just for understanding the necessity of this feature, thanks!

Hey @llorenspujol thanks for your answer.
Yes, I basically want the grid to occupy all the height I have on the display and also be able to add more height so it creates scroll in there (independently of the objects that I have on the grid). I guess that with an input that sets the height my problem would be solved.

In PR #69 it has been added the option to add the height to the grid with an Input, so this issue should be solved. It would be available in the next published version.

Thanks!