Must authenticate when docker container is restarted

Repro:

Run with docker-compose up -d
Sign in to localhost:5001 and authenticate
Restart presencelight with docker-compose restart

Expected behavior: PresenceLight starts running with authentication from the previous run
Actual behavior: User must authenticate with PresenceLight again before PresenceLight can do its thing

Rewriting this issue to be an issue instead of rambling.

Microsoft Identity makes my head hurt. This might be relevant, I think?
https://github.com/AzureAD/microsoft-identity-web/wiki/token-cache-serialization

I need to use a different caching mechanism I think.... Looking more into it

@CamSoper can you try the fix in this PR?
https://github.com/isaacrlevin/presencelight/pull/616
You will have to build the image and container but see if that resolves your issue

Will do. Give me a day or two.

@isaacrlevin Well, I tried. I can't get it built. If I try to docker build ./PresenceLight.Web at the CLI, it gets blows up on copying the csproj files. If I right-click on the Dockerfile and try to build, it gets to the end but then the PressenceLight.Web.csproj blows up complaining that the docker CLI isn't installed (it is).
I'm not out of tricks to try, I thought about just forking your repo and using your GH actions to build it, but if you could just tag a build from this PR "experimental" and push to Docker Hub, that'd probably save me some troubleshooting...

The docker build worked when I used your parameters. I'm not sure why your docker command worked and mine didn't.
Anyway I tagged it as camsoper/camthegeek:pl-custom and sent it to my public docker hub (I actually run on my Ubuntu server, x64 FYI) and then used this docker-compose.yaml:
version: '3.7'

services:
  presencelight:
    #image: isaaclevin/presencelight:latest
    image: camsoper/camthegeek:pl-custom
    container_name: presencelight
    restart: unless-stopped
    ports:
      - 5000:80
      - 5001:443
    volumes:
      - ./https:/https:ro
      - ./config:/app/config
      - ./aspnet:/root/.aspnet
    environment:
      ASPNETCORE_HTTPS_PORT: "5001"
      ASPNETCORE_URLS: "https://+;http://+"
      ASPNETCORE_Kestrel__Certificates__Default__Password: "presencelight"
      ASPNETCORE_Kestrel__Certificates__Default__Path: "/https/presencelight.pfx"
      Serilog__MinimumLevel: "Error"

Did a docker-compose up and it barfed thusly:
presencelight    | [19:35:16 ERR] An unhandled exception has occurred while executing the request.
presencelight    | System.ArgumentNullException: IDW10106: The 'ClientId' option must be provided.
presencelight    |    at Microsoft.Identity.Web.MergedOptionsValidation.Validate(MergedOptions options)
presencelight    |    at Microsoft.Identity.Web.MicrosoftIdentityWebAppAuthenticationBuilderExtensions.<>c__DisplayClass5_0.<AddMicrosoftIdentityWebAppInternal>b__3(OpenIdConnectOptions options, IServiceProvider serviceProvider, IOptionsMonitor`1 mergedOptionsMonitor, IOptionsMonitor`1 msIdOptionsMonitor, IOptions`1 msIdOptions)
presencelight    |    at Microsoft.Extensions.Options.ConfigureNamedOptions`5.Configure(String name, TOptions options)
presencelight    |    at Microsoft.Extensions.Options.OptionsFactory`1.Create(String name)
presencelight    |    at Microsoft.Extensions.Options.OptionsMonitor`1.<>c__DisplayClass10_0.<Get>b__0()
presencelight    |    at System.Lazy`1.ViaFactory(LazyThreadSafetyMode mode)
presencelight    |    at System.Lazy`1.ExecutionAndPublication(LazyHelper executionAndPublication, Boolean useDefaultConstructor)
presencelight    |    at System.Lazy`1.CreateValue()
presencelight    |    at System.Lazy`1.get_Value()
presencelight    |    at Microsoft.Extensions.Options.OptionsCache`1.GetOrAdd(String name, Func`1 createOptions)
presencelight    |    at Microsoft.Extensions.Options.OptionsMonitor`1.Get(String name)
presencelight    |    at Microsoft.AspNetCore.Authentication.AuthenticationHandler`1.InitializeAsync(AuthenticationScheme scheme, HttpContext context)
presencelight    |    at Microsoft.AspNetCore.Authentication.AuthenticationHandlerProvider.GetHandlerAsync(HttpContext context, String authenticationScheme)
presencelight    |    at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware.Invoke(HttpContext context)
presencelight    |    at Microsoft.AspNetCore.Diagnostics.ExceptionHandlerMiddleware.<Invoke>g__Awaited|6_0(ExceptionHandlerMiddleware middleware, HttpContext context, Task task)
presencelight    | [19:35:16 ERR] An exception was thrown attempting to execute the error handler.
presencelight    | System.ArgumentNullException: IDW10106: The 'ClientId' option must be provided.
presencelight    |    at Microsoft.Identity.Web.MergedOptionsValidation.Validate(MergedOptions options)
presencelight    |    at Microsoft.Identity.Web.MicrosoftIdentityWebAppAuthenticationBuilderExtensions.<>c__DisplayClass5_0.<AddMicrosoftIdentityWebAppInternal>b__3(OpenIdConnectOptions options, IServiceProvider serviceProvider, IOptionsMonitor`1 mergedOptionsMonitor, IOptionsMonitor`1 msIdOptionsMonitor, IOptions`1 msIdOptions)
presencelight    |    at Microsoft.Extensions.Options.ConfigureNamedOptions`5.Configure(String name, TOptions options)
presencelight    |    at Microsoft.Extensions.Options.OptionsFactory`1.Create(String name)
presencelight    |    at Microsoft.Extensions.Options.OptionsMonitor`1.<>c__DisplayClass10_0.<Get>b__0()
presencelight    |    at System.Lazy`1.ViaFactory(LazyThreadSafetyMode mode)
presencelight    | --- End of stack trace from previous location ---
presencelight    |    at System.Lazy`1.CreateValue()
presencelight    |    at System.Lazy`1.get_Value()
presencelight    |    at Microsoft.Extensions.Options.OptionsCache`1.GetOrAdd(String name, Func`1 createOptions)
presencelight    |    at Microsoft.Extensions.Options.OptionsMonitor`1.Get(String name)
presencelight    |    at Microsoft.AspNetCore.Authentication.AuthenticationHandler`1.InitializeAsync(AuthenticationScheme scheme, HttpContext context)
presencelight    |    at Microsoft.AspNetCore.Authentication.AuthenticationHandlerProvider.GetHandlerAsync(HttpContext context, String authenticationScheme)
presencelight    |    at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware.Invoke(HttpContext context)
presencelight    |    at Microsoft.AspNetCore.Diagnostics.ExceptionHandlerMiddleware.HandleException(HttpContext context, ExceptionDispatchInfo edi)
presencelight    | [19:35:16 ERR] Connection id "0HMG17B1M5ITI", Request id "0HMG17B1M5ITI:00000001": An unhandled exception was thrown by the application.
presencelight    | System.ArgumentNullException: IDW10106: The 'ClientId' option must be provided.
presencelight    |    at Microsoft.Identity.Web.MergedOptionsValidation.Validate(MergedOptions options)
presencelight    |    at Microsoft.Identity.Web.MicrosoftIdentityWebAppAuthenticationBuilderExtensions.<>c__DisplayClass5_0.<AddMicrosoftIdentityWebAppInternal>b__3(OpenIdConnectOptions options, IServiceProvider serviceProvider, IOptionsMonitor`1 mergedOptionsMonitor, IOptionsMonitor`1 msIdOptionsMonitor, IOptions`1 msIdOptions)
presencelight    |    at Microsoft.Extensions.Options.ConfigureNamedOptions`5.Configure(String name, TOptions options)
presencelight    |    at Microsoft.Extensions.Options.OptionsFactory`1.Create(String name)
presencelight    |    at Microsoft.Extensions.Options.OptionsMonitor`1.<>c__DisplayClass10_0.<Get>b__0()
presencelight    |    at System.Lazy`1.ViaFactory(LazyThreadSafetyMode mode)
presencelight    |    at System.Lazy`1.ExecutionAndPublication(LazyHelper executionAndPublication, Boolean useDefaultConstructor)
presencelight    |    at System.Lazy`1.CreateValue()
presencelight    |    at System.Lazy`1.get_Value()
presencelight    |    at Microsoft.Extensions.Options.OptionsCache`1.GetOrAdd(String name, Func`1 createOptions)
presencelight    |    at Microsoft.Extensions.Options.OptionsMonitor`1.Get(String name)
presencelight    |    at Microsoft.AspNetCore.Authentication.AuthenticationHandler`1.InitializeAsync(AuthenticationScheme scheme, HttpContext context)
presencelight    |    at Microsoft.AspNetCore.Authentication.AuthenticationHandlerProvider.GetHandlerAsync(HttpContext context, String authenticationScheme)
presencelight    |    at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware.Invoke(HttpContext context)
presencelight    |    at Microsoft.AspNetCore.Diagnostics.ExceptionHandlerMiddleware.<Invoke>g__Awaited|6_0(ExceptionHandlerMiddleware middleware, HttpContext context, Task task)
presencelight    |    at Microsoft.AspNetCore.Diagnostics.ExceptionHandlerMiddleware.HandleException(HttpContext context, ExceptionDispatchInfo edi)
presencelight    |    at Microsoft.AspNetCore.Diagnostics.ExceptionHandlerMiddleware.<Invoke>g__Awaited|6_0(ExceptionHandlerMiddleware middleware, HttpContext context, Task task)
presencelight    |    at Microsoft.AspNetCore.Server.Kestrel.Core.Internal.Http.HttpProtocol.ProcessRequests[TContext](IHttpApplication`1 application)

Ugh, that is my bad. You need the config settings. Put this in appsettings.json
{
  "Logging": {
    "LogLevel": {
      "Default": "Information",
      "Microsoft": "Information"
    }
  },
  "SendDiagnosticData": false,
  "AzureAd": {
    "Instance": "https://login.microsoftonline.com/",
    "TenantId": "common",
    "ClientId": "f8119f58-4523-44c1-ab72-b2e0c815bd6a",
    "CallbackPath": "/signin-oidc",
    "RedirectHost": "https://localhost:5001",
    "SignedOutCallbackPath ": "/signout-callback-oidc",
    "ClientSecret": "io2~p.3-2~x646_TMC-uo4tL9610mA2RxO"
  },
  "DownstreamApi": {
    "BaseUrl": "https://graph.microsoft.com/beta",
    "Scopes": "user.read presence.read offline_access"
  },
  "ApplicationInsights": {
    "InstrumentationKey": "63a1f0f1-4e30-4dd2-86dc-9acc718e675b"
  },
  "SnapshotCollectorConfiguration": {
    "IsEnabledInDeveloperMode": true,
    "ThresholdForSnapshotting": 1,
    "MaximumSnapshotsRequired": 3,
    "MaximumCollectionPlanSize": 50,
    "ReconnectInterval": "00:15:00",
    "ProblemCounterResetInterval": "1.00:00:00",
    "SnapshotsPerTenMinutesLimit": 1,
    "SnapshotsPerDayLimit": 30,
    "SnapshotInLowPriorityThread": true,
    "ProvideAnonymousTelemetry": true,
    "FailedRequestLimit": 3
  },
  "AppType": "Web",
  "AppVersion": "Local"
}

Well we're a lot closer.
Got it all built and working. Started with docker-compose up -d, connected, authed, and verified it was picking up statuses and sending them to my custom API.
Then I did a docker-compose restart and it restarted. I test-changed my Teams status, but it wasn't picked up.
So I went back to the browser tab with the web app and hit F5 and observed the network traffic. It never redirected to AAD, it just lit up and started working again. The downside is that I had to hit F5 in the browser before it'd work.
(I also tested again to make sure.)

When you run it the 2nd time, can you do a screengrab of the console logs BEFORE you refresh the page, and than when you do? I have an idea what the issue is but want to confirm

@CamSoper
This may be a challenge with how Microsoft Identity works. When you have a token cache, there is a requirement that you can access it with the account that the token represents. In the case when the app restarts, at start time it does not know the user account (as you "could" have multiple accounts). The reason why the browser refresh makes it all work is because of cookies.
So the flow is

Token cache file is persisted, representing some user
When app starts, it has no users, so it does not load the cache
Popping the browser triggers a population of the user account
Next Graph poll the cache is loaded and we are on our merry way

I am looking at my options of serializing the user account and persisting that is well, but I can assume that is a terrible practice. At lease if I have a serialized file with the account I can load that and maybe the cache gets read

@cmatskas can you help me hunt down what I might need to do here? Long story short I want to persist a user account in cache through a docker restart WITHOUT having to trigger the repopulation of that UserAccount? Here are the bits in code where I am messing interacting with Identity
https://github.com/isaacrlevin/presencelight/blob/main/src/PresenceLight.Razor/Services/UserAuthService.cs
Is what I am doing possible?

Hmm, probably not possible. Can you hit up our dev support team for this? HTTPS://Aka.ms/425Show/help. Ping me when you get your issue raised so I can chase up. Thx

Not possible after research :(


Make the number of nominations configurable

Builds on top of https://github.com/paritytech/substrate/pull/8113, and makes the number of votes per nominator configurable, while adding integrity checks to prevent footguns.
I don't intend to change the value to 24 in the companion yet. That can happen afterwards.
cc @wpank @gavofyork
This has already been audited (almost identical logic in #7929) and the main feedback was that this will need re-benchmarking.

this needs to merge master to be easier to review

Done, but it was quite a mess, I have to re-review myself and make sure I didn't mess things up.

Does someone know a way to also remove duplicate commits, while keeping the PR? I could just generate diff from this branch, and apply it to a new branch to get cleaner commit history, but I prefer keeping the PR.

git rebase -i HEAD~3 lets you omit / squash / rearrange the last 3 commits. Of course you can specify whatever number of them you want. Then just git push -f.

cheers @coriolinus

bot merge


Add metrics when poll for decision task fail due to bad binary

for https://github.com/uber/cadence/issues/2465

Coverage increased (+0.02%) to 68.987% when pulling f6a14273ebe5d1d88ef9d356a51a53cec53bae5b on longquanzheng:bad-build-metric into 6ab93242d30f2d41f3bb2b6470e94b318f21e6c7 on uber:master.


unit tests using the realm

Hello everyone, I hope you are all well!
I would like to know if there is any way to integrate the realm with the unit testing features with mockito/mocktail.
I'm trying to test the realm mapping class, but I'm not having success.
Is there any way to mock the return results?
Thanks!

I find it easier to open an actual in-memory realm for unit tests. Would that work for you?

Good morning @nielsenko!
Thank you very much for answering.
Sorry for taking so long to respond.
One question (sorry for my ignorance, I'm new to Flutter), in case, wouldn't this qualify as an integrated test?
As I only need a stub to know the return, would I need the realm in memory so I can test?

Hi Fernando
Since you explicitly mentioned mockito let me quote the author on best practices:

Testing with real objects is preferred over testing with mocks - if you can construct a real instance for your tests, you should!

I totally agree. Don't stub unless you have to. Since you can easily spin up a realm in-memory, I suggest you do that for your tests.
Now "all" you need to stub/fake is the backend synchronisation, if you want to avoid the burden of doing an actual integration test with the backend.
One approach is to have a fake that implements Realm  simply delegating to an actual in-memory realm, but faking Realm.subscriptions, Realm.syncSession, SubscriptionSet.waitForDownload, SubscriptionSet.waitForUpload and Session.getProgressStream to the extend you need it. For many test-cases that may not even be necessary.

For the test of the SDK itself we actually spin-up a fresh backend and do proper integration tests whenever we test sync related functionality. It is not too hard to automate setup and teardown using realm-cli. That is one of the joys of SaaS 😄

Thanks for replying again, @nielsenko.
We are currently using mocktail for testing.
As mockito currently needs generated files, we chose to use mocktail even to not have this dependency.
Sorry to bother you, but I was wondering if you have any issues with the Realm framework and how it should be used (in this case, best practices for collection names and relationships)?

About your question regarding relationships, I have a structure like this in my collection:
{
    "_id": "object-id",
    "name": "name",
    "created_at": {
        "user_id": "object-id",
        "username": "name"
    },
    "updated_at": {
        "user_id": "object-id",
        "username": "username"
    }
}

And about the structure, would it be correct to have relationships where the collection has its name separated by an underscore?
I created 2 private classes (within my target class) defined as _created_at and _updated_at (following the template in the doc)
and when generating the g.dart file, the attribute is transformed like this in the get and set methods:
@override
createdat? get createdAt =>
    RealmObject.get<createdat>(this, 'created_at') as createdat?;
@override
set createdAt(covariant createdat? value) =>
    RealmObject.set(this, 'created_at', value);

@override
updatedat? get createdAt =>
    RealmObject.get<updatedat>(this, 'created_at') as createdat?;
@override
set createdAt(covariant updatedat? value) =>
    RealmObject.set(this, 'created_at', value);

And when running the project, it is informed that the above attributes were not found
The main class looks like this:
// main collection
@RealmModel()
@JsonSerializable()
@_ObjectIdConverter()
class _collection {
  @PrimaryKey()
  @MapTo('_id')
  late ObjectId id;
  @MapTo('name')
  late String name;
  @MapTo('created_at')
  late _created_at? createdAt;
  @MapTo('updated_at')
  late _updated_at? updatedAt;
}

// relationship classes
@RealmModel()
@JsonSerializable()
@_ObjectIdConverter()
class _created_at {
  @MapTo('id')
  late ObjectId userId;
  @MapTo('name')
  late String userName;
}

@RealmModel()
@JsonSerializable()
@_ObjectIdConverter()
class _updated_at {
  @MapTo('id')
  late ObjectId userId;
  @MapTo('name')
  late String userName;
}

Please forgive me if I'm being annoying, it's just that I would like to spend the possibilities of use without having to write anything native to be able to support (but if there is a need, I'll have to).
Thank you very much for your attention.

Hi Fernando
Unfortunately the above schema would require embedded objects, which we don't support yet. Also, adding json annotations at the model level will not do what you probably expect (see #683).
In general, whenever you use embedded objects (which are very common in MongoDB) you need to hoist out the object into a separate collection and make a link. That is until we support embedded objects 😒
I'm not sure what the @_ObjectIdConverter does in the above code, but if we forget about that and @JsonSerializable() then the following model might work for you (assuming you can change the schema on Atlas):
// main collection
import 'package:realm_dart/realm.dart';

part 'issue870.g.dart';

@RealmModel()
@MapTo('collection') // you need a collection in Atlas to match
class _Collection {
  @PrimaryKey()
  @MapTo('_id')
  late ObjectId id;

  // @MapTo('name') // <-- not needed, since field has the same name
  late String name;

  @MapTo('created_at')
  late _Change? createdAt;

  @MapTo('updated_at')
  late _Change? updateAt;
}

@RealmModel()
@MapTo('change') // you need a collection in Atlas to match
class _Change {
  @PrimaryKey()
  @MapTo('_id') // As this is hoisted into its own collection, it needs a _id field of its own
  late ObjectId id;

  @MapTo('user_id')
  late ObjectId userId;

  @MapTo('username')
  late String userName; // why not username both places? (or user_name on Atlas?)

  late DateTime at; // I assume you forgot the actual timestamp in your model
}

Notice I have introduced a new collection called change(on Atlas) and Change (in Dart) that is used for both the createdAtandupdatedAt` fields.
Is this a feasible way forward for you?

BTW, unless you are already bound to a schema on Atlas, I suggest enabling developer mode under sync, which will allow the client to do additive changes to the schema.
Eventually you will do a destructive change, in which case the easier path forward is just to delete the old Atlas App Service application and start again.
I find that to be the fastest way to experiment with the schema - if you do greenfield development.
Also, getting familiar with realm_cli is time well spent. It will allow you to script setup and teardown of your apps on Atlas.

Thank you very much for replying.
You gave a great help.
I talked to my techlead and we'll set up a meeting with the backend team so we can rethink the structure.
As there is still no support for Map types, there would be a need to change to the model you presented me.
Again, thank you so much for your help and God bless you.

You are welcome. I don't know about your timelines, but we are actively adding missing features. Unfortunately I cannot commit to any particular date, but embedded objects are high on our list of priorities.

@nielsenko Is there any example available for unit testing either with Mockito or without?
I've created a singleton class inside database_service.dart
class DatabaseService {
  static final DatabaseService _instance = DatabaseService._internal();

  factory DatabaseService() {
    _instance._openRealm();
    return _instance;
  }

  final Logger log = Logger('DatabaseService');

  DatabaseService._internal();

  late Realm _realm;

  Realm get realm => _realm;

  final Configuration _config =
      Configuration.local([Patient.schema]);

  _openRealm() {
    _realm = Realm(_config);
  }

  closeRealm() {
    if (!realm.isClosed) {
      realm.close();
    }
  }
}

from the tests if I try to call DatabaseService(), I'm getting this error
/test/data/repository/patient_repository_test.dart": Invalid argument(s): Failed to load dynamic library '/usr/local/Caskroom/flutter/3.0.4/flutter/bin/cache/artifacts/engine/darwin-x64/../Frameworks/realm.framework/Resources/librealm_dart.dylib': dlopen(/usr/local/Caskroom/flutter/3.0.4/flutter/bin/cache/artifacts/engine/darwin-x64/../Frameworks/realm.framework/Resources/librealm_dart.dylib, 0x0001): tried: '/usr/local/Caskroom/flutter/3.0.4/flutter/bin/cache/artifacts/engine/darwin-x64/../Frameworks/realm.framework/Resources/librealm_dart.dylib' (no such file)

@tushkaty I can reproduce your issue wrt. loading librealm_dart.dylib when invoking flutter test to run widget tests etc. I will get back to you, when I know more.

Just to add that I'm facing the same problem reported by  @tushkaty  when trying to uniting test my application. But one thing is different: I'm not even calling/using Realm in my test file.
I'm getting this:

Failed to load "[path]/X_controller_test.dart": Invalid argument(s): Failed to load dynamic library '/Users/germano/fvm/versions/stable/bin/cache/artifacts/engine/darwin-x64/../Frameworks/realm.framework/Resources/librealm_dart.dylib': dlopen(/Users/germano/fvm/versions/stable/bin/cache/artifacts/engine/darwin-x64/../Frameworks/realm.framework/Resources/librealm_dart.dylib, 0x0001): tried: '/Users/germano/fvm/versions/stable/bin/cache/artifacts/engine/darwin-x64/../Frameworks/realm.framework/Resources/librealm_dart.dylib' (no such file)

@wilsleygermano It's because some of your test cases using the Realm class object.

@tushkaty you are correct. I thought running a specific test file, that doesn't use realm, would avoid that error. Anyway, now you remind me that I'm facing problems with my realm tests the same way you have (using Mockito).

@tushkaty I deleted my test case that used the Realm class and the problem is still there. Any thoughts on that?

Hi @tushkaty and @wilsleygermano,
The new release is available here realm 0.10.0+rc.
You can upgrade the realm to 0.10.0+rc and then to run the new available command flutter pub run realm install from the project root folder. This will download the library and you will be able to run your tests.
I hope this will help.


Lack of proper new lines make some messages unreadable

Hi!
I am not sure if this is a Puppetboard or a PuppetDB issue, so sorry for possible misreporting!
We are using Puppetboard v 1.0 plus my old PRs, from here: https://github.com/gdubicki/puppetboard/tree/with_my_prs
PuppetDB is in version 5.2.8.
The lack of new lines in some messages make them unreadable.
Example from Puppetboard:

The same output from the puppet run terminal output logged to a file:

Note how the the latter it's pretty clear that puppet has added a single empty line while in the former it's not clear at all.

I finally updated to Puppetboard 2.2.0 and I see that the problem is resolved:

Sorry for bothering you!


Allow maxBuffer overwrite by NODE_ENV

Implement overwrite of maxBuffer for child_process.exec by environment variableprocess.env.SFPOWERSCRIPTS_PROCESS_MAX_BUFFER
https://github.com/Accenture/sfpowerscripts/blob/develop/packages/core/src/command/commandExecutor/ExecuteCommand.ts#L27
https://github.com/Accenture/sfpowerscripts/blob/develop/packages/core/src/command/commandExecutor/ExecuteCommand.ts#L27
https://github.com/Accenture/sfpowerscripts/blob/develop/packages/sfpowerscripts-cli/src/commands/sfpowerscripts/orchestrator/publish.ts#L144

Thanks @pelim for raising this. Would you be happy to raise a PR for this?


Make system prompt customizable.

I felt that clipea didn't respect my OS nearly enough (always recommended I use apt even though I am on Mac.
So I think making the system prompt customizable for everyone would be beneficial.
I'll probably submit a PR myself soon.

Thanks. Can you give any examples?
The OS gets provided as part of the environment that's sent to Clippy, so it should know what to use to install stuff. It might be that the prompt needs to be a bit more forceful there.
Having it fully customisable would be cool though. Maybe a ~/.clipea file.

I've had varying responses for these kind of prompts. e.g.
?? deinstall clipea from my system
📎🟢 $ pip uninstall clipea

or sudo apt-get remove clipea
they went away after editing the last line of system-prompt to this:
Important: Every command you output will automatically be executed in this env, be careful to only output commands which will work on this operating system and shell:
But will still output apt-get stuff if my prompt is deinstall clipea from my ubuntu system which seems good.


Update ensemble_schema.json for existing screens to validate against it

Changes to make existing screen definitions validate against the schema.
Main changes:

Widget can now be a string/null (there is usage of widgets in a string/null fashion)

Widget can be just an object to support user-defined widgets

Reworked Conditional widget to support the syntax of existing screens

Added Firestore API definition

Fixed some properties to be values or formulas - not the full list but just the things that were breaking in the validation

Added definitions for the following Action types: Log Event, Get Device Token, Execute Action Group, Execute Conditional Action

Had to update some screens - attaching the full list of screens and script I'm using to validate screens against schema.
attachment for EnsembleUI pull request.zip

Hi @v-krupsky,
The ensemble_schema.json is no longer in use. We now have TypeScript interfaces that automatically generate the schema in a different repo.
@kmahmood74 shouldn't we remove the ensemble_schema.json file from here to avoid confusing people?

Hi @v-krupsky,
The ensemble_schema.json is no longer in use. We now have TypeScript interfaces in a different repo that automatically generate the schema for studio.
@kmahmood74 shouldn't we remove the ensemble_schema.json file from here to avoid confusing people?

As @sharjeelyunus noted, the ensemble_schema.json is an output of the typescript schema. Even that is not the one here so we should remove from this repo.

Thank you @sharjeelyunus @kmahmood74 - could you please advise where and how can I get the current schema?


[Release] sdk/resourcemanager/nginx/armnginx/3.0.0

https://github.com/Azure/sdk-release-request/issues/4723

The breaking changes are introduced and approved in this PR.

/azp run go - armnginx - ci

@ArcturusZhang, @lirenhe, @tadelesh could you please review this PR for NGINX sdk release? Thank you!

/azp run

/azp run go - armnginx - ci


Components

The components page has been created. Few things to be noted are:

The page is completely responsive and dynamic. It is making two API calls, one for the top 10 components and another for the top 3 components of various categories.
The "Author name" is static as of now since it is not there in the JSON received. Few changes need to be done in the backend model for the same.
The search bar is not yet active since a search query API is yet to be made.

@rowlandm @DennisSchwartz
Kindly review the PR. I have added a few details.


Hotstar donot working Please Fix the issue

Please follow the guide below

You will be asked some questions and requested to provide some information, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your issue (like this: [x])
Use the Preview tab to see what your issue will actually look like

Make sure you are using the latest version: run youtube-dl --version and ensure your version is 2018.12.03. If it's not, read this FAQ entry and update. Issues with outdated version will be rejected.

[*] I've verified and I assure that I'm running youtube-dl 2018.12.03

Before submitting an issue make sure you have:

[*] At least skimmed through the README, most notably the FAQ and BUGS sections
[ ] Searched the bugtracker for similar issues including closed ones
[ ] Checked that provided video/audio/playlist URLs (if any) are alive and playable in a browser

What is the purpose of your issue?

[*] Bug report (encountered problems with youtube-dl)
[ ] Site support request (request for adding support for a new site)
[*] Feature request (request for a new functionality)
[ ] Question
[ ] Other

The following sections concretize particular purposed issues, you can erase any section (the contents between triple ---) not applicable to your issue

If the purpose of this issue is a bug report, site support request or you are not completely sure provide the full verbose output as follows:
Add the -v flag to your command line you run youtube-dl with (youtube-dl -v <your command line>), copy the whole output and insert it here. It should look similar to one below (replace it with your log inserted between triple ```):
C:\youtube-dl>youtube-dl --verbose https://www.hotstar.com/tv/kaal-bhairav-rahasya/s-1287/veer-hurts-his-family-members/1000225208
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: ['--verbose', 'https://www.hotstar.com/tv/kaal-bhairav-rahasya/s-1287/veer-hurts-his-family-members/1000225208']
[debug] Encodings: locale cp1252, fs mbcs, out cp437, pref cp1252
[debug] youtube-dl version 2018.12.03
[debug] Python version 3.4.4 (CPython) - Windows-10-10.0.17134
[debug] exe versions: ffmpeg 4.1, ffprobe 4.1, phantomjs 2.1.1
[debug] Proxy map: {}
[hotstar]<PHONE_NUMBER>: Downloading webpage
ERROR: An extractor error has occurred. (caused by KeyError('title',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpflqtkr62\build\youtube_dl\extractor\common.py", line 508, in extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpflqtkr62\build\youtube_dl\extractor\hotstar.py", line 91, in _real_extract
KeyError: 'title'
Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpflqtkr62\build\youtube_dl\extractor\common.py", line 508, in extract
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpflqtkr62\build\youtube_dl\extractor\hotstar.py", line 91, in _real_extract
KeyError: 'title'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpflqtkr62\build\youtube_dl\YoutubeDL.py", line 792, in extract_info
  File "C:\Users\dst\AppData\Roaming\Build archive\youtube-dl\rg3\tmpflqtkr62\build\youtube_dl\extractor\common.py", line 521, in extract
youtube_dl.utils.ExtractorError: An extractor error has occurred. (caused by KeyError('title',)); please report this issue on https://yt-dl.org/bug . Make sure you are using the latest version; type  youtube-dl -U  to update. Be sure to call youtube-dl with the --verbose flag and include its complete output.

...
<end of log>

If the purpose of this issue is a site support request please provide all kinds of example URLs support for which should be included (replace following example URLs by yours):
Description of your issue, suggested solution and other information
Hotstar is not working anymore please fix this issue.
youtube-dl extract hotstar link without token.
example:-https://hsprepack.akamaized.net/videos/bharat/rakrsb/58/master_Layer5.m3u8
but actual full link is
https://hsprepack.akamaized.net/videos/bharat/rakrsb/58/master_Layer5.m3u8?hdnea=st=1544138330~exp=1544140130~acl=/*~hmac=1f3a10e7eb3d43ca68eaff80598d22e43aad8b5ac53aad89f3388a2621fc9e21

youtube-dl http://hotstar.com/tv/kaal-bhairav-rahasya/s-1287/veer-hurts-his-family-members/1000225208
[hotstar]<PHONE_NUMBER>: Downloading webpage
[hotstar]<PHONE_NUMBER>: Downloading JSON metadata
[hotstar]<PHONE_NUMBER>: Downloading m3u8 information
[download] Destination: Veer Hurts His Family Members!-1000225208.mp4
ffmpeg version n4.1 Copyright (c) 2000-2018 the FFmpeg developers
  built with gcc 8.2.1 (GCC) 20180831
  configuration: --prefix=/usr --disable-debug --disable-static --disable-stripping --enable-fontconfig --enable-gmp --enable-gnutls --enable-gpl --enable-ladspa --enable-libaom --enable-libass --enable-libbluray --enable-libdrm --enable-libfreetype --enable-libfribidi --enable-libgsm --enable-libiec61883 --enable-libjack --enable-libmodplug --enable-libmp3lame --enable-libopencore_amrnb --enable-libopencore_amrwb --enable-libopenjpeg --enable-libopus --enable-libpulse --enable-libsoxr --enable-libspeex --enable-libssh --enable-libtheora --enable-libv4l2 --enable-libvidstab --enable-libvorbis --enable-libvpx --enable-libwebp --enable-libx264 --enable-libx265 --enable-libxcb --enable-libxml2 --enable-libxvid --enable-nvdec --enable-nvenc --enable-omx --enable-shared --enable-version3
  libavutil      56. 22.100 / 56. 22.100
  libavcodec     58. 35.100 / 58. 35.100
  libavformat    58. 20.100 / 58. 20.100
  libavdevice    58.  5.100 / 58.  5.100
  libavfilter     7. 40.101 /  7. 40.101
  libswscale      5.  3.100 /  5.  3.100
  libswresample   3.  3.100 /  3.  3.100
  libpostproc    55.  3.100 / 55.  3.100
[hls,applehttp @ 0x55e7bee65080] Opening 'https://hses.akamaized.net/videos/bharat/kbrs2sb/15/phone/media-6/segment-0.ts' for reading
[hls,applehttp @ 0x55e7bee65080] Opening 'https://hses.akamaized.net/videos/bharat/kbrs2sb/15/phone/media-6/segment-1.ts' for reading
Input #0, hls,applehttp, from 'https://hses.akamaized.net/videos/bharat/kbrs2sb/15/phone/media-6/index.m3u8':
  Duration: 00:20:54.16, start: 0.111111, bitrate: 0 kb/s
  Program 0
    Metadata:
      variant_bitrate : 0
    Stream #0:0: Audio: aac (LC) ([15][0][0][0] / 0x000F), 48000 Hz, stereo, fltp
    Metadata:
      variant_bitrate : 0
    Stream #0:1: Video: h264 (Main) ([27][0][0][0] / 0x001B), yuv420p(tv, bt709), 1280x720 [SAR 1:1 DAR 16:9], 25 fps, 25 tbr, 90k tbn, 50 tbc
    Metadata:
      variant_bitrate : 0
Output #0, mp4, to 'file:Veer Hurts His Family Members!-1000225208.mp4.part':
  Metadata:
    encoder         : Lavf58.20.100
    Stream #0:0: Video: h264 (Main) (avc1 / 0x31637661), yuv420p(tv, bt709), 1280x720 [SAR 1:1 DAR 16:9], q=2-31, 25 fps, 25 tbr, 90k tbn, 90k tbc
    Metadata:
      variant_bitrate : 0
    Stream #0:1: Audio: aac (LC) (mp4a / 0x6134706D), 48000 Hz, stereo, fltp
    Metadata:
      variant_bitrate : 0
Stream mapping:
  Stream #0:1 -> #0:0 (copy)
  Stream #0:0 -> #0:1 (copy)
Press [q] to stop, [?] for help
[https @ 0x55e7bf495cc0] Opening 'https://hses.akamaized.net/videos/bharat/kbrs2sb/15/phone/media-6/segment-2.ts' for reading
frame=   99 fps= 16 q=-1.0 size=     256kB time=00:00:04.01 bitrate= 523.0kbits


High GPU usage w/ GPU plugin

I'm not sure if it's normal, but the GPU plugin uses 45-60% of my gpu. Whether I'm logged in or just on the title screen it seems to have high usage. Even if I turn the draw distance to 1 with anti aliasing off, the usage is still around 40%. Is this normal? Only asking because it makes my gpu begin to heat up a bit. Thanks for your help :)

OS: Windows 10
RuneLite version: 1.5.34
Launcher version: 2.0.2
RTX 2080, i7-9700k, Asus Z390-F

We have recently replaced the GL library used by the GPU plugin. Given that change and that we are not able to reproduce this issue, we are closing this. Please comment if you still experience this issue.


mat-tab-nav-bar loses selection when focus lost

Bug, feature request, or proposal:
The selected tab of a mat-tab-nav-bar is shown with an underline. When the focus is lost, the selected tab item is no longer shown as selected
What is the expected behavior?
Expected behavior would be the indicated selected tab stays selected
What is the current behavior?
When the focus is lost, the selected tab item is no longer shown as selected
Please advice.
Thank you!

@thehtcodes - I'm not able to reproduce this. Can you make a demo showing the issue?

Closing as this issue is out of date.

Hello
The problem (or as designed) is still exists.
When selecting tab (mat-tab in mat-tab-group), the tab is highlighted with animation and underline color is shown. If user click click outside (losing focus), the highlight is gone. Only underline color remains. It is shown in angular documentation : [https://material.angular.io/components/tabs/overview].
How can I override this behavior? Thank you in advance

I also have this issue as described by @mariannafld The text color of the active tab goes back to inactive when the tab loses focus.  It is seen in the material docs linked above by @mariannafld
This issue shouldn't have been closed.


Solve the bad character in the dex file name

This merge request is aim to fix the issue #3
strncpy() not put the 0 after the string keyword, put it by ourself.

Already fixed

Merge the patch.


[Question] Any chnace of getting this to work with avalonia

I am writing an Avalonia application, is there any chance of getting this to work with that in the future?

Hi, Avalonia is a different framework which means that the entire codebase must be rewritten (and possibly rethought) in order to make Nodify work with it.
I don't have the time, nor the energy to do this.

that's fair enough thanks for getting back to me :)

Thank you for your interest in Nodify.

As info, here a port of Nodify (with some changes though) for Avalonia : https://github.com/MakesYT/NodifyM.Avalonia

There is now a recommended Avalonia port that has all the functionality the WPF version has and tries to stay compatible with this code base.
https://github.com/BAndysc/nodify-avalonia


"Non-transitive" promotion rules with One and Zero.

The following calls have different results depending on the order of the arguments to promote_type:
import VectorInterface as VI
promote_type(VI.One, VI.Zero, Float64) # -> StackOverflowError
promote_type(VI.One, Float64, VI.Zero) # -> Float64
promote_type(Float64, VI.One, VI.Zero) # -> Float64

Due to some changes in internals, in 1.10 this instead results in:
promote_type(VI.One, VI.Zero, Float64) # -> Float64
promote_type(VI.One, Float64, VI.Zero) # -> Float64
promote_type(Float64, VI.One, VI.Zero) # -> StackOverflowError

Due to chance, the matmul code happen to call the first combination of this:
https://github.com/JuliaLang/julia/blob/58ed79361f4faad0da8d96aeaffbd8d75ebbb386/stdlib/LinearAlgebra/src/matmul.jl#L500
which means it works in 1.10 but not in 1.11 (here is the PkgEval error for an upstream package https://s3.amazonaws.com/julialang-reports/nanosoldier/pkgeval/by_hash/af49d9f_vs_08e1fc0/FiniteMPS.primary.log). In theory, the promote call in LinearAlgebra could be made to call the one with the number in the middle but it is a quite brittle situation.

Thanks for reporting this and my apologies for the delay. I think adding
Base.promote_rule(::Type{Zero}, ::Type{One}) = Bool

fixes this, so I will commit this fix asap.

This is fixed in the upcoming 0.4.6. Thanks again for the report.

:+1:


Update S3 tutorial and related docs

Description
While reading the S3 tutorial I noticed some inconsistencies that made me want to run it again.

the ingestion was crashing on t4g.medium (even after tweaking the commit_timeout_secs config), so increasing the suggested config to t4g.large.
section on time pruning was removed as this dataset does not present clear benefits from using it. To balance that removal, I added a mention to it in the REST API docs.

How was this PR tested?
Running the tutorial step by step on EC2 instances.

Note: I am not addressing https://github.com/quickwit-oss/quickwit/issues/3939 because there are a few missing things:

we should test whether it still works if we public dataset bucket is in a different regions as the user
the dataset used hdfs-logs-multitenants.json.gz is zipped which isn't supported by the file source. Note that hosting the uncompressed version might become a bit costly when users download it from outside AWS or another region (0.15 per download)

While running the bench again I noticed that the tutorial was failing. As the problem I noticed probably also applies to other docs, I opened an issue instead of addressing it here: https://github.com/quickwit-oss/quickwit/issues/3984


Update add_designer.py

@m4rc1e
In the spreadsheet, the "Name" column became "Designer Name" instead of "Name", we need to update it in the script.
I hope I changed the right line :)

Hi Emma, I changed the column name in the Spreadsheet back to "Name" for it to work okay with the add-designer tool, and it was working the last time I used it (last week).
If you are using a local .xlsx file you can change the column name there and it would work ok. Is there any specific purpose for the column to be named "Designer Name" in the spreadsheet?

Fair enough :)


feat: run mkdocs via Docker Engine API

Hey :wave:!
I wanted to create a draft PR to gauge interest in this approach from the Backstage devs before fixing all the tests I've broken.
My goal was to migrate from calling the Docker CLI to making calls against the Engine API, such that nothing changes for existing users, but techdocs-cli serve starts working for users that have a different engine implementation at the docker socket path, or have DOCKER_HOST set in their environment.
Is this something you'd be interested in merging, or is there a preference for a different implementation?

The system may not have the Docker CLI installed while still having access to an implementation of the Docker Engine API. For example, if the daemon is running in a VM with shared folders, or when another container runtime is being used.
This changeset modifies techdocs-cli to interact with Docker via the Engine API instead of the Docker CLI. The local Docker Engine UDS is used by default, but another location can be specified with the DOCKER_HOST environment variable.
Fixes: #12028
Hey, I just made a Pull Request!

:heavy_check_mark: Checklist

[x] A changeset describing the change and affected packages. (more info)
[x] Added or updated documentation
[ ] Tests for new functionality and regression tests for bug fixes
[ ] Screenshots attached (for UI changes)
[x] All your commits have a Signed-off-by line in the message. (more info)

I'm still waiting for feedback, any feedback.

Hi @terinjokes, I'll try and take a look at the end of the week. The TechDocs team is pretty small and also busy right now so that might be why they haven't given it a look yet 👍


Typo in the TcpListener Class code snippet

https://docs.microsoft.com/en-us/dotnet/api/system.net.sockets.tcplistener?view=netframework-4.8
In the code snippet of the example it says

The "user" should probably be replaced with "use"

Thanks for your feedback @Storm312. I've opened a couple of PRs to fix this issue.


apache-datasketches: init at 1.6.0

Description of changes
PostgreSQL extension providing approximate algorithms for distinct item counts, quantile estimation and frequent items detection.
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

You must add the missing hooks in each phases of your derivation.

This should be done too

Don't forget to squash the commits.

@GrahamcOfBorg build apache-datasketches
@GrahamcOfBorg test apache-datasketches

@GrahamcOfBorg build apache_datasketches
@GrahamcOfBorg test apache_datasketches

@GrahamcOfBorg build apache_datasketches
@GrahamcOfBorg test apache_datasketches

@drupol Thanks a lot for reviews and advice!

Thanks for your contribution and patience :)


Increase restart modal timeout to 5 seconds

Summary
This increases the timeout for the restart modal to 5 seconds.
Fixes #11087

Technical notes summary

While troubleshooting this issue, I noticed that we were receiving a 200 response from the server before the Rancher backend finished restarting. It must be that the backend is taking longer to shutdown and responding to our request when it does so.
I'm not a fan of increasing the timeout to an arbitrary number, but this is the most straightforward approach to resolve the issue at this point. The obvious issue with this approach is that we will see the same regression in the future if the backend takes any longer to shut down.
I also considered that we might be able to implement a function that initially polls at a greater duration and then incrementally steps down the timeout for each successive attempt. This would give us the flexibility to increase our timeout to something larger to give us more certainty that the backend has stopped before we first polled. I think this refactoring would be out of scope for resolving this issue.
Areas or cases that should be tested

See the description in #11087
Areas which could experience regressions

See the description in #11087
Checklist

[x] The PR is linked to an issue and the linked issue has a Milestone, or no issue is needed
[x] The PR has a Milestone
[x] The PR template has been filled out
[x] The PR has been self reviewed
[x] The PR has a reviewer assigned
[x] The PR has automated tests or clear instructions for manual tests and the linked issue has appropriate QA labels, or tests are not needed
[x] The PR has reviewed with UX and tested in light and dark mode, or there are no UX changes

It's a little bit more complicated, but we could wait for a non-200 before then waiting for a 200

That's not a bad idea, we already know that Rancher must go down before coming back up and it could resolve my issue with just increasing a timeout to resolve the issue. Thanks for the suggestion!

Hmm, it might not be that simple. A quick test makes it look like the timeout is waiting in the queue while the event loop gets flooded while polling the websocket.


Nether service fails on Samsung Artik (Tizen 3.0)

I have the following issue. I installed Tizen 3.0 on Samsung Artik 710. Nether service fails to load during the startup.
When I try to
systemctl start nether.service
I get:
Job for nether.service failed. See "systemctl status nether.service" and "journalctl -xe" for details.
The result of "systemctl status nether.service":
● nether.service - nether service
   Loaded: loaded (/usr/lib/systemd/system/nether.service; disabled; vendor preset: enabled)
   Active: failed (Result: start-limit) since Tue 2017-06-20 11:07:16 MSK; 2min 47s ago
  Process: 2729 ExecStartPost=/usr/sbin/ip6tables-restore /etc/nether/nether_ipv6.rules (code=exited, status=1/FAILURE)
  Process: 2728 ExecStart=/usr/bin/nether -l JOURNAL -P policy=/etc/nether/cynara.policy -B /etc/nether/file.policy -r /etc/nether/nether.rules (code=killed, signal=TERM)
 Main PID: 2728 (code=killed, signal=TERM)

Jun 20 11:07:16 artik systemd[1]: Failed to start nether service.
Jun 20 11:07:16 artik systemd[1]: Unit nether.service entered failed state.
Jun 20 11:07:16 artik systemd[1]: nether.service failed.
Jun 20 11:07:16 artik systemd[1]: nether.service holdoff time over, schedul...t.
Jun 20 11:07:16 artik systemd[1]: start request repeated too quickly for ne...ce
Jun 20 11:07:16 artik systemd[1]: Failed to start nether service.
Jun 20 11:07:16 artik systemd[1]: Unit nether.service entered failed state.
Jun 20 11:07:16 artik systemd[1]: nether.service failed.
Hint: Some lines were ellipsized, use -l to show in full.

And on journalctl -xe:
Jun 20 11:07:16 artik systemd[1]: Unit nether.service entered failed state.
Jun 20 11:07:16 artik systemd[1]: nether.service failed.
Jun 20 11:07:16 artik systemd[1]: nether.service holdoff time over, scheduling r
Jun 20 11:07:16 artik systemd[1]: Cannot add dependency job for unit resize2fs@d
Jun 20 11:07:16 artik systemd[1]: Cannot add dependency job for unit resize2fs@d
Jun 20 11:07:16 artik systemd[1]: Cannot add dependency job for unit resize2fs@d
Jun 20 11:07:16 artik systemd[1]: start request repeated too quickly for nether.
Jun 20 11:07:16 artik systemd[1]: Failed to start nether service.
-- Subject: Unit nether.service has failed
-- Defined-By: systemd
-- Support: http://lists.freedesktop.org/mailman/listinfo/systemd-devel
--
-- Unit nether.service has failed.
--

What can be the problem?

Probably, the problem is that your kernel doesn't support IPv6 (check if you have the /proc/net/if_inet6 file). The simple solution is to adjust the systemd configuration to get rid of applying  nether_ipv6.rules. You can also recompile the kernel with appropriate options turned on. The code on GitHub is a little bit outdated. You can find the recent version of Nether on https://review.tizen.org/git/?p=platform/core/security/nether.git;a=summary


temp working directory is not deleted

The temporary folder (crxXXXXX-XXXXX-XXXXXXX) is not being deleted after the process is finished.
Is it known?
Maybe it's only on my working station?
I'm using the crx module (3.0.3) inside my module on node v0.12.7

It is clearly a bug because the temp module should remove them after the process ended.

Temporary folder are not created anymore so you should not encounter the problem :-)


Why not netty for networking?

Hi!
Why the current implementation doesn't use netty?

@sasbury would have insight here..

Someone just ported to NIO, i am not sure that there is any performance benefit. As far as Netty goes I would say it is heavy dependency to add, but someone could build a version that uses that.

Netty is not only reactive but also has "native transport" implementation, I'm no expert but if I recall correctly it instead of using NIO has some JNI magic that handles networking more directly which results in lower latency. And I'd love to see it implemented as part of this project.

Netty is not only reactive but also has "native transport" implementation, I'm no expert but if I recall correctly it instead of using NIO has some JNI magic that handles networking more directly which results in lower latency. And I'd love to see it implemented as part of this project.

My concern is three fold. First, JNI brings new deployment issues to anyone using the library. Second, since this is a client library and not a server library many of the NIO type optimizations don't actually add much value. The library doesn't open lots of sockets, just the one to the server. Someone just did an NIO re-write, I am curious how it performs in comparison to old socket code. Netty itself brings a big dependency (with possible down stream dependencies).

My concern is three fold. First, JNI brings new deployment issues to anyone using the library. Second, since this is a client library and not a server library many of the NIO type optimizations don't actually add much value. The library doesn't open lots of sockets, just the one to the server. Someone just did an NIO re-write, I am curious how it performs in comparison to old socket code. Netty itself brings a big dependency (with possible down stream dependencies).

Closing, added will not fix label. sasbury's comments pretty well sums it up, so no need to elaborate.


mask-size

eg
mask-image: url('../images/external-link.svg');
mask-size: cover;

Covered by http://caniuse.com/#feat=css-masks?

Yup.


Identify an  IPv4 range without knowing its URI

Scenario/Intent
Want to find an ipV4_range by name. as the ipv4pools-range does not get a function get_by
When I look at the library the get function is defined as follow
"""
Gets an IPv4 range by ID or URI.
    Args:
        id_or_uri: Can be either the range ID or URI.

What is the range ID? and how can I find it?
The goal is to identify an IPv4 range without knowing its URI.
Environment Details

OneView SDK Version: 4.8
**OneView Appliance Version:**4.20
OneView Client API Version: 500
Python Version: 3.7
Platform: RedHat

@DungKHoang,
Refer the below module to get ranges.
https://github.com/HewlettPackard/python-hpOneView/blob/release/v4.8.0/hpOneView/resources/servers/id_pools_ranges.py


Add commit tag to the frontend and the API

Provide a clear and concise description of what you want to happen.
Add commit tags to the API and the frontend to easily know what is currently deployed.
For example the API could have an endpoint that would return:
{commit-hash: hash, commit-message: message}
And the frontend could have a similar data made accessible, maybe hidden in the "about page" or as a standalone page. Thoughts on this @laurentS @echaidemenos since we did something similar on Ovio?

Good idea, always comes in handy at some point when you need to debug things. Better do it before we need it :)


test comment

          test comment

Originally posted by @TylerLeonhardt in https://github.com/microsoft/vscode/pull/186633#discussion_r1246721523

Is it open right now for contribution?


ci(rust): Add release workflow for rust

This PR adds release workflow for rust which will do cargo publish while tagged.

[ ] Did you create an Apache Jira ticket?  (Request account here, not required for trivial changes)
[ ] If a ticket exists: Does your pull request title follow the pattern "THRIFT-NNNN: describe my issue"?
[x] Did you squash your changes to a single commit?  (not required, but preferred)
[x] Did you do your best to avoid breaking changes?  If one was needed, did you label the Jira ticket with "Breaking-Change"?
[ ] If your change does not involve any code, include [skip ci] anywhere in the commit message to free up build resources.

cc @Jens-G, Would you like to take a look, thanks!
To make this workflow work, we also need to set a new secret call CARGO_REGISTRY_TOKEN.

LGTM but I have not really too much expertise with Rust, so a second pair of eyes is more than appreciated.

LGTM but I have not really too much expertise with Rust, so a second pair of eyes is more than appreciated.

Hi, @Jens-G, it seems no one else has reviewed this yet. However, this workflow has been successfully used in many other projects. For example:

OpenDAL: https://github.com/apache/opendal/blob/main/.github/workflows/release_rust.yml
Iceberg-Rust: https://github.com/apache/iceberg-rust/blob/main/.github/workflows/publish.yml#L46

Would this be valuable for your review?

this CARGO_REGISTRY_TOKEN needs to be added to GitHub repo itself right? i as a committer would not be able to do that, it requires some admin role in the repo itself.

this CARGO_REGISTRY_TOKEN needs to be added to GitHub repo itself right? i as a committer would not be able to do that, it requires some admin role in the repo itself.

Yes. We will need one of our PMC members to contact with the INFRA team to get it setup.

We can take those tickets as examples:

https://issues.apache.org/jira/projects/INFRA/issues/INFRA-25132
https://issues.apache.org/jira/browse/INFRA-25951

https://issues.apache.org/jira/browse/INFRA-26083

issues.apache.org/jira/browse/INFRA-26083

Thank you, @Jens-G, for this!

INFRA has added the token.

INFRA has added the token.

Thanks!


Keyword "Get page source": Add selector for partial source code

I want to propose to add an optional selector to the keyword Get Page Source.
The current implementation only supports to get the complete page source which can be a huge amount of data.

Long use case description:
The idea for a selector came when I was trying to find a selector for Get Text keyword.
My current way is to F12 into the browser devtools, pick the element and then inspect where I could start to locate. As long as the selector is pure XPath/CSS, I develop the selector in the console wia $x() or $$().
The conveniance feature of Browserlib to concatenate Xpath and CSS with >>, as well as frame selectors (>>>) comes at the cost to built them via the Debug Console of VS code, because the browser console does not understand this syntax.
That requires to switch between VS Code and the Browser Devtools multiple times (or to work with tiled windows...)
It would relly help to do something like this:
Get Page Source  p.availability

  Durchschnittliche Erreichbarkeit des Dienstes in den letzten 7 Tagen:
  100,00 %

Get Text  p.availability span.value
100,00 %

Here, I used Get Page Source to get a HTML snippet as output right into the debug console of VS Code.
From there I can inspect the interesting HTML source code easily and go on to refine the selector - without switching back to the browser.

So are you looking what this Playwright API returns https://playwright.dev/docs/api/class-locator in innerHTML ?

So are you looking what this Playwright API returns https://playwright.dev/docs/api/class-locator in innerHTML ?

Yes, exactly.

And lastly, when you want to assert a certain content in the page source, you normally know where it should roughly be. No need to search the complete DOM or even worse, dump it in case of a failure.
This is where a selector also would help.

@allcontributors please add @simonmeggle for ideas

@simonmeggle
With Get Property you can just do that.
Get Property    <selector>    innerHTML

This just includes the content of an element.
Get Property    <selector>    outerHTML

This includes the element itself.
I think Get Page Source is really the wrong keyword for that.
It literally says in the name Page source. not element.
So maybe we could add a new keyword for innerHTML and outerHTML so that it is more obvious for users.
On the other hand, where do we stop?

At least we could document that one can already get the element html in this way. Most likely many people do not see it.


[Idea] Callback before insertion

https://github.com/yogiben/meteor-autoform-file/blob/master/lib/client/autoform-file.coffee#L67
It would be nice to have a callback before insert so be can configure metadata for instance. Useful if we want to associate an ownerId with a file for example.

Yes, I need this too!

+1

Implemented in fc3ca05344f5.


Android Basics: Forage app

page: link
typo in step 1:
private val viewModel: ForageableViewModel by activityViewModels {
   ForageableViewModelFactory(
       (activity?.application as BaseApplication).database.foragableDao()
   )
}

foragableDao should be forageableDao

Also, could not get this particular code to work.
Working implementation with Factory Pattern:
 private val viewModel: ForageableViewModel by lazy {
    ForageableViewModelFactory(
        (requireActivity().application as BaseApplication).database.forageableDao()
    ).create()
}


MC reset process, allow MC to stay running when MPF exits

Implement an MC reset process

reset process is implemented in 0.33+. only need to change mc?

probably not worth it for now


cicd: specify unittest directory

Description
cicd: specify unittest directory
Type of change

[ ] Bug fix
[ ] New feature
[ ] Document
[x] Test
[x] CI/CD
[x] Refactor

How has this been tested

[x] Unittest
[ ] Others:

Checklist:

[x] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Let's make the world better✨😋🐍🌎

Codecov Report

Merging #37 (9336a49) into main (c60b059) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #37   +/-   ##
=======================================
  Coverage   95.56%   95.57%
=======================================
  Files          21       21
  Lines         609      610    +1
  Branches       36       36
=======================================
+ Hits          582      583    +1
  Misses         18       18
  Partials        9        9

Impacted Files
Coverage Δ

pypj/task/githubactions.py
100.00% <100.00%> (ø)

Continue to review full report at Codecov.

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


[Compression] fix speedup randomize bool

Description
If tensor.dtype is torch.bool, the range of it should be [0, 2)
Test Options

[ ] fast test
[ ] full test - HPO
[ ] full test - NAS
[ ] full test - compression

Checklist

[ ] test case
[ ] doc

How to test

should the pr be merged into master before merged into v2.10?

should the pr be merged into master before merged into v2.10?

no, we should merge v2.10 back to master after release


Disable and redirect legacy app users alpha

https://www.notion.so/saladtech/Disable-and-redirect-legacy-app-users-ca96c55783cb4fc7b7b8951bf059e43c?pvs=4
https://www.figma.com/file/IV8bjY0Oe393t8c0EWizk8/Product-Designs?node-id=6212%3A56683&t=aV78t11KKPg5X13B-4

I don't see new updates screen. Could you please tell me how I can check it?


Mint NFT directly from URI

Add a feature to mint a NFT directly from a URI rather than having to provide the on-chain formatted JSON file.

Added in v0.3.1.


All hard-coded strings should be abstracted to a captions module

There are lots of captions which are hard-coded into the XSLT, dating back years (for example, the captions in the left column of the Metadata table). These should be abstracted into a central captions module.

Started work on this, and abstracted all the obvious captions to a central captions file. There will be more lurking in plain text nodes.

I've done everything I can find in XHTML output nodes, but there are another few dozen in the TEI processing stages that need to be done.

Completed first pass at rev 11532.

Did a bit in the PDF rendering in rev 11686. I believe that's clean now.

I believe this ticket is now complete. Closing.


add chat history support for rerank LLM

We changed rerank LLM to rerank Pipeline.
So, it will support chat history when you use adequate LLM instance.


Journal plugin does nothing

I've installed journal plugin and does nothing (neither by keyboard shortcut nor by menu).
Notes are encrypted.
OS: Linux Mint 20.3.

Is there any other plugin working as expected?
The following is my joplin version:
Joplin 2.8.8 (prod, win32)
Client ID: 57b929924c8644f6a5dc4a0c4e96051b
Sync Version: 3
Profile Version: 41
Keychain Supported: Yes
Revision: c2a6a13

Is there any other plugin working as expected?
No other plugins.

Joplin 2.8.8 (prod, linux)
ID del Cliente: fe763aedda734cef8d1b16c62fb9e753
Versión de la Sincronización: 3
Versión del Perfil: 41
Llavero Soportado: No
Revisión: c2a6a13
I'm syncronizing with OneDrive, encrypted mode.
When syncronizing I've observed /home/juliux/.config/joplin-desktop directory. It's created a file named as database.sqlite-journal, and is immediately deleted automatically.

It's solved now. Thanks!!!!!


fix: [M3-8112] - Reset errors in PlacementGroupDeleteModal

Description 📝
In the PlacementGroupDeleteModal, API error messages are persisting when encountering an error, closing the dialog and reopening it. This PR resets error messages on Close so the user the does see then when reopening the modal
Changes  🔄

Reset delete and unassign queries errors on modal close

Preview 📷

How to test 🧪
Prerequisites

ℹ test in ALPHA
have at least one placement group created

Verification steps

Navigate to http://localhost:3000/placement-groups/delete/{your placement group ID}
In your network tab, block network request for DELETE | https://api.dev.linode.com/v4/placement/groups/
Click on the "Delete" button
Confirm "An unexpected error occurred." error message in dialog
Close dialog
Reopen by clicking on "Delete" for the same PG in the table row
Confirm error is no longer in the modal (was without the fix)

As an Author I have considered 🤔
Check all that apply

[ ] 👀 Doing a self review
[ ] ❔ Our contribution guidelines
[ ] 🤏 Splitting feature into small PRs
[x] ➕ Adding a changeset
[ ] 🧪 Providing/Improving test coverage
[ ] 🔐 Removing all sensitive information from the code and PR description
[ ] 🚩 Using a feature flag to protect the release
[x] 👣 Providing comprehensive reproduction steps
[ ] 📑 Providing or updating our documentation
[ ] 🕛 Scheduling a pair reviewing session
[ ] 📱 Providing mobile support
[ ] ♿  Providing accessibility support

To test this PR, are we suppose to cause the API to fail intentionally? I was able to delete 2 Placement Groups without encountering the API error shown in the Preview. Also, should the unit test be updated for this scenario?
The code changes make sense and tests are passing, I'm just a little confused because I wasn't able to replicate the corresponding issue and don't recall running into it in the past.

@carrillo-erik did you read the reproduction steps?

In your network tab, block network request for DELETE | https://api.dev.linode.com/v4/placement/groups/

@carrillo-erik did you read the reproduction steps?

In your network tab, block network request for DELETE | https://api.dev.linode.com/v4/placement/groups/

@abailly-akamai Okay, got it. I was able to verify that blocking the request url caused the error notice to appear in the modal.  After closing the modal and clicking the delete button, the error notice did not appear until the delete click to action was activated again.


Fix README flag and expose orphan network peers

Readme example was using wrong flag
Network peers were not exposed properly

Signed-off-by: Flavio Crisciani<EMAIL_ADDRESS>

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@2bf6330). Click here to learn what that means.
The diff coverage is 0%.

@@            Coverage Diff            @@
##             master    #2125   +/-   ##
=========================================
  Coverage          ?   40.32%
=========================================
  Files             ?      139
  Lines             ?    22436
  Branches          ?        0
=========================================
  Hits              ?     9048
  Misses            ?    12059
  Partials          ?     1329

Impacted Files
Coverage Δ

networkdb/networkdb.go
67.8% <0%> (ø)

networkdb/networkdbdiagnostic.go
0% <0%> (ø)

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 2bf6330...d8f432b. Read the comment docs.


feat: disable telemetry with env var from v2

restore functionality from v2. I've not actually built this yet and just made the edits on Github, sorry.
I think this should be added to the documentation, but not sure if it should be added to the CLI description of telemetry?

Thanks.
Can we test this?
Also, lets open an issue for this.

by test, do you mean write tests or just make sure it builds etc.?
I've updated to include DO_NOT_TRACK and explicitly check the variables = 1.

Yeah can we also open an issue for this?

Oh I see it.


Adds the Typelevel Unconference - Lambda World [2017]

Add the Typelevel Unconference that is going to be co-located with LW.
Could you check it @larsrh, thank so much!!

Thanks, Jorge!


Improve lint driver entrypoint

This makes the bevy_lint_driver binary behave a bit more like clippy-driver, which means it also behaves a bit more like rustc. Some changes include:

Panicking will now result in an ICE, which can be used to debug issues.
Logging is now setup, so you can set the RUSTC_LOG variable to configure logging levels and scope. (It uses tracing internally!)
Exit code logic now mirrors the functionality recommended for rustc drivers: specifically using catch_with_exit_code().

I'm currently debugging an issue with this where serde_derive fails to compile (I believe it's the rustc_driver::args::raw_args() call), so I'll update this when I fix it. :)

although I don't really have any experience with the rustc internals :)

That's fine! I don't expect you to know rustc internals, I just want a second set of eyes for testing and code quality reasons. :)


New Performance report to measure sniff run time performance

Description
Recreation of upstream PR https://github.com/squizlabs/PHP_CodeSniffer/pull/3810:

This PR depends on PR squizlabs/PHP_CodeSniffer#3804 to allow for automated testing of the new report.
Either PR squizlabs/PHP_CodeSniffer#3804 should be merged first and this PR rebased to remove that commit after the merge or the EndToEnd test included in this PR should be removed from the last commit and (after this PR has been merged), the test can be added to PR squizlabs/PHP_CodeSniffer#3804.
☝🏻 The above is the only reason to open this PR in draft. The actual change in the PR is ready for review.

Util\Timing: split printRunTime() method
... into logical parts to allow the Performance report access to the total run time information (at the point of report creation) as well.
File: add getListenerTimes() method
... to allow for access to the recorded listener times from within a report class.
Config: add trackTime toggle for whether or not to track listener times
As recording the time taken by each sniff has a performance impact in and of itself on a CS run, only record the time taken by each sniff when needed. [*]
Originally, this was already done conditionally based on the PHP_CODESNIFFER_VERBOSITY > 2 condition.
However, adding the Performance report would add a second criteria.
This commit adds a new (internal) Config setting trackTime, which will be set to true when PHP_CODESNIFFER_VERBOSITY > 2.
This commit paves the way for adding the second criteria in the next commit.

[*] I've done some unscientific benchmarks for this by running PHPCS multiple times, with and without tracking the listener times, over a 300+ file codebase.
Without tracking listener times, the run time was always around 39 seconds with 56Mb memory use.
With tracking listener times, the run time was always around 54 seconds with 64Mb memory use.
This, to me, shows a significant enough difference and sufficient reason to put this toggle in place to only track time when needed.

New Performance report to measure sniff run time performance
The report will print the sniff name, cumulative listener run time and % of the total sniff run time for each sniff triggered during a run.
The report is ordered by cumulative listener run time in descending order.
Additionally, it will highlight sniffs which have a cumulative listener run time more than twice the average run time per sniff in orange and sniffs with a cumulative listener run time of more than three times the average run time per sniff in red.
At the bottom of the report it will also compare the total sniff relative run time with the total run time.
Fixes squizlabs/PHP_CodeSniffer#3784
Includes end-to-end test (depends on PR squizlabs/PHP_CodeSniffer#3804).
Includes mention of the report in the CLI help text.
The report output is as shown in the screenshots in https://github.com/squizlabs/PHP_CodeSniffer/issues/3784#issuecomment-1520203761

More information from the original PR:

As per my comment in PR squizlabs/PHP_CodeSniffer#3804, I've removed the End2End test for the report from this PR to unblock it and am marking the PR as ready for review

Suggested changelog entry

New Performance report, mostly useful for sniff maintainers, to allow for finding "slow" sniffs.

To run this report, add --report=Performance for a normal PHPCS run on the command line.

Related issues/external references
Fixes #28

🎗  Reminder: when this PR gets merged, the wiki will need to be updated, both the Usage page as well as the Reports page!

The wiki has been updated to include information about the new report.


Add matomo for in-house analytics

closed with https://github.com/lgactna/ncc-web/commit/5436b4b6b14242930e86c3ed3438cc7a3ab65ddb, target domain is ncc-web.tech


DARK MODE

If you want to add dark mode to the site, please dm me on Slack or Discord, and we can make a design together, and you can code dark mode in using the dark: utility. This should be too hard, but I don't want to keep requesting changes without a design so let me know.

I'm down to do this if you want.

Hi, can you assign this issue to me? Would love to work on this too 👉👈


@feathersjs/commons dependency

Steps to reproduce
install feathers-hooks-common and @feathersjs/feathers in client app.
Expected behavior
@feathersjs/commons 4.0.0 should get installed
Actual behavior
feathers-hooks-common has a dependency of @feathersjs/commons 1.4.0 but @feathersjs/feathers has a dependency of @feathersjs/commons 4.0.0. npm installs @feathersjs/commons 1.4.0 instead of 4.0.0.
System configuration
Tell us about the applicable parts of your setup.
Module versions (especially the part that's not working):
NodeJS version: 9.11.1
Operating System: Windows 10
Browser Version: Chrome WebView 71.x

You have to run feathers-hooks-common thru WebPack if you want to use it client-side.


⚠ ECDC Shop has degraded performance

In 4c402e4, ECDC Shop (https://shop.memmingen-indians.de) experienced degraded performance:

HTTP code: 200
Response time: 11892 ms

Resolved: ECDC Shop performance has improved in 3d32747 after 36 minutes.


Error! Unable to update, Error code:900001

Hi, When I use fw to update fw.pkg, it reports the above error code as the title shows.

it's supposed to be like that

ok. when i again use fw to update 1.4 version, it also reports this error and directly entering the previously opened  doc. i checked the memu, no app menu exist. is it means it fails

I need you to be precise. If you just following the root guide, it should
not give you anything in the menu.
To check if root is successful, 1) reboot and see if there are three bars
showing up during the boot, and 2) check if connect to pc, adb can
recognize your device in normal boot up.
To get the menu hack, you need to follow the app launcher hack wiki.
On Thu, Jul 25, 2019, 00:11 yogi08<EMAIL_ADDRESS>wrote:

ok. when i again use fw to update 1.4 version, it also reports this error
and directly entering the previously opened doc. i checked the memu, no app
menu exist. is it means it fails
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/HappyZ/dpt-tools/issues/121?email_source=notifications&email_token=ABDVEKXHHYUJ6ULMKKH7DF3QBEYW5A5CNFSM4IGQLAIKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD2YLWAY#issuecomment-514898691,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABDVEKRP3BO3SWS6AFB6Z6TQBEYW5ANCNFSM4IGQLAIA
.

thanks it works


replace 'problems' key with 'exercises' in config.json

see #7

I forgot to point out that as of exercism/x-api#137, it is safe to remove problems.

:tada: thanks :)


Templates

Agency one-page HTML/CSS theme built with Bootstrap 5
Fabio said they typically use:

https://www.evl.uic.edu/shadows/
https://www.evl.uic.edu/vis-chicago/

AVA used this:

https://accessibility-cv.github.io/
https://github.com/accessibility-cv/accessibility-cv.github.io

We went with Agency one-page HTML/CSS theme built with Bootstrap 5


Searching with queries that returns a lot of books

Hello guys, there is an issue where searching for "ste" then continuing to "stefan".
The query for retrieving the books for "ste" is much longer than the time it takes to return the books by "stefan" and results in the books from "ste" overriding the books from "stefan" in the search field.
This is because the function that searches for and returns the books is asynchronous and allows for both of the search "ste" and "stefan" to run simultaneously. This means that the longer query will the be one that will be shown in the end.
Please fix.

Is there any way to cancel queries? If so when typing a word, you'd cancel all current coroutines before issuing a new query.

@Danryg came with a fix which sort of solves the problem but it will not abort any queries. See 4cfd7fd26e33365b6f111a83b28549be3376deb6.
So the underlying problem is still there but the user will not notice anything and the search works as intended for the user.


Doc: Adjust language about metadata order

Ref: https://github.com/dotnet/dotnet-api-docs/issues/7467

@David-Engel Will this change make its way into the dotnet-api-docs repo somehow?

@gewarren We run the documents publication process with each GA release into the sqlclient-api-docs repo.

@gewarren We run the documents publication process with each GA release into the sqlclient-api-docs repo.

Okay thanks, so sounds like this needs to be updated manually then: https://github.com/dotnet/dotnet-api-docs/blob/b54f4425aef3adb6a3b5d871240931b5ec822900/xml/System.Data.SqlClient/SqlDataReader.xml#L1878


Puppeteer error in docker

[0] [info] [2023-10-31 11-10:35] [scrape:all]:  runAll
[0]     Error: Failed to launch the browser process! undefined
[0]     [34:50:1031/114056.228042:ERROR:bus.cc(407)] Failed to connect to the bus:
[0]      Failed to connect to socket /run/dbus/system_bus_socket: No such file or
[0]     directory
[0]     [34:34:1031/114057.464120:ERROR:ozone_platform_x11.cc(239)] Missing X
[0]     server or $DISPLAY
[0]     [34:34:1031/114057.464132:ERROR:env.cc(255)] The platform failed to
[0]     initialize.  Exiting.
[0]
[0]
[0]     TROUBLESHOOTING: https://pptr.dev/troubleshooting
[0]
[0] docudigger scrape all exited with code 1

:tada: This issue has been resolved in version 2.0.1-dev.3 :tada:
The release is available on:

npm package (@dev dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 2.0.3-dev.1 :tada:
The release is available on:

npm package (@dev dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This issue has been resolved in version 2.0.3-dev.5 :tada:
The release is available on:

npm package (@dev dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


FPS compared to official python implementation

Hi ! @viplix3 ,
Thank you for your superb work here.
Wile doing your performance analysis compared to python code, have you noticed some performance improvement (in fps) ?
C++ should be faster here, right ?

Hi @arnaud-nt2i
Yip, the C++ code was significantly faster than the Python version I was using as a reference, especially on the Jetson NX.
I don't remember the exact performance gains though.
The performance I was able to achieve using the current implementation is detailed here
This is the reference Python implementation I was using to compare both the performance and accuracy.
It should be noted that the reference Python implementation has evolved quite a lot by now thanks to the open-source nature of the project.
If C++ is not a restriction for you I'd recommend checking out the above repo as that has ReID and multiple tracker support.
I have plans of adding ReID support but unfortunately, I haven't been able to find time for this project in the past few months.
Additionally, if you could compare the performance results of the reference implementation I shared, please feel free to raise a PR with relevant experiments!

ok thank for quick answer.
Another fast question:
Do you plan to run Camera Motion compensation in parallel of Detection for further improvements ?
https://github.com/NirAharon/BoT-SORT/issues/37#issuecomment-1278909530

Ideally in a perception pipeline, yes, object detection would be running in one thread, and camera motion compensation would be executed parallelly in another thread as they are independent of each other.
The tracker would get the results of both object detection and camera motion compensation for processing.
This would significantly reduce the computation time.
This perfectly aligns with the BoT-SORT author's comments on the issue link you've provided.
As this is a standalone application, executing CMC in a parallel thread doesn't provide any benefit as the tracker needs those results to process further.
So even if I spawn a new thread for CMC, the tracker would have to wait for CMC to complete its execution to utilize the output.
I hope that clears things up.

ok thank you very much!

I have plans to improve this project further so please feel free to report any issues you observe.
Closing this for now!


sort files in alphabetic order

It is great script and would like to use it but it doesn't sort files alphabetically so instead of organized playlist I've got random list of files.
It would be great to have little enhancement :-)
Thank you so much in advance!

Hmm, that's strange... on my machine, it sorts alphanumerically. I keep my music organized in folders by artist, and then album, with track names such as "01 Track 1.mp3". When I run this script on an artist folder, I end up with something like:

Artist/Album 1/01 Track 1.mp3
Artist/Album 1/02 Track 2.mp3
Artist/Album 1/...
Artist/Album 2/01 Track 1.mp3
Artist/Album 2/02 Track 2.mp3
Artist/Album 2/...

This doesn't happen for you?

I have the same structure in my folders and most of the files started as track number and then track name.
I have following output:
13-Van Morrison-Madame George.mp3
03-Van Morrison-Astral Weeks.mp3
11-Van Morrison-Ballerina.mp3
05-Van Morrison-Cyprus Avenue.mp3
09-Van Morrison-Slim Slow Slider.mp3
07-Van Morrison-The Way Young Lovers Do.mp3
10--Комментарии БГ.mp3
06--Комментарии БГ.mp3
04--Комментарии БГ.mp3
12--Заключение.mp3
08--Комментарии БГ.mp3
14--Заставка.mp3
01--Заставка.mp3
02--Вступление.mp3

Hmm, I've made a small tweak to the script that I think might solve your problem. You can install the latest version by running the following in a Terminal window:
cd ~/Library/Services/Create-Playlist.workflow
git pull

yeahh...
That works great!
I was trying to fix this error with "| sort" but I have no experience with bats script so it was no success.
Your solution is simple and works great!!!!

Thank you very very much :-))))

Ya, sort was the next thing I would have tried if that didn't work =) Glad to help!


add relation_map property to only calculate one time the relation_map

#337

it will fail if something has a different relation_map


Rearrange ush with subdirs

Rearranging the ush directory would make it more clear which script to run (master_metplus.py) and the organization would be moving in the directory of eventually components of the METplus wrappers into a package that could be installed by a user.
-\ ush
- master_metplus.py (only script in top level so it is easy to tell what to run)
-\ metplus (or should this be outside of ush so it can eventually be used as a package?)
-\ wrappers
|  init.py (used to determine what to import when import metplus.wrappers is called)
| command_builder.py
| point_stat_wrapper.py
| pcp_combine_wrapper.py
...
| tc_pairs_wrapper.py
      -\ util
        | time
        | config
        | met_util (should be broken up into time/config/io/etc.)
        | feature_util
        | grid_to_obs_util

Wrappers and utils would be referenced in the code like this:

In Python3 you can create namespace packages that can contain files in various different places. This could be useful for making METplus into a package since the files will be under ush and not under the top level directory. More info here:
https://www.python.org/dev/peps/pep-0420/


Any plans to address these?
warning node-telegram-bot-api ><EMAIL_ADDRESS>request has been deprecated, see https://github.com/request/request/issues/3142
warning node-telegram-bot-api ><EMAIL_ADDRESS>request-promise has been deprecated because it extends the now deprecated request package, see https://github.com/request/request/issues/3142
warning node-telegram-bot-api > request ><EMAIL_ADDRESS>this library is no longer supported
warning node-telegram-bot-api > request ><EMAIL_ADDRESS>Please upgrade  to version 7 or higher.  Older versions may use Math.random() in certain circumstances, which is known to be problematic.  See https://v8.dev/blog/math-random for details.

Hi this is not a error or problem at all. You can use normal the library without any problem.
But yes, we have plans to replace this package.


Minor fixes and style changes.

I also added the camper bot in challenge 1.

These look good to me.


datalad/git-annex: Install commands that fail with WinError 740 are retried with elevation

Closes #82.

note: lots (but not all) of CI errors, didn't look inside

oh, tests are failing in other PR #83 as well and seems to boil down to datalad not being yet available for 3.9 on conda forge:
UnsatisfiableError: The following specifications were found

to be incompatible with the existing python installation in your environment:
Specifications:

  - datalad -> python[version='>=3.6,<3.7.0a0|>=3.7,<3.8.0a0|>=3.8,<3.9.0a0']

which sucks... pushed some commit to https://github.com/conda-forge/datalad-feedstock/pull/61 so we could may be get a bit closer to having 3.9 supported after that build/PR merged (even if not on windows)

meanwhile lets merge/release and hope that no other gotchas slipped through in CIs


PM and server window not maximized on iOS

PM's and server window isn't maximized, as shown below
Apple iPhone 8 Plus, running iOS 11.2.1. Same issue with Apple Safari and Opera Mini 16.0.6. Using TheLounge 2.7.0-pre.3. Same issue was found in 2.4.0.

Issue is found in a theme. Sorry for the issue.


WFLY-14462 exclude-local-receiver and local-receiver-pass-by-value no…

…t in ejb3 subsystem schema
JIRA: https://issues.redhat.com/browse/WFLY-14462

@chengfang I don't think it needs to block this PR, but I don't see these attributes tested in the ejb3 subsystem tests

These 2 attributes are tested in https://github.com/wildfly/wildfly/blob/master/ejb3/src/test/java/org/jboss/as/ejb3/subsystem/Ejb3SubsystemUnitTestCase.java#L253, which was recently added. Is there anything else needed?

@chengfang thanks - I was lazy and just checked the xml


rm: bug with trailing backslash on windows

I have a bad habit of providing options at the end of the command:
rm .\some-directory\ -rf

and since I use Tab to autocomplete	the path in powershell, it will add trailing backslash, and will interpret the -rf as part of the path.
rm: cannot remove '.\some-directory\ -rf': No such file or directory

It works fine if I removed the trailing backslash.

tried to track this down a bit, and it looks like it is an issue with using clap::App::get_matches_from with wild::args_os(). using clap::App::get_matches() parses the args correctly.

Opened an upstream issue https://gitlab.com/kornelski/wild/-/issues/7

@amrbashir  Thank you! Let's hope it gets fixed!


New ws extension spi

See specification Server-side WebSocket Extension SPI  for details. Here's a summary:

New module transport/ws/extensions contains the new SPI.
WebSocketExtensionFactory loads available extensions using service loader and also contains the method to negotiate extensions
Most classes from the previous framework have been removed or repurposed, notably:

ActiveWsExtensions is removed, replaced by the List returned by WebSocketExtensionFactory.negotiate).
WsExtension, WsExtensionBuilder and WsExtensionParameter have been renamed to ExtensionHeader, ExtensionHeaderBuilder and ExtensionParameter and are only used for extension header parsing and construction.

GatewayContextResolver is enhanced to create and inject the WebSocketExtensionFactory instance as a resource, and inject resources into it
WebSocketExtensionFactory injects resources into the individual WebSocketExtensionFactorySpi instances
Robot tests ExtensionFilterIT (for wsn) and WsebExtensionFilterIT (for wseb) show a test extension being loaded dynamically and modifying traffic in both directions.

Coverage remained the same at 46.63% when pulling a6137d76696cb5714824dfaf5a8849f88022173a on cmebarrow:new_ws_extension_api into ab2da6ad42c8ed7b2e95ec219badc2aaaa841c25 on kaazing:develop.

Coverage remained the same at 46.64% when pulling 3e54f68e397bff0c7b91d12c06446cc3b256edda on cmebarrow:new_ws_extension_api into ab2da6ad42c8ed7b2e95ec219badc2aaaa841c25 on kaazing:develop.


[Flutter SDK] Show current speech from user

I have seen in some of your demos that you are able to display the words being spoken on other platforms. This is something I'd like to add to my Flutter app.
I plan to start looking into how to accomplish this but wanted an issue to track if anyone else had ideas, etc.
After further investigation, I have no idea how to accomplish this. I would love to help if someone is willing to guide me.

@AlexHartford, the Picovoice SDK does not transcribe speech and thus there is no way to show the words being spoken. The Rhino Speech-to-Intent engine performs recognition solely in the acoustic domain. To achieve the effect you are looking for, you would have to use a Speech-to-Text engine.

Okay, thanks!


fix: accurately determine unix time string

Avoid content like abc123456789abc from being parsed as a timestamp, resulting in nothing being parsed.
I'm not sure why spaces were allowed before, so I won't handle spaces in this scenario.

Thanks!


Using row ID values in rows is inconsistent when the table has row groups

When the table has row groups, rows tend to lose their original ordering due to migration of rows to the different groups. This makes using a row index value in a rows argument less recommended (you have to know the pre-shuffled index). Using the row ID is the recommended way, however, this fails and it seems that the row ID maps to the wrong row. Here is an example of that using tab_style():
from great_tables import GT, md, style, loc
from great_tables.data import towny

towny_mini = towny[["name", "website", "land_area_km2", "csd_type"]].tail(4)

(
    GT(towny_mini, rowname_col="name", groupname_col="csd_type")
    .tab_style(
        style=style.fill(color="Beige"),
        locations=loc.body(columns="land_area_km2", rows=["Woodstock"]),
    )
)

From pairing, it sounds like the row ID should always be the original position (as passed into GT)

This was partially fixed by @machow  in https://github.com/posit-dev/great-tables/pull/202. However, I'm seeing that the Pandas table behaves incorrectly compared to a Polars one. What follows is a pair of examples that involves a GT table having a stub and row groups, and, some basic styling.
Let's take a portion of the towny dataset as a Pandas DataFrame. From the output we see that 'Woodstock' is the sole city under the csd_type column:
from great_tables import GT, md, style, loc
from great_tables.data import towny
import polars as pl

towny_mini = towny[["name", "website", "land_area_km2", "csd_type"]].tail(4)

towny_mini

Let's write the GT statement for this Pandas DF; notice that 'Woolwich' is in the 'city' group.
# With Pandas -- incorrectly classifies Woolwich as a city; styling the 'Woodstock' row is also incorrect
(
    GT(towny_mini, rowname_col="name", groupname_col="csd_type")
    .tab_style(
        style=style.fill(color="Beige"),
        locations=loc.body(columns="land_area_km2", rows="Woodstock"),
    )
)

With a Polars DF, everything is correct:
# With Polars -- correctly uses Woodstock in the city grouping and
#   styling is fine using the `rows=` reference
towny_mini_pl = pl.from_pandas(towny_mini)

(
    GT(towny_mini_pl, rowname_col="name", groupname_col="csd_type")
    .tab_style(
        style=style.fill(color="Beige"),
        locations=loc.body(columns="land_area_km2", rows="Woodstock"),
    )
)


interaction with thirdweb

it has transaction failing issues when trying to use thirdweb smartContract
would be awesome if u right down on readme file how to make  it to work.

I think the error is related to the minting cost or price. ThirdWeb uses Ether while Hashlips minting DApp uses Wei (smallest denomination of Ether).
Let's say you want to sell a Polygon NFT.
In ThirdWeb, the currency is 1 (1 MATIC).
In Hashlips minting DApp, the cost is<PHONE_NUMBER>000000000<PHONE_NUMBER>000000000 WEI or 1 MATIC)

Sadly it's not about that

Sth strange :D

Anyway thanks for your attention

I think the error is related to the minting cost or price. ThirdWeb uses Ether while Hashlips minting DApp uses Wei (smallest denomination of Ether).
Let's say you want to sell a Polygon NFT.
In ThirdWeb, the currency is 1 (1 MATIC).
In Hashlips minting DApp, the cost is<PHONE_NUMBER>000000000<PHONE_NUMBER>000000000 WEI or 1 MATIC)

u can preview repo: testzcp


Fix release build of sbt-precog

Release build failed with:
[warn] /home/runner/work/sbt-precog/sbt-precog/version.sbt isn't formatted properly!
Error:  1 files must be formatted
Error:  (Compile / scalafmtSbtCheck) 1 files must be formatted

This is because in sdmerge the generated version.sbt isn't formatted correctly, which causes scalafmtSbtCheck that runs as part of the ci target to fail.

<0>


Specifying aliases in OWNERs files

--alias-file option can specify team aliases of the following kind:
aliases:
  team/t1:
    - team/t2
    - u2
  team/t2:
    - team/t3
    - u3
  team/t3:
    - u4

This should let us use OWNERs files better with an extra level of indirection.
Things that need further thought:

Do we want to support nested team definitions like above? (If yes, what do we do if there are cyclic loops?)
Should the yaml file containing aliases be within the repository, or be globally consistent and supplied to mungegithub at the time of invocation?

This change is

@pwittrock PTAL and let me know if this is what we had in mind earlier.

so these are distinct from the github teams...    i'm just thinking out loud trying to decide if I like that...

@eparis An advantage over github teams would be the audit-trail it leaves behind when there are modifications (in the form of PRs). I think could be valuable.

@eparis There are definitely different groups backends we could use for membership - GH teams, Google Groups, etc.  I am open to discussing the trade offs of each, but don't think this is a decision we need to commit to right now.  I asked @foxish to see if he could put together a Proof Of Concept in 1/2 a day using a file-based backend so that we can provide some notion of a group as an abstraction layer to the OWNERS files efforts.  If we do this right, it should be relatively simple to swap out one backend for another without creating a bunch of extra work.

Do we want to support nested team definitions like above? (If yes, what do we do if there are cyclic loops?)

No.  Lets maximize our ROI and do the the minimum required for our goals right now.

Should the yaml file containing aliases be within the repository, or be globally consistent and supplied to mungegithub at the time of invocation?

Good question.  Lets start with the simplest thing (a file in the repo) and add support for something globally when need.  We may ultimately decide to use GH teams or some other backend for the memberships.

@pwittrock PTAL. The code was generic enough that I moved it to a feature that can be used by multiple mungers. I've addressed the issues cited in your comments as well.

Reviewed 1 of 6 files at r1.
Review status: all files reviewed at latest revision, 4 unresolved discussions.

Comments from Reviewable

Review status: all files reviewed at latest revision, 6 unresolved discussions.

mungegithub/features/aliases.go, line 51 [r2] (raw file):
Previously, pwittrock (Phillip Wittrock) wrote…
> would you add a `var _ Feature = &Aliases{}` line to indicate that this implements the Feature interface (or whatever interface it implements)?

Done.

mungegithub/features/aliases.go, line 86 [r2] (raw file):
Previously, pwittrock (Phillip Wittrock) wrote…
> Is this guaranteed to be called serially, or do we need to protect a.data against concurrent writes?

It is guaranteed to be called serially.

mungegithub/features/aliases_test.go, line 44 [r1] (raw file):
Previously, foxish (Anirudh Ramanathan) wrote…
> Removed.

Done.

mungegithub/mungers/blunderbuss.go, line 53 [r1] (raw file):
Previously, pwittrock (Phillip Wittrock) wrote…
> I'd encapsulate the alias related functionality into its own object for maintainability.

Done.

mungegithub/mungers/blunderbuss.go, line 54 [r1] (raw file):
Previously, pwittrock (Phillip Wittrock) wrote…
> Make this an interface instead of a generic function.  It helps with maintainability.

Done.

mungegithub/mungers/blunderbuss.go, line 93 [r1] (raw file):
Previously, pwittrock (Phillip Wittrock) wrote…
> Better to define a golang struct and parse it into that so we get compile time checks.

Done.

mungegithub/mungers/blunderbuss.go, line 179 [r1] (raw file):
Previously, pwittrock (Phillip Wittrock) wrote…
> Move this into `b.expandAliases`

Done.

Comments from Reviewable

Reviewed 1 of 1 files at r4.
Review status: all files reviewed at latest revision, all discussions resolved.

Comments from Reviewable

+lgtm

Review status: all files reviewed at latest revision, all discussions resolved, some commit checks pending.

Comments from Reviewable

Review status: all files reviewed at latest revision, all discussions resolved, some commit checks pending.

Comments from Reviewable