[FEATURE] Improve Event Service Efficiency by Removing Direct Calls to Friends Service in Get Event Handler

Description:
Currently, the Event Service is making direct calls to the Friends Service in the GetEventHandler whenever an event is fetched. This process introduces unnecessary latency and potential points of failure, as the entire event retrieval process is dependent on the availability and response time of the Friends Service. This inefficiency affects the performance of the Event Service, especially when handling requests at scale.
Problem:
When the Event Service queries an event, it calls the Friends Service to retrieve the friends data of the user requesting the event. This dependency introduces the following issues:

Latency: The response time of the GetEvent query is extended by the need to call the Friends Service.
Reliability: If the Friends Service is unavailable or experiences high latency, it impacts the Event Service's ability to return the event data in a timely manner.
Unnecessary Overhead: The Friends Service is called for every GetEvent request, even when the friends data is not critical for all use cases.

Proposed Solution:
Refactor the Event Service to decouple the Friends Service calls from the GetEvent query:

Remove Direct Calls: The GetEventHandler should no longer call the Friends Service when retrieving events.
Asynchronous Processing: If friend-related data is necessary (e.g., for showing friends who are attending or interested), this data should be fetched asynchronously and separately from the main event details. This can be handled either by a separate request or during the background processing.
Optional Data: The friends' data (e.g., friends who are signed up or interested in the event) can be retrieved and displayed in a separate request or cached for efficiency, without blocking the main event retrieval.
Improve Efficiency: The Event Service should focus on efficiently retrieving event data without depending on external services during the GetEvent query.

Alternatives Considered:

Caching Friends Data: One alternative was to cache the friends data, but this would add additional complexity in managing cache expiration and synchronization with the actual Friends Service.
Batch Requests: Another alternative was to batch requests to the Friends Service, but this still introduces unnecessary dependencies and may not fully resolve the performance issue.

Additional Context:

Events are often queried in high volume, especially in scenarios involving user feeds, where the performance impact is most visible.
A decoupled solution will allow for more modular services, improving overall system resilience and performance.

This issue will also be aligned with the Paralax development.
We will have to eliminate possible null refferencial exceptions.
o": null,
  "Pageable": null
}
[20:54:02 INF] Executed endpoint 'HTTP: GET events/search'
[20:54:02 ERR] An error occurred while processing the request.
MongoDB.Driver.Linq.ExpressionNotSupportedException: Expression not supported: Convert(e.Organizer.OrganizationId, Guid).
   at MongoDB.Driver.Linq.Linq3Implementation.Translators.ExpressionToFilterTranslators.ToFilterFieldTranslators.ConvertExpressionToFilterFieldTranslator.Translate(TranslationContext context, UnaryExpression expression)
   at MongoDB.Driver.Linq.Linq3Implementation.Translators.ExpressionToFilterTranslators.ToFilterFieldTranslators.ExpressionToFilterFieldTranslator.Translate(TranslationContext context, Expression expression)
   at MongoDB.Driver.Linq.Linq3Implementation.LinqProviderAdapterV3.TranslateExpressionToField[TDocument,TField](Expression`1 expression, IBsonSerializer`1 documentSerializer, IBsonSerializerRegistry serializerRegistry, Boolean allowScalarValueForArrayField)
   at MongoDB.Driver.ExpressionFieldDefinition`2.Render(RenderArgs`1 args)
   at MongoDB.Driver.SingleItemAsArrayOperatorFilterDefinition`2.Render(RenderArgs`1 args)
   at MongoDB.Driver.AndFilterDefinition`1.Render(RenderArgs`1 args)
   at MongoDB.Driver.AndFilterDefinition`1.Render(RenderArgs`1 args)
   at MongoDB.Driver.PipelineStageDefinitionBuilder.<>c__DisplayClass35_0`1.<Match>b__0(RenderArgs`1 args)
   at MongoDB.Driver.DelegatedPipelineStageDefinition`2.Render(RenderArgs`1 args)
   at MongoDB.Driver.AppendedStagePipelineDefinition`3.Render(RenderArgs`1 args)
   at MongoDB.Driver.AppendedStagePipelineDefinition`3.Render(RenderArgs`1 args)
   at MongoDB.Driver.MongoCollectionImpl`1.AggregateAsync[TResult](IClientSessionHandle session, PipelineDefinition`2 pipeline, AggregateOptions options, CancellationToken cancellationToken)
   at MongoDB.Driver.MongoCollectionImpl`1.UsingImplicitSessionAsync[TResult](Func`2 funcAsync, CancellationToken cancellationToken)
   at MongoDB.Driver.IAsyncCursorSourceExtensions.ToListAsync[TDocument](IAsyncCursorSource`1 source, CancellationToken cancellationToken)
   at MiniSpace.Services.Events.Infrastructure.Mongo.Repositories.Extensions.AggregateByPage[TDocument](IMongoCollection`1 collection, FilterDefinition`1 filterDefinition, SortDefinition`1 sortDefinition, Int32 page, Int32 pageSize) in /home/kaliuser/Documents/portfolio/commercial_apps/distributed_minispace/MiniSpace.Services.Events/src/MiniSpace.Services.Events.Infrastructure/Mongo/Repositories/Extensions.cs:line 40
   at MiniSpace.Services.Events.Infrastructure.Mongo.Repositories.EventMongoRepository.BrowseAsync(FilterDefinition`1 filterDefinition, SortDefinition`1 sortDefinition, Int32 pageNumber, Int32 pageSize) in /home/kaliuser/Documents/portfolio/commercial_apps/distributed_minispace/MiniSpace.Services.Events/src/MiniSpace.Services.Events.Infrastructure/Mongo/Repositories/EventMongoRepository.cs:line 47
   at MiniSpace.Services.Events.Infrastructure.Mongo.Repositories.EventMongoRepository.BrowseEventsAsync(Int32 pageNumber, Int32 pageSize, String name, String organizer, DateTime dateFrom, DateTime dateTo, Nullable`1 category, Nullable`1 state, IEnumerable`1 organizations, IEnumerable`1 friends, Nullable`1 friendsEngagementType, IEnumerable`1 sortBy, String direction) in /home/kaliuser/Documents/portfolio/commercial_apps/distributed_minispace/MiniSpace.Services.Events/src/MiniSpace.Services.Events.Infrastructure/Mongo/Repositories/EventMongoRepository.cs:line 72
   at MiniSpace.Services.Events.Infrastructure.Mongo.Queries.Handlers.GetPaginatedSearchEventsHandler.HandleAsync(GetSearchEvents query, CancellationToken cancellationToken) in /home/kaliuser/Documents/portfolio/commercial_apps/distributed_minispace/MiniSpace.Services.Events/src/MiniSpace.Services.Events.Infrastructure/Mongo/Queries/Handlers/GetPaginatedSearchEventsHandler.cs:line 61
   at Paralax.CQRS.Queries.Dispatchers.QueryDispatcher.QueryAsync[TQuery,TResult](TQuery query, CancellationToken cancellationToken)
   at Paralax.WebApi.CQRS.Builders.DispatcherEndpointsBuilder.<>c__DisplayClass3_0`2.<<Get>b__0>d.MoveNext()
--- End of stack trace from previous location ---
   at Paralax.WebApi.EndpointsBuilder.BuildQueryContext[T](HttpContext httpContext, Func`3 context)
   at Microsoft.AspNetCore.Routing.EndpointMiddleware.<Invoke>g__AwaitRequestTask|7_0(Endpoint endpoint, Task requestTask, ILogger logger)
   at Microsoft.AspNetCore.Authorization.AuthorizationMiddleware.Invoke(HttpContext context)
   at Microsoft.AspNetCore.Authorization.AuthorizationMiddleware.Invoke(HttpContext context)
   at Microsoft.AspNetCore.Authentication.AuthenticationMiddleware.Invoke(HttpContext context)
   at App.Metrics.AspNetCore.Tracking.Middleware.ApdexMiddleware.MeasureTransaction(HttpContext context)
   at App.Metrics.AspNetCore.Tracking.Middleware.PerRequestTimerMiddleware.TimeTransaction(HttpContext context)
   at App.Metrics.AspNetCore.Tracking.Middleware.RequestTimerMiddleware.TimeTransaction(HttpContext context)
   at App.Metrics.AspNetCore.Tracking.Middleware.ErrorRequestMeterMiddleware.Invoke(HttpContext context)
   at App.Metrics.AspNetCore.Tracking.Middleware.ActiveRequestCounterEndpointMiddleware.Invoke(HttpContext context)
   at Swashbuckle.AspNetCore.SwaggerUI.SwaggerUIMiddleware.Invoke(HttpContext httpContext)
   at Swashbuckle.AspNetCore.Swagger.SwaggerMiddleware.Invoke(HttpContext httpContext, ISwaggerProvider swaggerProvider)
   at Paralax.WebApi.Exceptions.ErrorHandlerMiddleware.InvokeAsync(HttpContext context, RequestDelegate next)
[20:54:02 INF] Request finished HTTP/1.1 GET http://localhost:5000/events/search?OrganizationId=3be66f89-

@SaintAngeLs, I remember you've mentioned the nullable values in convert query method in Paralax. it has nothing to deal with it. The issue is that the Organizer(Enumerator Type).OrganizationId(Also nullable Type) is compared as the string to Guid.


Bug in the scanner

Hello!
First of all, I love this repo. As somebody trying to use Sphero robots for research and student projects in academia, mostly using ROS, your Python driver is a godsend. Some time ago, my colleague and I developed a driver for Sphero SPRK+ and a ROS wrapper for it (check it out here), so I know how much work it can be.
I think I found a bug in scanner.py.
This line returns a generator:
https://github.com/artificial-intelligence-class/spherov2.py/blob/899e7bdba3178aba8f5f3ff10a9a76dbe19e13af/spherov2/scanner.py#L54
Once you iterate through "all elements" of the generator here:
https://github.com/artificial-intelligence-class/spherov2.py/blob/899e7bdba3178aba8f5f3ff10a9a76dbe19e13af/spherov2/scanner.py#L61
it will be empty. For any subsequent toy in found toys, the generator doesn't have any more elements and everything will be skipped.
The result is that, by using find_toys without toy_types, you can find maximally one toy, if you are lucky. If you are not, and there is some other BLE device nearby that is not Sphero, you will find nothing.
I would recommend simply wrapping line 54 with a list:
toy_types = list(all_toys())

It's not a long list, so there shouldn't be any memory problems.

Thank you. I think this bug is fixed here https://github.com/artificial-intelligence-class/spherov2.py/commit/c983ece45ef4307f08253dc6d1edd83126d1fc54

I am sorry If I'm delusional, but I think the linked commit is where the bug was introduced. That commit removed the list so we are left with an empty generator after the first loop.
I am running the code on main (source install) and I can confirm that I have the described problem.

ah, i can’t believe this bug is hunting me again (this was a fix to the same bug previously). can you check the latest commit which should have fixed it

Yes, set works perfectly as well. Thank you for the super quick response :smile:


Fix handling of Avro timestamps created in Hive 3.1

Hive 3.1 has changed the semantics of timestamps in backward incompatible way for a number of file formats (see https://issues.apache.org/jira/browse/HIVE-21002).  The impact of this change on Presto was addressed for Parquet and RCBinary, but it still manifests itself for Avro.
0: jdbc:hive2://localhost:10000/default> insert into test_avro values (1, '2020-01-01 12:34:56.123');

presto:default> select * from test_avro;
 id |           ts
----+-------------------------
  1 | 2020-01-01 18:19:56.123
(1 rows)

I though this gonna require  Hive 3.1.2 (see https://issues.apache.org/jira/browse/HIVE-21002, https://issues.apache.org/jira/browse/HIVE-21291)
But maybe we just do the same as @dain did for Parquet, ORC and RCFile?
OTOH, having 4 toggles (3 existing + 1 new) seems like may be hard for users to set correctly.

@dain have a look please.

@aalbu or @electrum is this still an issue

Yes, it can be reproduced in the HDP3 environment.


Create scafolding ruff Ruff config

As a suggestion,
I have not enabled most rule, I and invite you to try them (some of them might be overzealous).
Here for example I enable some rule that warn about default mutable argument and why it's bad:
In [1]: def f(a=[]):
   ...:     return a
   ...:

In [2]: x = f()

In [3]: x.append('coucou')

In [4]: f()
Out[4]: ['coucou']

note that ruff has a --fix option for many of those.

@JbIPS

Aow, it looks really bad indeed. How can it happen ?


[Thread] Ideas & Tweaks

Vanilla items

BridgeTrigger: Need to adjust bridgeDurability in Update() to scale according to player size
FlashlightItem: Adjust flashlightBulb & flashlightBulbGlow according to item size
JetpackItem: Adjust jetpack strength according to item size
KeyItem: Display "It won't fit. Scaling it was not the key to success.." when resized, as a reminder that scaling isn't always helpful
KnifeItem: Deal damage according to size
LockPicker: Increase unlocking speed if enlarged
SprayPaintItem: Spray radius according to item / spray-can size
TetraChemicalItem: Adjust duration according to item size
WhoopsieCustionItem: Adjust loudness according to item size

Ship

Scaling ship furniture

Might be fun if flashlights scaled up beyond a certain point dealt damage to players / monsters that are too close

Suggestion: Random sized enemies, and random sized on join/respawn between set values. That would be amazing!

Suggestion: Random sized enemies, and random sized on join/respawn between set values. That would be amazing!

There is a mod named RandomEnemiesSize which is compatible with this mod


CSHARP-2287: RetryWrites=true in a transaction throws exception when …

…using async writes.

Rebased and pushed to master.


go version

# github.com/gin-contrib/graceful
../../go/pkg/mod/github.com/gin-contrib/graceful@v0.0.0-20230904152018-e56ed94cd808/graceful.go:109:25: undefined: context.WithCancelCause
../../go/pkg/mod/github.com/gin-contrib/graceful@v0.0.0-20230904152018-e56ed94cd808/graceful.go:138:18: undefined: context.Cause
../../go/pkg/mod/github.com/gin-contrib/graceful@v0.0.0-20230904152018-e56ed94cd808/graceful.go:170:32: undefined: context.WithCancelCause
../../go/pkg/mod/github.com/gin-contrib/graceful@v0.0.0-20230904152018-e56ed94cd808/graceful.go:209:16: undefined: context.Cause
note: module requires Go 1.20

my default version is 1.18, so must update my version?

Hum, I guess I could drop that. It was nice to have. Is there any reason to use an old Go for you?

fixed in #8


Adapt HtmlWebpackPlugin to Craco

Hello devs !
My problem
I wanted to use an index.html file from my src directory (not from the public folder) which is one of the features in the html-webpack-plugin.
I came up with the following solution which works.
const {getPlugin} = require("../../src/lib/webpack-plugins")

module.exports = {
    webpack: {
        configure: (c) => {
            // Change html-webpack-plugin's template path
            const htmlWebpackPlugin = getPlugin("HtmlWebpackPlugin");
            if (htmlWebpackPlugin) htmlWebpackPlugin.userOptions.template = path.resolve(__dirname, 'src/index.html');
            return c.resolve?.extensions?.push('.png', '.jpg', '.jpeg'), c;
        }
    },
    plugins: [
        {
            plugin: {
              overrideWebpackConfig: ({ webpackConfig, context: {  paths } }) => {
                // Change the CRA public path / html file path
                paths.appPublic = path.resolve(__dirname, 'src');
                paths.appHtml = path.resolve(__dirname, 'src/index.html');
                return webpackConfig;
              }
            },
          },
    ]
};

Although I do not know how to implement it in craco.
What I can tell you is that :

The html-webpack-plugin needs the CRA paths (public path / html path) to be changed for it to work properly.
You need to override the html-webpack-plugin userOptions.template from the config for it to work.

What I tried

Adding html-webpack-plugin manually in the craco-config. The react-scripts package seams to always use the projects public folder event after changing it in the plugin options.
Just changing the template path from the html-loader which resulted in the same error as in the logs down below. Looks like react-scripts isn't happy when the index.html file is missing in the public folder.
Using the craco-html-webpack-plugin

Solutions

Find a way to change CRA paths easily
Ability to change plugins options directly from craco (without having too much code like the one I sent...) and make sure that it works with react-scripts
Create an options somewhere in the config to reference the index.html and/or the favicon for the html-webpack-plugin.

Some logs
Here is the output from npm start when I don't have the correct config and with the index.html file in my src folder
> env-cmd --verbose -e common,dev craco start --verbose

Options: {"command":"craco","commandArgs":["start","--verbose"],"rc":{"environments":["common","dev"]},"options":{"expandEnvs":false,"noOverride":false,"silent":false,"useShell":false,"verbose":true}}
Found environments: [common,dev] for default .rc file at path: ./.env-cmdrc.json
Project root path resolved to:  /home/alan/dev/apv11_frontend
Override started with arguments:  [
  '/home/alan/.nvm/versions/node/v16.14.2/bin/node',
<EMAIL_ADDRESS>  '--verbose'
]
For environment:  development
Config file path resolved to:  /home/alan/dev/apv11_frontend/craco.config.ts
Overrided craco config with plugin.
Overrided craco config with plugin.
Applied craco config plugins.
Found Webpack dev config at:  /home/alan/dev/apv11_frontend/node_modules/react-scripts/config/webpack.config.js
Overrided ESLint config to enable an ignore file.
Applied CSS loaders options.
Overrided CSS loader.
Applied CSS loaders options.
Overrided CSS loader.
Applied CSS loaders options.
Overrided CSS loader.
Applied CSS loaders options.
Overrided CSS loader.
Overrided PostCSS loader.
Overrided PostCSS loader.
Overrided PostCSS loader.
Overrided PostCSS loader.
Merged webpack config with 'webpack.configure'.
Overrided webpack config with plugin.
Applied webpack config plugins.
Overrided require cache for module: /home/alan/dev/apv11_frontend/node_modules/react-scripts/config/webpack.config.js
Overrided Webpack dev config.
Found dev server config at:  /home/alan/dev/apv11_frontend/node_modules/react-scripts/config/webpackDevServer.config.js
Overrided require cache for module: /home/alan/dev/apv11_frontend/node_modules/react-scripts/config/webpackDevServer.config.js
Overrided dev server config provider.
Starting CRA at:  /home/alan/dev/apv11_frontend/node_modules/react-scripts/scripts/start.js
Could not find a required file.
  Name: index.html
  Searched in: /home/alan/dev/apv11_frontend/public
Child process exited with code: 1 and signal:. Terminating parent process...

Similar
https://github.com/dilanx/craco/issues/279
Thank you for your time.

You should be able to use html-webpack-plugin as normal as I described in #413.

Here is my craco config file export :
module.exports = {
  style: {
    css: {
      loaderOptions: {
        url: false,
      },
    },
  },
  plugins: [
    {
      plugin: CracoAliasPlugin,
    },
  ],
  webpack: {
    plugins: {
      add: [
        new HtmlWebpackPlugin({
          filename: path.resolve(__dirname, 'src/index.html'),
        }),
      ],
    },
    configure: (c: Configuration) => {
      return c.resolve?.extensions?.push('.png', '.jpg', '.jpeg'), c;
    },
  }
};

This is supposed to use the index.html template located in 'src', but it will actually look in the 'public' folder as specified in error in the console :
Options: {"command":"craco","commandArgs":["start","--verbose"],"rc":{"environments":["common","dev"]},"options":{"expandEnvs":false,"noOverride":false,"silent":false,"useShell":false,"verbose":true}}
Found environments: [common,dev] for default .rc file at path: ./.env-cmdrc.json
Project root path resolved to:  /home/alan/dev/apv11_frontend
Override started with arguments:  [
  '/usr/bin/node',
<EMAIL_ADDRESS>  '--verbose'
]
For environment:  development
components/* components/$1
interfaces/* interfaces/$1
utils/* utils/$1
scss/* scss/$1
fonts/* fonts/$1
assets/* assets/$1
Config file path resolved to:  /home/alan/dev/apv11_frontend/craco.config.ts
Overrided craco config with plugin.
Applied craco config plugins.
Found Webpack dev config at:  /home/alan/dev/apv11_frontend/node_modules/react-scripts/config/webpack.config.js
Overrided ESLint config to enable an ignore file.
Applied CSS loaders options.
Overrided CSS loader.
Applied CSS loaders options.
Overrided CSS loader.
Applied CSS loaders options.
Overrided CSS loader.
Applied CSS loaders options.
Overrided CSS loader.
Overrided PostCSS loader.
Overrided PostCSS loader.
Overrided PostCSS loader.
Overrided PostCSS loader.
Added webpack plugins.
Merged webpack config with 'webpack.configure'.
Applied webpack config plugins.
Overrided require cache for module: /home/alan/dev/apv11_frontend/node_modules/react-scripts/config/webpack.config.js
Overrided Webpack dev config.
Found dev server config at:  /home/alan/dev/apv11_frontend/node_modules/react-scripts/config/webpackDevServer.config.js
Overrided require cache for module: /home/alan/dev/apv11_frontend/node_modules/react-scripts/config/webpackDevServer.config.js
Overrided dev server config provider.
Starting CRA at:  /home/alan/dev/apv11_frontend/node_modules/react-scripts/scripts/start.js
Could not find a required file.
  Name: index.html
  Searched in: /home/alan/dev/apv11_frontend/public
Child process exited with code: 1 and signal:. Terminating parent process...
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>start: `env-cmd --verbose -e common,dev craco start --verbose`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR!
Name: index.html
Searched in: /home/alan/dev/apv11_frontend/public"
Thank you

@AlanBlanchetNeovision  this is how I did the workaround
Inside craco.config.js
const HtmlWebpackPlugin = require('html-webpack-plugin');

module.exports = {
  ...
  webpack: {
    configure: (webpackConfig) => {
      const htmlWebpackPluginInstance = webpackConfig.plugins.find(
        webpackPlugin => webpackPlugin instanceof HtmlWebpackPlugin
      );

      if (htmlWebpackPluginInstance) {
        // put the custom path of the index.html here
        htmlWebpackPluginInstance.userOptions.template = path.join(__dirname, 'src/index.html');
      }

      return webpackConfig;
    },
  }
}

Unfortunately, I also tried to delete the public folder as it is not needed anymore, but craco will throw the following
Could not find a required file.
  Name: index.html
  Searched in: /<your-project-absolute-path>/public
 ELIFECYCLE  Command failed with exit code 1.

Not sure what's the best way to do this,  but hope it helps.


Enhancement: Enable multiline_array_trailing_comma fixer

This PR

[x] enables the multiline_array_trailing_comma fixer
[x] runs make cs

Follows #242.
:information_desk_person: See https://github.com/FriendsOfPHP/PHP-CS-Fixer/tree/1.11#usage:

multiline_array_trailing_comma [symfony]
PHP multi-line arrays should have a trailing comma.

Thanks!


Add fastero to gallery

The profiling category is only there because I can't think of any others. Suggestions appreciated

Thanks @wasi-master !
The image is a little too large. The max size is 800x600

@willmcgugan is that a necessity? Because now I would need to further lower the resolution or crop the image.
The image size is pretty low already though

Afraid so. Think of it as a thumbnail. You can put a full-res version on the repo...

@willmcgugan Done


npm run test fails during CI

Because of the match pattern for the ESLint command, the test command fails in production after building the application. We could fix this in a few ways:

Remove the linting script from the test command. Since we have lint-staged, linting should not be a concern when deploying our application (in theory).
Change the matching pattern. This makes sense, but it takes more effort than the 5 minutes I’ve put into thinking about it.
Change the order of steps: Install deps, test, then build. But, when we implement integration tests, we’ll need to have the application built before we test.

We're just not going to run lighting in production, but we'll still want to revisit this later for local development, because it can cause issues there as well.


@conda-forge-admin, please add bot automerge

@conda-forge-admin, please add bot automerge

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/types-termcolor-feedstock#4.


[Platinum] Mt. Coronet has the wrong encounters (Only Regice)

Describe the bug
Mt. Coronet's default encounter lists only Regice.
To Reproduce
Steps to reproduce the behavior:

Make a new Platinum Nuzlocke
Scroll down to the Mt. Coronet encounters and click "find encounters"

Expected behavior
The actual encounters found in Mt. Coronet should appear instead. Here is a link which contains them: https://m.bulbapedia.bulbagarden.net/wiki/Mount_Coronet
Screenshots
(https://i.gyazo.com/e2edbd01c826ef87d9d5ab32d0230c66.png)

I've regenerated a lot of the Platinum route encounters based on Pokemon DB
They should be more correct now!


MSID: 90690 Version: 1 DOI: 2<IP_ADDRESS>1596

"msas": "Biochemistry and Chemical Biology", "Neuroscience"
"msid": "90690"
"preprintDoi": "10.1101/2<IP_ADDRESS>1596"
"Reviewed Preprint posted": "2023-09-26"
"Sent for peer review": "2023-07-25"
"Posted to bioRxiv": "2023-08-02" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>1596v1/")
[PLACE PDF URL HERE WHEN AVAILABLE] See step 7
Step 1. Inform bioRxiv
Who can help: @QueenKraken, @nlisgo, @scottaubrey

[x] Manuscript is in data hub index (https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v1/by-publisher/elife/get-by-manuscript-id?manuscript_id=90690)

Step 2. Create preview of manuscript
Who can help: @fred-atherden, @nlisgo, @scottaubrey

[ ] Manuscript is available for preview (https://prod--epp.elifesciences.org/preview/90690/v1)

[ ] Trigger import for staging: https://prod--epp.elifesciences.org/import
[ ] Sync data folder with s3://prod-elife-epp-data/data
[ ] Trigger import for staging: https://staging--epp.elifesciences.org/import
[ ] Sync data folder with s3://staging-elife-epp-data/data
[ ] Pull request created to enhanced-preprint-data
[ ] Post link to pull request below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-manuscript
Step 3: Awaiting public reviews and QC
Who can help: Production team

[ ] QC passed or QC not needed

Who can help: Editorial team

[x] Public reviews available on sciety (https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>1596)
[x] Manuscript and editor details at the top of this issue (Supplied by Editorial team)
[x] Reviews returned by EPP - https://prod--epp.elifesciences.org/api/reviewed-preprints/90690/v1/reviews

Example
"msas": "Genetics and Genomics", "Neuroscience"
"msid": "84628"
"version": "1"
"preprintDoi": "10.1101/2<IP_ADDRESS>4241"
"articleType": "Reviewed Preprint"
"status": "Published from the original preprint after peer review and assessment by eLife."

"Reviewed Preprint posted": "2023-01-02"
"Sent for peer review": "2022-10-28"
"Posted to bioRxiv": "2022-11-21" (link: "Go to bioRxiv": "https://www.biorxiv.org/content/10.1101/2<IP_ADDRESS>4241v1")

Editors:
Reviewing Editor
Michael B Eisen
University of California, Berkeley, United States
Senior Editor
Michael B Eisen
University of California, Berkeley, United States

Step 4: Modify manuscripts.json (no PDF)

[ ] Reviewed preprint is published to EPP (https://elifesciences.org/reviewed-preprints/90690)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[ ] Request that a doi be registered

[ ] Post link to #enhanced-preprint comment below

Pull request: [PLACE LINK TO PULL REQUEST HERE]
#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]

Instructions to modify manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/publish-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/334/files
Once the pull request is merged in it should be available a few minutes later.

Request that a doi
Post the following in #enhanced-preprint:
@Fred can you register a doi for https://elifesciences.org/reviewed-preprints/90690

Step 5: Awaiting search reindex

[ ] Reviewed preprint is avaliable on journal homepage (https://elifesciences.org)

The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.

Additional info
If needed, the jenkins pipeline to reindex search can be triggered sooner.
https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 6: Published! Request PDF generation

[ ] Post the link to the PDF url at the top of the issue

[ ] Post link to #enhanced-preprint comment below

#enhanced-preprint comment thread: [PLACE LINK TO COMMENT HERE]
Post the following to the #enhanced-preprint on slack:
@Ryan Dix-Peek please can you generate a PDF for https://elifesciences.org/reviewed-preprints/90690

Step 7: Introduce PDF to data folder and git repo

[ ] PDF is avaliable at https://github.com/elifesciences/enhanced-preprints-data/raw/master/data/90690/v1/90690-v1.pdf

Detailed instructions: https://github.com/elifesciences/enhanced-preprints-data#add-a-pdf
Step 8: Add PDF url to manuscripts.json

[ ] Reviewed preprint PDF is available for download (https://prod--epp.elifesciences.org/reviewed-preprints/90690)

[ ] Pull request created to enhanced-preprint-client
[ ] Pull request switched to "Ready for review"
[ ] Post link to pull request below

[PLACE LINK TO PULL REQUEST HERE]

Instructions to add PDF url to manuscripts.json

Visit: https://github.com/elifesciences/enhanced-preprints-client/actions/workflows/add-pdf-url-to-manuscript.yaml
Click: Run workflow
Complete the form and click "Run workflow"
A successful run should result in a new pull request at https://github.com/elifesciences/enhanced-preprints-client/pulls
Open the pull request and click the "Ready for review" button to trigger tests
Once the tests pass and you are happy with the changes the PR can be merged

Example pull request: https://github.com/elifesciences/enhanced-preprints-client/pull/397/files
Once the pull request is merged in it should be available a few minutes later.

Step 9: Done!

[ ] Kettle is on!

Chávez-Gutiérrez Lucía has wrong corresponding email address, also is it Chávez-Gutiérrez Lucía or Lucía Chávez-Gutiérrez both names have been used in preprint

Nice sport Rebecca - thanks

Authors still not updated. Chased again.

No real luck with the authors so I am going to modify the file with an xsl so this can proceed.


Installation of requirements.txt fail because of pyocd issue

Installation of requirements.txt fails because the latest release of pyocd is not installable via pip. See mbedmicro/pyOCD#442.
I suggest locking the pyocd dependency to a version that is now to least be installable, ie. version 0.12.0.

Thanks! This was fixed via #11280. Closing.


fix(lint/noUnsafeDeclarationMerging): recoggnize ambient classes in ambient namespace

Summary
This is related to #19 but concerns noUnsafeDeclarationMerging.
This correctly recognizes ambient classes in ambient context.
Thus, the following code is not reported as unsafe because merging an ambient class and an interface is safe:
declare namespace A {
  namespace B {
    export interface Foo {}
    export class Foo {}
  }
}

Test Plan
Tests included.

reminder that we should update the changelog with this fix

Actually, the changelog doesn't need to be updated because the rule already has an entry in the "New rules" section. It would be confusing for a reader to have a fix entry for a new rule.


⚠ Spryr has degraded performance

In 66640ab, Spryr (https://spryr.com) experienced degraded performance:

HTTP code: 200
Response time: 4716 ms

Resolved: Spryr performance has improved in e2c3de0 after 16 minutes.


Record on chart not updated

Good afternoon,
I have a question about enriching entities on the chart via connector.
i2 Analyze v4.4.3 with fixpack
i2 containers 2.8.0
ANBP 10.0.1

I added a Vehicle to the chart with License plate 192KMB

I called a car registration connector. The car is found including the color etc

I copied the output of the connector to the chart

And the vehicle is updated as expected.

Now I have a vehicle from another connector where the license plate is identically. But also the Type and Model are filled.

I added the Vehicle to the chart

Via the connector I found the same details

But when I copied this connector output to the chart nothing happens.

I would expect the same results for the second scenario. Why is the color added in the first scenario and not in the second scenario?
Also the Identifier from both vehicles is the same.

It seems that all properties are assigned correctly to the seed.

I tested it also in i2 Notebook, there it works as expected in both scenario's.

So looking at your issue, starting with the first part, first charting 192KMB with the first connector and it's coloured red.

I presume to change the colour you have a Conditional Formatting Specification active in Live Formatting with a rule looking for "Vehicles" records from a given deployment and set the shading colour for the chart item containing the record based on the "Vehicle Color".
All the source identifiers in the connector are marked as persistent/reproducible from the connector.
Are there any other system matching rule in place? By default ANBP will match i2Connect results on source identifier and update accordingly, otherwise you are able to deploy system matching rules to to discriminator property based matching, for example Make + Model + License Plate.

With at least 1. above in place 192KMB from connector 1 that has a Red colour hits the chart and obtains a red icon shade.
Now the next step. Finding and charting the "same" 192KMB from connector 2. Is this a seeded query using 192KMB on the chart found in connector 1?
If so you should be able to modify the seed record received and return it in the results. The i2 Connect Gateway will have kept back some additional data ANBP attached to the seed record that you won't see in what is set to the connector and mark up the returned seed with any changes it has noticed in the round trip of that record. This allow ANBP to effectively rehydrate the record we asked the gateway to cache and then we apply the changes the gateway has detected in the returned seed (this allow us to preserve notes and source references that may have been applied, along with other metadata should the record be a store, IS, record).
However if you used the seed to, in effect, find a record that happened to have the same License Plate Number and return that in the results, unless there are system matching rules in place, recombining the returned record to what is already on the chart will be down to sourceID matching. Given it's from a different connector, the source ID the i2Connect gateway returns may include the connector ID and connector query ID. Therefore although both connectors may have given the same single string sourceID in both cases, the i2Connect gateway has decorated the result thus making the sourceIDs look different and not matching. This could be why from connector 2 192KMB was charted as a different chart item to the original seed, and given it didn't have colour information (I am assuming from the attributes put on the chart snapshot) the Live Formatting rule didn't change it to red.

@Andrew-Stawarz-i2 thanks for your reply!

I use indeed the Live Formatting for the color.
How can I check that?
I have one match rules for the entity person.

Connector 1: custom car registration connector. I used the service where use a vehicle as seed. The service will get extra information about the vehicle (like color etc) and returns this extra information.
Connector 2: iBase connector. I retrieved the vehicle from iBase in the 2nd scenario.
For both 192KMB entities I use the same connector (connector 1) where the 192KMB is the seed.
In i2 Notebook it works as expected with the same connectors.

Get vehicle from the User Guide db via the iBase connector

Retrieve extra information via the connector 1

Copy to chart -> properties are added (as expected)

Hi @sdorresteijn88,
I've ploughed through the code in ANBP to check the seeding behaviour. Using our test i2Connect connectors I see the expected behaviour when you use one connector query to first search for a record and then use that result to seed a second query that will "enrich" the seed record with additional values. The enriched record result correctly merges back with the seed when set to the chart.
The same works when the above two operations are executed in i2Notebook.
What I have found, taking a leftfield approach, is if I first search for a record in i2Connect from within i2Notebook, chart the result, and then share that record so that it can be opened in ANBP, that shared record will appear as read-only (not user modifiable):

This is a bug. If I then try to execute the same seeded i2Connect query using the shared record in ANBP and chart the enriched resulting record then record on the chart is not updated. Why? In ANBBP we have logic that controls the updating of user or system governed (those typically ETL'd into the Information Store) records. Because i2Notebook marked the shared record as not user modifiable, ANBP interpreted this as a system governed record on charting.

@Andrew-Stawarz-i2 it's for me not clear if the last part is also part of the description of the bug.
'This is a bug. If I then try to execute the same .... system governed record on charting.'
Can it be that this logic changed after 20-9-2023? At that moment I assume I had the most recent versions of all software.  I have a note in the code of my connector that it was working in ANBP but not in i2 Notebook at that moment. Now it's the other way around it's working in i2 Notebook and not in ANBP.

@sdorresteijn88 That's interesting to hear, you've observed a difference in ANBP External Searches/i2Connect behaviour between our 10.0.0 and 10.0.1 releases. The most significant change in ANBP in the 10.0.1 release (release notes: https://docs.i2group.com/release-material/anbp/10.0.1/i2-anbp-10.0.1.html) that would have impacted on Analysis Studio/Hub deployments was the addition of "Flexible chart item types and properties" from the palette, but that is still restricted to chart only operations.
I will need to diff the code base between those releases to see if there where any "unexpected" changes made around i2Connect interactions (seeding, charting and fusion/matching).
As for the bug I mentioned, this was found whilst I tried various combinations of seeding, charting of records and sharing records between our web and desktop notebook offerings. Records shared from i2Notebook with ANBP had an incorrect flag set on the shared record that would subsequently affect ANBP's charting of results from seeded i2Connect queries using those shared records.

@sdorresteijn88 I've ploughed through the ANB and ANBP code changes between the 10.0.0 and 10.0.1 release and there has been no changes made that would have impacted on charting for i2Connect.
The last significant change was between 9.4.0 and 9.4.1 when handling charted iBase records as seeds for /i2Connect. Prior to that in August 2022 we fixed up behaviour in identifying i2Connect source identifiers that were reported by the connector as being persistent.

@Andrew-Stawarz-i2 I used ANB 9.4.x at that time because iBase 9.2.1 was not released yet.

Hi @sdorresteijn88 (Steven)
It seems that what you are experiencing here is related to differences in how i2 Notebook and Analyst's Notebook Premium are behaving with respect to your i2 Connect charting.
Andrew has spent several hours looking in to this for you and we feel that as this is product related,  and not SDK coding issues for example, we would be best to have this handled as a standard product related case.
Can you please raise this via support in the normal way and I will let our folks know that development are already aware of what you are experiencing here and it can be escalated immediately.
If you include a link to this GitHub Issue it will help avoid going over things again from your perspective  unnecessarily.
Thank you.

@Anthony-Johnson-i2 I'll raise a support ticket for this. Thanks for the investigation so far!

Seems to be fixed in 2.9.0


data

Great work, can you tell your code to use the training data in the txt file and some other data? I am a beginner in this direction and do not know much about it. I hope you can help me

The datasets used in this project are commonly known datasets in remote sensing, you can easily download them online. The dataloaders are provided in ./data/.


SDL: propagate X11 libs even when building statically against them on Darwin

Packages like SDL_image, SDL_mixer, SDL_net, SDL_ttf depend on this.
This reverts a piece of 19130ebc5d05cc7720335c92d68a4fd1faf28dc0.
Motivation for this change
https://github.com/NixOS/nixpkgs/commit/19130ebc5d05cc7720335c92d68a4fd1faf28dc0#commitcomment-27987219, search "SDL" in https://hydra.nixos.org/jobset/nixpkgs/trunk#tabs-jobs.
Things done

[X] It evaluates.
[X] Nothing changes on Linux.
[X] Fits CONTRIBUTING.md.

Please run ofborg build SDL SDL_image SDL_mixer SDL_net SDL_ttf on darwin.

@GrahamcOfBorg eval

@GrahamcOfBorg eval

Success on x86_64-linux (full log)
Partial log (click to expand)
/nix/store/p26kggyw4s40cn0180i3519d7pfzlsrw-SDL-1.2.15

Success on aarch64-linux (full log)
Partial log (click to expand)
these paths will be fetched (0.13 MiB download, 0.43 MiB unpacked):
  /nix/store/daijpnf7hpc6309lghxj3926vq4a08vm-SDL-1.2.15
copying path '/nix/store/daijpnf7hpc6309lghxj3926vq4a08vm-SDL-1.2.15' from 'https://cache.nixos.org'...
/nix/store/daijpnf7hpc6309lghxj3926vq4a08vm-SDL-1.2.15

@GrahamcOfBorg eval

@GrahamcOfBorg build SDL SDL_image SDL_ttf SDL_mixer

Success on x86_64-linux (full log)
Partial log (click to expand)

*** Downloading ‘https://cache.nixos.org/nar/0p553r92nvcq8izdwna3cxhshv15qyf8kpw9cvd00dr7c2jpgqkx.nar.xz’ (signed by ‘cache.nixos.org-1’) to ‘/nix/store/4rg74yd7vm9jij2akz1wp7xcwxm3vajb-SDL_mixer-1.2.12’...
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed

  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
100 70768  100 70768    0     0   330k      0 --:--:-- --:--:-- --:--:--  329k

/nix/store/p26kggyw4s40cn0180i3519d7pfzlsrw-SDL-1.2.15
/nix/store/ryhabjvmpqnbyymsd9qs4957vam29dxv-SDL_image-1.2.12
/nix/store/rfy7vqvfpy6rcsgr5ah0f3kjnrrs5i5r-SDL_ttf-2.0.11
/nix/store/4rg74yd7vm9jij2akz1wp7xcwxm3vajb-SDL_mixer-1.2.12

Success on aarch64-linux (full log)
Partial log (click to expand)
  /nix/store/yfaxm5a645z9cbjsz23rx0jr4k4pywax-SDL-1.2.15-dev
copying path '/nix/store/6cgcnlddj9f4ldg4zh11br508b117bfb-SDL_image-1.2.12' from 'https://cache.nixos.org'...
copying path '/nix/store/mlpqydsz13a5glcbfin1vz3qk3ihjjyy-SDL_ttf-2.0.11' from 'https://cache.nixos.org'...
copying path '/nix/store/yfaxm5a645z9cbjsz23rx0jr4k4pywax-SDL-1.2.15-dev' from 'https://cache.nixos.org'...
copying path '/nix/store/135kyd2gki19205b2c09dl6h4s9l2xj1-smpeg-svn390' from 'https://cache.nixos.org'...
copying path '/nix/store/sa7v98ksfy4q4nwd9xvr5l8gvx5gm9dv-SDL_mixer-1.2.12' from 'https://cache.nixos.org'...
/nix/store/daijpnf7hpc6309lghxj3926vq4a08vm-SDL-1.2.15
/nix/store/6cgcnlddj9f4ldg4zh11br508b117bfb-SDL_image-1.2.12
/nix/store/mlpqydsz13a5glcbfin1vz3qk3ihjjyy-SDL_ttf-2.0.11
/nix/store/sa7v98ksfy4q4nwd9xvr5l8gvx5gm9dv-SDL_mixer-1.2.12

Success on x86_64-darwin (full log)
Partial log (click to expand)
/nix/store/ikcdk6s24zccg5gdav1f3bhyig5hr6l5-SDL_mixer-1.2.12/lib/libSDL_mixer-1.2.0.dylib: fixing dylib
/nix/store/ikcdk6s24zccg5gdav1f3bhyig5hr6l5-SDL_mixer-1.2.12/lib/libSDL_mixer-1.2.0.dylib: fixing dylib
/nix/store/ikcdk6s24zccg5gdav1f3bhyig5hr6l5-SDL_mixer-1.2.12/lib/libSDL_mixer-1.2.0.dylib: fixing dylib
/nix/store/ikcdk6s24zccg5gdav1f3bhyig5hr6l5-SDL_mixer-1.2.12/lib/libSDL_mixer-1.2.0.dylib: fixing dylib
/nix/store/ikcdk6s24zccg5gdav1f3bhyig5hr6l5-SDL_mixer-1.2.12/lib/libSDL_mixer-1.2.0.dylib: fixing dylib
/nix/store/ikcdk6s24zccg5gdav1f3bhyig5hr6l5-SDL_mixer-1.2.12/lib/libSDL_mixer-1.2.0.dylib: fixing dylib
/nix/store/kclmnd47ckm9clbz1lffk1hsjhclwydh-SDL-1.2.15
/nix/store/vvpn16bak4j94vr35lf90fqbf5iywllz-SDL_image-1.2.12
/nix/store/h552r695gmb7xa6y17gqq8lcc0p3ay3p-SDL_ttf-2.0.11
/nix/store/ikcdk6s24zccg5gdav1f3bhyig5hr6l5-SDL_mixer-1.2.12

Thanks!


Uncaught Exception

I am not very knowledgable about JS/TS so there's a very likely chance I did something wrong here.
I see the following error trying to connect to an invalid address:
FetchError: request to http://localhost:32820/api/ failed, reason: connect ECONNREFUSED <IP_ADDRESS>:3282
upon spamming .connectNetwork(). Was wondering if there's a way to handle this with a defined Exception?
to catch const resp = await fetch(http://localhost:${WEBSERVER_PORT}/api/)

Thanks for the report, will look into it shortly 👍

Can you check if this PR ☝  fixes the issue?
You can install it with npm install -g micropython-ctl@beta. Afterwards mctl version should print v1.13.8-beta2.

Let me give this a try! You are right, I am not able to figure out how to effectively do try catch in JavaScript ☹; been mostly coding in Java/scala.
My use case it to convert input image and display it to to an OLED i2c connected to esp32. I have been using node-red for the flows and wrote a function node to do this for me:
I am super interested in adding node-red nodes for the apis you have created to add drag and drop UI nodes for it.
I performed spam calls to check resiliency but would crash node-red with:

I think it would be really great to have those node-red nodes, and I'm happy to support you however I can.
A remarks on your code:
If connectNetwork succeeds without an exception, then you are guaranteed that the device is connected. Therefore the next 2 lines (check on isConnected shouldn't be necessary.
How do you import the module, actually? Is the execution environment web-based or Node.js-based? I don't have experience with node-red :)

One more remark -- it is possible to reuse one long-lasting connection, if you don't want to connect and disconnect every time. You can do this by calling device.startInternalWebserver(), although that name should probably change to device.allowConnectionReuse()

I think it would be really great to have those node-red nodes, and I'm happy to support you however I can.

Give node red a try. It is nodejs based with webui to do flow programming . I have been fooling around it with it for 4 months for ifttt and iot but the possibilities are endless 😊.
Having said that, just need to find time between jobs and 2 kids 😂.

A remarks on your code:
If connectNetwork succeeds without an exception, then you are guaranteed that the device is connected. Therefore the next 2 lines (check on isConnected shouldn't be necessary.

 They were debug code that can be removed.

How do you import the module, actually? Is the execution environment web-based or Node.js-based? I don't have experience with node-red :)

This is a bit tricky in node red. A settings.js needs to be updated so that it can be used in a UI function  node or alternatively use user defined nodes to import packages.

One more remark -- it is possible to reuse one long-lasting connection, if you don't want to connect and disconnect every time. You can do this by calling device.startInternalWebserver(), although that name should probably change to device.allowConnectionReuse()

That is a great idea and thank you for the tip! I was been thinking about using a config node in node red. Will look up some plug-in code to reverse engineer 🤩


TypeScript w/ out-of-the-box defaultTheme + jsx attributes

Hi,
following the readme's and guides on how to set up xstyled with either styled components or emotion (but avoiding using the 'x' notation, we know you recommend this approach, but we want to avoid using the 'x' notation and keep the 'styled' notation) we are bumping into an issue with TypeScript that we cannot seem to solve.
We aim to get the typed values from the defaultTheme, and in the future, from a custom theme, instead of the default css values.

when using the prop function notation we get the correct types, we also understand that the string interpolation method with th or hooks with useTheme or the "magic" notation does not have strong typing.

we understand that https://github.com/gregberge/xstyled/issues/163 `but I was wondering if the prop notation were at least supposed to infer possible values from the theme? from the typings it seems that this is possible, or at least intended, but we cannot figure out how to make xstyled infer them from the SystemProps' defaultTheme, instead of from the CSS default props.

Silly, for anyone bumping into this issue, when you work using the styled notation, make sure to extend your component from the SystemProps<Theme> or SystemProps<DefaultTheme> interface, where Theme or DefaultTheme are your custom emotion or styled components theme or the default theme respectively.

Silly, for anyone bumping into this issue, when you work using the styled notation, make sure to extend your component from the SystemProps<Theme> or SystemProps<DefaultTheme> interface, where Theme or DefaultTheme are your custom emotion or styled components theme or the default theme respectively.

By following TypeScript guide everything should work fine with v3.


FIX: Change ci to fail-fast=False

This will enable running the slack webhook for the transformers tests

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Unable to move terminal window on Windows 10 + WSL

[x] I am on the latest Hyper.app version
[x] I have searched the issues of this repo and believe that this is not a duplicate

OS version and name: Microsoft Windows 10 + WSL + VcVxsrv
Hyper.app version: 3.0.2
Link of a Gist with the contents of your .hyper.js: Default Hyper JS (deleted files and tried fresh).
Relevant information from devtools (CMD+ALT+I on macOS, CTRL+SHIFT+I elsewhere):
The issue is reproducible in vanilla Hyper.app: Is Vanilla

Issue

After starting up Hyper on WSL + VcVxsrv, I cannot move or resize the window. Maximize and restore work, but resize doesn't change the window size and I cannot drag the window on the screen or to another monitor. Standard Windows shortcuts, like Win+Arrow don't work either.
There are no errors in developer tools and bringing the window to front with the menu doesn't change anything.

is Hyper installed on windows or wsl?

WSL via the debian package.
On Thu, Mar 26, 2020, 03:55 Labhansh Agrawal<EMAIL_ADDRESS>wrote:

is Hyper installed on windows or wsl?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/zeit/hyper/issues/4426#issuecomment-604307265, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHNLNV6M4IJEHW6SMCI2CDRJMJ7VANCNFSM4LTSOVWQ
.

You should install Hyper in windows instead and set term to wsl.exe
x11 is not 'officially' supported by wsl https://github.com/Microsoft/WSL/issues/2356 so it might have a few kinks, maybe some apps work but all are not guaranteed to.

Setting it to Windows bash works (config settings below, if they are ever useful to anyone). It still feels odd for X11 not to work. I run gvim, zeal and terminator on X11 every day without issue. Either way, it works.
shell: 'C:\\Windows\\System32\\bash.exe',
shellArgs: ['~'],

I agree most of the apps work through Vcxsrv, I have used some of them myself. But I had issues trying to run vs code like that. So maybe it's an issue with electron apps currently as the window frame and dragging are done in a roundabout way for electron apps (ones without the native frame like hyper, vscode etc.)
I think you can close the issue as it's resolved now.


Mod manifest is missing some information

The mod manifest in 5he .Content project is misconfigured (author, ...)

Oh my B. Fixed now


JobCreator not creating final jobs for a specific workflow

This workflow
vlimant_task_SMP-RunIISummer15wmLHEGS-00164__v1_T_170926_193056_9589
is facing the same issues on both vocms0310 and vocms0303 (and maybe submit2).
These agents are being drained for a couple of weeks now, they have 0 jobs queued and/or in the global pool, however, there are still some subscriptions not finished and files available for further processing.
I've put vocms0310 JobCreator component into debug mode, but I can't see anything weird (besides a few things that I still don't know how it works...).

Jean-Roch noticed something weird going on with vocms0254, regarding this request:
pdmvserv_task_EXO-RunIISummer16NanoAOD-01867__v1_T_180210_152217_1516
it has only two WQE, both in Running state on vocms0254. One of them is completed, the other is Running in the agent. The wf_id 16179 and sub_id 34350 is still not finished, see
SELECT wmbs_workflow.id AS wf_id, wmbs_subscription.id AS sub_id, wmbs_workflow.task, wmbs_subscription.finished
       FROM wmbs_workflow INNER JOIN wmbs_subscription
       ON wmbs_workflow.id=wmbs_subscription.workflow
       WHERE wmbs_workflow.name='pdmvserv_task_EXO-RunIISummer16NanoAOD-01867__v1_T_180210_152217_1516';

both Processing filesets are no longer open, but only one of them has the subscription finished.
Poking around the wmbs tables, we can see that subscription still has THAT file available for job creation, the fileset has the correct fileid associated to it, AND it has valid locations, see
select * from wmbs_sub_files_available where subscription in (34350, 34394);
select * from wmbs_fileset_files where FILESET=197956;
select * from wmbs_file_location where fileid=4968732;

Now I'm going to add some debugging statements to the JobCreator code to see why it doesn't create a job/jobgroup for that available file
...
2018-02-28 10:54:02,205:140288507098880:INFO:JobCreatorPoller:Retrieved 0 jobGroups from jobSplitter
2018-02-28 10:54:02,206:140288507098880:INFO:JobCreatorPoller:Found end in iteration over subscription 34350

I found the problem, but I don't know how it happened!
The file available for job splitting does NOT have any run/lumi map information:
select * from wmbs_file_runlumi_map where FILEID=(select id from wmbs_file_details where lfn='/store/mc/RunIISummer16MiniAODv2/ADDGravToGG_MS-8000_NED-4_KK-1_M-500To1000_13TeV-sherpa/MINIAODSIM/PUMoriond17_80X_mcRun2_asymptotic_2016_TrancheIV_v6-v1/100000/9EA30DC4-82EF-E611-A0B7-7845C4F92EC7.root');

which makes this file not to have any Run object inside the job splitting algo, which causes no job to be created:
https://github.com/dmwm/WMCore/blob/1.1.8_wmagent/src/python/WMCore/JobSplitting/EventAwareLumiBased.py#L120
The component responsible for creating these wmbs information is WorkQueueManager (WorkQueueManagerWMBSFileFeeder thread), which according to the logs (and thankfully some debugging added on that branch) everything went smooth (including a graceful cycle completion):
...
2018-02-14 14:39:57,097:140288919463680:INFO:WMBSHelper:"pdmvserv_task_EXO-RunIISummer16NanoAOD-01867__v1_T_180210_152217_1516" Injecting block /ADDGravToGG_MS-8000_NED-4_KK-1_M-500To1000_13TeV-sherpa/RunIISummer16MiniAODv2-PUMoriond17_80X_mcRun2_asymptotic_2016_TrancheIV_v6-v1/MINIAODSIM#da7ed1b0-ef88-11e6-93bf-002590494c06 (1 files) into wmbs
2018-02-14 14:39:57,182:140288919463680:INFO:WMBSHelper:WMBS File: /store/mc/RunIISummer16MiniAODv2/ADDGravToGG_MS-8000_NED-4_KK-1_M-500To1000_13TeV-sherpa/MINIAODSIM/PUMoriond17_80X_mcRun2_asymptotic_2016_TrancheIV_v6-v1/100000/9EA30DC4-82EF-E611-A0B7-7845C4F92EC7.root
 on Location: set([u'T1_US_FNAL_MSS', u'T2_IN_TIFR', u'T2_US_UCSD', u'T1_US_FNAL_Buffer'])
2018-02-14 14:39:57,384:140288919463680:INFO:File:AMR about to add 1 non-unique files for fileset 197956
2018-02-14 14:39:57,395:140288919463680:INFO:File:AMR run and # of lumis to be recovered: {1: 4}
2018-02-14 14:39:57,398:140288919463680:INFO:AddRunLumi:AMR received 1 file entries
2018-02-14 14:39:57,399:140288919463680:INFO:AddRunLumi:AMR inserting 4 lumis from sliceBinds
2018-02-14 14:39:57,531:140288919463680:INFO:WorkQueue:Created top level subscription 34350 for pdmvserv_task_EXO-RunIISummer16NanoAOD-01867__v1_T_180210_152217_1516 with 1 files
...
2018-02-14 14:41:15,939:140288919463680:INFO:WorkQueueManagerWMBSFileFeeder:53 of units of work acquired for file creation
2018-02-14 14:41:15,941:140288919463680:INFO:BaseWorkerThread:WorkQueueManagerWMBSFileFeeder took 154.646 secs to execute

this is where the run/lumi information is added to the database (see log above):
https://github.com/dmwm/WMCore/blob/1.1.8_wmagent/src/python/WMCore/WMBS/File.py#L617
this is the DAO adding it (see log above, no problems!!!!):
https://github.com/dmwm/WMCore/blob/1.1.8_wmagent/src/python/WMCore/WMBS/MySQL/Files/AddRunLumi.py
this is the higher level code that starts all these insertions, where we can see that top level subscription was successfully created (thus the db transaction had been committed!!!):
https://github.com/dmwm/WMCore/blob/1.1.8_wmagent/src/python/WMCore/WorkQueue/WorkQueue.py#L449
@ticoann can you please take some time and have a look at this issue and my findings? I'm happy to know that I'm wrong and I missed a subtle detail, unfortunately I cannot see anything wrong in the code nor in the logs.
Then we can see how to manually fix this (these?) cases.

And I have a conceptual comment, even though code-wise it seems to be correct.
I can't make much sense of this code
https://github.com/dmwm/WMCore/blob/1.1.8_wmagent/src/python/WMCore/WorkQueue/WMBSHelper.py#L438
haven't we started a new transaction object a few lines above? Thus, I'd expect existingTransaction to be True, (problem is that it would not commit anything in that case).
It seems this code should instead be
        existingTransaction = self.beginTransaction()
...
        self.commitTransaction(existingTransaction)

however, if there is a transaction on going (e.g. myThread.transaction.transaction != None), existingTransaction would be True and again, a commit would be triggered only by the highest level class which initiated this beginTransaction(), IF called from there.
Am I missing anything?

 existingTransaction = self.beginTransaction()

...
self.commitTransaction(existingTransaction)

Alan, you are right. The code should be above.  I think it was mistake.
But since this should be (is) a top level function, maybe someone want to make sure it commits.
I would change the code for clarity.

what can we do in the short term to release the missing jobs ?

Retrieve the run/lumi for the affected files from DBS, then insert the missing run/lumi database records in WMBS. Next job splitter data discovery should find it. @amaltaro , do you agree ?

Yes, I'll try this procedure for that one file/workflow. If things go well, then we'll have to see how many we're missing and automatically fetch/insert the missing info.
Sorry, I had the pleasure to be stuck in the car for the whole morning...

@vlimant the agent had only that file without the run/lumi map. I manually inserted it, the job was created and that workflow is now announced.

is there a way to detect other such cases ?
https://cmsweb.cern.ch/couchdb/workqueue/_design/WorkQueue/_rewrite/elementsInfo?request=pdmvserv_task_B2G-RunIISummer16NanoAOD-01730__v1_T_180228_041828_870
https://cmsweb.cern.ch/couchdb/workqueue/_design/WorkQueue/_rewrite/elementsInfo?request=pdmvserv_task_B2G-RunIISummer16NanoAOD-01233__v1_T_180227_201834_636
https://cmsweb.cern.ch/couchdb/workqueue/_design/WorkQueue/_rewrite/elementsInfo?request=pdmvserv_task_B2G-RunIISummer16NanoAOD-01180__v1_T_180228_001820_2891
look stalled

there are several others with same stalling apparent behavior that can be spotted visually, let me know if you find a way to track them all down

It seem above workflows are stuck with different reason than Alan described.
It merge task create the job but not yet submitted. (Not sure why)
/pdmvserv_task_B2G-RunIISummer16NanoAOD-01730__v1_T_180228_041828_870/B2G-RunIISummer16NanoAOD-01730_0/B2G-RunIISummer16NanoAOD-01730_0MergeNANOEDMAODSIMoutput
http://vocms0253.cern.ch:5984/wmagent_jobdump%2Fjobs/_design/JobDump/_show/jobSummary/500188
Task: /pdmvserv_task_B2G-RunIISummer16NanoAOD-01730__v1_T_180228_041828_870/B2G-RunIISummer16NanoAOD-01730_0/LogCollectForB2G-RunIISummer16NanoAOD-01730_0
http://vocms0253.cern.ch:5984/wmagent_jobdump%2Fjobs/_design/JobDump/_show/jobSummary/486630
I will investigate

Seangchan, for the case you looked up, that processing job ran at T1_US_FNAL, so the merge and logCollect jobs are waiting to run there. Problem is that FNAL has more jobs pending than the number of slots
cmst1@vocms0253:/data/srv/wmagent/current $ $manage execute-agent wmagent-resource-control --site-name=T1_US_FNAL -p
T1_US_FNAL - 1843 running, 5012 pending, 18836 running slots total, 3531 pending slots total, Site is Normal:
...

that's why even these very high priority jobs haven't been submitted to condor yet.
Question is, why we have exceeded the site pending slots? Possible because the agent pulled higher priority work. I'm not sure whether late binding can contribute to it as well.
Coming back to the question whether we can track these no run/lumi information. I cooked this query and ran it on vocms0253, unless I made a mistake in the query, there are NO problematic files (missing the run/lumi map).
SQL> SELECT DISTINCT avail.fileid
FROM wmbs_sub_files_available avail
  INNER JOIN wmbs_file_details details
    ON avail.fileid = details.id
  LEFT JOIN wmbs_file_runlumi_map map
    ON avail.fileid = map.fileid
WHERE
  map.fileid IS NULL AND
  details.lfn not like '%logArchive.tar.gz%';  2    3    4    5    6    7    8    9

no rows selected

BTW, I've just checked all the other agents in drain and there are no cases of files without run/lumi map. Which makes it very unlikely to be a bug in the WMCore code btw...

apologies for dumping an unrelated set of workflow in the issue. good if no other such cases are in no other agents (did you check also the running ones?)

Yep, it's all good!

Thanks Alan for checking this one but don't work while you are on vacation.

that's why even these very high priority jobs haven't been submitted to condor yet.
Question is, why we have exceeded the site pending slots? Possible because the agent pulled higher priority work. I'm not sure whether late binding can contribute to it as well.

It seems there are a lot of higher priority jobs running on FNAL but I haven't specifically checked for the merge job. Anyway, if it is the real problem, I will create the separate issue.


Update CODEOWNERS

The test failure is unrelated to this PR: https://input-output-rnd.slack.com/archives/C029GMYP1B7/p1726561570609479?thread_ts=1725441419.831849&cid=C029GMYP1B7


⚠ Bofrost.lu has degraded performance

In 916445c, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1589 ms

Resolved: Bofrost.lu performance has improved in 94bd40c after 11 minutes.


SW-18561 - Fixed too many properties crash Elasticsearch populate

Contribute Guidellines?! Sorry - no time for this. Take it or not.

Rules are for everyone. This is how communities work. Feel free to create a new PR which fits our contribution guidelines.
https://developers.shopware.com/contributing/contribution-guideline/


Optimize the antrea client wait and retry mechanism

Add new WaitForAntreaClientErrNil func to optimize the antrea client wait and and retry mechanism.
Signed-off-by: Wu zhengdong<EMAIL_ADDRESS>

/test-all

Codecov Report

Merging #3194 (a8e7b4c) into main (2ee6ad1) will decrease coverage by 30.05%.
The diff coverage is 81.81%.

@@             Coverage Diff             @@
##             main    #3194       +/-   ##
===========================================
- Coverage   60.59%   30.53%   -30.06%
===========================================
  Files         297      291        -6
  Lines       25434    24585      -849
===========================================
- Hits        15411     7508     -7903
- Misses       8345    15948     +7603
+ Partials     1678     1129      -549

Flag
Coverage Δ

kind-e2e-tests
30.53% <81.81%> (-16.44%)
:arrow_down:

unit-tests
?

Flags with carried forward coverage won't be shown. Click here to find out more.


KafkaSource - Change Topics to Topic

Closes #1210

We discussed this with Fran - supporting multiple topics in a single source makes it hard to identify the source of the messages, so this PR makes "1 source = 1 topic" equation true. LGTM.
@odacremolbap could you also lgtm this as it brings incompatible change to current KafkaSource.


[Android] Refine the default values of mUseWideViewport and mLoadWith…

…OverviewMode
XWalk enable the wide viewport quirk in 2e9640b and implemented corresponding APIs
in ebfc438, and the default values of mUseWideViewport and mLoadWithOverviewMode were
set to false to keep the same with Android WebView.
However, XWalk has the different usage in H5 game development, it no need to require
developer set those valuse via APIs, so set the default values to true will have benefit
and convenience.
BUG=XWALK-6995
BUG=XWALK-6963
BUG=XWALK-7051

@lincsoon @wuhengzhi please help to review, thanks


add OperatorGroup

So far the reconcile loop only looks at the namespace selector and writes the results to the status and an annotation on each deployment.

The failing images test will pass once https://github.com/operator-framework/operator-lifecycle-manager/pull/489 is merged.

/test images

@ecordell I'm working on rebasing next, but I just pushed the simple e2e and associated fixes.

@ecordell Please review when you get a chance. Although it's probably best to review holistically, I tried to make each commit reviewable by itself.

A few concerns that might need special attention:

rbac - luckily the rules can just be fixed as needed
out of sync issues - there's bound to be some type of problem with non-first edits
CSV copying - currently it copies CSVs no matter the phase

/retest

/retest
currently timing out on yum update failures

/retest
Looks like CI operator is having a bad day:
panic: runtime error: index out of range

goroutine 135 [running]:
github.com/openshift/ci-operator/pkg/steps.waitForPodCompletion(0x1285700, 0xc4204b55c0, 0xc4204ac330, 0xe, 0x1275cc0, 0xc4202aa2d0, 0x7ffe49a0613e, 0xf, 0x0, 0xc420859520)
	/go/src/github.com/openshift/ci-operator/pkg/steps/template.go:572 +0x3f7
github.com/openshift/ci-operator/pkg/steps.(*podStep).Run(0xc4205aa1c0, 0x1275a40, 0xc4200517c0, 0x0, 0x5, 0xc42036a400)
	/go/src/github.com/openshift/ci-operator/pkg/steps/test.go:120 +0xc4f
github.com/openshift/ci-operator/pkg/steps/release.(*assembleReleaseStep).Run(0xc4204315e0, 0x1275a40, 0xc4200517c0, 0xbeef49f166006400, 0x65f0f3ff12, 0x1974400)
	/go/src/github.com/openshift/ci-operator/pkg/steps/release/create_release.go:83 +0x773
github.com/openshift/ci-operator/pkg/steps.runStep(0x1275a40, 0xc4200517c0, 0xc420213140, 0xc420322f00, 0x0)
	/go/src/github.com/openshift/ci-operator/pkg/steps/run.go:105 +0x92
created by github.com/openshift/ci-operator/pkg/steps.Run
	/go/src/github.com/openshift/ci-operator/pkg/steps/run.go:71 +0x3a1```

/test e2e-aws-olm

/approve
/lgtm

/test e2e-aws-olm


gitgraph.lua:987: assertion failed!

Thanks for making this plugin!
I wanted to report that I get the following error when I try to clone https://github.com/ratatui-org/ratatui and run gitgraph:
E5108: Error executing lua: .../.local/share/nvim/lazy/gitgraph.nvim/lua/gitgraph.lua:987: assertion failed!
stack traceback:
        [C]: in function 'assert'
        .../.local/share/nvim/lazy/gitgraph.nvim/lua/gitgraph.lua:987: in function 'commit_cell_symb'
        .../.local/share/nvim/lazy/gitgraph.nvim/lua/gitgraph.lua:1012: in function 'row_to_str'
        .../.local/share/nvim/lazy/gitgraph.nvim/lua/gitgraph.lua:1150: in function 'gitgraph'
        .../.local/share/nvim/lazy/gitgraph.nvim/lua/gitgraph.lua:1851: in function 'draw'
        /Users/USERNAME/.config/nvim/lua/kd/plugins/git-graph.lua:29: in function </Users/USERNAME/.config/nvim/lua/kd/plugins/git-graph.lua:28>
Press ENTER or type command to continue

Same here for me in ubuntu

Thank you.
Will try to reproduce this and get to the bottom of it.

Was able to reproduce this, btw that repo looks amazing :) <3 Rust

This is an interesting bug, it seems related to a branch called (origini/images), ... gitgraph tries to render this branch, but perhaps this branch has no common ancestors with the other branches 🤔
With git log --graph --oneline --all, I am unable to find the branch origin/images ...

Oh cool, I think I figured out what it is.
Commit 6754f92 has no parents, so has no ancestors in common with the rest of the repo. I just never considered handling this case. I'll probably have a fix in today. BTW git log --graph --oneline --dateorder shows this, but git log --graph --oneline  show ssomething quite confusing.

So at least gitgraph is failing rather than trying to show something confusing :)
Will be great to have this fixed though! 🎉
Diving deeper now

Should now be fixed, please git it a try 🤞
Thanks again for reporting this, this is what makes open source so great!

Yay! Thanks for the quick fix! Works perfectly for me now!


flush() call doesn't actually flush, but waits for the next automatic flush at configured interval instead?

Hello,
I'm experiencing weird behavior of setGroup and flush methods of Amplitude NodeJS client in case when batching is enabled (useBatch: true).
Expected Behavior
I expect one of the following:

setGroup method sends request and resolves returned promise without any delays
Calling flush after setGroup sends all the requests and resolves all pending promises, including promise from setGroup

Current Behavior
The setGroup method waits until flush interval is over and only then it resolves the promise. The timeout is 10000ms by default (managed by flushIntervalMillis configuration option).
Manual calling of flush() doesn't help either. I've tried the following, no success:

const promise = ampli.setGroup(...);  await Promise.all([ampli.flush(), promise]);
const promise = ampli.setGroup(...);  ampli.flush();  await promise;

I digged into amplitude code and found this line - it's being reached when context.timeout is 0, and this is hardcoded when invoking method via execute() (see here).
What I'm trying to do here is to ensure the group is assigned in the amplitude before any other events both in service backend and frontend are sent. Thus, I need to wait until setGroup request is executed, and it might take up to 10 seconds which is not a satisfactory delay for a backend entrypoint. I've decreased flushIntervalMillis option to 1000ms for now and it helped (well, sort of), but it doesn't look like a proper solution.
Possible Solution
Please confirm if this is a bug or it's an expected behavior.
Environment

JS SDK Version: "@amplitude/analytics-node": "^1.0.1", actual installed version is 1.0.1. I'm pretty sure all behavior described above exists in latest version, according to what I can see in current master branch.
Installation Method: yarn
Browser and Version: Node 18.19.0

Hi @ctizen, setGroup() will just send an identify event. The backend will process events based on the order of receiving. So you don't need to await after setGroup(). The SDK will make sure the identify call of setGroup() is before events tracked after it.


emit additional metrics about invoker ContainerPool

Peeling off some metrics changes from a larger PR #4326
This PR adds additional metrics to emit from invoker ContainerPool regarding:

runbuffer size and memory footprint
active pool size and memory footprint

Description

Related issue and scope

[ ] I opened an issue to propose and discuss this change (#????)

My changes affect the following components

[ ] API
[ ] Controller
[ ] Message Bus (e.g., Kafka)
[ ] Loadbalancer
[x] Invoker
[ ] Intrinsic actions (e.g., sequences, conductors)
[ ] Data stores (e.g., CouchDB)
[ ] Tests
[ ] Deployment
[ ] CLI
[ ] General tooling
[ ] Documentation

Types of changes

[ ] Bug fix (generally a non-breaking change which closes an issue).
[x] Enhancement or new feature (adds new functionality).
[ ] Breaking change (a bug fix or enhancement which changes existing behavior).

Checklist:

[x] I signed an Apache CLA.
[x] I reviewed the style guides and followed the recommendations (Travis CI will check :).
[ ] I added tests to cover my changes.
[ ] My changes require further changes to the documentation.
[ ] I updated the documentation where necessary.

Codecov Report

Merging #4625 into master will decrease coverage by 7.79%.
The diff coverage is 100%.

@@            Coverage Diff            @@
##           master    #4625     +/-   ##
=========================================
- Coverage   84.55%   76.76%   -7.8%
=========================================
  Files         183      189      +6
  Lines        8308     8716    +408
  Branches      571      615     +44
=========================================
- Hits         7025     6691    -334
- Misses       1283     2025    +742

Impacted Files
Coverage Δ

...in/scala/org/apache/openwhisk/common/Logging.scala
70.32% <100%> (-2.66%)
:arrow_down:

...e/openwhisk/core/containerpool/ContainerPool.scala
95.97% <100%> (+0.51%)
:arrow_up:

...core/database/cosmosdb/RxObservableImplicits.scala
0% <0%> (-100%)
:arrow_down:

...ore/database/cosmosdb/cache/CacheInvalidator.scala
0% <0%> (-100%)
:arrow_down:

...core/database/cosmosdb/CosmosDBArtifactStore.scala
0% <0%> (-95.89%)
:arrow_down:

...tabase/cosmosdb/cache/CacheInvalidatorConfig.scala
0% <0%> (-94.74%)
:arrow_down:

...sk/core/database/cosmosdb/CosmosDBViewMapper.scala
0% <0%> (-92.6%)
:arrow_down:

...e/database/cosmosdb/cache/ChangeFeedListener.scala
0% <0%> (-86.67%)
:arrow_down:

...e/database/cosmosdb/cache/KafkaEventProducer.scala
0% <0%> (-76.48%)
:arrow_down:

...whisk/core/database/cosmosdb/CosmosDBSupport.scala
0% <0%> (-74.08%)
:arrow_down:

... and 44 more

Continue to review full report at Codecov.

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


Group descendant teams

Fixes #189
I'm just not sure what to display when the filter is set to 'Direct Children' and 'Teams'.
For now, an empty list is returned

I'm just not sure what to display when the filter is set to 'Direct Children' and 'Teams'.
For now, an empty list is returned

Just the group children with type "Team". I don't understand what is the problem.


Prompt+Confirmation before formatting the HDD

Not sure because the drive has been formatted by raspiblitz.. possibly I accidentally formatted it to NTFS or something when I set it up initially to sync blockchain on my desktop. it was a while ago now.
Any recommended file recovery tools for Ubuntu? I'm hoping the blitz hasn't written over the data and corrupted it.. there is a chance it has because it formatted the drive and added ssh folder + raspiblitz.conf file.
Maybe for future release it would be good to have a notification saying blockchain not found and need a Y/N confirmation before it formats the drive in case we put the chain in the wrong folder or set the wrong filesystem on the drive.

Prob not worth the time/trouble to try to fix it.  I would download the torrent again if you don't already have it and sync again.
I agree it should prompt.  @rootzoll see : https://github.com/rootzoll/raspiblitz/blob/e38c48e54536a6233a0062b99858d43685635d80/home.admin/10setupBlitz.sh#L256  It prob should not reformat the drive without prompting the user to confirm.
https://github.com/rootzoll/raspiblitz/blob/4ccdd8bb22a1a0805588cf994735275871a51d48/home.admin/30initHDD.sh#L78
Originally posted by @fluidvoice in https://github.com/rootzoll/raspiblitz/issues/434#issuecomment-473569138

TODO: The script should check the HDD and ...

when its the wrong format .. ask to reformat in ext4 or cancel
when its ext4 and not empty .. info on how to prepare a correct Blockchain HDD and ask if format or cancel

I would like to ask for an extra confirmation as well.
I just accidentally formatted a HDD with a complete blockchain downloaded on it, and an LND wallet on it as well.

Starting with v1.2 the RaspiBlitz will ask before formatting the HDD.
Closing this issue.


OIDC - OpenId Connect authentication fail for application behind NGINX

Checklist

Have you pulled and found the error with jc21/nginx-proxy-manager:latest docker image?
Yes
Are you sure you're not using someone else's docker image?
Yes
Have you searched for similar issues (both open and closed)?
Yes

Describe the bug
NGINX Proxy Manager is facing internet. I tried to use OpenID Connect for authentication behind it.
Internet --- NGINX proxy manager --- APISIX with openid-connect --- Web app
Keycloak is used for OIDC server.
I tried to use APISIX to manage the authentication (behind NGINX Proxy Manager) without success.
https://github.com/apache/apisix/issues/10149
We get to the conclusions that was a cookies issue with cross-site access.
But, keycloak cookie is set properly and I didn't find anywhere in NGINX Proxy Manager:
'set $session_cookie_samesite Strict;'
If I bypass NGINX Proxy Manager with ngrok tunnelling. Callback authentication works.
So, I tried with a small nodejs app directly behind the NGINX Proxy Manager to simplify the problem.
I get the same issue. Callback to the app can not be decoded properly, an information is missing.
Nginx Proxy Manager Version
v2.10.4
To Reproduce
This is the nodejs code:
const express = require('express')
const { auth } = require('express-openid-connect');
require('dotenv').config()

const path = require('path')

const app = express()
const port = process.env.NODEPORT

app.use(auth({
    issuerBaseURL: process.env.ISSUER_BASE_URL,
    clientID: process.env.CLIENT_ID,
    clientSecret: process.env.CLIENT_SECRET,
    baseURL: process.env.BASE_URL,
    secret: process.env.SECRET,
    authorizationParams: {
        response_type: 'code',
        audience: process.env.BASE_URL,
        scope: 'openid',
    },
    idpLogout: true,
    auth0Logout: true
  }))

app.use(express.static(path.join(__dirname, 'htdocs')))

app.get('/profile', async (req, res) => {
    res.send(`hello ${JSON.stringify(req.oidc.user, null, 2)}`);
});

app.listen(port, () => {
  console.log(`Server listening on port ${port}`)
})

go to app external URL <BASE_URL>
it is redirect to login page on keycloak
when user credential are accepted, callback to:
<BASE_URL>/callback?state=eyJyZXR1cm5UbyI6Ii8ifQ&session_state=a7fd9407-e2c3-474c-8625-117276ed6bc0&code=a10612fc-664f-41f5-aaa3-9900d38433eb.a7fd9407-e2c3-474c-8625-117276fc6bc0.ab703194-4f67-4d9e-8a26-eff7867dc310
and page:

502 Bad Gateway
_______________
openresty

Operating System
Linux Debian

I am also seeing this issue with AzureAd auth
Setup
Public -> NPM -> Blazor Docker App
502 bad gateway response when auth call back is returned at /signin-oidc
Some research suggests default payload size is 4kb and may need to be bumped up.


Publish a separate multicontainer project

Ship a separate project that allows users to try out multi container easily & safely, with similar installation/usage to the current CLI.

Separate package name
Separate binary name (mresin)
Uses the multicontainer backend by default
Ok if it clobbers settings/token

Done a while ago: https://www.npmjs.com/package/resin-cli-mc


Bug: Added the required challenge text but nothing happens it says i havent done it.

Challenge Change the Font Size of an Element has an issue.
User Agent is: Mozilla/5.0 (Windows NT 6.3; WOW64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/54.0.2840.99 Safari/537.36.
Please describe how to reproduce this issue, and include links to screenshots if possible.
 Please consider asking for help in the chat room by clicking the Help button on the challenge.

An error? I think I know what it could be but I've been coding for over a
year this is just a side project essentially to improve my skills and get
to know clients.
On Tue, Nov 29, 2016 at 5:00 PM Dylan<EMAIL_ADDRESS>wrote:

@jdc20181 https://github.com/jdc20181 you have an error in your code.
Please consider asking for help in the chat room by clicking the Help
button on the challenge.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/11866#issuecomment-263712957,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMphBgEH6dXGfmbJK6yWhn3I4zWLEu-uks5rDKBugaJpZM4K_hQz
.

Your CSS attribute does not have a ; at the end. While not strictly required for a single CSS attribute, the tests enforce it here because it's an assumed best practice and beginners should get in the habit of using semicolons at the end of CSS attributes as there are usually more than one being applied.
We try to refrain from offering code help here in Github issues as that is the purpose of the chat room. Happy coding.

I figured it out asking for help.
It wasn't any error in my CSS. I even checked for errors.
Further more- it was the text in the  element that caused the issue but
no where on my screen said what o was told to place there.
Thanks
On Tue, Nov 29, 2016 at 10:21 PM Dylan<EMAIL_ADDRESS>wrote:

Your CSS attribute does not have a ; at the end. While not strictly
required for a single CSS attribute, the tests enforce it here because it's
an assumed best practice and beginners should get in the habit of using
semicolons at the end of CSS attributes as there are usually more than one
being applied.
We try to refrain from offering code help here in Github issues as that is
the purpose of the chat room. Happy coding.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/11866#issuecomment-263772188,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMphBr8XSYmq4l-GtAHDvfITGy0Kgo_9ks5rDOurgaJpZM4K_hQz
.

Not a beginner added it up and with what I said earlier I have around 2.5
years coding experience under my belt.
About 1.5 years web and 2.5 years Non web.
On Tue, Nov 29, 2016 at 10:22 PM jeffrey crowder<EMAIL_ADDRESS>wrote:

I figured it out asking for help.
It wasn't any error in my CSS. I even checked for errors.
Further more- it was the text in the  element that caused the issue but
no where on my screen said what o was told to place there.
Thanks
On Tue, Nov 29, 2016 at 10:21 PM Dylan<EMAIL_ADDRESS>wrote:
Your CSS attribute does not have a ; at the end. While not strictly
required for a single CSS attribute, the tests enforce it here because it's
an assumed best practice and beginners should get in the habit of using
semicolons at the end of CSS attributes as there are usually more than one
being applied.
We try to refrain from offering code help here in Github issues as that is
the purpose of the chat room. Happy coding.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/FreeCodeCamp/FreeCodeCamp/issues/11866#issuecomment-263772188,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMphBr8XSYmq4l-GtAHDvfITGy0Kgo_9ks5rDOurgaJpZM4K_hQz
.


[Script]:

Aim
Small code for getting weather of a location (using openweather api)
Details
Anybody can use this code as reference for using open weather api
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

Already Present in the Repo


info on table borders

Some info and code is provided on table borders and adjusting them.

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

Looks good @allenpbiju, thanks for contributing!


Virtual Machines Reorg

Docs PR Template
Why this should be merged
The current precompile generation tutorial is overwhelming. Furthermore, some concepts, such as the AllowList are explained multiple times with a different degree of detail.
How this works
Split up the precompile tutorial and create a better structure. Furthermore, I moved some to a single place that can be referenced
How these changes were tested
Workflow checks

[] ran vale /docs/file/path on all changed .md files to ensure all grammar rules pass
[] ran markdownlint /docs/file/path on all changed .md files to ensure all linting rules pass
[] completed the above two checks and all additional rules outlined in style-checker-notes.md to
ensure all checks pass
[] if a doc was removed/deleted, the path to that doc must be redirected to a valid URL via the
_redirects file
[] all files that were moved from their current directory to a new path have had their paths
redirected via the _redirects file
[] _redirects were manually verified with the cloudflare preview link
[] sidebars.json reflects all changes made to file path

this whole PR is 🔥 🔥 🔥

Please restructure the folder structure and use the naming convention present in the docs revamp plan

Guidelines for folder structure are outlined in the Docs Revamp Plan (link pinned to #avalanche-dev-docs channel) more info here: https://github.com/ava-labs/avalanche-docs/pull/1418#issuecomment-1636130200


Works only while Online/ No Offline training

I managed to train 2 people yesterday and today since my Internet was out I tried training another character, but it said it could not connect to an address.
Is this extension not possible to be executed offline, AFTER  the initial setup when you already have gotten all the models required?

Loading weights [e9d3cedc4b] from C:\Users\freel\AppData\Roaming\Visions of Chaos\Examples\MachineLearning\Text To Image\stable-diffusion-webui\models\Stable-diffusion\realisticVisionV40_v40VAE.safetensors
Applying attention optimization: xformers... done.
Traceback (most recent call last):
File "C:\Users\freel\AppData\Roaming\Visions of Chaos\Examples\MachineLearning\Text To Image\stable-diffusion-webui\venv\lib\site-packages\urllib3\connection.py", line 203, in _new_conn
sock = connection.create_connection(
File "C:\Users\freel\AppData\Roaming\Visions of Chaos\Examples\MachineLearning\Text To Image\stable-diffusion-webui\venv\lib\site-packages\urllib3\util\connection.py", line 60, in create_connection
for res in socket.getaddrinfo(host, port, family, socket.SOCK_STREAM):
File "C:\Users\freel\AppData\Local\Programs\Python\Python310\lib\socket.py", line 955, in getaddrinfo
for res in _socket.getaddrinfo(host, port, family, type, proto, flags):
socket.gaierror: [Errno 11001] getaddrinfo failed
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "C:\Users\freel\AppData\Roaming\Visions of Chaos\Examples\MachineLearning\Text To Image\stable-diffusion-webui\venv\lib\site-packages\urllib3\connectionpool.py", line 790, in urlopen
response = self._make_request(
File "C:\Users\freel\AppData\Roaming\Visions of Chaos\Examples\MachineLearning\Text To Image\stable-diffusion-webui\venv\lib\site-packages\urllib3\connectionpool.py", line 491, in _make_request
raise new_e
File "C:\Users\freel\AppData\Roaming\Visions of Chaos\Examples\MachineLearning\Text To Image\stable-diffusion-webui\venv\lib\site-packages\urllib3\connectionpool.py", line 467, in _make_request
self._validate_conn(conn)
File "C:\Users\freel\AppData\Roaming\Visions of Chaos\Examples\MachineLearning\Text To Image\stable-diffusion-webui\venv\lib\site-packages\urllib3\connectionpool.py", line 1096, in _validate_conn
conn.connect()
File "C:\Users\freel\AppData\Roaming\Visions of Chaos\Examples\MachineLearning\Text To Image\stable-diffusion-webui\venv\lib\site-packages\urllib3\connection.py", line 611, in connect
self.sock = sock = self._new_conn()
File "C:\Users\freel\AppData\Roaming\Visions of Chaos\Examples\MachineLearning\Text To Image\stable-diffusion-webui\venv\lib\site-packages\urllib3\connection.py", line 210, in _new_conn
raise NameResolutionError(self.host, self, e) from e
urllib3.exceptions.NameResolutionError: <urllib3.connection.HTTPSConnection object at 0x000002C9104033D0>: Failed to resolve 'pai-aigc-photog.oss-cn-hangzhou.aliyuncs.com' ([Errno 11001] getaddrinfo failed)
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "C:\Users\freel\AppData\Roaming\Visions of Chaos\Examples\MachineLearning\Text To Image\stable-diffusion-webui\venv\lib\site-packages\requests\adapters.py", line 486, in send
resp = conn.urlopen(
File "C:\Users\freel\AppData\Roaming\Visions of Chaos\Examples\MachineLearning\Text To Image\stable-diffusion-webui\venv\lib\site-packages\urllib3\connectionpool.py", line 844, in urlopen
retries = retries.increment(
File "C:\Users\freel\AppData\Roaming\Visions of Chaos\Examples\MachineLearning\Text To Image\stable-diffusion-webui\venv\lib\site-packages\urllib3\util\retry.py", line 515, in increment
raise MaxRetryError(_pool, url, reason) from reason  # type: ignore[arg-type]
urllib3.exceptions.MaxRetryError: HTTPSConnectionPool(host='pai-aigc-photog.oss-cn-hangzhou.aliyuncs.com', port=443): Max retries exceeded with url: /webui/ChilloutMix-ni-fp16.safetensors (Caused by NameResolutionError("<urllib3.connection.HTTPSConnection object at 0x000002C9104033D0>: Failed to resolve 'pai-aigc-photog.oss-cn-hangzhou.aliyuncs.com' ([Errno 11001] getaddrinfo failed)"))
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "C:\Users\freel\AppData\Roaming\Visions of Chaos\Examples\MachineLearning\Text To Image\stable-diffusion-webui\venv\lib\site-packages\gradio\routes.py", line 488, in run_predict
output = await app.get_blocks().process_api(
File "C:\Users\freel\AppData\Roaming\Visions of Chaos\Examples\MachineLearning\Text To Image\stable-diffusion-webui\venv\lib\site-packages\gradio\blocks.py", line 1431, in process_api
result = await self.call_function(
File "C:\Users\freel\AppData\Roaming\Visions of Chaos\Examples\MachineLearning\Text To Image\stable-diffusion-webui\venv\lib\site-packages\gradio\blocks.py", line 1103, in call_function
prediction = await anyio.to_thread.run_sync(
File "C:\Users\freel\AppData\Roaming\Visions of Chaos\Examples\MachineLearning\Text To Image\stable-diffusion-webui\venv\lib\site-packages\anyio\to_thread.py", line 33, in run_sync
return await get_asynclib().run_sync_in_worker_thread(
File "C:\Users\freel\AppData\Roaming\Visions of Chaos\Examples\MachineLearning\Text To Image\stable-diffusion-webui\venv\lib\site-packages\anyio_backends_asyncio.py", line 877, in run_sync_in_worker_thread
return await future
File "C:\Users\freel\AppData\Roaming\Visions of Chaos\Examples\MachineLearning\Text To Image\stable-diffusion-webui\venv\lib\site-packages\anyio_backends_asyncio.py", line 807, in run
result = context.run(func, *args)
File "C:\Users\freel\AppData\Roaming\Visions of Chaos\Examples\MachineLearning\Text To Image\stable-diffusion-webui\venv\lib\site-packages\gradio\utils.py", line 707, in wrapper
response = f(*args, **kwargs)
File "C:\Users\freel\AppData\Local\Programs\Python\Python310\lib\contextlib.py", line 79, in inner
return func(*args, **kwds)
File "C:\Users\freel\AppData\Roaming\Visions of Chaos\Examples\MachineLearning\Text To Image\stable-diffusion-webui\extensions\sd-webui-EasyPhoto\scripts\easyphoto_train.py", line 90, in easyphoto_train_forward
check_files_exists_and_download(check_hash.get("base", True), "base")
File "C:\Users\freel\AppData\Roaming\Visions of Chaos\Examples\MachineLearning\Text To Image\stable-diffusion-webui\extensions\sd-webui-EasyPhoto\scripts\easyphoto_utils\common_utils.py", line 494, in check_files_exists_and_download
if os.path.exists(_filename) and compare_hash_link_file(url, _filename):
File "C:\Users\freel\AppData\Roaming\Visions of Chaos\Examples\MachineLearning\Text To Image\stable-diffusion-webui\extensions\sd-webui-EasyPhoto\scripts\easyphoto_utils\common_utils.py", line 507, in compare_hash_link_file
r = requests.head(url)
File "C:\Users\freel\AppData\Roaming\Visions of Chaos\Examples\MachineLearning\Text To Image\stable-diffusion-webui\venv\lib\site-packages\requests\api.py", line 100, in head
return request("head", url, **kwargs)
File "C:\Users\freel\AppData\Roaming\Visions of Chaos\Examples\MachineLearning\Text To Image\stable-diffusion-webui\venv\lib\site-packages\requests\api.py", line 59, in request
return session.request(method=method, url=url, **kwargs)
File "C:\Users\freel\AppData\Roaming\Visions of Chaos\Examples\MachineLearning\Text To Image\stable-diffusion-webui\venv\lib\site-packages\requests\sessions.py", line 589, in request
resp = self.send(prep, **send_kwargs)
File "C:\Users\freel\AppData\Roaming\Visions of Chaos\Examples\MachineLearning\Text To Image\stable-diffusion-webui\venv\lib\site-packages\requests\sessions.py", line 703, in send
r = adapter.send(request, **kwargs)
File "C:\Users\freel\AppData\Roaming\Visions of Chaos\Examples\MachineLearning\Text To Image\stable-diffusion-webui\venv\lib\site-packages\requests\adapters.py", line 519, in send
raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPSConnectionPool(host='pai-aigc-photog.oss-cn-hangzhou.aliyuncs.com', port=443): Max retries exceeded with url: /webui/ChilloutMix-ni-fp16.safetensors (Caused by NameResolutionError("<urllib3.connection.HTTPSConnection object at 0x000002C9104033D0>: Failed to resolve 'pai-aigc-photog.oss-cn-hangzhou.aliyuncs.com' ([Errno 11001] getaddrinfo failed)"))

webui\venv\lib\site-packages\requests\adapters.py", line 519, in send
raise ConnectionError(e, request=request)
requests.exceptions.ConnectionError: HTTPSConnectionPool(host='pai-aigc-photog.oss-cn-hangzhou.aliyuncs.com', port=443): Max retries exceeded with url: /webui/ChilloutMix-ni-fp16.safetensors (Caused by NameResolutionError("<urllib3.connection.HTTPSConnection object at 0x000002C9104033D0>: Failed to resolve 'pai-aigc-photog.oss-cn-hangzhou.aliyuncs.com' ([Errno 11001] getaddrinfo failed)"))


Updates

Is there a permissions problem @jaywonchung? You should be able to merge after approving otherwise.

I've given you maintainer access now. The default was read.

Sorry, I was using my previous email for notification (it is separated from primary email configuration) so I didn't see it. Please let me check my PR permission later. @mosharaf


feat: multiple queues

This is my first version of multi-queue implementation, and the support of taskiq project is also needed to realize multi-queue.
#16

It seems that you only need to increase the queue_name parameter in BrokerMessage.
https://github.com/vvanglro/taskiq-aio-pika/blob/feat/multiple_queues/taskiq_aio_pika/broker.py#L223-L229