Service Fabric & ASPNET Core 2.0 fails to run/start site

when creating a Service Fabric application using a ASP.NET Core 2.0 Stateless template,
the site fails to run under service fabric.
https://github.com/aspnet/Hosting/issues/1124
github repo https://github.com/grahamehorner/ServiceFabric.ASPNETCORE2.git

I think this is a Service Fabric issue.
See Partition is below target replica or instance count. https://github.com/Azure/service-fabric-issues/issues/39

@campersau it may be, however I'm only seeing this when attempting to host ASPNET Core 2.0 applications; ASPNET Core 1.1 work fine; and the source code is exactly the same only expectation is that newer nugget packages having been referenced.

Are you using the All package? Do you have the 2.0 runtime store installed on the nodes?

@benaadams I'm running the ServiceFabric on a local developer install using a temple generated, then updated the nugget packages to preview3

@campersau @benaadams I can confirm that ASPNET Core 1.1 template generated Stateless Web Applications work with Service Fabric using the Service Fabric v2.7-preview5718 and the issues I'm seeing are when using the ASPNET Core 2.0 preview2-final & preview3-latest, I've also added the Microsoft.NETCore.All
I think this could be a result of a silent exception
System.TypeLoadException: 'Method 'StartAsync' in type 'Microsoft.AspNetCore.Server.WebListener.MessagePump' from assembly 'Microsoft.AspNetCore.Server.WebListener, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60' does not have an implementation.'

more info switched to KestrelCommunicationListener and UseKestral and ServiceFabric reports the node
Error event: SourceId='System.FM', Property='State'.
Partition is below target replica or instance count.
fabric:/UtiliteamGB.Xanadu/ManagementPortal 1 1 da376ff5-82b5-45f2-b026-77544628f081
IB _Node_3 Up<PHONE_NUMBER>60293288
(Showing 1 out of 1 replicas. Total available replicas: 0.)
however running the code as a Kestral console application gives
Exception thrown: 'System.IO.IOException' in mscorlib.dll
An unhandled exception of type 'System.IO.IOException' occurred in mscorlib.dll
Failed to bind to address http://<IP_ADDRESS>:5000: address already in use.
as it is running in service fabric

Debug output shows
Creating application...
...
The application has started.
Service Status:
fabric:/.. is not ready, 1 partitions remaining.
Service Status:
fabric:/... is ready.
The application is ready.
Finished executing script 'Get-ServiceFabricApplicationStatus'.
Time elapsed: 00:00:02.7763707
'ManagementPortal.exe' (CLR v4.0.30319: DefaultDomain): Loaded 'C:\Windows\Microsoft.Net\assembly\GAC_64\mscorlib\v4.0_<IP_ADDRESS>__b77a5c561934e089\mscorlib.dll'. Symbols loaded.
'...' (CLR v4.0.30319: DefaultDomain): Loaded 'C:\SfDevCluster\Data_App_Node_3...'. Symbols loaded.
The program '[14208] ...' has exited with code 0 (0x0).
'...' (CLR v4.0.30319: DefaultDomain): Loaded 'C:\Windows\Microsoft.Net\assembly\GAC_64\mscorlib\v4.0_<IP_ADDRESS>__b77a5c561934e089\mscorlib.dll'. Symbols loaded.
'...' (CLR v4.0.30319: DefaultDomain): Loaded 'C:\SfDevCluster\Data_App_Node_3...'. Symbols loaded.
The program '[12804] ManagementPortal.exe' has exited with code 0 (0x0).
'...' (CLR v4.0.30319: DefaultDomain): Loaded 'C:\Windows\Microsoft.Net\assembly\GAC_64\mscorlib\v4.0_<IP_ADDRESS>__b77a5c561934e089\mscorlib.dll'. Symbols loaded.
'...' (CLR v4.0.30319: DefaultDomain): Loaded 'C:\SfDevCluster\Data_App_Node_3...'. Symbols loaded.
is interesting because a Thread.Sleep(Timeout.Infinite) shouldn't allow the application to exit ?

Failed to bind to address http://<IP_ADDRESS>:5000: address already in use.

You want to pass the address/port/FQDN that SF gives you and listen on that, not <IP_ADDRESS>:5000 (hence port clashes)

@benaadams yes; it does, it code is take the url, but still can't get the service to run? has anyone got a project running with ASPNET Core 2 ? using the template generated code ?
        protected override IEnumerable<ServiceInstanceListener> CreateServiceInstanceListeners()
        {
            return new ServiceInstanceListener[]
            {
                new ServiceInstanceListener(serviceContext =>
                {
                    return new  WebListenerCommunicationListener(serviceContext, "ServiceEndpoint", (url, listener) =>
                    {
                        ServiceEventSource.Current.ServiceMessage(serviceContext, $"Starting WebListener on {url}");
                        return new WebHostBuilder()
                            // .UseWebListener()
                            .UseHttpSys()
                            .ConfigureServices(
                                services => services
                                    .AddSingleton<StatelessServiceContext>(serviceContext))
                            .UseContentRoot(Directory.GetCurrentDirectory())
                            .UseStartup<Startup>()
                            .UseServiceFabricIntegration(listener, ServiceFabricIntegrationOptions.None)
                            .UseUrls(url)
                            .Build();
                    });
                })
            };
        }

    protected override IEnumerable<ServiceInstanceListener> CreateServiceInstanceListeners()
    {
        return new ServiceInstanceListener[]
        {
            new ServiceInstanceListener(serviceContext =>
            {
                return new  WebListenerCommunicationListener(serviceContext, "ServiceEndpoint", (url, listener) =>
                {
                    ServiceEventSource.Current.ServiceMessage(serviceContext, $"Starting WebListener on {url}");
                    return new WebHostBuilder()
                        // .UseWebListener()
                        .UseHttpSys()
                        .ConfigureServices(
                            services => services
                                .AddSingleton<StatelessServiceContext>(serviceContext))
                        .UseContentRoot(Directory.GetCurrentDirectory())
                        .UseStartup<Startup>()
                        .UseServiceFabricIntegration(listener, ServiceFabricIntegrationOptions.None)
                        .UseUrls(url)
                        .Build();
                });
            })
        };
    }

this is a bit odd as I've not specified the port 5000 ?! anywhere

Q: is there a symbol server for ASPNET Core 2.0 ?

@benaadams @DamianEdwards @davidfowl The issue seems to be that Start, StartAsync doesn't run the service; however Run, RunAsync works?

Just updated to the ASP.NET Core 2.0.0-preview3-26054 packages; and this looks to have resolved whatever the root cause was ?

@benaadams @davidfowl @DamianEdwards I spoke too soon :'( trying to findout why with no code changes this broke this morning ?

@grahamehorner did you get this running in ASP.NET Core 2.0? I'm having the same issue as you. I update my SF runtime with the latest and installed the .NET Core 2.0 Runtime & SDK.
I'm getting the following exception when trying to create the WebListener:
Method 'StartAsync' in type 'Microsoft.AspNetCore.Server.WebListener.MessagePump' from assembly 'Microsoft.AspNetCore.Server.WebListener, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60' does not have an implementation.
protected override IEnumerable<ServiceInstanceListener> CreateServiceInstanceListeners()
      {
          return new ServiceInstanceListener[]
          {
              new ServiceInstanceListener(serviceContext =>
                  new WebListenerCommunicationListener(serviceContext, "ServiceEndpoint", (url, listener) =>
                  {
                      ServiceEventSource.Current.ServiceMessage(serviceContext, $"Starting WebListener on {url}");
                      return new WebHostBuilder()
                                  .UseWebListener()
                                  .ConfigureServices(
                                      services => services
                                          .AddSingleton<StatelessServiceContext>(serviceContext))
                                  .UseContentRoot(Directory.GetCurrentDirectory())
                                  .UseStartup<Startup>()
                                  /* .UseApplicationInsights() */
                                  .UseServiceFabricIntegration(listener, ServiceFabricIntegrationOptions.None)
                                  .UseUrls(url)
                                  .Build();
                  }))
          };
      }

UseWebListener has been renamed to UseHttpSys. It seems that you are still referencing WebListener 1.0.0. See if that fixes anything?

@jkotalik @codematrix yes, I managed to get everything working with some help for the team; it was some changes in the hosting packages that broke a few things, all good now.
BTW I'm personally using kestrel as this is now the default in the newer templates.

@jkotalik @grahamehorner Awesome! UseHttpSys did the trick. UseKestrel() works just as well and will be using instead if this is the default.
Thanks for the speedy response. Much appreciated.
codematrix

Yeah, both Kestrel and HttpSys have the ups and downsides. I'd recommend checking out this to understand the difference between HttpSys and Kestrel.

Currently if neither UseKestrel or UseHttpSys is called, we default to UseKestrel.

Hey @jkotalik thanks for the link. It's always good to understand the differences be web service implementations.

This is still an issue with Microsoft.AspNetCore.Server.WebListener, Version=<IP_ADDRESS>:
System.TypeLoadException: 'Method 'StartAsync' in type 'Microsoft.AspNetCore.Server.WebListener.MessagePump' from assembly 'Microsoft.AspNetCore.Server.WebListener, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60' does not have an implementation.'

Method does not have an implementation. generally means you have mixed versions of libs; so for example Hosting is expecting a different version than the Server

@codematrix @jkotalik the link has some old information with regards Kestrel and will no doubt be updated soon; kestrel is has been hardened so is now supported as for edge servers; it is also going to be allowed to run inside IIS (inproc) and will support HTTP2 (@DamianEdwards) tonights #live.asp.net

I am using the package Microsoft.ServiceFabric.AspNetCore.WebListener v 2.7.198 and it has dependency on
Microsoft.AspNetCore.Server.WebListener >= v 1.0.0. It doesn't work with neither 1.0.0 nor with <IP_ADDRESS>. It is exactly the same error folks reported above.

Why is this thread closed if the issue has not been resolved? It seems an apparent bug to me.

inside my ServiceFabric projects .csproj file is the following
  <ItemGroup>
    <PackageReference Include="Microsoft.AspNetCore" Version="2.0.0" />
    <PackageReference Include="Microsoft.AspNetCore.Mvc" Version="2.0.0" />
    <PackageReference Include="Microsoft.AspNetCore.Mvc.Razor.ViewCompilation" Version="2.0.0" PrivateAssets="All" />
    <PackageReference Include="Microsoft.AspNetCore.StaticFiles" Version="2.0.0" />
    <PackageReference Include="Microsoft.ServiceFabric" Version="5.7.198" />
    <PackageReference Include="Microsoft.ServiceFabric.AspNetCore.Kestrel" Version="2.7.198" />
    <PackageReference Include="Microsoft.ServiceFabric.AspNetCore.WebListener" Version="2.7.198" />
    <PackageReference Include="Microsoft.ServiceFabric.Data" Version="2.7.198" />
    <PackageReference Include="Microsoft.ServiceFabric.Services" Version="2.7.198" />
    <PackageReference Include="Microsoft.VisualStudio.Web.BrowserLink" Version="2.0.0" />
  </ItemGroup>

I'm able to run both using Kestrel and WebListerner without any issues

Below is mine:
 <ItemGroup>     <PackageReference Include="Microsoft.AspNetCore" Version="2.0.0" />     <PackageReference Include="Microsoft.AspNetCore.Mvc" Version="2.0.0" />     <PackageReference Include="Microsoft.AspNetCore.Mvc.Razor.ViewCompilation" Version="2.0.0" PrivateAssets="All" />     <PackageReference Include="Microsoft.AspNetCore.StaticFiles" Version="2.0.0" />     <PackageReference Include="Microsoft.Cloud.InstrumentationFramework" Version="<IP_ADDRESS>" />     <PackageReference Include="Microsoft.Extensions.Caching.Memory" Version="2.0.0" />     <PackageReference Include="Microsoft.ServiceFabric" Version="5.7.198" />     <PackageReference Include="Microsoft.ServiceFabric.AspNetCore.Kestrel" Version="2.7.198" />     <PackageReference Include="Microsoft.ServiceFabric.AspNetCore.WebListener" Version="2.7.198" />     <PackageReference Include="Microsoft.ServiceFabric.Data" Version="2.7.198" />     <PackageReference Include="Microsoft.ServiceFabric.Services" Version="2.7.198" />     <PackageReference Include="Microsoft.VisualStudio.Web.BrowserLink" Version="2.0.0" />   </ItemGroup>
I can run Kastrel, but not WebListener:
//this works return new WebHostBuilder()                             .UseKestrel()                             .ConfigureServices(                                 services => services.AddSingleton(serviceContext))                             .UseContentRoot(Directory.GetCurrentDirectory())                             .UseServiceFabricIntegration(listener, ServiceFabricIntegrationOptions.None)                             .UseStartup<Startup>()                             .UseUrls(url)                             .Build();                         })) //this fails return new WebHostBuilder()                             .UseWebListener()                             .ConfigureServices(                                 services => services.AddSingleton(serviceContext))                             .UseContentRoot(Directory.GetCurrentDirectory())                             .UseServiceFabricIntegration(listener, ServiceFabricIntegrationOptions.None)                             .UseStartup<Startup>()                             .UseUrls(url)                             .Build();

@apryiomka WebListener was renamed to HttpSys in 2.0.

@apryiomka In 2.0.0 wave, WebListener is not HttpSys. The issue is resolved in 2.0.

HttpSys is a separate package. What do you "renamed"?
https://www.nuget.org/packages/Microsoft.AspNetCore.Server.HttpSys/

There is no 2.0 version for WebListener https://www.nuget.org/packages/Microsoft.AspNetCore.Server.WebListener/ v1.1.2 is the latest

@apryiomka @Tratcher and I are talking about our webserver rename. Please see https://github.com/aspnet/Announcements/issues/211 for the rename.
The problem seems to be with the package Microsoft.ServiceFabric.AspNetCore.WebListener (this is the specific ServiceFabric package).
@glennc Do you know if ServiceFabric folks plan to release a 2.0.0 version with HttpSys support?
@apryiomka are you able to use Kestrel instead?

@muratg, can i use Kestrel with internet facing app/service? It works locally with my local cluster, but doesn't seem to work when i deploy to azure cluster. If i remote the node, i can access it from the node directly and local ip address (<IP_ADDRESS>:9012), but if i try http://clustername.centralus.cloudapp.azure.com:9012, it doesn't work. I am more than happy with stick with Kestrel, but i don't seem to be able to access it from the internet.

@apryiomka @codematrix @muratg I think some confusion here is due to the naming of the package; but the team are aware of this https://github.com/Azure/service-fabric-aspnetcore/issues/15

I can successfully use Kestrel as well.

Two Changes Needed to resolve it:

Use this package Microsoft.AspNetCore.Server.HttpSys  (I used version 2.0.2)
In WebHostBuilder, replace .UseWebListener() with .UseHttpSys()

I hope you already know how to configure ApplicationManifest.xml and ServiceManifest.xml to bind SSL policy. if not see this link which has old implementations but the xml configs are still same : https://dzimchuk.net/setting-up-https-endpoints-in-asp-net-core-services-in-service-fabric/


tabs标签页位置设置为left的时候，prefix和suffix位置没有同步切换到标签列的上下位置，还是在标签列的左右两边

Describe the bug
tabs标签页位置设置为left的时候，prefix和suffix位置没有同步移动到上下位置，还是在标签页的左右两边

Steps to reproduce
无
Link to minimal reproduction
无
System Info
无

Used Package Manager
npm
Validations

[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion.
[X] The provided reproduction is a minimal reproducible example of the bug.

请提供最小复现

This issue does not have any recent activity. If you are still experiencing similar problems, open a new error, including a minimal copy of the problem


Header Tweaks

Header needs to be tweaked slightly:

[ ] Update Style Guide Header and Release
[ ] Left align the logo
[ ] Right align the nav links
[ ] Slightly smaller padding between the nav links
[ ] Update Style Guide Library in xyz
[ ] Able to set the logo to be wrapped in an anchor tag and linked

Figma: https://www.figma.com/design/DVjqjKrUWwFzgZzXCsVdGv/Frequency_Marketing-Site_?node-id=2210-1385&node-type=canvas&t=JvEcfiRCowfKicGx-0

@claireolmstead I added one more thing to the list if you haven't fully started "Able to link the logo" As I keep trying to click on it to go to a place.

@wilwade I added an href to the header so all you have to do is add that in. wasn't sure if we ever wanted to link externally (seems weird) so not sure if we also need a target or something. Since the release isn't cut, I didn't update the version in the .xyz site.


requirements.txt - Use minimal instead of exact versions

requirements.txt - Use minimal instead of exact versions
As suggested in #174
I took the current version in requirements.txt (assuming nothing breaks with them) and changed them to the minimal version.
Downgrading to the real minimal version that works at this point is too complicated and might introduce new undiscovered bugs.
Tested locally on a fresh virtualenv.

This was so needed! Merging and uploading into pypi asap


Macros key positions are incorrect for the Adv360

Added some macros to my adv360 layout, some did not work. Example:

results in
backslash {
    bindings = <&kp BACKSLASH>;
    key-positions = <31 51>;
    timeout-ms = <50>;
    layers = <0 2 3>;
};

but according to the keymap taken by urob's config, the Adv360 has this map:
/*               ADVANTAGE 360 KEY MATRIX
 ╭────────────────────────────┬────────────────────────────╮
 │  0   1   2   3   4   5   6 │  7   8   9  10  11  12  13 │
 │ 14  15  16  17  18  19  20 │ 21  22  23  24  25  26  27 │
 │ 28  29  30  31  32  33  34 │ 35  36  37  38  39  40  41 │
 │ 42  43  44  45  46  47 ╭───┴───╮ 48  49  50  51  52  53 │
 │ 54  55  56  57  58╭────╯       ╰────╮59  60  61  62  63 │
 ╰───────────────────┼────────┬────────┼───────────────────╯
                 ╭───╯ 64  65 │ 66  67 ╰───╮
                 │ 68  69  70 │ 71  72  73 │
                 ╰───────╮ 74 │ 75 ╭───────╯
                         ╰────┴────╯ */

According to the combo, it's selecting two keys on opposite sides of the board. I haven't looked through the codebase for a solution but I at least wanted to make the bug known. Great work on this tool btw!

So, uh, the Advantage 360 layout has been a bit of a mess but:

Are you sure that you're using urob's matrix transform?
The pre-defined layout wouldn't be set up to match that matrix transform (a simple bit crucial example is the omission of foot pedals). You could try adjusting it or creating a new one with the help of https://nickcoutsos.github.io/keymap-layout-tools/ but it's not trivial so it's worth validating your other options first.

Any news on this @RATIU5?

My config was a direct fork from Kinesis' repo. The only thing I did was open the repo inside your layout tool and configured away. I referenced urob's matrix after I realized the combos did not work correctly.
I don't believe I used his matrix transforms in this project, I was however using them in a different repo.

Based on the current JSON in the repo you would use these key positions:

Note that some positions are skipped -- my understanding is that those positions in the transformation matrix are intended for foot pedals but aren't displayed in the editor.


renoise: 3.0.1 -> 3.1.0

Motivation for this change

Bump to 3.1.0.
Renoise non-demo builds have a licence baked into them, which means each is a totally individual bundle so SHAs can't be used.

Things done

Updated the version to 3.1.0.
Replaced the demo parameter with the releasePath parameter so that a path to the full version can be supplied.

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

[x] NixOS
[ ] macOS
[ ] Linux

[ ] 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 nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

While I remember, it would be amazing if this could be streamlined into the 17.03 branch as well.

@the-kenny Is the 17.03 channel frozen? The current commit for it looks to be way behind even this PR.

@seanparsons The channel might be frozen, yeah. I personally don't use channels and can't say much on that issue.


fix(launcher): send sigkill on timeout when force killing

@dignifiedquire This is in response to bug we are seeing where karma hangs because it is unable to force kill chrome, possibly https://github.com/karma-runner/karma/issues/24#issuecomment-216249098

I signed it!

Thanks, can you change the commit message to befix(launcher: instead of bug please?

@dignifiedquire, i updated the commit message, thanks.

@dignifiedquire Any ETA on this one landing?


Is it possible to support generic custom mapping?

Our situation is: we have two large set of beans, most of which has field name like:
field-a: getObject
field-a: getObjects
I don't want to manually write up all the custom mapping in XML configuration (we have 100+ types to map). Is it possible to support such generic custom mapping / converter?
com.my.package.
com.my.package2.
${field}
${field}s
I have went through the source code and it seems possible if I can implement a custom mapping generator, however, I don't how to do that.

@hienchu ; is this still an issue?

Closing due to inactivity.


Support for PHPUnit v10

$ composer why-not phpunit/phpunit 10.1
sanmai/phpunit-legacy-adapter 8.2.1    requires         phpunit/phpunit (^7 || ^8 || ^9)

Please provide support for PHPUnit 10. It's a blocker for PHPUnitGoodPractices/Traits#63

Thank you! 🍻


Add ADAMContext APIs to create genomic RDDs from dataframes

In some cases, it's convenient to use vanilla Spark APIs to load genomic data and apply some basic transformations before creating an ADAM object. This PR adds methods to ADAMContext to load each type of genomic RDD from a Spark SQL dataframe and a metadata path. We look for metadata like the sequence and record group dictionaries in the metadata path.

Can one of the admins verify this patch?

Thanks for the contribution, @henrydavidge!

Jenkins, test this please

Closing in favor of #2158


Issues with snapshots containing "complicated" transparent images

In a project I'm working on, we have lots of issues with "glitches" in captures images when the transparent images contain shadows or text with rounded fonts.
It results in image comparison failure and diff images with those "glitches" that are very small differences.
I made a fix that seems to solve the issue using swizzling so far. I can't make a PR on the concerned files on this repository at this point because it might impact other projects using this library.
If anybody has an idea how we can make it configurable, that would be good.
The fix:
In + (UIImage *)fb_imageForLayer:(CALayer *)layer and + (UIImage *)fb_imageForView:(UIView *)view from the file UIImage+Snapshot.m I changed the context to capture only opaque images by changing the following line:
from
UIGraphicsBeginImageContextWithOptions(bounds.size, NO, 0);

to
UIGraphicsBeginImageContextWithOptions(bounds.size, YES, 0);

Note: I went in OpenRadar to try to find a bug related to this but didn't find any screenshot glitches posts. As this issue could maybe come from some compression algorithm that I'm not aware of, I'm not filing a radar yet.

I've been noticing this a lot lately, too. Image Diffs that look completely blank.

In a project I'm working on, we have lots of issues with "glitches" in captures images when the transparent images contain shadows or text with rounded fonts.
It results in image comparison failure and diff images with those "glitches" that are very small differences.
I made a fix that seems to solve the issue using swizzling so far. I can't make a PR on the concerned files on this repository at this point because it might impact other projects using this library.
If anybody has an idea how we can make it configurable, that would be good.
The fix:
In + (UIImage *)fb_imageForLayer:(CALayer *)layer and + (UIImage *)fb_imageForView:(UIView *)view from the file UIImage+Snapshot.m I changed the context to capture only opaque images by changing the following line:
from
UIGraphicsBeginImageContextWithOptions(bounds.size, NO, 0);

to
UIGraphicsBeginImageContextWithOptions(bounds.size, YES, 0);

Note: I went in OpenRadar to try to find a bug related to this but didn't find any screenshot glitches posts. As this issue could maybe come from some compression algorithm that I'm not aware of, I'm not filing a radar yet.

Can you simply share a patch file of the necessary changes?


Show metadata in hierarchy lists

When showing a list of images, include selected metadata as additional columns.
Ideally, we should be able to sort by column.

Added in a series of PRs in large_image.


Can the background be blurred?

I haven't seen any issues related to using a background blur. Can this be possible with the SPStorkController? I assume simply putting a blur view in the view controller being presented would not work because of the snapshot. Maybe I'm wrong.
I'll try to see if this solution works when I get a chance: http://justabeech.com/2014/10/22/using-uivisualeffectview-in-a-modal-view-controller/

Yes, blur not work because SPStorkController use snapshots

Any recommendations on how it can be implemented? I'll be happy contribute if I figure it out. Thank you.

You can try to add a blur before the view to be presented.


TSLint throws an error on test directory.

I have encountered error Cannot find module '~~~' on my test/ directory.
First, My directory structure below.
directory structure
- /
  - .vscode/
  - node_modules/
  - src/
    - libs/ (my util libraries)
    - errors/ (my custom exceptions)
    - models/ (sequelize models)
    - routes/ (route scripts)
    - index.ts (main script)

  - test/ (test directory with mocha)
  - typings/ (for type definition of mysql2 package)
  - package.json (also, package-lock.json)
  - tsconfig.json
  - tslint.json

and I use paths in tsconfig.
tsconfig.json
{
  "compilerOptions": {
    "target": "es6",

    "module": "commonjs",
    "moduleResolution": "node",

    "strict": true,

    "baseUrl": "./",
    "paths": {
      "@libs/*": ["./src/libs/*"],
      "@errors/*": ["./src/errors/*"],
      "@models/*": ["./src/models/*"],
      "@test/*": ["./test/*"],
      "@/*": ["./src/*"],
    },
    "typeRoots": ["./typings", "./node_modules/@types"],
    "esModuleInterop": true,
    "noResolve": false,

    "plugins": [
      {
        "name": "typescript-tslint-plugin"
      }
    ]
  },
  "include": ["./src/*", "./typings/*", "./test/*"]
}

tslint.json
{
  "extends": "tslint:recommended",
  "rulesDirectory": ["src/", "test/"],
  "rules": {
    "max-line-length": {
      "options": [120]
    },
    "new-parens": true,
    "no-arg": true,
    "no-bitwise": true,
    "no-conditional-assignment": true,
    "no-consecutive-blank-lines": false,
    "quotemark": [true, "single"],
    "no-console": {
      "severity": "warning",
      "options": ["debug", "info", "log", "time", "timeEnd", "trace"]
    }
  }
}

then, I trying to import the modules using paths.
src/index.ts

src/routes/api/1.0.0/users/index.ts

test/reset.ts

test/api/1.0.0/user/User.test.ts

but, tslint throws the error on only test/api/user.ts file.
also, I found some weird situation.
Change src/ directory name to xxx/
I changed src directory name to xxx and all configs too.
but, same problem occurred in xxx/ directory.
Move all test/**/.test.ts file to test/.test.ts
I moved all test file to test root directory.
then, in my test/*.test.ts file, tslint doesn't throw the error!!
Is tslint handle src directory specially?
or is my mistake?

Closing as TSLint has been deprecated in favor of ESLint and this extension is no longer being maintained
Please look into migrating to use ESLint and the ESLint VS Code extension


Added 4 Sum Problem and updated readme

Have you read the Contributing Guidelines on Pull Requests?
Yes
Description
Added 4 Sum Problem
Checklist

[x] I've read the contribution guidelines.
[x ] I've checked the issue list before deciding what to submit.
[x] I've edited the README.md and link to my code.

Related Issues or Pull Requests
#2480

Hii , I have made the changes and also commited but it is not showing here. Is there any way to commit to a particular pull request.I know its  very childish but since I am a newbie kindly help me!
Thanks in advance.

Changes have been done


Unable to set 'expires_in' for token

When I generate my token:
 api_client.request_jwt_user_token(
      ENV["DOCUSIGN_JWT_INTEGRATION_KEY"],
      ENV["DOCUSIGN_IMPERSONATED_USER_GUID"],
      ENV["DOCUSIGN_RSA_PRIVATE_KEY"],
      60,
      "signature impersonation"
    )

It doesn't matter what the 4th arg is. The code always sets it to 3600.
Also, for what it's worth, the syntax of the example here https://developers.docusign.com/docs/esign-rest-api/sdk-tools/ruby/auth/ (token = api_client.request_jwt_user_token(Rails.configuration.jwt_integration_key, Rails.configuration.impersonated_user_guid, rsa_pk, expires_in=3600, "signature impersonation")) does not need the expires_in=.

The expires_in variable has a different use then you may expect. The value that is provide in this request is only used for the JWT assertion that we use to create an Access token. The expiration of JWT assertion does not have any impact on when the Access token expires.  All JWT Access tokens generated will always be valid for 36000 seconds regardless of when the JWT assertion is set to expire.
For example if you set the expires_in  value to 60, you can use the same JWT assertion(the "token" variable) to generate new access tokens for the next 60 seconds. After the 60 seconds pass you will need a new JWT assertion to get new Access tokens. Any Access token generated during the 60 seconds  will be valid for 1 hour after they are generated.
It is not possible to modify the expiration of the Access token that is generated with this process

Thanks for the fast response @EdwinMoralesDS
I don't entirely understand the different between the expiration of the JWT assertion compared to the JWT token. I had expected the token to expire with the passed in expires on because that was getting written into the token token's claim here.
I'll do a little bit more research to better understand this. But still, I'm not sure that the code is correct for being able to pass in the expires_in value:
expires_in = 3600 if expires_in > 3600 (source)
If I test this out with different values, this is what I get:
Less than 3600 we get nil
2.7.2 :004 > expires_in = 60
 => 60
2.7.2 :005 > 3600 if expires_in > 3600
 => nil

More than 3600 we get 3600:
2.7.2 :006 > expires_in = 1000000
 => 1000000
2.7.2 :007 > 3600 if expires_in > 3600
 => 3600

So I suppose I still don't understand what is supposed to be happening, but this logic doesn't seem quite right.

@aturkewi   can you try to run the following code:
expires_in = 60; expires_in = 3600 if expires_in > 3600; puts  expires_in

Oh my goodness 🤦 Thank you for that code example. I was misreading that line in the source.
I had read it as expires_in = (3600 if expires_in > 3600) (where expires_in was being reset to the right side of the =). But I see now that it's really (expires_in = 3600) if (expires_in > 3600) (where expires_in is only being reset if it's too large).
Well that was embarrassing. I appreciate you taking the time to help me better understand this. I'll go ahead and close this out. Thank you again.


Update virtual-machines-linux-how-to-attach-disk.md

What I had to do in the end to make the disk writable:
cd /datadisk
sudo chmod go+w datadisk/

Hi @woutercx, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no contribution license agreement is required at this point. Real humans will now evaluate your PR.

TTYL, AZPRBOT;

Thank you for this contribution - sending to the author for review.

@woutercx this is for attaching an existing data disk?

Hi,
I don't remember exactly what I did, but yes, I think it was an existing
data disk.
I took an image from the VM Depot Azure Gallery:
http://www.hanselman.com/blog/Over400VirtualMachineImagesOfOpenSourceSoftwareStacksInTheVMDepotAzureGallery.aspx
and in the end I had to make it writable.
On 06/11/15 19:37, Cynthia Nottingham [MSFT] wrote:

@woutercx https://github.com/woutercx this is for attaching an
existing data disk?
—
Reply to this email directly or view it on GitHub
https://github.com/Azure/azure-content/pull/4822#issuecomment-154495621.

@woutercx thanks for your contribution. I just tried this with the new disk, and you do need to make it writable with that extra command. Please go ahead and merge this change, after making these changes:

Replace datadisk by datadrive as that is used in the article
Fix the typo in the second step, datadrive/ should be replaced with /datadrive

Thanks again for doing this.

@woutercx - Dhanashri submitted some patches to your fork for this - if you could accept her pull request in your fork of this project, we'll be able to move ahead and accept your update :)
Here's the link to that pull request: https://github.com/woutercx/azure-content/pull/1

Thanks!


Any chance to get fabric support?

Any chance to get fabric support?

Hi, before I answer let me take a look at Fabric ;)

Many mods use a kind of "multi compile" thing. But I am not so familiar with minecraft modding.
See maybe a example here? https://github.com/jaredlll08/Clumps

Thanks for checking this out. The main work will likely be the compatibility of the java code, and how I can project the library I have already for Forge to Fabric. Some things seem easy to port, others (like registries, block/item overrides, Forge-events, config, rendering, etc) are quite differently handled. I'll experiment a bit what is possible and where the problems are located ;)

Oky, I managed to compile and very briefly test a first 1.19 Fabric version for the mod, do you like to assist with testing? ;)
redstonepen-1.19-fabric-1.0.11.zip

Quick correction, I forgot something that causes a server crash in the initial build above. Correction is here ...
redstonepen-1.19-fabric-1.0.11.zip

Okay, the initial beta build is on Curse.


testing application components with jest fails

Good day,
we are using this amazing library in our app, a bare expo app and we faced 2 challenges. specifically with jest.

the main key in package.json points to the esmodule build and not the commonjs build as expected
after loading the commonjs build, tests fail with this message: Native RNSkia Module cannot be found when testing a component that uses any of react-native-skia exports

Hi, @salzhrani! Thanks for your kind word :)
Have you tested changing the main key to point to the commonjs build folder and verified that it fixes the error with Jest? Does this in any way affect the rest of the library?

@salzhrani the library won't run in jest. Can you create a mock for it?

We might provide our own mock part of the package in the future. We assume that you simply want to test regular RN components that happen to contain a React Native Skia canvas? For anything more complex, you might want to look at how we run Skia on jest via CanvasKit: https://github.com/Shopify/react-native-skia/blob/main/package/src/renderer/tests/Simple.spec.tsx

we currently mock components we use in our code. will try to figure out a way to automate it and hopefully share it

@wcandillon, @chrfalch  definitely we need a package like:
@shopify/react-native-skia/testing where it provides the mocks for jest.
Currently I am facing this problem where my testing is failing because of skia and I am not sure how to mock it correctly.

Hi @flo-sch, super awesome findings and write-up, we really appreciate this! Would be super happy if you could help with creating Mocks for the library. Maybe we could collaborate and find a solution for the NativeSetup issue you're seeing - I would imagine that we could create some kind of Platform test for this - the same goes for the global object issue. We already have Platform switches for Native / Web in the code!

Hi @flo-sch, super awesome findings and write-up, we really appreciate this! Would be super happy if you could help with creating Mocks for the library. Maybe we could collaborate and find a solution for the NativeSetup issue you're seeing - I would imagine that we could create some kind of Platform test for this - the same goes for the global object issue. We already have Platform switches for Native / Web in the code!

My solution which I can share is to mock views and methods exposed and bypass everything and return plain views for views and noop function for everything else
import { View } from 'react-native';
import { createElement } from 'react';

const PlainView = ({ children, ...props }: any) =>
  createElement(View, props, children);
const noop = () => null;

jest.mock('@shopify/react-native-skia', () => {
  const mock: { [key: string]: any } = {
    // other props can be added which
    // aren't handled properly
    // by the handler
    Canvas: PlainView,
  };
  const handler = {
    get(_: any, prop: string, __: any) {
      // first look for the prop in the mock
      if (prop in mock) {
        return mock[prop];
      }
      // class case? return a view
      if (prop[0] === prop[0].toUpperCase()) {
        return PlainView;
      }
      // probably a method
      return noop;
    },
  };
  return new Proxy(mock, handler);
});

if you think it is useful I can make a PR

@salzhrani using this solution still has some issues like:
   TypeError: Cannot read properties of undefined (reading 'Make')

      53 |
      54 |      const straightLine = useMemo(() => {
    > 55 |              const path = Skia.Path.Make();
         |                                     ^
      56 |              path.moveTo(0, height / 2);
      57 |              for (let i = 0; i < width - 1; i += 2) {
      58 |                      const x = i;

Not sure how to fix this one.

You can add any missing properties to the mock object
 Path: ()=> ({
  Make: () => ({
    moveTo: noop
    })
  })

Not optimal but should help


Increase memory request and limit to 50Mi and 100Mi respectively

Issue #, if available:
Description of changes:
In the management cluster, the capc memory usage starts from 16~21Mi without having any workload clusters. Therefore, 20Mi resource request is too tight so I am increasing it to 50Mi/100Mi. Here are the memory usage trend per the number of workload clusters. I couldn't find an increasing trend of the memory usage per the # of clusters.
Testing performed:

1-1-1 workload clusters

# of WL
Memory

0
16Mi

1
19Mi

2
20Mi

3
20Mi

4
21Mi

5
20Mi

6
21Mi

3-3-3 workload clusters

# of WL
Memory

0
19Mi

1
20Mi

2
24Mi

3
26Mi

4
27Mi

5
26Mi

6
27Mi

7
27Mi

8
28Mi

9
25Mi

10
24Mi

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/retest

/retest

/approved


How to get group or roles information by ID_Token？

SECURITY NOTICE: If you have found a security problem in the UAA, please do not file a public github issue. Instead, please send an email to<EMAIL_ADDRESS>Thanks for taking the time to file an issue. You'll minimize back and forth and help us help you more effectively by answering all of the following questions as specifically and completely as you can.
What version of UAA are you running?
UAA version:74.11.0
I curl the UAA API to get id_token,access_token info.
How are you deploying the UAA?
I am deploying the UAA
locally only using gradlew
What did you do?

I use the uaa.yaml like this https://github.com/cloudfoundry/uaa/blob/develop/uaa/src/main/resources/required_configuration.yml
I use uaac creates a client:
uaac client add webappclient -s webappclientsecret \
--name WebAppClient \
--scope roles,openid,profile,email,address,phone \
--authorized_grant_types authorization_code,refresh_token,client_credentials,password,implicit \
--authorities uaa.resource \
--redirect_uri http://localhost:8081/login/oauth2/code/uaa

3.I create a user named appuser and secret is appusersecret
4.I add the user to roles
5.I curl the uaa api get Id_token:
eyJhbGciOiJSUzI1NiIsImprdSI6Imh0dHBz...Jmq2Wg
6.I use jwt to decode id_token and get the payload data:
{
  "sub": "8b029dc5-3a23-4664-8308-81a03e5b5c76",
  "aud": [
    "webappclient"
  ],
  "iss": "http://uaa-dev.mstech.com.cn/uaa/oauth/token",
  "exp":<PHONE_NUMBER>,
  "iat":<PHONE_NUMBER>,
  "amr": [
    "pwd"
  ],
  "azp": "webappclient",
  "scope": [
    "openid"
  ],
  "email"<EMAIL_ADDRESS>  "zid": "uaa",
  "origin": "uaa",
  "jti": "66456ec7bf2b429bbfc9659f52dcc80d",
  "previous_logon_time":<PHONE_NUMBER>852,
  "email_verified": true,
  "client_id": "webappclient",
  "cid": "webappclient",
  "grant_type": "implicit",
  "user_name": "appuser",
  "rev_sig": "220f04ed",
  "user_id": "8b029dc5-3a23-4664-8308-81a03e5b5c76",
  "auth_time":<PHONE_NUMBER>
}

The id_token do not contains information about roles?
What did you expect to see? What goal are you trying to achieve with the UAA?
I hope I can get roles information from id_token and I think it is necessary.
Thanks!
What did you see instead?
Please include UAA logs if available.

Thanks for the question. Long story short, the roles claim in the id_tokens minted by the UAA represents role data present in an external IdP. The user you're working with is in the internal user store, so the roles claim is empty.
In the UAA, groups represent the scopes that can be requested on a user's behalf.
I agree it's a bit confusing and we get a lot of questions about it.


update free for life link

Hey, I'm the owner of Free For Life and have updated our website URL (you can confirm on the repo: https://github.com/wdhdev/free-for-life). Thanks!

@JuanPabloDiaz

Thank you @wdhdev


"Implement interface" codefix missing newline after first member

TypeScript Version:  3.7.3
Search Terms: implement interface codefix
Code
The "Implement interface" codefix fails to add a newline character after the first newly-added member when the class body is split across multiple lines:

Actual behavior:

#26550


Clarification on GetObjects FOREIGN KEY

What would you like help with?
Is the intent of the FOREIGN KEY constraints returned by AdbcConnectionGetObjects that they include relational constraints in both directions? That is, for a CUSTOMER table defined as in TPC-H, should it return both "ORDERS"."CUSTKEY" (which references "CUSTOMER"."CUSTKEY") and "NATION"."NATIONKEY" (which is referenced by "CUSTOMER"."NATIONKEY")?
If so, the documentation could probably be improved by indicating this explicitly.
If not, if it only returns references in one direction then this is a feature gap with ODBC and JDBC.

I think from reading the GetObjects schema definition, only JDBC's getImportedKeys is supported, effectively. We would need to add something to cover getExportedKeys, probably by extending the constraint_type (adding EXPORTED KEY?)


Fix flake TestEndpointsDeduping, it is caused by running kube controller twice

Please provide a description of this PR:
Fix https://github.com/istio/istio/issues/36503

9m15s: 2680 runs so far, 0 failures
9m20s: 2705 runs so far, 0 failures
9m25s: 2730 runs so far, 0 failures
9m30s: 2755 runs so far, 0 failures

ping @stevenctl


Fix junk data in ffprobe

Hello, this is my attempt at fixing issue #39 .
I've made a proper file using Kid3 and then compared it to tag from this library and made the following changes:

The padding in file should be 1024, however your code somewhere adds an extra byte, so made it 1023
The header size (and 1 extra byte) isn't included in the good tag, so removed them, however the 1024b padding is included.

I've also added a quiet.mp3 file, which after tagging works in ffprobe and didn't without my changes.
Thank you.

Hello, I've made changes to the test and added the padding to the encoder builder. I've also included ffmpeg in the Github Actions workflow, so the test can pass.
However I didn't solve the last problem, as I am not really familiar with the entire codebase to do it properly. Hopefully you can fix that and merge soon. Thank you.

I've actually tested it right now on different (actual music files) and the issue still seems to be present.

Okay, so this commit was tested on ~1000 MP3 files, all were issue free, so hopefully that's it.
However I had to change the assert in storage.rs on line 263, which am not completely sure what does. However without the change it crashes after trying to write multiple times to same file, and with the change I didn't have any issues, so I am assuming it's safe, but let me know. Thank you.

Thanks! I try to schedule some time to push this along the coming weekend :)

Reviewed and merged! Although I have squashed your changes and fixed to off-by-one and storage padding issues in another commit :)
Once again, thank you for your effort!


dev-docs.bevyengine.org branch should be force-pushed to avoid bloating the repo

gh-pages appears to be the branch in question and this appears to be the workflow.
We don't care about history here, so we should just erase this to make git pull faster for contributors and stave off hitting Github's repo size limit.
Discovered by @james7132.

Seems like we can just use single-commit and force together to wipe out the history here.


[feature] Does right mouse button drag support

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

No, I'd think that's against normal HCI


i18n

Hi @mmistakes,
I'd like to translate theme into my language. I made some but I wonder whether they are reverted back while fetching updates from upstream. Can you tell me how I made this theme i18n'ed?

My other theme Skinny Bones supports being localized better as it makes use of some _data files.
I have no plans to do the same with this theme. You'll just have to go through all the layouts and includes and localize any English text into the language you want.

Closing because theme text can now be localized via a data file.


Bump document deletion date by 3 months

Review app deployed to https://teaching-vacancies-review-pr-3447.london.cloudapps.digital


Fixture creation from class with validation annotation fails when using ConstructorPropertiesArbitraryIntrospector

Describe the bug
Fixture creation from class with validation annotation fails when using ConstructorPropertiesArbitraryIntrospector
Your environment

version of Fixture Monkey: 1.1.5
version of Java/Kotlin: JDK 21

Steps to reproduce
Please check my commit from my forked repository

https://github.com/doljae/fixture-monkey/commit/f99e3eaa55488b2e977792c616240bf73ee7f8ba#diff-562c4f972c1f26857310b0b93cd9c5c3aee43d71a8b2698607713716c155e604R63

Make test class which has fields with validation annotation (TestClassV1, TestClassV2)
Add two test cases based on existing test cases in JakartaValidationFixtureMonkeyTest (sampleInteger1, sampleInteger2)
Run tests

Codes
@Getter
@Setter
public class TestClassV1 {

	@Max(3)
	@Min(3)
	private int value1;

	@Max(100)
	@Min(100)
	private int value2;

	@ConstructorProperties({"value1", "value2"})
	public TestClassV1(int value1, int value2) {
		this.value1 = value1;
		this.value2 = value2;
	}
}

@Getter
@Setter
@NoArgsConstructor
@AllArgsConstructor
public class TestClassV2 {

	@Max(3)
	@Min(3)
	private int value1;

	@Max(100)
	@Min(100)
	private int value2;
}

@Property(tries = 100)
void sampleInteger1() {
	FixtureMonkey SUT = FixtureMonkey.builder()
		.plugin(new JakartaValidationPlugin())
		.objectIntrospector(ConstructorPropertiesArbitraryIntrospector.INSTANCE)
		.defaultNotNull(true)
		.build();

	TestClassV1 actual = SUT.giveMeOne(TestClassV1.class);

	then(actual.getValue1()).isEqualTo(3);
	then(actual.getValue2()).isEqualTo(100);
	then(actual.getValue1()).isBetween(3, 3);
	then(actual.getValue2()).isBetween(100, 100);
}

@Property(tries = 100)
void sampleInteger2() {
	FixtureMonkey SUT = FixtureMonkey.builder()
		.plugin(new JakartaValidationPlugin())
		.defaultNotNull(true)
		.build();

	TestClassV2 actual = SUT.giveMeOne(TestClassV2.class);

	then(actual.getValue1()).isEqualTo(3);
	then(actual.getValue2()).isEqualTo(100);
	then(actual.getValue1()).isBetween(3, 3);
	then(actual.getValue2()).isBetween(100, 100);
}

Expected behaviour
sampleInteger1 & sampleInteger2 should be passed
Actual behaviour
sampleInteger2 passed, but sampleInteger1 didn't
Additional info

Both TestClassV1 and TestClassV2 used the same validation annotation, the only difference is the use of @ConstructorProperties.

In my debugging, it seems that if I set the ConstructorPropertiesArbitraryIntrospector to call the object constructor with @ConstructorProperties and the constructor parameter has the validation annotation, the object is created as intended.

However, it seems a bit odd to me that the fixture is not created correctly when the user has set the validation annotation on the field just fine, like TestClassV1, and just changed the method used to create the object.

In fact, if I set up a ConstructorPropertiesArbitraryIntrospector and use it, it seems that the validation annotation I used for the field internally is not working, although I'm not sure if that's accurate.

The sampleInteger1 test also passes if the @Min and @Max values have some range, I guess it is related to iteration logic for generating random value from jqwik.

I applied your PR(https://github.com/naver/fixture-monkey/pull/1119) and found that both test cases pass.
Thanks for your quick adjustment and please feel free to close the issue when you finished 🙂


Seg Fault due to hub model and external inflow

Feature or improvement description

The hub index is 1 in most of the code (first entry in the array), but here it is the last one.  I've had segfaults in nalu-wind and amr-wind that both trace back to this line of code when running actuators with the hub turned on.
Interestingly we only see this issue with GCC, and in testing a RelWithDebugInfo build of OpenFAST the failures were intermittent as many memory leaks are.
Related issue, if one exists

https://github.com/Exawind/nalu-wind/issues/979
Impacted areas of the software

Any turbine run using the external inflow and hub model.
Additional supporting information

Pinging @andrew-platt @rafmudaf @gantech
Test results, if applicable

I am not an openfast developer and am not aware of the tests. I assume this is just not getting covered in the test suite.  We've had a segfault in the nalu-wind test suite for months now.
https://my.cdash.org/test/60805600

I'm going to merge this into the dev branch.  It will be released with v3.3.0

I think this is a fine temporary fix, but technically this last point is the nacelle and not the hub. The hub point (index 1) gets sent to ServoDyn (see SrvD_InputSolve), so it is omitted in subroutine AD_InputSolve_IfW and is the reason the node index starts at 2 there instead of 1.
We added a nacelle point in AeroDyn for some other work (hydropower? + some syncing with Envision), but the OpenFOAM interface portion of that in OpenFAST apparently didn't get updated. I was actually working on a fix for this interface last week, but it will also require that everything interfacing with this (nalu-wind, amr-wind, and similar) will need to supply another point for wind at the nacelle.

Hmm this is interesting. I would request that you add an option to make the addition of the new point for the nacelle optional. From an actuator line implementation standpoint this will require us to add an additional actuator point, code infrastructure etc. This is inconsistent in terms of restarts and can potentially add larger impacts on the codes coupling to OpenFAST. @tonyinme what are your thoughts?

hey all, @psakievich had a conversation offline and agreed that it is best to bundle the hub and nacelle contribution into one for the CFD coupling.

@psakievich, could you add a comment in the code about using the hub point as the nacelle point for the CFD coupling?  This could help prevent confusion when we are looking at that code later.  I'll merge after that.

sorry. I forgot to turn off the auto-formatting on my neovim config.  I don't ever work in fortran. will fix


Bugs encountered in Preview 1.0

What happened?
Items:

Potions occasionally do not heal past a certain amount of health. The potion will be consumed but you can't heal over for example 25% of Max HP. You'll gain nothing. Healing spells work.

 No potion will be consumed but you don't heal either.

Controller:

Cannot charge soul stones in Inventory, this can be fixed by editing the controlmap.txt in Tsukiro\mods\The Ultimate Control Scheme\Interface\Controls\PC. Changing '!0,Shout' to '0x0200' for ChargeItem fixes it. It should look like this:
// Inventory ChargeItem    !0,Wait    !0,Wait    0x0200        0    0    0    0x8

Sprinting doesn't work. Changing the key bind from R1/RB to anything else doesn't solve this either.

Gameplay:

Sekiro Deathblow stays on the enemy for multiple hits, is that intended?

Fast Travel sometimes bugs out on sign posts. You cannot fast travel anymore.

Reading a book or letters sometimes flicks around the camera.

While they do no damage, walking on bones accelerates you with the force of physics.

Misc:

Katana, when nude, still wears underwear. Was that intended?

My Specs:
DxDiag.txt
Modlist version
Preview 1.0

Cannot reproduce any of this, signs are fixed, bones are just bethesda physics, camera is fixed, Katana should now be nude


updated org-admin and plan create pages to reflect schema change

Added a new boolean 'migrated' to the templates model
This distinguishes between templates created to fill in the back end of a plan, and those templates which reflect the live data from the org-admin or plan creation interfaces
This is accessed in the templates model by a new scope valid.  To get only non-migrated templates, simply being a query with Template.valid and chain your query from there.

Make sure to run rake db:migrate after pulling this.


Follow up PR of fixing dotnet resource leak assign

Hi,
We create this PR as a follow up PR to address the comments from a previous PR

This comment is addressed in the new push. Now it is ready for another review.
Thanks!

In addition, I have improved the readability of the resource leak message. Now it looks like something attached below:

A make fmt issue: when I run make fmt it reports the following error:

FYI, I am developing in a docker container.

A make fmt issue: when I run make fmt it reports the following error:

FYI, I am developing in a docker container.

It says that parallel is not found so just install it? :)
The PR fails the dotnet resource leaks test that's checked in, is it expected? You can test with make direct_dotnet_resourceleak_test, it introduces some false positives:
--- a/mnt/btrfs/trunk-git-infer-15-1649327228/infer/tests/codetoanalyze/dotnet/resourceleak/issues.exp
+++ b/mnt/btrfs/trunk-git-infer-15-1649327228/infer/tests/codetoanalyze/dotnet/resourceleak/issues.exp.test
@@ -1,2 +1,4 @@
{+codetoanalyze/dotnet/resourceleak/ResourceLeak.cs, Void TestCodeResourceLeak.ResourceLeakInitInterprocOk(), 0, DOTNET_RESOURCE_LEAK, no_bucket+}
codetoanalyze/dotnet/resourceleak/ResourceLeak.cs, Void TestCodeResourceLeak.ResourceLeakInitInterprocBad(), 0, DOTNET_RESOURCE_LEAK, no_bucket
{+codetoanalyze/dotnet/resourceleak/ResourceLeak.cs, Void TestCodeResourceLeak.ResourceLeakInterprocOk(), 0, DOTNET_RESOURCE_LEAK, no_bucket+}
codetoanalyze/dotnet/resourceleak/ResourceLeak.cs, Void TestCodeResourceLeak.ResourceLeakInterprocBad(), 0, DOTNET_RESOURCE_LEAK, no_bucket
@@ -6,2 +8,3 @@ codetoanalyze/dotnet/resourceleak/ResourceLeak.cs, Void TestCodeResourceLeak.Res
codetoanalyze/dotnet/resourceleak/ResourceLeak.cs, Void TestCodeResourceLeak.ResourceLeakStaticSingletonBad(), 0, DOTNET_RESOURCE_LEAK, no_bucket
{+codetoanalyze/dotnet/resourceleak/ResourceLeak.cs, Void TestCodeResourceLeak.ResourceLeakGlobalResourceInterprocOk(), 0, DOTNET_RESOURCE_LEAK, no_bucket+}
codetoanalyze/dotnet/resourceleak/ResourceLeak.cs, Void TestCodeResourceLeak.ResourceLeakGlobalResourceInterprocBad(), 0, DOTNET_RESOURCE_LEAK, no_bucket

Yes, I made a small mistake. Now all the unit tests passed. Meanwhile I have installed parallel and formatted the code. Thanks for finding them out!


New Linux build ready & tested for Godot 3.1

;)

Sorry for the delay, I have been without my laptop for over a week :(


remove public network interface on warmer pool instances

Worker creates job VMs without a public IP, so that all traffic goes through the NAT. This change ensures pre-warmed instances have the same behaviour.
refs travis-ci/reliability#195

Confirmed working on staging!


About motion_cnn.py,why data loading needs so much time ?it seems to be close to the batch time.

hello,When I run the motion_cnn.py,I get some questions,As follows,
1、About motion_cnn.py,When I set the bath_size as more than 32 such as 64, I find  the program stuck at the training stage, and some time it run fast ,but then it stuck.(My GPU is GTX1080ti,when i run the command  nvidia-smi,It shows that I just use less than 4G memory)
2、Also, I find that the data loading time  is similar to the batch time,It makes me confused,Why data loading needs so long ,and similiar to batch time,does the model trainning (batch time-data loading time) need little time?
hope to get your answer,thanks!

Hi,
For the first question, the reason why the training process will temporally stuck is that the model is still processing on CPU(including I/O and resizing image to the size of model input). If you raise the number of  batch size, the time of preprocessing will also increase.  Therefore, it is normal that the time on CPU will increase, due to the hardware limitation.
For the second question, to make it more precise, batch time means the time consumption on GPU, and the data time means the time consumption on CPU.
If you would like to this verify this question, you can test the code in  here and figure out the reason why it takes so much time on CPU.
I hope my clarification has been helpful.
Jeffrey


Added IE polyfills to support IE9-11 (fix apply on undefined error)

Added IE polyfills for IE9-11

new ClientApp/polyfills/polyfills.ts file containing polyfills
added core-js to package.json
added core-js to webpack.config.vendor.js
added polyfills.ts import to ClientApp/boot-client.ts

@AmrineA,
Thanks for your contribution.
To ensure that the project team has proper rights to use your work, please complete the Contribution License Agreement at https://cla2.dotnetfoundation.org.
It will cover your contributions to all .NET Foundation-managed open source projects.
Thanks,
.NET Foundation Pull Request Bot

Will resubmit with additional updates


GLIBC_2.34' not found (required by /host/lib/liblarod.so.1) error with Q1615 MK III (AXIS OS version 10.11.65)

Describe the bug
pose-estimator-with-flask example not working with Q1615 MM III (AXIS OS version 10.11.65)
To reproduce
Follow the readme file from this project
$ export ARCH=armv7hf
$ export CHIP=tpu
$ export AXIS_TARGET_IP=<IP_ADDRESS>
$ export DOCKER_PORT=2375
$ export APP_NAME=acap4-pose-estimator-python
$ export MODEL_NAME=acap-dl-models
$ docker -H tcp://$AXIS_TARGET_IP:$DOCKER_PORT system prune -af
$ docker build . -t $APP_NAME --build-arg ARCH
$ docker save $APP_NAME | docker -H tcp://$AXIS_TARGET_IP:$DOCKER_PORT load
$ docker build . -f Dockerfile.model -t $MODEL_NAME --build-arg ARCH
$ docker save $MODEL_NAME | docker -H tcp://$AXIS_TARGET_IP:$DOCKER_PORT load
$ docker-compose -H tcp://$AXIS_TARGET_IP:$DOCKER_PORT --env-file ./config/env.$ARCH.$CHIP up
Screenshots

Same error with minimal-ml-inference example:

Same error with object-detector-cpp:

Environment

Axis device model: [Q1615 Mk III]
Axis device firmware version: [10.11.65]
Stack trace or logs:
/usr/bin/acap_runtime: /lib/arm-linux-gnueabihf/libc.so.6: version `GLIBC_2.34' not found (required by /host/lib/liblarod.so.1)
OS and version: [Ubuntu 20.04 LTS]

Thanks for the report, we're looking into what is causing this. If possible, try falling back to FW 10.10 to see if this resolves the issue for you.

Thank you for suggesting the old firmware. It is working with AXIS OS 10.10.69.


Workspace - Skeleton loader not displayed in workspace member page

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: v9.0.3-1
Reproducible in staging?: Y
Reproducible in production?: N
Found when validating PR : https://github.com/Expensify/App/pull/43893
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Logs: https://stackoverflow.com/c/expensify/questions/4856
Issue reported by: Applause-Internal team
Action Performed:

Open app.
Go to Account settings > workspace > member page
Notice that Skeleton loader is not displayed in workspace member page when data is loading

Expected Result:
Skeleton loader is displayed in workspace member page when data is loading
Actual Result:
Skeleton loader is not displayed in workspace member page when data 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
[ ] MacOS: Desktop

Screenshots/Videos
https://github.com/Expensify/App/assets/115492554/21cfb55f-336a-4b5f-9854-943bbdde3361

View all open jobs on GitHub

@jliexpensify I haven't added the External label as I wasn't 100% sure about this issue. Please take a look and add the label if you agree it's a bug and can be handled by external contributors.

Difference between stg and prod
https://github.com/Expensify/App/assets/115492554/bf2a2d65-dda7-427c-875c-e76208df8f29

Hmmm, so the skeleton view is implemented in https://github.com/Expensify/App/pull/43893
But it doesn't look like the PR is deployed to staging yet?
cc: @truph01 @jjcoffee @blimpich

I see that the PR is not deployed to staging.

@izarutskaya did we test the PR prematurely? 🤔

It seems to be an automation issue bcoz the changes of that PR are reflected in the staging branch code.

Hmm so from my testing there is no issue on staging. The loader shows when the connection speed is throttled enough for it to show, exactly the same as on production. @izarutskaya Do you mind retesting with a throttled connection?

asked for retest here

Also I don't think this should block the deploy.

Hi, just moving to a Daily (I assume that's ok?)

Issue is not reproduced on a slow network
https://github.com/Expensify/App/assets/93399543/8e45f475-b60b-490d-9ead-88f4912358cf

Great lets close this then.


ALM Toolkit support for Tabular Editor serialized models

Is your feature request related to a problem? Please describe.
To use Tabular Editor's folder serialization option along with ALM Toolkit requires that you first take an additional step to convert back to a single .bim file before using ALM Toolkit for model comparisons. It would be nice to be able to skip that step.
Describe the solution you'd like
I am hoping that Tabular Editor could have a new feature added so that any time you click Save when working with folder serialized models that both the folder serialized version and a normal .bim version can both be generated/saved at the same time.  For source control purposes, .bim files could then be added to git ignore files so they don't get checked in but would still be available for ALM Toolkit comparisons.
Describe alternatives you've considered
This problem could be solved by ALM Toolkit instead of by Tabular Editor.  I have submitted this similar request to the ALM Toolkit team: https://github.com/microsoft/Analysis-Services/issues/117

This would be a huge help! +1

I'm a huge fan of Tabular Editor folder-format support in ALM toolkit. Will reach out to Christian Wade to see what can be done in this regard.

@otykier I've submitted a Pull Request for your review that addresses this for now by generating a .bim file automatically.  I would also prefer the feature be added to ALM Toolkit, but adding this feature to Tabular Editor gets us what we need (although in a slightly less than ideal way).


A little typo in the repository tag "awesome"

A little typo in the repository tag "awesome"
Related to #7 and #8

@ianbjacobs , could you please help us one more time with this topic correction

Fixed (I think).

#8 is passing now. Thanks @ianbjacobs


fix(code) Set Default Bool True & Returning Data Into Json Format

… Countries

Changes Done in this Whole Package is under:

Return Data Into JSON Format for three function not in array.
Bind all the functionality in three main function.
Created separate filter models for countries,cities & states
Worked on code conciseness.
Further more worked on suggestions that were requested.
@HamzaAnis Review requested please!


Error running "rubocop --require rubocop-rspec"

I'm trying to update rubocop after getting the following deprecation warning when running my spec:

"DEPRECATION WARNING: Static attributes will be removed in FactoryBot 5.0. Please use dynamic
attributes instead by wrapping the attribute value in a block:

admin { true }

To automatically update from static attributes to dynamic ones,
install rubocop-rspec and run:
rubocop
--require rubocop-rspec
--only FactoryBot/AttributeDefinedStatically
--auto-correct"

I've installed the rubocop gem successfully but when I run 'rubocop --require rubocop-rspec" as above I get a failure.
Mention the following information in the issue report:
0.61.1 (using Parser <IP_ADDRESS>, running on ruby 2.3.4 x86_64-linux)`
It's actually much longer but I'm not sure how to get the whole backtrace to come up in the terminal so I can copy/paste it across.
Thanks!

@mellansquare I hope the directions provided solved your issue. If not, feel free to re-open. 🙇


Assigning a file key while some volumes are full

Hello guys,
We're currently testing Seaweed for a possible future use in a file hosting solution.
We have 3 master servers, and 3 volumes servers ATM.
On each of these 3 volume servers, we have 6 volumes.
3 of these 6 volumes are full, and the remaining 3 are empty.
The problem is, when we send a GET request to /dir/assign, it keeps on returning us file keys that are linked to the full volumes. The "Volume Layout" structure's "Writable" attribute still contains the full volumes when they shouldn't be there. Logically, it should only contain volumes 4, 5 and 6 but it currently has 1, 2, 3, 4, 5 and 6, making that half of our file keys assignations are leading to an error when uploading because the volume is full.
What is the condition for a volume not to be included in the writable volumes array?
My guess is that the volume still has some bytes free, so it stays writable, but the free space isn't large enough to upload any normal file. Could I be right ?
Any solution for this ?
Thanks in advance.
Post scriptum: Another question: what amount of data is taken in memory for a single 10 MB file ? I've read in your "Optimization" page that 1.5 Million 20 KB files are taking 24bytes per entry in memory, making it 36 MB per volume. What would be the values for 3k 10MB files ? (KB/file)
Is there a formula to calculate the memory needed per volume server in terms of average file size?

Please share the screenshot of http://master:9333/ to see the details of the volumes.

We just fixed it, had a wrong -volumeSizeLimitMB when starting the volume server. It was over the theoretical maximum size of a volume which is 35 GB, and when we assign a file key it considers the volume as not full because it is not reaching the value we passed.
Sorry for the long post.
For now SeaweedFS seems to be a pretty robust solution, congratulations for your hard work on it.
Any answer to my second question?
Thanks again.

The formula is right. Actual memory is about 24 bytes(including possible overhead) per file entry.


Having a generic authorization contract for each key

Currently when you approve a new device to use your app, it is by default a "management key" which means that it has unlimited powers to do anything they want to the contract. The only limits on keys are given by the purpose and the requiredApprovals number set by ERC725 standard. This is insufficient.
Consider the case in which you might want to add a phone that is able to add and approve new devices on the contract, but you want to set limits on how many new keys they can add in one day to prevent abuse. Or maybe you want to be able to add access to a very limited scope, maybe a key that can only interact with a single contract. To do that we need to either replace or improve the key purpose concept with an "authorization contract" that authorizes a key to do things according to a given set of rules.
We want to keep the types of authorization contracts open, so for the purpose of this issue we would simplify with 2 example authorization contracts:

a contract that limits which contracts that key can interact with. It can be set to no contracts, only one specific contract, or any contract
a contract that limits the amount of "addKey" operations that each key can do, on the identity contract, per day. By default the logic would be: after adding N new keys, no new "addKeys" operations are accepted for N days

A proposed layout for how this would look for the user would be similar to the authorization screens of android or facebook apps, to be added on the "pendingAuthorization" screen:

@alexvandesande Here is my implementation plan, would love some feedback on it!
Additions:

GeneralAuthorizationInterface.sol

isActionAuthorized() (param list for each will be updated later)
initAuthorization()
addAuthorizedAccount()
revokeAuthorizedAccount()
actionExecuted()

AuthorizationWhiteListedAddress.sol (Checks if address is interacting with a whiteListed address)
AuthorizationKeyAdditionLimit.sol (Checks and keeps track of how many keys have been added by a certain address)
SignedExecutionScheme.sol:
** Modifications **

add a new parameter bytes extraData in executeSigned() similar to what is mentioned in https://eips.ethereum.org/EIPS/eip-1077 for future compatibility. This extraData parameter will include an address which points to authorizationContract to check.
before any action is executed, the contract will check the authorizationContract, with the approvers and action provided in the call,
if action is successfully executed, the actionExecuted() function in the authorization contract which will update the state in authorization contract.

Rationale:
This solution avoid any conflicts with the existing requiredApprovals construct already implement in the code base and since each key is not tied to a specific authorization contract, each key can be given many different authorized actions (using different authorization contracts)

@shine2lay you have been approved to start work on this bounty :)
@alexvandesande please provide feedback on @shine2lay's action plan at your earliest convenience. Thanks!

I support Shine's action plan and we've talked previously about it. I would like to share a diagram description of the functions he describes:

(in this case, the identity is the `executor` and an external chosen contract would be the `authorizationContract` (sometimes also called "terms of use")

@alexvandesande Diagrams look awesome! Great way to share info 👍

Something that we need to think about, which @mkosowsk raised, is how to handle multiple authorization contracts. If key A needs to call authorization contract A1 and key B calls authorization contract B1, then how does the identity handle when user A and B try to do action C? Does the contract call each authorization contract with both users on it? What if A1 doesn't know anything about user B? What if they are both authorized to do it in A1 but not in B1?
We could only allow one authorization contract per identity, but I feel it would be less flexible. Another option would be to make actionExecuted a function that is only "write only" meaning that the authorization contract is never supposed to "throw" it, only to acknowledge.

The idea here being that auth contracts act as advisors, it's ultimately the identity's job to decide either to do something or not, so each identity contract could decide how to deal with that. In this case, they could simply ask all auth contracts for all users and if a single auth contract authorizes it, then it proceeds with the action, and lets the other know that the action was performed anyway

Hey @shine2lay, Ryan from Gitcoin here. How is this coming along? I saw your PR, #119, but haven't seen much activity on this thread or the PR in a week or so.

@ryan-shea I was at DevCon last week, met some folks there and discussed solution for this PR. We are not entirely sure what is the best solution yet.

Hey @shine2lay any additional updates on this? Snoozing the bot :)

Hey @shine2lay and @alexvandesande, any news here? Want to check in 😄

@ryan-shea no updates, can't progress until marek and i have a chat.
@ryan-shea could you remove me from the task? I feel bad that i can't put much time on it now. I don't want to hold up if anyone else wants to take on this task.

@shine2lay do you have a rough estimate of how long it would take you if you were to stay on?

@ceresstation unknown. like i mentioned the main reason is that i am blocked. We are attempting to unblock it but communication has been very slow.


(eos_designs) Invalid characters allowed in VRF, SVI, VLAN name.

Issue Summary
Invalid characters are permitted in VRF, VLAN, and SVI names when generating tenant configurations with AVD. The tested example was with parentheses in a VRF name.
Which component(s) of AVD impacted
eos_designs
How do you run AVD ?
Ansible CLI with AVD Runner
Steps to reproduce
1. Create a VRF with a parentheses in the name.

  Lab-Tenant:
    vrfs:
      Lab-(Tenant)-_Internal:
        vrf_vni: 13

Run the ansible-playbook for the fabric.
View pending tasks in CVP and note the invalid configuration error.

### Relevant log output

```shell
Example of editing a VLAN in EOS directly with the parentheses in the name:

lab-bl-01a(config)#vlan 3100
lab-bl-01a(config-vlan-3100)#description test(VLAN)
% Invalid input

### Contributing Guide

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

We will add format validation in the schema for both eos_designs and eos_cli_config_gen.
This will require us to enhance the schema tooling for formats.


Update the README file section for heartbeat

Resolved the issue with the setup of heartbeat. Need to set a list of lists as the argument in the worker. Update the documentation accordingly.

Awesome!


Add Opa Middleware Docs

Describe the proposal
Add basic document and examples for the new Open Policy Agent Middleware: https://github.com/dapr/dapr/issues/1617

@RichiCoder1 Will you be able to get to these docs? We like to ensure that docs are there to support the code in the release. Thanks

@msfussell My pr https://github.com/dapr/docs/pull/787/files should have all the basics. Might expand on it, but not sure what else I should add that wouldn't be redundant.


allow explicitly setting authentication scheme for HttpClient

For cases where our out-of-the-box authentication does not work, it might be useful to allow users to explicitly specify the authentication scheme(s) to use for both proxy and http authentication.
Something like:
http.auth.scheme=ntlm
http.auth.scheme.proxy=basic
This has originally been raised here: https://issues.gradle.org/browse/GRADLE-2589
Expected Behavior
I want to be able to somehow explicitly say to my gradle build that I want to only use BASIC authentication when publishing Maven artifact.
Current Behavior

Currently, I am facing a problem where maven-publish is not letting me publish artifacts on Nexus using BASIC authentication. Gradle chooses NTLM, but the credentials I provide are NOT from NTLM-user, so authentication fails, and BASIC authentication is never attempted, even though it could work. Setting -Dhttp.keepAlive=true, as suggested in https://discuss.gradle.org/t/cannot-access-repositories-via-basic-auth-proxy/7601/37, does not help.
Context

I am trying to publish a Java library into corporate Nexus repository, which is authenticated. The process works for my personal user, because it's an AD user and NTLM authentication succeeds; but when I use dedicated CI user (bot) account credentials, these fail with NTLM, even though Nexus accepts them via BASIC HTTP Auth.

hmm, I came across some suggestion to use
            authentication {
                basic(BasicAuthentication)
            }

and now it says in the logs that

2021-02-16T16:37:56.408+0200 [DEBUG] [org.gradle.internal.resource.transport.http.HttpClientConfigurer] Using Credentials [username: ci.kn.thirdparty] for authenticating against 'repository.my.domain:-1' using Basic

but there's still this bit in the logs:
2021-02-16T16:37:56.511+0200 [DEBUG] [org.apache.http.impl.execchain.MainClientExec] Target auth state: UNCHALLENGED
2021-02-16T16:37:56.511+0200 [DEBUG] [org.apache.http.impl.execchain.MainClientExec] Proxy auth state: UNCHALLENGED
2021-02-16T16:37:56.595+0200 [DEBUG] [org.apache.http.impl.auth.HttpAuthenticator] Authentication required
2021-02-16T16:37:56.596+0200 [DEBUG] [org.apache.http.impl.auth.HttpAuthenticator] repository.int.kn:443 requested authentication
2021-02-16T16:37:56.596+0200 [DEBUG] [org.apache.http.impl.auth.HttpAuthenticator] Authorization challenge processed
2021-02-16T16:37:56.596+0200 [DEBUG] [org.apache.http.impl.auth.HttpAuthenticator] Authentication failed

is it possible to see wire logs somehow?

yes, if someone could provide a working example how to configure wire logging for Apache Http Client used by maven-publish plugin, that would be awesome. I don't understand why is it so insanely difficult to customize system properties for JVM processes that execute Gradle build tasks. Some guide on "advanced troubleshooting" would certainly be a huge step forward

apparently the original issue has to be closed - there is a way to allow for explicit authentication scheme configuration


Refactor for dataset

dataset is well supported and can otherwise be polyfilled, no need to use get/setAttribute for the data attributes.

#7


Ignore foreign key columns for updated_at

This could alleviate unexpected updated_at values by making the host and the guest behave the same.

Atm I compare the new row against the old using NEW and OLD. This would require looping and checking just the columns I want, which would likely be slower. But maybe I can do an analogy to _.pick to reduce the columns of what is compared without it being much slower?
Docs here: https://www.postgresql.org/docs/9.2/static/plpgsql-trigger.html

Related: http://dba.stackexchange.com/questions/1957/sql-select-all-columns-except-some

Based on the above Q&A, it may be best to just write out the values to check at the time that the thing is created. For instance,
NEW.a, NEW.b, NEW.c,
leaving out NEW.foreign_key_column. All of these values would be known at the time that I'm configuring the trigger. Migrating the table would require deleting and re-adding a new trigger. And performance may be a consideration?

Another idea would be to store all relationships in associative tables. There would be three benefits to this:

updated_at is consistent
code throughout the code base is much more consistent (1 code path for all relationships rather than 5 code paths)
all relationships would be set up to have extra data associated with the relationship itself

The cons are:

performance

I'd need to measure the performance impact to see if it's worth it.


hcm: log before onDestroy

Signed-off-by: Kuat Yessenov<EMAIL_ADDRESS>Commit Message: call log() before onDestroy(), as well as populate final stream info details at the time of log() rather than the deferred destruction.
Additional Description:
Risk Level: low (might affect timings of events)
Testing: unit
Docs Changes: none
Release Notes: none
Fixes #12616
Fixes #12840

Windows failure looks like a flake in ext_authz test. Can someone re-kick windows CI?

/retest
should work for Azure pipelines now

Oh actually already tagged for Snow, so I'll leave that as is :-P


Cannot restore data using etcd v3 when all 3 hosts are powered off in a 3 node setup

Rancher versions:
rancher/server: master
kubernetes (if applicable):v1.7.2
**Docker version: (docker version,docker info preferred)**1.12.6
**Operating system and kernel: (cat /etc/os-release, uname -r preferred)**Ubuntu
**Type/provider of hosts: (VirtualBox/Bare-metal/AWS/GCE/DO)**DO
**Setup details: (single node rancher vs. HA rancher, internal DB vs. external DB)**Single node
**Environment Template: (Cattle/Kubernetes/Swarm/Mesos)**Kubernetes
Steps to Reproduce:

Create a 3 node setup with no isolated planes and create user pods
Power off all the 3 nodes
Add a  new host (h4)
Perform restore steps
In this case, we lose all data and we have no way to recover as there is no persistent storage.
We get a new one node cluster.

In etcd v2, we could restore from NFS storage.

Tested with rancher/etcd:v3.0.17-4.

Created a Kubernetes setup with non isolated planes and created user pods
Power off all the 3 nodes with etcd
Add a new host (h4)
Performed restore steps with persistent storage (NFS)
A healthy single node cluster was formed with no data loss. Creation of new pods was successful.

Tested the same with isolated planes. Restore from persistent storage(NFS) was successful. A healthy single node cluster was formed with no data loss. Creation of new pods was successful.


Api changes for explorer

This implements:

https://github.com/ooni/backend/issues/639
https://github.com/ooni/backend/issues/640

Potentially superseded by https://github.com/ooni/backend/pull/641


Missing symbol from the shared library

The symbol WebRtcIlbcfix_InitDecode is missed from the shared library. The following command produces no output:
$ nm -D libilbc.so | grep 'WebRtcIlbcfix_InitDecode'

As a result, applications that uses this function and tries to link against libilbc.so fails to build. For example, FFmpeg is showing me this error:
/usr/bin/ld: /tmp/ffconf.mxjpOf0E/test.o: in function `check_WebRtcIlbcfix_InitDecode':
test.c:(.text+0x3): undefined reference to `WebRtcIlbcfix_InitDecode'
collect2: error: ld returned 1 exit status
ERROR: libilbc not found

This can be fixed by the following modification, but I'm not sure if it would be desirable for the project. Probably there could be a better solution.
--- a/CMakeLists.txt
+++ b/CMakeLists.txt
@@ -44,7 +44,6 @@ if(NOT CMAKE_BUILD_TYPE)
     set(CMAKE_BUILD_TYPE Release)
 endif()

-set(CMAKE_C_VISIBILITY_PRESET hidden)
 set(CMAKE_CXX_VISIBILITY_PRESET hidden)
 set(CMAKE_VISIBILITY_INLINES_HIDDEN 1)

Darn, I fixed this for init_encode but not init_decode…

Thank you for fixing this! Working fine now :)


ciao-image: Fix open file issue on posix write ops

When writing a new image on posix backend, we open a new file
and drop all image data there, this opened file needs to be
closed before ending the function.
Fixes #661
Signed-off-by: Obed N Munoz<EMAIL_ADDRESS>

@rbradford, you're right, my bad, I forgot to rename it. I'm changing it to something more descriptive. Taking https://golang.org/pkg/io/#CopyBuffer as base, I'm thingking in something like length or file_length

Coverage increased (+0.05%) to 64.79% when pulling 6e29ac2cb512f6c36d9c7d20977a7af2e388bec3 on obedmr:issue-661 into e7e35eb09db9e1dfabb11ecd2cf5613f2d4aedb0 on 01org:master.

@obedmr Is there any value in returning that? Does it get used anywhere?

@rbradford, you have a good point, as far I can see, that lenght value is not used anywhere. So, We should change this to only returning the error. It would imply a change on https://github.com/01org/ciao/blob/master/ciao-image/datastore/ceph.go#L65 What do you think?

Coverage increased (+0.03%) to 64.764% when pulling 56b2b77ea3144ae12ddc7a50dfe713a187b0b5bc on obedmr:issue-661 into e7e35eb09db9e1dfabb11ecd2cf5613f2d4aedb0 on 01org:master.

Coverage increased (+0.006%) to 64.742% when pulling 56b2b77ea3144ae12ddc7a50dfe713a187b0b5bc on obedmr:issue-661 into e7e35eb09db9e1dfabb11ecd2cf5613f2d4aedb0 on 01org:master.

@obedmr Yes. You should change that file too.

Coverage increased (+0.02%) to 64.764% when pulling 58dea81101d5ac8c116b7da9a6ae99e08e5bd7b7 on obedmr:issue-661 into 14a0a250477afc65964f9cc32b2eebe8c462d475 on 01org:master.

Coverage decreased (-0.02%) to 64.727% when pulling 6949c9e01c3ec4043419863bedf3666d5d1a45eb on obedmr:issue-661 into a193e0734c70362bae33d0279765239448654be6 on 01org:master.

Coverage decreased (-0.01%) to 64.738% when pulling 6949c9e01c3ec4043419863bedf3666d5d1a45eb on obedmr:issue-661 into a193e0734c70362bae33d0279765239448654be6 on 01org:master.

Coverage decreased (-0.01%) to 64.738% when pulling 6949c9e01c3ec4043419863bedf3666d5d1a45eb on obedmr:issue-661 into a193e0734c70362bae33d0279765239448654be6 on 01org:master.

@rbradford @markdryan done, thanks for the feedback.

Coverage increased (+0.03%) to 64.797% when pulling bc0b292eb3ab645ed5791665738a1505042309c4 on obedmr:issue-661 into cd27b8e36a75b4961908a6f591f8e208eaf39dc0 on 01org:master.


Add files via upload

screens for homework Git session Task1
URL for Task 2 and 3 https://github.com/IhorKyl/git_hmwrk/pull/1
uoft-dsi-c3-homework-20240426
What changes are you trying to make? (e.g. Adding or removing code, refactoring existing code, adding reports)
What did you learn from the changes you have made?
Was there another approach you were thinking about making? If so, what approach(es) were you thinking of?
Were there any challenges? If so, what issue(s) did you face? How did you overcome it?
How were these changes tested?
A reference to a related issue in your repository (if applicable)
@mentions of the person or team responsible for reviewing proposed changes (At least 2 people)

Checklist

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

uoft-dsi-c3-homework-20240426


Elastic profile configuration

Added requested validations. Fixed CPU metadata settings. Additional container test.

Thank you so much for this, @jikuhn!


Include a way for services to ping Cabot and give status update for cron monitoring

There's a common pattern for cron job monitoring to ping a randomized http endpoint on success and the monitoring system make sure that a heartbeat is received every X interval.
Example would be https://deadmanssnitch.com/
Would you consider adding something like that into Cabot so that it can monitor cron jobs?

You might find the Coal Mine project our company open sourced useful: https://github.com/quantopian/coal-mine
It's written in Python and it's Apache licensed, but I haven't looked into how complex an integration would be.

Hi - we just built an open source project for Cabot to solve this very issue.  Basically it turns Cabot into a DeadmansSnitch    It uses redis to uses a Cabot check to poll redis keys, that expire when the process or snitch you are monitoring fails to report in.
https://github.com/objectstoragesolutions/cabot-check-tattletrail


No Action Available when executing 'doQuickfix'

The file in question is a javascript file but the filetype is set to javascript.jsx. The diagnostics do appear in the command line but when I run CocActionAsyn('doQuickfix') via <Plug>(coc-fix-current) I get *No action available in the commandline. The dianostic in question are identation errors which I can fix using ALE but I was under the impression that I could use code actions to fix it like I would in VSCode. Below are the logs that I retrieved from both tsserver and the LSP
LSP log (abbreviated for brevity)
[Trace  - 8:16:59 PM] Response received: getSupportedCodeFixes (14). Request took 4 ms. Success: true
Result: [
    "2352",
    "1329",
    "7051",
    "80004",
    "7034",
    ....
]

Tsserver (abbreviated for brevity)
Perf 486  [20:44:15.655] 14::getSupportedCodeFixes: elapsed time (in milliseconds) 0.2613
Info 487  [20:44:15.655] response:
    {"seq":0,"type":"response","command":"getSupportedCodeFixes","request_seq":14,"success":true,"body":["2352","1329","7051","80004","7034","7005","7006","7019","7033","7010","7032","7008","7046","7043","7044","7047","7048","7050","7049","7045","80002","80006","80001","...}

Perhaps the listed actions above do no fix indentations?

Please provide code and configuration to reproduce the problem.

Check if the diagnostic code inside the result code, I think not, so it's expected.