Static assets request path problems in preview6

Describe the bug
In preview 6 when there is a wwwroot folder in a blazor library project, the request path of server side and client side are different.
To Reproduce
Steps to reproduce the behavior:

dotnet new a blazor library project with a name that contains ..
Rename the content folder to wwwroot to make use of the new static assets feature add in preview6.
Change the references in the csproj file of the library project to use wwwroot instead of content (There would even not be any proper url path to the assets with client side hosting if this step is skipped. Question: Why doesn't blazor client side use the new wwwroot support? )
dotnet new a blazor client side and a blazor server side project that reference the library project.
Run and test both project.

Expected behavior
Static assets in both project can be accessed with the same url path.
Actual behavior
The file "background.png" is accessed with different url path:
Client side hosting: https://<hostname>/_content/<TheOriginal.LibraryName>/background.png
Server side hosting: https://<hostname>/_content/<lowercaselibrarynamewithoutdots>/background.png
Consequence
A blazor library with static assets cannot have a name with ., otherwise it would have different behavior when referenced by client side and server side hostings.
Is there any plan to fix this fault in the futural preview, or some quick fix for the current version?

I am also facing this issue with dot library application. The wwwroot assets are not loaded in server-side Blazor application.

The application is working as expected while renaming the library app into RazorLib1. Could you please provide some quick fix for this issue.?

I am also facing this issue with dot name library application. The wwwroot assets are not loaded in server-side Blazor application.

The application is working as expected while renaming the library app into RazorLib1. Could you please provide some quick fix for this issue.?

You can access by replacing the  Test.RazorLib1 to testrazorlib1 from the url and it should work on server side hosting, but it stops working on client side hosting. :-<

@AlseinX : Yes, it will work, but we need generic solution all the hosting models of Blazor.

@karthickthangasamy Me too.

Thanks for contacting us, @AlseinX.
@javiercn can you please look into this? Seems like something we'll need to normalize on.

It is expected that blazor client-side will also move to use _content/<> we just have been focusing on other work.

@javiercn

You can access by replacing the Test.RazorLib1 to testrazorlib1

Struggled with this as didn't know I had to replace the dotted project name with lowercase without dot.
AFAIK dots are allowed in the path part of a URL.
Also case is ignored in URLs
Whats the reason for this
Unlikely but if I had 2 projects of same name but only differing by dot this wopuld break.
Project.Test
ProjectTest

@mikes-gh dots are allowed, but normally cause issues, with many proxies and redirection rules, because people write simple rules like . to refer to files and that causes problems.
Urls are case-sensitive. Many static files implementations simply rely on the file system casing, that's why in some cases they can seem case insensitive, and you can choose to interpret them that way, but in HTTP they are case sensitive, if not, try to set a cookie path to /PATH and visit the site at /path and you'll see how the browser doesn't send the cookie.

@javiercn  thanks so is the plan to use lowercase no dots for both client and server.

@mikes-gh yes

@javiercn can I ask why the _content prefix for this feature?
It doesn't seem very intuitive to me..
Maybe the static path prefix could be made configurable in the csproj of the library
Whilst convention is good to keep code down in this case it may be less confusing to state the path particularly because the rule is a bit awkward.
Just my 2c and I am sure you have lots of other priorities :smile:

Another thought I had which may make more sence naming wise is to replace dots with hyphens.
Blazor.Lib.js becomes blazor-lib.js seems a bit better to me and looking at conventional js libraries this seems to be a  common pattern

@mikes-gh We use _content by default to clearly distinguish what's directly from your app with what comes through static web assets, as a safe way to avoid collisions and keep everything under a common subspace of the url address space.

@javiercn  any thoughts on replacing dots with hyphens instead of removing them
e.g.
_content/Blazor.Lib. becomes _content/blazor-lib

@javiercn

dots are allowed, but normally cause issues, with many proxies and redirection rules, because people write simple rules like . to refer to files and that causes problems.

But the main static asset for server side blazor contains a dot?
<script src="_framework/blazor.server.js"></script>

We are planing to go back to _content/$(PackageId) as the default convention.

@javiercn -  I agree with your desision.  At the moment the docs say _content/{LIBRARY NAME}/
Does LIBRARY NAME mean the csproj name?
I think its possible to use a RCL from a project reference that doesn't include a <PackageId> i.e. an RCL that has never been packed?  This might happen when testing.
Forgive me if I'm missing something just coming at this from an end-user perspective.

$(PackageId) is always defined, even if you don't pack your project. It will work whether you pack the project or reference it

Fixed in https://github.com/aspnet/AspNetCore-Tooling/commit/98b168c8947d8c7f7b3f2beb3a0561fb89cadfb5
This will be in preview7

@javiercn
And another problem is, making a blazor component library that supports client-side hosting still needs specifying the wwwroot folder instead of content (which is in the dotnet new blazorlib template) manually while the server-side support takes ``wwwroot` as default name convention.
<EmbeddedResource Include="wwwroot\**\*.js" LogicalName="blazor:js:%(RecursiveDir)%(Filename)%(Extension)" />

<EmbeddedResource Include="wwwroot\**\*.css" LogicalName="blazor:css:%(RecursiveDir)%(Filename)%(Extension)" />

<EmbeddedResource Include="wwwroot\**" Exclude="**\*.js;**\*.css" LogicalName="blazor:file:%(RecursiveDir)%(Filename)%(Extension)" />

dotnet blazorlib is going away in favor of simply using razor class library.
Regarding the embedded files I would have to take a look. It might have been something blazon specific that we didn’t remove.
Closing as we’ve taken action.

@javiercn
I am still confused that the razorclasslib template creates a library with .cshtml files while blazor uses .razor files. Does cshtml work also with blazor?

There is a flag for it. We simply default now to support pages and views, but the default will change.

Someone can help me how to make the Blazor SSR website work with a component project. The static files in content folder are not copied to wwwroot.

@minhhieugma  : Rename the content folder with wwwroot and generate the nuget package using dotnet pack.

nerate the nuget package using dotnet pack.

Thank guy, it works. Can you tell me how to include script/link automatically as blazor client side rendering can do? I would not like to manual add it!

@minhhieugma : I hope, for SSR we have to refer it manually like the below. Let's wait for others thoughts about including the assets automatically as like client-side application.
<script src="_content/LIBRARY_NAME/example.js


Only allow one WIFI connection in Config Mode

If WebConfig is accessed by more than one client, server must be able to resolve concurrent changes made by different clients OR only one client at a time must be allowed to connect.
Since WebConfig is relatively rarely used for device configuration by (single) user, limiting WIFI connections looks reasonable in this case.

WiFi connections limited to only one connection.

Problem
Consider the following scenario:

User owns two WiFi-enabled gadgets (e.g. Smartphone and Tablet) and both of them are connected to user's home WiFi only.
User switches on the IoT device, enters Config Mode and connects to device-established WiFi using Smartphone, sets up the configuration and switches the device to normal mode. Since device-established WiFi is no longer available, Smartphone automatically connects to user's home WiFi.
IoT device is switched to Config Mode while user's home WiFi is down. Smartphone automatically reconnects to device-established WiFi.
Since IoT device only allows one WiFi connection, user is now unable to connect to device-established WiFi using Tablet, and thus cannot modify parameters.

Solution
WiFi connection limit removed since version 0.6
Caveats

There is a hard-coded limit of 4 WiFi connection. If more than 4 different gadgets were used to set IoT device configuration, the same problem arises.
User has to make sure that no concurrent parameter modification are made.


[Dask.order] Memory usage regression for flox xarray reductions

Looks like I caught our first severe regression introduced by https://github.com/dask/dask/pull/10535
import xarray as xr
import dask.array as da
from dask.base import visualize
from dask.order import diagnostics
from dask.base import collections_to_dsk
import flox.xarray

dummy = xr.Dataset(
    {"foo": (("x", "y", "z"), da.ones((3, 10, 10), chunks=(1, 5, 5)))},
    {"x": ["a", "b", "c"], "y": range(10)},
)
result = flox.xarray.xarray_reduce(
    dummy,
    dummy.x,
    func="sum",
)
print(max(diagnostics(collections_to_dsk([result]))[1]))
visualize(
    result,
    filename='flox-reduction-order.png',
    color='order',
    optimize_graph=True,
)

Pre #10535
max_pressure: 6

Tasks are loaded as required and reducers are scheduled promptly. This is nice.
Main (w/ regression)
max_pressure: 8

We can see that dependents of root tasks 1 and 2 are scheduled greedily. This can become quite bad for very large graphs.

Raw graph for testing
a, b, c, d, e = abcde
    dsk = {
        (a, 0): 0,
        (a, 1): 0,
        (a, 2): 0,
        (b, 0, 0, 0): (f, (a, 0), (1, 5, 5)),
        (b, 0, 0, 1): (f, (a, 1), (1, 5, 5)),
        (b, 0, 0, 2): (f, (a, 2), (1, 5, 5)),
        (b, 0, 1, 0): (f, (a, 0), (1, 5, 5)),
        (b, 0, 1, 1): (f, (a, 1), (1, 5, 5)),
        (b, 0, 1, 2): (f, (a, 2), (1, 5, 5)),
        (b, 1, 0, 0): (f, (a, 0), (1, 5, 5)),
        (b, 1, 0, 1): (f, (a, 1), (1, 5, 5)),
        (b, 1, 0, 2): (f, (a, 2), (1, 5, 5)),
        (b, 1, 1, 0): (f, (a, 0), (1, 5, 5)),
        (b, 1, 1, 1): (f, (a, 1), (1, 5, 5)),
        (b, 1, 1, 2): (f, (a, 2), (1, 5, 5)),
        (c, 0, 0, 1): (f, [(b, 0, 1, 0), (b, 0, 1, 1), (b, 0, 1, 2)]),
        (c, 0, 1, 0): (f, [(b, 1, 0, 0), (b, 1, 0, 1), (b, 1, 0, 2)]),
        (c, 0, 1, 1): (f, [(b, 1, 1, 0), (b, 1, 1, 1), (b, 1, 1, 2)]),
        (c, 0, 0, 0): (f, [(b, 0, 0, 0), (b, 0, 0, 1), (b, 0, 0, 2)]),
        (d, 0, 0, 0): (c, 0, 0, 0),
        (d, 0, 0, 1): (c, 0, 0, 1),
        (d, 0, 1, 0): (c, 0, 1, 0),
        (d, 0, 1, 1): (c, 0, 1, 1),
    }

@dcherian most of the xarray graphs I've been looking at have this kind of pattern where there are data tasks that basically store an array. If I look at the raw graph of the above thing, I get something like

which are the root tasks in the above graphs

Those tasks are often the culprits that are throwing off dask.order and I wonder what they are, where they come from and if they can be avoided. I guess the fact that we're dealing here with an array of size one is since I'm running on a small toy example or is this a common thing? If that was true for large scale graphs, we should just inline this data and be rid of the more complex task graphs

we should just inline this data and be rid of the more complex task graphs

maybe I should just inline this kind of thing for the ordering part... I'll have to play with this idea a bit.
I still would like to learn more about the pattern that causes these kinds of graphs

cc @TomNicholas
Xarray calls dask.array.from_array on an array-like object. We could pass inline_array=True there. See https://github.com/pydata/xarray/pull/6566 . No one wants to make a decision on whether inline_array should be True or False by default. It would help if dask had a strong opinion here. @mrocklin has most context here, I believe

I caught our first severe regression introduced by https://github.com/dask/dask/pull/10535

This is also the major difference between examples that use random data, and examples that read files like a real workload.

No one wants to make a decision on whether inline_array should be True or False by default. It would help if dask had a strong opinion here.

Well, that depends on what that data actually is. I'm not familiar enough with xarray or zarr to make this decision for you.
Generally speaking, if one sticks to the best practice of storing any kind of sizable data remotely (i.e. anything beyond a couple MB) you should be better off with inlining. I strongly hope that a zarr array is typically just pointing to a remote storage location such that the actual payload data is not literally embedded in the graph.
The one point in the discussion that is a little too simplified in this discussion is that while pickle is smart, the scheduler still has to send this data N times to the workers. I'm not entirely sure if pickle on workers can actually deduplicate this data 🤔 (I'll run some tests)

a zarr array is typically just pointing to a remote storage location

It could be a local file too but it is not an array in memory.
HEre's an example:
import xarray as xr

xr.tutorial.open_dataset("air_temperature").to_zarr("test.zarr", mode="w")

ds = xr.open_zarr("test.zarr", chunks={})
ds

I think this was closed by https://github.com/dask/dask/pull/10660


Refine application of trans files

Refine application of the tfm tables in trans files by removing any row in generated tables that:

for an attribute with process in indices, includes a process that is not declared in the corresponding non-trans file;
for an attribute with commodity in indices, includes a commodity that is not declared in the corresponding non-trans file, base or syssettings.

This PR should enable #256

@Antti-L please feel free to comment.


deps: update php and laravel requirements

bump to support php 8.1 and laravel 10. This was done since this was a downstream dependency of ltree.

@jamesrusso
Has the handling of presenters changed in Laravel 10? The coverage dropped because the tests don't work anymore. I need time to investigate what the problem is. And then I'm dying branches in all repositories.

@jamesrusso
Has the handling of presenters changed in Laravel 10? The coverage dropped ( https://coveralls.io/builds/62705298 ) because the tests don't work anymore. I need time to investigate what the problem is. And then I'm dying branches in all repositories.

Not sure. I'll try and spend some time on it today.

@jamesrusso
Has the handling of presenters changed in Laravel 10? The coverage dropped ( https://coveralls.io/builds/62705298 ) because the tests don't work anymore. I need time to investigate what the problem is. And then I'm dying branches in all repositories.

Not sure. I'll try and spend some time on it today.

The unit tests and locally are showing no tests run. I've made some additional commits to this PR to handle some some other dep updates.

Looking back to 2020, I do not think the coverage was ever good on this project?

added support php 8.1/8.2 and laravel


Test on PHP 7.4 and HHVM 3

Tests are failing due to code style, not relating to this PR.

Thanks for this :-).
Code style issue in CI is known and needs fixing; agree it's unrelated.


Dynamo 3D Preview Capture

Currently Dynamo does not have a built in method for capturing the 3D preview (geometry), you can only capture the canvas.  There should be a way to modify the current method to capture the geometry as well.  This could be a pretty interesting node in itself for animations and such.

Moved to hydra-dynamo


Restrict sharing experience by domain for OneDrive for Business

Hello,
I am testing restrict sharing by domain on SharePoint Online and OneDrive for Business.
I have a question about experence when a shareable domain (allowlist) has been set up and is to be shared with existing guests.
According to the below article, it is noted in parentheses that when sharing to an existing guest in SPO site, no error is displayed on sharing dialog, but access will be blocked if the domain isn't allowed.

(If the user is already in your directory, you won't see the error, but they will be blocked if they attempt to access the site.)

As far as I tested, the behavior on OneDrive sites is as well. In other words, when sharing to an existing guest user that the user's domain is doesn't allowed, there is no error on sharing dialog, but the user can't access the shared content.
In the article, the below note specified sharing on SPO site only, but I think it can apply on OneDrive site as well.
If my understanding is correct, could you please clarify that the below note applies to both SPO and OneDrive sites?

(If the user is already in your directory, you won't see the error, but they will be blocked if they attempt to access the site.)

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

ID: f3e6adf5-c322-b7f1-9359-1f9e0723374b
Version Independent ID: 0071fb22-d32b-da3f-aec1-60772b341bd0
Content: Domain restrictions when sharing SharePoint & OneDrive content - SharePoint in Microsoft 365
Content Source: SharePoint/SharePointOnline/restricted-domains-sharing.md
Service: sharepoint-online
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

Thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


Basic Sea Of Thieves Support

Hey Man,
love your bot, played with it a little and added Sea Of Thieves support :)

Thanks for the contribution, glad you like the program.


FYP: "Don't recommend channel"

#1102

Johnny has the API Changes but needs to make a few tweaks. We'll get it live tomorrow.

Don't know why I can't force-push ... re-creating PR

Argh, just saw Niko's message ...


Fill between 2 line

Description
fill color between 2 line
Code example
fill(plot1, plot2, color, title, editable, show_last, fillgaps) → void
fill(hline1, hline2, color, title, editable, fillgaps) → void

Method of implementation
No response

Hey @DEKAGATO,
This comes under #180, so i'm marking as duplicate.
Louis


Use u8 and [u8], not char and str for printing methods on Console

Console’s printing methods don’t work with Unicode strings or characters, so it doesn’t make sense to use the char (Unicode scalar value) and str (UTF-8 string) types. This changes these methods to use u8 and [u8] instead. The ergonomic impact of this is minimal (although it breaks just about all code, unfortunately), due to the existence of byte string and character literals (b"foo" and b'c'), and additionally, this makes it possible to specify special characters directly inside byte strings (e.g., printing b"90\xf8" (0xf8 is chars::GRADE) now displays 90°, but previously displayed 90├© due to the fact that strings are UTF-8).

The ergonomic impact seems much higher to me, but yeah, this is a problem. I absolutely agree that we should allow b'c' and b"foo", but I'd like to keep the ability to pass in normal chars and strings as well (with the understanding that you mustn't pass in non-ascii values).
What do you think of accepting something like Into<u8> and Into<[u8]>, potentially with the char and str implementations calling std::ascii::AsciiExt::is_ascii during the conversion?
(we'd have to define a different trait because the types and the Into trait are in a different crate)
That conversion would have a performance hit, but you'd get a runtime error and you could always make it fast by using the ASCII types directly. I.e. easy for people getting started but not costing folks who know what they're doing.

Since libtcod actually does support Unicode printing functions: docs, we could statically dispatch the printing based on what argument it gets. I cannot test this right now, but something like this (the mutable trait object syntax might be incorrect, I can't remember if you need a 2nd &):
pub trait PrintString {
    fn print(&self, con: &mut Console);
}

impl PrintString for [u8] {
    fn print(&self, con: &mut Console) {
        // Print ASCII using the ffi
    }
}

impl PrintString for &str {
    fn print(&self, con: &mut Console) {
        // Print Unicode using the ffi
    }
}

impl PrintString for String {
    fn print(&self, con: &mut Console) {
        // Print Unicode using the ffi
    }
}

Using this, the definition for the actual print function would be something like this:
fn print<T>(&mut self, x: i32, y: i32, text: &T) where T: PrintString {
    assert!(x >= 0 && y >= 0);
    text.print(self);
}

This could also be repeated for char and u8.

@tomassedovic Yeah, I was just thinking about doing that. Potentially (perhaps later in a different PR, though) the char/str → u8/[u8] conversion implementation could also manually map specific Unicode glyphs to their equivalents in libtcod’s character set, and only if no such equivalent exists would it throw a runtime error.
One question, though: the ASCII control characters display differently in libtcod’s character set. These characters could easily be supported using char/&str, because they have the same representation as a C/byte string, but that’s potentially confusing, because they don’t display the same in Unicode, which is what char and &str are supposed to represent. So should a char/&str → u8/[u8] conversion method allow control characters?
@Gustorn I did try to use the _utf functions in the FFI, but they just didn’t work in my testing (e.g., é displayed as ú), and I had to manually convert &str to UTF-32 anyway, which is an O(n) operation.

@P1start Does libtcod really expect UTF-32 strings? That's a bit curious... Also, are you sure you were using a Unicode font for the Root console?

@Gustorn Are Unicode fonts available for libtcod? I was under the impression that only 256-character terminal.png-like fonts were supported. And I’m pretty sure they’re UTF-32 because the C API expects const wchar_t *, which is equivalent to *const i32 in Rust (wchar_t is 32 bits, at least on my platform).
Also, I just realised that a static-dispatch-based API (fn print<T: AsTcodStr> or whatever) isn’t going to work, since that would make Console not object safe, which is a big deal. You could use dynamic dispatch (fn set_char(x: i32, y: i32, glyph: &ToTcodStr) or something similar), but then you’d have to box/reference every string or character, making the ergonomic impact even worse, probably.

Here's a forum topic about an (apparently) full Unicode bitmap font. As for wchar_t and UTF: you actually have to store UTF-8 using a type that can hold a minimum of 32 bits, since Unicode code points can take up 1-4 bytes using that encoding (the original spec said up to 6 bytes, but if I remember correctly there is an RFC that specifies 4 as the current upper limit).
Object safety is actually a really good point, one which I haven't considered. To be completely honest, I haven't really seen a use-case for using trait objects, so object safety is doesn't seem a terribly big deal to me: the only function where Console types are actually interchangeable is blit, other than that I can't see why you'd store a Vec<Box<Console>> for example.

@Gustorn Looks like that ‘only’ supports the basic multilingual plane, but I now see that you can indeed support more than just 256 characters. I guess I now know how I was using the _utf functions incorrectly: the only difference is that they support 232 characters instead of just 256; they don’t necessarily map to Unicode values unless your font file is specifically organised that way.
One of the first things I wrote with tcod involved using Console as a trait object, in order to make sure another trait was object safe. It went something like this:
trait Draw {
    fn draw(&self, &mut Console);
}

And then there was going to be a Vec<Box<Draw>> somewhere which would be iterated over to draw every drawable object, but I never really got that far. It’s not a super compelling example, but it’s a good idea anyway to keep traits object safe whenever possible.

Yeah, unfortunaly that is a valid example. These are the times when I actually miss C++. A templated print wouldn't cause any problems there.
I still think that using u8 as default is not the greatest idea: it does have a significant ergonomic impact (either the byte literals or as_bytes (?)), and it's much more natural to work with String and &str. An other option would be to get rid of every function inside the trait (except for con), and define everything as free functions. Since there's no actual UFCS in Rust, this would be pretty inconvenient (even though it would mirror the C API quite faithfully):
root.print(args...);
// Would become
console::print(root, args...);

So here's some testing I've done with object safety, and I'm happy to say, we can actually do the generic print and have Console still be object safe. Here's the link to my little test: commit. Unforuntalely I don't really have time today, but tommorow I think I'll implement it.

I'm sorry, but I've had little time to participate today and I'll be completely off for the next few days. If you come up with something that doesn't require the u8 and [u8] literals, feel free to merge it.
I'll be back on Monday so if it doesn't come in by then, I'll read through this thread again and get this fixed one way or another.
Thanks both of you.

Proposing to close this since #163 was merged.

Agreed, the unicode concerns should be addressed by #163. Note that we still don't use b'@' and b"foo" directly, but that can be addressed in another pull request.
Thanks @P1start for starting this! If you feel we should re-open this, feel free to comment here.

I'll reopen this since I did a bit of research regarding the literals.
After a bit of experimentation I don't think we should accept ASCII literals, and here's why. With the current implementation we choose the printing function based on Rust's is_ascii function, which only returns true if the given characters are the part of the original (non-extended) ASCII table. This means that given the example in the opening post (b"90\xf8") is_ascii would return false, which would result in us trying to use the UTF-8 printing function with an invalid byte (the degree character is represented by two bytes in UTF-8). This is most certainly not the behaviour one would expect.
Now the current runtime dispatch could be combined with my earlier idea for static dispatch, but I'm not sure if introducing arbitrary rules for the different string types is such a good idea. Let's say we have the trait TcodString that controls if a value should be converted to UTF-8 or not. A possible implementation for the different string (or string-like) types:

&str -> convert if not is_ascii
String -> convert if not is_ascii
&[u8] -> never convert

This is probably close to the behaviour people would expect: since libtcod doesn't really care about encoding, using ASCII literals, even with the extended ASCII table should give correct results.  In the case of the two regular string types they are already guaranteed to be valid UTF-8 encoded strings at least, but we can save the conversion if they are in the original ASCII range (this is what we're doing now).
I can probably implement it in a way that still accepts user-defined string types, provided they implement AsRef<str>. Without negative trait bounds also accepting AsRef<[u8]> is impossible I think (it would have to be a concrete implementation for &[u8].


[SQL] 2017-10-24

FYI @jaredmoo

Includes:

https://github.com/Azure/azure-sdk-for-python/pull/1550
https://github.com/Azure/azure-sdk-for-python/pull/1555

Looks good, @emgu-msft and @maboja-msft to also review

Looks good from my perspective as well.

Codecov Report

Merging #1570 into master will decrease coverage by 0.32%.
The diff coverage is 26.92%.

@@            Coverage Diff             @@
##           master    #1570      +/-   ##
==========================================
- Coverage   54.17%   53.84%   -0.33%
==========================================
  Files        3612     3626      +14
  Lines       89128    89881     +753
==========================================
+ Hits        48285    48400     +115
- Misses      40843    41481     +638

Impacted Files
Coverage Δ

...mgmt/sql/models/elastic_pool_edition_capability.py
55.55% <0%> (-6.95%)
:arrow_down:

...mt-sql/azure/mgmt/sql/models/edition_capability.py
55.55% <0%> (-6.95%)
:arrow_down:

azure-mgmt-sql/azure/mgmt/sql/models/__init__.py
100% <100%> (ø)
:arrow_up:

azure-mgmt-sql/azure/mgmt/sql/models/database.py
100% <100%> (ø)
:arrow_up:

...e-mgmt-sql/azure/mgmt/sql/sql_management_client.py
100% <100%> (ø)
:arrow_up:

azure-mgmt-sql/azure/mgmt/sql/version.py
100% <100%> (ø)
:arrow_up:

...ure-mgmt-sql/azure/mgmt/sql/operations/__init__.py
100% <100%> (ø)
:arrow_up:

...mt/sql/operations/server_dns_aliases_operations.py
13.93% <13.93%> (ø)

...ure-mgmt-sql/azure/mgmt/sql/models/elastic_pool.py
33.33% <50%> (-2.39%)
:arrow_down:

...-mgmt-sql/azure/mgmt/sql/models/database_update.py
15.15% <50%> (-0.48%)
:arrow_down:

... and 52 more

Continue to review full report at Codecov.

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


Add instance timeout to poolnoodle

moves the PoolNoodle invocation to inside of local_uds instead of at the manager level to avoid repeating decision logic  specific to firecracker
adds an active pool when we start an instance that keeps track of the start_time
we now check if the pools are timed out and move them to the to_be_cleaned pool
this check is pretty lazy and only happens if we don't have new jails to make or old jails to clean. Given that the timeout is 60 minutes, another few seconds should be fine

bors try

bors try

bors try

bors r+


Where is the text images in CC-OCR?

Hello!
When I try to download the link OCR-CC Data (Huge, ~1.3T), I find the CC-OCR dataset does not contain text images. So I would like to know where to get these images.

We uploaded the GCC index file at https://tapvqacaption.blob.core.windows.net/data/GoogleCC/Train_GCC-training.tsv
The first index in "ocr_feat/visu_feat_resx" before "_" indicates the row number in the index file (both 0-indexed). E.g., "100000_1967358300" is the "100000" row of the soccer match image.

We uploaded the GCC index file at https://tapvqacaption.blob.core.windows.net/data/GoogleCC/Train_GCC-training.tsv
The first index in "ocr_feat/visu_feat_resx" before "_" indicates the row number in the index file (both 0-indexed). E.g., "100000_1967358300" is the "100000" row of the soccer match image.

Is there another way to download the OCR-CC Data? Such as Google Drive... I can not download the dataset stably due to my area. Many Thanks.

Unfortunately, the CC3M dataset does not allow sharing raw images due to copyright issues. If you have a copy of CC3M images, it should cover all images in OCR-CC. There are also various online tools for CC3M downloading, which might solve/alleviate the network issue.

Hello! When I try to download the link OCR-CC Data (Huge, ~1.3T), I find the CC-OCR dataset does not contain text images. So I would like to know where to get these images.

老哥，能够分享一下你下载的这个数据集吗？我按照他提供的这个azcopy下载一直不行。。能分享一个百度网盘链接不。。感谢感谢


Fix allocation profiler alloc_size for JDK 7-9

According to the signatures of both send_allocation_in_new_tlab and
send_allocation_in_new_tlab_event, the previous code was taking the
tlab size as the requested allocation size.
This commit fixes this by using the correct parameter.
Tested with https://github.com/pingtimeout/humongous-allocator.  Without
the fix, 8 instances of java.lang.Thread_[i] were reported with a -o collapsed=total above 12M, which cannot be correct.  The same happened
with other java.lang basic data types.

Measuring total TLAB size is intended.
The allocation sampling mechanism provides no means to count the exact number of allocations or the exact amount of allocated bytes.
Current async-profiler behavior is aligned with JFR which measures so called "allocation pressure". If I record the size of individual objects instead, the relative allocation pressure would be misleading. E.g. if there are three types of objects: A (100 bytes), B (200 bytes) and C (300 bytes), and the number of allocations of each object is the same, the allocation ratio shown by the profiler should be 1:2:3, which is the case with the current approach, but not with your suggested approach.

Without the fix, 8 instances of java.lang.Thread_[i] were reported with a -o collapsed=total above 12M, which cannot be correct. The same happened with other java.lang basic data types.

One more clarification: async-profiler shows the estimated total size of all allocated objects of the given class, not only the size of the sampled objects. On the long distance, TLAB size would give [statistically] more precise results. However, I admit that with a small number of allocated objects (like java.lang.Thread in your case) the collected results may seem incorrect.

Ah I see, thanks @apangin for the explanation.  I have a related question for you then.  I have tried to make async-profiler record humongous object allocations to precisely know where they are created, so that a codebase can become more G1GC friendly.  To do that, I needed some if(alloc_size > threshold) code and this this.
The code is still very hacky at this point.  But eventually I was thinking about submitting it here.  Do you have any recommendation for how I should implement this feature?  In my current version, that if logic is in AllocTracer::recordAllocation.  But that cannot work very accurately if the tlab size is used instead of the actual object size.

Humongous allocations are not related to TLAB at all. Async-profiler makes difference between allocations in new TLAB and outside TLAB. Humongous allocations are always outside.
Alternatively, you can profile humongous allocations directly, using async-profiler's feature to intercept arbitrary native functions. In this case, you'll need to intercept G1CollectedHeap::humongous_obj_allocate:
./profiler.sh -e _ZN15G1CollectedHeap22humongous_obj_allocateEmh --sync-walk ...

I've just added --sync-walk feature to make Java stack traces more accurate when profiling VM runtime functions. It's not yet tested well, use carefully.


CSV report (RM Squid 2.*-style) rounds 4cor_208Pb232Th_Age_RM to 4 significant figures

This is a very minor one, relating to the reporting of 4cor_208Pb232Th_Age_RM (and by association, its 1sigma absolute uncertainty). Everything looks fine when inside Squid3; below is the relevant Expression Manager window, which shows the full double-precision values in the Peek. These are correct in detail (i.e. "perfectly" matched to SQUID 2.50).

However, when the data is output to a Report Table (in my case, via Custom Report Tables... Reference Materials... Report Table per Squid2.*), these data have undergone symmetrical arithmetic rounding to 4 significant figures:

This issue persists across all 8 permutations of (Perm2/Perm4 + 204corr/207corr + Ucentric/Thcentric). It did not surface during Perm1 testing because in Perm1, neither SQUID 2.50 nor Squid3 perform calculations that give rise to 208Pb/232Th ages. I have not yet done any Perm3-based testing, but based on what I have seen, I would expect the problem to persist there.
It seems to be isolated; I have not yet found any other expression that is affected.

I found the culprit - a specification to show 4 instead of 15 digits in the report at those columns - a typo/bug.  Repaired in next release.


feat(bodyParser):added possibility to provide options that are passed to body parser

Notice! This was not fully tested! Please check it before merging to production.

in the latest version (0.4.0-beta.2) I changed body-parser to express-formidable because it also allows you to receive multipart data. So closing this one


Thread safety

Prepare KSMT for use in a multi-threaded environment:

[ ] Use concurrent data structures in KContext
[ ] Guard solver native APIs with locks

Concurrent data structures is a very good point.


error: no instance of overloaded function "atomicAdd" matches the argument list

Hi，Encountered the following questions when configuring the environment, and look forward to your answers.
-D_GLIBCXX_USE_CXX11_ABI=0 -gencode=arch=compute_52,code=sm_52 -gencode=arch=compute_60,code=sm_60 -gencode=arch=compute_61,code=sm_61 -gencode=arch=compute_70,code=sm_70 -gencode=arch=compute_75,code=sm_75 -gencode=arch=compute_80,code=sm_80 -gencode=arch=compute_86,code=compute_86 -gencode=arch=compute_86,code=sm_86 -std=c++14
/dfs/data/code_python/detection_3d/BEVDepth_sec/bevdepth/ops/voxel_pooling_train/src/voxel_pooling_train_forward_cuda.cu(60): error: no instance of overloaded function "atomicAdd" matches the argument list
argument types are: (half *, const half)
detected during instantiation of "void voxel_pooling_train_forward_kernel(int, int, int, int, int, int, const int *, const T *, T *, int *) [with T=half]"
(99): here
1 error detected in the compilation of "/dfs/data/code_python/detection_3d/BEVDepth_sec/bevdepth/ops/voxel_pooling_train/src/voxel_pooling_train_forward_cuda.cu".

Hello, have you solved this problem?

same error. any suggestion?

I sloved this by setting export TORCH_CUDA_ARCH_LIST="7.0 7.5 8.0 8.6+PTX" before use python setup.py develop
I think it is because the "atomicAdd" with Half type only support the GPU with 7.x+ Compute Capability


Add webdriver.io tests to terra-clinical-action-header

Summary

Consume webdriver.io for testing in terra-clinical-action-header and remove the nightwatch tests.
The test examples in the nightwatch folder are moved to examples folder

Additional Details
Resolves #278
Thanks for contributing to Terra.
@cerner/terra

@rakshith91 could you address the file conflicts?


feat: 🎸 Add new events and extrinsics with 5.2.0 support

Description
Add new extrinsics and events in chain 5.2.0
Breaking Changes
NA
JIRA Link
DA-493
Checklist

[ ] Updated the Readme.md (if required) ?

:tada: This PR is included in version 8.5.0-alpha.7 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 8.6.0-alpha.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 9.1.0-alpha.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

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


[Event Request] table 233 "Item Journal Batch" - OnBeforeSetupNewBatch

Hello, would it be possible to add OnBeforeSetupNewBatch event to table 233 "Item Journal Batch", like this:
procedure SetupNewBatch()
...
      var
         IsHandled: Boolean;
     begin
         IsHandled := false;
>         OnBeforeSetupNewBatch(Rec, IsHandled); //New Event
         if IsHandled then
             exit;
         ItemJnlTemplate.Get("Journal Template Name");
         "No. Series" := ItemJnlTemplate."No. Series";

     [IntegrationEvent(false, false)]
     local procedure OnBeforeSetupNewBatch(var ItemJournalBatch: Record "Item Journal Batch"; var IsHandled: Boolean)
     begin
     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. 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.
Build ID: 54020.


Enhance player statistics

It would be cool to enhance player statistics to include faceoffs won (FOW) as it is used quite often in Yahoo Fantasy Leagues.
Currently as a workaround I need to iterate over all games and check the boxscore and calculate on my own.

Hey @mlynarp
Thanks for giving me this great suggestion! I will try and put it in the back-log for myself and work on this when I have a chance

Hey @mlynarp
After releasing v3.0.0, there is a way to enable you to have the ability to retrieve the face off winning percentage statistic per player.
Within the C# class model PlayerCareerTotals there is the ability to give you access to the players regular season and playoff season face off winning percentage.
You can access this endpoint via Task<PlayerProfile>GetPlayerInformationAsync(PlayerEnum player) method, here is the documentation, https://github.com/Afischbacher/Nhl.Api?tab=readme-ov-file#getplayerinformationasyncplayer-method
Here is a small sample of the JSON returned with the specific information you are looking for with Connor McDavid's profile
    "careerTotals": {
        "regularSeason": {
            "faceoffWinningPctg": 0.4744
        },
        "playoffs": {
            "faceoffWinningPctg": 0.4635
        }
    },

Let me know if this helps, I can then close this issue :)
Thanks

Hey @Afischbacher ,
actually my goal was to collect statistics about the faceoffs won not the percentage. So for example I need to know that McDavid has collected throughout the 2023/2024 season 187 faceoffs he won as this category is used in Yahoo Fantasy league.
Thanks for future improvements.

Hey @mlynarp
Now I understand what you mean, and I have built something to accommodate this and other types of events (blocked shots, hits) so you can effectively count by the player and season, this should be live in 3.1.0
Thanks :)

Hei @Afischbacher,
I checked the source code and it seems you will have to integrate https://api.nhle.com/stats/rest/en endpoint for statistics data which is not part of the source code.
Do you need help with these or when do you think you will have API which will return statistics data per player per season?
Just to describe you in more detail what I need:
I have an app which loads all active players within active season (info about season and season dates would be great as well) and then loads statistics per each player. For skater it is GP, G, A, P, +-, PIM, Shots, FOW (int number), Hits, Blocks, and for goalies its GP, W, GAA, SV, SV%. Then I do some analysis and data visualization.
Therefore I can help you with the code if its too big for now. Currently I didn't find the way how to achieve it with current version.
Btw. I studied the API reference here https://github.com/Zmalski/NHL-API-Reference and not sure whether all data are available in API :-(
Thanks for you valuable work!

Hey @mlynarp
So, I have just released the 3.1.0 version of the Nhl.Api and think it will help you towards your goal of creating this app, I hope. It may require you to do API calls within the NuGet with minor LINQ manipulation, but it could work.
If I understand exactly what you're looking for. Take a look at the GetPlayerInformationAsync method to get all the statistics such as Goals, Assists and Points and call the GetTotalPlayerStatisticValueByTypeAndSeasonAsync for the Face offs Won and any other statistics you may need or want. Now I understand this might not give you the full picture of what you are looking for, but I think it's a step in the right direction and a welcome improvement for individuals looking for different statistics.
Here is an example of the Player Landing Endpoint for Connor McDavid
 "featuredStats": {
        "season": 20232024,
        "regularSeason": {
            "subSeason": {
                "gamesPlayed": 33,
                "goals": 14,
                "assists": 39,
                "points": 53,
                "plusMinus": 9,
                "pim": 16,
                "gameWinningGoals": 0,
                "otGoals": 0,
                "shots": 96,
                "shootingPctg": 0.145833,
                "powerPlayGoals": 4,
                "powerPlayPoints": 19,
                "shorthandedPoints": 0,
                "shorthandedGoals": 0
            },

Again, it does not have all the pieces of the puzzle, but I think it's close once you include the new endpoints I have added.
Let's keep this issue open and find a full solution for you, but please let me know what you think about the new changes.
Thanks
P.S. Please share this app, I would love to see it!!!

Hey @Afischbacher
thanks for new version. It looks good and I think I will be able to get all the data I need.
Just one note, I noticed the implementation of the total player statistics. This is perfectly fine for one player, however I need to collect it for all NHL players. It looks to me that it would cause a performance issue so I will rather iterate over all games boxscores and extract the particular fields for players.
Maybe it would be good to do it in API at least per team to avoid plenty of API calls and iterations.
P.S. My app is currently in very rough state, its like alpha alpha version :-) I can share later on when it is really in a good shape.

I started with the implementation but found that I had no way to collect information about the team. There is a list of team enums, but I can't map it to a team abbreviation. This function is private through the NhlTeamService class. The only way to collect information such as team name, conference name, division name, team abbreviation is through GetLeagueStandingsByDateAsync. This function includes team stats, which is great, but unfortunately it doesn't include the team ID, so I don't have a way to connect to the team enum programmatically.
Can you improve it and include this information to LeagueStanding object or make NhlTeamService interface available to use?
Thanks.

Hey @mlynarp
Thank you for the feedback. Let me see if I can open up the functionality of the service to be public and also see if I can get the statistics per player for all categories. I'll see if I can find time tonight to unblock you.

Hi @Afischbacher
any news regarding API extension?
Thanks

Hey @mlynarp
Let me try and release something for this weekend, apologies on the delays
Thanks

Hi @Afischbacher,
do you have time to release something? Or what is your expected time frame to provide above mentioned features.
Thanks for info.

Hey @mlynarp
I have local changes done. Just need to write tests and deploy.
My goal is to have this done for end of week. Other projects and priorities have been taking my time, sadly...
I'll keep you posted. But please keep bugging me.
Thanks
Andre

Hey @mlynarp
Check out the latest release of v3.2.0 and let's see if this does the trick for your app, I think it's what you're looking for!.
If it does, I can close the issue, apologies for this taking longer than expected!
Thanks for your patience!
Andre

Hey @Afischbacher,
thanks for you update. I checked the API and I think I should be able to fulfill my task. Nevertheless I started to work on it and found out that NHL API is extremely slow. Hope its temporarily. Do you experience same issue?
One simple request takes up to 10 seconds.

It looks like to be working normally again.

So during my implementation I discovered couple of troubles:

Calling GetAllTotalPlayerStatisticValueBySeasonAsync(8478465, "20232024") throws exception: Object reference not set to an instance of an object
Calling GetAllTotalPlayerStatisticValueBySeasonAsync(8482496, "20232024") returns wrong stats. It has 48 FOW, 13 Hits and 21 Blocked shots. However the stats on NHL or Yahoo says 44, 9, 14.

All above mentioned will be valid for more players, I just mentioned the reproducible cases.

Hey @mlynarp ,
Let me peek at these cases and see if I can get these bugs fixed soon.
Thanks,
Andre

Hey @mlynarp ,
Let me peek at these cases and see if I can get a bug fix out soon.
I have fixed most of the issues quickly, I think I can have a release out tonight.
Thanks

Hey @Afischbacher
do you plan to release your quick fixes or do you plan something bigger? :-)
Thanks for feedback.

Hey @mlynarp
Haha, if I were planning something bigger, I would tell you 🧑🍳😁😆
Let me try and get it out in the next 24 hours, otherwise it will be a couple of days.
Thanks again,
Andre

@mlynarp
Take a read here https://github.com/Afischbacher/Nhl.Api/pull/48/files
Make comments if you want!
If I have time I can merge and release the NuGet tonight 👍

@mlynarp
v3.2.1 is live, take a peak!

Great,
first test results:

Calling GetAllTotalPlayerStatisticValuesBySeasonAsync(8481523, "20232024", Nhl.Api.Enumerations.Game.GameType.RegularSeason) throws exception: Unknown NHL team: Montréal Canadiens (Parameter 'teamName')'  happens for all Montreal players
Still very slow. Any idea how to improve it? My idea would be to make a function to call total stats per team to reduce number of iterations.

Thanks for your work!

Number of blocks still not fit, e,g. Elias Pettersson

@mlynarp

Release, v3.2.2 is now live, this is for the Montreal fix due to the accent in the team's name, which should work now.
As for the performance, because I am pull each play by play in the game, it is not super performant at times, but I usually see performance in the range of 400ms to 1.5s which is not idea but not terrible, how slow is it for you?
Number of blocks for Elias Pettersson? Not sure what this means?

Thanks,
Andre

Montreal issue is fixed, great.
I am loading all NHL players, so I call GetAllTotalPlayerStatisticValuesBySeasonAsync roughly 1200 times. I run it in parallel but still it takes aprox. 120 seconds and sometimes when running in parallel it fails. Running in sequence would be a death.
The number of blocked shots returned by GetAllTotalPlayerStatisticValuesBySeasonAsync is not correct and doesn't fit the official numbers from NHL. The Faceoff won and hits seem to be correct.

Hey @mlynarp
Happy to see this works with the new fix
Yes well, you're making about 84 * 1200 =100,800 requests in the span in ~2 minutes, I am not surprised, I would consider looking into a caching/storing this data in a persisted state to reduce your dependency on the API and run this on an event based or timer-based model. Doing this on request is not the best idea because this data does not update until any game is live or active.
As for the blocked shots, I took a deep dive into Elias Pettersson blocks for this season, and it has 41 registered events for him blocking versus the 38, so I am not sure if it's a data issue, but go through each game and look to see if he blocks the puck (joking of course)
See my data below, there are 41 instances of him "blocking a shot"
sample.json
I am ready to close this issue,
Thanks

Hey @Afischbacher
current version is pretty good and I can start to use software again and save my fall in Fantasy League :-)
Anyway I checked the json and you are right there are 41 blocked shots for Elias. This is really strange. Only thing that come to my mind is that they do a stats correction after the game and maybe its not fixed in the play by play data.
Performance:
You do for every player  a call for profile information (that must be done), then call for team schedule and then for every game (84) you call GetGameCenter and then the iterative math. This means 1200x team schedule, 1200x84 game center.
I believe you can call once per team its schedule and then for every game(84) call game center. Then iterate and fill all team players. This would mean 32x team schedule, 32x84 game center. Iteration for team members will be more complicated but much faster. In the end you end up with 2688 API calls.
I will appreciate if you can try this as it would be then complete. I do some caching but irrespectively of this I believe it deserves this improvement ;-)
Thanks for your support.

Hey @mlynarp
I think I can create an entire endpoint dedicated to this, I think the difference here is instead of the entity being around a player, we can just focus it on the entire league and build a Dictionary of Dictionary<PlayerProfile, Stats> and I think it can be done.
Let me see what I can do, I won't close this just yet.
Thanks,
Andre

Hey @Afischbacher,
I redesigned my application and as a part of it I collect stats in different way. See the pseudo code:

I load team schedule for every team
For each game I load boxscore
From boxscore I iterate the stats of forwards, defense and goalies
I sum up the totals of statistics field

As a result all of this takes aprox 10 seconds and all the statistics are correct and fits to what is shown on Yahoo. I think that this is much better attitude to collect stats.
Petr

@mlynarp
Hey!
Look at v3.3.0 I have added an endpoint that is much more performant and gives all the statistics I believe you need!
The method is
GetAllPlayersStatisticValuesBySeasonAsync
Let me know what you think and if this works, we can finally close this!
Thanks,
Andre

Hey @Afischbacher
I checked new version and it is really a good progress, however I measured 21 seconds for whole league which is still worse than my iteration over boxscore (8seconds) and it still suffers from unprecise data. The hits given and blocked shots doesn't match the real statistics.
I think that game play by play is not the correct statistics. Rather use the boxscore where it seems to fit.
P.S.
Do you think you might provide an API to get player ID from player name? The problem is that getting the team roster will return just the active players on the roster. It means when a player is sent to AHL or being long term injured he's not part of roster and then this player is missing in overall stats. Alternatively you might return all player IDs who played within current season that would be the best.
Thanks for your work!

Hey @mlynarp
Interesting, during my testing I saw it grant me results within 10 seconds, which is not terrible, but it could be due to caching or latency from the NHL endpoint?
As for this question:
Do you think you might provide an API to get player ID from player name?
There are two endpoints that come to mind:
 public async Task<List<PlayerSearchResult>> SearchAllPlayersAsync(string query, int limit = 25, CancellationToken cancellationToken = default)
public async Task<List<PlayerSearchResult>> SearchAllActivePlayersAsync(string query, int limit = 25, CancellationToken cancellationToken = default)
With these endpoints you can just search for any name and have information returned including the PlayerId
Also remember the PlayerEnum is a direct dynamic mapping of all player names to their identifiers. Look for it, it may help you!
Let me know if this is any help, if it is, I can close this issue!
Thanks

Hey @Afischbacher
thanks for the API, I think that should do the job for me.
From my perspective the API offers what I really need, I will keep my own implementation but your API is fine.
Feel free to close the issue but I would consider to replace play by play statistics by boxscore as it delivers precise data.

Hey @mlynarp
Going to close this issue as it is resolved!
Thanks
P.S. Please do share the app you build!!


consolidate cosmos spring samples

Purpose
This is largely a copy/paste/edit exercise to consolidate existing disparate spring cosmos samples to this central repo. Once committed, we will re-direct documentation and eventually archive the original repos. Repos being moved are:
https://github.com/Azure-Samples/azure-spring-data-cosmos-java-sql-api-samples copied to \azure-spring-boot-samples\cosmos\azure-spring-data-cosmos*cosmos-quickstart-samples*
https://github.com/Azure-Samples/azure-spring-mvc-cosmos-db-java-sql-api-samples copied to \azure-spring-boot-samples\cosmos\azure-spring-data-cosmos*cosmos-mvc-sample*
https://github.com/Azure-Samples/azure-spring-data-cosmos-java-sql-api-aad copied to \azure-spring-boot-samples\cosmos\azure-spring-data-cosmos*cosmos-aad-sample*
https://github.com/Azure-Samples/cosmosdb-springboot-aks copied to  \azure-spring-boot-samples\cosmos\spring-cloud-azure-starter-data-cosmos*spring-cloud-azure-data-cosmos-aks-sample*

...

Does this introduce a breaking change?

[ ] Yes
[x] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[x] Other... Please describe:
adding samples copied from other repos

How to Test

Get the code

git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install

Test the code

What to Check
Verify that the following are valid

...

Other Information

Hi, @kushagraThapar

I added you as a maintainer of this repository, so you have authority merge this PR now.
Current PR will be merged to spring-cloud-azure-v4.4.1 branch. Please create a new PR which merge to main branch.
After this PR merged, please archive the original repos and add a description in original repos to tell customer to use this repository.


link should be self closing

Describe the Issue

whats the problem?
Affected Page
https://www.freecodecamp.org/learn/2022/responsive-web-design/learn-html-forms-by-building-a-registration-form/step-4
Your code
<title>Registration Form</title>
<link rel="stylesheet" type="text/css" href="styles.css" >

Expected behavior
submit and progress to next
Screenshots
No response
System

Device: [ Laptop]
OS: [Windows 10]
Browser: [Chrome, and opera]
Version: [e.g. latest both]

Additional context
No response

"/" - is it really needed?
worked okay without it in another codes.


excel-shared-runtime-scenario cannot be run on Excel Version 2008 (Build 13117.20000) Beta Channel

Hello,
My laptop is installed with Office 364 Beta Channel Version 2008 (Build 13117.20000) and when i try to follow the steps in https://github.com/OfficeDev/PnP-OfficeAddins/tree/master/Samples/excel-shared-runtime-scenario
start npm start
npm run sideload
The sample is not running properly with errors when the excel application starts.
Please find the attached screenshot below. The add-in is not loaded properly.

When the open taskpane is clicked, error message at the status bar saying failed to download resources.

The excel version is beta channel (insiderfast)

Hi @chanhohang,
Sorry for the late reply on this. The shared runtime samples were using an old beta endpoint and we recently updated them for release bits. This may have been the issue you were encountering. If you can try the latest update to the samples that would be great! Please let us know if that doesn't fix the issue.
Thanks!

Tested on 2011 13406.20000, all seems to work properly. I think this can be closed as soon as PR #70 is pushed in ... (probably already works but I tested it at least on the #70 code.

@AlexJerabek @davidchesnut Same here, I'm unable to close the issue (only comment). See above, tested it in my live stream and working so can be closed.

Hi @aafvstam, (@AlexJerabek as fyi) I just gave you access for issues. We give folks a few days to follow up if they still have questions or the issue isn't resolved. Then a bot will auto-close the issue. So no need to close these.


Update smoothed safegraph signal names

Description
Update the work-in-progress safegraph signal names to conform to our naming conventions.
Changelog

Signals ending in 7d_avg now end in 7dav

Fixes
part of #332

it looks like you're reusing your old weekday branch, which contains commits that got rebased when we fixed the safegraph stuff a month ago. The commits from "29 days ago" looked suspicious, so I checked, and there's already a commit with message "migrate safegraph.run back onto using functools.partial" in main.
Recommend doing the following:

make a new branch off main
cherry pick 3cb534c and 279ed47
delete outdated weekday branch
either submit a new PR with the new branch and close this one, or rename the new branch to weekday and force-push to update this one

it looks like you're reusing your old weekday branch, which contains commits that got rebased when we fixed the safegraph stuff a month ago. The commits from "29 days ago" looked suspicious, so I checked, and there's already a commit with message "migrate safegraph.run back onto using functools.partial" in main.
Recommend doing the following:

make a new branch off main
cherry pick 3cb534c and 279ed47
delete outdated weekday branch
either submit a new PR with the new branch and close this one, or rename the new branch to weekday and force-push to update this one

Done, see #553


Migration testing: Test 25% random processes

we checked all ingests and found some smaller issues, mostly related to the latest changes regarding the archive ingest:

19_century, Chemist_and_druggist: the task DDS API Call is missing
Archive backlog: change order of Edit METS and Image conversion JPG
Digitisation_on_Demand, Genetics_Books_Project, Glasgow_Crichton_3rd_Party_ftp, Glasgow_Gartnaval_3rd_Party_ftp, LMA_3rd_Party_ftp, Medieval_Manuscripts, RAMC_Books: delete task PDF creation
Migration_project: add tasks Image removal, jpeg deletion after 90 days delay and 15 days delay
Wellcome Audio: add task Image removal at the end


Package verify should validate contents of operator.yaml

If a task definiton doesn't contain appropriate data:
restore:
      resource:
      - restore.yaml

instead of
restore:
      resources:
      - restore.yaml

Maestro doesn't alert the user of incorrect data.

The YAML structure is still not being checked:
operators/repository/cowsay$ kubectl kudo package verify operator/
package is valid
operators/repository/cowsay$ vi operator/operator.yaml
operators/repository/cowsay$ git diff
diff --git a/repository/cowsay/operator/operator.yaml b/repository/cowsay/operator/operator.yaml
index ae88ec6..1f5de42 100644
--- a/repository/cowsay/operator/operator.yaml
+++ b/repository/cowsay/operator/operator.yaml
@@ -10,7 +10,7 @@ tasks:
   - name: app
     kind: Apply
     spec:
-      resources:
+      resource:
         - deployment.yaml
   - name: genwww
     kind: Pipe
operators/repository/cowsay$ kubectl kudo package verify operator/
Warnings
template "deployment.yaml" is not referenced from any task
package is valid
operators/repository/cowsay$

Adding to the 0.12.0 backlog

We validate all required task fields now so that e.g. empty resource list of an Apply task will lead to an error like:
Errors
task validation error: apply task 'app' has an empty resource list. if that's what you need, use a Dummy task instead
Error: found 1 package verification errors

@zen-dog this issue was written up as a package verify task to validate contents of the operator yaml.   The validation you refer to is during plan exec (best I can tell).   I don't see how that is related.
Specifically the yaml is not strict... so the provided example of
restore:
      resource:
      - restore.yaml

was suppose to be:
restore:
      resources:
      - restore.yaml

The term resource is allowed but does not configure resources which was expected.

@kensipe the error above will be shown by the task validation which in turn is called by the BuildVerifier on the client-side.

Oh @zen-dog are you saying it is part of package verify?   without hunting.. that isn't clear

Yep, it is.


[$500] IOU - Receipt shows attachment icon when it is opened right after uploading a receipt

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 1.4.8-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:

Go to staging.new.expensify.com
Go to 1:1 DM
Click + > Request money > Manual
Create a manual receipt without receipt
Click on the IOU preview twice to open IOU details page
Click on the upload receipt window and upload a receipt
Right after uploading the receipt, click on the receipt
Note that it shows attachment icon while the receipt is loading

Expected Result:
The receipt will show a spinner when it is loading
Actual Result:
The receipt attachment icon while it is loading
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android: Native
[x] Android: mWeb Chrome
[ ] iOS: Native
[x] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
Add any screenshot/video evidence

https://github.com/Expensify/App/assets/78819774/c7b47782-8820-43ba-9309-b209b8586cd4
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~01b2f18a17342b1055
Upwork Job ID:<PHONE_NUMBER>439260160
Last Price Increase: 2023-12-05

Seems like regression from https://github.com/Expensify/App/pull/31467

Confirmed regression came from https://github.com/Expensify/App/pull/31467.
cc: @dukenv0307 @mananjadhav @mountiny
Not happening on 0e3a005

Should we assign to @dukenv0307 to fix?

being reverted here so I think we can close this after deploy to staging and verify fix


Sort out how to embedded-postgresql with some extensions

For PM mode, we're enjoying the embedded postgres.
I'd like us to be able to use the Trusted Lang Extension and Apache AGE perhaps.
Can we somehow do the embedded route, but with baked-in extensions? Dunno!

upstream issue: https://github.com/theseus-rs/postgresql-embedded/issues/47

I believe the spirit of this has been addressed (and we are able to load extensions


OpenBLAS 0.2.18 does not compile with PGI

Running into several issues with trying to compile on x86-64 based Linux system with PGI:

lapack/potf2/zpotf2_L.c fails with an 'Illegal lvalue' error:

pgcc -c -O2 -DMAX_STACK_ALLOC=2048 -mp -tp p7-64 -DF_INTERFACE_PGI -fPIC -DDYNAMIC_ARCH -DSMP_SERVER -DUSE_OPENMP -DNO_WARMUP -DMAX_CPU_NUMBER=4 -DASMNAME=zpotf2_L -DASMFNAME=zpotf2_L_ -DNAME=zpotf2_L_ -DCNAME=zpotf2_L -DCHAR_NAME=\"zpotf2_L_\" -DCHAR_CNAME=\"zpotf2_L\" -DNO_AFFINITY -I../.. -DDOUBLE  -DCOMPLEX -DCOMPLEX -DDOUBLE zpotf2_L.c -o zpotf2_L.o
PGC-S-0058-Illegal lvalue (zpotf2_L.c: 71)
PGC/x86-64 Linux Rel Dev-r117179: compilation completed with severe errors
This error also occurs with zpotf2_U.c.
Problem seems to be related to this construct, in common.h:
#if defined(C_PGI) || defined(C_SUN)
#define CREAL(X)        (*((FLOAT *)&X + 0))
#define CIMAG(X)        (*((FLOAT *)&X + 1))
If I change the code in zpotf2_L.c to use the standard C99 complex macros creal() and cimag(), the error goes away.
This problem also affects lapack/lauu2/zlauu2_L.c and lapack/lauu2/zlauu2_U.c.

kernel/x86_64/cdot.c fails to compile:

pgcc -c -O2 -DMAX_STACK_ALLOC=2048 -mp -tp p7-64 -DF_INTERFACE_PGI -fPIC -DDYNAMIC_ARCH -DSMP_SERVER -DUSE_OPENMP -DNO_WARMUP -DMAX_CPU_NUMBER=4 -DASMNAME=cdotc_k_SANDYBRIDGE -DASMFNAME=cdotc_k_SANDYBRIDGE_ -DNAME=cdotc_k_SANDYBRIDGE_ -DCNAME=cdotc_k_SANDYBRIDGE -DCHAR_NAME=\"cdotc_k_SANDYBRIDGE_\" -DCHAR_CNAME=\"cdotc_k_SANDYBRIDGE\" -DNO_AFFINITY -DTS=_SANDYBRIDGE -I.. -DBUILD_KERNEL -DTABLE_NAME=gotoblas_SANDYBRIDGE -UDOUBLE  -DCOMPLEX -DCOMPLEX -UDOUBLE -DCONJ ../kernel/x86_64/cdot.c -o cdotc_k_SANDYBRIDGE.o
PGC-S-0040-Illegal use of symbol, __real__ (../kernel/x86_64/cdot.c: 103)
PGC-W-0156-Type not specified, 'int' assumed (../kernel/x86_64/cdot.c: 103)
PGC-S-0040-Illegal use of symbol, __imag__ (../kernel/x86_64/cdot.c: 104)
PGC-W-0156-Type not specified, 'int' assumed (../kernel/x86_64/cdot.c: 104)
PGC-S-0043-Redefinition of symbol, result (../kernel/x86_64/cdot.c: 104)
PGC-S-0040-Illegal use of symbol, __real__ (../kernel/x86_64/cdot.c: 167)
PGC-W-0156-Type not specified, 'int' assumed (../kernel/x86_64/cdot.c: 167)
PGC-S-0043-Redefinition of symbol, result (../kernel/x86_64/cdot.c: 167)
PGC-S-0040-Illegal use of symbol, __imag__ (../kernel/x86_64/cdot.c: 168)
PGC-W-0156-Type not specified, 'int' assumed (../kernel/x86_64/cdot.c: 168)
PGC-S-0043-Redefinition of symbol, result (../kernel/x86_64/cdot.c: 168)
PGC/x86-64 Linux Rel Dev-r117182: compilation completed with severe errors
make[1]: *** [cdotc_k_SANDYBRIDGE.o] Error 2
This is due to the use of a GNU-specific extension for C99 complex datatypes, which is not supported by PGI:
__real__ result = 0.0 ;
__imag__ result = 0.0 ;
One can fix this by changing the above as follows, to comply with the C99 standard:
result = 0.0 + _Complex_I*0.0;
This also affects kernel/x86_64/zdot.c.
Could you fix these issues for the next release?  Thanks!

Can you find in their documentation how to detect presence of cimag/creal support with ifdef?
Probably fix is one-liner switching recent enough PGI to modern side of C99 things.

1st issue:
_Complex _Imaginary etc are introduced by
#if ((defined(STDC_IEC_559_COMPLEX) macro right above.
It is required to show support for _Complex inside complex.h (plus dozen of quirk cases without mandatory macro defined)
Can you check it is the case with PGI 2016 (2012 is fine with 559 macro)?

Thanks for the response.  I know we do not define this macro in $PGI/linux86-64/include/complex.h, but there is an #include_next directive there which (I think) pulls in the standard /usr/include/complex.h file.  From there, it should be pulling in /usr/include/features.h, which is supposed to define this macro.  For some reason, the following test program I wrote indicates that this macro is still not defined:
$ cat testprog.c
#include <stdio.h>
#include <complex.h>
int main()
{
#if defined (__STDC_IEC_559_COMPLEX)
puts("yep");
#else
puts("nope");
#endif
return 0;
}
$ pgcc -c99 -o testprog testprog.c
$ ./testprog
nope
I'll check with one of our developers to see if I can better understand why this macro is not being defined.

??? lost in markdowns
int main()
{
#if defined (__STDC_IEC_559_COMPLEX__)
puts("yep");
#else
puts("nope");
#endif
return 0;
}

Thanks for the correction - yes, it turns out that we support the macro:
cparrott@sb-parrott ~ $ cat testcmplx.c
#include <complex.h>
#include <stdio.h>

int main()
{
#if defined (__STDC_IEC_559_COMPLEX__)
puts("yep");
#else
puts("nope");
#endif
return 0;
}
tcparrott@sb-parrott ~ $ pgcc -o testcmplx testcmplx.c
cparrott@sb-parrott ~ $ ./testcmplx
yep

But I cannot guess which IFDEF lands PGI compiler with very old PGI compiler and GCC v3

For what it's worth, we can only test as far back as PGI 9.0-0 here - anything before that requires operating system versions that we no longer support, and do not have installed anywhere in our lab.
I tried the test program with PGI 9.0-0, and it worked as expected:
=== compiling with PGI 9.0-0 === /proj/pgi/linux86-64/9.0-0/bin/pgcc -c99 -o testcmplx testcmplx.c ./testcmplx yep
So I would think PGI 9.0-0 would be a reasonable starting point as a cutoff for modern C99 complex support in PGI, as it is probably not likely that many people would still be running something much older than that.
pgcc -V will report the version of PGI being used:
`
$ /proj/pgi/linux86-64/9.0-0/bin/pgcc -V
pgcc 9.0-0 64-bit target on x86-64 Linux -tp nehalem-64
Copyright 1989-2000, The Portland Group, Inc.  All Rights Reserved.
Copyright 2000-2009, STMicroelectronics, Inc.  All Rights Reserved.
`

Could you convert the version number into ifdef statement?

I'm checking on this with our lead developer right now, will follow up when I have more info.

Very sorry, I forgot to follow up on this!  Yes, we do have version macros for testing the current PGI version.  These are PGIC, PGIC_MINOR, and PGIC_PATCHLEVEL.  For the 16.5 release, these would evaluate to 16, 5, and 0, respectively.

Here is a sample program:
danger3% more version_number_C.c
int main(){
int v = __PGIC__;
int m = __PGIC_MINOR__;
int p = __PGIC_PATCHLEVEL__;
printf("The main level __PGIC__ is %d\n",v);
printf("The minor level __PGIC_MINOR__ is %d\n",m);
printf("The patchlevel __PGIC_PATCHLEVEL__ is %d\n",p);
exit();
}

Better this (nd similar measurement right under):
./common.h
  #define openblas_make_complex_xdouble(real, imag)  ((real) + ((imag) * _Complex_I))
+  puts('good')
#else
+  puts('bad')
  #define OPENBLAS_COMPLEX_STRUCT
  typedef struct { float real, imag; } openblas_complex_float;

It looks like at some point ifdefs in common.h got so messed up that superold C_SUN line redefines good intent one screen before.

Bit late to the party, but would something like this work in common.h:
#if defined(C_PGI) || defined(C_SUN)
   #if defined(__STDC_IEC_559_COMPLEX__)
     #define CREAL creal
     #define CIMAG cimag
   #else
    #define CREAL(X)    (*((FLOAT *)&X + 0))
    ...

to get it to use C99 when available (obviating any specific version checks)
coupled with PR #916 to convert the instances of verbatim __real__ , __imag__
in cdot.c/zdot.c to use these definitions like everybody else ?
(Assuming this works, I guess it would make sense eventually to put the #ifdef STDC_IEC_559_COMPLEX on top and handle all the more archaic forms of dealing
with complex numbers in its #else branch - which should remove the need for
GNU extensions support)

Replying to myself, #916 does not help as it would place creal() in the lvalue of the assignment. Best portable solution would appear to be OPENBLAS_MAKE_COMPLEX_FLOAT(0,0), but from 8fade09 (interface/zdot.c line 164) it looks as if that did not work with some versions of MS Visual Studio ?

Compile issues now hopefully addressed correctly in #1126. Accuracy error in DTRSM test so far appears to be specific to HASWELL, not seen with either SANDYBRIDGE or NEHALEM on same (Kaby Lake mobile) hardware. And the message goes away when I drop COMMON_OPT to -O1, so possibly a compiler bug in their "community edition".

Hi Martin,
Thanks for the fix!
There is an open bug internally here involving PGI's handling of inline assembly in the file dtrsm_kernel_RN_haswell.c.  This may correspond to the test failures you are seeing on Haswell with the dtrsm() test.  I will note your observations on our bug tracking system, so hopefully we can escalate this issue and get it fixed soon.
Thanks,
+chris

FYI - this issue has now been elevated to TPR status in our bug tracking system.  It will now be tracked via TPR# 23606.  We will update here when the inline assembly issue has been fixed and included into a future PGI release.

Thanks for the feedback. I imagine that often such mis-optimizations will come about because the compiler misses some form of clue in the code. So if your coworkers happen to come up with some keyword to be added or a slight rearrangement that does not affect performance but would avoid triggering the bug I expect we would be happy to include it to avoid the problem across all versions of your compiler.


fix scale for mobilenet v2

@IRDonch, @eizamaliev  build failed due to the wrong link on VGG model license.
In the list_topologies.yml this file indicated: https://creativecommons.org/licenses/by/4.0/legalcode.txt
Althogh in  VGG16/19 description legal information section
used:
[https://raw.githubusercontent.com/keras-team/keras/master/LICENSE]()
Are you sure that all right?

The link in the VGG docs is incorrect, but that's not why the build failed. It failed because the BIT-Vehicle page is down.

@IRDonch

The link in the VGG docs is incorrect, but that's not why the build failed. It failed because the BIT-Vehicle page is down.
Oh.. do not pay attention on it...
But also for me, the way how to formated links in Legal Information section public models, looks strange... I do not sure that link which maps nothing acceptable... At least they are unclickable.

I think better no format them as hyperlink https://raw.githubusercontent.com/keras-team/keras/master/LICENSE or map it to itself [https://raw.githubusercontent.com/keras-team/keras/master/LICENSE](https://raw.githubusercontent.com/keras-team/keras/master/LICENSE) looks better


SSL Connect error on Windows

Steps to Reproduce
Running rye sync results in [35] SSL connect error on my Windows machine. I tried to set up #348 in my config.toml to see if it had an effect but no luck. I would appreciate any help debugging this behavior. The issue occurs without being connected to any VPN.
Expected Result
No error.
Actual Result
❯ rye sync -v
Bootstrapping rye internals
target dir<EMAIL_ADDRESS>download url: https://github.com/indygreg/python-build-standalone/releases/download/20230826/cpython-3.11.5%2B20230826-x86_64-pc-windows-msvc-shared-pgo-full.tar.zst
Downloading<EMAIL_ADDRESS>error: could not sync because bootstrap failed

Caused by:
    0: failed to fetch internal cpython toolchain<EMAIL_ADDRESS>    1: download of https://github.com/indygreg/python-build-standalone/releases/download/20230826/cpython-3.11.5%2B20230826-x86_64-pc-windows-msvc-shared-pgo-full.tar.zst failed
    2: [35] SSL connect error (schannel: failed to receive handshake, SSL/TLS connection failed)

Version Info
rye 0.14.0
commit: unknown (9da44223d 2023-09-16)
platform: windows (x86_64)
self-python<EMAIL_ADDRESS>symlink support: false

I am on Windows 11 Enterprise
Stacktrace
No response

I'm assuming you have a corporate proxy set by your system administrator. Could this be?

It could be, I'm actually not sure. I recently started having SSL/TLS issues on this Window machine and I'm still investigating what's going on. E.g. after upgrading to Chocolatey 2.0 I ran into this error. I followed the steps described but it didn't solve the issue.
I realize this is not a specific rye issue, but would you mind pointing me to the code responsible for downloading cpython that produced the error above? It'll help me investigate what's going on.

The specific download is taking place here: https://github.com/mitsuhiko/rye/blob/26931a97776e25ca34aab0c54e5d0ba67d53ee5d/rye/src/bootstrap.rs#L393-L454
The failing call is here: https://github.com/mitsuhiko/rye/blob/26931a97776e25ca34aab0c54e5d0ba67d53ee5d/rye/src/bootstrap.rs#L442-L444
I'm assuming an SSL certificate is presented which curl (via schannel) has not loaded.

Thanks! I will investigate a bit more and let you know if I make any progress.

@mitsuhiko, I encounter a similar error with my corporate MITM proxy (Cisco Umbrella):
~ on ☁  (us-east-1)
❯ rye self update
Updating to latest
error: could not download release latest for this platform

Caused by:
    0: download of https://github.com/mitsuhiko/rye/releases/latest/download/rye-x86_64-windows.exe failed
    1: [35] SSL connect error (schannel: next InitializeSecurityContext failed: Unknown error (0x80092012) - The revocation function was unable to check revocation for the certificate.)

~ on ☁  (us-east-1) took 2s

Something similar happens with a default conda/mamba installation that is set to use schannel. It uses the native certificates, but also does a revocation check. To get mamba to work, we need to set an option to prevent curl from doing the SSL revocation check. This may be needed for rye on corporate machines that are running these types of MITM proxies. Either that or an option to bypass SSL. (I know that sounds bad, but the Cisco Umbrella actually does all the SSL checks itself.)
I do have a copy of the 3rd party certificate. So, it would also work if I could configure rye to use that certificate file. I didn't see an option like that in the documentation, but maybe I missed it.
Anyway, I like where you are going with rye. On the same machine, I'm able to install and download toolchains under WSL in Ubuntu. But I think we would all like to get it working in the native Windows environment.
Thanks for all your efforts and contributions!

By bypass SSL, I mean bypass validation. But still use the https:// prefix.

It could be, I'm actually not sure. I recently started having SSL/TLS issues on this Window machine and I'm still investigating what's going on. E.g. after upgrading to Chocolatey 2.0 I ran into this error. I followed the steps described but it didn't solve the issue. Note that I can always download the url directly from my webbrowser (Firefox/Chrome).
I realize this is not a specific rye issue, but would you mind pointing me to the code responsible for downloading cpython that produced the error above? It'll help me diagnose the problem.

then please close. the only rye-related "issue" is how it might proxy settings.

I believe this is resolved in latest version for windows users. I'm closing this for now. Please reopen if this still happens in 0.18.


Add 'tcpdump' as a tool to BWCTL

From @arlake228 on March 14, 2015 12:17
Original issue 902 created by arlake228 on 2014-05-19T09:54:54.000Z:
This is a 'what if' bug, close if it doesn't make technical sense.
It was mentioned in a workshop yesterday that being able to invoke tcpdump (headers only, specific interface that traffic is traversing) via bwctl would be useful.
This bug is to figure out if it would be possible.
Copied from original issue: perfsonar/project#900

Comment #2 originally posted by arlake228 on 2014-05-19T11:58:41.000Z:
I think that would be the common and constrained use case (e.g. we would know ports involved, and could use that to limit what tcpdump is dumping).
I think just being able to run it absent of a performance test may be valuable, but I am not sure how valuable.
The other nagging question is related to the data this would generate, and how to manage that (e.g. a 10 second 10g iperf may be MBs in size uncompressed).

Comment #1 originally posted by arlake228 on 2014-05-19T11:52:42.000Z:
Is the idea that, along with the normal iperf/etc results, you'd get a tcpdump showing the traffic for the test?

Comment #4 originally posted by arlake228 on 2014-05-20T07:37:39.000Z:
10s iperf3 test on a 10G host:
[rootjz@star-pt1 ~]# ls -hals test1.dmp
825M -rw-r--r-- 1 root root 825M May 20 00:33 test1.dmp
[rootjz@star-pt1 ~]# ls -hals test1.dmp.gz
3.1M -rw-r--r-- 1 root root 3.1M May 20 00:33 test1.dmp.gz
20s iperf3 test on a 10G host:
[rootjz@star-pt1 ~]# ls -hals test2.dmp
1.2G -rw-r--r-- 1 root root 1.2G May 20 00:36 test2.dmp
[rootjz@star-pt1 ~]# ls -hals test2.dmp.gz
4.5M -rw-r--r-- 1 root root 4.5M May 20 00:36 test2.dmp.gz
So I guess the good news is that it compresses well.

Comment #3 originally posted by arlake228 on 2014-05-19T12:18:06.000Z:
Yeah, the general case would be fraught with security implications, though, given the protocol, probably a bit easier than having bwctl return two different sets of result data.
As to the 10g iperf, i'd be curious how big it will be compressed since we could probably set it up to compress the results before shipping back to the client. Either way, i'd presume they'd be big, but i'd be curious how big.

Comment #5 originally posted by arlake228 on 2015-01-27T14:39:53.000Z:
<empty>


Nebra Rock pi Indoor - beacon/witness pattern: can witness fine until a beacon is sent

The issue I am having is similar to this issue posted in 2021, I  am located in Australia if that makes any difference. Since Australia had a few frequency changes in the last about 9 months. AU915 to AS923 and back to AU915.
#https://github.com/helium/miner/issues/1133
I just recently onboarded in new Nebra Rock pi indoor version. The original firmware was 2<IP_ADDRESS> (4492c46) and asserted location.. Even after 24 hours of leaving it connected to the internet (ethernet), the dashborad still said " Awaiting Location Assertion"
I updated the firmware to the latest version from hub.nebra.com and now on the dashboard all looks OK, but the hotspot stops witnessing as soon as it beacons. The sequence is as follows

hotspot witnesses other beacons
hotspot sends own beacons
hotspot own beacon returns 0
hotspot stops witnessing

If i restart the hotspot the same sequence repeats. If I leave it ON and connected, it tries to beacon every 6 hours, but gets no witnessing. Also I have checked heliumgeek and moken explorer, the hotspot does not witness anything selected or unselected or invalid.

Hotspot looks fine to me and is still witnessing https://heliumtracker.io/hotspots/1026512

Ticket 430636

thanks for looking into it. It does not look fine to me at all. It only witnesses when I restart the hotspot and then stops witnessing after it beacons. https://explorer.moken.io/hotspots/112RWc6zYvRQLdZari4tAhusu6NVyiNU5QgEDHGZgNpRmD69HNTy/activity?activityRange=12H&selectedActivities=Witness,Beacon&selectedLayer=hotspots
If i flash an old firmware it keeps on working for say an hour and then as soon as it updates to new firmware, it stops soon after when it beacons.

Look at helium tracker link I sent. It looks fine.
Also, I see you have 5dbi antenna asserted.
Make sure antenna is connected properly.

Thanks for your message. I have been looking at helium tracker and the moken explorer as well for the past 1 week.  Helium tracker does not tell the full story. Even with helium tracker you can see that once the hotspot beacons there is no more witnesses. Below is the screenshot of moken explorer taken now 6.16am AEST ( sydney time), you can clearly see that it was witnessing after a restart and once it beacons it stops witnessing totally ( selected or unselected).  I also checked the Antenna nothing wrong it is a  6 dbi antenna on the roof, so I entered 5 dbi to account for cable loss. Also we can rule out the antenna, as the antenna works fine as soon as we restart the unit and also with an older version of firmware.

The firmware hasn't changed between v1.0.2-2 and v1.0.2-52 ... gateway-rs from helium is still the same as is the packet forwarder and there is no other changes there.
Your miner is also on correct frequency of au915.
It's almost certainly a loose antenna, concentrator or cables to the antenna.

Also rebooting it constantly likely is not helping things

Thanks for your reply/ I checked the antenna connectors and plugged it back in. Hopefully it helps. Is there any way to stop automatic updates. I want to run an older version of the firmware in order to troubleshoot this issue.

There is no way to turn off the updates I'm afraid.
In any case, as I said nothing has changed on the software that could cause what you are describing.

Thanks for your help Aaron in troubleshooting my hotspot. I can confirm that the problem has nothing to do with the firmware. The hotspot has been running fine for the last about 24 hours with the latest firmware.
It looks like the problem was over heating of the most likely the concentrator module. I felt the hotspot was too hot by touching, so I removed the plastic cover off and put a small fan next to it and it has been running fine since then.

Thanks for the update and good to know it's working ok now 👍


Is there a way to use the withAboutSpecial to create a web link?

Is there a possibility to make a special link directly to a web page? Or any other way to make a clickable weblink with the library?
Thanks a lot for all the work :)

Do you mean for every single library? or a top level button action?

Apologies - a button at the top I can use to go to a web URL - where I can give more detail about my project on my own website.
Thanks for the prompt reply!

So you'd enable the special button by setting it's name:
https://github.com/mikepenz/AboutLibraries/blob/bb3eb47a2f5a7358a6b7ffe8a616833c682ec378/library/src/main/java/com/mikepenz/aboutlibraries/LibsBuilder.kt#L294
and then you'd set the special button listener here:
https://github.com/mikepenz/AboutLibraries/blob/bb3eb47a2f5a7358a6b7ffe8a616833c682ec378/library/src/main/java/com/mikepenz/aboutlibraries/LibsConfiguration.kt#L18
https://github.com/mikepenz/AboutLibraries/blob/bb3eb47a2f5a7358a6b7ffe8a616833c682ec378/library/src/main/java/com/mikepenz/aboutlibraries/util/LibsListenerImpl.kt#L24-L26

King. I'll look into this tomorrow, thanks a ton for the help :)

Perfect, cracked it.
Huge thanks for this man, it's a fantastic library, especially for a newb codemonkey like me!

you are welcome :) glad it works


How should I use my dataset

For your own data, you can

generate HR-LR pairs and put them separately in two folders, LR_folder and HR_folder, with the same name.
modify config.py file with your LR/HR folder_path.

Thank you for your reply,I try it now.

Hi,you use the validation set, do you need test sets?

I did not implement evaluation code, but simply test code for single image. It should be easy to implement in a similar structure.

ok,thanks a lot.

do you use test sets in this code? @aipeteryao


Session object for Auth class

Hi,
first of all, thanks for the library. I'm having issue with the Auth class in identity.web package. The session object, that is one of the parameters, is described as what you can pass to it, but doesn't explicitly say what keys are actually required. The reason why I'm asking is because I'm using Tornado, and it would be really helpful to have more info than "if you're using Flask pass x and if you're using Django pass y".
Thanks again for the effort.

The session parameter is documented as a dict-like object. It does not "require" any keys to be preexist. The Auth object will maintain several key-value pairs it needs.
Currently, we mentioned Flask's session and Django's session, only because they are popular web frameworks in Python. But any other dict-like object in other web framework should work.
Tornado does not seem to provide a session API out-of-box. But perhaps you can somehow wrap its RequestHandler.get_secure_cookie() into a dict-like helper, and feed it into identity.web's Auth. Let us know how that goes. :-)

Thanks for the answer. I'm still having issues on what interface should be implemented as I can just inherit a Dictionary but apparently I cannot just give the dict or dict-like object (I tried that and I get an error like "Invalid grant - state missing from auth_code_flow"). From your source code, I can see that you're using the following keys in the dictonary:
    _AUTH_FLOW = "_auth_flow"
    _USER = "_logged_in_user"```

which probably means that I need to provide those three values in that dictionary in order for session object to be correct or maybe I'm misinterpreting something?

Sorry again for the hassle...

This library will write and subsequently maintain its data using those keys. But that is an implementation detail. You as the caller does not need to "provide those three values".  All you need is to provide a dict-like interface. Now, "dict-like" means both get and set. Does your dict-like object actually implement the __setitem__() to properly write the data back?

Understood. I guess I'll need to do my part of implementing session management myself or using 3rd party prior to using Auth object.
Thanks again.

That state parameter is defined in the documentation as optional, so, Auth class will automatically generate a random one and stick to it, or Auth will accept whatever input from you and still stick to it. In either case, you are not supposed to run into a "state mismatch" error. Now that you did run into that error, I believe your question of "what should that state hold" is irrelevant.
Can you double check your still-in-development homebrew session module first? Unit test it to make sure it does persist data?

That might be connected to the issue of Tornado being multithreaded async and not having a global store (I mean it has, but it's application wide). I need to keep Auth object persistent for the session for multiple tenants (clients) and one of the options is keeping it in the database.
However, I see that you're using caches (which might complicate things if they are continuously updated)for requests so my question is: is it possible to serialize and store Auth object and every time I need it deserialize it and use it for e.g log in and log out? Or is it mandatory for Auth object to reside in memory all the time?

Even if you could serialize the Auth object, where would you put it? A centralized database? Then why don't you just implement the session into your database (and your probably should), and everything should just work out of the box? From the very beginning, Auth object's document tells you to provide a session which, by definition, should be something that persists across different requests. I think the direction to go is to find or build a session mechanism for Tornado, and then plug it into Auth. We can go from there.


Feat: Add Form.Item to ComponentConfig

[ ] I'm sure this does not appear in the issue list of the repository

Basic Info
What are the similar cases of this feature
Hi, we currently want to customize our Form.Item behavior globally to normalize with customized string trimming. Would it be possible to add Form.Item to the list of configurable components so we can do this?
What problem does this feature solve?
Allows us to modify Form.Item props globally with ConfigProvider

Using normalize on input instead


Refactor: delete Tensor::mat2vec()

This method is initially proposed to flatten and clone mat to vector.
However, Tensor::getData() can cover most of it's purpose, it seems
no longer in need.
Changes proposed in this PR:

Delete Tensor::mat2vec().

See also:
https://github.com/nnstreamer/nntrainer/pull/149#discussion_r439172191
Self evaluation:

Build test: [X]Passed [ ]Failed [ ]Skipped
Run test: [X]Passed [ ]Failed [ ]Skipped

Signed-off-by: Jihoon Lee<EMAIL_ADDRESS>

:memo: TAOS-CI Version: 1.4.20191203. Thank you for submitting PR #162. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnsuite.mooo.com/.


python3Packages.ihatemoney: fix CVE-2020-15120

Motivation for this change
fix CVE-2020-15120
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

Maybe update to the new release instead.
https://github.com/spiral-project/ihatemoney/releases/tag/4.1.5

No we are already at 4.2 and it's not possible to downgrade (due to sqlite not being able to drop columns). The next release is 5.0 but is not ready yet.

@GrahamcOfBorg test ihatemoney


ui: Reload resources on socket reconnect

Signed-off-by: Ryan Northey<EMAIL_ADDRESS>Fix #121

Codecov Report

Merging #181 (a559086) into main (9b33fc6) will decrease coverage by 0.52%.
The diff coverage is 13.51%.

@@            Coverage Diff             @@
##             main     #181      +/-   ##
==========================================
- Coverage   43.41%   42.89%   -0.53%
==========================================
  Files          91       91
  Lines        2499     2532      +33
==========================================
+ Hits         1085     1086       +1
- Misses       1414     1446      +32

Impacted Files
Coverage Δ

ui/src/app/socket.js
12.29% <11.11%> (-0.10%)
:arrow_down:

ui/src/app/store.js
11.39% <11.11%> (-1.62%)
:arrow_down:

ui/src/app/playground.js
41.66% <100.00%> (ø)

Continue to review full report at Codecov.

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


Setting NewtonSoft MaxDepth in generators

Fixes #6405

Pull Request Test Coverage Report for Build 311537

0 of 0   changed or added relevant lines in 0 files are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.003%) to 79.038%

Files with Coverage Reduction
New Missed Lines
%

/libraries/Microsoft.Bot.Connector.Streaming/Application/TimerAwaitable.cs
1
68.25%

Totals

Change from base Build 311238:
-0.003%

Covered Lines:
25526

Relevant Lines:
32296

💛  - Coveralls

I think we are missing updating the generators\vsix-vs-win templates too, right?
nit, it may be good to add a comment on why maxdepth is set but not sure

Updated VSIX.  I'm open to suggestions for a comment.  Or not.

@EricDahlvang @gabog Per Erics suggestion, I am now using HttpHelper.BotMessageSerializerSettings.MaxDepth instead of hard coding 128.
Which means... I changed the MaxDepth on BotMessageSerializerSettings to be 128.

:heavy_check_mark: No Binary Compatibility issues for Microsoft.Bot.Builder.Integration.AspNet.Core.dll


exporter: only profile exporter, skip ebpf compilation

The ebpf compilation skews the overall picture of cpu usage by Kepler. This PR starts profiling after kepler is fully initialized so we can focus on the runtime cpu usage.

/lgtm


<View /> snippet should have a second tab trigger inside the component

It would be nice if the tab trigger would be like this:
<View style={styles.${1}}>${2}</View>
That way, I can start typing the View's children components as soon as possible.
More than willing to submit a PR for this if wanted.

Good call, will add that now. Cheers!


GLTF import = missing material

Godot version
3.3.2 stable
System information
Windows 10
Issue description
When importing GLB including several meshes, materials, textures (w/ alpha), and camera, only one material is displayed in Godot by default, the rest is not taken into account. When I try to import the same GLTF on https://gltf-viewer.donmccurdy.com/ all the materials are displayed properly. Screenshots attached.

Steps to reproduce
I model and texture in Blender and export using glTF 2.0 exporter with these settings :

In Godot, I simply double-click the GLB file to open it.
Minimal reproduction project
No response

@Henrydice Please upload a minimal reproduction project to make this easier to troubleshoot.

It would be helpful to have a test project to debug.

After cleaning the import files and redoing the whole process of export from Blender/import to Godot it finally works and takes into account all the materials. I therefore close the issue, sorry for the inconvenience and thanks for the quick feedbacks.