I could not find a link to download minified version in your homepage.

Hi there, I was looking for a simple way to download ractive minified version without downloading the repo and running npm run build. I was expecting a min version link, but only found the non minified version throught your CDN: http://cdn.ractivejs.org/latest/ractive.js.
Doing some research in internet I came upon this page which has all versions for your builds:
http://spmjs.io/package/ractive
Which is awesome, but I think you still should have these links in your web home page for those who are not in touch with node.js

Here is the minified file: http://cdn.ractivejs.org/latest/ractive.min.js
I do agree that there should be a clear description of all the different available builds somewhere on the website, though!

I've created an issue at the ractivejs.org repo, so I'm going to close this issue.


Option to handle async actions

Added support to thunk actions, now function will check if payload is a function it will add getState function to it

I think this is somewhat out-of-scope of this library - have you tried the example at https://redux-actions.js.org/docs/middleware/index.html?

I think this is somewhat out-of-scope of this library

+1
Closing this for now.


[Request] Can you please add bar plots features?

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
=> Current library does not contain Bar plot modules.
Describe the solution you'd like
A clear and concise description of what you want to happen.
=> Need bar plotting module, which includes stacked bar plots, etc.

I felt that the Bar Plot, which is basically provided by Splunk, has a lot of restrictions on visual expression. I asked if it is possible to implement this function because there are some functional parts that are not working well and the plot is much better visually. (ex. Bar Width Size Control, Font setting, etc.)
REF: https://community.splunk.com/t5/Dashboards-Visualizations/Chart-Bar-Chart-Configuring-Questions-Fonts-and-Formats/m-p/654599#M53841

Hi @euisuk-chung ,
If you need more customisations you can always extend the simpleXML code of your dashboard with CSS and JS. You got some examples as replies in the Splunk Answer you reported. 😉
Unfortunately the main object of this visualization is not offering "clones" of visualizations already available out-of-the-box by Splunk, but to offer new ones.


"A route without a path is always a match." ?

The Readme on Routes says

A route without a path is always a match.

I do not understand hat.

@urbanhop It's confusing, I agree. It means, however, that if you omit the path attribute, the route will always match, meaning the component specified via the render attribute will be rendered.
The following route is rendered oInly if path equals "/".
<Route path="/" render={Home} />

The following route is always rendered.
<Route render={Home} />


[local_auth] updated canCheckBiometrics behavior

On Android canCheckBiometrics returns true if the device doesn't have any biometric enrolled (For more info check https://github.com/flutter/flutter/issues/45497#issuecomment-1007310214). Now with this PR canCheckBiometrics returns false on Android if device has the biometric hardware but doesn't have any biometric enrolled.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the relevant style guides and ran the auto-formatter. (Unlike the flutter/flutter repo, the flutter/plugins repo does use dart format.)
[x] I signed the CLA.
[x] The title of the PR starts with the name of the plugin surrounded by square brackets, e.g. [shared_preferences]
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated pubspec.yaml with an appropriate new version according to the pub versioning philosophy, or this PR is exempt from version changes.
[x] I updated CHANGELOG.md to add a description of the change, following repository CHANGELOG style.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] All existing and new tests are passing.

If you need help, consider asking for advice on the #hackers-new channel on Discord.

Note that this PR will require native unit tests for the Java changes, since that's the important part of the PR.

Note that this PR will require native unit tests for the Java changes, since that's the important part of the PR.

No problem I will do these changes into the java code including native tests

@darwin-morocho Are you still planning on updating this PR?

@stuartmorgan I've closed this PR in favor to this new PR https://github.com/flutter/plugins/pull/4874


Find by regular expression with backreference - not working

Hi, i have two files with these strings
getTemplate('aa') and getTemplate("aa")
and backreference not works. Im trying $1, $0 , {$1}, \1 etc...
getTemplate\(('|")aa$1\)
Can you tell me how is correct syntax in VSC? Is it supported?

You need to use \1 instead of \\1

@IllusionMH Excellent! Thx.

Does it resolve problem?
Please close this issue if it's resolved.

@IllusionMH yes! Where can i find more info about these advanced things?
I will close immeadiatelly (tomorrow)

Find widget uses JS RegExp engine to perform search in single file.
Global search uses ripgrep under the hood.

OK, thx!


cmd/sync: fix

Welcome to Codecov :tada:
Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Issue #66

Following issue #66 I added a condition to check if a shadow root is already attached to the element and prevent the exception (https://developer.mozilla.org/en-US/docs/Web/API/Element/attachShadow#exceptions). I tested it with our apps and it seems to work fine.
I don't know if the shadow root already being attached should be handled a different way like removing and recreating it or any other action. But this check seems to work, the import-map ui is rendered and styled correctly on first load and refresh.

@joeldenning Are you able to merge this? 🙏

Is there any update on when this PR is going to be merged?

An update on this would be super appreciated, thank you!

I am seeing this error as well. If this PR isn't an appropriate fix, can we get some direction on how to correctly fix the issue?

@joeldenning Are you able to merge this? 🙏

Closed in favor of #86


README: Correct missing closing tag

What kind of change does this PR introduce?
Readme update syntax error fix

thanks @OrhanTozan 👍

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Update for react 17+

Im starting to get problems with Flex functions, as most of my dependancies require start requiring react 17. Specifically NextJS require react 16.9... and rebass requires react 16.8...
Is there simple alternative for rebass/reflexbox? I use emotion for JSS
I was thinking to go Material UI road but that would require a lot of changes.

Have you found something to replaced it?

Facing the same issue. I am unable to install reflexbox npm i reflexbox. Its throwing the following error,
npm ERR! Found<EMAIL_ADDRESS>npm ERR! node_modules/react
npm ERR<EMAIL_ADDRESS>from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer<EMAIL_ADDRESS>from<EMAIL_ADDRESS>npm ERR! node_modules/reflexbox
npm ERR!   reflexbox@"*" from the root project

I was wondering about this too.

any update in updating  rebass for react 18?

So...rebass is dead?

same error here, some update on this?


Respect comments between imports

They could be used to organize imports
Maybe some kind of a "group system" (with specific comments to group imports)
Or just ignore comments and place them again.

Due to the fact, that comments are ignored by the typescript  compiler there has to be another method of creating a "group system"


SteamOS ASCII

Would be nice to have an ASCII logo on my Steam Deck besides the tux

I tried making one, what do you think of this?

awesome!

I added some colors, is this good or should I leave it white?
The actual SteamOS logo looks like this:

purple is definitely better

heres the neofetch SteamOS logo for reference (taken from https://github.com/hykilpikonna/hyfetch)

Ok, thank you for your feedback :) I will leave it as is then

the only improvement I would make is to try and make it smaller, but idk how you would even do that, or if its possible without making compromises.

Yea it would be nice if it would be smaller, but I think this is pretty much the minimum to not lose too much detail. If you enable more pfetch output, it looks pretty good at its current size:


Integrate with Telescope buffers to show pinned buffers

This is probably a stretch, but I would love to be able to show when buffers are pinned when using Telescope buffers.

I was looking at the source for it, but I think it'd basically involve re-implementing it: https://github.com/nvim-telescope/telescope.nvim/blob/40c31fdde93bcd85aeb3447bb3e2a3208395a868/lua/telescope/builtin/__internal.lua#L868-L938C1

Thanks for opening an issue. @al-ce is already working on this #5

This is now in the latest release
https://github.com/axkirillov/hbac.nvim/releases/tag/v1.2.0
Just call Hbac telescope and you get the buffer picker experience + pin state and mappings to toggle pin state / delete buffer etc.

Be free to give us any feedback on whether it works as you expect it and if there is any friction!

That is great news, thanks!  Updated, and gave it a try. Some initial impressions:

having icons for both pinned and unpinned is kinda confusing. I think it's fine to leave it blank for unpinned.
I'm not sure what order it's showing them in, or if it's configurable at all

For comparison, I use these options for buffer:
			pickers = {
				buffers = {
					sort_lastused = true,
					sort_mru = true,
					ignore_current_buffer = true,
				},
			}

The end result of those is that it hides the buffer I'm currently in, and then default selected item is the most recent buffer.

Thanks for giving it a try and for the feedback @technicalpickles. Yes, @ditsuke pointed out the sorting / order issue as we were closing it and we have plans to open a PR to address that. That's my fault for not checking where each of the possible opts belonged in different parts of the picker.
Fair point about the icons, we can consider that in future commits. In the meantime, try setting the unpinned icon to an empty string in the opts (you can also change their hl groups)
require("hbac").setup({
  telescope = {
    pin_icons = {
      unpinned = { "" },
    },
  },
})

Thanks again! I'll try to remember to ping you when there's progress on the sorting.

That is great news, thanks! Updated, and gave it a try. Some initial impressions:
* having icons for both pinned and unpinned is kinda confusing. I think it's fine to leave it blank for unpinned.

* I'm not sure what order it's showing them in, or if it's configurable at all

For comparison, I use these options for buffer:
			pickers = {
				buffers = {
					sort_lastused = true,
					sort_mru = true,
					ignore_current_buffer = true,
				},
			}

The end result of those is that it hides the buffer I'm currently in, and then default selected item is the most recent buffer.

Thank you for the suggestions! I think all of your points make a lot of sense, and I would love to see them in hbac.
I agree that having only an icon for the pinned state would probably be more straightforward.
As for the ordering opts you mentioned I think it's also a good idea for us to implement them.


Change Discover and Contribute URLs for Open Beauty Facts

Depends on #279

Added in strings. Needs conditional code


Warn about missing bind group / vertex buffer indices in can_draw

This is useful to guide developers to why their custom shaders / meshes may not
be drawing.
I think also adding some debug!() in DrawContext::set_bind_groups_from_bindings_internal() in the cases where bindings don't match up could be useful so that one can see the warning, bump the log level to debug, and then maybe get some insight as to what exactly is wrong there.

With this you can see a warning when there is a missing bind group or vertex buffer, including seeing its index which corresponds to the GLSL layout parameters. You can then re-run with RUST_LOG=debug,wgpu=warn cargo run 2>&1 | grep "BindGroupDescriptor { index : 1" if the index of the missing bind group is 1 and you will see the full BindGroupDescriptor that is not fulfilled.

This is super useful, but theres one problem we'll want to solve before merging (at least if we want these to be warnings, which i think we do)
When assets that ultimately become gpu bindings are still loading and a draw is attempted, the new warning triggers. In a sense, this is "correct" because we tried to draw and the binding wasn't set.
However this is a "natural" part of asset loading, and isn't something that should matters to users. "warnings" should always be actionable (or at the very least, be worth presenting to the user). But users can't really do anything about this / nothing is "wrong".
(try the load_gltf example for a particularly noisy case)

Closing as this will require significant changes to avoid creating noise for loading assets.


upstream: Introducing close_connections_on_host_set_change property

Signed-off-by: Kateryna Nezdolii<EMAIL_ADDRESS>Description: This change introduces property close_connections_on_host_set_change for common lb config (default value is false). If property is set, every time on host set change (hosts added or removed), TLS cluster managers will drain their corresponding http and tcp connection pools. Initially this property was intended to fix split brain issue in hash based load balancers. For more detailed description please see RFC: https://docs.google.com/document/d/1yX8qRDXfbcOqNwSpaptFk1ru-bY_yHDLLOVBuLaclR4/edit?usp=sharing
Apart from addition of new config property, new metric cluster_manager.upstream_connections_closed_on_host_set_change (counter) in stats has been added.
Risk Level: Medium/Low
Testing: Tests added for new functionality
Docs Changes: TBD
Release Notes: TBD
Fixes #6730

@nezdolik please let us know when this is ready for review. Thanks!

@dio will do, thanks

@mattklein123 think this is ready for review. Apologies for generating lots of noise with WIP PR. Documentation changes will be added later on once code change looks good.

/retest

@envoyproxy/api-shepherds API LGTM

/retest

I'm OOO for the rest of the week. I will take a look next week unless @snowp is able to take a look first. Thanks!

@snowp updated existing test to use Http::Protocol::Http2 to test draining of persistent connections.

@snowp thanks for review. Believe I addressed all the comments

Thanks @nezdolik will take a look today.

Thanks for suggestions, @mattklein123. I will be on short vacation starting today and will get back to working on this next Tuesday.

Messed up DCO check, will have to do force push :(

@htuch fixed api comments, thanks

@snowp any further comments? If not can you approve/merge?


Create ag.txt

Add Tartu Grownups Gymnasium

@hashdealer First stage of verification passed, official domain is ag.tartu.ee as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@hashdealer Thank you for your patience while waiting for a review and processing your request. Pull request merged.


Can this work with custom media queries?

I am using cssnext, which includes postcss-custom-media. Is there a way to get this plugin to work well with postcss-custom-media, such that the media() can be passed a custom media query?
@custom-media --break-lg (width >= 48em);

h1 {
  font-size: media(
    1em,
    (--break-lg) 1.125em
  )
}

Currently, min-width: 48em does not get properly substituted.

Yes, this works with custom media queries, and (min-width: 48px) would work (with parenthesis). 😄


don't use hf.SetHandleHttp(Handle)  in the main function

don't use hf.SetHandleHttp(Handle)
how to run Handle automatically? (is it even possible?)
or create an Init function (executed at every HTTP call)

use reflection: → performance issue
create a new Init function: one extra call (main is used at each HTTP Call → CreateWasmRuntimeAndModuleInstances)


L515  depth message has no output in ROS

Required Info

Camera Model
L515

ROS version
Melodic

Firmware Version
<IP_ADDRESS>

Operating System & Version
Ubuntu18.04

Kernel Version (Linux Only)
5.4.0-89-generic

Platform
PC

SDK Version
2.0

Language
C++/Python

Serial No.
f0264437

Product ID
0B64

Issue Description
When I was running the command:
roslaunch realsense2_camera rs_camera.launch align_depth:=true device_type:=l515 tf_prefix:=measured/camera
Everything was fine. Both RGB and depth messages were OK, but something went wrong yesterday, I can still receive the depth message, but they are empty. To be more precise, there is no problem at the beginning, but after I run the ROS node, the above-mentioned problems will occur after a minute or so.
I upload two videos of the whole process on youtube; they can show the problem more clearly. Here are the links: when I run the realsense-viewer, it works perfectly.
Then I started running the ROS node, something happened. In the second video, before 1:14, the depth message is OK but with some flicker, but after that, it has just gone.
And when I try to check the message by running the command: rostopic echo -n1 /camera/aligned_depth_to_color/image_raw, the output :
header:
  seq: 1844
  stamp:
    secs:<PHONE_NUMBER>
    nsecs: 586819887
  frame_id: "measured/camera_color_optical_frame"
height: 720
width: 1280
encoding: "16UC1"
is_bigendian: 0
step: 2560
data: [0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 153, 11, 153, 11, 153, 11, 153, 11, 153, 11, 153, 11, 153, 11, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0, 0,

They are all 0, too long to copy them all.
And no specific output information when an error occurs. The terminal is just keeping output the:
e, number: 61
 19/01 15:13:28,549 WARNING [139816150759168] (messenger-libusb.cpp:42) control_transfer returned error, index: 768, error: No data available, number: 61
 19/01 15:13:29,799 WARNING [139816150759168] (messenger-libusb.cpp:42) control_transfer returned error, index: 768, error: No data available, number: 61
 19/01 15:13:31,050 WARNING [139816150759168] (messenger-libusb.cpp:42) control_transfer returned error, index: 768, error: No data available, number: 61
 19/01 15:13:32,301 WARNING [139816150759168] (messenger-libusb.cpp:42) control_transfer returned error, index: 768, error: No data available, number: 61

According to some replies, they can be ignored.
Every time such an error occurs, I can only temporarily solve the problem by plugging and unplugging the USB cable, because the problem will happen again after a minute.
So does someone know how to handle such things?

Case closed as duplicate of https://github.com/IntelRealSense/realsense-ros/issues/2226


HOCS-5668: Add COMP PSU categories

Should this be in the COMP2 file too?

Yeah, my bad. Added now


include Office 365 Management (Mobile App) app id: 00b41c95-dab0-4487-9791-b9d2c32c80f2

please include Office 365 Management (appid: 00b41c95-dab0-4487-9791-b9d2c32c80f2) it's getting blocked because Conditional Access Policy cannot scope to it.

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

ID: c83d2402-0113-e453-e103-a1abc4876531
Version Independent ID: d638de2c-15f1-546d-05c0-06e3d7e62287
Content: Office 365 App in Conditional Access reference - Microsoft Entra ID
Content Source: docs/identity/conditional-access/reference-office-365-application-contents.md
Service: active-directory
Sub-service: conditional-access
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

Blocked in what sense? This article is just a listing of what is included in the app bundle that you can target with Conditional Access.

Blocked in what sense? This article is just a listing of what is included in the app bundle that you can target with Conditional Access.

apologies John if this is the wrong place. Conditional Access Policies applied to the "office 365" cloud apps is missing "Office 365 Management (appid: 00b41c95-dab0-4487-9791-b9d2c32c80f2)" to be included in it. Without that, Conditional Access will fail because the Office 365 Management app is not recognized as part of the office 365 app.
Additionally I cannot find that app to exclude in Conditional Access Policies, it doesn't show up in the list. This app is the one that comes from the iOS app Store
https://apps.apple.com/us/app/microsoft-365-admin/id761397963

@jku-sr
Thanks for your feedback! We will investigate and update as appropriate.

Ahh. That is not part of this app group as that is part of the admin side of the apps. I would look at the service dependency that exists when you try to launch the app. I would bet it is actually calling the Windows Azure Service Management API.
You can find more info in the following troubleshooting article.
https://learn.microsoft.com/entra/identity/conditional-access/troubleshoot-conditional-access#service-dependencies

#remove-label:"cxp"
#label:"assigned-to-author"

Ahh. That is not part of this app group as that is part of the admin side of the apps. I would look at the service dependency that exists when you try to launch the app. I would bet it is actually calling the Windows Azure Service Management API.
You can find more info in the following troubleshooting article. https://learn.microsoft.com/entra/identity/conditional-access/troubleshoot-conditional-access#service-dependencies

actually it looks like it's calling Microsoft Graph?

I would suggest trying policies for the Windows Azure Service Management API or Microsoft Admin Portals and see if those allow you access to the app.
If those don't work the app has the following page on Learn with a support email address About the Microsoft 365 Admin mobile app. They would probably be able to help you further.

#please-close

#remove-label:"cxp"
#label:"assigned-to-author"
#reassign:@MicrosoftGuyJFlo


CFD has opening fee set?

Where is this coming from:

Symbol: BtcUsd
Tested locally.

Can't find a single instance of sending an offer with an opening fee set on Grafana over the past 30 days.

Not a bug, it's just hard-coded when going through the maker UI:
https://github.com/itchysats/itchysats/blob/23408a1e9d773909282cc9a663e34ab872d5070e/maker-frontend/src/MakerApp.tsx#L293


chore(main): release 1.0.0

:robot: I have created a release beep boop
1.0.0 (2024-07-07)
Features

init (b6dda30)

Bug Fixes

update template (62b9e30)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

v1.0.0
:sunflower:


Migrate namespace from kedro_datasets.datasets to kedro_datasets

Signed-off-by: Nok Chan<EMAIL_ADDRESS>Description

Pre-requisite of https://github.com/kedro-org/kedro/issues/1711
Previously the namespace is incorrect. We need to update the namespace first.
Development notes

Checklist

[ ] Opened this PR as a 'Draft Pull Request' if it is work-in-progress
[ ] Updated the documentation to reflect the code changes
[ ] Added a description of this change in the relevant RELEASE.md file
[ ] Added tests to cover my changes

The 1 failing test on Python 3.10 can be ignored for now - it's not a new thing. It's a bit strange as we don't seem to have this problem in kedro. We should address this separately, maybe after we sync everything when we work on Release 1.0.0 and see if this problem persists.


Sidebar-only extensions?

I guess this is a feature request. It'd be nice to be able to have sidebar-only extensions — i.e., extensions that are not tied to a specific field.
I want to create a UI extension that generates a code snippet for our publishing team to copy/paste. The snippet will include auto-generated tracking links that include the entry title and other fields. This extension should live in the Contentful sidebar.
However, the only way to include this extension is to create a new field that uses the extension, even if that field isn't needed for anything. For example, the Contentful URL checker UI extension requires that you add a Boolean field to your content model, even though that field won't be used for anything.
So, would it be possible to make UI extensions that are not tied to a field, live in the sidebar, and have access to the entry being edited?
Thanks!
EDIT: I noticed that the SDK docs mention entry-level extensions that just live in the sidebar. This is basically what I'm referring to — but I don't see any way to make those without tying them to a specific field...?

Hi @jessepinho! Thanks for the feature request.
We'll work on that. The way it works now is a quite hacky, all people agree with that :)

Future Googlers: see https://www.contentful.com/developers/docs/extensibility/custom-sidebar/


Implement early exit

The current logic was waiting until the timeout was triggered which means if you set your timeout pretty high you have to wait for a pretty long time (even if all pods came up). This PR simply checks if all Pods came up and then leaves the loop.
Also I added a small fix for the cleanup functionality to also delete Pods that are created by the kboom job.

Super, super, you're on a roll @johscheuer :)
Thank you!
And we really need a proper AUTHORS doc here …


Update dep on JMH in JmhPlugin to 1.11

Fixes #60

Thanks for the PR!
I think it'll be hard to get more more dry than 1 line per version of: project, jmh :)

I'll release it in a bit today

I was referring to the duplication of JMH version numbers in build.sbt and JmhPlugin.scala. I'm presuming those versions should be the same, no?

Ah yes, that's correct should be fixed, I'll  open a ticket.


#19210 Adding an option to revert the sorting of columns

Fixing #19210
Adding an option to sort the columns according to the original order.

I assume the feature request is about reverting the sorting of the "columns" table only, excluding the "Keys", "References" etc. Therefore, we just need to sort it according to the ascending order of the "#" column.
My current implementation is just to traverse all columns, and if there is a column with the name "#", add a menu option "Revert sorting".
                	@Override
                	public void run() {
                       GroupingViewerColumnController.this.sortByColumn(finalNumberColumnInd, SWT.UP);
                	}

This is kinda ugly but works. But if we do not use this trick, we have to override the whole SortListener of ViewerColumnController. Please let me know if there is any better implementation.

Thank you for contribution.
Please note that according to the Contribution guide it's required to add the ticket number in the commit comment.

Thank you for contribution.
Please note that according to the Contribution guide it's required to add the ticket number in the commit message.

Modified accordingly.

SiyaoIsHiding, thank you for your contribution. But we discussed this issue and decided to fix it another way without adding an option to revert the sorting.
What we want to do:

When clicking on a column header, sort the column ASC
On a second click, sort it DESC
On the third click return the original order

This PR is stale now, so I'm closing it. Please, let me know if you'll continue working on this - we can reopen it.


Added spring cache configuration

Description
This PR includes the following proposed change(s):

FLA-66 - finishing Store submission information in the cache for a limited time

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Refactoring / Documentation

if your change is a breaking change, please add breaking change label to this PR
How Has This Been Tested?
Running the app locally
Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have made corresponding changes to the documentation
[x] New and existing unit tests pass locally with my changes

[ERROR] Errors:
[ERROR]   SubmissionApiImplTest.withValidPayloadShouldReturnOk:94 » NullPointer
[INFO]
[ERROR] Tests run: 23, Failures: 0, Errors: 1, Skipped: 0


Backup

Is the backup script enough for backup everything (looping through all volumes). What about the master server, all the metadata etc., e.g. scenario of loosing the master and all volumes?

please ask questions on the mailing list.

no need to backup master.
backup volumes on the volume server will be good enough.

OK, thanks.


Initial aspect ratio and colormaps for ImageView?

Hi,
Maybe this is documented but I can't find an answer.
I have a class inherited from ImageView. I'd like to keep the data aspect ratio and have a logarithmic color map from start, without having to click the buttons every time. How do I do this?
Here's the code,
https://github.com/alexbjorling/nanomax-analysis-utils/blob/master/apps/limaLiveViewer.py

Hi,
You can set data aspect ratio with: self.setKeepDataAspectRatio(True) (http://www.silx.org/doc/silx/0.5.0/modules/gui/plot/plot.html#silx.gui.plot.Plot.Plot.setKeepDataAspectRatio).
And you can set the default colormap to log scale with: self.setColormap(normalization='log') (http://www.silx.org/doc/silx/0.5.0/modules/gui/plot/imageview.html#silx.gui.plot.ImageView.ImageView.setColormap)
It not clear enough from the doc that ImageView also provides all functions from the PlotWidget.
self.setColormap(normalization='log') is working in last release (0.5.0) but it is borken in current github version, I'm fixing it now.

Ah, I was only looking one level up in the inheritance tree, at PlotWindow.
And thanks for the fix.

Most of the plotting API is defined in silx.gui.plot.Plot in 0.5.0.
It will be defined in silx.gui.plot.PlotWidget in 0.6.0 as we merge those 2 classes (this should be transparent for the user).

BTW #639 should be closed with all those features, shouldn't it?


Extend math macros to enable nested macros

Which area is this feature request for?
LaTeX Export
Describe the feature you'd like to request
Given the math macros:
'\aNrm': "a"
'\aMat': '[\mathrm{\aNrm}]'

When only \aMat is used in the document body, it seems that only the macro defining '\aMat': '[\mathrm{\aNrm}]' is added to the tex preamble, this means that the symbol \aNrm is still undefined and causes and error when \aMat is used.
Describe the solution you'd like
Check for nesting of macros and include all dependencies.
Describe alternatives you've considered
Or we could just add all the defined math macros to the document, even if they are not used.
[originally reported by @alanlujan91]
cc @rowanc1

Should be released later this week!! Thanks @alanlujan91 for the bug report.


Execute error

added detail message if the input is missing necessary placeholders
expose both the input and output node names for the model graph

This change is

Review status: 0 of 4 files reviewed at latest revision, all discussions resolved.

src/executor/graph_executor.ts, line 110 at r1 (raw file):

  }

  private checkInput(inputs: NamedTensorsMap) {

it would also be good to throw an error if you provide an input that doesn't exist as a placeholder

src/executor/graph_executor.ts, line 114 at r1 (raw file):

    const missing: string[] = [];
    this.placeholders.forEach(name => {
      if (inputKeys.indexOf(name) === -1) missing.push(name);

use {}

src/operations/operation_mapper.ts, line 36 at r1 (raw file):

const CONTROL_FLOW_OPS =
    ['Switch', 'Merge', 'Enter', 'Exit', 'NextIteration', 'LoopCond'];

is this meant to be in this PR?

src/operations/operation_mapper.ts, line 78 at r1 (raw file):

      map[node.name] = this.mapNode(node);
      if (this.isControlFlow(node)) withControlFlow = true;
      if (node.op === 'Placeholder') placeholders.push(map[node.name]);

use {}

Comments from Reviewable

Review status: 0 of 4 files reviewed at latest revision, 4 unresolved discussions, some commit checks failed.

src/executor/graph_executor.ts, line 110 at r1 (raw file):
Previously, nsthorat (Nikhil Thorat) wrote…
it would also be good to throw an error if you provide an input that doesn't exist as a placeholder

good call, added.

src/executor/graph_executor.ts, line 114 at r1 (raw file):
Previously, nsthorat (Nikhil Thorat) wrote…
use {}

the ts style guide allow single line to omit the braces.
https://g3doc.corp.google.com/javascript/typescript/g3doc/styleguide.md?cl=head#control-flow-statements-blocks
let me know if you think otherwise.

src/operations/operation_mapper.ts, line 36 at r1 (raw file):
Previously, nsthorat (Nikhil Thorat) wrote…
is this meant to be in this PR?

not for this cl, I will revert the change.

src/operations/operation_mapper.ts, line 78 at r1 (raw file):
Previously, nsthorat (Nikhil Thorat) wrote…
use {}

same as earlier.

Comments from Reviewable

Review status: 0 of 4 files reviewed at latest revision, 4 unresolved discussions, some commit checks failed.

Comments from Reviewable

I am trying to convert the frozen inference graph of DeepLab hosted at their model zoo.
The conversion was successful but when I load the model I get the following output :
Error importing the graph: block_shape must be positive for 'MobilenetV2/expanded_conv_7/depthwise/depthwise/SpaceToBatchND' (op: 'SpaceToBatchND') with input shapes: [1,?,?,384], [2], [2,2] and with computed input tensors: input[1] = <0 0>, input[2] = <[0 0][0 0]>. Process finished with exit code 2
The script which I use for loading the converted is :
graph_def = None
graph = None
model_file = "tensorflowjs_model.pb"

print('Loading graph definition ...', file=sys.stderr)
try:
    with tf.gfile.GFile(model_file, "rb") as f:
        graph_def = tf.GraphDef()
        graph_def.ParseFromString(f.read())
except BaseException as e:
    parser.exit(2, 'Error loading the graph definition: {}'.format(str(e)))

print('Importing graph ...', file=sys.stderr)
try:
    assert graph_def is not None
    with tf.Graph().as_default() as graph:  # type: tf.Graph
        tf.import_graph_def(
            graph_def,
            name=''
        )
except BaseException as e:
parser.exit(2, 'Error importing the graph: {}'.format(str(e)))


add go-gen-test emacs extension to readme

I create new emacs extension and put it to MELPA emacs packages repo.

Coverage remained the same at 95.737% when pulling f245b36407d0b48d5312664de77604b865163165 on s-kostyaev:add-new-emacs-extention into 0dfc8c37122d873525c2be0b32108a77d2c2aba8 on cweill:master.

Coverage remained the same at 95.737% when pulling 355311d5458a843c6a3e4c18e568025dc15a3b8b on s-kostyaev:add-new-emacs-extention into 0dfc8c37122d873525c2be0b32108a77d2c2aba8 on cweill:master.


update data-model to a stable release

as a developer, I want our dependencies to be stable.
Currently, the data-model-SNAPSHOT is ahead of the data-model. The changes in data-model-SNAPSHOT should be pushed to data-model and the semantic versioning incremented
ACCEPTANCE CRITERION
-- [X] data model published
-- [X] PDX Finder dev switched to use newest stable release.

@zperova  Done


Tracking pip support in RustPython

Yes, we are almost there. But still many issues are scattered on multiple projects outside of RustPython.
This issue is tracking the status of related issues.
Submitting issue or patch for each project will be very helpful, thank you!

[ ] windows pip requires ctypes #2671

[x] patch platformdirs: https://github.com/platformdirs/platformdirs/pull/275
[x] vendor updated platformdirs in pip: https://github.com/pypa/pip/pull/12670
[ ] New pip release

[ ] macos wheel requires ctypes

[x] patch wheel: https://github.com/pypa/wheel/pull/613
[ ] New wheel release

[ ] setuptools requires ctypes #2721

[x] patch platformdirs: https://github.com/platformdirs/platformdirs/pull/275
[ ] vendor updated platformdirs in setuptools
[ ] New setuptools release

hey @youknowone, thanks for your work on this project. I'm interested in using rustpython in an open source project i'm working on. would closing this issue allow projects using rustpython as a crate be able to dynamically execute python files with import statements just like a jupyter notebook? how do i help with this?

@cs50victor  Hi, RustPython already can import a lot of pure python packages while it is compatible to RustPython.
This issue is about installing a new package using pip command. This is not related to import step.
To help this issue, checking if the new release of those project included the patched code would be helpful. Once all of them are solved, I will update it and test them again.


Custom filtering or manage requests yourself possible?

I have read through the documentation and there are still two questions open for my use case.

I need additional filtering, a date range picker together with the search filter
I would need the filtered data separately  to show some statistics in graphs

I assume that this is not possible. However would it be possible to manage the data fetching myself based on listening for actions and then passing the data in explicitly?

@arddor I'm not quite sure what you mean. But is this what you're looking for?

I don't think so. If I understood the intention of the module right data fetching is done through it. Filtering is done through Search which does a textual search. What I would need is an additional filter field for time range.
Secondly the data which is fetched from vue-table I would need to access them for additional visualization in graphs, e.g. considering the http://vuetable.ratiw.net/examples/semantic.html I would like to show a Pie Chart showing percentage of male / female based on the items in the table.
To summarise it:

Can other filters apart from text search be defined?
Can I somehow access the data fetched from vue-table

@arddor Actually, all filtering are done from the server. vuetable always work with the server side (via api-url prop) to request the data. In order for the filtering to work, the given API must support that via filter query string.
So,

you can filter anything as long as your API support it.
you can access the data fetched via tableData. see here


freeglut fix

Fix as hinted by @ilovezfs

Audit is failing.
        Error: 1 problem in 1 formula
homebrew/x11/freeglut:
  * cmake dependency should be
      depends_on "cmake" => :build
    Or if it is indeed a runtime dependency
      depends_on "cmake" => :run

Looks unrelated, but would you mind fixing it while you're in here?

@BrewTestBot test this please

Thanks for your patience!


Adding support for R files

The R statistical language

You need to update the test/list_file_types.t file and make sure that make test runs successfully.

Do people even look at the Travis CI status? :/ I'll fix it myself.


Having an issue with CelliesProjects/wm8978-esp32

@per1234
I am having issues updating https://github.com/CelliesProjects/wm8978-esp32 to a new version.
I don't see any log files at
http://downloads.arduino.cc/libraries/logs/github.com/CelliesProjects/wm8978-esp32

Hi @CelliesProjects. The wm8978-esp32 in the logs URL is actually a folder (the logs are in index.html under that path), so it's necessary to add a trailing slash to the URL:
http://downloads.arduino.cc/libraries/logs/github.com/CelliesProjects/wm8978-esp32/
With the URL fixed, the logs look just fine:

2021/10/19 11:12:49 Scraping https://github.com/CelliesProjects/wm8978-esp32.git
2021/10/19 11:12:49 Checking out tag: 1.0.0
2021/10/19 11:12:49 Release wm8978-esp32:1.0.0 already loaded, skipping
2021/10/19 11:12:49 Checking out tag: 1.0.1
2021/10/19 11:12:49 Release wm8978-esp32:1.0.1 already loaded, skipping
2021/10/19 11:12:49 Checking out tag: 1.0.2
2021/10/19 11:12:49 Release wm8978-esp32:1.0.2 already loaded, skipping
2021/10/19 11:12:49 Checking out tag: 1.0.3
2021/10/19 11:12:49 Release wm8978-esp32:1.0.3 already loaded, skipping
2021/10/19 11:12:49 Checking out tag: 1.0.4
2021/10/19 11:12:49 Release wm8978-esp32:1.0.4 already loaded, skipping

and Library Manager too:

So I think all is well. Please let me know if you have any questions or concerns.


Add global config parameter for flow

Right now, you can only define flow on per-host level. With this change you would also be able to define the default one for every host and then rewrite it when necessary.
To set it in config, add flow: '/flow' to the st2Config object like that
'use strict';
angular.module('main')
  .constant('st2Config', {

    hosts: [{
      name: 'Dev Env',
      url: '//<IP_ADDRESS>:9101'
    }],
    flow: '/flow'

  });

or append
angular.module('main').run(function (st2Config) { st2Config.flow = '/flow' });

at the bottom of the file.

:+1:


It brokes when you register and enter first time

When you start up the app setting the password, the first time page loads give you an error about wrong filetype.
This is about the processing of POST requests.
Original message:

al hacer el registro-primer login, lanza mensaje que ha dado error subir la carta

Cannot reproduce 👎

Problem comes at login time

Solved at https://github.com/palaueb/restacart/commit/d1096fb4d61ae6ce9434dbc9dbbbafbe51c037f6


Update README to fix the version to require

There are stable versions now so it's better to document using them. Also it was confusing, it lead me to believe there was no stable version released yet.

Maybe we should replace Compsoer installation instruction to use composer require command instead.

I've pushed another commit.


Skip upgrade this version

Hello ic005k!
You have mentioned several times that you like suggestions for the app. Latest update shows a prompt if there is an update, that is a nice feature. What would be very useful would be the ability to see the update window but have an option to not update this version (skip this version always). That way, if I want to stay on 0.7.4, it would not upgrade to 0.7.5 and would disable upgrade prompts. If the update is clicked in the menu bar, then the updates would appear. Small detail but I think it would be useful.￼￼ Many thanks!
Your app is truly the Swiss Army knife of OC!

Ok, will consider adding this feature in the next version, thanks for the suggestion. More suggestions or questions are welcome, thanks.


python310Packages.pywam: init at 0.0.1

Description of changes
Used by https://github.com/Strixx76/samsungwam
Things done

Built on platform(s)

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

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

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

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

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

[x] Fits CONTRIBUTING.md.

Nope. Nope. NOPE. N O P E.


Support metal inserts for personalized surgical guides

I don't know how to describe it. It's something that goes inside the surgical guide slot and that's made of stainless steel so it avoids the risk of generating debris while making the osteotomies.
@cmfsx maybe you could explain more?
Do surgeons have 3D models of these slot like instruments?  Or do they have standardized sizes and geometries?

It appears these type of stainless steel tools also are used for drilling the holes of the neomandible plate


Undefined (mode)

I'm getting
undefined
in 8 mins
undefined
in 28 mins
etc
when using my transit route origin: Larchmont, NY, destination: Grand Central Terminal, , NY
If I use the examples of Mannheim HBF to Frankfurt HBF, I get
ICE in an hour
RE in 2 hours
1CE in an hour

I'm sorry for that, I did't do a lot of testing. I'll have a look at it. You're also welcome put your hands on it yourself.

Would you like to have New Haven displayed there?

New Haven is the name of the train line, so yes, that would be consistent. Thanks.
I did start to look at the code, and I did get it working with routes on London busses, but not the NY Metro North.
Thank you for this module - Having this data is the raison d'etre for my mirror. I had started poking around but JS is not my forte.

Thanks, just send the commits to fix it.
SamLewis0602 did most of the work, I just put the data in the right place :)


Cannot delete a Job without orphaning Pods

Current and expected behavior
Expected behavior:
After deleting a Job, unless the PropagationPolicy::Orphan is set in the DeleteParams, all Pods that were owned by that Job should be deleted as well. However, they are not cleaned up currently after running the Jobs example.
Reproduce:
In one terminal:
git clone https://github.com/kube-rs/kube-rs.git
cd examples
export KUBECONFIG=/your/kube/config/pathc
cargo run --example job_api

In another terminal:
watch kubectl get pods,jobs

After the example successfully runs, a Pod still remains, even though the example concludes with:
    jobs.delete("empty-job", &DeleteParams {
        dry_run: false,
        ..DeleteParams::default()
    })
    .await?;

Possible solution
It appears somehow that the delete is always done in orphan mode. And after the Job is deleted the orphaned pod no longer contains any ownerReference to the preexisting Job
Additional context
The following should demonstrate the expected behavior and the behavior when orphan cascade deleting strategy is set, which somehow appears to be the immutable option in the kube-rs api:
With deletion of dependent Jobs
Traditional background or foreground deletion that deletes all dependent Pods with Job deletion.
Apply this Job to the cluster and watch as one Pod is created and runs until Completed:
kubectl apply -f job.yaml
watch kubectl get pods,jobs

Delete the job setting deletion cascade strategy to background (same result can be expected for foreground). Watch as the Job and dependent Pod are deleted:
kubectl delete jobs/pi --cascade=background
watch kubectl get pods,jobs

See how the Pods remain if the cascade strategy is set to orphan
kubectl apply -f job.yaml
watch kubectl get pods,jobs

Delete the job setting deletion cascade strategy to background (same result can be expected for foreground). Watch as the Job and dependent Pod are deleted:
kubectl delete jobs/pi --cascade=orphan
watch kubectl get pods,jobs

Environment
Client Version: v1.21.5+k3s2
Server Version: v1.21.5+k3s2
Running the jobs_api example locally on Ubuntu 20.04
Configuration and features
This was reproduced using the latest version:
kube = { version = "0.64.0", default-features = false, features = ["client", "rustls-tls"] }
k8s-openapi = { version = "0.13.1", default-features = false, features = ["v1_22"] }

I found this issue in my project that was using:
kube = { version = "0.59.0",  features = ["derive"] }
k8s-openapi = { version = "0.13.0", default-features = false, features = ["schemars", "v1_16"] }

Affected crates
No response
Would you like to work on fixing this bug?
No response

Thanks for this.
It appears that our default is different from kubectl delete job which logs this with -v=9:
Request Body: {"propagationPolicy":"Background"}
curl -v -XDELETE  -H "Accept: application/json" -H "Content-Type: application/json" -H "User-Agent: kubectl/v1.22.2 (linux/amd64) kubernetes/8b5a191" 'https://<IP_ADDRESS>:39301/apis/batch/v1/namespaces/default/jobs/empty-job'
DELETE https://<IP_ADDRESS>:39301/apis/batch/v1/namespaces/default/jobs/empty-job 200 OK in 5 milliseconds

whereas our body is the default empty body. Possibly we should have PropagationPolicy not be optional and instead set the default to be Background. It's possible that apimachinery does this, need to double check.
I don't think it is immutable though. The following seems to work, and it avoids orphans:
    use kube::core::params::PropagationPolicy;
    jobs.delete("empty-job", &DeleteParams {
        dry_run: false,
        propagation_policy: Some(PropagationPolicy::Background),
        ..DeleteParams::default()
    })
    .await?;

I don't think it is immutable though. The following seems to work, and it avoids orphans

@clux modifying it does work! I may have mistaken an ophaned Pod from a previous run. Sorry! It does seem weird that the default is not background. Does that mean that the Jobs finalizer defaults to orphan: "The default policy is decided by the existing finalizer set in the metadata.finalizers and the resource-specific default policy"? This seems to suggest that Kubernetes (not just kubelet) has a default of background deletion in general.
Regardless, it may be good to update the jobs_api.rs example to add the propagation_policy to the delete params -- I'm happy to put that in if you'd like.

It certainly looks that way. I experimented a bit now and can confirm that the kubernetes api-server is treating the empty body (no propation policy set) different depending on what resource we use:

deleting a `Deployment with default params -> deployment deleted, pods deleted
deleting a Deployment with Orphan policy -> deployment deleted, pods stays around
deleting a Job object with default params -> job deleted, completed pod stays around
deleting a Job object with Orphan policy -> job deleted, completed pod stays around

need the explicit Background set on Job, unlike on Deployment. certainly weird, but I guess it's also kind of unusual for pods in jobs to stay around after they are done as well.
At any rate, I think action items here are:

add builder DeleteParams::cascade(mut self, pp: PropagationPolicy)
document the findings better in the PropagationPolicy enum
change job_api.rs to use the explicit Backround policy (through builder)

If you think that makes sense, and have time to pick that up, that would be very helpful!

Actually, I think this is such a fundamental thing to decide for DeleteParams that we should make root constructors for it (in the same way we have a root constructor for PatchParams::apply) to avoid forcing everyone to import the long PropagationPolicy into scope. That way it's more likely people use explicit values.
I.e. I think we should have:

DeleteParams::background() -> Default + PropagationPolicy::Background
DeleteParams::foreground() -> Default + PropagationPolicy::foreground
DeleteParams::orphan() -> Default + PropagationPolicy::Orphan

and keep the default, but document that it is ambiguous with respect to kind.

Actually, I think this is such a fundamental thing to decide for DeleteParams that we should make root constructors for it

Great idea! Yeah, I could probably take this on. I might not get started on it for a few days, though.

That's great, really appreciate the help! I'll assign this to you and rename the issue.
And don't worry about the time, it's not attached to anything urgent.

That's great, really appreciate the help! I'll assign this to you and rename the issue.
And don't worry about the time, it's not attached to anything urgent. Just assign me to PR review when you get around to it 😸

Hey @clux, I don't have permissions to request your review on the PR but here it is: https://github.com/kube-rs/kube-rs/pull/757


adding Virginia Community College System

Virginia Community College System has multiple colleges that use email.vccs.edu as their email domain.

web: http://www.vccs.edu
NVCC (https://nvcc.edu) is one of the community colleges in the system.
CS program url: https://catalog.nvcc.edu/preview_program.php?catoid=2&poid=194&returnto=62

@mrshishirr I am sorry, the domain vccs.edu is blacklisted because the school allows to register an email address there even if you are not a student of the school. If you are the student of this school, please apply with official document (OFFICIAL DOCUMENT tab at https://www.jetbrains.com/shop/eform/students).


enhancement - repositories stargazed by a user

It could also be nice to be able to see stats on which repositories a user has starred themselves ( i.e. github.com/username?tab=stars )
and provide the same statistics:
NAME | URL | MAIN LANG | STARS | FORKS etc

Yes, I remember looking into it, however I could not fetch such information from the GitHub API. At the moment the program is calling this address whose response does not contain the required info. Do you know of the right endpoint to call to retrieve it?

Sorry I am not very knowledgable about the api. Just thought i would offer this as a suggestion because it is related to stargazing. The tool is still nice even without this feature.

I will look into it again these days, I am sure there must be a way to fetch this information (I will have a deeper look into the GitHub documentation).


At least 2 VScode  were started

VSCode Version:
OS Version:

Steps to Reproduce:

The same happened with me right now, after the latest update...
macOS Sierra 10.12.1
VSCode: Version 1.8.0-insider (25dc9851...)

Same here. I opened VS Code with three workspaces. I quit. I restart all three open. I close two of them. I quit. I restart. All three of them open.  None of the workspaces contained dirty files. I use autosave. No untiled files that need to be restored in any of the workspaces.
The ~/Library/Application Support/Code - Insiders/Backups/workspaces.json file contains all three workspaces in its folderWorkspaces property.

My configuration:

VSCode Version: Code - Insiders 1.8.0-insider (25dc9851ef6d41fb4939a37312590cadaee90327, 2016-12-08T21:42:14.353Z)
OS Version: Darwin x64 15.6.0
Extensions:

Extension
Author
Version

vscode-docker
PeterJausovec
0.0.9

theme-toothpaste
aalaap
0.0.1

vscode-eslint
dbaeumer
1.2.0

githistory
donjayamanne
0.1.2

tslint
eg2
0.7.1

vscode-npm-script
eg2
0.1.2

vsc-travis-ci-status
felixrieseberg
1.0.5

csharp
ms-vscode
1.5.3

debugger-for-chrome
msjsdiag
2.3.2

Ruby
rebornix
0.10.4

java
redhat
0.0.7

kustovscode
seanmcbreen
0.0.1

autoimport
steoates
1.2.2

gitblame
waderyan
1.3.0

vscode-open-in-github
ziyasal
1.2.2

@kieferrm you have window.reopenFolders = all?

@bpasero yes.

@Tyriar I can reproduce and it seems that your logic in removeBackupPathSync no longer works, I can see a lowercase workspace identifier getting in and thus we fail to remove it from the list of backup workspaces (where the case is normal).

@Tyriar please review my fix: https://github.com/Microsoft/vscode/commit/40243d1e01f426f54670f3dbaa6b77d18469310d

@bpasero LGTM, thanks for fixing.


[CI] - Tika + Quarkus main

This issue will be opened and closed depending on the state of https://github.com/quarkiverse/quarkus-tika build against Quarkus main SNAPSHOT.
If you are interested in being notified, please subscribe to this issue.

@quarkiverse/quarkiverse-tika any hint what's going on?


Line 90, Column 3: [Warning-link-out-of-scope] File '5.0/Microsoft.PowerShell.Management/Get-ChildItem.md' referenced by link '../../Microsoft.PowerShell.Management/Get-ChildItem.md' will not be built because it is not included in build scope.
Line 91, Column 24: [Warning-link-out-of-scope] File '5.0/Microsoft.PowerShell.Management/Set-Location.md' referenced by link '../../Microsoft.PowerShell.Management/Set-Location.md' will not be built because it is not included in build scope.
Line 92, Column 29: [Warning-link-out-of-scope] File '5.0/Microsoft.PowerShell.Management/Get-Location.md' referenced by link '../../Microsoft.PowerShell.Management/Get-Location.md' will not be built because it is not included in build scope.
Line 188, Column 2: [Warning-link-out-of-scope] File '5.0/Microsoft.PowerShell.Management/Get-Item.md' referenced by link '../../Microsoft.PowerShell.Management/Get-Item.md' will not be built because it is not included in build scope.
Line 189, Column 2: [Warning-link-out-of-scope] File '5.0/Microsoft.PowerShell.Management/Get-ChildItem.md' referenced by link '../../Microsoft.PowerShell.Management/Get-ChildItem.md' will not be built because it is not included in build scope.
Line 252, Column 3: [Warning-link-out-of-scope] File '5.0/Microsoft.PowerShell.Management/Get-ItemProperty.md' referenced by link '../../Microsoft.PowerShell.Management/Get-ItemProperty.md' will not be built because it is not included in build scope.
Line 253, Column 3: [Warning-link-out-of-scope] File '5.0/Microsoft.PowerShell.Management/Get-ItemProperty.md' referenced by link '../../Microsoft.PowerShell.Management/Get-ItemProperty.md' will not be built because it is not included in build scope.
Line 291, Column 3: [Warning-link-out-of-scope] File '5.0/Microsoft.PowerShell.Management/Set-Item.md' referenced by link '../../Microsoft.PowerShell.Management/Set-Item.md' will not be built because it is not included in build scope.
Line 292, Column 3: [Warning-link-out-of-scope] File '5.0/Microsoft.PowerShell.Management/Set-ItemProperty.md' referenced by link '../../Microsoft.PowerShell.Management/Set-ItemProperty.md' will not be built because it is not included in build scope.
Line 323, Column 32: [Warning-link-out-of-scope] File '5.0/Microsoft.PowerShell.Management/New-ItemProperty.md' referenced by link '../../Microsoft.PowerShell.Management/New-ItemProperty.md' will not be built because it is not included in build scope.
Line 352, Column 3: [Warning-link-out-of-scope] File '5.0/Microsoft.PowerShell.Management/Copy-Item.md' referenced by link '../../Microsoft.PowerShell.Management/Copy-Item.md' will not be built because it is not included in build scope.
Line 353, Column 3: [Warning-link-out-of-scope] File '5.0/Microsoft.PowerShell.Management/Copy-ItemProperty.md' referenced by link '../../Microsoft.PowerShell.Management/Copy-ItemProperty.md' will not be built because it is not included in build scope.
Line 381, Column 3: [Warning-link-out-of-scope] File '5.0/Microsoft.PowerShell.Management/Move-Item.md' referenced by link '../../Microsoft.PowerShell.Management/Move-Item.md' will not be built because it is not included in build scope.
Line 382, Column 3: [Warning-link-out-of-scope] File '5.0/Microsoft.PowerShell.Management/Move-ItemProperty.md' referenced by link '../../Microsoft.PowerShell.Management/Move-ItemProperty.md' will not be built because it is not included in build scope.
Line 412, Column 3: [Warning-link-out-of-scope] File '5.0/Microsoft.PowerShell.Security/Get-Acl.md' referenced by link '../../Microsoft.PowerShell.Security/Get-Acl.md' will not be built because it is not included in build scope.
Line 413, Column 3: [Warning-link-out-of-scope] File '5.0/Microsoft.PowerShell.Security/Set-Acl.md' referenced by link '../../Microsoft.PowerShell.Security/Set-Acl.md' will not be built because it is not included in build scope.
Line 489, Column 3: [Warning-link-out-of-scope] File '5.0/Microsoft.PowerShell.Management/Clear-Item.md' referenced by link '../../Microsoft.PowerShell.Management/Clear-Item.md' will not be built because it is not included in build scope.
Line 490, Column 3: [Warning-link-out-of-scope] File '5.0/Microsoft.PowerShell.Management/Clear-ItemProperty.md' referenced by link '../../Microsoft.PowerShell.Management/Clear-ItemProperty.md' will not be built because it is not included in build scope.
Line 491, Column 3: [Warning-link-out-of-scope] File '5.0/Microsoft.PowerShell.Management/Remove-Item.md' referenced by link '../../Microsoft.PowerShell.Management/Remove-Item.md' will not be built because it is not included in build scope.
Line 492, Column 3: [Warning-link-out-of-scope] File '5.0/Microsoft.PowerShell.Management/Remove-ItemProperty.md' referenced by link '../../Microsoft.PowerShell.Management/Remove-ItemProperty.md' will not be built because it is not included in build scope.
Line 505, Column 1: [Warning-link-out-of-scope] File '5.0/Microsoft.PowerShell.Management/Set-ItemProperty.md' referenced by link '../../Microsoft.PowerShell.Management/Set-ItemProperty.md' will not be built because it is not included in build scope.
Line 507, Column 1: [Warning-link-out-of-scope] File '5.0/Microsoft.PowerShell.Management/Set-Item.md' referenced by link '../../Microsoft.PowerShell.Management/Set-Item.md' will not be built because it is not included in build scope.
Line 526, Column 3: [Warning-link-out-of-scope] File '5.0/Microsoft.PowerShell.Management/Set-Item.md' referenced by link '../../Microsoft.PowerShell.Management/Set-Item.md' will not be built because it is not included in build scope.
Line 527, Column 3: [Warning-link-out-of-scope] File '5.0/Microsoft.PowerShell.Management/Set-ItemProperty.md' referenced by link '../../Microsoft.PowerShell.Management/Set-ItemProperty.md' will not be built because it is not included in build scope.
Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

archived-reference/4.0/PSScheduledJob/About/about_Scheduled_Jobs.md

Line 42, Column 67: [Warning-link-out-of-scope] File '4.0/PSScheduledJob/PSScheduledJob.md' referenced by link '../PSScheduledJob.md' will not be built because it is not included in build scope.
Line 243, Column 1: [Warning-link-out-of-scope] File '4.0/PSScheduledJob/PSScheduledJob.md' referenced by link '../PSScheduledJob.md' will not be built because it is not included in build scope.
Line 2, Column 1: [Suggestion-description-missing] Missing required attribute: 'description'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Blank page after Softaculous install

https://support.denisflorent.digital/

Hi,
I never used softculos, so I believe you should talk to them and see why it is failing, a default installation works on my side

Hey.
Thanks for the quick reply.
Will check that with them.
Cheers,
Denis.
Le 22 juin 2018 à 11:07, Jordi Puigdellívol <notifications<EMAIL_ADDRESS>a écrit :
Hi,
I never used softculos, so I believe you should talk to them and see why it is failing, a default installation works on my side
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FBadChoice%2Fhandesk%2Fissues%2F147%23issuecomment-399359108&data=02|01||445ad04e76bc415a07cb08d5d8173253|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636652516487195945&sdata=7Sh%2Bb1JYYiW37ebW2hOsWMhnDaQaFNiXQjNu5%2FCP%2FmU%3D&reserved=0, or mute the threadhttps://eur01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FALo01kjoZzSw0yzXNOFjhSUc0tFGoMp7ks5t_KW_gaJpZM4UuTUB&data=02|01||445ad04e76bc415a07cb08d5d8173253|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|636652516487195945&sdata=ULx23Qta%2BgfewTAi88UaJgPYK7fCQxGs31c%2B3DKxV8o%3D&reserved=0.


Please compile version for Scala 2.13.6

Ditto for 2.12.14, not sure it's worth a separate issue or not.

@mccartney Could you kindly check if everything was fine with publishing the built artifacts? They should be uploaded to maven cental by now, but it seems they are not.

You are right. Something must've gone wrong:
https://search.maven.org/search?q=g:com.sksamuel.scapegoat v:1.4.9
I don't see 1.4.9 there. Looking into it - I'll use #537 for keeping the record.


(misc): setup_termbg_sync breaks firenvim

Contributing guidelines

[X] I have read CONTRIBUTING.md
[X] I have read CODE_OF_CONDUCT.md

Module(s)
mini.misc
Description
Using setup_termbg_sync alongside firenvim breaks the latter because how it's designed to communicate with the browser.
My feature request is to add a FAQ or mention of taking this into account for end users of mini.misc.
Said FAQ could mentoin that to solve this issue a user simply has to conditionally enable setup_termgb_sync only if the current Neovim session hasn't been started by firenvim:
    if not vim.g.started_by_firenvim then require("mini.misc").setup_termbg_sync() end

Or do you think this should be mentioned on the firenvim side?

Thanks for the suggestion!
The general idea of the note about when it should be used sounds reasonable. I thought that having the mention of "terminal emulator" would be enough, but it might not be. There is now a dedicated note on latest main.
Mentioning any specific cases of when not to call it (like when Firenvim is used, etc.) seems to be too much.

@TheLeoP, while reading some unrelated parts of Neovim help, I noticed that there seems to be an automated way to make setup_termbg_sync() do nothing if Neovim is started not in compatible way. So I think guarding behind vim.g.started_by_firenvim should not be necessary anymore (at least it is not needed for Neovide and VS Code Neovim extension).

Thanks for the heads up :D. Could you link/mention the tag of the relevant part of the Neovim help?


[Fixed in GH-6052] safari firefox: Missing Visual Focus on PDP

The PDP needs a deep review. Most of the elements in this page don't have a visible focus or not receiving focus at all.

Chrome: ✔ (note: PDP tabbing order is fixed in a11y branch)
Firefox: ✔ (note: focus border is huge)
Safari: ❓

Question:
The focus on the thumbnail pictures are in red instead of blue? Intended?
For all browsers: I cannot actually navigate to another tab with the keyboard. It shows the focus, but can't actually go there.
Chrome: OK.
Safari:
Missing focus on "Add to cart", "Write a Review", More/Less, Cancel, Submit buttons
Firefox:
Missing focus on "Show Reviews" button. The focus for me is barely visible on the tabs, and the stars.

Tested on dev3.

Chrome ✅

Firefox ✅

Safari ✅

Could not actually select the elements in focus on PDP: #6574

Missing focus for cross-selling, upselling products: #6575

Could not see the focus for thumbnails on second page: #6576


Better simplified Chinese language package

The details
ad-addon.zip

Great. i applied this to all of versions.
And release completed.
Thanks for translation!


Adding pool memory management

We've landed samples with pools for memory management. We'll migrate this sample to this repo.
This will include changing the allocate and free, and the Utf8 and Utf16.
If you've any feedback on the sample before I merge this into the this package, please let us know.

The samples have been updated, and are not conflicting with package:ffi anymore.
We could opt to add a Pool extends Allocator here, but we don't have to. Users can also write their own pools.
Closing this issue. If there is a need for adding a Pool in this package, please reopen.


transitiontime is in multiples of 100ms

Tried setting transitiontime to 1000 and it took a very long time to change state. I checked the Philips Hue docs and it says the following:

transitiontime - uint16	The duration of the transition from the light’s current state to the new state. This is given as a multiple of 100ms and defaults to 4 (400ms). For example, setting transitiontime:10 will make the transition last 1 second.

Might be worth updating the documentation on this library to reflect this behaviour :)

Can you please elaborate on the actual function call you made, as I already perform conversion from milliseconds into the actual hue value that the API takes, but maybe I have missed a possible call in the code and am not transforming it correctly in all places.

I've only noticed it when using the setLightState function.. I haven't tried any other functions which may specify a transitiontime as well

How are you building the light state, manually as a JSON object, or via a light state object?


package on pypi (0.0.1) is borked

Added README.rst to MANIFEST.in. otherwise this package is not installable

thank you sir!


Update Ruuter to use project-based DSL mapping and routing

Add functionality to enable project-based paths in Ruuter, so that DSL's can be defined as:
<project>/<method>/<dsl path*>
for example, training/GET/stories-file.yml which maps to GET /training/stories-file

Currently in test together with other component rewrites (DM #18-#21, Resql #22 & #27, Ruuter #237)


If the length of the correct description is not equal to the predicted description length, the bleu evaluation cannot be performed.

    assert len(references) == len(hypotheses)

# Calculate BLEU-4 scores
bleu4 = corpus_bleu(references, hypotheses)

Hey @hubin111
This situation you're describing is impossible! Because, for each image, we will always have a generated caption (hypothesis). This is a simple list [hyp1, hyp2, ...]. Accordingly, we'll have references as a list of lists [[ref1a, ref1b, ref1c, ref1d, ref1e], [ref2a, ref2b, ref2c, ref2d, ref2e], ... ].
Now, if you compute the length of these two lists, then they will match.
This assert statement
assert len(references) == len(hypotheses)
just checks for whether we generated one caption for each image.
So, I don't see where the issue is coming from..

@kmario23 What you are suggesting makes complete sense. However, COCO dataset has 5 groundtruth captions per image. So since our model will generate one caption per image, how can their lengths be equal?

Python assertion checks for lengths only at the top level. See the example in my description above.

@kmario23 Yup, understood. Thanks :D


Move transpose subroutines in dedicated submodules

For a synthetic view of the modification, please look at the first commit. The fprettify script has modified many lines.

This was only tested with the GNU compiler

Passes tests locally with ifort


change go tools to use certain commit

vet tool was updated to work with go tip (unreleased, 1.5.0)
and it takes us a week or so to update our go version after they release sooo this will eventually resurface whether or not they revert the commit (see golang/go#10446)
fixes CI, ping @LK4D4 @tianon

:+1: (love the branch name too!)

@jfrazelle LGTM
but @tianon comment makes sense

updated with -v

LGTM

:heart:


[Feature Request] Select and Remember USB Devices for Redirection to VM

First of all:
great project, i'm loving it so far.
While I was discussing the use of your client for a new VDI project with my colleague we came across the USB pass through feature, which is essential for us. It needs to work properly and reliably.
Our Clients mainly will use Windows PCs to connect to the VDI.
I guess my question is if it's possible to build a select and remember function for the USB devices which need to be redirected to the VM and then pass that information to the virt-viewer at start?
For the user to select all needed USB devices in the virt-viewer after every connection is not a desired behavior.
Just my 2 cents:
The need in the market for a better and more open approach to VDI is huge. Especially after the prognosis for the dying Terminal Server from Microsoft and the acquisition of VMWare from Broadcom.
So to build a reliable VDI with Proxmox and other open-source tools would be huge, and for me personally, a huge win for the open-source community.

So the best I could come up with is to be able to define custom parameters in the ini file that are passed through to remote-viewer.
I have updated the example ini file with these changes and released a new MSI build, please let me know if this works for you.

I'll need to study the .vv file options for setting USB passthrough, if
they exist, so if you have any insight into this please respond with that
information.
On Fri, Aug 19, 2022, 6:24 AM Chris @.***> wrote:

First of all:
great project, i'm loving it so far.
While I was discussing the use of your client for a new VDI project with
my colleague we came across the USB pass through feature, which is
essential for us. It needs to work properly and reliably.
Our Clients mainly will use Windows PCs to connect to the VDI.
I guess my question is if it's possible to build a select and remember
function for the USB devices which need to be redirected to the VM and then
pass that information to the virt-viewer at start?
For the user to select all needed USB devices in the virt-viewer after
every connection is not a desired behavior.
Just my 2 cents:
The need in the market for a better and more open approach to VDI is huge.
Especially after the prognosis for the dying Terminal Server from Microsoft
and the acquisition of VMWare from Broadcom.
So to build a reliable VDI with Proxmox and other open-source tools would
be huge, and for me personally, a huge win for the open-source community.
—
Reply to this email directly, view it on GitHub
https://github.com/joshpatten/PVE-VDIClient/issues/28, or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABIP64VFV6DIYXWAYVYBDPLVZ5VHVANCNFSM57AMUOXA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

After a bit of research I think you're going to need to use a usb filter to do this
see: https://www.mankier.com/1/remote-viewer and search for usb-filter
You can add this option in the AdditionalParameters section of the ini file


Scatter plot looks meh

We need to change the scatter plot so it can be more atrractive especially when the values change from the dashboard

Scatter plot was replaced using a violin plot


Documentation

Thanks so much for coming here to raise an issue. Please take a moment to 'check' the below boxes:

[x] I'm up-to-date with the latest release:
pip install -U talos

[x] I've confirmed that my Keras model works outside of Talos.

If you still have an error, please submit complete trace and a code with:

output of shape for x and y e.g. (212,12)
Talos params dictionary
The Keras model wired for Talos
Description of extra variables in the model

You can provide the code in pastebin / gist or any other format you like.

If I am not making a mistake, the current example does not fit with the latest version. Can you please update the documentation on calling Scan?

Thank you,
import talos as ta
h = ta.Scan(X, Y, params=p, experiment_name='first_test', model=drug2gene_model, grid_downsample=0.5)
TypeError                                 Traceback (most recent call last)
 in ()
1 import talos as ta
----> 2 h = ta.Scan(X, Y, params=p, experiment_name='first_test', model=drug2gene_model, grid_downsample=0.5)
TypeError: init() got an unexpected keyword argument 'experiment_name'

That's true. I will update the examples. Now you have to use:
h = ta.Scan(x, y,
                    params=p,
                    dataset_name=icu_stays,
                    experiment_no='2',
                    model=icu_stays,
                    grid_downsample=0.1)

Ok done. This will be updated in the next merge to Master. Thanks for flagging it.
Closing here.


Bevy mesh terrain integration

Hi @rewin
okay i got it working !! but i had to do something funky with downcasting so if you want to change the way the EditorTools work to avoid that then you can do that .
now it works 😄 😄   it is a bit funky because

the chunks highlight white when brushing which is annoying , could use a WIthout to avoid  this
the camera orbits around while brushing ... i avoid this in my editor by reserving Left Click to brushing and not panning or orbiting the camera ,, and i use WASD to fly
if you click a lot while brushing, its possible to make the editor crash because one of your systems is trying to do something to my chunk which gets removed (its entity)  in another thread.   May have to use  try_insert or something creative there idk .  my chunk entities do get auto-removed in an async thread  so that is likely related.

please fix cargo fmt -- --check and cargo clippy.
I also dont see Cargo.toml in the changes, would like to see it to review libraries and lint configuration

Its merge to integration branch (bevy_mesh_terrain_integration), not main. So we can merge it now without checking


Add JavaScript client

This is heavily based on the Python client. Let me know if there are any changes you want me to make!

Thanks for the contribution! I'm no Javascript expert, so I will trust that if users notice anything they would like to see done differently, then they will comment and request alterations.


Update specifying-virtual-wifi-bus-dependencies.md

Small changes to be consistent with the typical way we refer to INF sections

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

Thanks for the clarification on terminology, @zlockard.


How to have a new row appear in the correct (sorted) order?

When I add a new row to a table, it does not show up in the expected sorted place.
I have to sort a column to see the new row show up where it should be.
How can I do this the correct way?
This is my table definition:
<vue-good-table
        title="Shows"
        :columns="columns"
        :rows="rows"
        :paginate="true"
        :perPage="15"
        :paginateOnTop="true"
        :lineNumbers="false"
        :onClick="onRowClick"
        rowStyleClass="row-class"
        styleClass="table condensed table-bordered"/>

I assign values to the rows property after I get data from a server:
this.rows = this.$store.state.showsList
When I add a new row via a different component I add it to a vue store collection that is also assigned to the row property of the table by pushing it to the store collection.
addShow: (state, payload) => {
  state.showsList.push(payload)
}

I can see in the debugger that the row is added to the rows property and to the store collection.
But how do I make it appear in the correct location?
Thanks,
Eric

JS uses references. You need to create a new Array instead of assign this one, like the example below.
this.rows = [ ...this.$store.state.showsList ]

Just a tip: Use getters instead of direct assign Store's state, it avoid a lot of issues coming from mutable methods (ex. Array.sort) and reference assignments.

Vitor,
Thanks for your reply.
I'll try to use Getters like you suggested.
Creating a new array would be inefficient as the showList is very large.
I don't want to double the memory footprint of my app as I have 4 of these
large list.
Cheers,
Eric
On Fri, Dec 22, 2017 at 3:07 AM Vitor Cavalcanti<EMAIL_ADDRESS>wrote:

JS uses references. You need to create a new Array instead of assign this
one, like the example below.
this.rows = [ ...this.$store.state.showsList ]
Just a tip: Use getters instead of direct assign Store's state, it avoid
a lot of issues coming from mutable methods (ex. Array.sort) and reference
assignments.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/xaksis/vue-good-table/issues/148#issuecomment-353445809,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABl8IQLWcPtuwMoGBo8ijDkOQEd6-18pks5tCrp8gaJpZM4RJtZ0
.

@eric-naguras I'm going to close this for now. Please re-open if you are still experiencing a problem with this.


Testing

Automated changes by create-pull-request GitHub action

Closing, sorry for the noise :sweat_smile:


Add typmod validation in [VAR]BINARY input function

Description
Task: BABEL-4956
Signed-off-by: Rishabh Tanwar<EMAIL_ADDRESS>Test Scenarios Covered

Use case based -

Boundary conditions -

Arbitrary inputs -

Negative test cases -

Minor version upgrade tests -

Major version upgrade tests -

Performance tests -

Tooling impact -

Client tests -

Check List

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is under the terms of the Apache 2.0 and PostgreSQL licenses, and grant any person obtaining a copy of the contribution permission to relicense all or a portion of my contribution to the PostgreSQL License solely to contribute all or a portion of my contribution to the PostgreSQL open source project.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

1 of 2 (50.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.001%) to 72.769%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

contrib/babelfishpg_common/src/varbinary.c
1
2
50.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.001%

Covered Lines:
41779

Relevant Lines:
57413

💛  - Coveralls


[Snyk] Upgrade next from 12.1.5 to 12.2.5

This PR was automatically created by Snyk using the credentials of a real user.Snyk has created this PR to upgrade next from 12.1.5 to 12.2.5.

:information_source: Keep your dependencies up-to-date. This makes it easier to fix existing vulnerabilities and to more quickly identify and fix newly disclosed vulnerabilities when they affect your project.

The recommended version is 111 versions ahead of your current version.
The recommended version was released a month ago, on 2022-08-12.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open upgrade PRs.
For more information:
🧐 View latest project report
🛠 Adjust upgrade PR settings
🔕 Ignore this dependency or unsubscribe from future upgrade PRs

Closed by #1001


errors when run ionic serve

I'm submitting a ...  (check one with "x")
[x ] bug report
[ ] feature request
[ ] support request
hello,
I upgraded to ionic 3.2.1 and when i run your app, I run before npm install and then ionic serve and I get errors

Hey @adirzoari I'm sorry for the late reply. I have updated the repository and it should be fixed. Give it a try and reopen the issue in case it didn't work for you.


Add support for OCI-based registry

Is this a BUG REPORT or FEATURE REQUEST? (choose one):
FEATURE REQUEST
Add the support for using oci registry for pushing helm charts. This feature is supported by Helm 3

https://helm.sh/docs/topics/registries/#enabling-oci-support

It would be great if this plugin supports this feature. We are using this feature in our CI/CD by manually setting the HELM_EXPERIMENTAL_OCI=1 flag and using helm cli to push the chart to OCI registry.
The use of oci registry for storing helm charts is increasing day by day and it will be the most used repository type in near future.
So please add support for this feature in this plugin. Thanks.
Environment (plugin version, maven version, OS, ...):
helm 3
Anything else we need to know:
This feature is experimental for now, and requires to have the below flag set in env variable
export HELM_EXPERIMENTAL_OCI=1

We could really use this feature as we use the GitLab Docker Registry as our helm repo, which only supports OCI. Having a dedicated Helm Repo would add unnececcesary overhead and maintance costs, so we would really appreciate OCI support.

I also need this feature 👍

+1 - We really like the plugin but... without an OCI support we won't be able to publish anything into either Harbor or an Azure Container Registry for our firm (big asset manager ;-))

Fixed with #169, will be released this week with 6.1.0.


[WIP] Refactor Invoke-SQLiteBulkCopy

This PR:

Optimizes regular expressions, especially replaces Get-ParameterName with one-liner
Fixes bug with CommandTimeout that was not assigned to SQL command
Adds breaks in switch statements to exit switch block when match is found
Adds support for milliseconds when converting to/from DateTime
Fixes bug in AppendDataSource parameter that was sending boolean from -match operator to output
Updates test cases
Fixes misspellings

I can't test it against PowerShell 2.0.


[10.0.x] NO-ISSUE: Fix weekly image tags

Fix image tags for the weekly deploy jobs.
Pattern = Stream-YYYYMMDD

PR job #5 was: UNSTABLE
Possible explanation: This should be test failures
Please look here: https://ci-builds.apache.org/job/KIE/job/kogito/job/10.0.x/job/pullrequest/job/kogito-images.build-and-test/5/display/redirect
See console log:

Console Logs
Starting building: KIE » kogito » 10.0.x » pullrequest » kogito-images.build-image #22Starting building: KIE » kogito » 10.0.x » pullrequest » kogito-images.build-image #28Starting building: KIE » kogito » 10.0.x » pullrequest » kogito-images.build-image #25Build KIE » kogito » 10.0.x » pullrequest » kogito-images.build-image PR #1797 - kogito-base-builder: https://github.com/apache/incubator-kie-kogito-images/pull/1797 completed: SUCCESS[Pipeline] }[Pipeline] // stage[Pipeline] }Build KIE » kogito » 10.0.x » pullrequest » kogito-images.build-image PR #1797 - kogito-jit-runner: https://github.com/apache/incubator-kie-kogito-images/pull/1797 completed: SUCCESS[Pipeline] }[Pipeline] // stage[Pipeline] }Build KIE » kogito » 10.0.x » pullrequest » kogito-images.build-image PR #1797 - kogito-jobs-service-postgresql: https://github.com/apache/incubator-kie-kogito-images/pull/1797 completed: SUCCESS[Pipeline] }[Pipeline] // stage[Pipeline] }Build KIE » kogito » 10.0.x » pullrequest » kogito-images.build-image PR #1797 - kogito-data-index-postgresql: https://github.com/apache/incubator-kie-kogito-images/pull/1797 completed: SUCCESS[Pipeline] }[Pipeline] // stage[Pipeline] }Build KIE » kogito » 10.0.x » pullrequest » kogito-images.build-image PR #1797 - kogito-jobs-service-ephemeral: https://github.com/apache/incubator-kie-kogito-images/pull/1797 completed: SUCCESS[Pipeline] }[Pipeline] // stage[Pipeline] }Build KIE » kogito » 10.0.x » pullrequest » kogito-images.build-image PR #1797 - kogito-data-index-ephemeral: https://github.com/apache/incubator-kie-kogito-images/pull/1797 completed: UNSTABLE[Pipeline] unstableWARNING: Tests on kogito-data-index-ephemeral seems to have failed[Pipeline] }[Pipeline] // stage[Pipeline] }Build KIE » kogito » 10.0.x » pullrequest » kogito-images.build-image PR #1797 - kogito-jobs-service-allinone: https://github.com/apache/incubator-kie-kogito-images/pull/1797 completed: UNSTABLE[Pipeline] unstableWARNING: Tests on kogito-jobs-service-allinone seems to have failed[Pipeline] }[Pipeline] // stage[Pipeline] }[Pipeline] // parallel[Pipeline] }[Pipeline] // dir[Pipeline] }[Pipeline] // script[Pipeline] }[Pipeline] // stage[Pipeline] stage[Pipeline] { (Declarative: Post Actions)[Pipeline] script[Pipeline] {[Pipeline] sh+ wget --no-check-certificate -qO - 'https://ci-builds.apache.org/job/KIE/job/kogito/job/10.0.x/job/pullrequest/job/kogito-images.build-and-test/5/api/json?depth=0'[Pipeline] readJSON[Pipeline] sh


Docker image should not require TARGET_URL

Since you can specify the target host on the Locust class I don't see a need to having to specify it again with an environment variable. Could this restriction be removed, or am I missing something?

Just specify locust [arguments] as the COMMAND for docker run.

But you still need arguments to make it run correctly. Env vars are one quite regular way of doing this.
I think perhaps we should set /usr/local/bin/locust as the ENTRYPOINT and leave CMD unset. This is quite conventional and means the user must pass all required arguments to docker run, e.g:
docker run -p 8089:8089 --volume $PWD/dir/of/locustfile:/mnt/locust locustio/locust --locustfile /mnt/locust/locustfile.py --no-web -c 1000 -r 100

AWS CLI does this for example.

Ah, yes. ENTRYPOINT is what we should use of course :).


Does not compile on Android arm

Steps to reproduce:
# In ~/.cargo/config
[target.armv7-linux-androideabi]
linker = "path/to//android-arm-api-27-toolchain/bin/arm-linux-androideabi-gcc"

$ cargo build
/home/dmerej/.cargo/registry/src/github.com-1ecc6299db9ec823/terminal_size-0.1.8/src/unix.rs:33:30
   |
33 |         ioctl(STDOUT_FILENO, TIOCGWINSZ, &mut winsize);
   |                              ^^^^^^^^^^ expected i32, found u32

I think there's a #[cfg] missing somewhere
Update For now I edit the file by hand to add TIOCGWINSZ as i32 but this is not ideal ...
As to why I need terminal size on android, well, it's a long story :)

By the way, term_size does not have the problem, so I'm using this instead now.

Thanks will take a look at! But yeah with cross-platform I did not have Android in my mind ;) but I'll see what I can do.

As to why I need terminal size on android, well, it's a long story :)

Or a short one! I want to try termpix in Termux! And terminal-size is a dependency! That's it 😄

This build error should be fixed when #14 is merged


Security 密码明文传输

In GitLab by @luotian-github on May 27, 2020, 08:22
问题 1/2 http://<IP_ADDRESS>/account/sign-in － user_password    问题 2/2 http://<IP_ADDRESS>/api/v1/users/ {userId}/password /pas sword －  user_password_old
一般修订建议
使用 https； 使用加密算法如 AES、SHA-256 等

In GitLab by @luotian-github on May 27, 2020, 08:23
assigned to @sokril

In GitLab by @luotian-github on May 27, 2020, 09:31
assigned to @luotian-github and unassigned @sokril

In GitLab by @luotian-github on May 28, 2020, 08:47
will use https for the security mode

In GitLab by @luotian-github on Jun 22, 2020, 03:11
assigned to @cuiaq123 and unassigned @luotian-github

In GitLab by @cuiaq123 on Jun 30, 2020, 05:57
assigned to @luotian-github and unassigned @cuiaq123

In GitLab by @luotian-github on Aug 31, 2020, 06:31
removed milestone


Migration create stops without helpful error message

EdgeDB Version: 2.14+5561c9b
EdgeDB CLI Version: 2.3.1+5d93f42
OS Version: macOS Monterey 12.5.1

Steps to Reproduce:

Create a new project and a simple ESDL like

module default {

	type MyType {
		required property name -> str;
		property deleted -> bool {
			default := false;
		};
		property deleted_at  -> datetime;
		constraint exclusive on (.name) except (.deleted);
	}
}

Apply the schema:

> edgedb migration create
Created dbschema/migrations/00001.edgeql, id: m1jz5fffe7eo7i4e7q7ran5zvbvwwtrigm7mjwykdmsol5rbexqrwa
> edgedb migrate
Applied m1jz5fffe7eo7i4e7q7ran5zvbvwwtrigm7mjwykdmsol5rbexqrwa (00001.edgeql)

You decide that deleted is somewhat redundant, so change the constraint to constraint exclusive on (.name) except (exists .deleted_at);
Yes, the whole property deleted should be removed, too, but that leads to another problem, so let's stick to the constraint in this example.
Migrate the change:

> edgedb migration create
edgedb error: EdgeDB could not resolve migration with the provided answers. Please retry with different answers.
>

No question were asked, no answers were given, no chance to provide different answers, and no additional hints to the potential issue with the change were given.
I don't know if my updated contraint is terribly wrong, but anyway, I'd expect that

a hint to the migration issue is given
a message referring to provided answers is printed only if there were answers provided (= questions asked)

Moved the issue to EdgeDB as this is server issue mostly.

Hello,
Do you guys have any plans to improve the messaging here? The workaround often ends up being find some workarounds by slicing and dicing the changes into a couple of migration by dropping fields and recreating fields altogether. Which in our case we can afford to do now but certainly won't be able to once we run in production.

Another problem related to this issue is when I have except in a exclusive constraint, the generated js/python code from my EdgeQL might be wrong on the cardinality. Here is an example:

The schema:

module default {
    type User {
        required name: str;
        deleted: bool;

        constraint exclusive on (.name) except (.deleted);
    }
};

The EdgeQL:

select User { id, name } filter .name = <str>$name and not .deleted;

The code generated from the EdgeQL:

// GENERATED by @edgedb/generate v0.5.6

import type {Executor} from "edgedb";

export type GetUserByNameArgs = {
  readonly "name": string;
};

export type GetUserByNameReturns = Array<{
  "id": string;
  "name": string;
}>;

export function getUserByName(client: Executor, args: GetUserByNameArgs): Promise<GetUserByNameReturns> {
  return client.query(`\
select User { id, name } filter .name = <str>$name and not .deleted;`, args);

}

You can see the GetUserByNameReturns is an array, instead of a single object. The name is unique, so the returned value should be a single object. I also tried to create the constraint on both name and deleted, and the result is the same.
constraint exclusive on ((.name, .deleted)) except (.deleted);

Without the except in the constraint, the result is correct.


Support Cortex-A55

I have a linux machine arm64. Can you support Cortex-A55?

If this for a personal project include your board information and change the title accordingly, please. If a lot of people ask for the same board then we can consider if. If this is for a commercial project please close the issue and submit a formal request https://picovoice.ai/contact/sales/

If this for a personal project include your board information and change the title accordingly, please. If a lot of people ask for the same board then we can consider if. If this is for a commercial project please close the issue and submit a formal request https://picovoice.ai/contact/sales/

I'm working on a device at school. I want to use wake word to wake it up. Unfortunately you do not support this chip yet.
It uses arm64 architecture (chip s905 x3). Can you support it in future?

I'm working on a device at school. I want to use wake word to wake it up. Unfortunately you do not support this chip yet.
It uses arm64 architecture (chip s905 x3). Can you support it in future?

Same issue. I try it on linux arm machine. But it not work

Same issue. I try it on linux arm machine. But it not work

Same issue. I try it on linux arm machine. But it not work

Same issue. I try it on linux arm machine. But it not work

The repo's readme mentions the list of the supported platform. What you are using is not supported at this time.

The repo's readme mentions the list of the supported platform. What you are using is not supported at this time.


No SpotifyNoPremium option

Describe the bug

No SpotifyNoPremium option
To Reproduce
Steps to reproduce the behavior:

open the setting
main setting

Screenshots

Desktop Setup
OS: Osx v12.3.1
Spotify: v<IP_ADDRESS>6
Spicetify: v2.9.7
Dribbblish: v4.2.0-69f283c
Logs

[paste logs here]

it's been disabled here (https://github.com/JulienMaille/dribbblish-dynamic-theme/commit/ea97d6924590c748ed8f0dffe9a1d1ce69d507e0) for the moment due to this issue -> #213


build: set dockerhub credentials for Nutanix examples

Adding dockerhub credentials in the examples, to make it easier to bring up Nutanix clusters. Otherwise, the limit is hit for almost every cluster.

Closing in-favor of https://github.com/nutanix-cloud-native/cluster-api-runtime-extensions-nutanix/pull/501


feat: columns should be a dictionary not list of tuples

Goals:
Since List[Tuple] is not something that can be passed via YAML to a Flow, we need to make sure that this is doable, so having a Dict like structure for columns is the best way to go.

[ ] Change the code
[ ] Test both ways to initialize
[ ] Write documentation

Linked to https://github.com/jina-ai/annlite/pull/160


queryOptions() helper using select doesn't type-check when used with useQueries

Describe the bug
I'm trying to use a QueryOptions object created with the new queryOptions() helper in a useQueries() call.
This doesn't typecheck as long as the options object contains a select-fn where the return type doesn't equal the return type of the queryFn.
It works fine when the same options object is used in a useQuery(). Same story with useSuspenseQueries() and useSuspenseQuery()
Your minimal, reproducible example
https://www.typescriptlang.org/play?#code/JYWwDg9gTgLgBAbzgRwK4FMoE8DyYbAQB2AzgDRyonoCKG2FVt9w6JcAvnAGZQQhwARAAEYAQ1LiAxgGsA9FHRipMALRpMWQQChtU4iXgbsAaXRY4AXjgAKAJRWAfHADagmecEBdPQaP0sADEiK1sHS2cABT4QYGoAOkUSCAAbADd0GwBGO19JFACAZXQU9BVoUJsAEzFxAC44IlQQACNMcOca8ThnLN19fIh8QlJQ41xhgxsEbTgCzTMsBvHF+zJZ+exg5YDg9bnqUpUdzWKjmGh1jjsAbn6-Taws0KY6TRshggNb+-zxgCYXtQ3lBWCRppswQ0XJ8RiQvJwfkA
Steps to reproduce

Hover over the error reported in the useQueries call in query2
Observe that it is expecting select to return the same type as queryFn

Expected behavior
I would expect that the useQueries call would type-check the same way as the useQuery call
How often does this bug happen?
None
Screenshots or Videos
No response
Platform
Any platform
Tanstack Query adapter
react-query
TanStack Query version
5.8.7
TypeScript version
5.3.2
Additional context
No response

@TkDodo I think that I have a fix for this. It looks as though it's because select should be optional. I think it affects a few other places too. Could you please assign to me and I'll get the PR ready.

@TkDodo I've managed to get this working - although I couldn't find type tests for most of the methods changed. Let me know if I've done anything wrong as it's my first time opening a PR here.

@charlotte-bone thank you for this ❤

although I couldn't find type tests for most of the methods changed

yeah with useQueries, we still have an older method of type tests where we include them in the normal test file. At some point, I'd like to refactor this to be similar to [useQuery.types.test.tsx](https://github.com/TanStack/query/blob/c014ea7a29db5bdd0770733e3c531c9050d7e4e9/packages/react-query/src/__tests__/useQuery.types.test.tsx)
If that sounds interesting to you, please go ahead :)
Also: We have the useQueries types copied in multiple adapter packages. I think vue-query etc would have the same bug. Not sure how much we could abstract here, but we have about 5 usages of very similar types. If you want to take a stab at this too, it would be much appreciated.

Thank you @charlotte-bone! I tried fixing this myself, but my Typescript-fu didn't suffice in this instance. 😄
And thank you @TkDodo for merging & releasing this so quickly. 👍