Method not found: 'System.String DotNetNuke.Services.Url.FriendlyUrl.FriendlyUrlProvider.FriendlyUrl(DotNetNuke.Entities.Tabs.TabInfo, System.String, System.String, DotNetNuke.Entities.Portals.PortalSettings)'

Description of bug
9.4.2 has broken all of our modules due to Method not found: 'System.String DotNetNuke.Services.Url.FriendlyUrl.FriendlyUrlProvider.FriendlyUrl(DotNetNuke.Entities.Tabs.TabInfo, System.String, System.String, DotNetNuke.Entities.Portals.PortalSettings)'.
Error log
2019-11-13 10:05:23.285+01:00 [WIN-SVOFJIQAQ17][D:8][T:9][FATAL] DotNetNuke.Framework.PageBase - An error has occurred while loading page.
System.MissingMethodException: Method not found: 'System.String DotNetNuke.Services.Url.FriendlyUrl.FriendlyUrlProvider.FriendlyUrl(DotNetNuke.Entities.Tabs.TabInfo, System.String, System.String, DotNetNuke.Entities.Portals.PortalSettings)'.
at Mandeeps.DNN.Libraries.URL.Factories.URLFactory.GetFriendlyURL(Nullable1 ModuleID, PortalSettings pS, String QueryParameters, URLEntity u)    at Mandeeps.DNN.Modules.LiveArticles.Components.LiveArticlesController.GetArticleLink(ArticleVersion article, VersionPage vPage)    at Mandeeps.DNN.Modules.LiveArticles.Components.LiveArticlesController.ReplaceArticleTokens(ArticleVersion Article, String Template, Int32 UserId, Dictionary2& RenderControlsDefinition, Boolean ThumbnailReplace, Int32 CurrentPageId, Widget wt, Boolean UseStoredProcedure, Boolean IsAjaxPreview)
at Mandeeps.DNN.Modules.LiveArticles.Components.LiveArticlesController.RenderRecentArticles(Widget widget, Int32 UserId, Dictionary`2& RenderControlsDefinition)
at Mandeeps.DNN.Modules.LiveArticles.UI.LiveArticlesWidget.View.Initialize()
at System.Web.UI.Control.OnLoad(EventArgs e)
at DotNetNuke.Entities.Modules.PortalModuleBase.OnLoad(EventArgs e)
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Control.LoadRecursive()
at System.Web.UI.Page.ProcessRequestMain(Boolean includeStagesBeforeAsyncPoint, Boolean includeStagesAfterAsyncPoint)
Additional context
See https://github.com/dnnsoftware/Dnn.Platform/blob/8151da229c803ea82cb7956d3c36bb322d4634ed/DNN Platform/Library/Services/Url/FriendlyUrl/FriendlyUrlProvider.cs#L50
Affected version

[ x ] 9.4.2

Affected browser

[x] Chrome
[x] Firefox
[x] Safari
[x] Internet Explorer
[x] Edge

A suggested solution is to revert this in 9.4.3?

Ok, so if I understand correctly, in #3160 the overload was modified to use the IPortalSettings abstraction instead of PortalSettings, but from the comments on there, PortalSettings does Inherit from IPortalSettings already and for that reason this was not expected to be a breaking change. It would be really nice to get some testing from module vendors before/during the RC stages to spot these issues before releases.

@valadas Mariette did reported this issue about 24 hours prior to official release of DNN 9.4.2. Our team confirmed the issue and we were still gathering more information regarding the cause; however, official release was rolled out the same day.
Perhaps, we can have designated minimum days after RC before release?

Really? We have been discussing PR #3160 since mid-october including possible breaking changes. The RC has been published on November 5th, the release on November 12th and this issue reported today... Did I miss something?

Perhaps, I'm missing the boat. We don't individually track PR requests or individual issues. We only test our products when RC is released. Considering we have over 60+ products; that's a significant undertaking and takes some resources.
I simply suggested that we should have a defined minimum days prior to release so we're aware that if an RC is released on 1st; it will not be officially released until 7th or 15th. Just a suggestion... I'm not stepping on your toe.
Issue was originally reported by Mariette on Slack. We were working with her and we did point out to her that this is a DNN issue. DNN 9.4.2 was released while we were investigating.
Again, my post here was to suggest to avoid this in future. But if suggestions are not welcome then I'm very sorry.

No, no worries, just trying to figure this out because I did not see that reported before final went out and it was not expected to be a breaking change. That being said it would be nice to have more module vendors follow the progress of the platform and chime in on those types of risks on their end... I think nobody makes breaking changes on purpose and we try to test as much as possible but it has been a couple times that module vendors report issues right after the official release. We had 2-3 weeks RC stages and it happened too...
9.4.0 was in RC stage from July 19th and got out on September 11th and issues with some modules where reported one or two days after the official release... I am not blaming anyone but looking into solutions and I believe the RC period is not the major issue, it is more to get some module vendors to follow on ideas and upcoming changes...

I agree. We can commit to testing for each RC; but we do need a minimum defined RC period. We only had 4 business days between Nov 5th and 12th; it's extremely hard to re-allocate resources to test 60+ products within such a short span.
We can discuss this further in our next Tech Advisory meeting.

but from the comments on there, PortalSettings does Inherit from IPortalSettings already and for that reason this was not expected to be a breaking change.

@valadas My understanding is that the breaking change is caused due to introduction of IPortalSettings in DotNetNuke.Abstractions.dll

we do need a minimum defined RC period

This is what was in the roadmap though https://dnncommunity.org/road-map it only got 1 day later than planned for both the RC and the release...

introduction of IPortalSettings in DotNetNuke.Abstractions.dll

Yes it appears this is the cause, reverting the change might be a big thing, maybe we can find a way to have some sort of alias to it or an additional overload deprecated for a future release or some such...

Mariette did reported this issue about 24 hours prior to official release of DNN 9.4.2

This is the part I did not get, knowing it before the release, for sure would have been a good reason to not publish it yet, but I did not see that issue before the release myself.

This was the first version I was involved in testing and the result shows. I am happy with this though more time to correct major issues like the one we see now is really needed. I had a slow start testing because it was the first time, next time will be better and quicker.

I will try to test these modules on one of my test sites as well to help with these.  My beta sites do not have much on it currently because I start them from scratch a lot.
I will load up 2sxc for those users and Mandeeps collections.  I can't say I will be there for every RC but I will try to help here as they both seem to have been hit by updates.  I can be an extra voice when I can.

This would be good to put as Known Issues to help stop others now.

Yeah, test early, test often :) Any testing hands are welcome specially regarding upgrades and third party modules.

@valadas Can you add this to the release notes please?

Sure can

Done

Thank you @valadas

I can confirm also the same errors using Ventrian News Articles.
https://github.com/ventrian/News-Articles/issues/42

Maybe we can push out a 9.4.3 RC fix soon with the fix and start a 9.4.4 milestone.
I have a  PR I would like to see working that are in 9.4.3 and one I will put together today that was my first accepted in 10 but should be in 9 I will redo right now.

We certainly have a better setup now for quick release cycles. Now the thing is how do we resolve this issue? @ahoefling @mitchelsellers opinions? Add an overload marked deprecated for Dnn10? Or do you guys have better ideas to handle this?

I can confirm also the same errors using Ventrian News Articles.
ventrian/News-Articles#42

I will add this module as well.
Thanks @b-creative @marietteknap for stepping up here with testing efforts for the community... YOUR AWESOME!

I'm working on a PR that will add back the overloads but mark them as deprecated.  Still testing that it fixes the issue and doesn't cause compilation issues going forward (e.g. ambiguous overload)

Merged, will be out with 9.4.3 soon...


[BUG] Unable to set alert profile due to [illegal_state_exception] in Opensearch-Dashboard 2.0

Describe the bug
Whenever I am trying to create a new alert profile in opensearch-dashboard 2.0, then I am getting [illegal_state_exception]. This error appears when you try to create a "action" in your alert profile.
To Reproduce
Steps to reproduce the behavior:

Go to 'Alerting'
Click on 'Monitors' and then click the "Create monitor" button
Next fill the "Monitor Details", "Data Source" & "Tigger"
When you start filling the details at the "Action" block and when you press on create then this error occurs.

Expected behavior
It should save the alert profile
OpenSearch Version
OpenSearch 2.0
Dashboards Version
Opensearch-dashboard 2.0
Plugins
I am using default plugins out of the box
Screenshots

Host/Environment (please complete the following information):

OS: CentOS

@kavilla - Thanks for acknowledging my issue. Is there any workaround ?

@atifafsar I found that you can manually create them using the API. Dashboards seems to be sending an invalid request body.

@atifafsar I found that you can manually create them using the API. Dashboards seems to be sending an invalid request body.

Thanks @DogukanL, Will try that.

@atifafsar We have identified the issue and currently are working on a fix. More details can be found on issue #254. I will also be closing it in favor of #254 as that will be the main issue to track this problem.


List-like union as a struct to save on allocations

This PR converts the list-like union via a class hierarchy to a struct for the sole purpose of saving GC allocations. There is no other benefit. In fact, I don't expect there will ever be more than one allocation saved per list and per invocation of an operator so the savings are really pathetic. The implementation also feels a bit clumsy but there is not much choice there.

I believe the performance benefit has not been demonstrated to offset the more awkward code.

Benchmark code:
[ParamsSource(nameof(Sources))]
public IEnumerable<int> Source { get; set; }

public IEnumerable<IEnumerable<int>> Sources
{
    get
    {
        var xs = Enumerable.Range(1, 1_000_000);
        yield return xs;
        yield return xs.ToArray();
    }
}

[Benchmark]
public void CountDown()
{
    Source.CountDown(1, (_, _) => 0).Consume();
}

Benchmarks for CountDown using last release (3.3.2):
BenchmarkDotNet=v0.13.2, OS=Windows 11 (10.0.22621.1105)
Intel Core i7-1065G7 CPU 1.30GHz, 1 CPU, 8 logical and 4 physical cores
.NET SDK=7.0.102
  [Host] : .NET 7.0.2 (<IP_ADDRESS>605), X86 RyuJIT AVX2

Method
Source
Mean
Error
StdDev
Allocated

CountDown
Int32[1000000]
22.20 ms
0.507 ms
1.488 ms
119 B

CountDown
Syste(...)rator [36]
20.09 ms
0.479 ms
1.413 ms
179 B

Notice that the run with an array as a source is slower.
Following is the benchmark for changes in this PR up to 064fb6151586c2e467920662678c1dc8e0231f47:
BenchmarkDotNet=v0.13.2, OS=Windows 11 (10.0.22621.1105)
Intel Core i7-1065G7 CPU 1.30GHz, 1 CPU, 8 logical and 4 physical cores
.NET SDK=7.0.102
  [Host] : .NET 7.0.2 (<IP_ADDRESS>605), X86 RyuJIT AVX2

Method
Source
Mean
Error
StdDev
Median
Allocated

CountDown
Int32[1000000]
18.76 ms
0.455 ms
1.334 ms
18.45 ms
115 B

CountDown
Syste(...)rator [36]
19.77 ms
0.477 ms
1.383 ms
19.20 ms
179 B

The run with an array as a source is slightly faster.

📎 benchmark-3.3.2.log.txt
📎 benchmark-pr-064fb615.log.txt

@atifaziz If you would, please compare your latest results to SuperLinq 4.6.0? I did a comparison against latest public release (3.3.2), and I show that SuperLinq is 40% faster on array and essentially equivalent (4% faster) on enumerables.
For CountDown specifically, it appears to be faster to use the corelib Reverse() operator than to do any allocation of our own.
// * Summary *

BenchmarkDotNet=v0.13.2, OS=Windows 11 (10.0.22623.1180)
Intel Core i7-1065G7 CPU 1.30GHz, 1 CPU, 8 logical and 4 physical cores
.NET SDK=7.0.102
  [Host] : .NET 7.0.2 (<IP_ADDRESS>605), X64 RyuJIT AVX2

|             Method |               Source |     Mean |    Error |   StdDev | Allocated |
|------------------- |--------------------- |---------:|---------:|---------:|----------:|
| SuperLinqCountDown |       Int32[1000000] | 10.30 ms | 0.187 ms | 0.166 ms |     138 B |
|  MoreLinqCountDown |       Int32[1000000] | 17.24 ms | 0.305 ms | 0.339 ms |     186 B |
| SuperLinqCountDown | Syste(...)rator [36] | 13.02 ms | 0.250 ms | 0.234 ms |     229 B |
|  MoreLinqCountDown | Syste(...)rator [36] | 13.58 ms | 0.240 ms | 0.224 ms |     237 B |

I had already removed ListLike<> from SuperLinq code, which allows for the possibility of comparison. I did a further comparison for all three methods (AggregateRight, CountDown, ScanRight). I did find a performance improvement for SuperLinq in ScanRight; after fixing ScanRight, I found that in only one case does the ListLike<> behavior improve performance:
// * Summary *

BenchmarkDotNet=v0.13.2, OS=Windows 11 (10.0.22623.1180)
Intel Core i7-1065G7 CPU 1.30GHz, 1 CPU, 8 logical and 4 physical cores
.NET SDK=7.0.102
  [Host] : .NET 7.0.2 (<IP_ADDRESS>605), X64 RyuJIT AVX2

Method
Source
Mean
Error
StdDev
Gen0
Gen1
Gen2
Allocated

SuperLinqCountDown
Int32[1000000]
10.055 ms
0.1166 ms
0.1090 ms
-
-
-
138 B

MoreLinqCountDown
Int32[1000000]
16.673 ms
0.2415 ms
0.2259 ms
-
-
-
186 B

SuperLinqScanRight
Int32[1000000]
12.884 ms
0.1779 ms
0.1577 ms
281.2500
281.2500
281.2500
4000642 B

MoreLinqScanRight
Int32[1000000]
13.821 ms
0.1599 ms
0.1335 ms
375.0000
375.0000
375.0000
4000912 B

SuperLinqAggregateRight
Int32[1000000]
6.466 ms
0.0895 ms
0.0837 ms
265.6250
265.6250
265.6250
4000500 B

MoreLinqAggregateRight
Int32[1000000]
5.326 ms
0.0596 ms
0.0557 ms
-
-
-
38 B

SuperLinqCountDown
Syste(...)rator [36]
10.958 ms
0.2174 ms
0.2232 ms
-
-
-
149 B

MoreLinqCountDown
Syste(...)rator [36]
13.263 ms
0.2488 ms
0.2206 ms
-
-
-
237 B

SuperLinqScanRight
Syste(...)rator [36]
15.848 ms
0.3051 ms
0.3858 ms
375.0000
375.0000
375.0000
8000878 B

MoreLinqScanRight
Syste(...)rator [36]
16.365 ms
0.3096 ms
0.3180 ms
343.7500
343.7500
343.7500
8000949 B

SuperLinqAggregateRight
Syste(...)rator [36]
6.637 ms
0.0931 ms
0.0871 ms
289.0625
289.0625
289.0625
4000552 B

MoreLinqAggregateRight
Syste(...)rator [36]
7.955 ms
0.1582 ms
0.1693 ms
328.1250
328.1250
328.1250
4000636 B

Data: https://gist.github.com/viceroypenguin/ec71e7adb8124c455e8a2925bee1b987

@atifaziz If you would, please compare your latest results to SuperLinq 4.6.0? I did a comparison against latest public release (3.3.2), and I show that SuperLinq is 40% faster on array and essentially equivalent (4% faster) on enumerables.
For CountDown specifically, it appears to be faster to use the collection foreach rather than use the list-like structure.

I shared those benchmarks to get back to the comment from @fsateler, where he rightly challenged that the performance & allocation benefits were not demonstrated (trade-off being code legibility). So this PR is not about CountDown, but since it uses ListLike<T> and scans the entire source, I wanted to use it as the basis of the demonstration. The (separate) problem with CountDown is that its loop is poorly written with respect to performance. It accesses the list Count property 3 times and twice during each iteration of the loop. Each access is a virtual dispatch so it probably adds up. If the list count is taken into a local variable like so:
diff --git a/MoreLinq/CountDown.cs b/MoreLinq/CountDown.cs
index f7b8ade1..16797781 100644
--- a/MoreLinq/CountDown.cs
+++ b/MoreLinq/CountDown.cs
@@ -65,15 +65,11 @@ static partial class MoreEnumerable

             IEnumerable<TResult> IterateList(ListLike<T> list)
             {
-                var countdown = Math.Min(count, list.Count);
+                var count = list.Count;
+                var countdown = Math.Min(count, count);

-                for (var i = 0; i < list.Count; i++)
-                {
-                    var cd = list.Count - i <= count
-                           ? --countdown
-                           : (int?) null;
-                    yield return resultSelector(list[i], cd);
-                }
+                for (var i = 0; i < count; i++)
+                    yield return resultSelector(list[i], count - i <= count ? --countdown : null);
             }

             IEnumerable<TResult> IterateCollection(int i)

then it makes a stark improvement:

Method
Source
Mean
Error
StdDev
Allocated

MoreLinqCountDown
Int32[1000000]
11.30 ms
0.244 ms
0.712 ms
96 B

SuperLinqCountDown
Int32[1000000]
13.24 ms
0.288 ms
0.849 ms
93 B

MoreLinqCountDown
Syste(...)rator [36]
19.77 ms
0.491 ms
1.433 ms
179 B

SuperLinqCountDown
Syste(...)rator [36]
18.70 ms
0.537 ms
1.582 ms
179 B

In fact, it's almost 15% faster than SuperLinq (which I'm guessing is just iterating the list via the enumerator).

For CountDown specifically, it appears to be faster to use the collection foreach rather than use the list-like structure.

So perhaps not? I haven't investigated in detail as to why, but if I were to take a somewhat educated guess, then I'd say iterating a sequence makes two virtual dispatches per loop iteration (MoveNext + Current) whereas with a list, there's just one, the indexer. Moreover, even if ListLike<T> indexer is coded for maximum probability of getting in-lined, it makes no discernible difference.

📎UserQuery-20230123-003056.log.txt


[CT-687] [Informational Test Severity] Idea.. Additional Test Severity Levels

Is this your first time opening an issue?

[X] I have read the expectations for open source contributors

Describe the Feature
Applies to DBTCloud most notably.
We've identified the need to mark some referential integrity testing as Informational rather than a Warn or Fail Severity.
All test results in our Production environment have been moved to a Warn severity to alleviate Job Failure notifications during schedule run times.  A failed execution should only apply to actual failures and not failed test results so this leaves us with one severity level of Warn remaining.
Warn is ideal for most tests and should be capable of triggering alerts from DBTCloud as well but we are running into scenarios where we know there's a test that will trigger a result set and count, but can't be fixed immediately or are more informational in nature.
Describe alternatives you've considered
Omitting the test from execution seems to be our only option at the moment while we create a ticket and address it but that leaves us with an inconsistent testing strategy when compared to other tests.
Who will this benefit?
DBTCloud users as well as scheduled production executions that are setup to trigger notifications from test results.
Are you interested in contributing this feature?
No
Anything else?
No response

Normally "warn" is the standard severity level to inform people that the test is failing but not derail the run. Have you looked at creating custom tests that will succeed instead of warn or fail?

@MDamon Thanks for sharing your idea! I agree with Gerda, I'm inclined to think that "informational tests" which still find failing rows, but shouldn't result in actual run failure, is exactly what the warn test severity is for. It sounds like you see a need for finer distinction:

Warn is ideal for most tests and should be capable of triggering alerts from DBTCloud as well but we are running into scenarios where we know there's a test that will trigger a result set and count, but can't be fixed immediately or are more informational in nature.

Could you share more about what you're thinking here? Is the concern around having too many tests with warnings, such that the signal is getting lost in the noise? Have you thought about using the conditional warn_if threshold (docs)?

+1 dbt Cloud user - they would like to have additional test severity levels so as to help differentiate between 'important' warns that shouldn't get in the way of the run versus 'less important' warns that shouldn't get in the way of the run. Today, they are getting 100s of warns in their runs, and it is hard to get through the noise and triage these based on the type of test failure. Possible workaround for them is to add meta tags to their models that they can pull in alongside their test status data to help filter their warnings in their warehouse.


update readme with details

add details

hi @victoryeo, thanks for your interest in contributing! Sorry about the lack of response. Do you mind rebasing to the latest and submit an update? The work in the github actions to perform the build and tests are especially useful. Note that there's a better way to generate the ZKP artifacts from the circuits by using npm run gen from inside zkp/circuits

Sure. i will close this PR.


PFQuery fromLocalDatastore() returns [Error]: The Internet connection appears to be offline.

Please consider this code:
if !Reachability.isConnectedToNetwork() {
    query.fromLocalDatastore()
}
query.findObjectsInBackgroundWithBlock { (objects, error) -> Void in

}

I'm expecting the objects to be received from the local datastore when the device is completely offline, but I'm getting this error instead of desired objects:
[Error]: The Internet connection appears to be offline. (Code: 100, Version: 1.11.0)

Does local datastore requires the Internet connection?

PS: I have called Parse.enableLocalDatastore()


[stable/grafana] Adds delete hook annotation to test pod

What this PR does / why we need it:
Adds an annotation for helm to delete the test pod after it finished successfully, as expected.
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] (not needed) Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

Will use helm test --cleanup


Edit documents

Codecov Report

Merging #311 (347d45f) into master (e4ee331) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #311   +/-   ##
=======================================
  Coverage   51.31%   51.31%
=======================================
  Files         123      123
  Lines       17586    17586
=======================================
  Hits         9024     9024
  Misses       8562     8562

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 e4ee331...347d45f. Read the comment docs.


Manually set status for Web Bluetooth

@ddbeck that'll be my next burndown list, and I left it out here knowing that it wouldn't have any computed status or compat keys.


Support for batching functionality of influxdb-java

This module should provide a way to enable the batching functionality of influxdb-java and gzip compression as well. It can be through configuration.

Great idea, any chance that you could provide a pull request?

Feature implemented with the latest release.


feat: masonry component

Implemented a homegrown masonry component (based on DeSandro's masonry layout that correctly works in React 18.
Fixes: https://github.com/manoldonev/react-todo-app-ts/issues/46

👍🏻


Embedded WebBrowser just like expo's

I have local html file to be rendered in an embedded webview
I was looking for the same look and feel as that of Expo's WebBrowser
https://user-images.githubusercontent.com/95462949/210719844-01b4f121-29bd-4e87-a142-71ef8953e10b.mp4

@SupriyaPKalghatgi I think you want this project instead: https://github.com/proyecto26/react-native-inappbrowser

@TheAlmightyBob Forgot to mention that, my project is purely Expo

@SupriyaPKalghatgi Ah, I missed that it's a local file. Then I think you'll need to build an equivalent of that screen yourself using your preferred navigation framework (I assume React Navigation).
Such a UI is outside the scope of this library. The API is there so you can implement forward/back buttons... unfortunately I don't know of a way to implement the text zoom control. And of course native Android has a different UI, dunno if that would be a concern.

@TheAlmightyBob I got this working with react navigation's modal presentation
https://reactnavigation.org/docs/modal


Should be able to customize the resources in the Helm Chart

For example the cpu, mem, etc

Do you mean for the Backstage pod? This should already be possible.
https://github.com/backstage/charts/blob/main/charts/backstage/values.yaml#L159
I think this issue comes from OCP Helm UI, where the OCP Form renderer is unable to render this specific field due to https://issues.redhat.com/browse/OCPBUGS-14875
I'll investigate further.

@christophe-f can you please clarify resources for which entity we're unable to configure at this moment? I guess it's postgresql, correct? The Backstage itself should be fine (in both form and yaml views). Postgresql is not configurable in Form view in OCP (however it is configurable via yaml)

Great, we will add that to the documentation regarding the resource attribute

Should be fixed upstream via: https://github.com/bitnami/charts/pull/21237 (+ propagated back)


feat: add strategy support for state archives

Motivation
Make the state-archive process more generic so we can extend in future.
Description

Make the state-archive process to generic
Add feature flag to the cli to select a strategy

Steps to test or reproduce

Run all tests

:tada: This PR is included in v1.23.0 :tada:


Chart changes ordering when X axis type is "Category"

I want the X values to be ordered like the result ordering.
Thanks

You can now disable X axis ordering via chart options.

:+1: thanks


fix: Use usize time for usize::from_le_bytes buffer

Fixes https://github.com/smartcorelib/smartcore/issues/87

I would appreciate @ronniec95 if you confirm me that this fixes the problem. I am going to check which images we can use to run the build tests with other architectures

Before merging this @VolodymyrOrlov I would like to add a way in the CI to check this tool for different architectures and OS, what do you think if we move to Github Actions?

@morenol I am fine as long as Github Actions supports all build steps.


Parsing error if the document starts with a dash

Hello! I get a NestedTextError exception when I try to parse an example from the documentation starting with a dash:
-
    cmd: emborg version
    expected: emborg version: \d+\.\d+(\.\d+(\.?\w+\d+)?)?  \(\d\d\d\d-\d\d-\d\d\)
    expected_type: regex
-
    cmd: emborg --quiet list
    expected: home-\d\d\d\d-\d\d-\d\dT\d\d:\d\d:\d\d
    expected_type: regex

My code:
import nestedtext as nt

n = nt.load('test.nt')

print(n)

I have found out experimentally that this error appears whenever the dash is the first character in the document. Even such a document causes this error:
- a
- b

But this one parses without errors:
a:
 - b
 - c

It looks like a bug.

By default, the top-level data structure is required to be a dictionary. If you want to allow lists or strings instead, use the top argument to nt.load():
$ cat list.nt
- a
- b
$ python
Python 3.10.0 (default, Oct 20 2021, 17:23:57) [Clang 12.0.1 ] on linux
Type "help", "copyright", "credits" or "license" for more information.
>>> import nestedtext as nt
>>> nt.load('list.nt', top='list')
['a', 'b']

That said, the error message should be much clearer about this, and the documentation example should probably mention this issue as well.

The representation of all parameters in the form of text, without casting to other types, is exactly what I was looking for

Us too, glad you like the format!


Add  markup in helptexts where applicable

It results in nicer styling in the project's admin page if an appropriate markup is used in the help text strings.
For example:

:+1:


publish 1.8.1: Exclude PMMs from 1Inch

Any reason the 1inch test is removed?

There is another file, oneInch.test.ts which covers 1Inch test.


Please change Immersive Controls for vivecraft from ✅ to ➖

they're not immersive, especially if you're left handed, but even then the right handed controls are just genuinely bad
hope mcxr will have better controls ;3

They've technically created immersive controls, they're just not good immersive controls. It happens, just have to deal with it until we have real immersive controls in MCXR. 😋


Add support for Streaming APIs

This PR adds support for defining streaming APIs.
It introduces a new root-level object called stream. It looks like this:
stream:
  framing:
    type: 'chunked'
    delimiter: '\r\n'
      read:
        - $ref: '#/components/messages/chatMessage'
        - $ref: '#/components/messages/heartbeat'

From now on, the topics object is not required if stream is present.
\cc @jkarneges @MikeRalphson @kinlane

I have the Streamdata.io reviewing to give me feedback. I'm modeling based upon a couple websocket and SSE APIs I'm working on. I'll have more thoughts shortly.

Next thing in the queue is WebSockets APIs support a.k.a. APIs that don't need or have the concept of topic and everything happens in the same channel.

Makes sense, consider how topics could be externally defined by 3rd parties, even if the topics don't directly feed into defining the feed in any way. aka Tagging. I find API providers just take for granted, but I still want to apply topics.

I'm merging it. However, feel free to leave feedback here.

@fmvilas Thank you for merging this PR, do you plan to update the documentation https://www.asyncapi.com/v1/guide/#Core-concepts to show & explain the stream root-level object ?

Yes, I'll be working on the website the next month.


Charts Shrinked

Hi everyone,
I'm facing a strange issue. Some graphs are shrinked. Do you have any idea about what can be happening? I noticed that after I load a vertialBarchart all the other chart will be shrinked. If I change to lineChart it's more consistent but still some times it happens

security issues were causing this behavior


Fix errors in CI about documentation

Signed-off-by: Sebastian Malton<EMAIL_ADDRESS>

How do you have docker set up?

I'm using the docker daemon from minikube

Following https://apple.stackexchange.com/a/439938 worked for me just now. Might be a configuration issue


Add colors param to SimplePrinter

SimplePrinter(colors: true)

Reason: color is easier than having to search for [E] ...

Wouldn't the PrettyPrinter be best suited for this purpose?

Perhaps. The problem is that I have limited console space and PrettyPrinter eats up a lot of it. It would make work easier to have the option to output single-line logs which would stand out clearly from the rest of the non-log output... even if it was a single hard-coded color other than gray.

Yesterday I've done quite a bit of research into colouring console output. The way I'm thinking about it now is that this would be a good feature to have. I want to implement it as follows:

detect if a terminal is attached and if that terminal supports ANSI escapes
iff so color the output. I'm planning to use the stdlib io for this an phase out our custom coloring code

So it would be an automatic process. I hope I have time for this end of this week/beginning of next week.


Local role to service provider role mapping support in IS 7.0.0

Is your suggestion related to an experience ? Please describe.
In prior versions of IS, we have supported local role to service provider role mappings. We need to analyze whether we need to support  local role to service provider role mappings or use application roles introduced with the new role v2 implementation instead of service provider role mappings.

Fix with : https://github.com/wso2/carbon-identity-framework/pull/5368 https://github.com/wso2-extensions/identity-inbound-auth-oauth/pull/2320

Fixed the issue where the sub-organization admins cannot login to console using SSO https://github.com/wso2/carbon-identity-framework/pull/5396/commits/82a79fe3bbc02b6116ff7bd59b1485ce01598731
Here, in OrganizationAuthenticator, roles claim is taken from the user attributes, and set it to the identity provider mapped user roles claim.


index-url missing in manifest-entry.xsd?

Shouldn't there be an index-url element in manifest-entry.xsd for the URL at which index endpoint is being served?

There should be!


WIP call vendor/configure and vendor/make to create libzmq.a

Signed-off-by: Axel Nennker<EMAIL_ADDRESS>I suggest to call zeromq's autogen.sh && configure && make to build libzmq.a
How to use this:

clone the repository
git clone --recursive https://github.com/AxelNennker/zeromq-src-rs
cd
cd zeromq-src-rs
Install Android Studio if you haven't
Please see https://developer.android.com/studio/
set environment (on Ubuntu)

export HOST_TAG=linux-x86_64
export NDK=$HOME/Android/Sdk/ndk-bundle

call cargo build
ANDROID_API=28 PATH=$NDK/toolchains/llvm/prebuilt/linux-x86_64/bin:$PATH CC=$NDK/toolchains/llvm/prebuilt/linux-x86_64/bin/aarch64-linux-android28-clang CXX=$NDK/toolchains/llvm/prebuilt/linux-x86_64/bin/aarch64-linux-android28-clang++ cargo build  --target=aarch64-linux-android
the resulting library is in vendor/src/.libs/libzmq.a

Possible improvements:

copy vendor code to OUT_DIR/vendor before configuring and building so that cargo clean removes everything built
tell crate using this crate where the new static library is

Hi Axel,
Could you explain to me what problem using cmake causes that using make directly would solve?
From what I recall you were unable to cross-compile for android using cmake. Why not open an issue for that?

Added WIP to the title of this PR.
I will follow openssl-rs's example and copy the source_dir to OUT_DIR
https://github.com/alexcrichton/openssl-src-rs/blob/master/src/lib.rs#L80
This way it gets removed when cargo clean is called.
Also integrate with testcrate again following openssl_rs's example.

I now build from the testcrate to see who the src-crate is used.
Sub-problem is now that the cloned git submodule is not on the stable branch and the configure script is complaining about that.
--- stderr
configure: WARNING: You are building an unreleased version of 0MQ and asciidoc or xmlto are not installed.

Is that something you could change for the git submodule?

Yes the master branch uses a unreleased version for zmq. The latest_release branch tracks the lastest zmq stable release.

I suggest to add the stable branch here as a submodule like described here.
https://stackoverflow.com/questions/1777854/how-can-i-specify-a-branch-tag-when-adding-a-git-submodule
Maybe even a fixed version (which would be needed to be updated from time to time)

Currently latest_release tracks the stable branch. However the stable branch is not a fixed version.
I'm not sure I understand your previous comment.

I think that the submodule should be 'stable' not master.
cd zeromq-src-rs/vendor
git checkout stable
cd ..
git add zeromq-src-rs
git commit -m "moved zeromq-src-rs to stable"
git push

This way the submodule directory 'vendor' contains the branch 'stable' when zeromq-src-rs is checked out.

The master branch mirrors libzmq's master branch. It is a preview branch.
The lastest_release mirros follows libzmq's lastest_release branch. This is a the stable branch.
Do you mean that you think either of these branches should be renamed?


Update links from individuals to IETF drafts

and a few typos and formatting

PR preview available at: https://stevelasker.github.io/scitt-web/


NIFI-7890 - Added record support to ConsumeMQTT processor

This also adds the option to define a delimiter instead of record reader/writer so that multiple messages can be appended into a single flow file. This greatly increases the performances of the processor when the use case requires high throughput in terms of events per second.
Thank you for submitting a contribution to Apache NiFi.
Please provide a short description of the PR here:
Description of PR
Enables X functionality; fixes bug NIFI-YYYY.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically main)?

[x] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[x] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[x] Have you written or updated unit tests to verify your changes?
[x] Have you verified that the full build is successful on JDK 8?
[x] Have you verified that the full build is successful on JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[x] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[x] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

Thanks for the improvement @pvillard31. I think this will give us MUCH better performance for any MQTT related use cases. @exceptionfactory had a few comments but otherwise I'm a +1.

Thanks for the improvement @pvillard31. I think this will give us MUCH better performance for any MQTT related use cases. @exceptionfactory had a few comments but otherwise I'm a +1.

Thanks for the review @exceptionfactory and @markap14 - I made some changes following your recommendations.

Thanks for the review @exceptionfactory and @markap14 - I made some changes following your recommendations.

@pvillard31 Thanks for the feedback and changes, looks good.

@pvillard31 Thanks for the feedback and changes, looks good.

Squashed and rebased against main to fix the conflicts caused by the 2 others PRs related to this processor being merged today.

Squashed and rebased against main to fix the conflicts caused by the 2 others PRs related to this processor being merged today.

Thanks for the review @turcsanyip - I think I addressed your comments, let me know if there is something else.

Thanks for the review @turcsanyip - I think I addressed your comments, let me know if there is something else.


Generate a constructed EEType for interfaces implemented by constructed types

These are visible to reflection: things visible to reflection need to be
considered fully constructed because we can't predict how the code will
use them.

@sergiy-k Another thing for the ASP.NET benchmark; metadata availability just falls out of having the constructed EEType there.


Resolving of Sentry failed

Platform:

[ ]  Dart
[x]  Flutter Android or iOS
[ ]  Flutter Web

IDE:

[x]  VSCode
[ ]  IntelliJ/AS
[ ]  XCode
[ ]  Other, which one?

split-debug-info and obfuscate (Flutter Android or iOS) or CanvasKit (Flutter Web):

[ ]  Enabled
[ ]  Disabled

Platform installed with:

[x] pub.dev
[ ] GitHub

Output of the command flutter doctor -v below:

Not relevant to this issue

The version of the SDK (See pubspec.lock):
6.12.1

I have the following issue:
Because app depends on sentry_flutter ^6.12.1 which depends on package_info_plus ^1.0.0, package_info_plus ^1.0.0 is required.
So, because app depends on package_info_plus ^2.0.0, version solving failed.
pub get failed (1; So, because app depends on package_info_plus ^2.0.0, version solving failed.)
The description goes here ...
Steps to reproduce:

add sentry 6.12.1
add package_info_plus 2.0.0

Actual result:

pub get fails

Expected result:

pub get succeeds

@vlazdra thanks for reporting.
You can use dependency_overrides for now.
dependency_overrides:
  package_info_plus: ^2.0.0

package_info_plus v2 only adds a new field, so it's compatible, it's just a dependency resolution issue.
v2 has just been released, we'll wait a bit more till upgrading it, otherwise we will get issues with people using v1 and our SDK v2.

@ueman that could be just a minor =D

Since it's compatible, you can change the dependency constraints in Sentry to
dependencies:
  package_info_plus: '>=1.0.0 <3.0.0'

I didn't make the choice to go for a major bump in package_info_plus :D


docs: added MIT licience badge in README.md

Description
#minor
Task Item

Screenshots

Coverage remained the same at 55.715% when pulling 919b4b5a4627a93eee32197a502b2dc4a6b633a9 on harsh1868:main into 846d18f1d2c817b01e12271f553a4f3e1f2909a3 on microsoft:main.


store images/containers on the host?

Upsides:

You don't have to re-download base images if you destroy and re-create the VM
You aren't limited by the VM's disk size

Downsides:

Destroying the VM is no longer enough to clean up everything -- images will still take up space on the host
There's likely a performance penalty

It looks like the layers are stored on a separate .vmdk in the boot2docker vm. It may be preferable to come up with a way to keep that around when upgrading the volume itself. (Optionally of course - there will be cases where you want to start completely fresh.)

Big upvote on this. I just found out doing a dinghy upgrade deleted all of my custom images and containers.
If the layers are stored on a .vmdk folder, can we just remote mount this on the host?
I'm wondering if this would also be a possible alternative: doing something that works like git stash. Where you can execute a dinghy stash which will copy the layer data to the host. Then do a dinghy stash apply which will copy them back after a dinghy upgrade is complete.

I'm not sure this is worth the effort anymore, now that we're using docker-machine and upgrades don't delete images and containers. The only reason would be to preserve stuff across a VM destroy/create, which is much less common.

Gonna close this out, not worth it anymore


Creation of Parallel Polyline at a certain distance

Please tell us your new feature idea
It would be interesting to be able to make a parallel line at a distance that we pass and that we can draw another line in reference to a polyline ... it is very interesting because it can be used for many things ... for example for agriculture ... parallel itineraries ..etc..
Why do you need it?
I am doing an agricultural GPS project with your library and I need to be able to draw parallel lines to mark where the tractor will go on the next line at a defined distance from the first line.
How helps other people?
Very useful to create parallel paths in many activities.

https://github.com/mapsplugin/cordova-plugin-googlemaps-doc/blob/master/v2.6.0/class/utilities/geometry/spherical/README.md

If I know those functions exist and I have tried by:
spherical.computeOffset ()
but this function does it on the same angle ... there is no possibility to draw a parallel line because the distance is on the same line, it is on the same latitude and the displacement should be on the longitude so that the line is parallel ..
I do not see how to do it and it is very important in my project.

I think spherical utility has enough functions to draw parallel lines.
Please do your best.

Solution:
//On our master line we calculate the heading:
var P_A = { lat: longuitud_AA, lng: latitud_AA, };
var P_B = { lat: longuitud_BB, lng: latitud_BB, };

var heading_calculado = plugin.google.maps.geometry.spherical.computeHeading(P_A, P_B);

//Calculate Lines Parallel
//Line 1
posicion1_a = plugin.google.maps.geometry.spherical.computeOffset(punto_origen_A, ancho_trabajo, heading_calculado);
posicion1_b = plugin.google.maps.geometry.spherical.computeOffset(punto_destino_B, ancho_trabajo, heading_calculado);
datos_linea_1 = [posicion1_a, posicion1_b];
linea1.setPoints(datos_linea_1);
//Line 2
posicion1__a = plugin.google.maps.geometry.spherical.computeOffset(punto_origen_A, -ancho_trabajo, heading_calculado);
posicion1__b = plugin.google.maps.geometry.spherical.computeOffset(punto_destino_B, -ancho_trabajo, heading_calculado);
datos_linea__1 = [posicion1__a, posicion1__b];
linea_1.setPoints(datos_linea__1);


Support for Received DMX Packet (Label=5)

Hi,
I'm building a DMX remote with an arduino. I found that a good part of labels were supported by the library. So it would be interesting to add the support of a DMX input to the existing code instead of starting from scratch.
Do you already have an idea of how to implement this feature?

Hi there! I have not yet implemented DMX input.
Input is a bit more complicated to deal with. The main difficulty is getting a fixed message rate, but this is difficult to do when other things are running on the Arduino, and different Arduino-compatibles are faster/slower more powerful/less powerful than others. Ideally this would be handled in a separate thread so that it is not hindered at all by the user-programmed main loop. This is theoretically possible on the Teensy, but then the library also needs to be made thread-safe, which would be a bit of work.
I should also note that implementing label 5 is only the first part. You would also need to implement labels 8 and 9. A simple driver or program may only rely on label 5, but label 9 (sending only changes) is much more efficient. However I have not tested that so I honestly do not know how widely used it is.
What I am working on right now are some performance improvements, but unfortunately I don't currently have access to the hardware needed to finish that.
You are welcome to work on adding that and I will help you as much as I can. I would definitely review and merge a pull that adds DMX input if you are able to get it to work reliably.
May I ask what specifically you are trying to do? Depending on what software and platforms you are trying to control there may be a much easier way to do this with other protocols and adapting methods. If you would rather it be wireless then it would definitely be easier with sACN and something like an esp8266.


4 more accurate ranking algorithm

Pull Request Test Coverage Report for Build 68

58 of 58 (100.0%) changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 64:
0.0%

Covered Lines:
812

Relevant Lines:
812

💛  - Coveralls

Pull Request Test Coverage Report for Build 68

58 of 58 (100.0%) changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 64:
0.0%

Covered Lines:
812

Relevant Lines:
812

💛  - Coveralls

Pull Request Test Coverage Report for Build 68

58 of 58 (100.0%) changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 64:
0.0%

Covered Lines:
812

Relevant Lines:
812

💛  - Coveralls

Pull Request Test Coverage Report for Build 68

58 of 58 (100.0%) changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 64:
0.0%

Covered Lines:
812

Relevant Lines:
812

💛  - Coveralls


Feature/cfa single partners cms

Description
This PR

Migrates Single partners page to CMS
Implements pagify for single page view

Fixes #582
Type of change
Please delete options that are not relevant.

[x] New feature (non-breaking change which adds functionality)

Screenshots

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation

@koechkevin Update the Partners block preview image

@koechkevin Update the Partners block preview image

https://codeforafrica-ui-pr-585.dev.codeforafrica.org/about

@koechkevin Update the Partners block preview image

https://codeforafrica-ui-pr-585.dev.codeforafrica.org/about

@Koech I meant the image preview in the CMS similar to https://github.com/CodeForAfrica/ui/blob/35caf0d5b6673873ede01130240aec2497469df4/apps/codeforafrica/src/payload/blocks/CustomPageHeader.js#L5

👍🏽
There is our partners on homepage and our partners on about page. How are we handling these?

@kilemensi My thinking is, users can add a block on each page and select from a list of partners(collection) to show.

👍🏽
There is our partners on homepage and our partners on about page. How are we handling these?

@kilemensi My thinking is, users can add a block on each page and select from a list of partners(collection) to show.

We have this function for that @koechkevin


[Bug]: $nin inclusion operator returns all documents when filtering metadata

What happened?
vectordb.get(where={"sha256":{"$nin":[sha256]}})['metadatas'][0]['sha256'] == sha256 returns True.
It seems that the $nin inclusion operator doesn't work correctly - it returns the entire collection. In the example above, sha256 is assigned the value of the sha256 metadata property of the first document in the collection - I would expect the returned documents to not include that document when filtering using the $nin operator.
Versions
Chroma v0.4.13, Python 3.11.5, MacOS 13.6
Relevant log output
No response

@avri-schneider, PR is up.


[image_picker] iOs GIF file Support

Hi, I cannot find any related discussion here, but is there any way to pick GIF file on iOs device? Mine always converted to JPEG.

Hello,
Do you mean selecting an image from the gallery(or Camera Roll, etc), right?

Yes, from the iOs gallery. Everytime I select the GIF file, it's always converted to JPEG. I ended up using file_picker library. But it would be nice to be able to select GIF file on iOs gallery using this library too.

I'm using 0.5.0,it's always converted to PNG. Hope fix it soon.

As workaround you can use the file_picker with the FileType.IMAGE flag.

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.


how to append a bigint number just like<PHONE_NUMBER>3812481

our user_id is very big , number type is bigint,like<PHONE_NUMBER>3812481
when i  excutor
R.APPENDINTARRAY test<PHONE_NUMBER>3812481
i  get is  R.GETINTARRAY test result is<PHONE_NUMBER>

Hello @sunyaf thank you for the bug report.
Indeed I believe this is being caused by an integer overflow, since<PHONE_NUMBER>3812481 % 2^32 =<PHONE_NUMBER>
I'll look into this and see if this limitation can be removed, otherwise I'll update the documentation to be explicit about this.

thank you，it would be soon to be happy.
because i will use it as soon as possible.
it would be very helpful to me.
thank you very much.

Hello @sunyaf thank you for the bug report.
Indeed I believe this is being caused by an integer overflow, since<PHONE_NUMBER>3812481 % 2^32 =<PHONE_NUMBER>
I'll look into this and see if this limitation can be removed, otherwise I'll update the documentation to be explicit about this.
thank you，it would be soon to be happy.
because i will use it as soon as possible.
it would be very helpful to me.
thank you very much.

Hello
I have confirmed that this issue is due to R.APPENDINTARRAY using internally (1, 2) 32-bit integer arrays, which is the default implementation of the CRoaring build.
From a quick look on their repo it seems that it is possible to create a 64-bit version of the library for C++, but I didn't find anything for C specifically, which could put us in trouble.
In any case, I think this issue won't be very easy to fix.
I'll probably just update the README for now to make it clearer that integers are capped at 32-bits.
I'll keep you updated.

Update: I have confirmed that CRoaring do not support 64-bit integers: https://github.com/RoaringBitmap/CRoaring/issues/232, https://github.com/RoaringBitmap/CRoaring/issues/1
We'll have to monitor their progress, so there isn't much we can do on this front for now.
I'll update the README.

Update: I have confirmed that CRoaring do not support 64-bit integers: RoaringBitmap/CRoaring#232, RoaringBitmap/CRoaring#1
We'll have to monitor their progress, so there isn't much we can do on this front for now.
I'll update the README.

i find   this
`
#include
#include "roaring.hh"
#include "roaring.c"
int main() {
Roaring r1;
for (uint32_t i = 100; i < 1000; i++) {
r1.add(i);
}
std::cout << "cardinality = " << r1.cardinality() << std::endl;
Roaring64Map r2;
for (uint64_t i =<PHONE_NUMBER>0000000100ull; i <<PHONE_NUMBER>0000001000ull; i++) {
r2.add(i);
}
std::cout << "cardinality = " << r2.cardinality() << std::endl;
return 0;
}
`
https://github.com/RoaringBitmap/CRoaring
you can search 'Roaring64Map'

The C library is 32-bit only but there is a 64-bit extension in C++.

i find this
`
#include
#include "roaring.hh"
#include "roaring.c"
int main() {
Roaring r1;
for (uint32_t i = 100; i < 1000; i++) {
r1.add(i);
}
std::cout << "cardinality = " << r1.cardinality() << std::endl;
Roaring64Map r2;
for (uint64_t i =<PHONE_NUMBER>0000000100ull; i <<PHONE_NUMBER>0000001000ull; i++) {
r2.add(i);
}
std::cout << "cardinality = " << r2.cardinality() << std::endl;
return 0;
}
`
https://github.com/RoaringBitmap/CRoaring
you can search 'Roaring64Map'

I think C can code with C++


Fixes: #462: Fix race condition in method sendResponse in com.android.identity.wallet.transfer.Communication

Fixes #462
This PR changes the method sendResponse in class Communication to only close the underlying mTransport after the deviceResponse has been fully sent.

Closing due to inactivity, please reopen if this is still an issue.


Still installs some static libs when BUILD_SHARED_LIBS=ON

1.8.0 installs the mix of static and shared libraries:
lib/libtdactor.a
lib/libtdapi.so
lib/libtdapi.so.1.8.0
lib/libtdclient.so
lib/libtdclient.so.1.8.0
lib/libtdcore.a
lib/libtddb.a
lib/libtdjson.so
lib/libtdjson.so.1.8.0
lib/libtdjson_private.a
lib/libtdjson_static.a
lib/libtdnet.a
lib/libtdsqlite.a
lib/libtdutils.a

The cmake's standard variable BUILD_SHARED_LIBS should be driving what kind of libraries is built, unless there is a good reason otherwise.

libtdjson_static is meant to be static. Other libraries are internal libraries and aren't supposed to be used directly.

Other libraries are internal libraries and aren't supposed to be used directly.

They shouldn't be installed then?

Likely, yes, but they don't harm.


[Bug report]:

Platform
Desktop
Version
1.5.1
Description
Updated my firefly wallet (desktop windows) to the latest version 1.5.1 and after a successfull update, the SHIMMER $SMR token which I earned as rewards (from staking IOTA) didn't appear in my holdings.  Only the ASSEMBLY $ASMB token appeared.
Expected behaviour
Was expecting ASSEMBLY $ASMB token and SHIMMER $SMR token , both of which were earned as rewards from STAKING IOTA, would still appear and be reflected in my holdings after the successful update of FIREFLY Desktop (windows) version 1.5.1 wallet.
Actual behaviour
Only the Assembly $ASMB token was reflected in my token holdings after the update.
Can the issue reliably be reproduced?
No
Steps to reproduce the issue
No response
Operating System
Windows
Diagnostics
na

Developer Profile
No
Profile Type
Software (Stronghold)
Staking
Previously staked
Crash Status
None of the above
Error Messages
na
Error Logs
na

Duplicate declaration

[X] I have searched the issues tracker this issue and there is none

Code of Conduct

[X] I agree to follow this project's Code of Conduct

Have you already tried to use the balance finder in the settings a few times in a row? I´m going to close this issue as it is a duplicate to https://github.com/iotaledger/firefly/issues/4016. Feel free to open the issue again if you have further problems.


ACM-3940 Fix Ansible icon

https://issues.redhat.com/browse/ACM-3940
As per product icons brand standards, the AAP logo should have a drop shadow when shown on a dark background.
https://www.redhat.com/en/about/brand/standards/icons/product-icons

/cc @Randy424

/test unit-tests-sonarcloud


Incorrect DB config, or could not connect to DB / PDOException: SQLSTATE[HY000] [2002] No such file or directory

Hello,

[X] I've read the Troubleshooting First Steps
[X] This request isn't a duplicate of an existing issue, opened or closed
[X] I've read the docs and followed them (if applicable)
[X] This is not a personal support request that should be posted on the YOURLS Discourse community

Describe the bug
When I try to install Yourls (after completed config.php and create manually the DB, and download the fr_FR.mo  and fr_FR.po and put them in languages directory, I have this message when I try to install the script:

Fatal error
Incorrect DB config, or could not connect to DB
PDOException: SQLSTATE[HY000] [2002] No such file or directory

I have switch the debug mode, but the message error is the same
define( 'YOURLS_DEBUG', true );
Thanks
//EDIT : oh, sure I can connect to phpMyAdmin with password and user to this database

Needs more logs. https://github.com/YOURLS/YOURLS/wiki/Troubleshooting-first-steps#check-server-logs
As a guess, following the instructions later in that troubleshooting wiki page you said you've read, I googled the error. Are you perchance using localhost as the DB server address?

Hello,
I have that in logs:

[Thu Oct 01 10:52:49.908623 2020] [autoindex:error] [pid 15285:tid<PHONE_NUMBER>40352] [client <IP_ADDRESS>:42356] AH01276: Cannot serve directory /home/clients/8b44057ae55374496743b90224a9/sites/domain.tld/: No matching DirectoryIndex (index.html,index.cgi,index.pl,index.php,index.xhtml,index.htm,index.php,index.php5,index.php4,index.php3,default.php,home.php,welcome.php) found, and server-generated directory index forbidden by Options directive

Yes, I use localhost

That log line looks unrelated.
Try other solutions on the question page I linked. This kind of troubleshooting is really outside the scope of YOURLS' issue tracker, so if you really can't figure it out I suggest starting a Discourse thread.

Ok, solved. I have use a automatic installation that my hoster gives.


Adds padding for access code in item view

Description
Fixes #1682
Notes (out of scope, known isues, hints for reviewing code, ...)  (optional)
...
Test cases

[ ] Case 1:

Given I am the usual user
When I go to this page
Then I see bottom padding for group code section

[auto-generated message] E2E test report on: https://algorea-static.s3.eu-central-1.amazonaws.com/branch/feature/add-padding-for-group-code-access/playwright-report/index.html

  1. Given I am the usual user
  2. When I go to [this page](https://dev.algorea.org/branch/feature/add-padding-for-group-code-access/en/a/1667525548838093512;p=4702,7528142386663912287,944619266928306927;pa=0)
  3. Then I see bottom padding for group code section

I see " This chapter has no content visible to you, so you can't validate it for now. " on that page.


Combine the 2 applescripts into a single script, or call the second from within the first when appropriate

Since the server config generator needs to be run on the computer running the server (to get the correct path to the root folder), and I would quite like this to be cross-platform to work on Windows system PCs or Linux Raspberry Pis, is it worth reworking that in JS / as an executable file?
NB need to work out how paths are formatted on other systems.

Perhaps continue with applescript for Mac and find another way to do it for other devices?


add more ci

continuation of #1

The status embed will only work if its on the default branch, so to test that, it has to be merged.


Consolidating Several PR's

Getting caught up on PR's, consolidating several into a single PR to avoid cascading CI checks.

https://github.com/Unidata/netcdf-c/pull/954
https://github.com/Unidata/netcdf-c/pull/953
https://github.com/Unidata/netcdf-c/pull/850
https://github.com/Unidata/netcdf-c/pull/957

Seeing a compiler fail under parallel; will push the fix shortly.

Seeing a downstream issue in netcdf-fortran, f03tst_open_mem failing.  Will open an issue over there since we are about to do a new release anyways, if necessary.  Will follow up with more info.
If the issue is non-trivial, will back out that particular PR.

The issue with netcdf-fortran is external to this consolidation, so merging this and then looking at fortran.


[WIP] Updated and fixed the C template

Checklist

[ ] I tested all changes and all related features with different users (student, tutor, instructor, admin) on the test server https://artemistest.ase.in.tum.de.
[ ] Server: I added multiple integration tests (Spring) related to the features (with a high test coverage)
[ ] Server: I added @PreAuthorize and check the course groups for all new REST Calls (security)
[ ] Server: I implemented the changes with a good performance and prevented too many database calls
[ ] Server: I documented the Java code using JavaDoc style.
[ ] Client: I added multiple integration tests (Jest) related to the features (with a high test coverage)
[ ] Client: I added authorities to all new routes and check the course groups for displaying navigation elements (links, buttons)
[ ] Client: I documented the TypeScript code using JSDoc style.
[ ] Client: I added multiple screenshots/screencasts of my UI changes
[ ] Client: I translated all the newly inserted strings into German and English

Motivation and Context

The C template was using a quite outdated version of the tester and had some flaws related to potential vulnerabilities.
Description

I've updated the C template to fix those vulnerabilities and fixed those tests.
Steps for Testing

Create a new C exercises
The solution should build successfully

We might have to update the solution used within the K6 tests. I'll have a look at this.

@COM8 I invited you to get access to the Artemis Github repository.
Can you please recreate this PR based on a feature branch in the Artemis repo?
Then, you and we can test the changes on our test server.
With external PRs, this is not possible

@krusche ACK 👍.

Closed in favour of #2150.


Initial implementation of generate-google-map in Deno

Ref: appwrite/appwrite#1905
Execution log:

Generated file:

Kotlin In Appwrite
#kotlin #appwrite #android #cloud
Appwrite is an end-to-end backend server that is aiming to abstract the complexity of common, complex, and repetitive tasks required for building a modern app.
Appwrite is both cross-platform and technology agnostic, meaning it can run on any operating system, coding language, framework, or platform. Although Appwrite can easily fit the definition of serverless technology, it's designed to run well in multiple configurations.
It provides various libraries for major programming languages and platforms so you don't have to write code for interacting with our HTTP APIs from scratch.
Appwrite has divided SDKs into two categories
Client SDKs
Server SDKs
Client SDKs
Web SDK
Flutter SDK
Apple SDK
Android SDK
Server SDKs
Kotlin SDK
Dart SDK
Swift SDK
Python SDK
Ruby SDK and etc
Here we will be seeing how to use kotlin in Appwrite functions
So what is Kotlin?
Kotlin is a cross-platform, statically typed, general-purpose programming language with type inference.
Kotlin is designed to interoperate fully with Java, and the JVM version of Kotlin's standard library depends on the Java Class Library, but type inference allows its syntax to be more concise.
So how to write kotlin in appwrite function and for this install Kotlin SDK by following the below instructions
GRADLE
Appwrite's Kotlin SDK is hosted on Maven Central. In order to fetch the Appwrite SDK, add this to your root-level build.gradle(.kts) file:
repositories {
mavenCentral()
}
Next, add the dependency to your project's build.gradle(.kts) file:
dependencies {
implementation("io.appwrite:sdk-for-kotlin:")
}
MAVEN
Add this to your pom.xml

io.appwrite
sdk-for-kotlin
0.1.1

Add the function
You can add a new function from your Appwrite project's dashboard. Access your Functions settings from your project's left navigation panel. Click the Add Function button and choose your function name and code runtime.
Init your SDK
Initialize your SDK with your Appwrite server API endpoint and project ID which can be found in your project settings page and your new API secret Key project API keys section.
This can be found out when you install Appwrite according to the Operating System and access the Appwrite Console UI
import io.appwrite.Client
val client = Client()
client
.setEndpoint('https://[HOSTNAME_OR_IP]/v1') // Your API Endpoint
.setProject('5df5acd0d48c2') // Your project ID
.setKey('919c2d18fb5d4...a2ae413da83346ad2') // Your secret API key
Make your First Request
import io.appwrite.services.Users
val users = Users(client);
val res = users.create(email =<EMAIL_ADDRESS>password = 'password');
println(res.body?.string())
As of now, we have seen how to set up the functions and Kotlin SDK.
Now, we will be seeing different operations which can be performed on these functions.
Create functions : To create a new function with help of Kotlin
import io.appwrite.Client
import io.appwrite.services.Functions
suspend fun main() {
val client = Client(context)
.setEndpoint("https://[HOSTNAME_OR_IP]/v1") // Your API Endpoint
.setProject("5df5acd0d48c2") // Your project ID
.setKey("919c2d18fb5d4...a2ae413da83346ad2") // Your secret API key
val functions = Functions(client)
val response = functions.create(
    name = "[NAME]",
    execute = listOf(),
    runtime = "dotnet-5.0",
)
val json = response.body?.string()

}
List functions : To get a list of all the project's functions. You can use the query params to filter your results.
import io.appwrite.Client
import io.appwrite.services.Functions
suspend fun main() {
val client = Client(context)
.setEndpoint("https://[HOSTNAME_OR_IP]/v1") // Your API Endpoint .setProject("5df5acd0d48c2") // Your project ID .setKey("919c2d18fb5d4...a2ae413da83346ad2") // Your secret API key
val functions = Functions(client)
val response = functions.list( )
val json = response.body?.string() }
Get function : It will get a function by its unique ID.
val response = functions.get( functionId = "[FUNCTION_ID]" )
Update function : It will update the function by its unique ID.
val response = functions.update(
functionId = "[FUNCTION_ID]",
name = "[NAME]",
execute = listOf(),
)
Delete function : It will delete a function by its unique ID.
val response = functions.delete( functionId = "[FUNCTION_ID]" )
Create execution : Trigger a function execution. The returned object will return you the current execution status. You can ping the Get Execution endpoint to get updates on the current execution status. Once this endpoint is called, your function execution process will start asynchronously.
val response = functions.createExecution( functionId = "[FUNCTION_ID]", )
So this is the basic idea of how to create functions in the Appwrite using the Kotlin SDK.
For more you can always refer to appwrite docs
Thank You


Add code to display thumbnails of gallery associated

Closes #293

Moved documentation to the target folder

UT fails because of w3c_validator

Failure seems to be with 'error: line 140: Bad value “/Supported Plugins/photogallery.png” for attribute “src” on element “img”: Illegal character in path segment: space is not allowed.'

Asking for advice on https://github.com/getpelican/pelican/issues/2581

Alternatively we could rename the folders in our repo to contain no spaces until this is handled as it should want me to raise issue against repo to rename them?

"Fixed" the issue by moving the image to another folder /images/ ready for review @talha131


[Web] No intellisense or colorization from modules in extra paths

repro steps:

Go to: https://github.dev/vritant24/pylance-intellisense-repro
open project-name/main.py

expected: see intellisense and be able to go to definition on class Dog in
dog = Dog()

actual: No intellisense

Some additional notes:

I am able to see intellisense for the same version of the python extension in insiders.vscode.dev but not on vscode.dev
The intellisense breaks if there is a space in the provided extra path. eg: in the above case, if the project-directory folder is changed to project directory

Currently pylance is not able to read files in the workspace from the browser so the intellisense and imports are not supported. @rchiodo is working on an enhancement to support virtual workspaces. The reason it works for you in the insiders version is likely due to the experiment we have enabled, which sets the "python.analysis.enableSyncServer" to true.


[Event Request] codeunit 7324 "Whse.-Activity-Post" - OnPostOutputLineOnAfterInitItemJournalLine

local procedure PostOutputLine(ProdOrder: Record "Production Order"; ProdOrderLine: Record "Prod. Order Line")
var
    ItemJnlLine: Record "Item Journal Line";
    ItemJnlPostLine: Codeunit "Item Jnl.-Post Line";
    ReservProdOrderLine: Codeunit "Prod. Order Line-Reserve";
begin
    with TempWhseActivLine do begin
        ItemJnlLine.Init();
        OnPostOutputLineOnAfterInitItemJournalLine(ItemJnlLine, SourceCodeSetup); //A108B0045.Event19.05
        ItemJnlLine.Validate("Entry Type", ItemJnlLine."Entry Type"::Output);

[IntegrationEvent(false, false)]
local procedure OnPostOutputLineOnAfterInitItemJournalLine(var ItemJournalLine: Record "Item Journal Line"; SourceCodeSetup: Record "Source Code Setup")
begin
    //A108B0045.Event19.05
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release.
We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues.
If you have follow-up questions or requests, please create a new issue where you reference this one.


Add related content for oversight reports

The Section 215 report is done, but the other three need their placeholder text replaced with actuals. This will involve building out the dummy press-releases.yml file and the associated markdown in the _press folder, as well as the as-yet-uncreated fact-sheets data. (Might want to make fact sheets a less specific data type like related-documents?)

I've done a bunch of this in #42, but because that PR did things by migrating content over from the legacy site in an automated way, the structured data might not be in an ideal format for you. Hopefully it should be easier than manually transferring stuff over from the legacy site's HTML, though!


display the indexes in the string reprs

With the flexible indexes refactor indexes have become much more important, which means we should include them in the reprs of DataArray and Dataset objects.
This is a initial attempt, covering only the string reprs, with a few unanswered questions:

how do we format indexes? Do we delegate to their __repr__ or some other method?
should we skip PandasIndex and PandasMultiIndex?
how do we present indexes that wrap multiple columns? At the moment, they are duplicated (see also the discussion in #6392)
what do we do with the index marker in the coords repr?

(also, how do we best test this?)

[ ] Tests added
[ ] User visible changes (including notable bug fixes) are documented in whats-new.rst

A few thoughts:

how do we format indexes? Do we delegate to their repr or some other method?

Like for variable data, Xarray indexes could implement _repr_inline_ and __repr__ to have both a summarized and detailed representation. they could also have a _repr_html_ (for fancy representation of complex indexes).

should we skip PandasIndex and PandasMultiIndex?

I'd skip it for the plain text DataArray / Dataset reprs (not a good information / verbosity ratio), but I'd keep it for the html reprs (the index section could be collapsed by default) as well as for the Indexes repr.  We could also provide a display option for more control on this (e.g., a display_default_indexes option set to False by default).

how do we present indexes that wrap multiple columns? At the moment, they are duplicated

Assuming that all coordinates related to a given index are shown next to each other, we could render the inline repr for the 1st coordinate and then use a short symbol (e.g., --, or a unicode symbol?) below that means "it's the same index".

what do we do with the index marker in the coords repr?

I think we can keep it as-is. It helps to identify at a glance which coordinates are indexed and which aren't. And it's still relevant if we skip PandasIndex and PandasMultiIndex in the plain text DataArray / Dataset reprs.

Shall we point this at the main branch instead?

From #6867 (https://github.com/pydata/xarray/issues/6867#issuecomment-1202535745), we might want to update the "Dimensions without coordinates" line too.

should we skip PandasIndex and PandasMultiIndex?

If we're encouraging people to look at these / subclass them to create custom indexes, then they should have a repr too.

I think we should move forward here.
Are there any real blockers?

not really, I wanted to wait until set_xindex was in main (the PR has been merged last week) and have not looked at it since.
There's two issues left that might need a bit of discussion: in 8f21df3 I skipped displaying default PandasIndex instances because that's basically redundant with the * on coordinates (and it would have required me to update a lot of doctests, which will probably make this a breaking change). If we decide to revert that, should we mark every coordinate with an index with a *?
Another proposal we had was to replace the "dimensions without coordinates" line with a "coordinates without index" line. @benbovy, this might be a misunderstanding on my part, but I thought "dimension coordinates" (and in particular their indexes) are still used for alignment? If so, I think we might need both lines.

Looks good to me @keewis. Thanks for your work on the indexes repr!
Yes I think we can skip displaying default indexes for now... The question is which indexes are considered as default, i.e., all PandasIndex and PandasMultiIndex instances (like in this PR) or just the single pandas indexes automatically created for the dimension coordinates? We can decide this later, though, it's not a problem adding more indexes in the text repr later (we'll probably need it when dropping the multi-index dimension coordinate with tuple elements). For the html repr it's easier: we could display all indexes and collapse the section by default.

but I thought "dimension coordinates" (and in particular their indexes) are still used for alignment?

Yes that's a good point. Let's keep "dimensions without coordinates".


[Storage] Bump versions for upcoming GA release

API change check
API changes are not detected in this pull request.


svg_loader: define the scaling factors for units changing

The values in absolute units are calculated based on the pixel unit.
Till now the default value of 90dpi was used. The necessary constants
are kept, but the new ones for 96dpi are defined as well and used by default.

Neat code :) I just have questions.
Are there any cases that 90DPI needed? what cases is it used?
If it is necessary, shouldn't it be defined in the common header? (PX_PER_IN)

@mgrudzinska ping?

Neat code :) I just have questions. Are there any cases that 90DPI needed? what cases is it used? If it is necessary, shouldn't it be defined in the common header? (PX_PER_IN)

As I understand it, the values of 90 or 72 were used in CSS1 and CSS2, but now the units are related to each other by the fixed ratio and any other values used to switch between the units can be considered as obsolete. So since CSS defines 1 inch to be equal to 96 pixels, I modified the code and kept only the 96 value.
Nevertheless please read the text below, maybe I misunderstood something.
From CSS1 (https://www.w3.org/TR/CSS1/#units):
"Pixel units, as used in the last rule, are relative to the resolution of the canvas, i.e. most often a computer display. If the pixel density of the output device is very different from that of a typical computer display, the UA should rescale pixel values. The suggested reference pixel is the visual angle of one pixel on a device with a pixel density of 90dpi and a distance from the reader of an arm's length. For a nominal arm's length of 28 inches, the visual angle is about 0.0227 degrees."
But since CSS3 (the most recent version: https://www.w3.org/TR/css-values-4/#absolute-lengths):
"Note: This definition of the pixel unit and the physical units differs from the earlier editions of CSS1 and CSS2. In particular, in previous versions of CSS the pixel unit and the physical units were not related by a fixed ratio: the physical units were always tied to their physical measurements while the pixel unit would vary to most closely match the reference pixel. (This unfortunate change was made because too much existing content relies on the assumption of 96dpi, and breaking that assumption broke the content.)"


Training

Hi, could you provide more detail about how you put together the dataset and trained the model?
I am interested in trying to train it, but am confused about the structure of the dataset.

Nevermind, I had somehow missed the info at the very bottom to download your actual dataset. It's starting to make sense now.
I am putting together a new dataset to try to enhance the model even further. Would you be interested in working with me to train it?

@SkyTNT would you be able to explain a bit more about how the dataset works? I have finally gotten around to downloading the full thing and exploring the code.
My understanding is that the dataset is generated on the fly by compositing the BG and FG images with some augmentations. Really quite clever! But, if that is the case, then what is the purpose of the imgs & masks folders? Those masks are of extremely poor quality, so I'm not sure what they are supposed to be for.

@SkyTNT would you be able to explain a bit more about how the dataset works? I have finally gotten around to downloading the full thing and exploring the code.
My understanding is that the dataset is generated on the fly by compositing the BG and FG images with some augmentations. Really quite clever! But, if that is the case, then what is the purpose of the imgs & masks folders? Those masks are of extremely poor quality, so I'm not sure what they are supposed to be for.

The imgs & masks folder contains manually labeled images. But when I try to train using only the generated dataset, the accuracy of the model drops. I guess the generated datasets deviate from the real ones, so I kept them. According to my training, convert to sketch data augmentation seems to be very helpful to improve the accuracy. But my convert to sketch implementation should still need to be improved to make it closer to the real sketch. I also found that compositing and data augmentation at low resolutions can make image detail look unnatural. But doing it at high resolution will be slow. If your cpu is powerful enough, you can increase the resolution of DatasetGenerator, and add RescalePad(image_size) at transform_generator.
https://github.com/SkyTNT/anime-segmentation/blob/c98f338ce402a1e7bfccb93b66c9626c741b28c5/data_loader.py#L272-L273

Thanks, that is helpful to know.
I will experiment with the augmentations, and I think I can put together an alternate dataset of the manually labeled images.
Currently I have been able to execute the training code, but I am running into a problem with export.py. RuntimeError: Error(s) in loading state_dict for AnimeSegmentation
Do you know what might cause this?

Can you provide more detailed information?

Oh, it was my mistake. I didn't realize the training was 2 steps (gt encoder, then the 2nd training), so I did not train long enough to be able to export.

After updating pytorch version to cuda 11.7, now I get an error when trying to train:
  File "D:\Downloads\anime-segmentation\train.py", line 113, in training_step     loss0, loss = self.net.compute_loss(loss_args) TypeError: ISNetGTEncoder.compute_loss() missing 1 required positional argument: 'targets'
Is it a bug with using cuda 11.7 pytorch? Shall I need to downgrade?

Thank you, it had changed somehow, so I was able to download the file again and it works. I am not sure what happened, but maybe I accidentally copied the file from the original DIS repo.
If I want to write the generated dataset images to files on my disk, can you recommend where would be the best place to insert the code for that? I would like to see the images so I can tweak it more easily.

You can insert the code at
https://github.com/SkyTNT/anime-segmentation/blob/2373527d745755fbe2987ba146d9326fed8e8881/dataset_generator.py#L311-L312
with argparse.ArgumentParser

A few more questions, if you don't mind.

How many epochs did you have to train the model? Both the gt model and the standard model.
Is it possible to train the gt model separately?  I assume I could train using --net isnet_gt, and then next use --net isnet? How would I tell the isnet which gt model to use?
My goal is to train a model that is optimized for real anime video frames, at 1080p resolution. Do you think its better to leave the model --img-size at 1024? Or should I use 1080? (And I assume I would need to change some of the code, such as in dataset_generator.py to accommodate the changed resolution)

Thanks!

Gt model only needs few epochs to train. But standard model needs hundreds or even thousands epoch to train.
Training the gt model is just the first training stage. It will never chage after standard model starts training. So you don't need to train them separately.
1080 should be ok. And the augmentations in dataset_generator is auto adapted to resolution. But i'm not sure it is right. You can check the output of dataloade by dataloader_test.py ot test.py.


Backend data changes, but table does not react to data change

Hi,
Have an issue trying to implement a search function (on various fields) and have the table automatically reduce to that searched list.
Using Vue3 sfc component (and ts).
My setup is
const searchTerm = ref('')

linked to an input
<input v-model="searchTerm" />

Setup data and add some random entries
// Setup Reactive data
const data = reactive([])
// Load Random data
for (let i = 0; i < 126; i++) {
    data.push({
        id: i,
        email: faker.internet.email(), // Using Faker.js
        name: faker.name.findName(),
        address: faker.address.streetAddress(),
    })
}

created a computed value
const searchedData = computed(() => {
    return data.filter(
        x =>
            x.email.toLowerCase().includes(searchTerm.value.toLowerCase()) ||
            x.name.toLowerCase().includes(searchTerm.value.toLowerCase())
    )
})

Then return this from the setup()
return {
    searchTerm,searchedData,
}

And back in the template I watch the data

    Data: {{searchedData}}

So, when I enter say "hotm" (or any search), the displayed data gets filtered as expected.
Yet the table does not update.
However, the table does update (reduced to the searched value list) when I change page or sort the columns (i.e. when the table components emits a do-search)
I actually was able to get it working (feels like a cludge) by using a watch then update "table.rows", as order and sort pageSize values are exposed so I was able to reslice from 0 to pageSize.
But as I said, feels a little hackish...
watch(searchedData, () => {
    table.rows = searchedData.value
        .sort(dynamicSort(table.sortable.order, table.sortable.sort))
        .slice(0, table.pageSize)
    table.totalRecordCount = searchedData.value.length
})

Any suggestions or workarounds?
!!! Feature Request !!!
Would be good to include a search function as part of the component.
Much like how you implement the sort function but have the search field emit an event.
PS: Would it make more sense to have the table sorting emit a do-sort event, and the a new search field emit a do-search event.
Many thanks in advance.
Cheers.
PS: for reference my dynamicSort is
const dynamicSort = (property, order) => {
    let sort_order = 1
    if (order === "desc") { sort_order = -1 }
    return function (a, b){
        if (a[property]  b[property]) {
            return sort_order
        } else {
            return 0
        }
    }
}

Hi,
Have an issue trying to implement a search function (on various fields) and have the table automatically reduce to that searched list.
Using Vue3 sfc component (and ts).

Hi @wattsie
I added new featrue for your request.
Has sample code on here
Template:

Static mode sample

    SearchBy:<input v-model="searchTerm" />

setup() :
    const searchTerm = ref("");
    const data = reactive([]);
    for (let i = 0; i < 126; i++) {
      data.push({
        id: i,
        name: "TEST" + i,
        email: "test" + i +<EMAIL_ADDRESS>      });
    }

    const table = reactive({
      columns: [
        // something...
      ],
      rows: computed(() => {
        return data.filter(
          (x) =>
            x.email.toLowerCase().includes(searchTerm.value.toLowerCase()) ||
            x.name.toLowerCase().includes(searchTerm.value.toLowerCase())
        );
      }),
      totalRecordCount: computed(() => {
        return table.rows.length;
      }),
      sortable: {
        order: "id",
        sort: "asc",
      },
    });

    return {
      searchTerm,
      table,
      updateCheckedRows,
    };

Hope this feature can help you.
If you have any question, let me know. ;)

@linmasahiro great job adding the search functionality, is the search functionality now part of the entire package cos I can't seem to see it action on the docs

@linmasahiro great job adding the search functionality, is the search functionality now part of the entire package cos I can't seem to see it action on the docs

@olufemioyedepo
If you want to check actions and demo, you can watch this page.
https://linmasahiro.github.io/vue3-table-lite/dist/advanced.html#filter


Is this lib maintained still maintained ?

I see few issues not being tackled, latest release failing. Is this project still maintained ?
Any good alternative if so ?

@stychu what is failing?

Just checking coz I've seen reports in issues regarding some old dependencies which cause possible memory leaks

Yes, the inflight package which is a dependency of<EMAIL_ADDRESS>causes memory leaks - it would be good to get rid of that!

This is also being reported by
https://security.snyk.io/vuln/SNYK-JS-INFLIGHT-6095116
Missing Release of Resource after Effective Lifetime

@stychu
You raise a good question! Would you like to edit the title, which says "Is this lib maintained still maintained ?" which has got the word "maintained" in it once too much.


Fleet: Add notification icon to Conditions tab on git repo to alert for an error

Add a notification icon to the Conditions tab on the git repo details page, so that a user is alerted for the presence of any errors in the items belonging to that particular group (check red faded box on image, which refers to the tabs where icon should be added)

Test Environment:
Rancher version: v2.6-head 4df2214
Rancher cluster type:  HA

Testing:
Confirmed:
red icon with an exclamation point (so a user notices) appears on the "Conditions" tab if there is an error in conditions

Confirmed with @catherineluse and @gaktive to add release note label.


Error: app_not_configured_for_user when user has another session on Google

I'm using SAML to validate into Google Apps.  Everything works fine if user try to login only to our GSuite, but if their have another session instance (like their personal Gmail account) it reloads to 403 page error.  How can we force to AddSession from the login request?

The error that you experienced is from Gsuite, not from php-saml.
It should be an issue with Gsuite and how it handle multiple session.


"Cannot read property 'match' of undefined" on ng build --prod after updating to 6.0.0-rc.*

Versions
Angular CLI: 6.0.0-rc.2
Node: 8.11.1
OS: win32 x64
Angular: 6.0.0-rc.3
... animations, common, compiler, compiler-cli, core, forms
... http, platform-browser, platform-browser-dynamic, router

@angular/cdk: 6.0.0-rc.1
@angular/cli: 6.0.0-rc.2
@angular/flex-layout: 5.0.0-beta.14
@angular/material-moment-adapter: 6.0.0-rc.1
@angular/material: 6.0.0-rc.1
@angular-devkit/architect: 0.5.4
@angular-devkit/build-angular: 0.5.4
@angular-devkit/core: 0.5.4
@angular-devkit/schematics: 0.5.4
@ngtools/json-schema: 1.1.0
@schematics/angular: 0.5.4
@schematics/update: 0.5.4
typescript: 2.7.2
webpack: 4.5.0

Repro steps

I'm not sure if the steps below are necessary to see the bug happening in Windows 10 system. These are the steps I did on my machine, and since 6.0.0-rc.0 I cannot perform ng build --prod.

Migrate a project from earlier versions to 6.0.0-rc.0  (installing cli also globally)
ng update @angular/cli --migrate-only --from=1.7.1
upgrade cli to 6.0.0-rc.1 and then 6.0.0-rc.2
Try to ng build --prod

If it helps, the project I used in the tests: https://github.com/julianobrasil/material2-test
Observed behavior
Cannot read property 'match' of undefined
TypeError: Cannot read property 'match' of undefined
    at Object.normalize (E:\******\node_modules\@angular-devkit\build-angular\node_modules\@angular-devkit\core\src\virtual-fs\path.js:178:14)
    at options.fileReplacements.forEach (E:\******\node_modules\@angular-devkit\build-angular\src\browser\index.js:99:55)
    at Array.forEach (<anonymous>)
    at BrowserBuilder.buildWebpackConfig (E:\******\node_modules\@angular-devkit\build-angular\src\browser\index.js:98:34)
    at Observable.rxjs_1.Observable.obs [as _subscribe] (E:\******\node_modules\@angular-devkit\build-angular\src\browser\index.js:31:38)
    at Observable.subscribe (E:\******\node_modules\rxjs\internal\Observable.js:162:69)
    at E:\******\node_modules\rxjs\internal\util\subscribeTo.js:22:31
    at Object.subscribeToResult (E:\******\node_modules\rxjs\internal\util\subscribeToResult.js:7:45)
    at MergeMapSubscriber._innerSub (E:\******\node_modules\rxjs\internal\operators\mergeMap.js:132:38)
    at MergeMapSubscriber._tryNext (E:\******\node_modules\rxjs\internal\operators\mergeMap.js:129:14)
    at MergeMapSubscriber._next (E:\******\node_modules\rxjs\internal\operators\mergeMap.js:112:18)
    at MergeMapSubscriber.Subscriber.next (E:\******\node_modules\rxjs\internal\Subscriber.js:103:18)
    at MergeMapSubscriber.notifyNext (E:\******\node_modules\rxjs\internal\operators\mergeMap.js:141:26)
    at InnerSubscriber._next (E:\******\node_modules\rxjs\internal\InnerSubscriber.js:30:21)
    at InnerSubscriber.Subscriber.next (E:\******\node_modules\rxjs\internal\Subscriber.js:103:18)
    at MergeMapSubscriber.notifyNext (E:\******\node_modules\rxjs\internal\operators\mergeMap.js:141:26)

Desired behavior

Run ng build --prod without errors
Mention any other details that might be useful (optional)
As the error message mentions rxjs a lot, I think it's worth to tell my rxjs versions:
"rxjs": "6.0.0-rc.1"
"rxjs-compat": "6.0.0-rc.1"

Heya, I believe you ran into https://github.com/angular/angular-cli/issues/10141. It was fixed for upgrading projects in rc.1 (or maybe 2).

I had the exact same issue.  Seems fixed in rc.2.  Just create new from the CLI rc.2 and reintegrate your files.


Suggestions: More cyber mobs with mob themed augments.

For a few examples, Cyberware Enderman have augments that allow you to blink around like teleportation wands. Cyberware Skeletons have augments that give you regeneration when standing under a tree, a bonemeal aura around you, etc. Cyberware Spiders have augments that let you climb walls, deal poison damage when hitting with fists, etc. Cyberware Zombie Pigman have augments that give you fire resistance, strength, etc.

More Cybermobs were definitely planned - but the specialized augments are something I hadn't thought of. It's a cool idea, I think I'll try to implement something like that.

Having it based on what the mob does makes little sense, considering that the mobs obviously don't need the augments in order to do any of the things listed. However, randomly giving mobs things like a fire breathing augment or one that accelerates healing in low light levels but also causes burning in the sun would be interesting.


There is no trailing slash in the Redirect URI field in the OIDC Configuration page

Describe the bug
If you leave the default value or copy the suggested Redirect URI value, Azure will give a Redirect URI error mismatch after creating App Registration.
To Reproduce
Add OIDC settings for Firezone and leave the Redirect URI field as default, i.e. without filling it.
Expected behavior
Add a slash to the default value for Redirect URI at the end. Also add it to the description field below.
Screenshots / Logs

Platform (please complete the following information):

Firezone Version 0.7.20
OS: Linux
Distro: Ubuntu Server 22.04
Deployment method: Docker

Hey @kinseii. I know it's a bit confusing that you need to manually add a slash for Azure, but If we change the default it will break a lot of connections that are already working for all other providers, so I don't think that we will be tackling this.
On the upside, we do have it described in our Azure AD docs so maybe it will help people struggling with it: https://www.firezone.dev/docs/authenticate/oidc/azuread/#step-1-obtain-configuration-parameters.

@AndrewDryga
You described the slash only for adding App Registration. The OIDC configuration page itself says that you don't need to change it for most cases. This misled me and I spent several hours trying to find the reason.


Nested lists do not close

This one is possible related to #3. This issue isn't with list item closures but with closing nested list containers ul and ol. This leads to a behavior where onece yo have a nested list, the list never returns to the original level:

This image shows the original doc, note that “Security Event” returns to the previous level.

This is the HTML with the unclosed sub-list. The “Security Event” just continues on as there is no closing list element.

Could this bug please be prioritized? Markdown lists are generated with good fidelity, but not HTML lists. Thanks.
(Alternatively, I'm happy to try to fix it - is the source code for gd2md-html available somewhere?)

Latest version (1.0β16) updated Feb. 28, 2019 supports simple nested lists properly in HTML.  See https://github.com/evbacher/gd2md-html/wiki/Demo-(HTML-conversion)#lists for an example HTML conversion.
For nested list items that include paragraphs or codeblocks, we still have a problem. I will open another bug for that.

Thank you for the fix!


orchestra/daemon: add 'silent' option to signal()

Default to False (not silent) to preserve existing behavior.
Signed-off-by: Sage Weil<EMAIL_ADDRESS>

Refer to this link for build results (access rights to CI server needed):
http://jenkins.ceph.com//job/teuthology-pull-requests/1739/
Test PASSed.