SQL Server Connection Resiliency

Hi, I have configured my DbContext to use SQL Server Connection Resiliency feature in EF Core.
When using an UAT server for testing (only using 1 SQL Server 2016 instance), this feature works fine, without throwing any error regarding this feature. But when using production servers (yes, we have 2 SQL Server 2016 instances grouped by SQL Server AlwaysOn feature), there are so many error messages regarding this feature.
Logs
Here are one of our log (cannot post the exception message below what I have posted since it will tell my project's class name):
System.InvalidOperationException: The configured execution strategy 'SqlServerRetryingExecutionStrategy' does not support user initiated transactions. Use the execution strategy returned by 'DbContext.Database.CreateExecutionStrategy()' to execute all the operations in the transaction as a retriable unit.
   at Microsoft.EntityFrameworkCore.Storage.ExecutionStrategy.OnFirstExecution()
   at Microsoft.EntityFrameworkCore.Storage.ExecutionStrategy.Execute[TState,TResult](TState state, Func`3 operation, Func`3 verifySucceeded)
   at Microsoft.EntityFrameworkCore.Query.Internal.QueryingEnumerable`1.Enumerator.MoveNext()
   at System.Linq.Enumerable.TryGetFirst[TSource](IEnumerable`1 source, Boolean& found)
   at lambda_method(Closure , QueryContext )
   at Microsoft.EntityFrameworkCore.Query.Internal.QueryCompiler.<>c__DisplayClass17_1`1.<CompileQueryCore>b__0(QueryContext qc)
   at System.Linq.Queryable.FirstOrDefault[TSource](IQueryable`1 source, Expression`1 predicate)

Codes
My AddDbContextPool() configurations are like this
services.AddDbContextPool<MyContext>(options =>
{
    options.UseSqlServer(Configuration.GetConnectionString("MyDb"), strategy =>
    {
        strategy.EnableRetryOnFailure(
            maxRetryCount: 5,
            maxRetryDelay: TimeSpan.FromSeconds(30),
            errorNumbersToAdd: null
            );
    });
});

Then, my CreateExecutionStrategy() codes are like this
await MyContext.Database.CreateExecutionStrategy().ExecuteAsync(async () =>
{
    using (var transContext = await this.MyContext.Database.BeginTransactionAsync())
    {
        // Ma codes....
        // First SaveChangesAsync() call.
        await this.MyContext.SaveChangesAsync();

        // Second SaveChangesAsync() call.
        await this.MyContext.SaveChangesAsync();

        // Transaction commit.
        transContext.Commit();
    }
}

Project's frameworks and dependencies
EF Core version: 2.0.1
Database Provider: Microsoft.EntityFrameworkCore.SqlServer
Operating system: Windows Server 2012 R2
IDE: Visual Studio 2017
I've searched on the internet, but haven't found any similar situations like what I have now.
Thanks in advance.

@Tieantono Trying just the code you posted does not produce the exception. Could you post a small repro project?
Also is there a particular reason you are not using the latest EF Core version?

@AndriySvyryd Ok, I'll create a small repro project.
Yes, the problem was not found when using personal computer and UAT server (using Windows Server 2012 R2 too).
We gladly like to use the newest version of EF Core! But sadly, our project is huge, and we don't have time to upgrade our dependencies's version, and we are in production phase right now.
We have problem when we want to save a blob file into our SQL Server database (with VARBINARY(MAX) data type), and we suspect it might be coming from this exception.

@Tieantono Just to be sure you are aware, EF Core (and all of .NET Core 2.0) is out of support. It's not getting any updates, even for security issues.

@AndriySvyryd I'm sorry, but I haven't made the repo yet, but considering the current situation of our project, which is better now after we replaced AddDbContextPool() to AddDbContext(), we know the what are the reasons:

When using AddDbContextPool(), we still curious about its behaviour. In my codes, there are some DataReader (SELECT query) that are in the BeginTransaction() scope. Not only that, some of our transaction codes have a very long processes

Now it's time to refactor all codes to only run SaveChanges() once only if it possible. Previously we have to using BeginTransaction() since there are some header and detail tables that must be inserted simultaneously, but in order to do that, the header ID must exist first. We just found out recently that we don't need to save the header table first now in EF Core 2.0.
@ajcvickers Oh I didn't know about that, but I've prepared migration branch in order to update our frameworks versions

EF Team Triage: Closing this issue as the requested additional details have not been provided and we have been unable to reproduce it.
BTW this is a canned response and may have info or details that do not directly apply to this particular issue. While we'd like to spend the time to uniquely address every incoming issue, we get a lot traffic on the EF projects and that is not practical. To ensure we maximize the time we have to work on fixing bugs, implementing new features, etc. we use canned responses for common triage decisions.


Log Levels

Is there a way to set the log level to just Error.
I have a bunch of Deprecations (waiting for libraries) and would like
to exclude these to rollbar.
rollbar: {
  accessToken: 'MY_CODE',
  enabled: environment !== 'development',
  captureUncaught: true,
  logLevel: 'error',
  payload: {
    environment: environment
  }

I was hoping we could add logLevel but its not used.

Have you tried setting the reportLevel option to 'error'? The Rollbar docs seem to indicate that it controls what level of logging is actually sent to Rollbar, and the default is 'warning', which would report deprecations.

That looks like it has done the trick, thanks.


Implement example of library usage

currently do no see much sense on implementing an example application for DI container. Will reopen or create new if needed


Issue with k8s.io/docs/tutorials/kubernetes-basics/update/update-interactive/

This is a Bug Report
Problem:
Another continuity problem where the ending state of the previous tutorial does not match the starting point of this one. In the last tutorial, we scaled to 4 instances and then back to 2 but at the start of this tutorial, the environment is already configured for 4. Although it doesn't massively affect the results, it will affect what you see in the console compared to what is shown interactively.
Proposed Solution:
Either make this tutorial start with 2 instances, which makes the problem go away, make the last tutorial increase from 2 back to 4 before finishing, add a comment that the exact number of instances is not important or tell the user to scale back out to 4 before continuing (if they haven't already)
Page to Update:
Issue with k8s.io/docs/tutorials/kubernetes-basics/update/update-interactive/

Thanks for the feedback, seems like a good idea to fix it.
/language en

/priority backlog
/kind bug


Make dynamic property assignment in DataRecord safer

Fix all lint errors in elements/
Add | None typing to class values that initialize to None in logical.py
Explicit pass of None in for input_schema on CacheScan

Pushed a small commit to:

Apply a sanity check (to replace our previous setting of kwargs['inputSchema'] = None) to formally ensure that we never initialize CacheScan or BaseScan with a value other than None
Give inputSchema a default setting of None in LogicalOperator. Without this, the call to the BaseScan constructor in the Optimizer would throw an exception. I considered just adding inputSchema=None to that constructor call, but I want to enforce correct behavior where possible.
Heeded the advice of Ruff to remove [] as a default value in function signatures
Made schema a property of the Set class; this has been a long-needed fix (kudos to @ramipellumbi for planting the property seed elsewhere) and is also needed for L142 in optimizer.py
Fixed a fun bug 🐛 where the definition of __setattr__ and __getattr__ in DataRecord created an infinite recursion loop b/c when we initialized self._data = {} in the constructor, it would call __setattr__(self, "_data", {}). This would in turn call __getattr__(self, "_data") (from the else branch of __setattr__) which would then call itself (__getattr__) infinitely many times on the line if name in self._data:
Removed scan_idx from list of class attributes in __setattr__ b/c we really want it to be discarded after the constructor

Pushed a small commit to:
* Apply a sanity check (to replace our previous setting of `kwargs['inputSchema'] = None`) to formally ensure that we never initialize `CacheScan` or `BaseScan` with a value other than `None`

* Give `inputSchema` a default setting of `None` in `LogicalOperator`. Without this, the call to the `BaseScan` constructor in the `Optimizer` would throw an exception. I considered just adding `inputSchema=None` to that constructor call, but I want to enforce correct behavior where possible.

* Heeded the advice of Ruff to remove `[]` as a default value in function signatures

* Made `schema` a property of the `Set` class; this has been a long-needed fix (kudos to @ramipellumbi for planting the `property` seed elsewhere) and is also needed for L142 in `optimizer.py`

* Fixed a fun bug 🐛 where the definition of `__setattr__` and `__getattr__` in `DataRecord` created an infinite recursion loop b/c when we initialized `self._data = {}` in the constructor, it would call `__setattr__(self, "_data", {})`. This would in turn call `__getattr__(self, "_data")` (from the `else` branch of `__setattr__`) which would then call itself (`__getattr__`) infinitely many times on the line `if name in self._data:`

* Removed `scan_idx` from list of class attributes in `__setattr__` b/c we really want it to be discarded after the constructor

Dang, the recursion bug is a goofy mistake -- my bad

Pushed a small commit to:
* Apply a sanity check (to replace our previous setting of `kwargs['inputSchema'] = None`) to formally ensure that we never initialize `CacheScan` or `BaseScan` with a value other than `None`

* Give `inputSchema` a default setting of `None` in `LogicalOperator`. Without this, the call to the `BaseScan` constructor in the `Optimizer` would throw an exception. I considered just adding `inputSchema=None` to that constructor call, but I want to enforce correct behavior where possible.

* Heeded the advice of Ruff to remove `[]` as a default value in function signatures

* Made `schema` a property of the `Set` class; this has been a long-needed fix (kudos to @ramipellumbi for planting the `property` seed elsewhere) and is also needed for L142 in `optimizer.py`

* Fixed a fun bug 🐛 where the definition of `__setattr__` and `__getattr__` in `DataRecord` created an infinite recursion loop b/c when we initialized `self._data = {}` in the constructor, it would call `__setattr__(self, "_data", {})`. This would in turn call `__getattr__(self, "_data")` (from the `else` branch of `__setattr__`) which would then call itself (`__getattr__`) infinitely many times on the line `if name in self._data:`

* Removed `scan_idx` from list of class attributes in `__setattr__` b/c we really want it to be discarded after the constructor

The ruff lint point should be a public PSA. Python default arguments to functions are evaluated WHEN THE FUNCTION IS DEFINED NOT CALLED. So, if you have a mutable default argument like a list or set, that will be shared across all calls. Very bad. E.g.,
def append_to(element, to=[]):
    to.append(element)

my_list  = append_to(12) # [12]
my_other_list = append_to(42) # [12, 42]


Test for Debezium mysql - passing additionalProperties may fail.

fixes https://github.com/apache/camel-quarkus/issues/3488
Problem is caused by the bug in camel - see https://issues.apache.org/jira/browse/CAMEL-17658
It is already fixed in Camel for 3.16.0
This PR adds test among debezium tests to cover reported bug in CQ.

We need to wait for a new 3.16.0-SNAPSHOT until the camel-main branch can build successfully again.
I'll probably merge this manually as there's likely other stuff that will need fixing up.

I merged this manually with a minor change.
If you try to refer to a JSON key like .body("database.connectionTimeZone", is("CET")), then you need to wrap it in single quotes like .body("'database.connectionTimeZone'", is("CET")).
Otherwise RestEasy interprets it as you want to access the object database with key connectionTimeZone.


chore: Upgrade to XGBoost v2.0.2. Fixes #3310

Fixes https://github.com/kserve/kserve/issues/3310. This brings all the improvements in recent XGBoost https://github.com/dmlc/xgboost/releases/tag/v2.0.2

@terrytangyuan Can you help create an issue and link to this PR?

@terrytangyuan Can you help create an issue and link to this PR?

Done.

/lgtm
/approve


Install Mods in PE 1.0b

What is your question?
How to add mods in PE 1.0b?

https://github.com/MobilePorting/FNF-PsychEngine-Mobile/wiki/Installing-Mods

https://github.com/MobilePorting/FNF-PsychEngine-Mobile/wiki/Installing-Mods

Thanks


SKTypeface.FromStream raises exception in Release mode only

Description
Hi, first of all, thx for this amazing library! I've been using it for some time now without any issues up until now that I'm building in release mode.  The code snippet below works perfectly in Debug mode but not Release, where it's raising an exception when calling SKTypeface.FromStream(stream) (stack trace details down below)
Am I missing anything or should I change this approach?
Thanks in advance!
Code
using (var asset = App.assets.Open($"fonts/someFont.otf"))
{
    var fontStream = new MemoryStream();
    asset.CopyTo(fontStream);
    fontStream.Flush();
    fontStream.Position = 0;
    typeface = SKTypeface.FromStream(fontStream);
    if(typeface != null)
    {
        //Do something...
    }
}

Expected Behavior
typeface form returned without exception
Actual Behavior
Exception raised in release mode only.
Basic Information

Version with issue:  1.60.0
IDE:  VS 2017 Enterprise 15.6.6
Platform Target Frameworks: API 25 (Android 7.1)

Android:  Testing on a 6.0 android device.

Target Devices:

Honor 8

Stack trace details
05-15 00:23:48.144 E/mono-rt ( 2747): Stacktrace:
05-15 00:23:48.144 E/mono-rt ( 2747):
05-15 00:23:48.144 E/mono-rt ( 2747):   at  <0xffffffff>
05-15 00:23:48.144 E/mono-rt ( 2747):   at (wrapper managed-to-native) SkiaSharp.SkiaApi.sk_typeface_create_from_stream (intptr,int) <0x0003b>
05-15 00:23:48.144 E/mono-rt ( 2747):   at SkiaSharp.SKTypeface.FromStream (SkiaSharp.SKStreamAsset,int) [0x00018] in <5a812555f55f40c0a29350793a12a8ac>:0
05-15 00:23:48.144 E/mono-rt ( 2747):   at SkiaSharp.SKTypeface.FromStream (System.IO.Stream,int) [0x00019] in <5a812555f55f40c0a29350793a12a8ac>:0

Sorry, ignore that one. I upgraded to SkiaSharp 1.60.0 from 1.56.2, but SkiaSharp.Views and SkiaSharp.Views.Forms were still on 1.56.2.  After upgrading them and do a full clean & rebuild it's working fine now in Release.
Sorry about that! :)


[Feature Request] Dedicated subtitle button

Many remotes controls (real and software) have a dedicated subtitle button that we can use to quickly turn on/off the subtitle as well as select the language.
This is useful because some sections of a movie, we want to just check what they said then turn subtitles off again.
Right now the implementation is not very user friendly
requiring going through a hidden drop down menu then choose subtitle then we can choose  the subtitle language or turn on/off subtitle.
This is extremely cumbersome.

We need a dedicated subtitle button always showing.
It would also be good if we can hide the other unused buttons as well e.g. Play single, play all button,
Ideally we should be able to choose what buttons we want to show/hide e.g. subtitle, audio etc..

I would like to bump this and also ask that a menu is added to get to the parent subtitle menu which lets you select local subtitles (as well as download, sync and disable them). Currently you have to bring up the player's OSD to get to that menu and bringing up that OSD is sometimes very hard (sometimes the middle button does nothing and I have to switch to the TV remote to bring it up).

@bendem Check here
https://github.com/xbmc/Kore/pull/427
Someone implemented it, but one of the main coder of this app decided now to merge the pull request.
Pretty disappointing outcome.
I suggest you post in that thread and reply to that coder to let them know your opinion.

@n123al
Unfortunately the main coder has decided to not merge your code.
Can you add code that allows us to put any button on the main UI, so we can assign it to any function we want?
Can you fork this to your own kore project with the code you made.
Disappointing to see your code not get the merge request approved.


KI Landing Page - Add Video Layer

The current version of the KI Landing page (https://www.klimadao.finance/infinity) has an image placeholder where I video needs to be:

This is meant to be the DeFi that defies climate change video - https://www.youtube.com/watch?v=eRmmDh1ingU&t=2s
Next steps:

[x] Need to add a play button to the current graphic that’s there so users knows to click
[x] Need GH issue to add video layer to page

@Atmosfearful here is the figma design for this, ready to hand off to dev
https://www.figma.com/file/6CUpojwfvdV4R3ACk85JYG/KI-Landing-Page?node-id=109%3A12#241339005

Closed in #676


fix(prometheus): revert prometheus client_golang from from v1.12.2 to v1.11.0

What problem does this PR solve?

Issue Number: close #5494
What is changed and how it works?
revert the prometheus client_golang to v1.11.0
None

/check-issue-triage-complete

/merge


Pipe raw request data in a file

I can't seem to find how to actually pipe data sent from client into a file on the server.
I don't want to use any third party plugin, I try to remain raw here if that's possible.
On the front I send blob data (audio) from the fetch function
const response = await fetch('/api/audio', {
  method: 'POST',
  body: this._blobData,
});

_blobData is just a Blob object. When it's sent it appears to be raw data representing an audio.

How can I pipe this on the server-side into a file ?
I tried to find a property in the context object that may hold/contain the data but I can't find.
Thanks in advance.

Sorry that was a silly question... I just had to use ctx.req.pipe(fileStream) ...


Vite dev and build modes having trouble with webc packaging

NodeJS 19, Vite 3.2.3. I'm using the Vite vanilla scaffold with most demo stuff removed. It has:

  "type": "module", in package.json
 <script type="module" src="/main.js"></script> in index.html
A single line of import { WebC } from "@11ty/webc"; in main.js

The dev server rendering in Chrome gives an exception:
Uncaught TypeError: require_path().posix is undefined
    js index.js:4
    __require2 @11ty_webc.js:16
    js pattern.js:5
    __require2 @11ty_webc.js:16
    js index.js:12
    __require2 @11ty_webc.js:16
    js tasks.js:4
    __require2 @11ty_webc.js:16
    js index.js:2
    __require2 @11ty_webc.js:16
    <anonymous> webc.js:2
index.js:4:23

This next one is likely for Rollup, not here, but nonetheless...the Vite build command logs this to the console:
vite v3.2.4 building for production...
✓ 369 modules transformed.
'createHash' is not exported by __vite-browser-external, imported by<EMAIL_ADDRESS>file: /Users/pauleveritt/WebstormProjects/webc_testing/node_modules/@11ty/webc/src/ast.js:4:9
2: import fs from "fs";
3: import os from "os";
4: import { createHash } from "crypto";
            ^
5: import { DepGraph } from "dependency-graph";
error during build:
Error: 'createHash' is not exported by __vite-browser-external, imported by<EMAIL_ADDRESS>    at error (file:///Users/pauleveritt/WebstormProjects/webc_testing/node_modules/rollup/dist/es/shared/rollup.js:1858:30)
    at Module.error (file:///Users/pauleveritt/WebstormProjects/webc_testing/node_modules/rollup/dist/es/shared/rollup.js:12429:16)
    at Module.traceVariable (file:///Users/pauleveritt/WebstormProjects/webc_testing/node_modules/rollup/dist/es/shared/rollup.js:12788:29)
    at ModuleScope.findVariable (file:///Users/pauleveritt/WebstormProjects/webc_testing/node_modules/rollup/dist/es/shared/rollup.js:11440:39)
    at ChildScope.findVariable (file:///Users/pauleveritt/WebstormProjects/webc_testing/node_modules/rollup/dist/es/shared/rollup.js:6372:38)
    at ClassBodyScope.findVariable (file:///Users/pauleveritt/WebstormProjects/webc_testing/node_modules/rollup/dist/es/shared/rollup.js:6372:38)
    at ChildScope.findVariable (file:///Users/pauleveritt/WebstormProjects/webc_testing/node_modules/rollup/dist/es/shared/rollup.js:6372:38)
    at FunctionScope.findVariable (file:///Users/pauleveritt/WebstormProjects/webc_testing/node_modules/rollup/dist/es/shared/rollup.js:6372:38)
    at ChildScope.findVariable (file:///Users/pauleveritt/WebstormProjects/webc_testing/node_modules/rollup/dist/es/shared/rollup.js:6372:38)
    at Identifier.bind (file:///Users/pauleveritt/WebstormProjects/webc_testing/node_modules/rollup/dist/es/shared/rollup.js:7439:40)

Hmm… here’s what I see, looks like an incompatibility with fast-glob and Vite.
Module "path" has been externalized for browser compatibility. Cannot access "path.posix" in client code.
get @ browser-external:path:9
node_modules/glob-parent/index.js @ index.js:4
__require2 @ @11ty_webc.js?v=1a76a99b:16
node_modules/fast-glob/out/utils/pattern.js @ pattern.js:5
__require2 @ @11ty_webc.js?v=1a76a99b:16
node_modules/fast-glob/out/utils/index.js @ index.js:12
__require2 @ @11ty_webc.js?v=1a76a99b:16
node_modules/fast-glob/out/managers/tasks.js @ tasks.js:4
__require2 @ @11ty_webc.js?v=1a76a99b:16
node_modules/fast-glob/out/index.js @ index.js:2
__require2 @ @11ty_webc.js?v=1a76a99b:16
(anonymous) @ webc.js:2
index.js:4 Uncaught TypeError: Cannot read properties of undefined (reading 'dirname')
    at node_modules/glob-parent/index.js (index.js:4:40)
    at __require2 (@11ty_webc.js?v=1a76a99b:16:50)
    at node_modules/fast-glob/out/utils/pattern.js (pattern.js:5:20)
    at __require2 (@11ty_webc.js?v=1a76a99b:16:50)
    at node_modules/fast-glob/out/utils/index.js (index.js:12:17)
    at __require2 (@11ty_webc.js?v=1a76a99b:16:50)
    at node_modules/fast-glob/out/managers/tasks.js (tasks.js:4:15)
    at __require2 (@11ty_webc.js?v=1a76a99b:16:50)
    at node_modules/fast-glob/out/index.js (index.js:2:21)
    at __require2 (@11ty_webc.js?v=1a76a99b:16:50)
node_modules/glob-parent/index.js @ index.js:4
__require2 @ @11ty_webc.js?v=1a76a99b:16
node_modules/fast-glob/out/utils/pattern.js @ pattern.js:5
__require2 @ @11ty_webc.js?v=1a76a99b:16
node_modules/fast-glob/out/utils/index.js @ index.js:12
__require2 @ @11ty_webc.js?v=1a76a99b:16
node_modules/fast-glob/out/managers/tasks.js @ tasks.js:4
__require2 @ @11ty_webc.js?v=1a76a99b:16
node_modules/fast-glob/out/index.js @ index.js:2
__require2 @ @11ty_webc.js?v=1a76a99b:16
(anonymous) @ webc.js:2

Changing the test case to import fastglob from "fast-glob"; instead of WebC gives back the same error. Is this a Vite incompatibility?
fast-glob should only be used with the WebC defineComponents method—we could dynamically import it? Hmm…

Just to take a step back, this isn’t trying to bundle WebC to the client is it? This is using WebC ƒor server rendering?

Correct. More specifically, I'm experimenting with a test-oriented development experience using Vitest. I have it working with Eleventy itself and happy-dom. Experimenting with removal of Eleventy and plugging the data cascade with something like µsignal.

I will recreate a bug with just fast-glob and remove WebC. I'll then report to either fast-glob or throw it on the infinite pile at Vite.
Thus, closing this.

I wouldn’t be opposed to a dynamic import on fast-glob, if that helps—but it’d probably be a lower priority task.
Regardless, thanks Paul!

Let's leave this closed. I'll make a video series on this idea -- productive WebC development with Vitest -- but with 11ty. If it seems a useful idea, we can revisit directly-with-webc.
OT, I joined Mastodon last night, primarily because you stopped tweeting. 😀

Awesome! What’s your profile? I think I missed it!

When there's something worth your time to look at, I'll give you a nudge. 😇


core: Add start-after for OpList and add into capability

cc @lexcao

We can discuss the detials in this issue

Hi @Xuanwo , this is my first time contributing to OpenDAL, I would like to discuss the details before coding, please feel free to correct me.
To support start-after param for List operation, we could make these changes:

Add start_after arg (with getter and setter) for OpList
Add list_with method with param OpList for Operator

I need help on how to

add into capability

What does that mean?
I just found the Capability is used on operations rather than a single argument.

add into capability

We need to add a new field called list_with_start_after: bool to indicate that service support this arg.

Thanks.
I find there is a option for batch parameter max_batch_operations in AccessorInfo
So the list_with_start_after is added to the AccessorInfo?
Then to add a warning log when the accessor does not support the arg but is presented in the OpList when calling list_with method.

I just found there is a way to support custom args by leveraging AccessorCapability
enum AccessorCapability {
    List { start_after: bool }
    ...
}

This needs to extend AccessorCapability to support this since currently it is just u32 enum.
With that, the max_batch_operations could be replaced by
Batch { max_operations: uszie }

But this is another topic, I don't know if this is a good practice or not.

I'm guessing you need to update the main branch. We have a new Capacity design now.

Great, thank you! My bad.
I am ready to code. 💪


Genome folds

Dear developers,
Thanks for releasing your model, I'm sure I can speak for many in the community to say that it's looking hugely impressive!
To use and validate it, I'd like to see know what regions of the genome are in each of the test/validation folds that were used to the four models. For Enformer/Basenji, that was easily reconstructed from the helpfully shared sequences_[human|mouse].bed files in the public Google Storage bucket with 'supplementary' small files here, but I don't believe that's available for Borzoi yet?
Of course, it could be reconstructed from the large training dataset files, but given that I'm only looking for the genomic coordinates rather than the fully processed tracks corresponding to those, I was hoping there is an easier way.
Related to that though, all files in the borzoi-paper bucket currently don't seem to be available, as it returns the following error:
<Error>
 UserProjectMissing
 <Message>
  Bucket is a requester pays bucket but no user project provided.
 </Message>

  Bucket is a requester pays bucket but no user project provided.

</Error>

Although I'm hoping to not need those files at the moment, I figured I'd still mention it to let you know.
I'm sure the public release has left everyone swamped with questions coming in and issues popping up, so I appreciate any bit of time you are willing to spend on this!

Thanks for your interest! I add the sequences and targets files into a data/ directory from the github, too, so you don't have to figure out GCP for that.
For model f0, sequences labeled fold0 form the test set and fold1 form validation.
For model f1, sequences labeled fold1 form the test set and fold2 form validation. Etc

Hi David,
Just to clarify, the test/val/train splits for each of the models is set up as:
f0: test=fold0, val=fold1, train=rest
f1: test=fold1, val=fold2, train=rest
f2: test=fold2, val=fold3, train=rest
f0: test=fold3, val=fold4, train=rest
Thanks!


Support ability to merge objects in OpenAPI.NET. for example merge requests, payloads..

Can you clarify that you mean create a merged document by unioning the components from one document with the components from another document?  Merging of objects means something very different and we are not likely to do that.

In Dev Proxy we offer the ability to generate OpenAPI specs from intercepted requests and responses. Say, we intercept a request to /v1.0/me?$select=id,name and then another to /v1.0/me?$select=name,phone. We'd like to be able to merge both responses into one with 3 properties: id, name and phone.


update default token_secret_signature_key

Updated the default token_secret_signature_key from Rails.application.secrets.secret_key_base to Rails.application.credentials.secret_key_base since it was moved to an encrypted file due to this .

Closing since there's a PR for this on #225 .


travis - before_script.sh - alternative gpg key

This seems to be the suggested solution for the gpg --keyserver errors.

https://rvm.io/rvm/security
In rare cases the gpg --keyserver is failing, use this instead:
curl -sSL https://rvm.io/mpapis.asc | gpg --import -

Thanks!

Could a maintainer please run sync_templates_and_ci to push this and https://github.com/caskroom/homebrew-cask/pull/34701 to the other repos?

Ping @vitorgalvao

Running the sync. Missed the PR before. Thank you.

Thanks!


Incorrect complex filter output

Same file as in https://github.com/RazrFalcon/resvg/issues/268

Seems to be working somewhat fine now.


PD Fixup Round: i18n across step edit form

There are a number of TODO's surrounding using i18n for copy across the step edit form. They can be removed once the copy is put into i18n.

This ticket should maybe be split up?

Marked as stale b/c I did some of this in the last PD usability sprint, but not closing this yet b/c it should get a second pass to make sure


Documentation for linking a work item to pull request is gone

Documentation for linking a work item to pull request is gone
In the old documentation, there used to a page for linking a work item to pull request, now the link just reroute to the top git level page.
Issue Details
Please either add it to the pull request section or add it as an example to the work item update section

@hellosnow Can you take a look to see if it is by design?

This feels like a @erickson-doug ask 😄

Close as transferring the issue.


Support Git Stash unstaged changes

It would be great to have "Git: Stash unstaged changes" command shortcut in command pannel
This would avoid typing manually git stash save -k.
The "Stash All changes" button in the right pannel has the same function in STAGED CHANGES and in CHANGES parts. Maybe it could be implemented an other way on the CHANGES part to only stash changes not staged.

Cross issue with https://github.com/Microsoft/vscode/issues/71597

Any updates on my suggestion ? @eamodio ? 😄

Any considerations @eamodio ?

@julian-poidevin Sorry, I somehow missed this before. Good idea -- and I'm also going to be adding stash support to the new Git Commands command as well

@eamodio Thanks for implementing this ! 👍🏻


Failed tasks searchable by Args

As a debugging dev I would like the ability to search for args in the Django Admin interface for failed tasks and see failed tasks which have these args.

Also came here to ask for this. Even for successful tasks.
In my case some emails are not going through. We pretty much know it is due to the SMTP server but we would like to trace certain email addresses so that we can follow from the SMTP server logs what is going wrong.
Since scenarios are not always reproducible, it would be handy to be able to search out the args and thereby find out the times of the 'successful' tasks.


CmdLet name - small issue

The cmdlet name for getting the Organization Config is not in the right case. It should be Get-OrganizationConfig   and not Get-Organizationconfig

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

ID: 7bad8864-5f08-90ae-a6d1-a23e58fc1843
Version Independent ID: 7bad8864-5f08-90ae-a6d1-a23e58fc1843
Content: Bookings in Outlook
Content Source: microsoft-365/bookings/bookings-in-outlook.md
Service: bookings
GitHub Login: @kwekuako
Microsoft Alias: kwekua

PR merged. Closing


MUST use standard data formats (2)

solves https://github.com/zalando/restful-api-guidelines/issues/642 -- replaces outdated https://github.com/zalando/restful-api-guidelines/pull/650

:+1:

@tfrauenstein could you please update the changelog? For example that Severity of Rule 238 changed from SHOULD to MUST.

@vadeg  change log history is now up-to-date, thx.

👍

:+1:


Add manpage to the extra/ directory

This patch adds a manpage to the extra/ directory.
Signed-off-by: David Polverari<EMAIL_ADDRESS>

Thanks!


feat: Light mode theme

Frameless white background

🤖 Generated by Copilot at c2d1ae4
Summary
🎨🚚🐛

This pull request improves the UI consistency and appearance of various apps and components by applying the baseSurface class from the aurora theme. It also fixes some minor issues with the kanban, chess, and gravity apps, and changes the title of the labs app. Additionally, it adds some comments to the nav tree components to suggest future refactoring.

baseSurface class
adds consistency and style
to many apps now

Walkthrough

Changed the title of the labs app to "DXOS Labs" in index.html (link).

Approved, pending the last changes Rich and I paired on.


Failed to build in ios

** BUILD FAILED **
The following build commands failed:
CompileC /Users/kwan/Library/Developer/Xcode/DerivedData/reactnative_eyepetizer-aiggwvtvbboeutgupqcnpsteinzy/Build/Intermediates.noindex/Pods.build/Debug-iphonesimulator/react-native-webview.build/Objects-normal/x86_64/RNCWebView.o /Users/kwan/Programming/Javascript/react/react-native/ReactNative_Eyepetizer/node_modules/react-native-webview/apple/RNCWebView.m normal x86_64 objective-c com.apple.compilers.llvm.clang.1_0.compiler
(1 failure)

For anyone who are facing this issue,
Downgrading react-native-webview to 11.14.3 fixes the issue.


ga.tests.ui.pr : Bump Playwright

Questions
Answers

Description?
ga.tests.ui.pr : Bump Playwright

Type?
bug fix

BC breaks?
no

Deprecations?
no

Fixed ticket?
N/A

Sponsor company
@PrestaShopCorp

How to test?
Tested on PrestaShop/PrestaShop#33958

thanks @Progi1984


Consider switching from Chromium to Servo

This might solve problems like #490, #371
On the downside:

dependency on Rust

which we would somehow need to manage as well
which could make building Servo clean again impossible
=> no widespread OS has Servo dev support per default

Servo is still immature

we might at least want to wait till WebRender is landed in Firefox

https://hackmd.io/BwJhEMQMwZgBgLQDYCsBTKCAsBjAjJsFgJxoLErEjAzF4gAmMaQA#
https://bugzilla.mozilla.org/show_bug.cgi?id=1311790

installing the rust compiler could be done via conan.
downloading Rust is only necessary if you build the "CEF-Servo" package from source
otherwise we only link the prebuilts.
Yes true, we might want to switch later.
The deadline for Firefox 60 is May 2018, see https://wiki.mozilla.org/RapidRelease/Calendar
We will develop against the CEF API anyways till then.

Servo removed the support for CEF for the moment. https://github.com/servo/servo/issues/9147#issuecomment-411944715
Waiting for it to (hopefully) come back...

Hopefully.

Okay, I looked into servo some more regarding the issues:
Rust

Rust functions can be executed pretty easily from C -> This is not a problem
The rust toolchain needs to be installed. -> This needs an additional conan dependency

Dependencies
There are a lot less dependencies for Servo than for CEF, building should be much more straight-forward.

Rust
gstreamer

easily installable through package managers on all target OS

openssl
bz2
harfbuzz

@Croydon: what do you mean by

Blocked by https://bugzilla.mozilla.org/show_bug.cgi?id=1386669 & https://bugzilla.mozilla.org/show_bug.cgi?id=1386674

Servo is used in firefox already, so why are these blocking?

Servo is used in firefox already, so why are these blocking?

There is a huge difference between using parts of Servo in Firefox (this is the case) and having full Servo in Firefox, which will still take same time and maybe a long time.
These meta bugs I linked to can be a good indicator and how stable Servo as a whole is.

As far as we evaluated on the hackathon #608 Servo will remain unfit for our use in the realistic future.


Reference the IRI schema:rangeIncludes

The skos:scopeNote on gist:rangeIncludes indirectly mentions schema:rangeIncludes but shouldn't Gist also explicitly reference it with its IRI?
Can we do the following?
gist:rangeIncludes owl:equivalentProperty schema:rangeIncludes
If we can't because of an OWL profile constraint that is fine but it seems we should do some form of
gist:rangeIncludes ?p schema:rangeIncludes to be a team player in LOD and to be thingy rather than stringy.
Oh, this also applies to gist:domainIncludes.

gist:rangeIncludes is an OWL annotation property. Schema predicates are RDFS properties, therefore it would be incorrect to assert equivalence. An rdfs:seeAlso would be a good idea.

An rdfs:seeAlso would be a good idea.

I agree. That seems like the minimal way be to thingy rather than just stringy.

I don't know why the scope note on a single property would reference the schema predicates; that seems arbitrary.

My guess is because schema:rangeIncludes came first. Also I think the skope note is still useful albeit stringy:
"This property means the same thing as the Schema.org version, which is an rdf:Property and not compliant with OWL-DL."
So we are saying we'll add the following if we get consensus:
gist:rangeIncludes rdfs:seeAlso schema:rangeIncludes .

Oh, @rjyounes, there is also skos:closeMatch that might be even more to the point.

@justin2004 I disagree. The skos predicates are meant to relate concepts, not RWOs.

gist:rangeIncludes rdfs:seeAlso schema:rangeIncludes .
gist:domainIncludes rdfs:seeAlso schema:domainIncludes .

I am in agreement with adding these triples, and would like to remove individual references to the schema predicates within specific term definitions.

PROPOSAL:
Add schema namespace prefix
@prefix schema: <https://schema.org/>
Replace
gist:domainIncludes
     skos:scopeNote
		"This property means the same thing as the Schema.org version, which is an rdf:Property and not compliant with OWL-DL."^^xsd:string
		;
	.

with
gist:domainIncludes rdfs:seeAlso schema:domainIncludes ;
	.

and Replace
gist:rangeIncludes
     skos:scopeNote
		"This property means the same thing as the Schema.org version, which is an rdf:Property and not compliant with OWL-DL."^^xsd:string
		;
	.

with
gist:domainIncludes rdfs:seeAlso schema:rangeIncludes ;
	.

Alternatively, leave the scope note intact and just add the seeAlso triple.

I'd suggest editing the skos:scopeNode slightly to say it also 'means the same' as dcam:rangeIncludes (where prefix dcam: <http://purl.org/dc/dcam/>) and add an rdfs:seeAlso dcam:rangeIncludes triple too (just to emphasise that the notion of domain/rangeIncludes is not just a Schema.org notion!).
And the fundamental reason I'd support keeping the scope note is for the explanation that "an rdf:Property is not compliant with OWL-DL" (and maybe also an rdfs:seeAlso <HTML page that explains gist's intent to maintain OWL-DL compliance, and therefore the need to avoid rdfs:Class and rdf:Property> triple!).

My current view is that asserting rdfs:seeAlso to other ontologies initiates a bad practice. I recommend closing this issue.

My current view is that asserting rdfs:seeAlso to other ontologies initiates a bad practice. I recommend closing this issue.

I'm good with this. It raises the question of when else do we refer to external ontologies?


Persist Caddy data

From the dockerhub description (https://hub.docker.com/_/caddy?tab=description):

However, it's very important to persist the data directory.

@ostcar @gsiv Is it necessary (The warning seems to be really important)? Did you care about this?

I did not know about it.
But it does not seem necessary in our case. If I understand correctly, it is only necessary for TLS data. We don't use TLS with caddy

We don't use TLS with caddy

Not in our own prod setup, but we support it for other users

We don't use TLS with caddy

Not in our own prod setup, but we support it for other users

I concur with @ostcar; I don't see why we would need to persist the data directory.
I must admit, though, that I'm also unaware of how other users would go about enabling TLS.  To me, it looks like they would need to customize the provided Caddyfile and build a new image.
Granted, it would be nice to include an example TLS setup with a note saying that, when enabling TLS, Caddy's data directory must be persisted.

This is the basic https setup: https://github.com/OpenSlides/OpenSlides/pull/5894
It should be sufficient for other users to quikcly use https and provides enough flexibility I think.

Granted, it would be nice to include an example TLS setup with a note saying that, when enabling TLS, Caddy's data directory must be persisted.

Ok, I'll add it there.


Move student program related files to mentorship folder

Closes https://github.com/chaoss/community/issues/320
Signed-off-by: Venu Vardhan Reddy Tekula<EMAIL_ADDRESS>

Thanks @vchrombie


Query: Avoid calling ValueBuffer.Equals()

Only values should be compared--not entire buffers. Making this method throw shows a few test failures.

Functional Tests
Failures

InMemory
4

Sqlite
6

SqlServer
8

We run into using Equals method due to client side implementation.
For query like below, (untested)
context.Blogs.Where(b => ClientMethod(b.Id)).Distinct().Select(b => b.Value).ToList();

Here, we have a client Where which causes us to evaluate Distinct on client. But at the same time we are not projecting out the entity, so we use ValueBuffer rather than entity. Doing operations like Distinct/Contains etc on enumerable causes call to Equals method. We should be materializing entity for such cases but currently materialization is separated from client eval so we don't have a way to fix it.


[Bug]: Empty Space on the right

Bug Description
There is an empty space on the right side and a lack of space on the left side bar.
Relevant Screenshot

To Reproduce
Colapse the right and the left side to see the empty space on the right.
Show the left side bar to see some space lacking there.
Obsidian Version
0.14.6
Installer Version
0.14.6
Font Size
16
Which editor are you using?

[X] New Editor
[ ] Legacy Editor

In which mode(s) does the problem occur? (if applicable)

[ ] Source (Edit)
[ ] Reading (Preview)
[X] Live Preview

Which operating system?

[X] macOS
[ ] Linux
[ ] Windows
[ ] iOS
[ ] Android

Plugin-Specific Problems

[ ] I took note of the specific instructions regarding those plugins.

Checklist

[X] I updated to the latest version of the theme (Settings → Appearance → Themes - Manage → Update).
[X] I made sure that the issue is related to this theme by checking that the problem does not occur with the default theme.
[X] In case I am missing an UI element, I have checked that it cannot be re-enabled with the Style Settings plugin.
[X] If the bug occured after updating the theme, I read the last couple of entries in the changelog to make sure the bug is not an intentional change. (In that case, there will most likely be a style setting to turn it off again.)

this is caused by hider plugin when you enable hide app ribbon in the plugin settings

Oh, thank you!

andreafaggion.com
On 18 May 2022 09:50 -0300, imeed166 @.***>, wrote:

this is caused by hider plugin when you enable hide app ribbon in the plugin settings
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you authored the thread.Message ID: @.***>

This theme doesn't need to support the plugin because It already has all the features the plugins provides. So you should probably close this issue.

np, had the exact same thing happened to me in the past


Simple console based UI

Previously, UI was used which is blocking userinput. We have to implement using GUI.

In newer implementation a FLTK (Flash Light Tool-Kit) API/Library is used for
interactive GUI (Graphical User Interface).


feat: added message to forward event handler

Signed-off-by: Peter Strobel<EMAIL_ADDRESS>

After some more investigation with @PeterStrob, we've deemed these changes to be not needed at this time. See further discussion below.
Problem

Implementers of external undelivered queues lack the mediated key AND outbound message in one event.

acapy::outbound-message::undeliverable has the outbound message but not the mediated key. (Key faulty assumption)
acapy::forward::received has the mediated key but not the whole outbound message.

It was incorrectly assumed that the mediated key was not to be found in the undeliverable event. However, it can be decoded from the outbound message.

Mediated Key and Outbound Message
Below is an example payload for the undelivered message event:
{

    "topic": "acapy::outbound-message::undeliverable",
    "payload":  <OutboundMessage(connection_id='b114b86f-2e08-4e7f-a898-b79d7812af11', enc_payload=None, _endpoint=None, payload='{"@type": "did:sov:BzCbsNYhMrjHiqZDTUASHg;spec/trust_ping/1.0/ping_response", "@id": "c85481d5-8f77-48f0-9a82-cd258e4aa5b8", "~thread": {"thid": "ce461eba-f363-4ef4-9a71-1d29dbd29046"}}', reply_session_id='9d4594ae-cea2-40cf-abce-510439223ae2', reply_thread_id='ce461eba-f363-4ef4-9a71-1d29dbd29046', reply_to_verkey='48DT12mtyQNMMAtJJ6ygS9ZBVJ4bEyZsYrqY5VW8or9B', reply_from_verkey='51xjfZqVZ2Fkq6HvvuJTeCuwXpnRrhLTEbsZnNJGV82F', target=None, target_list=[<ConnectionTarget(did='6jV2NCwZ1VxGAd5vwZwJpG', endpoint='', label='test-runner', recipient_keys=['48DT12mtyQNMMAtJJ6ygS9ZBVJ4bEyZsYrqY5VW8or9B'], routing_keys=[], sender_key='51xjfZqVZ2Fkq6HvvuJTeCuwXpnRrhLTEbsZnNJGV82F')>], to_session_only=False)>
}

Critically, this is an example of a message that was sent from the mediator to the mediated client. Therefore, the enc_payload is not populated.
An example payload of an undeliverable message event with an enc_payload, meaning that it originated from a forward message, can be found below:
{
    "topic": "acapy::outbound-message::undeliverable",
    "payload":  <OutboundMessage(
        connection_id='a010bb16-0ce0-4f42-9e47-295bc950182 5',
        enc_payload=b'{"protected": "eyJlbmMiOiAieGNoYWNoYTIwcG9seTEzMDVfaWV0ZiIsICJ0eXAiOiAiSldNLzEuMCIsICJhbGciOiAiQXV0aGNyeXB0IiwgInJlY2lwaWVudHMiOiBbeyJlbmNyeXB0ZWRfa2V5IjogIjMwN3J6TE84TzZ3M3ZTc3ZwVkM2blREZnFWdVA3dXE3ZXJZWHA 2VVNTeFBINVFYSzZLLVM3cnFwZUpEalRnZWIiLCAiaGVhZGVyIjogeyJraWQiOiAiNDhEVDEybXR5UU5NTUF0Sko2eWdTOVpCVko0YkV5WnNZcnFZNVZXOG9yOUIiLCAic2VuZGVyIjogImVhWDFDRUNCQUdyWGtmUm9fQ0NONmsyei1oaW1aYTZiMTQ3TjF2Qm9tRFl0Mnl5bW1wZjlJRUExcVBkbDJ Bcm9ISkJVc2tQbmJVLVppWUM1NjM2Ym15eDc2TVF1VjBGLTB4WVl2RllvdW5aRGVkSU90eWR6eklGYm1Cdz0iLCAiaXYiOiAiYVFyZlQ5TlBwYUF2TkQ0ZTFwUy1wRFB5SS1OR1pCRVUifX1dfQ==", "iv": "xi6P42zYEU9l69zx", "ciphertext": "PN0Cc_zqEen1QtrW8P-1iiPjlOxRdzr FrjxpezEF_luW626kqV9GVexf9EBnsLe_Hsm-UIgLu993WeRQGh0DILN3Hs96qId2deYWcI1QevJYuSqsypFQuZSpZ_4Cdmspe-hk8T1RTq6SpFTU_2g1cBKxZnl6mLueSudvcgzIrpm6hGX3sqIC4eAOTXmTSnj1fPAdfQ4ILWck6o1p7dkvZfRfZajmy_i5Vd5j7Hp-3jlD4Z40gc2xAc3rwFUmMVk IfZYnnyaxOMaqaXE6DbPBX-M=", "tag": "LE7lOwVYe3cOBRNaR4oO8w=="}',
        _endpoint=None,
        payload=None,
        reply_session_id=None,
        reply_thread_id=None,
        reply_to_verkey='48DT12mtyQNMMAtJJ6ygS9ZBVJ4bEyZsYrqY5VW8or9B',
        reply_from_verkey='FE6K1Cheo3YnTMc4p3ZpkMdrjnLBTiqG9fUSrqhsy6ST',
        target=None,
        target_list=[
            <ConnectionTarget(
                did='6jV2NCwZ1VxGAd5vwZwJpG',
                endpoint='',
                label='test-runner',
                recipient_keys=['48DT12mtyQNMMAtJJ6ygS9ZBVJ4bEyZsYrqY5VW8or9B'],
                routing_keys=[],
                sender_key='51xjfZqVZ2Fkq6HvvuJTeCuwXpnRrhLTEbsZnNJGV82F'
            )>
        ],
    to_session_only=False
    )>
}

The insides of the enc payload is:
{
    "protected": "eyJlbmMiOiAieGNoYWNoYTIwcG9seTEzMDVfaWV0ZiIsICJ0eXAiOiAiSldNLzEuMCIsICJhbGciOiAiQXV0aGNyeXB0IiwgInJlY2lwaWVudHMiOiBbeyJlbmNyeXB0ZWRfa2V5IjogIjMwN3J6TE84TzZ3M3ZTc3ZwVkM2blREZnFWdVA3dXE3ZXJZWHA 2VVNTeFBINVFYSzZLLVM3cnFwZUpEalRnZWIiLCAiaGVhZGVyIjogeyJraWQiOiAiNDhEVDEybXR5UU5NTUF0Sko2eWdTOVpCVko0YkV5WnNZcnFZNVZXOG9yOUIiLCAic2VuZGVyIjogImVhWDFDRUNCQUdyWGtmUm9fQ0NONmsyei1oaW1aYTZiMTQ3TjF2Qm9tRFl0Mnl5bW1wZjlJRUExcVBkbDJ Bcm9ISkJVc2tQbmJVLVppWUM1NjM2Ym15eDc2TVF1VjBGLTB4WVl2RllvdW5aRGVkSU90eWR6eklGYm1Cdz0iLCAiaXYiOiAiYVFyZlQ5TlBwYUF2TkQ0ZTFwUy1wRFB5SS1OR1pCRVUifX1dfQ==",
    "iv": "xi6P42zYEU9l69zx",
    "ciphertext": "PN0Cc_zqEen1QtrW8P-1iiPjlOxRdzr FrjxpezEF_luW626kqV9GVexf9EBnsLe_Hsm-UIgLu993WeRQGh0DILN3Hs96qId2deYWcI1QevJYuSqsypFQuZSpZ_4Cdmspe-hk8T1RTq6SpFTU_2g1cBKxZnl6mLueSudvcgzIrpm6hGX3sqIC4eAOTXmTSnj1fPAdfQ4ILWck6o1p7dkvZfRfZajmy_i5Vd5j7Hp-3jlD4Z40gc2xAc3rwFUmMVk IfZYnnyaxOMaqaXE6DbPBX-M=",
    "tag": "LE7lOwVYe3cOBRNaR4oO8w=="
}

The contents of the protected header are:
{
  "enc": "xchacha20poly1305_ietf",
  "typ": "JWM/1.0",
  "alg": "Authcrypt",
  "recipients": [
    {
      "encrypted_key": "307rzLO8O6w3vSsvpVC6nTDfqVuP7uq7erYXp6USSxPH5QXK6K-S7rqpeJDjTgeb",
      "header": {
        "kid": "48DT12mtyQNMMAtJJ6ygS9ZBVJ4bEyZsYrqY5VW8or9B",
        "sender": "eaX1CECBAGrXkfRo_CCN6k2z-himZa6b147N1vBomDYt2yymmpf9IEA1qPdl2AroHJBUskPnbU-ZiYC5636bmyx76MQuV0F-0xYYvFYounZDedIOtydzzIFbmBw=",
        "iv": "aQrfT9NPpaAvND4e1pS-pDPyI-NGZBEU"
      }
    }
  ]
}

The forward event looked like:
{
    "topic": "acapy::forward::received",
    "payload": {
        'connection_id': 'a010bb16-0ce0-4f42-9e47-295bc9501825',
        'status': 'undeliverable',
        'recipient_key': '48DT12mtyQNMMAtJJ6ygS9ZBVJ4bEyZsYrqY5VW8or9B',
    }
}

Therefore, we see that the mediated key (recipient_key in the forward event and the enc_payload.protected | b64-decode.recipients[].header.kid in the undeliverable event) is known without needing to modify the forward event.


Resize in SI units

Currently it's possible to define the size of an image by pixels. However it would be great if it would be possible to define the size in centimeters or another typical unit because this reflects more knowledge of the average user.

https://github.com/T8RIN/ImageToolbox/issues/33


fix(venue-page): Fix the width of map

(Thanks for sending a pull request! Please fill in the following content to let us know better about this change.)
Types of changes
Please put an x in the box that applies

[x] Bugfix
[ ] New feature
[ ] Refactoring
[ ] Breaking change (any change that would cause existing functionality to not work as expected)
[ ] Documentation Update
[ ] Other (please describe)

Description
Describe what the change is
You don't need to explain HOW you change the code. Your code, including comments in the code should explicitly
self-describe enough about how you change the code.

Fix oversized width of map in mobile
Also update the format of code by using SCSS formatter

Steps to Test This Pull Request
Steps to reproduce the behavior:

Go to venue page in responsive mode
check the result

Expected behavior
A clear and concise description of what you expected to happen.
The view should looks as following screenshots

Related Issue
If applicable, refernce to the issue related to this pull request.
#873
More Information
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here. You may also want to refer
to how to wirte the perfect pull request

Codecov Report

Merging #874 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #874   +/-   ##
=======================================
  Coverage   75.98%   75.98%
=======================================
  Files          70       70
  Lines        2727     2727
=======================================
  Hits         2072     2072
  Misses        655      655

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 10c1fc6...462f808. Read the comment docs.

天阿我是 review 到斷片嗎? 我怎麼記得我有回過這一個但實際上卻找不到紀錄。見鬼了。
當初在實作這一頁的時候，我是用 firefox 看手機寬度的，所以沒發現有問題，看到你的 issue 用 chrome 看就有問題了，iphone / android 都是用 webkit 所以看起來應該都一樣有問題。
原則上是希望長這樣

after messing around and some testing, I have the following conclusion:
the shove-left() mixin needs another line:
margin-right: calc(50% - 50vw);

You can use the original code, add the margin-right to shove-left, and then get rid of the whole width: 100vw;.
It should work.
Some reference that I found: https://css-tricks.com/full-width-containers-limited-width-parents/

Hi @yychen, Thank you for your advice. I've updated the width of the map to using full width. Please have a check, Thanks!


colorValue not updating without re-render of component.

The value passed into v-model doesn't update the color picker value unless the component is re-rendered.
As a workaround, I can force a re-render of the component, but this doesn't feel right.
This looks like:
<Vue3ColorPicker :key="componentKey"
      v-model="colorValue"
      mode="solid"
      :show-color-List="false"
      :show-eye-drop="false"
      type="RGB"
      show-color-list="true"
      color-list-count="10"
    />

<script setup lang="ts">
import { Vue3ColorPicker } from '@cyhnkckali/vue3-color-picker'

const conponentKey = ref(0)
const colorValue = ref('rgba(255, 255, 255, 1)')

watch(
  computedButtonId,
  (newButtonId, oldButtonId) => {
    if (newButtonId !== oldButtonId) {
      colorValue.value = ledControlState.value[newButtonId] || 'rgba(255, 255, 255, 1)';
      // Increment componentKey to force re-render
      componentKey.value += 1;
    }
  }
);
</script>

Hi @christywastaken,
I'll look into it as soon as I can.

@christywastaken is it the only color picker on the page?

@christywastaken is it the only color picker on the page?

Yeah, the only one.

@christywastaken  @cyhnkckali understood, I'll fix.


Add onWheel event

Adds onWheel event (like onClick, but for mouse wheels). Seems to be in DOM.Event.Types already, but not anywhere else.
Seems simple and works with my use-case, but not really tested. slamdata/purescript-dom-indexed#10 is required for use of the tag to type-check.
Reference: https://developer.mozilla.org/en-US/docs/Web/Events/wheel

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

I've signed the CLA, but is there a reason it requires access to private email addresses? The public one really should be sufficient.

Thanks, I'll merge this when we make the release that updates for new Aff, as both are breaking changes and the Aff one shouldn't be long in coming.
I have no idea why the CLA thing wants a private email 😕


Trigger event when seed data is available

Create a uniform end point for libraries to subscribe to, which need
to load data either from the JSON seed or the Backbone collections.

Coverage remained the same at 99.095% when pulling cfe75838b96e9cade9ac60121496969b2b082bb0 on tf:seed-loaded-event into 1114ffb9c49f30fbfa735417a30ef5317acd51a9 on codevise:master.

Coverage remained the same at 99.095% when pulling cfe75838b96e9cade9ac60121496969b2b082bb0 on tf:seed-loaded-event into 1114ffb9c49f30fbfa735417a30ef5317acd51a9 on codevise:master.


Writing markup with jsx- translation completed

Revert the commits. We can't bring any change to yarn and package files.


Support CFI augmentation without data

In a specific ARM build of libc.so, we've encountered CFI augmentations that neither use the eh nor the z format, and just define a signal trampoline ("S"). This patch changes parsing of the z operand to make it optional according to:
http://refspecs.linuxbase.org/LSB_5.0.0/LSB-Core-generic/LSB-Core-generic/ehframechpt.html

The test failure on Rust 1.32.0 also happens on master and is unrelated.

Coverage decreased (-86.2%) to 0.0% when pulling 75996bf9aebd5cfc48376bdf06f4a0b4d7e4f8ce on jan-auer:fix/cfi-augmentation-no-z into 1f1a9eb23993d3787a4e3389eade28a104bd91d0 on gimli-rs:master.

Coverage decreased (-86.2%) to 0.0% when pulling 75996bf9aebd5cfc48376bdf06f4a0b4d7e4f8ce on jan-auer:fix/cfi-augmentation-no-z into 1f1a9eb23993d3787a4e3389eade28a104bd91d0 on gimli-rs:master.

@philipc Regarding the CI failure: looks like flate2 doesn't support rust 1.32 anymore, which is depended upon by object. If we want to guarantee 1.32 support then we should pin object's dependency to an older version, I guess.
   Compiling flate2 v1.0.11
error[E0658]: use of unstable library feature 'try_from' (see issue #33417)
  --> /Users/travis/.cargo/registry/src/github.com-1ecc6299db9ec823/flate2-1.0.11/src/ffi.rs:47:9
   |
47 |     use std::convert::TryFrom;
   |         ^^^^^^^^^^^^^^^^^^^^^
error[E0658]: use of unstable library feature 'try_from' (see issue #33417)
   --> /Users/travis/.cargo/registry/src/github.com-1ecc6299db9ec823/flate2-1.0.11/src/ffi.rs:109:27
    |
109 |             .and_then(|i| usize::try_from(i).ok())
    |                           ^^^^^^^^^^^^^^^

Going to go ahead and merge this, since CI failure is unrelated to this PR.

If we want to guarantee 1.32 support then we should pin object's dependency to an older version, I guess.

No, we can bump that version whenever it is reasonable to do so. That test is more for simply knowing what the minimum version is.


Load templates from embedded dir if missing from user config dir

Is your feature request related to a problem? Please describe.
I was customizing a device config file, but it failed to load because a template file was not available in the user config directory. I don't have any intention of editing the template file, so I would just like to store the single device file. One of the easy things about the user config directory is I can just throw in a device file w/o having to worry about naming and directory structure. For a file that requires templates that becomes slightly more complicated. It also means if the config file is being used long-term, I don't need to maintain the template, only the single device file.
Describe the solution you'd like
When loading a user config file and the template file is not found, try to fallback to the embedded template file.
Describe alternatives you've considered
Save the template(s) into the user config dir with an identical directory structure compared to the embedded dir. This reduces the relative ease of customizing a device config though.
Additional context
2021-10-05T05:52:32.925Z CONFIG   Error parsing config file zw062.json: Could not parse config file /usr/src/app
                                  /store/config/templates/master_template.json: ENOENT: no such file or director
                                  y, open '/usr/src/app/store/config/templates/master_template.json'
                                  Import stack:
                                    in #base_0-100_nounit
                                    in /usr/src/app/store/config/zw062.json

$ pwd
/srv/apps/z2m/data/config
$ ls
14287.json  zw062.json

If anyone wants to work on this, go for it.


Energy conduits not extract by default

Is there an existing issue for this?

[X] I did not find any existing issues.

Current Behavior
In certain scenarios (thermal dynamos for example) energy conduits are not set to extract by default
Expected Behavior
Conduits extract energy by default
How to Test
Load a thermal dynamo with fuel, attach energy conduits to the end and see that the conduits don’t extract by default
Local Environment
- **Operating System**:
- **Java version**:
- **Minecraft Version**: 1.20.1
- **Version**: 6.1.8
- **Other Mods**: Thermal

Any Additional Information?
No response

The issue with thermal dynamos has been fixed and will be in the 6.0.11-beta release.


devos/darwin: refine users

I meant to include these changes in #438:

Demonstrates the ability to define reusable home-manager user configurations
via home.users and import them into any arbitrary home-manager user within a
nixos or darwin profile.
Rename the system/home users to primary to distinguish them from all the other
references to 'darwin' and also to aid in demonstrating the flexibility of
"portable" home-manager users
Use Digga's collectHosts function to... collect hosts for deriving homeConfigurations
Remove some extraneous comments
Add some extraneous comments

Could we remove the nixos user and use admin/primary everywhere? It would make writing docs easier too, since we only have to reference one user to change.

yeah, good point. will update.

Could we remove the nixos user and use admin/primary everywhere?

I took a halfway approach, keeping the nixos user around but only using it in the bootstrap-iso host since that's the expected user for live media.

Now I'm remembering that one of the reasons I split up the users was because nix-darwin lacks a lot of the basic users.users.<name>.* options. extraGroups, for one. Builds are failing during checks for that reason.
https://github.com/divnix/digga/runs/7312053064?check_suite_focus=true#step:7:312


blog: add a blogpost talking about v2 plan & work

Motivation
Add a blog post talking about the plan & work for Docusaurus 2.0.
Have you read the Contributing Guidelines on pull requests?
Yes
Test Plan
https://deploy-preview-952--docusaurus-preview.netlify.com/blog/2018/09/09/towards-docusaurus-2.0
P.S : Still need revision on wording, content, etc.

We definitely need revision on the content 😭 / wording wise. Appreciate it if you guys could give some suggestion !! @yangshun @JoelMarcey

Hmm.. i was thinking that maybe more details should be put in the actual Docusaurus 2 release post. This blog post aims to

Point user to RFC issue
Give an overview of planned v2
Informing user that we are working on v2, so less features/ changes on v1

Wdyt ?
Is this good for you @joelmarcey ?

@yangshun added a 'frequently asked questions'. Please see it once again. 👍

Just two spelling improvements:

Howver, one thing that I'll highlight > However ...
Crowdin for translations,so >> translations, so ...
we do noot want to make Docusaurus better > we do not want ...

Other than those, it looks good! 👍

Merging this since @JoelMarcey has proodread it too 😄

Are there any other changes you want to make? If not I'll just add that line in.

Not at this time -- I can make any further changes if I come up with any.


Update Changelog for 4.8.0 release

Coverage remained the same at 100.0% when pulling 9b10b4d1dc8c5c9d389ee68c7d90ea8e8aefdaf1 on l0gicgate:update-changelog into c9bdc9e0d2f8613055632334ec6711b965d5fdf3 on slimphp:4.x.


Fixed wrong CommonJS jQuery UI dependency

Right npm package name is jquery-ui, not jquery.ui.
https://www.npmjs.com/package/jquery-ui

These changes have been merged with the correct jquery-ui strings. Thanks!


SF-1317: gb-carousel now sets tiles to the carousel on page load

https://groupby.atlassian.net/browse/SF-1317

Codecov Report

Merging #81 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #81      +/-   ##
==========================================
+ Coverage    93.1%   93.11%   +<.01%
==========================================
  Files         201      201
  Lines        5556     5563       +7
  Branches      701      701
==========================================
+ Hits         5173     5180       +7
  Misses        293      293
  Partials       90       90

Impacted Files
Coverage Δ

<EMAIL_ADDRESS>100% <100%> (ø)
:arrow_up:

...storefront/breadcrumbs/src/original-query/index.ts
100% <0%> (ø)
:arrow_up:

...orefront/breadcrumbs/src/refinement-crumb/index.ts
100% <0%> (ø)
:arrow_up:

...torefront/breadcrumbs/src/corrected-query/index.ts
100% <0%> (ø)
:arrow_up:

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>100% <0%> (ø)

... and 4 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 d2a52e7...489490d. Read the comment docs.

Codecov Report

Merging #81 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #81      +/-   ##
==========================================
+ Coverage    93.1%   93.11%   +<.01%
==========================================
  Files         201      201
  Lines        5556     5563       +7
  Branches      701      701
==========================================
+ Hits         5173     5180       +7
  Misses        293      293
  Partials       90       90

Impacted Files
Coverage Δ

<EMAIL_ADDRESS>100% <100%> (ø)
:arrow_up:

...storefront/breadcrumbs/src/original-query/index.ts
100% <0%> (ø)
:arrow_up:

...orefront/breadcrumbs/src/refinement-crumb/index.ts
100% <0%> (ø)
:arrow_up:

...torefront/breadcrumbs/src/corrected-query/index.ts
100% <0%> (ø)
:arrow_up:

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>

<EMAIL_ADDRESS>100% <0%> (ø)

... and 4 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 d2a52e7...489490d. Read the comment docs.

Codecov Report

Merging #81 into master will increase coverage by 0.92%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #81      +/-   ##
==========================================
+ Coverage   93.18%   94.11%   +0.92%
==========================================
  Files         207      209       +2
  Lines        5623     5866     +243
  Branches      701      724      +23
==========================================
+ Hits         5240     5521     +281
+ Misses        293      250      -43
- Partials       90       95       +5

Impacted Files
Coverage Δ

<EMAIL_ADDRESS>100% <100%> (ø)
:arrow_up:

<EMAIL_ADDRESS>97.72% <0%> (-2.28%)
:arrow_down:

...ont/flux-capacitor/src/core/reducers/is-running.ts
100% <0%> (ø)
:arrow_up:

<EMAIL_ADDRESS>100% <0%> (ø)
:arrow_up:

...capacitor/src/core/reducers/data/productsLoaded.ts
100% <0%> (ø)
:arrow_up:

...ux-capacitor/src/core/reducers/data/site-params.ts
100% <0%> (ø)
:arrow_up:

...storefront/navigation/src/refinement-pill/index.ts
100% <0%> (ø)
:arrow_up:

<EMAIL_ADDRESS>100% <0%> (ø)
:arrow_up:

<EMAIL_ADDRESS>100% <0%> (ø)
:arrow_up:

...ront/flux-capacitor/src/core/reducers/data/area.ts
100% <0%> (ø)
:arrow_up:

... and 32 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 d76b907...c5a5e69. Read the comment docs.


Changes applied to NgrokModuleSet do not propagate to managed Edge

What happened
After applying NgrokModuleSet with e.g. oauth set to google email address and changing the email address there's no change in the managed Edge.
What you think should happen instead
After applying NgrokModuleSet with e.g. oauth set to google email address and changing the email address, managed Edge should be changed
How to reproduce

Apply (e.g. via kubectl apply ...) below listed manifest (substitute domain and email address with your own)
apiVersion: v1
kind: Service
metadata:
  name: proxy
  namespace: infra
spec:
  ports:
  - name: http
    port: 80
    targetPort: 80
  selector:
    app: httpbin
---
apiVersion: apps/v1
kind: Deployment
metadata:
  name: httpbin
  namespace: infra
spec:
  replicas: 1
  selector:
    matchLabels:
      app: httpbin
  template:
    metadata:
      labels:
        app: httpbin
    spec:
      containers:
      - image: kong/httpbin:0.1.0
        name: httpbin
        ports:
        - containerPort: 80
---
apiVersion: networking.k8s.io/v1
kind: Ingress
metadata:
  name: proxy
  namespace: infra
  annotations:
    k8s.ngrok.com/modules: oauth
spec:
  ingressClassName: ngrok
  rules:
  - host: <DOMAIN>.ngrok-free.app
    http:
      paths:
      - path: /
        pathType: Prefix
        backend:
          service:
            name: proxy
            port:
              number: 80
---
kind: NgrokModuleSet
apiVersion: ingress.k8s.ngrok.com/v1alpha1
metadata:
  name: oauth
  namespace: infra
modules:
  oauth:
    google:
      emailAddresses:
      - <EMAIL>

Verify that you can access <DOMAIN> using <EMAIL> address via google authentication.

Change <EMAIL> to <EMAIL2> and apply the changes (e.g. kubectl apply ...)

Verify that you can still access <DOMAIN> using <EMAIL> address via google authentication and not via <EMAIL2>

Thank you for the report @pmalek ! Currently, the controller is watching NgrokModuleSet CRs for changes, but it is not triggering reconciliation. We’ll be looking into addressing this gotcha next, but for the time being, operators either need to restart the controller or change some of the ingress objects to trigger reconciliation. One neat way you could do this is to create and set a new NgrokModuleSet CR for your ingress, instead of updating the old CR.
Also, we are happy to look over a PR if you wanna explore the fix yourself.

Thanks for response. I'll wait until you provide the fix as it's not obvious to me where and how to change (ingress controller https://github.com/ngrok/kubernetes-ingress-controller/blob/main/internal/controllers/ingress_controller.go I suppose) the code to make it work.


Download fanbox articles in HTML

Prerequisites

[x] Did you read FAQ section?
[x] Did you test with the latest releases or commit ?

Description
I want to store and view the article's text with the pictures embedded.
Downloading article to an HTML file would preserve this.
The formatting is like a list of text lines and images.
I'm not really sure what would be the best way to go about this, but I'd really appreciate it.

Hey. I am actually quite interested in this idea.
I think technically it won't be toooooooo difficult. But I wanna ask a few questions.

How would you like the html pages to be like?  Exactly as what is shown on fanbox or a really simple html page with no style or whatsoever would do?
What information would like to be shown on the page? Would you have the old txt files replaced by html files? Or do you want to control them in config.ini?

And 1 more, this should only apply to article type posts, right?

Hey. I am actually quite interested in this idea.
I think technically it won't be toooooooo difficult. Hmm.
I will probably start trying it out tomorrow on my side, and maybe pull a request?

#659

Hey. I just made it apply to article posts. is that enough for you?

Hey. I just made it apply to article posts, is that enough?
And firefox, chrome and edge seem to be working fine though, IE seems to be not so compatible. What browser do you use?

PR merged


Security Configuration improvements

The current security configuration API is flexible enough for now, but some improvements are needed.

Being able to define several request matchers that use the same instance of an Authenticator. (Some form of this is currently achievable because the path matcher is a regex, but not all of it)
Add role validation. This could be a required roles field in a request matchers

Done in https://github.com/airamhr9/citrine/commit/7224f90da20a75da038f58630738819b29841e7e


Incorporate alternative sparse matrix-vector multiplication implementations

Jax has a sparse matrix-vector multiplication (SpMV) and it's performance is OK when batched, but in some cases, we'll want the best performance for small batch sizes.  In those cases, we can use DLPack to move tensors out of Jax, apply the fast SpMV, and move the result back into Jax w/ DLPack.  Of course the custom vip should be registered, which is just the transpose of the sparse matrix, or the same matrix if the matrix is symmetric.
For Nvidia GPUs we want to call their cuSparse routine via CuPy, and for CPU the fastest seems to be an ISPC kernel I wrote or MKL if ISPC not available.
However, as a first proof of concept, it would already be useful to have SciPy's CSR matrix done in this manner, to check the overhead.

if we assume the CSR data is static, then we can cache it as part of the closure, so that we only need to move the dense vector/matrix w/ dlpack.


jupyter lab install path not flexible

When installing jupyter via pip3 install --upgrade --no-cache-dir --no-warn-script-location jupyterlab , the install path may not be in $HOME/.local/bin/jupyter which run.sh current depends.
A solution could be to provide a parameter to the fully qualified path.

When installing jupyter via pip3 install --upgrade --no-cache-dir --no-warn-script-location jupyterlab , the install path may not be in $HOME/.local/bin/jupyter

When is this possible? I have not seen this yet.
We can make the path an input to the module to support/fix cases affected by the assumption.

the script assumes that the jupyter exec in located in $HOME/.local/bin/jupyter but this won't always be true.
So when it tests command -v jupyterlab it does a pip3 install --upgrade --no-cache-dir --no-warn-script-location jupyterlab but then jupyter might be installed in a python shim.

Would this be resolved by #294?


src\client\app\main-prod.ts(5,36): error TS2307: Cannot find module './app.module.ngfactory'

at run serve.e2e show
src\client\app\main-prod.ts(5,36): error TS2307: Cannot find module './app.module.ngfactory'.

Since app.module.ngfactory creates at (gulp) run-time, vscode also shows same error when opening this file.
In my opinion this module has to import at run-time. Changing main-prod.ts from
import AppModuleNgFactory from './app.module.ngfactory';
import { platformBrowser } from '@angular/platform-browser';

platformBrowser().bootstrapModuleFactory(AppModuleNgFactory);

to
import { platformBrowser } from '@angular/platform-browser';

let ngFactory = require('./app.module.ngfactory');
let AppModuleNgFactory = ngFactory.AppModuleNgFactory;

platformBrowser().bootstrapModuleFactory(AppModuleNgFactory);

will resolve tslint issue in vscode. May be your issue too. I actually could not reproduce your issue.


Fix #285

Basically, if there's more than 5 tabs in the UITabBarController and the selected index is >=4 we need to use the UIMoreNavigationController to determine the correct value instead of using selectedViewController on the tab bar. What a hassle.
Closes #285

how does one find this issue? lol

Look at the tests :)

O, I see the tests... what app did you have to write to make this happen!?  lol
Looks legit to me.   I'll merge if no one has anything to say.

A client project I'm on for InfiniteRed. o_O
Requires a UITabBar that needs to be contained inside a UINavigationController so we can push the entire tab bar off the stack.


detect-secrets-hook mutates baseline and fails

Problem statement
When detect-secrets-hook runs and mutates the secrets baseline for timestamp and update of the line numbers it also terminates with exit code 1 which means an error, and looks like this:
The baseline file was updated.
Probably to keep line numbers of secrets up-to-date.
Please `git add .secrets-baseline`, thank you.

This means that if I use it in a pre-commit hooks setup it fails my commit process and requires me to manually re-add the secrets baseline and run it again. It is very unintuitive and requires manual work for developers.
Proposal
Show the same error message about baseline being up to date and adding it to the repository but terminate with exit code 0 to make sure a pre-commit script doesn't fail. This is also a strong point IMO because the error isn't that secrets were detected, just that the baseline gets mutated which is not an exception but rather expected behavior of the tool.
Alternative
If terminating with an error code is required at least use a different error code so a scripting on top of it can differentiate between the different errors and act accordingly.

Our development of this hook was under the assumption that the hook would be used together with a specific pre-commit framework. IIRC, this framework does not display output unless the error code is non-zero.
I agree that this isn't the best option for it, but this is the design reason for that choice. If reducing manual effort is the problem, maybe we can auto-add this? The issue is that I'm not sure whether auto-adding is expected behavior with the framework.
Personally, I use the following script in my workflow:
function main() {
    # Run the pre-commit in the tox environment, to mitigate OS issues
    tox -e pre-commit -- run

    # Add changed files (assuming pre-commit modified files)
    if [[ $? == 1 ]]; then
        git diff --staged --name-only --diff-filter=ARM | xargs git add
    fi
}

main

As always, we're open to suggestions, but please keep this compatibility in mind with your changes.

Other frameworks like lint-stated has the same kind of behavior and will not show a message unless there's an error. My current workaround for this is to wrap the detect-secrets-hook with my own to parse the stderr output and then return a 0 exit code. However as I mentioned, I don't think this specific circumstance is of an error'ish nature. The hook updates the timestamp/line numbers of the file and then requests to re-stage it - that's IMHO not something you want to break the commit for.
How about my suggestion of at least changing the exit code number so that we can differentiate between errors (1) and say file mutations like in this topic and exit with say (2) ?

The alternate exit code sounds great to me 👍
As an aside, once #92 is added you’ll have the option of detect-secrets not modifying the baseline on line number and timestamp changes.

Sure thing, see PR at https://github.com/Yelp/detect-secrets/pull/214


Scheduling WakeUp

Is it possible to set wake up schedule for multiple days - say every Sunday at 06:30 Hrs for rsync backup target.

Not yet possible but I also thought about it. I plan on rewriting the frontend (and some of the backend) to make including new features more easy, but I need to find the time for it.

Thanks for the feedback , look forward to new features when you get some spare time...


Allow additional MAF columns

Allow additional MAF columns. Separately, keep track of filterable_effect_function and only_nonsynonymous for easy querying.

Coverage increased (+0.1%) to 52.994% when pulling ff62712ed0e6b88490be9e98ea75d6c60c5b741c on additional_maf into 5ed28a4d54c0646cd01acf47aa55ad30c7d61ac8 on master.


Selection Pane (replacing Scroller Pane)

Description
The scroller panes are cool n all, but I feel like they're lacking some features for me to actually implement them in my designs.

Custom labels instead of lua number (see Brass Funnel)
pre-/suffix for nums, not visible unless in gui (see Brass Funnel)
Additional rows (see Pulse Extender)
Item slots; either item and selection or only one (see Brass Funnel)
Non-numerical items (see Mechanical Roller; probs without the icon, string only; each item got an id similar to <select> in html so that you can reference them by id)
Changing the gap of the focused options (shift)

Describe alternatives you've considered
n/a

I wouldn't neccesairly replace scroller panes, but add the suggested selection panes as an additional block.

Woah woah there, for each custom interface you see on a create block, that is it's whole own behaviour class.
That would be a lot of work, doing via their code. This is why each block only has one.
Besides, switching between them would be difficult as well, as those are not really easy to dynamically switch.
At this point I would need to do something custom.
I actually like the idea of having a way of making easy (and good looking) interfaces. I had this flowing in my mind for a while now.
But I don't think that a "Selection Scroller Pane" is the way.
I would also just use a monitor for more complex stuff at this point.


Indexer API

Phython API for ETH indexer
Storage API
Init Connection to Storage
store(indexing, ...)
There might be different implementations of this, eg. sqlAlchemy storage, postgres storage, later messagebroker storage
indexing: Used to pass information on what parameter needs to be indexed of the events. Probably necessary to make the right query, if not we do not need it.
Events Retrieval
getEvents(address?, eventName?, filter: {parameter=>value}, from_block?, to_block?)
Method to retrieve indexed events and parameters to only return certain events
filter is used to filter for arbitrary event properties.
Event
an event should be
{
   transactionHash,
   blockNumber,
   address,
   eventName,
   args: {
             argname: value,
          },
   blockHash,
   transactionIndex,
   logIndex,
   timestamp  # needs to be queried with the blockNumber
}

later: Transaction Retrieval
getTransactions(filter: from, to)
Get transactions of a certain address
Store events
storeEvents(events)
function to store events in the database
later: Get Event
getEvent(id)
retrieve an event based an some kind of id
later: Delete Event
deleteEvent(id)
delete event based on some kind of an id. Needed to remove events that were removed due to a chain reorg.
Indexer API
Init Indexer Service
Indexer(store, ...)
Inits the indexer service and connects it to the storage. There might be different indexer based on the source of the events to index, eg. NodeIndexer to index it directly from the blockchain, later maybe messagebrokerindexer to index it from a message broker stream.
Index all occured events
indexEvents(addresses, ABIs, eventNames,fromBlock, toBlock)
Update new occured events
eventsUpdater(addresses, ABIs, eventNames) returns function
Returns a function that can be used to update the events that happend since the last call to that same function. Can then be called for example in a gevent loop to update the the database.
later:
Index Transactions
indexTransactions(addresses, onlyValueTransactions=true, fromBlock, toBlock)
Update Transactions
transactionsUpdater(addresses, onlyValueTransactions=true)
Also we probably have to save the information which blocks we indexed.

We did this different and therefore this can be closed.


boxplot outliers

I'm trying to hide outliers in boxplot:
x=dataset("lattice", "singer");
x[:OUT] = false;
plot(x, x="VoicePart", y="Height",outliers="OUT",Geom.boxplot)

However, I got following error:
type Data has no field outliers
while loading In[42], in expression starting on line 3

 in set_mapped_data! at /home/hanny/.julia/v0.3/Gadfly/src/Gadfly.jl:256
 in plot at /home/hanny/.julia/v0.3/Gadfly/src/Gadfly.jl:340

Is this a bug or my misunderstanding? Thanks

I haven't added support for manually specifying outliers yet, I'm afraid. Right now it uses the standard Tukey boxplot rules: fences are median +/- 1.5 times the inter-quartile range, and outliers are anything outside the fences. It should be more customizable though.


Feature: Add follow button to profile

Type of feature
🍕 Feature
Current behavior

We will need a table in the api for follows and unfollows.
A user will group follows (in a future settings feature).
User will not be able to see who is following them (for now)
Users will be able to see who they are following.

Suggested solution
The follow button should show on verified and non-verified profiles.
Note: there willl need to be an unfollow button
Additional context
No response
Code of Conduct

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

Contributing Docs

[ ] I agree to follow this project's Contribution Docs

duplicate https://github.com/open-sauced/insights/issues/715


[ipython] magic command for boilerplate imports

https://ipython-books.github.io/14-creating-an-ipython-extension-with-custom-magic-commands/
https://stackoverflow.com/a/64791087

def import_all(modules: Dict[str, Optional[str]]) -> None:
    """modules = {full_name: alias | None}"""
     globals().update({(alias or full_name): __import__(name) for name, alias in modules.items()})

Setup done. Will add imports as needed. Example:

import all

%magic_import_common

import specific packages identified by key (e.g. plotting)

%magic_import_common key


Incremental mode

Hi, was able to use the aggregator actually, thank you very much!
It has squeezed out 1.8M responses into 500K labels using 4.5 hours on 1 thread on server and 35Gb of memory ;) I think we can incorporate the solution, but I need to implement some enchancements to make it more usefull in production scenario. I will share my thoughts here just to let you know what we think would be useful in our real situation:

for the MLtoRank scenario we need to constantly get new labels and aggregate new judgements. 5 hours delay for adding an extra 1000 labels may be too long and too much electricity to burn. So we need to learn how to perform incremental step. That may include a possibility to backup all distributions and other state, prefill new cells with defaults and perform 1-2 extra iterations.

the prior may be enchanced by implementing a "partial" steps that updates only part of the rows imperically close to the changed ones. Then after one partial step we can perform one full to settle down if needed.

we have a method to order extra 3 marks if the first 3 do not give a confident label. So we will need to output a confidence level for chosen label for the decision to make an extra order.

And one extra off topic:

actually we have a binary labels with extra "grey" option. This is not a true ordinal, because "grey" option is rare, a few percent: it is allowed to use it in complex situations, also it can be produced if we have a lot of confident answers with trues and false. I think we can write some sort of heuristic based on labels probability distribution. E.g. calculate P(white) * 0.3 + P(gray) + P(black) * 0.3 +  and compare with P(white) and P(black).

I would be happy to hear any thoughts regarding this, thank you!

Just to let you know. I was involved once in one of the search giants in calculating kind of freshness PageRank over constantly changing web graph. The algorithm somehow accumulated weight diff and distributed it to peers when weight exceeded some threshold. Also there were some heuristics to intensify processing near new nodes with big weights.
Regarding convergence in incremental scenario: perhaps we can backup values from previous steps and update peers of the worker / task in case there is a big change in value with a flag "include in the next partial iteration". Then run some partial iterations with full ones every 5 partial. If believe that such a technic could produce a VERY fast dawid skene algorithm implementation. ;) Especially for the incremental scenario.

Hi,
One way to achieve the first two points would be to use an online algorithm.

One could use an online algorithm as described in the paper. The idea here would be to first estimate the true labels for all the questions that are available at the beginning. Then you could save the current state (class_marginals, error_rates, question_classes, counts). When you get a new batch of responses, you could load the saved variables, and do an EM pass on the new batch to estimate its correct labels. This would use the state learnt from previous batches and so will help for the current batch. You can then save the new set of parameters and repeat. This has not yet been implemented in this code.
To obtain the correct response of each question, we first calculate a term proportional to the probability (confidence) that a particular label would be correct and then choose the label which has the highest probability. So you can print question_classes[i, :] before this line to view the confidence that each label is correct for the ith question.


create option to ignore line or key (if json)

I think I'm against adding stuff like this to the library because there's an almost limitless set of kinda-arbitrary ways to pre-transform objects or tweak how we convert them to JSON that could in theory be useful in some particular use case. If we add this, why not an option to sort all arrays in the object, or convert numbers to strings, or convert all strings to lowercase, or treat an array of strings as equal to a string containing those same elements separated by commas, etc etc etc.
IMO all these ideas share a few common characteristics with what's asked for in this issue:

they're easy to implement in userland with a preprocessing step
they're kinda arbitrary and will probably only be useful to like 2% of users of diffJson or less
I worry that adding options for them will give the misleading impression to a user that the options are clever in some way and that they should be leaning on the library's functionality for these kind of rules... when in reality the whole diffJson method is basically a crude hack (we just serialize and then diff with diffLines) and if we were to implement these options it'd just be with a preprocessing step that the user could've easily written themselves in userland.

Instead I think the right answer is for users of the library to just do that preprocessing step themselves - e.g. in your case I guess you'd want to deep-clone the objects you're diffing, recurse over the clones deleteing the keys you want to ignore, and then diff the clones.


Sample SecurityContextConstraints does not seem to work certain Enterprise Openshift platforms

Sample SecurityContextConstraints provided in sidb does not seem to work on  IBM Cloud Openshift container Platform
https://github.com/oracle/oracle-database-operator/blob/main/config/samples/sidb/openshift_rbac.yaml
Can someon help me with this, as i am not able to figure out why the CR is not able to find the sidb-scc
oc get SecurityContextConstraints | grep sidb
sidb-scc                       false   ["*"]                                                                                RunAsAny    MustRunAsRange     MustRunAs   MustRunAs   <no value>   false            ["awsElasticBlockStore","azureDisk","azureFile","cephFS","cinder","configMap","csi","downwardAPI","emptyDir","ephemeral","fc","flexVolume","flocker","gcePersistentDisk","gitRepo","glusterfs","iscsi","nfs","persistentVolumeClaim","photonPersistentDisk","portworxVolume","projected","quobyte","rbd","scaleIO","secret","storageOS","vsphere"]

oc get sa | grep sidb
sidb-sa                    2         9m32s

oc get role | grep sidb
use-sidb-scc                                                      2023-03-05T04:45:03Z

oc get rolebinding | grep sidb
use-sidb-scc                                                      Role/use-sidb-scc                                                      5m16s

Stack trace:
oc get SingleInstanceDatabase prebuiltdb-sample
NAME                EDITION   STATUS    VERSION       CONNECT STR   OEM EXPRESS URL
prebuiltdb-sample   Express   Pending   Unavailable   Unavailable   Unavailable

status:
  cloneFrom: Unavailable
  conditions:
  - lastTransitionTime: "2023-03-05T04:53:44Z"
    message: 'pods "prebuiltdb-sample-f58im" is forbidden: unable to validate against
      any security context constraint: [provider "anyuid": Forbidden: not usable by
      user or serviceaccount, provider "pipelines-scc": Forbidden: not usable by user
      or serviceaccount, provider "db2u-c-db2wh-iot-scc": Forbidden: not usable by
      user or serviceaccount, spec.containers[0].securityContext.runAsUser: Invalid
      value: 54321: must be in the ranges: [1000730000,<PHONE_NUMBER>], provider "ibm-restricted-scc":
      Forbidden: not usable by user or serviceaccount, provider "nonroot": Forbidden:
      not usable by user or serviceaccount, provider "sidb-scc": Forbidden: not usable
      by user or serviceaccount, provider "ibm-anyuid-scc": Forbidden: not usable
      by user or serviceaccount, provider "hostmount-anyuid": Forbidden: not usable
      by user or serviceaccount, provider "ibm-anyuid-hostpath-scc": Forbidden: not
      usable by user or serviceaccount, provider "hostnetwork": Forbidden: not usable
      by user or serviceaccount, provider "hostaccess": Forbidden: not usable by user
      or serviceaccount, provider "nfd-worker": Forbidden: not usable by user or serviceaccount,
      provider "ibm-anyuid-hostaccess-scc": Forbidden: not usable by user or serviceaccount,
      provider "nvidia-driver": Forbidden: not usable by user or serviceaccount, provider
      "nvidia-gpu-feature-discovery": Forbidden: not usable by user or serviceaccount,
      provider "nvidia-mig-manager": Forbidden: not usable by user or serviceaccount,
      provider "nvidia-node-status-exporter": Forbidden: not usable by user or serviceaccount,
      provider "nvidia-operator-validator": Forbidden: not usable by user or serviceaccount,
      provider "node-exporter": Forbidden: not usable by user or serviceaccount, provider
      "ibm-privileged-scc": Forbidden: not usable by user or serviceaccount, provider
      "nvidia-dcgm": Forbidden: not usable by user or serviceaccount, provider "nvidia-dcgm-exporter":
      Forbidden: not usable by user or serviceaccount, provider "privileged": Forbidden:
      not usable by user or serviceaccount]'

Fixed, Jus add to link the SecurityContextConstraints to Users

sidb-sa
oracle-database-operator

---
kind: SecurityContextConstraints
apiVersion: v1
metadata:
  name: sidb-scc
  namespace: oracle-database-operator
allowPrivilegedContainer: false
users:
  - system:serviceaccount:oracle-database-operator:sidb-sa
  - system:serviceaccount:oracle-database-operator:oracle-database-operator
runAsUser:
  type: MustRunAsRange
  uidRangeMin: 0
  uidRangeMax: 60000
seLinuxContext:
  type: RunAsAny
fsGroup:
  type: MustRunAs
  ranges:
    - min: 0
      max: 60000
supplementalGroups:
  type: MustRunAs
  ranges:
    - min: 0
      max: 60000
volumes:
  - "*"
allowedCapabilities:
  - "*"

/assign


315 unclear error message when you have mapped temporary locations but no location map

Codecov Report

Merging #316 (4e53573) into main (48aa7d8) will decrease coverage by 0.01%.
The diff coverage is 44.44%.

@@            Coverage Diff             @@
##             main     #316      +/-   ##
==========================================
- Coverage   60.87%   60.85%   -0.02%
==========================================
  Files          78       78
  Lines        7680     7684       +4
==========================================
+ Hits         4675     4676       +1
- Misses       3005     3008       +3

Flag
Coverage Δ

unittests
60.85% <44.44%> (-0.02%)
:arrow_down:

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

Impacted Files
Coverage Δ

..._migration_tools/migration_tasks/loans_migrator.py
19.59% <0.00%> (ø)

...n_tools/mapping_file_transformation/item_mapper.py
17.32% <50.00%> (+0.24%)
:arrow_up:

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 48aa7d8...4e53573. Read the comment docs.


Getting blank video in Allure report using wdio-video-reporter lib

Describe the bug
I am using Webdriver.IO v7 for automation testing and configuring the video support in Allure report for failed testcases.
I followed the wdio-video-reporter documentation on Webdriver.IO official website,
but once e2etest case completes it just adds a blank video in the allure report.
To Reproduce
Add below code in wdio.conf.js:

const video = require('wdio-video-reporter');
reporters: [
[video, {
saveAllVideos: true,       // If true, also saves videos for successful test cases
videoSlowdownMultiplier: 3, // Higher to get slower videos, lower for faster videos [Value 1-100]
videoRenderTimeout: 5,      // Max seconds to wait for a video to finish rendering
}],
[
'allure',
{
outputDir: './results/allure-raw',
disableWebdriverStepsReporting: true,
disableWebdriverScreenshotsReporting: true,
},
],
],

Expected behavior
It should attach a full video of the automation test in allure report
Environment (please complete the following information):
Using below packages:

"wdio-video-reporter": "^3.5.0",
"@types/mocha": "^8.2.0",
"@wdio/allure-reporter": "^7.20.3",
"@wdio/cli": "^7.20.3",
"@wdio/local-runner": "^7.20.3",
"@wdio/mocha-framework": "^7.20.3",

Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Smartphone (please complete the following information):

OS: Windows 10
Browser: Google Chrome

It seems to be a bug with recent webdriver.io versions.
Please take action on this.

Thanks for reporting!

We greatly appreciate any contributions that help resolve the bug. While we understand that active contributors have their own priorities, we kindly request your assistance if you rely on this bug being fixed. We encourage you to take a look at our contribution guidelines or join our friendly Discord development server, where you can ask any questions you may have. Thank you for your support, and cheers!

@arpitgoyal43 increase the reporterSyncTimeout to 30000 default was 5000. In addition increase videoRenderTimeout to around 80 sec

@bhanuagarwal73 : Tried the suggested way, but still getting an empty video. PFA the snapshots of wdio.conf.js

reporters: [
'Spec',
[video, {
reporterSyncTimeout: 3000,
saveAllVideos: true,       // If true, also saves videos for successful test cases
videoSlowdownMultiplier: 3, // Higher to get slower videos, lower for faster videos [Value 1-100]
videoRenderTimeout: 80,      // Max seconds to wait for a video to finish rendering
}],
[
'allure',
{
reporterSyncTimeout: 3000,
outputDir: './results/allure-raw',
disableWebdriverStepsReporting: true,
disableWebdriverScreenshotsReporting: false,
},
],
],

@arpitgoyal43 mind providing a reproducible example to help us investigate the issue?


Issue #936 - packets experiencing parsing errors are not automaticall…

…y dropped

Intended to address issue https://github.com/p4lang/p4-spec/issues/936

Reviewers, please feel free to forward a link to this PR to anyone you think should know about it or comment on it.  Thanks.  I do not plan to merge this in before we have a public P4.org architecture work group meeting to ask for comments, probably within a few weeks from now, announced in p4-arch email list.