Only use safe options when packing CSS assets

I've noticed z-index changes were overwritten.
It turns out that css-nano was used when optimizing css assets and safe option was not passed in.
https://cssnano.co/optimisations/zindex/
Verified that z-index no longer changes with this option.

Can you merge next into this and see if we can get CI passing?

Meh, I trust you that this works.


fix: Subtasks

This is to fix #34. Doing some additional final touches before ready for PR.

This also resolves #32, which I see as the first iteration towards dealing with #5.


Panic when adding candidate

This panic is fairly rare, but in certain (yet unknown) circumstances it is more prevalent.
panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x18 pc=0xe1758f]

goroutine 6592 [running]:
github.com/pion/ice/v2.(*Agent).addCandidate.func1({0xc0011adf90?, 0xc0011adf48?}, 0xc000004300?)
	/go/pkg/mod/github.com/pion/ice/v2@v2.3.4/agent.go:798 +0x26f
github.com/pion/ice/v2.(*Agent).taskLoop(0xc000004300)
	/go/pkg/mod/github.com/pion/ice/v2@v2.3.4/agent.go:240 +0xc2
created by github.com/pion/ice/v2.NewAgent
	/go/pkg/mod/github.com/pion/ice/v2@v2.3.4/agent.go:363 +0xd92

@m1k1o It looks like candidateConn is nil? https://github.com/pion/ice/blob/master/agent.go#L817C1-L818C1
I think it would be worth adding a defensive if candidateConn != nil check
It would be nice to figure out why this is happening though!

From my findings, I see that right before panic, the code is frozen for a couple of seconds (up to 10). Probably because of some TCP connection blocking when write buffer is full. When it finally unblocks, it is flooded with all messages at once and most likely ocurrs some race condition that leads to panic.
Last message seems to be Ignore duplicate candidate: tcp4 host <ip>:0, where the IP is NAT1TO1 of the server. The candidate is announced as candidate:4225724578 1 tcp<PHONE_NUMBER> <ip> 0 typ host tcptype active. I saw recent changes in the code about Active ICE TCP Candidates, but this is v2.3.4 what should predate those changes. I might try to upgrade to latest version to see if it happens again.

That is a strange one! I think we might be able to reproduce this though. Can you create a TCP client that doesn't disconnect properly. Maybe as simple as connecting from phone then turning on Airplane mode?
I am very confident that Active ICE TCP changes will not have fixed this. This is the passive code.

Simply connecting using TCP on mobile and truning on Airplane Mode did not cut it:
6:09PM WRN write error: write tcp <IP_ADDRESS>:52100-><IP_ADDRESS>:58735: use of closed network connection module=webrtc submodule=pion subsystem=ice-tcp
6:09PM INF error reading streaming packet: read tcp <IP_ADDRESS>:52100-><IP_ADDRESS>:58735: use of closed network connection module=webrtc submodule=pion subsystem=ice-tcp
6:09PM WRN error closing connection: close tcp <IP_ADDRESS>:52100-><IP_ADDRESS>:58735: use of closed network connection module=webrtc submodule=pion subsystem=ice-tcp
6:09PM WRN Failed to read from candidate tcp4 host <IP_ADDRESS>:52100: io: read/write on closed pipe module=webrtc peer_id=1 session_id=user submodule=pion subsystem=ice
6:09PM WRN Failed to discover mDNS candidate 14e7f6f2-8a05-4212-8024-e43a6032d72a.local: mDNS: connection is closed module=webrtc peer_id=1 session_id=user submodule=pion subsystem=ice

I traced it back to this commit, that added connConfigs = append(connConfigs, connConfig{nil, 0, TCPTypeActive}), where nil was passed down to addCandidate.

It was since then reverted by and changed by you. Therefore I believe, upgrading to the latest should fix the issue.

After upgrading pion/ice to v2.3.9 i can confirm, that it does not panic anymore.
But actually I am not able to connect using UDPMux or TCPMux. Only using ephemeral UDP the connection is possible. It has something to to again with the fact, that my server has 2 interfaces (first for the Internet, other for Management) and I am connecting from Management. After disabling the other Interface, it works like a charm.
This is issue on its own, so maybe it should be extracted and this one closed.

Closing, no work to be done here!


Merging dev to master for 6.5.3-preview Release

Codecov Report

Merging #713 into master will decrease coverage by 0.06%.
The diff coverage is 26.19%.

@@             Coverage Diff             @@
##             master    #713      +/-   ##
===========================================
- Coverage     48.17%   48.1%   -0.07%
+ Complexity     2585    2582       -3
===========================================
  Files           114     113       -1
  Lines         26578   26553      -25
  Branches       4453    4457       +4
===========================================
- Hits          12803   12773      -30
- Misses        11635   11648      +13
+ Partials       2140    2132       -8

Flag
Coverage Δ
Complexity Δ

#JDBC42
47.98% <26.19%> (-0.11%)
2571 <10> (-12)

#JDBC43
48.04% <26.19%> (-0.08%)
2581 <10> (-2)

Impacted Files
Coverage Δ
Complexity Δ

...a/com/microsoft/sqlserver/jdbc/SQLJdbcVersion.java
0% <ø> (ø)
0 <0> (ø)
:arrow_down:

.../microsoft/sqlserver/jdbc/SQLServerDataSource.java
45.19% <ø> (ø)
66 <0> (ø)
:arrow_down:

...soft/sqlserver/jdbc/SQLServerDatabaseMetaData.java
32.26% <ø> (-0.16%)
58 <0> (ø)

...m/microsoft/sqlserver/jdbc/KerbAuthentication.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...m/microsoft/sqlserver/jdbc/KeyVaultCredential.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

...m/microsoft/sqlserver/jdbc/SQLServerStatement.java
59.36% <0%> (-0.19%)
135 <0> (ø)

...om/microsoft/sqlserver/jdbc/SQLServerBulkCopy.java
52.61% <0%> (-0.19%)
240 <0> (ø)

...QLServerColumnEncryptionAzureKeyVaultProvider.java
0% <0%> (ø)
0 <0> (ø)
:arrow_down:

.../microsoft/sqlserver/jdbc/SQLServerConnection.java
45.76% <31.25%> (+0.06%)
288 <0> (ø)
:arrow_down:

...oft/sqlserver/jdbc/SQLServerPreparedStatement.java
50.61% <53.12%> (-0.08%)
160 <10> (+3)

... and 24 more

Continue to review full report at Codecov.

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


No profile uri

Add please profile uri: "content://com.android.contacts/profile/data"

You can add your own content provider URIs via the "Add Content Provider" option in the overflow menu. :wink:


Docker images

Hi!
Great project! Are you considering to dockerize it and build images on dockerhub or github container registry? Thanks!

Thanks for your attention and advice, we are planning to add that later on

And if you want to take part in dockerizing the platform, just let us know in advance😄

I have implemented application orchestration using docker compose. Can I submit a PR?

Welcome! That's so cool. Just submit a PR and briefly describe your implementations if possible. And we can discuss more details there.

@BlankCheng, I can do the github action to build and upload to ghcr the image

Is there an update on this? I am searching for docker-related good first issues and it seems this issue should be closed?


remove a few templates from DEV, those starting with lc:RT

Hi Jeremy could you please remove the 9 templates on DEV whose IDs start with lc:RT?
please leave all the others (which start with ld4p:RT) in place
I'm using DEV as a place to show people the starter set of LC templates, and mistakenly uploaded a few before changing their names to ld4p
thanks!

The following resource templates have been removed from DEV:

lc:RT:bf2:AdminMetadata:BFDB
lc:RT:bf2:AdminMetadata
lc:RT:bf2:AdminMetadata:GenerationProcess
lc:RT:bf2:ReferenceInstance
lc:RT:bf2:RelatedExpression
lc:RT:bf2:RelatedInstance
lc:RT:bf2:RelatedWork
lc:RT:bf2:Relation
lc:RT:bf2:AdminMetadata:Status


The MI extension landing page pops up for non-MI projects

Description:
The MI extension landing page appears in the following scenarios when users have non-MI-related projects or XML files open:

When a user creates an XML file that is not part of an MI project.
When a user reloads the window with XML files from a non-MI project open.

This behaviour occurs only once after the VS Code window is loaded and will only reoccur if the window is reloaded again.

https://github.com/user-attachments/assets/9d40772e-9f6b-4441-9fc9-8f2a2bd95974
Steps to reproduce:
Affected Versions:
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

Duplicate: https://github.com/wso2/mi-vscode/issues/279


Misleading message for workflows coming back via lambda logs

Describe the bug
Error message claims that the needed property is "workflow" when it should be "workflows"
To Reproduce
Register a .dockstore.yml using "workflow" rather than "workflows" as a key
Look in lambda logs
Expected behavior
Term should be workflows
Screenshots
Use
version: 1.2
workflow:
   - subclass: NFL
     primaryDescriptorPath: /nextflow.config

(which is wrong)

Additional context
dev.dockstore.org
┆Issue is synchronized with this Jira Bug
┆Issue Type: Bug
┆Fix Versions: Dockstore 1.10
┆Sprint: Backlog
┆Issue Number: DOCK-1418

check webservice /workflows/github/release

closing because the message is saying it can't create the property workflow because it's not able to find the property workflow in our class definition of a dockstoreyml12. It's not saying the property should be workflow.

The error message could be a lot more user-friendly... What is a JavaBean and why is it in the error message? What is the signficance of c23b79? Why is it trying to create a property called workflow? Said a mythical scientist, probably.
A more user friendly message would be something like

Unrecognized key "workflow" in .dockstore.yml
Unrecognized property "workflow" in .dockstore.yml

As a bonus, which is probably not worth the work, you could say "Did you mean workflows?".
Not saying we should do all of this, especially the last part, or maybe not even any of it, but it could be improved.
BTW, the current error message is from the state I left the code in.

Seems better now

Error reading .dockstore.yml: Unrecognized property "workflow"
--

on dev.dockstore.org


WIP

For #

[ ] Pull request represents a single change (i.e. not fixing disparate/unrelated things in a single PR).
[ ] Title summarizes what is changing.
[ ] Has a news entry file (remember to thank yourself!).
[ ] Appropriate comments and documentation strings in the code.
[ ] Has sufficient logging.
[ ] Has telemetry for enhancements.
[ ] Unit tests & system/integration tests are added/updated.
[ ] Test plan is updated as appropriate.
[ ] package-lock.json has been regenerated by running npm install (if dependencies have changed).
[ ] The wiki is updated with any design decisions/details.

Codecov Report

Merging #13237 into master will decrease coverage by 0.02%.
The diff coverage is 57.14%.

@@            Coverage Diff             @@
##           master   #13237      +/-   ##
==========================================
- Coverage   59.63%   59.60%   -0.03%
==========================================
  Files         670      670
  Lines       36823    36843      +20
  Branches     5188     5189       +1
==========================================
+ Hits        21960    21962       +2
- Misses      13770    13784      +14
- Partials     1093     1097       +4

Impacted Files
Coverage Δ

src/client/common/application/notebook.ts
10.16% <0.00%> (-0.74%)
:arrow_down:

src/client/common/application/types.ts
100.00% <ø> (ø)

src/datascience-ui/renderers/constants.ts
0.00% <0.00%> (ø)

...ient/datascience/jupyter/kernels/kernelSelector.ts
68.10% <64.70%> (+0.34%)
:arrow_up:

...nt/datascience/jupyter/kernels/kernelSelections.ts
87.15% <66.66%> (-2.54%)
:arrow_down:

src/client/datascience/notebook/constants.ts
100.00% <100.00%> (ø)

src/client/common/utils/platform.ts
64.70% <0.00%> (-11.77%)
:arrow_down:

src/client/datascience/crossProcessLock.ts
79.41% <0.00%> (-11.77%)
:arrow_down:

src/client/linters/pydocstyle.ts
86.66% <0.00%> (-2.23%)
:arrow_down:

src/client/datascience/debugLocationTracker.ts
76.56% <0.00%> (-1.57%)
:arrow_down:

... and 1 more

Continue to review full report at Codecov.

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


Website isn't mobile optimized.

Fixed in https://github.com/auchenberg/coldfrontconf.com/commit/e6bbe2cf56f1b5c27805449ec9e396147baad120


Top predictions have the same title and para.

I am trying to return top 10 predictions by :
returning final_predictions_sorted i.e. in bertqa_sklearn.py
final_predictions_sorted = collections.OrderedDict(
sorted(final_predictions.items(),
key=lambda item: item[1]['start_logit'] + item[1]['end_logit'],
reverse=True))
But all different predictions have the same title and para attached with it,
Could @andrelmfarias  or @fmikaelian  help me with this.

Stupid question, figured it out.


Respect Newlines

I'm using the Autolink library to parse out links/phone numbers for certain blobs of text that may contain them, however it looks like every newline gets parsed out, regardless of if I use multiple lines, add \n, ,  etc etc.
Example:

This is a test for newlines

some text.\n
some other text

becomes

This is a test for newlines, 1. some text.\n 2. some other text

How do I fix this?

Looked more into this and it turns out there was a function elsewhere that stripped out \r's and \n's before passing it to autolink
Issue is resolved, I did notice that I would get a crash if i did something like
<Autolink text={'This is some text
with some newlines
to see what happens'} />

but that may just have more to do with how react-native handles strings


nothing is showing after launching the lidar.

I am using the rplidar a1 on ubuntu 22.04 with ros humble. I have also already done this sudo chmod 777 /dev/ttyUSB0
I am not getting anything to show up on rviz after inputting the command ros2 launch sllidar_ros2 view_sllidar_a1_launch.py

I have just pushed a PR that might help. This is for the A1: https://github.com/Slamtec/sllidar_ros2/pull/43 . I have tested on Ubuntu 22.04 and Humble


TimberPost should use PostGetter::get_post_class instead of relying on second argument

When using {{TimberPost}} inside a twig file, an instance of \Timber\Post is returned regardless of whether a PostClassMap filter exists.
Ideally, the TimberPost php implementation would make the $PostClass argument optional and call PostGetter::get_post_class as a default while still retaining the ability to specify the class manually.
Would you welcome a pull request for this feature?

@kshaner I'm totally open to seeing a PR. If it's pretty simple — go for it! But if you're finding it's a big re-write to make it happen, take a look at maybe doing the PR against the 2.x branch. We're trying to really shift our focus over there so we can get it released soon

@kshaner more thoughts — I'd be interested to see where else the class map could be used. Is this something where defaults could be set at the theme level, and then (probably via a filter) used throughout Timber so a dev doesn't have to constantly re-send the value?

@jarednova, Agreed wholeheartedly with this approach.  We're attempting to use it that way and have run into some strange class override scenarios, such as when using {{post.parent}}. It would be nice to get rid of all the $PostClass properties of the various classes and instead reply on the class map.
With this route, it may be a bigger PR and should probably target 2.x.

Yeah, {{ post.parent }} is another good example of where that appears. I think going for 2.x makes sense. Thanks for taking this on!

Hey all!
Making sure that the post class map is used in all Timber functionality while still being able to use custom classes in certain situations will be part of the object factory refactor in #1793. That would speak against making a separate pull request for this.

Thanks @gchtr. That's exactly the route I would have gone. Thanks for letting me know before I started.

NOW I get why the object factory PRs matter so much — thanks for clarifying @gchtr

See also #2073, #2090


Update the game over message

Update the game over message so people know how to play again!

This pull request is fine


Adding dataset Incunabula Reichenau

Hello HTR-United team!
please consider the following data set description for inclusion in your directory.
As the referenced data set contains pages from ~200 prints, a full description would
be impractical. Please let me know if any changes are required.
Thank you for your amazing work!
Here is our dataset YAML file:
schema: https://htr-united.github.io/schema/2023-06-27/schema.json
title: Incunabula Reichenau
url: https://doi.org/10.5281/zenodo.11046061
authors:
 - name: Annika
   surname: Stello
   orcid: 0000-0002-6305-4810
   roles:
     - project-manager
 - name: Gerit
   surname: Heim
   orcid: 0000-0002-5820-7771
   roles:
     - project-manager
 - name: Katharina
   surname: Ost
   orcid: 0000-0002-6234-9721
   roles:
     - transcriber
institutions: []
description: >-
 This data set contains the training data for the following three published
 Transkribus models:

 German Incunabula (Reichenau)
 Latin Incunabula (Reichenau)
 Latin/German Bilingual Incunabula (Reichenau)

 This data set represents an excerpt of a collection of incunabula and post-incunabula
 of the   former Reichenau monastery, now held at the Badische Landesbibliothek in
 Karlsruhe (see https://digital.blb-karlsruhe.de/topic/view/7530707). As, typically,
 1-20 pages were drawn from single prints, it reflects a wide range of typefaces used
 by early printers from the German language area and Northern Italy.

 The data was created as part of the project Digitalisierung und Volltexterkennung
 der ehemals Reichenauer Inkunabeln at the Badische Landesbibliothek, which was
 funded by the Stiftung Kulturgut Baden-Württemberg.
project-name: Digitalisierung und Volltexterkennung der ehemals Reichenauer Inkunabeln
language:
 - lat
 - deu
production-software: Transkribus
automatically-aligned: false
script:
 - iso: Latn
 - iso: Goth
script-type: only-typed
time:
 notBefore: '1470'
 notAfter: '1510'
hands:
 count: more-than-10
 precision: exact
license:
 name: CC-BY-SA 4.0
 url: https://creativecommons.org/licenses/by-sa/4.0/
format: Page-XML
volume:
 - metric: pages
   count: 2200
transcription-guidelines: Abbreviations are represented through special characters, please see the project repository for a full documentation.

Hello Katharina,
This is really nice, many thanks for your contribution! As you can see, I created the corresponding pull request, so the dataset is now added to the database ! :)


Homework 3

What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
What did you learn from the changes you have made?
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
How were these changes tested?
A reference to a related issue in your repository (if applicable)
Checklist

[ ] I can confirm that my changes are working as intended

Good Job with HW3

For HW2, For Where query task, use the order by command to sort the result.

HW 4 missing.

HW 4 missing.

sorry I will submit all the HW tomorrow.


Wrong item counter

Steps to reproduce the problem

Add 2 pieces of one product
Open cart tab

What I expected to happen
Same counter both on cart icon and 'items counter' in sidebar
Screenshots
Same cart:

System information
Operating system: OSX
Browser: Chrome

i think you misunderstand the concept here, you are buying 1 item with quantity of 2, not 2 items.

I've discussed this with the author of design. Both counters should be the same.


List parameter to WebAPI action shows up as ObservableCollection in generated client

Given the following WebAPI action
[ResponseType(typeof(List<int>))]
[Route("Report/{ids}/{from}/{to}")]
public IHttpActionResult Get(List<int> ids, DateTime from, DateTime to)
{
    return Ok(ids);
}

When you generate a client with swagger2csclient the parameter of type List<int> is converted to ObservableCollection<object>
And the following code cannot compile
var result = client.GetAsync(new ObservableCollection<int>(new List<int> { 1, 2, 3 }),
    new DateTime(2016, 6, 1), new DateTime(2016, 7, 1)).Result;

I have tried specifying the type in the route, e.g. [Route("Report/{ids:List<int>}/{from}/{to}")] but no change.
Any ideas?

What about:
var result = client.GetAsync(new ObservableCollection<int>() { 1, 2, 3 },
    new DateTime(2016, 6, 1), new DateTime(2016, 7, 1)).Result;

You can also change the array type to list (ArrayType property on settings)

... but yes, the array operation parameters should always be of type IEnumerable<...>

That example doesn't work either because the generated client is expecting ObservableCollection even though the actual WebAPI parameter is List.
I've changed the ArrayType to "List" but it still loses the T type and converts to List instead. Maybe the current code doesn't support generics?

The swagger spec only knows array and the generator uses the type from the settings to generate the .NET type... I think the problem is, that the currently generated code does not support path parameters of type array/list. Can you post the generated GetAsync method?

Fixed in https://github.com/NSwag/NSwag/commit/995c20fccd866bb95f49ff9ae53aabe07c0b2cde, NuGet v3.23, please reopen if not solved...


Use UV and Justfiles

Description
We need to switch to UV for python, which will improve build times and venv creation times and swap Makefile out with a Justfile for simplicity.
Related Issue(s)

I'm not familiar with uv but I don't understand what makes Justfiles simple? I've seen a few before since I believe `@roeap likes the tool, but I don't understand the benefit over GNU/Make.
I tend to prefer GNU/Make because it's on every Linux and macOS (gmake) install by default, but I'm not married to it

I'm not familiar with uv but I don't understand what makes Justfiles simple? I've seen a few before since I believe `@roeap likes the tool, but I don't understand the benefit over GNU/Make.
I tend to prefer GNU/Make because it's on every Linux and macOS (gmake) install by default, but I'm not married to it

I found the syntax more simple, and its written in Rust 😆🦀


Adding lateral style icon and collapsed text

Closes #985
Within the screener improvements project there are a few tweaks that would be great to have in AccordionToggle.

Allow the chevron to not go from full up to full down, instead do a quarter turn from sideways to down
Display collapsed text which only shows if the accordion is closed

Mega short loom of them in action: https://www.loom.com/share/5b159ce1c36749869f663a885005fc63
I achieved these changes with a small amount of code:

Added chevronLateral prop which when present adds a class that is used to do a little rotation to the SVG icon, note it doesn't look great when on the right, more natural when to the left.
Add collapsedText prop which only shows if present and when isCollapsed is true

Hopefully I can get away with these changes! I think they are nice additions and work well but let's iterate if not... definitely want to keep using Accordion system instead of building my own show hide this this one is loooovely. Moar flattery but stellar notes and scripts to link the live version of the DS with rails server. So straightforward! 🙌
Also!!! If you look at the screenshots from figma you can see there is going to be a dropdown in the AccordionToggle as well. I played around with passing in a dropdown prop but kinda thinking the best way to do this might be to use absolute positioning instead but your thoughts would be great.
Figma: https://www.figma.com/file/kY6sLQRhR5FZ70fUZvCmdp/Screener-Improvements?node-id=2056%3A79169&mode=dev
Figma screenshots

Used within Rails Server

Storybook screenshots

@jasonbasuil mind merging? I don't have access but it should be all ready to go.
I do need this for the work I am doing in the screener can we talk getting it released? 😸  Heading out for a coffee walk will ping when I return.
Also super joy to work in this code base, thanks for all you do! 🙌


Sched combiner closures instead of running to avoid data races

A change similar to https://github.com/grpc/grpc/pull/18491.
GRPC_CLOSURE_RUN runs the closure immediately. In our case, if we do that, we can end up with a use-after-free kind of bug, since we need to reference some elements in the op structure even after scheduling the closure.
By scheduling instead of running, we guarantee that the closures will not be run before we are done using the struct since we are using a combiner.
Fixes internal bug - b/132890439

Known issues : #19422, #19170


(Failed to compile) Module not found:

Web browser and version
Chrome 90.0.4430.212 (Official Build) (64-bit) (cohort: Stable)
Operating System
Windows 10
Steps to reproduce this

clone repository
run npm install to download relevant dependencies
npm run start
Failure to Compile occurs

Additional context

A quickfix:
Go into node_modules/react-data-grid/ and create a new folder 'dist' containing 'react-data-grid.css'. I left the css file empty, and the site locally compiled sucessfully! Hopefully these are just missing files that can be added to the repo, and it seems like they aren't necessary for the most part.

Mixing package manager is not advisable and should be avoided. The issue you are experiencing depends on you using npm, while we use yarn to manage packages: hence, we provide the yarn.lock and not the package-lock.json. Switching to yarn as stated in the readme fixes the issue.

@makhas as suggested by @osioalberto use yarn (and not npm) to install dependencies. Here the link to the install instruction: https://github.com/rawgraphs/rawgraphs-app#instructions-mac-os

Only for the records, I had the same problem here even using yarn. @makhas log messages show that he is running Windows. I am using Linux. Maybe npm/yarn repos are missing something.  Quickfix worked just fine!


add "Vue USWDS" to list of packages

can you resolve the conflicts?

@vuesomedev conflicts should be fixed now.


Missing permission instructions

[Enter feedback here]
Required permissions for ARO deploy is absent from this document. Please see https://docs.microsoft.com/en-us/azure/openshift/tutorial-create-cluster#verify-your-permissions. Thanks

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

ID: 93aa980e-1970-235c-228a-7b19880171b8
Version Independent ID: 6f7937a4-12e0-d61c-3bd8-c6347ec48385
Content: Tutorial - Create an Azure Red Hat OpenShift 4 cluster
Content Source: articles/openshift/tutorial-create-cluster.md
Service: container-service
GitHub Login: @sakthi-vetrivel
Microsoft Alias: suvetriv

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

Thank you for your feedback. This article has been revamped since this feedback was submitted, and it seems like this issue has been addressed through those changes. Please let me know if it's not.
#please-close


No menu items available in  Selection menu

Menu "Selection" available, but all options aregreyed out. Should we have this menu option available?

Most of these don't make sense for the diagram. Which ones would you like to use?
I don't think that Theia can deactivate entire menus.

If we use the diagram display and if none of the menu items apply to any diagram, then the menu "Selection" should not clutter the menu bar, it should be invisible when the user is editing/viewing a diagram

Then this issue should go to Theia :-)

Issue move to THEIA: https://github.com/theia-ide/theia/issues/731


one thing for result

The yolo can check many things at the same time, how can i get the one thing from the results

In the .yaml file you could delete all the names that you don't want and leave the single object that you want as a result.
You could also train yolo for a custom object from here https://github.com/AlexeyAB/darknet#how-to-train-to-detect-your-custom-objects and then add it to darknet_ros with these instructions https://github.com/WalkingMachine/sara_wiki/wiki/Create-a-custom-dataset-(YOLO)

Thank you very much for your help.I delete the object which I don't want in the .yaml.It's successd,but today I roslaunch again, but it dossn't work,it can't dectect the things in the .yaml. Why ?

What is the error message?


Improve serialiser handling of attributes, text nodes

Resolves https://github.com/manoelcampos/xml2lua/issues/41

[x] Add attributes to rendered tags that are missing them
[ ] Support text node rendering without <%d+></%d+>
[ ] Don't unescape CDATA when parsing and serialising tree.

I'm thinking the DOM handler is easier to serialise. It would also permit other XML types to be serialised appropriately.

Hello @lmn8
Sorry for the late reply, but I'm really busy these days.
Are you planning to fix the other items?
Because that would be new issues.
If you are done already, I'll try the code for merging.
I checked quickly and seems to be OK.
Thanks for contributing.

You can merge it. The other issues (CDATA, text node rendering) are going to require handler changes too, so you would need to bump the version.
At some point, I can open a pull that will allow serialising DOM handler roots.


perf(client): request support query in url

What this PR does / why we need it:
perf(client): request support query in url
Which issue(s) this PR is related to (optional, link to 3rd issue(s)):
Fixes #
Reference to #

Special notes for your reviewer:
/cc @iawia002

Release note:

Client request now supports query in url

/cherrypick release-0.3
/lgtm


Integrate recent changes from old repo

@majordamage,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/concepts/bots/bots-context.md
:warning:Warning

Details

msteams-platform/concepts/bots/bots-conversations.md
:warning:Warning

Details

msteams-platform/publishing/office-store-guidance.md
:warning:Warning

Details

msteams-platform/concepts/bots/bots-context.md

[Warning] Ignore metadata: search.ms_sitename, search.ms_product, search.ms_docsetname. They are generated by open publish.
Line 23: [Warning] Invalid file link:(~/concepts/bots/botevents.md#bot-or-user-added-to-a-team).
Line 119: [Warning] Invalid file link:(~/concepts/bots/botevents.md#bot-or-user-added-to-a-team).

msteams-platform/concepts/bots/bots-conversations.md

[Warning] Ignore metadata: search.ms_sitename, search.ms_product, search.ms_docsetname. They are generated by open publish.
Line 17: [Warning] Invalid file link:(~/concepts/bots/bots-notifications).
Line 72: [Warning] Invalid file link:(~/concepts/bots/bots-conversations#cards).
Line 133: [Warning] Invalid file link:(~/concepts/bots/bots-notifications#channel-updates).
Line 183: [Warning] Invalid file link:(~/scenarios/bots-channel-conversations#mentions).
Line 189: [Warning] Invalid file link:(~/concepts/bots/bots-notifications#channel-updates).
Line 193: [Warning] Invalid file link:(~/concepts/bots/bots-notifications#team-name-updates).
Line 196: [Warning] Invalid file link:(~/concepts/bots/bots-notifications#channel-updates).
Line 235: [Warning] Invalid file link:(~/concepts/cards-actions).
Line 305: [Warning] Invalid file link:(~/scenarios/bots-personal-conversations#starting-a-11-conversation).
Line 306: [Warning] Invalid file link:(~/scenarios/bots-channel-conversations#creating-a-channel-conversation).

msteams-platform/publishing/office-store-guidance.md

[Warning] Ignore metadata: search.ms_sitename, search.ms_product, search.ms_docsetname. They are generated by open publish.
Line 9: [Warning] Invalid file link:(~/publishing/apps-publish).
Line 72: [Warning] Invalid file link:(~/publishing/apps-publish#microsoft-teams-app-approval-process).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[Tooltip]Tooltip is invalid on ios & android

Description
I try https://snack.expo.dev/@viraj_10/tooltip-native-onpress, but I still can't open tooltip on ios&android
CodeSandbox/Snack link
https://snack.expo.dev/@viraj_10/tooltip-native-onpress
Steps to reproduce
I try https://snack.expo.dev/@viraj_10/tooltip-native-onpress, but I still can't open tooltip on ios&android
gluestack-ui Version
1.0.35
Platform

[ ] Expo
[ ] React Native CLI
[ ] Next
[ ] Web
[X] Android
[X] iOS

Other Platform
No response
Additional Information
No response

@BooYeu
The 'tooltip-native-onpress' feature is operational on both iOS and Android, activated by a long press on the respective views. However, it's worth noting that when using a mobile device, it is necessary to hold the button for a brief duration. If the press is too quick, the hover effect may not become visible. Likewise, on the web, the hover effect might not be triggered if the cursor is moved rapidly, adhering to the anticipated behavior.
https://github.com/gluestack/gluestack-ui/assets/108934337/3f292e4c-878e-4e22-ba35-9b18ce01d273

Can you add some props to show tooltip when use click it ? Because users rarely long press something in ios&android

@BooYeu, As shown in example. we are using onPressIn and onPressOut to control isOpen prop in tooltip.
You can choose any event which you feel is right for your application and control isOpen prop.
You can use onPress to toggle isOpen or fire a custom event which makes isOpen false.
Closing this issue, Since tooltip is working on native platform. You can further open discussion.


can support download .git folder ?

.git is meaningful when check history of some code

also .git is useful as many editors rely on on .git to figure out the project root folder where things like LSP will be based on


Theme: Table icons randomly change on-hover in Microsoft Edge

When I set:
Animate table icons on focus: Yes
Grayscale table icons and bring color on focus only: Yes
and mouse around the page, all of the icons except for the one I might be hovering over flicker and change images randomly. I'm using MS Edge.

Hi,
Yes, I remember Edge didn't render it properly. I will try to fix it by the next release in case it's technically possible.
Thanks for reporting.

Wow! Certainly appreciate the lightning fast response. Awesome, professional theme. Thank you!

Thanks, you're welcome.

I have no idea why this is happening. :disappointed:

I disabled icon animations and all is well since. Very visually appealing theme. I use Webmin a lot and this theme a very comfortable.

I will try fixing it but I'm not sure, that it's technically possible.
I would highly recommend using Google Chrome or Mozilla Firefox for the best experience!

Hello,
@qooob, same behavior on Firefox 47.x
@RedstoneFiend : How do you disabled icon animation?
Thx
PS I love the theme very much :-)

Not sure what you mean by saying that it's the same in Firefox.
Animation can be disabled in theme settings.
Please provide videoscreencast, I would like to take a look as I can't reproduce it - Firefox works perfect.
What version of the theme are you using?

@qooob yes, you're right, it works well with Firefox... Actually when I stated the issue, I thought I was under Firefox (which I use by default) , but instead I was using IE... Sorry!

lebrach, the settings are in my original post above. Webmin Configuration -> Webmin Themes -> Table Options section.

Alright, it will be fixed in 18.20.
Support for 4 different browsers is really extensively funny work to do.

Version 18.20 is out, please upgrade.


Clarify conversion of storage account

This page is mixing different concepts, at first it talks about data replication however going deeper you're talking about storage account conversion.
My advise would be to:

split storage account conversion to a new distinct page
clarify if while we convert a storage account the source account is: lost to a new one, replaced by the new configuration and so on..

Thanks for your help
Regards
Olivier

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

ID: 66ebb48e-c896-a747-bc42-aa203978a1cd
Version Independent ID: 6e7bfefb-e67c-b5d5-89f7-6f1b6a2e8b61
Content: Change how a storage account is replicated - Azure Storage
Content Source: articles/storage/common/redundancy-migration.md
Service: storage
Sub-service: common
GitHub Login: @jimmart-dev
Microsoft Alias: jammart

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

@olileger
Thanks for bringing this to our attention.
I'm going to assign this to the document author so they can take a look at it accordingly.

@jimmart-dev
Can you please check and add your comments on this doc update request as applicable.

@jimmart-dev @AjayBathini-MSFT do you have any update please ?

Thanks for your contribution to our documentation
Unfortunately, we have been unable to review this issue in a timely manner. We sincerely apologize for the delayed response. We are closing this issue. If you feel that the problem persists, please respond to this issue with additional information.
?
Please continue to provide feedback about the documentation. We appreciate your contributions to our community.
#please-close


Fury-parent 0.8.0 artifact was not uploaded to Maven central

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
0.8.0
Component(s)
Java
Minimal reproduce step
Search at Maven Central: https://central.sonatype.com/artifact/org.apache.fury/fury-parent
you will find 0.8.0 for fury-core but you won't find 0.8.0 fury-parent.
What did you expect to see?
0.8.0 for fury-parent present in Maven central
What did you see instead?
fury-parent 0.8.0 is not present, the latest one is 0.7.1
Anything Else?
No response
Are you willing to submit a PR?

[ ] I'm willing to submit a PR!

Hi @juliojgd  does https://repository.apache.org/content/repositories/orgapachefury-1050/ work for you? If it works, I can publish it to maven central now

Looks good

fury-parent pom released now


Private Disclosure

Hello, Do you have an email for private disclosure?

Hi @omriinbar,
Dularion would be the best person to deal with any disclosure of information her discord is dularion#6854
https://discord.gg/CJEHWX9
If you are unable to send the message or need someone else you can send it to me however I am not a contributing member to streama my discord is Wiruses#4612 and if you require I can provide you an email over a github DM if you do not have discord and I can pass it on to dularion


An error ocurred while starting the kernel

Issue Report Checklist

[ ] Searched the issues page for similar reports
[ ] Read the relevant sections of the Spyder Troubleshooting Guide and followed its advice
[ ] Reproduced the issue after updating with conda update spyder (or pip, if not using Anaconda)
[ ] Could not reproduce inside jupyter qtconsole (if console-related)
[ ] Tried basic troubleshooting (if a bug/error)

[ ] Restarted Spyder
[ ] Reset preferences with spyder --reset
[ ] Reinstalled the latest version of Anaconda
[ ] Tried the other applicable steps from the Troubleshooting Guide

[ ] Completed the Problem Description, Steps to Reproduce and Version sections below

Problem Description
Traceback (most recent call last):
File "C:\ProgramData\Anaconda3\lib\runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "C:\ProgramData\Anaconda3\lib\runpy.py", line 85, in run_code
exec(code, run_globals)
File "C:\ProgramData\Anaconda3\lib\site‑packages\spyder_kernels\console_main.py", line 3, in
start.main()
File "C:\ProgramData\Anaconda3\lib\site‑packages\spyder_kernels\console\start.py", line 270, in main
import_spydercustomize()
File "C:\ProgramData\Anaconda3\lib\site‑packages\spyder_kernels\console\start.py", line 30, in import_spydercustomize
import spydercustomize
File "C:\ProgramData\Anaconda3\lib\site‑packages\spyder_kernels\customize\spydercustomize.py", line 17, in
import pdb
File "C:\ProgramData\Anaconda3\lib\pdb.py", line 76, in
import code
File "C:\Users\seaso\code.py", line 1, in
import sample
File "C:\Users\seaso\sample.py", line 2, in
print(hello(TOM));
NameError: name 'hello' is not defined
Traceback (most recent call last):
File "C:\ProgramData\Anaconda3\lib\runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "C:\ProgramData\Anaconda3\lib\runpy.py", line 85, in run_code
exec(code, run_globals)
File "C:\ProgramData\Anaconda3\lib\site‑packages\spyder_kernels\console_main.py", line 3, in
start.main()
File "C:\ProgramData\Anaconda3\lib\site‑packages\spyder_kernels\console\start.py", line 270, in main
import_spydercustomize()
File "C:\ProgramData\Anaconda3\lib\site‑packages\spyder_kernels\console\start.py", line 30, in import_spydercustomize
import spydercustomize
File "C:\ProgramData\Anaconda3\lib\site‑packages\spyder_kernels\customize\spydercustomize.py", line 17, in
import pdb
File "C:\ProgramData\Anaconda3\lib\pdb.py", line 76, in
import code
File "C:\Users\seaso\code.py", line 1, in
import sample
File "C:\Users\seaso\sample.py", line 2, in
print(hello(TOM));
NameError: name 'hello' is not defined
Traceback (most recent call last):
File "C:\ProgramData\Anaconda3\lib\runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "C:\ProgramData\Anaconda3\lib\runpy.py", line 85, in run_code
exec(code, run_globals)
File "C:\ProgramData\Anaconda3\lib\site‑packages\spyder_kernels\console_main.py", line 3, in
start.main()
File "C:\ProgramData\Anaconda3\lib\site‑packages\spyder_kernels\console\start.py", line 270, in main
import_spydercustomize()
File "C:\ProgramData\Anaconda3\lib\site‑packages\spyder_kernels\console\start.py", line 30, in import_spydercustomize
import spydercustomize
File "C:\ProgramData\Anaconda3\lib\site‑packages\spyder_kernels\customize\spydercustomize.py", line 17, in
import pdb
File "C:\ProgramData\Anaconda3\lib\pdb.py", line 76, in
import code
File "C:\Users\seaso\code.py", line 1, in
import sample
File "C:\Users\seaso\sample.py", line 2, in
print(hello(TOM));
NameError: name 'hello' is not defined
Traceback (most recent call last):
File "C:\ProgramData\Anaconda3\lib\runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "C:\ProgramData\Anaconda3\lib\runpy.py", line 85, in run_code
exec(code, run_globals)
File "C:\ProgramData\Anaconda3\lib\site‑packages\spyder_kernels\console_main.py", line 3, in
start.main()
File "C:\ProgramData\Anaconda3\lib\site‑packages\spyder_kernels\console\start.py", line 270, in main
import_spydercustomize()
File "C:\ProgramData\Anaconda3\lib\site‑packages\spyder_kernels\console\start.py", line 30, in import_spydercustomize
import spydercustomize
File "C:\ProgramData\Anaconda3\lib\site‑packages\spyder_kernels\customize\spydercustomize.py", line 17, in
import pdb
File "C:\ProgramData\Anaconda3\lib\pdb.py", line 76, in
import code
File "C:\Users\seaso\code.py", line 1, in
import sample
File "C:\Users\seaso\sample.py", line 2, in
print(hello(TOM));
NameError: name 'hello' is not defined
Traceback (most recent call last):
File "C:\ProgramData\Anaconda3\lib\runpy.py", line 193, in _run_module_as_main
"main", mod_spec)
File "C:\ProgramData\Anaconda3\lib\runpy.py", line 85, in run_code
exec(code, run_globals)
File "C:\ProgramData\Anaconda3\lib\site‑packages\spyder_kernels\console_main.py", line 3, in
start.main()
File "C:\ProgramData\Anaconda3\lib\site‑packages\spyder_kernels\console\start.py", line 270, in main
import_spydercustomize()
File "C:\ProgramData\Anaconda3\lib\site‑packages\spyder_kernels\console\start.py", line 30, in import_spydercustomize
import spydercustomize
File "C:\ProgramData\Anaconda3\lib\site‑packages\spyder_kernels\customize\spydercustomize.py", line 17, in
import pdb
File "C:\ProgramData\Anaconda3\lib\pdb.py", line 76, in
import code
File "C:\Users\seaso\code.py", line 1, in
import sample
File "C:\Users\seaso\sample.py", line 2, in
print(hello(TOM));
NameError: name 'hello' is not defined
What steps reproduce the problem?

Start the spyder ide
It fails to connect to kernel and displays the above error

What is the expected output? What do you see instead?
Expected output is ipython kernel gets connected.
Paste Traceback/Error Below (if applicable)

PASTE TRACEBACK HERE

Versions

Spyder version: 3.3.0
Python version: 3.6.6 64bits
Qt version: 5.9.4
PyQt version: 5.9.2
Operating System name/version:  Windows 10 Pro

Dependencies

IPython >=4.0     :  6.5.0 (OK)
cython >=0.21     :  0.28.4 (OK)
jedi >=0.9.0      :  0.12.1 (OK)
matplotlib >=2.0.0:  2.2.2 (OK)
nbconvert >=4.0   :  5.3.1 (OK)
numpy >=1.7       :  1.15.0 (OK)
pandas >=0.13.1   :  0.23.3 (OK)
pycodestyle >=2.3 :  2.4.0 (OK)
pyflakes >=0.6.0  :  2.0.0 (OK)
pygments >=2.0    :  2.2.0 (OK)
pylint >=0.25     :  1.9.2 (OK)
qtconsole >=4.2.0 :  4.3.1 (OK)
rope >=0.9.4      :  0.10.7 (OK)
sphinx >=0.6.6    :  1.7.6 (OK)
sympy >=0.7.3     :  1.2 (OK)

The problem is the file C:\Users\seaso\code.py. This file has the same name as a standard library in Python; see https://docs.python.org/3/library/code.html . This confuses Python and Spyder. Please do not use code.py as a file name and rename the file C:\Users\seaso\code.py; this should fix your problem.

Yes, that's the solution @HYL-Dave.


the context biasing that has not yet the end boundary does not subtract the previous offset score.

Describe the issue
At the end of the ctc prefix search, the context that has not yet the end boundary does not subtract the previous offset score.
To Reproduce
My test context is "千玺机器人" ，context_score= 3.0, and my test pronouncing is "qianxi" .  The recognition results are as follows：

When the ctc search was over,  a relatively low score for "迁徙" because the context state of "千玺" was not set to the start state with negative score.
Expected behavior
By overwritting 'CtcPrefixBeamSearch.FinalizeSearch()' to fix it ?

@pengzhendong , what do you think?

@luchuanze We've talked about this before. You could commit a pr, thanks!

@luchuanze我们以前讨论过这个。你可以提交一个公关，谢谢！

@luchuanze We've talked about this before. You could commit a pr, thanks!

OK,


feat: Detect Chargebee Access Tokens

Description:
This PR adds a detection for Chargebee access tokens.
Checklist:

[x] Does your PR pass tests?
[x] Have you written new tests for your changes?
[x] Have you lint your code locally prior to submission?

Followed Guidelines mentioned in Contributing.md file

@zricethezav @baruchiro Can one of you assist with reviewing this PR please ?

Thanks @baruchiro for the quick response. 🙌 . I hope this PR will be merged and changes will be part of upcoming releases.

Thanks @baruchiro for the quick response. 🙌 . I hope this PR will be merged and changes will be part of upcoming releases.


brew services start opener errors out

Running on Ubuntu 20.0.4
❯ brew install superbrothers/opener/opener
==> Tapping superbrothers/opener
Cloning into '/home/linuxbrew/.linuxbrew/Homebrew/Library/Taps/superbrothers/homebrew-opener'...
remote: Enumerating objects: 19, done.
remote: Counting objects: 100% (19/19), done.
remote: Compressing objects: 100% (13/13), done.
Receiving objects: 100% (19/19), 7.50 KiB | 7.50 MiB/s, done.
Resolving deltas: 100% (6/6), done.
remote: Total 19 (delta 6), reused 16 (delta 4), pack-reused 0
Tapped 1 formula (14 files, 72.9KB).
==> Downloading https://github.com/superbrothers/opener/releases/download/v0.1.0/opener-darwin-amd64.zip
==> Downloading from https://objects.githubusercontent.com/github-production-release-asset-2e65be/343804754/c76a3fcc-e4af-47f5-b2cf-cd410087704c?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAIWNJYAX4CSVEH53A%2F20220621%2Fus-east-
######################################################################## 100.0%
==> Installing opener from superbrothers/opener
==> Caveats
Warning: opener provides a launchd plist which can only be used on macOS!
You can manually execute the service instead with:
  opener
==> Summary
🍺  /home/linuxbrew/.linuxbrew/Cellar/opener/0.1.0: 6 files, 4.1MB, built in 1 second
==> Running `brew cleanup opener`...
Disable this behaviour by setting HOMEBREW_NO_INSTALL_CLEANUP.
Hide these hints with HOMEBREW_NO_ENV_HINTS (see `man brew`).
❯ brew services start opener
==> Tapping homebrew/services
Cloning into '/home/linuxbrew/.linuxbrew/Homebrew/Library/Taps/homebrew/homebrew-services'...
remote: Enumerating objects: 2025, done.
remote: Counting objects: 100% (534/534), done.
remote: Compressing objects: 100% (191/191), done.
remote: Total 2025 (delta 362), reused 456 (delta 334), pack-reused 1491
Receiving objects: 100% (2025/2025), 554.78 KiB | 18.49 MiB/s, done.
Resolving deltas: 100% (917/917), done.
Tapped 1 command (45 files, 815.4KB).
Error: Formula `opener` has not implemented #plist, #service or installed a locatable service file
❯ brew services start opener
Error: Formula `opener` has not implemented #plist, #service or installed a locatable service file

Thanks for trying opener. The formula for opener does not support Linux (systemctl). I will try to support it.
However, let me check. Are you correct that you want to use opener on Linux Desktop? opener does not need to be installed on a Linux server.

thanks @superbrothers, it installed successfully. Will complete rest of installation steps


Steaming with ijkplayer

Add this library if it meet criteria https://github.com/Bilibili/ijkplayer

I think the README is too complex.
@dkhamsing @naeemshaikh90 what do you guys think?

Let's skip it


Mid-July Tasks

[ ] Verify that issue with jupyter lab vs jupyter notebook was taken care of.
[ ] Think about adding instructions on package compiler + Plots to advanced setup instructions, or something like that.
[ ] Think through whether using the built-in conda/jupyter is better for users to start out.
[ ] Update the setup instructions once we pin down the best solution for activate_github_path etc.

I am going to spin out a separate issue for the activate_github_path from this metaissue for discusison

These are being spun out, so will close


Allow additional waiting strategy for the Database to be available

Feature description
Unfortunately, there is something different with timing when Rancher Desktop is used which makes Postgres (probably Oracle) container tests fail because it's reported as ready(it uses LogMessageWaitStrategy), but it still takes some time for the port to be exposed or something...
It would be nice to have some additional availability checks that can be enabled. I can see two options:

Await for successful SQL query, that is the default Testcontainers check:
https://github.com/testcontainers/testcontainers-java/blob/7ba6b1c518d8f181b5831229977c260dbff803dc/modules/jdbc/src/main/java/org/testcontainers/containers/JdbcDatabaseContainer.java#L143-L175
Unfortunately, it's overridden by Postgres and Oracle with LogMessageWaitStrategy.
This check might be the most bulletproof but might need a custom configuration of the SQL check query.

Wait for the port to be available. We can probably reuse Testcontainer's HostPortWaitStrategy.

Relates to #123

Just chiming in with some experiences:

This check might be the most bulletproof but might need a custom configuration of the SQL check query.

One would think so, but it is explicitly flaky in the case of the default PostgreSQL image. This originates from the fact, that the PostregeSQL database restarts after bootstrapping its internal tables if the database was started without a data directory being present. This allowed for a small window of a race condition, where the JDBC query would succeed, but the database engine would proceed to perform the restart. The LogMessageWaitStrategy was introduced as a more stable criteria for readiness and from our experience, it works more reliably in the case of PostgreSQL.
Regarding the different behavior, I like to point to this comment:
https://github.com/testcontainers/testcontainers-java/issues/5986#issuecomment-1275496406
Rancher Desktop has vastly different runtime characteristics compared to Docker Desktop and Testcontainers does not plan to support them explicitly.
That being said, for the issue at hand, using WaitAllStrategy can be a way to check for multiple indicators of readiness.

@kiview Great insights! Maybe you can suggest what would be the solution to it. @melix created a PR for custom wait strategies https://github.com/micronaut-projects/micronaut-test-resources/pull/141


Inconsistent in Generating Dataset Interface and Query Detail in Dashboard

Hi @dabico,
I found that the information in the interface of generating the dataset is inconsistent with the information in the query detail of the dashboard.
For example, I select "at least 10 contributors" on the interface of generating the dataset and click the generating dataset. However, the query detail in the dashboard shows me that "min_contributors = 0" and "min_issues = 10". (see following Figures)

This situation confuses me. Could you kindly take a look at what happened to this issue? Is the problem in the front end?

https://github.com/seart-group/DL4SE/blob/32b4a0c2a099e1718fbbf0816e1ea614a723c957/dl4se-website/src/views/TaskCreateView.vue#L49-L66
Thank you for reporting, this is an actual bug introduced by refactoring. The issues filter was mapped to modify the state of the min_contributors value and vice-versa (🤦). I'll create a fix, and let you know as soon as it is live.

@SpanShu96
I pushed the fix and redeployed, took a while because of airport WIFI. You may have to re-run your query though, since that last one used min_issues = 10. Apologies for the inconvenience. I'll mark this as closed. If you encounter more issues with this, feel free to drop a line.

@SpanShu96
I pushed the fix and redeployed, took a while because of airport WIFI. You may have to re-run your query though, since that last one used min_issues = 10. Apologies for the inconvenience. I'll mark this as closed. If you encounter more issues with this, feel free to drop a line.

@dabico
I am sorry for replying late. Thank you for your generous help in time!
I can now generate the dataset and download it. No need to apologize! I should thank you for creating this tool, which brings convenience to my research (save time to write my own crawler):bow:. Thank you very much!
I see. I will update you if I encounter more issues.


feat: add @vuepress/bundler-farm

🚧 Experimental + Work in Progress 🚧

Before submitting the PR, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Provide a description in this PR that addresses what the PR is solving. If this PR is going to solve an existing issue, please reference the issue (e.g. close #123).

What is the purpose of this pull request?

[ ] Bug fix
[x] New feature
[ ] Documentation update
[ ] Other

Description
Ref: #1552

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 2 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.03%) to 38.576%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/vite-kit/src/plugins/vuepressVuePlugin.ts
0
2
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
631

Relevant Lines:
1663

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 2 (0.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.03%) to 38.576%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/vite-kit/src/plugins/vuepressVuePlugin.ts
0
2
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.03%

Covered Lines:
631

Relevant Lines:
1663

💛  - Coveralls

Looking forward to it (๑•ૅω•´๑)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 28 (0.0%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.6%) to 38.04%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/bundler-farm/src/resolveFarmConfig.ts
0
2
0.0%

packages/vite-kit/src/plugins/vuepressVuePlugin.ts
0
2
0.0%

packages/bundler-farm/src/farmBundler.ts
0
3
0.0%

packages/bundler-farm/src/plugins/vuepressUserConfigPlugin.ts
0
3
0.0%

packages/bundler-farm/src/dev.ts
0
7
0.0%

packages/bundler-farm/src/build.ts
0
11
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.6%

Covered Lines:
631

Relevant Lines:
1689

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 29 (0.0%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.6%) to 38.025%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/bundler-farm/src/resolveFarmConfig.ts
0
2
0.0%

packages/vite-kit/src/plugins/vuepressVuePlugin.ts
0
2
0.0%

packages/bundler-farm/src/farmBundler.ts
0
3
0.0%

packages/bundler-farm/src/plugins/vuepressUserConfigPlugin.ts
0
3
0.0%

packages/bundler-farm/src/dev.ts
0
8
0.0%

packages/bundler-farm/src/build.ts
0
11
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.6%

Covered Lines:
631

Relevant Lines:
1690

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 29 (0.0%)  changed or added relevant lines in 6 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.6%) to 38.025%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/bundler-farm/src/resolveFarmConfig.ts
0
2
0.0%

packages/vite-kit/src/plugins/vuepressVuePlugin.ts
0
2
0.0%

packages/bundler-farm/src/farmBundler.ts
0
3
0.0%

packages/bundler-farm/src/plugins/vuepressUserConfigPlugin.ts
0
3
0.0%

packages/bundler-farm/src/dev.ts
0
8
0.0%

packages/bundler-farm/src/build.ts
0
11
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.6%

Covered Lines:
631

Relevant Lines:
1690

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 30 (0.0%)  changed or added relevant lines in 7 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.6%) to 37.965%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/vite-kit/src/plugins/vuepressMainPlugin.ts
0
1
0.0%

packages/bundler-farm/src/resolveFarmConfig.ts
0
2
0.0%

packages/vite-kit/src/plugins/vuepressVuePlugin.ts
0
2
0.0%

packages/bundler-farm/src/farmBundler.ts
0
3
0.0%

packages/bundler-farm/src/plugins/vuepressUserConfigPlugin.ts
0
3
0.0%

packages/bundler-farm/src/dev.ts
0
8
0.0%

packages/bundler-farm/src/build.ts
0
11
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.6%

Covered Lines:
631

Relevant Lines:
1690

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 30 (0.0%)  changed or added relevant lines in 7 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.7%) to 37.935%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/vite-kit/src/plugins/vuepressMainPlugin.ts
0
1
0.0%

packages/bundler-farm/src/resolveFarmConfig.ts
0
2
0.0%

packages/vite-kit/src/plugins/vuepressVuePlugin.ts
0
2
0.0%

packages/bundler-farm/src/farmBundler.ts
0
3
0.0%

packages/bundler-farm/src/plugins/vuepressUserConfigPlugin.ts
0
3
0.0%

packages/bundler-farm/src/dev.ts
0
8
0.0%

packages/bundler-farm/src/build.ts
0
11
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.7%

Covered Lines:
631

Relevant Lines:
1690

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 30 (0.0%)  changed or added relevant lines in 7 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.7%) to 37.935%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/vite-kit/src/plugins/vuepressMainPlugin.ts
0
1
0.0%

packages/bundler-farm/src/resolveFarmConfig.ts
0
2
0.0%

packages/vite-kit/src/plugins/vuepressVuePlugin.ts
0
2
0.0%

packages/bundler-farm/src/farmBundler.ts
0
3
0.0%

packages/bundler-farm/src/plugins/vuepressUserConfigPlugin.ts
0
3
0.0%

packages/bundler-farm/src/dev.ts
0
8
0.0%

packages/bundler-farm/src/build.ts
0
11
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.7%

Covered Lines:
631

Relevant Lines:
1690

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 30 (0.0%)  changed or added relevant lines in 7 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.7%) to 37.935%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

packages/vite-kit/src/plugins/vuepressMainPlugin.ts
0
1
0.0%

packages/bundler-farm/src/resolveFarmConfig.ts
0
2
0.0%

packages/vite-kit/src/plugins/vuepressVuePlugin.ts
0
2
0.0%

packages/bundler-farm/src/farmBundler.ts
0
3
0.0%

packages/bundler-farm/src/plugins/vuepressUserConfigPlugin.ts
0
3
0.0%

packages/bundler-farm/src/dev.ts
0
8
0.0%

packages/bundler-farm/src/build.ts
0
11
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.7%

Covered Lines:
631

Relevant Lines:
1690

💛  - Coveralls

This PR may take longer to complete. Vite SSR and Farm SSR are not very similar :(

Any update about this?

I'm sorry that I'm preparing for 中考 and have to put aside some of my WIP open-source contributions :(

I fully understand that, I will probably pick up your work because we can not delay this to July 2025.


Increasing 503 error codes over time

We have a cluster that has experienced an error where requests increasingly return 503 HTTP codes over time.
The image below shows the errors generated by requests to a K8S cluster with Voyager. This cluster was up for a number of days, and then starting at 11:00 requests started returning 503 (ignore the peak around 9:00, that was an unrelated problem). At about 15:30 the Voyager ingress controller pods were deleted and restarted, and the errors went away.
We had 2 Voyager ingress controller pods, which would explain the zigzag pattern as each pod returns more (but not the same number of) 503 response codes.

I've experienced this problem a few times since, and deleting the Voyager pods always resolves the issue with no additional changes to the cluster.
The log files from the Voyager ingress controller pods don't show anything that indicates an error, other than listing the 503 error responses themselves.
I can set up a cron job to recycle the Voyager pods easily enough, which does seem to prevent the problem from happening.
So I have two questions:

Has anyone else experienced these 503 errors?
Is there any downside to using cron to delete the Voyager ingress controller pods?

We are using Voyager 7.40 in AKS, and routing TCP and HTTP traffic.

Happened again. We're using Google Kubernetes Engine with Voyager 9.0.0.

@mkozjak For some reason voyager ingress still sends requests to terminated backend pods, even though they aren't in /etc/haproxy/haproxy.cfg anymore. - How did you check that? By doing exec to the voyager pod or by describing the ingress's configmap?

@kfoozminus ingress' configmap values were updated. haproxy.cfg was ok too (yes, exec'd). I tailed voyager pod logs and it'd still hit backend pods that weren't alive anymore.

@mkozjak Does this problem after a certain period of time automatically? Or do you have to delete the controller pod every time?
I'm trying to reproduce the problem. Any details that can help me do that is much appreciated :)

@kfoozminus You mean does it go away? No. And it starts happening automatically in random intervals. Maybe @mcasperson has more info.

@mcasperson

I've experienced this problem a few times since, and deleting the Voyager pods always resolves the issue with no additional changes to the cluster.

Does this problem never go away? Or it takes too long to be fixed up so you just delete the pod to avoid down time?

@mkozjak

And it starts happening automatically in random intervals.

So here's what I gathered:

You had a working voyager ingress with (let's say) one backend service which has 4 pods.
You updated the backend deployment. So all of your 4 pods are terminated and 4 new are created.
Ingress-configmap and /etc/haproxy/haproxy.cfg inside the ingress-pod shows updated configuration, but your log shows traffic is going to terminated pods.
This problem never go away. You restart the ingress-pod (the one generated from Ingress object) and everything works well.

Is that ok?
@mcasperson Same like this?

@kfoozminus all good, except that the traffic also goes to new ones. Sometimes it chooses new ones and sometimes old ones.

@kfoozminus all good, except that the traffic also goes to new ones. Sometimes it chooses new ones and sometimes old ones (3rd bullet).

But the haproxy.cfg contains only the new pods?

Exactly!

Ok, thanks. I'm trying to reproduce this with load testing and restarting the backend pods over and over.

@mcasperson @mkozjak While we are trying to understand what's the problem - if this problem happens again - maybe we can arrange a virtual meeting to see what's going wrong.

Yes, that'd be great.

@mkozjak Are you using the default alpine based haproxy-image or did you specify --haproxy-image-tag while installing voyager?

@kfoozminus
helm install appscode/voyager --name voyager-operator --version 9.0.0 \
--namespace kube-system --set cloudProvider=gke \
--set apiserver.enableValidatingWebhook=false

@kfoozminus Want me to upgrade to 10.x?

No it's ok.

@kfoozminus Note that I'll soon need to update to 10.x so maybe try running tests on that version also.

@kfoozminus I'm having a reproduced issue currently. Anyway to do a live debugging session together? I will need to restart everything in half an hour..

@mcasperson What did you do to tackle this one in the mean time?

@kfoozminus Upgraded to 10.x, I guess.
kubectl exec -it voyager-voyager-operator-76778cc5f9-4ctln -n kube-system voyager version

Version = 10.0.0
VersionStrategy = tag
Os = alpine
Arch = amd64
CommitHash = b866601fd012b57feb048e8eb3caa954ae5280af
GitBranch = release-10.0
GitTag = 10.0.0
CommitTimestamp = 2019-04-29T00:07:03

A really confusing part is:
kubectl describe pod voyager-k8s-ingress-66f7d879fd-2lfv9

Image:         appscode/haproxy:1.9.2-9.0.0-alpine

Shouldn't it be appscode/haproxy:1.9.6-10.0.0-alpine?

@mkozjak sorry I wasn't online. The problem ia gone by now, I guess? :(

Yup.
On Mon, Jun 3, 2019, 21:29 Jannatul Ferdows Jenny<EMAIL_ADDRESS>wrote:

@mkozjak https://github.com/mkozjak sorry I wasn't online. The problem
ia gone by now, I guess? :(
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/appscode/voyager/issues/1334?email_source=notifications&email_token=AA2X77BAZC2TK4MSV2EL2FDPYVWJJA5CNFSM4GTNHL2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODW2OE4Q#issuecomment-498393714,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA2X77B7CJJCXNBSWTDRNQ3PYVWJJANCNFSM4GTNHL2A
.

Hi Mario,
Our offices are closed this week for Eid vacation. We will be back to
regular schedule from next Monday (June 10). We are going to pick up this
issue once we are back.
Regards,
Tamal
On Mon, Jun 3, 2019 at 12:31 PM Mario Kozjak<EMAIL_ADDRESS>wrote:

Yup.
On Mon, Jun 3, 2019, 21:29 Jannatul Ferdows Jenny <
<EMAIL_ADDRESS>wrote:

@mkozjak <https://github.com/mkozjak
https://mailtrack.io/trace/link/9bf68dde576f938c5bc2fa49acb3f9199a1e71fc?url=https%3A%2F%2Fgithub.com%2Fmkozjak&userId=3471932&signature=44c99f75681d2963>
sorry I wasn't online. The problem
ia gone by now, I guess? :(
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<
https://github.com/appscode/voyager/issues/1334?email_source=notifications&email_token=AA2X77BAZC2TK4MSV2EL2FDPYVWJJA5CNFSM4GTNHL2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODW2OE4Q#issuecomment-498393714
https://mailtrack.io/trace/link/aea2715ed170ff57c475167fe165bab578536635?url=https%3A%2F%2Fgithub.com%2Fappscode%2Fvoyager%2Fissues%2F1334%3Femail_source%3Dnotifications%26email_token%3DAA2X77BAZC2TK4MSV2EL2FDPYVWJJA5CNFSM4GTNHL2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODW2OE4Q%23issuecomment-498393714&userId=3471932&signature=0a5d8b8f7d42420a
,
or mute the thread
<
https://github.com/notifications/unsubscribe-auth/AA2X77B7CJJCXNBSWTDRNQ3PYVWJJANCNFSM4GTNHL2A
https://mailtrack.io/trace/link/1c7a4b92462c5996ae6f3d84361c85fec3a7d416?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAA2X77B7CJJCXNBSWTDRNQ3PYVWJJANCNFSM4GTNHL2A&userId=3471932&signature=bc6bb7821ca468af
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://mailtrack.io/trace/link/4a0522c37d523d361cee5aa3424b410bc4e3634d?url=https%3A%2F%2Fgithub.com%2Fappscode%2Fvoyager%2Fissues%2F1334%3Femail_source%3Dnotifications%26email_token%3DAAAXEXVYQPAMZUORCS2EQCLPYVWQLA5CNFSM4GTNHL2KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODW2OKCI%23issuecomment-498394377&userId=3471932&signature=1945a156f2a7081f,
or mute the thread
https://mailtrack.io/trace/link/71acfaf596d85990f25ac32f65cc426f71ea9855?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAAAXEXWCBW3JG4EMKO7M7Y3PYVWQLANCNFSM4GTNHL2A&userId=3471932&signature=c88fa998820b9248
.

Hey everyone, just wanted to mention that I'm having a very similar issue, if not the same exact issue. Everything works great until large amounts of traffic are moved through the ingress, then the ingress will continue working properly for 10-15 minutes before intermittently throwing 503s.
I'm using Azure Kubernetes Service 1.14.0 running Voyager 10 pointed at external backends (Azure app services). Nothing else is having trouble reaching these services when we see this issue. Unfortunately, I'm unable to confirm whether this issue scales out in the same way since I'm only able to replicate the issue in prod. If there's anything I can do to help with this issue, I'm happy to help.

Hey guys, just curious if you have any updates on this issue. Thanks!

We are assuming for now that this issue is happening because of haproxy reload issue and pods being terminated without grace. I'm writing a detailed doc about it. But we've only been able to reproduce this issue in where after a certain while it goes away. (Basically we are assuming no 503 for new requests.)
Are you facing this problem right now?

Yes, we're able to reproduce this issue any time we ramp up traffic to our ingress. We have it down for the time being but I can bring it back up if you need something.

Yes, that'd be great. Join our slack: https://appscode.slack.com/

Great, thanks! Just messaged you over there.

@kfoozminus @lanserver Maybe any updates on this one? In which slack channel did you do the debugging in not in a direct message?

@mkozjak we used direct message.
His issue seems different from this though.

@kfoozminus So it was something different than sudden 503s after certain pods die and 200s after the ingress controller pods are restarted?

Yeah.

Well, ok. The next time I reproduce the issue I'm gonna try sending SIGUSR2 signal to a haproxy process so it reloads its configuration in runtime to see if that'll solve the issue.
Can you please continue your tests on this internally, @kfoozminus?

@mkozjak Yes, of course! We are testing and trying to reproduce this continuously!
Were you able to see how many haproxy process was running in that moment?

@kfoozminus In one pod or you mean the number of pods? It was 4 running pods at the time.

Next time it happens again, give us the output to ps aux | grep haproxy from inside the voyager pod. Want to see how many haproxy processes were running.
I hope to do a live session though.

kubectl describe pod voyager-k8s-ingress-66f7d879fd-2lfv9
Image:         appscode/haproxy:1.9.2-9.0.0-alpine
Shouldn't it be appscode/haproxy:1.9.6-10.0.0-alpine?

@mkozjak yes, it should be. What exact command did you use to upgrade to 10.0.0?

@kfoozminus
helm upgrade voyager-operator appscode/voyager --version 10.0.0 --namespace kube-system --set cloudProvider=gke --set apiserver.enableValidatingWebhook=false

@kfoozminus: Maybe I did something wrong with the update procedure?

I upgraded with same command. Mine shows appscode/haproxy:1.9.6-10.0.0-alpine.
Does yours still show appscode/haproxy:1.9.2-9.0.0-alpine?

Seems to have managed to pull the latest one in the end, since I restart the ingress all the time.
appscode/haproxy:1.9.6-10.0.0-alpine

After upgrading, it takes a few seconds to update the image (time new voyager operator takes to go into running state and update all ingress deployments).

@kfoozminus: Maybe had any luck reproducing the issue?

Hi, it seems like the issue is still present in version 10.0.0. I got several 503 errors on frontend files today right after upgrading my stack, and killing the Voyager pod solved the issue.

@Simon3: next time you reproduce, can you please try running ps aux | grep haproxy and sending SIGUSR2 to the process inside the pod instead of killing it and see if it'll start working again? Thanks!

Sure I will if it happens again.

Hey, I'm still on Voyager 10.0.0 and the bug keeps occurring, like twice a month. We are doing lots of helm upgrades though, like 20 per day. I tried the SIGUSR2 signal, but it didn't change anything, here is my history in case I would have missed something:
bash-4.4# ps | grep haproxy
    9 root      0:00 runsv haproxy-controller
   12 root     43:03 voyager haproxy-controller --enable-analytics=true --burst=1000000 --cloud-provider=gke --ingress-api-version=voyager.appscode.com/v1beta1 --ingress-name=flowr-527-rc-latest-ingress --qps=1e+06 --logtostderr=false --alsologtostderr=false --v=3 --stderrthreshold=0
 1002 root      0:10 haproxy -f /etc/haproxy/haproxy.cfg -p /var/run/haproxy.pid
 1051 root      0:03 haproxy -f /etc/haproxy/haproxy.cfg -p /var/run/haproxy.pid -x /var/run/haproxy.sock -sf 1048
 1057 root      0:01 haproxy -f /etc/haproxy/haproxy.cfg -p /var/run/haproxy.pid -x /var/run/haproxy.sock -sf 1054
bash-4.4#
bash-4.4# kill -SIGUSR2 12

It would be nice if we could get an update on this issue, is there a point in upgrading to v11.0.1 for this issue? Are you still investigating it?
Thanks in advance!

@Simon3 I believe the SIGUSR2 needs to go to the haproxy processes and not the voyager wrapper
@kfoozminus Is there an update on this issue?

Ok I applied it to every haproxy process this time, still lots of 503 errors.
I then deleted the Voyager pod, and it worked like a charm as usual.

We are seeing the same issue on version 11.0.0. Every release, one or two Voyager pods (out of 65) will flood 5xx errors. Each of those affected Voyager pods has two haproxy processes running, as opposed to one haproxy process on the normal Voyager pods.
bash-4.4# ps aux | grep haproxy

10 root 0:00 runsv haproxy-controller

12 root 7:40 voyager haproxy-controller --enable-analytics=true --burst=1000000 --cloud-provider=gke --ingress-api-version=voyager.appscode.com/v1beta1 --ingress-name=rlcdn --qps=1e+06 --logtostderr=false --alsologtostderr=false --v=3 --stderrthreshold=0

8437 root 13:31 haproxy -f /etc/haproxy/haproxy.cfg -p /var/run/haproxy.pid -x /var/run/haproxy.sock -sf 8434

8461 root 6:04 haproxy -f /etc/haproxy/haproxy.cfg -p /var/run/haproxy.pid -x /var/run/haproxy.sock -sf 8458

8469 root 0:00 grep haproxy

Issuing SIGUSR2 doesn't work; however issuing SIGKILL to the older process (8437 in the example above) works. Issuing SIGKILL to the newer process will just cause another haproxy process to spawn and the errors to continue.
Another observation is that right before the errors begin, the affected Voyager pods log errors like these:
E0213 18:32:18.845822      12 controller.go:276] error reading haproxy.pid file: open /var/run/haproxy.pid: no such file or directory

E0123 00:04:52.399698      12 controller.go:274] error reading haproxy.pid file: EOF

Those errors are thrown from here. My guess as to what's happening is that for whatever reason, /var/run/haproxy.pid does not properly capture the pid of a running haproxy process, so the controller starts a new one when it actually should just reload the already-running process.


/auth/logut url and guest middleware

/auth/logut didn't work.
Override the getLogout method in AuthController and print some text, nothing outputed.
I noticed that there is guest middleware about the logout action in the constructor of AuthController.
Comparing the code with another project Scrumwala based on laravel 5.1, I found that except option is defferent.
In Laravel 5.2.32, the constructor of AuthController is:
public function __construct()
{
    $this->middleware($this->guestMiddleware(), ['except' => 'logout']);
}

While in Scrumwala (Laravel 5.1.*):
public function __construct()
{
	$this->middleware('guest', ['except' => 'getLogout']);
}

By using getLogout instead of logout as the value of except option, /auth/logout works in Laravel 5.2.32
I ran aonther test.
In Laravel 5.2.23, ['except' => 'logout'] works fine....  /-_-\

Feel free to discuss on the forums.


Failure to upload after flashing

This worked seamlessly on my first attempt. The second ESP8266 NodeMCU module I tried has worked with other firmware. Every time I try to flash it with Konnected firmware it fails to reboot. The LED is flashing rapidly as if it is rebooting and not finding the bootloader. Is there a way around this? If I re-flash with the Arduino API it seems to work still, so the chip does not seem to have any permanent damage.
./scripts/flash
esptool.py v2.2-dev
Connecting....
Detecting chip type... ESP8266
Chip is ESP8266EX
Uploading stub...
Running stub...
Stub running...
Configuring flash size...
Auto-detected Flash size: 4MB
Flash params set to 0x0240
Compressed 479444 bytes to 311295...
Wrote 479444 bytes (311295 compressed) at 0x00000000 in 27.5 seconds (effective 139.6 kbit/s)...
Hash of data verified.
Leaving...
Hard resetting...
opening port /dev/tty.SLAB_USBtoUART with 115200 baud
Traceback (most recent call last):
File "//anaconda/bin/nodemcu-uploader", line 11, in
sys.exit(main_func())
File "//anaconda/lib/python2.7/site-packages/nodemcu_uploader/main.py", line 252, in main_func
uploader = Uploader(args.port, args.baud, start_baud=args.start_baud)
File "//anaconda/lib/python2.7/site-packages/nodemcu_uploader/uploader.py", line 78, in init
__sync()
File "//anaconda/lib/python2.7/site-packages/nodemcu_uploader/uploader.py", line 76, in __sync
raise DeviceNotFoundException('Device not found or wrong port')
nodemcu_uploader.exceptions.DeviceNotFoundException: Device not found or wrong port

It sometimes takes a minute or two to reboot. It looks like you're using my flash script, which is designed to wait and retry after a couple seconds for it to reboot after flashing. Can you try giving it a minute and see if it eventually does upload successfully?

I noticed it was in a loop but I waited ages to no avail. After some searching I decided the flash was failing and found that a complete erase of the flash and fresh run of the script solved the issue.
Thanks for your help and for the great software!


[Event] May 2023 Voxxed Days Brussels: how can we participate

A task to discuss how we can participate in Voxxed Days Brussels: 23 May 2023
Thoughts:

Promo?
Speaker Training?
Booth?
Surrounding Java activity?

Received mail from the organization:

They offer a discount code we can send to our members in exchange for some promo.
The also give us the opportunity to fill a slot with some Quicky Sessions

Both to be discussed in the next community hangout.

Ideas:

Toastmasters?
5 minute session?  Feedback!
Help with abstract?

Guidelines on who gets the slots

Voxxed: 3*15 minutes
Training: Community (those present) can vote on the most popular talk (top 3 gets selected)
Training: 1 week before event, we let you know how long the training presentation should be. (but mention it is going to be a short preview, not a 50 minute session!)
Voxxed: Fallback in case not enough sign up: fill up rest of slots with experienced speakers?

TODO:

[ ] Figure out how the voting could work?
[ ] Check with Olivier what the options are for beginning of may? @TomCools
[ ] Ask Toastmasters if they want to participate in this training. @TomCools

@TomCools, I got a reply from the school.
They are closed the first week of May.
Should I ask for a reservation on the second week, or should we try to advance this in April?

@ohubaut: Maybe move it up a week to April? We need enough time for people to practice their talks.

@TomCools Done. Venue reserved for April 27th.

❤


Creating landing page

Landing page should have navigation bar with branding
Create new project input box
Project description Textarea

Done. @shanip1


[Bug]: "Label" does not work as include or exclude filter

Version
3.0.0-alpha.52
Matter Controller
Alexa
Steps to reproduce
Choose any supported entity which is not already included via any filter in the matter bridge and add a label, eg "Voice". Then go to the bridge and add an include for the label "Voice". It will not appear in the list of devices.
For excluded, make certain you have a filter in place that includes devices. Then go to one entity which is included, apply a label, eg "Novoice". Go to the bridge and create an exclude filter for label "Novoice". The device will not be excluded.
State and attributes
No response
Relevant log output
No response
Documentation & Issues

[X] I have reviewed the documentation and the linked troubleshooting guide.
[X] I have searched the issues for a similar problem.

In the background home assistant is not using the name of the label, but its "slug".
You have to use that slug for the filters.
You can find the slug by writing {{ labels() }} into a template (HA -> Developer tools -> Template).

I see. Basically it seems to be a lowercase version of a label if it doesn't contain other than just letters.
You can't use the list from {{ labels() }} for selection? Else a hint would be helpful.

Thanks you all. Now IT works.

Geht trz nicht

Ist HA das HAOS basierend?


fix hiding

previously clicking on an element outside of the datepicker didn't go through the usual hide method, thus forceParse was ignored, leaving invalid dates in the input field

possible duplicate of #1464

@hebbet, #1464 didn't fix that issue though. As you can see, my changes are on top of master, which includes #1464.
I've confirmed with master, that the issue is still there:

change the date field to have invalid date
click somewhere outside the date picker
the field will keep the invalid value

On the other hand if you use 'tab' key to jump to next field, the invalid date will get replaced with a valid date.

thanks! :)


shotcut: 21.03.21 -> 21.08.29 and clarify license

Motivation for this change
This works with newer mtl++. License is gpl3Plus: https://github.com/mltframework/shotcut/blob/master/src/abstractproducerwidget.cpp#L1-L17
Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.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.

Oh, thanks for the heads-up. I put it in draft because I was having trouble exporting videos — does that work for you?

you are right; it tries to execute melt-7 which does not exist; https://github.com/woffs/nixpkgs/commit/9a6e971b25ce45bd037a651a99fbb50b6599047f would fix that

Thanks! Added you as Co-Authored-By.

Looks good to me, export is working. I'd suggest to update to 21.09.20 and then let it be merged.

Updated; re-tested a handful of things (import mp4, import jpg, blur filter, playlist, crop, rearrange, save, load, export); seems to work fine.


Configurable max number of columns from config

We have a use case for a client to increase a number of colums in a grid.
So, we have modified this block to set the maximum number of columns from the configuration instead of a fixed number of columns set to four.
In addition to that, we have added the templates svg files from 5 to 16 because Semantic have a 16 column grid and we think it's enough :smile:

@ionlizarazu thanks for your contribution! More than four columns is a very valid use case and should indeed be configurable.


Improvement of docstring for Collection or fiona.open

Hey, i just spent WAY too long on a trivial problem. First time Fiona/Shapely and actually also first time using Python to do anything a bit more sophisticated than a script.
I did not realise that the fiona.open Collection was "emptying" as I was looping over it. So I used two nested loops and wondered why it just would not work. The outer loop was supposed to look at each record once, the inner loop used the same collection and compared each record with the current record of the outer loop. Naturally, this lead to very confusing behaviour.
I come from Java with JTS and OpenJUMP where a FeatureCollection would have work as I thought this worked.
I am using PyCharm, the docstring for Collection just told me that it is "A file-like interface to features in the form of GeoJSON-like mappings" (as it says on http://toblerity.org/fiona/fiona.html).
If I had known that it was an iterator, I would probably have spotted my mistake earlier. As I said, I am new to Python, so this should be a trivial thing but the rest was easy, this was just unexpected.
Long text, what I mean is: Could you add some line to the Collection (or fiona.open?) saying that it iterates this way? :)

@fnorf The Collection() docstring could certainly be improved but I think your confusion may be a by-product of being new to Python.  References to a file-like object are fairly common in Python documentation and imply a certain set of features and behaviors, one of which is that the object is an iterator.
If you're not familiar,
By saying file-like object, we're assuming you're already familiar with the object returned by Python's builtin open(), which is what Collection() is modeled after.
In other words, we want this:
with fiona.open(input_file) as src:
    for feature in src:
        # Do something with a feature

to be familiar and behave like this:
with open(input_file) as f:
    for line in f:
        # Do something with a line

Thanks for the suggestion, @fnorf. Here's what you'll see in 1.6.3:
Help on class Collection in module fiona.collection:

class Collection(__builtin__.object)
 |  A file-like interface to features of a vector dataset
 |
 |  Python text file objects are iterators over lines of a file. Fiona
 |  Collections are similar iterators (not lists!) over features
 |  represented as GeoJSON-like mappings.
 |
 |  Methods defined here:
...

@sgillies The Fiona docs on Toblerity.org are out of date as well.  What would it take to do an update?

I am late, but thank you very much!


SPONSOR - Shipment - Remove Kit ID List(.csv) from Shipment Details

the download is already implemented in Kits tab

Removed Kit ID List(.csv) from Shipment Detail and created PR for this


optimize lettuce plugin

Fix

[x] Add a unit test to verify that the fix works.

[x] Explain briefly why the bug exists and how to fix it.

[x] Update the CHANGES log.

This is a big change for the lettuce plugin
Lettuce is an async Redis client base on netty, and wide use in our company.
Recently I found almost every lettuce span has the time elapsed less than 1ms and has a long gap between two spans. After reading the lettuce source code, I found the plugin some bug.
because  lettuce is an async framework base on netty, old interceptor named 'RedisChannelWriterInterceptor' stop span in after method can not record the time elapsed, it should be close in 'RedisCommand#complete' or 'RedisCommand#completeExceptionally' or 'RedisCommand#cancel'   called when Redis server response
lettuce plugin record nothing when lettuce work on reactive mode (project reactor)
record Redis peer in skywalking dynamic field can simplify
The new plugin can work with Redis standalone and Redis Sentinel and Redis cluster(old version can not).  Redis cluster and Redis sentinel need to use docker-compose. I do not know how to make scenarios test in this two cluster model

The new plugin can work with Redis standalone and Redis Sentinel and Redis cluster(old version can not). Redis cluster and Redis sentinel need to use docker-compose. I do not know how to make scenarios test in this two cluster model

It is fine to skip that. If you want, it is better to update support list doc to mention all these.

LGTM. @darcydai Do you need to update the document to mention this?

The new plugin can work with Redis standalone and Redis Sentinel and Redis cluster(old version can not). Redis cluster and Redis sentinel need to use docker-compose. I do not know how to make scenarios test in this two cluster model

where should I write this？

How about adding (cluster..) after redis on this page?
https://skywalking.apache.org/docs/skywalking-java/latest/en/setup/service-agent/java-agent/supported-list/


Iox 1815 add determinism label to selected types

Pre-Review Checklist for the PR Author

[x] Code follows the coding style of CONTRIBUTING.md
[x] Tests follow the best practice for testing
[ ] Changelog updated in the unreleased section including API breaking changes
[x] Branch follows the naming format (iox-123-this-is-a-branch)
[x] Commits messages are according to this guideline

[x] Commit messages have the issue ID (iox-#123 commit text)
[x] Commit author matches Eclipse Contributor Agreement (and ECA is signed)

[x] Update the PR title

Follow the same conventions as for commit messages
Link to the relevant issue

[x] Relevant issues are linked
[x] Add sensible notes for the reviewer
[ ] All checks have passed (except task-list-completed)
[x] All touched (C/C++) source code files from iceoryx_hoofs are added to ./clang-tidy-diff-scans.txt
[x] Assign PR to reviewer

Notes for Reviewer

Checklist for the PR Reviewer

[ ] Commits are properly organized and messages are according to the guideline
[ ] Code according to our coding style and naming conventions
[ ] Unit tests have been written for new behavior
[ ] Public API changes are documented via doxygen
[ ] Copyright owner are updated in the changed files
[ ] All touched (C/C++) source code files from iceoryx_hoofs have been added to ./clang-tidy-diff-scans.txt
[ ] PR title describes the changes

Post-review Checklist for the PR Author

[ ] All open points are addressed and tracked via issues

References

Closes #1815

@elBoberido @FerdinandSpitzschnueffler Reduced the scope to the two experimental modules as discussed. All functions are "deterministic" there, as in many other modules (except for POSIX and arguably some concurrency stuff).
Still, we need more experience with the test validation and I documented the reason for scope reduction to make it clear to users that this is not fully rolled out for good reason.
I can squash all the commits if desired (a better solution, as nothing much is happening). Let me know if this is OK before or after review.


Error creating native image: Invalid path entry [target\native-image-maven-plugin\20.2.0\windows-amd64]

I am trying to use the Maven plugin to generate a native image following the documentation, but I always get the following error:
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  49.965 s
[INFO] Finished at: 2020-09-18T11:29:35+02:00
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.graalvm.nativeimage:native-image-maven-plugin:20.2.0:native-image (default) on project graalvm-demo: Error creating native image:: Requirements for building native images are not fulfilled [cause: Invalid Path entry C:\development\DEVEL\Work\Playground\GraalVM\graalvm-demo\target\native-image-maven-plugin\20.2.0\windows-amd64] -> [Help 1]

I am using latest 20.2.0 plugin and also JDK openjdk1.8.0_252-jvmci-20.1-b02-fastdebug with latest Spring Boot 2.4.0-M3.
My Maven options are:
            <plugin>
                <groupId>org.graalvm.nativeimage</groupId>
                <artifactId>native-image-maven-plugin</artifactId>
                <version>20.2.0</version>
                <configuration>
                    <buildArgs>
                        -Dspring.spel.ignore=true
                        -Dspring.native.remove-yaml-support=true
                        --no-fallback
                        --allow-incomplete-classpath
                        --report-unsupported-elements-at-runtime
                        -H:+ReportExceptionStackTraces
                        --no-server
                        -Dspring.native.mode=reflection
                        -Dspring.native.dump-config=/tmp/computed-reflect-config.json
                        -Dspring.native.verbose=true
                        -Dspring.native.skip-logback=true
                        --initialize-at-run-time=org.springframework.data.r2dbc.connectionfactory.ConnectionFactoryUtils
                        --initialize-at-build-time=io.r2dbc.spi.IsolationLevel,io.r2dbc.spi
                        --initialize-at-build-time=io.r2dbc.spi.ConstantPool,io.r2dbc.spi.Assert,io.r2dbc.spi.ValidationDepth
                        --initialize-at-build-time=org.springframework.data.r2dbc.connectionfactory
                        -H:+TraceClassInitialization
                        --initialize-at-build-time=org.reactivestreams.Publisher
                        --initialize-at-build-time=com.example.reactive.ReservationRepository
                        --initialize-at-run-time=io.netty.channel.unix.Socket
                        --initialize-at-run-time=io.netty.channel.unix.IovArray
                        --initialize-at-run-time=io.netty.channel.epoll.EpollEventLoop
                        --initialize-at-run-time=io.netty.channel.unix.Errors
                        --initialize-at-build-time=org.apache.commons.logging.LogFactory
                        --initialize-at-build-time=jdk.vm.ci.services.Services
                        -H:DeadlockWatchdogInterval=10
                        -H:+DeadlockWatchdogExitOnTimeout
                    </buildArgs>
                </configuration>
                <executions>
                    <execution>
                        <goals>
                            <goal>native-image</goal>
                        </goals>
                        <phase>package</phase>
                    </execution>
                </executions>
            </plugin>

Attaching a ZIP file with the same project --> https://filebin.net/7n92474no5622rw1/graalvm-demo.zip?t=r536u2rf

I haven't tried with that particular JDK variant. What worked for me right now, on windows, is:

run vcvars64.bat to get the right cl.exe on my PATH:

c:\Program Files (x86)\Microsoft Visual Studio\2017\BuildTools\vc\Auxiliary\Build\vcvars64.bat

using graalvm-ce-java11-20.2.0

SET JAVA_HOME=E:\graalvm-ce-java11-20.2.0\
SET PATH=%JAVA_HOME%\bin;%PATH%

build the commandlinerunner sample:

cd spring-graalvm-native\spring-graalvm-native-samples\commandlinerunner
..\..\mvnw -Pnative clean package
.\target\commandlinerunner.exe

I tried to grab the zip but maybe it expired. I don't see anything in your buildArgs that looks like it might cause an issue. (Oh, I was just running from a command prompt too, not powershell).

I managed to make it work, but when I run it, I get endless ClassNotFoundExceptions. Do I have to add them one by one, rebuild, run and repeat the process all the time?
This is a minimal application with Spring Cloud Stream with Kafka binding, Spring Cloud Config Server, Spring Security and everything reactive with Webflux, ibcluding R2dbc with PostgreSQL.

spring-graalvm-native analyses the app and tries to work out what configuration needs to be added, but if you are using technology we have not dealt with yet then we may miss things. We haven't done anything with spring-cloud-stream and minimal stuff with spring cloud config server.  In these cases you can run the app with the GraalVM agent to collect the config as you run it in JVM mode then attempt to compile to a native-image using that config. How to do it is discussed here:
https://repo.spring.io/milestone/org/springframework/experimental/spring-graalvm-native-docs/0.8.1/spring-graalvm-native-docs-0.8.1.zip!/reference/index.html#_missing_configuration

@emedina Please create more specific issues when you identify that a starter you need is not supported like "Add Spring Cloud Config Server support" or "Add Spring Cloud Stream support". Spring Security support is already WIP via #247.


[Fix][sql] fix sql symbol

descripe:
The symbol used by key should be '`' but use '''
Purpose of the pull request
fix sql symbol

@jefferyeven
Thank you very much for your contribution, this issue has already been submitted by a small partner to PR #5474

so sorry, this problem has resolved. if you have interesting，you can find other task and leave a message, I will assign to you.
deeply thx again.


Can't resolve references of tutorial-code

After downloading the tutorial project, and compiling it, Visual Studio 2015 couldn't resolve the dependencies specified in project.json

Havin the same issue, can someone help us.

Hmm that's strange, it definitely worked when I built the example. Maybe some of the version numbers are wrong. I'll try to take a look when I get a chance.
What you can do is create a new ASP.NET Core MVC project and compare the project.json files.

Closing issues older than a year, please re-open if you think this is still relevant.
The tutorial project restores successfully for me, it's possible we fixed this issue when updating to .NET Core 2.0