repository name

I suggest renaming this repository from sdk to dart-sdk or dart and adding a Getting The Source button in the GET STARTED menu at the dart's homepage.
You see, you cann't find this repository by just type dart in github's search engine(https://github.com/search?utf8=✓&q=dart).

Thanks for the suggestion. We have a lot of tools that look for this repository by name. It's unlikely that we'll change it now, unfortunately.


Implement kernel version of ebpf_delete_registry_tree()

This issue is to track implementation of kernel mode version of ebpf_delete_registry_tree()

This is no longer needed as km store APIs are being deleted in PR #3109


test1

Change-Id: Ia3513e117cf07dd48eb490f0bbcc65f8969a8b77

This pull request is part of a stack:

test1 (#1)
test2 (#2)
test3 (#3)

This pull request is part of a stack:

test1 (#1)
test2 (#2)
test3 (#3)


Add more complete example of multiple client configuration

It took me a bit of digging and research to figure out how to get this to work. I hope that the example expanded below would help others implement this more easily.

Thank you for this. This is something I will be using soon

Thanks! :+1:


NIFI-6766: Escape policy label

NIFI-6766:

Ensuring policy label is properly escaped when populating the user's access policy listing.

+1 merged to master


HOT RELOAD: OSR related reload crash

How to reproduce (may take minutes of reloading):
In your top level sdk/ run the following once:
$ ln -s sdk/lib/ lib
In pkg/compiler run the following:
$ gdb --args ../../out/ReleaseX64/dart --trace-reload --background_compilation=false --reload-every=500000 bin/dart2js.dart --categories=all bin/dart2js.dart
The crash I observed was:
#0  Write<dart::RawObject*> (value=<synthetic pointer>, ptr=0x7ffff7e86348) at runtime/vm/verified_memory.h:55
#1  dart::RawObject::StorePointer<dart::RawObject*> (this=0x7ffff7e86321, addr=0x7ffff7e86348, value=0x7fffaafeb731)
    at runtime/vm/raw_object.h:578
#2  0x000000000072604e in StorePointer<dart::RawObject*> (value=<optimized out>, addr=<optimized out>, this=0x7ffff4dbb550)
    at runtime/vm/object.h:647
#3  set_owner (function=..., this=this@entry=0x7ffff4dbb550) at runtime/vm/object.h:4703
#4  dart::Function::AttachCode (this=this@entry=0x7ffff4dbb4f0, value=...) at runtime/vm/object.cc:5263
#5  0x0000000000887460 in DRT_HelperStackOverflow (zone=<optimized out>, thread=<optimized out>, isolate=<optimized out>, arguments=...)
    at runtime/vm/code_generator.cc:1385
#6  dart::DRT_StackOverflow (arguments=...) at runtime/vm/code_generator.cc:1217
#7  0x00007ffff7e4341b in ?? ()
#8  0x0000000001104c20 in ?? ()
#9  0x0000000000000000 in ?? ()

Digging in, we are looking at the function "Function '_emptyDeferredQueue@74414697':." and the code is "[stub: LazyCompile]".
The crash happens when trying to associate the code with the function (set_owner). The code object is read only in the vm-isolate heap and we can't touch that memory.

Note that during a reload we replace many functions's code with the lazy compile stub.


Merge Image Display UpdateFrom* functions.

Desired behavior
There is quite a bit of duplicate code in the various UpdateFrom* functions in Image Display. Let's try to keep the same Image Display behavior and functionality but with some refactoring underneath the hood.
Implementation suggestion
Merge the UpdateFrom* function. We could templates or just refactor out the code from converting an image array of the input type (int8, int16, float32) to an rgb image buffer:
https://github.com/ignitionrobotics/ign-gui/blob/ign-gui4/src/plugins/image_display/ImageDisplay.cc#L261-L280
Additional context
related pull request comment

I'm closing this since #212 has been merged.


use full path to rustdoc as for rustc

If user is using system-wide rust/cargo and rustup'ed one and tries to run
cargo test, rustc is executed through /usr/bin/rustc, but rustdoc
is executed without full path which leads to:
error[E0514]: found crate XXX compiled by an incompatible version of rustc
Fixes: a4c0c06275889ea4231a4fccd1180e6ab5907428
Reported-by: Martin Sehnoutka<EMAIL_ADDRESS>Signed-off-by: Igor Gnatenko<EMAIL_ADDRESS>

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @brson (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

Is this patch perhaps missing somethign? I think as-is this just doesn't compile?

I must missed something, checking....

After some debugging I can say, that the fault is on our side. When we build a package, a .cargo/config file is created and it contains this:
[build]
rustc = "/usr/bin/rustc"
rustflags = ["-Copt-level=3", "-Cdebuginfo=2", "-Clink-arg=-Wl,-z,relro,-z,now", ]

[term]
verbose = true

[source]

[source.local-registry]
directory = "/usr/share/cargo/registry"

[source.crates-io]
registry = "https://crates.io"
replace-with = "local-registry"

which is the source of this problem, because rustc is defined here, but rustdoc is not. It can be fixed simply by adding this line into the config file:
rustdoc = "/usr/bin/rustdoc"

So I propose to close this PR and fix it in the RPM macro instead, because there is nothing to be done on cargo side.
https://pagure.io/fedora-rust/rust2rpm/blob/master/f/data/macros.cargo

uh oh, thanks for debugging


Spacing and paragraph breaks on articles

Type

🐛 Bug

Kirsten spotted this bug yesterday. Doesn't seem to be a problem on any other templates (base page or otherwise).

Hi there, just noticed when I looked at the articles just now on Stories that all the line breaks/ spaces between paragraphs seem to have been removed. They're now looking more like one big block of text...

Fixed by #2842.


not work on IE11

in IE11 it genarated a data-uri link like this
data:text/html;charset=utf-8,%3Chtml%3E%3Chead%3E%3Cmeta%20http-equiv%3D'Refresh'%20content%3D'0%3B%20URL%3Dhttp%3A%2F%2Fwww.whatismyreferer.com%2F'%20%2F%3E%3C%2Fhead%3E%3Cbody%3E%3C%2Fbody%3E%3C%2Fhtml%3E
and then. IE don't support data-uri html, it says can't display this page

I came here after googling for the same issue! The prototype.js trick suggested by @gitcnd fixed the problem!
Here are some more details, for those who need it:

Download prototype.js and put it in your scripts folder
Edit prototype.js, and find change the line:

    return {
      IE:             !!window.attachEvent && !isOpera,

to:
    return {
      IE:             ua.indexOf('Trident') >-1 || (!!window.attachEvent && !isOpera),

Now, use noreferrer.js instead of jquery.noreferrer.js, and it should work. You'll still get an ugly white screen when using IE11, but at least it will work!


Search in Navbar does not animate

The search overlay should show with an animation. Currently, it just displays the overlay without any transition.

@egunther39 Could you provide guidance on how the search overlay should behave?

This requires more guidance from a design and implementation perspective.


AttributeError: 'LlamaRotaryEmbedding' object has no attribute 'cos_cached'

I'm trying to fine-tune unsloth/yi-6b-bnb-4bit on a custom dataset but as soon I initiate the training using trainer.train wrapped in an learning_rate scheduler optimizer using CosineAnnealing learning rate scheduler I get the error shown in the image. The error "AttributeError: 'LlamaRotaryEmbedding' object has no attribute 'cos_cached'" seems to be related to the LLaMA model of the Transformers library. It appears that the issue is with the implementation of the rotary embedding, specifically in the rotate_half function, where the slicing should have been interleaved.  I'll appreciate help solving the issue above.

@Ronnie-Leon76 I'll check this out today!! Sorry on the issue!

I'll really appreciate it

Having the same issue with unsloth/llama-2-7b-chat-bnb-4bit

@Ronnie-Leon76 @hbernie Apologies could not look into it yesterday - was a bit inundated with stuff :(( Will 100% do it today! :) Sorry again!

@Ronnie-Leon76 @hbernie
I found the issue! A new push a few days ago was made to the latest HuggingFace transformers branch https://github.com/huggingface/transformers/pull/27931 which broke it!!
I'm assuming you used the HF notebooks which we shared - for now I would comment out !pip install "git+https://github.com/huggingface/transformers.git" - we will install the correct version anyways.
Also if you're on a local PC, I would downgrade transformers via !pip install transformers==4.37.2
I will for now edit my notebooks to remove the lines - hope the temporary fix solves it!

@Ronnie-Leon76 @hbernie I think I fixed it!! Hope you all can try it out :) I also updated all the notebooks on our HuggingFace branch https://huggingface.co/datasets/unsloth/notebooks/tree/main and on our blog posts.
No need to change your notebooks!
If you're using a local machine (Colab no need to), upgrade Unsloth via pip install --upgrade --force-reinstall --no-cache-dir git+https://github.com/unslothai/unsloth.git
Hope it works!

@danielhanchen It works fine. Thanks a lot.

AttributeError: 'MistralRotaryEmbedding' object has no attribute 'cos_cached'
came back

@spydaz Best to uninstall then reinstall Unsloth

i am facing the same issue , Can some help. I tried uninstalling unsloth the install . still

@SnehaKumari14 So
pip uninstall unsloth -y
pip install --upgrade --no-cache-dir "unsloth[colab-new] @ git+https://github.com/unslothai/unsloth.git"

Does not work? :(


Fix for KeyID digest generation

Original issue:
https://github.com/ProtonMail/gopenpgp/issues/10
Issue explanation:
https://github.com/ProtonMail/gopenpgp/issues/10#issuecomment-498066306

Hi @L11R! Thanks for catching this, and for investigating. The PR looks good to me. I'm just trying to figure out why the tests failed in Go 1.10, do you have any ideas?

I guess your test are just not fully reproducible from time to time. Restart might help.

@twiss how soon you will merge it? I think it's sorta critical issue and should be fixed ASAP, would be nice also prepare new GopenPGP release with new replace directive.

@L11R this MPI code is not quite correct when the MPI has leading zeros. That's also what caused the failing test - there's a small chance of producing a broken MPI. Give me a moment, I'll push a fix.

Thanks, got it, will wait.


Pass site or siteId to @netlify/config

- Summary
Fixes: https://github.com/netlify/cli/issues/959
- Test plan

- Description for the changelog
Use siteId or site flags as siteId as parameters to resolveConfig
- A picture of a cute animal (not mandatory but encouraged)
🐠

CI tests seem to be failing due to linting.


05_Redux challengeText more concise

Updates the challenge text to be a little more direct.

Looks good to me!

👍


spring-security-ldap Version Specification

This patch simply removes the version specification of
spring-security-ldap from the userdirectory-ldap module as it should be
defined in the main pom.xml file only.
Your pull request should…

[ ] have a concise title
[ ] close an accompanying issue if one exists
[ ] be against the correct branch (features can only go into develop)
[ ] include migration scripts and documentation, if appropriate
[ ] pass automated testing
[ ] have a clean commit history
[ ] have proper commit messages (title and body) for all commits
[ ] have appropriate tags applied

'Gratz on the awesome pr number :D


Placeholder not updating with selected option in Windows 8 (IE/Chrome)

On SOD elements with a placeholder, the label shows the first option on select, rather than the placeholder or the selected option, both in Internet Explorer 10 and Chrome 43, on Windows 8.
Works fine in Firefox though, oddly.

I guess you've moved on, @Steviant. But would you mind creating a fiddle where I can take a look at this issue? Here's one that you can fork https://jsfiddle.net/vestman/g7pvajc2/
The reason I'm asking is because I'm not able to reproduce it in IE10 nor Chrome (I'm testing on Windows 7)

Ack, ye gods, this was a while back. I don't have a Windows rig to test on now, as I stumbled on this issue while working as a contractor for a particular client. I'll ping it over to them, though, as it was them that raised it with me in the first place.

He, yeah, SoD haven't been getting a lot of attention from me the last year. Thought it was time to change that... Hehe.
Any way, thanks!

Hi! Any solution for this bug ? :(

Hey man, i found the error. On windows 8 Chrome if IF is TRUE
// If it's a touch device
if (_private.isTouch()) {
$sod.addClass("touch");
}
In Edge is false, thats why edge works fine. In my case i only comment this condition, but if u use in mobile you have to test the mobile header to :)
hope it helps.


What is the source of WIT  video dataset?

WIT for WebImageText.

https://arxiv.org/pdf/2103.00020

This is the dataset to train CLIP. The dataset detail is undisclosed. If you'd like to know more about that, please consult the CLIP developers.


Update about page translations

Key changes

Consolidated territory manager and customer success manager titles into generic "title" translations
Updated about page layout to use flexbox for team list to support word wrapping for longer title names
Accidentally translated "Customer Success Manager" in locales that had translations - I can remove these if we've prefer the community update (I can also commit and add a change flag to them so they're reviewed).

We also usually pull request directly into the master branch.
Master branch is then merged into the production branch to be deployed.

So this is branched off of my development environment branch so that I could use that during development.  I've opened against development-environment so that the diff is simpler / does not show the development-environment changes as well.  Once I get approval I'll point to master and remove development-environment changes from this branch so that it follows the standard merge process.

Updated to remove nested title block

Rebased development-environment commits out, merging.


Project dependency cleanup

Which specific dependency we must exclude?

@huafengw  jettison-1.1.jar

@huafengw  Remove activation-1.1.jar if possible, thanks!

Already done!


FOR TESTING ONLY: Testing pr extendr/libr-sys#141

extendr/libr-sys#141

Alright; I'm re-running everything, but it already worked out, and I've addressed comments from you @multimeric.


add a theme using photoswipe (http://photoswipe.com/).

This PR is linked to https://github.com/saimn/sigal/pull/164
Photoswipe library needs image size to render image (see http://photoswipe.com/documentation/faq.html). So, I have to add a specific property to return image size (disclaimer : I am a python beginner). We can't use 'img_size' property because depending on image orientation, image size can differ.
The first image is highlighted for each album.
Pictures are lazy loaded thanks to echo (https://github.com/toddmotto/echo).
TODO

[x] improve CSS and HTML
[x] improve responsive design
[ ] add support for video (not supported)
[x] add support for albums
[x] add links
[x] update documentation

It is really great to see a new theme coming to sigal, thanks @franek! I will try it soon.

Thanks it is a basic theme. I am not a designer. It answers to my need. This theme depends on this PR https://github.com/saimn/sigal/pull/164

since the dependent PR #164 is already merged, are there any plans to merge this soon ?

@iemejia I have rebased the PR but there is still work on html/css (responsive design). I don't know if I will have time to work on it in the next days. Do not hesitate to propose improvement :p

It looks already good ! A few suggestions to improve the design (at least for me ;)) :

put the album-list and breadcrumb divs in the content div, so that these get aligned with the text below
remove the white border below the album-list

@saimn I have applied what you have suggested. There are still some little things to do (video, links).

At the moment, Photoswipe does not support video. Is possible to release this theme without video support ?!?

At the moment, Photoswipe does not support video. Is possible to release this theme without video support ?!?

Yes, if you can just mention this in the docs, then it is not a problem for me.

Yes, if you can just mention this in the docs, then it is not a problem for me.

done. Thx

This is a wonderful contribution. Thank you @franek!
I have used this new theme for a gallery I've posted: http://lukeandmichelle.ca/cycamatic/
I did some work to improve the alignment of the thumbnails and to ensure the layout is responsive to small screens. I posted my changes here. If they're a useful addition to this PR, I can submit a PR to this branch so that they are amalgamated.
I tested these changes on Mac Chrome, Firefox, Safari, and Mobile Safari on iPhone 4S, and simulated a few other mobile devices using Chrome Devtools.

Hi @lukecyca,
Nice. Seems very good. You can submit a PR to this branch and I will check.
Thx,

thanks @lukecyca, I have merged your contribution and fix an alignment problem for menu.
@saimn If you are OK with this theme, you can merge this PR. thanks.

Merged ! Thanks for your work on this, I guess it is time to bump a new version :)

\o/
I wanted to fix image path for photoswipe theme but you did it (see https://github.com/saimn/sigal/commit/1639154801f606e63c0c142c24d826a51488f47f). Thx

New version, new version, yeeesss please, pypi !


Closes #559
Closes #604

CI failing seems to be a github issue? Just rerun it.

Oh weird the CI shows as passing now.

Haven't tested the latest fixes and I'm not going to bother to until this PR actually gets reviewed.

Replaced by #894


I can not sign up

Thank you for your interest in the Phenotype Database.
What was the url (dashin.eu/interventionstudies I assume), date and username of your failed sign up?


fix(ds): cache traefik routes

Fixes RVTEE-504
Changes

#1081  👈
main

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

Join @MasterPtato and the rest of your teammates on  Graphite

just comment why redis is in the same step


feat: Adding more flexibility for pauses and delays

Overview
The problem statement is currently how to best integrate manual steps into an Opentrons workflow. Currently with a delay/pause folks need to use a stop watch to track timing for when to do manual steps.

Use this section to describe your issue at a high level. Please include some context about why you want this feature. What would this feature allow you to accomplish? Are there any workarounds you use to partially achieve your goal? Are there any issues you could find that may be related?
-->
Implementation details
I would defer to the user who recommended this feature, but I'd say this could be addressed in two ways:

Having a clock under the runtime tracker for longer pauses/delays. This would be hard to implement because 1 to 10 second delays are common in liquid handling steps. So we might need to either find a reasonable minimum seconds, or use method 2.
What we could do is add an alert function potentially in the middle of a delay

Design
I wrote a little custom function that can do this.
Please note:

I'm not sure if this would be implementable in the Python API based solely on the code below. There are a fair amount of adjustments that I'd imagine this likely would use.
I can help provide a satisfying alarm sound.

import subprocess
from opentrons import protocol_api, types

metadata = {'apiLevel': '2.8'}

metadata={"apiLevel": "2.9"}

AUDIO_FILE_PATH = '/etc/audio/speaker-test.mp3'

def run_quiet_process(command):
     subprocess.check_output('{} &> /dev/null'.format(command), shell=True)

def test_speaker():
     print('Speaker')
     print('Next\t--> CTRL-C')
     try:
         run_quiet_process('mpg123 {}'.format(AUDIO_FILE_PATH))
     except KeyboardInterrupt:
         pass
         print()

def alarm_delay(self, seconds=0, minutes=0, msg=None, alert = True, alert_time = 0):
       """Delay protocol execution for a specific amount of time.
       :param Boolean alert: Whether someone wants an alert or not
       :param float [seconds] alert_time: The time remaining prior to the alarm going off

       If `seconds` and `minutes` are specified, they will be added.
       if 'Alert' is not specified default to false
       if 'alert_time' is not specified default to 0 seconds

       """
      delay_time = seconds + minutes * 60
       self._implementation.delay(seconds=delay_time-alert_time, msg=msg)
      if alert_time > 0:
         self._implementation.delay(seconds = alert_time, msg = f"the protocol is {alert_time} away from completion")
         if not protocol.is_simulating:
            test_speaker()
        else:
          protocol.comment('the delay has completed')
          if not protocol.is_simulating:
            test_speaker()

Acceptance criteria
I would have to research more on the use cases whether it is more of an API feature request to address the issue or something that would need to take place on the app to be useful.
(

Put a good alarm sound on the OT-2
Set an "alarm function"
Use the general principles in transfers.py where we break a complex action (make an alarm + a delay) into its separate components that are housed in a single function which would make it cleaner than the above implementation
Test it on the OT-2
Release it

I have a similar issue. I have a 2+ hour protocol with some critical timing steps at the very end that require manual intervention. It would be great if the pause() function gave a more obvious visual indication that the protocol is paused. For example, including an option to blink the deck lights until the protocol is resumed.
-Dan


add package info

What does this PR do?
reaction to #358
It would be fine to make a homepage for example from the main README file, as Github offers
see: https://guides.github.com/features/pages/
it is about %min and couple clicks... :)

btw, why is the package name pytorch_lightning not just torch_lightning the same way as PyTorch repo but imported as import torch

seems to he be failing on unrelated code...

fixed tests. can you rebase master?

@Borda I had to revert this because it broke the install.
(pl3) [user@machine5 pytorch-lightning]$ pip install numpy
Requirement already satisfied: numpy in /misc/scratch1/_Group/falcon/software/miniconda3/envs/pl3/lib/python3.6/site-packages (1.16.4)
(pl3) [user@machine5 pytorch-lightning]$ pip install -e .
Obtaining file:///home/user/Developer/pytorch-lightning
  Installing build dependencies ... done
  Getting requirements to build wheel ... error
  ERROR: Command errored out with exit status 1:
   command: /home/user/media/falcon_scratch1/software/miniconda3/envs/pl3/bin/python /home/user/media/falcon_scratch1/software/miniconda3/envs/pl3/lib/python3.6/site-packages/pip/_vendor/pep517/_in_process.py get_requires_for_build_wheel /tmp/tmpw4qf53yy
       cwd: /home/user/Developer/pytorch-lightning
  Complete output (22 lines):
  Traceback (most recent call last):
    File "/home/user/media/falcon_scratch1/software/miniconda3/envs/pl3/lib/python3.6/site-packages/pip/_vendor/pep517/_in_process.py", line 207, in <module>
      main()
    File "/home/user/media/falcon_scratch1/software/miniconda3/envs/pl3/lib/python3.6/site-packages/pip/_vendor/pep517/_in_process.py", line 197, in main
      json_out['return_val'] = hook(**hook_input['kwargs'])
    File "/home/user/media/falcon_scratch1/software/miniconda3/envs/pl3/lib/python3.6/site-packages/pip/_vendor/pep517/_in_process.py", line 54, in get_requires_for_build_wheel
      return hook(config_settings)
    File "/tmp/pip-build-env-30cwdsvd/overlay/lib/python3.6/site-packages/setuptools/build_meta.py", line 146, in get_requires_for_build_wheel
      return self._get_build_requires(config_settings, requirements=['wheel'])
    File "/tmp/pip-build-env-30cwdsvd/overlay/lib/python3.6/site-packages/setuptools/build_meta.py", line 127, in _get_build_requires
      self.run_setup()
    File "/tmp/pip-build-env-30cwdsvd/overlay/lib/python3.6/site-packages/setuptools/build_meta.py", line 237, in run_setup
      self).run_setup(setup_script=setup_script)
    File "/tmp/pip-build-env-30cwdsvd/overlay/lib/python3.6/site-packages/setuptools/build_meta.py", line 142, in run_setup
      exec(compile(code, __file__, 'exec'), locals())
    File "setup.py", line 10, in <module>
      import pytorch_lightning
    File "/home/user/Developer/pytorch-lightning/pytorch_lightning/__init__.py", line 1, in <module>
      from .trainer.trainer import Trainer
    File "/home/user/Developer/pytorch-lightning/pytorch_lightning/trainer/trainer.py", line 9, in <module>
      import numpy as np
  ModuleNotFoundError: No module named 'numpy'

Please fix, resubmit PR and test (inluding pip install -e .)


lds: cidr equality only checks significant bits

Partially solves https://github.com/istio/istio/issues/29197. The specific case laid out there has CIDRs that have a different prefix, but the prefix length only includes equal parts.
This will make sure we accurately detect conflicting filter chains.

@stevenctl: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
35f1830c5291564c22612d15a882d92ace0aac13
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@stevenctl: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
1624ebc75a97c26abcf68cf78de3e4e87a37337d
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@stevenctl: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
1624ebc75a97c26abcf68cf78de3e4e87a37337d
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

I'll add a release note when the larger bug is fixed.


Map persistence when losing focus

Since V1.8.0 I've noticed a slight issue.
Half the time I'm using a local setup where as a DM I'm using a laptop connected to a secondary monitor. The monitor is placed flat on the table showing a browser window with the "Player" screen. On my own screen I'm controlling the DM screen. However when I'm changing focus between the two windows, the unfocused one goes inactive. This also resets the zoom / focus on the player map when refocused afterwards. Is it possible to be able to retain focus on both windows?

Have you tried open the second window in an incognito tab?

I have not! Awesome, it works exactly as I hoped! Thank you for the quick reply, awesome application!

Glad you enjoy using this app 😍. Feel free to join our discord for questions or feedback https://discord.gg/ypDpJM


Assignment X icon : It should remove the home from the Assignment...??

https://staging.bostonhpa.org/admin/assignment/assignProfile/2
Surveymode: Homes: Assignment # column : click: Unassign X column

In talking with the group, we don't see a need at the moment to have to remove Homes from Assignment since Data optimized the walking routes for the Homes.  We recall if nobody is home at an address we will not revisit it.
Will close this and inform Chris S.

Merged via 'Removed unassigned column' #545


Use CSS Layout

This gives us a nearly-complete Flexbox feature set, worthy of calling v1.0.0.

If anyone can help, here are my goals to accomplish before I take in this poll and release v1.0.0.
[ ] - Document any not-fully-working features/combinations in IE8 in ISSUES.md
[ ] - Document any not-fully-working features/combinations in IE9 in ISSUES.md
[ ] - Document any other quirks (like when changing flex props live or during resizes) in ISSUES.md
[ ] - Document the flexibility API in API.md


Replaces filterSortedByKey with faster ArrayQueue.

This PR fixes part of #127313.
No worries, I did it in my free time yesterday 😉
Initially, for N being the number of lines and M the number of injected texts, it had runtime O(N + M), then O(N * log M) and now something like O(N * log (M / N)) which is better than both.
Note that O(N + M) is not always faster than O(N * log M) - consider N = 1 or M = N^2.

Initially, for N being the number of lines and M the number of injected texts, it had runtime O(N + M), then O(N * log M) and now something like O(N * log (M / N)) which is better than both.
Note that O(N + M) is not always faster than O(N * log M) - consider N = 1 or M = N^2.

Sorry about my debt item https://github.com/microsoft/vscode/issues/127313 being incorrect. I believe the current complexity is O(N + NlogM) where N is the number of changed lines and M is the number of injected texts in all the lines, not O(NlogM) as stated in #127313.
I am now indeed very curios to understand the algorithm, I have rarely seen a big O complexity use division i.e. O(N * log (M / N)).

Sorry about my debt item #127313 being incorrect. I believe the current complexity is O(N + NlogM) where N is the number of changed lines and M is the number of injected texts in all the lines, not O(NlogM) as stated in #127313.

O(N + NlogM) = O(NlogM), since NlogM grows at least as fast as N, but if M depends on N even faster.

I am now indeed very curios to understand the algorithm, I have rarely seen a big O complexity use division i.e. O(N * log (M / N)).

This is due to N being the number of lines and M being the number of injected texts.
M / N is the average number of injected texts per line.
We don't need to do a linear search to find all injected texts on the same line if they are sorted - we can do an exponential search.
Assuming all injected texts are equally distributed among the lines, we don't need to walk over M / N injected texts for each line, but only have a look at log2(M / N) many.
I think equal distribution is actually the worst case here. If the first line contains all the injected text, we actually have a runtime of O(N + log M), which is even better.
I don't know the runtime of slice (O(1) or O(n)), but at least the skipping part can make use of this improved runtime (even though the current implementation returns a slice in both cases).

I'll merge it - we can continue discussing here at a later point in time.

:+1: Looks very good!


Thermostat/HeaterCooler: possible to hide cool/auto modes?

Hi,
Looks like by default a Thermostat will have four modes:

Off
Heat
Cool
Auto
If I have a device which only support heating, is there a way to disable other modes? Or even disable the modes selection possibility? Thank you

ask plugin author

Yes, you just have to change the property characteristic.props.validValues for tote TargetState characteristic.

Yes, you just have to change the property characteristic.props.validValues for tote TargetState characteristic.

Hi I tried that and can restrict values to what I need, works great except for a minor thing. For Heater/Cooler with auto mode I can't seem to find a way to remove "OFF" option. So if I allow 0=AUTO, 1=HEAT and 2=COOL, it will show 3=OFF even though the value is not allowed. But the interesting thing is, if I only allow 1 and 2, then both 0 and 3 are invisible. Guess some minor bug somewhere, no big deal though.

For Apple‘s newer services, like HeaterCooler, HumidifierDehumdiifier, AirPurifier, Valve, Fanv2,... they completely removed the OFF option from the target states (like in Thermostat) and use an additional characteristic „Active“ for this.
I think the OFF option in the user interface (Home App) relates to this characteristic and therefore cannot be restricted by validValues for ***TargetState.
The accessory should always have a chance to be switched off. I wonder why OFF disappears, if you allow only Cool and Heat. I have not seen this yet

For Apple‘s newer services, like HeaterCooler, HumidifierDehumdiifier, AirPurifier, Valve, Fanv2,... they completely removed the OFF option from the target states (like in Thermostat) and use an additional characteristic „Active“ for this.
I think the OFF option in the user interface (Home App) relates to this characteristic and therefore cannot be restricted by validValues for ***TargetState.
The accessory should always have a chance to be switched off. I wonder why OFF disappears, if you allow only Cool and Heat. I have not seen this yet

My understanding is active shows the current state of the device (if it’s running). For thermostat I leave only one target state HEAT, and in Homekit “modes” menu is completely gone (because only one option). For Heater/cooler if I only keep HEAT and COOL options, I get a switch in Homekit. So I believe target state is still the way to control the options.

Characteristic „Active“ is very similar to „On“. Definitely from the Homekit framework view, for HeaterCooler you have to use both „TargetHeaterCoolerState“ (which does not include OFF) and „Active“ (which can be ACTIVE or INACTIVE. From the Home App view, it seems like OFF is a fourth mode, but in fact, if you select mode to OFF, „Active“ is set to INACTIVE and the target mode is not touched. The advantage is, that you can turn on/off the device directly from the room view and the last target mode is retained.
If you only allow one target state, there will be switch. If you allow only Heat and Cool, there will also be an OFF field.

You have to add the maxValue to .setProps like below:
this.service.getCharacteristic(Characteristic.TargetHeatingCoolingState)
      .setProps({
        maxValue: Characteristic.TargetHeatingCoolingState.HEAT
      })

Take a look at my homebridge-web-boiler plugin if you want a full example.

You have to add the maxValue to .setProps like below:
this.service.getCharacteristic(Characteristic.TargetHeatingCoolingState)
      .setProps({
        maxValue: Characteristic.TargetHeatingCoolingState.HEAT
      })

Take a look at my homebridge-web-boiler plugin if you want a full example.

Thank you! Tried that without luck. For Thermostat the trick works, but not for Heater/Cooler for some reason.

As I said before, you can not make „OFF“ disappear for HeaterCooler, because it is linked to the required Characteristic.Active and is not/never part of TargetState. It‘s only about how the UI of the Home App is designed. Try Eve App for comparison.

As I said before, you can not make „OFF“ disappear for HeaterCooler, because it is linked to the required Characteristic.Active and is not/never part of TargetState. It‘s only about how the UI of the Home App is designed. Try Eve App for comparison.

Hi thank you I fully understand what you're saying, but as I stated earlier on I can hide the OFF button if I only allow 1=HEAT and 2=COOL, so it must be possible.

I also tried
maxValue: 1, minValue: 0, validValues: [0,1],
and with this you can select HEAT, COOL and OFF in Apples Home App. In Eve you can choose between Heat and Cool and have a separate On/Off switch. This reflects the underlying characteristics logic better...

I also tried
maxValue: 1, minValue: 0, validValues: [0,1],
and with this you can select HEAT, COOL and OFF in Apples Home App. In Eve you can choose between Heat and Cool and have a separate On/Off switch. This reflects the underlying characteristics logic better...

Try validValues: [1,2] or validValues: [1]

Ok, only way is to disallow value 0 for Active:
primaryService.getCharacteristic(this.Characteristic.Active).setProps({
maxValue: 1,
minValue: 1,
validValues: [1]
});
It works differently from Thermostat which I can hide the OFF button completely:


limit maximum width of man content to 80 characters

See https://www.reddit.com/r/linux/comments/5oqrb9/show_rlinux_manpagesdebianorg_has_been_modernized/dcld26d/ for context.

A minimum width would also be useful, see e.g. https://manpages.debian.org/jessie/manpages-zh/intro.1.zh_CN.html

This is fixed, but will only be live with the next full re-render in the next few days.


Add high avaliablity support for captive portal

This pull request implement synchronization for connected users when enabling High Availability, and fix multiple bugs related to HA with captive portal in general.

[x] Main redmine issue : https://redmine.pfsense.org/issues/97 (Captive Portal should sync its database to other members of clusters)
[x] Also fix :

https://redmine.pfsense.org/issues/8807 (HA sync : files voucher_{$cpzone}.cfg and voucher_{$cpzone}.public are not created on secondary node)
https://redmine.pfsense.org/issues/8809 (HA sync : changing a voucher roll on master does not reset active tickets on secondary)
https://redmine.pfsense.org/issues/9303 (HA sync : disabling captive portal HA sync does remove all zones on secondary)

[x] Ready for review
[x] Forum thread : https://forum.netgate.com/topic/136525/usertransfer-in-ha-modus/3 . I am looking for testers, even if you are not using High Availability. Please refer to the forum for more info

Hi,
Sorry for the delay !
I've updated the PR accordingly.

I guess ready for merge/testing?

Still needs a more thorough review.

Main redmine issue : https://redmine.pfsense.org/issues/97 (Captive Portal should sync its database to other members of clusters) - synced, all config is copied to the secondary;
https://redmine.pfsense.org/issues/8807 (HA sync : files voucher_{$cpzone}.cfg and voucher_{$cpzone}.public are not created on secondary node) - works, I can see /var/etc/voucher_{$cpzone}.cfg and /var/etc/voucher_{$cpzone}.public on the secondary after sync;
https://redmine.pfsense.org/issues/8809 (HA sync : changing a voucher roll on master does not reset active tickets on secondary) - works as expected, active tickets reset on secondary is OK;
https://redmine.pfsense.org/issues/9303 (HA sync : disabling captive portal HA sync does remove all zones on secondary) - works as expected, after disabling captive portal HA sync, zones remain on the secondary;

only one issue,- after entering the voucher code it's not redirected to the web site, browser URL
shows https://<IP_ADDRESS>:8002/index.php?zone=zone1
tested on 2.5.0.a.20200512.2320 cluster

Thanks @vktg for your feedback !

only one issue,- after entering the voucher code it's not redirected to the web site, browser URL
shows https://<IP_ADDRESS>:8002/index.php?zone=zone1. No such issue with 2.4.5 captive portal.

Nice catch ! ...I wasn't paying attention to this during my own tests
It's fixed now.

@Augustin-FL please rebase your fork and resolve merging conflicts so @vktg can re-test it and we can get it merged

Hi,
It's done.

Main redmine issue : https://redmine.pfsense.org/issues/97 (Captive Portal should sync its database to other members of clusters) - primary to secondary is OK, but regardless of whether I set up on the secondary->primary sync page, when primary returns back, it removes all active users:

logportalauth | 15154 | Zone: zone1 - DISCONNECT: wVF6NMsuzef, 0c:dd:ae:8c:e7:00, <IP_ADDRESS>

and I don't see any active vouchers from the secondary;

https://redmine.pfsense.org/issues/8807 (HA sync : files voucher_{$cpzone}.cfg and voucher_{$cpzone}.public are not created on secondary node) - OK;
https://redmine.pfsense.org/issues/8809 (HA sync : changing a voucher roll on master does not reset active tickets on secondary) - OK;
https://redmine.pfsense.org/issues/9303 (HA sync : disabling captive portal HA sync does remove all zones on secondary) - OK;
after entering the voucher code it's not redirected to the web site, browser URL
shows https://<IP_ADDRESS>:8002/index.php?zone=zone1. - OK,  no such issue now;

Main redmine issue : https://redmine.pfsense.org/issues/97 (Captive Portal should sync its database to other members of clusters) - primary to secondary is OK, but regardless of whether I set up on the services_captiveportal_hasync.php page, when primary returns back, it removes all active users:

logportalauth | 15154 | Zone: zone1 - DISCONNECT: wVF6NMsuzef, 0c:dd:ae:8c:e7:00, <IP_ADDRESS>

and I don't see any active vouchers from the secondary;

I'm sorry to ask this but....Could check if you didn't simply reach the time limit of your voucher, and that your XMLRPC sync is correctly configured on the primary node?
I do confirm that this is working (I just made the same test to make sure)
Here are the logs I am getting while performing the test
(on the primary)

(On the secondary)

(Not sure if it is related, but while updating pfSense to the latest developpment version - before applying my PR - I got an error about config.xml was missing. It could be somehow related to this? )

I'm sorry to ask this but....Could check if you didn't simply reach the time limit of your voucher, and that your XMLRPC sync is correctly configured on the primary node?

You are right, this is my configuration error, after previous test.
All works fine now!
tested on 2.5.0.a.20200518.1031 HA pair


[release-2.7] Cherrypicks for Flink Runner

@charlesccychen Here the requested commits from the mailing list thread.
https://issues.apache.org/jira/browse/BEAM-5239
https://issues.apache.org/jira/browse/BEAM-5246
Post-Commit Tests Status (on master branch)

Lang
SDK
Apex
Dataflow
Flink
Gearpump
Samza
Spark

Go

---
---
---
---
---
---

Java

Python

---

---
---
---
---

Should we add the port number fix:  https://github.com/apache/beam/commit/84a571ef8452d273310ff6309c8bc17242e5c5fb

Sure, I added it.

Run Java PostCommit


[Transform][BUG] remove static when forward sourceTypeStrs

Sorry, it is not what I expected.  some_array_ref = std::array<...>{...} is always wrong...
std::array<...>{...} is a temp value. It is different from below, which is correct:
std::array<...> a{...};
some_array_ref = a;

Please note that a is a named variable, which is killed at the end of the scope, while some_array_ref = std::array<...>{...} kills the array after the end of the statement.
I re-post the comments in the previous PR:
Please always avoid writing code like
ArrayRef<std::string> x = {"abc"};
somefunc(x);

Please note that "abc" has type of char[4] which is not std::string. The compiler inserts an conversion here, and the code looks like
ArrayRef<std::string> x = {std::string("abc")};

Also note std::string("abc") is a temp value and the lifetime ends at the full evaluation of the statement, which means it is dead after the ; at this line. And x outlives the temp value and points to a dead pointer.
However, the following code is valid:
ArrayRef<const char*> x = {"abc"};
ArrayRef<StringRef> y = {"abc"};

Below also valid
void somefunc(ArrayRef<std::string> x);
....
somefunc({"bf16"});

Also I suggest using std::array<...>{x,y,z} over std::array<...>({x,y,z}). The () are redundant, and will trigger ambiguous-error in some old compilers.


register this thanks

Requirements

[x] You have completed your website.
[x] The website is reachable.
[x] The CNAME record doesn't contain https:// or /.
[x] There is sufficient information at the owner field.

Description

I use this amazing domain for my own blog
Link to Website

https://skyshadowhero.github.io

CLA Assistant Lite bot: Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.

I have read the CLA Document and I hereby sign the CLA

You can retrigger this bot by commenting recheck in this Pull Request

I have read the CLA Document and I hereby sign the CLA

Added skyshadowhero.is-cool.dev using the CLI.

I have read the CLA Document and I hereby sign the CLA

CLA Assistant Lite bot: Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.
I have read the CLA Document and I hereby sign the CLA
You can retrigger this bot by commenting recheck in this Pull Request

I have read the CLA Document and I hereby sign the CLA

recheck


Make code samples runnable/editable in the browser

At the YAPC::NA Perl 6 hackathon someone (@Util maybe?) had the suggestion that the code samples on the front page of perl6.org should be runnable/editable. I believe the glot.io API could be used for this and @zoffixznet has already implemented such functionality on his site.

Would be really nice if we make it possible not to just run the examples, but make them editable so that users can tweak them and run again. That'd be soooo cool… :)

but make them editable

They are editable on perl6.party

@finanalyst, @Util, this ticket may be of interest to you both. (Please coordinate first)


Allow to get IP from X-Forwarded-For header

I'm running my app in AWS with a load balancer. The load balancer overwrites the IP header in req.ip, so, if express-brute locks a request, it will be locked for all clients. I need a way to get the IP from X-Forwarded-For header. I have a middleware that does this:
const clientIp = req.get("X-Forwarded-For") || req.connection.remoteAdress;
Is it possible to add an option to use this IP instead the IP in req.ip?

Yeah I know that the header can be overwritten, but it's always possible for me to get the last ip in the chain, which will be the real ip. A whitelist is not an option for us since the proxy ip is not predictable, the infrastructure is created and scaled on demand, so the load balancer ip changes.
Maybe add an option to pass a function to get the ip any way the user want? something like this:
var bruteforce = new ExpressBrute(expressBruteStore, {
    freeRetries: 3,
    minWait: 1000,
    maxWait: 10000,
    lifetime: 30,
    getIp: req => (req.get("X-Forwarded-For") || req.connection.remoteAdress || req.ip)
});

I think the behavior you're asking for is provided by this:
app.set('trust proxy', 1);

I think the behavior you're asking for is provided by this:
app.set('trust proxy', 1);

This should be added to the documentation.


feat: Added ignoreFiles option to CLI

Description
Adding an option to pass extra list of .*ignore files to install command, to get more flexibility to end user:
xdpm install --ignore-files ".gitignore, .xdignore, .npmignore"

More details in related issue #38
Related Issue
#38
Motivation and Context
Described in related issue #38
How Has This Been Tested?
Manually on Windows, but I believe there is no cross-platform issues.
Types of changes

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

Checklist:

[x] I have signed the Adobe Open Source CLA.
[x] My code follows the code style of this project.
[x] My change requires a change to the documentation.
[x] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

@ashryanbeats @kerrishotts Friendly reminder

ping-ping

🙃


chore: bumps docker-cli-js v2.5.3

Bumps docker-cli-js to the latest release

Thank you very much for the update!
I'll do it myself in the next hour.
It's just that package-lock.json (where I manually fixed the CVE issue with v2.5.2) needs to change or be removed from the repo (I removed it from .gitignore in some version)
Thanks again,
I'll drink my coffee and do it.

I submitted this in v1.8.9, thanks again ^_^


IO error on Android when sqlite uses the temp store

Describe the bug
When using op-sqlite on Android version 12 or lower, SQLite returns IO error (code 10) for large queries/transactions. This is likely caused by sqlite trying to write a temporary store file. It seems to be the same issue as https://github.com/ospfranco/react-native-quick-sqlite/issues/80 : the workaround described there solves the issue. In our case, several users were affected by this issue and it was hard to figure out the cause from an opaque [op-sqlite] SQLite error code: 10, description: disk I/O error. happening to some users and not to others (depending on their DB size), for the same queries.
Versions:

OS and version: Android 12 and lower. We don't see it happening on 13 and above
op-sqlite version: 3.0.7. Have you tried the latest version? No
RN version: 0.71.14
New Architecture: No

Reproducible example
In our case, a transaction with ALTER TABLE ... DROP COLUMN on a large table (60k+ rows and 20 columns) was triggering the issue. Reproducible on the Android 12 emulator.
Proposed fix
Set the SQLITE_TEMP_STORE=2 flag in the Android CMakeLists.txt (currently done only if sql cipher is enabled). IMHO there is no value in keeping the temp store on files knowing that it will fail (on a sufficiently large amount of devices) as soon as sqlite needs that. Nor to expect users to configure each time the SQLite connection to mitigate it, when it can be done at compile time by the library, and when not documented. We patched the library this way to fix the issue.

You can pass compilation flags yourself in the object in the package.json:
{
	// your normal package.json
	// ...
	"op-sqlite": {
	  "sqlcipher": true,
	  "crsqlite": true,
	  "performanceMode": "1",
	  "iosSqlite": false,
	  "sqliteFlags": "-DSQLITE_TEMP_STORE=2", // Add it here
	  "fts5": true,
	  "libsql": true
	}
}

Let me know if this works!

Thanks, we will try that as soon as we update to a recent version 🙏
The point of the issue was to bring up the fact that by default configuration, the bundled sqlite doesn't seem to work on all Android versions and maybe it would be a better option if the default configuration of op-sqlite would do so instead, by defaulting to SQLITE_TEMP_STORE=2 (seems like Android uses it as well). Or at least mention this in the op-sqlite documentation. Your choice of course.
I'll close the issue, for us it's solved, hopefully it will help someone else getting to the cause of the IO error faster than we did.


chore: Update Material UI and Emotion

Description 📝

Updates MUI and Emotion to latest
Updating MUI fixes the issues where scrolling with a Menu open would not move the menu correctly. This allows us to re-enable scrolling while action menus are open.

https://github.com/linode/manager/pull/9540#issuecomment-1677665164
https://github.com/mui/material-ui/pull/37773

Preview 📷

Before
After

Notice how you can't scroll 😣
Notice how you can scroll and the action menu remains in the correct position 🎉

How to test 🧪

Checkout the PR
Run yarn for good measure
Run yarn dev and verify the app works and looks normal

I'll be sure to follow up about the other Menus causing scroll lock @jdamore-linode.
The MacOS scroll bounce is unfortunate. I'll keep an eye out for fixes for that.


GCM Notifications doesn't register a client

Hi! After pressing Register button message Registering ..... is shown, and nothing after that. The SenderId is valid. The register.js script can't be viewed in the debugger

@jianli-chromium might be able to help.


Bug: mismatched selection color

Describe the bug
By default selection color is blue.
It would be nice if we make it match the brand theme color. 😃
Screenshots:
Default:

Expected (I injected CSS locally):

Steps to reproduce

Click, hold, and drag the mouse cursor to highlight the text content (most people do this to select and copy the text content).

Browsers
Chrome
Additional context (Is this in dev or production?)
No response
Code of Conduct

[X] I agree to follow this project's Code of Conduct

Contributing Docs

[X] I agree to follow this project's Contribution Docs

You can achieve this by adding the following selection class in the body: selection:bg-light-orange-8 selection:text-light-slate-12
https://github.com/open-sauced/insights/blob/a4e27169833ca91fcb86852e9f091104a5283739/pages/_document.tsx#L7
Feel free to play with the values and propose something with a nice contrast :pizza:

Thanks, @takanome-dev! That's super helpful.

@wasimapinjari  Would you like to open a PR in the landing page too?

@wasimapinjari Would you like to open a PR on the landing page too?

Sure, I think we can do that everywhere:
opensauced.pizza
hot.opensauced.pizza
docs.opensauced.pizza
api.opensauced.pizza

I am not sure about the API, but the other repos are a good idea.

Yeah, we need different colors for the API page, orange wouldn't work there. We can go ahead and skip that for now.

Sure, feel free to open a PR in the other repos.


How to correctly configure the backup path

@clcc2019 I hope the access key ID and secret have been disabled.


⚠ Bofrost.de has degraded performance

In b3c4858, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1891 ms

Resolved: Bofrost.de performance has improved in c07bd9a after 6 minutes.


WIP: fix tests for real wallets

Codecov Report

Merging #1761 (f5d0fc7) into main (8ed2422) will increase coverage by 1.85%.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main    #1761      +/-   ##
==========================================
+ Coverage   49.08%   50.94%   +1.85%
==========================================
  Files          47       47
  Lines        7112     7112
==========================================
+ Hits         3491     3623     +132
+ Misses       3621     3489     -132

see 10 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

Got a fix for the 2 eclair tests:
diff --git a/lnbits/wallets/eclair.py b/lnbits/wallets/eclair.py
index 97d39064..7eea5a37 100644
--- a/lnbits/wallets/eclair.py
+++ b/lnbits/wallets/eclair.py
@@ -71,8 +71,6 @@ class EclairWallet(Wallet):
     ) -> InvoiceResponse:
         data: Dict = {
             "amountMsat": amount * 1000,
-            "description_hash": b"",
-            "description": memo,
         }
         if kwargs.get("expiry"):
             data["expireIn"] = kwargs["expiry"]
@@ -81,6 +79,8 @@ class EclairWallet(Wallet):
             data["descriptionHash"] = description_hash.hex()
         elif unhashed_description:
             data["descriptionHash"] = hashlib.sha256(unhashed_description).hexdigest()
+        else:
+            data["description"] = memo

         async with httpx.AsyncClient() as client:
             r = await client.post(


Does not respect Xmodmap Settings

Issue Type: Bug
I have an Xmodmap setting configuration file with the below in it, however, within Azure Data Studio (And VSCode, which is installed via Snap) this is not respected. Buttons 6 and 7 are not reversed. This means that the thumb scroller I have on my mouse needs to be scrolled right to scroll left (and left for right).
.Xmodmap file:
pointer = 1 2 3 4 5 7 6

These settings are respected by everything else I have installed that supports the functionality (Allows side scrolling). It's just Azure Data Studio and VSCode that does not.
Azure Data Studio version: azuredatastudio 1.7.0 (e1280022d69b651cfff04b30e830904575c8acda, 2019-05-07T21:35:31.938Z)
OS version: Linux x64 5.0.0-16-generic

System Info

Item
Value

CPUs
Intel(R) Core(TM) i7-8700 CPU @ 3.20GHz (12 x 4299)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: disabled_softwaresurface_synchronization: enabled_onvideo_decode: unavailable_offwebgl: enabledwebgl2: enabled

Load (avg)
4, 4, 4

Memory (System)
15.33GB (8.58GB free)

Process Argv
--unity-launch

Screen Reader
no

VM
0%

Extensions (1)

Extension
Author (truncated)
Version

vscode-icons
vsc
8.7.0

@LarnuUK  Hi, Azure Data Studio is a fork of VSCode, and we implement the database related features, could you please open an issue in VSCode github repo? https://github.com/microsoft/vscode, once they fix the issue we can get it into Azure Data Studio.
closing as an upstream issue.
Thanks

Thanks @alanrenmsft , looks like it's a known issue on VSCode: #23911. Changing the Keyboard: Dispatch doesn't fix, as it appeared it used to as well, so I'll watch that bug for now.
Cheers.


[1.12.2] Game crashes when you look at the mob: 'Botfly Larva' from Erebus.

Minecraft: 1.12.2
Forge: <IP_ADDRESS>29
The One Probe: theoneprobe-1.12-1.4.21
Crash Report: https://pastebin.com/wTHk3qDX

Figure I post this here as well, but anyways... in the Erebus there's a mob called "Botfly Larva which... when looked at will cause the game to crash. If you disable the 'isVisible' in the in-game config it won't crash.

This is a bug in The Erebus. There's nothing that can be done about it on our side. I opened ganymedes01/TheErebus#596 to fix it on their side.


Consolidate config types and YAML annotations

This change consolidates tracing.Config and tracing.TracerConfig into one type with corresponding YAML tags which would allow allow us configure the tracing logger. In addition, this provides YAML tags for any other configuration type.
Closes #364

Oh and we also need to add yaml tags to log.SentryConfig, as it's currently used in baseplate.Config: https://pkg.go.dev/github.com/reddit/baseplate.go#Config.Sentry


arrowMarkup bindings

Nested elements in gallery.arrowMarkup trigger close.
Adding the class "mfp-prevent-close" to them fixes the issue, but this seems like a weird default behaviour.

Still an unfixed Issue in 2016 :(


[PAGINATION] Create Pagination UI components

Figma Link:- https://www.figma.com/file/JPcD0ljV8yyfLBnzkiqMVc/Flutter-UI-Component?node-id=159%3A45465&t=VEhHTa8C85SrPyM7-0

I want to work on this issue under JWOC.


Allow easy apppending of file upload types in app.js

Per #145:
"We should provide a neat and tidy option for it, and for adding new files to an existing group without restating everything"

This already exists in the 0.6 refactor. See the fileGroups option in apostrophe-files/index.js

Reopening because currently you can't just add to / manipulate the existing list, you have to start from scratch.

Yeah, I think this is more about having easily appended file types from app.js.

Right... what we have in 0.6 already makes it easy to override the file
types from app.js, but it is not yet easy to append.
On Tue, Apr 28, 2015 at 11:33 AM, benirose<EMAIL_ADDRESS>wrote:

Yeah, I think this is more about having easily appended file types from
app.js.
—
Reply to this email directly or view it on GitHub
https://github.com/punkave/apostrophe/issues/193#issuecomment-97106853.

--
*THOMAS BOUTELL, *DEV & OPS
P'UNK AVENUE |<PHONE_NUMBER>  |  punkave.com

Correct, I'll change the name of this issue to reflect that, and remove it from the milestone, since that's really more a nice to have than something we should hinge 0.6's completeness on.

Whoops, this wasn't even the 0.6 version of the ticket. Putting this one in the backlog.

This was addressed for 2.0. It will not be changing in 0.5 which is in maintenance mode.


404 error on the generated XML

Hi,
I have install the Geta.Seo.Sitemap in my project, after installation we are getting some issue when generate sitemap. It seems all xml is created when running the sitemap job but when you click on url it give 404 error.
Following is the structure of our site
XYZ_PublicWeb -- In manage Website, and under that we have all the language domain
www.xyz.com
www.xyz.ae
www.xyz.zh
www.xyz.fr
etc
In Serach engine sitemap Settings tools, we define domain as
www.xyz.com/en/sitemap.xml
www.xyz.ae/ae/sitemap.xml
www.xyz.zh/zh/sitemap.xml
www.xyz.fr/fr/sitemap.xml
language is add in url becasue we selected language in each domain while creating sitemap.
Thanks in Advance

Hi @mkadiwar . Do you still need help on this issue?
Regards,
Rui Vaz

No, Thank you ☺
Br,
Malik Kadiwar
Consultant | AppsTwo CSD
Capgemini India | Mindspace, Airoli
Tel.: +91 2 26 68 60 500 (EXT: 2219406) – Mob.: + 91 70453 70317
www.capgemini.comhttp://www.capgemini.com/
[https://visualidentity.capgemini.com/capgemini.png]

Connect with Capgemini:
[cid:image010.png@01D35EC5.D54D2F50]https://www.capgemini.com/insights-and-resources/blogs[cid:image011.png@01D35EC5.D54D2F50]https://www.twitter.com/capgemini<EMAIL_ADDRESS>https://www.facebook.com/capgemini<EMAIL_ADDRESS>https://www.linkedin.com/company/capgemini<EMAIL_ADDRESS>https://www.youtube.com/capgeminimedia<EMAIL_ADDRESS>https://www.slideshare.net/capgemini<EMAIL_ADDRESS>https://plus.google.com/+CapgeminiGlobal
Please consider the environment and do not print this email unless absolutely necessary.
Capgemini encourages environmental awareness.
From: Rui Vaz<EMAIL_ADDRESS>Sent: Monday, May 14, 2018 8:42 PM
To: Geta/SEO.Sitemaps
Cc: Kadiwar, Malik; Mention
Subject: Re: [Geta/SEO.Sitemaps] 404 error on the generated XML (#68)
Hi @mkadiwarhttps://github.com/mkadiwar . Do you still need help on this issue?
Regards,
Rui Vaz
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/Geta/SEO.Sitemaps/issues/68#issuecomment-388853084, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ANrrEvRC9oP0MZ9U25AjRDgz9XPabPj5ks5tyZ7PgaJpZM4QhsV2.
This message contains information that may be privileged or confidential and is the property of the Capgemini Group. It is intended only for the person to whom it is addressed. If you are not the intended recipient, you are not authorized to read, print, retain, copy, disseminate, distribute, or use this message or any part thereof. If you receive this message in error, please notify the sender immediately and delete all copies of this message.


Roll aws sdk to v1.7.4

TBR

Codecov Report

Merging #3254 into master will not change coverage.
The diff coverage is n/a.

Continue to review full report at Codecov.

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


[Twilio Auth Provider] SMS messages display a wrong Sender

Bug report
Describe the bug
OTP codes sent via SMS comprise a title (ie. “Presence") and a body (ie. your “Your Presence code is: 827347”).
However, when Twilio's messages pass through Supabase, the Alphasender gets substituted by a generic phone number (not to be confused with Twilio Service SID). Twilio Support says that these numbers belong to their batch of phone numbers used under the hood to dispatch the SMS. For some reason, Supabase is assigning these numbers to Sender.
Users will be suspicious if they receive OTP codes from generic phone numbers that are not clearly linked to the app they have signed up.
To Reproduce

Sign in with phone number
Wait for the SMS
See the number as the alphasender instead of Presence

Issue is not reproducible when sending messages directly from Twilio console via CURL method. Issue occurs only when messages pass through Supabase.
Expected behavior
It depends on the countries, but usually Sender should be either Customer’s custom Alphasender (in our case: “Presence”) or Twilio’s default Alphasender (“AUTHMSG”).
Screenshots

System information

Version of supabase-flutter: 1.2.2

Additional context
n/a

Hey I'm sorry to be bouncing this back to Twilio but GoTrue (the software behind Supabase Auth) is not able to control the phone number. Please check your Twilio configuration.
Here's the code that does this. Twilio translates the SID into a phone number. We are unable to choose the phone number for you.


Light Rail trains issue with door side

Expected Behaviour
When train pulls into depot, it is flipped, since it is expected that the train will come in front first, the train is flipped so that it can leave front first again.
Actual Behaviour
On Light Rail train (and all variants, phase 1-5), the doors are only on one side. When using a turnback rail for the route, the train ends up going into the depot back first (instead of the expected front first, since it never actually completed a loop). This means that when it is flipped, it ends up leaving the depot "backwards", with the doors facing the wrong side. This is fine on all other trains since the doors are on both sides anyway. But not with Light Rail variants. What end up happening is the train car has doors on the opposite side every new route.
Reproduction Steps
Steps to reproduce the behaviour: Depot --> platform 1 --> turnback --> platform 2 --> turnback --> depot
Screenshots attached below of route that causes the issue

Possible solution: Instead of just blindly flipping the train every time it enters the Depot, reset to a standard position (front facing), so that no matter if the train enters frontwards or backward, the output from depot is the same.

Actually I think this is a non-issue, since that's the expected behaviour.


Refactor frontend grant access form section into reusable component

Describe the task
Refactor the frontend grant access form sections into reusable components so we can reusable them when grant application admin and delegated admin

[x] create a new folder under grantaccess folder called "form"
[x] separate the user domain radio selection into a new component
[x] separate the username and verification into a new component, add validation
[x] separate the adding forest client number and verification into a new component, add validation
[x] update the current GrantAccess.vue component to use the reusable form component, make sure it works as expected

Additional context

Add any other context about the task here.
Or here

created a new ticket #1092 to keep improve the logic when handle submissions and push notifications.


WebView does not render content

I can't get the WebView control to render anything, not even hardcoded HTML. I've created a sample project by creating a new project with dotnet new fabulous-mauicontrols -n WebViewTest and replaced the ScrollView with the following code:
VStack(spacing = 5.) {
    Label("Title")
    Label("Date")
    WebView(html = "
Test Test Test
")
    WebView(url = "http://www.google.com")
    Label("After WebView")
}

Debugging the app with the profile "Windows Machine" I can only see the labels, but not the WebView:

Here's the solution I used:
WebViewTest.zip

@causa-prima. Tested this on android, iOS, macCatalyst and works as expected, based on your screenshot Im assuming that you are running this in Windows right ?. I think is not a Fabulous issue might be windows related maybe permissions. Also have you read https://learn.microsoft.com/en-us/dotnet/maui/user-interface/controls/webview?view=net-maui-8.0&pivots=devices-windows ?

@edgarfgp : Yes, I'm running on windows (couldn't get the android emulator to work correctly yet). And yes, I've read the documentation you linked to. At the end of the first paragraph you can find the following sentence:

By default, .NET MAUI projects include the platform permissions required for a WebView to display a remote web page.

Thus, I'd assume that it's not a permission-related problem. I'll still dig around to see if I can find any related info that will help (any hints are appreciated).
Btw: Your android screenshot seems to be missing the Title and Date labels, and both display the second Test as bold in the hard coded HTML example. I guess you adjusted the code, right?

Well, this is awkward. In my example, you just need to set the height of the WebView to some fixed value, e.g. WebView(html = "
Test Test Test
").height(50.). I guess the behavior of VStack differ between platforms in the way the sizes are calculated. With your findings, I also gave the android emulator a try, and with my original example the layout in my actual app was switching between two different states, but with a fixed size everything is fine.
So my lesson learned for today: If you can't see something that should be there, try explicitly changing the size to something that must be visible.
Anyways, thanks for the quick support that lead me to the right solution!

No worries. Happy that you figure out this :). Feel free to raise more issues if necesary


Consider Qt options for Mac Retina Display

Boyce Griffith reported this Are there settings that can be used to control the resolution scaling for VisIt.app in OS X? I am trying to use a prebuilt executable of VisIt on a MacBook with retina display, and the default scaling generates very grainy text. This can be fixed by changing the scaling of everything to More Space, but Id prefer to keep the scaling at Best for everything but VisIt. If this can be addressed by building from source, I can do that I am using the prebuilt executables because I ran into some issues between clang and Qt when trying to build VisIt 2.6.3. (Maybe these are fixed with 2.7?) Thanks! Honestly, I am not able to see the difference on my MacBook Pro but my vision isn't that good either. The attached image, when you zoom in, you see a clear difference in the text rendered in VisIt's GUI and the text rendered by, for example, the terminal app. Note that I had to zoom in on the image before my eyese could see the difference. I breifly looked on-line and ran across this dialog http://blog.qt.digia.com/blog/2013/04/25/retinadisplaysupportformacosiosandx11/
-----------------------REDMINE MIGRATION-----------------------
This ticket was migrated from Redmine. As such, not all
information was able to be captured in the transition. Below is
a complete record of the original redmine ticket.
Ticket number: 1679
Status: Pending
Project: VisIt
Tracker: Bug
Priority: Normal
Subject: Consider Qt options for Mac Retina Display
Assigned to: -
Category: -
Target version: -
Author: Mark Miller
Start: 12/07/2013
Due date:
% Done: 0%
Estimated time:
Created: 12/07/2013 01:29 pm
Updated: 12/17/2013 05:18 pm
Likelihood: 3 - Occasional
Severity: 2 - Minor Irritation
Found in version: 2.7.0
Impact:
Expected Use:
OS: OSX
Support Group: Any
Description:
Boyce Griffith reported this Are there settings that can be used to control the resolution scaling for VisIt.app in OS X? I am trying to use a prebuilt executable of VisIt on a MacBook with retina display, and the default scaling generates very grainy text. This can be fixed by changing the scaling of everything to More Space, but Id prefer to keep the scaling at Best for everything but VisIt. If this can be addressed by building from source, I can do that I am using the prebuilt executables because I ran into some issues between clang and Qt when trying to build VisIt 2.6.3. (Maybe these are fixed with 2.7?) Thanks! Honestly, I am not able to see the difference on my MacBook Pro but my vision isn't that good either. The attached image, when you zoom in, you see a clear difference in the text rendered in VisIt's GUI and the text rendered by, for example, the terminal app. Note that I had to zoom in on the image before my eyese could see the difference. I breifly looked on-line and ran across this dialog http://blog.qt.digia.com/blog/2013/04/25/retinadisplaysupportformacosiosandx11/
Comments:
From Boyce


Colorpicker closing after selecting a color, when another picker is open

Description
Colorpicker closes on any click when you have navigated/clicked from another open colorpicker

Context

bootstrap-colorpicker version: 3.1.2
bootstrap version: 4.2.1
jQuery version: 3.4.1
Browser name and version: Chrome 80.0.3987.149
Operative System name an version: Windows 10

Expected behavior

Have two colorpickers created on the page, but not shown
Open one of them and select a color
Whilst still having the first one open, click the element that will show the second colorpicker. The first colorpicker will now close and the second one will open.
Click anywhere on the second colorpicker and remains open, and can still be interacted with

Actual behavior

Have two colorpickers created on the page, but not shown
Open one of them and select a color
Whilst still having the first one open, click the element that will show the second colorpicker. The first colorpicker will now close and the second one will open.
Click anywhere on the second colorpicker and it will close (unexcepted behaviour, it should stay open)

It should only close when the user has clicked outside of the second colorpicker.
It works correctly if you click outside of the first colorpicker anywhere on the page where another colorpicker won't be shown.
Live Example
Couldn't get jsfiddle to work with the links provided for a template.
Just create two input which each have a colorpicker.
Click on one to get the corresponding colorpicker to open.
Then click on the other input whilst the first colorpicker is still shown.
Then click anywhere on the second colorpicker and it will close

This is expected behavior. The same happens with browser's HTML select elements (combobox): If you use one, the other is collapsed.

Sorry thats not what I was getting at.
Let me try to explain it again.
Given two dialogue boxes that each open separate colourpicker instances, clicking one dialogue box and choosing a colour, then immediately clicking into the other dialogue box, closes the first colourpicker instance (expected behaviour) and then opens and immediately closes the second colourpicker instance before allowing you to choose a colour (unexpected behaviour).
This is a bug.
I can try and record whats happening if you are still unsure.

@itsjavi This is reproducible on the demo page as well: https://itsjavi.com/bootstrap-colorpicker/tutorial-Basics.html
Repro steps;

Click on a colorpicker
Click on another colorpicker

As you say it's expected is that the first color picker is closed. All good here.
However, what's not expected is that as soon as you interact with the new colorpicker, it closes immediately after that interaction. The expected behaviour is for it to stay open.
Note that this does not happen when popover.animated = false, however turning off animation causes other problems, like colorpicker not opening after it has been closed.
It's possible it might be a bug in popover, any thoughts on potential causes are welcome.
Thanks!

For your convenience, I've recorded the bug here: https://drive.google.com/file/d/1hbuxS7TpPK17wjwO-WYEjW-g0x5Mac0Z/view

Thanks for the example, it's clearer now. I will reopen the issue. @macduy


add onTransition function to transitions

should take dispatcher, source and target state and fsm data as parameters
allows for custom behaviour during transition over and above OnExit/OnEnter

added as an effect as per uml spec


[Shortcut Guide] Guide mis-numbers taskbar items when using 7+ Taskbar Tweaker

Environment
Windows build number:  Microsoft Windows [Version 10.0.18363.657]
PowerToys version: 0.15.1
PowerToy module for which you are reporting the bug (if applicable): Shortcut Guide

Steps to reproduce

Install 7+ Taskbar Tweaker
Enable "Don't group pinned items"

Open a pinned application and drag the open application tile next to the pinned icon tile.  In this example, the terminal is next to the pinned terminal icon.

Open the Shortcut Guide and look at the numbers.

Press Win+4 and expect to open up Explorer
Actually re-focuses the open terminal window because that's the 4th distinct item in the taskbar.
Move Terminal window so it's not adjacent to the Terminal icon
Open up the Shortcut Guide and see they're numbered correctly.

Expected behavior
Shortcut guide should properly take into account taskbar tiles which aren't attached to the pinned icon.
Actual behavior
When an application is right next to it's own pinned icon it should be numbered independently.  My assumption is that there's some special logic to collapse icons and applications to deal with something that happens when the pinned apps are open.

I understand that this is not exactly a "supported" scenario since it's using 7+ Taskbar Tweaker which is essentially a dll hijack and it depends on internal implementation details.  But having the shortcut guide work for this scenario would be great!

we'll look into it, no promises

Looked at this a bit today and here's what I found:
The UIA_NamePropertyId contains the name of the button (e.g. Windows Terminal (Preview) when the application is not launched, or if app is running it contains the name of the window title along with a "X running window(s)" suffix (e.g. Untitled - Notepad - 1 running window).  This could be used to skip de-duping with pinned buttons,
Unfortunately (aside from being a little hacky), this is probably localized so wouldn't be cross platform.
Other problem is that the hotkey activation applies assigns a single hotkey to groups, so depending on the grouping option you select you get different results.

If you are using the "Group by Application ID" option, the pinned button gets one number, and the group of active apps gets another.

If you are using the "Don't group", the button and each application gets it's own number.  And because they're both running apps with the "running windows" suffix, there's no obvious way to tell the difference.

So to make it work properly you'd actually need some way to identify which items were grouped together.  I'm sending the 7+ Taskbar Tweaker dev a message to see if he's willing to point out how he's handling it.

This is due to a 3rd party software, don't plan to fix due to fix would be dependent on their implementation and could impact other scenarios then


device status not updating

The problem
I can see my Tuya devices with the integration,  however, if the light is actually on.  HomeAssastant shows the light is off.
I can turn the light on with HA,  however, the UI will return to "off".
if the light is on,  in HA it will show as off,  and turn turn the light off with HA,  I have to click the light to on, and then quickly to off

after several hours,  it seems to be working..


Disable autogenerating DatadogMetric for each HPA

For the e2e tests, configure the Datadog Cluster Agent to not create autogenerated DatadogMetric objects for each HPA that is created.
This is the recommended way to deploy the Cluster Agent when using KEDA with the DCA, but this wasn't reflected in the tests configuration.
Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)

/run-e2e datadog

Let's move the agent e2e test from sequential to the folder where the other is (as the problem with the self-generated metrics is solved)


InputAccessoryView doesn't work inside React Native Modal

Environment
Environment:
OS: macOS Sierra 10.12.6
Node: 8.6.0
Yarn: 1.7.0
npm: 5.3.0
Watchman: 4.9.0
Xcode: Xcode 9.2 Build version 9C40b
Android Studio: 2.3 AI-162.4069837
Packages: (wanted => installed)
react: 16.3.1 => 16.3.1
react-native: https://github.com/expo/react-native/archive/sdk-30.0.0.tar.gz => https://github.com/expo/react-native/archive/sdk-30.0.0.tar.gz
Description
InputAccessoryView doesn't work when used inside a react-native Modal.
Reproducible Demo
https://snack.expo.io/@kadoshms/inputaccessoryview-inside-modal

Facing the same issue. Any idea?

I'm facing the same issue and it's actually blocking in my situation.
Are there any ways to workaround this issue?

any news on this? i'm facing the problem as the numeric textinput is inside a modal and it's blocking me

@euroclydon37 are you able to preare a PR with a fix using your findings?

This is a real bug but it is unlikely we'll be able to spend any time on a fix so I'm going to close this issue. It seems that you already have a way to fix it so if you could send a Pull Request, we are happy to review and land it.

I know this is closed but am blocked by this myself and would love a fix. @euroclydon37 any chance of a PR?

Interested as well! I'm running into this problem

any update with this ?


Patron Controller receives incorrect object from MVC binding handler

When attempting to Put to the Patron Controller, the binding handler creates a CreatePatron object and attempts to pass that to the action. Putting a break point into the CommandInputFormatter proves that at some point MVC is binding to a UpdatePatronHeader command and the Model.Type is picking the proper class.
{
  "ExceptionType": "Microsoft.AspNetCore.Diagnostics.ExceptionHandlerFeature",
  "Message": "Unable to cast object of type 'Vigil.Patrons.Commands.CreatePatron' to type 'Vigil.Patrons.Commands.UpdatePatronHeader'.",
  "StackTrace": [
    "   at lambda_method(Closure , Object , Object[] )",
    "   at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.<InvokeActionMethodAsync>d__27.MoveNext()",
    "--- End of stack trace from previous location where exception was thrown ---",
    "   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()",
    "   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)",
    "   at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.<InvokeNextActionFilterAsync>d__25.MoveNext()",
    "--- End of stack trace from previous location where exception was thrown ---",
    "   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()",
    "   at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.Rethrow(ActionExecutedContext context)",
    "   at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.Next(State& next, Scope& scope, Object& state, Boolean& isCompleted)",
    "   at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.<InvokeNextResourceFilter>d__22.MoveNext()",
    "--- End of stack trace from previous location where exception was thrown ---",
    "   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()",
    "   at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.Rethrow(ResourceExecutedContext context)",
    "   at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.Next(State& next, Scope& scope, Object& state, Boolean& isCompleted)",
    "   at Microsoft.AspNetCore.Mvc.Internal.ControllerActionInvoker.<InvokeAsync>d__20.MoveNext()",
    "--- End of stack trace from previous location where exception was thrown ---",
    "   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()",
    "   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)",
    "   at Microsoft.AspNetCore.Builder.RouterMiddleware.<Invoke>d__4.MoveNext()",
    "--- End of stack trace from previous location where exception was thrown ---",
    "   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()",
    "   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)",
    "   at Microsoft.AspNetCore.Diagnostics.EntityFrameworkCore.MigrationsEndPointMiddleware.<Invoke>d__5.MoveNext()",
    "--- End of stack trace from previous location where exception was thrown ---",
    "   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()",
    "   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)",
    "   at Microsoft.AspNetCore.Diagnostics.EntityFrameworkCore.DatabaseErrorPageMiddleware.<Invoke>d__6.MoveNext()",
    "--- End of stack trace from previous location where exception was thrown ---",
    "   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()",
    "   at Microsoft.AspNetCore.Diagnostics.EntityFrameworkCore.DatabaseErrorPageMiddleware.<Invoke>d__6.MoveNext()",
    "--- End of stack trace from previous location where exception was thrown ---",
    "   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()",
    "   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)",
    "   at Microsoft.AspNetCore.Diagnostics.ExceptionHandlerMiddleware.<Invoke>d__6.MoveNext()"
  ]
}

Turns out, the issue was with caching in the CommandInputFormatter. Problem has been fixed.


rmw_cyclonedds: 0.23.0-1 in 'rolling/distribution.yaml' [bloom]

Increasing version of package(s) in repository rmw_cyclonedds to 0.23.0-1:

upstream repository: https://github.com/ros2/rmw_cyclonedds.git
release repository: https://github.com/ros2-gbp/rmw_cyclonedds-release.git
distro file: rolling/distribution.yaml
bloom version: 0.10.7
previous version for package: 0.22.2-1

rmw_cyclonedds_cpp
* Fix zero copy issues. (#309 <https://github.com/ros2/rmw_cyclonedds/issues/309>)
* Handle allocation errors during message deserialization. (#313 <https://github.com/ros2/rmw_cyclonedds/issues/313>)
* Update includes after rcutils/get_env.h deprecation. (#312 <https://github.com/ros2/rmw_cyclonedds/issues/312>)
* Contributors: Christophe Bedard, Michel Hidalgo, Sumanth Nirmal

Held for Rolling sync.


Dry run option

Is there an option to not have it write in place?
If I just want to dry run this application without actually changing anything is that possible?

There is not currently a dry run option, but one way to "fudge" would be to commit the files you want to run it on before running undebt. Then you can jump between both versions.

Would you be opposed if I added one?

@seemethere In addition to using git like @ajm188 suggested, If you pipe input to undebt over stdin, it will write the output to stdout. Thus, you could do
cat ./test_file.txt | undebt -p ./some_pattern.py

to just print the modified file instead of actually changing it in-place.

@seemethere That being said, I wouldn't at all be opposed to adding an actual dry run option, if that's something you wanted to try to do.

@seemethere of course you're welcome to submit a PR :)


Use sphinx directive to generate output of full examples in the docs

Something like .. click:run:: directive.
https://github.com/pallets/pallets-sphinx-themes/blob/main/src/pallets_sphinx_themes/themes/click/domain.py

I just managed to do it in click-extra. Here is how I did it:
Add Pallets-Sphinx-Themes dependency
Add a little workaround in Sphinx's conf.py
And now you can use both .. click:example:: and .. click:run:: in your docs.
Note that both directives needs to resides in the same {eval-rst} MyST block.
It's kind of hackish but OK to me as it lighten up a bit the burden of documentation maintenance.

For the record, I added a bug report to the sphinx-click project to implement this feature instead of having it embedded within the Pallets-Sphinx-Themes project.

Thanks for the useful info, @kdeldycke!

I just made something really cool for the upcoming click-extra v3.0.0 release: I managed to have the .. click:run:: directive output ANSI colored shell session in Pygments. It's a little janky but does the job.

FYI, I just streamlined and documented the setup of .. click:example:: and .. click:run:: directives at: https://kdeldycke.github.io/click-extra/sphinx.html


Fails on download

Traceback (most recent call last):
File "D:/pythonWork/daily/youtube/3.py", line 10, in
pl.download_all()
File "C:\software\Python\Python36\lib\site-packages\pytube\contrib\playlist.py", line 172, in download_all
dl_stream.download(download_path, filename_prefix=prefix)
File "C:\software\Python\Python36\lib\site-packages\pytube\streams.py", line 206, in download
filename = filename or self.default_filename
File "C:\software\Python\Python36\lib\site-packages\pytube\streams.py", line 176, in default_filename
title = self.player_config_args['title']
KeyError: 'title'

See #434


Food for thought: Some type of media page?

I'm putting together a presentation for an upcoming workshop and I'd like to add some MBON related graphics (logos, pretty pictures, etc.). Right now I have to wonder around the existing site and try to copy paste images of interest. It would probably be beneficial to others if we had a resource somewhere on this site for media, where it has links to various images and branding pieces folks might use in presentations.

Coming back around to this as I'm looking for MBON related images for other activities. Would be nice to have a media page with some high resolution images/photos to use for other efforts.


KubeVIP Static Pod - Unable to find Namespace (serviceaccount mount missing)

I have KubeVIP running perfectly fine on Ubuntu 18 with K8S 1.20.2
On Ubuntu 20 it does not work as it seems it can't find the Namespace.
184 func returnNameSpace() (string, error) {
185     if data, err := ioutil.ReadFile("/var/run/secrets/kubernetes.io/serviceaccount/namespace"); err == nil {
186         if ns := strings.TrimSpace(string(data)); len(ns) > 0 {
187             return ns, nil
188         }
189         return "", err
190     }
191     return "", fmt.Errorf("Unable to find Namespace")

My understanding is that Static Pod can't mount secrets so not sure how this is supposed to work.
Am I missing something ?

This is correct a static pod doesn't run "in" Kubernetes.. can you attach your kube-vip manifest so I can attempt to reproduce?

Sorry for the delay. I use this manifest:
apiVersion: v1
kind: Pod
metadata:
  creationTimestamp: null
  name: kube-vip
  namespace: kube-system
spec:
  containers:
  - args:
    - start
    env:
    - name: vip_arp
      value: "true"
    - name: vip_interface
      value: ens192.4011
    - name: vip_cidr
      value: "32"
    - name: vip_leaderelection
      value: "true"
    - name: vip_leaseduration
      value: "50"
    - name: vip_renewdeadline
      value: "30"
    - name: vip_retryperiod
      value: "10"
    - name: vip_address
      value: <IP_ADDRESS>
    image: plndr/kube-vip:latest
    imagePullPolicy: Always
    name: kube-vip
    resources: {}
    securityContext:
      capabilities:
        add:
        - NET_ADMIN
        - SYS_TIME
    volumeMounts:
    - mountPath: /etc/kubernetes/admin.conf
      name: kubeconfig
    - mountPath: /etc/ssl/certs
      name: ca-certs
      readOnly: true
  hostNetwork: true
  volumes:
  - hostPath:
      path: /etc/kubernetes/admin.conf
    name: kubeconfig
  - hostPath:
      path: /etc/ssl/certs
    name: ca-certs
status: {}

In the logs, in the working cluster I see this:
root@fab2-k8s-1:~# docker logs 90a1c933c26e
time="2021-02-22T00:49:01Z" level=info msg="Beginning cluster membership, namespace [kube-system], lock name [plndr-cp-lock], id [fab2-k8s-1]"

Closing this is there has been no reply.

Hello @thebsdbox ,
Would you mind elaborating on the problem you can see please? I am following the steps you have linked to generate the manifest, and it's giving me exactly the same "unable to find Namespace" error.

Can you attach your manifest?

Sorry for the delay. I don't seem to be getting notifications from github...!
Here's the manifest I was using when trying the static-pod:
`apiVersion: v1
kind: Pod
metadata:
creationTimestamp: null
name: kube-vip
namespace: kube-system
spec:
containers:

args:

manager
env:
name: vip_arp
value: "true"
name: port
value: "6443"
name: vip_interface
value: ens160
name: vip_cidr
value: "32"
name: svc_enable
value: "true"
name: vip_leaderelection
value: "true"
name: vip_leaseduration
value: "5"
name: vip_renewdeadline
value: "3"
name: vip_retryperiod
value: "1"
image: ghcr.io/kube-vip/kube-vip:v0.4.0
imagePullPolicy: Always
name: kube-vip
resources: {}
securityContext:
capabilities:
add:

NET_ADMIN
NET_RAW
SYS_TIME
volumeMounts:

mountPath: /etc/kubernetes/admin.conf
name: kubeconfig
hostAliases:

hostnames:

kubernetes
ip: <IP_ADDRESS>
hostNetwork: true
volumes:

hostPath:
path: /etc/kubernetes/admin.conf
name: kubeconfig
status: {}`

@thebsdbox Any clues from the above, please? Thanks in advance.

I thought I had responded to this, apologies. Looking at the manifest it appears that the control plane load-balancing isn't enabled only services. If you're only doing services then it should be deployed as a daemonset and isn't required to be deployed as a static pod.

Got it, thank you!


Azure Functions Runtime is Unreachable caused by setting a Read-Only app setting

Add another section/cause for the Azure Functions Runtime is Unreachable
Setting a Read-Only Environment Variable
When setting environment variables or app settings, which are marked as "Read-Only" the function host will fail to start, resulting in the Azure Function Runtime is Unreachable.
Review the function app's application settings to ensure that there are no "Read-Only" Environment Variable getting set.
Steps to Reproduce the issue:
On the Function App -> Configuration blade -> Add a Read-Only app setting "REGION_NAME" with any value.

Then go to the Functions blade in the portal and you will see the Azure Functions runtime is unreachable.

Note:
The portal has some validation checks for some of these app settings, but not all.
I have tested the following and the result of passing the validation - All of them caused the runtime unreachable error:
WEBSITE_SITE_NAME - Fails
WEBSITE_RESOURCE_GROUP - Succeeds
WEBSITE_OWNER_NAME - Succeeds
REGION_NAME - Succeeds
WEBSITE_PLATFORM_VERSION - Succeeds
HOME - Fails
SERVER_PORT - Succeeds
WEBSITE_COMPUTE_MODE - Fails
WEBSITE_SKU - Fails
SITE_BITNESS - Succeeds
WEBSITE_HOSTNAME - Fails
WEBSITE_VOLUME_TYPE - Succeeds
WEBSOCKET_CONCURRENT_REQUEST_LIMIT - Succeeds
WEBSITE_SCM_ALWAYS_ON_ENABLED - Succeeds
WEBSITE_SCM_SEPARATE_STATUS - Succeeds
There are some settings that I did not test, as there is a good bit more Read-only settings that can be set and will be environment/stack dependent.

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

ID: 75028922-3176-8802-9f85-62c92c381784
Version Independent ID: 1e1c0232-aee5-59c9-fc1e-f8d7a8cf73a8
Content: Troubleshoot error: Azure Functions Runtime is unreachable
Content Source: articles/azure-functions/functions-recover-storage-account.md
Service: azure-functions
GitHub Login: @ggailey777
Microsoft Alias: glenga

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

Duplicate MicrosoftDocs/azure-docs#101678