Time-consuming to run RSeQC geneBody_coverage2.py on bigwig

Running geneBody_coverage2.py on bigwig files instead of the entire BAM as input requires much less memory but is still time-consuming (32hrs for 15M reads,1.1G BAM).

More details:

For human data, the .command.sh takes ~6hrs to finish but takes >50hrs to finish by running as a nextflow task.
Sometimes nextflow couldn't detect the exit of genebody_coverage:

all result/output files generated properly
no .exitcode generated
job failed due to time limit

Thanks, Jun for looking into this.
Maybe it's a good point to think about using something else, QualiMap can do genebody coverage, too:
http://kokonech.github.io/qualimap/kidney_rnaseqqc/qualimapReport.html#Coverage Profile Along Genes (Total)

Thank you for the suggestion. I will look at it.
I have no idea why nextflow took so long time to run genebody_coverage2.py.  Any ideas?

MultiQC also doesn't run with this output. That can be fixed, but I wonder if we should look into using a different tool to do this step (and others..?).

Parts could be replaced at least by QualiMap:
http://qualimap.bioinfo.cipf.es/doc_html/command_line.html#cmdline-rnaseqqc

Reverting would be a quick fix, but it feels dirty. The BigWig outputs are useful and I feel that we should be able to generate the gene body plots with this pretty easily. I'd quite like to have a proper look into this / profile the job and try to see what's going wrong. It's a really short script so I'm curious as to where the time is going.
Alternatively, we can indeed switch wholesale to another package. I'd prefer not to mix up QC packages too much as it can get pretty complicated and the software requirements are tougher.

I did quite a lot tests for this and feel like it might be a nextflow problem.
Actually, using bigwig file takes much less memory and shorter time (~6hrs, human) than using bam file (~25hrs).  But if ran it as a nextflow task, it took very long time (>50hrs), and sometimes nextflow couldn't detect the exit of the task. I looked at the trace file and saw rchar > 10TB, maybe the problem caused by this?

Aha! @jun-wan you are confirming my suspicions! Now we just need to figure out what's happening 😆

Can you be a bit more explicit what you mean with rchar > 10 TB? Then I could also have a look at my issues... happy to help here - we're quite affected by this unfortunately :-(

It's I/O counter: chars read, written in .command.trace and also summarised in nfcore-rnaseq_trace.txt

To narrow things down - same input dataset on same pipeline (1.0) release on same computing system binac: Still high rchar values of 3.5 TB for a single sample (human rnaseq).
I had a power outage today, so the run didn't finish. Running again with 0.30.1 to test whether its version specific.

Update: Nextflow 0.30.1 (double checked, same input script - same behaviour). didn't fail yet but
rchar<PHONE_NUMBER>995 , so around 2.7TB already.

Gah, why the hell is that process doing some much disk access? Can anyone see a reason for this in the code? What happens if we try running the job outside of nextflow - does it take a comparable amount of time?

I'll take one of the BAMs and run it on a workstation to test. can't really test it on the cluster atm...

I think I'm having the same problem: I'm running the pipeline on Uppmax/rackham, with singularity, using nextflow v 30.1.
geneBody_coverage2.py finished ok according to log file, and all results are there. Still it was cancelled due to time limit.
Looking at the trace file, the disc io is not so bad: rchar is 12.2 MB and vchar is 663.4 KB
.command.trace in the work directory says

CollectRnaSeqMetrics from picard can do this too if you set CHART=<OUTPUT_FILE>:
https://broadinstitute.github.io/picard/command-line-overview.html
It looks like picard is already part of the software requirements but as Phil mentioned it wont be as tidy...

CollectRnaSeqMetrics could be a nice solution actually if we can't figure this out soon... 👍

Yes, thats what I routinely use. You can also feed it ribosomal intervals to look for rRNA contamination (mainly in total RNA preps) - another metric that is useful to have but you need to create a *.list file containing the intervals from the gtf. The parameters like strandedness can also be set.

Hi there,
I took the time to read the script and performed some changes, being the main one replacing the python package bx.bbi for pyBigWig for reading the BigWig file. This resulted in a 21X performance increase on the script.
I've sent the script to its creator to see if he can make the changes permanent in a new release.
In the meantime, I'm attaching the script geneBody_coverage3.txt to this post (the extension txt should be changed back to py). If you add it to the bin folder of the pipeline and add the conda package pybigwig=0.3.12 to the environment.yml and to the Docker image, we should be able to fix this issue. Or at least it will give us enough time to properly test alternatives and think if we want to change.
I've also noticed that MultiQC is not producing several of the RSeQC plots, like:

bam_stat
gene_body_coverage: for this one I've noticed that the folder where files are being stored (geneBodyCoverage ) is different than the one expected by MultiQC (gene_body_coverage); also, when publishing the files to the results folder, some of the rules have incorrect file names;
inner_distance
junction_annotation
read_gc

On other order of things, I've seen the rchar issue (it being GB/TB big) in other processes. Maybe it could be an overhead created by Nextflow or other intermediary (I'm using Singularity)?
Please, give it a try and let me know if it works for you.
Cheers,
Santiago

gene_body_coverage: for this one I've noticed that the folder where files are being stored (geneBodyCoverage) is different than the one expected by MultiQC (gene_body_coverage); also, when publishing the files to the results folder, some of the rules have incorrect file names;

Sorry, regarding this point, "folder name" is unrelated to why this metric is not being produced. This was part of my ignorance regarding how MultiQC works.
However, there are a couple of errors in the module, but I will open a ticket in MultiQC for that.
@ewels, do you have a test to check if all the plots for the tools in the pipeline are being properly generated (or generated at all)?

@santiagorevale - no pipeline tests like this yet, no. It could be nice to add. We've thought about validating outputs, perhaps just grepping the MultiQC log output would be an easy start on that road..

Hello everyone! Does anyone know if this is still an issue in rnaseq v1.1 with nextflow>=0.32.0?

Probably still an issue yes, hoping to get to it very soon.

@apeltzer has now updated the dev branch to use RSeQC 3.0 🎉
Would be great if anyone gets a chance to try running the pipeline with some big inputs to see if there's an improvement.
Thanks all!
Phil

grep -e "genebody*" execution_trace.txt
165     29/ee512c       88280   genebody_coverage (QPB05024AJ__AS184983LR28503AlignedByCoord.out)       COMPLETED       0       2019-03-19 13:57:29.587 3h 28m 51s      1h 37m 51s      101,5%  5042 MB 7301 MB 4846 GB 22 MB
166     ac/a264b6       88281   genebody_coverage (QPB05022A3__AS184981LR28503AlignedByCoord.out)       COMPLETED       0       2019-03-19 13:57:29.612 3h 29m 41s      1h 38m 31s      101,6%  504 MB  7304 MB 4974 GB 23 MB
164     d1/47f623       88282   genebody_coverage (QRDGM008ALAS225023LR34148AlignedByCoord.out) COMPLETED       0       2019-03-19 13:57:29.633 4h 6m 16s       1h 34m 49s      101,4%  5037 MB 7292 MB 4806 GB 22 MB
163     1c/2ae0ad       88283   genebody_coverage (QRDGM011A6AS225026LR34148AlignedByCoord.out) COMPLETED       0       2019-03-19 13:57:29.668 4h 6m 46s       1h 35m 12s      100,8%  5038 MB 7296 MB 4799 GB 22 MB
167     30/0767d9       88285   genebody_coverage (QPB05018AA__AS184977LR28503AlignedByCoord.out)       COMPLETED       0       2019-03-19 13:57:29.713 4h 46m 36s      1h 38m 57s      100,6%  5041 MB 7288 MB 4842 GB 22 MB
168     6f/0ef199       88286   genebody_coverage (QRDGM010AWAS225025LR34148AlignedByCoord.out) COMPLETED       0       2019-03-19 13:57:29.736 5h 1s   1h 31m 9s       101,2%  5033 MB 7288 MB 4681 GB 21 MB
169     86/74b6df       88287   genebody_coverage (QRDGM009ATAS225024LR34148AlignedByCoord.out) COMPLETED       0       2019-03-19 13:57:29.757 5h 6m 16s       1h 36m 33s      101,7%  5041 MB 7302 MB 4884 GB 22 MB
170     a5/9a0053       88288   genebody_coverage (QPB05013A4__AS184972LR28503AlignedByCoord.out)       COMPLETED       0       2019-03-19 13:57:29.777 5h 39m 1s       1h 32m 47s      101,4%  504 MB  7294 MB 4766 GB 22 MB
171     6d/c0a079       88289   genebody_coverage (QPB05017A2__AS184976LR28503AlignedByCoord.out)       COMPLETED       0       2019-03-19 13:57:29.797 5h 42m 16s      1h 35m 24s      100,9%  5036 MB 7325 MB 4849 GB 22 MB
173     50/1d6001       88290   genebody_coverage (QPB05023AB__AS184982LR28503AlignedByCoord.out)       COMPLETED       0       2019-03-19 13:57:29.823 5h 53m 36s      1h 38m 9s       101,1%  5041 MB 7294 MB 4914 GB 23 MB
172     8d/274122       88291   genebody_coverage (QPB05014AC__AS184973LR28503AlignedByCoord.out)       COMPLETED       0       2019-03-19 13:57:29.856 6h 27m 21s      1h 40m 45s      100,7%  504 MB  7297 MB 4843 GB 22 MB
175     9c/b0217d       88292   genebody_coverage (QPB05015AK__AS184974LR28503AlignedByCoord.out)       COMPLETED       0       2019-03-19 13:57:29.886 6h 35m 51s      1h 35m 47s      100,8%  504 MB  7298 MB 4764 GB 22 MB
176     8e/00bae9       88293   genebody_coverage (QRDGM012AEAS225027LR34148AlignedByCoord.out) COMPLETED       0       2019-03-19 13:57:29.914 6h 40m 26s      1h 34m 6s       100,9%  5039 MB 7295 MB 4833 GB 22 MB
174     77/65462b       88294   genebody_coverage (QRDGM007ADAS225022LR34148AlignedByCoord.out) COMPLETED       0       2019-03-19 13:57:29.943 7h 12m 21s      1h 33m 16s      100,7%  5037 MB 7297 MB 4719 GB 22 MB
178     83/9091e1       88295   genebody_coverage (QPB05016AS__AS184975LR28503AlignedByCoord.out)       COMPLETED       0       2019-03-19 13:57:29.973 7h 20m 51s      1h 38m 33s      100,8%  5042 MB 7295 MB 4961 GB 23 MB
180     c2/d3f413       88296   genebody_coverage (QPB05020AL__AS184979LR28503AlignedByCoord.out)       COMPLETED       0       2019-03-19 13:57:30.002 7h 38m 36s      1h 44m 54s      100,7%  5047 MB 7307 MB 5119 GB 24 MB
177     90/41d727       88297   genebody_coverage (QPB05019AI__AS184978LR28503AlignedByCoord.out)       COMPLETED       0       2019-03-19 13:57:30.031 8h 8m 7s        1h 40m 41s      101,4%  5039 MB 7297 MB 4741 GB 22 MB
179     a3/686b0c       88298   genebody_coverage (QPB05021AT__AS184980LR28503AlignedByCoord.out)       COMPLETED       0       2019-03-19 13:57:30.059 8h 17m 47s      1h 41m 52s      100,7%  5047 MB 7302 MB 5144 GB 24 MB

These were some mouse samples that I just ran for integration testing of the dev branch these days. Can't really say whether this is much faster than before unfortunately.

Human Data:
79      18/f4cc73       88447   genebody_coverage (I16R003f01_01_S5_L003_R1_001AlignedByCoord.out)      COMPLETED       0       2019-03-22 16:41:30.167     2h 14m 41s      2h 1m 14s       100,6%  9395 MB 11 GB   9504 GB 25 MB
76      3f/6c2192       88444   genebody_coverage (I16R003f02_01_S6_L002_R1_001AlignedByCoord.out)      COMPLETED       0       2019-03-22 16:41:30.031     2h 27m 2s       2h 26m 58s      101,6%  9393 MB 11 GB   9321 GB 24 MB
77      47/63af20       88443   genebody_coverage (I16R003f02_01_S6_L004_R1_001AlignedByCoord.out)      COMPLETED       0       2019-03-22 16:41:29.988     2h 28m 47s      2h 28m 36s      101,5%  9393 MB 11 GB   9348 GB 24 MB
80      ab/1bee49       88446   genebody_coverage (I16R003f01_01_S5_L002_R1_001AlignedByCoord.out)      COMPLETED       0       2019-03-22 16:41:30.134     3h 28m 38s      3h 28m 31s      100,7%  9397 MB 11 GB   9506 GB 25 MB
78      32/a4c8ba       88449   genebody_coverage (I16R003f02_01_S6_L003_R1_001AlignedByCoord.out)      COMPLETED       0       2019-03-22 16:41:30.227     3h 42m 33s      2h 27m 13s      100,5%  9395 MB 11 GB   9278 GB 24 MB
82      75/c10a0e       88450   genebody_coverage (I16R003f01_01_S5_L001_R1_001AlignedByCoord.out)      COMPLETED       0       2019-03-22 16:41:30.258     4h 17m 13s      2h 2m 31s       101,6%  9397 MB 11 GB   9494 GB 25 MB
75      76/a10aa6       88445   genebody_coverage (I16R003f02_01_S6_L001_R1_001AlignedByCoord.out)      COMPLETED       0       2019-03-22 16:41:30.078     4h 20m 23s      4h 20m 17s      100,3%  9395 MB 11 GB   9272 GB 24 MB
81      85/b42b38       88451   genebody_coverage (I16R003f01_01_S5_L004_R1_001AlignedByCoord.out)      COMPLETED       0       2019-03-22 16:41:30.290     4h 29m 58s      2h 2m 55s       100,9%  940 MB  11 GB   9445 GB 25 MB

I think its fine now - we should keep an eye on it maybe still...


Priorities and nested subtasks

Hi, first of all cool project. Thought of writing a calcurse clone myself, but yours seems pretty good already, at least from a ui standpoint.
For tasks, wouldn't it make sense to at least have a priority integer that goes from like 0-9 by default, or a mask that says sth like low medium high?
Also, I can't seem to create subtasks for a subtask.
Maybe tags are a good idea?
How many features are necessary for keeping a good todo list are highly subjective, but maybe listing the features you would consider useful in a seperate segment of the readme could help clarify things.
Your tool is too simple for me, while taskwarrior is overly complex.
I may just write my own task management tool anyway, but please write up your desired scope / workflow.

Hi, thank you for the feedback!
From a philosophical point of view, one of my goals with this app was to achieve simpler and faster operation than in calcure. That is why some functions, like priority, are reduced to just two states: normal/important. But indeed, I think I could add one more state, like low priority, so it would be a choice between low/normal/high priority, which should cover most needs.
Similarly, about the subtasks, I thought it would be better to sacrifice sub-sub-tasks for the sake of simple toggling task/sub-task state. Personally, I never even use subtasks, but I imagine some people might need it. So, how many levels of nested subtasks would you realistically use?

Personally, I rarely even use sub-sub tasks myself, but I think they are about as deep as is practical and I have used them a few times in the past.
For priorities, I really like the "unset/low/normal/high" paradigm used by tasks, the android app I used for task management for several months.
For simplicity, low / normal / high is enough with normal being a default priority

Just added these two features in the last commit. Will appear in the next release

Third level of subtasks. Now subtask added to a subtask will create a subsubtask. However, toggling with s will toggle only first level - kind of a reasonable compromise.
Low priority tasks with l. So now it can be normal/important/unimportant/done.

Just added these two features in the last commit (already in repositories).

Third level of subtasks. Now subtask added to a subtask will create a subsubtask. However, toggling with s will toggle only first level - kind of a reasonable compromise.
Low priority tasks with l. So now it can be normal/important/unimportant/done.

Greatly appreciate it, I consider this issue resolved :)


Fix small spelling mistake in the documentation

What does this PR do?
It fixes a small spelling mistake in the cookbook about "Compute new values"
It should be table instead of tables

Thanks, Mila!


@searchable with @auth - query returns items.size() instead of total hits

Describe the bug
Note - this only happens for models with the @auth directive, the models without @auth works fine.
In the Query.searchX.res.vtl resolver that the CLI generates the end looks like this -
#set( $es_response = {
  "items": $es_items
} )
#if( $es_items.size() > 0 )
  $util.qr($es_response.put("nextToken", $nextToken))
  $util.qr($es_response.put("total", $es_items.size()))
#end
$util.toJson($es_response)

but it should be this -
#set( $es_response = {
  "items": $es_items
} )
#if( $es_items.size() > 0 )
  $util.qr($es_response.put("nextToken", $nextToken))
  $util.qr($es_response.put("total", $ctx.result.hits.total))
#end
$util.toJson($es_response)

because it now currently returns the length of the array of the items, which is incorrect.
Amplify CLI Version
4.27.2
To Reproduce
add @model,
add @searchable to model,
add @auth to model.
add some items to DB (for example 3 items),
perform search query with limit set to 2,
the total will be equal to 2 even though there are 3 total results
Expected behavior
the total field is equal to the total number of documents that fit the filter condition

The PR which added this change has a comment for why it is done this way. The elasticsearch result is filtered by auth rules. You don't want to leak information about the total size of the response if a user is only supposed to see a partial list of the response.
If your use case is not concerned about this data leakage, you can customise the template to remove these lines. But Amplify should operate as secure-by-default
https://github.com/aws-amplify/amplify-cli/pull/2602#issuecomment-543844821

Hi this should now be addressed with Transformer v2. https://docs.amplify.aws/cli/graphql/search-and-result-aggregations/#aggregate-values-for-search-result-minimum-maximum-average-sum-terms We also added support for other aggregation queries. The auth rules are now applied at query-time instead of result-post-processing time.


Using NLog for ASP.NET Core to write custom information to the database

In the previous versions of NLog it was easily possible to map custom log properties to custom database columns using LogEventInfo and ${event-properties} layout renderer:
LogEventInfo evt = new LogEventInfo(logLevel, "", message);
evt.Properties["custom"] = "some data";
log.Log(evt);

I cannot make this work using ASP.NET Core and NLog.Extensions.Logging.NLogLogger.
I am using standard ASP.NET Core logging infrastructure and NLog.Extensions.Logging package. At the runtime, my Microsoft.Extensions.Logging.ILogger contains additional NLog.Extensions.Logging.NLogLogger instance, configured to write all the events to the database. I want to pass some dictionary-like object to Microsoft.Extensions.Logging.ILogger.Log() to make this information available to NLog layout renderers. Is it supported?

Hi @dr-noise
May be you can use the state parameter of Log method to send the dictionary but you should have to write your own ILogger implementation to write it in NLog logger.
Actually we can't handle this scenario because the ILogger interface is part of .NET Core and we can't extend the method signature

Thanks @phenixdotnet
I'll have deeper look into this, I thought that state parameter already supported by the NLog.Extensions.Logging.NLogLogger.
For example, NLogLogger can check if state is instance of Dictionary<string, string> and push all data from there to the underlying LogEventInfo.

For example, NLogLogger can check if state is instance of Dictionary<string, string> (or Dictionary<object, object>) and push all data from there to the underlying LogEventInfo.Properties.

It will fit your need but it seem hard to generalize this. That why I suggest you use your own implementation. We can work on push some extension point on the actual class but I don't think its a good idea to implement this idea directly in NLog lib

I agree on this @phenixdotnet — it's not a good idea to enforce API users to be aware of this implementation detail.

I think we should do something about this is the near future.  The aspnet logging is prepared for structural logging, but NLog not.
Related http://nlog.userecho.com/topics/21-make-structural-logging-possible/

I think it's a good idea to allow programmatically overwrite the current behaviour.  (by DI into nloglogger)

@304NotModified Agree.
We can define an interface to allow injection or add a secondary constructor which allow passing an Action.
Something like that
public NlogLogger(Action<LogEventInfo, object> stateProcessor)

I think DI is the new way, but I have to agree I don't know how to set-up the default DI binding (so that we user only have to set the DI binding if he needs to overrule the default).
I was thinking of an interface like this
interface INLogConvertToLogEventInfo
{
 LogEventInfo convert(Microsoft.Extensions.Logging.LogLevel logLevel, int eventId, object state, Exception exception);
}

I think we can fix this with some c# extensions.  Will try to look at this when we update the NLog.Extensions.Logging package for rc2
(structural logging is also still on the road)

Did we figure out this or is there any example which i can look onto?

Added documentation about how to include NLog properties:
https://github.com/NLog/NLog.Extensions.Logging/wiki/NLog-properties-with-Microsoft-Extension-Logging


Rocket creating/reading the folder outside the server folder

Since last halloween update, Rocket is creating/reading the files on a directory outside the server folder:
servers
SERVER 1
SERVER 2
Rocket\
...instead of...
servers
SERVER 1
Rocket
SERVER 2
Rocket\

Just have been trying it and couldn't seem to replicate. Whats the instance name you tried? Please share the full command line u started the server with.. also check an instance name without spaces.

I am sorry, seems to be fixed already, I just downloaded the version before the Halloween patch and for some reason was having that behaviour. You can close this issue.


What is the results of XLSR-53 model?

Thank you for your valuable sharing. Recently, I saw that you published the XLSR-53 model. What is the result of it on babel, common voice, and MLS?

Besides, can you share the config for running XLSR-53 (finetuning and pretraining)?

@iamxiaoyubei Hey sorry I missed this. You can find the results of the XLSR-53 model on MLS, CommonVoice and BABEL in our update XLSR paper here: https://arxiv.org/abs/2006.13979
The pretraining is similar to the pretraining of the Librivox model (same setting, with lower learning rate). Except the dataset is preprocessed in such a way that it oversamples lower-resource languages, just like in XLM, and as described in the XLSR paper.
For finetuning, the config are the same than for wav2vec 2.0 Librivox model: so you can find scripts for finetuning on 1 hour, 10 hours, 100 hours and 960 hours. Depending on the size of your dataset, you can choose to pick the right finetuning script.

@iamxiaoyubei Hey sorry I missed this. You can find the results of the XLSR-53 model on MLS, CommonVoice and BABEL in our update XLSR paper here: https://arxiv.org/abs/2006.13979
The pretraining is similar to the pretraining of the Librivox model (same setting, with lower learning rate). Except the dataset is preprocessed in such a way that it oversamples lower-resource languages, just like in XLM, and as described in the XLSR paper.
For finetuning, the config are the same than for wav2vec 2.0 Librivox model: so you can find scripts for finetuning on 1 hour, 10 hours, 100 hours and 960 hours. Depending on the size of your dataset, you can choose to pick the right finetuning script.

For pretraining, the code remains the same, the only thing that changes is in the ".tsv" file that you need to prepare.
And basically, in there, you simply add the path to the .wav (for instance) files of all your languages. So you have a single .tsv file for the 53 languages. But you "upsample" the lower-resource ones, by having multiple times low-resource language files for instance. This is a bit hacky but that means you don't have to make any change in the code, you simply have to modify the preprocessing.
For fine-tuning, you can download the MLS data and CommonVoice data online (BABEL you have to sign something to get it). Then you can do the preprocessing exactly how it's done for English: each character is space-separated and words are separated by a "|" (that creates your output .wrd file for instance).
For training language models for all those languages, I would suggest this data we recently released called "CC100": http://data.statmt.org/cc-100/. Not perfect because you need to filter it to fit the domain of (e.g.) spoken language, but still a good resource.

For pretraining, the code remains the same, the only thing that changes is in the ".tsv" file that you need to prepare.
And basically, in there, you simply add the path to the .wav (for instance) files of all your languages. So you have a single .tsv file for the 53 languages. But you "upsample" the lower-resource ones, by having multiple times low-resource language files for instance. This is a bit hacky but that means you don't have to make any change in the code, you simply have to modify the preprocessing.
For fine-tuning, you can download the MLS data and CommonVoice data online (BABEL you have to sign something to get it). Then you can do the preprocessing exactly how it's done for English: each character is space-separated and words are separated by a "|" (that creates your output .wrd file for instance).
For training language models for all those languages, I would suggest this data we recently released called "CC100": http://data.statmt.org/cc-100/. Not perfect because you need to filter it to fit the domain of (e.g.) spoken language, but still a good resource.

@aconneau Thank you~!


Frontend not updating the list of organizations

The frontend is not showing/updating the list of organizations when doing a second entry into the Create New course page. Also it doesn't update when switching between the gitlab vs github toggle buttons. Not totally sure about the exact behaviors that cause this, but it looks pretty easy to reproduce. Again I'm in Chrome on macOS.

This seems to have been caused by the ssh tunnel:
packet_write_wait: Connection to <IP_ADDRESS> port 22: Broken pipe
packet_write_wait: Connection to UNKNOWN port 65535: Broken pipe

But I suppose the frontend should indicate an error message, if it can get a response from the server. Hence I let this remain as a bug.


path cant be a boolean

How path can be String if savePhotoInMemoryDevice return boolean?
String path = magicalCamera.savePhotoInMemoryDevice(magicalCamera.getMyPhoto(),"myPhotoName","myDirectoryName", MagicalCamera.JPEG, true)
public boolean savePhotoInMemoryDevice

This method is telling whether the photo is saved in memory device perfectly or not. To get path you need to use getRealPath ().

Hello, in the next version of magical camera you can obtain the path, the
boolean function now return the path, but I dont know when I upload this
version, because I dont have an Android 6.0 and I have problems with
genymotion....
please be patience ;)
2016-11-23 5:07 GMT-06:00 Prateek Vijay<EMAIL_ADDRESS>
This method is telling whether the photo is saved in memory device
perfectly or not. To get path you need to use getRealPath ().
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/fabian7593/MagicalCamera/issues/12#issuecomment-262486260,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AMua5drVFnJYemh8Ijv1lrh-yMY_UGLKks5rBB6EgaJpZM4K6aQQ
.

--
Fabián Rosales Esquivel
*Analyst and Software Developer *
Frosquivel Developer
If you can dream, it you can do it.
Desamparados, San José, Costa Rica
Mobile: (506) 8346 1166
*GitHub: *https://github.com/fabian7593 https://github.com/fabian7593

<EMAIL_ADDRESS>[image: skype]fabianre7593

Hello @ShBogdan  and @prateekvijay  I upload the new version of magical camera 4.8, and this issue is completelly solved, you can the posibillity of obtain the path string if the photo is save in memory device, and if not save the string is equals to empty...
Please Im sorry for the incovenience and thanks for report the issue and for use Magical Camera.


when the commitLog is overlarge，the broker startup very very very slow，eg. 10 minutes

when the commitLog is overlarge，the broker startup very very very slow，eg. 10 minutes 。
spend 10 minutes on load commitLog

has other ways to solve this problem except reduce the message accumulation？

@iNanos what's your version of rocketmq, and this restart rocketmq is master or slave? could you provide more information about this issue?


Creating Individual Contribution Report - Bahadir

Write the individual contribution report for Bahadir.
Add the following topics:

Member: Info about myself (name, group)
Responsibilities: Overall description of the responsibilities that are assigned to me
Main contributions: Overall description of my contributions to the project until Milestone. Provide the links to your work and related issues in your project repository.
Management related significant issues: My issues that contribute to the
management of our software project (Please only lists the most-significant ones, we DO track all your issues separately.)

Creating related issues for use in the contribution report.

Finished the general info for the report. Commit link.

Added some links and issue redirections to the report. Commit link

Done.


Allow Entity deaths to be Cancellable

SpongeAPI | SpongeCommon | SpongeForge
This is mostly to bring in compatibility of entity deaths to match Forge's requirements, while this is somewhat of an implementation leak that it's "possible", it's just not recommended as the health status of the entity could have adverse effects.

Are we going to add the behavior on cancel to the JavaDocs?

Spoke to @Zidane and @kashike and some forge members about the change:
It's currently unimplemented in Forge to handle the cancel of the entity death with the following logic: Modders who decide to cancel the entity death should "undeadify" the entity dying. It would otherwise not be the fault of the implementation if the entity does indeed die. What this will mean is that if the event is canceled, drops will still not occur, but every tick thereafter will still perform the death animation and process. What can happen however, is that the event is then rethrown during death ticks, at which point, if those are cancelled, death doesn't happen.
What @kashike and @Zidane did agree on however is that if the event is thrown 3 times for the same entity, and they're cancelled but the entity is not "undeaded", the entity will ultimately be allowed to die.
I'm comfortable with handling the implementation in Common to support these changes, but I'd like to get some more @SpongePowered/developers input on the idea since the API would react a little differently in this case compared to other events.


terraform plan issue on recently clone repo

What did you expect to happen?
I expected to see the terraform execution plan and see what ressources will be deployed
What actually happened?
I got an error saying that some arguments are missing
Output
terraform plan --var "billing_account=$BILLING_ACCOUNT" --var "org_id=$ORG_ID"

Error: Missing required argument

  on gcp.tf line 278, in resource "google_container_cluster" "vault":
 278:     workload_metadata_config {

The argument "mode" is required, but no definition was found.

Error: Unsupported argument

  on gcp.tf line 279, in resource "google_container_cluster" "vault":
 279:       node_metadata = "SECURE"

An argument named "node_metadata" is not expected here.

Error: Unsupported argument

  on gcp.tf line 293, in resource "google_container_cluster" "vault":
 293:     username = ""

An argument named "username" is not expected here.

Error: Unsupported argument

  on gcp.tf line 294, in resource "google_container_cluster" "vault":
 294:     password = ""

An argument named "password" is not expected here.

Additional context
I just clone the repo and set the required variables: billing_account and org_id
terraform --version
Terraform v0.12.31
+ provider.google v3.90.1
+ provider.google-beta v4.5.0
+ provider.kubernetes v2.7.1
+ provider.random v3.1.0
+ provider.tls v3.1.0

@pdomineaux hello, did you manage to bypass those issues?

@pdomineaux hello, did you manage to bypass those issues?

Nope, I still don't have a solution to this problem. Any clue?


how to get driving video

Hi, if I have a random video from net, how to generate a driving video which only a face inside it?

I believe that when you run the code, there is a script called "crop-video.py" that already tracks the face but the face-alligment library is needed

thanks, just notice this script added.


hitSlop is not filtered out when Platform.OS is web

Current Behavior
Tamagui doesn't filter out the RN-only prop "hitSlop" on web, resulting in the console error:
Warning: React does not recognize the `hitSlop` prop on a DOM element. If you intentionally want it to appear in the DOM as a custom attribute, spell it as lowercase `hitslop` instead. If you accidentally passed it from a parent component, remove it from the DOM element.

I've confirmed this on both Button and RadioGroup.Item
Expected Behavior
Tamagui only applies a provided hitSlop prop on native devices.
Tamagui Version
1.84.2

Platform (Web, iOS, Android)
Web

Reproduction
https://github.com/kpcurai/super-duper-octo-engine/commit/2b6bfdf04398d8e247d2bfc2b4f2163fad431b9e

System Info
System:
    OS: macOS 13.6.1
    CPU: (8) arm64 Apple M1 Pro
    Memory: 106.11 MB / 16.00 GB
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 21.6.0 - /opt/homebrew/bin/node
    Yarn: 4.0.2 - /opt/homebrew/bin/yarn
    npm: 10.2.4 - /opt/homebrew/bin/npm
    Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
  Browsers:
    Chrome: 120.0.6099.234
    Safari: 17.1
  npmPackages:
    @babel/runtime: ^7.18.9 => 7.21.0
    @manypkg/cli: ^0.19.1 => 0.19.2
    check-dependency-version-consistency: ^3.0.3 => 3.0.3
    eslint: ^8.21.0 => 8.39.0
    node-gyp: ^9.3.1 => 9.3.1
    prettier: ^2.7.1 => 2.8.8
    turbo: ^1.10.3 => 1.10.3
    typescript: ^5.3.3 => 5.3.3

Fixed in recent versions.

I observe this still in v1.88.13

Should be fixed in 1.88.17 lmk


fix: hilbish may segfault on startup

i do not know the exact reason, but sometimes on startup hilbish will segfault for some reason. since this only happens when i open up my terminal it quickly flashes away so that doesnt allow me to fully read the stack trace. what is known is that it is related to bait (i think)

ok turns out it's because tym (my terminal) with a daemon resizes right on startup, which would then cause hilbish to try and emit a signal.resize hook, but it segfaults because bait doesnt get initialized yet


zdf.de frequently gets audiodescription track instead of plain german

Checklist

[x] I'm reporting a broken site support
[x] I've verified that I'm running youtube-dl version 2020.02.16
[x] I've checked that all provided URLs are alive and playable in a browser
[x] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[x] I've searched the bugtracker for similar issues including closed ones

Verbose log
$ youtube-dl --verbose "https://www.zdf.de/serien/arctic-circle-der-unsichtbare-tod/arctic-circle---der-unsichtbare-tod-1-100.html"
[debug] System config: []
[debug] User config: []
[debug] Custom config: []
[debug] Command-line args: [u'--verbose', u'https://www.zdf.de/serien/arctic-circle-der-unsichtbare-tod/arctic-circle---der-unsichtbare-tod-1-100.html']
[debug] Encodings: locale UTF-8, fs UTF-8, out UTF-8, pref UTF-8
[debug] youtube-dl version 2020.02.16
[debug] Python version 2.7.17 (CPython) - Linux-5.4.0-3-amd64-x86_64-with-debian-bullseye-sid
[debug] exe versions: ffmpeg 4.1.4, ffprobe 4.1.4, phantomjs 2.1.1, rtmpdump 2.4
[debug] Proxy map: {}
[debug] Using fake IP <IP_ADDRESS> (DE) as X-Forwarded-For.
[ZDF] arctic-circle---der-unsichtbare-tod-1-100: Downloading webpage
[ZDF] arctic-circle---der-unsichtbare-tod-1-100: Downloading JSON content
[ZDF] arctic-circle---der-unsichtbare-tod-1-100: Downloading JSON metadata
[ZDF] arctic-circle---der-unsichtbare-tod-1-100: Downloading m3u8 information
[ZDF] arctic-circle---der-unsichtbare-tod-1-100: Downloading m3u8 information
[ZDF] arctic-circle---der-unsichtbare-tod-1-100: Downloading m3u8 information
[ZDF] arctic-circle---der-unsichtbare-tod-1-100: Downloading m3u8 information
[ZDF] arctic-circle---der-unsichtbare-tod-1-100: Downloading f4m manifest
[ZDF] arctic-circle---der-unsichtbare-tod-1-100: Downloading f4m manifest
[debug] Default format spec: bestvideo+bestaudio/best
[debug] Invoking downloader on u'https://zdfvodde-vh.akamaihd.net/i/zdf/20/02/200216_2215_sendung_arc/5/200216_2215_sendung_arc_a,3a4_476k_p9,3a4_776k_p11,1a2_776k_p11,3a4_1496k_p13,3a4_2296k_p14,3a4_3296k_p15,v14.mp4.csmil/index_5_av.m3u8?null=0'
[hlsnative] Downloading m3u8 manifest
[hlsnative] Total fragments: 275
[download] Destination: Arctic Circle - Der unsichtbare Tod (1_10)-arctic-circle---der-unsichtbare-tod-1-100.fhls-2623-1.mp4
[download] 100% of 900.79MiB in 08:23
[debug] Invoking downloader on u'https://zdfvodde-vh.akamaihd.net/i/zdf/20/02/200216_2215_sendung_arc/5/200216_2215_sendung_arc_a,3a4_476k_p9,3a4_776k_p11,1a2_776k_p11,3a4_1496k_p13,v14.mp4.csmil/index_0_a.m3u8?null=0'
[hlsnative] Downloading m3u8 manifest
[hlsnative] Total fragments: 275
[download] Destination: Arctic Circle - Der unsichtbare Tod (1_10)-arctic-circle---der-unsichtbare-tod-1-100.fhls-95-3.mp4
[download] 100% of 37.32MiB in 03:07
[ffmpeg] Merging formats into "Arctic Circle - Der unsichtbare Tod (1_10)-arctic-circle---der-unsichtbare-tod-1-100.mp4"
[debug] ffmpeg command line: ffmpeg -y -loglevel 'repeat+info' -i 'file:Arctic Circle - Der unsichtbare Tod (1_10)-arctic-circle---der-unsichtbare-tod-1-100.fhls-2623-1.mp4' -i 'file:Arctic Circle - Der unsichtbare Tod (1_10)-arctic-circle---der-unsichtbare-tod-1-100.fhls-95-3.mp4' -c copy -map '0:v:0' -map '1:a:0' 'file:Arctic Circle - Der unsichtbare Tod (1_10)-arctic-circle---der-unsichtbare-tod-1-100.temp.mp4'
Deleting original file Arctic Circle - Der unsichtbare Tod (1_10)-arctic-circle---der-unsichtbare-tod-1-100.fhls-2623-1.mp4 (pass -k to keep)
Deleting original file Arctic Circle - Der unsichtbare Tod (1_10)-arctic-circle---der-unsichtbare-tod-1-100.fhls-95-3.mp4 (pass -k to keep)

$

Description
There repeatedly are videos on zdf.de which get downloaded with audiodescription track instead of the default german track without audiodescription. The example plays the default german track when I open the page with torbrowser (I'm geoblocked). youtube-dl downloads with the audiodescription track. It makes no difference if use it's built-in geoblock circumvention or torsocks for the download.
The behavior is reproducible for each individual video, but does not happen on every video available. In the example above, the audiodescription version is easily recognizable because a voice describes already the yellow logo during the first seconds of the video.
youtube-dl -F … lists several mp4 audio formats, I assume one with "Audiodeskription" gets used instead of "TV_Ton":
format code                              extension  resolution note
hls-aac-Audiodeskription-0               mp4        audio only [deu]
hls-aac-Audiodeskription-1               mp4        audio only [deu]
hls-aac-Audiodeskription-2               mp4        audio only [deu]
hls-aac-Audiodeskription-3               mp4        audio only [deu]
hls-aac-TV_Ton-0                         mp4        audio only [deu]
hls-aac-TV_Ton-1                         mp4        audio only [deu]
hls-aac-TV_Ton-2                         mp4        audio only [deu]
hls-aac-TV_Ton-3                         mp4        audio only [deu]

It seems mediathekview had a similar problem, with some discussion on https://forum.mediathekview.de/topic/2444/reich-oder-tot/22 (in german) and the bugfix on https://github.com/mediathekview/MServer/pull/504/files but I don't know if the cause for the issue is the same.

just use youtube-dl -f videocode+hls-aac-TV_Ton-0 link etc.
where videocode of course is video track you want to dl

@kuco28 that's what I do when I discover that I downloaded with the wrong audio track. It just frequently happens that I download a series part by part and start watching when I have all episodes of a season, then discover that I downloaded the first episodes with the wrong audio track, and those are not available any more. I see no reason why the default audio track should be the one that nearly nobody uses, especially given the fact that this is completely different behavior to what the zdf.de web interface as well as other tools such as MediathekView provide. I also frequently recommend the tool to people that are not too familiar with command-line usage, but manage to type in that one single youtube-dl command with an URL appended, but don't know how to find out the details about audio and video formats. It should work for them as well.

@kuco28 [...] I also frequently recommend the tool to people that are not too familiar with command-line usage, but manage to type in that one single youtube-dl command with an URL appended, but don't know how to find out the details about audio and video formats. It should work for them as well.

Interesting. Like to know from where they get those dl-links??
Well, I'd recommend a service like "vavideo.de" for those who are not keen to fiddle with a command-line tool like yt-dl. It provides "man-in-the-middle" access to the media portals of German public TV broadcasters like ARD/ZDF/BR/HR/SWR/NDR/.../3SAT/ARTE/... .
Once you've found a video you've searched for, simply use the button "Direktlinks" to decide what video quality you prefer and a) select it to see the video (and its URL) immediately or b) using the right-click "Save as ..."-option ...
PS.  Live-Channels ermitteln kann (zur Verwendung in einem VCR, welchen ich mit Autohotkey gescripted habe)??


Fix the firewalld_version fact

Work around a bug in firewall-cmd and use raw python to get the
firewalld version if firewalld-cmd does not return anything
Incorporated the simp-beaker-helpers gem to get access to the helper
methods

Fixes #263

Pull Request (PR) description

This Pull Request (PR) fixes the following issues

LGTM.  Have you tested on both EL7 and EL8?

@alexjfisher Yeah, it works on 7 and 8 properly. They're both in the Beaker nodeset


Remove input variable from manual trigger

This wasn't working as I expected it to, so let's remove for now. We can still deploy manually, it will just always deploy to GitHub Pages.

Self-merging to unbreak CI


Test suite

I have made:

A test suite that verifies the conversion process automatically from examples persisted on disk in the test directory. I used Jest.
An automatic generation script for the test suite, which is very helpful while developing, to verify that the changes are acceptable.
github's CI integration

I updated the branch:

More tests
Support for xfail to mark known failures of conversions

Also npm run bugs to list know bugs from the test suite in the command line

VSCode settings so we have formatting when needed only.

I also refactored the test suite runner to use ES modules.


Not possible to create ServiceMonitor if running in DaemonSet mode

It should be possible to monitor collectors if they run in DaemonSet mode.
https://github.com/open-telemetry/opentelemetry-helm-charts/blob/ad613643be0db1a18cc9bf8ccd0fd22a9b734f89/charts/opentelemetry-collector/templates/servicemonitor.yaml#L1 condition won't allow to create ServiceMonitor if in DaemonSet mode.

This is a hold-over from when we didn't allow the daemonset mode to create a service.  Now that we allow daemonset to have a service, we should allow the service monitor as well.


gives an error when import tensorflow

import tensorflow
Traceback (most recent call last):
File "C:\Users\домашний\AppData\Local\Programs\Python\Python37\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 58, in
from tensorflow.python.pywrap_tensorflow_internal import *
File "C:\Users\домашний\AppData\Local\Programs\Python\Python37\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 28, in
_pywrap_tensorflow_internal = swig_import_helper()
File "C:\Users\домашний\AppData\Local\Programs\Python\Python37\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 24, in swig_import_helper
_mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description)
File "C:\Users\домашний\AppData\Local\Programs\Python\Python37\lib\imp.py", line 243, in load_module
return load_dynamic(name, filename, file)
File "C:\Users\домашний\AppData\Local\Programs\Python\Python37\lib\imp.py", line 343, in load_dynamic
return _load(spec)
ImportError: DLL load failed with error code -1073741795

During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "<pyshell#1>", line 1, in
import tensorflow
File "C:\Users\домашний\AppData\Local\Programs\Python\Python37\lib\site-packages\tensorflow_init_.py", line 24, in
from tensorflow.python import pywrap_tensorflow  # pylint: disable=unused-import
File "C:\Users\домашний\AppData\Local\Programs\Python\Python37\lib\site-packages\tensorflow\python_init_.py", line 49, in
from tensorflow.python import pywrap_tensorflow
File "C:\Users\домашний\AppData\Local\Programs\Python\Python37\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 74, in
raise ImportError(msg)
ImportError: Traceback (most recent call last):
File "C:\Users\домашний\AppData\Local\Programs\Python\Python37\lib\site-packages\tensorflow\python\pywrap_tensorflow.py", line 58, in
from tensorflow.python.pywrap_tensorflow_internal import *
File "C:\Users\домашний\AppData\Local\Programs\Python\Python37\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 28, in
_pywrap_tensorflow_internal = swig_import_helper()
File "C:\Users\домашний\AppData\Local\Programs\Python\Python37\lib\site-packages\tensorflow\python\pywrap_tensorflow_internal.py", line 24, in swig_import_helper
_mod = imp.load_module('_pywrap_tensorflow_internal', fp, pathname, description)
File "C:\Users\домашний\AppData\Local\Programs\Python\Python37\lib\imp.py", line 243, in load_module
return load_dynamic(name, filename, file)
File "C:\Users\домашний\AppData\Local\Programs\Python\Python37\lib\imp.py", line 343, in load_dynamic
return _load(spec)
ImportError: DLL load failed with error code -1073741795
Failed to load the native TensorFlow runtime.
See https://www.tensorflow.org/install/errors
for some common reasons and solutions.  Include the entire stack trace
above this error message when asking for help.

C:\Users\домашний>pip freeze
absl-py==0.7.1
astor==0.7.1
et-xmlfile==1.0.1
gast==0.2.2
grpcio==1.20.1
h5py==2.9.0
jdcal==1.4.1
Keras==2.2.4
Keras-Applications==1.0.7
Keras-Preprocessing==1.0.9
Markdown==3.1
mock==3.0.5
numpy==1.16.3
openpyxl==2.6.2
Pillow==6.0.0
protobuf==3.7.1
PyYAML==5.1
scipy==1.2.1
six==1.12.0
tensorboard==1.13.1
tensorflow==1.13.1
tensorflow-estimator==1.13.0
termcolor==1.1.0
Werkzeug==0.15.2
python 3.7.0
What could be the problem?

I don't think tensorflow supports python 3.7.0 right now.

@igorgertman   Please provide details about what platform you are using (operating system, architecture). Also include your TensorFlow version. Also, did you compile from source or install a binary? Thanks!

Windows 7 64 bit operating system. AMD phenom ii n930 processor.  on the K10 architecture, tensorflow version 1.13.1. Installed through PIP install tensorflow.

pip 19.1.1 from c:\users\домашний\appdata\local\programs\python\python37\lib\sit
e-packages\pip (python 3.7)
Python 3.7.0 (v3.7.0:1bf9cc5093, Jun 27 2018, 04:59:51) [MSC v.1914 64 bit (AMD64)] on win32

@igorgertman Please check your processor supports AVX instructions?

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!

Automatically closing due to lack of recent activity. Please update the issue when new information becomes available, and we will reopen the issue. Thanks!


Upgrade to latest testcontainers

Currently, a testcontainers version (9baedef17589ff70f7a585c809e0b1beb27eff62) is taken from GitHub directly using jitpack. This was required to make the build succeed under Mac M1.

[ ] Replace this version with a release and remove the jitpack repository again

done


spv: v0.0.4

Updated to convert delegated.key to WIF and derive an address from it instead of generating a mnemonic; added parsing and reporting for UTXOs and transactions.

Added host to resolve seed.multidoge.org, removed static IP.

Closing, superseded by https://github.com/dogeorg/pups/pull/9.


Ping pong Ball Score Keeper

This is A pull request for issue number #49 .
@chicken-biryani @Ayushparikh-code  Please review my PR.

Add your project Title and description in index.md

@chicken-biryani  Done all the necessary changes.


[Suggestion] Add Forge support

Currently, there are no plans for providing Forge support for the newest Minecraft versions. Lighty is available on NeoForge however, and when I feel like Lighty is "done", I might offer a Forge version again. For now, the main focus is on improving Lighty for the platforms it is available on, which might take some time to complete.
That said, if someone wants to PR Forge support, I'll gladly accept the PR if it fits into the existing project structure.
Then again, as far as I can tell most big mods are transitioning away from Forge to NeoForge, so maybe it is time to move on too.

Thank you for such a fast and elaborate answer.
I must admit, I was not aware of the Forge/NeoForge situation until I saw your answer. I totally get why you would want to put your efforts elsewhere


Do we still need test-apiserver.sh to run as part of integration tests?

contrib/hack/test-apiserver.sh is run as part of our integration tests, and it drives kubectl commands to simulate creating resources. Two questions:

These don't appear to simulate any functionality beyond CRUD actions on an etcd-based API server. What was the original intent for this script?
Because it drives kubectl commands, it looks more like an end-to-end test. Can we move it there?

cc/ @duglin @MHBauer since I think one or both of you wrote this script originally

its original intent was to make sure we didn't regress on the apiserver as we made changes.
While it starts etcd, we could easily make it use TPRs - the same kubectl cmds should still work, right?
I'm ok with moving it to an e2e bucket if that's where it should live. I'm also ok with removing it if we have other tests that do the same thing.

Effectively this was the forst version of the clientset test.
i don't think we absolutely need to keep it around, because we do have the
client set test.
On Fri, May 12, 2017 at 1:45 PM Doug Davis<EMAIL_ADDRESS>wrote:

its original intent was to make sure we didn't regress on the apiserver as
we made changes.
While it starts etcd, we could easily make it use TPRs - the same kubectl
cmds should still work, right?
I'm ok with moving it to an e2e bucket if that's where it should live. I'm
also ok with removing it if we have other tests that do the same thing.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes-incubator/service-catalog/issues/842#issuecomment-301141549,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAWXmHnGxKHlCc_KDjrG2K6FT2_OLHGJks5r5Jq5gaJpZM4NZkaw
.

I will remove the script entirely in a PR to come

Looking at this and #865 we wanted to keep this as it tests compatibility with kubectl.


chore(release): bump to v2.1.1

Details
Release version 2.1.1
Motivation
Address CG issues that have been fixed in the code but not in the dist
Context

4 tests in task-config-spec are hardcoded for linux and fail on Windows. The same is true at the v2.1.0 tag, so it's unrelated to any changes since then.
The NOTICES.TXT file was generated via the pipeline's CG option, and removes notices for 5 package versions that we no longer use.

Pull request checklist

[n/a] Addresses an existing issue:
[n/a] Added relevant unit test for your changes. (yarn test)
[n/a] Verified code coverage for the changes made. Check coverage report at: <rootDir>/test-results/unit/coverage
[x] Ran precheckin (yarn precheckin)
[] (after PR created) The Accessibility Checks (pull_request) check should fail. All other checks should pass.

Closing this to reopen from the Microsoft repo (see #629)


Tls script

script to set config-deployment root CA + audience
script to unset config-deployment  root CA + audience
Cleanup in guard-service deployment

/hold

/cherrypick release-0.3


Move helm tests to use --profile=ambient

Fixes https://github.com/istio/istio/issues/49242
Please provide a description of this PR:

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-helm_istio
37d7169207f57d368ad7519fbce7ff3e8f3b3c75
link
true
/test integ-helm

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-helm_istio
37d7169207f57d368ad7519fbce7ff3e8f3b3c75
link
true
/test integ-helm

integ-ambient_istio
37d7169207f57d368ad7519fbce7ff3e8f3b3c75
link
true
/test integ-ambient

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

blocked on https://github.com/istio/istio/pull/50336

Related https://github.com/istio/istio/issues/49242 I thought we can do this only after 1.22 as we have an upgrade test from previous version and profile changes made it to only 1.21

Related #49242 I thought we can do this only after 1.22 as we have an upgrade test from previous version and profile changes made it to only 1.21

I think at this point master is "1.22" so its fine now

PR needs rebase.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-helm_istio
72c047cf7054e395f3b015bd83b68974700bb139
link
true
/test integ-helm

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

integ-helm_istio
80a93cd737b4af531a2f7278c0fd1c6b1b66e620
link
true
/test integ-helm

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Flutter crashes when pressing the Escape key on a Lenovo Yoga Chromebook

Flutter doctor information:
Doctor summary (to see all details, run flutter doctor -v):[✓] Flutter (Channel beta, v1.5.4-hotfix.1, on Linux, locale en_US.UTF-8)[✓] Android toolchain - develop for Android devices (Android SDK version 28.0.3)[!] Android Studio (not installed)[✓] VS Code (version 1.33.1)[✓] Connected device (1 available)
ChromeOS version: 73.0.3683.114
Flutter crashes with the following error when the escape key is pressed:
W/EGL-MAIN( 3320): Could not dequeue buffer from native window
F/libc ( 3320): Fatal signal 11 (SIGSEGV), code 1, fault addr 0x68 in tid 3337 (1.gpu)

Build fingerprint: 'google/nami/nami_cheets:7.1.1/R73-11647.154.0/5465689:user/release-keys'
Revision: '0'
ABI: 'x86_64'
pid: 3320, tid: 3337, name: 1.gpu >>> dev.flutter.devRpg <<<
signal 11 (SIGSEGV), code 1 (SEGV_MAPERR), fault addr 0x68
rax 00007f3a90371320 rbx<PHONE_NUMBER>000000 rcx<PHONE_NUMBER>000000 rdx 00007f3a989b6600
rsi 00007f3ac0a46070 rdi<PHONE_NUMBER>000000
r8<PHONE_NUMBER>000001 r9<PHONE_NUMBER>000000 r10 00007f3a8e525700 r11<PHONE_NUMBER>000001
r12 00007f3ac04dc108 r13<PHONE_NUMBER>000000 r14 00007f3aacdd2e78 r15 00007f3ac04d91f0
cs<PHONE_NUMBER>000033 ss<PHONE_NUMBER>00002b
rip 00007f3aacaa251a rbp 00000000ffffffff rsp 00007f3ac0a45ef0 eflags<PHONE_NUMBER>010246
backtrace:
#00 pc<PHONE_NUMBER>42f51a /vendor/lib64/dri/i965_dri.so
Lost connection to device.

cc @cbracken

/cc @jason-simmons

Our current understanding is that this is a device-specific crasher for the particular Chromebook model in question and that the crasher is very reproducible. @mjohnsullivan has one of these devices that he's promised to let us take a look at next week.

I am informed that @mjohnsullivan gave @cbracken the offending device last week. Reassigned accordingly.

Hi @mjohnsullivan
Just checking to see if this is still a valid issue using latest Flutter version ?

Without additional information, we are unfortunately not sure how to resolve this issue.
We are therefore reluctantly going to close this bug for now.
Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact description of what happens, logs, and the output of flutter doctor -v.
All system setups can be slightly different, so it's always better to open new issues and reference related issues.


DOC: More single to double backtick

A quick scan of sphinx errors when building with default role being py:obj, a few references look impossible as they are invalid python identifiers, and most of them need double backticks.

ps. for future doc only change feel free to add [docs only] to the end of the commit message to save on CI

ps. for future doc only change feel free to add [docs only] to the end of the commit message to save on CI

Thanks for merging, and yes i'm supposed to know that, but I keep forgetting...


Ensure csrftoken cookie is set inside nevercache tag

I'm currently hunting down occasional CSRF failures for users and noticed that Mezzanine isn't setting the CSRF cookie when it should for logged in users. It's not the cause for the issues I'm tracking, but it is a potential issue for other setups.
When the nevercache tag is used, any {% csrf_token %} inside the tag will not trigger a set cookie for the csrftoken cookie because the CsrfViewMiddleware has already run by the time UpdateCacheMiddleware renders the contents of the nevercache tag. Since the standard Mezzanine template include core/templates/includes/form_fields.html wraps the {% csrf_token %} with a nevercache tag this affects most forms being rendered by Mezzanine. It can be observed by logging into the demo site and going to 'Update Profile'. If you inspect the HTTP response there is no Set-Cookie for csrftoken.
Generally this isn't a problem because the cookie is set on login and has an expiration time of 1 year. Clearing cookies will also clear the session cookie and log the user out, so it's hard to end up in a situation where the session cookie is set (so the user is logged in) but not the CSRF cookie. In theory if the CSRF token is not refreshed within a year it will start failing, but it's unlikely for a user to be logged in for a year. This length is however configurable and there may be use cases (the docs mention IE problems) where setting it to None for a session-length cookie is needed, which could result in the situation where the user is logged in but the CSRF token cookie is not set after the the user closes their browser and reopens it.
Another possible case this would be problematic would be if a server is setup to use HTTP Basic Auth and Django is using a Remote User Auth. In that case there is no login form to cause the CSRF token to be set so the first time the user tries to use a form with this bug it would fail.
The pull request simply runs the CsrfViewMiddleware again for the response to set the cookie if {% csrf_token %} was used in a nevercache tag. There are no observable side effects from running it twice in the case that the cookie has already been set by a {% csrf_token %} not in a nevercache tag.

Confirmed that the easiest way to reproduce the issue this fixes (I've modified the pull request to remove an over limiting check for an existing CSRF token) is to put CSRF_COOKIE_AGE = None in your settings, log in, close the browser (I used FireFox, Chrome wasn't keeping me logged in between closing and opening) and then immediately go to the 'Update Profile' page and try to submit that form. It will fail with a CSRF token cookie not set error. Applying this patch makes it work as expected.

Added a testcase, although I don't love the hack I had to put around nevercache. Since it gets set at module load time I had to find a way to force it to be the caching version for that one specific test and then go back to being the no-op version.

Great work David, thank you.


make the card pictures square

#23

Hi! The size is okay but some images look stretched like this:

Can you adjust the aspect ratio? Probably object-fit: cover; will do

Hi! The size is okay but some images look stretched like this:

Can you adjust the aspect ratio? Probably object-fit: cover; will do

Fix in https://github.com/vspedr/developets/pull/27/commits/6c12f0ca45cbd9e506d4e13c4c58ab4a7bc3c833

Looks good now! Thanks :)


Remove unused setting should_sync_alerts_as_incidents

Why do we need this ?

We currently have usages of a setting with key should_sync_alerts_as_incidents  in the bff. This API is not being used in the BFF and the setting type is not in the backend anymore.
Unused code and tech debt

Acceptance Criteria

[ ] Remove any usages and api calls for the setting with key  should_sync_alerts_as_incidents

Further Comments / References

Hey @samad-yar-khan
Can you assign this to me
I would like to explore our backend.
And remove unused code.

Hey @VipinDevelops, apologies for the delay on this one, but yes, you can take this up. :)

Can I work on this? @jayantbh

Sure @Kamlesh72. I'll assign this to you.

Hey @Kamlesh72  Yeah you can work on this


Added a param to bypass permissions.

This is for servers who want spells (that modify blocks) that should be allowed to be done in protected areas.

Doesn't bypass_build do this already?

Oh, I think it does. I did take a quick look around spells.default and I didn't catch that!


Flask g cache cause "The CSRF session token is missing" error

Logging in to one account with multiple browsers will cause CSRF token verification to fail because of the cache of flask "g"。

My backend code to generate csrf :

sass_blueprint = Blueprint("sass", __name__, url_prefix="/api/cli")

@sass_blueprint.after_request
def after_request(response):
    csrf_token = generate_csrf()
    response.set_cookie("csrf_token", csrf_token)
    return response

The first browser I log into CSRF works fine

I log into the same account with a second browser got  an error: "The CSRF session token is missing"

I observed that flask g cache can cause this problem

My temporary solution

sass_blueprint = Blueprint("sass", __name__, url_prefix="/api/cli")

@sass_blueprint.before_request
def before_request():
    if 'csrf_token' in g:
        delattr(g, 'csrf_token')

I found that flask "g.xxx" is not reset after a request

test code:
sass_blueprint = Blueprint("sass", __name__, url_prefix="/api/cli")

@sass_blueprint.before_request
def before_request():
    try:
        print('xxxxxx in g:', 'xxxxxx' in g)

        if 'xxxxxx' not in g:
            print('set g xxxxxx......')
            setattr(g, 'xxxxxx', 'xxxxxx')

Then request the service several times，found that flask "g.xxx" is not reset after a request
Flask version:  2.0.1

I found that g is not destroyed after a request，so this caused the problem.
# test code
sass_blueprint = Blueprint("sass", __name__, url_prefix="/api/cli")

@sass_blueprint.before_request
def before_request():
    try:
        print('xxxxxx in g:', 'xxxxxx' in g)
        if 'xxxxxx' not in g:
            print('set g xxxxxx......')
            setattr(g, 'xxxxxx', 'xxxxxx')

Flask: 2.0.1
Link:  https://flask.palletsprojects.com/en/2.0.x/api/#flask.g

There is something else wrong with your code. g is not persisted across requests. Perhaps you're pushing an app context globally rather than letting Flask manage it per-request?

@davidism Thanks a lot ，I found that I use app.app_context().push() can cause this problem.


refactoring of internal criteria-related APIs

This is a bit saner.

Ugggg DONT MERGE PLEASE

Ugggg DONT MERGE PLEASE

OK, that's better. I futzed the rebase.


Description of ParserRules missing

Could you please add some more description of how parser rules work. I think this would help a lot of people to work with your great editor.
There is a very steep learning curve and it is so hard to understand how to proceed when defining parser rules.
Could you please give some guidance. Is there a debug mode which steps should I use to add new rules?

+1

+1 looking for that too

+1


Referenced type variable incorrectly inferred as "{}" in function calls

TypeScript Version:  2.6.0-dev.201xxxxx
Code
// Fails:  error TS2339: Property 'a' does not exist on type '{}'.
infer({ thing: { a: 10 } }).a;

// With explicit type parameters, it works
infer<{a:number}, Holder<{a: number}>>({ thing: { a: 10 } }).a;

interface Holder<T> {
	thing: T;
}
function infer<T, H extends Holder<T>>(holder: H): T {
	return holder.thing;
}

Expected behavior:
The type parameter T should be inferred to be the type of thing in the instance of Holder passed to the function infer().
Perhaps there is another way to get this type of inference to work? I'm building a builder-pattern class and really need this to correctly return the right types.
Actual behavior:
The type parameter, T, is inferred to be {} when calling infer()

You don't actually need H here.
function infer<T>(holder: Holder<T>): T {
	return holder.thing;
}

Fair, though in my actual code I do need it. Slightly contrived, but closer to my actual use case:
const result = infer({ thing: { a: 10 } });
result.holder.thing += result.value.thing;

interface Holder<T> {
	thing: T;
}
function infer<T, H extends Holder<T>>(holder: H): { holder: H, value: T} {
	return { holder, value: holder.thing };
}

How about function infer<T, H extends Holder<T>>(holder: H & Holder<T>): { holder: H, value: T} {?

@andy-ms Yeah, that's perfect, actually. Thanks. It seems to me that it shouldn't be necessary, but this is a great work around.

@sandersn Could we infer a more specific T here?

Another option
function infer<H extends Holder<any>>(holder: H): { holder: H; value: H['thing'] } {
  return { holder, value: holder.thing }
}

you can define your function as:
function infer<T, H extends Holder<T>>(holder: Holder<T> & H): T

Automatically closing this issue for housekeeping purposes. The issue labels indicate that it is unactionable at the moment or has already been addressed.

@sandersn Could we infer a more specific T in the original example? It seems silly to add & Holder<T> when H already extends that.

constraints are not inference positions.. they are merely for checking constraints after an inference has occurred.
Now, the general question is i want to capture a container type "and" the type it contains.. this is doable through one of two ways.. either, you infer the container type, and you access the element type using indexed access type, e.g.:
function infer<H extends Holder<any>>(holder: H): H['thing'];

or through making two inferences using intersection types, e.g.:
function infer<T, H extends Holder<T>>(holder: Holder<T> & H): T;


Replace use of any with types for query from controller-utils

The query function in controller-utils uses any for its arguments as well as its return type. This means that any will infect other functions that use query
query is used in the following places throughout core:

assets-controllers

AccountTrackerController.getBalanceFromChain
AccountTrackerController.syncBalancesWithAddresses

user-operation-controller

getSuggestedGasFees
PendingUserOperationTracker#onUserOperationConfirmed
Bundler.estimateUserOperations

gas-fee-controller

fetchEthGasPriceEstimate
fetchLatestBlock
fetchBlockFeeHistory

transaction-controller

updatePostTransactionBalance
getSuggestedGasFees
TransactionController.approveTransaction
TransactionController.publishTransaction
estimateGas
getCode
getLatestBlock
readAddressAsContract
PendingTransactionTracker#getTransactionReceipt
PendingTransactionTracker#getBlockByHash
PendingTransactionTracker#getNetworkTransactionCount

Changing the type of the arguments that query takes or its return type may affect all of these packages, or any other packages downstream.

The attached PR (#3912) records an initial approach discussed by the team during refinement. The ticket is available for anyone to take over and build on the PR.


Eliminate ordered_query

The ordered_query function is confusingly-named, because it invites comparison with p2p_query (which reads data from a list of Derecho members) but can modify state on group members and can only be called on exactly one subgroup. It's really just syntactic sugar for an ordered_send that has a return value. We should eliminate ordered_query and allow ordered_send to have a non-void return type.

It turns out, as Matthew pointed out in #65, that combining ordered_send and ordered_query is actually pretty important for ensuring that void-returning RPC functions have a way to wait for stability. The only way for a caller to guarantee that a state update performed by an RPC function has completed at all nodes in the subgroup is to wait for the std::future of the QueryResults::ReplyMap to be fulfilled, since this means the RPC message was delivered in the current View. However, ordered_send doesn't return anything, ostensibly because there are no replies to track. It should at least return a QueryResults<void> that callers can wait on, with an empty ReplyMap that gets fulfilled when the RPC message is delivered in the current View.


Update quilt_rails (possibly react-server) to be compatible with react@18.

Overview
Original slack thread: https://shopify.slack.com/archives/C01H7LC0EEL/p1650556018972339
TLDR; projects using quilt_rails will have client side hydration errors when using react 18 which deprecates reactDOM.hydrate.
Given: react 18 is installed
When: hydrating the client
Then: use hydrateRoot instead of hydrate
We should update our client to use hydrateRoot when we detect react@18 is being used.
Consuming repo
repo: https://github.com/Shopify/m2m-prototype
PR: https://github.com/Shopify/m2m-prototype/pull/53
Checklist

[x] Please delete the labels section before submitting your issue
[x] I have described this issue in a way that is actionable (if possible)

Hey @KevinMind I was curious if you had any updates with this. I'm facing this same issue today. The repo and PR you linked show a 404 if I click on them, by the way.


Update nvarchar field length for entities

Update nvarchar field length instead of nvarchar(max) for:

Providers. Reference issue#54
Workshops. Reference issue#55
Teachers. Reference issue#55 section 4 & 5
Addresses. Reference issue#55
Social groups (name limit to 100)
User as per recommendations
Make sure if any entity has a key for UserId the field is capped at nvarchar(450)

done


Change order of objects to avoid scrollbar over visualisation chooser

Pull Request Description
Fixes the scrollbar of the text visualization overlaying the visualization chooser.  Fixes #5089
Also fixes the scrollbar not overlaying the visualization action bar.
https://user-images.githubusercontent.com/1428930/223997913-3a9c84ae-6675-4f8e-b2e2-99c9e9c98565.mp4
Checklist
Please include the following checklist in your PR:

[x] The documentation has been updated if necessary.
[x] All code conforms to the
Scala,
Java,
and
Rust
style guides.
All code has been tested:

[x] Unit tests have been written where possible.
[x] If GUI codebase was changed: Enso GUI was tested when built using BOTH
./run ide build and ./run ide watch.

QA report
Overall the issue is fixed. 🟢 The dropdown is always the topmost element on the scene, which is nice.
Side issues
However, I can't help but notice that it is virtually impossible to grab the vertical scrollbar with the mouse – the action bar always covers it.
https://user-images.githubusercontent.com/6566674/224679744-609dc292-4dfc-4c3a-9d6c-87d3c99b8030.mp4
Another issue is that the visualization has rounded corners, but the scroll area does not – and the tip of the scrollbar is visible in the corner. I didn't find an issue for that.

Side issues
However, I can't help but notice that it is virtually impossible to grab the vertical scrollbar with the mouse – the action bar always covers it.

It is still possible to scroll by clicking into the scrollbar. But we might want to think about adjusting the minimal length of the scrollbar, to make it longer than the action bart height.


feat: using the docker file that has Madara + Kakarot + hive genesis state + kakarot-RPC, launch hive and run tests.

world boss

@ftupas what is the status of this?

So technically we are now able to use the Dockerfile to run the hive tests but they are not passing due to the EOA funding issue 😥 Should we close this and add the debugging issues to the epic?


Dvclive clarifications

Closes #4644.
This separates how dvclive tracks results and works with git and dvc into its own page. Before merging, we should decide where these explanations are sufficient, and where we need to make product updates to simplify.

I like the idea of explaining things better (even if we decide to change the product later), but why do we need to have two pages? I really liked the simplicity and the title "how it works" vs this more formal and more advanced approach?

No strong opinion on two pages vs one page. The idea came from @alex000kim in https://iterativeai.slack.com/archives/C039PTZCM39/p1688063883826619.


Fix typo

Fixes typo in endless battle warning

...in what sense is this a typo?

Ohhh, turnsLeftText already has "turns" in it.

Thanks!


missing NSG on RRAS to allow the traffic in between the local and remote vnet

[Enter feedback here]
One customer followed this doc to deploy the S2S VPN tunnel using RRAS on two of this azure stack stamps. The tunnel was setup successfully, but tenant VM cannot talk to its peer on another stamp.
We figured out the problem is that we were miss the NSG rule on the RRAS to allow the traffic in between the two vNets in both inbound and outbound. Without this NSG, the RRAS will drop the packets when its IP is not either the source or destination IP.
We should allow source "LocalVNetAddressSpace", destination "RemoteVNetAddressSpace", protocal Any, port Any in both inbound and outbound.
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: c745c925-5ab8-7b3d-0f1b-4a60a63bd831
Version Independent ID: aeb3a9fa-fe38-ac0d-f166-49354010562d
Content: Set up multiple site-to-site VPN tunnel in Azure Stack Hub - Azure Stack Hub
Content Source: azure-stack/user/network-howto-vpn-tunnel.md
Service: azure-stack
Sub-service: azure-stack-hub
GitHub Login: @mattbriggs
Microsoft Alias: mabrigg

@Donnie-Zhang Thanks for the feedback! I have assigned the issue to content author to check and update the document as appropriate. We will update you accordingly.

@mattbriggs Can you please check and add your comments on this doc update request as applicable


Renamed isAt* funcs to at* in Parser and Parser.Lookahead

Motivation
Closes #1976. This pull request is a step towards making the Parser API consistent, and renaming some of the boolean isAt* functions with at*.
Changes

[x] Lexer.Cursor.isAtStringInterpolationAnchor — private, so I'll rename it and change any spots where it's invoked. Deprecating the existing function should not be necessary.
[x] Lexer.Cursor.isAtEscapedNewline — same as above.
[x] Parser.Lookahead has two fileprivate functions isAtStartOfPostfixExprSuffix and isInBindingPatternPosition — I think it's best to rename both of them so the code in that particular extension is consistent. It would be weird to have atStartOfPostfixExprSuffix and isIn*.
[x] Parser.isAtPythonStyleInheritanceClause
[x] In SwiftParser/Statements.swift we have Parser.Lookahead extension with isAtStartOfSwitchCase, isStartOfStatement, isStartOfConditionalSwitchCases.

These are not marked as private, so I think we should keep the original version and deprecate it.
Should I rename isStartOf* as well for consistency?

[x] SwiftParser/Types.swift has Parser.Lookahead.isAtFunctionTypeArrow — also not marked as private.

TODO

[x] Decide if we need deprecated versions for non-private funcs
[x] Tests pass
[x] Formatting OK

@ahoppen, thank you for reviewing this! Cleaned up Lexer, so I think it should be good to go now.

@swift-ci Please test

@ahoppen cleaned it up. Parser and Parser.Lookahead use at* notation. Cursor is back to isAt* notation.

@swift-ci Please test

@ahoppen I see the formatting issues on the CI output, and I can clean them up. I'll build swift-format from main as you advised and work through them.

Great. Thank you for fixing the formatting issue. Also: The macOS CI failure is unrelated to your changes.

My local swift-format is still building, but I think I've fixed the only problem it found. Rebased over main to see if the macOS test is fixed there.

OK, let’s try again.
Also, just a note: We prefer to squash the commits of a PR because it just creates a nicer git history. I’ll put together some documentation with reasons for that in the upcoming days.
@swift-ci Please test

@ahoppen, agreed re: squashing — I usually end up doing that on the GitHub side when merging, but I'm happy to git rebase -i and squash so that after a review there's a single commit per pull request!
I usually keep commits separate to retain the step by step history of a PR while we're still talking through it, but once the review concludes, I can absulutely squash them.

There's Parser.isContextualExpressionModifier — should we rename it as well for consistency? @ahoppen

@swift-ci Please test Windows

There's Parser.isContextualExpressionModifier — should we rename it as well for consistency? @ahoppen

Yes, I think that would be a good idea as well.

Cleaned it up, and squashed into a single commit.

@swift-ci Please test

@swift-ci Please test Windows


Docker config: publishing DB port to the host

Is your feature request related to a problem? Please describe.
The Docker config publishes the database port to the host in the standard mysqld port. This can (as in my case) conflict with an already running mysqld and prevent the container from starting. I commented out the published ports in the docker config, which seems to have no effect on the running seonaut instance later.
Describe the solution you'd like
If there's no need to expose the DB to the host: remove the port publishing for the DB.
If the DB needs to be exposed to the host: use a port other than 3306 on the host.
Describe alternatives you've considered
Additional context

I changed the docker file in ea43e63  so it does not expose the mysql and redis ports to the host by default.
There's also a new docker-compose.dev file which will expose the ports, as I find it useful in some cases. Also, it is now possible to set the port numbers via env variables, which will make it easier to change.


Support for Elasticsearch versions other than 5.x

Feature Description
For those of us that can't upgrade to ES 5.x (yet), it would be super helpful if there was an official 2.X version of this Docker image that we could build upon. Any plans to support versions prior to 5.x?

I have to be honest and say there's no plans, sorry.
I don't imagine the basic image would be that hard to create, but we demand (of ourselves), full documentation, acceptance tests and release automation. The total body work to release 2.x images at a standard that we'd be happy with is out of scope.


composer refuses to install library

Hi all,
I'm trying to install and use the IGDB (v3) api for PHP for a school project however composer doesn't seems to like it very much.
I installed composer with their .exe file from the official website and tried
composer.phar require igdb/api
However it will revert the changes when I try to specify a version
Ex: composer.phar require igdb/api:*
This is what composer outputs:
> ./composer.json has been updated
> Loading composer repositories with package information
>Updating dependencies (including require-dev)
> Your requirements could not be resolved to an installable set of packages.

>Problem 1
> - The requested package igdb/api * is satisfiable by igdb/api[dev-master] but these conflict with your > requirements or minimum-stability.
                                                                                >Installation failed, reverting ./composer.json to its original content.

Is there any solution (or alternative) for this?
Kind regards,
Ruben

The owner of this repository needs to tag a release.

On 3 Mar 2019, at 15:40, RFlintstone<EMAIL_ADDRESS>wrote:
Hi all,
I'm trying to install and use the IGDB (v3) api for PHP for a school project however composer doesn't seems to like it very much.
I installed composer with their .exe file from the official website and tried
composer.phar require igdb/api
However it will revert the changes when I try to specify a version
Ex: composer.phar require igdb/api:*
This is what composer outputs:

./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

Problem 1

The requested package igdb/api * is satisfiable by igdb/api[dev-master] but these conflict with your > requirements or minimum-stability.
Installation failed, reverting ./composer.json to its original content.
Is there any solution (or alternative) for this?

Kind regards,
Ruben
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Also note that this library hasn’t been updated since v1 and almost certainly won’t work with v3. There doesn’t seem to be much interest in officially supporting this, and I’be not signed up for an API account until I know for sure the API isn’t going to move once again. Aiming at a moving target isn’t much fun :P

On 3 Mar 2019, at 15:40, RFlintstone<EMAIL_ADDRESS>wrote:
Hi all,
I'm trying to install and use the IGDB (v3) api for PHP for a school project however composer doesn't seems to like it very much.
I installed composer with their .exe file from the official website and tried
composer.phar require igdb/api
However it will revert the changes when I try to specify a version
Ex: composer.phar require igdb/api:*
This is what composer outputs:

./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

Problem 1

The requested package igdb/api * is satisfiable by igdb/api[dev-master] but these conflict with your > requirements or minimum-stability.
Installation failed, reverting ./composer.json to its original content.
Is there any solution (or alternative) for this?

Kind regards,
Ruben
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Also note that this library hasn’t been updated since v1 and almost certainly won’t work with v3. There doesn’t seem to be much interest in officially supporting this, and I’be not signed up for an API account until I know for sure the API isn’t going to move once again. Aiming at a moving target isn’t much fun :P

True, is it better to use node.js in this case?
-Ruben

In this case, I'd say so yes.


Make the title of social events black

Number of Volunteers: 1
(cc @Robert-Alonso @prabhat1081)
Can we make the tile of these events black? They are not clickable.

Will do


Client secret

Although the documentation is clear "The Amazon Cognito JavaScript SDK does not use the app client secret" I dont understand why dont add the header "Authorization" in the function getCodeQueryParameter(httpRequestResponse) (amazon-cognito-auth.js) when the user configure its sdk with "client secret". This not solve the problem?

I can't commet on the suggestion above, but I would like to see support for applications with a "client secret".

I will mark this as a feature request. However I cannot estimate the delivery time at this point.

@cristianedb is correct. If the client was issued a secret, the client must pass its client_id and client_secret in the authorization header through Basic HTTP authorization. The secret is Basic Base64Encode(client_id:client_secret).


App Crashes when initializing location provider

Hi, I am trying to integrate location provider into my app...I added the swift package and I added the @ObservedObject and init code from the sample app to my main view in swiftui, and the plist entries as specified, and when I run my app I get this error:
2020-09-14 11:25:59.764437-0700 MyApp[24106:754941] *** Assertion failure in -[CLLocationManager setAllowsBackgroundLocationUpdates:], CLLocationManager.m:927
2020-09-14 11:25:59.767850-0700 MyApp[24106:754941] *** Terminating app due to uncaught exception 'NSInternalInconsistencyException', reason: 'Invalid parameter not satisfying: !stayUp || CLClientIsBackgroundable(internal->fClient) || _CFMZEnabled()'
*** First throw call stack:
(0x1944c9114 0x1a7c7fcb4 0x1943d8308 0x1956bc2c8 0x199fc94f8 0x104b7a9bc 0x104b7aad0 0x104b7a404 0x104984f7c 0x104a82328 0x19a95b2cc 0x104a81984 0x19ae737a0 0x104a81368 0x104a83414 0x19a925bdc 0x19abfb7ac 0x19abfb61c 0x19a946d8c 0x1ba6b6064 0x1ba6b6474 0x1ba6bf060 0x19aea7774 0x19aeaa448 0x19aea9044 0x19aeaa3e4 0x19aa368a8 0x19ae15054 0x19ae0db20 0x19aa334c8 0x19aeaa3c4 0x19aeab560 0x19aa50f78 0x19a7a8ec0 0x19a7a8e28 0x19a7a3ef4 0x1b7955f40 0x19a7a8e0c 0x19a7a8f40 0x194447ecc 0x1944425b0 0x194442af8 0x194442200 0x1aa4bc598 0x196d081b0 0x196d0d784 0x104b5ab4c 0x194121598)
libc++abi.dylib: terminating with uncaught exception of type NSException
Any ideas what might cause this?

Hi,
Have you added Location to UIBackgroundModes in the plist as described in the Readme? Seems like your app is not getting permission to run in the background.
Alternatively, you can also tick the corresponding checkbox in the project settings.
Please tell me if that resolved the issue. If not, you could you check if the linked minimal example project runs for you?

That seemed to fix the crash (I hadn't had background services enabled, sorry for the lack of reading the readme heh)...for some reason my app doesn't require background services so I don't know if there is a granular way to disable the need for background services...
Now, I am having the same issue as the user in #3, here is my log for your reference:
start() No location authorization status set by delegate yet. Try to start updates anyhow. locationManager(_:didChangeAuthorization:) authorizedWhenInUse
I have a swiftui view, and I am calling locationProvider.requestAuthorization() in my .onAppear method, but the authorization request pop up window isn't "popping up"...also, when i check for lat and lon via locationProvider.location?.coordinate.latitude, both come back nil...

No worries...

For setting different, more fine-grained properties, the easiest would be to just copy LocationProvider.swift into your project instead of using it as a package. It is only a single file anyhow.
Then set self.lm.allowsBackgroundLocationUpdates = false in the init() function.

For the problem with the popup not appearing: you can try deleting and reinstalling the app on Simulator/Device. There is sometimes problems with caching of old location permission settings when resetting the app permissions (background, etc.)

Gotcha...yeah i deleted and reinstalled several times, but it seems like any time I call locationManager.requestWhenInUserAuthorization() it doesn't pop up the notification...
also any word on issue #3? seems to still be plaguing me as well...

Would you mind running the minimal code example
https://github.com/himbeles/LocationProviderExample
to see if that works for you?

Closed due to inactivity and missing feedback.

Hiya,
I want to use didChangeAuthorization and am a little confused about how to use it in conjunction with your implementation.
Do you have a code example please ?
Thanks


Update OpenCV calls to OpenCV2 style for Buster

New Raspbian installs are now using Debian Buster with OpenCV 3.2 which doesn't like the old style IplImage calls anymore, which were deprecated in OpenCV 3.0. So updated the code to use OpenCV 2 style Mat calls and this code compiles with both 2.4 in Stretch and 3.2 in Buster. I hope I haven't broken anything, as I could only test with my 120MM-S so not sure about color cameras.
Also ran through clang-format to unify the styling, causing some diff noise unfortunately.

Thanks! I used this, and it compiled successfully for me. Thank you!

Nice job Jarno! I'll test this with ASI224MC on my current Raspbian, then install Buster and test again.


Store and load models and learned weights

Pass filename of trained model to load as command line argument. Store every few training examples based on the store_every parameter in the problem definition. Also store after all training has been done.


Scanner skipping almost all files

Almost all files are skipped while scanning, Please have a look!
Thanks!

@Shafeeqts89 Thats intended behavior. It finds JAR (or WAR, EAR, ..) first and investigate it.

Can we scan .js files as well? our security team is pointing to some .js files like "log4javascript.min.js" also. I'm not sure if that is relevant or not.

@Shafeeqts89  That is not relevant at all.


Allow user to adjust routes that he intends to share

Here we see the unnamed road has been chosen, with no way available to the
user to share the way via the named road.

What is your question? Please note, that for discussion you should use our forum.

Ok, please use our forum for discussion.

OK I made https://discuss.graphhopper.com/t/allow-influencing-route-chosen/3403 .


fixed: Hamburger menu and logo overlap in smaller screens

This pull request addresses the issue where the hamburger menu and logo were overlapping on smaller screens. I have made the necessary changes to fix this bug and ensure proper alignment and display of the elements.
Changes made:

Added responsive classes to adjust the layout of the hamburger menu and logo on smaller screens.

Hey ! Please star this project :) @sudip57


[5.3] Add MailEvents for listening sent and not sent

I add two more MailEvents for notify AFTER an Email is sent or not.
The main reason ist, that I want a secured MailLog in the database and to know if a email is sent or not and it's not enought to now that the email was planed to send.
The second benefit is the result of the transport, cause Amazon SES return a MessageID for identifying the message in their system which I need to match the bounces I get per a JSON Post and to monitor the ses sending activity
The MailingNotSent event is only the missing part to make the MailEventSystem complete.
Any suggestions?

This should probably target 5.3 rather than 5.2.

cannot change the base branch on my repo, but there is no difference between 5.2 and 5.3, so it should work

https://github.com/laravel/framework/blob/v5.3.0/src/Illuminate/Mail/Mailer.php#L162-L197

my fault, looked in 5.2... so, remove result and transport?

Just hold off until Taylor has a look first. :)

What I would suggest is to define your own class in your application that you send all mail through, then in that centralized location you can do exactly what you want if the message is successfully sent or not.

thanks Taylor for your response.

What I would suggest is to define your own class in your application that you send all mail through, then in that centralized location you can do exactly what you want if the message is successfully sent or not.

Thats the way I already used it, but I thought it was a good thing for every Developer to know when an email is really sent,  and not only to know that its planed to send.


Test

This is a test of our new github action

Verified that this issue automatically had the project assigned.


Add BTRST to bridge UI

What changed? Why?
Notes to reviewers
How has it been tested?

@wbnns FYI This token is already on the STL


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-source-version-target/pull/176 is merged into live branch.
Auto archive content to https://github.com/opstest2/docs-archive-test-source-version-target.git

Docs Build status updates of commit d13f1e9:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

versioningArchiveSourceFolder/TOC.yml
:white_check_mark:Succeeded
View (sql-server-2014 || azuresqldb-current || azuresqldb-mi-current || azure-sqldw-latest || >= aps-pdw-2016 || >= sql-server-2016 || >= sql-server-linux-2017 || sql-server-previousversions || sqlallproducts-allversions)

versioningArchiveSourceFolder/docfx.json
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel


Implement MulAssign for Mat and MatMul

Same as PR https://github.com/sarah-ek/faer-rs/pull/48, I am not sure if this is the correct way of implementing these traits. Nonetheless, I have been playing around with faer a lot lately, and having these traits has helped me to prototype and test stuff easily.
If this (or PR https://github.com/sarah-ek/faer-rs/pull/48) is something that you don't want in faer-core it is OK to close this as "wont fix".

the implementation looks good to me. if the CI runs successfully then i'll be merging this. (it's fine if it complains about code coverage. it does that sometimes)


Tp2000 1025 measures tab for certificates

TP-??? Your PR title here

Why

What

Codecov Report
Attention: 19 lines in your changes are missing coverage. Please review.

Comparison is base (031a5da) 92.75% compared to head (cf32987) 92.72%.
Report is 3 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1063      +/-   ##
==========================================
- Coverage   92.75%   92.72%   -0.03%
==========================================
  Files         460      460
  Lines       34062    34174     +112
  Branches     2596     2606      +10
==========================================
+ Hits        31593    31689      +96
- Misses       1971     1987      +16
  Partials      498      498

Files
Coverage Δ

certificates/urls.py
100.00% <100.00%> (ø)

certificates/views.py
88.46% <64.15%> (-9.99%)
:arrow_down:

... and 6 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Add new rule for parsing time values.

Add following to deye_hybrid.yaml:

Grid Current L1
Grid Current L2
Grid Frequency
Time of Use Time [1 - 6]
Time of Use Power [1 - 6]
Time of Use SOC [1 - 6]
Time of Use Enabled [1 - 6]

Update customization.md with new rule as well as missing rules
This allows using the Sunsynk dashboard (https://github.com/slipx06/Sunsynk-Home-Assistant-Dash) as well as the Sunsynk Power Flow Card (https://github.com/slipx06/sunsynk-power-flow-card):

Also have a Deye 8 kW and its working great. Thanks.