Question: How to add COM based UI-Controls (like ExplorerBrowser) to WinUI3

Ages ago, Microsoft released the Windows API Code Pack for using native Windows Controls like ExplorerBrowser in Windows Forms Applications.
There, they've built an C#-Wrapper for the underlying COM-Based UI-Control, that then was hosted in an WinForms UserControl.
Is it possible to adopt this approach using WinUI3 / WinRT? I know, I can get the underlying native HWND  handle for an WinUI3 main window. Using that, in my understanding, it should at least be possible to bind the COM Control against the main window.
However, I couldn't find any more docs or examples how to connect the message loop for handling drawing, focus issues, etc pp.
Any ideas and comments on this?
Regards, and thanks in advance!

I did a test in C# and ExplorerBrowser can work, but I had to use a Win32 Region (SetWindowRgn) to enable its UI
(and I did not set the DarkMode on right part...) :

@castorix Thanks for your reply. Could you share the snippet?
I can get the ExplorerBrowser to be drawn into the window, but the UI still doesn't work for me 😞
I'm new to WinUI and really appreciate your time spent on this. Thanks in advance! 👍

@castorix Thanks for your reply. Could you share the snippet?

I made a more complete sample, but still using a Region : WinUI3_ExplorerBrowser.zip
(maybe there is a better way because I must remove/set it for Popup windows like ContentDialog (otherwise they are hidden by the Region)...)

@castorix Thank you very, very much on this! 👍
I wonder, if there's a way to handle these overlays correctly... If you have an idea, you're welcome. However, and again, many thanks. This really helps to find out how it could be done.

The problem is that this control needs user interaction.
I don't see atm how to allow mouse without a region on the underlying  window...

Hallo, @castorix,
thank you so much for your efforts.
Yes, I understand the Focus-Problem. My thought was, when the control has the focus (via keyboard or some kind of OnMouseOver(), then set the region to true.
On the other hand, when the control has lost keyboard focus, or another XAML-Control got the focus via mouse click, then set the region to false again.
The SwapChainPanel would just handle the correct drawing issues, when the form has to repaint any stuff like overlays, etc. pp.
As said, I'm a WinUI-newbie, and just want to make sure that I don't drop the possibility of using it for my own projects in the future, although there would be any chance that I can get this combination up and running, at least with some hacks 🤔
Thank you very, very much in advance,
Regards, tajbender


Players might spawn outside of map if map is too small.

Players will also spawn outside of the map if they have a high enough player id.
Players will get high player ids if player joins and leaves the game many times.

Fixed in #501


More syllable rules

Opening this PR now to make you aware of it. I'm still chipping away at it.
todo:

[x] more test cases & more precise assertion
[x] fix #2 (do cu ment, some thing)
[x] fix https://github.com/nlp-compromise/nlp_compromise/issues/130 (eu lo gy)
[x] fix trailing consonants not being attached to previous syllables (reasons is bugged as rea son s, and is bugged as an d)
figure out how to interpret -shed words (finished, unpolished, crashed)

do this in another branch

yes! go nuts.
let me know when i can pull-er in

nudge :wink: *nudge

Okay this branch is now complete. However there's a merge conflict since #5. I can update this branch to remove let/const, however, wouldn't it be better to put this code through an ES5 transformation build step, so the source can still use let/const?

Basically I'm saying #4 was a mistake...

ah, this is brilliant. thank you.
yeah, I don't mind monkeying around with different build ideas. No sweats.
I'm no git-master, but happy to force-merge this, or whatever it's called. That cool?

Well if you merge it in now, it'll effectively revert the other two pr's, since I am using let/const, so as-is, this project will break in browsers. What I'm saying is there's basically two options: I can add a commit to replace let/const with var, here in this branch, or the es5 build process can be restored. I'm okay either way.

But anyway if you are fine returning to a build process, then yes this can be merged

got it. thanks!


Emojify pipeline names in a few places that were missing it

Pipeline names were emojified in some places but not all. I grepped for pipeline.name and converted the remaining places where emojification wasn't happening, with the exception of things like the document title where emojification doesn't make sense.
Fixes #343
Tested only with unit tests

Moving to #350!


Update Node.js version on Travis

Update Node.js to version 12 on Travis.
Fixes #11

Hey @TiagoDanin, just updated it as you requested. Can you please review again?
Thanks!


permissions logic fix

the previous logic was for exact matches, but that failed on MOD_STREAMER when you are the streamer and not a mod. just need overlap, not exact match.

actually screwed this up. it should be an == not >=

Was a little too fast there ^^


ActionView::Template::Error (undefined method `title' for nil:NilClass)

when visiting this work //resources/cb98bbcf-2a13-4638-91c3-c1c778cac610 rails throws the error above.
1.) https://scholarsphere-migration.dsrd.libraries.psu.edu/resources/cb98bbcf-2a13-4638-91c3-c1c778cac610
2.) https://app.datadoghq.com/apm/trace/8844229904587825234?colorBy=service&env=migration&graphType=span_list&shouldShowLegend=true&sort=time&spanID=8925952396436761481

The collection has one work, which is unpublished. Therefore:
> collection.works.first.latest_published_version
=> nil

We should suppress listing unpublished works for migrated collections, and disallow adding unpublished works to collections.

This appears to have been fixed in #473, but the issue still exists in the dashboard collection view page. However, that page should go away with #492.


Waiting for server bootup

Thanks for the great project!
I have been using this library with great success, however, I have a usecase which I don't think is covered by the current API.
I'm doing integration tests for my gRPC service, where I launch the server, then connect to it with a client and assert the response. Simplified code:
    #[tokio::test]
    async fn service_test() {
        // Start server
        tokio::spawn(async move {
            Server::builder()
                .add_service(...)
                .serve("[::1]:50051".parse().unwrap())
                .await
                .unwrap();
        });

        // How to replace this sleep with a signal once server is ready?
        tokio::time::sleep(Duration::from_millis(500)).await;

        // Connect to the service as client
        let mut client = RPCServiceClient::connect("http://[::1]:50051")
            .await
            .expect("Failed to connect to gRPC server");

        let response = client.rpc_method(...).await.expect("Request failed");
        assert!(...);
    }

The problem is that if I don't include a delay between server and client, the test fails with Connection refused, because the server was not ready yet. Use of delay is not ideal because it can still fail due to server lag and slows down tests in general. Is there a way to somehow wait until server is ready?

I've been doing the same (although waiting only 1ms seems to be enough in my tests).

We actually do the same thing in tonic's test suite but yeah it is kinda annoying.
I haven't tested this (on my phone) but I'm thinking it's possible to use serve_with_incoming to create the tcp listener up front and pass it to the async task. That should remove the need for a sleep and it's what I do with hyper in another project. Should also make it possible to use ephemeral ports.
I'll try it later today unless someone beats me to it.

Reporting to tell that serve_with_incoming works! There is a bit of boilerplate to get the ephemeral ports to work.
    #[tokio::test]
    async fn service_test() {
        let socket = tokio::net::TcpSocket::new_v6().unwrap();
        socket.bind("[::1]:0".parse().unwrap()).unwrap();
        let port = socket.local_addr().unwrap().port();
        let listener = tokio_stream::wrappers::TcpListenerStream::new(socket.listen(1).unwrap());

        // Start server
        tokio::spawn(async move {
            Server::builder()
                .add_service(...)
                .serve_with_incoming(listener)
                .await
                .unwrap();
        });

        // Connect to the service as client
        let mut client = RPCServiceClient::connect(format!("http://[::1]:{}", port))
            .await
            .expect("Failed to connect to gRPC server");

        let response = client.rpc_method(...).await.expect("Request failed");
        assert!(...);
    }


barcode is not show in https

barcode is not show in https

Yo can check this repo https://packagist.org/packages/elminson/barcode
with UnitTest and new features as Set text color an background color. I hope you like it.

same issue barcode is not showing in https .


hello

the darkened version is pretty good,
can you make the keywords of rust language italic and a brighter purple color?
the type annotation must be yellow-gold !!
like this :

Hi, I'm glad you're liking it. However, my aim is to provide theme that is consistent with the One Dark theme that zed comes with by default. Purpose of this theme is to just enhance the default contrast which is for some of zed' users pretty low.

brighter purple color?
the type annotation must be yellow-gold !!

Currently, I am not aware of the language-specific color theme customization, colors are being assigned for the different types of tree-sitter tokens that are language-agnostic. So even if I would want to implement your  suggestions for different colors for rust code, that would be impossible at the moment.

can you make the keywords of rust language italic

There is a feature to make certain parts of the code appear in different font style/weight, but as I said, since that is not the way default One Dark theme works, I have no incentive to make change.
Feel free to copy one-dark-darkened.json contents and place it in ~/.zed/themes directory, and you can adjust it to your preferences. https://zed.dev/docs/themes#local-themes


Add mobilenetv3 support for classification and detection

Adding mobilenet models from new release. Closes #94

Well mobilenet small isn't done yet


Add primary_key to issue_index

Make the group_id a primary key in issue_index. This already has an unique index
and therefore is a good candidate for becoming a primary key.
This PR also changes all other uses of this table to add the group_id as the
primary key.
Fix #16802
Signed-off-by: Andrew Thornton<EMAIL_ADDRESS>

We can probably just backport this as is just without the migration.

Codecov Report

Merging #16813 (76cd01f) into main (e891d68) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main   #16813   +/-   ##
=======================================
  Coverage   45.48%   45.48%
=======================================
  Files         762      762
  Lines       85975    85975
=======================================
+ Hits        39106    39108    +2
+ Misses      40555    40553    -2
  Partials     6314     6314

Impacted Files
Coverage Δ

models/index.go
49.05% <ø> (ø)

modules/util/timer.go
42.85% <0.00%> (-42.86%)
:arrow_down:

services/pull/pull.go
41.78% <0.00%> (-0.41%)
:arrow_down:

modules/queue/workerpool.go
53.81% <0.00%> (+0.76%)
:arrow_up:

modules/git/blame.go
67.14% <0.00%> (+1.42%)
:arrow_up:

modules/charset/charset.go
75.75% <0.00%> (+4.04%)
:arrow_up:

Continue to review full report at Codecov.

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


Feature/media scaffolding

This introduces the basic database tables, models, views, and controllers for media objects, including their relationships to source sets and sources.  It addresses #8025 and follows the ER Diagram.  This does NOT allow admins to upload actual media files, just their related metadata.
Media objects (images, video, audio, and documents) form a conceptual group of "assets".  Assets belong to "attachable" objects (sources or source sets).  "Assets" and "attachables" don't lend themselves to explicit representation in the relational database, nor did it seem helpful to make "asset" and "attachable" classes or mixins in the application.  Instead, "assets" and "attachables" are defined through relationships and accessors in the models.  These conceptual object types are helpful because they allow you to refer generically to, say, a source's asset without needed to know whether that assets is an image, video, etc.  There is a shared_examples spec for "assets" that describes the expected behavior of an "asset".
This refactors some controller specs, introducing some shared_examples to DRY them out.
To run tests locally:  remember to do rake db:test:prepare.
To run the server locally:  remember to clean up your database to get rid of any old data that conforms to previous database schema.  I find it easiest to do bundle exec rake db:drop db:create db:migrate.

I'm not sure what metadata are needed yet for the assets.  Some tables might benefit from a meta column, of type text, that we can use for storing JSON for properties and other encoding-related data.
Each audio or video asset will have derivatives.  For example, a video will have a H.264 (.mp4) file and a Webm (.webm) file for cross-browser compatibility.  There should probably be separate tables for those, but I'm not sure yet.  I can add those later.  I have to figure out what's most efficient for the transcoding process.  It depends on whether we handle transcoding internally as separate jobs for each derivative.  Each derivative will be created as a separate job if we use Amazon Elastic Transcoder, and we would probably benefit from multiple records for those, but I'm uncertain.

Regarding the @attachable properties and attachable methods:
Can you explain the advantage of using these properties versus just <Model>.source?
I'm not clear as to what ApplicationController#load_attachable provides because it seems that, for any given thing like an audios controller or a videos controller it should always know what kind of object the audio or video belongs to: e.g. a Source. I know that Images can belong to Sources or SourceSets, but maybe there should be two different models for what are different kinds of images, or maybe there needs to be some kind of one-to-many relationship.
There is also this line in ApplicationController where it appears to be instantiating an object based on a string parameter received in the querystring and I'm wondering if that's safe.

@markbreedlove - in response to your comments above, I've made some [changes].(https://github.com/dpla/primary-source-sets/commit/e2641df6accd85673cb9a4b66aafdb2b457c9dbe)
I think your instinct is correct that it would be simpler to refer to attachables as <Model>.source.  I added an Image.source method and did away with Document.attachable, Video.attachable, and Audio.attachable.
I retained Image.attachable so that I could continue to enforce the one-to-one relationship between Image and Source, or between Image and SourceSet - for now.  Using attachable in this way seems to be an accepted Rails pattern for dealing with this type of polymorphic relationship.  As per our earlier discussion, we may reconsider the one-to-one relationship between Image and attachable, along with the one-to-one relationships between Source and other assets.  I'll discuss with FA this afternoon.
I also fixed the line in ApplicationController where it was instantiating an object based on a string parameter received in the querystring.  Good catch!  The the new method specifies which params to use.

I think this is good to go -- ready to squash. We've discussed pretty well what we still have to work on, and I think it will be good to have this merged now so there's a multimedia foundation and iterate over various additions later.

:squash:ed and rebased to develop


Blog App: Setup controllers

In this PR, I added controllers.
Changes

For each URL I  created:

A route.
An action in the respective controller.
A view file including only placeholders written in plain HTML.

Thank you for the feedback, @mckent05. 🙃


[#6156] Remove DeprecatedConfiguration

Codecov Report

Merging #6157 into master will decrease coverage by 0.86%.
The diff coverage is 97.61%.

@@            Coverage Diff            @@
##           master   #6157      +/-   ##
=========================================
- Coverage   40.47%   39.6%   -0.87%
=========================================
  Files        3026    3025       -1
  Lines       91391   91278     -113
  Branches    11896   11878      -18
=========================================
- Hits        36992   36153     -839
- Misses      51327   52098     +771
+ Partials     3072    3027      -45

Impacted Files
Coverage Δ

...nt/collector/config/StatReceiverConfiguration.java
62.83% <100%> (-12.32%)
:arrow_down:

...nt/collector/config/SpanReceiverConfiguration.java
63.51% <100%> (-9.16%)
:arrow_down:

...tor/config/AgentBaseDataReceiverConfiguration.java
63.91% <91.66%> (-9.24%)
:arrow_down:

...or/receiver/thrift/DispatchHandlerFactoryBean.java
0% <0%> (-100%)
:arrow_down:

...ervice/StreamExecutorServerInterceptorFactory.java
0% <0%> (-100%)
:arrow_down:

...ctor/receiver/grpc/service/StatServiceFactory.java
0% <0%> (-100%)
:arrow_down:

...int/collector/dao/hbase/TableDescriptorConfig.java
0% <0%> (-100%)
:arrow_down:

...oint/collector/cluster/AbstractClusterService.java
0% <0%> (-100%)
:arrow_down:

...ctor/receiver/grpc/service/SpanServiceFactory.java
0% <0%> (-100%)
:arrow_down:

...ver/grpc/service/AbstractServerServiceFactory.java
0% <0%> (-88.24%)
:arrow_down:

... and 103 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 716bcba...758dc93. Read the comment docs.


CAG expand / contract are creating degenerate CAG

The order of faces after these operations becomes wrong, and leads to unusable shapes afterwards (for extrude etc)

@kaosat-dev do you have any quick examples? i'd like to add some tests as part of V2.

@z3dev hmm that is going to be very hard , this was ages ago, I can try to reproduce but I honestly do not remember how I found the issue above


⚠ Secret Website 3 has degraded performance

In 744ddb8, Secret Website 3 ($SECRET_WEBSITE_3) experienced degraded performance:

HTTP code: 200
Response time: 2097 ms

Resolved: Secret Website 3 performance has improved in 3766468.


Thumbnails on search results

One of four thumbnails should appear in the search results for records in Open Vault:
Unique thumbnail - includes all records for digitized video, some records for digitized audio, and records for images
Digitized Audio Icon Thumbnail - records for digitized audio that do not have a unique thumbnail
Non-Digitized Audio Thumbnail - records for non-digitized audio
Non-Digitized Video Thumbnail - records for non-digitized video

@caseyedavis12 : will someone be providing the icons?

@mccalluc They are provided [here] (https://wgbh365-my.sharepoint.com/personal/michael_muraszko_wgbh_org/_layouts/15/guestaccess.aspx?guestaccesstoken=d2WP3VYrolkAfZ3yvHvAfVRAGHjMkN5JBg%2BnO%2B8LF6M%3D&docid=2_12ec2364298cc42938cd04028c14877cb) and in an email I sent you last week with the subject, "OpenVault Redesign deliverables"

I think this is complete.


TPV: alert, numerous problem annotations generated  under "regulation of cell size"

regulation of actin nucleation has a parentage to regulation of cell size (via regulation of actin filament length).
Not sure why (possibly should be "cellular component size"...(in fact  regulation cell size has been placed as a child of cellular component size which is also incorrect).
Anyway its making my "regulation of cell size" annotations look a bit crazy.

It looks like this is coming from the OBA import. Actin filament length is part_of some cell size. This seems incorrect. It needs to be fixed in oba. @cmungall  do you know who we contact?
As for the second part of this ticket. I think it is ok. A cell is a cellular component.

@ukemi How did you look up where Actin filament length is part_of some cell size is coming from?
thanks, Karen

You can follow the parents in Protege.

OBA has a general design pattern in which the quality hierarchy shadows the partonomy of the entities.
So for example:

plasma membrane morphology is part-of cell morphology
bone mass part-of organism mass
leaf shape part-of plant shape

Note that is_a is not the correct relationship type to use here. We're following a general idea of qualities being made up of other qualities.
I think we need to break this up for different qualities. Mass is additive over all parts of a cell. However, length is not.
This may take a bit of time. In the interim, I suggest importing only the subclass hierarchy from OBA. This may mean that we will need to manually add some links in GO, but it seems better than the false positives.

I'm going to do #13396 first so we can see the diff


Is the Library still maintained

I was planning to use it for one of my projects rather than writing java interfaces from python to the original library. Was just wondering is the library actively supported/used

@prateek05 I ended up exposing the java version through a jruby API and use it from my python apps via REST. I would love to eventually move over to using the python only implementation but I haven't had time to check this project's test coverage compared to the original simplenlg. please let me know if you end up using this in your project! would be interested to re-visit

Hi @prateek05
We are not working on it anymore but it's stable and tested, so you can use it if you'd like, it avoids the complexity of exposing the library behind an API, but if you want the latest features, you can consider using the java version.

@jdeniau  Thanks. I ll work on a wrapper over the java version.

@prateek05 check out https://github.com/jeremybmerrill/simplernlg

Hi guys. The Java version is no longer supported either. I have my own nlg library that was using simplenlg (https://github.com/roman-kutlak/nlglib) to do surface realisation, but I've now decided to extend this to support English instead. Will let you know when I have something working.

Is the library still maintained ? Good question! I see it was not tested for Python 3.4 yet, what about Python 3.7 or more ?

Hi @Catadanna , my previous response is still valid three years after 😄
In addition, if you are willing to upgrade it to python 3.x, feel free to open a PR, I will gladly release it 👍

OK I shall see! PS: I live in Lyon, too :-)


the azure dev support mentiones i cant trigger/start logic apps with azure cli on dictionary parameters

if that is true why they are here and documented here?
https://docs.microsoft.com/en-us/cli/azure/logicapp?view=azure-cli-latest#az_logicapp_start
If the logic apps can't be used in automation with parameters why do they exist in the first place ?
this trigger to take in a dictionary parameter to start

### This is regarding the ticket you have raised with Microsoft -<PHONE_NUMBER>000932

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

ID: 81e21e02-e42f-a0aa-e437-0ec8e73ab81b
Version Independent ID: 29316eb6-a99d-6ccf-6e4d-5e352e391c31
Content: az logicapp
Content Source: latest/docs-ref-autogen/logicapp.yml
GitHub Login: @rloutlaw
Microsoft Alias: routlaw

I need this start with a dictionary parameter to work from my dockercontainer

route to service team

The commands starting with az logicapp are used for Logic App Standard SKU offering only. az logicapp start starts the logic app site. This doesn't trigger the workflows in it. To trigger a workflow, we need workflow level commands which are not released yet.
From the screenshot, it looks like you are using the LogicApp Consumption SKU. For that these are the commands you can use-
https://docs.microsoft.com/en-us/cli/azure/logic?view=azure-cli-latest

I understand, so your ask for logicapp CLI extension is of a command to trigger a workflow with custom payload(like a dictionary). As of latest version, we do not have a command for that.
only workaround I can think of is to get the callback URI from portal or from ARM client, ad then call it again using ARM client. What are your security concerns with ARMClient?


Error: Debug Failure. False expression. in 'findPrecedingToken' during 'quickinfo'

This issue comes from crash dumps in telemetry. We've tried to de-duplicate issues on a best-effort basis, comparing the sequence of methods called and the command requested while ignoring line numbers.
TypeScript Version Prefix: 2.7.1
VSCode Version 1.19.1
command requested: quickinfo
hitting sessions: 1327
proportion of all sessions: 0.004953840066001426
stack
Error: Debug Failure. False expression.
    at findPrecedingToken (tsserver.js:64624:18)
    at getTokenAtPositionWorker (tsserver.js:64586:41)
    at getTouchingToken (tsserver.js:64558:16)
    at Object.getTouchingPropertyName (tsserver.js:64554:16)
    at getQuickInfoAtPosition (tsserver.js:82852:27)
    at Object.getQuickInfoAtPosition (template-language-service-decorator.js:56:24)
    at IOSession.Session.getQuickInfoWorker (tsserver.js:89781:62)
    at Session.handlers.ts.createMapFromTemplate._a.(anonymous function) (tsserver.js:88934:61)
    at unknown (tsserver.js:90355:88)
    at IOSession.Session.executeWithRequestId (tsserver.js:90346:28)
    at IOSession.Session.executeCommand (tsserver.js:90355:33)
    at IOSession.Session.onMessage (tsserver.js:90375:35)
    at Interface.<anonymous> (tsserver.js:91576:27)
    at emitOne (events.js:96:13)
    at Interface.emit (events.js:191:7)
    at Interface._onLine (readline.js:241:10)
    at Interface._normalWrite (readline.js:384:12)
    at Socket.ondata (readline.js:101:10)
    at emitOne (events.js:96:13)
    at Socket.emit (events.js:191:7)
    at readableAddChunk (_stream_readable.js:178:18)
    at Socket.Readable.push (_stream_readable.js:136:10)
    at Pipe.onread (net.js:560:20)

@aozgaa The line numbers don't seem to match 2.7.1, is this possibly an older or newer version?

It could be an insiders build whose version starts with 2.7.1. I could try to get the exact version in the future, but requires some work. How useful would that be, on a scale from 1-10?

Insider builds have different version anyways. this could either be 2.7.1 or a local build. We have changed the version in master, so moving forward we should not have this confusion.

Looks like a duplicate of #20788.

This issue reproduced in the following context:
TypeScript Version Prefix: 2.7.2
VSCode Version 1.21.0-insider
command requested: quickinfo
hitting sessions: 435
proportion of all sessions: 0.008930587776386294
stack
Error: Debug Failure. False expression.
    at findPrecedingToken (tsserver.js:64794:18)
    at getTokenAtPositionWorker (tsserver.js:64756:41)
    at getTouchingToken (tsserver.js:64728:16)
    at Object.getTouchingPropertyName (tsserver.js:64724:16)
    at getQuickInfoAtPosition (tsserver.js:83050:27)
    at Object.getQuickInfoAtPosition (template-language-service-decorator.js:59:24)
    at IOSession.Session.getQuickInfoWorker (tsserver.js:89979:62)
    at Session.handlers.ts.createMapFromTemplate._a.(anonymous function) (tsserver.js:89132:61)
    at unknown (tsserver.js:90553:88)
    at IOSession.Session.executeWithRequestId (tsserver.js:90544:28)
    at IOSession.Session.executeCommand (tsserver.js:90553:33)
    at IOSession.Session.onMessage (tsserver.js:90573:35)
    at Interface.<anonymous> (tsserver.js:91774:27)
    at emitOne (events.js:96:13)
    at Interface.emit (events.js:191:7)
    at Interface._onLine (readline.js:241:10)
    at Interface._normalWrite (readline.js:384:12)
    at Socket.ondata (readline.js:101:10)
    at emitOne (events.js:96:13)
    at Socket.emit (events.js:191:7)
    at readableAddChunk (_stream_readable.js:178:18)
    at Socket.Readable.push (_stream_readable.js:136:10)
    at Pipe.onread (net.js:560:20)


Registry tool: registry apply doesn't support updating fields to their zero value

It makes API calls to the Update methods for each resource and uses an implied field mask that only modifies fields that aren't set to their zero value. There's no mechanism for providing a field mask or performing a full replacement with registry apply.
Some ideas

Allow field masks somewhere in the registry object YAML
Allow field masks as an option to the apply command
Use a special value to signal that the field should be cleared
Allow full replacement of resources as an option to the apply command

Another idea: The information is in the YAML. I wonder if we could be smart and determine what fields were actually declared?


Bolding a simple list causes TypeError in browser console

Describe the bug
Clicking the UL (list) action after bolding a simple list throws the error TypeError: Cannot read property 'nextElementSibling' of null.
To Reproduce
Steps to reproduce the behavior:

In Chrome, launch the browser debug tools (F12) and then open the basic editor example: https://nhn.github.io/tui.editor/latest/tutorial-example01-editor-basic
Select the WYSIWYG editor (problem also happens in the Markdown editor).
Click the UL action to create a bulleted list.
Type "foo" to create the first list item and press enter.
Type "bar" to create a second list item and press enter twice (so that there is an empty line after the list.
Select all text (CTRL-A on windows) including the blank lines below the list.
Press the Bold button to bold all items in the list.
Press the UL button.
All items are bolded, but the following error is logged to the browser debug console:

tui-editor-Editor-full.js:109523 Uncaught TypeError: Cannot read property 'nextElementSibling' of null
    at WwListManager._getNextLine (tui-editor-Editor-full.js:109523)
    at WwListManager._getNextLine (tui-editor-Editor-full.js:109538)
    at WwListManager.getLinesOfSelection (tui-editor-Editor-full.js:109501)
    at Command.exec (tui-editor-Editor-full.js:151069)
    at CommandManager.exec (tui-editor-Editor-full.js:15251)
    at tui-editor-Editor-full.js:15201
    at tui-editor-Editor-full.js:36952
    at forEachArray (tui-editor-Editor-full.js:11653)
    at Object.forEach (tui-editor-Editor-full.js:11730)
    at EventManager.emit (tui-editor-Editor-full.js:36951)

Expected behavior
No exception when clicking the UL action after adjusting the formatting of the list.
Screenshots

Desktop (please complete the following information):

OS: Windows 10
Browser Chrome
Version 79.0.3945.130 (Official Build) (64-bit)

@rwhepburn Thanks for the detailed explanation. I checked and it doesn't seem to be reproduced in Markdown. Also, the error code indicates that it is happening in the module related WYSIWYG. I've checked on the problem at WYSIWYG!
And as for the bug fix plan, the editor is working on a major version update. So we'll fix the bug after this is done. It will only be patched in the current version when it is determined to be a really critical issue.
I would appreciate it if you consider this plan. 🙏


[WIP] use crun instead of runc

Switching to crun, as crun is the defacto reference implementation of OCI for cgroup v2.
Both containerd and CRI-O are failing due to incompatibility related to /sys/fs/cgroup structure:
containerd:
  message: 'failed to create containerd task: OCI runtime create failed: mount `
/sys/fs/cgroup/devices//docker/2b9dfd11943b6ef3fdd16888d6589e797ec39c5443d4972d4
75ac489553869bf` to ''/run/.ro436300186/user/1000/containerd/io.containerd.runti
me.v2.task/k8s.io/be6bdbf67401d16fd6b35305af915def13725133af2b5f95f1ed7e662d905b
60/rootfs/sys/fs/cgroup/devices'': No such file or directory: unknown'

crio:
  message: container create failed: mount `/sys/fs/cgroup/devices//docker/e8abdf
947d3c3f27dcd40874224605cbacebaa978f1ada7644ce0d4a86cd9b53` to '/home/user/.loca
l/share/containers/storage/vfs/dir/fa622496b8a0cb247bb4e31572835e449a3299e4ef193
6c317ec80c9e7bd6c0b/sys/fs/cgroup/devices': No such file or directory

Reproducer:
$ make up
$ export KUBECONFIG=$(pwd)/config/localhost.kubeconfig
$ kubectl get nodes -o wide
NAME           STATUS   ROLES    AGE   VERSION             INTERNAL-IP    EXTERNAL-IP   OS-IMAGE       KERNEL-VERSION     CONTAINER-RUNTIME
297bdd1e1f87   Ready    <none>   20s   v1.18.0-usernetes   <IP_ADDRESS>   <none>        Ubuntu 19.10   5.3.0-29-generic   cri-o://1.17.0-dev
f19367ef5286   Ready    <none>   19s   v1.18.0-usernetes   <IP_ADDRESS>   <none>        Ubuntu 19.10   5.3.0-29-generic   containerd://1.3.0-317-g7811aa75
$ kubectl create deployment --image=nginx:alpine foo
$ kubectl scale deployment --replicas=2 foo
$ kubectl get pods -o yaml

crun version: v<IP_ADDRESS>

@AkihiroSuda have you a chance to check the crun git master version?

Attempted to use github artifact binary from master, but couldn't wget/curl 😢
Will try again later.

Restarted CI with https://github.com/AkihiroSuda/crun/releases/tag/snapshot-c69ea8c


BasicAuth not adding to generated doc (BasicAuth security scheme not working)

Hello. I use spring-fox 3.0 and spring-boot 2.3.2
<dependency>
	<groupId>io.springfox</groupId>
	<artifactId>springfox-boot-starter</artifactId>
	<version>3.0.0</version>
</dependency>

When i try adding basic authorization to my API descpription this is not working (no securityScheme block added to generated doc). I use springfox.documentation.service.BasicAuth class for aim my goal.
This is my api descpition:
@Configuration
public class SwaggerConfig {
    @Bean
    public Docket openApi() {
        return new Docket(DocumentationType.OAS_30)
                .groupName("MYAPI")
                .apiInfo(apiInfo())
                .select()
                .apis(RequestHandlerSelectors.basePackage("com.mypackage"))
                .paths(PathSelectors.regex("/.*"))
                .build().securitySchemes(Arrays.asList(new BasicAuth("basicAuth")) //THIS IS NOT WORKING. NO securityScheme block generated
                .securityContexts(Arrays.asList(securityContext()));
    }

    private ApiInfo apiInfo() {
        return new ApiInfoBuilder()
                .title("MYAPI")
                .description("My api")
                .version("1.0")
                .build();
    }

    private SecurityContext securityContext() {
        return SecurityContext.builder()
                .securityReferences(Arrays.asList(basicAuthReference()))
                .forPaths(PathSelectors.ant("/**"))
                .build();
    }

    private SecurityReference basicAuthReference() {
        AuthorizationScope authorizationScope = new AuthorizationScope("basicAuth","basicAuth");
        return new SecurityReference("basicAuth", new AuthorizationScope[0]);
    }
}

I small research source code and finded this class: springfox.documentation.oas.mappers.SecuritySchemeMapper
next method not covering BasicAuth class!!!
void mapScheme(Map<String, SecurityScheme> map, springfox.documentation.service.SecurityScheme scheme) {
    SecurityScheme mapped = null;
    SecurityScheme securityScheme = new SecurityScheme()
        .extensions(new VendorExtensionsMapper().mapExtensions(scheme.getVendorExtensions()));
    if (scheme instanceof HttpAuthenticationScheme) {
      mapped = securityScheme
          .type(SecurityScheme.Type.HTTP)
          .description(scheme.getDescription())
          .bearerFormat(((HttpAuthenticationScheme) scheme).getBearerFormat())
          .scheme(((HttpAuthenticationScheme) scheme).getScheme());
    } else if (scheme instanceof OAuth2Scheme) {
      OAuthFlows flows = new OAuthFlows();
      Scopes scopes = new Scopes();
      ((OAuth2Scheme) scheme).getScopes()
                             .forEach(s -> scopes.addString(s.getScope(), s.getDescription()));
      OAuthFlow flow = new OAuthFlow()
          .authorizationUrl(((OAuth2Scheme) scheme).getAuthorizationUrl())
          .refreshUrl(((OAuth2Scheme) scheme).getRefreshUrl())
          .tokenUrl(((OAuth2Scheme) scheme).getTokenUrl())
          .scopes(scopes);
      switch (((OAuth2Scheme) scheme).getFlowType()) {
        case "password":
          flows.password(flow);
          break;
        case "clientCredentials":
          flows.clientCredentials(flow);
          break;
        case "authorizationCode":
          flows.authorizationCode(flow);
          break;
        case "implicit":
        default:
          flows.implicit(flow);
          break;
      }
      mapped = securityScheme
          .type(SecurityScheme.Type.OAUTH2)
          .description(scheme.getDescription())
          .flows(flows);
    } else if (scheme instanceof ApiKey) {
      mapped = securityScheme
          .type(SecurityScheme.Type.APIKEY)
          .name(scheme.getName())
          .in(mapIn(((ApiKey) scheme).getPassAs()));
    } else if (scheme instanceof OpenIdConnectScheme) {
      mapped = securityScheme
          .type(SecurityScheme.Type.OPENIDCONNECT)
          .name(scheme.getName())
          .openIdConnectUrl(((OpenIdConnectScheme) scheme).getOpenIdConnectUrl());
    }
    if (mapped != null) {
      map.put(scheme.getName(), mapped);
    }
  }

this only accept HttpAuthenticationScheme classes.
When i change BasicAuth to HttpAuthenticationScheme all working fine.
return new Docket(DocumentationType.OAS_30)
                .groupName("APIEKD")
                .apiInfo(apiInfo())
                .select()
                .apis(RequestHandlerSelectors.basePackage("com.mypackage"))
                .paths(PathSelectors.regex("/.*"))
                .build().securitySchemes(Arrays.asList(HttpAuthenticationScheme.BASIC_AUTH_BUILDER.name("basicAuth").description("Basic authorization").build())) //THIS IS WORKING NOW!
                .securityContexts(Arrays.asList(securityContext()));

Please fix it or update documentation and delete BasicAuth class from libs. Thank you!
P.S. Swagger2 not working too with BasicAuth

Thank you for pointing that out. BasicAuth should've been deprecated as it is superseded by HttpAuthenticationScheme for swagger2. Seems like its still not working with swagger 2.0. I'll look at this. Thanks for reporting

Thank you for pointing that out. BasicAuth should've been deprecated as it is superseded by HttpAuthenticationScheme for swagger2. Seems like its still not working with swagger 2.0. I'll look at this. Thanks for reporting

Any update on this ? I am facing the same issue

Any workaround would be much appreciated. Stuck at this since a week.

Any workaround would be much appreciated. Stuck at this since a week.

Use HttpAuthenticationScheme.BASIC_AUTH_BUILDER

Any workaround would be much appreciated. Stuck at this since a week.

Use HttpAuthenticationScheme.BASIC_AUTH_BUILDER

Worked Perfect! Thanks a lot.


Add cancel event

In my use case, using hand tracking to fire the blink events sometimes the user points for a few frames but not long enough to trigger the blink so in that situation I want to cancel it so that the line and hit entity disappear.
This allows it to listen for a cancel event which stops the blink.

This looks very useful and I like the implementation!
Any chance you could add an example of how it can be used too?

Sure, where do you want the example?

The examples live on this GitHub Pages site (https://jure.github.io/aframe-blink-controls/), and the source for them is in this repo's docs folder (https://github.com/jure/aframe-blink-controls/tree/main/docs). In order to not increase the number of examples for this, I think the terrain example would be a good place to demonstrate the cancel functionality, since the title of it is 'Example with curved terrain and events'.
What do you think?

Sure.

If it detects a squeeze or gripdown event it cancels the navigation

Thank you so much, Ada! Apologies for the late response - to try and make it right I've now also released a new version of aframe-blink-controls.


[WIP][BugFix] fix short circuit behavior for several operators

What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] I have added user document for my new feature or new function

@Mergifyio backport branch-2.2 branch-2.3 branch-2.4

BE Build Success

run starrocks_admit_test


Blub

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.

Comment

2

3

4

/support

/support

/support

/support

/test

/support

/close

/close

/close

/support

/close

/close

/close


Segmentation fault

~/.bundler/ruby/1.8/gems/aws-s3-0.6.2/lib/aws/../../support/faster-xml-simple/lib/faster_xml_simple.rb:162: [BUG] Segmentation fault
ruby 1.8.7 (2009-06-12 patchlevel 174) [x86_64-linux], MBARI 0x6770, Ruby Enterprise Edition 2009.10

I've run into this regularly and randomly as well.  Very annoying when it occurs during long-running jobs.


Unable to clone repo

Dev Home version
0.503.261.0
Windows build number
10.0.22621.2361
Other software
OS Build Version: 10.0.22621.2361.amd64fre.ni_release.220506-1250
.NET Version: .NET 6.0.21
Steps to reproduce the bug
Trying to clone a repo from bitbucket I get this error code and message

Expected result
Successfully cloning the repo
Actual result
just displays error code 80131500
Included System Information
Physical Memory: 31.69GB (15.32GB free)
Processor Architecture: x64
Included Extensions Information
Extensions:
Microsoft.Windows.DevHome_0.503.261.0_x64__8wekyb3d8bbwe

Hello @AlthalusDGr8
What is the full repo URL you are trying to clone?  Is the repo public or private?

Its a private repo on bitbucket

@krschau 80131500 is the generic HRESULT for "a .net exception happened" - could we please special case that and provide a meaningful error message (from the exception itself) instead? @dhoehna

@AlthalusDGr8 Dev Home does not currently support private repositories outside of GitHub (and ADO is coming).  The extensions are responsible for authentication and cloning with that account.  So a BigBucket extension will need to be created to enable private repository cloning.
That said, I agree with @asklar that we need to better display that to the user.  Changing the title of this issue to reflect that.

@AlthalusDGr8 this is an issue with older versions of the extension and DevHome.  Can you please upgrade both the extension and DevHome to their most recent versions and reproduce the error?  The message should be more descriptive.

Hello we have interest into use private repos from bitbucket, does cloning have any change?

Issue #954 is tracking BitBucket support. I don't think it's on our current timeline, but anyone is free to create their own Dev Home extension that would support it!

This issue should be reopened. Leaving aside actual support for private repos, the error I get should not be just "80131500" but an explanation of what is going wrong.


ENH: Add missing getter method to itk::LaplacianOperator ivar

Add missing getter method to itk::LaplacianOperator class ivar.
PR Checklist

[X] No API changes were made (or the changes have been approved)
[X] No major design changes were made (or the changes have been approved)

Ain't there a way to use Get macros to return a const pointer?

There's itkGetConstMacro(DerivativeScalings, const double *) using the macro defined in Modules/Core/Common/include/itkMacro.h.

There's itkGetConstMacro(DerivativeScalings, const double *) using the macro defined in Modules/Core/Common/include/itkMacro.h.

Thanks @Leengit. Changed in 787827e. Let's see if the CIs are happy.


Remove workmanager

It is Just Test
In our app workmanager doesn’t work For IOS anyways and works not properly for Android (Xiomi, Honor).
I suggest to use without workmanager: scheduleNotification,  pendingNotificationRequests and compute.
I search and tried I couldn’t register scheduleNotification in worknamager,
But there are minuses:

user sometimes should open up the app.
if user would not use the app long time notification will not come.

And plusses are:

It works for IOS and Android. I tested it.
shown before messages will not be shown again
Feed ids no need to be cache
If the time is not yet come notification will not be shown
We can give the meetup time and notification time separately
Via scheduleNotification id registers notification only once

Notifications come right on time.

Not quite ready now, I need to test on real device and fix unit tests, finally run real_app integration test

I think we are ready. But check again. Also guide me on
final _feeds = await compute(FeedRepo().fetchData, langCode);


List also panels nested inside a "row" type of panel

Hello,
Found this one while testing internally. My problem is that I have plenty of dashboards that have collapsed rows, meaning the panels are nested this way:
{
    "panels": [
        {
            "type": "row",
            "panels": [...]
        }
    ]
}

This PR ensures those panels are parsed and linted too.

Thanks @wuzuf!


How to slide to next and previous month, from outside the calendar.

I want to add controls (>> << to the header) to slide to next  and previous month for the desktop user.
For the month on the year, if the 1st date falls on Sunday, there is no way to go to the previous month for the desktop user.

@rahulthakare Have a look at the changeDate event in the README. When user clicks the >> << controls, in your click handler you can broadcast the changeDate event.


Restore the previous end time generation

The current date picker with the "latest" selection always discard the last druid interval.
My guess is that the bug has been introduce with this PR https://github.com/implydata/pivot/pull/358 so I have reverted the problematic line. Testing against a running druid cluster seems to work

Thank you for submitting this pull request, however I do not see a valid CLA on file for you. Before we can merge this request please visit https://yahoocla.herokuapp.com/ and agree to the terms. Thanks! :smile:

@vmaurin thanks for the PR. However, I don't get what the problem is. Here's the screenshot of swiv app we have -

The latest interval has end time - 9:52 PM and current time is also 9:52 PM. Am I missing anything ?

@pjain1 If you have a druid backend, when querying the max time, swiv will be 9:52 PM, Then for querying the data, swiv will send an interval to Druid with 9:52, or max time in Druid interval are exclusive

I see, this sounds correct for the latest interval. I am not familiar with the code so can you make sure that this function is not used when interval is chosen manually otherwise it would be incorrect to change end time from what is manually chosen.

I am not so familiar either, I have reverted this line changed by the Timekeeper commit
Looking to the code, it seems that this function is used in two different cases :

directly, with dataCube.getMaxTime(timekeeper)
with interval clauses, only for relative interval with the following code

public evaluate(now: Date, maxTime: Date, timezone: Timezone): FilterClause {
    if (!this.relative) return this;
    return this.changeSelection(r(FilterClause.evaluate((this.selection as Expression), now, maxTime, timezone)));
  }

So it looks like it is not applying to non relative selection. Unfortunately, with my data cube, as the lower druid granularity we have is hours, I have no selection with a minute level. But when I am selecting a specific interval, it doesn't took data out the specified interval, so it seems to behave as expected

Released the patch in 0.9.41

Actually use 0.9.42 instead. 0.9.41 is busted, sorry for that.


Function 1 skeleton - Add team-members

Create a skeleton and documentation for the add_team_members function.

Completed with https://github.com/UBC-MDS/pokehelpyer/pull/15.


Improved overall style constuction & improved file structure

Lots of small changes and improvements to GUI, such as

Global color variables
Drawer styling redone and improved upon
More closely follows google's guidelines

Styled two more views

Shotcaller view
Upload view

EJS file content moved to element files. This has a few implications:

Everything except style.css is dumped overboard. Can't use variables there, so don't use it.
Don't add content to EJS files, as I'll just move it out anyways.

Created some subfolders in /Elements/ because it was a mess.

Can't find any weird things, great work!

Some small issues, looks good to me.

Thanks!


Cannot collapse code in admin plugin anymore

Describe the bug
In the JSON payload in the admin plugin, when trying the collapse a chunk of code, the resource view closes.
In staging, 1.7.0-M2
https://user-images.githubusercontent.com/11705901/144201610-08ea0093-bb9e-44ab-8c11-c8565c3699f5.mp4
What part of Nexus is affected: Fusion
To Reproduce

Go to the project view, browse tab
Click on any resource
Open the admin plugin
Collapse any chunk of code in the JSON payload
The resource view closes

Expected behavior
1-4. same
5. the chunk is collapsed

Bug fixed in 1.7.0-M3


https redirection for http connections

fixed the previous flawed PR for issue #648

@rohitnairtech Rohit, the problem is actually even bigger: in-front of the python server we have a Go caching http-frontend server, and so http redirection had to be handled in its code. The python server that you started fixing always gets insecure HTTP queries, and never HTTPS, because HTTPS is terminated by the front-end server. The problem is fixed now in the frontend server. Still thanks a lot for your interest to the project, and feel free to send new bugs, feature requests or contributions.


Change highlight color

For those who are searching for this issue, there is a file _sass/_variables.scss including all the color settings.


104 write getting started

@David--K I've reworked this with the MS-Styleguide, please review again :)


Fix Docker+SSH protocol with private key

What does this PR do?
Fix Docker+SSH protocol with private key.
Motivation
Fix docker+ssh access in CI.
Additional Notes
Possible Drawbacks / Trade-offs
Describe how to test/QA your changes
E2E APM tests work.
Reviewer's Checklist

[x] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[x] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[x] A release note has been added or the changelog/no-changelog label has been applied.
[x] Changed code has automated tests for its functionality.
[x] Adequate QA/testing plan information is provided. Except if the qa/skip-qa label, with required either qa/done or qa/no-code-change labels, are applied.
[x] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[x] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[x] If applicable, the need-change/operator and need-change/helm labels have been applied.
[x] If applicable, the k8s/<min-version> label, indicating the lowest Kubernetes version compatible with this feature.
[x] If applicable, the config template has been updated.

/merge


Command "game" is not defined.

What steps will reproduce the problem?
type in the command line "./yii game"
What is the expected result?
A little game like Yii invaders
What do you get instead?
Error.

Try ./yii yii


[Suggestion] It would be better to add the "Deleting database account 'xxx'..." log message next to the "Delete Database Account 'xxx'"

OS: Win10
Build Version: 20220628.2
Repro Steps:

Right click a database server -> Delete it.
Check the log message in the Activity Log.

Actual:
There is no "Deleting database account 'xxx'..." log message next to the "Delete Database Account 'xxx'".

Suggestion:
It would be better to add the "Deleting database account 'xxx'..." log message next to the "Delete Database Account 'xxx'", like below:

The additional message doesn't seem to carry any additional information. Let's keep the behavior as is for now.


Multiple step search not working

A questions came in from the field to us.  Any thoughts?
The requirement is to find all military installations within 100 miles of a particular point and be able to download a report with all the names listed.
Here's what I'm doing:

Click on Search.
Click on By Attribute.
Type in airfield - I used Mc Connell, as they're the requesting location, so I select Mc Connell AFB.
Clear the results tab.
Click on By Shape.
Change the Buffer to 100 Miles. (Note: Display Buffer Only box doesn’t work, nor does Add to Existing Results box, even when the results aren't cleared).
Click on the point and then click on McConnell's airfield.
Nothing happens. No results display.

What am I doing wrong? How do I get a list to display?

I tried this at home with a different service.  It worked.  I need to test this layer at work.  I am having trouble Add to Existing Results.  The check box is disabled.  Gremlins.


Handle different failure/error scenarios during socket write

Similar to issue #225
We need to handle different error/failure scenarios for connection handling and socket writes.

@JyotinderSingh @lucifercr07 @AshwinKul28 I fixed issue #225 and am working on this one as well. Please assign it to me

@JyotinderSingh @lucifercr07 @AshwinKul28 I fixed issue #225 and am working on this one as well. Please assign it to me

Done

Hello @ayushsatyam146,
There has been no activity on this issue for the past 5 days.
It would be awesome if you keep posting updates to this issue so that we know you are actively working on it.
We are really eager to close this issue at the earliest, hence if we continue to see the inactivity, we will have to reassign the issue to someone else. We are doing this to ensure that the project maintains its momentum and others are not blocked on this work.
Just drop a comment with the current status of the work or share any issues you are facing. We can always chip in to help you out.
Thanks again.

I am still working on it. Will raise a PR by tomorrow

Hello @ayushsatyam146,
There has been no activity on this issue for the past 5 days.
It would be awesome if you keep posting updates to this issue so that we know you are actively working on it.
We are really eager to close this issue at the earliest, hence if we continue to see the inactivity, we will have to reassign the issue to someone else. We are doing this to ensure that the project maintains its momentum and others are not blocked on this work.
Just drop a comment with the current status of the work or share any issues you are facing. We can always chip in to help you out.
Thanks again.


Integration Tests in Jenkins (pure K8s)

The T2 clusters were rebuilt to use pure K8s instead of only the API/control plane in #182.
We now have to make sure that the integration tests also run in this environment.

[ ] superset (as spike)
[ ] solve problem w/ hcloud clusters' DNS
[ ] rework 'available-versions' image to provide helm versions
[ ] Add superset build job to Jenkins CI
[ ] other operators (add list here)

Although overall progress is blocked by the Helm chart stuff, I can start by working on the available versions Docker image


Upstream / Origin

The pull / push from upstream-buttons work on the default repository, which is called origin per default. wouldn’t it be better to let the tooltips reflect this and call then push / pull from origin?

@vonloxley: Origin is not the default. You can have multiple origins. For example to pull from repo x, but push to repo y.
I leave it that way. This plugin only provides a prepared toolbar, which you could have configured yourself within the IDE. The label comes from the build-in NB actions

No problem, but please note, that you can have multiple remotes, of which the default one, per convention, is called origin.


Feat: apply RFC3339 standard to typia.tags.Format<"date-time"|"date"|"time">

this pr is about #1059

Great job, would be published to the next version.


stripe: Add page to show list of past customer invoices.

Generally looks pretty reasonable to me, aside from the data issues that existing invoices look weird, but there's likely nothing to be done about that. Posted one comment.

Updated!

Looks great, merged, thanks @amanagr!


Temperature sensor

Is it possible to use the Temperature as a sensor for automation? In my setup I don't seem to be able to use it for anything (although can read it)

Unfortunately, this is rather a limitation of Apple Home App - https://github.com/KhaosT/HAP-NodeJS/issues/375


Better sessions

Given this change one can now use the technique as describe by the SQLAlchemy docs regarding joining a session into an external transaction. For example, and bearing in mind I also have SQLALCHEMY_COMMIT_ON_TEARDOWN = True as a default setting on my app, I can now rollback everything as one transaction:
import pytest

def pytest_runtest_call(item):
    # Flush all fixtures before the test is called
    if hasattr(item, 'funcargs') and 'db_session' in item.funcargs:
        item.funcargs.get('db_session').flush()

@pytest.yield_fixture()
def db_session(app, db):
    with app.app_context():
        connection = db.engine.connect()
        transaction = connection.begin()
        options = dict(bind=connection, binds={})
        session = db.create_scoped_session(options=options)
        db.session = session
        yield session
        transaction.rollback()
        connection.close()
        session.remove()

@pytest.fixture()
def user(db_session):
    user = MyUserClass(...)
    db.session.add(user)

This would take care of #243.

Now merged, unfortunately automagic was lost in the rebase.

No worries. Thanks, though!

Hi @immunda, could you consider releasing new version containing this change?

In the meantime, this is the workaround that I use for transactional testing with Flask-SQLAlchemy. This is a py.test fixture, but the same approach would work with any testing framework supporting setup/teardown. This code assumes that you're already inside the Flask application context:
    connection = db.engine.connect()
    transaction = connection.begin()

    # Patch sessionmaker to use a predefined connection
    from flask import _app_ctx_stack
    db.get_binds = lambda app: {}
    db.session = scoped_session(
        session_factory=lambda: SignallingSession(_db, bind=connection),
        scopefunc=_app_ctx_stack.__ident_func__)

    try:
        yield db
    finally:
        db.session.remove()
        transaction.rollback()
        connection.close()

In case anybody is interested, I found a more elegant way of temporarily patching Flask-SQLAlchemy to use a specific connection without replacing the whole session.
Assuming create_app is your application factory and _db is a SQLAlchemy instance:
@pytest.yield_fixture
def app():
    with create_app().app_context():
        yield _app

Using monkeypatch from py.test:
@pytest.yield_fixture
def db(app, monkeypatch):
    connection = _db.engine.connect()
    transaction = connection.begin()

    # Patch Flask-SQLAlchemy to use our connection
    monkeypatch.setattr(_db, 'get_engine', lambda *args: connection)

    try:
        yield _db
    finally:
        _db.session.remove()
        transaction.rollback()
        connection.close()

Using patch from unittest.mock:
@pytest.yield_fixture
def db(app, monkeypatch):
    connection = _db.engine.connect()
    transaction = connection.begin()

    # Patch Flask-SQLAlchemy to use our connection
    with patch('app.db.get_engine') as get_engine_mock:
        get_engine_mock.return_value = connection

        try:
            yield _db
        finally:
            _db.session.remove()
            transaction.rollback()
            connection.close()

This is now out in the wild with version 2.1

Thanks much, Phil!
On Oct 23, 2015 5:58 AM, "Phil Howell"<EMAIL_ADDRESS>wrote:

This is now out in the wild with version 2.1
—
Reply to this email directly or view it on GitHub
https://github.com/mitsuhiko/flask-sqlalchemy/pull/249#issuecomment-150532382
.

In case anybody is interested, I found a more elegant way of temporarily patching Flask-SQLAlchemy to use a specific connection without replacing the whole session.
Assuming create_app is your application factory and _db is your SQLAlchemy instance:
@pytest.yield_fixture
def app():
    with create_app().app_context():
        yield _app

Using monkeypatch from py.test:
@pytest.yield_fixture
def db(app, monkeypatch):
    connection = _db.engine.connect()
    transaction = connection.begin()

    # Patch Flask-SQLAlchemy to use our connection
    monkeypatch.setattr(_db, 'get_engine', lambda *args: connection)

    try:
        yield _db
    finally:
        _db.session.remove()
        transaction.rollback()
        connection.close()

Using patch from unittest.mock on Python 3 (mock on Python 2):
@pytest.yield_fixture
def db(app):
    connection = _db.engine.connect()
    transaction = connection.begin()

    # Patch Flask-SQLAlchemy to use our connection
    with unittest.mock.patch('app.db.get_engine') as get_engine_mock:
        get_engine_mock.return_value = connection

        try:
            yield _db
        finally:
            _db.session.remove()
            transaction.rollback()
            connection.close()

This is all that worked for me, unless I'm being dumb and missed some other option.

@adambyrtek 's approach works, exception a minor fix due to the newer version of SQLAlchemy is expecting a different call signature for db.get_engine

In case anybody is interested, I found a more elegant way of temporarily patching Flask-SQLAlchemy to use a specific connection without replacing the whole session.
Assuming create_app is your application factory and _db is your SQLAlchemy instance:
Using monkeypatch from py.test:
@pytest.yield_fixture
def db(app, monkeypatch):
    connection = _db.engine.connect()
    transaction = connection.begin()

    # Patch Flask-SQLAlchemy to use our connection
    monkeypatch.setattr(_db, 'get_engine', lambda *args: connection)

    try:
        yield _db
    finally:
        _db.session.remove()
        transaction.rollback()
        connection.close()

The monkeypatch line should be
monkeypatch.setattr(_db, 'get_engine', lambda *args, **kwargs: connection)

Or else you would see an error like this:
self = <sqlalchemy.orm.session.SignallingSession object at 0x1088feb80>, mapper = <Mapper at 0x10755e640; MasterProduct>
clause = <sqlalchemy.sql.selectable.Select at 0x1089a4730; Select object>

    def get_bind(self, mapper=None, clause=None):
        """Return the engine or connection for a given model or
        table, using the ``__bind_key__`` if it is set.
        """
        # mapper is None if someone tries to just get a connection
        if mapper is not None:
            try:
                # SA >= 1.3
                persist_selectable = mapper.persist_selectable
            except AttributeError:
                # SA < 1.3
                persist_selectable = mapper.mapped_table

            info = getattr(persist_selectable, 'info', {})
            bind_key = info.get('bind_key')
            if bind_key is not None:
                state = get_state(self.app)
>               return state.db.get_engine(self.app, bind=bind_key)
E               TypeError: <lambda>() got an unexpected keyword argument 'bind'

../../../.venvs/server-D2-ixepO/lib/python3.8/site-packages/flask_sqlalchemy/__init__.py:166: TypeError

I am running Flask-SQLAlchemy==2.4.1 and SQLAlchemy==1.3.16.


Fixes changing keyboard layout in vscode web is broken

This PR fixes #140291 though not sure if it's 100% correct.

@alexdima can we merge/fix this for this january release :pray:

:+1: I think the change is good, but it looks like the code might have other problems. For example, I was unable to get it to re-detect when going from a user selected layout to "Auto Detect".
fyi @rebornix


Which OAuth2 flow to use for a UWP app?

Hi, I'm attempting to create an UWP (Win10) app as a client to meetup.com. I'm currently stuck at authenticating to meetup.com, since I think you can't handle my redirect_uri on the oauth2/authorize endpoint. My app will be a native app, so the redirect_uri is the actual appId in the form of a URI like ms-app://<incredibly long appID string>.
I get to the pint where I can chose to grant or deny my app access to my meetup account. When I chose allow I'm presented with an error message stating that

We can't connect to the service you need right now. Check your network connection or try this again later.

The exception I get mentions that the protocol is unknown. Am I stuck with using implicit flow or is there something you can do to fix this? How are the official iOS and Android apps handling this?

Nevermind, I found a way to work around the issue. WebAuthenticationBroker can monitor just any URI to regain control of the authentication flow, so I'm using https://www.meetup.com as a redirect.


[5.1] Allow Arr::get from objects that implements ArrayAccess

Allow to use Arr::get method on objects that implement ArrayAccess. It's something that I stumbled upon and is really useful.

Looking at it again, it probably needs an additional check after the instanceof.

Yeah it has no key check after instanceof check. Also needs unit tests.

Will adjust it and write some tests once I have a free moment.

Please send new features to 5.2. 5.1 is open for bug fixes only at this point.


Set Mongo client to prefer reads from secondaries

This will spread the read requests across the secondary Mongo instances
automatically and spare the primary from taking all the load as is
happening currently. This should obviate the need to increase the sizes
of the Mongo instances and should also improve performance.

I suspect this was switched to "prefer_primary" when the problem of a third of requests going to the DR replica in the DR environment with extra high-latency penalty was discovered. We don't have this in AWS so it should be fine to read from any secondaries there.


Extension breaks Stadia website

I was having trouble with the Google Stadia site not loading. I was surprised to find it was this extenxion causing the issue. I'm not exactly sure why this issue exists but it does.

I am afraid that if I address this issue I will be providing more Stadia support than Google ever did.

Lol, honestly fair enough. I respect that.


FFmpeg options for swaygrab

Related to #1233

Document this in the man page too

Will that work? If it does I'll squash the commit

Yes, thanks

Thanks!


Fix info bar tool tip

The tool tip on the information bar bellow the navbar doesn't unwrap or display anything when it's clicked. I would like to get assigned this issue, for GsSoC22.

Sure, That would be great! Make sure you follow the Google Design Pattern , This tooltip has been inspired from it only.


Different Output with other library

I was using geralt because it uses libsodium implementation but it seems it produce different output with other library such as Isopoh.Cryptography.Argon2 .
       public IActionResult Test(string password)
       {
           byte[] passwordBytes = Encoding.UTF8.GetBytes(password);
           // salting is hex string
           string salting = "6c6f62616c6f62613264616477696639"; // plain text : lobaloba2dadwif9
           byte[] salt = Convert.FromHexString(salting);

           var config = new Argon2Config
           {
               Type = Argon2Type.HybridAddressing, //argon2id
               Version = Argon2Version.Nineteen,
               TimeCost = 2, // iterations
               MemoryCost = 8192, // memory
               Lanes = 1,  // paralism
               Threads = Environment.ProcessorCount,
               Password = passwordBytes,
               Salt = salt,
           };
           using var argon2client = new Argon2(config);
           using var hash = argon2client.Hash();
           string hashstring = config.EncodeString(hash.Buffer);

           Span<byte> okm = new byte[32];
           Argon2id.DeriveKey(okm, passwordBytes, salt, 2, 8192);

            string geraltString = Convert.ToHexString(okm);
            string IsopohCryptographString = Convert.ToHexString(hash.Buffer);

           return Ok(new{ IsopohCryptographString, geraltString });
       }

If i call the function with password = "string" , the output is :
isopohCryptographString: "E6274D67634B68F62F8764F7DD011E34F9B208003B153AAD69E03185BAB963C3"  (correct)
geraltString: "295C6D6889530B62F28980FA79208C869E0835D4F3D632F008D08C5447169E63"  (false)
whats wrong with my code ?

Thanks for the code snippet and inputs/outputs. Isopoh.Cryptography.Argon2 interprets MemoryCost as the number of 1024-byte blocks, whereas Geralt does memorySize in bytes. Thus, memorySize = 8192 with Geralt is equivalent to MemoryCost = 8 in Isopoh. The strings then match.
However, if you can, I recommend using the Argon2id.ComputeHash() function for a string output.

Thank you . So Argon2id.computehash function is to encode the hash to argon2 output format ?
like $argon2id$....

Thank you . So Argon2id.computehash function is to encode the hash to argon2 output format ?
like $ argon2id $....

Correct, and Argon2id.VerifyHash() will verify such a string.


Issue with IR sensor/image

Camera Model: D415
Firmware Version: 5.10.13
Operating System & Version: MacOS 10.14.2
Platform: PC
SDK Version: 2.2.17
Language: python
Segment: Robot
Issue Description
I've had an issue where depth quality degrades periodically and it has become more frequent over time. My depth images have increasingly common 0 depth readings scattered randomly throughout the image and depth precision is reduced. I think I've narrowed the issue to the IR sensor. I've included a capture of the sensor in normal working condition and when it's misbehaving (in one image, the IR projector is on and the other it's off, but the overall image appearance is very similar when it's on or off in each state, with the exception of the pattern showing up). I haven't (knowingly) changed any settings when the issue manifests.
good:

bad:

EDIT: I just observed the quality drop mid-stream. All I'm doing right now is using wait_for_frames() to capture frames and doing post-processing on those frames to try to track down a different issue.
EDIT2: I've also just observed that the depth quality drop is not as well correlated with the IR image behavior as I thought. My IR image looks normal right now, but the depth quality is still poor.

Hi

From your Infrared1 camera image, I don't think such pattern noise is related to temperature. It looks some certain registers setting not correct, or sensor itself became defective now.
Do you see same issue on Infrared 2? (although in Y8 mode, such noise pattern should be visible).
If No, probably the Infrared1 sensor is defective now. If Yes, can you try other format like 640x360 ....? see same issue?
If Yes, can you try firmware update again? (can be one version earlier or newer).
And check camera again with RealSense Viewer. Run Infrared1 camera only under same format: RGB8 and 1280x720 30FPS. If the issue is still there, I guess this camera sensor is defective now.

Thanks

Hi
Is there any update for this issue?
Thanks

Unfortunately I haven't had a chance to try these things, I've moved on to other parts of the project with a workaround to identify and skip frames with bad depth data. I'll likely come back to this eventually, but you can close it for now if you want to keep things tidy.

Thank you for the status update. I'd appreciate if you can share the root cause with us if you get it fixed.
OK, I'm going to close it for now and you're always welcome to re-open it in the future.
Thanks and have a good weekend

i saw the same pattern noise on my D415, which I just received today. It looks like the noise is caused by the embedded IR LED. i.e., if I cover the internal LED and use an external IR to illuminate the object, the patter noise went away. Can anyone help on this? btw - I did upgraded my D415 to the latest firmware.

The pattern on your image is consistent with one produced by the camera's IR Emitter component.  I took a snapshot of myself to illustrate this.

The IR Emitter projects a pattern on semi-random dots onto objects in a scene.  This helps to add analyzable texture to low-texture objects such as doors and walls.  The emitter can have its Laser Power turned down to reduce the pattern's visibility or be turned off altogether.
The emitter can be altered with scripting.  If you are using the RealSense Viewer though, you can change the Laser Power setting or disbale / enable the emitter in the options side-panel
To turn the emitter off in the Viewer, left-click on the blue box beside the 'Emitter Enabled' option so the box turns to black, indicating that it is turned off.  The image then resembles the one you captured with external IR.

The Laser Power setting can be found in the Viewer by expanding open the 'Controls' sub-option in the 'Stereo Module' section of the side-panel.

@MartyG-RealSense thanks! appreciate.


add Cargo.lock to .gitignore

Description:
.gitignore generated by swc plugin new ... lacks Cargo.toml

BREAKING CHANGE:
none

Related issue (if exists):
none

I will never accept this PR.
SWC even uses cargo-crev

Btw, why are you trying this?

Ah, sorry. I misread it because there was no decription.
But I'll not accept this, as it should not be ignored.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


[MWPW-133515] [Template X]: Enhance/replace {{heading_placholder}} logic

Please always provide the JIRA issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix MWPW-133515
Description:
authors now have an option to define exactly how they want the sentence after the number of templates to be with an extra column in the metadata sheet called 'toolbar-heading'; If the column is empty for the row, the old placeholder from the placeholder will be used.
Test URLs:

Before: https://stage--express--adobecom.hlx.page/express/templates/?lighthouse=on
After: https://mwpw-133515--express--adobecom.hlx.page/express/templates/?lighthouse=on

Codecov Report

Merging #87 (cd2be81) into stage (8be2739) will decrease coverage by 0.29%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##            stage      #87      +/-   ##
==========================================
- Coverage   39.57%   39.29%   -0.29%
==========================================
  Files           9        9
  Lines        4986     4986
==========================================
- Hits         1973     1959      -14
- Misses       3013     3027      +14

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

pipeline is failing
please provide a test url where toolbar-heading is in metadata. I couldn't see it on the test page.

Done & done!

pipeline is failing
please provide a test url where toolbar-heading is in metadata. I couldn't see it on the test page.

Done & done!

nice!


Add --dep ninja, more fine-grained dependencies with --dep_ninja

I am currently building a standalone Windows binary package for EverParse (see project-everest/everparse@aa4b3c2e6e449612b3733ebcc34a783447de032e) containing F*, z3, KReMLin and EverParse, removing all other dependencies such as Cygwin, MinGW, OCaml, etc.
This means that the end user has no access to GNU make. However, F* code generated by EverParse needs a build system to verify and extract to KReMLin AST. So, that binary package also includes ninja (https://ninja-build.org/), which is a very fast, standalone, small subset of GNU make with different syntax for build files.
Thus, this pull request proposes to output the rules of dependencies in a format supported by ninja, via --dep ninja.
Moreover, I do not need to output everything: I only need to output dependencies for .krml files. Thus, this pull request adds an option, --dep_ninja, which takes as argument one of: checked, ml, cmx, krml, or the default all, to let the user choose which dependency output they need.
I claim that my patch also allows generalizing --dep_ninja to the GNU make format as well, but I have not done so yet, because I have not found out the right option structure and naming that we should expose to users.
Thank you in advance for your feedback.

We spent a lot of time carefully optimizing the time spent doing dependency analysis. Can you measure the impact of your patch on generating the .depend for miTLS on branch dev?
Thanks!

This pull request does not change anything in the dependency analysis itself, nor does it change the way dependencies are printed for --dep full. The only changes are to reuse the same dependency analysis as --dep full for Ninja and to print them slightly differently; those changes are strictly limited to the printing of each dependency rule.

This doesn't seem very invasive to me, so I'm fine with the changes modulo the tidbits I pointed out as single comments. @nikswamy may want to give this a quick cursory glance for the handling of options which I'm not super familiar with

If it is just reformatting of output, then in what way are the dependences more fine-grained?

@tahina-pro , is this PR still in play or we don't need it anymore?


create new networks beside sleepy bike

would be nice to create a new network inside the app

Thank you for opening this!
It is a great point, and it's on our TODO list. It's likely that there will be a separate app for this purpose. (new communities are managed a bit differently than personal profiles - a stable community member list should probably not belong to a specific user, but to an organization with a separate solid pod)
Another option we're considering is that you'll be able to create new communities, but they'll have to have their own website. E.g. sleepy.bike will be a community for cyclists, without an option to add new communities. Showing other communities could be optional...

This is duplicate of #6


Usage in Node.js

is there a way to use this in Node.js without reading the whole file like in fast-exif?

Actually the code is compatible with Node.js 20 when certain requirements those includes
when XMP segments are present.

Actually it works in Node.js when certain requirements are met, I just added some examples, see examples-nodejs/.
Concerning reading file fully or progressively into memory:
For JPEG format, blob/file is read progressively. 64KB data is initially read (Exif metadata are restricted in size to 64 kB in JPEG images), if necessary more and more data will be read until 2MB.
For other formats: progressive reading is planned (with initial buffer size = 64KB, buffer increasing multiplier = 2), it may not necessaryly gain any benifits except instant memory usage, I'll try it later.

Thanks for adding an example! My specific case is parsing AVIF metadata. Seems like there's no other project doing that


Context for weights

Allowing setting contexts for weights eg. server=bungee world=skyblock etc.

This might be possible using the web editor.

if the weight permission has a context it will get ignored completely as of now

Usecase?

Per server 'ranks', eg. skyblock & prison have different 'ranks'
Would make it possible for eg. BungeeTabListPlus to sort them correctly. Instead of having none-ranked with ranked players at the top. (because they have a rank with a high weight on the other server)
Aswell as displaying the right primary group on the right server

@lucko it would be awesome solution for BungeeTabListPlus, which now makes huge problems with displaying players because of global weights. Could you reconsider adding this feature?
@Vankka Have you somehow managed to get rid of this BungeeTabList problem?

BTLP has many more options for sorting which sure would be useful for you.

They are not in case described by Vankaa

That if somebody has rank with ex. 100 priority on one server, even if he does not have permissions on another - he would have very high priority

@Kryniowesegryderiusz, no I never did. This suggestion is still valid

@Andre601 Is it hard to implement that? Is that why are you avoiding this?

Yeah, I'm nervous too about that. I can't set ranks :/

Same here, I would also like to see that feature added

Unlikely to happen, the contexts system wasn't ever designed to support this.
I'm still toying with the idea of implementing node priority, which would cover this.


Is moment required?

Using version 21.5.1. I'm having a hard time telling. I found several issues and a merged PR related to removing moment, but AFAIK from the docs and TS compiler messages I'm getting, the input and returned values are of type moment.Moment. What's the lowdown?
edit: I think it was react-datepicker that excisised moment.

This package uses moment, and is likely to until Temporal is stage 3 and a polyfill/shim is thus viable in production. Moment is the defacto standard, and once Temporal arrives, nothing else will serve a purpose, so the churn in switching to any alternative (or supporting multiples) doesn't seem worth it in the interim.
Closing as a philosophical duplicate of #208.


Replace react-markdown with Markdoc

Currently we are using react-markdown and hitting some limitations that could be fixed if we used Markdoc instead.

The reasoning is we want to have custom nodes to allow us to achieve #30 and #31 so if anyone has better suggestions I would love to hear them. 🦾

Working on a list of changes we need to make in application code to accomplish this.
will share the update in the issue thread soon.


refactor: Remove validator-import container from testdata

Changes:

Remove validator-import from the testdata files

Testing
Requires testing

[x] No

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 88.06%. Comparing base (000cc79) to head (a012e95).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #412   +/-   ##
========================================
  Coverage    88.06%   88.06%
========================================
  Files           99       99
  Lines         5673     5673
========================================
  Hits          4996     4996
  Misses         553      553
  Partials       124      124

Flag
Coverage Δ

unittests
88.06% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


[ASK] Are video thumbnails tracks supported?

Are video tracks with thumbnails supported?
tracks: [{
                file: "https://cdn.jwplayer.com/strips/xzxKu91o-120.vtt",
                label: 'track1',
                default: true,
                kind: "thumbnails",
              }].

Web seems to support such feature when user moves the current position of timeslider.

Hi @fdobre, yes you can use thumbnail tracks. See the below config to be used in our Example app.
This should also work in legacy configs on both platforms.
  let jwConfig = {
    "title": "Single Inline Linear Preroll",
    "playlist": [
      {
        "file": "https://content.bitsontherun.com/videos/q1fx20VZ-52qL9xLP.mp4",
        "tracks": [
          {
            file: "https://cdn.jwplayer.com/strips/xzxKu91o-120.vtt",
            label: 'track1',
            default: true,
            kind: "thumbnails",
          }
        ]
      }
    ]
  }

@Jmilham21 Tested, it works great!


incorrect boundary used for multipart-form requests

Since RapiDoc 7.5.0, requests generated using Content-Type: multipart/form-data contain an incorrect boundary header. Because of this incorrect header value, all multipart/form-data requests will fail on probably any backend.
Steps to reproduce this issue:

have a swagger YML file containing a multipart/form-data element (see https://github.com/skwakman/rapidoc-multipart-issue/blob/master/test.yml )
have a html file loading rapidoc version 7.5.0 or higher (see https://github.com/skwakman/rapidoc-multipart-issue/blob/master/api-test.html )
load up a local webserver, for example using python's python http.server
open up your browser's dev tools and open the 'network' tab
open up the api-test.html, fill the fields and click 'try'

Find the sent request in the network tab. The request's content-type header will be multipart/form-data; charset=utf-8; boundary=RAPIDOC. However, looking at the request's body, the actual boundary looks like a randomly generated string created by the browser, such as -----------------------------3779300221716527953402675240.
The request generated by rapidoc sets the boundary value to RAPIDOC, but does not actually use this boundary for its request body. This behaviour was introduced since 7.5.0, version 7.4.3 does not seem to have this issue.

thanks for reporting,
we were testing to support some pipeline integrations where we needed some predictable form boundaries, seems like it somehow mistakenly got merged. will remove it in our next patch, which will be coming soon


#172 Support for image extra options

-
-

Issue
https://github.com/ripe-tech/ripe-design/issues/172

Dependencies
https://github.com/ripe-tech/ripe-sdk/pull/213

Decisions
• Added prop extraOptions that let the user pass ripe's image options such as mirror, flip, rotation, etc • Added documentation based on https://docs.platforme.com/#render-endpoints-compose and https://github.com/ripe-tech/plugin-base/blob/master/templates/config/applier.html.tpl

Screenshot
Mirrored and rotated ripe-image example:Example of test toggling mirror and changing rotation every 500ms:Updated documentation:

I understand the reasoning but this approach is not an option. If we started size/crop/format as separate props then this must use the same strategy.

Just to make sure, add all props with comments and examples right?

I understand the reasoning but this approach is not an option. If we started size/crop/format as separate props then this must use the same strategy.

Just to make sure, add all props with comments and examples right?

Comments are enough I guess. What kind of examples were you thinking of? Should be added to the storybook as well.

I understand the reasoning but this approach is not an option. If we started size/crop/format as separate props then this must use the same strategy.

Just to make sure, add all props with comments and examples right?

Comments are enough I guess. What kind of examples were you thinking of? Should be added to the storybook as well.

I was talking about examples in the comments, like we do in other components.
I asked because some options, like bounding_box are lists or other types of structures, a little more abstract so I think that atleast those should have an example in the comments on how to use them.

I understand the reasoning but this approach is not an option. If we started size/crop/format as separate props then this must use the same strategy.

Just to make sure, add all props with comments and examples right?

Comments are enough I guess. What kind of examples were you thinking of? Should be added to the storybook as well.

I was talking about examples in the comments, like we do in other components.
I asked because some options, like bounding_box are lists or other types of structures, a little more abstract so I think that atleast those should have an example in the comments on how to use them.

Agree. It makes sense for those, no doubt.

This is waiting on docs.


Still accepting entries / PR?

Seems like inactive repo // cc @agarrharr

It's still active, I've just been a little busy IRL. I could use some help with reviewing PRs and issues though.

Well, thing is I wanna add my PR to the project, but I was a bit hesitant with the other open ones. If I were to do that, would cstate/cstate be a good contender? :)

So, it looks like cstate is a static site that shows the status of any website correct? This probably isn't a good fit for this list because it focuses on services specifically for static sites.

it’s a tool like any other. (especially if you take into account that stuff like soundcloud is on the list.)
should i make a PR or move on? 🤔

True. This list focuses more on things that you can embed on the page to extend the functionality of the site. I think the only exception is "Functions as a Service", but that still extends the functionality. It's just a bit more custom.
It might be better on this list: https://github.com/lvwzhen/tools

By the way, it does look awesome! I just want to limit the scope of this list.

Thanks for the alternative and thanks for your time! ♥


fix: dialog window bug

Description
fixes:
https://www.notion.so/oaknationalacademy/Feedback-Form-Sizing-10426cc4e1b18022addddbdf27ec49f9?pvs=23

:tada: This PR is included in version 1.9.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


have to manually provide registry secret in 2.0

Stood up a fresh 2.0 beta environment yesterday. Have a private docker registry (internal Gitlab). Images fail to automatically pull correct credentials.
4 hosts, all on the same subnet and all running near bare-bones Xenial installs. 1 is rancher, 1 is k8s etcd/control, and 2 are workers. RKE. 1.8.10-rancher1. Docker 17.03.2-ce.
Created a Registry entry in UI open to all namespaces in the default project.
kubectl create -f pod.yml that uses several containers from the private registry.
Get 403 errors trying to pull images.
Verified that docker login with same credentials from command line of worker node in question functions correctly.
After trying several different users, passwords, and tokens, eventually explicitly added the created secret as a imagePullSecret to the yml allowed the containers to pull.

Useful
Info

Versions
Rancher v2.0.0-beta2 UI: v2.0.34

Access
`` admin

Route
authenticated.project.dns.index

@allaryin Hi, since you're using kubectl directly - pods will get created without imagePullSecrets set. If you do it via Rancher, like creating a workload with those images, we'll fetch matching secrets from registries and set imagePullSecrets for you. Hope that helps!

I had created this ticket on the recommendation of one of the developers in Slack, and I had brought it up as an issue because this is a different behavior from 1.6 - where creating a pod via kubectl does automatically discover the pull secrets that you configure the registry in the UI.


refactor: lake: simplify load code

Simplifies the Lake dependency resolution code. Largely split from #3998.

!bench

!bench


Added some addons to the index

you are right, all addons are small, but then there would be no addon if we remove small addons... Only staffchat would remain.. Also staffchat can work alone so i personally don't consider it an addon. Feel free to correct if im wrong :)

You're not going to get this PR merged by trying to redefine addon. And also, DiscordSRV Staff Chat can't run on its own. It is an addon. It adds on functionality.

I never tried to redefine the meaning... Anyways i don't really care about it getting merged.. Its just a PR with a few lines to docs

Give me 1 minute

https://github.com/DiscordSRV/Staff-Chat/blob/master/src/main/java/com/rezzedup/discordsrv/staffchat/StaffChatPlugin.java#L124

I never heard the word "Addon" defines same developers...

It doesn't on its own, but if we list "addons" on our wiki then there will be the expectation that they are managed by us.

you are right, all addons are small, but then there would be no addon if we remove small addons... Only staffchat would remain.. Also staffchat can work alone so i personally don't consider it an addon. Feel free to correct if im wrong :)

If we add small and unknown plugins to our list then it will only clutter it up. We would rather have a small list of reliable and well known plugins.
StaffChat does work on its own, however it is intended to be used with DiscordSRV. This will be the same for the alerts module when it is released as a standalone project.

Ok, do i remove all except interactive chat? This one is known

I can get them all? Or there is an exception to the ones developed by you?

Alright 👍 (but btw you forgot about the note granny left about if someone wants his addon on list)


Crashing IE 9

It seems the latest version of the library is crashing the IE 9 because of "site is not responding due to a long-running script.".
I checked older versions as the demo site seems to work and the latest version without having the issue occurring for me was:
https://github.com/glinford/ellipsis.js/tree/bd192fa62cfff1a3eb40d72b31ae2285a61ab87d
Also when trying to pause the code in the debugger of the developer tools right before the browser crashes it showed the same line loading that has been added after the code started to crash:
https://github.com/glinford/ellipsis.js/commit/c6363e44cd232e64cc4daa310d0b36dcab7537f4
Let me know if you need any additional information.

I ran into an issue with Chrome becoming indefinitely unresponsive (just the tab it's running in). I'm guessing it's related. Since I don't need a generalized solution, I'm rolling my own.

@glinford any idea on merging this?


implement call to let raft know that a follower's db has been updated

Author Name: Jaime Melis (@jmelis)
Original Redmine Issue: 5363, https://dev.opennebula.org/issues/5363
Original Date: 2017-09-13

If a follower is offline for a very long time, the only recovery possible is to manually copy DB to the follower. In this case it's also necessary to inform the leader that the follower's DB is up to date, otherwise it will send the records since the last time the follower was online, which is unnecessary and can be problematic because if the record is too old, it might be purged and not available.

Original Redmine Comment
Author Name: Jaime Melis (@jmelis)
Original Date: 2017-09-13T10:41:52Z

A possible workaround is to simply delete the server from the zone and add it again. The server_id may change, in that case in must be reflected in oned.conf


bcc: update prog_load to new name (introduced in iovisor/bcc@e60438d)

Without this, fails to compile with current bcc master

I cherry-picked your PR into https://github.com/iovisor/gobpf/pull/175 which has other updates for bcc v0.9.0 and will merge as part of that. Thanks again!


updating dev tunnels initial run screenshot

The image that is shown when first running a dev tunnel has been updated.
In this version of the image, I blurred out the URL. There is a chance that this URL may be used later in the future so it's best to just blur it out.
cc @curib

FYI @tdykstra @Rick-Anderson I updated the screenshot in the dev tunnels doc. I reviewed the preview URL and it looks good to me.


When a file or folder with the same name exists in the documentRootPath directory, there is a problem with the link of the generated result.

Thank you for developing this plugin. I found an issue while using it:
The directory structure is as follows:
├───docs
│   │   api-examples.md
│   │   markdown-examples.md
│   │   website.md
│   │
│   ├───.vitepress
│   │   │   config.mts
│   │   ├───theme
│   │   │       index.ts
│   │
│   ├───docs
│   │       index.md
│   │       VitePress.md

The configuration is as follows:
const vitePressSidebarOptions = {
	documentRootPath: '/docs',
	collapsed: false,
	capitalizeFirst: true,
	debugPrint: true,
	useTitleFromFrontmatter: true,
	excludeFilesByFrontmatterFieldName: "exclude",
	useFolderLinkFromIndexFile: true,
};

The generated structure is:
[
	{
		"text": "Api-examples",
		"link": "/api-examples"
	},
	{
		"text": "Docs",
		"link": "/docs/index.md",
		"items": [
			{
				"text": "VitePress",
				"link": "/VitePress"
			}
		],
	},
	{
		"text": "Markdown-examples",
		"link": "/markdown-examples"
	}
]

I traced the issue and found it in the following code:

Should the logic in this part of the code be modified?

Hello! Thank you for using vitepress-sidebar.
If I understand correctly, the path /docs in the first step does not show up in the sidebar results? Normally, the path in documentRootPath is treated as the root path of the document and does not show up in the sidebar results.
If not, can you be a little more specific about what you're seeing, and what part of the code you've shown is believed to be wrong?

Hello, thank you for using vitepress-sidebar.
I just checked the issue again after I commented and realized your issue.
I'll comment again once I've fixed it, thank you.

Hello! Thank you for your patience.
Your case was one that I hadn't considered, which is why I was slow to spot the issue. I'll be sure to look for it in more situations in the future. Thank you for reporting the issue!
This issue has been fixed and released in version 1.29.1. Could you please upgrade to that version and check if the issue is resolved?
Regards,

I updated the version and verified that the problem has been solved, thank you!
I apologize for the distress caused by this as the description I provided earlier may not have been clear enough.

No problem, the title and screenshot gave me a quick clue :)
I'm closing this issue, but if you have any new problems in the future, feel free to create a new issue!
Regards,


Fixed bug with setting CLI options from a config file

Overview: What does this pull request change?
format was never included in the str keys when digesting the config file.
output_name was always overwritten.
These two behaviours have now been fixed.

Further Information and Comments

Do you want me to add tests for this?

Reviewer Checklist

[ ] The PR title is descriptive enough for the changelog, and the PR is labeled correctly
[ ] If applicable: newly added non-private functions and classes have a docstring including a short summary and a PARAMETERS section
[ ] If applicable: newly added functions and classes are tested

I didn't bother running pytest because of course there is no way I broke something.
Well well...

This is probably not worth the effort, format and file extensions are deeply intertwined and I think there are already ways to do it with --format={gif,mp4,mov,webm} through the command-line.