On press of marker pin i am not getting modal?

hey guys i am trying to get modal on click of map marker but could not see any modal if i am fetching marker from backend. i am using es6 syntax to i think there is some binding issue i tried various of binding but could not find success
Here is the code in which i can see modal easily
openModal4(id){
    this.refs.modal4.open();
  }

<MapView
          ref="map"
          .....
          >
          <MapView.Marker coordinate={this.state.region} onPress={this.openModal4.bind(this)}/>
        </MapView>
      <Modal style={[styles.modal,showUserLocation={true}
          region={this.state.region}
          style={styles.map}
          rotateEnabled={false}
          showsCompass={false} styles.modal4]} position={"bottom"} ref={"modal4"}>
      </Modal>

and here is the code in which i am fetching marker dynamically from backend but could not see modal on marker press
openModal4(id){
    this.refs.modal4.open();
  }
<MapView
          ref="map"
          .....
          >
          {this.state.markers.map(function(marker){
            return(
              <MapView.Marker coordinate={marker.latlng} key={marker.id} onPress={this.openModal4} />
            );
          })}
        </MapView>
      <Modal style={[styles.modal, styles.modal4]} position={"bottom"} ref={"modal4"}>
      </Modal>

can any body help me solve this issue it would be a great help thanks a lot in advance.

The problem is in your modal. Not related to the lib.
First try your modal without map.


test: Enable contract creation through a deposit test

Description
This was failing due to deposits not getting any gas. This was fixed
in https://github.com/ethereum-optimism/optimistic-specs/pull/386.

nice!

Edit: I may have spoken too soon. There's no reason that this test should be failing and it needs to be dealt with at some point.

Closing to follow up in the mono repo later.


Feat: Want to enhance the login page

Describe the feature
I want to improve the header and footer of the login page to enhance its appearance.
Add ScreenShots

Record

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

@Bhumika1312 Assigned!
Make sure to attach before & after screenshots in your PR. Also, star the repo before working on it. While creating your PR, ensure you pull all existing changes beforehand to avoid conflicts.
Happy open-source contributing!


Best Practice - Usage of CultureInfo in  IRequestHandler Handle

Hi,
In the handle method of the IRequestHandler I want to be sure that the
Thread.CurrentThread.CurrentCulture = culture;
Thread.CurrentThread.CurrentUICulture = culture;
are set to the "users" culture.
Now I'm setting the in the IApplicationBuilder.Use handler, but it seems that the Handle method is handled on a other thread (so it's not taken over the culture settings).
What is the best way to solve this.

No idea! Try StackOverflow.

FYI if you're in ASP.NET Core, there are other ways of setting the culture:


better toml handling

This is taken from nushell's repo but instead of being for better toml handling of the configuration system, it's for from toml and to toml, since we're using nu script for the config system. The real key to "better toml handing" is really just using the toml_edit crate.

Awesome, thanks for opening this issue. I don't use nushell a lot but fixing this use case would make me use it more. Will rewrite what I wrote in old issue:
I want to modify some toml files (change the values under keys). These toml are configurations for another ecosystem (this is not to do with the config command).
Currently with open x.toml | modify_thing | save x.toml the saved toml file has comments stripped and its keys and sections(/tables) sorted to be in alphabetic order. These are developer facing files, so reordering entries and removing comments is a negative.
Also does reordering happen for JSON modification...? Being able to modify these files directly from the term is super helpful but retaining entry order, non-data and their formatting (indent width,) is necessary

If our json tools were wrote with our nu-json crate then I believe ordering should be maintained as well as handling json comments.

well, as no one claimed this I can get it

you got it. have fun!

Bumping this up, seems a good issue to fix.
@GabrielBG0 - you managed to fix this problem?

No, I started to some changes but I think I bit more than I could chew here.
I'm afraid I lack both Rust and project knowledge to be able to fix it.
But if someone wants to check out what I did until now and give it a try u can finde the code here


AssimpNet source code

Hello, what is the version number of assimpnet? Where can I get the source code? Thank you

This is a slightly improved version of the original AssimpNet from https://github.com/assimp/assimp-net
The Ab3d.PowerToys.Assimp is using a slightly improved version. The source for that version can be obtained by accessing the AB4D branch: https://github.com/ab4d/assimp-net
In the near future, I am planning to provide a new Ab3d.AssimpNet assembly that will be based on the existing code but will be improved by using new .Net features, especially Span. This will be used so it will be possible to access the original positions, indices and other arrays without copying data to managed memory.


update metadata publishing requirement (#179)

Fixes #179.
For @wmo-im/tt-wismd review.
cc @golfvert

I fully support that change. Subtopic for metadata is not useful, and can create confusion.


[#5509] Allow cryptography 3.2 with OpenSSL 1.0.2.

Scope
Allow cryptography 3.2 on platforms with OpenSSL 1.0.2.
This is an issue on python-package, where the necessary CRYPTOGRAPHY_ALLOW_OPENSSL_102 env var is set for normal compat tests, but not propagated for elevated tests run through sudo.
Changes
Added a check for the relevant env var and pass it to sudo if existing.
Drive-by change:

support for FreeBSD 12.

How to try and test the changes
reviewers: @adiroiban

needs-review

I think that this should be fixed by updating the compat based on chevah/compat#103

Fine by me. Closing this…


add debounced in search

Fixes #1579

docs/.vitepress/theme/components/icons/IconsOverview.vue and docs/.vitepress/theme/components/icons/IconsCategoryOverview.vue need to be updated after this change.

updated IconsCategoryOverview and IconsOverview


Exception thrown when tryting to use serviceaccount in EKS to access CloudWatch

Describe the bug
I am using a third-party library which uses AWSSDK.Cloudwatch to send logs( https://github.com/Cimpress-MCP/serilog-sinks-awscloudwatch/issues/110 ).
I am running my code in a pod on EKS and have a service account set up using AssumeRoleWithWebIdentity to get the correct permissions to communicate with Cloudwatch. The problem is that when AmazonCloudWatchLogsClient tries to use the credentials in the pod it fails with an exception.
Expected Behavior
That the credentials from the service account is used without throwing an exception
Current Behavior
I get this exception:
2022-11-07T10:21:08.9073970Z Error initializing log stream. No logs will be sent to AWS CloudWatch. Exception was System.InvalidOperationException: Assembly AWSSDK.SecurityToken could not be found or loaded. This assembly must be available at runtime to use Amazon.Runtime.AssumeRoleAWSCredentials.
 ---> System.IO.FileNotFoundException: Could not load file or assembly 'AWSSDK.SecurityToken, Culture=neutral, PublicKeyToken=null'. The system cannot find the file specified.
File name: 'AWSSDK.SecurityToken, Culture=neutral, PublicKeyToken=null'
   at System.Reflection.RuntimeAssembly.InternalLoad(ObjectHandleOnStack assemblyName, ObjectHandleOnStack requestingAssembly, StackCrawlMarkHandle stackMark, Boolean throwOnFileNotFound, ObjectHandleOnStack assemblyLoadContext, ObjectHandleOnStack retAssembly)
   at System.Reflection.RuntimeAssembly.InternalLoad(AssemblyName assemblyName, RuntimeAssembly requestingAssembly, StackCrawlMark& stackMark, Boolean throwOnFileNotFound, AssemblyLoadContext assemblyLoadContext)
   at System.Reflection.Assembly.Load(AssemblyName assemblyRef)
   at Amazon.Runtime.Internal.ServiceClientHelpers.GetSDKAssembly(String assemblyName)
   at Amazon.Runtime.Internal.ServiceClientHelpers.LoadServiceConfigType(String assemblyName, String serviceConfigClassName)
   at Amazon.Runtime.Internal.ServiceClientHelpers.CreateServiceConfig(String assemblyName, String serviceConfigClassName)
   at Amazon.Runtime.AssumeRoleWithWebIdentityCredentials.CreateClient()
   --- End of inner exception stack trace ---
   at Amazon.Runtime.AssumeRoleWithWebIdentityCredentials.CreateClient()
   at Amazon.Runtime.AssumeRoleWithWebIdentityCredentials.GenerateNewCredentialsAsync()
   at Amazon.Runtime.RefreshingAWSCredentials.GetCredentialsAsync()
   at Amazon.Runtime.Internal.CredentialsRetriever.InvokeAsync[T](IExecutionContext executionContext)
   at Amazon.Runtime.Internal.RetryHandler.InvokeAsync[T](IExecutionContext executionContext)
   at Amazon.Runtime.Internal.RetryHandler.InvokeAsync[T](IExecutionContext executionContext)
   at Amazon.Runtime.Internal.CallbackHandler.InvokeAsync[T](IExecutionContext executionContext)
   at Amazon.Runtime.Internal.CallbackHandler.InvokeAsync[T](IExecutionContext executionContext)
   at Amazon.Runtime.Internal.ErrorCallbackHandler.InvokeAsync[T](IExecutionContext executionContext)
   at Amazon.Runtime.Internal.MetricsHandler.InvokeAsync[T](IExecutionContext executionContext)
   at Serilog.Sinks.AwsCloudWatch.CloudWatchLogSink.CreateLogGroupAsync()
   at Serilog.Sinks.AwsCloudWatch.CloudWatchLogSink.EnsureInitializedAsync()
   at Serilog.Sinks.AwsCloudWatch.CloudWatchLogSink.EmitBatchAsync(IEnumerable`1 events).

Reproduction Steps
Create a service account in and use it in an EKS pod when instantiating AmazonCloudWatchLogsClient using the empty construtor.
Possible Solution
Make the AWSSDK.CloudWatchLogs nuget package have a dependency on the AWSSDK.SecurityToken nuget package.
Additional Information/Context
No response
AWS .NET SDK and/or Package version used
AWSSDK.CloudWatchLogs <IP_ADDRESS>
Targeted .NET Platform
.:NET 6.0
Operating System and version
Windows 11

Hi @skovsende,
Good morning.
Thanks for reporting the issue. Does your third-party library which uses AWSSDK.Cloudwatch also includes dependency to AWSSDK.SecurityToken? AssumeRoleWithWebIdentity uses AmazonSecurityTokenServiceClient.AssumeRole() behind the scenes which resides in Amazon.SecurityToken, that requires AWSSDK.SecurityTokenpackage. Could you manually add packageAWSSDK.SecurityToken` and see if it resolved the issue?
Thanks,
Ashish

Yes, if I add a dependency to AWSSDK.SecurityToken it works. But that is very unintuitive and doesn't follow the principle of least surprise. If AWSSDK.CloudWatchLogs needs AWSSDK.SecurityToken to function properly I would suggest adding a dependency to that package.
If you don't want to add that dependency, might I suggest that the code in AssumeRoleWithWebIdentityCredentials.CreateClient throws an exception that is more informative when it fails to instantiate AmazonSecurityTokenServiceConfig? Perhaps mention the fact that the AWSSDK.SecurityToken nuget package needs to be referenced as well?
But really - the first suggestion is the "correct" solution - at least it's the one that would surprise the least .net developers IMHO.

AssumeRoleWithWebIdentity

@skovsende

Yes, if I add a dependency to AWSSDK.SecurityToken it works. But that is very unintuitive and doesn't follow the principle of least surprise. If AWSSDK.CloudWatchLogs needs AWSSDK.SecurityToken to function properly I would suggest adding a dependency to that package.
If you don't want to add that dependency, might I suggest that the code in AssumeRoleWithWebIdentityCredentials.CreateClient throws an exception that is more informative when it fails to instantiate AmazonSecurityTokenServiceConfig? Perhaps mention the fact that the AWSSDK.SecurityToken nuget package needs to be referenced as well?
But really - the first suggestion is the "correct" solution - at least it's the one that would surprise the least .net developers IMHO.

@skovsende AWSSDK.CloudWatchLogs is the package to send log messages to CloudWatch and perform API operations on CloudWatch. It doesn't assume any role and hence should not add dependency to AWSSDK.SecurityToken. The permissions are attached to EKS pod via role which lets assume a role. That role could have permission to send messages to CloudWatch logs, or, let's say, access S3 bucket. Also refer IAM roles for service accounts), this functionality allows application to pass JWT token to the AWS STS AssumeRoleWithWebIdentity API operation and receive IAM temporary role credentials. Applications which make use of this feature should include dependency on AWSSDK.SecurityToken package which enables to assume the role.
Thanks,
Ashish

I understand how and why it works like it does. The problem is that when instantiating the client without specifying AWSCredentials it uses FallbackCredentialsFactory which has hardcoded credentials in the REset() method. Instead of figuring out which Credentials are actually available, it just uses a set list even though dependant assemblies aren't present.
What would happen if I instantiate AmazonCloudWatchLogsClient with the empty constructor, runs in a pod under a service account that doesn't have permissions to access CloudLogs so I instead set EnvironmentVariables with credentials? My guess is that we would never get to reading the environment credentials because we fail AssumeRoleWithWebIdentityCredentials.Create() before I even get to trying to load EnvironmentVariablesAWSCredentials. So now my code fails because I don't include a dependency that I don't need...
Anyway - clearly you think this works as it should so I'm closing the issue, and then I just hope the next person running into the problem falls over this issue.

AWSSDK.SecurityToken

this one worked for me

I have same issue, with EKS Pods communicating with S3 service. Bad design, that dependency on Token library should be handled in the package.

This post saved me. I agree that this is annoying. The dep is only use case specific and not code specific.


Show private key

It would be quite useful to be able to retrieve private key (probably only after hitting some reveal button) so that it can be easily copied into 3rd party wallet.

Generally that's not a good idea so it would be better to understand the use-case. If it's to be copied into a 3rd party wallet app, it's better to use the encrypted JSON V3 wallet format. Otherwise this would require having an API endpoint on Bee that exposes the raw private key, which may end up a security issue.


Add labels to policies

Currently policies are enriched with metadata via annotatations.
Instead (or in addition), could we add labels to the policies?

Would provide a workaround for https://github.com/kubernetes-sigs/kustomize/issues/3887#issuecomment-1229717161
Would allow easily show metadata when using kubectl via the  -L, --label-columns=[] argument. e.g. kubectl get clusterpolicy -L policies.kyverno.io/category

Annotations are quite important right now on Kyverno policies as they're the means by which the whole filtering system works on the policy webpage. That plus the rendering program we use to generate Markdown from the policies relies upon it. We'll consider it, but it's not that easy to just switch over.


Support arrays of mixed-dimensionality

At the moment all our image arrays are 4D (CZYX) and each one of our label arrays is 3D (ZYX). This property is visible in the .zarray files, and in the folder structure. When the dimension along Z is dummy (a single Z plane), we still use the 4D/3D structure, with shape like (num_channels, 1, num_y, num_x) or (1, num_y, num_x). Also ROIs are defined in the same way: they are always 3D shapes (defined by 6 numbers), and in some cases the Z part is dummy (starting at 0 and ending at pixel_size_z, corresponding to a single pixel).
The perspective is that we will handle arrays with mixed dimensions, which can be up to 5D (TCZYX) but also lack some of the intermediate channels (like TCYX), see https://github.com/fractal-analytics-platform/fractal-tasks-core/issues/149#issuecomment-1289379988:

Also, while we are not tackling time-data yet, maybe we should start thinking about this topic for such design decisions. Eventually, we will also process time-resolved data, so data may be 2D, 3D, 4D or e.g. 2D + time (=> 3 actual dimensions, but maybe saved as 4D with Z dimension = 1)

Broadly speaking, a possible (preliminary!) plan to support this general case would be to

Have some custom handling of the dimensionality in the zarr-creation tasks.
Consistently use named axis in all other tasks.
Make sure that the relevant functions/tasks are capable of handling arrays of different shapes

Re: point 1
This means that create_zarr_structure and yokogawa_to_zarr would include more logic, to choose the right structure of the target zarr array. This may include something like explicit user-provided parameters on the structure one should expect, or inference from the metadata if that's sufficiently robust. As always, the simplest is to have a couple of small test folders with different cases (e.g. CZYX, TCZYX, TCYX,  and YX?)
Re: point 2
This may be a bit complex, but the nice advantage is that we would be moving even closer to OME-NGFF specs.
Note that sometimes we already have to specify named axes in the OME-NGFF metadata, e.g. in
https://github.com/fractal-analytics-platform/fractal-tasks-core/blob/f85f88032701f06df3ee7ac3ddcf6941540a005f/fractal_tasks_core/napari_workflows_wrapper.py#L204-L215
Re:  point 3
It should not be too challenging for functions with numpy arrays as inputs/outputs (thanks to broadcasting rules). It could be a bit trickier with dask arrays, but my feeling is that we are currently moving towards a direction where dask is mostly used to lazily-load arrays and organize the processing of several small parts (note that this could change, e.g. if we push towards in-task ROI-parallelization, and we may need to depend more heavily on dask arrays.. to be assessed).

Very good overview @tcompa

The perspective is that we will handle arrays with mixed dimensions, which can be up to 5D (TCZYX) but also lack some of the intermediate channels (like TCYX)

Actually, arrays can be n-dimensional. We always expect YX to be there. Anything else is optional. There will often be Z (though not always, we'll need to make the 2D only case work as well, see https://github.com/fractal-analytics-platform/fractal-tasks-core/issues/124). There often will be multiple channels (those typically can just be looped over) and there may be time information (sometimes to be looped over, i.e. process timepoint by timepoint, e.g. for segmentation. Some other times we'll need to process whole time series at once, e.g. to do tracking).
And users may come up with extra dimensions at some point. We don't need to support processing those as long as we don't have clear use cases for them, but in an optimal case, we should fail when we get such OME-Zarr files / it should be easy to adapt a task to them.

Have some custom handling of the dimensionality in the zarr-creation tasks.

That seems good to me. We can be somewhat conservative in adding dimensions. Let's make sure 2D only (https://github.com/fractal-analytics-platform/fractal-tasks-core/issues/124) and time data (https://github.com/fractal-analytics-platform/fractal-tasks-core/issues/169) can be parsed, but hold off on more complex logic.

create_zarr_structure and yokogawa_to_zarr would include more logic

=> Sounds good to me. Let's add complexity where needed for the two issues above. I'll work on small test sets. The 2D is ready, the time one I will need to look into.

Consistently use named axis in all other tasks.

The seems like a very good approach to make sure we're stable when users start introducing different dimensions, when we only have specific ones.

Make sure that the relevant functions/tasks are capable of handling arrays of different shapes

Lets:
a) Find a good way to define what input a task can handle, e.g. in its docstring
b) Let's make sure the tasks then run on the different shapes they are supposed to work + explicitly load them

It could be a bit trickier with dask arrays

Good point. But our current approach should scale quite a while, I hope. Let's re-asses this if it becomes necessary

A lot of discussion is ongoing in:

#398
#420
#403

Adding to this issue, work in https://github.com/fractal-analytics-platform/fractal-tasks-core/pull/557/files introduces the functions get_single_image_ROI and get_image_grid_ROIs which (in the current versions) do require a set of ZYX pixel sizes.
These are obtained through the NgffImageMeta.pixel_sizes_zyx property, which is setting the Z pixel size to 1 if the corresponding channel is missing - and for this reason the import-ome-zarr task remains flexible.
In the future, also these new functions will need to be made more flexible (that is, they should not always require the Z pixel size).


Exiting.In function: 'ParsePAF'

This is the error information:
#5: Unexpected value found! Overlaps are not formatted in the PAF format. Exiting.In function: 'ParsePAF'.
And my paf file looks like:
m170110_073744_42228_c101136512550000001823249906131712_s1_p0/30/0_13974	13974	507	13859	+	utg000001l	88984	24891	38334	2073	13443	255	cm:i:182
m170110_073744_42228_c101136512550000001823249906131712_s1_p0/68/0_3300	3300	204	2272	+	utg000697l	314885	115884	118067	145	2183	255	cm:i:12
m170110_073744_42228_c101136512550000001823249906131712_s1_p0/163/0_4021	4021	22	3956	+	utg000002l	43991	38915	42826	660	3934	255	cm:i:66
m170110_073744_42228_c101136512550000001823249906131712_s1_p0/163/0_4021	4021	158	3980	+	utg000003l	16020	9065	12833	666	3822	255	cm:i:56
m170110_073744_42228_c101136512550000001823249906131712_s1_p0/163/0_4021	4021	115	3879	+	utg002031l	46243	42607	46223	536	3764	255	cm:i:47
m170110_073744_42228_c101136512550000001823249906131712_s1_p0/163/0_4021	4021	22	3879	+	utg002471l	38305	2907	6571	452	3857	255	cm:i:41
m170110_073744_42228_c101136512550000001823249906131712_s1_p0/163/0_4021	4021	158	3653	-	utg002789l	20197	5756	9105	357	3495	255	cm:i:28
m170110_073744_42228_c101136512550000001823249906131712_s1_p0/163/0_4021	4021	1373	3351	-	utg000573l	96945	84380	86308	165	1978	255	cm:i:16
m170110_073744_42228_c101136512550000001823249906131712_s1_p0/163/0_4021	4021	2462	3358	-	utg001664l	52850	1969	2871	58	902	255	cm:i:7
m170110_073744_42228_c101136512550000001823249906131712_s1_p0/163/0_4021	4021	695	1039	+	utg002221l	63915	21629	21945	64	344	255	cm:i:6
m170110_073744_42228_c101136512550000001823249906131712_s1_p0/163/0_4021	4021	669	892	-	utg000531l	192555	100182	100380	49	223	255	cm:i:5
m170110_073744_42228_c101136512550000001823249906131712_s1_p0/163/0_4021	4021	732	890	-	utg002451l	79420	30732	30873	47	158	255	cm:i:4
m170110_073744_42228_c101136512550000001823249906131712_s1_p0/194/0_5043	5043	503	4875	-	utg000004l	55874	37728	50373	557	12645	255	cm:i:50
m170110_073744_42228_c101136512550000001823249906131712_s1_p0/194/0_5043	5043	460	5041	-	utg002719l	29432	101	5867	496	5766	255	cm:i:45
m170110_073744_42228_c101136512550000001823249906131712_s1_p0/194/0_5043	5043	387	5016	-	utg000005l	35283	160	7127	424	6967	255	cm:i:38
m170110_073744_42228_c101136512550000001823249906131712_s1_p0/194/0_5043	5043	283	2987	+	utg000004l	55874	37699	45545	278	7846	255	cm:i:26

Hi,
the only reason this error would happen is if a line could not be fully loaded here:
https://github.com/isovic/racon/blob/master/src/mhap.h#L52-L54
This doesn't include empty lines, because they are removed here:
https://github.com/isovic/racon/blob/master/src/mhap.cc#L206
Are you sure your PAF is formatted correctly?
Best regards,
Ivan.

The PAF file was created by minimap as described in the racon README.md file like this:
minimap -t 24 reads.gfa.fasta all.reads.fq > mappings.paf

Hello Ivan,
Thanks a lot for your response. Do find some error in the PAF file I parsed above?

Hello,
the snip of your *.paf file looks fine to me. Did minimap finish without errors? Can you check the end of your *.paf file with 'tail -n 15'?
Best regards,
Robert

Hello,
I'm having additional issues parsing paf files, as I see racon expects a cm column in the end of each line, but I don't have those, because I generated paf files from canu ovStoreDump utility.
My paf file looks like:

1         9473    1915    9192  -       2         7516     285    7503<PHONE_NUMBER>        7277     255
1         9473     137    1608  +       106914    4602    2862    4280<PHONE_NUMBER>        1471     255
1         9473    6407    6469  +       603386    3919    2075    3157<PHONE_NUMBER>          62     255
1         9473    2437    6293  +       968119    3849     128    3836<PHONE_NUMBER>        3856     255
1         9473       0    8901  +       1084988  11309    2700   11296<PHONE_NUMBER>        8901     255
1         9473      12    8606  -       1084989  11201     275    8664<PHONE_NUMBER>        8594     255
1         9473     339    8713  +       1084990  10999    2919   10986<PHONE_NUMBER>        8374     255
1         9473    5450    6141  -       1318683   5442     141<PHONE_NUMBER>203         691     255
1         9473      60    1772  +       1647386   8047    6306    7965<PHONE_NUMBER>        1712     255
1         9473      12    1758  -       1647389   6763      94    1813<PHONE_NUMBER>        1746     255

Does racon really use this information, or may I just create one additional column with random info written there?
Could you update the tool to accept 12 column TAB-delimited files, as Heng Li describes here ?
Thanks in advance,
Pedro

Hello Pedro,
racon consensus mode filters reads by the number of shared minimizers in order to obtain a single mapping per read. Unfortunately, number of shared minimizers is in the 13th field of the PAF format which is an optional field in the format specification but minimap does produce it. Can the canu utility generate MHAP format instead of PAF?
On the other hand, if you are using racon in the error correction mode, all valid mappings are used for each read. You can add a dummy column in format "cm:i:" to your PAF file, because it will not be used in this mode.
Sorry for the delayed answer and best regards,
Robert
P.s. In your file snip the 10th column seems to have quite large numbers (the column should contain equal bases in the mapping which should be smaller then the read lengths). You should double check if you run the utility or other programs as intended.

Hello Robert,
I did try to generate MHAP output from the ovStoreDump utility, but didn't deepen on that (it should be possible, as overlaps were done with MHAP within canu). I preferred to add an extra column in the paf (used cm:i:15).  I'm running in the error correction mode. Paf file is 433Gb big (I replaced read IDs with numbers) and it's still on the parsing PAF stage (more than 17hours). 560Gb of RAM used, let's see if It will finish.
I'll take a look on the 10th column. Does racon need that column in the --erc mode?
Best,
Pedro

Hello Pedro,
the 10th column is not used in this version of racon.
Best regards,
Robert

Racon 1.0.0 does not use the cm column from paf anymore. Overlaps are filtered by length rather than by number of shared minimizers.


README example has wrong if condition

Which does not trigger issue creation for failing links.
if: ${{ steps.lychee.outputs.exit_code != 0 }}

should be
if: ${{ steps.lychee.outputs.exit_code }} != 0

Hey @pat-s,
I saw some cases where the current syntax was suggested:

https://docs.github.com/en/actions/learn-github-actions/expressions#example-for-composite-action-step
https://timheuer.com/blog/manually-force-a-failure-in-github-action-step/#evaluating-your-condition-and-failing-the-step-manually

Perhaps both work?
Although I think you ran into issues with the existing syntax, right?
Actually maybe even removing the ${{ and }} would work.
https://docs.github.com/en/actions/using-jobs/using-conditions-to-control-job-execution
It seems to be a special case for conditions.

Cool, I've merged the PR. We should also test the version without the curly braces at some point.
Thanks a lot for the fix @pat-s.


How to add X-Amzn-Trace-Id in logger log4j. #Spring #microservice

Like Spring sleuth trace id (X-B3-TraceId), i want to put inside the logger format.
How to add X-Amzn-Trace-Id into logger?
This will enable to watch the request related logs in cloudwatch as in xray, we dont see logs information when we search for trace id.

@prasad8mhatre
If you have access to the global AWS X-Ray recorder (AWSXRay.getGlobalRecorder()), you can call AWSXRay.getGlobalRecorder().currentTraceId().toString() within your code to get the current trace id.
The only prerequisite would be that the AWSXRayServletFilter has already processed the X-Amzn-Trace-Id for the request you're processing. The ServletFilter will check if the incoming request has the X-Amzn-Trace-Id header and create a segment using that id, or will generate a new Trace Id otherwise.
If you only want to log the trace id if a request is sampled, then you can grab the current segment from the recorder and check if it is sampled first:
AWSXRay.getGlobalRecorder().getCurrentSegment().isSampled());

Closing. Please feel free to provide extra comments or open a new issue should more help or suggestion needed.


Players using it

Hey, I hope I write in the right space, GIT Hub is new to me.
I was wondering. Would it be possible for players to use AVE to gleen information about enemies. When a lot of abilities and speels are flying around, my players sometimes ask things like "What does the chalice symbol mean?" and I would love for them just to be able to look for themselves. Would that be somehow possible?

It is possible, What would be the intended workflow? Using targeting?

Targeting would probably be the best idea I think. Since players can't click enemie tokens.


Add keyboard shortcuts to app

Tab to change fields
Esc to disselect.
del to delete
etc...

Closing since this is overly broad. We can file more specific issues for particular parts of the app, like #272.


feat: move devices between locations

My current thinking is that moving devices in bulk should be done on the Location object and moving a single device should be on the Device.
Currently:

Deduplicating UDIDs happens in Location.moveDevices, but not API.moveDevices
Validating UDIDs and array length checking happens in the API method
The Location method chunks its input into multiple arrays to get around the hard limit of 20 devices, then Promise.all’s the multiple API method calls

To do:

[x] Validator for PUT /devices/migrate
[x] ^ Test it
[x] API.moveDevices
[x] Location.moveDevices
[ ] Test how moving a device + owner where the owner owns multiple devices works…
[ ] Better docs for Location.moveDevices (what happens in the above scenario?)
[ ] Add option to disable moving device owner
[ ] Validator for PUT /devices/:udid/migrate
[ ] ^ Test it
[ ] API.moveDevice
[ ] Device.setLocation(location: { id: number })

Moving a device + owner where the owner owns multiple devices will move the other devices with them. Makes sense, but wasn’t what I would’ve guessed. Interesting!


chore: Rename the "doc" folder to "docs"

Rename the "doc" folder to "docs" to be consistent with the tox env
name. RTD also seems to expect "docs" as its first choice, even though
it will discover "doc" as well.

Current coverage is 100% (diff: 100%)

Merging #875 into master will not change coverage

@@           master   #875   diff @@
====================================
  Files          30     30
  Lines        1875   1875
  Methods         0      0
  Messages        0      0
  Branches      305    305
====================================
  Hits         1875   1875
  Misses          0      0
  Partials        0      0

Powered by Codecov. Last update 5493bf8...127329b


CI Build fails for PRs outside of this repo

The CI build fails for PRs made from outside of this repo. It works for branches pushed into this repo directly only.
See https://github.com/Kentico/kontent-sample-app-net/pull/123#issuecomment-848664462

The first thing would be to add -Force argument to the Remove-Item command in publish.ps1 - https://github.com/Kentico/kontent-sample-app-net/blob/master/publish.ps1#L3

Fixed and tested in #133

Fixed and tested in #133


replace the non-word characters or first digit with underscores and print warning message

Fix https://issues.redhat.com/browse/ACM-14296
oc logs clusterlifecycle-state-metrics-v2-7c8968d85d-g2kdh | grep W0
W0918 08:42:29.812111       1 client_config.go:618] Neither --kubeconfig nor --master was specified.  Using the inClusterConfig.  This might not work.
W0918 08:42:29.924393       1 managedclusterlabels.go:53] Label key 'cluster.open-cluster-management.io/clusterset' was modified to 'cluster_open_cluster_management_io_clusterset'
W0918 08:42:29.924445       1 managedclusterlabels.go:53] Label key 'velero.io/exclude-from-backup' was modified to 'velero_io_exclude_from_backup'
W0918 08:42:29.924484       1 managedclusterlabels.go:53] Label key 'feature.open-cluster-management.io/addon-application-manager' was modified to 'feature_open_cluster_management_io_addon_application_manager'
W0918 08:42:29.924537       1 managedclusterlabels.go:53] Label key 'feature.open-cluster-management.io/addon-cert-policy-controller' was modified to 'feature_open_cluster_management_io_addon_cert_policy_controller'
W0918 08:42:29.924590       1 managedclusterlabels.go:53] Label key 'feature.open-cluster-management.io/addon-managed-serviceaccount' was modified to 'feature_open_cluster_management_io_addon_managed_serviceaccount'
W0918 08:42:29.924650       1 managedclusterlabels.go:53] Label key 'feature.open-cluster-management.io/addon-work-manager' was modified to 'feature_open_cluster_management_io_addon_work_manager'
W0918 08:42:29.924685       1 managedclusterlabels.go:53] Label key 'openshiftVersion-major-minor' was modified to 'openshiftVersion_major_minor'
W0918 08:42:29.924748       1 managedclusterlabels.go:53] Label key '5g-test' was modified to '_5g_test'
W0918 08:42:29.924794       1 managedclusterlabels.go:53] Label key 'feature.open-cluster-management.io/addon-config-policy-controller' was modified to 'feature_open_cluster_management_io_addon_config_policy_controller'
W0918 08:42:29.924861       1 managedclusterlabels.go:53] Label key 'feature.open-cluster-management.io/addon-governance-policy-framework' was modified to 'feature_open_cluster_management_io_addon_governance_policy_framework'
W0918 08:42:29.924906       1 managedclusterlabels.go:53] Label key 'local-cluster' was modified to 'local_cluster'
W0918 08:42:29.924977       1 managedclusterlabels.go:53] Label key 'feature.open-cluster-management.io/addon-cluster-proxy' was modified to 'feature_open_cluster_management_io_addon_cluster_proxy'
W0918 08:42:29.925037       1 managedclusterlabels.go:53] Label key 'feature.open-cluster-management.io/addon-hypershift-addon' was modified to 'feature_open_cluster_management_io_addon_hypershift_addon'
W0918 08:42:29.925088       1 managedclusterlabels.go:53] Label key 'openshiftVersion-major' was modified to 'openshiftVersion_major'

/assign  @qiujian16 @elgnay

/lgtm

/cherrypick backplane-2.5

/cherrypick backplane-2.6


Add support to socket timeout, health check and keepalive

Hello,
When TCP connection lost without proper disconnection, existing redis connection may become broken and the next request may take a long time before it reaches system default socket timeout, which is around ~15 mins with default linux config ( https://stackoverflow.com/questions/20403118/tcp-ceases-retransmissions-before-reaching-the-default-of-15-attempts-after-phy ).
In this situation, redis operation will be blocked for a long time before an exception is thrown. This is observed in 1.3.1
  File "/home/bolt/.local/lib/python3.8/site-packages/example-0.0.0-py3.8.egg/example/registry.py", line 63, in get_redis
    get_redis= await self.redis_client.smembers(key)
  File "/home/bolt/.local/lib/python3.8/site-packages/aioredis/connection.py", line 186, in _read_data
    obj = await self._reader.readobj()
  File "/home/bolt/.local/lib/python3.8/site-packages/aioredis/stream.py", line 102, in readobj
    await self._wait_for_data('readobj')
  File "/usr/local/lib/python3.8/asyncio/streams.py", line 517, in _wait_for_data
    await self._waiter
  File "/usr/local/lib/python3.8/asyncio/selector_events.py", line 846, in _read_ready__data_received
    data = self._sock.recv(self.max_size)
TimeoutError: [Errno 110] Operation timed out

With issue #184 , connection timeout is added. It would be nice to support socket timeout, socket keepalive and health check to better handle connection loss.
Thanks!

i met the same problem when i use redis pool execute some command, it blocked。hope add timeout option too

I have the same error, it is related to the keepalive in the server configuration

@guoyiang -
Thank you for your submission! Would you be willing to submit proposed implementation in a PR? This would be a great addition to the library.

@seandstewart I like copying others' code, so this is a perfect place to copy from: redis-py implements it in the exact same way we should: https://github.com/andymccurdy/redis-py/blob/8c176cdc7f36e2cbcd3254768766035a7b7cd8b3/redis/connection.py#L589-L593
We would add this kind of code (not the same but...) in asyncio.connection: https://github.com/aio-libs/aioredis/blob/f5907fa8631a6f36684e081ec78e4d9267c6a992/aioredis/connection.py#L128-L143
Passing in the methods would be the same as passing in timeout (for connection timeout).

This issue should be resolved as of #891 - please feel free to pull in the latest master and test to ensure this is the case.


Make sure ssl_opts are passed-through when connecting in the mc_monit…

mc_monitor should include ssl_opts. Closes #114

Good job, thanks!


Create LinkDestinationSummary elements with multi-language variant content for symbols

Bug/issue #, if applicable: SR-15574 rdar://86298116
Summary
This change creates LinkDestinationSummary elements with multi-language variant content for symbols. Before this change only the main summary element was created without its variant content.
Dependencies
n/a
Testing
Convert a multi-language documentation catalog and include the --emit-digest flag so that Swift-DocC will create the linkable-entities.json file.
(For testing purposes it may be convenient to also set the DOCC_JSON_PRETTYPRINT environment to YES to get output that's more easily inspectable.)
The content of the linkable-entities.json file should include the variant content of each interface language for each symbol "entity". Each variant should contain only the information that's different in the variant compared to the summarized element.
Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.

[x] Added tests
[x] Ran the ./bin/test script and it succeeded
[x] Updated documentation if necessary

@swift-ci please test

@swift-ci please test

@swift-ci please test

@swift-ci please test

@swift-ci please test

@swift-ci please test

@swift-ci please test


Translator Text API Languages and Authentication

 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.:x: Jann-Skotdal sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

@Jann-Skotdal : Thanks for your contribution! The author, @v-pawal, has been notified to review your proposed change.

@nitinme - I've assigned this PR to you for review and approval. It's in the public repo (which is unusual for CogSvcs prs) and I am unable to assign it to @v-pawal for some reason.

@Jak-MS : I still work for the Translator team, but off-network. I have asked @v-pawal to sign off on the PR

#sign-off


Modular Limb Interaction

Enable the movement system to adapt dynamically to limb configurations.

[x] Integrate limb detection from the entity creator into the movement system.
[x] Implement logic for reduced speed and stability when limbs are removed or damaged.
[ ] Simulate limping or movement inefficiencies based on missing limbs.
[x] Ensure performance scales efficiently for creatures with multiple limbs (e.g., centipedes).

PR #100


Error "startswith first arg must be str or a tuple of str, not list"

Hi,
I tested plex_assistant.command service found in Developer Tools > Services with the example and I have the following error: "Failed to call service plex_assistant/command. startswith first arg must be str or a tuple of str, not list"
I installed plex_assistant using hacs and I am on freenas.
Thanks

Do you mean you used the default example by hitting "Fill Example Data"? So, like this:
command: Play Evil Dead on the Downstairs TV
cast_delay: 6

Or did you send a different command and if so, what?

Also could you paste the whole error (click on it and copy paste the result)?

Hi, same problem here with the log
Logger: homeassistant.components.websocket_api.http.connection.140536386052304
Source: custom_components/plex_assistant/process_speech.py:20
Integration: Home Assistant WebSocket API (documentation, issues)
First occurred: 8:54:09 AM (2 occurrences)
Last logged: 8:54:16 AM

startswith first arg must be str or a tuple of str, not list
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/components/websocket_api/commands.py", line 130, in handle_call_service
    connection.context(msg),
  File "/usr/src/homeassistant/homeassistant/core.py", line 1260, in async_call
    task.result()
  File "/usr/src/homeassistant/homeassistant/core.py", line 1299, in _execute_service
    await self._hass.async_add_executor_job(handler.func, service_call)
  File "/usr/local/lib/python3.7/concurrent/futures/thread.py", line 57, in run
    result = self.fn(*self.args, **self.kwargs)
  File "/config/custom_components/plex_assistant/__init__.py", line 109, in handle_input
    PA
  File "/config/custom_components/plex_assistant/process_speech.py", line 20, in process_speech
    if command.startswith(controls[control]):
TypeError: startswith first arg must be str or a tuple of str, not list```

Sorry for the delay, this was an error in the French translation. Should be fixed in latest version ( 0.1.8 )


Difference between Nord and Northem

Hi guys,
Thank you for the atom themes. They are awesome.
Can you tell me which is the difference between Nord theme and Northem? Is Northem the precursor of Nord? Or is it a separate theme?

Hi Adrian,
nice to hear you like it.
Northem can be seen as a kind of precursor, but basically they are separate themes.
Nord has been designed with more precisely calculated colors and a focus on the bluish ambient. It is also generally more advanced and provides a lot of port projects. Nord will be updated frequently and is now the main color palette project.
There will be hotfixes for Northem port projects, but it will not be continued. Maybe some Northem port projects will be created by adopt it from the specific Nord project, but this is not yet clear.
In summary you should prefer Nord since it is the "better" and more advanced color palette.


Scaffold Teams tab app with SSO fails to install<EMAIL_ADDRESS>

I am trying to scaffold a tab app with SSO for teams. When it is scaffolding out the application it fails when installing packages. The process stops when it attempts to install<EMAIL_ADDRESS>To Reproduce
Steps to reproduce the behavior:

Complete Azure AD app registration
Create a new directory in Windows
Using cmd cd to new directory
yo teams
Select the following options:

App Gen 2.17.1
Create in current directory
manifest v1.8
Tab (application)
Loading indicator on tab = Y
Without header bar = Y
Test Framework = N
App Insights = N
Configurable tab
In a team
SSO = Y
Sharepoint = N

Expected behavior
I expected the scaffolded app to be created without any errors. I have tried several suggested solutions on the internet to fix it but nothing works.
I installed the last version of node-sass (^5.0.0) and when I tried gulp ngrok-serve (with a valid ngrok account) it gives the same error with the dependency on version 4.14.1 and will not start.
 You must install peer dependencies yourself. 33618 warn<EMAIL_ADDRESS>No repository field. 33619 warn<EMAIL_ADDRESS>No license field. 33620 verbose stack Error<EMAIL_ADDRESS>postinstall: node scripts/build.js33620 verbose stack Exit status 1 33620 verbose stack     at EventEmitter.<anonymous> (C:\Users\windows.user\AppData\Roaming\npm\node_modules\npm\node_modules\npm-lifecycle\index.js:332:16) 33620 verbose stack     at EventEmitter.emit (events.js:315:20) 33620 verbose stack     at ChildProcess.<anonymous> (C:\Users\windows.user\AppData\Roaming\npm\node_modules\npm\node_modules\npm-lifecycle\lib\spawn.js:55:14) 33620 verbose stack     at ChildProcess.emit (events.js:315:20) 33620 verbose stack     at maybeClose (internal/child_process.js:1021:16) 33620 verbose stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:286:5) 33621 verbose pkgid<EMAIL_ADDRESS>33622 verbose cwd C:\Users\windows.user\source\repos\TestTabSSO 33623 verbose Windows_NT 10.0.19042 33624 verbose argv "C:\\Program Files\\nodejs\\node.exe" "C:\\Users\\windows.user\\AppData\\Roaming\\npm\\node_modules\\npm\\bin\\npm-cli.js" "install" "--cache-min" "86400" 33625 verbose node v12.18.2 33626 verbose npm  v6.14.10 33627 error code ELIFECYCLE 33628 error errno 1 33629 error<EMAIL_ADDRESS>postinstall:node scripts/build.js`
33629 error Exit status 1
33630 error Failed at the<EMAIL_ADDRESS>postinstall script.
33630 error This is probably not a problem with npm. There is likely additional logging output above.
33631 verbose exit [ 1, true ]
Detailed information

Generator version: [2.17.1]
OS: [Windows 10 Enterprise, Version: 20H2, OS build: 19042.685]
Node version: [12.18.2]
Npm version: [6.14.10]

@dickiebowuk  - thanks for reporting this. I did a quick test with the same node version and could not replicate this. Have you been able to reproduce this multiple times?

Yes this has happened three times. Deleted the project, ran through the tutorial and get the same issue every time.
I’m happy to demonstrate it, if it would help.
Get Outlook for iOShttps://aka.ms/o0ukef

From: Wictor Wilén<EMAIL_ADDRESS>Sent: Saturday, December 26, 2020 12:23:39 PM
To: pnp/generator-teams<EMAIL_ADDRESS>Cc: Richard Bowen<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: #External# Re: [pnp/generator-teams] Scaffold Teams tab app with SSO fails to install<EMAIL_ADDRESS>(#165)
@dickiebowukhttps://github.com/dickiebowuk - thanks for reporting this. I did a quick test with the same node version and could not replicate this. Have you been able to reproduce this multiple times?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/pnp/generator-teams/issues/165#issuecomment-751350965, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASHRQPXEXM3YRCQUEWMBUCTSWXIUXANCNFSM4VFZGECA.

Thanks for the update. Would you mind sharing the full logs and the output from the console when scaffolding. Also detailed logs from npm install node-sass --loglevel verbose would be great if you could share.

Hi
Attached is the full log file produced from scaffolding the app. I ran the node-sass command below, but it didn’t produce a log file.
Best regards,
Richard Bowen
From: Wictor Wilén<EMAIL_ADDRESS>Sent: 27 December 2020 13:43
To: pnp/generator-teams<EMAIL_ADDRESS>Cc: Richard Bowen<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: #External# Re: [pnp/generator-teams] Scaffold Teams tab app with SSO fails to install<EMAIL_ADDRESS>(#165)
Thanks for the update. Would you mind sharing the full logs and the output from the console when scaffolding. Also detailed logs from npm install node-sass --loglevel verbose would be great if you could share.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/pnp/generator-teams/issues/165#issuecomment-751469507, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASHRQPSNSVGABATVLZZTYUTSW42VRANCNFSM4VFZGECA.

I captured the output from the cmd window when installing node-sass. The log is attached.
Best regards,
Richard Bowen
From: Wictor Wilén<EMAIL_ADDRESS>Sent: 27 December 2020 13:43
To: pnp/generator-teams<EMAIL_ADDRESS>Cc: Richard Bowen<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: #External# Re: [pnp/generator-teams] Scaffold Teams tab app with SSO fails to install<EMAIL_ADDRESS>(#165)
Thanks for the update. Would you mind sharing the full logs and the output from the console when scaffolding. Also detailed logs from npm install node-sass --loglevel verbose would be great if you could share.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/pnp/generator-teams/issues/165#issuecomment-751469507, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASHRQPSNSVGABATVLZZTYUTSW42VRANCNFSM4VFZGECA.

Attached is the cmd output when I attempt to scaffold the teams tab app.
Best regards,
Richard Bowen
From: Wictor Wilén<EMAIL_ADDRESS>Sent: 27 December 2020 13:43
To: pnp/generator-teams<EMAIL_ADDRESS>Cc: Richard Bowen<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: #External# Re: [pnp/generator-teams] Scaffold Teams tab app with SSO fails to install<EMAIL_ADDRESS>(#165)
Thanks for the update. Would you mind sharing the full logs and the output from the console when scaffolding. Also detailed logs from npm install node-sass --loglevel verbose would be great if you could share.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://github.com/pnp/generator-teams/issues/165#issuecomment-751469507, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASHRQPSNSVGABATVLZZTYUTSW42VRANCNFSM4VFZGECA.

I looked again at the errors when it was trying to install node-sass 4.14.1. npm couldn't hit the url and download the required files due to proxy settings for npm. I could download the files using chrome, so I could access it from my machine. I reset the proxy settings to null and tried again and the app scaffolded as expected and gulp ngrok-serve worked as well.

I looked again at the errors when it was trying to install node-sass 4.14.1. npm couldn't hit the url and download the required files due to proxy settings for npm. I could download the files using chrome, so I could access it from my machine. I reset the proxy settings to null and tried again and the app scaffolded as expected and gulp ngrok-serve worked as well.


Review sniff handling of PHP 7.4 spread operator in array expressions

Since PHP 7.4 you can use the spread operator (aka splat operator ...) to unpack array expressions.
$parts = array( 'apple', 'pear');
$fruits = array( 'banana', 'orange', ...$parts, 'watermelon'];
// ['banana', 'orange', 'apple', 'pear', 'watermelon'];

This is already covered upstream in PHPCS 3.5.0 so we need to add Generic.WhiteSpace.SpreadOperatorSpacingAfter to cover the case for array expression unpacking, since Core ruleset uses Squiz.Functions.FunctionDeclarationArgumentSpacing sniff already.
Refs:

Rules about formatting the spread operator: #1762
PHP RFC: https://wiki.php.net/rfc/spread_operator_for_array

Related to #764

All sniffs have been reviewed and fixed to support modern PHP in as far as my imagination reached.
If any more situations would be discovered which need to be taken into account, a new issue should be opened with code samples etc.
Closing this now as fixed.


Add date / time selector to proposal generator

Story
As a proposer
I want a datetime selector not the weird duration setter
So that I dont get a constantly changing timestamp that is confusing and requires me to hit a precise value if I want a round number
Acceptance Criteria

[ ] I can select the date and time that I want it to enact from some datetime picker thingy
[ ] I can see how far away that is
[ ] I am warned if that duration is less than the min duration or is within say 2 hours of that duration to give time to finish the proposal

Tasks
Sketch
Additional details / background info

This may be a dupe ...

@JonRay15 A datetime selector isn't an ideal element - the 'constantly changing timestamp' is the way the proposal's limits actually work. They're based on a fixed amount of time away from 'now', defined by a network param. Therefore, using a datetime selector to try to choose the minimum value will automatically get declined when the proposal's submitted, because by the time the user's hit the 'submit' button, the selected value will now be closer than the updated minimum.

We discussed this one previously @ckpringle, I think it needs some UX love


How to combine semiologies #2

original issue on combining semiologies #8 used MinMaxScaler and softmax. MinMaxScaler is sensitive to range and doesn't account for variance. Softmax is only great for the top prediction.
After proportions were supported akin to probabilities #118, we can now combine semiologies in a statistically more sound manner by using proportions and inverse variance weighting as alluded to in #8

in branch Bayesian-correction pull #222 this allows inverse variance weighting by modelling the GIF parcellations as binomial random variables (and the marginals p(semiology) from SS and p(GIF) from TS data), as well as an equal weightings (mean) method.
variance of binomial proportions = p(1-p)/n
I discussed these methods with Chibueze, Tim Stone, Normal WIliams from UCL, and they all agreed should prefer the above (cf both marginals from all-data, or using the data-query results as the estimate of the marginals).
NB this is an approximation as the GIFs are not independent (e.g.if it involves amygdala, likely will involve hippocampus)


Nukes any mention of CollisionGroup.Clickable

About the PR
Just removes it from everything that had it.
Closes  #2181.

Reviews applied, I moved stuff like the chargers and the morgue stuff over to Constructible while I was at it.


Using SAMM to train automatic segmentation

Hello, I was trying to use SAMM to train a model for automatic artery segmentation. I have collected the data needed for this. Is it possible to do this on the 3D slicer extension?

Hi thanks for using the extension. It doesn't support auto-segmentation yet. It's a pending feature. You could use other packages like MedSam. It is also open sourced.

Hi thanks for using the extension. It doesn't support auto-segmentation yet. It's a pending feature. You could use other packages like MedSam. It is also open sourced.

Could I use SAMM to train for semi-automatic segmentation? Also would it be possible to train a model with SAMM outside of 3D slicer? Thank you for the help.

You should be able to. The community has been publishing some pretty interesting SAM variants models with the emphasis on medical images. MedSam as I recommended earlier is one of them. It allows you to fine tune. I'm sure the others would have the same/similar.

Is MedSam on 3d slicer?


Fix Invalid URL

Edited the URL format from "/file" to "/design"

DANG thank you very muchhh!!!💖

This error has apparently not yet been fixed in the tk designer script, but you can run the command python -m tkdesigner.cli

GOAT


Is there a way to see if the notification was received by the device?

I'm trying to see if the notification was received, so that I may send it again if not.   Is there some sort of success or error logic that I could work with?  I'm afraid that if I send multiple notifications within a short time period some may be overwritten.
https://developer.apple.com/library/ios/technotes/tn2265/_index.html#//apple_ref/doc/uid/DTS40010376-CH1-TNTAG23

When the client receives the notification, it can update a Meteor collection or call a Meteor method.  That's the best way for the client to communicate with the server.


Eliminate a pandas normalize_json deprecation warning

Use the currently recommended way to normalize a json by pandas.

Sorry I did not realize this would break the tests under Python 3.5


Add support of #~

https://github.com/leanprover-community/lean4game/issues/210
Easy to fix, just need time for it. (modify the parser!)

We just drop/delete them now in e493a4f7f0d51b6707c9d11bd0488d2f26b58bd0


Running the sample with battery voltage doesn't handle not having an ExHub connected correctly

Instead of printing an error message and exiting, the program just continues, and ultimately fails when it tries to call getBatteryVoltage() on an undefined variable.

Not reporducible on main. Possibly fixed in #66


Abort merge if it fails

This is on top of PR #15, and closes issue #14 by aborting the merge if it fails

@timn let me know what you think, and I'll merge :)


[BUG] Mac M1 environment unable to use zenml CLI

How should I solve this problem?

zsh: illegal hardware instruction  zenml

Could you please add the full traceback as well as the ZenML version and MAC OS version?

Thank you very much for your answer.
My installation steps：
pip  install  -i  https://pypi.doubanio.com/simple/  --trusted-host pypi.doubanio.com  zenml
pip install pytorch-lightning==1.1.8

# add this to ~/.zshrc
eval "$(_ZENML_COMPLETE=source_zsh zenml)"

zenml version

My environment：
mac os version: 11.1

zenml version: 0.3.8
conda version: 4.9.2
python version: 3.8

Could try running this script:
from zenml.version import __version__
print(__version__)

And let us know what you get?

Running this script is normal.

python

Python 3.8.5 (default, Sep  4 2020, 02:22:02)
[Clang 10.0.0 ] :: Anaconda, Inc. on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> from zenml.version import __version__
>>> print(__version__)
0.3.8

As it seems to be working installation wise, I feel like the CLI problem might be due to a zsh configuration rather than a ZenML installation one. Were you able to run zenml init on the CLI? Could you try alternatively using simple bash than zsh?

Run zenml init on the CLI, same error.

When I use bash, I still get an error：

Thank you for the info. I feel like this an underlying problem with Tensorflow on Mac M1. See the following issues:
https://github.com/apple/tensorflow_macos/issues/143
https://github.com/ludwig-ai/ludwig/issues/1101
https://github.com/openeventdata/mordecai/issues/97
Is it possible for you to try the fixes mentioned here https://github.com/apple/tensorflow_macos/issues/143 and let us know if that works? Otherwise I'd ask to please wait until we get rid of the tensorflow dependency (circa ~1 month) for our big refactored release and it should then work!

Hi, there is a version conflict in installing zenml using tensorflow_macos virtual environment。

This will be solved when we get rid of the Tensorflow dependency. Another suggestion: Can you try using our Docker Image instead to run ZenML: https://hub.docker.com/u/zenmldocker

Hi, I tried this way, but I still get an error:

This is strange - it seems like a segmentation fault in the C++ core of tensorflow now. It seems like they have just stopped supporting Mac altogether: https://github.com/tensorflow/tensorflow/issues/42387
Unfortunately at this point, I would suggest you wait for our updated release as it will get us away from the Tensorflow dependency. Sorry for the inconvenience here!


Determine what to do about 'floating' App.Ref packs in source-build repo legs

In the source-build repo legs (at least SDK), we get restores of Microsoft.NETCore.App.Ref, at a recent released version. This is most likely an implicit version that is used when building net6 or net7 targeted binaries.
It's infeasible to generate SBRPs for these, since:

We'd have to do so every month
In most cases they may not even be available publicly until after release day, and so couldn't be checked into SBRP.

My guess is that these are coming in via tests but investigation is warranted.
I think we have a few options for this:

Forcing an older, SBRP compatible ref pack version during repo source-build.
Avoiding building any code that may use a ref pack of an older release (if the code is non-critical)
Simply excluding these prebuilts in repo-level source-build.

I'm a fan of the first or third options there. If these are tests, the behavior they are exhibiting is intentional. We build plenty of code that attempts to target downlevel, because that's a core feature of the SDK. The version of the ref pack probably matters less than the fact that it is a 6.0/7.0 ref pack.
If the issues are coming from non-test code, most likely the TFM should be upgraded.
@dotnet/source-build-internal

@mmitche - Can you link to a concrete occurrence of this that I can take a look at.  I am interesting in knowing the details here on where the differences are coming from the full product source-build and the repo build.  Ideally we would want to eliminate this difference.

I was seeing this in sdk repo-level source-build when I was enabling the baseline there. Let me see whether I can pull it up again.

@MichaelSimons https://dev.azure.com/dnceng-public/public/_build/results?buildId=224422&view=logs&j=2f0d093c-1064-5c86-fc5b-b7b1eca8e66a&t=d2b639a6-cb19-5f1f-66fd-8047f66b3026&l=297

Thanks. Commenting on that issue. That might be a difficult one to solve, if this is a downlevel targeting issue.

[Triage]  As noted we don't think this is going to be a wide spread issue and likely each occurrence will need to be scrutinized carefully.  This particular occurrence will be handled/discussed via https://github.com/dotnet/sdk/issues/30272.


[feature request] Adding a keyboard shortcut

This extension is neat, thank you! It's simple and does one job well.
Do you think it would be possible to add a shortcut as well?

Thank you for the kind words! Keyboard shortcuts should be easy to add, I'll take a look!

Added keyboard shortcut into 1.1.0, it's already in store for Firefox, will be available shortly for Chrome and Edge!

Awesome! Many thanks


Question: Is there a way to customize styling?

Just found this project and been playing with it for a couple of hours, love it!
I was curious if there is a way to customize the styling at all?
Thanks for such a great tool!

@yorch thanks for your interest in the project!
Unfortunately, currently it's not possible to change the styling of the admin panel.
This feature might come with some motor-admin enterprise version for web-development companies so they can white-label the admin panel to make it look like it's a completely custom built panel.


Add more tests for web

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

NONE

/approve


log login errors for fail2ban

#67

do I have to put the logfile path to a configrable variable ?

I am not shure if it would be better to use a seperate log file instead. I am currently at a rewrite of the software so I wont merge that change, but I will leave the issue open as it seems to be a good idea.

Hello
Separate log file ? is there an existing one filled by pdns-manager ?
2018-02-13 10:23 GMT+01:00 Lukas Metzger<EMAIL_ADDRESS>
I am not shure if it would be better to use a seperate log file instead. I
am currently at a rewrite of the software so I wont merge that change, but
I will leave the issue open as it seems to be a good idea.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/loewexy/pdnsmanager/pull/68#issuecomment-365201813,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMVhk3sQDNjdMaA6YkbyrWlC3SZdry6Eks5tUVSQgaJpZM4SDbzn
.

No currently there is no existing logfile. But if I understood correctly you are posting the messages to the apache error log. Maybe it would be an better option to write a separate logfile for PDNS Manager...

Hey, I have to intervene here. This PR adds no additional security at all! :warning:
As the logging is initiated from the client side, it can be disabled by an attacker.
(In fact, attackers would send login requests directly to api/index.php without using the actual login form.)
The only place, where this logging would help is at api/index.php, where the actual login request is processed.

You are definitely right... in the current version the code is completely useless. I did not review the code as I am currently reimplementing complete pdnsmanager, thus the PR is not really needed. But I let the issue open to implement it in a new version.

It's cool that you want to reimplement it. This feature is really worth to be implemented in the next version.

The logging in version 2.0 should be sufficent for this purpose. If not you can then open a new issue.


Addresses Issue #243

This is my attempt at addressing issue #243. It adds a 't' option to the -p (print) command so that the elapsed time can be displayed at the end of the output.
It works as-is with one problem: the body gets printed by default (even with JUST "-p t"). I wasn't able to track down why that was happening.

See https://github.com/dlynch158/httpie/pull/1


GoogleCloudPlatform/github-actions/setup-gcloud has been deprecated, please switch to google-github-actions/setup-gcloud.

While investigating #3, I've noticed that e.g. on https://github.com/apache/fineract/actions/runs/452581732 there is an annotation saying:
Thank you for using setup-gcloud Action. GoogleCloudPlatform/github-actions/setup-gcloud has been deprecated, please switch to google-github-actions/setup-gcloud.
I'll see what needs to be done about that some time (but possibly not today). If anyone is interested in helping, please feel free to send PRs - I suspect this means the uses: GoogleCloudPlatform/github-actions/setup-gcloud@master in https://github.com/apache/fineract/blob/develop/.github/workflows/fineract.dev.yaml needs to be changed.

It now (also or instead?) says the following, but after #8 I've had enough and will stop now:
Deploy on Fineract.devgoogle-github-actions/setup-gcloud is pinned at HEAD. We strongly advise against pinning to "@master" as it may be unstable. Please update your GitHub Action YAML from:      uses: 'google-github-actions/setup-gcloud@master'  to:      uses: 'google-github-actions/setup-gcloud@v0'  Alternatively, you can pin to any git tag or git SHA in the repository.
Deploy on Fineract.dev"service_account_key" has been deprecated. Please switch to using google-github-actions/auth which supports both Workload Identity Federation and Service Account Key JSON authentication. For more details, see https://github.com/google-github-actions/setup-gcloud#authorization
@ptuomola if you to raise a PR to update https://github.com/apache/fineract/blob/develop/.github/workflows/fineract.dev.yaml for whatever it's trying to want us to do above, feel free. (But "it works" as is, so not urgent - but it sounds like it may break sooner or later.)

Hi @vorburger - if I understood correctly what this is complaining about, then https://github.com/apache/fineract/pull/2014 should fix it. But as far as I can see there's no way to test this action as it doesn't get run unless we merge it. So should I just go ahead and try it out?
Btw how did you get the secrets for GCP set up for Fineract? I thought Apache github projects did not support secrets... if we can get a secret set up for SonarQube we can get rid of the whole Travis build altogether.

@vorburger unfortunately it looks like the deployment to fineract.dev is no longer working. I thought it was because of the changes I've made to address this issue. But I've just rolled my changes back, and it is still failing - so obviously that was not the case.
Looking at the timing of the last successful build and the one after that, I think the change that broke the deployment was this one: Replace command line Docker image build with Gradle Jib plugin https://github.com/apache/fineract/commit/6e052e536be5119b68ae095b923538912fde716a
Unfortunately I can't see the logs in the Google Cloud Build Console - I just get an error saying "Error: The caller does not have permission" - so I can't see what's broken by that. Perhaps you could grant me the required permission and then I can see if I can help to fix this?
FYI @vidakovic

@vorburger unfortunately it looks like the deployment to fineract.dev is no longer working.

Let's sort that out in #11.

Unfortunately I can't see the logs in the Google Cloud Build Console - I just get an error saying "Error: The caller does not have permission" - so I can't see what's broken by that. Perhaps you could grant me the required permission and then I can see if I can help to fix this?

Let's sort that out in #10.
That way we can keep this issue for the original goal (and you can re-revert your https://github.com/apache/fineract/pull/2014), after we fixed #10 and #11.

Btw how did you get the secrets for GCP set up for Fineract? I thought Apache github projects did not support secrets... if we can get a secret set up for SonarQube we can get rid of the whole Travis build altogether.

@ptuomola Ha! That's MAGIC... 😈 See this slide from my ApacheCon prez linking to this old blog post of mine, which explains how to set up a Secret on repos one only has Write but not Admin to, such as ASF. I've added a link to the README in https://github.com/vorburger/www.fineract.dev/commit/5c8020b00db738d29d21091ec10393c1817e1708. (Feel free to give me a shout if you need more about this; perhaps best in a separate issue from this one?)

Brilliant - thanks! Will give this a go a bit later, would be great to get completely rid of Travis...

This should now be resolved - I've reintroduced the patch that fixes the two actions that were marked as deprecated.
On another note, I wonder if you still need to use the gcloud beta commands or whether the build ones are now "mainstream"?

I've verified that the latest Fineract.dev related GitHub Action runs (e.g. here) don't show these warnings anymore, and am thus resolving this issue now.

wonder if you still need to use the gcloud beta commands or whether the build ones are now "mainstream"?

see #14


add vocational college varia teacher domain

@JoniRinta-Kahila Thank you for your patience while waiting for a review and processing your request. Pull request merged.
Please start requesting the licenses in about an hour, to let the changes to propagate through JetBrains system.
Please notice that the free educational licenses are for teachers and students only, and solely for educational purpose.
If other people who are not teachers, nor students, but possessing @eduvantaa.fi email addresses, need JetBrains licenses, they can send an email to<EMAIL_ADDRESS>to get an academic discount for the licenses.


Fix Linux build errors due to new folder structure

After changing from a folder structure where all files were located in Sources/ directly and not in any subfolders, Linux builds always failed.

Somehow not even moving all files back into Sources seems to fix the issue...


Suggestion to deal with omission of periods

There is a frequent hallucination in Whisper in which segments of the transcript are stripped of a period or full stop. Example (not a real transcription, just to illustrate the issue:

Meghan Elizabeth Trainor is an American singer-songwriter and television personality She rose to prominence after signing with Epic Records in 2014 and releasing her debut single All About That Bass, which reached number one on the U.S. Billboard Hot 100 chart and sold 11 million copies worldwide Trainor has released five studio albums with the label and has received various accolades, including the 2016 Grammy Award for Best New Artist.

I have found that adding about 5 seconds of whitenoise to the beginning of the affected excerpt and retranscribing it usually corrects the punctuation.
Perhaps this could be incorporated to the code. Or, if there were a way to separate the affected region (e.g. with information from the VAD), a separate function could be written to check for this hallucination, export the WAV for the affected region and retranscribe.

Hi @busdriverbuddha
Yeah these are some of the failure modes of whisper's LLM based decoder.

I have found that adding about 5 seconds of whitenoise to the beginning of the affected excerpt and retranscribing it usually corrects the punctuation.

Interesting, do you have any detailed evaluation on it? Like how much punctuation accuracy improves after adding this? Also any effects on the WER? One issue I see in this approach is that it will unnecessarily increase the inference time.
Can you try this:
files = ['audio.wav']
lang_codes = ['en']
tasks = ['transcribe']
initial_prompts = ['This is a documentary about Meghan Elizabeth.']

out = model.transcribe_with_vad(files,
                                lang_codes=lang_codes,
                                tasks=tasks,
                                initial_prompts=initial_prompts,
                                batch_size=32)

PS: This thing is also in my roadmaps on how to use prompting with whisper model to align the transcription format.

If you can provide me one sample file, I can try looking into it if VAD margins can be somehow used to improve these issues.

I can supply an MP3 file in which this issue happens predictably. How can I share it with you privately?

Hi got your email. I will get back to you by coming weekend.

All right, thank you!
On Tue, Feb 6, 2024 at 5:15 AM Shashi Kant @.***> wrote:

Hi got your email. I will get back to you by coming weekend.
—
Reply to this email directly, view it on GitHub
https://github.com/shashikg/WhisperS2T/issues/9#issuecomment-1928987164,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACBPVXSXCX4G3MDZJTEX3KLYSHRBHAVCNFSM6AAAAABCTPCSJCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMRYHE4DOMJWGQ
.
You are receiving this because you were mentioned.Message ID:
@.***>


Unable to create folder from save dialog

Operating system or device - Godot version:
Fedora 23 64bit, godot master 85d8000449be06affcd686838d498be186594c14
Issue description (what happened, and what was expected):
Opening the save file dialog and trying to create a folder using the create folder button results in this message

Steps to reproduce:
Link to minimal example project (optional but very welcome):

@reduz The problem is here 61655d6dc2912424de937e2870b6a5ad346c3daf, the recursive directories problem is solved but disables the creation of directories from the save dialog, this happends because p_dir becomes res://SOME_DIR, and that cant be resolved by mkdir

I think this is fixed

This is not fixed though. You can create folders from the Project Manager, but not in a Save Dialog to the res:// path.

Actually I tested against the current HEAD (a41a70ea) and it seems to be fixed. Probably thanks to ffbc2e56, as it was still broken in 729e61e1.

yes this has been fixed


add prisma component for code highlighting

add a new component to show the code sample with syntax highlighting

the updated version can be previewed on https://svelte-toast.vercel.app/

the updated version can be previewed on https://svelte-toast.vercel.app/

Wow nice! Thanks for doing this! Could you remove the docs/build/* files, as well as package-lock.json, from the PR so I can rebuild/update dependencies? Otherwise, looks good to me! 👍

Wow nice! Thanks for doing this! Could you remove the docs/build/* files, as well as package-lock.json, from the PR so I can rebuild/update dependencies? Otherwise, looks good to me! 👍

Thanks - can you revert docs/build/bundle.js.map and docs/build/global.css too?

Thanks - can you revert docs/build/bundle.js.map and docs/build/global.css too?


Making into a semi circle

Hello,
Nice plugin here.
Wondering how I can make this into a semi circle instead of a circle and set the values so it doesn't go past the semi circle.
I understand how semi circles are created using just canvas. I tried tweaking and it doesn't really work here.
Will really appreciate the help.

Hi!
Is this what you need?

Hello,
Yes exactly. Your awesome man!!. .. Thanks

I hope I've helped you. I'm closing this ticket, but feel free to reopen.


firebase_analytics: ARC Semantic Issue (Xcode)

Is there an existing issue for this?

[X] I have searched the existing issues.

Which plugins are affected?
Analytics, App Installations
Which platforms are affected?
iOS
Description
Anyone faced this issue?
I am using version -> 11.2.0
ARC Semantic Issue (Xcode): No known class method for selector 'initiateOnDeviceConversionMeasurementWithHashedEmailAddress:'
/Users/noob/.pub-cache/hosted/pub.dev/firebase_analytics-11.2.0/ios/Classes/FLTFirebaseAnalyticsPlugin.m:199:18
ARC Semantic Issue (Xcode): No known class method for selector 'initiateOnDeviceConversionMeasurementWithHashedPhoneNumber:'
/Users/noob/.pub-cache/hosted/pub.dev/firebase_analytics-11.2.0/ios/Classes/FLTFirebaseAnalyticsPlugin.m:203:18
Reproducing the issue
Launching lib/main.dart on iPhone 15 in debug mode...
Running Xcode build...
Xcode build done.                                           29.1s
Failed to build iOS app
ARC Semantic Issue (Xcode): No known class method for selector 'initiateOnDeviceConversionMeasurementWithHashedEmailAddress:'
/Users/noob/.pub-cache/hosted/pub.dev/firebase_analytics-11.2.0/ios/Classes/FLTFirebaseAnalyticsPlugin.m:199:18
ARC Semantic Issue (Xcode): No known class method for selector 'initiateOnDeviceConversionMeasurementWithHashedPhoneNumber:'
/Users/noob/.pub-cache/hosted/pub.dev/firebase_analytics-11.2.0/ios/Classes/FLTFirebaseAnalyticsPlugin.m:203:18
Could not build the application for the simulator.
Error launching application on iPhone 15.
Firebase Core version
firebase_core: ^3.2.0
Flutter Version
3.22.2
Relevant Log Output
No response
Flutter dependencies

Expand Flutter dependencies snippet

Replace this line with the contents of your `flutter pub deps -- --style=compact`.

Additional context and comments
No response

Hello @spn107 I'm not reproducing this. Which version of XCode are you using? Have you tried to remove your Pods folder and Podfile.lock and run flutter clean?

Hello @spn107 I'm not reproducing this. Which version of XCode are you using? Have you tried to remove your Pods folder and Podfile.lock and run flutter clean?

Hi @Lyokone  I am using Xcode 15.4. Yes I tried to remove your Pods folder and Podfile.lock and run flutter clean and also deleted pub.caches.


feat: initial support for dependent resources

This PR provides annotation-based configuration of dependent resources, which the controller then automatically handles, taking care of propagating creation/update of said dependent resources depending on how they are configured. The functional scope is still pretty limited and more changes will be needed but this sets up the architecture for more changes.
In particular, handling of readiness / status reporting of dependent resources will also be tackled at some point.

Closing this in favor of #785


component/directive selector rules doesn't work with es6 template string literals

I know it's strange to use string literals for selectors, but it's valid js :)
used rules:
    "directive-selector-name": [true, "camelCase"],
    "component-selector-name": [true, "kebab-case"],
    "directive-selector-type": [true, "attribute"],
    "component-selector-type": [true, "element"],

Thank you for mentioning. We will take a look.

I'm already working on fix ;) PR incoming

Soon we'll add the evaluator from the metadata collector and such issues will be gone.


Static methods doesen't work

Hi,
I try used your nuget on my job, replace the static methods doesn't work.
I will want replace the static methods in the tests unitarys.
I will want  adding test end to end on a new branch, i doesn't work. I dont have the rights.
On this branch, I habe add a example simplify  on my context and le the example with Console.Writline both are doesnt work.
Regards,
Anthony

@AnthonyDegroote What version were you using?

@AnthonyDegroote Please see my comment on #63

@AnthonyDegroote This has been fixed in v2.0 which can be found here: https://www.nuget.org/packages/Poser/2.0.0


added profile tooltip

While scrolling through feeds, users might want to quickly view a profiles description without having to navigate to their profile page. The PR provides a tooltip functionality where if the user hovers the mouse pointer over an avatar image for 1.5 seconds a tooltip pops up showing the avatar, username, and profile description.

Changes made (only one file BaseUserAvatar.vue):
q-tooltip was added to BaseUserAvatar with a delay of 1500 which creates a pop upof the users avatar image, name, and profile description using BaseUserName and store.getters.profileDesctiption(). Allows users to access this tooltip anywhere BaseUserAvatar is referenced (followerFeed, globalFeed, Follows, and Messages)

i definitely agree with the idea but i don't like using tooltip for this. I would want a card that you can then click and interact with. also the grey tooltip background makes links hard to see.
do you know if there is a way to delay the hiding of tooltips?

I was thinking that we might want clickable, maybe have the follow buttons included as well. I think we can do that with qmenu. I'll try to work it up soon!

yeah exactly! would be nice if you could follow, copy pubkey, send lightning tip all from that card. sounds good keep me posted


Multiple slashes in include paths messes up the mechanism in rpmbuild…

… when extracting debuginfo. This blocked the open source releases.

Assigning @vekterli as this was introduced in https://github.com/vespa-engine/vespa/pull/11507.
FYI @baldersheim


Investigate creating a jobs.graphql.org job board

similar to https://jobs.cncf.io/ or https://jobs.cd.foundation

will this be a paid service or a free open for all website?

for us to decide who gets to post... we generally make it free for all members to post and highlight their job openings more than others, but everyone is welcome to post

hmm.. let's get started? I feel a Gatsby site is good for this. It can run powered by Github as DB. What do we need to get started?


vulkan-tools: add dependency on vulkan-headers

This fixes the build on NixOS.

Motivation for this change
The vulkan-tools package did not build.
Things done

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

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

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

@GrahamcOfBorg build vulkan-tools
I suggested the removal of vulkan-headers from vulkan-loader in https://github.com/NixOS/nixpkgs/pull/55959#issuecomment-478296371
I think it's still alright, we'll just fix breakages as they come.
Ping @Ralith


Weekly resync 2021-09-02

Deploy Blockers
Jobs that have doubled in the past week

Triggered auto assignment to @joekaufmanexpensify (Supportal++), see https://stackoverflow.com/c/expensify/posts/tagged/648 for more details.

Test


Improve producer ID expiration performance

Skip using steam when expiring the producer ID. This can improve the performance significantly when the count is high.
Before
Benchmark                                        (numProducerIds)  Mode  Cnt      Score       Error  Units
ProducerStateManagerBench.testDeleteExpiringIds             10000  avgt    3    101.253 ±    28.031  us/op
ProducerStateManagerBench.testDeleteExpiringIds            100000  avgt    3   2297.219 ±  1690.486  us/op
ProducerStateManagerBench.testDeleteExpiringIds           1000000  avgt    3  30688.865 ± 16348.768  us/op

After
Benchmark                                        (numProducerIds)  Mode  Cnt     Score     Error  Units
ProducerStateManagerBench.testDeleteExpiringIds             10000  avgt    3    39.122 ±   1.151  us/op
ProducerStateManagerBench.testDeleteExpiringIds            100000  avgt    3   464.363 ±  98.857  us/op
ProducerStateManagerBench.testDeleteExpiringIds           1000000  avgt    3  5731.169 ± 674.380  us/op

Also, made a change to the JMH testing which excludes the producer ID populating from the testing.

Thanks Calvin -- just to be clear, we are also changing the benchmark to not include loading all the producer IDs as well?

@jolshan Yes, the loading part is excluded. I guess there is not much performance concern for the loading.


Add response type parameter

This PR partially addresses issue #63. Currently, the response type defaults to 'token' for renewels. These changes add a parameter to acquireTokenSilent to enable specification that a token of type 'id_token' get requested during renewal.

@austindyoung,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@austindyoung fixed in dev


⚠ St. Lawrence Global Observatory CKAN Sitemap has degraded performance

In be6c61e, St. Lawrence Global Observatory CKAN Sitemap (https://catalogue.ogsl.ca/sitemap/sitemap-1.xml) experienced degraded performance:

HTTP code: 200
Response time: 11712 ms

Resolved: St. Lawrence Global Observatory CKAN Sitemap performance has improved in fc97551 after 36 minutes.


typesCache deadlocks

@e-orz added the following commit:
https://github.com/thomsonreuters/CM-Well/pull/258/commits/5df365c002753bab6d0135e8b591b3c1fa279cb6
to help understand the sporadic failures of sorted search test.
It appears we get into some deadlock...
here's what's the test output is for that commit.
[info] - get sorted by latitude (all share property) *** FAILED ***
[info]   java.util.concurrent.TimeoutException: Futures timed out after [60 seconds]
[info]   at scala.concurrent.impl.Promise$DefaultPromise.ready(Promise.scala:223)
[info]   at scala.concurrent.impl.Promise$DefaultPromise.result(Promise.scala:227)
[info]   at scala.concurrent.Await$$anonfun$result$1.apply(package.scala:190)
[info]   at scala.concurrent.BlockContext$DefaultBlockContext$.blockOn(BlockContext.scala:53)
[info]   at scala.concurrent.Await$.result(package.scala:190)
[info]   at cmwell.it.SearchTests$$anonfun$1$$anonfun$14$$anonfun$apply$17.apply(SearchTests.scala:142)
[info]   at cmwell.it.SearchTests$$anonfun$1$$anonfun$14$$anonfun$apply$17.apply(SearchTests.scala:141)
[info]   at scala.util.Success$$anonfun$map$1.apply(Try.scala:237)
[info]   at scala.util.Try$.apply(Try.scala:192)
[info]   at scala.util.Success.map(Try.scala:237)

Are you sure it's a dealock in a code and not in tests? Runnng
https://github.com/thomsonreuters/CM-Well/blob/5df365c002753bab6d0135e8b591b3c1fa279cb6/server/cmwell-it/src/it/scala/cmwell/it/SearchTests.scala#L142
inside a test might never happen because the execution context only allows one thread.

It might be. Easy enough to test. Lets just use scala.concurrent.ExecutionContext.global directly (and explicitly) in the test.


Only two JSON files referenced

Minor fix

@juanandmsft : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#label:"aq-pr-triaged"

This change is now reflected in the live article. Thanks for your contribution!


Optionally process sub-assets during htl parsing #16

This PR adds HTL preprocessing based on the parcel's HTML asset. resolving references to local files and subsequently processing them as well.
eg, in the following html.htl the links to the static files...
$ fgrep link src/html.htl
    <link rel="shortcut icon" type="image/x-icon" href="static/favicon.ico">
    <link rel="stylesheet" href="static/bootstrap.min.css"/>

are rewritten:
$ hlx build src/html.htl
✨  Built in 1.03s.
.hlx/build/bootstrap.min.ef87a570.css    117.91 KB     25ms
.hlx/build/html.js                         8.44 KB    578ms
.hlx/build/favicon.64238bb4.ico            2.57 KB     24ms

See #16

If we can get the generated files into a directory that is

not starting with .
can be checked back into git

Then this change would tie in nicely with adobe/helix-cli#112


Storybook and webpack-dashboard

I am trying to add to my project the webpack-dashboard plugin, which makes a nicer output from webpacks default build output. But with storybook it fails, because I can't figure out how to make storybook be quiet when it has some errors in build process.
Here is a sample project, with intentional error:
https://github.com/PTihomir/storybook-test-project
npm run storybook will start the build, but at the end the listing will override the dashboard.
By removing the error and running npm run dev, the desired output will be presented.
I am not sure that the issue is with storybook, but I assume storybook just prints errors from webpack, even if in webpack.config the quiet parameter is set to true.

Looks like we should forward the devServer field as options for the webpack-dev-middleware. This is what the webpack-dev-server seems to be doing (check this link and this link). Storybook creates the middleware config here.
@PTihomir If you can send a pull-request for this I can merge it to react-storybook and do a release. Let me know if you need any help.

I'm gonna close this since we've a clear set of actions.
@PTihomir feel free to ask more questions or re-open if you need.

Ok, thanks. I will make the modifications and create the PR.

@PTihomir awesome.

Closing since we are open to accept a PR.

@PTihomir Did you filled a PR for this? Can't find any track of it. I'd be happy to take a look if not.


Feature/gh 542

The issuer in #542 wanted to manipulate the default properties of TaskExecuter. So it is possible to set a new one with help of a builder class. The logic which was implemented in
SimpleMessageListenerContainer was moved into this builder class, so the user must not implement it twice.

Thanks @n00bi14 for contribution! I'll review & test it some time this week.

Hi @maciejwalkowiak,
i use the spring cloud code style-guides in my IntelliJ IDE.
But if I try to compile the code with mvn compile, the checkstyle plugin detect eight issues.
Can you tell me please how I have to configure my IDE to compile the project? Mainly because my local build fails because of style-guide issues and the CI build fails because of missing classes.
Thanks very much!

Just a regular maven project import should eliminate any possibility of having compilation issues especially related to missing classes. When it comes to check style I haven't figured out yet the setup - if I have checkstyle issue reported I fix it manually. Check readme for Intellij setup - it's very detailed I'm not sure if it's up to date though.

I am sorry, in the end I did not manage to get my head around this issue. Due to the moving to another repository (https://github.com/awspring/spring-cloud-aws) I must close it for now, but once we get back to this topic I'll make sure to give you credits.

I am sorry, in the end I did not manage to get my head around this issue. Due to the moving to another repository (https://github.com/awspring/spring-cloud-aws) I must close it for now, but once we get back to this topic I'll make sure to give you credits.


Header Images: fix cropping for users without the 'customize' capability

I think I found a Core bug here. There's absolutely no reason for the ajax request to include wp_customize=on, the logic is entirely handled in the wp_ajax_crop_image() handler.
This commit appears to have moved the logic from the Customizer into a more general purpose context while leaving the constraining wp_customize=on parameter.
I guess this could be a core patch
Fixes #846
Proposed changes:
Make a custom controller but without wp_customize=on
Other information:

[ ] Have you written new tests for your changes, if applicable?

Testing instructions:

Be logged in as an author-level user (or any other user who can create posts but not edit others' posts)
Ensure that this user is ActivityPub-enabled
Go to profile and attempt to upload and crop an image. Any image uploaded or chosen from the library should work (the library will now only show the author user's own posts)

I fixed the remaining problem in https://github.com/Automattic/wordpress-activitypub/pull/854/commits/3000bf52cb0e17b74edcf12310f51aec2bc89f08 - you can't crop an image that you didn't upload when you can't edit others' posts, failing this check that supplies the nonces.
This is definitely ready for testing again @pfefferle


Is it possible to create database.config outside the /vendor (composer)?

I think it would be more convenient to work with composer when I can store config file as part of my repository (outside /vendor dir, which is not included). As example: phinx library.
And another one about installation: if i already have database (without rbac tables) install script just telling me "ok, congrats...", but actually it wasn't, because mysql file was not executed.

Not sure how to upvote the issue, but totally agree with @battis and @kroder. Would be great if one could pass own database configuration to the lib.

I was also having issues with the fact that there's the need to manually (and not programmatically) pass on the database configuration parameters into the constructor.
Here's my commit to solve temp the issue:
https://github.com/sergiodlopes/rbac/commit/bbcc3d99be030fff78a96de30676a35502c287d6
Falls back to default configuration variables in repository database.config.
Hopefully this is a temp fix. Looking forward for version 3 with this issue solved.
Any feedback is welcome.

Cool. Thanks!

+1 for @sergiodlopes, that solution works great!

Thank you again @sergiodlopes :-)
I just need to review all PR's, Issues and discussion concerning the database connection values. I'll get back ASAP regarding the best way to merge this PR into the v2.x branch.
This weekend is going to be a bit hectic for me, but I'll do my best to get in front of my keyboard as much as I can ;-)

+1 for this.
It would be great to be able to specify the location of the configuration file. This is especially useful when dealing with multiple environments (I.e dev, staging and prod).

Absolutely. Just ran into this now. The user db config should not be sitting in the vendor library.
@sergiodlopes solution is a good fix - even better would be to accept an existing PDO object. I already have a connection open - I don't want or need another one. It could then also take advantage of persistent connections (sort of like connection pooling).
Lastly, maybe this will help somebody - if you're using composer, you can use a custom fork like Sergio's in your composer.json:
"repositories": [
    {
        "type": "vcs",
        "url": "https://github.com/sergiodlopes/rbac",
    }
],
"require": {
    "owasp/phprbac": "dev-master"
}


Update doc for SessionBuilder::control_directory

The doc is outdated and needs some update.
Signed-off-by: Jiahao XU<EMAIL_ADDRESS>

This change is

Codecov Report

Merging #127 (82c80e9) into master (3319a81) will decrease coverage by 0.02%.
The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files

Files Changed
Coverage Δ

src/builder.rs
83.59% <ø> (ø)

... and 1 file with indirect coverage changes

LGTM.


Lack of request names after update

Expected Behavior
See the URL as the request name when name is empty (like it was done before)
Actual Behavior
Empty name, nothing is shown

Reproduction Steps

Create an empty request
Fill the url
You should see the URL as the name

Is there an existing issue for this?

[X] I have searched the issue tracker for this problem.

Additional Information
For rapid request creation it was valuable to have the option to skip naming and have just a bunch of URLs to change. I don't care about their names as their URLs are verbose enough to tell me what they represent
Insomnia Version
v8.1.0
What operating system are you using?
macOS
Operating System Version
All
Installation method
Web

Hi @pilotpirxie do you confirm there were custom names before the migration? Or did it just show the URL of the request?

Yes, before migration it showed the URL of the request and now I lost that and my huge collection is useless as I can't navigate.