Precision on genome indexes

Hi I was reading your article and trying to set up a pipeline following the preprocessing steps in your supplementary material
Would you mind sharing with me how you built your genome indexes for both the rRNA,tRNA and sn(o)RNA filtering and for the alignement. I tried building a genome index for the filtering part but after filtering instead of 2M read remaining I had 400k, I probably did something wrong
I hope to be able to implement your tool on a large amount of studies
Thank you
Nicolas

Thank you for your interest in my research!
The set-up i use is as follows:
to set up reference genomes for riboseq:
STAR --runThreadN 10 \
    --runMode genomeGenerate \
    --genomeDir ${folder}/STAR/ \
    --genomeFastaFiles $1 \
    --sjdbGTFfile $2 \
    --sjdbOverhang 35

To set up the xRNA libraries:
STAR --runThreadN 10 \
    --runMode genomeGenerate \
    --genomeDir "$folder/STAR" \
    --genomeFastaFiles $1 \
    --genomeSAindexNbases 6

Note that RIBO-former finetunes on separate data sets, so the tool is therefore generally not sensitive to changes to some of these pre-processing steps. Note that I'll be updating several parts of the library this month, so keep an eye out on the RIBO-former repository.

I was wondering more precisely about what Fasta and gtf did you use to make the index, did you have to make your own snRNA, rRNA, tRNA, snoRNA fasta, did you make a fasta from gtf you filtered to contained only those, did you take the ncRNA fasta from esembl and removed lncRNA (because we recently found that those can sometime be coding)
Thank you for your fast answer
Nicolas

Hey Nicolas, If you feel this issue is properly answered, please close the issue!

Oh sorry,
Meant to close it yesterday
Thanks a lot for the precisions
I'll wait patiently for your modification
Best
Nicolas

Already pushed (v0.5). Make sure to download the latest version.


Add undo/redo functionality

Usage:
Press 'z' to undo
Press 'y' to redo
Implementation:
When the user takes action such as zoom in/zoom out/pan left/pan right, the value of this.unitController.viewRange will be stored in undo stack.
When the user presses 'z' to undo, the most recent viewRange is popped from undo stack and pushed to redo stack.
When the user presses 'y' to redo, the most recent viewRange is popped from redo stack and pushed to undo stack.
Signed-off-by: Yining Wang<EMAIL_ADDRESS>

.

This is really really cool. I would argue it should be in the Theia extension.
I would also argue that undo should be ctrl-z and redo should be ctrl-shift-z.

Do I squash the last two commits?

Closing this one. The feature was done on the application level of theia-trace-extension. Feel free to re-open this one if needed.


Add CORS policy for Open Match published Swagger docs.

If you use Swagger UI to read the Swagger docs in Open Match you get this error.
Failed to load API definition.
Errors
Hide
Fetch errorFailed to fetch https://open-match.dev/api/v0.0.0-dev/frontend.swagger.json
Fetch errorPossible cross-origin (CORS) issue? The URL origin (https://open-match.dev) does not match the page (http://localhost:8080). Check the server returns the correct 'Access-Control-Allow-*' headers.

https://github.com/GoogleCloudPlatform/open-match/pull/373 addressed this.


Update header when retrying

Hi,
I'm using the RetryClient() to refresh the access token if expired and then try the request again.
However when retrying the request, the old access token is sent and not the new one.
This is how I configured my RetryClient:
RetryClient client = RetryClient(
  http.Client(),
  retries: 1,
  when: (response) => response.statusCode == 401,
  onRetry: (req, res, retryCount) async{
    await authService.refreshToken();
    req.headers['Authorization'] = sharedPrefs.accessToken;
    }
);

I'm guessing that the req element passed to onRetry can't be used to update the req that is actually sent when retrying.
How can I solve this?
Thanks :)

Found the problem. Currently the _onRetry function is a void Function, wheras it should be a Future function in order to accept future functions as well.
This
final void Function(BaseRequest, BaseResponse?, int)? _onRetry;

should be changed to this
final Future<void> Function(BaseRequest, BaseResponse?, int)? _onRetry;

Then at line 121 of RetryClient, _onRetry should be called like this instead:
await Future.delayed(_delay(i));
//_onRetry?.call(request, response, i); THIS DOESN'T SUPPORT FUTURE FUNCTION
if(_onRetry != null) await _onRetry!(request, response, i);
i++;

@SeriousMonk I have the same issue, did you find a fix?


checking nested lists

check lists which had problem ( Issue #4 )

[ ] amss
[x] amss2
[x] amss3
[x] arellano
[x] calvo
[x] career
[ ] chang_credible
[x] discrete_dp
[x] dyn_stack
[x] getting_started
[x] jv
[x] kalman
[x] linear_models
[x] lqcontrol
[x] lqramsey
[x] lu_tricks
[x] lucas_model
[x] markov_perf
[x] multiplicative_functionals
[ ] opt_tax_recur
[ ] optgrowth
[ ] orth_proj
[ ] pandas
[ ] perm_income
[ ] perm_income_cons
[ ] python_advanced_features
[ ] python_by_example
[ ] python_essentials
[ ] python_oop
[ ] rational_expectations
[ ] rob_markov_perf
[ ] robustness
[ ] scipy
[ ] smoothing
[ ] stationary_densities

arellano was

the first one fixed but the second one is

thanks @fkazemian it looks like I've messed up the list construct when trying to add index support.

chang_credible was

the first one fixed but the second one is:

hi @fkazemian the instance has been updated without index support. Can you check those cases you found above and let me know if they have been fixed by adding a comment to each. Thanks!

@mmcky I just checked, as you can see bellow arellano is fixed:

but the others not, for example amss:

opt_tax_recur
multi-level lists are fixed but those one with numerical index have still some problem:

If we want to be too strict, optgrowth is:

but I suppose there is no problem, so I checked it as fixed

I think I know what the problem is with index support. I will work on a fix now.
Can you post the source for optgrowth?

@mmcky , optgrowth is a list here. the code for  is
In our setting, we have the following key result

    * A feasible consumption policy is optimal if and only import numpy as np

python_advanced_features:    (I'll check it, I think the problem is related to rst )

for python_by_example, all of them are fixed, but I think this list is not beautiful which has 1, 2 and again 1,2, anyway it is fixed.

[ ] compare-lectures has been updated with all the fixes discussed above

@fkazemian would you mind checking these issues on the latest set of files on compare?

Hi @mmcky , sure

[ ] amss
[ ] amss2
[ ] amss3
[ ] arellano
[ ] calvo
[ ] career
[ ] chang_credible
[ ] discrete_dp
[ ] dyn_stack
[ ] getting_started
[ ] jv
[ ] kalman
[ ] linear_models
[ ] lqcontrol
[ ] lqramsey
[ ] lu_tricks
[ ] lucas_model
[ ] markov_perf
[ ] multiplicative_functionals
[ ] opt_tax_recur
[ ] optgrowth
[ ] orth_proj
[ ] pandas
[ ] perm_income
[ ] perm_income_cons
[ ] python_advanced_features
[ ] python_by_example
[ ] python_essentials
[ ] python_oop
[ ] rational_expectations
[ ] rob_markov_perf
[ ] robustness
[ ] scipy
[ ] smoothing
[ ] stationary_densities

amss is like

but I mark it as fixed.
chang_credible

opt_tax_recur is like

I marked it as fixed.

python_oop
it seems code is not supported in nested lists:

code:
* Any instance data should be prepended with `self`

    * e.g., the `earn` method references `self.wealth` rather than just `wealth`

* Any method defined within the class should have `self` as its first argument

    * e.g., `def earn(self, y)` rather than just `def earn(y)`

* Any method referenced within the class should be called as  `self.method_name`

Hi @mmcky I fixed the problem for chang_credible in PR  #56

The only problem as you can see is supporting code inside of lists, if we don't attention to that here I have to tell this issue is fixed. I change the label to question and fixed.

@natashawatkins for python_oop example listed above. It appears that many of the items are using a single quote. Do you agree they should be double single quotes (http://www.sphinx-doc.org/en/master/usage/restructuredtext/basics.html)?

Are you referring to the highlighting? I think throughout most of the lecture's we've used single quotes  - why doesn't this work when the notebook is compiled from rst?

@natashawatkins have these been fixed?

Ah, I don't believe so - I've done a few lectures related to jit but not the above one

once you have -- can you close this issue? Thanks.


Compatibility with GridSearchCV

Hi,
I wanted to know if there is a possibility in the future for a compatibility for our metric learning algorithms with GridSearchCV in the pipeline. This is wrt the following
http://scikit-learn.org/dev/developers/contributing.html#rolling-your-own-estimator

That would be nice!

This already works since version '0.2.1'.
I would encourage you to use my HierarchicalGridSearchCV for this where you can save a lot of time if you have a long Pipeline. https://github.com/svecon/HierarchicalGridSearchCV
If you still want to use the scikit version for this (warning: inefficient) then example code could look like this:
from sklearn.model_selection import GridSearchCV
from sklearn import svm, datasets
from sklearn.pipeline import Pipeline
from metric_learn import LFDA

iris = datasets.load_iris()
clf = GridSearchCV(
    Pipeline([('lfda', LFDA()), ('svr', svm.SVC())]),
    {'lfda__k':(1, 3), 'svr__C':(.1, .5)}
)
clf.fit(iris.data, iris.target)
print(clf.best_score_, clf.best_params_)


How to attach headers to graphql request?

I'm attempting to use the client generation to automate queries to a hasura api. The issue is hasura's api requires a header x-hasura-admin-secret and on top of this I need to send an Authorization Bearer token from keycloak to hasura's auth hook. The generated web client code from mvn graphql:generateClientCode is here:
			JsonResponseWrapper responseJson = webClient//
					.post()//
					.contentType(MediaType.APPLICATION_JSON)//
					.body(Mono.just(jsonRequest), String.class)//
					.accept(MediaType.APPLICATION_JSON)//
					.retrieve()//
					.bodyToMono(JsonResponseWrapper.class)//
					.block();

Obviously I can add the headers manually but they're overwritten on next run of the maven plugin and I don't think the solution is to maintain auto generated code in source control. What flexibility is there to add custom headers to the graphql endpoint POST request without having to edit and maintain auto generated code?

Hum,
I guess this is a usefull use case, from time to time.
Below, I assume you know Spring
What you'll have to do is to:

Use the Spring version of the plugin
Declare a ExchangeFilterFunction Spring exchange filter function. It allows to filter the http request. With this component, you can act on the http request, or on its response

To add such a filter, you'll have to override the WebClient that is provided in GraphQLAutoConfiguration..
So basically, in your Main app, add this Bean:
	/**
	 * The Spring reactive {@link WebClient} that will execute the HTTP requests for GraphQL queries and mutations.
	 */
	@Bean
	@ConditionalOnMissingBean
	public WebClient webClient(String graphqlEndpoint, @Autowired(required = false) HttpClient httpClient,
			@Autowired(required = false) ServerOAuth2AuthorizedClientExchangeFilterFunction oauthFilter) {
		Builder webClientBuilder = WebClient.builder()//
				.baseUrl(graphqlEndpoint)//
				.defaultHeader(HttpHeaders.CONTENT_TYPE, MediaType.APPLICATION_JSON_VALUE)
				.defaultUriVariables(Collections.singletonMap("url", graphqlEndpoint));

		if (httpClient != null) {
			webClientBuilder.clientConnector(new ReactorClientHttpConnector(httpClient));
		}
		if (oauthFilter != null) {
			webClientBuilder.filter(oauthFilter);
		}

		// Add you filter(s) here:
		webClientBuilder.filter(yourFilter);

		return webClientBuilder.build();
	}

Hello!
Thanks a lot for getting back to me on this. I must admit I am relatively new to Spring which might be apparent from the issues I've raised. Thanks for the solution - I'll add and close.

for version 1.17.3 this does not work, just WebClient was not overridden.
`/**
*
*/
package org.graphql_forum_sample.client;
import java.util.Collections;
import org.forum.client.util.QueryExecutor;
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.boot.SpringApplication;
import org.springframework.boot.autoconfigure.SpringBootApplication;
import org.springframework.boot.autoconfigure.condition.ConditionalOnMissingBean;
import org.springframework.context.annotation.Bean;
import org.springframework.http.HttpHeaders;
import org.springframework.http.MediaType;
import org.springframework.http.client.reactive.ReactorClientHttpConnector;
import org.springframework.security.oauth2.client.web.reactive.function.client.ServerOAuth2AuthorizedClientExchangeFilterFunction;
import org.springframework.web.reactive.function.client.ExchangeFilterFunction;
import org.springframework.web.reactive.function.client.WebClient;
import org.springframework.web.reactive.function.client.WebClient.Builder;
import com.graphql_java_generator.client.GraphQLConfiguration;
import reactor.core.publisher.Mono;
import reactor.netty.http.client.HttpClient;
@SpringBootApplication(scanBasePackageClasses = { Main.class, GraphQLConfiguration.class, QueryExecutor.class })
public class Main {
private static final Logger log = LoggerFactory.getLogger(Main.class);
public static void main(String[] args) {
SpringApplication.run(Main.class, args);
}
@Bean
@ConditionalOnMissingBean
public WebClient webClient(String graphqlEndpoint, @Autowired(required = false) HttpClient httpClient,
		@Autowired(required = false) ServerOAuth2AuthorizedClientExchangeFilterFunction oauthFilter) {
	Builder webClientBuilder = WebClient.builder()//
			.baseUrl(graphqlEndpoint)//
			.defaultHeader(HttpHeaders.CONTENT_TYPE, MediaType.APPLICATION_JSON_VALUE)
			.defaultUriVariables(Collections.singletonMap("url", graphqlEndpoint));

	if (httpClient != null) {
		webClientBuilder.clientConnector(new ReactorClientHttpConnector(httpClient));
	}
	if (oauthFilter != null) {
		webClientBuilder.filter(oauthFilter);
	}

	// Add you filter(s) here:
	webClientBuilder.filter(logRequest());

	return webClientBuilder.build();
}

private static ExchangeFilterFunction logRequest() {
    return ExchangeFilterFunction.ofRequestProcessor(clientRequest -> {
        log.info("Request: {}:=:{}", clientRequest.method(), clientRequest.url());
        clientRequest.headers().forEach((name, values) -> values.forEach(value -> log.info("{}={}", name, value)));
        return Mono.just(clientRequest);
    });
}

}
`
log info is not reflected, I checked to debug mode but the breakpoint was not hit.

Yes, debugging the Spring conf can be tricky.
Can you take a look at these issue, and their answers:
https://github.com/graphql-java-generator/graphql-maven-plugin-project/issues/107
https://github.com/graphql-java-generator/graphql-maven-plugin-project/issues/105
Etienne


Implementing foreign links on generic errors

Hi.
Is there any way to accomplish this?
For something like std::sync::PoisonError<T> for example.

No, there is no polymorphism. You can foreign link PoinsonError<Thing> though for example.

Thank you, yes, that much I succeed to do.
The problem arises when there is a generic error, especially with lifetime
parameters.
I think it could be implemented with addiitional parametric macro
implementation - but I worked around it, so I wouldn't be able to give it a
try.
On Mon, 24 Apr 2017 at 7:09 Yamakaky<EMAIL_ADDRESS>wrote:

No, there is no polymorphism. You can foreign link PoinsonError
though for example.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/brson/error-chain/issues/146#issuecomment-296520596,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAeZwBlSOUp5WyOV5Ybz5gdxC5QPmmH9ks5rzCB0gaJpZM4NFqWi
.

Is there a plan to implement this? It has come up a few times for me.

Yeah, I know... If you have an idea of implementation, it's more than welcome ^^

There is parse_generics_shim, which is macro from closed RFC, that allows parsing generics and where clause in macro, it's limited, but possible solution.


[#22] Receive file from client

After trying several methods to send a file from the client, I will now use our extension to receive the file from the client. These are functions on the extension side, which will be called from the client side to send the file chunks. The logic is very similar to the pgmoneta_ext_get_file function.

Would be better to do this in one function - as there could be concurrent operations using this API

The logic of using three functions is clearer than relying on just one. I am currently resolving concurrency issues to ensure that the FileTransferContext structure is isolated in different sessions. If necessary, we can also implement a locking mechanism.

I combined the separate functions into a single function, making it simpler to manage.

Merged.
Thanks for your contribution !


bugfix: when building materialized-view, if taskCount>1, may cause concurrentModificationException

In checkSegmentsAndSubmitTasks:
for (Map.Entry<Interval, HadoopIndexTask> entry : runningTasks.entrySet()) {
        Optional<TaskStatus> taskStatus = taskStorage.getStatus(entry.getValue().getId());
        if (!taskStatus.isPresent() || !taskStatus.get().isRunnable()) {
          runningTasks.remove(entry.getKey());
          runningVersion.remove(entry.getKey());
        }
      }

if  taskCount>1, while iterating runningTasks and removing entry from runningTasks will happen  concurrent, this will cause concurrentModificationException.
use ConcurrentHashMap instead  of HashMap     for runningTasks, will fix

@pzhdfy thanks for the patch and sorry for late review. Do you know what threads can read and remove at the same time? It's currently guarded by taskLock.
    synchronized (taskLock) {
      for (Map.Entry<Interval, HadoopIndexTask> entry : runningTasks.entrySet()) {
        Optional<TaskStatus> taskStatus = taskStorage.getStatus(entry.getValue().getId());
        if (!taskStatus.isPresent() || !taskStatus.get().isRunnable()) {
          runningTasks.remove(entry.getKey());
          runningVersion.remove(entry.getKey());
        }
      }
      if (runningTasks.size() == maxTaskCount) {
        //if the number of running tasks reach the max task count, supervisor won't submit new tasks.
        return;
      }
      Pair<SortedMap<Interval, String>, Map<Interval, List<DataSegment>>> toBuildIntervalAndBaseSegments = checkSegments();
      SortedMap<Interval, String> sortedToBuildVersion = toBuildIntervalAndBaseSegments.lhs;
      Map<Interval, List<DataSegment>> baseSegments = toBuildIntervalAndBaseSegments.rhs;
      missInterval = sortedToBuildVersion.keySet();
      submitTasks(sortedToBuildVersion, baseSegments);
    }

Oh I see. You’re right. Thanks.
I think it would be better to first find the entries to be removed, and then remove then all at once after iteration. I think this is better than using ConcurrentMap because there’s no concurrency issue. What do you think?
Also, all bug fix PRs should have unit tests. Would you please add one?

Or how about using .remove() on the iterator? I think if these maps are not actually going to be used from multiple threads, then we don't need to make them ConcurrentMaps.

Hi @pzhdfy, were you able to check this issue again by any chance?

@jihoonson


RDP Not listening

What I can't understand is the fact that after I tried the recommended solution, which was copying a new rfxvmt.dll to the System32 files and then update RDPWrap, and I tired that but I still can't connect to my desktop via RDPCheck, Any solutions?

#645
Try this.
You can find .ini in  %ProgramFiles%\RDPWrap\rdpwrap.ini

No system information provided in issue description, closing as invalid.
Use Telegram chat for support questions (see readme).


Initial PR

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 81.336%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
420

Relevant Lines:
522

💛  - Coveralls


STEP-1203: Add Maximum Test Repetitions step input

Checklist

[x] I've read and accepted the Contribution Guidelines
Requires MINOR version update

Context
We are adding Xcode 13's Test Repetition features to the Xcode Test step.

This includes choosing test repetition mode, setting maximum test repetitions and rerunning tests in a different process.

Changes

New step input: Maximum Test Repetitions (maximum_test_repetitions).

Works exactly like Maximum Test Repetitions in Xcode's Test Plan Configuration window.
Possible values: 2..< (default: 3).
Sets the appropriate flag on xcodebuild: -test-iterations.

Added E2E tests that test the 3 Test Repetition Modes and Maximum Test Repetitions together (more details below).
Separated flaky tests in sample app to eventually succeeding & eventually failing: https://github.com/bitrise-io/sample-swift-project-with-parallel-ui-test/pull/7.
Renamed existing E2E tests for should_retry_test_on_fail to make it easy to distinguish from new Test Repetition tests.
Reformatted the entire file to make the indentation of list items consistent.

Testing
There are 9 new E2E tests designed to test Test Repetition Modes and Maximum Test Repetitions together.

Given
When
Then

Sample app test fails 1 time(s), then always succeeds Test Repetition Mode is Retry on Failure Maximum Test Repetitions is 2
Step is executed
Step should succeed

Sample app test fails 2 time(s), then always succeeds Test Repetition Mode is Retry on Failure Maximum Test Repetitions is 2
Step is executed
Step should fail

Sample app test fails 2 time(s), then always succeeds Test Repetition Mode is Retry on Failure Maximum Test Repetitions is 3
Step is executed
Step should succeed

Sample app test succeeds 2 time(s), then always fails Test Repetition Mode is Until Failure Maximum Test Repetitions is 2
Step is executed
Step should succeed

Sample app test succeeds 1 time(s), then always fails Test Repetition Mode is Until Failure Maximum Test Repetitions is 2
Step is executed
Step should fail

Sample app test succeeds 3 time(s), then always fails Test Repetition Mode is Until Failure Maximum Test Repetitions is 3
Step is executed
Step should succeed

Sample app test succeeds 3 time(s), then always fails Test Repetition Mode is Up Until Maximum Repetitions Maximum Test Repetitions is 3
Step is executed
Step should succeed

Sample app test succeeds 3 time(s), then always fails Test Repetition Mode is Up Until Maximum Repetitions Maximum Test Repetitions is 3
Step is executed
Step should fail

Sample app test succeeds 3 time(s), then always fails Test Repetition Mode is Up Until Maximum Repetitions Maximum Test Repetitions is 3
Step is executed
Step should succeed

To make testing easier, the sample app now contains 2 kinds of simulated "flaky" tests:

One that succeeds a predefined number of times, then always fails. This can be used to test if retry_on_failure works as intended.
One the fails a predefined number of times, then always succeeds. This can be used to test if until_failure and up_until_maximum_repetitions works as intended.

Out of Scope

Finalizing description: description will be finalized in a separate PR before release.

Next steps:

Wait for CI to pass.
Merge https://github.com/bitrise-io/sample-swift-project-with-parallel-ui-test/pull/7.
Update sample app branch to master in E2E tests.
Merge this PR.


runtime: Add a simple File class for basic I/O

Here's something a little bit nicer than using the standard C library.
Note that we're limited by what's supported on all of macOS, Linux and Windows.

Also, do we perhaps want to start adding copyright headers to source files like in serenity? :)

assuming you mean the C++ we pull in and not the C++ we emit, that's probably not a bad idea...

Right, human-written files only! :^)
In serenity we use SPDX headers, like this: https://github.com/SerenityOS/serenity/blob/74695ce76ea5f6b88d8cb166621edafd1abedb47/Userland/Libraries/LibJS/Runtime/Temporal/ZonedDateTime.cpp#L1-L6

Also, do we perhaps want to start adding copyright headers to source files like in serenity? :)

Roger that!


suggestions not working in firefish/catodon? (empty results)

catodon.social, calckey.world

removed


Implement Fast HotStuff Immediate Next Steps When Chain Tip Is Updated

Current dependencies on/for this PR:

feature/pos-fast-hotstuff-consensus

PR #640

PR #641

PR #642

PR #643   👈

This comment was auto-generated by Graphite.


conn.uid in spicy-noise.log?

The uid field exists in conn.log for the wireguard entries, but the connection UID is not available for the corresponding entry/entries in spicy-noise.log. Is it possible to include the uid field for the connection in spicy-noise.log?

Great catch.  I've added the connection uid to the spicy-noise.log.


Line ending handling with sort functions

When inverting a selection of lines that that were selected by line (vs by character), the final line break at the end of the selection is included in the sorting, resulting in an unintended line break at the top of the selection, with the first unselected line following the sorted region being concatenated onto the last sorted line. Sorting options exhibit the same behavior, but without the blank line at the top.
I know that was a mouthful, so hopefully hopefully the gifs below make more sense. EOL sequence for the file is CRLF.
How to repro the issue (shows both sorting and invert behavior):

Expected behavior would be identical to when selecting by character, as supposed to by line:

Thanks for reporting this, the gifs do help.
I have an idea on the cause, I'll see if I can fix it tomorrow

Excluding from the sort operation any empty line that contains only an EOL sequence and is the last line in the selection, coupled with decreasing the selected line count by 1 would fix it I'd imagine. The invert option is already filtering out the empty line, it just needs the line count adjusted to not include it also.

Just a quick observation for what it's worth:
Single-line selections, and multi-line selections made by dragging up (resulting in the EOL capture at the top, instead of the bottom) emit the same behavior. (Technically, a whole-line selection made on a single line would count as two lines anyways even if perceived as a single line, but worth noting nonetheless.)

Released with 2.10.2 (pre-release version), it should be available in the Marketplace shortly

Just a quick observation for what it's worth:
Single-line selections, and multi-line selections made by dragging up (resulting in the EOL capture at the top, instead of the bottom) emit the same behavior. (Technically, a whole-line selection made on a single line would count as two lines anyways even if perceived as a single line, but worth noting nonetheless.)

e906199544adecfdafcf96d7b832821ae7eccd7e

Testing prerelease v2.10.3:
Line-selections are still including the orphan EOL in the sort, producing unexpected results. Note that "Line-selections" are referring to a full-line selection, made by clicking on the line number to the left. Notice how moving lines up and down using Alt with an up/down arrow exclude the line where the cursor appears below the selection. The sort option should do likewise.

Ok, for the first gif the problem is the cursor at the beginning on the new file (but without characters selected): the current logic uses the cursor's position to determine if the line should be sorted or not, regardless of an actual selection. I'll change that.
The secont gif is more surprising, line 13 (zz1) should not be touched at all... 🤔

The cursor position is important for placing it correctly after the sort operation, but it's of no relevancy to what gets sorted. In the first gif, it has to go on the next line since the line-selections include the EOL with the lsat line (seen by the extra non-existent space at the end of each highlighted line). Doing a normal char selection (click-drag the actual text, not the line numbers) will omit the final line's EOL in the selection because your selection area (which the cursor follows) stops on that last line (before EOL) not below it (after EOL).
Note that the cursor position is different after the sort, in the first gif, and in the second gif it is correct but the last line no longer shows the EOL as part of the selection.

586343f7a8c2126a5b4a2d80a3ffcf618e56a5b3
Pre-release 2.10.4 should be available in the Marketplace shortly

💯 works perfectly


JAVA-1100: Exception when connecting with shaded java driver in OSGI

Thank you for your contribution!  We were explicitly working around this in our tests, it didn't register with us at the time that this was something we could fix in our build.   I think no PR for 3.0 is needed, we will take care of merging this upstream when we merge 2.1 into 3.0

Confirmed, if can remove bootDelegationPackages("javax.security.*") from our test and our shaded configuration passes without it, where previously I could not.  :+1:

Great that it solved another problem as well! I can remove it in the patch directly.

Looks great!  :+1:

Thanks for solving 2 issues with one PR! :+1:

Merged, thanks.


NG Bootstrap UI - cannot apply a class on the DynamicRadioGroupModel options

Hello,
As you can see HERE, the class btn-outline-primary is not applied on the control options.
I think in this case, the class should be applied on the label field which is representing the option:
<fieldset *ngSwitchCase="7" ngbRadioGroup ...>
    <legend *ngIf="model.legend" [innerHTML]="model.legend"></legend>

    <label *ngFor="let option of model.options" class="btn btn-primary"> <-------APPLY THE CLASS HERE

        <input type="radio" ..../><span [innerHTML]="option.label"></span>
    </label>
</fieldset>

What do you think ?

@jogelin Yes, it think you found another weakness!
Maybe you'd like to improve the current template file on your own and provide a pull request?

I would like to help with PR but:

Impossible to install without kendo...why do you put libraries dependencies in the parent package.json ? It should be installed in each module no ?
The travis build is failing because of kendo

@jogelin

why do you put libraries dependencies in the parent package.json

Because otherwise you're unable to make the sample application running via npm install. I haven't figured out a better solution for Kendo yet, although I'm aware that it's available open source.


Using multiple OIDC issuers  or multiple client-id+secret?

Hi!
First off - thanks for writing this excellent package! I'm successfully using a my own fork based on @funkypenguin's fork to allow using with my own OIDC provider (in this case dex).
As I've separated my LAN in to different VLAN segments (e.g. admin services and normal services), I'd like to use be able to give different user groups access to those different services. This would prevent e.g. a use that has access to say Radarr to also access Portainer. For this I guess I would need to set a different OIDC provide and/or different client_id+secret.
It's trivial to just use another instance of traefik-forward-auth with different credentials, but would it make sense to also try and do this with a single traefik-forward-auth instance (using different cookie-names as well, as I'm using the same domainname for all services)?
My questions:

would it make sense to use a single traefik-forward-auth for multiple OIDC clients?
if so, is there anything being done already to support multiple client_id+secrets w/ traefik-forward auth?

Thanks!

Hi!
With tfa v0, the easiest fix would be multiple instances of tfa.
With tfa v2 you can pass credentials on a per-provider basis, however we still only support Google! OIDC support is being tracked in #47
I'm going to close this issue as there's a solution and path forwards tracked separately, but please feel free to re-open if you have any further questions!

Hey @thomseddon just asking the same question as above, as I'm not sure we answered the 'multiple OIDC issuers, is it actually possible to create multiple OIDC providers within a single TFA instance?
Maybe like being able to set the issuer-url/etc on a per rule basis? Or would I need to run multiple TFA instances per OID issuer?

Hi!
With tfa v0, the easiest fix would be multiple instances of tfa.
With tfa v2 you can pass credentials on a per-provider basis, however we still only support Google! OIDC support is being tracked in #47
I'm going to close this issue as there's a solution and path forwards tracked separately, but please feel free to re-open if you have any further questions!

Hi! First of all, thanks a lot for this awesome tool! I was wondering if by any chance there was documentation on the OP answer in the case of one tfa for multiple client?
Thanks a lot!!


Sims Recipe

Reorganized doc/Cookbook to include Sims Recipe, fixed links etc. I think this is ready for @danielsf and @rbiswas4 to take over.

Note that I moved the phosim data generation part of the original cookbook into a new Sims_Recipe.md file, but I left the reference catalogs for calibration part in the DM Level 2 recipe (now with a new filename!).


Fix hex view duplication issue

Fix hex view duplication issue

Issue was caused by variable not being set to empty string on each new debug session being started.

Closes #616

@stevedlawrence @arosien @mbeckerle Would one of you guys be able to give this look over. Believe once this PR is merged we can try to get an rc2 out for testing.


Transaction plasma

fixed all test for standard exits and transfer

*fyi merged into start-standard-exit branch


too many types of errors

What is the expected enhancement?
Right now we have separate error types for different components: MapperError, DAGCircuitError, QASMError, ExtensionError, TranspilerError, CouplingError. Why do we need these? Can't we make every error just be a QiskitError?
If there is truly a benefit, then we should be consistent. For example add a QobjError as well. But I don't think this is needed.

I'm closing this issue in favour of #601. Ali, if you don't mind, move the conversation there.


remove some prices out of the product

@lauraluiz  which prices should be removed or kept?

We should remove every country we are not supporting, as now it feels we are having have of the world there. It really kills performance without the price improvements in the platform.

What are the countries we are supporting?

@lauraluiz @schleichardt if I can support you please let me know.
At the moment we have DE, US, FR, IT and ES. I could remove FR, IT and ES.

@schleichardt @trangng we support those countries, but our data is holding 28 countries per variant. So yes, we have to remove these additional countries that we do not officially support. 5 countries in total should be fine.

@lauraluiz ok we can remove it manually or are there other faster ways?

I think I can remove it very fast.

For now we keep GB, IT and DE + the price without country.

done in f8b7198a6631c0cda9a6290a4adfcb60e6ec4615.


Open Dataset dialogue keeps values from previous selection

From @gunbra32 on June 22, 2017 8:25
Expected behavior
I would expect the Open Data Set dialogue to be either empty or pre-filled with values corresponding to the actual dataset to be opened.
Actual behavior
When choosing to open a second dataset, the values for variable and time constraint from the previous dataset are showing up in the dialogue box. In the example shown below, the proposed time constraint is Out of range and the variable constraint is from the previously opened aerosol product and not from the FIRE dataset.

Steps to reproduce the problem

Open an arbitrary dataset.
Open a second arbitrary dataset.

Note that I observed this behaviour when step 1 failed dut to the unavailability of esa_cci_odp. I am currently not able to test if it the described behaviour also occurs after an successful opening as the esa_cci_odp is nor working for me at the moment.
Specifications
Windows 10
<EMAIL_ADDRESS>Provide information such as the version of the project, operating system, or hardware.
Copied from original issue: CCI-Tools/cate-desktop#20

The idea was to let users open/download data from the same spatial/temporal subset across multiple data sources. I agree, that this approach must be intersected with the actually available spatial/temporal coverage.
What about having named subset configurations that are stored with the app and which you can apply to any data source just by selecting the subset's configuration name? Each subset configuration contains

spatial range
time range
variable(s) names

From @mzuehlke on August 15, 2017 8:44
Bug filed by @HerzogStephan and @HelenClifton mentioning related problems:

Overwritten 'Time constraints' in consecutive download attempts #37
Values persisted in "Download Data Source" window CCI-Tools/cate-core#307
Values persisted in "Open Local Dataset" window CCI-Tools/cate-core#308


dvdbackup: fix build

Motivation for this change
fix build
Things done

[X] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS linux)
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 nixpkgs-review --run "nixpkgs-review wip"
[X] 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)
[ ] Ensured that relevant documentation is up to date
[ ] Fits CONTRIBUTING.md.

NB this is #97479 related :-)

Review request acknowledged. I need to build a bit more context on that patch to give a reasonable review, but I will do my best!
Thank you for your contribution @woffs !

@bradediger :-)
I'm not sure if it's ok to simply refer to a patch from slackware, but it just solves the build problem.

I'm not sure if it's ok to simply refer to a patch from slackware, but it just solves the build problem.

I've seen people refer to patches from mailing list archives before.


[ROX-24638] Add vuln request migrations

Description
These migrations are based on vuln request migrations that were previously added and reverted in #8621 when release of exception management was delayed until 4.5.

m_200-201 : Migrated legacy vuln request fields to new fields
m_201-202: Creates globally scoped vuln requests for CVEs that are snoozed using legacy "Defer and Approve" workflow from ImageCVEs page

Checklist

[ ] Investigated and inspected CI test results
[X] Unit test and regression tests added
~[ ] Evaluated and added CHANGELOG entry if required~
~[ ] Determined and documented upgrade steps~
~[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)~

If any of these don't apply, please comment below.
Testing Performed
Here I tell how I validated my change

Unit tests
Manual test by creating vuln requests and snoozed CVEs in 4.3 and then upgrading to current image.
Same as above with 4.2
Manual test by creating vuln requests and snoozed CVEs in 4.3, upgrading to current image, doing a rollback and then re-upgrading.

Reminder for reviewers
In addition to reviewing code here, reviewers must also review testing and request further testing in case the
performed one does not seem sufficient. As a reviewer, you must not approve the change until you understand the
performed testing and you are satisfied with it.

be warned I have one in progress too.
https://github.com/stackrox/stackrox/pull/11532


Add jest-serializer-vue

Opening this issue as a place for discussion on this matter.
Planning to discontinue expectToMatchSnapshot() in utils.js and use jest-serializer-vue instead.
Below are some of my thoughts. Comments are welcome.
Current Snapshot
exports[`Carousel should mount successfully 2`] = `
".VueCarousel-wrapper
  .VueCarousel-inner(role='listbox', style='transform: translate(0px, 0); visibility: hidden; height: auto;')
|
.VueCarousel-pagination(style='display: none;')
  ul.VueCarousel-dot-container(role='tablist')
//
"
`;

Native Jest Snapshot
exports[`Carousel should mount successfully 1`] = `"

"`;

jest-serializer-vue Snapshot
exports[`Carousel should mount successfully 1`] = `

`;

Though, I find little to no advantage in making this change. I had hoped jest-serializer-vue to optimize the output so it's pull request friendly but it was a pretty straightforward serializer.
What I had hoped for
This will reduce noise when reviewing pull requests with updated snapshots.

Perhaps, a serializer with prettier might be able to help. See Sample.
Even though it's not exact as what I had hoped for but it will still wrap when a line exceeds the print width which seems fair to me.

Fair enough. I'll stick to using the existing jest-serializer-vue then.
I couldn't find any serializer with prettier support so it'll be custom if that is to be added.

Hmmm ok cool, thanks for looking into it. Could be an interesting package to build... maybe I'll give it a shot when I have some spare time


Auto-doc uses cleaner custom formatting rather than prop/attribute sphinx formatting

See #110

Current coverage is 96.25% (diff: 100%)

Merging #119 into feat/imagefileprop will not change coverage

@@           feat/imagefileprop       #119   diff @@
====================================================
  Files                      10         10
  Lines                    1387       1387
  Methods                     0          0
  Messages                    0          0
  Branches                    0          0
====================================================
  Hits                     1335       1335
  Misses                     52         52
  Partials                    0          0

Powered by Codecov. Last update c08affb...b3fdc18


[Feature Request]: Can I talk to LLM directly

Is there an existing issue for the same feature request?

[X] I have checked the existing issues.

Is your feature request related to a problem?
目前用ragflow检索能力很强，但是还要部署一个dify去做一些直达大模型的实例，与ragflow系统

Describe the feature you'd like
我尝试过编排里面去掉检索直接对接LLM，发现报错，不能正常工作
Describe implementation you've considered
No response
Documentation, adoption, use case
No response
Additional information
No response


Fix compiler warnings in native code

Overview
Fix various warnings from future compiler errors.

See: https://jira.bf.local/browse/ECR-XYZ
Definition of Done

[ ] There are no TODOs left in the code
[ ] Change is covered by automated tests
[ ] The coding guidelines are followed
[ ] Public API has Javadoc
[ ] Method preconditions are checked and documented in the Javadoc of the method
[ ] Changelog is updated if needed (in case of notable or breaking changes)
[ ] The continuous integration build passes

Coverage remained the same at 85.891% when pulling 7bcb43604c49985aa006ad093caee965df7f9f22 on vitvakatu:update_native_code into e34be40d01a03e07b70b1b5475ef7d935dec6cd4 on exonum:master.


Misc code clean up

This commit contains a lot of changes modernizing the code. All of them are hopefully uncontroversial.

Fix some Clippy warnings for arch through hostname.
Remove redundant 'static annotations.
Remove utf8 feature and use std instead.

LGTM :+1:


Android - NetSuite - App freezes then crashes when connecting to NetSuite

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.58-0
Reproducible in staging?: Y
Reproducible in production?: N
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: Y
If this was caught during regression testing, add the test name, ID and link from TestRail: https://expensify.testrail.io/index.php?/tests/view/5188341
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause Internal Team
Action Performed:

Login to the hybrid app with an account
Create a new workspace
Go to workspace settings > More features > Enable accounting
In the accountings page connect to NetSuite > Upgrade to control > Finish
Go through the flow of entering the credentials for connecting to NetSuite

Expected Result:
The workspace connects to NetSuite
Actual Result:
App freezes then crashes
Workaround:
Unknown
Platforms:

[x] Android: Standalone
[x] Android: HybridApp
[ ] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

Screenshots/Videos
0611_7.txt
https://github.com/user-attachments/assets/45b5419b-99db-43d9-8abc-9afb026edc80

View all open jobs on GitHub

@srikarparsi do you think this is a deployblocker? I Definitely think it's high priority but wonder if we can downgrade as it's only affecting Android users setting up an NS connection?

Hey! Taking a look

Checking if this is still reproducible since a lot of the crash errors got fixed by this revert

Yeah, I think this was also fixed by this CP. I'm going to close this out since it's not reproducible on staging anymore.


Errors when obfuscating input with string/hex_hash -> string/file_glob or string/folder_glob

When using bashfuscator in a command like this:
bashfuscator -c "echo hi" --choose-mutators string/hex_hash string/file_glob --test --clip
The test output will sometimes look something like this:
/bin/bash: line 42: printf: missing hex digit for \x
/bin/bash: line 51: printf: missing hex digit for \x
/bin/bash: line 56: printf: missing hex digit for \x
/bin/bash: line 62: printf: missing hex digit for \x
/bin/bash: line 117: printf: missing hex digit for \x
/bin/bash: line 213: printf: missing hex digit for \x
/bin/bash: line 266: printf: missing hex digit for \x
/bin/bash: line 0: printf: missing hex digit for \x
/bin/bash: line 0: printf: missing hex digit for \x
/bin/bash: line 0: printf: missing hex digit for \x
/bin/bash: line 0: printf: missing hex digit for \x
/bin/bash: line 0: printf: missing hex digit for \x
/bin/bash: line 0: printf: missing hex digit for \x
/bin/bash: line 0: printf: missing hex digit for \x
hi

This stems from the fact that the string/hex_hash Mutator generates printf statements like printf "\x$(...)", and the string/file_glob and string/folder_glob Mutators break the input commands into chunks and write them to files using printf.
Therefore, string/file_glob or string/folder_glob will inevitably try to use printf a string with \x in it, and raise the missing hex digit for \x error because no digit precedes the \x string.
@cpbarker and I started to find a fix by changing all occurances of printf -- to printf '%s' in the string/file_glob and string/folder_glob Mutators. This helps because the '%s' argument makes printf interpret the next argument as a literal string, so that \x isn't parsed by printf as an escape sequence.

@343iChurch I think I (accidentally) fixed this issue in one of the commits I made on the dev branch today, can you test and verify?

@capnspacehook verified this issue is "fixed" as of 6891379, but the fix appears to only work in Debian-based distros. Have tested in Ubuntu, Debian, and Ubuntu WSL. Fedora/RHEL/Centos still have the old issue, may be releated to issue #5.


LBF results with IA2C and MAA2C: Foraging-15x15-4p-5f-v1

Hi,
I would like to congratulate the authors for the article and for the initiative of maintaining those benchmark environments the community sorely needs. For the first time ever, I haven't had any major issues replicating the results from a reinforcement learning article. With the exception of the Level-based Foraging environment, namely for the task Foraging-15x15-4p-5f-v1 and algorithms IA2C and MAA2C with parameter sharing. The plots that I got where:

When we contrast with yours:

We can draw the following conclusions:

For both algorithms there is a lot more of standard-error. Prominently at the beginning of the training.
Judging by the average returns: My IA2C slightly over-performs yours while my MAA2C slighlty under-performs yours.
As a result, while your plots show that MAA2C is a better option we cannot draw the same conclusion by looking at mine.

By using the evaluation metrics I was able to get:

IA2C
IA2C
MAA2C
MAA2C

Mine
Yours
Mine
Yours

Maximum Return
0.92 ± 0.05
0.93 ± 0.03
0.92 ± 0.02
0.95 ± 0.01

Average Return
0.65 ± 0.04
0.59 ± 0.06
0.67 ± 0.04
0.73 ± 0.02

The random seeds used for both experiments are:

291174067
392184168
402285178
493194269
503295279

The configuration for the IA2C:
{
  "action_selector": "soft_policies",
  "add_value_last_step": true,
  "agent": "rnn",
  "agent_output_type": "pi_logits",
  "batch_size": 10,
  "batch_size_run": 10,
  "buffer_cpu_only": true,
  "buffer_size": 10,
  "checkpoint_path": "",
  "critic_type": "ac_critic",
  "entropy_coef": 0.001,
  "env": "gymma",
  "env_args": {
    "key": "lbforaging:Foraging-15x15-4p-5f-v1",
    "pretrained_wrapper": null,
    "state_last_action": false,
    "time_limit": 50
  },
  "evaluate": false,
  "gamma": 0.99,
  "grad_norm_clip": 10,
  "hidden_dim": 128,
  "hypergroup": null,
  "label": "default_label",
  "learner": "actor_critic_learner",
  "learner_log_interval": 10000,
  "load_step": 0,
  "local_results_path": "results",
  "log_interval": 500000,
  "lr": 0.0005,
  "mac": "basic_mac",
  "mask_before_softmax": true,
  "name": "ia2c",
  "obs_agent_id": true,
  "obs_individual_obs": false,
  "obs_last_action": false,
  "optim_alpha": 0.99,
  "optim_eps": 1e-05,
  "q_nstep": 5,
  "repeat_id": 1,
  "runner": "parallel",
  "runner_log_interval": 10000,
  "save_model": true,
  "save_model_interval": 500000,
  "save_replay": false,
  "seed": 291174067,
  "standardise_returns": false,
  "standardise_rewards": true,
  "t_max": 20500000,
  "target_update_interval_or_tau": 0.01,
  "test_greedy": true,
  "test_interval": 500000,
  "test_nepisode": 100,
  "use_cuda": false,
  "use_rnn": true,
  "use_tensorboard": true
}

Note that I wasn't sure why the configs in src/config/algos/ had a larger t_max than the reported. In this case I dropped the last evaluation from this analysis.
The diff ia2c/1/config.json maa2c/1/config:
11,12c11,12
<   "critic_type": "ac_critic",
<   "entropy_coef": 0.001,
---
>   "critic_type": "cv_critic",
>   "entropy_coef": 0.01,
34c34
<   "name": "ia2c",
---
>   "name": "maa2c",
40c40
<   "q_nstep": 5,
---
>   "q_nstep": 10,
50c50
<   "t_max": 20500000,
---
>   "t_max": 20050000,

Unless I missed something the critic_type isn't covered in the article the difference in configuration are compatible with the different configurations reported on Tables 14 and 24.
Could you clarify what may be happening? And how can I approximate your results?

To verify the results we might need more independent runs. In that particular case I ran about 20 independent runs from which I collected 5 seeds that yield the reported performance as per Henderson, et al. 2019 Random Seeds and Trials


.Net: AzureCosmosDBNoSQLMemoryStore GetAsync with embeddings false implementation

Motivation and Context
GetAsync is returning with embedding even if the withEmbedding parameter is false.
Description
Current implementation will return MemoryRecord based on withEmbedding flag.
Note
The method isn't being called yet, but I updated it because the implementation was missing.
Contribution Checklist

[ ] The code builds clean without any errors or warnings
[ ] The PR follows the SK Contribution Guidelines and the pre-submission formatting script raises no violations
[ ] All unit tests pass, and I have added new tests where possible
[ ] I didn't break anyone :smile:

@atiq-bs23 we have recently announced a new Microsoft.Extensions.VectorData.Abstractions package which we have developed in partnership with the .NET team. We have also started migrating our memory connectors to use these new abstractions and plan to obsolete the old implementations. So we are not taking any new changes to the older memory connectors. Thanks for your interest in the Semantic Kernel and your contributions.


Checkbox isn't checkboxing (Visual)

When you check a box it doesn't show any difference from the checked or not checked box. It's annoying in "Browse Channels" cuz you can't see which is checked. I will send screenshots.
 = Checked
 = Unchecked

There is the same problem with the checkbox for trusted links


fixes for running on older versions of .net and windows

 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.Mark Reed seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

I needed to comment out the following lines in WebRtcSession.cs to get the example working:
Spitfire.OnDataChannelConnecting += Spitfire_OnDataChannelConnecting;
Spitfire.OnDataChannelClosing += Spitfire_OnDataChannelClosing;
Spitfire.OnDataMessage += Spitfire_OnDataMessage;
Spitfire.OnDataBinaryMessage += Spitfire_OnDataBinaryMessage;

Also in Program.cs it may be more informative to change
Console.WriteLine("Meesage from peer:" + ex); to Console.WriteLine("Meesage from peer:" + ex.Data);
But other than that this should really be merged in


Build Phases Shell Script.

For me with React Native in xcode 10, I had to modify the script as so:
export NODE_BINARY="'node --max_old_space_size=8192' ../node_modules/react-native/scripts/react-native-xcode.sh"
Note the addition of double quotes.

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

ID: 6826cbd3-5ac7-94f7-6708-5f9c7f9a8d5a
Version Independent ID: 5e5287a4-d3f0-70f9-0f1f-c0f6b8ab8938
Content: App Center Build Code React Native Troubleshooting - Visual Studio App Center
Content Source: docs/build/troubleshooting/react-native.md
Service: vs-appcenter
GitHub Login: @nrajpurkar
Microsoft Alias: nirajpur

@potatowave thanks, I'll have the PM take a look at making an update. @nrajpurkar

@potatowave thanks for the catch here. I'll update the docs to include this difference for RN and Xcode 10.


[next] Docs update loading assets page

Admittedly, when I first saw this attach function, I didn't really understand it.
<T.MeshStandardMaterial
    map={texture2}
    attach={(parent, self) => {
      if (Array.isArray(parent.material)) parent.material = [...parent.material, self]
      else parent.material = [self]
    }}
  />

I've created a second alternative, which hopefully helps others figure it out sooner 😅 It's also a example of how we can mix and match 'normal' three.js code 👍

prettier is only worried about xr files
Checking formatting...
[warn] packages/xr/src/lib/plugins/pointerControls/hook.ts
[warn] packages/xr/src/lib/plugins/pointerControls/setup.ts
[warn] packages/xr/src/lib/plugins/teleportControls/hook.ts
[warn] Code style issues found in 3 files. Run Prettier to fix.
 ELIFECYCLE  Command failed with exit code 1.
Error: Process completed with exit code 1.

this PR is fine 😅


Don't animate multiLine TextField height while mounting.

When a multiLine TextField has an initial value longer than a single line the element mounts as a single line and then animates the change in height to the full text height as it's matched to the shadow text area.
For one it looks a bit weird, but it also causes problems when reading the container's height immediately after a mount since that uses the incorrect height of the text field at the start of said animation. My solution is to only enable the transition style once the component has mounted.

Apparently React doesn't like the use of isMounted() in the render method, so I'm using a manual _mounted flag instead.

I'm wondering. Is this animation triggered by a TextField rendering twice?
I can't see any animation on the material-ui.com website.

On the Material-UI website the text fields are all empty. I think this only happens when you already have text in the input (set via value prop) when it renders. Then it renders with a height of one line and animates its height to match that of the set text.
If you type in one of the multi-line inputs on material-ui.com until you go over into a new line you will see that the change in height is animated. It is that animation, which also happens on the initial render, that is causing my problems.
(I must admit, though, that it's been some time since I filed this. I'm busy sorting out some other issues with overflow: hidden/visible margin collapsing at the moment too, so if you want to wait with this until I confirm that I've sorted everything else and this is definitely the source of my problem that will be fine with me.)

Then it renders with a height of one line and animates its height to match that of the set text.

Should it render the first time with the right height? I feel like we can fix this issue in a better way.

Yes, that's the idea. When it mounts for the first time it should already have the proper height once we get to componentDidMount, which it doesn't if there is an initial animation. If the value changes at any time thereafter, be it via user input, value or valueLink the animation should be applied. This is just about that initial render.
If you have ideas on how to implement a better fix please let me know. I'll also take another look at the code to see what I can come up with, it's been a while since looked at the source.

@marnusw This PR is old. I'm closing it in favor of an issue.


Any way to filter the request_path ?

Hello
Is possible that any way to filter the request_path? For example I want to see analyze the just ".css" files (hit count /4xx 3xx 2xx errors)
Are there any parameter ?

Can I be assigned this feature

@maxiceptr hi there, sorry for this late reply but you can use --filter param for this, e.g. ngxtop --filter 'request_path.endswith(".css")'


fix(display): small and xs size

fix(display): small and xs size
Before:

After fix:

@dingnanco This is intentional. I've not built mobile versions for the other examples either. I think even on mobile the desktop screenshot shows a lot of what the UI looks like when put together.


Auto send message to unknow caller.

When module receives call from unknow number it should reject the call immediately and send message to that caller too to contact the owner number, currently it just sends message to owner and don't cut the call, it would be a best option to manage this from orange pi, what do you think @wasey-rao?

Regarding you question No, I don't think we should manage this from orange pi what if only ttgo is in working state and orange pi is disconnecteed due to any reason? I think we should implement this functionality in ttgo module only just ping orange pi and send data to save records not to manage this functionality and I think you should first make sure #136 this issue is close as a lot of issues can be reported due to this issue, so please do complete it as soon as possible. please do let me know which calls will be white/black listed for this issue @mhamidjamil

I follow these rules:

Don't send message to company calls (length of caller number less then 10)
Don't send messages to PTCL number stat from 042 or any.

It only sends message to mobile number with length greater then 10 and start from +923 or 03
I filter out the annoying number like if some one call more then 3 times a day then I mark that as a black listed number and will not send message to that number.

these are the points I'll follow and hope fully I close this issue on this weekend, do you have any other suggestions @wasey-rao  and you can mark these points a task list.


Update Scala to 3.2.0

This currently relies on a 2.13.9 snapshot version (2.13.9-bin-d578a02), that is able to read TASTy generated by 3.2.0. Let's wait for 2.13.9 final prior to merging this.


A method to get names and vectors of an EmbeddingSet

Suggested:
names, X = embset.to_names_vecs()

The most concise way to achieve it with current API:
names = list(embset.embeddings.keys())
X = embset.to_X()

Besides its potential use case, this could replace the utility function embset_to_X which is used in transformers.
Further, I have a feeling, which I don't know is good or not, that the need for the end-user to access embeddings property of EmbeddingSet should be minimal since it's mostly an implementation logic.

Although, this could be achieved another way as well:
X,  names = embset.to_X_y('name')

Though, that's a bit weird...... but works anyways!

In this case name can also refer to a property?

In this case name can also refer to a property?

Yes, that's basically what you have implemented for that purpose, i.e. to get custom properties.

I'm certainly open to a method like embset.to_names_vecs() I'm wondering if we can come up with a better name though.
Thesaurus mode!

embset.names_vectors ?
embset.to_X_names ?

Preference?

I think embset.to_names_X and embset.from_names_X would be good.

That certainly feels consistent. Agreed!


reduceBy, reduceWhile, tail, F, T function length is incorrect

All functions in the title have 0 length.
console.log(`${R.reduceBy.length} ${R.reduceWhile.length} ${R.tail.length} ${R.F.length} ${R.T.length}`)
> 0 0 0 0 0

Function length seems generally right, so I wonder if these functions' behaviour is intended.

I've fixed reduceBy and reduceWhile as part of https://github.com/ramda/ramda/pull/1843. I believe F and T are correct having a length of 0. tail does appear to be wrong. Would you like to submit a pull request to fix tail?

I don't know if I did well, but I push it anyway.

I noticed my pull request has conflict with your #1843 removing _curry1.
But I'm not sure what method is right and I have enough knowledge to do it.
Can I use curryN(1)? If this method is also problematic, I think it's better for others to do it.

There's no guarantee #1843 will be merged and I'll have to do a ton of rebasing anyway at this point, so your use of _curry1 is fine :)


Props to customise text in Stepper buttons

Reformat and refactor code in the component

could you please add documentation to use the new props?

I have added the documentation now but I think someone else made a pull request for this afterwards so I will close this one


Sleek says it archives to done.txt, but actually archives to todo_done.txt

Is it an actual bug?
Yes
Did you check if the bug has already been reported?
Yes
Describe the bug
Sleek says it archives to done.txt, but actually archives to todo_done.txt
To Reproduce
Steps to reproduce the behavior:

Create a task
Complete the task
Archive the task, it will say it will write the task in done.txt
See that it in fact writes to todo_done.txt

Do you see any error entries in sleeks developer tools?
No errors as far as I can see
Expected behavior
done.txt was written to instead of todo_done.txt
Screenshots
Not applicable.
Desktop (please complete the following information):

OS: Windows 10
Version of sleek 1.1.9
Source: Github

@dragos240 In fact it archives to NAMEOFYOURTODOFILE_done.txt, so I will need to add some dynamic wording here. Thanks for pointing it out.

I ran into this issue as well.
Personally I use done.txt, so it would really be great if we could change this as a setting. I expect there are a number of people that use todo.txt because there are a lot of options when it comes to integrating it on different platforms. But if this is a static setting it sort of hamstrings that compatibility. As done files are not really a part of the spec.
But it's not really a show stopper for me. I use sleek because it's text is slightly bigger and easier to read.

@ransome1, In my case, no file is created. It says it moves to done.txt. It removes all the completed tasks from the todo.txt file, but no file is created anywhere. I searched the entire file system, but don't see any new .txt files created. Right now, it seems like those tasks are just gone. Is there a way to figure out where they got stored, if at all?
I am on Windows 10.

I can't confirm this. I downloaded 1.2.9 from GitHub and the done.txt file being created in the same directory in which the actual todo file is located.
Can you please check the developer console, if there are any errors related to this? Press the alt key once and the sun menu will appear. In there click on About and Toggle developer tools.

Looked at the developer console, but I do not see any errors in the console tab. But there is no log mentioning that a new file is being created, though.

I assume the issue is the name of your folder, which contains a period. Maybe this breaks something on windows. I'll look into it tomorrow.
Can you please try the rename the directory and remove the period character. I'm wondering if the issue would still occur.

Tried without the period in the name. It is working now! Windows doesn't seem to have any problem with the folder. Does the issue come from Javascript?
Also, is there anyway to get the completed todos that failed to get archived?

Looking at the code, I see this line is the cause. It is replacing the name in the entire file path string instead of doing just the filename.
https://github.com/ransome1/sleek/blob/aad500c0d727294f2be0579b2a76e27394c9e291/src/js/helper.mjs#L169

I went ahead and created the PR. You can take a look - https://github.com/ransome1/sleek/pull/446

@durgaswaroop amazing, thank you so much :) I'll take a look at it as soon as I am on a computer!


Quick input smoke test failure/timeouts

when testing #119524
Running the smoke tests on windows, I'm seeing a few quick input test timeouts/failures.
using yarn smoketest --build <path to latest version> --stable-build <path to stable version>
  1) shows results from all folders is failing with this message:
1) VSCode Smoke Tests (Electron)
      VSCode Smoke Tests (Electron)
        Multiroot
          shows results from all folders:
    Error: Timeout: get elements '.quick-input-widget .quick-input-list .monaco-list-row .label-name' after 20 seconds.
     at poll (C:\Users\merogge\repos\vscode\test\automation\out\code.js:173:19)
     at async Code.waitForElements (C:\Users\merogge\repos\vscode\test\automation\out\code.js:250:16)
     at async QuickInput.waitForQuickInputElements (C:\Users\merogge\repos\vscode\test\automation\out\quickinput.js:24:9)
     at async Context.<anonymous> (out\areas\multiroot\multiroot.test.js:39:13)

using yarn smoketest --web --browser chromium
 ** Timeout! page.evaluate: Evaluation failed: Error: Active element not found. Current active element is 'HTML > BODY.windows.web'. Looking for .search-view .search-widget .search-container .monaco-inputbox textarea     at BrowserWindowDriver.isActiveElement (http://localhost:9888/static/out/vs/platform/driver/browser/baseDriver.js:63:23)     at eval (eval at evaluate (:300:31), <anonymous>:1:15)     at eval (<anonymous>)     at UtilityScript.evaluate (<anonymous>:300:31)     at UtilityScript.<anonymous> (<anonymous>:1:44)     at Connection.sendMessageToServer (C:\Users\merogge\repos\vscode\node_modules\playwright\lib\client\connection.js:69:15)     at Proxy.<anonymous> (C:\Users\merogge\repos\vscode\node_modules\playwright\lib\client\channelOwner.js:44:61)     at C:\Users\merogge\repos\vscode\node_modules\playwright\lib\client\frame.js:157:48     at Frame._wrapApiCall (C:\Users\merogge\repos\vscode\node_modules\playwright\lib\client\channelOwner.js:72:34)     at Frame.evaluate (C:\Users\merogge\repos\vscode\node_modules\playwright\lib\client\frame.js:156:21)     at C:\Users\merogge\repos\vscode\node_modules\playwright\lib\client\page.js:344:60     at Page._attributeToPage (C:\Users\merogge\repos\vscode\node_modules\playwright\lib\client\page.js:205:20)     at Page.evaluate (C:\Users\merogge\repos\vscode\node_modules\playwright\lib\client\page.js:344:21)     at Proxy.isActiveElement (C:\Users\merogge\repos\vscode\test\automation\out\playwrightDriver.js:72:55)     at Proxy.<anonymous> (C:\Users\merogge\repos\vscode\test\automation\out\code.js:208:39)

So the first issue I couldn't repro and the second one I did hit but it passed on retry. @meganrogge can you try running the tests again?

@TylerLeonhardt we'd want to fix any flakiness

So I think the second issue should be moved to #115244 and then the first issue... I've run the smoke tests so many times and can't repro :(

Saw the following once when running desktop smoke test. Curiously that did not fail the test. This happened when the same multiroot test as above was running:
** Timeout!
Error: Active element not found. Current active element is 'HTML > BODY.windows > DIV.file-icons-enabled.monaco-workbench.windows.chromium.nopanel.vs-dark.vscode-theme-defaults-themes-dark_plus-json > DIV.monaco-grid-view > DIV.monaco-grid-branch-node > DIV.monaco-split-view2.vertical > DIV.monaco-scrollable-element > DIV.split-view-container > DIV.split-view-view.visible > DIV.monaco-grid-branch-node > DIV.monaco-split-view2.horizontal > DIV.monaco-scrollable-element > DIV.split-view-container > DIV.split-view-view.visible > DIV.monaco-grid-branch-node > DIV.monaco-split-view2.vertical > DIV.monaco-scrollable-element > DIV.split-view-container > DIV.split-view-view.visible > DIV#workbench.parts.editor.part.editor.has-watermark > DIV.content > DIV.grid-view-container > DIV.monaco-grid-view > DIV.monaco-grid-branch-node > DIV.monaco-split-view2.vertical.separator-border > DIV.monaco-scrollable-element > DIV.split-view-container > DIV.split-view-view.visible > DIV.editor-group-container.active > DIV.editor-container > DIV.editor-instance > DIV.gettingStartedContainer.height-constrained.animationReady > DIV.gettingStarted.showDetails > DIV.gettingStartedSlideDetails.gettingStartedSlide.detail > DIV.monaco-scrollable-element.full-height-scrollable > DIV.gettingStartedDetailsContent > DIV.getting-started-detail-columns > DIV.getting-started-detail-left > DIV.monaco-scrollable-element.full-height-scrollable > DIV.getting-started-detail-container > BUTTON.getting-started-task.expanded'. Looking for .quick-input-widget .quick-input-box input
/out/vs/workbench/workbench.desktop.main.js:570:78507)
    at v.call (file:///C:/Users/Administrator/AppData/Local/Programs/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:2793:64099)
    at w.onPromise (file:///C:/Users/Administrator/AppData/Local/Programs/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:75:68877)
    at w.onRawMessage (file:///C:/Users/Administrator/AppData/Local/Programs/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:75:68236)
    at file:///C:/Users/Administrator/AppData/Local/Programs/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:75:67489
    at c.fire (file:///C:/Users/Administrator/AppData/Local/Programs/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:60:1836)
    at EventEmitter.N (file:///C:/Users/Administrator/AppData/Local/Programs/Microsoft VS Code Insiders/resources/app/out/vs/workbench/workbench.desktop.main.js:58:60411)
    at EventEmitter.emit (events.js:315:20)
    at Object.onMessage (electron/js2c/renderer_init.js:117:979)
        √ shows results from all folders
        √ shows workspace name in title

@TylerLeonhardt I suggest to close this issue and monitor newer builds, there were many changes around quick pick recently. Specifically I am tweaking how retry works:
https://github.com/microsoft/vscode/commit/38f026eb35735f22602d1919a87a785b561e6dcc
Besides, I cannot find shows results from all folders in the list of failing tests lately: https://monacotools.visualstudio.com/DefaultCollection/Monaco/_test/analytics?definitionId=111&contextType=build


Support Jar deploy

[x] Merge PR: #167
[x] Re-organize the deploy logic, to make sure war, jar & ftp can works for all kinds of Web Apps (Windows, Linux, Containers)
[x] Add unit test
[x] Add integration test
[ ] Update documents & README
[ ] Run tests & verification

We also need to change checking method for war & jar deploy to make sure it will not block container Web App.

Hi @brunoborges,
We are working on the integration works for the Jar deploy and find some problems. May need your suggestions.
Currently if the user does not set the deployment type in the plugin configuration, we will leverage the <packaging> field to decide which deployment type will be used. This decision may have several problems in the following scenario:

When user does not want to deploy anything, just update the application settings, provision a resources, etc..
When user is using Web App for Containers, he only wants to change the image name setting for the Web App.

Can we still needs to check if a valid deployment type is set during the execution. And do jar/war/ftp deploy if user explicitly set the deployment type. What do you think?

I believe what we need is a new Deployment Type called "AUTO". This way developers can set NONE for the use case you are talking about, and AUTO would be default (to look into ).
What do you think?

Although frankly, I think this plugin should only be about deployment, and not configuration changes. It is weird to have the functionality of building a Maven project that will not deploy the artifact to somewhere, just update the runtime configuration.
We should consider removing the feature about updating config without redeployment. It does not make sense in the context of a Maven lifecycle.

I see. But I still do not want to block those container Web App users due to our AUTO deployment. Maybe we can check what the runtime user is using. And printing a warning instead of error if the runtime is Container?

The flow is like this:

Plugin checks deploymentType.
If set to NONE, nothing is deployed
If set to JAR/WAR/FTP, does those
If set to AUTO not set at all (AUTO is default), it inspects packaging of the Maven artifact (jar/war)

This should be sufficient.

@brunoborges I see, that makes sense!

I will code that one for you. PR coming soon.

Alright, check PR #216


Bug(scope): Data path "" must NOT have additional properties(inlineLoader) when generating a new scope

Is there an existing issue for this?

[X] I have searched the existing issues

Which Transloco package(s) are the source of the bug?
Transloco
Is this a regression?
No
Current behavior
When I run ng g @ngneat/transloco:scope my-scope I get :

Node.js version v17.3.0 detected.
Odd numbered Node.js versions will not enter LTS status and should not be used for production. For more information, please see https://nodejs.org/en/about/releases/.
Schematic input does not validate against the Schema: {"name":"my-scope","project":"my-project","langs":["en"],"translateType":"JSON","routing":false,"routingScope":"Child","flat":false,"commonModule":true,"lintFix":false}
Errors:
Data path "" must NOT have additional properties(inlineLoader)

Expected behavior
I hoped to generate a new scope.
Please provide a link to a minimal reproduction of the bug

Transloco Config
module.exports = {
  rootTranslationsPath: 'src/assets/i18n/',
  langs: ['en'],
  keysManager: {}
};

Please provide the environment you discovered this bug in
Transloco: 4.0.0
Angular: 13.3.2
Node: 17.3.0
Package Manager: 8.3.0
OS: Linux 5.11 Ubuntu 20.04.3 LTS (Focal Fossa)

Browser
No response
Additional context
No response
I would like to make a pull request for this bug
No

+1

+1


Horizontal toolbars?

Maybe I have missed something but is it possible to easily make horizontal toolbars with this plugin? If not yet, I would like to suggest it as a feature request.

yes! it's definitely possible. I'm assuming by "horizontal toolbar" you
mean a horizontal control-style toolbar (popup toolbars are horizontal by
default). Like this?
[image: Screen Shot 2019-06-24 at 6.46.10 PM.png]
You can achieve that by setting adding some CSS to change  tags from
display: list-item to display: inline-block (list-item cascades down by
default, whereas inline-block flows from left to right).
I recommend that you create a new class (say, L.Toolbar.Control.Horizontal,
or something similar) and set a new CSS base class (e.g.
.leaflet-toolbar-control-horizontal). You'll want to inherit all the styles
from .leaflet-toolbar-control, and also make the CSS change I described
above (see Toolbar.less
https://github.com/Leaflet/Leaflet.toolbar/blob/master/src/Toolbar.less#L83).
You'll also need to tweak a height attribute somewhere (not quite sure
where) to get the bottom border to look right on the horizontal toolbar.
If you want to submit this back upstream, I'd be happy to review a PR :).

The image doesn't show but basically, I want to create a toolbar for animation control and that is a bit awkward vertically. I'll try to make it work with your hints - thank you for those! - though I'm more or less a JS newbie. Modernizing my weather website saakeskus.fi is kinda also a huge JS learning project for me :D.

sorry about that! Here's the picture I tried to send before:

The approach I described above should allow you to create a horizontal toolbar by extending the Leaflet.toolbar library -- you shouldn't need to modify the library code at all. If you decide to submit a PR upstream (modifying the library to provide this new functionality), we would handle it a bit different -- probably creating an option on the Leaflet.Toolbar.Control class, rather than creating a new Leaflet.Toolbar.Control.Horizontal class.

Jup, that's what I meant. Thank you!

add the following css:
.leaflet-draw a {
float: left;
}


Sync pages between qiskit-community/qiskit-textbook

Some notebooks are out of sync between this repo and qiskit-community/qiskit-textbook. E.g. "Representing qubit states" in this repo still uses execute, whereas this was removed in https://github.com/qiskit-community/qiskit-textbook/commit/a4c5360fa01927c72ab455f3612d5476ce784fbd. We need to pull in updates from qiskit-community/qiskit-textbook, being careful not to lose any updates made on this repo.

@frankharkins, do you think I can try this? As you said, it requires being very careful.

@gprs1809 thanks, but I'm already working on it in https://github.com/Qiskit/platypus/pull/967

@frankharkins , I see. That's fine. Thank you!


Fix a test which gives a JDK-dependant output

Task/Issue URL: https://app.asana.com/0/0/1204998623539898/f
Description
Steps to test this PR
Feature 1

[ ]
[ ]

UI changes

Before
After

!(Upload before screenshot)
(Upload after screenshot)

Current dependencies on/for this PR:

develop

PR #3322   👈

This comment was auto-generated by Graphite.


Feature request: Drawings provide optional cover.

No idea if this is possible, or how difficult it would be if it is, but I would like to see drawings (especially polygonal drawings) have the option to provide cover, similar to how tiles currently can.
Many people use pre-made maps, with terrain features or object assets baked-in, thus making it impossible to assign cover to them like one could on a clean map using tiles for the terrain features or object assets. Drawings that offer cover provisions could make this possible, allowing the DM to create a drawing outline of an object (set to hidden), the boundaries of which would then be taken into consideration in cover calculations, similar to walls, or the non-transparent parts of tiles.

Should be able to support point-based drawings. Putting it on the docket.

Gah, as much as I love the idea, I'm still going to have to punt 😩 .
Tiles providing cover is a one solution. Ethereal+invisible (i.e. blank walls) WITH a cover value is also a supported solution as of the next release.

Fair enough. Thanks for this!


(Feature) Add all FV events to Amplitude

Description
Send the following events to amplitude:

FV_INTRO
FV_CAMERAPERMISSION
FV_GETREADY_ZOOM
FV_PROGRESS_ZOOM
FV_ZOOMFAILED
FV_SUCCESS_ZOOM
FV_TRYAGAIN_ZOOM
FV_GENERALERROR
FV_WRONGORIENTATION
FV_DUPLICATEERROR
FV_TRYAGAINLATER
FV_CANTACCESSCAMERA

Added the following features:

onReady & onCaptureDone callbacks to useZoomVerification hook (for send FV_TRYAGAIN_ZOOM & FV_PROGRESS_ZOOM)
onPrompt callback to usePermissions hook (for send FV_CAMERAPERMISSION)

About #1986
How Has This Been Tested?

Pass FV flow with different cases (success, poor photo quality, duplicate found, different errors etc)
check Amplitude dashboard

Checklist:

[x] PR title matches follow: (Feature|Bug|Chore) Task Name
[x] My code follows the style guidelines of this project
[x] I have followed all the instructions described in the initial task (check Definitions of Done)
[x] I have performed a self-review of my own code
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] I have added reference to a related issue in the repository
[x] I have added a detailed description of the changes proposed in the pull request. I am as descriptive as possible, assisting reviewers as much as possible.
[ ] I have added screenshots related to my pull request (for frontend tasks)
[ ] I have pasted a gif showing the feature.
[x] @mentions of the person or team responsible for reviewing proposed changes

Done

@serdiukov-o-nordwhale is my master merge correct? (ie this.logger)

@serdiukov-o-nordwhale is my master merge correct? (ie this.logger)

Yep, you've merged ZoomSDKweb correctly - params order is exact:


SQL Normalised cache on disk

Is your feature request related to a problem? Please describe.
Seeing a lot of mobile apps require cache persistence, I think it's mandatory to have optional support for SQL storage out of the box. I have started building the API here following Apollo client android pattern here https://github.com/apollographql/apollo-android/blob/master/apollo-android-support/src/
Describe the solution you'd like
I want the cache argument to accept any class that implements Cache interface internally, with support for multiple caches or chaining such as in-memory cache and disk cache  I would like full support for iOS and Android.
I would love feedback and feature you think we should add to it

The cache argument should definitely be able to support anything that implements Cache. Adding an SQL cache seems fine enough. I'd note that we have a persist-able, normalized cache already. But still, nothing wrong with more options

Closing as this is not a priority anytime soon. Would be interesting to do with angel's orm, maybe


go/importer: backport importer.For("source") fixes for 1.10

Tracking issue for 1.10 point release fix.
See #24392 and https://github.com/golang/go/issues/24392#issuecomment-373994843 for details.

CL 83415 OK for Go 1.10.1

@griesemer to save time in the future you can re-open the issue and put it in the appropriate milestone for backporting instead of going through the trouble of creating a new issue.

@andybons ACK


:bug: Set ARCH using go env

Signed-off-by: Jian Qiu<EMAIL_ADDRESS>
Summary
make build fails on mac os due to run ARCH. Change to use ARCH in go env instead
Related issue(s)
Fixes https://github.com/kcp-dev/kcp/issues/2088

Thanks! Going to go with #2091 as that came in slightly earlier 😄
/close


new issue

compiled apktool from source u can decompile but cant recompile here is error
brut.androlib.AndrolibException: brut.common.BrutException: could not exec: [C:\Users\sac\AppData\Local\Temp\BAT_temp\brut_util_Jar_872098322233231128.tmp, p, —forced-package-id, 1, —min-sdk-version, 26, —target-sdk-version, 26, —version-code, 26, —version-name, 8.0.0, —no-version-vectors, -F, C:\Users\sac\AppData\Local\Temp\BAT_temp\APKTOOL8770275689377228556.tmp, -x, -0, arsc, -0, png, -0, res/
ican decompile with 2.3.2
and recompile with 2.3.4
but issue just wanted to put out there thanks

There is not enough information here to go on. If you follow the issue template, we can see if there is anything we can do.


Update Axis.cs

Add newline before class TicksDateTimeSignal so that .cs file parses correctly for the website.

Hi @chrisxfire, this PR may set the record for the smallest number of characters required to fix a bug 🤣
The CI pipeline won't let me merge this yet because it wants that file you edited to be autoformatted.
In visual studio you can do that with Edit > Advanced > Format Document, or you could run this console command in the ScottPlot4 folder:
dotnet tool update -g dotnet-format
dotnet format

... actually I think you may have used the web editor to make that change, so rather than have you fire up an IDE to fix whitespace I'll just do it on my end right now and merge this. Thanks again for the fix! 🚀

Copy!  I’d do it from VS but I’m away from my machine 😊
Chris
From: Scott W Harden @.>
Sent: Thursday, March 9, 2023 5:44 PM
To: ScottPlot/ScottPlot @.>
Cc: Christian @.>; Mention @.>
Subject: Re: [ScottPlot/ScottPlot] Update Axis.cs (PR #2479)
... actually I think you may have used the web editor to make that change, so rather than have you fire up an IDE to fix whitespace I'll just do it on my end right now and merge this. Thanks again for the fix! 🚀
—
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FScottPlot%2FScottPlot%2Fpull%2F2479%23issuecomment-1463041169&data=05|01||a1cee6e181d4408d980508db21009847|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|638140058666887543|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=OfHG9%2BpwSBspphewwAoW9CMYLUFtTbVQbi2Ytd8aVsM%3D&reserved=0, or unsubscribehttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAQ3QF2UNUJNOPHGAFEHCB4TW3J2OPANCNFSM6AAAAAAVVYXA3M&data=05|01||a1cee6e181d4408d980508db21009847|84df9e7fe9f640afb435aaaaaaaaaaaa|1|0|638140058666887543|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=Nj1aLZXPgYzyM0LfGkafdhKtipe9hryZb3GnOGt%2BBpQ%3D&reserved=0.
You are receiving this because you were mentioned.Message ID: @.***>


Please export the ColorPickerProps for typescript

Trying wrap the color picker into another component, and need access to the type ColorPickerProps which is currently not exported.

@rishisharma08 thats easy enough, will push an update ASAP

@rishisharma08 please try v3.0.8 and lmk if that works


not turning blue

Challenge Use CSS Selectors to Style Elements has an issue.
User Agent is: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:53.0) Gecko/20100101 Firefox/53.0.
Please describe how to reproduce this issue, and include links to screenshots if possible.
My code:

Kitty ipsum dolor sit amet, shed everywhere shed everywhere stretching attack your ankles chase the red dot, hairball run catnip eat the grass sniff.

@shajilphilip
Thanks for reporting this. But your code is incorrect.
GitHub Issues are for reporting bugs on the website only.
Please use the Help chat room for getting challenge related help.
Happy Coding.


NearMenu 4x1 has 2 OnClick entry instead on just 1

Describe the bug
The prefab NearMenu 4x1 has 2 on click function for ButtonFour

To reproduce
Steps to reproduce the behavior:

Create a new project.
Add a new Near Menu 4x1 from toolbox.
From hierarchy window navigate to NearMenu4x1(Clone) > ButtonCollection > ButtonFour.
From the Inspector window, navigate to Button Config Helper > Basic Events.
Notice that it has two added entry instead of one.

Expected behavior
The On Click event should only have 1 initial entry.
Screenshots

Your setup (please complete the following information)

Unity Version 2020.3.37f1
MRTK Version v2.8.2

Target platform (please complete the following information)

HoloLens 2
OpenVR

We appreciate your feedback and thank you for reporting this issue.
Microsoft Mixed Reality Toolkit version 2 (MRTK2) is currently in limited support. This means that Microsoft is only fixing high priority security issues. Unfortunately, this issue does not meet the necessary priority and will be closed. If you strongly feel that this issue deserves more attention, please open a new issue and explain why it is important.
Microsoft recommends that all new HoloLens 2 Unity applications use MRTK3 instead of MRTK2.
Please note that MRTK3 was released in August 2023. It features an all new architecture for developing rich mixed reality experiences and has a minimum requirement of Unity 2021.3 LTS. For more information about MRTK3, please visithttps://www.mixedrealitytoolkit.org.
Thank you for your continued support of the Mixed Reality Toolkit!


Entry point module

@mostaphaRoudsari,
pollination-apps mandates that the entry point module's name has to be "app.py" else it raises an error in CLI. This needs to be relaxed. I have an app where I would like the entry point module to be named "Introduction" or anything I like for that matter.

For now, I just disabled this check locally. I am not sure what's the best alterative for a check for app.py. May be this check should not happen. Else, it will stop anyone from having another name for the entry point module.

I thought I could run Pollination-apps locally and just comment out this check. But when I deployed, I got the message on CLI that the app is successfully deployed to Pollination, but when I headed over to Pollination, there was no app.  I waited 10 mins and refreshed several time. The app wasn't there. I saw 502 in console. This tells me that somewhere other than pollination-apps, "app.py" is hardcoded as a name for the entry point module.

Hi, @devngc - the app.py is hard coded and is not an easy change. Can you use this library to get around the issue:
https://github.com/blackary/st_pages

@mostaphaRoudsari, I tried this solution and have to report that it is shaky for now. It does not always work. It would be best to think about removing the hard-coded "app.py"

Hi @devngc I am looking into to this now. Before I dive to deep I would need to better understand your use-case. Why does the file have to be renamed to introduction in this case?

Hi @AntoineDao - This is mainly because of how the multi-page Streamlit apps work. They use the name of each module for each page. A poor design decision if you ask me but I think they are selling it to keep it simple.
docs.streamlit.io/library/get-started/multipage-apps/create-a-multipage-app#convert-an-existing-app-into-a-multipage-app
Right now the name for the first page of the app will be set app. @devngc is trying to set it to a better name like introduction.

Right, thanks for the docs, that helps me understand the context of this request a bit better 😄

Hi @AntoineDao,
I tried to deploy a new version of the app using the latest version of pollination-apps. I had renamed the my entry point module to "Introduction.py". However, I was not able to deploy. I got the following error.
"Error: Application folder is missing a required file: app.py"

That's a check that I put in the CLI. You can either remove it or put a dummy app.py file to deploy your app and then change the settings from the browser

Thank you both. Closing the issue.


[grasshopper] Let procedures / post-condition refer to pre-condition footprint

Right now we existentially quantify the footprint separately for the pre/post. We might want to rearrange this so we quantify the pre-footprint in the procedure argument, because this would mean we could compare the pre/post footprint, and refer to it explicitly in procedure calls.

Would this at all help with problems relating to assignment?  Eg. being able to pure-assume that e is not in the pre-condition footprint.  Or is the problem here entirely due to the postcondition?


Tsserver tests can be baselined

Also ensures inferred and auto import projects have name per project service so they are same across any type of runs
This is the PR I had started as part of #41004 and is partially done but nothing wrong with merging this without converting many more tests to baselines. That can be done as and when needed.

Can i get review on this so i can baseline #41004 on top of main after merging this.


Error with stripe package

Using the https://esm.sh/stripe package gives the error,
error: TS2439 [ERROR]: Import or export declaration in an ambient module declaration cannot reference module through relative module name.
    export * from '/v33/<EMAIL_ADDRESS>    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    at https://cdn.esm.sh/v33/@types/node@14.14.37/http.d.ts:2:5

TS2439 [ERROR]: Import or export declaration in an ambient module declaration cannot reference module through relative module name.
    import * as stream from '/v33/node:stream';
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    at https://cdn.esm.sh/v33/@types/node@14.14.37/http.d.ts:6:5

TS2439 [ERROR]: Import or export declaration in an ambient module declaration cannot reference module through relative module name.
    import { URL } from '/v33/node:url';
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    at https://cdn.esm.sh/v33/@types/node@14.14.37/http.d.ts:7:5

TS2439 [ERROR]: Import or export declaration in an ambient module declaration cannot reference module through relative module name.
    import { Socket, Server as NetServer } from '/v33/node:net';
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    at https://cdn.esm.sh/v33/@types/node@14.14.37/http.d.ts:8:5

Trying with https://esm.sh/stripe?no-check gives,
error: Uncaught ReferenceError: require is not defined
    at https://cdn.esm.sh/v33/stripe@8.141.0/deno/stripe.js:4:12741
    at https://cdn.esm.sh/v33/stripe@8.141.0/deno/stripe.js:1:763
    at https://cdn.esm.sh/v33/stripe@8.141.0/deno/stripe.js:4:23152
    at https://cdn.esm.sh/v33/stripe@8.141.0/deno/stripe.js:1:763
    at https://cdn.esm.sh/v33/stripe@8.141.0/deno/stripe.js:4:29463
    at https://cdn.esm.sh/v33/stripe@8.141.0/deno/stripe.js:1:763
    at https://cdn.esm.sh/v33/stripe@8.141.0/deno/stripe.js:5:10580
    at https://cdn.esm.sh/v33/stripe@8.141.0/deno/stripe.js:1:763
    at https://cdn.esm.sh/v33/stripe@8.141.0/deno/stripe.js:5:15340
    at https://cdn.esm.sh/v33/stripe@8.141.0/deno/stripe.js:1:763

stripe is using the child_process that is not supported by deno currently

According to this issue and this pull request in the Stripe repository it seems that they are moving towards removing the dependency on child_process so it can be used in CloudFlare workers, which has API similar to Deno with regards to Stripe according to the maintainers. It seems the only thing left is Node http/https either being polyfilled or converted to use fetch() instead.
I am uneducated on the process of converting modules to Deno, but seeing as the Stripe package is important to my projects I am willing to help out with the porting if needed.

still happens on v47
Check file:///tmp/test.ts
error: TS2439 [ERROR]: Import or export declaration in an ambient module declaration cannot reference module through relative module name.
    import * as stream from '/v47/node:stream';
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    at https://cdn.esm.sh/v47/@types/node@16.7.10/http.d.ts:43:5

TS2439 [ERROR]: Import or export declaration in an ambient module declaration cannot reference module through relative module name.
    import { URL } from '/v47/node:url';
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    at https://cdn.esm.sh/v47/@types/node@16.7.10/http.d.ts:44:5

TS2439 [ERROR]: Import or export declaration in an ambient module declaration cannot reference module through relative module name.
    import { Socket, Server as NetServer } from '/v47/node:net';
    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    at https://cdn.esm.sh/v47/@types/node@16.7.10/http.d.ts:45:5

TS2439 [ERROR]: Import or export declaration in an ambient module declaration cannot reference module through relative module name.
    export * from '/v47/<EMAIL_ADDRESS>    ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    at https://cdn.esm.sh/v47/@types/node@16.7.10/http.d.ts:1357:5

https://esm.sh/stripe?no-check should work

I will check the types transformer

seems to work fine in v53
feel free to reopen if it persists


Support Deno Deploy

Title says it all.
Fixes #15
Closes #16

Actually we can just as well drop the interactivity. There is not much a point to it since --prompt.


onMarkerPress event is bubbled up to parent components onPress events

Hi! I'm using MapView inside one of the views that is inside TabBar. It looks something like this:
<TabBarIOS>
  <TabBarIOS.Item onPress={this.handleTabBarItemPress}>
    <View>
      <Mapview>
        <MapView.Marker></MapView.Marker>
      </Mapview>
    </View>
  </TabBarIOS.Item>
</TabBarIOS>

Now when pressing on MapView.Marker, handleTabBarItemPress method gets called, even without adding any event handlers to marker element itself. The event.nativeEvent.action has "marker-press" as value. Luckily, this is my current workaround how i can bypass handling this event handler :)
Also i'd like to take a chance and thank you guys for converting this repository into library. I seriously needed working map view for Android this week and it could not come at a better time than it is. I hope i get a chance to contribute to this repository as well. Cheers!

Hey @priithaamer,
This behavior is because the onPress event is a bubbling event by default in iOS, and will propogate up the render tree.
I agree though that this isn't exactly the behavior you're expecting... especially with a TabBar item...
In the near term, you can terminate the bubbling by calling event.stopPropogation().  We should consider removing this though.

Thanks! I haven't seen onPress being bubbled up elsewhere, e.g with TouchableOpacity, that's why i raised the question. If it is still the case, i think onPress should not be used as it may cause lot of confusion for others as well.
I believe this issue may be closed for now.

@priithaamer i agree. I'll dive deeper into what other RN components are doing and try to follow whatever is "idiomatic". Thanks for bringing the issue up!