Modifier -> SwiftUI -> Clipboard: onPasteCommand(of:validator:perform:)

Doc: https://developer.apple.com/documentation/swiftui/view/onpastecommand(of:validator:perform:)-4r73j

[ ] Swift implementation
[ ] Elixir implementation

According to the @supernintendo @carson-katri @shadowfacts we won't support this


Program received segment fault in configuration and initialization

When I ran lightLDA on a single machine, I get the following message.
[INFO] [2016-06-30 11:49:39] INFO: block = 0, the number of slice = 1
[INFO] [2016-06-30 11:49:39] Server 0 starts: num_workers=1 endpoint=inproc://server
[INFO] [2016-06-30 11:49:39] Server 0: Worker registratrion completed: workers=1 trainers=1 servers=1
[INFO] [2016-06-30 11:49:39] Rank 0/1: Multiverso initialized successfully.
[INFO] [2016-06-30 11:49:43] Rank 0/1: Begin of configuration and initialization.
Segmentation fault (core dumped)
In other issues, say issue#15, I find that segmentation fault is usually caused by wrong tf count, which happened after "End of configuration and initialization". But my case happened before  "End of configuration and initialization", so I wonder what can cause the fault of my case?
BTW, when I use gdb to debug, I get the following:
#0  __memset_sse2 () at ../sysdeps/x86_64/memset.S:65
#1  0x000000000041b023 in multiverso::RowFactory::CreateRow(int, multiverso::Format, int, void*) ()
#2  0x000000000041f07d in multiverso::Table::GetRow(int) ()
#3  0x000000000042c4ca in multiverso::Server::StartThread() ()
#4  0x0000003e1feb6470 in std::(anonymous namespace)::execute_native_thread_routine (__p=) at ../../../../libstdc++-v3/src/thread.cc:44
#5  0x00000035248079d1 in start_thread (arg=0x7f054cfc2700) at pthread_create.c:301
#6  0x00000035244e88fd in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:115
Anyone can tell what has happened?

I find maybe this problem was caused by too large total TF count. If I reduce some words' TF count to less than 1 billion, then I will not get "segment fault" problem.


Update xxd comments and version string

The changelog in the top comment record multiple changes between 1998 and 2020. Thanks to everybody for contributing!
I'd like to change some wording in the comment and in the version string:

update my email-address,
strip "Small changes made afterwards" -- I want to avoid to diminuish any contributers achivements.
remove the silly poem, which atributed credit exclusively to myself.

Updates to the version string were neglected in the past. I agree with Bram, that the version number 1.10 is now meaningless.
I propose to put the date in the form yyyy-mm-dd of the last change instead of a version number.
That should help with https://github.com/vim/vim/issues/8472
Thanks for raising the issue!

It's might be better to add your account in .github/CODEOWNERS.

It's might be better to add your account in .github/CODEOWNERS.

I suppose that means adding this at the end:
src/xxd/				@jnweiger

--
hundred-and-one symptoms of being an internet addict:
49.
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///

I suppose this only makes sense, if @jnweiger is actually maintaining xxd

Legally, I own my share of the copyright on xxd. The CODEOWNERS files appears to be more a maintenance tool.
For the last 20 years I have not touched anything in xxd and I am absolutely happy to see how it prospered over time - probably through the help of younger hands. So, please don't make me the main maintainer now. For an occasional review like this, of course, feel free to ping me.

Jürgen wrote:

Legally, I own my share of the copyright on xxd. The CODEOWNERS files
appears to be more a maintenance tool.
For the last 20 years I have not touched anything in xxd and I am
absolutely happy to see how it prospered over time - probably through
the help of younger hands. So, please don't make me the main
maintainer now. For an occasional review like this, of course, feel
free to ping me.

The name CODEOWNERS is misleading, it's not like this is a list of
github users that own the code.  It's a mechanism to bring pull requests
to someone's attention automatically.  For xxd I can't think of anyone
else.
--
hundred-and-one symptoms of being an internet addict:
50. The last girl you picked up was only a jpeg.
/// Bram Moolenaar --<EMAIL_ADDRESS>-- http://www.Moolenaar.net   \
///                                                                      \
\\        sponsor Vim, vote for features -- http://www.Vim.org/sponsor/ ///
\\            help me help AIDS victims -- http://ICCF-Holland.org    ///


add option in "mne report" to get sensitivity maps of forward models

it's a good quality insurance plot for forward models.

something along these lines (copied from a script of @dengemann)
mag_map = mne.sensitivity_map(fwd, ch_type=fwd_ch_type, mode=fwd_ori)
brain = mag_map.plot(subject=subject, time_label='Magnetometer sensitivity',
                     hemi='rh', subjects_dir=subjects_dir, **fwd_plot_args)
brain.save_image('sensitivity_mag.png')
report.add_images_to_section('sensitivity_mag.png', 'sensitivity map',
                             'forward')
os.remove('sensitivity_mag.png')

fig = plt.figure()
plt.hist(mag_map.data.ravel(),
         bins=20, label=['Magnetometers'],
         color=['b'])
plt.title('Normal orientation sensitivity')
plt.xlabel('sensitivity')
plt.ylabel('count')
plt.legend()

+1

@jasmainak do you have time for this in the next two weeks?

okay I'll give some time for fixing MNE stuff on Monday. I've a couple of things to do.

+1
On Fri, Sep 9, 2016 at 2:30 PM Mainak Jas<EMAIL_ADDRESS>wrote:

okay I'll give some time for fixing MNE stuff on Monday. I've a couple of
things to do.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/mne-tools/mne-python/issues/2828#issuecomment-245899394,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AB0fiiaVcWckZu6BFXgoJmmjaOHSIgVkks5qoVFOgaJpZM4HPzH3
.

@jasmainak any time for this one in the next couple of days? If not I'll remove the milestone (or you can)

I have a local branch @Eric89GXL but I fear it's going to take more time. I'll remove it from the milestone

hi,
I’m interested in solving this issue, but will need some help through it.
If I understand correctly, a Report class method needs to be added to append 'sensitivity map' to the report? and is ‘sensitivity/count histogram’ mentioned above also required to be appended?
and if a method, will the input parameters to it would be same as that of 'mne.sensitivity_map()+SourceEstimate.plot()' or something else?

hi @parulsethi I'd suggest that you start from my work in progress commit here. I don't think I got it working, but it might be easier than starting from scratch.

Also, make sure you understand how to generate a report. You can read the tutorial on the mne report here

@jasmainak thanks for the suggestion, I've forked from there.

@drammock is this really in progress? It seems to me that @jasmainak self-assigned in 2016 :-) I think that's no longer up to date.
I think this is still "up for grabs"

@drammock is this really in progress? It seems to me that @jasmainak self-assigned in 2016 :-) I think that's no longer up to date.
I think this is still "up for grabs"

I was just hoping to inspire @jasmainak to finally get to it 😆

I did say, I'll get to it by Monday .......
2021? :)


I love this extention but for my very specific case i would love to know how i can achieve this:

is there any way i can make it so that the fallback and home colours can have alpha such as rgba or #FFFFFF1e. if not possible is there a way i can make it a gradient ? thanks!

@lunar-os Thanks and you have such a cool setup! It might be hard to achieve, but I’ll see what I can do.


Add locales for fil(filipino) and gsw(swiss german) to locale list, Fixes #60631

Description
This fixes a bug that broke localization generation for the flutter gallery which uses fil and gsw locales.
Related Issues
This fixes #60631
Tests
This script has no tests.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I signed the CLA.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I updated/added relevant documentation (doc comments with ///).
[x] All existing and new tests are passing.
[x] The analyzer (flutter analyze --flutter-repo) does not report any problems on my PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Did any tests fail when you ran them? Please read Handling breaking changes.

[x] No, no existing tests failed, so this is not a breaking change.
[ ] Yes, this is a breaking change. If not, delete the remainder of this section.

[ ] I wrote a design doc: https://flutter.dev/go/template Replace this with a link to your design doc's short link
[ ] I got input from the developer relations team, specifically from: Replace with the names of who gave advice
[ ] I wrote a migration guide: https://flutter.dev/go/breaking-changes-template Replace this with a link to a pull request that adds the migration guide to https://flutter.dev/docs/release/breaking-changes

LGTM. This is sort of a hack, but the main underlying issue is actually with the filename parsing algorithm that was recently introduced in #60185.
I'll spend some time thinking about what we can do to have a smart filename parsing algo. @Renzo-Olivares Do you happen to know if fil and gsw are language identifiers that are particular to Google?

I don't believe they're particular to Google. I did a quick search to see how Android handles locales and came it came up with this https://developer.android.com/reference/java/util/Locale#getISO3Language() .

Three-letter language/country(region) codes
The Locale constructors have always specified that the language and the country param be two characters in length, although in practice they have accepted any length. The specification has now been relaxed to allow language codes of two to eight characters and country (region) codes of two to three characters, and in particular, three-letter language codes and three-digit region codes as specified in the IANA Language Subtag Registry. For compatibility, the implementation still does not impose a length constraint.

link to registry https://www.iana.org/assignments/language-subtag-registry/language-subtag-registry

Seems like an unrelated tools test is failing on windows. Since flutter-build is also failing, maybe wait until the tree is green, then rebase/merge from master to see if that fixes the issue


Incorrect error message after removing the VM

While setup finished
$ ./crc setup
INFO Checking if oc binary is cached
INFO Checking if podman remote binary is cached
INFO Checking if CRC bundle is cached in '$HOME/.crc'
INFO Unpacking bundle from the CRC binary
INFO Checking if running as non-root
INFO Checking if Virtualization is enabled
INFO Checking if KVM is enabled
INFO Checking if libvirt is installed
INFO Checking if user is part of libvirt group
INFO Checking if libvirt is enabled
INFO Checking if libvirt daemon is running
INFO Checking if a supported libvirt version is installed
INFO Checking if crc-driver-libvirt is installed
INFO Installing crc-driver-libvirt
INFO Checking for obsolete crc-driver-libvirt
INFO Removing older system-wide crc-driver-libvirt
INFO Will use root access: remove old libvirt driver
INFO Checking if libvirt 'crc' network is available
INFO Setting up libvirt 'crc' network
INFO Checking if libvirt 'crc' network is active
INFO Starting libvirt 'crc' network
INFO Checking if NetworkManager is installed
INFO Checking if NetworkManager service is running
INFO Checking if /etc/NetworkManager/conf.d/crc-nm-dnsmasq.conf exists
INFO Checking if /etc/NetworkManager/dnsmasq.d/crc.conf exists
INFO Writing dnsmasq config for crc
INFO Will use root access: write dnsmasq configuration in /etc/NetworkManager/dnsmasq.d/crc.conf
INFO Will use root access: execute systemctl daemon-reload command
INFO Will use root access: execute systemctl stop/start command
Setup is complete, you can now run 'crc start' to start the OpenShift cluster

Unable to perform delete
$ ./crc delete
Do you want to delete the OpenShift cluster? [y/N]: y
ERRO Driver "libvirt" not found.  Use 'crc start' to create it.

$ ls -al ~/.crc/bin
total 94228
drwxr-x--- 2 gbraad gbraad       56 12 mrt 15:53 .
drwxr-xr-x 5 gbraad gbraad      133 12 mrt 15:02 ..
-rwxr-xr-x 1 gbraad gbraad 11988744 12 mrt 15:53 crc-driver-libvirt

$ ~/.crc/bin/crc-driver-libvirt
This is a hypervisor plugin binary for CodeReady Containers.
Please use this plugin through the main 'crc' binary.
(Driver version: 0.12.7, API version: 1)

So likely an error got returned and resulted in the wrong message?
If seems a domain exists, so start fails:
$ ./crc start
$INFO Creating CodeReady Containers VM for OpenShift 4.3.1...
ERRO Error creating host: Error creating the VM: Error creating machine: Error in driver during machine creation: virError(Code=9, Domain=20, Message='operation failed: domain 'crc' already exists with uuid f57311c6-821f-455c-890f-ccdb0c23c971')

Probably we check for a wrong state. first we need to add a domain check. I think this is an exsting issue, but also make sure the message is not vague: Do you have the plugin binary accessible in your PATH?. We should not have to ask this, as this should live in the ~/.crc/bin folder...

@gbraad I am only able to reproduce this if I delete the crc-driver-libvirt from the ~/.crc/bin/ directory.
$ rm -fr ~/.crc/bin/crc-driver-libvirt

$ ./crc delete
Do you want to delete the OpenShift cluster? [y/N]: y
ERRO Driver "libvirt" not found.  Use 'crc start' to create it.

$ sudo virsh list
[sudo] password for prkumar:
 Id   Name   State
----------------------
 3    crc    running


Add xib2Storyboard

Project URL
https://github.com/novemberfiveco/xib2Storyboard
Category
Development
Description
A tool to convert Xcode .xib to .storyboard files.
Why it should be included to Awesome macOS open source applications  (optional)
Checklist

[x] Only one project/change is in this pull request
[x] Addition in chronological order (bottom of category)
[x] Appropriate language icon(s) added if applicable
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

          1 Warning

:warning:
Found 1 link issue, a project collaborator will take care of these, thanks :)

Link issue by awesome_bot

Line
Status
Link

124
301
https://github.com/jamiestraw/streaker redirects tohttps://github.com/jamieweavis/streaker

  Generated by :no_entry_sign: Danger


Fix to use ":" in xml attribute name.

@RickardNorman Thank you for your PR. Unfortunately, I can't accept it for these reasons:

You are fixing ":" issue, but for unknown reason upgrading libraries versions. I would prefer to have atomic commits. So if you want to upgrade xunit or other libraries, it would be better to have another commits/PR.
For some reason, I can't see correct diff for src/MagicChunks/Documents/XmlDocument.cs. Take a look your commit. It just removes whole file content and adding very similar, but new. I guess it could be tab/space issue or something.

If you're ok to fix these issues I'm happy to accept your PR.
Thank you!


switch to angular#master^HEAD

Let's use angular#master^HEAD as our main version.
NOTE: this is forcing us to stay up to date but also, this may introduce some breaking changes, so we should be reactive and fix bugs as they arise ^^

Closed by #50


nixos/mosquitto: add bind_interface listener option, fix assertion messages

Description of changes
the bind_interface listener option is not currently supported by the mosquitto module. add that setting and a test for it, and fix assertion messages that mention the wrong attribute path for invalid settings as well.
closes #172944
Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

I'm not particularly familiar with how the nixos tests work, but it looks like it's just checking that mosquitto is listening on en0. Surely it should check to make sure it's not listening on any other interface as well (such as the loopback interface)?

indeed, the test only checks whether bind_interface does something that looks roughly correct (picking addresses from the given interface and listening on those). checking whether mosquitto behaves exactly as the docs say would belong in the mosquitto test suite, here we only check whether everything is hooked up correctly that bind_interface can work at all. (eg at first we hadn't allowed the AF_NETLINK yet, so the server just didn't start with that setting given)

My concern is that by just checking eth0 you aren't actually ensuring that bind_interface was specified correctly in the mosquitto conf at all. The reason being that a listener block { port = 1889; } will result in listener 1889, which will listen on all interfaces including eth0. So if the bind_interface command is mis-specified (omitted, written wrong, doesn't actually apply to the listener in question, etc) then the test won't catch that.
Or to put it another way, bind_interface doesn't mean "bind to this interface", it means "don't bind to any other interface but this one", and so we should be testing that it's not binding to other interfaces. But we do still need the eth0 test to make sure we're actually listening somewhere.

I haven't run it but this new test looks better, thanks.

@ofborg test mosquitto

@lilyball have you had the chance to test whether this fixes your problem?

going to merge this now since this only adds more allowed config keys, the tests pass, and we've already missed the branch-off anyway. once independently verified we can backport as well.

I finally tested this in my local setup and it does appear as though mosquitto is binding just to the interface specified. Thanks!


Query BDD Test by productId

As a Developer
I need to describe the behavior of listing all promotions by productId
So that I can test the feature under DBB
Details & Assumptions:

Write the feature using Gherkin syntax that is understood by the behave tool

Acceptance Criteria:
When run tests using `behave`
Then I should see “list all promotions by productId query” scenario pass

IBM Cloud toolchain: Delivery Pipeline deployed promotions to prod, including commits 9fc2661199993d8990995905067cd7dac4192450, 594968bcce6b80cd84ae9d4c43896a7c0c1dd42a


Remove the dependency on Jupyter Book or sphinx-external-toc

We've just released a new version of sphinx-external-toc and realized that we have a circular dependency between jupyterbook-latex, jupyter-book, and sphinx-external-toc.
From the PR where we discovered this, @chrisjsewell noted:

Really jupyterbook-latex no longer needs to depend on jupyter-book for testing (now that it uses this package for toc based stuff and is "independent" of jupyter-book), so those tests can be removed to remove the circular dependency

So I think we have two options:

remove tests in this repository that depend on Jupyter Book, remove the dependency on jupyter book, and pin only to a specific version of sphinx-external-toc
remove the dependency on sphinx-external-toc and inherit it from jupyter-book

I think we need to resolve this (or at least release a new version of this repo) before we can release a new version of Jupyter Book so that we don't create a version conflict.

@choldgraf the way we have been thinking of this repo is as an extension to jupyter-book so I think option 2 makes sense. @chrisjsewell has suggested we turn this into a pure sphinx extension so we could at that stage look at dropping dependency on jupyter-book and sphinx-external-toc
cc: @AakashGfude

the way we have been thinking of this repo is as an extension to jupyter-book so I think option 2 makes sense. @chrisjsewell has suggested we turn this into a pure sphinx extension so we could at that stage look at dropping dependency on jupyter-book and sphinx-external-toc

What, no option 1 is the only option. I've already turned it into a pure sphinx extension, that was the whole point of my refactor 😬, there is already no dependency on jupyter-book, just some legacy testing. There is also no direct dependency on sphinx-external-toc

But, jupyterbook-latex does use some config variables declared in jupyter-book like jblatex_captions_to_parts ?
But also, currently when implementing tableofcontents directive for latex, I am using the site_map provided by sphinx-external-toc. So, it kinda depends on both 👀 ?

You can see this already in the install_requires, the only requirement is sphinx.
In fact really you can just relax the version pinning of sphinx-external-toc (to include v0.2.0) in the test extra and that should be fine

@chrisjsewell just digging into this and looking at the setup.cfg.
Was thinking of updating
sphinx-external-toc~=0.1.0

to be
sphinx-external-toc>=0.1.0

I've already turned it into a pure sphinx extension

Right -- so we should probably also rename this repo something like sphinx-jupyterbook-latex.

jupyterbook-latex does use some config variables declared in jupyter-book like jblatex_captions_to_parts

@AakashGfude just looking at events.py and it seems those config values are being pulled from sphinx.app now.

But, jupyterbook-latex does use some config variables declared in jupyter-book like jblatex_captions_to_parts?

No it doesn't, that's a variable of this extension, it's not in any way dependant on jupyter-book.

sphinx-external-toc>=0.1.0

Always include a top pinning, so sphinx-external-toc>=0.1.0,<0.3.0

Right -- so we should probably also rename this repo something like sphinx-jupyterbook-latex.

Yep that's a possibility 👍

Right -- so we should probably also rename this repo something like sphinx-jupyterbook-latex.

Yep that's a possibility 👍

I have opened #61 to discuss the options

So I’m a bit confused because https://github.com/executablebooks/jupyterbook-latex/pull/60 didn’t seem to remove the Jupyter-book dependency. Can somebody confirm that this issue is no longer an issue? Or post what needs to be done to close this issue?

@choldgraf the setup.cfg only lists jupyter-book in testing dependencies. So it shouldn't depend on jupyter-book via standard install. Where do you see the dependency issue?

@choldgraf does this make sense? Is there anything else I can help with here?
I think the change suggested by @chrisjsewell resolves a version pinning conflict (causing the circular dependency issue?)

I have setup a release https://github.com/executablebooks/jupyterbook-latex/pull/63 once approved by @chrisjsewell I can merge and push to PyPI

@choldgraf I went ahead and merged #63 so should be available for the jupyter-book=0.11 release


Create a demo scene with experiments

The corridor scene is almost ready, but not pushed yet.
While working on this, some related cleanup can be done as well:

[ ] fix "item" vs "object" naming (e.g. InteractiveItem is wrong)
[ ] create a get_state_name method
[ ] provide access to current scene name (remote only in the room)

This is especially relevant with many new GOAT features being implemented recently. However, the corridor scene will probably not be used. Instead, there will be one "showcase" scene redirecting to individual scenes presenting specific features.


License of Dark Destiny?

Is it open source?
It's worth to be clarified.
https://sourceforge.net/projects/darkdestiny/

https://sourceforge.net/p/darkdestiny/code/HEAD/tree/doc/readme.txt mentions the license is GPL but no source file has any notice. However, the project uses MIT and GPL licensed code.

Sent a small message on the discussions part of the Sourceforge project page.
https://sourceforge.net/p/darkdestiny/discussion/636398/thread/1c049f93/

Sent a message to user zhdophanti (https://sourceforge.net/u/zhdophanti/profile/) who was last active on the project like two years ago.

I didn't get a reply from zhdophanti, so I wrote to the other author desertratx (http://sourceforge.net/u/desertratx/).

I converted the SVN sources to Git and put them on Gitlab (https://gitlab.com/osgames/darkdestiny) with a reference to the original project. I will add an entry and inform the authors about it.

I updated the existing entry and informed the authors about it.


Remove write permission from pages after write out the generated code from recompiler

Right now the permission is read, write and execute. Better to remove write just in case so we don't have any strange bugs in the future.

Due to #144 we don't need this anymore.


Value's returning 'NULL_VALUE' for all fields as well as other data types

Previously if Value.nullValue was equal to "NULL_VALUE" then the Value was null, however since version 4.0.0 all my firestore documents field values have nullValue == "NULL_VALUE" in addition to their correct values such as a integerValue or mapValue

Could you give me more information, @jimmyff ?
Here's the source bug: https://github.com/google/googleapis.dart/issues/211
Here's the corresponding fix: https://github.com/google/googleapis.dart/commit/560d4b32db5fdd641d55f24209755022d6115414

The commit didn't fix the issue. https://github.com/google/googleapis.dart/commit/560d4b32db5fdd641d55f24209755022d6115414#r56290820


Stars not rendering in production

I have Vite + React + Typescript application and I am using this react-rating library for displaying ratings in a modal.
Versions:
"react": "^18.2.0",
"@smastrom/react-rating": "^1.5.0",
It is working as expected in in dev but as soon as I build it, the stars disappears.
Here's what I found:
Svgs in development:
<svg aria-hidden="true" class="rr--svg" xmlns="http://www.w3.org/2000/svg" viewBox="0 0 60 58" preserveAspectRatio="xMidYMid meet"><g shape-rendering="geometricPrecision" transform="translate(-2 -3)"><path d="M62 25.154H39.082L32 3l-7.082 22.154H2l18.541 13.693L13.459 61L32 47.309L50.541 61l-7.082-22.152L62 25.154z"></path></g></svg>
The same svg in production:
 <svg aria-hidden="true" class="rr--svg" xmlns="http://www.w3.org/2000/svg" viewBox="0 0 0 0" preserveAspectRatio="xMidYMid meet"><g shape-rendering="geometricPrecision"><path d="M62 25.154H39.082L32 3l-7.082 22.154H2l18.541 13.693L13.459 61L32 47.309L50.541 61l-7.082-22.152L62 25.154z"></path></g></svg>
I noticed that viewBox is set to "0 0 0 0", if I manually edit the html, I can see my svg icons.
Oh! and I should also mention that the Rating is rendered inside a headlessui modal.

I have the same issue (
@naumanafsar do you have some workaround ?

I'm having the same problem.

@eshimischi The project is dead, look for alternatives.


test(Checkbox): remove incorrect event combinations

In #3435 we introduced changes to Checkbox tests, but these was a mistake in cycles.
We should pass event names instead of array to simulant.fire().

Codecov Report

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

@@           Coverage Diff           @@
##           master    #3691   +/-   ##
=======================================
  Coverage   99.83%   99.83%
=======================================
  Files         175      175
  Lines        3116     3116
=======================================
  Hits         3111     3111
  Misses          5        5

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 c884fa9...28de573. Read the comment docs.


[11.x] Fix 'pushProcessor method not found on LoggerInterface' error

I came across this trying to unit test a package I'm working on where I want to use a logger spy. Though the only requirement stated is that the logger must be of Psr\Log\LoggerInterface, in practice, the log channel cannot be built.
Due to the change when adding the Context functionality, there's an implicit dependency that the LoggerInterface used by Logger must have a pushProcessor() method (provider by Monolog's ProcessableHandlerInterface).
The alternative approach is to add a method for Logger@pushProcessor().
todo:

[x] add unit tests

Can this be formatted differently? I can't really tell what is going on.

@taylorotwell hopefully this is easier to read. I find use of tap() inside of tap() to be very hard to read and to reason about, so I removed a tap statement.
For clarity, the goal is to only call Logger::pushProcessor() if the Logger::$logger instance has a method called pushProcessor.
As I was modifying this, another thought would be to allow Logger::withContext() to accept a Closure. Seems like a bigger, more error-prone lift.
The failing tests look like they're related to sqlite changes from earlier today.

Thanks!


ARROW-11134: [CI][C++] Always run tests on Travis-CI

Also try to fix a compiler crash issue by limiting build parallelism.

Travis-CI builds on my fork:

https://travis-ci.com/github/pitrou/arrow/builds/214424780
https://travis-ci.com/github/pitrou/arrow/builds/214563104

Note that s390x seems to have problems on Travis-CI currently.

Travis-CI builds on my fork:

https://travis-ci.com/github/pitrou/arrow/builds/214424780
https://travis-ci.com/github/pitrou/arrow/builds/214563104

Note that s390x seems to have problems on Travis-CI currently.


Stopped working on Dell UP3017 on MBP16 since v2.2.0

Describe the bug
monitorcontrol has been working great on my MBP16 on Catalina and later Big Sur and my Dell UP3017 monitor. I had installed it with brew. Yesterday I did brew update and it installed v2.2.0. The sliders seem grayed out however I can move them, but they do nothing. Only occasionally, if you move them around a lot, a change in brightness might register. I tried different polling settings etc to no avail. I then tried the v3.0.0 betas and they behaved the same way. I then installed the v2.1.0 dmg which works flawlessly.
To Reproduce

Install v2.2.0 (or later release)
Move the brightness slider around

Expected behavior
I expect to see change in brightness on my monitor, but it doesn't happen.
Environment Information (please complete the following information):

macOS version: 11.5.0
Mac model: 2019 Macbook Pro 16
MonitorControl version: v2.2.0
Monitor(s): Dell UP3017 (HDMI connected via StarTech DKT30CHCPD adapter)
Apple Silicon/M1 (yes or no): no

See #478 🙂

@JoniVR this is not #478, I'm not using function keys; I am moving the sliders with my mouse but they do nothing. Only rarely will I see a change in brightness actually happen as I move them.

Your best bet is following the steps in the linked thread for brew.
I've been thinking about deleting 2.2.0 altogether but I'm not sure if that'd help brew users.
I'm still thinking it's the same issue though.. have you tried 3.0.0 beta 4? It helps alleviate it a bit for some people. If it's unrelated I'll reopen it but for now I think it's probably the same issue..

Yes, I tried v3.0.0-beta4 and it has the same, very intermittent behavior, it's hit or miss, as in 5% hit, 95% miss :-) . v2.1.0 is flawless: clicking or dragging the sliders changes the brightness immediately. Running ddcctl on the terminal always works (it's my last resort). I don't mind following #478 if it will fix the problem, although that one focuses on function key behavior and states moving sliders with the mouse in the menu works fine which is not the case for me. I guess I'm seeing what's reported reported in #238. Thank you for looking into this, and for a really great utility!

Yeah sliders worked but that was only because the slider move basically spams it, your specific config might be even more sensitive to it.
I'm not sure what the issue is yet but I plan on either fixing it in DDC.swift or just swapping out the entire implementation soon anyways so hopefully that'll fix your issue too. If it still works intermittently then I hope that'll fix it for you, if it doesn't then feel free to reopen or create a new issue 🙂


Expand search to include results for all custom diagrams

Currently, we're looking at tiles and the single model. Expand to include each diagram in search results. Navigation to that diagram on result click.

Search now includes diagram and tile names, and inventory, diagrams and tiles content.


Documentation

Right now there are no JSDoc comments in the exposed API. This, along with better TypeScript type definitions, is key for DX as it speeds up workflows by making it so that developers don't have to go to the browser to have explanations about the functions and parameters they want to use/pass to those functions. IDEs are just going to show you the documentation on hover.
Also having more descriptive names for functions, parameters and generic parameters/types is a self documenting way of helping the developers have a lesser overhead when developing

Majority is addressed in #252, now only a few host functions (hashing, crypto primitives, etc.) are missing and JSDoc site generation


Use admin-on-rest

Hi, and thanks for this project.
Part of what you're trying to achieve (a React admin based on a REST API) now exists as a standalone repository, in a slightly more advanced version. It's called admin-on-rest.
I think this repo would benefit a lot from reusing admin-on-rest components.

Thanks for bringing my attention to this.
I haven't yet thought about integrating an admin functionality to this boilerplate, but if I do, I'll look into admin-on-rest.
For now, this project is meant to serve as a starting point for building any SPA based modern web application.

Also, admin-on-rest is not based on Flask like React-Redux-Flask is.

admin-on-rest is a client-side component that can adapt to any server API, including any API you can build with Flask.

Ahh true, I did not notice that the intention was to be backend-agnostic (and got pretty caught up with the fact that "someone had already rolled a flask backend" because this was pretty close to what we wanted)
I recognize the importance of a backend agnostic admin panel in many cases.


Fetch images with api

Dependent on https://github.com/praekelt/molo/pull/411
This functionality attempts the following:

fetch all of the images from a site, that do not currently exist on the current site
update a dict referencing foreign image IDs to local IDs

Note:

Images are matched by title, as the image titles have not been changed per site - according to the content team. (Future enhancement could include fuzzy matching and/or an interface to check the import of images)
If there is an error with naming, the content will be duplicated, as this is a better use case than missing content.
I have not handled connection issues when requests are made. Given the time constraints and the fact that we host both sites in the interaction this seems like a fair trade-off. Suggestions welcome.

https://praekeltorg.atlassian.net/browse/MOLO-529

Changes since the inital PR comment:
Images are no longer matched by title, but by image dimension and hash. If there are different titles for matched images, the local version of the title is used.

👍


metrics-server on metal not available: FailedDiscoveryCheck

What happened:
In our shoot clusters, for our Gardener metal cloud (team here at FI-TS), the metrics-server is being deployed. However, with our infrastructural setup, the metrics server does not seem to work properly anymore, as e.g. the apiservice v1beta1.metrics.k8s.io is not available, deletion of namespaces is thus hanging and any auto-scaling (horizontal/vertical) won't work.
However, with the PodSecurityPolicy being deployed and hostNetwork enabled (see https://github.com/helm/charts/blob/master/stable/metrics-server/values.yaml) the metrics-server is functional.
We would like to be able to enable/configure such, as it seems like for our Metal Cloud that might be required. How can we go about it and fix the deployment of metrics server?
To me, that would induce a change/extension to the metrics server deployment chart, right?
kubectl describe apiservice v1beta1.metrics.k8s.io
Status:
    Message:               no response from https://<IP_ADDRESS>:8443: Get https://<IP_ADDRESS>:8443: dial tcp <IP_ADDRESS>:8443: connect: connection refused
    Reason:                FailedDiscoveryCheck
    Status:                False
    Type:                  Available

What you expected to happen:
Metrics-server apiservice availability is True.
How to reproduce it (as minimally and precisely as possible):
Reproducible within our metal infrastructure (see https://github.com/metal-pod); guess that's not as reproducible on your end as it is on our, however, hopefully we can find the issue together :)
Anything else we need to know?:
I am part of the Metal Cloud team here at FI-TS.
Approach also via Slack (see https://kubernetes.slack.com/archives/CB57N0BFG/p1568975926015200)
Environment:

Gardener version: currently looking up
Kubernetes version (use kubectl version): v1.14.3 (in shoot cluster), v1.15.3 (client)
Cloud provider or hardware configuration: see https://github.com/metal-pod

Hi @sjentzsch, interesting, thanks for filing the issue.
So, I understood that the main problem for you is that you need to run the metrics-server in the host network, right? The PSP is needed in this case as well, right?
I think it makes sense to add the PSP generally directly in Gardener (we already do this for most other components). I'm not so sure whether we should make hostNetwork=true the default as well, probably not, but if Gardener would deploy the PSP then you could mutate the PSP + the metrics-server deployment to run in the host network if that's required in your environment.
This way we would a) enhance Gardener (because it should deploy the PSP anyways) and b) get rid of the problem that your webhook would need to create additional resources - instead, it can just mutate the few fields for the host networking. WDYT?

Hi @rfranzke Exactly, that makes sense and sounds reasonable.
If you could ensure the PSP for the metrics-server gets deployed via Gardener, we can mutate hostNetwork=true on our end.

Hi @sjentzsch, sounds good. Would you like to file a PR against the metrics-server chart?

Hey @sjentzsch, any update on this?

Hi @rfranzke, we discovered a network issue on our side, which very likely caused such FailedDiscoveryCheck. Thus, we might not need the feature anymore. Please feel free to close the issue and we re-open if it turns out we still need some more customization on this end. Thanks a lot so far, nonetheless!

@sjentzsch what was the network issue?

This was a misconfigured network between the seed and the shoot actually.


Update viewModel syntax

When 2.3.0-pre.3 is released the generator should be updated to the new syntax. Instead of:
<can-import from="appstate" [.]="{value}" />

It becomes:
<can-import from="appstate" as="viewModel" />

could as="{viewModel}" work?

this doesn't work for me ... not sure what else I need to update.

Should just be as="viewModel", ping me today if it's still not working for you.

This has been fixed a while ago.


embulk(v0.10.20) guess with -o option doesn't work correctly

@dmikurube Please take a look when you get a chance.
Issue Type: Bug Report

Write the following environmental information.

OS version: macOS 10.15.7
Java version: 1.8.0_251
Embulk version: 0.10.20

Write all what you did, e.g. your commands executed
embulk guess (v0.10.20) with -o option doesn't work correctly.
% embulk example guess_test
Creating guess_test directory...
  Creating guess_test/
  Creating guess_test/csv/
  Creating guess_test/csv/sample_01.csv.gz
  Creating guess_test/seed.yml

Run following subcommands to try embulk:

   1. embulk guess guess_test/seed.yml -o config.yml
   2. embulk preview config.yml
   3. embulk run config.yml

% embulk guess guess_test/seed.yml -o config.yml
embulk: 'guess' needs exact 1 argument(s).
Usage: embulk [common options] guess [command options] <partial-config.yml>

"embulk guess" guesses missing parameters to complete configuration.

Common options:
   -h, --help                Print help
   -version, --version       Show Embulk version
   -l, --log-level LEVEL     Set log level (error, warn, info, debug, trace)
       --log-path PATH       Output log messages to a file (default: -)
   -X KEY=VALUE              Set Embulk system properties
   -R OPTION                 Command-line option for JRuby. (Only '--dev')

Plugin options:
   -L, --load PATH           Add a local plugin path
   -I, --load-path PATH      Add Ruby script directory path ($LOAD_PATH)
   -C, --classpath PATH      Add $CLASSPATH for JRuby separated by ':'

Other 'guess' options:
   -o, --output              Path to a file to write the guessed configuration
   -g, --guess NAMES         Comma-separated list of guess plugin names

%

The command embulk-0.10.x guess -o config.yml guess_test/seed.yml also output the same result.
( put -o option after guess command)
Write what you expected and observed actually
embulk 0.9.23 generates the configuration file.
% embulk example guess_test_v09
2020-11-19 09:11:42.278 +0900: Embulk v0.9.23
Creating guess_test_v09 directory...
  Creating guess_test_v09/
  Creating guess_test_v09/csv/
  Creating guess_test_v09/csv/sample_01.csv.gz
  Creating guess_test_v09/seed.yml

Run following subcommands to try embulk:

   1. embulk guess guess_test_v09/seed.yml -o config.yml
   2. embulk preview config.yml
   3. embulk run config.yml

% embulk guess guess_test_v09/seed.yml -o config.yml
... snip
in:
  type: file
  path_prefix: /private/tmp/guess_test_v09/csv/sample_
  decoders:
  - {type: gzip}
  parser:
    charset: UTF-8
    newline: LF
    type: csv
    delimiter: ','
    quote: '"'
    escape: '"'
    null_string: 'NULL'
    trim_if_not_quoted: false
    skip_header_lines: 1
    allow_extra_columns: false
    allow_optional_columns: false
    columns:
    - {name: id, type: long}
    - {name: account, type: long}
    - {name: time, type: timestamp, format: '%Y-%m-%d %H:%M:%S'}
    - {name: purchase, type: timestamp, format: '%Y%m%d'}
    - {name: comment, type: string}
out: {type: stdout}

Created 'config.yml' file.

Thanks for reporting the issue! I found that there is a mistake in the option definition. Fixing it in #1344.

It'll be released as a part of next v0.10.21.

@dmikurube Thank you!
It worked as I expected.  Should I keep this issue as open?
./build/executable/embulk-0.10.21-SNAPSHOT.jar guess -o /tmp/config.yml /tmp/guess_test/seed.yml
2020-11-19 13:08:17.656 +0900 [INFO] (main): Started Embulk v0.10.21-SNAPSHOT
2020-11-19 13:08:17.779 +0900 [INFO] (0001:guess): Listing local files at directory '/private/tmp/guess_test/csv' filtering filename by prefix 'sample_'
2020-11-19 13:08:17.780 +0900 [INFO] (0001:guess): "follow_symlinks" is set false. Note that symbolic links to directories are skipped.
2020-11-19 13:08:17.781 +0900 [INFO] (0001:guess): Loading files [/private/tmp/guess_test/csv/sample_01.csv.gz]
2020-11-19 13:08:17.793 +0900 [INFO] (0001:guess): Try to read 32,768 bytes from input source
2020-11-19 13:08:19.796 +0900 [INFO] (0001:guess): Gem's home and path are set by system configs "gem_home": "/Users/hsato/.embulk/lib/gems", "gem_path": ""
2020-11-19 13:08:20.704 +0900 [INFO] (0001:guess): Loaded plugin embulk (0.10.21.snapshot)
2020-11-19 13:08:20.721 +0900 [INFO] (0001:guess): Loaded plugin embulk (0.10.21.snapshot)
2020-11-19 13:08:20.747 +0900 [INFO] (0001:guess): Loaded plugin embulk (0.10.21.snapshot)
2020-11-19 13:08:20.763 +0900 [INFO] (0001:guess): Loaded plugin embulk (0.10.21.snapshot)
in:
  type: file
  path_prefix: /private/tmp/guess_test/csv/sample_
  decoders:
  - {type: gzip}
  parser:
    charset: UTF-8
    newline: LF
    type: csv
    delimiter: ','
    quote: '"'
    escape: '"'
    null_string: 'NULL'
    trim_if_not_quoted: false
    skip_header_lines: 1
    allow_extra_columns: false
    allow_optional_columns: false
    columns:
    - {name: id, type: long}
    - {name: account, type: long}
    - {name: time, type: timestamp, format: '%Y-%m-%d %H:%M:%S'}
    - {name: purchase, type: timestamp, format: '%Y%m%d'}
    - {name: comment, type: string}
out: {type: stdout}

Created '/tmp/config.yml' file.

Thanks for checking! As you confirmed, I think it's okay to close.

Thank you!


Do you have plans for supporting parent-children relationship in ES?

I've already add this function in my fork.
I can send a pull request if you want to add this.

I see your fork and guess that if we create a table with column ID and its parent ID, and create the associated mapping in ES, we can support parent-children model, am I right?
Btw, does parent ID  need to support multi columns like ID?

@siddontang
Yes.
Currently, my implementation will pick up the value of a parent id field and use that as the id of the parent type in ES.
For example
blog table
id 1
title 't'
content 'c'

comment table
id 1
blog_id 1
content 'c'

Here, in the config, blog_id should be specified as parent id field. When syncing from mysql to ES, the value of blog_id, which is 1, will be used as the id of its parent.
I don't see the reason why parent id field need to be constructed with multi columns.

greate, can you add an example in readme and then send the PR?

@siddontang
OK. No problem.

pull request sent: https://github.com/siddontang/go-mysql-elasticsearch/pull/21


chore: expose WaitForUpdateResult

What type of PR is this?
/kind cleanup

What this PR does / why we need it:

This PR makes WaitForUpdateResult of controllerCommon public for abstraction of update failure handling outside of cloud-provider-azure

Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 79.172%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
17691

Relevant Lines:
22345

💛  - Coveralls

Could we finalize on this discussion before getting ahead with this change?
https://github.com/kubernetes-sigs/azuredisk-csi-driver/pull/1667#discussion_r1084404753

Could we finalize on this discussion before getting ahead with this change? kubernetes-sigs/azuredisk-csi-driver#1667 (comment)

This PR should no longer be needed.


Feature/write in suggestion template

Writing to the template works so far. The problem that still exists is that if the key is nested, this is not displayed. This can lead to errors such as 'name':

name
subject - name
sources - licenses - name
licenses - name
resources - name
resources - schema - fields - name
resources - schema - fields - isAbout - name
resources - schema - fields - valueReference - name

The same for path and others
Translated with www.DeepL.com/Translator (free version)


I have to run boot enable again after update

After updating the software or the system ? In both cases, what is your distro ?

I am using Manjaro KDE. It seems to happen after AUR updates that include linux-enable-ir-emitter. Thank you

Can you paste the return of journalctl -u linux-enable-ir-emitter.service ?

vuth@vuth-xps159500 Linux 5.13.11-1-MANJARO x86_64 21.1.0 Pahvo
~ >>> journalctl -u linux-enable-ir-emitter.service
-- Journal begins at Mon 2021-04-05 12:08:29 CEST, ends at Tue 2021-08-24 14:58:46 CEST. --
Jul 23 18:24:51 vuth-xps159500 systemd[1]: Starting enable the infrared emitter...
Jul 23 18:24:52 vuth-xps159500 systemd[1]: linux-enable-ir-emitter.service: Deactivated successfully.
Jul 23 18:24:52 vuth-xps159500 systemd[1]: Finished enable the infrared emitter.
-- Boot acbbc7bb5a9548609169654675fd891e --
Jul 23 18:46:55 vuth-xps159500 systemd[1]: Starting enable the infrared emitter...
Jul 23 18:46:56 vuth-xps159500 systemd[1]: linux-enable-ir-emitter.service: Deactivated successfully.
Jul 23 18:46:56 vuth-xps159500 systemd[1]: Finished enable the infrared emitter.
-- Boot b8035f93ebf643f1ababb0964aae4c7e --
Jul 23 18:54:42 vuth-xps159500 systemd[1]: Starting enable the infrared emitter...
Jul 23 18:54:43 vuth-xps159500 systemd[1]: linux-enable-ir-emitter.service: Deactivated successfully.
Jul 23 18:54:43 vuth-xps159500 systemd[1]: Finished enable the infrared emitter.
Jul 23 18:55:39 vuth-xps159500 systemd[1]: Starting enable the infrared emitter...
Jul 23 18:55:40 vuth-xps159500 systemd[1]: linux-enable-ir-emitter.service: Deactivated successfully.
Jul 23 18:55:40 vuth-xps159500 systemd[1]: Finished enable the infrared emitter.
-- Boot fe58a8d5887d4b138777ddfd453f39ae --
Jul 24 06:00:35 vuth-xps159500 systemd[1]: Starting enable the infrared emitter...
Jul 24 06:00:36 vuth-xps159500 systemd[1]: linux-enable-ir-emitter.service: Deactivated successfully.
Jul 24 06:00:36 vuth-xps159500 systemd[1]: Finished enable the infrared emitter.
Jul 24 10:11:28 vuth-xps159500 systemd[1]: Starting enable the infrared emitter...
Jul 24 10:11:29 vuth-xps159500 systemd[1]: linux-enable-ir-emitter.service: Deactivated successfully.
Jul 24 10:11:29 vuth-xps159500 systemd[1]: Finished enable the infrared emitter.
-- Boot 4154cf6e0f1b47a9ad8291702cde9b45 --
Jul 24 16:04:09 vuth-xps159500 systemd[1]: Starting enable the infrared emitter...
Jul 24 16:04:10 vuth-xps159500 systemd[1]: linux-enable-ir-emitter.service: Deactivated successfully.
Jul 24 16:04:10 vuth-xps159500 systemd[1]: Finished enable the infrared emitter.
-- Boot cd3e60e7d5e845f98091222a19332978 --
Jul 25 04:19:32 vuth-xps159500 systemd[1]: Starting enable the infrared emitter...
Jul 25 04:19:35 vuth-xps159500 linux-enable-ir-emitter[902]: Unable to open a file descriptor for /dev/video2
Jul 25 04:19:36 vuth-xps159500 systemd[1]: linux-enable-ir-emitter.service: Deactivated successfully.
Jul 25 04:19:36 vuth-xps159500 systemd[1]: Finished enable the infrared emitter.
-- Boot 49608150adcf4b19b6b83a6c717f22c0 --
Jul 25 04:20:35 vuth-xps159500 systemd[1]: Starting enable the infrared emitter...
Jul 25 04:20:36 vuth-xps159500 systemd[1]: linux-enable-ir-emitter.service: Deactivated successfully.
Jul 25 04:20:36 vuth-xps159500 systemd[1]: Finished enable the infrared emitter.
Jul 25 06:52:17 vuth-xps159500 systemd[1]: Starting enable the infrared emitter...
Jul 25 06:52:18 vuth-xps159500 systemd[1]: linux-enable-ir-emitter.service: Deactivated successfully.
Jul 25 06:52:18 vuth-xps159500 systemd[1]: Finished enable the infrared emitter.
-- Boot 4471d997b90c4986951d9d1ee2010da9 --
Jul 26 03:53:31 vuth-xps159500 systemd[1]: Starting enable the infrared emitter...
Jul 26 03:53:32 vuth-xps159500 systemd[1]: linux-enable-ir-emitter.service: Deactivated successfully.
Jul 26 03:53:32 vuth-xps159500 systemd[1]: Finished enable the infrared emitter.
-- Boot f442292469ca4a74aa712c045ec88fae --
Jul 26 03:55:45 vuth-xps159500 systemd[1]: Starting enable the infrared emitter...
Jul 26 03:55:46 vuth-xps159500 systemd[1]: linux-enable-ir-emitter.service: Deactivated successfully.
Jul 26 03:55:46 vuth-xps159500 systemd[1]: Finished enable the infrared emitter.
-- Boot ce42123c04074cf1b042b97e38669c3b --
Jul 26 04:40:14 vuth-xps159500 systemd[1]: Starting enable the infrared emitter...
Jul 26 04:40:15 vuth-xps159500 linux-enable-ir-emitter[880]: Unable to open a file descriptor for /dev/video2
Jul 26 04:40:16 vuth-xps159500 systemd[1]: linux-enable-ir-emitter.service: Deactivated successfully.
Jul 26 04:40:16 vuth-xps159500 systemd[1]: Finished enable the infrared emitter.
Jul 26 04:40:16 vuth-xps159500 systemd[1]: linux-enable-ir-emitter.service: Consumed 1.926s CPU time.
Jul 26 05:04:04 vuth-xps159500 systemd[1]: Starting enable the infrared emitter...
Jul 26 05:04:05 vuth-xps159500 linux-enable-ir-emitter[2611]: Unable to open a file descriptor for /dev/video2
Jul 26 05:04:06 vuth-xps159500 systemd[1]: linux-enable-ir-emitter.service: Deactivated successfully.
Jul 26 05:04:06 vuth-xps159500 systemd[1]: Finished enable the infrared emitter.
Jul 26 05:04:06 vuth-xps159500 systemd[1]: linux-enable-ir-emitter.service: Consumed 1.755s CPU time.
Jul 26 06:29:32 vuth-xps159500 systemd[1]: Starting enable the infrared emitter...
Jul 26 06:29:34 vuth-xps159500 linux-enable-ir-emitter[6686]: Unable to open a file descriptor for /dev/video2
Jul 26 06:29:36 vuth-xps159500 systemd[1]: linux-enable-ir-emitter.service: Deactivated successfully.
Jul 26 06:29:36 vuth-xps159500 systemd[1]: Finished enable the infrared emitter.
Jul 26 06:29:36 vuth-xps159500 systemd[1]: linux-enable-ir-emitter.service: Consumed 2.829s CPU time.
Jul 26 07:22:57 vuth-xps159500 systemd[1]: Starting enable the infrared emitter...

Humm, I've no idea :/ .  It's strange because I don't remember having this problem.
And the other maintainers of the AUR package not having opened any issue about this,this leads me to think that they did not have this problem.
I will leave this issue open, if the systemd service is disabled again; feel free to send a new comment !

thank you, I'll post another comment if it happen again

@EmixamPP thanks for your answer where can I find enable-ir-emitter.service?

Oh yes, sorry! I forgot that the software as it is today did not exist at the time of this issue.
For me the problem is not with linux-enable-ir-emitter because of the Unable to open a file descriptor for /dev/video2 error. This means that the program could not access the camera.
The solution found in issue #1 ensures that the camera will be accessible when the program will be executed by systemd. I have not included this fix in the installation as the method used is not very recommended.
So, here is the path to the file : /usr/lib/systemd/system/linux-enable-ir-emitter.service.
If it works for you, I think I'll include this fix automatically in my install.bash script.

Hi unfortunately it didn't work. I added a new file called webcam.rules in /etc/udev/rules.d and edit /usr/lib/systemd/system/linux-enable-ir-emitter.service. But it didn't work

Can you try this, I have set up a restart of the service in case of an error :
[Unit]
Description=enable the infrared emitter
After=multi-user.target suspend.target hibernate.target hybrid-sleep.target suspend-then-hibernate.target

StartLimitInterval=10
StartLimitBurst=10

[Service]
Type=simple
ExecStart=/usr/bin/linux-enable-ir-emitter run

Restart=on-failure
RestartSec=1

[Install]
WantedBy=multi-user.target suspend.target hibernate.target hybrid-sleep.target suspend-then-hibernate.target

It is still not working. Anyways, when I try to activate IR.
vuth@vuth-xps159500 Linux 5.13.12-1-MANJARO x86_64 21.1.1 Pahvo
~ >>> sudo ls
[ WARN:0] global /build/opencv/src/opencv-4.5.3/modules/videoio/src/cap_gstreamer.cpp (2056) handleMessage OpenCV | GStreamer warning: Embedded video playback halted; module source reported: Could not read from resource.
[ WARN:0] global /build/opencv/src/opencv-4.5.3/modules/videoio/src/cap_gstreamer.cpp (1034) open OpenCV | GStreamer warning: unable to start pipeline
[ WARN:0] global /build/opencv/src/opencv-4.5.3/modules/videoio/src/cap_gstreamer.cpp (597) isPipelinePlaying OpenCV | GStreamer warning: GStreamer: pipeline have not been created
All frames were too dark, please check dark_threshold in config
Average darkness: [78.8554], Threshold: 50.0
Face detection image too dark
[ WARN:0] global /build/opencv/src/opencv-4.5.3/modules/videoio/src/cap_gstreamer.cpp (2056) handleMessage OpenCV | GStreamer warning: Embedded video playback halted; module source reported: Could not read from resource.
[ WARN:0] global /build/opencv/src/opencv-4.5.3/modules/videoio/src/cap_gstreamer.cpp (1034) open OpenCV | GStreamer warning: unable to start pipeline
[ WARN:0] global /build/opencv/src/opencv-4.5.3/modules/videoio/src/cap_gstreamer.cpp (597) isPipelinePlaying OpenCV | GStreamer warning: GStreamer: pipeline have not been created
All frames were too dark, please check dark_threshold in config
Average darkness: [78.720924], Threshold: 50.0
Face detection image too dark
[sudo] password for vuth:

The opencv warnings are "normal", you can hide them by adding export OPENCV_LOG_LEVEL=ERROR to the file ~/.bashrc.
Howdy show these message :
All frames were too dark, please check dark_threshold in config
Average darkness: [78.720924], Threshold: 50.0
Face detection image too dark

cause the camera does not detect a strong enough infrared signal, therefore the image is dark.
Some infrared emitter are also not very efficient, so you need to change the value of dark_threshold = to 100 in sudo howdy config.
Any way, our file descriptor problem persist. Are you sure that another program using the camera does not start up? If it hogs the camera, linux-enable-ir-emitter cannot access it.
I have another question :everything works correctly if you only use (boot and reboot) Linux ?

I just tried boot reboot linux, it didn't work either. so it's not related to dual boot with windows. :(. Maybe i'll try with a clean install of Manjaro

I developed this program using Manjaro, it is strange. Your distro is up to date (pacman + kernel) ? The issue #1 is about Manjaro, may be you have failed the manipulation ? I will recap in details :
Change /usr/lib/systemd/system/linux-enable-ir-emitter.service with :
[Unit]
Description=enable the infrared emitter
Requires=dev-video2.device
After=dev-video2.device multi-user.target suspend.target hibernate.target hybrid-sleep.target suspend-then-hibernate.target

[Service]
Type=oneshot
ExecStart=/usr/bin/linux-enable-ir-emitter run

[Install]
WantedBy=multi-user.target suspend.target hibernate.target hybrid-sleep.target suspend-then-hibernate.target

Create a new file /etc/udev/rules.d/webcam.rules
KERNEL=="video2", SYMLINK="video2", TAG+="systemd"

Execute sudo linux-enable-ir-emitter boot enable

It works with your last configs. Thank you so much!

Perfect ! With pleasure :smile:

@cchvuth could I ask you a favour ? Could you try changing the contents of /usr/lib/systemd/system/linux-enable-ir-emitter.service  with what is below ? Then rerun sudo linux-enable-ir-emitter boot enable. I'm trying to find a cleaner way to solve your problem and include it in the next software release. Thanks for your help!
[Unit]
Description=enable the infrared emitter
After=multi-user.target suspend.target hibernate.target hybrid-sleep.target suspend-then-hibernate.target

[Service]
Type=oneshot
ExecStartPre=/sbin/modprobe uvcvideo
ExecStart=/usr/bin/linux-enable-ir-emitter run

[Install]
WantedBy=multi-user.target suspend.target hibernate.target hybrid-sleep.target suspend-then-hibernate.target

If you don't have the time (or if you don't want to do that), no problem, feel free to close issue.

I tested your new config 3 times and it works. Thank you

Thank you very much for your participation !
I forgot to mention that you have to delete the file  /etc/udev/rules.d/webcam.rules .
If it was not done, can you try just one last time ?

yup still works when deleted that rules


Add "load more" functionality to lib

Request needs two parameters:
*end_date -> should be the date of the oldest time entry currently available
*num_of_days -> 5

end date should be in format 2016-01-20T11:04:48+02:00. Maybe also urlencoded, but we have to double check.

This is currently waiting for implementation of the required v9 api by the Toggl backend team.

Implemented on load-more-button.


Partial rendering of insight dashboard

SQL Operations Studio Version: 0.26.4

Steps to Reproduce:
Right click server and select Manage. Sometimes the dashboard gets rendered partially. For example Backup Status and Database Size gets shown, but the Explorer widget and Tasks insight doesn't get displayed. When I try to manage a database from one of the servers where rendering doesn't work no insights get rendered there as well.
Btw, no changes to the config file have been applied for the insights.

I also saw an issue on this build last night were an Azure DB Server page was missing all its widgets.  When I cleaned out all my cached state under C:\Users\karlb\AppData\Roaming\sqlops the page started working correctly again and I didn't have a consistent repro after that.
@anthonydresser @AbbiePetcht @erickangMSFT we should track down what's happening here and determine if we need to hold tomorrow's release to get this fixed.

I've never seen this before, any ideas what can trigger it?

What I saw yesterday was that the Server page on an Azure DB only showed the Server properties, but didn't have the tasks or explorer widgets.  The Database page rendered fine.  As mentioned, when I cleared the cached AppData and recreated the connection the issue went away.
I was guessing it was some "bad" data in my cache somehow breaking things, and was hoping it was related to something in my dev env.  Now that we have another report of a similar issue I think there's a higher likely this will have broader impact.  I would guess it has something to do with upgrading between builds and difference in cached metadata formats, or something like that, but I don't have a current repro.
Is there any data we could collect in a repro state that would help root cause the issue?

@kburtram regarding the problem you were seeing, I think most likely it had nothing to do with your cache it just happened when you tried to repro it with your clean cache you didn't do the actual repro. The bug I fixed which I think is most likely causing the issue originally reported was when you render a dashboard it was setting the edition in the configuration settings for the widgets for the current edition of the dashboard (which is intended). However, this edition was being persisted accross different dashboards, which meant when you are viewing dashboards of different sql server editions, (on prem vs azure) you would see sporadic behavior regarding which widgets appear.
Another important note is that some of the default insights we have for the dashboard won't appear for azure as intended since those insights don't make sense for azure.

@anthonydresser sounds good, I didn't have a good repro for what I hit yesterday which is why I didn't log an issue.
@SebastianPfliegel 0.26.5 build at https://github.com/Microsoft/sqlopsstudio/releases/tag/0.26.5 has @anthonydresser's proposed fix.  Please let us know if you still hit the issue with the latest build.

Just used 0.26.6 and this particular bug has been fixed, but it might have opened another one. If I manage a  connection for a database where I don't have access to master db (as it is an isolated database; don't know the actual feature name) I get following error (though insights get displayed correctly):
[Error - 10:13:51] Request admin/getdatabaseinfo failed.
  Message: System.AggregateException: One or more errors occurred. (Failed to connect to server AOCLGP11.) ---> Microsoft.SqlServer.Management.Common.ConnectionFailureException: Failed to connect to server AOCLGP11. ---> System.Data.SqlClient.SqlException: Fehler bei der Anmeldung für den Benutzer "APOLLO-OPTIK\Pfliegel.Sebastian".
   at System.Data.SqlClient.SqlInternalConnectionTds..ctor(DbConnectionPoolIdentity identity, SqlConnectionString connectionOptions, Object providerInfo, Boolean redirectedUserInstance, SqlConnectionString userConnectionOptions, SessionData reconnectSessionData, Boolean applyTransientFaultHandling)
   at System.Data.SqlClient.SqlConnectionFactory.CreateConnection(DbConnectionOptions options, DbConnectionPoolKey poolKey, Object poolGroupProviderInfo, DbConnectionPool pool, DbConnection owningConnection, DbConnectionOptions userOptions)
   at System.Data.ProviderBase.DbConnectionFactory.CreateNonPooledConnection(DbConnection owningConnection, DbConnectionPoolGroup poolGroup, DbConnectionOptions userOptions)
   at System.Data.ProviderBase.DbConnectionFactory.TryGetConnection(DbConnection owningConnection, TaskCompletionSource`1 retry, DbConnectionOptions userOptions, DbConnectionInternal oldConnection, DbConnectionInternal& connection)
   at System.Data.ProviderBase.DbConnectionInternal.TryOpenConnectionInternal(DbConnection outerConnection, DbConnectionFactory connectionFactory, TaskCompletionSource`1 retry, DbConnectionOptions userOptions)
   at System.Data.SqlClient.SqlConnection.TryOpen(TaskCompletionSource`1 retry)
   at System.Data.SqlClient.SqlConnection.Open()
   at Microsoft.SqlServer.Management.Common.ConnectionManager.InternalConnect()
   at Microsoft.SqlServer.Management.Common.ConnectionManager.Connect()
   --- End of inner exception stack trace ---
   at Microsoft.SqlServer.Management.Common.ConnectionManager.Connect()
   at Microsoft.SqlServer.Management.Common.ConnectionManager.PoolConnect()
   at Microsoft.SqlServer.Management.Common.ConnectionManager.GetServerInformation()
   at Microsoft.SqlServer.Management.Smo.Server.IsAzureDbScopedConnection(ServerConnection sc)
   at Microsoft.SqlServer.Management.Smo.Server.GetExecutionManager()
   at Microsoft.SqlServer.Management.Smo.SqlSmoObject.get_ServerVersion()
   at Microsoft.SqlServer.Management.Smo.Information.get_Version()
   at Microsoft.SqlTools.ServiceLayer.Admin.DatabaseTaskHelper.Initialize(CDataContainer context) in D:\a\1\s\src\Microsoft.SqlTools.ServiceLayer\Admin\Database\DatabaseTaskHelper.cs:line 53
   at Microsoft.SqlTools.ServiceLayer.Admin.AdminService.CreateDatabaseTaskHelper(ConnectionInfo connInfo, Boolean databaseExists) in D:\a\1\s\src\Microsoft.SqlTools.ServiceLayer\Admin\AdminService.cs:line 234
   at Microsoft.SqlTools.ServiceLayer.Admin.AdminService.GetDatabaseInfo(ConnectionInfo connInfo) in D:\a\1\s\src\Microsoft.SqlTools.ServiceLayer\Admin\AdminService.cs:line 186
   at Microsoft.SqlTools.ServiceLayer.Admin.AdminService.<>c__DisplayClass12_0.<<HandleGetDatabaseInfoRequest>b__0>d.MoveNext() in D:\a\1\s\src\Microsoft.SqlTools.ServiceLayer\Admin\AdminService.cs:line 158
--- 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.SqlTools.ServiceLayer.Admin.AdminService.<>c__DisplayClass12_1.<<HandleGetDatabaseInfoRequest>b__1>d.MoveNext() in D:\a\1\s\src\Microsoft.SqlTools.ServiceLayer\Admin\AdminService.cs:line 167
   --- End of inner exception stack trace ---
---> (Inner Exception #0) Microsoft.SqlServer.Management.Common.ConnectionFailureException: Failed to connect to server AOCLGP11. ---> System.Data.SqlClient.SqlException: Fehler bei der Anmeldung für den Benutzer "APOLLO-OPTIK\Pfliegel.Sebastian".
   at System.Data.SqlClient.SqlInternalConnectionTds..ctor(DbConnectionPoolIdentity identity, SqlConnectionString connectionOptions, Object providerInfo, Boolean redirectedUserInstance, SqlConnectionString userConnectionOptions, SessionData reconnectSessionData, Boolean applyTransientFaultHandling)
   at System.Data.SqlClient.SqlConnectionFactory.CreateConnection(DbConnectionOptions options, DbConnectionPoolKey poolKey, Object poolGroupProviderInfo, DbConnectionPool pool, DbConnection owningConnection, DbConnectionOptions userOptions)
   at System.Data.ProviderBase.DbConnectionFactory.CreateNonPooledConnection(DbConnection owningConnection, DbConnectionPoolGroup poolGroup, DbConnectionOptions userOptions)
   at System.Data.ProviderBase.DbConnectionFactory.TryGetConnection(DbConnection owningConnection, TaskCompletionSource`1 retry, DbConnectionOptions userOptions, DbConnectionInternal oldConnection, DbConnectionInternal& connection)
   at System.Data.ProviderBase.DbConnectionInternal.TryOpenConnectionInternal(DbConnection outerConnection, DbConnectionFactory connectionFactory, TaskCompletionSource`1 retry, DbConnectionOptions userOptions)
   at System.Data.SqlClient.SqlConnection.TryOpen(TaskCompletionSource`1 retry)
   at System.Data.SqlClient.SqlConnection.Open()
   at Microsoft.SqlServer.Management.Common.ConnectionManager.InternalConnect()
   at Microsoft.SqlServer.Management.Common.ConnectionManager.Connect()
   --- End of inner exception stack trace ---
   at Microsoft.SqlServer.Management.Common.ConnectionManager.Connect()
   at Microsoft.SqlServer.Management.Common.ConnectionManager.PoolConnect()
   at Microsoft.SqlServer.Management.Common.ConnectionManager.GetServerInformation()
   at Microsoft.SqlServer.Management.Smo.Server.IsAzureDbScopedConnection(ServerConnection sc)
   at Microsoft.SqlServer.Management.Smo.Server.GetExecutionManager()
   at Microsoft.SqlServer.Management.Smo.SqlSmoObject.get_ServerVersion()
   at Microsoft.SqlServer.Management.Smo.Information.get_Version()
   at Microsoft.SqlTools.ServiceLayer.Admin.DatabaseTaskHelper.Initialize(CDataContainer context) in D:\a\1\s\src\Microsoft.SqlTools.ServiceLayer\Admin\Database\DatabaseTaskHelper.cs:line 53
   at Microsoft.SqlTools.ServiceLayer.Admin.AdminService.CreateDatabaseTaskHelper(ConnectionInfo connInfo, Boolean databaseExists) in D:\a\1\s\src\Microsoft.SqlTools.ServiceLayer\Admin\AdminService.cs:line 234
   at Microsoft.SqlTools.ServiceLayer.Admin.AdminService.GetDatabaseInfo(ConnectionInfo connInfo) in D:\a\1\s\src\Microsoft.SqlTools.ServiceLayer\Admin\AdminService.cs:line 186
   at Microsoft.SqlTools.ServiceLayer.Admin.AdminService.<>c__DisplayClass12_0.<<HandleGetDatabaseInfoRequest>b__0>d.MoveNext() in D:\a\1\s\src\Microsoft.SqlTools.ServiceLayer\Admin\AdminService.cs:line 158
--- 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.SqlTools.ServiceLayer.Admin.AdminService.<>c__DisplayClass12_1.<<HandleGetDatabaseInfoRequest>b__1>d.MoveNext() in D:\a\1\s\src\Microsoft.SqlTools.ServiceLayer\Admin\AdminService.cs:line 167<---

  Code: 0
[Error - 10:13:51] Request admin/getdatabaseinfo failed.
  Message: System.AggregateException: One or more errors occurred. (Failed to connect to server AOCLGP11.) ---> Microsoft.SqlServer.Management.Common.ConnectionFailureException: Failed to connect to server AOCLGP11. ---> System.Data.SqlClient.SqlException: Fehler bei der Anmeldung für den Benutzer "APOLLO-OPTIK\Pfliegel.Sebastian".
   at System.Data.SqlClient.SqlInternalConnectionTds..ctor(DbConnectionPoolIdentity identity, SqlConnectionString connectionOptions, Object providerInfo, Boolean redirectedUserInstance, SqlConnectionString userConnectionOptions, SessionData reconnectSessionData, Boolean applyTransientFaultHandling)
   at System.Data.SqlClient.SqlConnectionFactory.CreateConnection(DbConnectionOptions options, DbConnectionPoolKey poolKey, Object poolGroupProviderInfo, DbConnectionPool pool, DbConnection owningConnection, DbConnectionOptions userOptions)
   at System.Data.ProviderBase.DbConnectionFactory.CreateNonPooledConnection(DbConnection owningConnection, DbConnectionPoolGroup poolGroup, DbConnectionOptions userOptions)
   at System.Data.ProviderBase.DbConnectionFactory.TryGetConnection(DbConnection owningConnection, TaskCompletionSource`1 retry, DbConnectionOptions userOptions, DbConnectionInternal oldConnection, DbConnectionInternal& connection)
   at System.Data.ProviderBase.DbConnectionInternal.TryOpenConnectionInternal(DbConnection outerConnection, DbConnectionFactory connectionFactory, TaskCompletionSource`1 retry, DbConnectionOptions userOptions)
   at System.Data.SqlClient.SqlConnection.TryOpen(TaskCompletionSource`1 retry)
   at System.Data.SqlClient.SqlConnection.Open()
   at Microsoft.SqlServer.Management.Common.ConnectionManager.InternalConnect()
   at Microsoft.SqlServer.Management.Common.ConnectionManager.Connect()
   --- End of inner exception stack trace ---
   at Microsoft.SqlServer.Management.Common.ConnectionManager.Connect()
   at Microsoft.SqlServer.Management.Common.ConnectionManager.PoolConnect()
   at Microsoft.SqlServer.Management.Common.ConnectionManager.GetServerInformation()
   at Microsoft.SqlServer.Management.Smo.Server.IsAzureDbScopedConnection(ServerConnection sc)
   at Microsoft.SqlServer.Management.Smo.Server.GetExecutionManager()
   at Microsoft.SqlServer.Management.Smo.SqlSmoObject.get_ServerVersion()
   at Microsoft.SqlServer.Management.Smo.Information.get_Version()
   at Microsoft.SqlTools.ServiceLayer.Admin.DatabaseTaskHelper.Initialize(CDataContainer context) in D:\a\1\s\src\Microsoft.SqlTools.ServiceLayer\Admin\Database\DatabaseTaskHelper.cs:line 53
   at Microsoft.SqlTools.ServiceLayer.Admin.AdminService.CreateDatabaseTaskHelper(ConnectionInfo connInfo, Boolean databaseExists) in D:\a\1\s\src\Microsoft.SqlTools.ServiceLayer\Admin\AdminService.cs:line 234
   at Microsoft.SqlTools.ServiceLayer.Admin.AdminService.GetDatabaseInfo(ConnectionInfo connInfo) in D:\a\1\s\src\Microsoft.SqlTools.ServiceLayer\Admin\AdminService.cs:line 186
   at Microsoft.SqlTools.ServiceLayer.Admin.AdminService.<>c__DisplayClass12_0.<<HandleGetDatabaseInfoRequest>b__0>d.MoveNext() in D:\a\1\s\src\Microsoft.SqlTools.ServiceLayer\Admin\AdminService.cs:line 158
--- 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.SqlTools.ServiceLayer.Admin.AdminService.<>c__DisplayClass12_1.<<HandleGetDatabaseInfoRequest>b__1>d.MoveNext() in D:\a\1\s\src\Microsoft.SqlTools.ServiceLayer\Admin\AdminService.cs:line 167
   --- End of inner exception stack trace ---
---> (Inner Exception #0) Microsoft.SqlServer.Management.Common.ConnectionFailureException: Failed to connect to server AOCLGP11. ---> System.Data.SqlClient.SqlException: Fehler bei der Anmeldung für den Benutzer "APOLLO-OPTIK\Pfliegel.Sebastian".
   at System.Data.SqlClient.SqlInternalConnectionTds..ctor(DbConnectionPoolIdentity identity, SqlConnectionString connectionOptions, Object providerInfo, Boolean redirectedUserInstance, SqlConnectionString userConnectionOptions, SessionData reconnectSessionData, Boolean applyTransientFaultHandling)
   at System.Data.SqlClient.SqlConnectionFactory.CreateConnection(DbConnectionOptions options, DbConnectionPoolKey poolKey, Object poolGroupProviderInfo, DbConnectionPool pool, DbConnection owningConnection, DbConnectionOptions userOptions)
   at System.Data.ProviderBase.DbConnectionFactory.CreateNonPooledConnection(DbConnection owningConnection, DbConnectionPoolGroup poolGroup, DbConnectionOptions userOptions)
   at System.Data.ProviderBase.DbConnectionFactory.TryGetConnection(DbConnection owningConnection, TaskCompletionSource`1 retry, DbConnectionOptions userOptions, DbConnectionInternal oldConnection, DbConnectionInternal& connection)
   at System.Data.ProviderBase.DbConnectionInternal.TryOpenConnectionInternal(DbConnection outerConnection, DbConnectionFactory connectionFactory, TaskCompletionSource`1 retry, DbConnectionOptions userOptions)
   at System.Data.SqlClient.SqlConnection.TryOpen(TaskCompletionSource`1 retry)
   at System.Data.SqlClient.SqlConnection.Open()
   at Microsoft.SqlServer.Management.Common.ConnectionManager.InternalConnect()
   at Microsoft.SqlServer.Management.Common.ConnectionManager.Connect()
   --- End of inner exception stack trace ---
   at Microsoft.SqlServer.Management.Common.ConnectionManager.Connect()
   at Microsoft.SqlServer.Management.Common.ConnectionManager.PoolConnect()
   at Microsoft.SqlServer.Management.Common.ConnectionManager.GetServerInformation()
   at Microsoft.SqlServer.Management.Smo.Server.IsAzureDbScopedConnection(ServerConnection sc)
   at Microsoft.SqlServer.Management.Smo.Server.GetExecutionManager()
   at Microsoft.SqlServer.Management.Smo.SqlSmoObject.get_ServerVersion()
   at Microsoft.SqlServer.Management.Smo.Information.get_Version()
   at Microsoft.SqlTools.ServiceLayer.Admin.DatabaseTaskHelper.Initialize(CDataContainer context) in D:\a\1\s\src\Microsoft.SqlTools.ServiceLayer\Admin\Database\DatabaseTaskHelper.cs:line 53
   at Microsoft.SqlTools.ServiceLayer.Admin.AdminService.CreateDatabaseTaskHelper(ConnectionInfo connInfo, Boolean databaseExists) in D:\a\1\s\src\Microsoft.SqlTools.ServiceLayer\Admin\AdminService.cs:line 234
   at Microsoft.SqlTools.ServiceLayer.Admin.AdminService.GetDatabaseInfo(ConnectionInfo connInfo) in D:\a\1\s\src\Microsoft.SqlTools.ServiceLayer\Admin\AdminService.cs:line 186
   at Microsoft.SqlTools.ServiceLayer.Admin.AdminService.<>c__DisplayClass12_0.<<HandleGetDatabaseInfoRequest>b__0>d.MoveNext() in D:\a\1\s\src\Microsoft.SqlTools.ServiceLayer\Admin\AdminService.cs:line 158
--- 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.SqlTools.ServiceLayer.Admin.AdminService.<>c__DisplayClass12_1.<<HandleGetDatabaseInfoRequest>b__1>d.MoveNext() in D:\a\1\s\src\Microsoft.SqlTools.ServiceLayer\Admin\AdminService.cs:line 167<---

  Code: 0

Is it related or should I open a new issue?

@SebastianPfliegel the exception looks unrelated as far as I can tell.  Could you please open a separate issue so we can get repro steps?  I think you're referring to connecting to a Contained Database.  I'll see if I can get the same error.

@MattIrv could you please take a look at the above exception and see if it's related to https://github.com/Microsoft/sqltoolsservice/pull/583?
I think a workaround may be to add the database name for the contained DB explicitly in the connection dialog (instead of the using default) which should avoid running queries against master, right?

@SebastianPfliegel and @kburtram: I've opened issue #715 to track this and can confirm it repros for contained databases. Unfortunately the workaround suggested above doesn't work because you need to enter the database name in order to connect to a contained database in the first place.

Contained database is the word I was looking for! And @MattIrv is right, I already had the database name selected (as it is required).
This one can be closed, though :)


Is mask decoder trained in the second stage?

Hello @zamling, it's mentioned that the mask decoder is trained in the paper. However, I noticed that in the code is_train_mask_decode is set to False by default, this means the mask decoder is not trained. I'm wondering which one should I use for my own dataset. Thanks for your attention.

Hi @lx709
The name, 'is_train_mask_decode', may be ambiguous. It means whether the mask decoder have been trained.
As the model initialized by LLM ckpt, mask decoder is not initailized by from_pretrained. The mask decoder will be initialized here
https://github.com/zamling/PSALM/blob/025fa75403fba97a50a857170396cbd0aec077e7/psalm/train/train.py#L378-L380
And 'is_train_mask_decode' will be set True after initializing, so that when inference or finetune based on another ft ckpt, the mask decoder can be loaded by from_pretrained. As shown below:
https://github.com/zamling/PSALM/blob/025fa75403fba97a50a857170396cbd0aec077e7/psalm/model/language_model/llava_phi.py#L170-L173
https://github.com/zamling/PSALM/blob/025fa75403fba97a50a857170396cbd0aec077e7/psalm/model/language_model/llava_phi.py#L165-L167

@zamling , thanks for your clarification.


Add log to connection check

When the connection check fails in the startup processing we don't log the problem.  This PR adds an info log.

LGTM - is there an easy way we can ratelimit this though? Would be nice if it printed out once every 30s or so instead of once every second...

ping @robbrockbank what's up with this guy?

i couldn't see an easy way to rate limit this - and didn't have the cycles available to investigate further.

didn't spot this PR, but I was thinking of adding this log as well whenever I touch this code next (maybe for BGP stability)

OK - I won't block this on rate limiting for now.
Once tests pass we can merge.


Remove with-cookie-auth

This example is something we don't and can't recommend to the community as it doesn't have any kind of security measures implemented, we currently have better examples for this so the readme has been updated to recommend those instead.

Stats from current PR

Default Server Mode

General

zeit/next.js canary
lfades/next.js examples/remove-with-cookie-auth
Change

buildDuration
10.9s
11.2s
⚠ +311ms

nodeModulesSize
55.5 MB
55.5 MB
✓

Client Bundles (main, webpack, commons)

zeit/next.js canary
lfades/next.js examples/remove-with-cookie-auth
Change

main-HASH.js gzip
6.25 kB
6.25 kB
✓

webpack-HASH.js gzip
746 B
746 B
✓

de003c3a9d30..a4ac.js gzip
10.2 kB
10.2 kB
✓

framework.HASH.js gzip
39.1 kB
39.1 kB
✓

Overall change
56.3 kB
56.3 kB
✓

Client Bundles (main, webpack, commons) Modern

zeit/next.js canary
lfades/next.js examples/remove-with-cookie-auth
Change

main-HASH.module.js gzip
4.78 kB
4.78 kB
✓

webpack-HASH..dule.js gzip
746 B
746 B
✓

de003c3a9d30..dule.js gzip
6.78 kB
6.78 kB
✓

framework.HA..dule.js gzip
39.1 kB
39.1 kB
✓

Overall change
51.5 kB
51.5 kB
✓

Legacy Client Bundles (polyfills)

zeit/next.js canary
lfades/next.js examples/remove-with-cookie-auth
Change

polyfills-HASH.js gzip
26.3 kB
26.3 kB
✓

Overall change
26.3 kB
26.3 kB
✓

Client Pages

zeit/next.js canary
lfades/next.js examples/remove-with-cookie-auth
Change

_app.js gzip
1.24 kB
1.24 kB
✓

_error.js gzip
3.15 kB
3.15 kB
✓

hooks.js gzip
664 B
664 B
✓

index.js gzip
222 B
222 B
✓

link.js gzip
2.03 kB
2.03 kB
✓

routerDirect.js gzip
279 B
279 B
✓

withRouter.js gzip
278 B
278 B
✓

Overall change
7.86 kB
7.86 kB
✓

Client Pages Modern

zeit/next.js canary
lfades/next.js examples/remove-with-cookie-auth
Change

_app.module.js gzip
594 B
594 B
✓

_error.module.js gzip
2.08 kB
2.08 kB
✓

hooks.module.js gzip
370 B
370 B
✓

index.module.js gzip
212 B
212 B
✓

link.module.js gzip
1.48 kB
1.48 kB
✓

routerDirect..dule.js gzip
271 B
271 B
✓

withRouter.m..dule.js gzip
270 B
270 B
✓

Overall change
5.28 kB
5.28 kB
✓

Client Build Manifests

zeit/next.js canary
lfades/next.js examples/remove-with-cookie-auth
Change

_buildManifest.js gzip
61 B
61 B
✓

_buildManife..dule.js gzip
61 B
61 B
✓

Overall change
122 B
122 B
✓

Rendered Page Sizes

zeit/next.js canary
lfades/next.js examples/remove-with-cookie-auth
Change

index.html gzip
927 B
927 B
✓

link.html gzip
935 B
935 B
✓

withRouter.html gzip
922 B
922 B
✓

Overall change
2.78 kB
2.78 kB
✓

Serverless Mode

General

zeit/next.js canary
lfades/next.js examples/remove-with-cookie-auth
Change

buildDuration
11.8s
12s
⚠ +159ms

nodeModulesSize
55.5 MB
55.5 MB
✓

Client Bundles (main, webpack, commons)

zeit/next.js canary
lfades/next.js examples/remove-with-cookie-auth
Change

main-HASH.js gzip
6.25 kB
6.25 kB
✓

webpack-HASH.js gzip
746 B
746 B
✓

de003c3a9d30..a4ac.js gzip
10.2 kB
10.2 kB
✓

framework.HASH.js gzip
39.1 kB
39.1 kB
✓

Overall change
56.3 kB
56.3 kB
✓

Client Bundles (main, webpack, commons) Modern

zeit/next.js canary
lfades/next.js examples/remove-with-cookie-auth
Change

main-HASH.module.js gzip
4.78 kB
4.78 kB
✓

webpack-HASH..dule.js gzip
746 B
746 B
✓

de003c3a9d30..dule.js gzip
6.78 kB
6.78 kB
✓

framework.HA..dule.js gzip
39.1 kB
39.1 kB
✓

Overall change
51.5 kB
51.5 kB
✓

Legacy Client Bundles (polyfills)

zeit/next.js canary
lfades/next.js examples/remove-with-cookie-auth
Change

polyfills-HASH.js gzip
26.3 kB
26.3 kB
✓

Overall change
26.3 kB
26.3 kB
✓

Client Pages

zeit/next.js canary
lfades/next.js examples/remove-with-cookie-auth
Change

_app.js gzip
1.24 kB
1.24 kB
✓

_error.js gzip
3.15 kB
3.15 kB
✓

hooks.js gzip
664 B
664 B
✓

index.js gzip
222 B
222 B
✓

link.js gzip
2.03 kB
2.03 kB
✓

routerDirect.js gzip
279 B
279 B
✓

withRouter.js gzip
278 B
278 B
✓

Overall change
7.86 kB
7.86 kB
✓

Client Pages Modern

zeit/next.js canary
lfades/next.js examples/remove-with-cookie-auth
Change

_app.module.js gzip
594 B
594 B
✓

_error.module.js gzip
2.08 kB
2.08 kB
✓

hooks.module.js gzip
370 B
370 B
✓

index.module.js gzip
212 B
212 B
✓

link.module.js gzip
1.48 kB
1.48 kB
✓

routerDirect..dule.js gzip
271 B
271 B
✓

withRouter.m..dule.js gzip
270 B
270 B
✓

Overall change
5.28 kB
5.28 kB
✓

Client Build Manifests

zeit/next.js canary
lfades/next.js examples/remove-with-cookie-auth
Change

_buildManifest.js gzip
61 B
61 B
✓

_buildManife..dule.js gzip
61 B
61 B
✓

Overall change
122 B
122 B
✓

Serverless bundles

zeit/next.js canary
lfades/next.js examples/remove-with-cookie-auth
Change

_error.js
557 kB
557 kB
✓

404.html
4.18 kB
4.18 kB
✓

hooks.html
3.81 kB
3.81 kB
✓

index.js
557 kB
557 kB
✓

link.js
593 kB
593 kB
✓

routerDirect.js
586 kB
586 kB
✓

withRouter.js
586 kB
586 kB
✓

Overall change
2.89 MB
2.89 MB
✓

Checked out the passport example, looks much better than the with-cookie-auth we had before.

Also https://github.com/zeit/next.js/tree/canary/examples/with-iron-session which brings Rails-like sessions to Next.js:

HTTP only
secure
samesite: lax
signed and encrypted
stateless (session data stored on the client)

:p

@vvo I forgot about that one!, free welcome to create a PR adding it to the list.


Address already in use

Failed to listen on <IP_ADDRESS>:33371 (reason: Address already in use) appear second Test unit class which in the same life cycle.
it seems that it have no release socket In teardown() really or Http-mock is not a singleInstance.

Sorry,I found the invoke of Symfony\Component\Process->stop() in HttpMockTrait::tearDownHttpMockAfterClass()
https://github.com/Codeception/Codeception/issues/3198
Oh,it's a “feature” of codeception framwork,not http-mock.

@JiankangMai alternatively you can indeed do set up and tear down in the per test steps, not in the per class steps. That’s less efficient as it needs to restart the server more often but should work.


Language list

Go to :
Edit upload
Language
Open the drop down list
Ελληνικά (Greek) is missing.
Please add it to the list.
Version 0.0.69 (latest on F-Droid)

This repo is for the native app, not the web app in the wild.
To add languages, typically we need to have translations for the site. You can help us at: https://explore.transifex.com/lbry/lbry-desktop/

Thanks for the answer, I'm talking about the android app.
I see "Slovencina" in the list although it's only translated at 4% according to your link.
Greek is translated 13% but unlisted.
Why is that?


UX Improvement - Don't stop coloring if I mouse off of the triangles and come back.

If I'm coloring and accidentally mouse off of the canvas area and then come back it won't keep coloring (even if I haven't let go of the mouse button)
Like this...

Oh I understand the other PR now. I made this explicitly, but you make me realize it wasn't wise at all. Gotcha.
Thanks for the nice issue.


Text box filter not working on Protein Family Sorter

I am comparing three groups and one private genome, selected PLFams, and there are 16650 PLFams.  I put iron in the text box and click filter...the same 16650.

Keyword search seems to be working on alpha. Please try again and reopen the ticket if you still see the problem.


fix cumsum docstring

added missing dedent statement

Codecov Report

Merging #71 into master will increase coverage by 0.04%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #71      +/-   ##
==========================================
+ Coverage   87.21%   87.26%   +0.04%
==========================================
  Files           3        3
  Lines         266      267       +1
  Branches       54       54
==========================================
+ Hits          232      233       +1
  Misses         19       19
  Partials       15       15

Impacted Files
Coverage Δ

xgcm/grid.py
90.04% <100%> (+0.04%)
:arrow_up:

Continue to review full report at Codecov.

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


CCG Flow Insecure (any client password accepted)

This stems back to #104, which was ultimately an explanation for how the interactive delegated flow was indeed secure.
So I understand the inherent security with using "any" email client password when using a delegated flow, since that requires immediate user interaction to complete the auth process, and therefore the resulting tokens are encrypted with the original email client password sent, so subsequent requests would need to use the same password to avoid a user interactive re-authentication. So that inherently keeps the delegated flow secure, HOWEVER if using client credential grant flow, that is simply not the case. With CCG, if I send a different password, sure the error retrying login due to exception while decrypting OAuth 2.0 credentials will be thrown, but the program will attempt to re-authenticate and generate new access tokens automatically, and since CCG doesn't require any user interaction at all and relies on the client/app id, tenant id, and client secret already stored in config, the new tokens are generated and stored w/o complaint.
This effectively allows a CCG flow to be used as an open relay for any system that can reach it with any specified password, as long as they specify the correct set of usernames.
One solution might be to allow specifying some sort of "app password" that we can add in config somehow to allow only certain email client passwords to be used. I'd imagine that adds some complexity and security risks since the same password would be used to encrypt the tokens in the same file later on, so more thought and consideration would need to be given for how to resolve that properly.
Another solution might be preventing CCG flows from automatically re-generating encryption keys based on current password when decryption fails, and maybe force manual user intervention to modify the config file in place to wipe out the bad keys (i.e. a way to get some user interaction when things go wrong).
Just some food for thought, but I do think this is a pretty serious issue that makes using the CCG flow insecure, especially if you're running other third party software on your server which could potentially exploit this (or perhaps they themselves were exploited), or god forbid you open this up on the public internet 😨...
Let me know if I'm just blatantly missing an easy way to make this secure. Since Microsoft's support for SMTP OAUTH2 w/ CCG flow was finally released earlier this year (ref this & that), setting up some sort of proxy like this would be pretty critical for finally retrofitting older SMTP clients that have no other ways of supporting this.
Thanks for the work you have put into this library! For the most part (minus some docker networking issues I'm investigating and this potential security flaw issue) it worked pretty well right out of the gate.

Great – glad you were able to resolve this. And yes, unfortunately, CCG is simply insecure by default without any preparation step (such as the encryption script) – it's an admin-type method that intentionally gives access to accounts with little interaction required. I don't think the proxy could really do much more here to secure this flow. But I do think anyone using the CCG flow is likely to be a relatively advanced user, and the proxy's readme is also pretty clear about the potential insecurity. I'll look at strengthening that warning, though.
Re: catch-all accounts and secret encryption – this is already on my list to fix. The intent is as outlined in the readme: this usage should be supported, but only when using the same password (which is potentially likely given the typical use of CCG).
It's also worth explaining why the proxy intentionally doesn't remove the unencrypted secret from any top-level inherited configuration. The reason is that you might be using this intentionally at first setup time – for example: you have a large number of accounts to configure via CCG, and you'd like each one to have a different password. After logging in once to each account, they would have their own encrypted versions of the secret, and you you can manually remove the unencrypted secret from the top-level section, which will prevent any new accounts from being created.
If, on the other hand, you do want to use the same password for all accounts, just replace the unencrypted secret at the top level with the encrypted one (and, once I've made the fix mentioned above, also the token_salt and token_iterations values) from an individual account, and this will be used for any new accounts. Note: it'll also be duplicated in the account itself, so if at any point you want to prevent the creation of new accounts you can remove it from the top level section (but keep token_salt and token_iterations at the top level).

All makes sense to me. Thanks for the explanation!
To make CCG flow secure by default, the only quick fix I can think of would be to maybe change encrypt_client_secret_on_first_use to be true by default and update readme/docs accordingly. Of course, that could be a breaking change for some when updating to the latest version. Having a changeme file which lists breaking changes per version would be helpful here.
Another solution (suggested in OP), but probably a bit more code changes required would be to only allow token generation once for a given account w/ CCG flow. How exactly, while allowing expired tokens to be re-generated, not 100% sure, but maybe by enforcing that the same token_salt must be used (once initially generated). I noticed that if I disable encrypt_client_secret_on_first_use (i.e. restore to default settings), then if I submit two different requests with different passwords, the two requests will have different token_salts. If I instead, use the same password, but force the access_token_expiry to be outdated, the access_token is regenerated, but the token_salt remains the same. So that is why I think we could probably try to enforce the same token_salt usage. I also don't think this would cause too many breaking changes as compared to the above. Anyway, just food for thought...
Either way, I think more direct warnings as you suggested would go a long way too, although not sure what the correct verbiage sweet spot would be. Perhaps at minimum a blurb that starts with ⚠WARNING⚠ in all caps 🤷 (since I completely missed it).

It's worth remembering that none of this is really an issue unless you're using the proxy in a shared environment, or on a publicly accessible server. If that's the case, then personally I think I'd advise against the more powerful methods entirely, and stick to normal per-user authentication.
The token_salt is not relevant – it's created afresh whenever a new password is used, and is used from that point forward as part of the encryption/decryption process. I also wouldn't want to change the default for encrypt_client_secret_on_first_use because that would affect all usage of the proxy, and would be very disruptive outside of the CCG case.
What actually may be worth considering is the default value for delete_account_token_on_password_error in the CCG (and ROPG) situation. The normal default is True because of issues with clients storing IMAP and SMTP passwords separately, and people being unaware of this, then finding themselves seemingly (but not actually) locked out of their account. I don't want to change that behaviour because it led to lots of unnecessary issues that I don't want to restart. However, if this value was changed so that it was ignored when using the CCG/ROPG method, it would mean that once the account had been authenticated, only the correct password would work; other attempts would be rejected (rather than resetting the account's credentials and allowing any password to be used for the next login).
This wouldn't help if you're using catch-all accounts in a situation where the client secret that's being used has access to many accounts, some of which are not set up with the proxy – new accounts are always going to be allowed in catch-all mode (that's the whole point!). So again it's a trade-off. But probably this change in conjunction with a bit more highlighting in the readme might improve things in general.