Fix: ConnectionManager gets disposed 3 times.

Tiny PR that prevents 2 unnecessary ConnectionManager.Dispose() calls.
I've also checked all the classes that inherit IDisposable if they dispose everything correctly. Only FullNode has a problem- it also gets disposed 3 times (2 of them are not intentional, caused by service provider automatically registering disposable objects), but the workaround already included in FullNode.Dispose() implementation, so it's fine.
Fix #691

When does the DI dispose of the connection manager?
Multi dispose calls is not such a big problem as long as the pattern makes sure its immutable (i.e. if disposed will not dispose again)

@dangershony i am not sure i agree. on one hand, sure, it can be implemented in a way that there is no actual problem in calling the dispose many times, but it feels very wrong principally - it seems that if we dispose things many times, we don't really control those components and they are being randomly disposed somewhere, maybe one time, maybe many times, maybe 0 times ...
so conceptually i think it is a problem despite the fact that it causes no actual damage

@noescape00 as this PR does not resolve the 3x FN dispose, do we have that information that FN is disposed 3 times somewhere? like in a separated issue

@noescape00 as this PR does not resolve the 3x FN dispose, do we have that information that FN is disposed 3 times somewhere? like in a separated issue

No, I only wrote it here and in slack.
I didn't create a separate issue because it looks like writer of FN.Dispose() knew that and specifically made a workaround.
Getting rid of that workaround would force us to include and change service provider's source code which is probably an overkill.

can you pls explain little bit more, i'm not sure i understand
what kind of workaround?
pls show more details about why do you think it is hard to dispose FN just once

please continue in #799

I would remove the IDisposable from ConnectionManager and call it explicitly on node shutdown.
I would also not add the LoggerFactory to the disposable list and explicitly dispose of it.

We have advanced much further in this conversation in #799
I would thus rather suggest to abandon this PR (not merge it) and deal with what we discuss in #799, then we can resolve #691 as a part of that big thing.

everyone happy to not-merge this PR?

Yep, I'm fine with it.


Do not require compat_resource

require 12.5 as a minimum version
docker --> dokken

So this one actually requires compat_resource for yum_repository that was introduced much later. If we drop compat_resource here we're dropping support for fairly recent chef releases.


#130: add by missing content length a default value to progress bar

Closes #130

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 59.031%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4249

Relevant Lines:
6922

💛  - Coveralls


Synthetic data size used for training

Can I know the size of synthetic data used for training huggingface model?
(https://huggingface.co/kha-white/manga-ocr-base)
Thank you for your awesome project.

5 million synthetic and 100k real images

5 million synthetic and 100k real images

Thany you :)


feat(deps): upgrade runtime dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-runtime-dependencies-1.x" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 1560

Questions ?
Please refer to the Backport tool documentation


Apply splitting to Azure Monitor metrics query with Azure.Monitor.Query

Library name and version
Azure.Monitor.Query 1.1.0
Query/Question
Is it possible to do splitting with Azure.Monitor.Query SDK when retrieving Monitor metrics?
I'm trying to retrieve metrics for a service bus account, split by EntityName. This is possible in the portal:

This is the code that I have. I can retrieve the metrics but don't know how to apply the splitting.
var credential = new ClientSecretCredential(_tenantId, _clientId, _clientSecret);
var metricsClient = new MetricsQueryClient(credential);
var resourceId = $"/subscriptions/{_subscriptionId}/resourceGroups/{_resourceGroupName}/providers/Microsoft.ServiceBus/namespaces/{_serviceBusName}";
var metrics = new[] { "ScheduledMessages", "ActiveMessages" };

var options = new MetricsQueryOptions();
options.Aggregations.Add(MetricAggregationType.Total);
options.TimeRange = TimeSpan.FromHours(4);

var result = await metricsClient.QueryResourceAsync(resourceId, metrics, options);

Environment
IDE: Visual Studio 17.5.3.
dotnet --info results:
.NET SDK:
 Version:   7.0.202
 Commit:    6c74320bc3

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.19045
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\Program Files\dotnet\sdk\7.0.202\

Host:
  Version:      7.0.5
  Architecture: x64
  Commit:       8042d61b17

I am interested in the same thing, is there any update?

Good point this should be better documented. Splitting and filtering actually both use the same filter parameter the only difference is splitting is where the filter value is equal to ''.
So an example of splitting is:
$filter=ApiName eq ''
An example of filtering is:
$filter=ApiName eq 'GetBlob'

I'll get this documented in our README file


Fix Fix decimal type check for ColumnValueRange issue and use primitive_type.

Proposed changes
Fix decimal type check for ColumnValueRange issue and use primitive_type in orc_reader. Because in #22842 the CppType of PrimitiveTypeTraits<TYPE_DECIMALXXX>  were changed.
Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall

(From new machine)TeamCity pipeline, clickbench performance test result:
the sum of best hot time: 44.96 seconds
stream load tsv:          544 seconds loaded<PHONE_NUMBER>9 Bytes, about 131 MB/s
stream load json:         20 seconds loaded<PHONE_NUMBER> Bytes, about 112 MB/s
stream load orc:          65 seconds loaded<PHONE_NUMBER> Bytes, about 16 MB/s
stream load parquet:          31 seconds loaded 861443392 Bytes, about 26 MB/s
insert into select:          30.1 seconds inserted 10000000 Rows, about 332K ops/s
storage size:<PHONE_NUMBER>2 Bytes

run buildall

run p0

run clickbench

run p0

run clickbench


Fix wrong error message being displayed for invalid descriptions

this is related to #155

Codecov Report

Merging #175 (f3a9d65) into master (21d940f) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##             master     #175   +/-   ##
=========================================
  Coverage     62.93%   62.93%
  Complexity      590      590
=========================================
  Files           111      111
  Lines          2118     2118
  Branches        231      231
=========================================
  Hits           1333     1333
  Misses          708      708
  Partials         77       77

Impacted Files
Coverage Δ

...in/java/seedu/address/model/group/Description.java
70.00% <ø> (ø)

...rc/main/java/seedu/address/model/ModelManager.java
72.00% <100.00%> (ø)

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 aac570c...f3a9d65. Read the comment docs.


release August

@matentzn and @twhetzel
I am seeing something very weird...
Canine sarcoma (MONDO:0700149) in mondo-edit.obo has a SubClassOf annotation (asserted) to "canine neoplasm" (MONDO:0700139) with a source (NCIT:C128195)
This annotation is not in the mondo.owl
I don't know why. I cannot think of what is redundant, and also, this subclass of is therefore missing in mondo.owl

UPDATE:
I did not see "canine sarcoma" in the "canine neoplasm" branch, but it is in it!
Therefore all is well.
We should talk about the new report reports/difference_release_base.md on a call because the "Relationships removed" section is confusing.


[Pomodoro] Add auto start feature

Extension
https://www.raycast.com/asubbotin/pomodoro
Description
Please add a tick mark or something to auto-start the next mode.
For example breaks should start automatically when the focus mode ends and focus mode should start automatically when the break time ends. Give the user option to enable this feature.
Ofcourse the notification should be shown in raycast window that next mode has started.
Who will benefit from this feature?
Anyone use this app.
Anything else?
No response

@shahnawaz323 I was also thinking about this. Right now we have to start the break ourselves which is less than ideal. I think having an option to enable auto-start along with an option to set the cycle count would be awesome. The cycle count can be used to start the long break.
Let me see if I can come up with something.


Fix the link to modeling exercise statistics

Checklist

[x] I tested all changes and all related features with different users (student, tutor, editor, instructor, admin) on the test server https://artemistest.ase.in.tum.de.

Motivation and Context
The instructor actions have a link to the statistics, which doesn't work right now:

Description
We correct the link to point to the correct page.
Steps for Testing

Create or find a modeling exercise
Go to the student view with an instructor account
Click on "statistics"

Review Progress

Code Review

[x] Review 1
[ ] Review 2

Manual Tests

[x] Test 1
[x] Test 2

I see, the component the old route uses doesn't do anything but throw a 404 at the moment, see https://github.com/ls1intum/Artemis/blob/5af5f40306fa126cca6d761dc8eb42c4ae940300/src/main/java/de/tum/in/www1/artemis/web/rest/ModelingExerciseResource.java#L241-L246 which was done by https://github.com/ls1intum/Artemis/commit/fa90f52fcdfd159faffb42a9abd7b7f82b571561#diff-add7a69d47dc03e894f18150f009b3b1a9290fc48ec484ee21b45c08f7054d24L237
Tbh I don't know what to do here.

Tbh I don't know what to do here.

@krusche told me to remove all the related code since they are no longer used/required.

Codacy made a remark about an unused import, so I removed that and the now unused model as well.


[FOOD-87][FOOD-89] Add default cog url

Possible solution to have a fixed cog url.

I just added back the possibility to put a cog url as an optional parameter. Maybe it could be handy to keep this functionality only when the tiler is in a debugging/dev environment so the FE and science can make tests with experimental datasets :thinking: (just thinking out loud)


Add Edit Module Command

Added EditModuleCommand class and EditModuleCommandParser to handle the editing of modules.
Added a method replaceModule(Module target, Module editedModule) to the Model interface, ModelManager class, AddressBook class and DistinctModuleList class to replace the target module with the editedModule.
Added a method updateFilteredModuleList(Predicate<Module> predicate) in the Model interface and ModelManager class to filter out modules in the filtered module list.

Codecov Report

Merging #64 (bdbb0c1) into master (8c683b0) will decrease coverage by 1.64%.
The diff coverage is 1.29%.

@@             Coverage Diff              @@
##             master      #64      +/-   ##
============================================
- Coverage     43.78%   42.13%   -1.65%
  Complexity      369      369
============================================
  Files           116      118       +2
  Lines          1914     1991      +77
  Branches        213      225      +12
============================================
+ Hits            838      839       +1
- Misses         1033     1109      +76
  Partials         43       43

Impacted Files
Coverage Δ

...main/java/seedu/address/commons/core/Messages.java
0.00% <ø> (ø)

...eedu/address/logic/commands/EditModuleCommand.java
0.00% <0.00%> (ø)

.../seedu/address/logic/parser/AddressBookParser.java
51.61% <0.00%> (-1.73%)
:arrow_down:

.../address/logic/parser/EditModuleCommandParser.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/model/AddressBook.java
42.85% <0.00%> (-2.43%)
:arrow_down:

...rc/main/java/seedu/address/model/ModelManager.java
46.57% <0.00%> (-4.18%)
:arrow_down:

...seedu/address/model/module/DistinctModuleList.java
25.80% <0.00%> (-8.98%)
:arrow_down:

...c/main/java/seedu/address/model/module/Module.java
0.00% <0.00%> (ø)

src/main/java/seedu/address/model/Model.java
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


Hanging on startup?

I've noticed since upgrading from 0.7.7 to 0.7.9 that Clojure Extras seems to hang IntelliJ on startup.
I'm not sure what debugging information I can provide to help further.

It hangs without any project open? Maybe you could create a repro repo? that would help


Fix updated on

Coverage remained the same at 81.93% when pulling 2847bd22828c58af5a088726cc1c7a182d5d804e on iksaif:fix-updated-on into c0a490cb80bb0019fdd0f8edd9ee4a96b2481db6 on criteo:master.


Add logical_disk latency metrics

This PR adds the metrics created in #329 with corrected metric names.

Thanks a lot! :tada:


spark: get column-level lineage from JDBC dbtable option

There were several problems with JDBC column level lineage.
First, it did not support dbtable option - which ment that one-to-one relationships weren't supported by column lineage collector.
Second, the JdbcColumnLineageCollector did not report the lineage when there was only one single input column.
Third, it used wrong, naive dataset name straight from the parser results, rather than fixed one from JdbcUtils.
Those changes fix that.

@mobuchowski Some tests are still failing ColumnLineageIntegrationTest > columnLevelLineageSingleDestinationTest for 3.2 and 3.4

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

Comparison is base (0be9239) 81.41% compared to head (d8795ea) 81.41%.
Report is 10 commits behind head on main.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff            @@
##               main    #2284   +/-   ##
=========================================
  Coverage     81.41%   81.41%
  Complexity      125      125
=========================================
  Files            90       90
  Lines          3804     3804
  Branches         33       33
=========================================
  Hits           3097     3097
  Misses          668      668
  Partials         39       39

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


Fix missing protobuf and pkg-config deps in Dockerfile

Fix Dockerfile and add Protobuf as dependency in README.md file

Thanks, I forgot about docker. I checked it and tested it. It works and I will update the image on dockerhub after I merge.


Cherrypick anti pull #1617 to release

Cherrypick anti pull #1617 to release. This should be updated without pushing nightly to release. Nightly is in progress anyways and not ready to be merged yet.

Fix issue of debuggers, pentesting tools, script analysers and exploit simulators kicking people in Studio. (#1617)

Fix studio dumb

Sceleratis

Fix typo

Fix conditions

Remove unnecessary warning

Remove unnecessary warn

Why

Why

Fixes annoyance in studio if people use debuggers, pentesting tools, script analysers and exploit simulators that people make issue tickets about.

This can probably just be included when v261 drops

Closed as no longer necessary


Fix description of add assignment command

fixes #165

Codecov Report

Merging #263 (6b9452f) into master (0810a46) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##             master     #263   +/-   ##
=========================================
  Coverage     75.92%   75.92%
  Complexity      683      683
=========================================
  Files            94       94
  Lines          2002     2002
  Branches        226      226
=========================================
  Hits           1520     1520
  Misses          415      415
  Partials         67       67

Impacted Files
Coverage Δ

...u/address/logic/commands/AddAssignmentCommand.java
88.88% <100.00%> (ø)

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 0810a46...6b9452f. Read the comment docs.


custom pre ATT dialog

Feature Description
Any chance we can
A. customize the Applovin pre ATT dialog?
or
B. disable it's use so that we can use our own?
I had already created my own that matched the design of my app, but disabled it once I installed AppLovin.
Same request for the Swift version of the plugin. I can make the request there too if needed.
Thanks!!

Hey - we're actually deprecating that flow completely and will be removed in a future SDK Release. You can disable it by reversing the steps you performed to enabled it.

Super! So with that reversal, the Applovin SDK will just read / respect the ATT selection after rolling our own?

Correct - AppLovin and all mediated SDKs you have integrated will automatically read the latest ATT value from the OS.


Service names - Dynamic DNS registration to Windows DNS Server?  Any value?

So, Ive been writing a python script that I will convert to go (at least I will when I get time) that watches the API server and any adds/deletes will trigger a script that updates a Windows 2016 DNS Server entry for the service.   I have debating adding other backends, but I'm not there yet.
Two questions for people - am I the only person who sees value to this?   and next question - Am I re-inventing the wheel?   Does something exist today already?  (Non Cloud - so we cant use the nice easy loadbalanced option)
What I am accomplishing in my hacked together script now - Whats the API (I use Calico in Routed mode) and the name of the service is added to the DNS zone identified in the script.  Right now it all goes to the root domain name for my AD - I may add other options as I run into a requirement, but Im not going to go the route that internal Kube names are created for kube-dns.   I may decide to take a label for the DNS entry since I have already run into way too may names I would rather have for DNS name over the service name already :)
In the end, I would foresee a simple docker container with the script that watches the API with some passed parameters to the Windows DNS Server. (Or other backend, if expanded)

Hi @robotica72, take a look at https://github.com/kubernetes-incubator/external-dns, it doesn't have support for Windows DNS Server, but I'm sure the community would find your work valuable, at the very least you can get some inspiration on how this project is watching the k8s resources.

Hey Dimitri - good idea, I knew of that project but saw it didn't support Windows DNS so I went off to do it myself - but never considered contributing to the project or even mentioning it to the team...  Might be about time to contribute :)

Closing this. Feel free to reopen if you disagree.


Items sort not correct on deploy

When i deploy my new website on vapid hosting, the sortable items in sections don't reflect the development order; so every time i do a new deploy, i have to reorder items with drag and drop every time.
Any tips?
Thank you very much!
EDIT: it seems that reordering items it's not working properly, when i drag and drop items, the one that i drag is ordered correctly, but the others are always jumping in a different order...

@Jack89ita This should be fixed in the most recent release, v0.5.1.
As a reminder, the NPM package name has changes from vapid-cli to @vapid/cli. To update:
npm uninstall -g vapid-cli
npm install -g @vapid/cli


tag in  is not handled properly on SSR

Describe the bug
/** @jsx Nano.h */
/** @jsxFrag Nano.Fragment */
/// <reference no-default-lib="true" />
/// <reference lib="dom" />
/// <reference lib="dom.iterable" />
/// <reference lib="deno.ns" />
/// <reference lib="deno.unstable" />

import * as Nano from<EMAIL_ADDRESS>
const App = () => (
  <>
    <Nano.Helmet>

      <meta charset="utf-8" />
      <title>Title</title>
      <meta property="og:type" content="website" />
    </Nano.Helmet>

Hello, world!

  </>
);
const { body, head } = Nano.Helmet.SSR(Nano.renderSSR(() => <App />));
console.log(head);  // ['<meta charset="utf-8" /><title>Title</title><meta conte...']
console.log(body);  //
Hello, world!

Nano.Helmet#didMount handles  tag, so it won't work for SSR, resulting to emit incorrect  tag in head property of the returning object of Nano.Helmet.SSR().
To prevent this, Nano JSX should at least delete  tag or raise an error on Nano.Helmet.SSR(). It would be better if it returns htmlAttributes and bodyAttributes like react-helmet does.

This is just a notice since your idea might be good and correct me if I'm wrong.
This is a notice from InertiaJS Title & meta:

Since JavaScript apps are rendered within the document , they are unable to render markup to the document , as it's outside of their scope. To help with this, Inertia ships with an  component, which can be used to set the page ,  tags, and other  elements.

Same goes for Nano JSX <Helmet> as far as I understand, it does not like React Helmet yet

Support attributes for html

If you check the template makeHTML, you see     ${head.join('\n')} is just additionally added/rendered to the html markup in side the . So  is out of scope for Helmet.

As of now Nano.Helmet.SSR returns { body, head, footer }.
I guess it should return { body, head, footer, html }, where html would be  (open tag only).
@aiotter What do you think?

@yandeu { prefix: 'og: https://ogp.me/ns#', lang: 'en' } seems nice!
How about implementing toString() method to this object, which returns prefix="og: https://ogp.me/ns#'" lang="en"? This will make people easier to embed the attributes into their HTML string. React-helmet has the similar one, helmet.htmlAttributes.toString().
FYR: https://github.com/nfl/react-helmet#as-string-output.
I afraid html attribute becoming different from that of body then. They can also set the attribute of body tag with Nano.Helmet, and it has the same problem. How should it be?

btw, I feel it better to have a option to return body and head as components instead of string.
I'm coding like following, but I feel it a bit too roundabout.
  const app = Nano.renderSSR(() => <App />);
  const { body, head } = Nano.Helmet.SSR(app);
  const html = Nano.renderSSR(() => (

  ));

react-helmet has nice way to accomplish this, say:
function HTML () {
    const htmlAttrs = helmet.htmlAttributes.toComponent();
    const bodyAttrs = helmet.bodyAttributes.toComponent();

    return (

                {helmet.title.toComponent()}
                {helmet.meta.toComponent()}
                {helmet.link.toComponent()}

                    // React stuff here

    );
}

If you feel this possible, I'll create a new issue for it.

The body returned from Nano.Helmet.SSR() does NOT include the body attributes.
This looks right:
const { body, head, footer, attributes: { html, body } } = Helmet.SSR(app)

What about this:
improve-helmet branch; test with npm run ssr
class App extends Component {
  render() {
    return (

Nano JSX App

        <Helmet>

          <title>My Title</title>

          <meta name="description" content="Nano-JSX application" />
        </Helmet>

    )
  }
}

const app = renderSSR(<App />)
const { attributes, head } = Helmet.SSR(app)

console.log(head) // <title>My Title</title><meta content="Nano-JSX application" name="description" />
console.log(attributes.body) // [Map] { 'class' => 'main', 'id' => 'id' }
console.log(attributes.html) // [Map] { 'lang' => 'en', 'amp' => 'true' }
console.log(attributes.body.toString()) // class="main" id="id"
console.log(attributes.html.toString()) // lang="en" amp="true"

Looks really nice! That makes us code like this:
const Html = () => (

        <App />

);

Or even simpler:
const app = renderSSR(<App />)
const { body, head, footer, attributes } = Helmet.SSR(app)

const Html = () => {
  return `
    <!DOCTYPE html>

        <meta charset="UTF-8">
        <meta name="viewport" content="width=device-width, initial-scale=1.0">
        ${head.join('\n')}

      >
        ${body}
        ${footer.join('\n')}

    `
}

@yandeu Thank you for implementing this!
I cannot wait to test it on my deno project; can you run denoify to update the deno_lib? So that I can test it before new release!

I just released v0.0.28


Add graph document operations to benchmark

Operations to add in benchmark:
-> Construction: add_single_node, add_nodes, add_single_edge, add_edges .
-> Getters: Features, num_nodes, num_edges, etc ...

To wait for https://github.com/jina-ai/jina/pull/3259 to be merged


Use Set-AzureADDirectorySetting

[Enter feedback here]
These are correct set of instructions:
$setting = (Get-AzureADDirectorySetting | where -Property DisplayName -Value "Group.Unified" -EQ)
$setting["EnableMIPLabels"] = "True"
Set-AzureADDirectorySetting -id $setting.id -DirectorySetting $setting
As documented, the update fails as follows
PS C:\Windows\system32> New-AzureADDirectorySetting -DirectorySetting $setting
New-AzureADDirectorySetting : Error occurred while executing NewDirectorySetting
Code: Request_BadRequest
Message: Another object with the same value for property templateId already exists.
InnerError:
RequestId: e7dacfc1-2938-4240-b1ed-1f91df7ab21d
DateTimeStamp: Sun, 10 Oct 2021 17:32:51 GMT
HttpStatusCode: BadRequest
HttpStatusDescription: Bad Request
HttpResponseStatus: Completed
At line:1 char:1

New-AzureADDirectorySetting -DirectorySetting $setting

  + CategoryInfo          : NotSpecified: (:) [New-AzureADDirectorySetting], ApiException
  + FullyQualifiedErrorId : Microsoft.Open.MSGraphBeta.Client.ApiException,Microsoft.Open.MSGraphBeta.PowerShell.New
 DirectorySetting

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

ID: ef84a3e0-879e-4312-a89d-7a3bb0cd9a51
Version Independent ID: 43851f8a-b71f-0a0a-8b1f-629e3528972f
Content: Assign sensitivity labels to groups - Azure AD
Content Source: articles/active-directory/enterprise-users/groups-assign-sensitivity-labels.md
Service: active-directory
Sub-service: enterprise-users
GitHub Login: @curtand
Microsoft Alias: curtand

@shanmugara Thank you for sharing feedback. We are actively validating it. We will get back with updates soon.


Containers not updated when showing the content

Hi,
First the example: http://codepen.io/anon/pen/EVNqZW?editors=110
Then the explanation:
I would like to apply this polyfill on a large set of regions that is hidden (except the first) then manually showed, but it appears that when i set display to other that none, the content is not fit into the container. Is there a way to ensure that even if my container is not display, he's still well filled ?
Thanks =)

CSS Regions cannot fill regions which are hidden, and I'm even having trouble filling regions as their layout is changed, whether or not this involves showing/hiding the region or not. Try to have your region positioned absolutely outside of the page, or use visibility: none?

That worked like charm, thanks =)

Glad it worked for you :-)


Improve UI of the Website

I would like to work on this issue and make the UI of the website more interactive and attractive by using a better color scheme, font style and some transitions as well.

Hello @priyankaa-2004,
Thank you for expressing your interest in improving the UI of our website.
Before you dive into the changes, I recommend taking a moment to analyze the current design and functionalities. Familiarize yourself with the existing styling conventions and user interface elements. This will help you align your enhancements with the overall project aesthetics and maintain consistency.
Feel free to share your proposed approach or any design ideas you have in mind. Additionally, you can create a draft or mockup to visualize the changes before implementation. This way, we can discuss and collaborate to ensure the improvements align with the project's vision.
Make sure to follow our Contribution Guidelines & Pull Request Guidelines.
#Happy Contribution! ⭐

If you have any questions or need clarification, don't hesitate to ask in the comments or on Discord [Find #sql-dev-tools channel].
#Good luck! ✌

@PranabKumarSahoo I am interested to solve this issue! I can improve the website's UI by integrating responsive design elements and implementing modern UX principles as well.

Hey @Sohan2213,
Thanks for your interest in improvement of UI in our website. I appreciate.
But in our website there's some unnecessary bugs occurred, the issue is also raised. (#60)
First of all try to examine the issues, If you're willing to fix, then I'll assign to you.
Thank you!

Yes, sure!! @PranabKumarSahoo

So, shall I start working on the bug!? @PranabKumarSahoo

Yah sure sure. Let me assign the task to you. ✌🏻


CTE STIX/TAXII Plugin 3.0.0-beta - Added [Pagination Method] configur…

…ation parameter.

Release Notes
Added "Pagination Method" configuration parameter.
In Taxii versions 2.0 and 2.1, data retrieval is divided into chunks. Each retrieval cycle restricted to fetching up to 100 bundles, each containing 1000 objects.


Getting "no queued message with id" error with missing Webhook events

Describe the bug
Messages sent do not always produce a webhook.  In worker.log I see numerous instances of
No queued message with ID 2945901 was available for processing.
The number of messages sent vs. number of webhooks attempted is off by about 60% (30k sent vs. 10k webhooks)
To Reproduce

Send email with webhooks configured

Expected behaviour
All messages with a sent status should trigger a webhook
Screenshots
MariaDB [postal-server-6]> select distinct event, count(event) from webhook_requests where timestamp>1639579836 group by event;
+-----------------------+--------------+
| event                 | count(event) |
+-----------------------+--------------+
| MessageBounced        |          580 |
| MessageDelayed        |          613 |
| MessageDeliveryFailed |          260 |
| MessageSent           |        15444 |
+-----------------------+--------------+
4 rows in set (0.83 sec)

MariaDB [postal-server-6]> select distinct status, count(status) from messages where timestamp>1639579836 group by status;
+-----------+---------------+
| status    | count(status) |
+-----------+---------------+
| Bounced   |           910 |
| HardFail  |           324 |
| Held      |           132 |
| Pending   |           473 |
| Processed |          1026 |
| Sent      |         19762 |
| SoftFail  |            52 |
+-----------+---------------+
7 rows in set (2.33 sec)

Environment details

OS: Ubuntu Server 18.04
Database Version: MariaDB 10.1
Postal Version: 1.0.0-2595481b26-stable

Additional information/context
Server is running in AWS on an m5.xlarge.  Avg. daily volume is about 30k messages but load seems to be pretty high on the system while going through the queue (load avg. around 4 with mysql taking most of the cpu).  Postal is configured with 2 workers with 4 threads.
We have another postal install on the same hardware running version 1.0.0-dev that does not have this problem and generally has lower system load despite sending 3x more volume.

Can you add a screenshot of the error message? Also have a look at the current retention settings, these are the defaults I think

Please can you upgrade to the latest version of Postal and confirm whether this issue still exists? If so, please open a new issue for triage. Be sure to include any relevant log data as well as details about your environment as prompted by the new issue template.


fix(ipfs-coord): Disabling at startup, but leaving dependency

:tada: This PR is included in version 1.5.1 :tada:
Your semantic-release bot :package::rocket:


fix Fix potential data race when a schema change jobs is set to cancelled but the table state is still SCHEMA_CHANGE (#39164)

Proposed changes
Set job cancel state after table state changed to normal.

run buildall

run buildall

run buildall


[TEST] Set up Jenkins server for upgrade automation test.

What's the test to develop? Please describe
A jenkins server is needed to implement the upgrade automation test so that Jenkins can trigger the upgrade test daily.
The details are https://github.com/harvester/tests/issues/620

Currently switching from distributed architecture on Kubernetes to plain old docker container just like others, should be completed by 12/09/22.


DatetimeEncoder fixes

Fix #741 Remove millisecond extraction in DatetimeEncoder (Turns out you pandas' DatetimeIndex don't have a millisecond attribute.)
Fix #745 Stop using the floor function and simplify the logic to check if we need total_time
Close #744 Adds a None option for extract_until to only extract total_time.

@jeromedockes

Hey @LeoGrin, I'd advocate not to change the scope of PRs once started because it's confusing to review. To fix new issues, let's open new PRs instead :)

I think returning the total number of seconds since Epoch when self.extract_until=None is misleading for the user.
I'd rather have self.extract_until=None as a no-op (not returning anything), and have an additional keyword init parameter like self.seconds_since_epoch.
What do you both think?

I agree, that seems better. If we don't want to add a parameter we could
even always return the seconds since epoch in addition to any other
features the user requests

Otherwise, I haven't specific remarks on the PR itself, but I plan to refactorize a lot of the DatetimeEncoder implementation that I'm not super happy with in a subsequence PR.

I agree the DateTimeEncoder could use some cleanup. For example it re-parses the whole date column to extract each feature.
Also the DateTimeEncoder and the TableVectorizer both parse date strings, and they do it in different ways so it may be worth thinking about their respective responsibilities and doing some refactoring.

Thinking about it a bit more, it would better for the shape of the output to not depend on the data, for instance if we want to use this inside a grid search, so it may be better to get rid of the logic we have now (check if columns are constant).

I think that's a good idea. Dropping constant columns can be done later
in the pipeline with a VarianceThreshold transformer from scikit-learn.
I guess the idea was to avoid adding eg an hour column full of 0 when
the column we are transforming contains dates not times. But for that it
may be better to rely on the date format detected when parsing the dates
than to check the variance a posteriori

I guess the idea was to avoid adding eg an hour column full of 0 when
the column we are transforming contains dates not times. But for that it
may be better to rely on the date format detected when parsing the dates
than to check the variance a posterior

Alternatively, we could let the user switch between these two behaviors with an additional drop_if_constant keyword argument.

I think that's a good idea. Dropping constant columns can be done later
in the pipeline with a VarianceThreshold transformer from scikit-learn.

We definitely need to add the VarianceThreshold estimator to the example!


Fix copyobject test

[CI] fix copyobject testcase

Codecov Report

Merging #1697 (3b9ab27) into master (8f3a385) will decrease coverage by 48.64%.
The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff             @@
##           master    #1697       +/-   ##
===========================================
- Coverage   74.43%   25.79%   -48.64%
===========================================
  Files         296      155      -141
  Lines       36098    38085     +1987
===========================================
- Hits        26869     9825    -17044
- Misses       7507    27095    +19588
+ Partials     1722     1165      -557

Impacted Files
Coverage Δ

blobstore/api/clustermgr/proto.go

blobstore/common/raftserver/snapshotter.go

blobstore/api/clustermgr/kv.go

blobstore/common/raftserver/wal/record.go

blobstore/blobnode/service.go

blobstore/common/config/reload.go

blobstore/clustermgr/volumemgr/volume_stat.go

...om/cubefs/blobstore/common/rpc/auth/authhandler.go

...thub.com/cubefs/blobstore/common/rpc/middleware.go

...r/github.com/cubefs/blobstore/api/access/client.go

... and 441 more

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

@tangdeyi  pls pass the DCO checking


[lab] Add missing peerDependencies

Added missing peerDependencies in order to not break projects using yarn pnp.
Closes #32620

Do you have a reproduction or reference material to sustain that this is the right direction (none was provided)? I personally don't buy it, I think that we are actually suffering from the opposite problem: #32668.

Well, I did provide steps to reproduce. The issue occurs when using yarn in pnp mode (Plug'n'Play). My understanding of dependencies:

dependencies: Everything this package needs and that should get installed when installing the package
devDependencies: Everything this package needs in order to build or work on it
peerDependencies: Things this package depends on, but not shipping with it, i.e. to leave open a degree of freedom which version is installed (i.e. react 16, 17, 18)

What should not happen from my point of view, that a package uses some other library, without declaring any dependency to it. And from what I see this is what was happening here...
But if you see a different way to fix the given problem, I'm open to suggestions :)

@merceyz We might need your help on this one around peer dependencies and Yarn PnP 🙏.
Here is a simplified version of the problem: @mui/lab nor @mui/material directly imports from @emotion/react, however, they have a dependency that does.
Should @mui/lab or @mui/material have @emotion/react as a peer dependency? Thanks!
If yes, then we need to move with the direction taken in this PR and many other changes.
If not, then we need to move forward with #32668.

Should @mui/lab or @mui/material have @emotion/react as a peer dependency?

Yes, if a dependency declares a peer dependency then you need to provide it in order for it to get access to it.
See https://dev.to/arcanis/implicit-transitive-peer-dependencies-ed0 for a more detailed version.

@merceyz Ok perfect, so if I understand your comment and the blog post correctly, we can close this PR and move forward with #32668. The packages we are considering, have, AFAIK no transitive dependency on emotion.

No, this PR is correct, @mui/lab needs to declare @emotion/react and @emotion/styled as peer dependencies.
You can see this by inspecting the dependency tree by running the following commands:
cd $(mktemp -d)
yarn init -2
yarn add @mui/lab @emotion/react @emotion/styled
yarn explain peer-requirements

Which gives the following complaints:
➤ YN0000: pf5d3d → @mui/lab@npm:5.0.0-alpha.80 [68b79] doesn't provide @emotion/react to @mui/system and 1 descendant
➤ YN0000: p6487f → @mui/lab@npm:5.0.0-alpha.80 [68b79] doesn't provide @emotion/styled to @mui/system and 1 descendant

@merceyz Ok in this case I'm not against the change, only to make sure we are covering all the cases.
Personally, it feels wrong to go in this direction. Can developers that depend on MUI fall into the bug described in https://dev.to/arcanis/implicit-transitive-peer-dependencies-ed0? From what I understand, no because MUI never has a direct dependency on @emotion/styled or @emotion/react that could be wrongly hoisted and take over the version installed at the root. I feel that in order to make a warning happy, we break the notion of dependency isolation (the parents of @mui/styled-engine, all need to know about the right version of the peer dependency that he expects).

Is someone still expecting anything from me here or is this going to be merged? :)

@nate-summercook It's up to the team working on MUI Core, they are responsible for it. My feedback is not binding.

sorry for being impatient here, but this issue really is a blocker for me (and anyone using @mui/lab through yarn pnp).

sorry for being impatient here, but this issue really is a blocker for me (and anyone using @mui/lab through yarn pnp).

@nate-summercook You can use packageExtensions while you wait for this to be released.

@nate-summercook You can use packageExtensions while you wait for this to be released.

Thanks! That's news to me! If only I had known this already two weeks ago 😅


Navbar improvement in donate animals section

add a better UI

Hey I would like to work on this issue during hacktoberfest.
Please assign this issue to me

@akshitagupta15june ,
What kind of UI improvements are expected? Also, can I add my own custom CSS file(s) if necessary?
I could observe so much empty space beside the logo-and-adopt-today context in the header. Can the navbar section be moved there?

Hey ,can I work on this issue please

@akshitagupta15june , is this issue still open to working? I see the UI for Navbar component has already been updated.


[Flaky Test] Controller Common Suite: [It] CheckerWatchdog

How to categorize this issue?

/area testing
/kind flake
Which test(s)/suite(s) are flaking:
Controller Common Suite: [It] CheckerWatchdog
CI link:
https://prow.gardener.cloud/view/gs/gardener-prow/logs/ci-gardener-unit/1511573836791812096
https://prow.gardener.cloud/view/gs/gardener-prow/logs/ci-gardener-unit/1517072393976156160
https://prow.gardener.cloud/view/gs/gardener-prow/logs/ci-gardener-unit/1518522396259127296
Reason for failure:
Multiple failures like:
Running Suite: Controller Common Suite - /home/prow/go/src/github.com/gardener/gardener/extensions/pkg/controller/common
========================================================================================================================
Random Seed:<PHONE_NUMBER>
Will run 18 of 18 specs
••••••••••••
------------------------------
• [FAILED] [0.002 seconds]
CheckerWatchdog [AfterEach]
/home/prow/go/src/github.com/gardener/gardener/extensions/pkg/controller/common/checkerwatchdog_test.go:55
  #Start / #Stop / #AddContext / #RemoveContext / #Result
  /home/prow/go/src/github.com/gardener/gardener/extensions/pkg/controller/common/checkerwatchdog_test.go:59
    should cancel the context returned by AddContext if the checker returns false
    /home/prow/go/src/github.com/gardener/gardener/extensions/pkg/controller/common/checkerwatchdog_test.go:82
  missing call(s) to *mock.MockChecker.Check(is anything) /home/prow/go/src/github.com/gardener/gardener/extensions/pkg/controller/common/checkerwatchdog_test.go:83
  In [AfterEach] at: /home/prow/go/src/github.com/gardener/gardener/extensions/pkg/controller/common/checkerwatchdog_test.go:56
------------------------------
•••••
Ran 18 of 18 Specs in 0.127 seconds
FAIL! -- 17 Passed | 1 Failed | 0 Pending | 0 Skipped
--- FAIL: TestCommon (0.13s)
FAIL
FAIL	github.com/gardener/gardener/extensions/pkg/controller/common	0.453s

Anything else we need to know:

I've spent quite some time to reproduce, debug and understand this problem. However, so far I haven't been able to.
Hence, https://github.com/gardener/gardener/pull/5873 adds some debug logs to help debug the flake if it occurs again.

@timebertt It happened again here: https://prow.gardener.cloud/view/gs/gardener-prow/pr-logs/pull/gardener_gardener/5876/pull-gardener-unit/1520017547204759552

The logs in above build are:
• [FAILED] [0.003 seconds]
CheckerWatchdog [AfterEach]
/home/prow/go/src/github.com/gardener/gardener/extensions/pkg/controller/common/checkerwatchdog_test.go:57
  #Start / #Stop / #AddContext / #RemoveContext / #Result
  /home/prow/go/src/github.com/gardener/gardener/extensions/pkg/controller/common/checkerwatchdog_test.go:61
    should cancel the context returned by AddContext if the checker times out
    /home/prow/go/src/github.com/gardener/gardener/extensions/pkg/controller/common/checkerwatchdog_test.go:126
  Begin Captured GinkgoWriter Output >>
    {"level":"debug","ts":1651236036.4442346,"msg":"Starting watchdog"}
    {"level":"debug","ts":1651236036.44455,"msg":"Adding context","key":"foo"}
    {"level":"Level(-2)","ts":1651236036.4446206,"msg":"Stopping timer"}
    {"level":"Level(-2)","ts":1651236036.4447029,"msg":"Resetting timer"}
    {"level":"Level(-2)","ts":1651236036.4447496,"msg":"Result requested"}
    {"level":"info","ts":1651236036.4449112,"msg":"Cancelling all contexts"}
    {"level":"Level(-2)","ts":1651236036.44[495](https://prow.gardener.cloud/view/gs/gardener-prow/pr-logs/pull/gardener_gardener/5876/pull-gardener-unit/1520017547204759552#1:build-log.txt%3A495)82,"msg":"Result requested"}
    {"level":"Level(-2)","ts":1651236036.44[499](https://prow.gardener.cloud/view/gs/gardener-prow/pr-logs/pull/gardener_gardener/5876/pull-gardener-unit/1520017547204759552#1:build-log.txt%3A499)28,"msg":"Using last result"}
    {"level":"debug","ts":1651236036.44[505](https://prow.gardener.cloud/view/gs/gardener-prow/pr-logs/pull/gardener_gardener/5876/pull-gardener-unit/1520017547204759552#1:build-log.txt%3A505)14,"msg":"Removing context","key":"foo"}
    {"level":"debug","ts":16[512](https://prow.gardener.cloud/view/gs/gardener-prow/pr-logs/pull/gardener_gardener/5876/pull-gardener-unit/1520017547204759552#1:build-log.txt%3A512)36036.4451535,"msg":"Stopping watchdog"}
  << End Captured GinkgoWriter Output
  missing call(s) to *mock.MockChecker.Check(is anything) /home/prow/go/src/github.com/gardener/gardener/extensions/pkg/controller/common/checkerwatchdog_test.go:127

Does this help @timebertt ?

/assign @timebertt since he wants to take another look

The debug logs were helpful indeed. I've managed to understand what's going on in these flakes:
In the successful case, the logs show this order of execution:
  Begin Captured GinkgoWriter Output >>
    {"level":"debug","ts":1651484912.7265851,"msg":"Starting watchdog"}
    {"level":"debug","ts":1651484912.726625,"msg":"Adding context","key":"foo"}
    {"level":"Level(-2)","ts":1651484912.7266822,"msg":"Stopping timer"}
    {"level":"Level(-2)","ts":1651484912.726752,"msg":"Resetting timer"}
    {"level":"Level(-2)","ts":1651484912.726759,"msg":"Result requested"}
    {"level":"info","ts":1651484912.726826,"msg":"Cancelling all contexts"}
    {"level":"Level(-2)","ts":1651484912.7268338,"msg":"Timer expired"}
    {"level":"info","ts":1651484912.726847,"msg":"Cancelling all contexts"}
    {"level":"Level(-2)","ts":1651484912.7268531,"msg":"Result requested"}
    {"level":"Level(-2)","ts":1651484912.726857,"msg":"Using last result"}
    {"level":"debug","ts":1651484912.726868,"msg":"Removing context","key":"foo"}
    {"level":"Level(-2)","ts":1651484912.726872,"msg":"Stopping timer"}
    {"level":"debug","ts":1651484912.726876,"msg":"Stopping watchdog"}
    {"level":"Level(-2)","ts":1651484912.726882,"msg":"Context done"}
  << End Captured GinkgoWriter Output

I.e., Result requested -> Timer expired -> Result requested (Using last result)
In the flake however, it looks like this:
  Begin Captured GinkgoWriter Output >>
    {"level":"debug","ts":1651236036.4442346,"msg":"Starting watchdog"}
    {"level":"debug","ts":1651236036.44455,"msg":"Adding context","key":"foo"}
    {"level":"Level(-2)","ts":1651236036.4446206,"msg":"Stopping timer"}
    {"level":"Level(-2)","ts":1651236036.4447029,"msg":"Resetting timer"}
    {"level":"Level(-2)","ts":1651236036.4447496,"msg":"Result requested"}
    {"level":"info","ts":1651236036.4449112,"msg":"Cancelling all contexts"}
    {"level":"Level(-2)","ts":1651236036.4449582,"msg":"Result requested"}
    {"level":"Level(-2)","ts":1651236036.4449928,"msg":"Using last result"}
    {"level":"debug","ts":1651236036.4450514,"msg":"Removing context","key":"foo"}
    {"level":"debug","ts":1651236036.4451535,"msg":"Stopping watchdog"}
  << End Captured GinkgoWriter Output

I.e., Result requested -> Result requested (Using last result)
Note, that Timer expired is not logged here.
This explains the cause of the flake:
Both case <-w.resultChan: and case <-timer.C(): in the watchdog's select statement can be unblocked/ready at the same time (same select execution) because the test code advances the fake clock and calls Result in quick succession (in a different goroutine). If that happens, a random case is chosen, as the select statement is not deterministic.
If the "timer expired" case is executed first, then there is a second checker.Check() call (expected by the mock) and after that the "result requested case is executed, which at the end makes the watchdog.Result() call return as expected by the test code. -> Test succeeds
If the "result requested" case is executed first, the watchdog.Result() call returns immediately without a new Result call and the test case (It) returns. There might not have been a checker.Check() call yet, which makes the mock fail the test case. Note however, that the watchdog's goroutine might still be running, so the Check() call is actually done, though too late for the mock's expectations.
In short, I would say that the root issue here is that we are testing asynchronous code (watchdog runs in another goroutine) with synchronous test code and mechanisms (mocks don't wait for calls to be done before returning from the test case).
I fixed this in PR by swapping out the mock with an asynchronous fake checker.


Bug: Commands cannot accept many paths

Describe the bug
Because CLI commands are actually routed their parameters go through some routing and filtering making it impossible to pass certain characters as a parameter. Of note that I ran into was trying to use a command that take a files path - the "/"s and "-"s cause it to be filtered and never arrives in $params.
CodeIgniter 4 version
Latest develop branch
Affected module(s)
CLI Commands
Expected behavior, and steps to reproduce if appropriate
Create a simple commands that calls var_dump($params) in run(). Call it:
php spark test:command /usr/bin/myfile.txt
Response is empty array.
Context

OS: Linux
Web server Apache
PHP version 7.2+

Have you tried quoting the offending string?
php spark test:command "/usr/bin/myfile.txt"

Yes, it still routes it:
class Test extends BaseCommand
{
    public function run(array $params)
    {
		dd($params);
    }

$ ./spark test "/usr/tmp/bin"

CodeIgniter CLI Tool - Version 4.0.0-rc.1 - Server-Time: 2019-09-24 10:36:06am

│ $params │
array (4) [
    0 => string (0) ""
    1 => string (3) "usr"
    2 => string (3) "tmp"
    3 => string (3) "bin"
]

Called from .../app/Commands/Test.php:14 [App\Commands\Test->run()]

Not a bug. Slashes need to be url-encoded, i.e. %2fusr%2ftemp%2fbin
The parameter needs to then be urldecode(...) in your code to see the original string with slashes.

Maybe "bug" isn't the correct term, but it's definitely a broken expectation. If I'm sitting at a shell prompt and want to call a command with a file argument I type ./command /path/to/file. For spark to require HTML encoding for paths is pretty weird.

Agreed, HTML encoding shouldn't be needed here.

@MGatner have you had any more chance to look into this?


Add necessary user permissions for Azure Bot creation

Please add necessary permissions required for creating an Azure Bot, Resource Group, or other steps if they differ.

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

ID: 7cf57ecf-7774-8f8b-1e47-cf628c3bfa25
Version Independent ID: ba30559c-36df-69c6-2703-cd67aa1380c2
Content: Create an Azure Bot resource in the Azure portal - Bot Service
Content Source: articles/v4sdk/abs-quickstart.md
Service: bot-service
GitHub Login: @mmiele
Microsoft Alias: kamrani

@matt852 -- I'm in the process of updating this topic. I'll ask the product team and let you know what I find out.

@matt852 -- as long as you're signed in to the Azure tenant, you should be able to create all these resources without any additional permissions needed.


Environmental variables are being ignored

Despite having environmental variables being set, on they are not honored.
Running a container from docker-compose, later:
docker exec -it container_id sh

and later:
~$ set

Gives, for example:
REDIS_DB='0'
REDIS_HOST='redis'

which is right, this is how I set it up, but
docker-compose logs

produces:
bullboard  | Error: connect ECONNREFUSED <IP_ADDRESS>:6379
bullboard  |     at TCPConnectWrap.afterConnect [as oncomplete] (node:net:1495:16) {
bullboard  |   errno: -111,
bullboard  |   code: 'ECONNREFUSED',
bullboard  |   syscall: 'connect',
bullboard  |   address: '<IP_ADDRESS>',
bullboard  |   port: 6379
bullboard  | }

Hi Vincent,
Thanks for fixing the bug so quickly! I appreciate your attention to detail
and willingness to help.
Best,
Tomasz
On Mon, Sep 11, 2023, 16:21 Vincent Le Quec @.***>
wrote:

Closed #49 https://github.com/Venatum/bull-board-docker/issues/49 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/Venatum/bull-board-docker/issues/49#event-10333439655,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABBI252PDIVLHYXSBCRBWXLXZ363RANCNFSM6AAAAAA3UBFPSE
.
You are receiving this because you authored the thread.Message ID:
@.***>


check-windows-service.rb problem when spaces in the service name

Needs quotes around the config service name when service name has spaces.
I fixed this problem by adding the following:
cmd= 'tasklist /svc|findstr /r /c:' + '"'+ config[:service] +'"'
#cmd= 'tasklist /svc|findstr /i ' + '"'+ config[:service] +'\"'
temp = system(cmd)

could you put together a pr?


How to expose Jaeger operator on VM

My use case is as below.
Cloud:GCP
I have plenty of applications deployed on a VM IN GCP
I have deployed JAEGER OPERATOR in GKE cluster
Now I want to trace my applications using the Operator deployed in GKE.
Is it possible to do that and how?

You can port forward from the jaeger query service, allowing you to see the traces via your browser.
kubectl port-forward svc/<jaeger-query-name> -n <namespace> 16686
But you'll probably only see the query service's own traces without additionally instrumenting your own running services separately.

If it is possible to use jaeger endpoint with client library instrumentation if we configure external DNS for UI (query) so that my application hosted on VM can talk to Jaeger

Now I want to trace my applications using the Operator deployed in GKE.

Not sure what you mean here. If you mean that you have a Jaeger Agent in your GCP, then you can certainly expose the port 14250 from the collector via an ingress. If the agent and collector aren't part of the same internal network, it might be tricky, as it means that you'd have an exposed collector. In which case, I see two options:

use the OpenTelemetry Collector as an agent in your GCP and as a first collector in your GKE, using the auth capabilities from it. Your GKE otelcol would then use your Jaeger Collector as the backend.
use the JAEGER_ENDPOINT + JAEGER_USERNAME/PASSWORD in the clients (GCP), bypassing the agent and sending data directly to the collector. You'd then need an auth proxy in front of the collector to validate the credentials.

If this isn't what you asked, please clarify (hence the needs-info). Feel free to close if the question has been answered.


How to only rendered visible canvas ?

when i add 10000 keframes in every row ,then the  Page begain lazy and lag. how Virtual scrolling improves efficiency

Hello @Myfyz
Only currently visible key frames are rendered.
It assumed that for the apps keyframes are distributed and you won't have 10.000 rendered at the same time.
What is your usecase?


Allow utils in themeMap for ease of typing for a more generic typing

Is your feature request related to a problem? Please describe.
Currently, trying to type a utility function implies repeating this pattern on almost every utility:
margin: (config) => (value: keyof typeof config['theme']['space'] | (string & {})) => ({ })
marginHorizontal: (config) => (value: keyof typeof config['theme']['space'] | (string & {})) => ({ })
marginVertical: (config) => (value: keyof typeof config['theme']['space'] | (string & {})) => ({ })

Describe the solution you'd like
What if themeMap could also include the utilities rather than only the existing CSS props? On that way, we could do something like this:
import createCss, { defaultThemeMap } from "@stitches/react";

createCss({
  ...,
  utils: {
    marginHorizontal: (config) => (value) => ({
      marginLeft: value,
      marginRight: value,
    }),
  },
  themeMap: {
    ...defaultThemeMap,
    marginHorizontal: "space",
  },
});

Describe alternatives you've considered
N/A
Additional context
N/A

Thanks for raising this. I have a note with the exact same idea and I'm really glad someone else mentioned it!

We've been working on types and are introducing some new types to deal with his. Will post an update when its out.

Fixed in V1. Please refer to migration guide and changelog


geoip - Schweiz/Switzerland: Cannot allocate memory

[x] I have read the contributing guide lines at https://github.com/opnsense/core/blob/master/CONTRIBUTING.md
[x] I have searched the existing issues and I'm convinced that mine is new.
Describe the bug
I want to create a block alias for every country in ipv4 and ipv6 except germany. When I tick everything except germany and apply, I'll get 'Cannot allocate memory'.
I switched RAM from 4GB to 8GB which increased the table from 405000 to 815000, but it's the same result. In Switzerland lies the bug.
To Reproduce
Steps to reproduce the behavior:
It works when I untick Switzerland, but additionally it should be then 47 out of 49 selected. Anyways, it shows 46 out of 49, which I guess could lead to the bug.

OPNsense 19.1.10 (amd64, LibreSSL).
AMD Opteron(tm) Processor 4234
Intel 82574L Gigabit Network Connection

Under Firewall: Settings: Advanced set "Firewall Maximum Table Entries" to e.g. 6000000.

6 zeroes, double-check.
But now:

Maybe reboot helps?
And it still says 47 out of 49 selected with only Germany unticked.

Still too low maybe? If you have IPv6 enabled table entry count could be huge.

I know that it will get huge, but I have no idea in GB what huge means. I have no experience with it. Isn't there a crude rule of thumb? ;)
After reboot, no message so far.

Why not choose only DE and CH and negate/invert the rule?

Sounds good, will try that. I really have no experience with the possibilities of more advanced fw-rules.

Just create the identical rule and hit "invert" checkbox .. (of course with new alias containing only the two).

Nice, thank you!

Helps understanding, what invert means. I only wanted to allow DE, so it should work now and problem is solved. :)

After some days and some machines testing...
only germany + invert works so nice and fast.


Review dir attribut in verifyDir

In verifyDir method review attribut dir, to case what exist more directry

Add point '.' before '=' and invert bar '' in this line '$this->dir = $this->url[$i];' to will be => '$this->dir .= ''. $this->url[$i];'


Apply identical styling to upper level nodes with/without children in tree grids

Closes #2414
Hoist P/R Checklist
Pull request authors: Review and check off the below. Items that do not apply can also be
checked off to indicate they have been considered. If unclear if a step is relevant, please leave
unchecked and note in comments.

[x] Caught up with develop branch as of last change.
[x] Added CHANGELOG entry, or determined not required.
[x] Reviewed for breaking changes, added breaking-change label + CHANGELOG if so.
[x] Updated doc comments / prop-types, or determined not required.
[x] Reviewed and tested on Mobile, or determined not required.
[x] Created Toolbox branch / PR, or determined not required.

If your change is still a WIP, please use the "Create draft pull request" option in the split
button below to indicate it is not ready yet for a final review.

Pull request reviewers: when merging this P/R, please consider using a squash commit to
collapse multiple intermediate commits into a single commit representing the overall feature
change. This helps keep the commit log clean and easy to scan across releases. PRs containing a
single commit should be rebased when possible.

I have some questions / concerns here - will reach out to review together momentarily, but please hold off on merging this until we get a chance to discuss - thanks!


Deprecate ClaimManagementService API

We need to mark the ClaimManagementService API as deprecated.
https://github.com/wso2/carbon-identity-framework/blob/master/components/claim-mgt/org.wso2.carbon.claim.mgt.ui/src/main/resources/ClaimManagementService.wsdl

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


[Feature]: Add manually generating session for tests

Feature type
New Feature
Description
With oidc 8, I've rewritten token validation. Now all is done via oidc. Tests have pregenerated data. Try to rewrite it, so each test will generate its own session

Fixed with  c8355ef


chore: more null checks

refactor useFee

refactor useFee

I've tried to untangled the current type error that made the build fail but I think there could be quite a bit of refactoring to do.
first PaymentType now refers to 2 differents things:
PaymentType from graphql codegen
PaymentType from galoy-client
those things are confusing so we should rename one of them
I also think we should stop using enum from js, and rely on const object instead. there is less type quirks IMO with plan js const object. declared with:
const obj = {
   LnUrl: "lnurl"
  Onchain: "onchain"
} as const

the key take away from the code below is the as const at the end. otherwise typescript would not infer properly that the set of keys are fixed
overall, I think we could extend the graphql.PaymentType with the additional "unified" and other property. The only thing I'm not sure how to really handle now is the fact that we're have the galoy-client library, but this library doesn't rely on codegen. so we can't really extend the PaymentType that graphql give us and have more internal only mobile type.
anyway, just some food for thought


Expecting a function in instanceof check, but got undefined

Hey all,
I've been getting this error:
/node_modules/jsdom/lib/jsdom/living/generated/EventInit.js:32
    if (obj instanceof Date || obj instanceof RegExp) {
                       ^
TypeError: Expecting a function in instanceof check, but got undefined
    at Object.convert (/Users/client/Development/7tennis-web/node_modules/jsdom/lib/jsdom/living/generated/EventInit.js:32:24)
    at new EventImpl (/Users/client/Development/7tennis-web/node_modules/jsdom/lib/jsdom/living/events/Event-impl.js:8:48)
    at Object.setup (/Users/client/Development/7tennis-web/node_modules/jsdom/lib/jsdom/living/generated/Event.js:199:17)
    at Object.create (/Users/client/Development/7tennis-web/node_modules/jsdom/lib/jsdom/living/generated/Event.js:183:10)
    at HTMLDocument.core.Document.createEvent (/Users/client/Development/7tennis-web/node_modules/jsdom/lib/jsdom/level2/events.js:156:16)
    at Window.process.nextTick (/Users/client/Development/7tennis-web/node_modules/jsdom/lib/jsdom/browser/Window.js:376:34)
    at _combinedTickCallback (internal/process/next_tick.js:67:7)
    at process._tickCallback (internal/process/next_tick.js:98:9)

This seems to happen during the .jsdom() call. I'm using Jest, Enzyme and sinon.
Here's the file structure:
import React from 'react';
import { shallow, mount } from 'enzyme';
import { MoreDropdown } from '../MoreDropdown';

// https://github.com/airbnb/enzyme/blob/master/docs/guides/jsdom.md
var jsdom = require('jsdom').jsdom;

global.document = jsdom('');
global.window = document.defaultView;
Object.keys(document.defaultView).forEach((property) => {
  if (typeof global[property] === 'undefined') {
    global[property] = document.defaultView[property];
  }
});

global.navigator = {
  userAgent: 'node.js'
};

const sinon = require('sinon');

^^^ that seems to fail before it reaches any tests.
Note: I don't get the
if (obj instanceof Date || obj instanceof RegExp)
                       ^

if I import sinon before jsdom, but Expecting a function in instanceof check, but got undefined error is still shown.
Please let me know if you have any tips on solving this.
Cheers!

This seems to be a consequence of the unsupported practice of mashing together two different global objects; see https://github.com/tmpvar/jsdom/wiki/Don't-stuff-jsdom-globals-onto-the-Node-global.


Determine if vendor specific plugin and samples should be part of MRTK repo

Should things like "Microsoft's MR Plugin" be checked into the MRTK repo

MS MR package provides hand joint data and controller models.  Maybe not ready to remove the dependency yet.


⚠ Secret Website 3 has degraded performance

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

HTTP code: 200
Response time: 2064 ms

Resolved: Secret Website 3 performance has improved in 5ffa29c.


Update centos_install_arm_ethosn_driver_stack.sh to use Python 3.7

As Dockerfile.package-cu102 does not have Python 3.6 (EOL), moving dependency installation script centos_install_arm_ethosn_driver_stack.sh to use Python 3.7 instead.
cc @tqchen @areusch @Mousius

Expected to fail until https://github.com/tlc-pack/tlcpack/pull/92 is merged.


automatic reconnection

automatic connection in the chat view this improvement solves having to enter and exit the chat view every time the connection was lost.

@jeandudey  There are no limits of attempts, so as not to make too many attempts in a row, a time of 8 seconds of interval was added in each attempt, this is only done when you are in the chat view of a contact.


Alert DSL

Love some opinions. I'm going to put a lot of effort into validating the config as best I can, so don't be afraid of a DSL being flimsy, that said currently you do:
  {
    "enabled": true,
    "metric": "api.errors",
    "statistic": "sum",
    "threshold": 5,
    "action": "email.me"
  }

It might be nice to optionally do:
"email.me if api.errors sum > 5 in 1m",
"email.me if api.latency avg >= 1.5s in 5m"

where "email.me" is an action defined elsewhere

JSON of course. Perhaps also ship a schema so that VS code can check the edits are OK.
Just to compare your design against: https://prometheus.io/docs/alerting/rules/


Add user-defined reporter ions.

New option: userReporterIons = [mass1, mass2, mass3, ...]
These masses are specifically annotated within lorikeet.
They are listed in the info section where mods are listed.

Going to resubmit request with a more sane diff.


About training CLIP with CC3M data

Hello,
I would like to train the CLIP model from scratch on CC3M data. However, if I try to create and use the dataloader, the process ends as the RAM usage increases significantly. Maybe it's because the data is large, have you experienced any similar problems?

Hey @Seung-B, thanks for your interest in our work! We did not encounter this issue. Have you tried reducing the batch-size?

Thanks for your response!
CC3M data cannot be fully downloaded now, so I am using the data in the hugging face(2.95M).
There are many links where CC3M data has expired, so I can get only 2.2M data.
Do you know where I can get full data?

Thank you for responding to my question!
I used the Hugging Face dataset, and even if it takes a long time, I obtained 2.95M cc3m data by saving them locally one by one.
https://huggingface.co/datasets/pixparse/cc3m-wds


unstable column order in rows.Scan() binding?

Apologies for the screenshots. I am wrapping up my experiments with this because I think the overhead in conforming to the sql.DB interface is unnecessary in my case. However I noticed odd behavior where the bindings in Scan don't seem to be stable. I can't really trace it down, maybe it's happening in the sql.DB standard library code?
You see the key and value being reversed across refreshes. I am calling stmt, err := db.N1QLConnection.Prepare(query) with each HTTP request to my server (not recommended but maybe part of reproducing what I see.) In the screenshots the JSON key should always correspond to the meta object, but it sometimes doesn't, and I can't figure out why.
I am calling rows.Scan(&meta, &value) to get the results of a query like SELECT META(default), beer from default WHERE beer IS NOT NULL but my results vary across browser refresh. It's like sometimes meta is getting bound to the first result column and sometimes they are switched. I haven't tried with results with more than two columns yet.

Here is a screenshot of my code, and a link to the branch.

I was able to reproduce the screenshots from fresh material as gists https://gist.github.com/jchris/a921a72f102aa5210d04

I tried putting an as statement into my query but the problem persists.
SELECT META(default) as meta, beer from default WHERE beer IS NOT NULL

Hi Chris,
We need to provide an additional API for use by our new shell. We should make sure that API meets your needs as well.
We will send you a proposal when ready.
Gerald
Sent from my T-Mobile 4G LTE Device
-------- Original message --------
From: Chris Anderson<EMAIL_ADDRESS>Date: 9/25/2015 7:07 AM (GMT-08:00)
To: couchbaselabs/go_n1ql<EMAIL_ADDRESS>Subject: Re: [go_n1ql] unstable column order in rows.Scan() binding? (#3)
I tried putting an as statement into my query but the problem persists.
SELECT META(default) as meta, beer from default WHERE beer IS NOT NULL
—
Reply to this email directly or view it on GitHubhttps://github.com/couchbaselabs/go_n1ql/issues/3#issuecomment-143233418.

What I need is pretty simple.
On config change, I'll validate and prepare some statements. At query time I will run them and filter the results to match the user's channels. Validation consists of making sure the query starts with selecting meta.id and probably current rev information from the _sync field, which I need for filtering.
It looks like this library has all the code that we need for service discovery etc, but breaking it free of the sql.DB interface would make it easier to adapt for our purposes, I wouldn't need to care how many Columns the query selects, etc.

Got something running with gocb. Thanks!

@jchris  Thanks for reporting this issue. Fixed by d3910182e5c925127a1424246ff0cba8c1d80108


Add significant whitespace

Fixes Markdown rendering of headers on Github

lol, there's already a dozen pull requests open that do the same thing


chore: add Slack notification for CI workflow

Not sure if this is needed, let's see how it goes.

Apparently the default GitHub Slack app that we use updates the PR notification with check information (which checks failed or "All checks have passed"), which is the information I'm after here, so I don't think we need this. Closing.

Apparently the default GitHub Slack app that we use updates the PR notification with check information (which checks failed or "All checks have passed"), which is the information I'm after here, so I don't think we need this. Closing.


WebGl: jagged edges of rounded shapes

Hi guys, I'm trying to draw rounded rect using PIXI.Graphics class, and I noticed one thing that webGl renderer draws this shape with some ugly jagged borders, but canvas renderer draws it good.
I know that this is known issue, does anyone have a solution how to draw it smoothly also for webGL renderer ?
Thanks
this.clear(); this.beginFill(color, 1); this.drawRoundedRect(0, 0, w, h, h/2); this.endFill();
webGl renderer

canvas renderer

try antialias:true in renderer options.
var renderer = PIXI.autoDetectRenderer(800, 600, {antialias:true});

Thank you very much. It works


include!(concat!(env!("OUT_DIR"), "/", file!())) does not include auto-suggestions

I have a project
project/
  Cargo.toml
  src/
    main.rs
  build.rs
  target/
    build/
      debug/
        project-xxxx/
          out/
            src/
              main.rs

In my main-file, I want to do:
// project/src/main.rs
mod test {
    include!(concat!(env!("OUT_DIR"), "/", file!()));
}

fn main() {
    test:: // no auto-suggestions
}

But no auto-suggestions are showing. However, when I do:
// project/src/main.rs
mod test {
    include!(concat!(env!("OUT_DIR"), "/", "src/main.rs"));
}

fn main() {
    test:: // a lot of auto-suggestions
}

It works great. Would it be possible to make the first example show auto-suggestions?

file!() expands to "".
https://github.com/rust-analyzer/rust-analyzer/blob/7f57a01b3d524a5ed85a30265b8aee2c81118f15/crates/hir_expand/src/builtin_macro.rs#L202-L204
Rustc returns a relative file name rather than an absolute one as the comment there suggests.

Rustc returns a relative file name rather than an absolute one as the comment there suggests.

Relative to crate, right ? If it is the case, maybe we could do something ?

Relative to the workspace dir. Cargo passes rustc the path of the crate root relative to the workspace dir.

Is Rust-Analyzer unaware of Cargo, and if so, does it mean this is not possible to solve?


Fails to compile against hscolor 1.21

Output from cabal install:
onfiguring hlint-1.9.16...
Building hlint-1.9.16...
Preprocessing library hlint-1.9.16...
[ 1 of 46] Compiling Test.Util        ( src/Test/Util.hs, dist/dist-sandbox-92d56781/build/Test/Util.o )
[ 2 of 46] Compiling HSE.Type         ( src/HSE/Type.hs, dist/dist-sandbox-92d56781/build/HSE/Type.o )
[ 3 of 46] Compiling HsColour         ( src/HsColour.hs, dist/dist-sandbox-92d56781/build/HsColour.o )

src/HsColour.hs:25:16:
    Couldn't match type ‘Int’ with ‘[Char]’
    Expected type: String -> String
      Actual type: Int -> String -> String
    Probable cause: ‘CSS.hscolour’ is applied to too few arguments
    In the expression: CSS.hscolour False
    In an equation for ‘hsColourHTML’:
        hsColourHTML = CSS.hscolour False

Confirmed, I see the same behavior with the latest hscolour, which was probably just released. I'll patch in about an hour. I've just edited the metadata, so in theory it should now work.

Fixed in hlint-1.9.17. Thanks for the speedy report!


About the reproduction and OpenAI deprecated  models

Hi,
Your project is exciting and I want to dig into it more. But when I tried to reproduce your project, there are two issues happened:

OpenAI has deprecated some models you have used in the paper and code. Can you update the code to make it up-to-date?
When I tried to reproduce your work, the result of PrOntoQA was around 55% (as mentioned in the #4) with gpt-3.5-turbo-instruct which is the OpenAI recommended alternative model for text-davinci-003.  If possible, may we discuss that? I want to figure out what I was doing wrong.

It's an awesome project! Hope to hear back from you!

Closed as it is solved in #4


select_interrupt_line_1 on stm32g4 is incorrect

The config implementation of select_interrupt_line_1 copies the Interrupts bitflags directly to the FDCAN_ILS register. This works on STM32H7 because this register's bits have the same mapping as other interrupt registers like FDCAN_IE.
On STM32G4 the bits are groups of interrupts, instead:

So currently this results in unpredictable interrupt-to-line mappings.

I might have time to submit a fix for this, my only question is whether the public interface needs to be the same across both families of chips. If it can be a different function signature for G4/etc then it seems relatively straightforward.
If they need to be compatible then maybe not, although a G4 compatible function could be implemented to work on H7 obviously (it would just be less flexible).


eas build:configure fails to run if no git installed

Summary
eas build:configure fails to run if no git command available (no git installed)
Environment
Windows 10
EAS CLI 0.37.0
Error output
eas build:configure
💡 The following process will configure your iOS and/or Android project to be compatible with EAS Build. These changes only apply to your local project files and you can safely revert them at any time.
√ Which platforms would you like to configure for EAS Build? » All
Error: git command not found, install it before proceeding
Reproducible demo or steps to reproduce from a blank project
Use any project and dont install git and try to run eas build:configure - it will fail to run
Expected behaviour
I think EXPO/EAS should not depend on git installed in system.

Hi @psnet,
Thanks for reporting. Actually, EAS CLI does not require you to have git installed. If you want to use the CLI without Git, you should set the EAS_NO_VCS=1 environment variable. See https://expo.fyi/eas-vcs-workflow for more details.

hi how can i set EAS_NO_VCS=1 environment variable
please say me

oi como posso definir EAS_NO_VCS = 1 variável de ambiente por favor me diga

https://stackoverflow.com/a/73500190


chore: remove unused write operations (insert, delete, batch_delete)

PR Type
Enhancement

Description

Removed unused write operations (insert, delete, batch_delete) from the RocksDB wrapper
Simplified BufferedBatchWriter by removing the delete method
Eliminated insert_impl, delete_impl, prepare_batch_deletion, and get_or_insert_with methods from RocksCf
Improved code maintainability by removing dead code and unused functionality
No new features added; purely a code cleanup operation

Changes walkthrough 📝
Relevant filesCode cleanup

rocks_batch_writer.rsRemove unused delete operation from BufferedBatchWriter

src/eth/storage/rocks/rocks_batch_writer.rs
Removed the delete method from the BufferedBatchWriter struct  Simplified the implementation by removing unused write operations

+0/-13

rocks_cf.rsRemove unused write operations from RocksCf

src/eth/storage/rocks/rocks_cf.rs
Removed insert, delete, prepare_batch_deletion, and get_or_insert_with methods  Eliminated unused write operations and their associated helper functions

+0/-64

💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information

Final benchmark:
RPS Stats: Max: 979.00, Min: 787.00, Avg: 839.29, StdDev: 24.43
TPS Stats: Max: 929.00, Min: 787.00, Avg: 838.96, StdDev: 22.82
Plot: View Plot


Oc 253 test implementation of new abstract appication context

All is well.

All is well.

All is well.

All is well.


Cursor return regardless if event is available?

Hi,
The old behavior of the rethinkdb driver where cursors not being returned until an event is available was changed, and I was wondering if rethinkdbdash shares that old behavior.
If so, can it be implemented that a cursor return regardless if event is available? It appears to be causing an issue in my server, where I cannot close a cursor unless an event occurred. If a client refreshes without an event being triggered, it would double up on responses.
(PS. @neumino I messaged you on slack and I think there's more context there if you want to check it out. )

Turns out there was an issue with how I was calling .map(...) with the cursor that was causing the issue!


Fix isSameEntityName

Fixes #38412
I misrefactored this utility function 8 months ago, resulting in the JS
export const moment = rollupMoment__default || moment;

being interpreted more like
export const moment = moment || anything;

when trying to decide if rollupMoment__default is ever referenced. In JS, we don’t count these kinds of conditional assignment as a usage, so it appeared that rollupMoment__default was never used at all, causing the import to be elided.

A tangential point is that the logic I meant probably doesn’t make sense for export variable statements anyway 🤷♂

The failing test is failing on master.


How to set maximum speaker for real time diarization?

[Enter feedback here]
How to set maximum speaker for real time diarization?
Document Details
⚠ Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

ID: 1b3ffa53-4033-103c-9988-7a28be2cfc91
Version Independent ID: 97c4b578-c048-f2aa-7abb-d06d805d1c34
Content: Real-time diarization quickstart - Speech service - Azure AI services
Content Source: articles/ai-services/speech-service/get-started-stt-diarization.md
Service: azure-ai-speech
GitHub Login: @eric-urban
Microsoft Alias: eur

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

@u7630991
To set the maximum number of speakers for real-time diarization in the Microsoft Speech Service, you can use the SpeakerRecognitionConfig class in the Speech SDK. Specifically, you can set the NumberOfChannels property of the SpeakerRecognitionConfig object to the maximum number of speakers that you want to support.
Here is an example of how to set the maximum number of speakers to 2 in C#:
// Create a SpeakerRecognitionConfig object with the maximum number of speakers set to 2
var config = SpeechConfig.FromSubscription("YourSubscriptionKey", "YourServiceRegion");
config.SpeechRecognitionLanguage = "en-US";
config.EnableDiarization = true;
config.DiarizationNumberOfSpeakers = 2;
// Create a SpeechRecognizer object with the SpeakerRecognitionConfig
using (var recognizer = new SpeechRecognizer(config))
{
// Start recognition and wait for a result
var result = await recognizer.RecognizeOnceAsync();
Console.WriteLine(result.Text);
}
In this example, the DiarizationNumberOfSpeakers property of the SpeakerRecognitionConfig object is set to 2, which means that the real-time diarization will attempt to identify up to 2 speakers in the audio stream.

@u7630991
I'd recommend working closer with our support team via an [Azure support request] (https://docs.microsoft.com/en-us/azure/azure-portal/supportability/how-to-create-azure-support-request). We'll follow up there.


Support flexfec, ulpfec, and any other non-video video codecs.

Relevant components:

[ ] Signalling server
[ ] Common library
[x] Frontend library
[ ] Frontend UI library
[ ] Matchmaker
[ ] Platform scripts
[ ] SFU

Problem statement:
The current code pruned the list of supported codecs to only include h.264, vpx, av1, but this meant error correction codecs like flexfec, ulpfec, and so on were removed from the browsers sdp (even though they are supported!).
Solution
This PR does not prune any codecs, but instead puts our preferred codec at the top and then adds all the other supported codecs.
Documentation
N/A
Test Plan and Compatibility
This has been tested on my machine and @gingernaz's machine and flexfec successfully is shown in the supported codecs and WebRTC stats using this PR.
This PR makes https://github.com/EpicGamesExt/PixelStreamingInfrastructure/pull/46 redundant.

Manually cherry-picked into 5.3 and 5.4
5.5 will get merged from master.


python3 compat

python3 doesn’t have exec
else kudos for your futureproof style!

Thanks for the PR!
Are you not seeing the following error when you run python -m mpl_style_gallery:

NameError: name 'np' is not defined

I think this issue is described in this stackoverflow answer. Basically, if the second and third arguments to exec are different objects, then global variables inside of exec'ed modules are not visible inside functions. By default, those arguments are different (globals() and locals() dicts). If those two arguments are the same object, which is the case when only the second argument is provided (see second paragraph of docs), then this behaves as expected.
In other words, I think exec(COMMAND) should become exec(COMMAND, {}). That way the empty dictionary is used as the locals and globals dict for the exec'd module. For some reason, I wasn't able to come up with a simple example that reproduced the issue, so I may be missing something.

done! does it work for you now?

I'm surprised that you don't see the same issue, but I've checked this out on both python 2.7 and 3.4, so I'm happy to sign off.
Thanks for the contribution!

well, i didn’t try with legacy python, so if the error was only there, i didn’t see it.


[mle] avoid link dead lock

The current MLE Link Request handler would result in dead lock if two routers sends Link Request to each other at very close time points, when either node tracks the other node in LinkRequest state, so rejects the request until timeout.
This commit changes the handler so that a node will accept the Link Request if the peer's router id is smaller than its own router id even if the peer is in LinkRequest state.

An alternative is both respond with Link Accept but not Link Accept and Request, and the bidirectional link is established.


bootstrap-datetimepicker range setting using mvc5 viewbag

This repo is no longer actively monitor or supported. All future work is being done to https://github.com/tempusdominus/bootstrap-3

This is a non issue please ask this at StackOverflow


/signups?campaigns=362&runs=z returns signup for campaign run nid = 0

BUG
Current Behavior

/signsups?campaigns=362&runs=z returns a signup with the run nid of 0

Desired Behavior

/signsups?campaigns=362&runs=z should return an empty data array

Steps to Replicate

Hit www.dosomething.org/api/v2/signsups?campaigns=362&runs=z

Why This Matters

Return value is not what we expect and gives incorrect data.

We originally reverted this code when we saw this issue on Thor but decided to re-add it so it isn't a blocker for Gladiator (right now, this bug shouldn't block anything for Gladiator).

Working PR open to check to make sure param is a number, like we expect: https://github.com/DoSomething/phoenix/pull/6367

Relevant Screenshots + Links

cc @aaronschachter @deadlybutter - let me know if I missed anything in the notes!

Closed with #6367


How many images are required for sufficent training?

Hello, from what I understand, this model has DETR components, so does it make it data hungry, like other transformer models, or smaller amount of images, requireed for CNN-based models, such as YOLO, with which this model is compared, is enough?

ofc it depends on your task, you need to experiment but yeah, as generally with Transformer-based models, you need more than some hundreds of samples. If you have a small datasets, first try with the nano config that builds a model of 2M-3M params

Thanks, got it


3.x: 403 errors for moved asset files

I am trying to implement 301 redirects for files stored in assets. I got into this issue when we rebuilt a site and discovered that we get a lot of 403 'not allowed' errors for files in the assets directory that have moved.
When a folder exists, but a file in that folder doesn't, you get a 404 error (e.g. www.domain.tld/assets/folder/nofile.pdf), as expected.
But when you hit an URL to a file in a non-existing folder, you get a 403 (e.g. www.domain.tld/non-existing/no-file.pdf).
The 403 errors are caused by ‘Deny from all’ in the assets/.htaccess file. This seems to have priority over any redirects added to the main .htaccess file for files that have changed path. It looks like the ‘Deny from all’ prohibits apache from checking whether the file actually exists.
To fix that, I have removed the global ‘Deny from all’ and changed the regex to the following using a negation:
<FilesMatch ".(!(?i:html|htm|xhtml|js|css|bmp|png|gif|jpg|jpeg|ico|pcx|...|jar|xml|pdf|gpx|kml))$">
Deny from all
</FilesMatch>

Is that safe to use? Should that be changed in core?
Also, any 404 handlers in SS (e.g. misdirection, etc) don't work for asset files because of
RewriteCond %{REQUEST_FILENAME} !-f

in the root .htaccess file.
If I add
RewriteCond %{REQUEST_URI} ^assets/ [NC,OR]

right before that line in the main .htaccess file, this works, but of course it also means that every request to a file goes through the whole SS controller stuff, which makes the site slower. With this the 404 handlers also works for asset files.

Is that safe to use? Should that be changed in core?

We probably aren't going to do any major changes to asset routing in 3.x; Given that 4.x has an entirely new asset system, we're likely to focus our efforts on optimising this.
Users are advised to, as you have done, manually modify and optimise your sites' root and assets .htaccess file in a way appropriate for your project.
If you can find a way to resolve the directory name issue causing 403 instead of 404 then we could PR it to the installer, but the fix you've tested seems to turn all disallowed file extensions into 404s instead of 403s (e.g. /assets/not-a-folder/program.exe -> 404).

Hi @tractorcow. Thanks. My fix is only turning requests to non existing files to 404 (e.g. your example of /assets/not-a-folder/program.exe), which is correct. If the file exists and the extension isn't allowed, you get a 403 (e.g. /assets/existing-folder/existing-file.exe).

Hm, I'd gotten a 404 not a 403. Perhaps I had not replicated your .htaccess correctly? Can you put it into a gist verbatim for me and I'll test it.

I'll just temporarily open this... not confirming this is a bug yet just want to finish investigation. :)

Thanks. Here the gist: https://gist.github.com/xini/502d988e7ccf1140d9bebd23be4be169

Oh, with this .htaccess I'm able to download .exe files without a 403.
Non-existing .exe queries are reported as 404.
Back to the drawing board?

I'm on apache 2.4 by the way.

Is there a problem with returning 404 for forbidden extensions? Isn't it a type of disclosure vulnerability otherwise?

It doesn't seem to be consistent either way... if the path itself is forbidden, it's not a disclosure. If the specific file is forbidden, yes it is a disclosure. Thus all forbidden extensions should be 403 regardless of whether they exist.

I'm marking this as a bug, for now, but I'm not 100% certain what the state of this bug is. It feels low priority, so we can come back to this and investigate it later.
As it stands, there is inconsistent behaviour in reporting of 403 / 404 for missing / disallowed assets in 3.x.

@tractorcow you're right, my .htaccess didn't work. sorry.
I think showing a 404 for everything except the allowed files would be preferable.
I have been digging into this, but I can't find a solution to not showing a 403 if a file doesn't exist.
I have come up with an update to the regex using a negative lookbehind (based on https://stackoverflow.com/questions/12926604/inverted-filesmatch):
Allow from all
<FilesMatch "(?<!i:\.html|\.htm|\.xhtml|\.js|\.css|\.bmp|\.png|\.gif|\.jpg|\.jpeg|\.ico|\.pcx|\.tif|\.tiff|\.au|\.mid|\.midi|\.mpa|\.mp3|\.ogg|\.m4a|\.ra|\.wma|\.wav|\.cda|\.avi|\.mpg|\.mpeg|\.asf|\.wmv|\.m4v|\.mov|\.mkv|\.mp4|\.ogv|\.webm|\.swf|\.flv|\.ram|\.rm|\.doc|\.docx|\.dotx|\.dotm|\.txt|\.rtf|\.xls|\.xlsx|\.xltx|\.xltm|\.pages|\.ppt|\.pptx|\.potx|\.potm|\.pps|\.csv|\.cab|\.arj|\.tar|\.zip|\.zipx|\.sit|\.sitx|\.gz|\.tgz|\.bz2|\.ace|\.arc|\.pkg|\.dmg|\.hqx|\.jar|\.xml|\.pdf|\.gpx|\.kml)$">
	Deny from all
</FilesMatch>

But it still shows a 403 for all files except the existing allowed ones. Where is my mistake in this one? This should work, shouldn't it?

If you don't want to show 403 then you can't use deny, pretty much.
It's ok to show 403, as long as it skips the file-exists check.

yeah, but how do I do that? I need redirects in the main htaccess to work for non-existing files, but the redirects don't work because of the deny.

For your app, remove the filesmatch, and use exclusively rewriterules to serve either 200, redirect, or manual 403 for disallowed extensions.

RewriteRule "\.exe" "-" [F]

See http://httpd.apache.org/docs/current/rewrite/flags.html#flag_f

Here a different solution for the '/assets.htaccess' file:
ErrorDocument 404 /error-404.html
ErrorDocument 500 /error-500.html

Allow from all
<FilesMatch "(?<!i:\.html|\.htm|\.xhtml|\.js|\.css|\.bmp|\.png|\.gif|\.jpg|\.jpeg|\.ico|\.pcx|\.tif|\.tiff|\.au|\.mid|\.midi|\.mpa|\.mp3|\.ogg|\.m4a|\.ra|\.wma|\.wav|\.cda|\.avi|\.mpg|\.mpeg|\.asf|\.wmv|\.m4v|\.mov|\.mkv|\.mp4|\.ogv|\.webm|\.swf|\.flv|\.ram|\.rm|\.doc|\.docx|\.dotx|\.dotm|\.txt|\.rtf|\.xls|\.xlsx|\.xltx|\.xltm|\.pages|\.ppt|\.pptx|\.potx|\.potm|\.pps|\.csv|\.cab|\.arj|\.tar|\.zip|\.zipx|\.sit|\.sitx|\.gz|\.tgz|\.bz2|\.ace|\.arc|\.pkg|\.dmg|\.hqx|\.jar|\.xml|\.pdf|\.gpx|\.kml)$">
	RewriteRule .* - [R=404,NC,L]
</FilesMatch>

It still uses a white list, it shows a 404 for everything except existing allowed files, and 404 handlers in the CMS like misdirection work too. This would prevent any information leak for existing but disallowed files.
Unfortunately I had to re-add the error documents, otherwise it shows the ugly default 404 page from Apache. Does anyone see a different solution for this?