Use a custom selection background and foreground color

Closes #105

cc: @rkdarst and @bast
Since the text color is implemented in https://github.com/AaltoSciComp/sphinx_rtd_theme_ext_color_contrast/, now I wonder if this is best added there. On the other hand if sphinx_rtd_theme_ext_color_contrast is a temporary package then this change is best added here.

This is good!  I assume you know more about colors and design than me, if you think this is good then it's good.
There is one thing though: sphinx-lesson could use different themes and this might interfere with others.  Most of my work with improving sphinx-rtd-theme's design has gone into here: https://github.com/AaltoSciComp/sphinx_rtd_theme_ext_color_contrast/tree/master/sphinx_rtd_theme_ext_color_contrast .  I made it as a way to do accessibility improvements that were too slow in sphinx_rtd_theme.  You could look there and simultaneously submit it there, if there isn't already something.  But there are many issues and handling is slow, so I wouldn't count on much.
Here are some thoughts you can probably figure out better:

I like the highlight of the yellow background: it's seems good for highlighting things when teaching.
When things are on a variety of different screens (screenshare, projected on a room screen, etc), do you know if the color choice is still visible?  (it doesn't have to be perfect, we can improve later)
Do you know if it works when it's in dark theme?  (if dark theme is even supported - I thought it was but don't seem to know enough about firefox to test that...)

Thanks for looking into this stuff.  It's very useful stuff that the rest of us aren't likely to ever get to...

This is good! I assume you know more about colors and design than me, if you think this is good then it's good.
There is one thing though: sphinx-lesson could use different themes and this might interfere with others. Most of my work with improving sphinx-rtd-theme's design has gone into here: https://github.com/AaltoSciComp/sphinx_rtd_theme_ext_color_contrast/tree/master/sphinx_rtd_theme_ext_color_contrast . I made it as a way to do accessibility improvements that were too slow in sphinx_rtd_theme. You could look there and simultaneously submit it there, if there isn't already something. But there are many issues and handling is slow, so I wouldn't count on much.

It could be added to upstream, but let's test it out here. Then it would be easy to change once we deploy it and if people have issues with it. Moreover this is very useful as a teaching aid, but less useful for a regular user reading the docs.

Here are some thoughts you can probably figure out better:
* I like the highlight of the yellow background: it's seems good for highlighting things when teaching.

Yes! That's the idea!

* When things are on a variety of different screens (screenshare, projected on a room screen, etc), do you know if the color choice is still visible?  (it doesn't have to be perfect, we can improve later)

Honestly, I haven't checked, but there are WCAG checkers online and here is one such for the pair of colours:
https://webaim.org/resources/contrastchecker/?fcolor=242424&bcolor=FCE762
We could make the foreground colour on shade darker just in case #121212 or go all in with #000000. To my eye, both are fine.

* Do you know if it works when it's in dark theme?  (if dark theme is even supported - I thought it was but don't seem to know enough about firefox to test that...)

Then we would need some other shade of dark yellow and white. We can try to address that using
https://developer.mozilla.org/en-US/docs/Web/CSS/@media/prefers-color-scheme
There are 2 kinds of dark theme, btw

that is natively implemented using CSS prefers-color-scheme queries
dynamically created using Dark Reader add-on.

Dark Reader can also sense the prefers-color-scheme query and use it, but I need to try it out.
sphinx-rtd-theme does not have a native CSS dark theme, but there are themes such as:

https://pradyunsg.me/furo/
https://pydata-sphinx-theme.readthedocs.io/

Which support this

Thanks for looking into this stuff. It's very useful stuff that the rest of us aren't likely to ever get to...

I confirm that this PR also works with sphinx v8 and sphinx-rtd-theme v3 - ofc it should, it is pure CSS after all.

Thanks!  I will assume you know this better than me and can fix stuff if it breaks.
But I really think that sphinx_rtd_theme_ext_color_contrast is a better place to put this.  It's hardly an "upstream", I also maintain it and develop it as a companion to almost every Sphinx project, including non-teaching ones.
Still... maybe this is mainly useful for teaching where it's seen on a screen?  Sounds good to merge and we can improve.  Send PRs when you want to move around.


Monster quantity buttons overlap in mobile

Via email:
On the encounter info manager portion, the first monster chosen will appear as normal but the + and - buttons to add more of the same monster is covered by the random encounter generator button. This is making it difficult to just add one more of the same monster.

@Asmor I was just looking into fixing, but it seems to not be an issue anymore. Can you confirm whether you still see it? Maybe it's a device-specific issue?

It only happens when your screen is narrower than 400px, and the Random button doesn't fit on the same line as "Encounter Info".
I put a CSS fix demo at https://snafuy.github.io/5e-monsters
(also JS fix for Issue #66 )

The fix for this is now live.


Fixes #317

Fixes #317 and uses biom.Table rather than pd.DataFrame for the feature table

It looks like the tests are failing because a few "test tables" are still being created as DataFrames:

self.table in tests/python/test_tools.py
table in tests/python/make-dev-page.py (looks like this can be fixed in one line, by replacing line 67 with table = table.view(biom.Table) (and importing biom ofc))

Thanks, @fedarko! I thought I had issued a full pytest call before the PR but had only tested out two of the modules. More shortly.

The following artifacts were built for this PR: empire-biplot.qzv, empire.qzv, empress-tree.qzv

Ok, everything looks good! Thanks so much @wasade! (╯°□°)╯︵ ┻━┻

woooo!!! thanks!!!

Thanks so much @wasade!


virtcontainers: improve algorithm to find containers

Do not iterate over a map to find a container, use map built-in
method instead.
fixes #2254
Signed-off-by: Julio Montes<EMAIL_ADDRESS>

/test-ubuntu


fix!: use correct path to extra file

#59
Fix incorrect path syntax

Codecov Report

Merging #63 (0365be9) into main (430ffc0) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #63   +/-   ##
=======================================
  Coverage   94.44%   94.44%
=======================================
  Files          18       18
  Lines         432      432
  Branches       36       36
=======================================
  Hits          408      408
  Misses         13       13
  Partials       11       11

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Tab names are invisible

Most of the time when I put my computer in sleep mode, all the tab names become invisible, and I have to restart Atom to make them visible again. This has been a problem for a long time now.
I can't reproduce it intentionally, because when I put my computer in sleep mode and use it in a few minutes, the problem doesn't occur.
Atom    : 1.12.5
Electron: 1.3.9
Chrome  : 52.0.2743.82
Node    : 6.5.0
Windows 10 x64
Image
Installed packages:
advanced-open-file
atom-beautify
atom-html-preview
atomic-chrome
atom-keyboard-macros
emmet
file-icons
git-time-machine
highlight-selected
jade-beautify
javascript-snippets
js-hyperclick
language-autohotkey
language-ejs
open-in-browsers
project-plus

Thanks for reaching out!
We require the template to be filled out on all new issues and pull requests. We do this so that we can be certain we have all the information we need to address your submission efficiently. This allows the maintainers to spend more time fixing bugs, implementing enhancements, and reviewing and merging pull requests.
Thanks for understanding and meeting us half way :grinning:


AMBARI-24566: Ambari-server cannot start

What changes were proposed in this pull request?
The layout in /var/lib/ambari-server/resource/stacks/ has been changed, the stackManager in ambari-server code cannot properly parse stackdirectory and get stackModule information, the fix is to recursively parse stackdirectory and create right stackModules map.
How was this patch tested?
Manually tested, patch the latest ambari-server code and it can start successfully

@jayush @swapanshridhar @jonathan-hurley @adoroszlai Please review this patch, thanks.


Include Telegram channel in page and/or Github

I belong to the Telegram channel, which is an excellent way to help new people.
It will be interesting to include the channel on the main page and this GitHub repository.

Hello @angoca ,
Not sure I understand your request.
But feel free to add a link to Telegram in the template messages sent to the new users in the region you're contributing.

No, I mean somewhere in https://welcome.osm.be/.
In fact, I have just discovered the Telegram group is a private one; therefore, it does not have a public URL.
I think an open Telegram group could be a way to help people to define new areas, improve translations, and get involved in platform maintenance. This way, more people could start using the platform.

Oh okay, you're talking about the "LCCWG - welcome tool group" Telegram group, right ?
That group wasn't really meant to be shared publicly but was setup in the early days of the new version of the Welcome Tool so I could share the progress of the development of the new version with people of LCCWG.


EDID page contains git merge conflicts.

This page is littered with git merge conflicts such as <<<<<<< HEAD and >>>>>>> master.
The markdown wasn't generated properly either. For example:
```registry
=======
```cpp

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

ID: 55e79e97-9c23-a7a4-f7e9-748525022e9c
Version Independent ID: ee067739-1ae0-4bd8-a206-4d36c31f17aa
Content: Overriding Monitor EDIDs with an INF - Windows drivers
Content Source: windows-driver-docs-pr/display/overriding-monitor-edids.md
Product: windows-hardware
GitHub Login: @nabazan-microsoft
Microsoft Alias: windowsdriverdev

@smithcoin Thank you for the issue! I've fixed the topic, and the fixes will be published by tomorrow.


Update index.md

I just added more info on C language...
Thank you for contributing! Please confirm this pull request meets the following requirements:

[X] Followed the contributing guidelines: https://github.com/github/explore/blob/master/CONTRIBUTING.md
[X] No affiliation with the project I am suggesting (as a maintainer, creator, contractor, or employee).

Which change are you proposing?

[X] Suggesting edits to an existing topic or collection
[ ] Curating a new topic or collection
[ ] Something that does not neatly fit into the binary options above

Editing an existing topic or collection
I'm suggesting these edits to an existing topic or collection:

[ ] Image (and my file is *.png, square, dimensions 288x288)
[X] Content (and my changes are in index.md)

I thought there was little information on C

Curating a new topic or collection

[ ] I've formatted my changes as a new folder directory, named for the topic or collection as it appears in the URL on GitHub (e.g. https://github.com/topics/[NAME] or https://github.com/collections/[NAME])
[ ] My folder contains a *.png image (if applicable) and index.md
[ ] All required fields in my index.md conform to the Style Guide and API docs: https://github.com/github/explore/tree/master/docs

Please replace this line with an explanation of why you think this topic or collection should be curated.

Something that does not neatly fit into the binary options above

[ ] My suggested edits are not about an existing topic or collection, or at least not a single one
[ ] My suggested edits are not about curating a new topic or collection, or at least not a single one
[ ] My suggested edits conform to the Style Guide and API docs: https://github.com/github/explore/tree/master/docs

Please replace this line with an explanation of your proposed changes.

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

Sorry closed this pull request... I am actually new to GitHub


Community Hangout - 22 October 2020

Our next 1h community hangout to chat about recent developments in Frictionless and data generally and next steps, involving our community members in the process!
Organizer: @sglavoie (Datopian) and @lwinfree (OKF)
Date: 22th October, 5pm BST
Agenda

Welcome and introductions: Icebreaker! [10m]
Link to meeting doc
Social Media @Frictionlessd8a and Discord Channel
Any items for the agenda?
Updates

frictionless-py — frictionlessdata/frictionless-py: Frictionless for Python is a framework to describe, extract, validate, and transform tabular data.

Community announcements/sharing

Opportunities to contribute - see the forum and "good first issue"
Keith: using frictionless to create infrastructure for sharing biology/genomics data packages.

Discussion

Frictionless-py questions/comments/feedback

AOB

Next meeting for November [link to be added]

Acceptance

[x] Blog post for the hangout (where we can also post updates afterwards) – DONE: https://frictionlessdata.io/blog/2020/10/28/october-virtual-hangout/
[x] Announce on social media
[x] Material prepped re how people can contribute.
[ ] An awesome 🎉 1h community hangout with at least 15 participants – Update: We had a great presentation from a new member! However, there were technical difficulties with Zoom. The link was apparently not working for some people and the attendance was 7-8 members on the call. Put in place: promote the hangout internally at Datopian and make sure a link is shared with an embedded password to avoid the need for a waiting room.

Tasks

[x] Zoom call link

[x] Reserve slot in Zoom account.
[x] Email guests ~2-3h before the event with the meeting details.

[x] Announce on Twitter – DONE: Oct 16 https://twitter.com/frictionlessd8a/status/1317090564140109825 + Oct 22 https://twitter.com/frictionlessd8a/status/1319252688534581250
[x] Send reminders on Discord – DONE: See https://discord.com/channels/695635777199145130/695635777199145133

[x] Send meeting link ~10 minutes before the event on Discord.

[x] Recording on YouTube. – DONE: Keith's presentation and the main event.

[x] Update blog post with link to YouTube video. – DONE: https://frictionlessdata.io/blog/2020/10/28/october-virtual-hangout/

[x] Material prepped re how people can contribute. – DONE: Letting the community know they can vote here https://github.com/frictionlessdata/frictionless-py/issues/486, linking to https://github.com/frictionlessdata/project/issues?q=is%3Aopen+is%3Aissue+label%3A"good+first+issue" and pointing to social media accounts so people can stay engaged.
[x] Schedule October Zoom meeting
[x] Update RSVP form with the correct hangout date. – DONE: https://forms.gle/3wEGBy2q4Q6pdNfK8
[x] Add previous guests to the Google Calendar invite
[x] Create a new entry in the public meeting doc with agenda for this hangout.

Note: The step of generating the list of people to invite was automated internally to facilitate the task and to respect people's privacy by taking into account their selection from their last entry for the column "Keep me up to date". We generate a string of comma-separated emails that can be pasted straight into Gmail/Google Calendar.

Tasks to add for next time

Advertise the hangout internally: announcements channel, All Hands meeting, reminder through email.
Make sure the Google Meet link is removed from the calendar. (This leads to confusion for some participants as there were two visible links one could join.)

I was thinking it might be cool to have @rufuspollock discuss the data views work at the next call. I think there are several groups that would find that interesting (including myself!). What do you think Rufus?

FIXED: Closing since we've had the community call and nothing more needs to be done specifically with this issue at this point – although further discussions are of course welcome! Issue for the next community call: #539.


Docker: Open in container fails. command in container failed.

VS Code Version: 1.65.1
Commit: 8908a9ca0f221f36507231afb39d2d8d1e182702
Date: 2022-03-08T02:06:27.846Z
OS Version: Windows_NT x64 10.0.19042

Steps to Reproduce:
The docker daemon runs on a remote host and the docker socket ports are forwarded to localhost. Such that any request from the docker desktop goes to the remote docker daemon.

Create Python 3.10 remote-container only modifications are setting: workspaceMount and wrokspaceFolder
Rebuild or reopen in container

Command in container failed: (dd iflag=fullblock bs=8192 count=6330 2>/dev/null; dd iflag=fullblock bs=6538 count=1 2>/dev/null) |
tar --no-same-owner -xz -C /vscode/vscode-server/bin/linux-x64/8908a9ca0f221f36507231afb39d2d8d1e182702_1647254460476

Frankly, may be two weeks ago the same procedure was working fine.
Dev Containers build output.
[9097 ms] Start: Run in container: mkdir -p '/home/vscode/.vscode-server/data/Machine' && cat >'/home/vscode/.vscode-server/data/Machine/settings.json' <<'settingsJSON'
[9105 ms] Start: Run in container: test -d /home/vscode/.vscode-server/bin/8908a9ca0f221f36507231afb39d2d8d1e182702
[9108 ms] Exit code 1
[9108 ms] Start: Run in container: test -d /vscode/vscode-server/bin/linux-x64/8908a9ca0f221f36507231afb39d2d8d1e182702
[9111 ms] Exit code 1
[9111 ms] Installing VS Code Server for commit 8908a9ca0f221f36507231afb39d2d8d1e182702
[9112 ms] Start: Installing VS Code Server
[9112 ms] Start: Run in container: mkdir -p /vscode/vscode-server/bin/linux-x64/8908a9ca0f221f36507231afb39d2d8d1e182702_1647254460476
[9166 ms] Start: Run in container: (dd iflag=fullblock bs=8192 count=6330 2>/dev/null; dd iflag=fullblock bs=6538 count=1 2>/dev/null) | tar --no-same-owner -xz -C /vscode/vscode-server/bin/linux-x64/8908a9ca0f221f36507231afb39d2d8d1e182702_1647254460476
gzip: stdin: invalid compressed data--format violated
tar: Skipping to next header
tar: Child died with signal 13
tar: Error is not recoverable: exiting now
[9221 ms] Exit code 2
[9232 ms] Command in container failed: (dd iflag=fullblock bs=8192 count=6330 2>/dev/null; dd iflag=fullblock bs=6538 count=1 2>/dev/null) | tar --no-same-owner -xz -C /vscode/vscode-server/bin/linux-x64/8908a9ca0f221f36507231afb39d2d8d1e182702_1647254460476
[9232 ms] gzip: stdin: invalid compressed data--format violated
tar: Skipping to next header
tar: Child died with signal 13
tar: Error is not recoverable: exiting now
[9233 ms] Exit code 2

Hi there,
I just updated vscode to the next version.

Version: 1.65.2 (user setup)
Commit: c722ca6c7eed3d7987c0d5c3df5c45f6b15e77d1
OS: Windows_NT x64 10.0.19042

and now it works again.


Add support for interfaces implementing other interfaces

To fix issue Add support for interfaces implementing other interfaces
To match new draft spec here: http://spec.graphql.org/draft/#sec-Interfaces.Interfaces-Implementing-Interfaces
Allows the following functionality:
interface Node {
  id: ID!
}

interface Resource implements Node {
  id: ID!
  url: String
}

The implementation here isn't massively efficient, I check multiple interfaces multiple times, mostly due to the requirement that

Transitively implemented interfaces (interfaces implemented by the interface that is being implemented) must also be defined on an implementing type or interface. For example, Image cannot implement Resource without also implementing Node:
https://spec.graphql.org/draft/#sel-FAHbhBHCAACGB35P

There's definitely a more efficient solution than the one here, really just wanted an opinion as to whether it is worth it.
Thanks
Tom

There are conflicts

👋  hey, what's the status with this PR? Any way I can help out? I just read through https://github.com/graphql/graphql-spec/pull/373 this could actually be very useful on a project I'm working on so keen to see this getting implemented. Awesome work so far though @tomemmerson 👍

@suessflorian I'd be happy to merge this change if you could resolve the conflicts

No worries @tomemmerson, your start here helped a lot - I've opened up a new PR continuing on from here and so I think we can close this PR @pavelnikolov 👍

Closing this PR because of #471


Inconsistant plugin sourcing order

Describe the bug
Optional plugins added with :packadd! foo are not sourced in the prescribed order.
To Reproduce

Create the following directory structure:
pack/brands/start/adidas/plugin/adidas.vim
pack/brands/start/nike/plugin/nike.vim
pack/brands/start/puma/plugin/puma.vim

Start Vim and do: :filter brands scriptnames:
 9: ~/.vim/pack/brands/start/adidas/plugin/adidas.vim
10: ~/.vim/pack/brands/start/nike/plugin/nike.vim
11: ~/.vim/pack/brands/start/puma/plugin/puma.vim

Plugins are sourced in filesystem order, which seems intuitive enough. All good.
Now…

Rename pack/brands/start to pack/brands/opt:
pack/brands/opt/adidas/plugin/adidas.vim
pack/brands/opt/nike/plugin/nike.vim
pack/brands/opt/puma/plugin/puma.vim

Add the following at the top of $MYVIMRC:
filetype on
packadd! adidas
packadd! nike
packadd! puma
finish

Start Vim and do :filter brands scriptnames:
 9: ~/.vim/pack/brands/opt/puma/plugin/puma.vim
10: ~/.vim/pack/brands/opt/nike/plugin/nike.vim
11: ~/.vim/pack/brands/opt/adidas/plugin/adidas.vim

Plugins are sourced in reverse :packadd order, which is not documented and rather counter-intuitive.
Expected behavior
Sourcing order should be the same as :packadd order:
 9: ~/.vim/pack/brands/start/adidas/plugin/adidas.vim
10: ~/.vim/pack/brands/start/nike/plugin/nike.vim
11: ~/.vim/pack/brands/start/puma/plugin/puma.vim

Environment

Vim version: 8.0.1365, 8.2.2681
OS: mac0S 10.14.16
Terminal: Irrelevant (happens in both GUI and TUI)

Additional info
In both cases, the plugin directories are added in the same order to &rtp.

This to be expected, ":packadd!" does not load the pack, that is postponed.  Thus the order can change.  And the implementation prepends the directory to 'runtimepath', thus the order is reversed.
I'll add a note in the documentation.  In general the order doesn't matter anyway.

And the implementation prepends the directory to 'runtimepath', thus the order is reversed.

The directories are added to &rtp in the same order in both cases, though:
[…]
/Users/romainl/.vim/pack/brands/start/puma,
/Users/romainl/.vim/pack/brands/start/nike,
/Users/romainl/.vim/pack/brands/start/adidas,
[…]

and:
[…]
/Users/romainl/.vim/pack/brands/opt/puma,
/Users/romainl/.vim/pack/brands/opt/nike,
/Users/romainl/.vim/pack/brands/opt/adidas,
[…]

and yet the plugins are sourced in different orders:

the same as &rtp when using opt and :packadd!,
the reverse of &rtp when using start.

which I can't explain.

In general the order doesn't matter anyway.

"In general", maybe, but apparently not in some specific cases involving popular plugins.

In the first case the packs are added when found, and the directory is added in 'runtimepath' in that order.
When using packadd! the directories are added to 'runtimepath' and only later the packs are added, when finding plugins in 'runtimepath'.

The problem with this is that, when adding those directories manually to &rtp, the plugins are sourced in the order they were added: you add adidas, then nike, then puma and they are sourced in that order.
If I can expect the outcome for the following snippet:
let rtp1 = &rtp->split(',')->filter({ idx, val -> val !~ "after$" })
let rtp2 = &rtp->split(',')->filter({ idx, val -> val =~ "after$" })

call add(rtp1, '/Users/romainl/.vim/pack/brands/foo/nike')
call add(rtp1, '/Users/romainl/.vim/pack/brands/foo/adidas')
call add(rtp1, '/Users/romainl/.vim/pack/brands/foo/puma')

let &rtp = (rtp1 + rtp2)->join(',')

to be that the plugins are sourced in the specified order, why can't I expect the same outcome for the following snippet?
:packadd! nike
:packadd! adidas
:packadd! puma


Update as upstream, improve grammar

Update as upstream to latest (15/9/1)
Improve grammar (KO) for more readability

Thanks!


added OTA status/progress reporting over mqtt

First cut at enhancement #171

Think I'm done with my first cut. Some Mosquitto logs running the latest homie-ota master (updated today):
device/$implementation/ota/payload <firmware blob>
device/$ota 1.0.0
device/$implementation/ota/status 202
device/$implementation/ota/status 206,1405/345520
device/$implementation/ota/status 206,2865/345520
device/$implementation/ota/status 206,4325/345520
...
device/$implementation/ota/status 206,343045/345520
device/$implementation/ota/status 206,344505/345520
device/$implementation/ota/status 206,345520/345520
device/$implementation/ota/status 200

Attempt to update to the firmware version currently running:
device/$implementation/ota/payload <firmware blob>
device/$ota 1.0.0
device/$implementation/ota/status 304

Bad firmware blob:
device/$implementation/ota/payload <patched firmware blob, first byte=FF>
device/$ota 1.0.0
device/$implementation/ota/status 202
device/$implementation/ota/status 206,1405/345520
device/$implementation/ota/status 206,2865/345520
...
device/$implementation/ota/status 206,344505/345520
device/$implementation/ota/status 206,345520/345520
device/$implementation/ota/status 400,OTA failed: ERROR[10]: Magic byte is wrong, not 0xE9

Questions:

Shall we use a space instead of comma between status code (206) and info (done/total)? Before Homie, I got very much used to 404 Not Found :-)

I am using error messages as provided by Updater::printError. Would you prefer simpler error messages with Homie users in mind? For example: OTA failed (invalid firmware binary) instead of OTA failed: ERROR[10]: Magic byte is wrong, not 0xE9?

I've been following the discussion, and like the use of HTTP status codes. I also prefer a space instead of a comma as separator between status code and info.

Yes, let's use spaces!
For the second point, simpler errors would indeed be better for parsing purposes. I would even use some sort of constants (e.g. 400 BAD_FIRMWARE), so that an UI could for example show i18n-ed messages.

+1 for the BAD_FIRMWARE codes!

Amazing feature! Thanks!
Sent from my iPhone

On 15 Oct 2016, at 11:04, JP Mens<EMAIL_ADDRESS>wrote:
+1 for the BAD_FIRMWARE codes!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@mrpace2 when you are done, can you rebase / merge against the develop branch? There should be a few conflicts that prevents me from accepting the PR.

Will do.

200 OK
(I'm done).


ARTEMIS-3485 Fix web plugin contexts

Support custom web plugin contexts defined with the url attribute
of the app element in bootstrap.xml.

I tested this, and it works great. Nice job, @brusdev!


Allow the association of other errors to FdbError

To make transact a little more flexible, I've added ErrorKind to FdbError.

renamed FdbError -> Error
introduced ErrorKind
introduced a From<failure::Error> for ease of conversion
transact now requires Into<foundationdb::error::Error> for it's future error result
Adds deref Vec<Element> to tuple

How about using Error: From<FdbError> for transact?
The patch will definitely increases usability of the crate, especially when user tries to mix tuple/subspace layers with other APIs. However where will be some inconvenience when user want to write a layer on top of core API. For example, now caller of Transaction::get might think they should handle non-core error codes, while they don't need to.

Sorry for late response. I wrote a comment to describe my concern. https://github.com/bluejekyll/foundationdb-rs/pull/77#discussion_r186436286
Except for Error type discussion, I'm great with the changes. Could you share your opinions about FdbError to Error change? If introducing Error is mainly for allowing error type other than FdbError in Database::transact, I think we can do that without Error. If the change is mainly for providing single error type over the crate for easy of use and If you think that's a right way to go, I'll follow your decision.
I'm sorry for not expressing my point clear earlier. I should have made my point clear at the first comment.

@bluejekyll Yes, It's better for me, thanks! Thanks for taking time to share the opinion.

Hit the wrong button there. Once tests pass this can be merged.

/ping @bluejekyll @yjh0502


[Snyk] Fix for 4 vulnerabilities

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to fix one or more vulnerable packages in the `npm` dependencies of this project.
Changes included in this PR

Changes to the following files to upgrade the vulnerable dependencies to a fixed version:

package.json
package-lock.json

Vulnerabilities that will be fixed
With an upgrade:

Severity
Priority Score (*)
Issue
Breaking Change
Exploit Maturity

608/1000  Why? Proof of Concept exploit, Recently disclosed, Has a fix available, CVSS 4.3
Information Exposure SNYK-JS-GATSBY-5671647
No
Proof of Concept

608/1000  Why? Proof of Concept exploit, Recently disclosed, Has a fix available, CVSS 4.3
Information Exposure SNYK-JS-GATSBYCLI-5671903
Yes
Proof of Concept

608/1000  Why? Proof of Concept exploit, Recently disclosed, Has a fix available, CVSS 4.3
Information Exposure SNYK-JS-GATSBYPLUGINSHARP-5671648
Yes
Proof of Concept

608/1000  Why? Proof of Concept exploit, Recently disclosed, Has a fix available, CVSS 4.3
Information Exposure SNYK-JS-GATSBYTRANSFORMERREMARK-5671901
Yes
Proof of Concept

(*) Note that the real score may have changed since the PR was raised.

Commit messages

Package name: gatsby
    The new version differs by 148 commits.

0c6cd61 chore(release): Publish
5e8e621 chore: Update main README (#36954)
7130cd4 test(gatsby): Slices API integration tests (#36747)
6496eed chore(release): Publish next
bc7ac84 chore: preserve previous webpack stats derived values, even if we restart webpack itself (#36980)
2b5af32 fix: drop `__renderedByLocation` prop when calculating slice props hashes and don't expose it to slice component (#36979)
cc1ee9b chore(release): Publish next
6a53861 chore(gatsby-link): Correct type export (#36968)
0ad6314 fix(gatsby-graphiql-explorer): Use upstream exporter package (#36966)
964265c chore(release): Publish next
b624442 chore: Update peerDeps (#36965)
b2ab092 chore(release): Publish next
e2a14bf feat(gatsby): Slices <> partial hydration interop (#36960)
0083e62 fix(deps): update starters and examples gatsby packages to ^4.24.7 (#36957)
68e9cab chore(changelogs): update changelogs (#36958)
b9eb8d2 chore(deps): update dependency autoprefixer to ^10.4.13 for gatsby-plugin-sass (#36934)
58c37ea chore(deps): update dependency @ jridgewell/trace-mapping to ^0.3.17 for gatsby-legacy-polyfills (#36933)
a5e4c47 fix(deps): update dependency body-parser to ^1.20.1 for gatsby-source-drupal (#36940)
c86aa7e chore(docs): Add clarification for Pro Tip on Part 4 of tutorial (#36918)
d5c775a feat(gatsby): handle graphql-import-node bundling (#36951)
59e2976 feat(gatsby-remark-embed-snippet): added csproj to language map so it will be recognized as xml (#36919)
c8a7dda chore(docs): Valhalla Content Hub Reference Guide (#36949)
3044280 fix(gatsby): stitch slices if just page html was regenerating without any of used slices regenerating (#36950)
10abdcb chore(release): Publish next

See the full diff

Package name: gatsby-plugin-sharp
    The new version differs by 250 commits.

ceb5527 chore(release): Publish
5bd48a5 chore(docs): update algolia guide (#38085)
89a3232 chore(gatsby-source-contentful): Fix docs pageLimit default value (#38082)
7501d18 chore(docs): Typo in v4 to v5 migration guide (#38081)
cbc0b35 feat(create-gatsby): Add Tailwind as a styling choice (#37944)
768581f chore(changelogs): update changelogs (#38077)
9284520 fix(deps): update starters and examples gatsby packages to ^5.9.1 (#38073)
22394b9 fix(deps): update e2e tests (major) (#38071)
9bb9037 chore(deps): update starters and examples (#38067)
ed5855e fix(gatsby): don't serve codeframes for files outside of compilation (#38059)
4cd23bf chore(release): Publish next
dfdeed4 fix(gatsby-source-drupal): add image cdn support for `files` type and `typePrefix` (#38057)
e5e2bb7 fix(gatsby-source-drupal): find mimetype field (#38056)
38fae7a chore(docs): Improve wording in main tutorial part 6 (#38054)
a9c54f7 chore(release): Publish next
d611439 fix(deps): update minor and patch dependencies for gatsby-source-graphql (#38028)
213d8b0 fix(deps): update minor and patch dependencies for gatsby-plugin-mdx (#38027)
272dacd fix(gatsby): handle cyclic chunkgroup children (#38052)
a83ed19 fix(deps): update minor and patch dependencies for gatsby (#38005)
d7cccfe fix(deps): update dependency sharp to ^0.32.1 (#38024)
2d2b7c2 chore: Fix dirty lock file
1a4b234 chore(deps): update dependency @ jridgewell/trace-mapping to ^0.3.18 for gatsby-legacy-polyfills (#37996)
ccecd9d chore(deps): update dependency rimraf to v5 for gatsby-page-utils (#38039)
94099a9 chore(deps): update dependency rimraf to v5 for gatsby-plugin-utils (#38040)

See the full diff

Package name: gatsby-transformer-remark
    The new version differs by 250 commits.

ceb5527 chore(release): Publish
5bd48a5 chore(docs): update algolia guide (#38085)
89a3232 chore(gatsby-source-contentful): Fix docs pageLimit default value (#38082)
7501d18 chore(docs): Typo in v4 to v5 migration guide (#38081)
cbc0b35 feat(create-gatsby): Add Tailwind as a styling choice (#37944)
768581f chore(changelogs): update changelogs (#38077)
9284520 fix(deps): update starters and examples gatsby packages to ^5.9.1 (#38073)
22394b9 fix(deps): update e2e tests (major) (#38071)
9bb9037 chore(deps): update starters and examples (#38067)
ed5855e fix(gatsby): don't serve codeframes for files outside of compilation (#38059)
4cd23bf chore(release): Publish next
dfdeed4 fix(gatsby-source-drupal): add image cdn support for `files` type and `typePrefix` (#38057)
e5e2bb7 fix(gatsby-source-drupal): find mimetype field (#38056)
38fae7a chore(docs): Improve wording in main tutorial part 6 (#38054)
a9c54f7 chore(release): Publish next
d611439 fix(deps): update minor and patch dependencies for gatsby-source-graphql (#38028)
213d8b0 fix(deps): update minor and patch dependencies for gatsby-plugin-mdx (#38027)
272dacd fix(gatsby): handle cyclic chunkgroup children (#38052)
a83ed19 fix(deps): update minor and patch dependencies for gatsby (#38005)
d7cccfe fix(deps): update dependency sharp to ^0.32.1 (#38024)
2d2b7c2 chore: Fix dirty lock file
1a4b234 chore(deps): update dependency @ jridgewell/trace-mapping to ^0.3.18 for gatsby-legacy-polyfills (#37996)
ccecd9d chore(deps): update dependency rimraf to v5 for gatsby-page-utils (#38039)
94099a9 chore(deps): update dependency rimraf to v5 for gatsby-plugin-utils (#38040)

See the full diff

Check the changes in this PR to ensure they won't cause issues with your project.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
🛠 Adjust project settings
📚 Read more about Snyk's upgrade and patch logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Learn about vulnerability in an interactive lesson of Snyk Learn.

👷 Deploy Preview for relaxed-rolypoly-ea9b8f processing.

Name
Link

🔨 Latest commit
a3de2c9cadcdfbb95a8aafa81143fa4d7b608cf8

🔍 Latest deploy log
https://app.netlify.com/sites/relaxed-rolypoly-ea9b8f/deploys/648263ec859b540008b1e3d2


Make end boundary of history optional (undefined)

Since None is used for the end boundary of a delta table history it could also easily be "transferred" up the call chain and be the default input value. That's the purpose of the PR.
Signed-off-by: Jacek Laskowski<EMAIL_ADDRESS>

May I have your take on this? @zsxwing @brkyvz @tdas @marmbrus (as the author)

This PR looks good. Thanks for improving the code quality! @jaceklaskowski


Suggested CSS font-family values should be quoted

VSCode Version: 1.18.1
OS Version: Windows 10

When VSCode suggests CSS font-family values with white spaces like Times New Roman, it's not quoted.
Edit : Actually, many are quoted. I just found the suggestion Cambria, Cochin, Georgia, Times, Times New Roman, serif where the quotes are missing.

fixed by https://github.com/Microsoft/vscode-css-languageservice/commit/ece7dbdeb9a16104d0d8952d3d99d1d8df01711a


Google ID registered in setting.cfg doesn't work

I edit the file setting.cfg as
google_userid = ganuongphap

or
google_userid =<EMAIL_ADDRESS>
but it still prompts me for the ID. Other settings work as expected. I use Python 3.11.3 on Windows 11.

@ooker777 - that is the correct setting. The most likely case is there is a unusual/unknown character in the settings.cfg file. I would just delete it and let KIM create a new one for you. Then, just type your id in the google_userid field.

Somehow it worked again. However at the time of asking I did delete and let KIM make a new one, but it didn't work.


Change description of chart

I would change the text into something like "Games in HLTB database", because that's what it is and not the completed games. Here 28 completed but 39 in db.

This screen is in the HowLongToBeat data. Therefore, the data comes from HowLongToBeat.
Isn't that explicit enough?

It‘s not about HltB, it’s about the word completed - that#s wrong


system_config: add source_only_buffer to SYSTEM_CONFIG_PARAMETERS

Which issue(s) this PR fixes:

Additional fix for #4661

What this PR does / why we need it:
Need this to the section handled in SystemConfig#dup.
This should be added to SYSTEM_CONFIG_PARAMETERS, though this is not affected to the actual behavior,
Note: I don't understand why log section is not added. Maybe it should be added as well.
(It should be done by another PR).
Docs Changes:
Not needed
Release Note:
Not needed

Thanks!


⚠ Bofrost.be has degraded performance

In 5a15075, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 1931 ms

Resolved: Bofrost.be performance has improved in 23f5017.


.translation() does not return the world position systematically

I'm facing an odd behaviour to me:
Description
✅ OK Case 1: position on <RigidBody>
<RigidBody ref={bodyRef} colliders="ball"
  position={[3, 3, 3]} // 👈🏻 `position` on RigidBody
>
  <mesh>
    <icosahedronGeometry />
    <meshStandardMaterial />
  </mesh>
</RigidBody>

useEffect(() => {
  console.log("translation=", bodyRef.current?.translation());
}, []);

outputs: translation= Vector3 {x: 3, y: 3, z: 3}

https://codesandbox.io/s/heuristic-ramanujan-emrms9?file=/src/App.tsx:529-549
❌ NOK Case 2: position on <mesh>
<RigidBody ref={bodyRef} colliders="ball">
  <mesh
    position={[3, 3, 3]} // 👈🏻 `position` on mesh
  >
    <icosahedronGeometry />
    <meshStandardMaterial />
  </mesh>
</RigidBody>

useEffect(() => {
  console.log("translation=", bodyRef.current?.translation());
}, []);

outputs: translation= Vector3 {x: 0, y: 0, z: 0}

https://codesandbox.io/s/quirky-microservice-ki7gr7?file=/src/App.tsx:553-573
Expected
The expected behaviour to me, should be .translation() always return 3,3,3 ie. the real, absolute, only one world position of the object

As discussed on Discord, and thanks to @JesproKing's message and @wiledal:

there's got to be more to say about this. How do you get the position of the rigid body? translation() doesn't update on every useFrame

@Seanmclem it seems you're talking about a different problem here. Opening a new issue with your specific details should help, especially with a codesandbox that reproduces it


why small batch_size

Hi there,
I notice your team since DAB-DETR. Your work leaves me great impression. However, I have a question about the small batch size. Why do you use such a small batch size? Do you run an experiment on it? Thanks for your help~

Batch size 16 is commonly used in many DETR-like models. We have tried other batch size before, but it seems batch size 16 achieves the best performance.
We are also curious about this phenomenon, but we can not provide an in-depth analysis of it now (:

Batch size 16 is commonly used in many DETR-like models. We have tried other batch size before, but it seems batch size 16 achieves the best performance.
We are also curious about this phenomenon, but we can not provide an in-depth analysis of it now (:

However, I reviewed the paper and the project on the github. They say that the batch_size you use is 2 for distributed mode.

Batch size 16 is commonly used in many DETR-like models. We have tried other batch size before, but it seems batch size 16 achieves the best performance.
We are also curious about this phenomenon, but we can not provide an in-depth analysis of it now (:

However, I reviewed the paper and the project on the github. They say that the batch_size you use is 2 for distributed mode.

We use batch size 2 or 4 per GPU and 8 or 4 GPUs in total.

Batch size 2 for each GPU, totally 8 GPUs, so total batch size is 16.

Batch size 2 for each GPU, totally 8 GPUs, so total batch size is 16.

Thanks for your help!


Fix missing actions for stock item

Fixes bug in recent UI overhaul which removed stock item actions for users who did not have "superuser" status


AppendChild adds extra empty lines in the csproj

This is an example of an app that reproes this. Notice that there may be other operations that cause this as well. AppendChild on ItemGroup maybe. So, please take a look at those other operations too.
This blocks the CLI from cleaning up the migrated csproj.
using System;
using System.IO;
using Microsoft.Build.Construction;
using Microsoft.Build.Evaluation;

namespace ConsoleApplication
{
    public class Program
    {
        public static void Main(string[] args)
        {
            Environment.SetEnvironmentVariable("MSBUILD_EXE_PATH", args[0]);

            var templateProjPath = args[1];

            var project = ProjectRootElement.Open(
                templateProjPath,
                ProjectCollection.GlobalProjectCollection,
                preserveFormatting: true);

            var propertyGroup = project.AddPropertyGroup();

            var outputProperty = propertyGroup.ContainingProject.CreatePropertyElement("some");
            propertyGroup.AppendChild(outputProperty);

            var outputProject = Path.Combine(AppContext.BaseDirectory, "output.csproj");

            project.Save(outputProject);
        }
    }
}

Its project.json:
{
  "version": "1.0.0-*",
  "buildOptions": {
    "debugType": "portable",
    "emitEntryPoint": true
  },
  "dependencies": {
    "Microsoft.Build.Runtime": "15.1.319-preview5"
  },
  "frameworks": {
    "netcoreapp1.1": {
      "dependencies": {
        "Microsoft.NETCore.App": {
          "type": "platform",
          "version": "1.0.1"
        }
      },
      "imports": "dnxcore50"
    }
  }
}

This is the output csproj. Notice the empty lines around some.
<Project ToolsVersion="15.0" xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
  <Import Project="$(MSBuildExtensionsPath)\$(MSBuildToolsVersion)\Microsoft.Common.props" />

  <PropertyGroup>
    <OutputType>Exe</OutputType>
    <TargetFramework>netcoreapp1.0</TargetFramework>
  </PropertyGroup>

  <PropertyGroup>

    <some />

  </PropertyGroup>

  <ItemGroup>
    <Compile Include="**\*.cs" />
    <EmbeddedResource Include="**\*.resx" />
  </ItemGroup>

  <ItemGroup>
    <PackageReference Include="Microsoft.NETCore.App">
      <Version>1.0.1</Version>
    </PackageReference>
    <PackageReference Include="Microsoft.NET.Sdk">
      <Version>1.0.0-alpha-20161019-1</Version>
      <PrivateAssets>All</PrivateAssets>
    </PackageReference>
  </ItemGroup>

  <Import Project="$(MSBuildToolsPath)\Microsoft.CSharp.targets" />
</Project>

When preserveFormatting is true and you add a new element, it tries to indent it by matching the whitespace around the previous or next sibling element, or by adding two spaces for indentation to the whitespace around the parent element if there are no siblings.  So what's happening here is that the whitespace it's matching includes blank lines.
The code that does this is here.

My problem is that during migration we are heavily modifying the csproj, so, in the end I am ending up with something like this:
<Project ToolsVersion="15.0" xmlns="http://schemas.microsoft.com/developer/msbuild/2003">
  <Import Project="$(MSBuildExtensionsPath)\$(MSBuildToolsVersion)\Microsoft.Common.props" />

  <PropertyGroup>

    <TargetFramework>netcoreapp1.0</TargetFramework>

    <OutputType>Exe</OutputType>

  </PropertyGroup>

  <ItemGroup>

    <Compile Include="**\*.cs" Exclude="bin\**;obj\**;**\*.xproj;packages\**" />

    <EmbeddedResource Include="**\*.resx" Exclude="bin\**;obj\**;**\*.xproj;packages\**" />

    <EmbeddedResource Include="compiler\resources\**\*" Exclude="bin\**;obj\**;**\*.xproj;packages\**" />

    <Content Include="Program.cs" Exclude="Program.cs">

      <Link>out/%(FileName)%(Extension)</Link>

      <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>

    </Content>

    <Content Include="testcontentfile2.txt">

      <Link>dir/mappingfile.txt</Link>

      <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>

    </Content>

    <Content Include="testcontentfile.txt">

      <CopyToOutputDirectory>PreserveNewest</CopyToOutputDirectory>

      <CopyToPublishDirectory>PreserveNewest</CopyToPublishDirectory>

    </Content>

  </ItemGroup>

  <ItemGroup>

    <PackageReference Include="Microsoft.NET.Sdk">

      <Version>1.0.0-alpha-20161019-1</Version>

      <PrivateAssets>All</PrivateAssets>

    </PackageReference>

    <PackageReference Include="Microsoft.NETCore.App">

      <Version>1.0.1</Version>

    </PackageReference>

  </ItemGroup>

  <PropertyGroup Condition=" '$(TargetFramework)' == 'netcoreapp1.0' ">

    <DefineConstants>$(DefineConstants);NETCOREAPP1_0</DefineConstants>

  </PropertyGroup>

  <PropertyGroup Condition=" '$(Configuration)' == 'Release' ">

    <DefineConstants>$(DefineConstants);RELEASE</DefineConstants>

  </PropertyGroup>

  <Import Project="$(MSBuildToolsPath)\Microsoft.CSharp.targets" />
</Project>

@livarcocc Yes, my explanation was intended to show where this is happening and help make it easier to fix.

Okay here's the issue.  The code that is appending the property is trying to match its parent element's whitespace.  In this case, the <PropertyGroup />'s whitespace is a newline and some indentation because your first <PropertyGroup /> has an extra linebreak between it and the <Import /> above.
So it's trying to make the children of the new <PropertyGroup /> have the same leading and trailing whitespace as the <PropertyGroup /> itself.
We could perhaps have it look for the closest relative and try to match the whitespace of the children of the first <PropertyGroup /> instead but that could be tricky.


Request was not matched when using urlPath with feing.@QueryMap

Description
It is not possible to match contract url using urlPath when using feing.@QueryMap as request param. Every other request without @QueryMap works fine.
version of Spring Cloud Contract: 2.2.4.RELEASE
version of Spring Cloud: Hoxton.SR7
Sample:
Contract:
Contract.make {

    request {
        method POST()
        urlPath("/v1/internal/ledgers/search?lastEntityId=111&pageSize=25&sortColumn=inputDate&direction=ASC")
        headers {
            contentType applicationJson()
        }

body and response are irrelevant
Interface used to post request:
@PostMapping("v1/internal/reports/search")
DataResultContainer<List<ReportRow>> postForReportsData(@Valid @RequestBody ReportSearchRequest request,
                                                            @RequestParam @QueryMap FeignNextPageRequest pageRequest);

Test output:
2020-10-08 14:17:46.462 ERROR 8840 --- [p1672012423-115] WireMock                                 :
                                               Request was not matched
                                               =======================

-----------------------------------------------------------------------------------------------------------------------
| Closest stub                                             | Request                                                  |
-----------------------------------------------------------------------------------------------------------------------
                                                           |
POST                                                       | POST
/v1/internal/ledgers/reversingEntry/9fa8c55f-11b7-4f7c-84  | /v1/internal/ledgers/search?lastEntityId=111&pageSize=25&<<<<< URL does not match
dd-b02af7879503                                            | sortColumn=inputDate&direction=ASC
                                                           |
                                                           |
-----------------------------------------------------------------------------------------------------------------------

As you can see request paths are identical.
If we modify contract as below, request is matched and everything is OK.
Contract.make {

    request {
        method POST()
        url '/v1/internal/ledgers/search?lastEntityId=111&pageSize=25&sortColumn=inputDate&direction=ASC'
        headers {
            contentType applicationJson()
        }

I'm not sure if it is bug but we spent a lot of time to find workaround for this problem and it may help someone in future ;)

I think that's because with urlPath you should be then calling queryParameters. In your case the urlPath is actually        /v1/internal/ledgers/search.
        urlPath("/v1/internal/ledgers/search) {
             parameter 'lastEntityId': '111'
             parameter 'pageSize': '25'
             parameter 'sortColumn': 'inputDate'
             paramtere 'direction' : 'ASC'
        }


SHAREFILE PROVISIONING

[Hi guys, can you add how to provision sharefile users please? There is nothing there]

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

ID: f0bc4f40-b67f-5d0d-88f3-7627963d6dd8
Version Independent ID: e2898011-786f-0157-a567-14e12c8cb384
Content: Tutorial: Azure Active Directory integration with Citrix ShareFile
Content Source: articles/active-directory/saas-apps/sharefile-tutorial.md
Service: active-directory
Sub-service: saas-app-tutorial
GitHub Login: @jeevansd
Microsoft Alias: jeedes

@leandmilo
Thanks for your feedback! I've assigned this issue to the author who will investigate and update as appropriate.

@v-rant Can you please work with ShareFile team and get this document updated?

Hi guys, any update with the documentation? Thanks

Hi guys, any update with this article about Provisioning? Thanks in advance @JamesTran-MSFT  @jeevansd  @v-rant?

Guys, any comments about this? Thank you

The Citrix Sharefile app is currently in an unsupported state, documentation will be added once the app is again in a stable state.

#please-close


YaRD editor validation

Tickets this change has:
YaRD editor: Create yard-validation project with model parser
YaRD editor: Use old dtable validation codes for subsumption
YaRD editor: Use J2CL to generate a JS module
I split the model to a separate project, but since this validation is the only module that will use it, I will investigate if the two modules should be merged together.

Temp commit to show progress. The code can now find subsumption that is the basic function for redundancy and conflict resolution. Needs more test data to be really really done, I will be adding that next.

Missing items needed for commit:

Licence headers
Inline rule parsing

Transpiling LocalDateTime correctly is more challenging than expected, but on it.

@paulovmr I could use some help on figuring out the formatting for the README and yml file. Can I run a formatter for them locally? Other than that I could also use a code review for the code.

Hi @Rikkola, could you please improve the PR title?

ubuntu-latest and macos-latest are giving eclipse dependency issue errors that are not related.

@paulovmr ok, all should be fine now

@Rikkola I just sent a PR to your fork with some suggestions related to the Serverless Logic Web Tools changes. Please check if they make sense.

Yes the PR makes sense, I merged it in.

Looks like I still lack the right files for formatting the pom.xml files.

Figured it out.


cannot import tape

I install tape use "pip install tape_proteins"and it shows that

but when "import tape" or "import tape_proteins"
it shows that

Could you repeatedly run pip uninstall tape_proteins until pip says it's not installed? You may have to run it multiple times. Then try running pip install tape_proteins.

On Mar 6, 2022, at 11:58 AM, willow-yll @.***> wrote:
I install tape use "pip install tape_proteins"and it shows that
https://user-images.githubusercontent.com/79819922/156933350-fab3ef39-0566-4bdc-9560-ed527bd77d6a.png
https://user-images.githubusercontent.com/79819922/156933355-b549fe9d-b581-405c-8e26-c6669806f8dd.png
https://user-images.githubusercontent.com/79819922/156933366-24a8997f-eec7-43f8-b1b4-2eac2c731862.png
but when "import tape" or "import tape_proteins"
it shows that
https://user-images.githubusercontent.com/79819922/156933394-66353232-49ee-4f35-9113-5139e5ec6aa4.png
—
Reply to this email directly, view it on GitHub https://github.com/songlab-cal/tape/issues/124, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABRSCXL7ZURERBUIVP7ACNDU6TP3TANCNFSM5QBLHHXA.
Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you are subscribed to this thread.

thank you !!!!  It is resolved!


Invalid operating system: netbsd

I am trying to run a build on NetBSD. But unfortunately, I always received the error message:
Invalid operating system: netbsd
https://github.com/gansm/finalcut/runs/4183358693?check_suite_focus=true

Yes, it's not ready yet. I used  the "Fix #1" syntax in the commit message which closes the issue automatically. But there are other repositories involved where I need to do some cleanup. Then I need to create a new releases/tags and you need to update to use the new tag, when it's available. I will hopefully get this done today or this weekend.

Excellent, it now works perfectly!
https://github.com/gansm/finalcut/runs/4201709257?check_suite_focus=true

Perfect, thanks.

Closed in https://github.com/cross-platform-actions/action/releases/tag/v0.3.0.


chore: add local development guide and shell script

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache-2.0 license
Summary
(describe what this merge request does)
Implementation highlights
(describe how the merge request does for feature changes, share the RFC link if it has)
Test checklist

[ ] add new test cases
[ ] all code changes are covered by unit tests
[ ] end-to-end tests

[ ] deploy web console with CloudFront + S3 + API gateway
[ ] deploy web console within VPC
[ ] deploy ingestion server

[ ] with MSK sink
[ ] with KDS sink
[ ] with S3 sink

[ ] deploy data processing
[ ] deploy data modeling

[ ] new Redshift Serverless
[ ] provisioned Redshift
[ ] Athena

[ ] deploy with reporting

Is it a breaking change

[ ] add parameters without default value in stack
[ ] introduce new service permission in stack
[ ] introduce new top level stack module

Miscellaneous

[ ] introduce new symbol link source file(s) to be shared among infra code, web console frontend, and web console backend

How about moving those changes to src/control-plane/?

done.


Add Firebase example

Quick PR for working with Firebase (REST API). Let me know if you'd like anything changed 🙌

Codecov Report

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

@@           Coverage Diff           @@
##           master     #313   +/-   ##
=======================================
  Coverage   99.29%   99.29%
=======================================
  Files          11       11
  Lines         427      427
=======================================
  Hits          424      424
  Misses          3        3

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 56fd29e...b3e909a. Read the comment docs.

So simple & elegant, great job @stursby


upgrade to latest Rails 5.0 (5.0.7)

Description of changes
It's good to keep up with latest point release of Rails.
Issue Resolved
No open issue. Resolves some out-of-date gem dependencies.

Let's give this a shot.


To avoid an E_NOTICE, check Ratelimit-Remaining header existence.

Pretty much says it all, the diff is simple enough.

Accidental duplicate of #153, didn't realize someone already requested this fix.


Text character count

Some of our text input have a limit. To help users, it would be great to be able to specify how many characters left before the user hits the limit. We are using this library GDS styling so something like this would be great: https://design-system.service.gov.uk/components/character-count/
Key features:

Developer should be able to specify the character limit number
Developer should be able to pass in the character limit prompt message (one when JS is enabled, and another for when non-js is disabled)
HTML structure should match GDS markup (with ability to set css classes) or the developer should be able to specify the HTML markup
Character should count down when JS is enabled

Progressive enhancements:

Please take into account how this component behaves without javascript
Without Javascript, the prompt message should be static
With Javascript, the prompt message should be dynamic to inform use how many characters left

closed by #419


multiple platform gems confusing Bundler when switching Rubies

at first I thought to be running into #4620 but as I tried to isolate I noticed its smt different.
when there are multiple platform specific gems (with deps changing btwn platforms) Bundler 1.12.5 fails to bundle install under a different platform (Ruby).
source 'https://rubygems.org'

gem 'pry-rails', '0.3.4', require: false
gem 'sauce-cucumber', '3.7', require: false

bundle installed under JRuby (<IP_ADDRESS>) fails to bundle install under MRI (2.3.1)
Here's the Gemfile.lock generated under JRuby just try to bundle install under MRI, it shall fail with :
Your Gemfile.lock is corrupt. The following gem is missing from the DEPENDENCIES section: 'ffi'
NOTE: that it works fine when I remove either gem declaration (keeping just one).
its something about having multiple platform gems that bundler fails to resolve correctly when switching ...

pry-rails depends on pry which has platform specific versions
sauce-cucumber -> sauce -> childprocess -> ffi (has -java as well as "ruby" gem version)

p.s. very nice job towards handling platform gems in 1.12.5, thank you :love_letter: !

I was unable to reproduce this with master, and I've touched a lot of the platform code recently, so it being fixed wouldn't entirely surprise me

ping @kares, is still a problem on the latest release?

1.13.6 failed the same (Your Gemfile.lock is corrupt. ...) but 1.14.4 seems to work under MRI 2.3.3, thanks!


addnwstr function in dvtm.c

What is the addnwstr function supposed to do? It is only called in one place and doesn't seem to get defined anywhere.

It's a curses function to "add a string of wide characters to a curses window and advance cursor". See https://linux.die.net/man/3/addnwstr .

Right. In that case, for Ubuntu 16.04 I can squash the complaint that addnwstr is implicitly declared:
diff --git a/dvtm.c b/dvtm.c
index 2b7ebdc..65dbe66 100644
--- a/dvtm.c
+++ b/dvtm.c
@@ -23,7 +23,7 @@
 #include <sys/time.h>
 #include <sys/types.h>
 #include <fcntl.h>
-#include <curses.h>
+#include <ncursesw/curses.h>
 #include <stdio.h>
 #include <stdarg.h>
 #include <signal.h>
@@ -396,8 +396,8 @@

The normal /usr/include/curses.h doesn't include reference to addnwstr.


Recover Data tab's onboarding tour

Context
We used to have an onboarding tour in the Data tab but after some of the recent changes, we lost it. We should recover it.

More context from @ivanmalagon:

first, every tab in layer section has the same key for the onboarding
that’s a result from this ticket
https://github.com/CartoDB/cartodb/issues/11586
(one year ago)
then, the latest changes on tours did that once a tour has been seen, it’s discarded from the time being
https://github.com/CartoDB/cartodb/issues/13003
as all the tours on layer sections share the same key, once you see one, you see all
that’s why you don’t see never the data onboarding
we should revert the unique key for every tour on layer section

@ramiroaznar adding this to the RT project!

We forgot to address this when solving this https://github.com/CartoDB/cartodb/issues/13415

It should be fixed too

sure, I'll take a look

I'm not sure if it's going to work for old users, it should, since the name we use to store those tours are different now, but I can't assure it 100%

Tested in a new user:

Hey guys, looks like this splash screen is appearing on old users (https://github.com/CartoDB/design/issues/1276#issuecomment-372769407). It shouldn't have. This should only appear during the onboarding process for new accounts.

This was what we implemented.

Right.
Layer content onboarding: ONCE PER USER
We need to assume that old accounts already saw the layer content onboarding. What's the point in showing an onboarding splash screen to already on-boarded users?

The keys we check for this have changed, before it was a generic layer-onboarding one, while now they are divided for each tab layer-data-onboarding, layer-style-onboarding, etc.
When we check those keys in old users they don't exist yet, so we show the onboarding.
I could've checked for old_key OR new_key, but I didn't think it was going to be such a problem to show it again to old users...

I could've checked for old_key OR new_key, but I didn't think it was going to be such a problem to show it again to old users...

Agree.

Thanks for the explanation @rubenmoya.
We should discuss those assumptions before implementing them, though. Onboarding screens are usually quite annoying. Users just want to get the job done and get out. If you take a look at Fullstory you can see how users rarely stop to read the onboarding and try to close it as fast as possible. And for old users, that already know how things work (we can only hope!), is an unnecessary burden.


Feature Request: Support for stock line chart

it would be great if you can add support for stock line charts.

https://www.highcharts.com/demo/stock/basic-line

Looks cool, but I believe you can create this with currently existing functionality. I'm trying not to bloat the library to much, so I probably won't implement this kind of specific chart anytime soon

I see, maybe you can add this issue to your repository's milestone, RN has a lack of js implementation for this kind of chart and it would be great have it out of the box. otherwise, it can be closed.
However, thanks for your attention and response.

I'm going to close it, but I'll keep it in mind :)


GCS & 1.10.2

Using dag provision through gcs and the latest release continously crash loop.
airflow.exceptions.AirflowConfigException: In kubernetes mode the following must be set in the `kubernetes` config section: `dags_volume_claim` or `dags_volume_host` or `dags_in_image` or `git_repo and git_branch and git_dags_folder_mount_point`

Is there anything config specific to bypass this? I've tried setting manually the dags_volume_host which does not seem to be passed to the actual pods.

Thanks for the PR. Would you elaborate and share the CRs ?

@barney-s This is more a question than a PR. Is there anything I can add?

Running into this problem as well.

@zifeo I got past it by adding config AIRFLOW__KUBERNETES__DAGS_IN_IMAGE: "True"
eg:
apiVersion: airflow.k8s.io/v1alpha1 kind: AirflowCluster metadata:   name: airflow-cluster spec:   config:     airflow:       AIRFLOW__SCHEDULER__DAG_DIR_LIST_INTERVAL: "100"       AIRFLOW__KUBERNETES__DAGS_IN_IMAGE: "True"
This was not the easiest to find might make sense to put it in the k8s executor examples? @barney-s

@Pritesh-Patel Could you share the full config file (except the bucket name of course)? I have tried the same without success.

@zifeo
apiVersion: airflow.k8s.io/v1alpha1
kind: AirflowCluster
metadata:
  name: airflow-cluster
spec:
  config:
    airflow:
      AIRFLOW__SCHEDULER__DAG_DIR_LIST_INTERVAL: "100"
      AIRFLOW__KUBERNETES__DAGS_IN_IMAGE: "True"
  executor: Kubernetes
  ui:
    replicas: 1
    version: "1.10.2"
  scheduler:
    version: "1.10.2"
  worker:
    version: "1.10.2"
  dags:
    subdir: ""
    gcs:
      bucket: "airflow-dags"
  airflowbase:
    name: airflow-base

Unfortunately after this I ran into SQL errors and decided to give up on this and use the helm chart. I would preferably like to use this - but i dont have the time to debug the issue. Let me know if you get further!

This is somewhat an aside - but if you want an config file which runs Airflow-in-Kubernetes on version 1.10.3 successfully, ours is open source: https://github.com/SixtyCapital/infrastructure/blob/master/docker/airflow/airflow.cfg
Our Kubernetes setup is more basic than this repo, unfortunately; though it works well

@max-sixty Looks interesting although it does not seem to support the Kubernetes executor?
@Pritesh-Patel Thanks. I will retry soon. I am curious to see whether you managed to run the Kubernetes executor with the helm chart. It seems to be unstable and was the reason we switched to the operator.

Yes @zifeo - we would love to use the Kubernetes Executor but haven't found it developed enough to use practically, yet

@barney-s is there something on the roadmap about this?


When adding files manually it does not count towards the max file count

I have a max file count of 1.
When I add a file in the interface it will only allow 1 file at a time. However if I programatically add an existing file on the server first, then proceed to upload another file it says the max file count is 0 and won't let me. Even if I delete the original image it is still now set to a max file count of 0.
Interestingly when I manually change the max file count this.$refs[this.dropzoneIdentifier].dropzone.options.maxFiles = this.maxFiles; before I manually add the files it set it properly to 1 but does not count the manually added file towards the max file count. Therefore I can add another file before it maxes out, totalling 2 files.
Is this a known issue or is there a way around it, thanks.

Hi @robertmylne
How are you manually adding the file? Are you using the manuallyAddFile method that we provide? A code snippet would help us understand exactly what you're doing.
Cheers
Rowan

Hi folks,
Version 3 is now released. Check out the new docs for more info.

Closing due to inactivity, feel free to reopen if you feel this is still an issue

Just tested this and can verify this issue still exists. Max files exceeded is not triggered by manually adding files.

I'm using manuallyAddFile and it's working inversely for me as you guys.
I have set maxFile:4 but when I manually add 2 files or 1 file and then try to upload file, it says max file upload limit reached.


Fix typo in the filename 50-infrastructure

Partly #267
Signed-off-by: Ramona Rautenberg<EMAIL_ADDRESS>

Depends on: https://github.com/osism/cfg-generics/pull/273


Update usage hints in afl-fuzz and supported options in README.md

Update usage hints in afl-fuzz and supported options in README.md, and fix some minor typos.

Ah, flag descriptions were indeed not up to date. Thanks for fixing!


[BUG] driver name driver.longhorn.io not found in the list of registered CSI drivers

Describe the bug

driver name driver.longhorn.io not found in the list of registered CSI drivers
The triggering reason may be related to the restart of the node
k8s v1.19.4
longhorn v1.6.1
Currently, this issue only exists on the Risc-verse-k8sw-02 node
Sep 09 16:47:11 risc-verse-k8sw-02 kubelet[3240779]: I0909 16:47:11.121692 3240779 operation_generator.go:622] "MountVolume.WaitForAttach entering for volume "pvc-e43dde24-fb36-4b0d-b59e-e09c52d94015" (UniqueName: "kubernetes.io/csi/driver.longhorn.io^pvc-e43dde24-fb36-4b0d-b59e-e09c52d94015") pod "my-pod" (UID: "82b92b9e-b770-4122-b662-fd5824282e33") DevicePath """ pod="default/my-pod"
Sep 09 16:47:11 risc-verse-k8sw-02 kubelet[3240779]: I0909 16:47:11.124724 3240779 operation_generator.go:632] "MountVolume.WaitForAttach succeeded for volume "pvc-e43dde24-fb36-4b0d-b59e-e09c52d94015" (UniqueName: "kubernetes.io/csi/driver.longhorn.io^pvc-e43dde24-fb36-4b0d-b59e-e09c52d94015") pod "my-pod" (UID: "82b92b9e-b770-4122-b662-fd5824282e33") DevicePath "csi-d92e28e3baf54e983b41251882fc605f20e1223b9d1e8186f27e802602fa7d06"" pod="default/my-pod"
Sep 09 16:47:11 risc-verse-k8sw-02 kubelet[3240779]: E0909 16:47:11.124948 3240779 nestedpendingoperations.go:348] Operation for "{volumeName:kubernetes.io/csi/driver.longhorn.io^pvc-e43dde24-fb36-4b0d-b59e-e09c52d94015 podName: nodeName:}" failed. No retries permitted until 2024-09-09 16:47:43.124933662 +0800 CST m=+1504181.615999143 (durationBeforeRetry 32s). Error: MountVolume.MountDevice failed for volume "pvc-e43dde24-fb36-4b0d-b59e-e09c52d94015" (UniqueName: "kubernetes.io/csi/driver.longhorn.io^pvc-e43dde24-fb36-4b0d-b59e-e09c52d94015") pod "my-pod" (UID: "82b92b9e-b770-4122-b662-fd5824282e33") : kubernetes.io/csi: attacher.MountDevice failed to create newCsiDriverClient: driver name driver.longhorn.io not found in the list of registered CSI drivers

@qww-ygg
Can you check the health of longhorn-csi-plugin pod in the namespace longhorn-system?
It is the responsibility of that pod to register driver.longhorn.io CSI drivers with kubelet. The error driver name driver.longhorn.io not found in the list of registered CSI drivers might related to the healthy of that pods. Kubectl describe it or getting the logs from it will be useful to troubleshooting

@qww-ygg
Can you check the health of longhorn-csi-plugin pod in the namespace longhorn-system?
It is the responsibility of that pod to register driver.longhorn.io CSI drivers with kubelet. The error driver name driver.longhorn.io not found in the list of registered CSI drivers might related to the healthy of that pods. Kubectl describe it or getting the logs from it will be useful to troubleshooting

The service status looks fine
# kubectl get pod -n longhorn-system -owide

longhorn-csi-plugin-78mqt                           3/3     Running   0             15h   <IP_ADDRESS>    risc-verse-k8sw-02   <none>           <none>
longhorn-csi-plugin-8764c                           3/3     Running   0             15h   <IP_ADDRESS>    risc-verse-k8sm-01   <none>           <none>
longhorn-csi-plugin-9mrp5                           3/3     Running   0             15h   <IP_ADDRESS>     risc-verse-k8sm-03   <none>           <none>
longhorn-csi-plugin-bh5tz                           3/3     Running   0             15h   <IP_ADDRESS>   risc-verse-k8sw-03   <none>           <none>
longhorn-csi-plugin-htzpx                           3/3     Running   0             15h   <IP_ADDRESS>   risc-verse-k8sw-05   <none>           <none>
longhorn-csi-plugin-lxbmr                           3/3     Running   0             15h   <IP_ADDRESS>    risc-verse-k8sw-04   <none>           <none>
longhorn-csi-plugin-vtgfl                           3/3     Running   0             15h   <IP_ADDRESS>     risc-verse-k8sm-02   <none>           <none>
longhorn-csi-plugin-x276q                           3/3     Running   0             15h   <IP_ADDRESS>   risc-verse-k8sw-01   <none>           <none>

# kubectl logs -n longhorn-system longhorn-csi-plugin-78mqt
Defaulted container "node-driver-registrar" out of: node-driver-registrar, longhorn-liveness-probe, longhorn-csi-plugin
I0909 08:33:58.607063       1 main.go:150] "Version" version="v1.12.0"
I0909 08:33:58.607245       1 main.go:151] "Running node-driver-registrar" mode=""
I0909 08:33:58.607250       1 main.go:172] "Attempting to open a gRPC connection" csiAddress="/csi/csi.sock"
I0909 08:34:08.608252       1 connection.go:253] "Still connecting" address="unix:///csi/csi.sock"
I0909 08:34:13.863643       1 main.go:180] "Calling CSI driver to discover driver name"
I0909 08:34:13.869204       1 main.go:189] "CSI driver name" csiDriverName="driver.longhorn.io"
I0909 08:34:13.869285       1 node_register.go:56] "Starting Registration Server" socketPath="/registration/driver.longhorn.io-reg.sock"
I0909 08:34:13.869562       1 node_register.go:66] "Registration Server started" socketPath="/registration/driver.longhorn.io-reg.sock"
I0909 08:34:13.869676       1 node_register.go:96] "Skipping HTTP server"

From the logs, looks like the registration has not been completed. This is an example when it is completed in my cluster:
I0828 22:59:33.540224       1 main.go:150] "Version" version="v2.11.1"
I0828 22:59:33.540334       1 main.go:151] "Running node-driver-registrar" mode=""
I0828 22:59:33.540342       1 main.go:172] "Attempting to open a gRPC connection" csiAddress="/csi/csi.sock"
2024-08-28T22:59:34.543174530Z I0828 22:59:34.542972       1 main.go:180] "Calling CSI driver to discover driver name"
2024-08-28T22:59:34.547456823Z I0828 22:59:34.547305       1 main.go:189] "CSI driver name" csiDriverName="driver.longhorn.io"
2024-08-28T22:59:34.547493814Z I0828 22:59:34.547366       1 node_register.go:56] "Starting Registration Server" socketPath="/registration/driver.longhorn.io-reg.sock"
2024-08-28T22:59:34.547690435Z I0828 22:59:34.547604       1 node_register.go:66] "Registration Server started" socketPath="/registration/driver.longhorn.io-reg.sock"
2024-08-28T22:59:34.547824226Z I0828 22:59:34.547777       1 node_register.go:96] "Skipping HTTP server"
2024-08-28T22:59:35.471963522Z I0828 22:59:35.471827       1 main.go:96] "Received GetInfo call" request="&InfoRequest{}"
2024-08-28T22:59:35.509345118Z I0828 22:59:35.509172       1 main.go:108] "Received NotifyRegistrationStatus call" status="&RegistrationStatus{PluginRegistered:true,Error:,}"

It seems that one of the following must be happening:

The node-driver-registrar container fails to create the registration socket at /var/lib/kubelet/plugins_registry/driver.longhorn.io-reg.sock. (This seems unlikely, since it would probably yield an obvious error in the logs we've already seen.)
Somehow the container and host don't share the same view of /var/lib/kubelet/plugins_registry/driver.longhorn.io-reg.sock, so kubelet doesn't see the registration socket.
The kubelet is looking in a different location than expected for device plugin registration sockets.
There is some kubelet error preventing it from attempting to communicate with the Longhorn node-driver-registrar over the socket. (Probably this one would be seen in the kubelet logs.)

Can we check for /var/lib/kubelet/plugins_registry/driver.longhorn.io-reg.sock from the host's perspective?

It seems that one of the following must be happening:

The node-driver-registrar container fails to create the registration socket at /var/lib/kubelet/plugins_registry/driver.longhorn.io-reg.sock. (This seems unlikely, since it would probably yield an obvious error in the logs we've already seen.)
Somehow the container and host don't share the same view of /var/lib/kubelet/plugins_registry/, so kubelet doesn't see the registration socket.
The kubelet is looking in a different location than expected for device plugin registration sockets.
There is some kubelet error preventing it from attempting to communicate with the Longhorn node-driver-registrar over the socket. For example, a permissions issues, etc. (Probably this one would be seen in the kubelet logs.)

Can we check for /var/lib/kubelet/plugins_registry/driver.longhorn.io-reg.sock from the host's perspective?

Thank you, I just restarted kubelet and the service has been restored to normal


404 File not found

When I do a pull, I get this error:
err: download: failed for url "". StatusCode: 404
This is for every file in my google drive. This was working a month ago or so. Running Debian Jessie. Any idea why this is happening? I can download the files from the web client just fine

Hello there @mhollands, thanks for reporting this issue and welcome to drive.
Google shutdown /host/fileId hosting at the end of August and we patched that to change the origin.  Please get the latest by
$ go get -u -v github.com/odeke-em/drive/drive-gen && drive-gen

This issue is a duplicate of closed ones https://github.com/odeke-em/drive/issues/776 https://github.com/odeke-em/drive/issues/766 and https://github.com/odeke-em/drive/issues/764.
I believe you are using an old version of drive.

Awesome! Thanks, I have now updated.
I thought that I had updated it, but turns out the update failed because I also hadn't updated my Go! Thanks for supporting this awesome tool - really is exactly what I need, instead of the annoying auto-syncing that google drive normally does.

Good to know it worked for you!
Thank you for the kind words! Yap, lots of people share your woes about auto sync, myself included :)
I'll close this issue then since the update solved it for you.

@mhollands worth mentioning that we have packages made for some distros already. Be sure to look at https://github.com/odeke-em/drive/blob/master/platform_packages.md so that you may be able to auto update drive.


Add isOutputRedirected parameter to GetFormatStyleString() method

PR Summary
Based on this feedback https://github.com/PowerShell/PowerShell/pull/13758#pullrequestreview-550380840, instead of checking if output is redirected within the engine, made it a parameter so that the console host and pass that information (and other hosts can do the same).
Also fixed an error in $PSStyle.Formatting formatting.
PR Checklist

[x] PR has a meaningful title

Use the present tense and imperative mood when describing your changes

[x] Summarized changes
[x] Make sure all .h, .cpp, .cs, .ps1 and .psm1 files have the correct copyright header
[x] This PR is ready to merge and is not Work in Progress.

If the PR is work in progress, please add the prefix WIP: or [ WIP ] to the beginning of the title (the WIP bot will keep its status check at Pending while the prefix is present) and remove the prefix when the PR is ready.

Breaking changes

[x] None
OR
[ ] Experimental feature(s) needed

[ ] Experimental feature name(s):

User-facing changes

[x] Not Applicable
OR
[ ] Documentation needed

[ ] Issue filed:

Testing - New and feature

[x] N/A or can only be tested interactively
OR
[ ] Make sure you've added a new test if existing tests do not effectively test the code changed

Tooling

[x] I have considered the user experience from a tooling perspective and don't believe tooling will be impacted.
OR
[ ] I have considered the user experience from a tooling perspective and enumerated concerns in the summary. This may include:

Impact on PowerShell Editor Services which is used in the PowerShell extension for VSCode (which runs in a different PS Host).
Impact on Completions (both in the console and in editors) - one of PowerShell's most powerful features.
Impact on PSScriptAnalyzer (which provides linting & formatting in the editor extensions).
Impact on EditorSyntax (which provides syntax highlighting with in VSCode, GitHub, and many other editors).

The PR should close #14387

The markdown test failed due to a broken link, which has nothing to do with this PR.

@daxian-dbw I think we can defer making those APIs public util we have a host that needs it

@SteveL-MSFT The host of PowerShell Jupyter kernel will need them. Once .NET interactive adopts PowerShell 7.2, it will be a matter of time for users to open the issue about $PSStyle not working in the PowerShell sub-kernel.


patch label in sidebar is wrapper

What happens?

What did you expect to happen?
no wrapping
How can we reproduce the issue?
seems like it's reproducible on the default swagger editor setup
Swagger/OpenAPI Example
https://sandbox.scalar.com/e/N2Uai

This doesn’t really happy for me on macOS, but font rendering is complex. I tried to improve the styling in #872. Are you able to check out the branch and test it? This is how you can

git@github.com:scalar/scalar.git
cd scalar
git checkout style/no-wrap-for-method-badges
pnpm install
make up
http://localhost:5050/api-reference


谷歌浏览器无法建立连接

 我的谷歌浏览器版本为：版本 90.0.4408.0（正式版本）dev （64 位）
 当采用ion-sdk-js进行访问连接时，ion-sfu总是连接失败：
 `[2021-02-07 14:02:02.568] [INFO] [146][peer.go][Join] => peer ckkuqoe4w000ehc98b9chkkkl join session test_session

[2021-02-07 14:02:02.790] [DEBUG] [63][publisher.go][func3] => ice connection state: checking
[2021-02-07 14:02:02.790] [INFO] [153][peer.go][Join] => peer ckkuqoe4w000ehc98b9chkkkl send answer
[2021-02-07 14:02:02.790] [DEBUG] [127][peer.go][func3] => on publisher ice candidate called for peer ckkuqoe4w000ehc98b9chkkkl
[2021-02-07 14:02:02.790] [DEBUG] [127][peer.go][func3] => on publisher ice candidate called for peer ckkuqoe4w000ehc98b9chkkkl
[2021-02-07 14:02:02.791] [DEBUG] [127][peer.go][func3] => on publisher ice candidate called for peer ckkuqoe4w000ehc98b9chkkkl
[2021-02-07 14:02:02.791] [DEBUG] [127][peer.go][func3] => on publisher ice candidate called for peer ckkuqoe4w000ehc98b9chkkkl
[2021-02-07 14:02:03.040] [DEBUG] [100][peer.go][func1] => peer ckkuqoe4w000ehc98b9chkkkl negotiation needed
[2021-02-07 14:02:03.210] [INFO] [109][peer.go][func1] => peer ckkuqoe4w000ehc98b9chkkkl send offer
[2021-02-07 14:02:03.210] [DEBUG] [115][peer.go][func2] => on subscriber ice candidate called for peer ckkuqoe4w000ehc98b9chkkkl
[2021-02-07 14:02:03.210] [DEBUG] [115][peer.go][func2] => on subscriber ice candidate called for peer ckkuqoe4w000ehc98b9chkkkl
[2021-02-07 14:02:03.210] [DEBUG] [115][peer.go][func2] => on subscriber ice candidate called for peer ckkuqoe4w000ehc98b9chkkkl
[2021-02-07 14:02:03.210] [DEBUG] [115][peer.go][func2] => on subscriber ice candidate called for peer ckkuqoe4w000ehc98b9chkkkl
[2021-02-07 14:02:03.213] [INFO] [190][peer.go][SetRemoteDescription] => peer ckkuqoe4w000ehc98b9chkkkl got answer
[2021-02-07 14:02:03.213] [DEBUG] [57][subscriber.go][func1] => ice connection state: checking
[2021-02-07 14:02:32.888] [DEBUG] [63][publisher.go][func3] => ice connection state: failed
[2021-02-07 14:02:32.888] [DEBUG] [68][publisher.go][func3] => webrtc ice closed for peer: ckkuqoe4w000ehc98b9chkkkl
[2021-02-07 14:02:32.889] [DEBUG] [63][publisher.go][func3] => ice connection state: closed
[2021-02-07 14:02:32.889] [DEBUG] [68][publisher.go][func3] => webrtc ice closed for peer: ckkuqoe4w000ehc98b9chkkkl
[2021-02-07 14:02:33.323] [DEBUG] [57][subscriber.go][func1] => ice connection state: failed
[2021-02-07 14:02:33.323] [DEBUG] [66][once.go][doSlow] => webrtc ice closed for peer: ckkuqoe4w000ehc98b9chkkkl
[2021-02-07 14:02:33.323] [DEBUG] [57][subscriber.go][func1] => ice connection state: closed`

测试ok
版本 90.0.4412.3（正式版本）dev (x86_64)
请检查是否开启了VPN？

关闭vpn


Clean up selenium after itself. Also report start up errors properly

Errors from seleniumStart were not reported properly (the relevant promise was already resolved), it would always time out.
This change also makes sure the selenium server is killed after the script run. Otherwise, I have to kill it manually after every run.
Previously we manually checked for the "Selenium Server is up and running" message. As far as I can tell, this is not needed anymore. I am guessing selenium-standalone has been updated?

Previously we manually checked for the "Selenium Server is up and running" message. As far as I can tell, this is not needed anymore. I am guessing selenium-standalone has been updated?

I have a feeling this was a Windows-only required thing. As such I'm going to hold off on merging this until I get back home to my Windows machine to test it. If so I'll be sure to add a comment this time around.

Works for me on windows 8.1

Tested and merged as 12072c7b6f7dddac7435ee8cb9956e3d1ac78b03; thanks!


Can't plot adjacency matrix of subgraph

Hi there,
Just getting started with Graphein and so far it's awesome, the docs are very helpful too -- my issue is with trying to plot a subgraph from a protein graph g.
Not sure if I'm doing something wrong, but I cannot plot the subgraph s_g using the adjacency matrix.  I get the following error from plotly:
ValueError: The length of the y vector must match the length of the first dimension of the img matrix.

It seems to be the fact that my subgraph has 7 nodes, but the adjacency matrix still has all 760 nodes from the original protein.
Not sure why this is the case since I am using the recompute_distmat option?
My code:
k = 3
s_g = extract_k_hop_subgraph(g, central_node=phos_site, k=k, recompute_distmat=True)

print(s_g) # 7 nodes
print(s_g.graph['dist_mat']) # all original 760 nodes?

fig = plot_distance_matrix(s_g)  # this line gives the error
fig.write_html("visual.html")

Thanks in advance, and apologies if i'm missing something.

could you true using filter_dataframe=True and maybe update_coords=True too?

Figured it out, typo here: https://github.com/a-r-j/graphein/blob/0a2d5e39787cf002c06b03615d9dd3fe62e0171d/graphein/protein/subgraphs.py#L70
should be "dist_mat", not "distmat". Have pushed a fix to master.

Hey @a-r-j , yep I figured the same thing but was waiting to hear back from you -- I actually modified visualisation.py to use g.graph["distmat"] as I wasn't sure what behaviour you wanted (i.e. the original adjacency matrix should still be accessible from a subgraph?)
Anyway, thanks for the fix :)

@a-r-j  another issue is when the subgraph is plotted, the node labels are assigned to the x and y axes randomly (but the actual distance matrix cols / rows are plotted in sequence order).  At the moment i've fixed this using the following:
# in graphein.protein.visualisation.plot_distance_matrix

x_range = list(g.nodes)
x_range = sorted(x_range, key=lambda x: int(x.split(':')[-1]))

Hope that makes sense!

Good spot. Seems like the start of a solution, but I think your solution breaks down with multichain proteins, no?

Yep, go ahead with a PR. Happy to review & support :)
That's an interesting use case you've described. I think it would be a good addition. The plot functions accept a distance matrix (separately from a graph) which I anticipated could be used for this sort of thing. However, if this is something that would be really useful for you it's probably useful for others and so we should support it.

Hiya,
I’m away at the moment but I’ll check this out for you in next few days.
In the meantime could you true using filter_dataframe=True and maybe update_coords=True too?
Sent from mobile

On 5 Jun 2022, at 15:55, Cam @.***> wrote:

Hi there,
Just getting started with Graphein and so far it's awesome, the docs are very helpful too -- my issue is with trying to plot a subgraph from a protein graph g.
Not sure if I'm doing something wrong, but I cannot plot the subgraph s_g using the adjacency matrix. I get the following error from plotly:
ValueError: The length of the y vector must match the length of the first dimension of the img matrix.
It seems to be the fact that my subgraph has 7 nodes, but the adjacency matrix still has all 760 nodes from the original protein.
Not sure why this is the case since I am using the recompute_distmat option?
My code:
k = 3
s_g = extract_k_hop_subgraph(g, central_node=phos_site, k=k, recompute_distmat=True)
print(s_g) # 7 nodes
print(s_g.graph['dist_mat']) # all original 760 nodes?
fig = plot_distance_matrix(s_g)  # this line gives the error
fig.write_html("visual.html")
Thanks in advance, and apologies if i'm missing something.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.


Keyboard Non-US Slash Bar key inconsistent across platforms

The Keyboard Non-US Slash Bar (USB HID Usage Name) key seems to be mapped to different key tokens across platforms - depending on the platform, it's either GLFW_KEY_WORLD_1 or GLFW_KEY_WORLD_2, making the distinction between the two arbitrary.

macOS: GLFW_KEY_WORLD_1 (kVK_ISO_Section = 10)
Windows: GLFW_KEY_WORLD_2 (86)
X11: GLFW_KEY_WORLD_1 (LSGT = 94 with evdev keymap)
Wayland: GLFW_KEY_WORLD_2 (KEY_102ND = 86 (94 when translated to xkb))

(I could not confirm this is the case on macOS, but the accepted comment to this Apple Community post says the "±/§" key is USB Usage ID 100, which is Keyboard Non-US Slash Bar).
Not sure if this is solvable at all without breaking existing keybinding configuration of these keys for API consumers. We'd either need to move Windows and Wayland to use GLFW_KEY_WORLD_1 for the key (seems more logical for the first one to be used, and the second deprecated), or X11 and macOS to use GLFW_KEY_WORLD_2 (might break less configurations).
I'd be happy to work on this after it's decided what should be done. Worst case, it'd need to be part of the breaking 4.0 release.

Thank you for the detailed bug report! I was going to make an issue for this bug for 3.5 but don't have anything to add to your description.


Exporting project

Hello,
Thank  you for your project who helped me a lot. Now that my model works properly i'm trying to export it into another app, unfortunatly, i'm facing some problems. All the files a compiled but the app cannot be launch. The following issues are thrown by xcode:
ld: warning: ignoring file /Users/actimage/Documents/Actelin/Actelin/caffe/lib/libprotoc.a, file was built for archive which is not the architecture being linked (armv7): /Users/actimage/Documents/Actelin/Actelin/caffe/lib/libprotoc.a
Undefined symbols for architecture armv7:
  "_cblas_dcopy", referenced from:
      void caffe::caffe_cpu_scale<double>(int, double, double const*, double*) in math_functions.o
  "_cblas_scopy", referenced from:
      void caffe::caffe_cpu_scale<float>(int, float, float const*, float*) in math_functions.o
  "_cblas_sasum", referenced from:
      float caffe::caffe_cpu_asum<float>(int, float const*) in math_functions.o
  "_cblas_dgemm", referenced from:
      void caffe::caffe_cpu_gemm<double>(CBLAS_TRANSPOSE, CBLAS_TRANSPOSE, int, int, int, double, double const*, double const*, double, double*) in math_functions.o
  "_cblas_sgemv", referenced from:
      void caffe::caffe_cpu_gemv<float>(CBLAS_TRANSPOSE, int, int, float, float const*, float const*, float, float*) in math_functions.o
  "_cblas_sscal", referenced from:
      void caffe::caffe_scal<float>(int, float, float*) in math_functions.o
      void caffe::caffe_cpu_scale<float>(int, float, float const*, float*) in math_functions.o
      cblas_saxpby(int, float, float const*, int, float, float*, int) in math_functions.o
  "_cblas_daxpy", referenced from:
      void caffe::caffe_axpy<double>(int, double, double const*, double*) in math_functions.o
      cblas_daxpby(int, double, double const*, int, double, double*, int) in math_functions.o
  "_cblas_ddot", referenced from:
      double caffe::caffe_cpu_strided_dot<double>(int, double const*, int, double const*, int) in math_functions.o
  "_cblas_saxpy", referenced from:
      void caffe::caffe_axpy<float>(int, float, float const*, float*) in math_functions.o
      cblas_saxpby(int, float, float const*, int, float, float*, int) in math_functions.o
  "_cblas_dgemv", referenced from:
      void caffe::caffe_cpu_gemv<double>(CBLAS_TRANSPOSE, int, int, double, double const*, double const*, double, double*) in math_functions.o
  "_cblas_dscal", referenced from:
      void caffe::caffe_scal<double>(int, double, double*) in math_functions.o
      void caffe::caffe_cpu_scale<double>(int, double, double const*, double*) in math_functions.o
      cblas_daxpby(int, double, double const*, int, double, double*, int) in math_functions.o
  "_cblas_sgemm", referenced from:
      void caffe::caffe_cpu_gemm<float>(CBLAS_TRANSPOSE, CBLAS_TRANSPOSE, int, int, int, float, float const*, float const*, float, float*) in math_functions.o
  "_cblas_dasum", referenced from:
      double caffe::caffe_cpu_asum<double>(int, double const*) in math_functions.o
  "_cblas_sdot", referenced from:
      float caffe::caffe_cpu_strided_dot<float>(int, float const*, int, float const*, int) in math_functions.o
ld: symbol(s) not found for architecture armv7
clang: error: linker command failed with exit code 1 (use -v to see invocation)

Any idea?

Xcode 7 changed its linker rules and it causes linker error with static libraries.
So please try it on Xcode 6.

Ok, but it's working fine with your sample project. I think that i'm missing something, all the errors are about math_functions.

Did you add Generate protobuf files in Build Rules section?
Please also check what's missing in the Build Phases and Build Settings of the project.

Thank you, I've just checked and everything seems right. Generate protobuf files is in Build Rules. Build Phases and Build settings are identical to sample-project. I've noticed something but i'm not sure if it matters.

It's hard to find what's missing.

I forgot to use Accelerate...

Is there anyway that I can get this working on xcode 7?

It's working on Xcode 7. I suggest you to open a new issue and explain your problem.

@thomasnierenactimage @thomasnieren I am getting a very similar error to what is stated above.
Xcode complains that the libprotoc.a was built for a different architecture.


5-pin sensor to Raspberry Pi?

The sensor I purchased has 5 pins, not 4 as in your diagram.  The 5th pin is labeled on the sensor as INT and the description online says "Interrupt output, connects to I/O pin".  The other 4 pins are connected per your specification (pin 1 "VCC" to RPI pin 4, pin 2 "GND" to RPI pin 6, pin 3 "SDA" to RPI pin 3, pin 4 "SCL" to RPI pin 5) and I've left this 5th wire disconnected.  The brand is CQRobot 3D Gesture Sensor, purchased from the major online retailer here in the US.
I haven't had any luck getting the sensor to respond yet and I'm not sure if I just don't have it configured correctly or if the 5th pin being disconnected could be preventing the sensor from working?  The documentation on the manufacturer's website isn't clear whether it has to be connected, as the diagrams show the wire not going to any particular pin, not sure if that means that it's optional to connect or just not depicted because there are multiple options.  Thoughts?

I think the chipset itself might be compatible, but the connection is different so the main script should be rewritten. Unfortunately, I don’t have experience with that sensor, so cannot help.

Sorry for the delayed response, I was on travel.  After my original post I discovered the test script (print.py) for the sensor and it worked beautifully with this sensor.  I discovered that the orientation of the sensor was 90 degrees off, so I adjusted that in the Config file, but still no luck.  That makes me hopeful that this sensor will still work and I just need to figure out what's up with the settings.  I'll dig deeper into the previous issues and try to find a solution.

I think  gestureMapFromTo would be for that purpose.


New command for scanning for PII information

Type of Change

[ ] Bug fix (non-breaking change, fixes #)
[X] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[X] Pester test is included
[ ] If new file reference added for test, has is been added to github.com/sqlcollaborative/appveyor-lab ?
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Purpose
The commands goes through a database and will based on known names and patterns asses if any of the columns in the database potentially contains PII information.
Approach
It's a new command
Commands to test
Invoke-DbaDbPiiScan

:bananadance:

ohh one round o passes!

sorry about the failure - looks like we may have to move toa different clone depth boo https://www.appveyor.com/docs/how-to/repository-shallow-clone/#setting-depth-of-git-clone-command

Looks bomb af! For those of you wondering:


Schemas

Adding schemas for transactions for serialization of request and response of Fastify
Related to https://github.com/frmscoe/General-Issues/issues/222

Parked until confirmed to be best way of doing this. @rtkay123 to unpark/approve


Read an account from the service

As a Account manager
I need a read route in the REST API
So that I can read an account from the database
Details and Assumptions
* REST API is already developed, but read method has to be implemented
* The service is programmed with the Flask framework

Acceptance Criteria
gherkin
Given I submit a GET request to the API
When I provide the account id of a customer
Then the service should return the information about the customer related to that account id

Finished: https://github.com/ccgalindog/devops-capstone-project/pull/9