Added basemap opacity slider

To get some experience with the new dashboard.py, I added a basemap opacity slider alongside the data opacity slider (as requested in issue #11).  This was relatively simple to do, though I wish it could be more atomic, with changes required at only a single location in the code, instead of three locations spread out.  I think that making that work would require some fundamental changes to Bokeh's architecture, which I'd be happy to discuss with Bokeh devs at some point, but not as a priority right now.  In any case, it's already much cleaner than having to switch to JavaScript!
The result works fine, but it's very wasteful of screen space, even after packing the two opacity sliders vertically so that they would fit on the page:

Is there any way to change the CSS (presumably) to make the packing much tighter?  There are other controls that need to be added, such as a checkbox for turning labels on and off, and the controls are already taking up 1/3 of the page, mostly with whitespace!

Closing the PR because it has been superseded by https://github.com/bokeh/datashader/pull/47


How to set basic minecraft permissions like /locate?

I couldn't find info about this anywhere. How?

/locate is not part of the Vanilla Permissions List.
Google had this to say about the command:
https://minecraft.gamepedia.com/Commands#locate
https://www.digminecraft.com/game_commands/locate_command.php

Do you mean that there is no way to set an internal minecraft permission to a group inside your plugin config?

minecraft.command.locate I found it. One dude in Sponge chat told me.

Do you mean that there is no way to set an internal minecraft command's permission to a group inside your plugin config?

Nope.  Just didn't look like Mojang included a node for it since it wasn't in the official wiki.

minecraft.command.locate I found it. One dude in Sponge chat told me.

Groovy.  :-)  Thanks for the update.


No define for OutputBucketName

https://github.com/aws-solutions-library-samples/guidance-for-multi-modal-data-analysis-with-aws-health-and-ml-services/blob/main/store-multimodal-data/medical-imaging/store-imagingdata-with-awshealthimaging.ipynb
The variable called OutputBucketName is undefined.

InputBucketName  is also undefined.


ThreadStatsd crashes when mysql replication thread is stopped

ThreadStatsd is crashing when SQL Slave is down, because slave.seconds_behind_master is Null
Exception in thread Thread-2:
Traceback (most recent call last):
File "/usr/lib/python2.7/threading.py", line 552, in __bootstrap_inner
self.run()
File "/home/ex/mysql-statsd/mysql_statsd/thread_statsd.py", line 71, in run
self.send_stat(item)
File "/home/ex/mysql-statsd/mysql_statsd/thread_statsd.py", line 55, in send_stat
sender(k, float(v))
TypeError: float() argument must be a string or a number
Here is workaround:
try:
sender(k, float(v))
except Exception, e:
print "k", k, "v", v
print e
$ python mysql_statsd/mysql_statsd.py -d -f -c ~ex/mysql-statsd.conf
k slave.seconds_behind_master v None
float() argument must be a string or a number

Hi Exus,
Thanks for reporting this bug.
It is kind of expected when a NULL is sent to the sending of metric it will crash as it explicitly expects a numeric value.
I'll ask Thijs to include the exception.
Art

Question is, is it normal for mysql to give this value? What does that mean? Can we then maybe convert the value to 0 if it's a normal value?

@thijsdezoete Yes it is the normal value. The value normally is the seconds the slave is lagging behind its master, however if replication breaks it uses NULL to determine it is broken.

Ok, and the desired action would be to not send the data(for this specific key) in the case of downtime I guess?

Oops, we merged this one a little soon, but I would still like to verify if this actually solves your issue @Exus

Yep, it's fine now
Thanks!


Is there a way to set language of returned address?

I've added a pull request to explicitly set the language a while ago but this repository seems inactive. You could checkout my PR or fork of the repository for a quick fix while waiting for the PR to go through. If you have not already found another solution.

@Johnstedt thank you for your contribution. Can you please solve the pending conflicts? I'd be very happy to merge the language support then.

@Johnstedt You can close this issue now since #11 merged.

Solution,please

@nostagy8 Read the exemple in the README.

It should be possible by passing the language as described below
Geocoder.init("xxxxxxxxxxxxxxxxxxxxxxxxx", {language : "en"});


All: Fix some outstanding indentation issues in files

New PR from my fork and with hopefully no problems this time around. (See #1045 for motivation, background, etc.)

Thanks for merging!
On Sep 21, 2016 11:18 PM, "Trent Willis"<EMAIL_ADDRESS>wrote:

Closed #1046 https://github.com/jquery/qunit/pull/1046 via e6e2ea6
https://github.com/jquery/qunit/commit/e6e2ea6ee8b21f237c302d0a8d93ed5ae09aba34
.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/jquery/qunit/pull/1046#event-798337024, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AARWekyQK9CPQ3GmfUSkc8IjF3_X9VBOks5qsgF4gaJpZM4KDeXq
.


How to update weights of CodeBERT for a downstream task?

I'm using CodeBERT for a downstream multi-class classification task. I'm training RobertaForSequenceClassification model, following #53. But this only updates the weights of the classification layer, but not the weights of CodeBERT layers.
I'm interested to evaluate if updating CodeBERT's weights (slightly) for a downstream task improves the results. Please share your thoughts.

The code updates the weights of the whole CodeBERT instead of only classification layer

Thank you for your response. I'll look into my code and check why CodeBERT's weights are not updating. I finetuned RobertaForSequenceClassification  for 5 epochs and in none of the epochs, CodeBERT's weights are updating.
Another related question - how is the loss calculated below? This is from finetuning code in #53
logits - (768,) CLS token
labels - [1, 0, 1, 0, .., 0] (one hot encode or class label for multiclass classification)
wouldn't logits and labels have different shapes?
def forward(self, input_ids=None,labels=None):
        logits=self.encoder(input_ids,attention_mask=input_ids.ne(1))[0]
        prob=torch.softmax(logits,-1)
        if labels is not None:
            loss_fct = nn.CrossEntropyLoss(ignore_index=-1)
            loss = loss_fct(logits,labels)
            return loss,prob
        else:
            return prob

Normally, CodeBERT is updated. You can use the following code to print which parameters are updated.
   for n, p in model.named_parameters():      if p.requires_grad:          print(n)
logits are (bs,labels_num) and labels are label index which shape is (bs,)


Seg fault in correcting reads

I tried wit 700 Mbp genome and got a fantastic results! Thank you!!
Now I'm now working on a plant genome of ~4 Gbp and got segmentation fault problem.
So if it is using too much memory, is there a way to save it?
The error message is like below;

/data/grc1/KanAoi/C1/PECAT/scripts/crr0_correct_1.sh: line 16: 1914431 Segmentation fault      (core dumped) /home/naitok785/miniconda3/envs/pecat/share/pecat-0.0.3-0/bin/fsa_rd_correct /data/grc1/KanAoi/C1/PECAT/1-correct/0/rd2rd.txt.sub.1.paf /data/grc1/KanAoi/C1/PECAT/0-prepare/prepared_reads.fasta /data/grc1/KanAoi/C1/PECAT/1-correct/0/corrected_reads_0.fasta.1 --output_directory=/data/grc1/KanAoi/C1/PECAT/1-correct/0 --thread_size=36 --read_name_fname=/data/grc1/KanAoi/C1/PECAT/1-correct/0/readname.core.1 --infos_fname /data/grc1/KanAoi/C1/PECAT/1-correct/0/corrected_reads_0.fasta.1.infos --score=weight:lc=15 --aligner edlib:bs=1000:mc=6 --min_coverage 4 --filter1 oh=1000:ohr=0.01 --candidate n=600:f=30 --min_identity 90 --min_local_identity 80
Plgd script end: 2023-12-13 11:03:58

the cfgfile is as below.
project=PECAT`
reads=./KanAoi_C1.fastq.gz
genome_size=4500000000
threads=36
cleanup=1
grid=local
prep_min_length=3000
prep_output_coverage=40
corr_iterate_number=1
corr_block_size=4000000000
corr_correct_options=--score=weight:lc=15 --aligner edlib:bs=1000:mc=6  --min_coverage 4 --filter1 oh=1000:ohr=0.01 --candidate n=600:f=30 --min_identity 90 --min_local_identity 80
corr_filter_options=--filter0=l=5000:al=2500:alr=0.5:aal=8000:oh=3000:ohr=0.3
corr_rd2rd_options=-x ava-ont -f 0.005 -I 10G
corr_output_coverage=40
align_block_size=12000000000
align_rd2rd_options=-X -g3000 -w30 -k19 -m100 -r500 -I 10G -f 0.002
align_filter_options=--filter0=l=5000:aal=6000:aalr=0.5:oh=3000:ohr=0.3 --task=extend --filter1=oh=300:ohr=0.03 --min_identity 0.95
asm1_assemble_options=--max_trivial_length 10000 --min_identity 0.95 --min_coverage 4 --min_contig_length 10000
phase_method=2
phase_rd2ctg_options=-x map-ont -c -p 0.5 -r 1000 -w 10 -k 19 -r 1000 -I 10G -K 8G
phase_use_reads=1
phase_phase_options= --coverage lc=10 --phase_options icr=0.1:icc=8:sc=10
phase_filter_options=--threshold 1000
phase_clair3_command=singularity exec --containall -B `pwd -P`:`pwd -P` clair3_v0.1-r12.sif /opt/bin/run_clair3.sh
phase_clair3_use_reads=0
phase_clair3_rd2ctg_options=-x map-ont -w10 -k19 -c -p 0.5 -r 1000 -I 10G -K 8G
phase_clair3_phase_options=--coverage lc=20 --phase_options icr=0.1:icc=3:sc=10 --filter i=90
phase_clair3_filter_options=--threshold=2500 --rate 0.05
phase_clair3_options=--platform=ont --model_path=/opt/models/r1041_e82_400bps_sup_v420 --include_all_ctgs
asm2_assemble_options=--reducer0 "best:cmp=2,0.1,0.1|phase:sc=3" --contig_format prialt,dual
polish_map_options=-x map-ont -w10 -k19 -I 10g -K 8G -a
polish_filter_options=--filter0 oh=2000:ohr=0.2:i=96
polish_cns_options=

hi, @drk0311.  It may be out of memory. Can you show the size of files in the  directory 1-correct/0? Blocks can be divided into smaller ones to reduce memory usage, for example corr_block_size=1000000000. You need to delete the directory 1-correct/0 first before running pecat again.

Thank you!
The PAF files in 1-corect/0 are more than 300 GB, maybe too big, right.
I'll try smaller corr_block_size option and see what happens.

Hello - I had a similar problem also using a sample with a large genome (6Gb) and using a smaller corr_block_size helped me. However, after solving this first problem I made it to crr0_correctafter 187 hours of running and 16Tb of intermediate files, but I ran out of memory again. Is there anyway to restart from where I'm at? In the scripts folder it looks like I've completed 86/212 of the crr0_correct_*.sh scripts. If the program doesn't have a built in way to restart, could I just kick off each of the remaining crr0_correct_*.sh scripts manually? I'm only interested in these precorrection steps from PECAT and it looks like I'm almost done. I really hope there is a way I can restart.

hi, @stubbsrl
PECAT can continue to run at the breakpoint using the same command pecat.pl correct cfg.  Using the cleanup=1 can reduce intermediate files.

Thank you @lemene!
I used both cleanup=1 and adding the parameter -f 0.005 to corr_rd2rd_options and align_rd2rd_options and at the peak I had 16Tb of files. It's now down to 12Tb as the program is using those large files and deleting them.

The analysis made it through 211 of the 212 blocks, but ran out of memory on one block (block #210). I copied over rd2rd.txt.sub.210.paf, prepared_reads.fasta, readname.core.210, and the crr0_correct_210.sh script to a computer with 500 gb memory. I changed the paths in the script and kicked off the analysis crr0_correct_210.sh script. It runs for 1 hour and 45 mins and still runs out of memory. I looked at the help flag for fsa_rd_correct and I tried in separate attempts adding the --use_cache flag, reducing --thread_size, and increasing --min_coverage. Any suggestions on how to get this final file to be corrected? Any other changes I can make to the crr0_correct_210.sh script? Could I split the paf somehow? My plan is once this paf turns into a fasta to move all of the #210 files back with the other corrected fastas and finish the analysis. Thanks in advanced for any suggestions!

Hi @stubbsrl
Reducing --thread_size should be helpful.  Moving files back is feasible. But you need to create file crr0_correct_210.sh.done and write 0 in the file.

Hi @lemene thanks for the feedback. I've tried various thread sizes all the way down to --thread_size 20 and I am still getting segmentation faults/core dumps. Any other suggestions?

Hi! I encountered similar error here. I'm trying to test this out with a bacteria with just 5.5mb size but got aborted core errors. I'd be very much happy to receive your inputs about this especially that I'm interested to try this program on a plant with ~400mb size.
Here's my options for correction
`project= smarcescens
reads= smarcescens_simplex.filtered.fastq
genome_size= 5500000
threads=4
cleanup=1
grid=local
prep_min_length=3000
prep_output_coverage=80
corr_iterate_number=1
corr_block_size=4000000000
corr_filter_options=--filter0=l=5000:al=2500:alr=0.5:aal=5000:oh=1000:ohr=0.1
corr_correct_options=--score=weight:lc=10 --aligner edlib --filter1 oh=1000:ohr=0.01
corr_rd2rd_options=-x ava-ont
corr_output_coverage=80`
Here's the terminal log.
2024-05-05 01:32:40 [INFO] Start Correcting 2024-05-05 01:32:40 [INFO] thread size 4, totalsize 12211 2024-05-05 01:32:40 [INFO] done 0, all 12211 free(): double free detected in tcache 2 /data/user/test_assemblers/smarcescens/scripts/crr0_correct_0.sh: line 16: 1473021 Aborted                 (core dumped) /data/user/miniconda3/envs/pecat/share/pecat-0.0.1-1/bin/fsa_rd_correct /data/user/test_assemblers/smarcescens/1-correct/0/rd2rd.txt.sub.0.paf  /data/user/test_assemblers/smarcescens/0-prepare/prepared_reads.fasta  /data/user/test_assemblers/smarcescens/1-correct/0/corrected_reads_0.fasta.0 --output_directory=/data/user/test_assemblers/smarcescens/1-correct/0 --thread_size=4 --read_name_fname=/data/user/test_assemblers/smarcescens/1-correct/0/readname.core.0 --infos_fname /data/user/test_assemblers/smarcescens/1-correct/0/corrected_reads_0.fasta.0.infos --score=weight:lc=10 --aligner edlib --filter1 oh=1000:ohr=0.01 Plgd script end: 2024-05-05 01:32:41 2024-05-05 01:32:43 [Error] Failed to run correcting reads 0, crr0
Here's the /usr/bin/time -v output
Command exited with non-zero status 1         Command being timed: "pecat.pl correct cfgfile_1"         User time (seconds): 506.21         System time (seconds): 9.50         Percent of CPU this job got: 322%         Elapsed (wall clock) time (h:mm:ss or m:ss): 2:40.15         Average shared text size (kbytes): 0         Average unshared data size (kbytes): 0         Average stack size (kbytes): 0         Average total size (kbytes): 0         Maximum resident set size (kbytes): 1534148         Average resident set size (kbytes): 0         Major (requiring I/O) page faults: 0         Minor (reclaiming a frame) page faults: 4461422         Voluntary context switches: 67738         Involuntary context switches: 5650         Swaps: 0         File system inputs: 3696         File system outputs: 1768432         Socket messages sent: 0         Socket messages received: 0         Signals delivered: 0         Page size (bytes): 4096         Exit status: 1

Hi @jp-jong
From the message free(): double free detected in tcache 2, it seems that this is not a lack of memory , but a bug in PECAT. But I cannot locate it. I suggest you try the version v0.0.3.

I checked and I am using v0.0.3. Here is the output using  /usr/bin/time - v. Also note I'm just running the shell script for the one paf where I keep getting a segmentation fault and I made separate readname files as you suggested (you'll see I'm using readname.core1.210 ). The machine I'm using has 500gb mem.
/usr/bin/time -v bash crr0_correct_210_1.sh
Plgd script start: 2024-05-09 21:55:06
2024-05-09 21:55:06 [INFO] Start
2024-05-09 21:55:06 [INFO] Arguments:
  ol_fname = /mydir/pecat/rd2rd.txt.sub.210.paf
  rr_fname = /mydir/pecat/prepared_reads.fasta
  cr_fname = /mydir/pecat/corrected_reads_0.fasta.210
  filter0 = l=2000:i=0.00:al=2000:alr=0.50:oh=10000000:ohr=1.00:aal=10000000:aalr=1.00:ilid=0
  filter1 = l=0:i=0.00:al=0:alr=1.00:oh=1000:ohr=0.01:aal=10000000:aalr=1.00:ilid=0
  graph_fname =
  infos_fname = /mydir/pecat/corrected_reads_0.fasta.210.inf
os
  output_directory = /mydir/pecat
  read_name =
  read_name_fname = /mydir/pecat/readname.core1.210
  aligner = edlib:bs=1000:mc=6
  score = weight:lc=10
  candidate = c=80:n=600:f=30:p=0.95:ohwt=0.1
  min_identity = 90
  min_local_identity = 80
  thread_size = 70
  min_coverage = 4
  variants =
  use_cache = 0
  debug = 0
  debug_flag = 0
  skip_branch_check = 0

2024-05-09 21:55:06 [INFO] Memory: 9124
2024-05-09 22:10:49 [INFO] Load<PHONE_NUMBER> overlaps from file /mydir/pecat/rd2rd.txt.sub.210.paf
2024-05-09 22:16:16 [INFO] Load 18244710 reads from file: /mydir/pecat/prepared_reads.fasta
2024-05-09 22:16:17 [INFO] Memory: 194866120
2024-05-09 22:57:00 [INFO] BuildIndex: memory=224501524
2024-05-09 22:57:00 [INFO] Memory: 224501524
2024-05-09 22:57:00 [INFO] Start Correcting
2024-05-09 22:57:00 [INFO] thread size 70, totalsize 33965
2024-05-09 22:57:00crr0_correct_210_1.sh: line 16: 1544632 Segmentation fault (core dumped) /mydir/miniconda3/envs/pecat-env/share/pecat-0.0.3-0/bin/fsa_rd_correct /mydir/pecat/rd2rd.txt.sub.210.paf /mydir/pecat/prepared_reads.fasta /mydir/pecat/correct
ed_reads_0.fasta.210 --output_directory=/mydir/pecat --threa
d_size=70 --read_name_fname=/mydir/pecat/readname.core1.210
--infos_fname /mydir/pecat/corrected_reads_0.fasta.210.infos
 --score=weight:lc=10 --aligner edlib:bs=1000:mc=6 --min_coverage 4 --filter1 oh=1000:ohr=0.01 --can
didate n=600:f=30 --min_identity 90 --min_local_identity 80
Plgd script end: 2024-05-09 22:57:15
 Command being timed: "bash crr0_correct_210_1.sh"
 User time (seconds): 10257.79
 System time (seconds): 1073.89
 Percent of CPU this job got: 303%
 Elapsed (wall clock) time (h:mm:ss or m:ss): 1:02:08
 Average shared text size (kbytes): 0
 Average unshared data size (kbytes): 0
 Average stack size (kbytes): 0
 Average total size (kbytes): 0
 Maximum resident set size (kbytes): 224502344
 Average resident set size (kbytes): 0
 Major (requiring I/O) page faults: 139
 Minor (reclaiming a frame) page faults: 662291363
 Voluntary context switches: 48761
 Involuntary context switches: 1138676
 Swaps: 0
 File system inputs:<PHONE_NUMBER>
 File system outputs: 8
 Socket messages sent: 0
 Socket messages received: 0
 Signals delivered: 0
 Page size (bytes): 4096
 Exit status: 0

@stubbsrl I still think that this error seems to be caused by the lack of memory. Is this machine running other programs simultaneously?  You can try using fewer threads, such as -- thread_size=10, and reducing the number of reads in the file readname.core1.210, such as only containing 100 reads.  If it still fails, please add the parameter '--debug', and keep only one read in  readname.core1.210, and run  bash crr0_correct_210_1.sh > debug_log. debug_log  will contain more information to help locate the bug.

@lemene the v0.0.3 seems to work without errors. Thanks!!

@stubbsrl
Have the files 1-correct/0/corrected_reads_X.fasta been deleted?  Can you show me what other files are in the directory  1-correct/0/?

No I still have the corrected reads fastas.
Inside the 1-correct/0/ dir I have: corrected_reads_0.fasta.X, corrected_reads_0.fasta.X.infos, and rd2rd.txt. Yesterday when I tried to restart pecat it made readname.core.X and rd2rd.txt.sub.X.paf in that dir.

crr0_split.sh is the previous step and should not be executed again, otherwise the corrected reads (corrected_reads_0.fasta.X) will be deleted. The time of the file may have been incorrect, and PECAT did not accurately recognize the current status. If you don't want to discard the corrected reads, it's best to manually execute the scripts.

oh wow I'm very glad corrected reads didn't get deleted. So, what is the next script that I should execute manually? Everything in the scripts dir has a done file with it so nothing jumps out to me.

Each file crr0_correct_X.sh should be executed and the corresponding files corrected_reads_0.fasta.X, corrected_reads_0.fasta.X.infos will be generated. corrected_reads_0.fasta.X are corrected reads. They are concatenated to a single file as the output of the correction step.  The  crr0_correct_X.sh requires file rd2rd.txt.sub.X.paf.  rd2rd.txt.sub.X.paf are generated from ccX.fasta.paf by crr0_split.sh.  Is rd2rd.txt.sub.X.paf  deleted? It's a bit cumbersome to regenerate them.
Following are scripts  in execution order and  the associated input and output files.
crr0_rd2rd_split.sh: ../../0-prepare/prepared_reads.fasta -> ccX.fasta
crr0_rd2rd_index.sh:  ../../0-prepare/prepared_reads.fasta -> reads.index
crr0_rd2rd_align_X.sh: ccX.fasta , reads.index-> ccX.fasta.paf
crr0_split.sh:  ccX.fasta.paf -> rd2rd.txt.sub.X.paf, readname.core.X
crr0_correct_X.sh: rd2rd.txt.sub.X.paf, readname.core.X ->  corrected_reads_0.fasta.X, corrected_reads_0.fasta.X.infos
crr0_cat.sh: corrected_reads_0.fasta.X, -> ../corrected_reads.fasta
To regenerate rd2rd.txt.sub.X.paf,  all  ccX.fasta.paf are required，which means  you need to redo the alignment between the reads.
`

Hi, I am the guy who opened this issue.
The cause of core dump or segmentation fault issues often come from organelle genome (chloroplast and mitochondrion).
So if you are working with a plant genome of quite a large size (>3 Gbp), you'd better remove the reads derived from organelle genomes from your fastq files.  Maybe your fastq files contain more then 10,000x of the chloroplast genome!!
So, if the reference organelle genomes are available, map your reads to it, extract unmapped reads or those mapped only <50% of its read length, and then run PECAT.


Emscripten support

Sharing my experimental Emscripten target support. Try it out:
https://snart.me/demos/emscripten/ft2-clone/
vær så god!

I mean, cool and all, but the mouse pointer has coordinate mapping issues for me on your web test link. It makes it quite unusable so far...

As long as it's a nice and simple patch that doesn't change the behavior of the FT2 clone on other platforms, I'll merge it eventually if you manage to iron out the bugs. :)


Fix "⌨Add autocomplete widget with command line completion"

Hey @NBTX,
Your changes look good to me!
View build log
TravisBuddy Request Identifier: 250848a0-cd6d-11e9-a9d8-278aeb0f3435


fix(shuttle): catch redis 'ERR no such key'

Motivation
This bug prevents shuttle from starting properly.
Change Summary
The command to check if a group exists can throw an "ERR no such key" if it hasn't been created yet. This fix catches the error and ignores it which lets the next step of creating the group take place.
Merge Checklist
Choose all relevant options below by adding an x now or at any time before submitting for review

[x] PR title adheres to the conventional commits standard
[ ] PR has a changeset
[ ] PR has been tagged with a change label(s) (i.e. documentation, feature, bugfix, or chore)
[ ] PR includes documentation if necessary.
[x] All commits have been signed

Additional Context

PR-Codex overview
This PR enhances the createGroup method in eventStream.ts by adding error handling for group existence and creation.
Detailed summary

Added error handling to check if the group already exists
Improved error handling for cases where the key doesn't exist or group creation fails

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

Thanks @kengoldfarb! Took your commit and added a changeset in #2133.
Next time you can include the changeset yourself by running yarn changeset. Thanks for the fix!


P1 analysis for decay analysis

Summary
This PR upgrades Decay common analysis class to support single point scan.
Details and comments
If one sets the analysis option raw_data_mode=True the analysis class will skip the curve fit and store each single data point as individual analysis result entry under the name "MyExperiment_single_point".
Note that current implementation is really inefficient, because of tight coupling of data processing to analysis. This should be updated after the refactoring of analysis so that it can call data processing and _run_analysis as consecutive callbacks, i.e. separation of processing from analysis.

This is also relevant to #536

What's special in Decay compared to other types of analysis? We could have a new analysis class, inheriting directly from CurveAnalysis, that outputs the raw data.

Because decay is monotonic to xval. I've never seen P1 type experiment with non-monotonic data. Probably @oliverdial knows other examples.

Thanks! The background of this work is that some people wanted to monitor only single (or few) point of T1 curve. We have other examples such as Stark shift spectroscopy https://arxiv.org/abs/2105.15201, or survival probability of RB as a cost function of heavy optimization task https://arxiv.org/abs/2003.05952
However I've never seen this approach is used with non-decay type experiment. So I believe it should be sufficient to support this analysis (i.e. omit fit and return data point) only for decay analysis.

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


don't make Logger write empty strings

Don't bother calling write (which will trigger a syscall) for nothing

http://jenkins01.arangodb.biz:8080/view/PR/job/arangodb-matrix-pr/2410/


Blender Cask fails install

A couple days ago I contributed PR #21580, which included Hbc.appdir, today @vitorgalvao contributed a00f4be3fdb42c22bcc0af05b22578e27ec0dbc0 which replaced Hbc.appdir with plain appdir. With this change I receive the following when trying to install:
==> Downloading https://download.blender.org/release/Blender2.77/blender-2.77-OSX_10.6-x86_64.zip
######################################################################## 100.0%
==> Verifying checksum for Cask blender
Warning: Unexpected method 'appdir' called during preflight on Cask blender.

  Most likely, this means you have an outdated version of homebrew-cask. Please run:

      brew uninstall --force brew-cask; brew untap phinze/cask; brew update; brew cleanup; brew cask cleanup

  If this doesn’t fix the problem, please report this bug:

      https://github.com/caskroom/homebrew-cask#reporting-bugs

Warning: Unexpected method 'appdir' called during preflight on Cask blender.

  Most likely, this means you have an outdated version of homebrew-cask. Please run:

      brew uninstall --force brew-cask; brew untap phinze/cask; brew update; brew cleanup; brew cask cleanup

  If this doesn’t fix the problem, please report this bug:

      https://github.com/caskroom/homebrew-cask#reporting-bugs

==> Moving App 'blender.app' to '/Applications/Blender.app'
==> Moving App 'blenderplayer.app' to '/Applications/Blenderplayer.app'
==> Symlinking Binary 'blenderwrapper' to '/usr/local/bin/blender'

Which breaks the shim script. I changed it back to my PR and it seemed to install fine. I'm raising this as an issue because I don't know why it breaks or how it's supposed to work. I also tried running that command and it had no effect.
 I was changing a bunch of casks with staged_path and got some with this change without thinking. appdir isn’t available inside preflight (and others) like staged_path. Apologies for the breakage, will fix now.

Fixed. Thank you for the report.

Thanks!


filter instances by state

I'd like to be able to find all instances sometimes like asg-instances --pending.

duplicate of #63


Container algorithms

added container based parallel::copy
fixed return types for parallel::copy
fixed seqmented parallel::copy for new return types
fixed parallel::replace, parallel::rotate, etc. for new return types for copy
added range based parallel::replace, added tests
added projections to parallel::replace
added range based parallel::remove_copy[_if] , added tests
changed return types for parallel::reverse
added range base parallel::reverse[_copy]

This solves part of #1668

Please note that the adaptation of our algorithms to the newer range proposal causes breaking changes related to some of the return types of the algorithms. This will be documented separately.


Refactor: Reorder functions and add comments to vm's class

rename builtIn -> builtin, BuiltIn -> Builtin for consistency

Reorder functions and add comments to boolean.go
Reorder functions and add comments to array.go
Reorder functions and add comments to channel.go
Reorder functions and add comments to error.go
Reorder functions and add comments to file.go
Reorder functions and add comments to go_object.go
Reorder functions and add comments to hash.go
Reorder functions and add comments to http.go
Reorder functions and add comments to integer.go
Reorder functions and add comments to json.go
Reorder functions and add comments to method.go
Reorder functions and add comments to null.go
Reorder functions and add comments to object.go
Reorder functions and add comments to plugin.go
Reorder functions and add comments to range.go
Fix comments
Reorder functions and add comments to simple_server.go
Reorder functions and add comments to string.go
Reorder functions and add comments to url.go
Fix comments
Change BuiltIn -> Builtin, builtIn -> builtin for consistency

@hachi8833 Can you also write a short note on wiki about how to organize and comment functions? And then put the link on contributing guideline.

@st0012 Add the description for organizing functions at the wiki https://github.com/goby-lang/goby/wiki/Contibuting-a-Method#2-keep-the-files-organized-and-add-comments
To show the screenshots on the wiki page, the PR, which contains screenshots under wiki/ dir, should be merged.
Thanks,

Ok, but can you fix those warning first?


Hidden details icon in mobile view and details is empty in case of composite properties in field

Here is sample of my code:
	grid = $("#grid").grid({
		primaryKey: "Name",
		uiLibrary: "bootstrap",
		notFoundText: "No records found",
		detailTemplate: '

',
		responsive: true,
		showHiddenColumnsAsDetails: true,
		columns: [
			{ field: "Name", title: "Name", minWidth: 150, priority: 1 },
			{ field: "Purpose", title: "Purpose", minWidth: 50, priority: 4 },
			{ field: "Description", title: "Description", priority: 5 },
			{ field: "Enabled", title: "Enabled", minWidth: 30, priority: 2 },
			{ field: "Time", title: "Time", tmpl: '{StartTime} to {EndTime}', minWidth: 150, priority: 3 },
		],
		pager: { enable: true, limit: 5, sizes: [2, 5, 10, 20] }
	});

In case of multiple properties in a single field ("Time"), the expanded detail is empty.
Responsive works fine upto a certain width. But in mobile view, the details icon is hidden. So there is no way to expand the details section.

Hello,
The issue with multiple properties in a single field exists and I will work on it in the new version.
The details icon shows in mobile view. You can see that in an example similar to your code at http://gijgo.com/Temp/37eb84b7-7ec0-485e-be57-06a97f7e8210.html
Could you check for some other javascript errors in your page. The display of this icon could be affected by something else.
Please use priority only with combination of minWidth. In the current version you need to have both in order to get it work.
Thanks for the feedback.
Best Regards,
Atanas


[Feature] Add TotalCount property to PageResult

Description
Describe the feature you would like to request:
[Please provide a clear and concise description of the feature you're requesting.]
Use Case
What problem would this feature solve?
[Explain the specific use case or problem that this feature would address.]
Proposed Solution
Describe the solution you'd like:
[Provide details about how you envision this feature being implemented in the Go wrapper.]
Additional Context
Any additional context or information:
[Include any additional information, context, or examples that might help in understanding or implementing the feature.]
Related Issues
Are there any existing issues or pull requests related to this feature request?
[If applicable, link to any existing issues or pull requests that are related to this feature request.]
Version
Do you have a specific version in mind for this feature?
[If you have a preference for which version of the Go wrapper should include this feature, please specify.]
Checklist
Before submitting this feature request, please make sure you have completed the following:

[ ] I have searched for similar feature requests in the repository's issue tracker.
[ ] I have checked the latest version of the Go SDK to see if this feature already exists.
[ ] I have provided a clear and descriptive title for this feature request.
[ ] I have provided a detailed description of the feature and its use case.
[ ] I have provided any relevant additional context or examples.
[ ] I have linked any related issues or pull requests.

Thank you for your contribution!

Possible duplicate of #63


running just 'iptb' should show subcommands

Current:
$ iptb
usage: iptb [<flags>] [<args>...]

Flags:
      --help              Show context-sensitive help (also try
                          --help-long and --help-man).
  -n, --n=N               number of ipfs nodes to initialize
  -p, --port=4002         port to start allocations from
  -f, --force             force initialization (overwrite existing
                          configs)
      --mdns              turn on mdns for nodes
      --bootstrap="star"  select bootstrapping style for cluster
      --utp               use utp for addresses
      --wait              wait for nodes to come fully online before
                          exiting

Args:
  [<args>]  arguments

I need to run something with the beginning of a subcommand to get the full help, e.g.
$ iptb e

Also, the subcommand instructions are being written to stdout, while the main usage re: arguments is being written to stderr -- everything should probably go to the latter.

done


bug: Delete last list item in collection-repeat fails

Type: bug
Platform: mobile webview
The new collection-repeat doesn't delete the last item in the list even though the list items array is empty.
I have an array with several items that are shown in the list by means of collection-repeat. When I delete whatever item but the last it works. But if I delete the last element, the item is still there until I tap or try to scroll.
I think it could be some view refresh that isn't executed or something like this.
I solved it by adding:

$ionicScrollDelegate.$getByHandle('listScroll').resize();

or (if deleting is done asynchronously):

$timeout(function() {
$ionicScrollDelegate.$getByHandle('listScroll').resize();
}, 150);

Fixing.

Here's the crazy logic we need:
    function makeChangeValidator() {
      var self;
      return (self = {
        dataLength: 0,
        width: 0,
        height: 0,
        resizeRequiresRefresh: function(newWidth, newHeight) {
          var requiresRefresh = self.dataLength &&
            newWidth && newWidth !== self.width &&
            newHeight && newHeight !== self.height;

          self.width = newWidth;
          self.height = newHeight;

          return !!requiresRefresh;
        },
        dataChangeRequiresRefresh: function(newData) {
          var requiresRefresh = newData.length > 0 || newData.length < self.dataLength;

          self.dataLength = newData.length;

          return !!requiresRefresh;
        }
      });
    }

Now all the cases are covered cleanly.

Thank you so much! You were so fast resolving it ;)


execute the transaction is an error

In general, there are many minor errors, even I can't enter 2fa because I can't enter the number because

I can't enter the number.

Do you still have the Google Authenticator entry? This "Recover" feature is meant to be used when you have lost it, therefore it wouldn't make sense to ask you to enter the authenticator code.
This particular error you have shown indicates the browser deleted part of the information stored for this wallet. This could happen when disk space is low and the browser silently cleans up some stored data to free up disk space. This may also happen for some older wallets or Safari iOS, which I intend to fix later. It seems your case falls under the former situation.

I am closing this issue since it is repetitious to #148


Element for Props Component

I'd like to propose to use the HTML  element for each entry in the list view of the Props component in gatsby-theme-docz.
This would bring a standardized as well as accessible markup structure which could even be extended by the use a
 for the whole props list.
Example

id

      string

The id global attribute defines an identifier (ID) which must be unique in the whole document. Its purpose is to identify the element when linking (using a fragment identifier), scripting, or styling (with CSS).

class

      string

A space-separated list of the classes of the element.

See this pen for 👉demo
Small print

Browser support includes all modern browsers except IE Edge
Support for IE Edge (and older) could be added through a polyfill
Degrades gracefully: Not supporting browsers will treat <details and summary elements as div so the content will be visible (and will have an appearance like the opened version)
Safari seems to have an issue with updating the open state (repaint occurs only after blur) when using pseudo element for caret

I'd like to implement that as soon as I find some spare time and I'm always open to feedback :)

In my opinion a polyfill for Edge would be to much. Without one the list is just displayed as opened and does not prevent the usage of it.
The polyfill would just be additional code to be maintained which hopefully will be unused within a few updates of Edge.

Hi there,
little status update before christmas:
The desktop view turned out quite good whereas the mobile view comes with some challenges.
Tricky is the part where on mobile the arrow is not next to the label but next to the prop type since this is how details works.
Would you mind if we would keep it like that for now or does it have big impact on usability?
I also tried using a label for  but that seems not to be supported…
Desktop

Mobile

Merry Christmas to you all! 🎁


NullPointerException

Hi,
It seems like I can't get jenkins-discord to work, at all. I built the .hpi file using mvn package, uploaded it to Jenkins and set up the Post built actions as described (http://i.imgur.com/aTlv5Dx.png).
However, when I schedule a build, it seems like jenkins-discord can't send the notification: when the build is finished and Jenkins starts processing the Post-Build actions, Java throws a NullPointerException:
ERROR: Build step failed with exception
java.lang.NullPointerException
	at nz.co.jammehcow.jenkinsdiscord.WebhookPublisher.perform(WebhookPublisher.java:42)
	at hudson.tasks.BuildStepMonitor$1.perform(BuildStepMonitor.java:20)
	at hudson.model.AbstractBuild$AbstractBuildExecution.perform(AbstractBuild.java:735)
	at hudson.model.AbstractBuild$AbstractBuildExecution.performAllBuildSteps(AbstractBuild.java:676)
	at hudson.model.Build$BuildExecution.cleanUp(Build.java:196)
	at hudson.model.Run.execute(Run.java:1782)
	at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43)
	at hudson.model.ResourceController.execute(ResourceController.java:97)
	at hudson.model.Executor.run(Executor.java:405)
Build step 'Discord Notifier' marked build as failure
Finished: SUCCESS

How should I fix that?
Thanks!

I'm a bit busy at the moment but I have a rough idea what it might be. Is your Jenkins URL in your global config set?

With your latest commit there is no more NullPtrException, however the plugin notifies me about 'Your Jenkins URL is not set (or is set to localhost)!' although it is correctly set imho: http://i.imgur.com/Y09ws6o.png
Thanks again.

I would try removing the URL from the field, saving it, then putting it back in. It must be returning null for the URL which should only happen if it's the loopback IP. Try a quick restart and if nothing works return back.

Weird. Might put in a feature to fix that if it's a prominent issue.
Enjoy.

Hmnnnn... I wonder it this is related to my issue? I can't get it to work at all either.

Hello, melt7777 here.
We keep getting this in our console output: I do not know what else to provide you for troubleshooting. I see this should have been fixed long ago. After this error, nothing is posted to the discord. It happens intermittently (the worst case to troubleshoot!)
ERROR: Build step failed with exception
java.lang.NullPointerException
	at nz.co.jammehcow.jenkinsdiscord.WebhookPublisher.perform(WebhookPublisher.java:102)
	at hudson.tasks.BuildStepMonitor$1.perform(BuildStepMonitor.java:20)
	at hudson.model.AbstractBuild$AbstractBuildExecution.perform(AbstractBuild.java:744)
	at hudson.model.AbstractBuild$AbstractBuildExecution.performAllBuildSteps(AbstractBuild.java:690)
	at hudson.model.Build$BuildExecution.cleanUp(Build.java:196)
	at hudson.model.Run.execute(Run.java:1866)
	at hudson.model.FreeStyleBuild.run(FreeStyleBuild.java:43)
	at hudson.model.ResourceController.execute(ResourceController.java:97)
	at hudson.model.Executor.run(Executor.java:429)
Build step 'Discord Notifier' marked build as failure
Finished: SUCCESS

We are using latest jenkins, and the latest Release version of the Discord Notifier "v1.4.1 - Enum result
@KocproZ KocproZ released this 5 days ago · 4 commits to master since this release"
Also, we do have the URL set properly in Jenkins settings. Any advice?


Web Inconsistency: In some cases, text is unexpectedly collapsed

As shown below, I have a couple of lines of text placed within some nodes which set flexGrow/flexShrink. One of the lines is unexpectedly invisible. Sorry I don't have a better description of the problem.
Here's a relatively small repro case I extracted from my team's app expressed using React Native components (there's a JSFiddle later):
<View style={{"position":"absolute","left":0,"right":0,"top":0,"bottom":0}}>
  <View style={{"alignSelf":"flex-start"}}>
    <View style={{"flexShrink":1}}>
      <Text>
        Top Text
      </Text>
    </View>
    <View>
      <View style={{"flexGrow":1}}>
        <Text>
          Bottom Text
        </Text>
      </View>
    </View>
  </View>
</View>

Expected Result
Both "Top Text" and "Bottom Text" are visible (JSFiddle: https://jsfiddle.net/pygpx2oq/):

Actual Result
Only "Bottom Text" is visible:

Reproed using a version of React Native (https://github.com/facebook/react-native/commit/4c7576e485973391d4042e694523dbfe70e49b50) which appears to be using a version of Yoga from today (March 8).

This is still reproducable in the latest RN. See https://snack.expo.io/r1V-wTceG


Using Metavoice when conda is installed?

I have conda installed for other purposes to use other parts of my application. Is it possible to make Metavoice work when conda is present?
I'm forced to keep conda. Please advise.

Hey! Yep, you can use:
poetry export -f requirements.txt --output requirements.txt
& install from that to unblock yourself until we get https://github.com/metavoiceio/metavoice-src/pull/119 into main.

Thank you so much! It was almost impossible before. It works now.


Eloquent raw query have different result and response with raw query into the database

Laravel Version
10.29.0
PHP Version
8.3.0RC1
Database Driver & Version
mysql  Ver 8.1.0 for macos13 on arm64 (MySQL Community Server - GPL)
Description
I want to select into table portfolios with a column date type of 'date'

I'm trying to select all of the portfolios rows within multiple values of 'years' like this: select * from portfolios where YEAR(date) in (2021,2012,2015,2020)

The problem is if I run the query from database client that will work but did not eloquent with same query select * from portfolios where YEAR(date) in (2021,2012,2015,2020) ;
shared error page
The conclusion :
Query works when used in database client but not in eloquent
Steps To Reproduce
the code
$portfolioByYear = DB::table('portfolios')->whereRaw('YEAR(`date`) in ?', '(2021,2012,2015,2020)')->get();
               // $portfolioByYear = Portfolio::whereRaw('YEAR(`date`) in ?', '(2021,2012,2015,2020)')->get(); have same result

Hi there,
Thanks for reporting the problem you are encountering, but it looks like this is a question which may be better suited for a support channel. We only use this issue tracker for reporting bugs with the library itself. If you have a question on how to use functionality provided by this repository you can try one of the following channels:

Laracasts Forums
Laravel.io Forums
StackOverflow
Discord
Larachat
IRC

However, this issue will not be locked and everyone is still free to discuss solutions to your problem!
Thanks.

what??, thats not  bug??. eloquent showing query and then the query is can work in the database client but the eloquent get error.


One suggestion

Maybe it would be better to integrate the pins together.
Thank you.

What do you mean?

I meant to put Pins in the same folder, Instead of being scattered in the list.

Got it, see #307.

Got it, see #307.

OK, Thank you


feat(register of applications): add search bar to applications regist…

…er page

:tada: This PR is included in version 2.15.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Improve performance of Search list

Details

this comment gives context to the PR: https://github.com/Expensify/App/issues/43396#issuecomment-2218017022
this is a bit of a test, to see whether memoization of getItemHeight function will lead to better list performance
the function getItemLayout is called multiple times by FlatList to get the sizes of items being rendered; This function is implemented inside our BaseSelectionList and in turn calls getItemHeight
in theory after memoizing it the subsequent renders of every item should be a bit lighter because the heights are unchanged once items are rendered, so we will not recall the function
this issue does not affect web in any way

results
To me results are inconclusive. I used React Profiler to profile rendering of list, then scrolling and lazy loading new items. When using the code with _.memoize rendering of whole List seems to be faster by around 2-3ms which is not a lot 🤷
I think a better test is for a few people build this and just get a feel for whether it looks good or bad.
NOTE:

during scrolling of list the Search component is not re-rendering a lot - only once when new list items are fetched, so I believe we should not be putting any improvements or memoization to Search component itself.

CC @luacmartins
Fixed Issues

$ https://github.com/Expensify/App/issues/43396
PROPOSAL:
Tests

open Search on mobile
scroll down multiple times
see if this PR improves the problems from original issue connected

Offline tests

QA Steps

open Search on mobile
scroll down multiple times
see if this PR improves the problems from original issue connected

PR Author Checklist

[x] I linked the correct issue in the ### Fixed Issues section above
[x] I wrote clear testing steps that cover the changes made in this PR

[x] I added steps for local testing in the Tests section
[x] I added steps for the expected offline behavior in the Offline steps section
[x] I added steps for Staging and/or Production testing in the QA steps section
[x] I added steps to cover failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[x] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)
[x] I tested this PR with a High Traffic account against the staging or production API to ensure there are no regressions (e.g. long loading states that impact usability).

[x] I included screenshots or videos for tests on all platforms
[x] I ran the tests on all platforms & verified they passed on:

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

[x] I verified there are no console errors (if there's a console error not related to the PR, report it or open an issue for it to be fixed)
[x] I followed proper code patterns (see Reviewing the code)

[x] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick)
[x] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[x] I verified that comments were added to code that is not self explanatory
[x] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[x] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method

[x] If any non-english text was added/modified, I verified the translation was requested/reviewed in #expensify-open-source and it was approved by an internal Expensify engineer. Link to Slack message:

[x] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[x] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[x] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[x] I verified the JSDocs style guidelines (in STYLE.md) were followed

[x] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[x] I followed the guidelines as stated in the Review Guidelines
[x] I tested other components that can be impacted by my changes (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar are working as expected)
[x] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[x] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[x] I verified that if a function's arguments changed that all usages have also been updated correctly
[x] If any new file was added I verified that:

[x] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[x] If a new CSS style is added I verified that:

[x] A similar style doesn't already exist
[x] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG))

[x] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[x] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[x] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[x] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[x] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[x] I verified that all the inputs inside a form are aligned with each other.
[x] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[x] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[x] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.

Screenshots/Videos

Android: Native
https://github.com/Expensify/App/assets/3929868/6a016de3-f60f-4746-83df-0a3c8c14d5ad

Android: mWeb Chrome

iOS: Native
https://github.com/Expensify/App/assets/3929868/da65b9f8-de70-4b10-b0be-2c632cc54523

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

@luacmartins Yes checking

The performance slightly improved, but I still see blank space when fast scroll
https://github.com/Expensify/App/assets/41129870/0950844d-c944-44bd-bc9a-c072745d06ce
https://github.com/Expensify/App/assets/41129870/9e8fba63-bc0b-4c3e-978e-fd769639b83a

@luacmartins I still see blank space when fast scroll for the first render. videos above ☝

I think that's fine for now. I'll take the small performance improvement while we still investigate other solutions.

Reviewer Checklist

[ ] I have verified the author checklist is complete (all boxes are checked off).
[ ] I verified the correct issue is linked in the ### Fixed Issues section above
[ ] I verified testing steps are clear and they cover the changes made in this PR

[ ] I verified the steps for local testing are in the Tests section
[ ] I verified the steps for Staging and/or Production testing are in the QA steps section
[ ] I verified the steps cover any possible failure scenarios (i.e. verify an input displays the correct error message if the entered data is not correct)
[ ] I turned off my network connection and tested it while offline to ensure it matches the expected behavior (i.e. verify the default avatar icon is displayed if app is offline)

[ ] I checked that screenshots or videos are included for tests on all platforms
[ ] I included screenshots or videos for tests on all platforms
[ ] I verified tests pass on all platforms & I tested again on:

[ ] Android: Native
[ ] Android: mWeb Chrome
[ ] iOS: Native
[ ] iOS: mWeb Safari
[ ] MacOS: Chrome / Safari
[ ] MacOS: Desktop

[ ] If there are any errors in the console that are unrelated to this PR, I either fixed them (preferred) or linked to where I reported them in Slack
[ ] I verified proper code patterns were followed (see Reviewing the code)

[ ] I verified that any callback methods that were added or modified are named for what the method does and never what callback they handle (i.e. toggleReport and not onIconClick).
[ ] I verified that the left part of a conditional rendering a React component is a boolean and NOT a string, e.g. myBool && <MyComponent />.
[ ] I verified that comments were added to code that is not self explanatory
[ ] I verified that any new or modified comments were clear, correct English, and explained "why" the code was doing something instead of only explaining "what" the code was doing.
[ ] I verified any copy / text shown in the product is localized by adding it to src/languages/* files and using the translation method
[ ] I verified all numbers, amounts, dates and phone numbers shown in the product are using the localization methods
[ ] I verified any copy / text that was added to the app is grammatically correct in English. It adheres to proper capitalization guidelines (note: only the first word of header/labels should be capitalized), and is either coming verbatim from figma or has been approved by marketing (in order to get marketing approval, ask the Bug Zero team member to add the Waiting for copy label to the issue)
[ ] I verified proper file naming conventions were followed for any new files or renamed files. All non-platform specific files are named after what they export and are not named "index.js". All platform-specific files are named for the platform the code supports as outlined in the README.
[ ] I verified the JSDocs style guidelines (in STYLE.md) were followed

[ ] If a new code pattern is added I verified it was agreed to be used by multiple Expensify engineers
[ ] I verified that this PR follows the guidelines as stated in the Review Guidelines
[ ] I verified other components that can be impacted by these changes have been tested, and I retested again (i.e. if the PR modifies a shared library or component like Avatar, I verified the components using Avatar have been tested & I retested again)
[ ] I verified all code is DRY (the PR doesn't include any logic written more than once, with the exception of tests)
[ ] I verified any variables that can be defined as constants (ie. in CONST.js or at the top of the file that uses the constant) are defined as such
[ ] If a new component is created I verified that:

[ ] A similar component doesn't exist in the codebase
[ ] All props are defined accurately and each prop has a /** comment above it */
[ ] The file is named correctly
[ ] The component has a clear name that is non-ambiguous and the purpose of the component can be inferred from the name alone
[ ] The only data being stored in the state is data necessary for rendering and nothing else
[ ] For Class Components, any internal methods passed to components event handlers are bound to this properly so there are no scoping issues (i.e. for onClick={this.submit} the method this.submit should be bound to this in the constructor)
[ ] Any internal methods bound to this are necessary to be bound (i.e. avoid this.submit = this.submit.bind(this); if this.submit is never passed to a component event handler like onClick)
[ ] All JSX used for rendering exists in the render method
[ ] The component has the minimum amount of code necessary for its purpose, and it is broken down into smaller components in order to separate concerns and functions

[ ] If any new file was added I verified that:

[ ] The file has a description of what it does and/or why is needed at the top of the file if the code is not self explanatory

[ ] If a new CSS style is added I verified that:

[ ] A similar style doesn't already exist
[ ] The style can't be created with an existing StyleUtils function (i.e. StyleUtils.getBackgroundAndBorderStyle(theme.componentBG)

[ ] If the PR modifies code that runs when editing or sending messages, I tested and verified there is no unexpected behavior for all supported markdown - URLs, single line code, code blocks, quotes, headings, bold, strikethrough, and italic.
[ ] If the PR modifies a generic component, I tested and verified that those changes do not break usages of that component in the rest of the App (i.e. if a shared library or component like Avatar is modified, I verified that Avatar is working as expected in all cases)
[ ] If the PR modifies a component related to any of the existing Storybook stories, I tested and verified all stories for that component are still working as expected.
[ ] If the PR modifies a component or page that can be accessed by a direct deeplink, I verified that the code functions as expected when the deeplink is used - from a logged in and logged out account.
[ ] If the PR modifies the UI (e.g. new buttons, new UI components, changing the padding/spacing/sizing, moving components, etc) or modifies the form input styles:

[ ] I verified that all the inputs inside a form are aligned with each other.
[ ] I added Design label and/or tagged @Expensify/design so the design team can review the changes.

[ ] If a new page is added, I verified it's using the ScrollView component to make it scrollable when more elements are added to the page.
[ ] If the main branch was merged into this PR after a review, I tested again and verified the outcome was still expected according to the Test steps.
[ ] I have checked off every checkbox in the PR reviewer checklist, including those that don't apply to this PR.

Screenshots/Videos

Android: Native

Android: mWeb Chrome

iOS: Native

iOS: mWeb Safari

MacOS: Chrome / Safari

MacOS: Desktop

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by https://github.com/luacmartins in version: 9.0.7-3 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Cherry-picked to staging by https://github.com/thienlnam in version: 9.0.7-4 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

@Expensify/applauseleads please QA this PR and check it off on the deploy checklist if it passes.

🚀 Deployed to production by https://github.com/thienlnam in version: 9.0.7-8 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to production by https://github.com/mountiny in version: 9.0.8-6 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


 Now that you've assigned yourself, people who drop by know that they're welcome to participate, but you'll be carrying this issue across the finish line. :sunglasses:.
Let's use GitHub Pages
Now, on to business! GitHub Pages allow you to serve a static site from a repository. We've filled this repository with some site content, but the rendered site isn't visible right now. Let's change that.
:keyboard: Activity: Enable GitHub Pages

Click on the Settings tab in this repository
Scroll down to the "GitHub Pages" section
From the "Source" drop down, select main branch

Note: Even though you'll see an option to choose a theme, do not apply a theme at this point. We've protected the code so you can't make unintended changes. You'll have the opportunity to apply a theme when the course is completed.

I'll respond in this issue when I detect GitHub Pages has deployed your site.

Turning on GitHub Pages creates a deployment of your repository. I may take up to a minute to respond as I await the deployment.

Originally posted by @github-learning-lab[bot] in https://github.com/ShahimaShahazada/master/issues/1#issuecomment-988763721

Hi


Update dg

Codecov Report

Merging #184 (21621d1) into master (dca65a6) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #184   +/-   ##
=========================================
  Coverage     72.04%   72.04%
  Complexity      755      755
=========================================
  Files           102      102
  Lines          2332     2332
  Branches        261      261
=========================================
  Hits           1680     1680
  Misses          561      561
  Partials         91       91

Continue to review full report at Codecov.

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


Editor pop-up menu unreadable black background

From<EMAIL_ADDRESS>on 2015-02-14T12:10:20Z
Please note that reporting bugs from Spyder's help menu ("?" > "Report
issue...") will fill version details for Spyder, Python and Qt in this description for you.
What steps will reproduce the problem?

Modify Spyder editor syntax coloring prefs 2. 3.

What is the expected output? What do you see instead?
Pop-up menu should have white background /or/ invert against text colors in pop-up.
Instead, background is black, text is black, pop-up commands thus unreadable What version of the product are you using? On what operating system? Spyder 2.3.2 for Python 3.4 on Mac OS 10.10
Please provide any additional information below
.
Original issue: http://code.google.com/p/spyderlib/issues/detail?id=2174

From ccordoba12 on 2015-02-15T15:20:15Z
Could you post a screenshot please? I really don't understand what you mean.

Closing because of lack of further info


make tracked? overridable

Hey there,
I am looking for a possibility to have more control over tracking.
This was the simplest approach I could think of to make it work.
I got a plug that I call where I want to track and simply add a enabled?: true to the current process directory.
Then i override the tracked? function to look for that key.
Any thoughts on this?

Hi there,
I'm working on the same code as @nduitz. Just want to add a few thoughts here.
Our use-case might be a bit special. Making tracked?/1 overridable also has the implication that users could implement schema deny logic instead of allow logic, as the application config currently allows.


[WIP] Modified overlay agent module behavior for handling Docker failures.

This PR changes the failure handling of the overlay agent module. The overlay agent modules no longer fails hard on an unsuccessful overlay configuration.
Issues
DCOS-10952
...
Checklist

[* ] Included a test which will fail if code is reverted but test is not
[*] Followed relevant code rules Rules for Packages and Systemd

If this is a component/package update, include

[ ] Change Log from last:
[ ] Test Results:
[ ] Code Coverage:

This is waiting PR #1100  since it needs the latest mesos to work.

@asridharan Please rebase on latest #1100 which includes the latest Mesos master

@asridharan It looks like these overlay modules changes were already merged into dcos-mesos-modules and will be included in train 118 ( #1131 ), from #1108 (dcos-mesos-modules bump) along with #1100 for the Mesos bump. Is there anything else we'd need (config changes, etc.)?

@adam-mesos you are right. #1108 already has these changes. Will go ahead and close this PR for the time being.


Do not clober group membership

The group inside the weblogic resource clobbers group members. It replaces the list of members with an empty set. oracle-inventory::default then appends a user which results in the group resource always running in both locations on every chef run.
  * weblogic[<IP_ADDRESS>.0] action create
    * group[weblogic_admin] action create
      - alter group weblogic_admin
      - replace group members with new list of members
    * linux_user[weblogic] action create (up to date)
  Recipe: oracle-inventory::default
    * group[weblogic_admin] action create (up to date)
    * linux_user[weblogic] action create (up to date)
    * directory[/ortlapp/inventory] action create (up to date)
    * template[/ortlapp/oraInst.loc] action create (up to date)
    * group[weblogic_admin] action create
      - alter group weblogic_admin
      - add missing member(s): weblogic
    * directory[/var/cache/chef/weblogic] action create (up to date)
    * directory[/ortlapp/middleware] action create (up to date)
    * remote_file[/var/cache/chef/weblogic/fmw_<IP_ADDRESS>.0_wls.jar] action create (up to date)
    * template[/var/cache/chef/weblogic/silent.rsp] action create (up to date)
    * execute[install weblogic server <IP_ADDRESS>.0] action run (skipped due to only_if)

This change permits current membership whilst ensuring the group exists. Group membership should not matter, group is only used for gid on user.
  * weblogic[<IP_ADDRESS>.0] action create
    * group[weblogic_admin] action create (up to date)
    * linux_user[weblogic] action create (up to date)
  Recipe: oracle-inventory::default
    * group[weblogic_admin] action create (up to date)
    * linux_user[weblogic] action create (up to date)
    * directory[/ortlapp/inventory] action create (up to date)
    * template[/ortlapp/oraInst.loc] action create (up to date)
    * group[weblogic_admin] action create (up to date)
    * directory[/var/cache/chef/weblogic] action create (up to date)
    * directory[/ortlapp/middleware] action create (up to date)
    * remote_file[/var/cache/chef/weblogic/fmw_<IP_ADDRESS>.0_wls.jar] action create (up to date)
    * template[/var/cache/chef/weblogic/silent.rsp] action create (up to date)
    * execute[install weblogic server <IP_ADDRESS>.0] action run (skipped due to only_if)

Thanks Adam.


Add 'resolved' event for getting service info

@Apercu not sure how you're handling versioning, but this is definitely a breaking change.

I use semver for the versioning, will release the 0.2.0 soon.
Thanks again!


[cleanup] Eliminate usage of GRPC_ASSERT(false...);

Introduce a new Crash function, use it everywhere for improved readability.
This also improves bloat.
Note: there's an excess of #include "src/.../crash.h" in tests - these don't hurt, and we'll fix them as we get to iwyu in those directories.

The addition was a run-away search and replace I used at some point... the files in src/ that retain it are files that are not managed by iwyu right now - mostly iomgr, binder - iomgr will go away, binder hopefully will get under iwyu at some point.


upgrade addon-framework to the latest version

Signed-off-by: Zhiwei Yin<EMAIL_ADDRESS>

/assign @philipwu08

/lgtm

/lgtm

/approve


IL, OH - Filter out negative results per 4/4 HHS Guidance

Illinois no longer requires reporting negative rapid test results. Since several SimpleReport senders would like to continue entering negative results so they can provide copies of the results to patients who request them, we need to filter them out much like we do for WA in our settings.
Including Ohio here, too. They have the same request for at-home test results.

@Adrian-Brewster let's keep the tickets in "needs analysis" until there in assignee on the ticket. We can discuss this and other during sprint planning

Hey team! Please add your planning poker estimate with ZenHub @brick-green @greene858 @jbiskie @jorg3lopez @MauriceReeves-usds @oslynn @TomNUSDS

Jurisdiction quality filter change.

From<EMAIL_ADDRESS><EMAIL_ADDRESS>Sent: Thursday, June 23, 2022 10:02 AM
To: Report Stream (CDC<EMAIL_ADDRESS>Cc: Reeves, Maurice (CDC/OD/OCS<EMAIL_ADDRESS>White, Sarah (CDC/DDPHSS/OD/HITSSU<EMAIL_ADDRESS>Kumar, Anshul (CDC/OD/OCS<EMAIL_ADDRESS><EMAIL_ADDRESS>Subject: RE: ReportStream: Filtering out negative COVID results
Hello,
If you are able to filter according to the HHS guidance, that would be great.
So, negatives should only be reported for CLIA-certified laboratories performing NAAT testing.  Negative test results can be filtered for all other testing from any lab/entity.
Thanks,
Bill
From: Report Stream (CDC<EMAIL_ADDRESS>Sent: Thursday, June 9, 2022 10:20 AM
To: Storm, William<EMAIL_ADDRESS>Cc: Reeves, Maurice (CDC/OD/OCS<EMAIL_ADDRESS>White, Sarah (CDC/DDPHSS/OD/HITSSU<EMAIL_ADDRESS>Kumar, Anshul (CDC/OD/OCS<EMAIL_ADDRESS>Subject: ReportStream: Filtering out negative COVID results
Hi Bill,
We have an open ticket to filter out negative COVID results for Ohio based off of the 4/4 HHS guidance that was released.  Is this still your preference? If so, we can have the settings change made today. Please confirm your preference and we will make the change.
Thanks!
Adrian

I have updated the jurisdictional filters for IL and OH. They are now only receiving positive COVID results.


fixing the names SimpleRNNLearner and AutoencoderLearner

Fixing the names in the notebook notebooks/chapter19.RNN.ipynb according to the names used in deep_learning4e.py
As an aside, the training run with SimpleRNNLearner gives a lower validation accuracy:
...
Epoch 10/10
 - 64s - loss: 0.5762 - accuracy: 0.6814 - val_loss: 0.6450 - val_accuracy: 0.5956

It was tested with the following packages:
Name: tensorflow
Version: 1.14.0
Summary: TensorFlow is an open source machine learning framework for everyone.
Home-page: https://www.tensorflow.org/
Author: Google Inc.
Author-email<EMAIL_ADDRESS>License: Apache 2.0
Location: /usr/local/lib/python3.6/dist-packages
Requires: grpcio, termcolor, gast, wheel, tensorboard, six, keras-preprocessing, tensorflow-estimator, wrapt, protobuf, absl-py, astor, keras-applications, numpy, google-pasta
Required-by: tensorflow-serving-api, Entailment
---
Name: Keras
Version: 2.3.0
Summary: Deep Learning for humans
Home-page: https://github.com/keras-team/keras
Author: Francois Chollet
Author-email<EMAIL_ADDRESS>License: MIT
Location: /usr/local/lib/python3.6/dist-packages
Requires: six, scipy, h5py, keras-preprocessing, keras-applications, pyyaml, numpy
Required-by:

Thanks for the PR, much appreciated! Could you remove the warning messages? I think these lines should do it:
import warnings
warnings.filterwarnings("ignore", category=FutureWarning)

Thanks Antonis and there is a new commit for that.
Meanwhile I updated keras, scipy and numpy and the eval accuracy is now indeed around 80%
$ pip show keras scipy numpy
Name: Keras
Version: 2.3.1
Summary: Deep Learning for humans
Home-page: https://github.com/keras-team/keras
Author: Francois Chollet
Author-email<EMAIL_ADDRESS>License: MIT
Location: /usr/local/lib/python3.6/dist-packages
Requires: pyyaml, keras-preprocessing, keras-applications, scipy, h5py, numpy, six
Required-by: hyperas
---
Name: scipy
Version: 1.3.1
Summary: SciPy: Scientific Library for Python
Home-page: https://www.scipy.org
Author: None
Author-email: None
License: BSD
Location: /usr/local/lib/python3.6/dist-packages
Requires: numpy
Required-by: tensor2tensor, scikit-learn, projectq, Keras, jaxlib, hyperopt, gym
---
Name: numpy
Version: 1.17.3
Summary: NumPy is the fundamental package for array computing with Python.
Home-page: https://www.numpy.org
Author: Travis E. Oliphant et al.
Author-email: None
License: BSD
Location: /usr/local/lib/python3.6/dist-packages
Requires:
Required-by: torchvision, torch, tfds-nightly, tf-agents, tensorflow, tensorflow-probability, tensorflow-hub, tensorflow-datasets, tensorboard, tensor2tensor, scipy, scikit-learn, projectq, pandas, opt-einsum, opencv-python, matplotlib, kfac, Keras, Keras-Preprocessing, Keras-Applications, jaxlib, jax, hyperopt, h5py, gym, autograd, Entailment


try (and fail) to parse type field comments as json schema descriptions

Purpose
Briefly describe the purpose of the change, and/or link to the JIRA ticket for context
Fixes MLPAB-####
Approach
Explain how your code addresses the purpose of the change
Learning
Any tips and tricks, blog posts or tools which helped you. Plus anything notable you've discovered about the Modernising LPA service
Checklist

[ ] I have performed a self-review of my own code
[ ] I have added relevant logging with appropriate levels to my code
[ ] I have updated documentation (Confluence/GitHub wiki/tech debt doc) where relevant
[ ] I have added tests to prove my work
[ ] I have added welsh translation tags and updated translation files
[ ] I have run an accessibility tool on any pages I have made changes to and fixed any issues found
[ ] The product team have tested these changes

POC


First try

Installed SwiftUICharts in the package manager in Xcode 11 beta 4. It is visible in the project navigator (Swift Package Dependencies).
I made a new SwiftUI file, called ChartView.
`import SwiftUI
import SwiftUICharts
struct ChartView: View {
var body: some View {

    ChartView(data: [12,17,24,33,23,56], title: "Chart two", form: Form.small)    }

}
#if DEBUG
struct ChartView_Previews: PreviewProvider {
static var previews: some View {
ChartView()
}
}
#endif
`
Now I'm getting an error "Argument passed to call that takes no arguments".

Sorry, but I couldn't reproduce this issue. It worked without errors for me. I'm using the latest (beta 4 Xcode)
Here is my code:
import SwiftUI
import SwiftUICharts

struct ContentView: View {
    var body: some View {
         ChartView(data: [12,17,24,33,23,56], title: "Chart two", form: Form.small)
    }
}

#if DEBUG
struct ContentView_Previews: PreviewProvider {
    static var previews: some View {
        ContentView()
    }
}
#endif

This is indeed a bug. The problem with a declarative framework such as SwiftUI is that debugging is really hard and often errors pop up in the weirdest of places. That being said, after reading the code line by line I found that the problem is with the struct named "Form" in ChartView. This conflicts with the built-in Form used in SwiftUI, so you get the bug in projects using Form. Simply renaming Form to ChartSize or something similar resolves the issue.

Thanks I'll do a renaming then


Review the need to adjust final selection in Expr::applyFunctionWithPeeling

See https://github.com/facebookincubator/velox/pull/2592#pullrequestreview-1114176395

@mbasmanova please correct me if I am wrong.
I looked into this yesterday, and was trying to come up with a scenario where not adjusting final selection in applyFunctionWithPeeling() would result in an issue. Turns out the actual value of the final selection does not matter at this point. What matters is whether it is set or not. This is because the final selection rowset is used to materialize lazy vectors, however when we get to this point, the input vectors would have been loaded and the applying function only needs to check whether finalSelection is set or not so that it can preserve partially populated results.
TLDR: we dont need to update final selection since its actual value is not used.
However, we would benefit from adding a comment here that explains why it is not required since this function is virtually identical to peelEncodings() and a similar question can come up in the future (the big difference here is that peelEncodings() only applied to input columns whereas this applies to intermediate results)

@bikramSingh91 Bikram, thank you for looking into this and writing up a detailed summary of your findings. This makes sense to me and I agree that adding a comment would be beneficial.


timeShift data is out of phase when switching from 343 to 350 days

Hi,
My data has a distinct 7-day pattern and everything shows as expected up to 343 days while looking back. As soon as I increase the number to 350 (and then 357, 364) the data is visibly out of phase. I would appreciate any clues.

Solved by creating a dedicate daily pattern in storage-schemas.conf


KitchenSink - Install saucectl when image does not provide it

LGTM until the pipeline passes :)

@alexplischke / @tianfeng92 I'm not sure if this is the correct approach and I'm not sure there is any alternative (currently) to chown (uid from runner & seluser)


feat: add custom summarizer argument in CLI run mode 在CLI启动模式中添加自定义Summarizer参数

Thanks for your contribution and we appreciate it a lot. The following instructions would make your pull request more healthy and more easily get feedback. If you do not understand some items, don't worry, just make the pull request and seek help from maintainers.
Motivation
The new CLI mode is very helpful. But it's lack of custom summarizer argument. This PR solves this. User can use python run.py --models hf_qwen_7b --datasets smoking_gen --summarizer configs/summarizers/medium.py to run with custom summarizer.
Modification

Add --summarizer argument
read related config file
fix related bug to compatible with command without custom summarizer

Use cases
The following cases are tested. smoking_gen is a short custom defined dataset to test quickly.

python run.py --models hf_qwen_7b --datasets smoking_gen --summarizer configs/summarizers/medium.py
python run.py --models hf_qwen_7b --datasets smoking_gen

Checklist
Before PR:

[x] Pre-commit or other linting tools are used to fix the potential lint issues.
[ ] Bug fixes are fully covered by unit tests, the case that causes the bug should be added in the unit tests.
[ ] The modification is covered by complete unit tests. If not, please add more unit test to ensure the correctness.
[ ] The documentation has been modified accordingly, like docstring or example tutorials.

After PR:

[ ] If the modification has potential influence on downstream or other related projects, this PR should be tested with those projects.
[ ] CLA has been signed and all committers have signed the CLA in this PR.

Thanks!


Emoji/high codepoint strings do not work with the html! macro

html! { <p>{"😐"}</p> }

Result:


This is a bug in stdweb, which I've fixed a minute ago. The fix will be available in 0.4 which will be released very soon.
Thanks for the report!

As 0.4 was just released this can be fixed now once yew updates to it.

Sorry to hijack this, but I've been hoping for a fix to #121 for a while. I didn't know that stdweb affected the html! macro. The upgrade to 0.4 didn't resolve #121, but this seems a bit similar in that the macro fails to render when Rust code is embedded. Please see my minimal example attached to #121 which demonstrates the specific problem I'm having.
Thanks!

Fixed as mentioned above


[Feature Request] Customize metric bucket sizes

Is your feature request related to a problem? Please describe.

From a user:

We noticed the metric caps out at 10 seconds. If we start a large batch workflows we are generally fine waiting for them to process for 10-15 minutes. Given the 10 second maximum it’s pretty hard to tell how busy the workflow queue is.

(small number of workflow task slots)

I have changed the title to reflect the general desire to customize bucket sizes. Unsure if this can be done on a per metric basis or just in general for all durations/histograms.

Also, if we are to record our own metrics, even those use the default buckets, which are not always ideal

It would be nice to override this for the activity execution latency metrics. Right now it caps out at 1 minute, but we have some activities that take longer than that.


[repology] 'pname' is not exported to https://nixos.org/nixpkgs/packages-unstable.json.gz

Describe the bug
Although now majority of packages have 'pname' it is not exported to https://nixos.org/nixpkgs/packages-unstable.json.gz so repology is not able to pick pname and still parses name to pname+version using an unreliable heuristics.
https://github.com/repology/repology/issues/854#issuecomment-530094388
To Reproduce
curl --silent https://nixos.org/nixpkgs/packages-unstable.json.gz

I'll also add that version is not always exported too. For example, uhubctl entry defines both pname and version:
https://github.com/NixOS/nixpkgs/blob/848ea3af656663e634a79f1fdab866bc77dca895/pkgs/tools/misc/uhubctl/default.nix#L7-L8
but neither is present in packages-unstable.json.gz

version from the derivation is never exported. Only in the cases it is added to meta like in
https://github.com/NixOS/nixpkgs/blob/848ea3af656663e634a79f1fdab866bc77dca895/pkgs/development/compilers/abcl/default.nix#L32
The files are generated here:
https://github.com/NixOS/nixos-homepage/blob/d6aa00d9f35e61396dc38416f821774f910c1c74/Makefile#L216-L245


SourceKitLSP: handle process termination better

This adjusts the process termination to handle termination on Windows,
which does not provide a kill operation, which is Unix specific

CC: @ahoppen

@swift-ci please test

Thanks for fixing this!


Set the HTTP user agent

Currently when we make an HTTP request, the User-Agent header we send is just whatever the default is that Dart's HTTP implementation gives us. This means e.g. that log lines on the server look like this, saying "via Dart":
2023-11-21 22:32:07.353 INFO [zr] <IP_ADDRESS>   POST    200 113ms (db: 35ms/41q) /api/v1/register (11@zulip via Dart)

Instead we should set a user-agent string that identifies this app. Ideally it should include the app's version number.
There are at least two different kinds of HTTP requests the app makes, both of which should have the user agent set:

Zulip API calls, which go through ApiConnection.send;
fetching images, including avatars, image emoji, and images included in messages, all of which go through RealmContentNetworkImage.

It's likely those will need to be handled separately.

The value should probably say "ZulipMobile" somewhere, because apparently (https://github.com/zulip/zulip-flutter/issues/453#issuecomment-1856146265) for send-message requests that will activate the server's heuristic for deciding we're a UI client and therefore the newly sent message should be marked as read for its author, solving #440.
For some discussion on how to implement this, see #453.

The value should probably say "ZulipMobile" somewhere

But this should be marked with a TODO(server-8) comment — once we're supporting only Zulip Server 8+, our use of read_by_sender (from #456) should always get respected, so we should no longer need to cater to that old heuristic and we can use a more transparent user-agent of our choice.


Switch to Js.Result.t instead of internally defined result

FYI this PR also has the update bs-platform commit from #13

This change was badly needed!


Issues found by PVS-Studio

Hello,
I have found a few bugs using PVS-Studio analyzer. PVS-Studio is a static code analyzer for C, C++ and C#: https://www.viva64.com/en/pvs-studio/
Analyzer warnings:

V3095 The '_services' object was used before it was verified against null. Check lines: 102, 104. MassTransit.Host MassTransitHostService.cs 102
A link to the source code.

V3022 Expression 'hostAddress' is always not null. The operator '??' is excessive. MassTransit.HttpTransport HttpTestHarness.cs 33
A link to the source code.

In addition, I suggest having a look at the emails, sent from @pvs-studio.com.
Best regards,
Sergey Hrenov,
PVS-Studio Team

Sorry, but repeated emails and then posting issues for your commercial product are not cool.


Simplify examples in documentation

Hi,
Some examples (two actually) in the repository still make use of the unnecessary projectId argument when instantiating a SpeechClient, which can somehow be a bit confusing when starting to use the client.
I'm issuing a PR to propose a fix that instantiates SpeechClient without any argument, thus following the examples in samples/recognize.js and samples/recognize.v1p1beta1.js.
Also, the page at the URL https://cloud.google.com/nodejs/docs/reference/speech/latest/ (linked in README.md) should be updated :

should point to https://cloud.google.com/nodejs/docs/reference/speech/1.2.x (latest version)
the corresponding page still uses projectId

Thanks a lot for this great program, we're loving it !

I'm glad to hear it's working well, and thanks for taking the time to help!
@alexander-fenster what do you think about removing the project ID from the examples? Without looking, I'm pretty sure our other APIs show this option. However, maybe it really isn't that vital anymore, since a project ID can be detected in all GCP environments as well as from a JSON service account key file. I suppose the only times it can't be are when:

A user provides credentials object, without a project ID
A user provides a PEM/P12 key file

@jmdobry 1.2.0 was released 3 days ago. I assume there is just some lag in when the docs site gets updated?

I'm OK - I approved #47. We'll see if samples tests work after this change is merged (I guess they should).


Refactored string resources introduced in #29:

... to make it much easier for translators to add a new language. Only 38 items have to be translated instead of 75 before

renamed string-ids to become more contextsesitive so translaters know more about their meaning

Example: strings for the settings activity are prefixed with "settings_"

renamed some resource-ids

Example: R.id.data becomes R.id.data_edit
Example: R.id.activities_layout becomes R.id.intent_matchin_activities_layout

moved non translatable strings.xml to donottranslate.xml

text_plain in strings.xml becomes mime_type_text_plain in donottranslate.xml

removed fromatting from string resources to code making translating much easier for translators

the new intent_extras_title replaces the old extras, extra, extra_count, extras_title_bold, bundle_title_bold, bundle_title_bold_uppercase

moved some string resources back to code.

for example preference-id: @string/intercept_enable_pref

added those german translations that are not Intend-api specific
reimplemented parts of getIntentDetailsStrings

this will refactor #25 #28 #29 #31

@k3b Thanks for all this.

thanks for adding it.


Auto Glyphs Are Always 0 (Relic Recovery)

Auto glyphs are always 0, even if there are keys or they are exist.
The final auton scoring is calculated including the auto glyphs.

In 1718-ISR-CMP1-Q002-1 match, the total number of red auton points is 55. But if you add all the points it goes out 40, Which means there are 15 points missing from the glyphs(0).


Change address forms to not have duplicate IDs

1. Why is this change necessary?
2. What does this change do, exactly?
3. Describe each step to reproduce the issue or behaviour.
4. Please link to the relevant issues (if any).
5. Checklist

[x] I have rebased my changes to remove merge conflicts
[ ] I have written tests and verified that they fail without my change
[ ] I have created a changelog file with all necessary information about my changes
[ ] I have written or adjusted the documentation according to my changes
[ ] This change has comments for package types, values, functions, and non-obvious lines of code
[x] I have read the contribution requirements and fulfil them.

@ndzoesch @shyim What to do, to get this merged?

@ndzoesch @shyim What to do, to get this merged?

Hi @Schrank
your commit is currently in our pipeline, but it fails due to concurrent changes currently. We are on it and it shouldn't take long now.
Regards

@Schrank
Thank you for your Contribution. Your commit was merged 👍 🚀


iRODS: Check if checksum already exists before calculating

In the checksum calculation rule on our iRODS server, we should add a check to see if isync has already calculated the checksum on put. If it has, we should skip generating it twice and slowing down batch transfers.
Will look into this next week, might be totally trivial or not.
Not really a SODAR codebase related issue, but adding it here in lack of a better place.

We are no longer using said rule, so I'm considering this closed.


Unbound: root-hints and aggressive-nsec switches

Hi!
Just a couple of suggestions:
- Add a root-hints switch:
ref. https://forum.opnsense.org/index.php?topic=37840.msg187795#msg187795
Looks like there may be a strange thread race on unbound start, leading to an error loading root hints, high CPU load and unbound inoperability. Adding a switch should not cause harm imho since the internal hints list is up-to-date and unbound is used in forwarding mode in many configurations.

Add an aggressive-nsec switch:
ref. https://forum.opnsense.org/index.php?topic=38888.0
ref. https://github.com/NLnetLabs/unbound/issues/824
ref. https://unbound.docs.nlnetlabs.nl/en/latest/topics/privacy/aggressive-nsec.html
Bad nsec records may lead to name resolution problems with  aggressive-nsec (yes by default).

Thanks!

To be honest I'd like to solve the root hints thing differently, perhaps with something along the lines of https://github.com/opnsense/core/commit/845fbd384 Using the copy should always work unless unbound has larger issues and adding an opt-out knob for less controllable results has no benefits.

@fichtner Hi!)

unless unbound has larger issues

thats what im talking about). from what I saw during experiments (https://forum.opnsense.org/index.php?topic=37840.msg187795#msg187795) i don’t understand how to solve this on the OPNsense side: it doesn't look like it's a matter of using templates

(I'd like to avoid two workardounds for this now at once, maybe you can remove that bit from your PR to merge the other thing)

Merged, thanks!

@fichtner
Thanks!
Although I like my theory about thread safety and reading the file on each thread init, I'm afraid that my knowledge is not enough to create a proper request at upstream...
But I'll try to dig into the code some more )

efc9b09c769 then

technically it's probably still not perfectly safe, ouch... let me do that later

@fichtner no no. I talked about race and thread safety on unbound side itself.
(https://github.com/NLnetLabs/unbound/blob/352245160058e9419565f922d62ce01634280b9d/daemon/worker.c#L2278 on each thread with https://github.com/NLnetLabs/unbound/blob/master/iterator/iter_hints.c#L307)
so I don't expect that even locking the entire seup/start procedure will help to prevent this.
I talked about making a request in unbound repo )
sorry if I expressed unclearly

@kulikov-a ok but how can this screw up reading a file? unless the threads use a shared structure which is... err... bad?

@fichtner but the errors content: sudden attempts to parse a RR type in a comment line or a mismatch between the record type and the ip type (for the A-record suddenly there is an ipv6 address ?) leads to the assumption that .... yes :wink:

hmm, I am starting to not understand why people in the forum keep insisting on fixing this with us


Is it possible to delete specific images

Hey, I was integrating galleria with my django project. But apart from showing images, I also need the functionality to delete images. Like from the gallery view, I will choose an image and then there will be a delete button. How can I do this? Any ideas?
Thanks

Try API method .splice()
https://galleriajs.github.io/docs/api/methods.html#manipulation
pi 3. 4. 2020 o 7:32 FiazAbrar<EMAIL_ADDRESS>napísal(a):

Hey, I was integrating galleria with my django project. But apart from
showing images, I also need the functionality to delete images. Like from
the gallery view, I will choose an image and then there will be a delete
button. How can I do this? Any ideas?
Thanks
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/GalleriaJS/galleria/issues/418, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAMR3URESK3YL73UCMWLGKLRKVYGJANCNFSM4L3OKR6Q
.

--
Samuel Mikláš

Not sure it that is ideal, can you take a look here: https://stackoverflow.com/questions/10433444/galleria-deletion-icon
Is this something that should be done?

Sure it can be done. In my opinion, Internal API is best for this. My previous post.
Here you go - live example.

You can buy me a bear. Chears

This works great with the twelve theme, but not with the folio theme.  Any idea why? https://jsbin.com/litikiyoje/1/edit?html,output


Server HTTPS assets in the docs

The docs serve the main stylesheet over HTTP, if we load this over https the entire site can be loaded over https.

Done


[BUG] - cabal fails to satisfy tasty-discover dependency in nix-shell

External
Area
Other  nix-shell environment
Summary
cabal commands such as cabal build cardano-node fail within a nix-shell because tasty-discover cannot be found. tasty-discover is explicitly included in the shell here https://github.com/input-output-hk/cardano-node/blob/c7f03233a4b69b4809bf9cf88d7e3f34f0886d30/nix/haskell.nix#L103 but because of https://github.com/input-output-hk/haskell.nix/issues/231 (also reported as https://github.com/input-output-hk/haskell.nix/issues/839) this does not satisfy the dependency.
Steps to reproduce
~/work/iohk/cardano-node% nix-shell                                             (git)-[master]
trace: Using IOHK default nixpkgs
trace: To make project.plan-nix for cardano-node-src a fixed-output derivation but not materialized, set `plan-sha256` to the output of the 'calculateMaterializedSha' script in 'passthru'.
trace: To materialize project.plan-nix for cardano-node-src entirely, pass a writable path as the `materialized` argument and run the 'updateMaterialized' script in 'passthru'.
building '/nix/store/cn3mxdjwsq87s2r0pa5fwxrxcpg41d0p-git-ls-files.drv'...
trace: To make project.plan-nix for cardano-node-src a fixed-output derivation but not materialized, set `plan-sha256` to the output of the 'calculateMaterializedSha' script in 'passthru'.
trace: To materialize project.plan-nix for cardano-node-src entirely, pass a writable path as the `materialized` argument and run the 'updateMaterialized' script in 'passthru'.
trace: To make project.plan-nix for hoogle a fixed-output derivation but not materialized, set `plan-sha256` to the output of the 'calculateMaterializedSha' script in 'passthru'.
trace: To materialize project.plan-nix for hoogle entirely, pass a writable path as the `materialized` argument and run the 'updateMaterialized' script in 'passthru'.

[nix-shell:~/work/iohk/cardano-node]$ ./scripts/cabal-inside-nix-shell.sh

[nix-shell:~/work/iohk/cardano-node]$ cabal build cardano-node
Warning: No remote package servers have been specified. Usually you would have
one specified in the config file.
Resolving dependencies...
cabal: Could not resolve dependencies:
[__0] trying: cardano-node-chairman-1.24.2 (user goal)
[__1] rejecting: cardano-node-chairman:!test (constraint from config file,
command line flag, or user target requires opposite flag selection)
[__1] trying: cardano-node-chairman:*test
[__2] unknown package: cardano-node-chairman:tasty-discover:exe.tasty-discover
(dependency of cardano-node-chairman *test)
[__2] fail (backjumping, conflict set: cardano-node-chairman,
cardano-node-chairman:tasty-discover:exe.tasty-discover,
cardano-node-chairman:test)
After searching the rest of the dependency tree exhaustively, these were the
goals I've had most trouble fulfilling: cardano-node-chairman,
cardano-node-chairman:test,
cardano-node-chairman:tasty-discover:exe.tasty-discover

Expected behavior
The dependency should be satisfied and cabal operations work.
System info (please complete the following information):

OS: Nixos 20.09
cardano-node master

Can someone confirm if removing tasty-discover helps?
https://github.com/input-output-hk/cardano-node/pull/2265

Can someone confirm if removing tasty-discover helps?
https://github.com/input-output-hk/cardano-node/pull/2265

I've merge my PR. Please let me know if I can close this issue.

I've merge my PR. Please let me know if I can close this issue.

Yes! Thank you.

Yes! Thank you.


When will new release issue and which is actual state of project?

Hi!
I'm looking for a solution for sync between PostgreSQL and others storages.
I've read posts about bottledwater and it looks as appropriate tool for my task, but I want to know
what's going on with project now. Will you develop it or not?

Hi, I unfortunately don't have time to maintain Bottled Water at the moment, and there doesn't seem to be anybody else currently wanting to do it either. You're of course welcome to maintain it — it's open source after all :)
Alternatively, take a look at Debezium, which uses a very similar approach to Bottled Water, and seems more actively developed

@Budulianin I've recently forked this repo to ucalgary/bottledwater-pg, because we're using it in a project. There's also corresponding Docker builds at ucalgary/bottledwater and ucalgary/postgres-bw on Docker Hub.
We are actively making some improvements to meet our internal needs. But, I can't promise any sort of external support or development.

@ept thanks for reply and recommendation about Debezium. I'm going to check Debezium too.
I'm sorry that you don't support this repository any more. I read your articles from confluent.io, they are very interesting.
@kinghuang thanks for your suggestion. I'm going to look at it.
It will be better if you support this repository instead of fork it :)