Adding progress bar to demultiplex functions

This pull request will be for adding a progress bar to the demultiplex functions of pyinseq, one of the heavy performance task. It will contain bars for demultiplexing fastq and writing trimmed/not-trimmed reads to fastq files. In addition, a CustomStream class was added to allow the logging class to use tqdm writing module so that log messages do not overlap with progress bars.

Travis is throwing a fail for "make test". This is internally cause by pytest not properly tearing down after test_genome_script function. I do not know why this is happening. I will look into it this week.

Let's discuss briefly this afternoon.

Codecov Report

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

@@           Coverage Diff           @@
##           master      #75   +/-   ##
=======================================
  Coverage   92.02%   92.02%
=======================================
  Files           3        3
  Lines         188      188
=======================================
  Hits          173      173
  Misses         15       15

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 f3559fc...f97ac0f. Read the comment docs.

Passed TravisCI. If code looks good, go ahead and squash/merge.

@eburgoswisc Have you tried this with large input files (100-200 M reads), or with .gzipped input files?
Here is what I get when I run it. When I run our simple test it completes (and the status bars look awesome!)
(pyinseq) DMI-MBPro2-mandel:pyinseq markmandel$ pyinseq -i pyinseq/tests/data/input/example01.fastq -s pyinseq/tests/data/input/example01.txt -g pyinseq/tests/data/input/ES114v2.gb -e TEST_TQDM3
2019-03-13 16:17 - INFO - runner - Process command line arguments
2019-03-13 16:17 - INFO - utils - Make directory: results/TEST_TQDM3
2019-03-13 16:17 - INFO - utils - Make directory: results/TEST_TQDM3/raw_data/
2019-03-13 16:17 - INFO - utils - Make directory: results/TEST_TQDM3/genome_lookup/
2019-03-13 16:17 - INFO - runner - Demultiplex reads
Demultiplexing Reads: 100%|███████████████████████████████████████████████████████████████████████████████████████████████████████| 534/534 [00:00<00:00, 61118.77reads/s]
2019-03-13 16:17 - INFO - demultiplex - Writing reads to E001_01.fastq
Writing E001_01 reads: 100%|█████████████████████████████████████████████████████████████████████████████████████████████████████| 252/252 [00:00<00:00, 167003.41reads/s]
2019-03-13 16:17 - INFO - demultiplex - Writing reads to E001_02.fastq
Writing E001_02 reads: 100%|█████████████████████████████████████████████████████████████████████████████████████████████████████| 274/274 [00:00<00:00, 234634.40reads/s]
2019-03-13 16:17 - INFO - demultiplex - Writing reads to _other.fastq
Writing _other reads: 100%|███████████████████████████████████████████████████████████████████████████████████████████████████████████| 8/8 [00:00<00:00, 15040.09reads/s]
2019-03-13 16:17 - INFO - demultiplex - Writing trimmed reads to E001_01.fastq
Writing E001_01 trimmed reads: 100%|██████████████████████████████████████████████████████████████████████████████████████████████| 252/252 [00:00<00:00, 80127.71reads/s]
2019-03-13 16:17 - INFO - demultiplex - Writing trimmed reads to E001_02.fastq
Writing E001_02 trimmed reads: 100%|██████████████████████████████████████████████████████████████████████████████████████████████| 274/274 [00:00<00:00, 66665.08reads/s]
2019-03-13 16:17 - INFO - demultiplex - Total records demultiplexed: 534
2019-03-13 16:17 - INFO - runner - Prepare genome features (.ftt) and fasta nucleotide (.fna) files
  Nucleotide file output file: results/TEST_TQDM3/genome_lookup/genome.fna
  Feature table output file: results/TEST_TQDM3/genome_lookup/genome.ftt
2019-03-13 16:17 - INFO - runner - Prepare bowtie index
2019-03-13 16:17 - INFO - runner - Building bowtie index files in results/TEST_TQDM3/genome_lookup
2019-03-13 16:17 - INFO - runner - Map with bowtie
2019-03-13 16:17 - INFO - runner - Sample E001_01: map reads with bowtie
2019-03-13 16:17 - INFO - runner - # reads processed: 252
# reads with at least one reported alignment: 252 (100.00%)
# reads that failed to align: 0 (0.00%)
Reported 252 alignments to 1 output stream(s)

2019-03-13 16:17 - INFO - runner - Sample E001_01: summarize the site data from the bowtie results
2019-03-13 16:17 - INFO - runner - Sample E001_01: map site data to genes
2019-03-13 16:17 - INFO - runner - T50 result for E001_01: 3
2019-03-13 16:17 - INFO - runner - Sample E001_02: map reads with bowtie
2019-03-13 16:17 - INFO - runner - # reads processed: 274
# reads with at least one reported alignment: 274 (100.00%)
# reads that failed to align: 0 (0.00%)
Reported 274 alignments to 1 output stream(s)

2019-03-13 16:17 - INFO - runner - Sample E001_02: summarize the site data from the bowtie results
2019-03-13 16:17 - INFO - runner - Sample E001_02: map site data to genes
2019-03-13 16:17 - INFO - runner - T50 result for E001_02: 3
2019-03-13 16:17 - INFO - runner - Aggregate gene mapping from all samples into the summary_data_table
2019-03-13 16:17 - INFO - runner - T50 E001_01: 3
2019-03-13 16:17 - INFO - runner - T50 E001_02: 3
2019-03-13 16:17 - INFO - runner - Print samples info: results/TEST_TQDM3/samples.yml
2019-03-13 16:17 - INFO - runner - Print summary log: results/TEST_TQDM3/log.txt
2019-03-13 16:17 - INFO - runner - Print command entered
pyinseq -i pyinseq/tests/data/input/example01.fastq -s pyinseq/tests/data/input/example01.txt -g pyinseq/tests/data/input/ES114v2.gb -e TEST_TQDM3
2019-03-13 16:17 - INFO - runner - Print settings
command: pyinseq
experiment: TEST_TQDM3
path: results/TEST_TQDM3/
parse_genbank_file: True
genome_path: results/TEST_TQDM3/genome_lookup/
organism: genome
raw_path: results/TEST_TQDM3/raw_data/
generate_bowtie_index: True
process_reads: True
process_sample_list: True
map_to_genome: True
samples_yaml: results/TEST_TQDM3/samples.yml
summary_log: results/TEST_TQDM3/log.txt
write_trimmed_reads: True
summary_table: results/TEST_TQDM3/summary_gene_table.txt
min_counts: 3
max_ratio: 10
keep_all: False
barcode_length: 4
disruption: 1.0
2019-03-13 16:17 - INFO - runner - Print samples detail
E001_01:
  barcode: GAAG
E001_02:
  barcode: CTTT

2019-03-13 16:17 - INFO - runner - ***** pyinseq complete! *****

When I try with a gzipped file it yields an error. We need to add a test for this.
(pyinseq) DMI-MBPro2-mandel:pyinseq markmandel$ pyinseq -i ~/src/pyinseq/private/example02.fastq.gz -s pyinseq/tests/data/input/example02.txt -g pyinseq/tests/data/input/ES114v2.gb -e TEST_TQDM4
2019-03-13 16:19 - INFO - runner - Process command line arguments
2019-03-13 16:19 - INFO - utils - Make directory: results/TEST_TQDM4
2019-03-13 16:19 - INFO - utils - Make directory: results/TEST_TQDM4/raw_data/
2019-03-13 16:19 - INFO - utils - Make directory: results/TEST_TQDM4/genome_lookup/
2019-03-13 16:19 - INFO - runner - Demultiplex reads
Traceback (most recent call last):
  File "/Users/markmandel/anaconda3/envs/pyinseq/bin/pyinseq", line 7, in <module>
    exec(compile(f.read(), __file__, 'exec'))
  File "/Users/markmandel/Dropbox/repos/pyinseq/scripts/pyinseq", line 7, in <module>
    runner.main(args)
  File "/Users/markmandel/Dropbox/repos/pyinseq/pyinseq/runner.py", line 434, in main
    demultiplex_fastq(reads, samples_dict, settings)
  File "/Users/markmandel/Dropbox/repos/pyinseq/pyinseq/demultiplex.py", line 53, in demultiplex_fastq
    results = fastq_generator(reads)
  File "/Users/markmandel/Dropbox/repos/pyinseq/pyinseq/utils.py", line 104, in fastq_generator
    for i in screed.fastq.fastq_iter(mmap.mmap(f.fileno(), 0, prot=mmap.PROT_READ)):
  File "/Users/markmandel/anaconda3/envs/pyinseq/lib/python3.6/site-packages/screed/fastq.py", line 21, in fastq_iter
    line = to_str(line.strip())
  File "/Users/markmandel/anaconda3/envs/pyinseq/lib/python3.6/site-packages/screed/utils.py", line 6, in to_str
    line = line.decode('utf-8')
UnicodeDecodeError: 'utf-8' codec can't decode byte 0x8b in position 1: invalid start byte

When I first gunzip the file, the program then hangs on the Demultiplex Reads step. It's been here for 20 minutes with no status bar or update.
(pyinseq) DMI-MBPro2-mandel:pyinseq markmandel$ pyinseq -i ~/src/pyinseq/private/example02.fastq -s pyinseq/tests/data/input/example02.txt -g pyinseq/tests/data/input/ES114v2.gb -e TEST_TQDM5
2019-03-13 16:20 - INFO - runner - Process command line arguments
2019-03-13 16:20 - INFO - utils - Make directory: results/TEST_TQDM5
2019-03-13 16:20 - INFO - utils - Make directory: results/TEST_TQDM5/raw_data/
2019-03-13 16:20 - INFO - utils - Make directory: results/TEST_TQDM5/genome_lookup/
2019-03-13 16:20 - INFO - runner - Demultiplex reads

Small other point I noticed when looking at the output... this section:
2019-03-13 16:17 - INFO - demultiplex - Writing reads to E001_01.fastq
2019-03-13 16:17 - INFO - demultiplex - Writing reads to E001_02.fastq
2019-03-13 16:17 - INFO - demultiplex - Writing reads to _other.fastq
2019-03-13 16:17 - INFO - demultiplex - Writing trimmed reads to E001_01.fastq
2019-03-13 16:17 - INFO - demultiplex - Writing trimmed reads to E001_02.fastq

should probably be:
2019-03-13 16:17 - INFO - demultiplex - Writing reads to raw_data/E001_01.fastq
2019-03-13 16:17 - INFO - demultiplex - Writing reads to raw_data/E001_02.fastq
2019-03-13 16:17 - INFO - demultiplex - Writing reads to raw_data/_other.fastq
2019-03-13 16:17 - INFO - demultiplex - Writing trimmed reads to E001_01_trimmed.fastq
2019-03-13 16:17 - INFO - demultiplex - Writing trimmed reads to E001_02_trimmed.fastq

I will look into it tomorrow afternoon. To handle bigger files and gzipped, I will move to using ScreedDB class to open them and process the reads (with progress bar). However, the progress bar is handled for de-multiplex. Ill also add tests.

Sounds good. Here are couple of things to consider:

Is mmap required for the tqdm function? If not, then consider making this PR just about tqdm, and then add mmap later if desired. This would make it easier to evaluate and merge the tqdm PR, and also easier to troubleshoot if behavior changes in the future.

You mentioned that you made some changes due to speed. I think we should wait to optimize until we are a bit further along, then set up some standard benchmarks for this. For example in this case, it may work much faster on smaller data sets but flail on larger ones.

Can you do squash/merge? I'll make a separate pull request for adding new fastq.gz test files and methods, unless its ideal to keep committing to this one.

Hi @eburgoswisc, great to have this. Please make the following changes:

The current implementation is very confusing for the user. We don't want progress bars for each of the buffered reads. See the screenshot below. The demultiplexing is only 5% done but the user will think it is complete. I think the benefit of the progress bar is that you can hide those other details in the background. tqdm takes care of the number of reads, how many are demultiplex, and how long the remaining portion is predicted to take. Once the progress bar makes it to 100% then all of the demultiplexing should be done.
Change the log entry of "Counting sequences in input for progress bar" to "Preparing to demultiplex reads."
Remove the .gitignore and VSCode files from the commit. Try to manage these on your local machine for now. If needed we can add a bunch of stuff to the .gitignore in a future commit.

Should I remove the logger.info that is called in the writing methods too? I can leave that message in the progress bar and it will not show in the log.txt. The trimmed reads bar will disappear.


Do not mix power monitoring config files with telemetry

With recent healthcheck changes, attempts are made to start kepler before it's even deployed because the config files were created under same directory.

/test pre-commit-test

/test pre-commit-test

/test pre-commit-test

/lgtm


Error: Cannot find module 'electron-media-service'

https://sentry.io/gpmdp/electron-client/issues/259651104/
Error: Cannot find module 'electron-media-service'
  File "module.js", line 470, in Module._resolveFilename
  File "Player.app/Contents/Resources/electron.asar/common/reset-search-paths.js", line 35, in Function.Module._resolveFilename (/Applications/Google Play Music Desktop
  File "module.js", line 418, in Function.Module._load
  File "module.js", line 498, in Module.require
  File "internal/module.js", line 20, in require
...
(30 additional frame(s) were not displayed)

Error: Cannot find module 'electron-media-service'

This error started flooding Sentry immediately after the last release, determining severity and cause now

Issue identified as being the native module doesn't compile on the version of XCode that Travis runs and as an optional dep no error is thrown during install.  Trying alternate xcode versions 👍

Fixed in master


0.65.4 – possible breaking change – "partially initialized module 'strawberry' has no attribute 'type'" (circular import?)

Hey, upgrading to 0.65.4 throws me the following error:
Exception in thread django-main-thread:
Traceback (most recent call last):
  File "/usr/local/lib/python3.9/threading.py", line 954, in _bootstrap_inner
    self.run()
  File "/usr/local/lib/python3.9/threading.py", line 892, in run
    self._target(*self._args, **self._kwargs)
  File "/venv/lib/python3.9/site-packages/django/utils/autoreload.py", line 64, in wrapper
    fn(*args, **kwargs)
  File "/venv/lib/python3.9/site-packages/django/core/management/commands/runserver.py", line 110, in inner_run
    autoreload.raise_last_exception()
  File "/venv/lib/python3.9/site-packages/django/utils/autoreload.py", line 87, in raise_last_exception
    raise _exception[1]
  File "/venv/lib/python3.9/site-packages/django/core/management/__init__.py", line 375, in execute
    autoreload.check_errors(django.setup)()
  File "/venv/lib/python3.9/site-packages/django/utils/autoreload.py", line 64, in wrapper
    fn(*args, **kwargs)
  File "/venv/lib/python3.9/site-packages/django/__init__.py", line 24, in setup
    apps.populate(settings.INSTALLED_APPS)
  File "/venv/lib/python3.9/site-packages/django/apps/registry.py", line 91, in populate
    app_config = AppConfig.create(entry)
  File "/venv/lib/python3.9/site-packages/django/apps/config.py", line 224, in create
    import_module(entry)
  File "/usr/local/lib/python3.9/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1030, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
  File "<frozen importlib._bootstrap>", line 972, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 228, in _call_with_frames_removed
  File "<frozen importlib._bootstrap>", line 1030, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
  File "<frozen importlib._bootstrap>", line 986, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 680, in _load_unlocked
  File "<frozen importlib._bootstrap_external>", line 790, in exec_module
  File "<frozen importlib._bootstrap>", line 228, in _call_with_frames_removed
  File "/venv/lib/python3.9/site-packages/strawberry/__init__.py", line 1, in <module>
    from . import django, experimental, federation
  File "/venv/lib/python3.9/site-packages/strawberry/django/__init__.py", line 4, in <module>
    from strawberry_django import *  # type: ignore # noqa: F401, F403
  File "/venv/lib/python3.9/site-packages/strawberry_django/__init__.py", line 1, in <module>
    from . import auth, filters, ordering
  File "/venv/lib/python3.9/site-packages/strawberry_django/filters.py", line 9, in <module>
    from .fields.types import is_auto
  File "/venv/lib/python3.9/site-packages/strawberry_django/fields/types.py", line 13, in <module>
    @strawberry.type
AttributeError: partially initialized module 'strawberry' has no attribute 'type' (most likely due to a circular import)

Downgrading back to 0.65.3 resolved this issue for me.
I take usage of the strawberry.django package, include auto from it and use the strawberry.django.type and field definitions

Something must have broken in #994. @la4de any ideas?

Have the same problem with 0.65.4

I propose to revert my previous change. See PR #1019. We need to find out better solution for symbol resolution issue with IDEs.

Workaround is to use 0.65.3

We merged and released a fix for this.
Closing this, let us know if it still an issue with recent Strawberry versions.


MSID: 101613 Version: 1 DOI: 10.1101/2<IP_ADDRESS>0010

MSID: 101613
Version: 1
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>0010
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=101613
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>0010

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/101613v1
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>0010
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/101613/v1/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

@fred-atherden This doesn't seem to have made it to the meca/reviewed-preprints folder.

Hmm - the reviews were posted to the v2 which was only published a few days ago.
We don't have access to the MECA package via the API yet:
{
  "status": [
    {
      "count": 1,
      "messages": "ok"
    }
  ],
  "results": [
    {
      "msid": "610010",
      "tdm_doi": "10.1101/2<IP_ADDRESS>0010",
      "ms_version": "1",
      "filedate": "2024-07-25",
      "tdm_path": "s3://transfers-elife/biorxiv_Current_Content/August_2024/28_Aug_24_Batch_2083/dec97075-702c-1014-96e5-b8d40e0ed520.meca",
      "transfer_type": "api"
    }
  ]
}

My suggestion would be to wait until tomorrow to see if the MECA path appears for the V2 in the above API response. if not, then we'll need to contact bioRxiv and ask them to provide access.


remove spaces

Ref: Conversation #3131 / Benoît

I have a prototype in use now. Do you have some more information on helpscout.net?

Ref: Conversation #5494 / Jie

Ref: Conversation #5539 / Paco

Ref: Conversation #6119 / Jianwen

Ref: Conversation #6221 / Jparr.com


package author has not included the right index files etc so npm can load the ts using the shorthand notation.

package author has not included the right index files etc so npm can load the ts using the shorthand notation.

Documentation dosent seems correct eg:
the README says MultiselectDropdownModule, but I think it's a typo and should be MultiselectDropdown

I implemented this package in my local test-application. importing MultiselectDropdownModule worked just fine in the app.module.ts file
The actual component you build should work with IMultiSelectOption, IMultiSelectTexts, IMultiSelectSettings imported.


Property 'globEager' does not exist on type 'ImportMeta'.ts(2339)

Describe the bug

Reproduction

System Info
Output of npx envinfo --system --npmPackages vite,@vitejs/plugin-vue --binaries --browsers:

Used package manager:
Logs

Before submitting the issue, please make sure you do the following

[x] Read the Contributing Guidelines.
[x] Read the docs.
[x] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[x] Provide a description in this issue that describes the bug.
[x] Make sure this is a Vite issue and not a framework-specific issue. For example, if it's a Vue SFC related bug, it should likely be reported to https://github.com/vuejs/vue-next instead.
[x] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.

//index.ts
import * from 'vite'
const modules = import.meta.globEager('./src/baseCmpts/*.svelte')

please give a reproduction，it seems that something went wrong with your code,


Contributions Bot Work

@all-contributors please add Abhijit2505 for python

@all-contributors please add Abhijit2505 for mentoring

@all-contributors please add Abhijit2505 for code

@all-contributors please add pixan198 for code

@all-contributors please add hk2608 for code

@all-contributors please add Himanshi2016 for code

@all-contributors please add ishi0301 for code

@all-contributors please add chaitali0210 for code

@all-contributors please add AfraMP for code

@all-contributors please add MercyRabecal for code

@all-contributor please add pg00019 for code

@all-contributors please add pg00019 for code

@all-contributors please add Rsr0 for code


Ignore id when on_update is true

For example, I have this sequence when on_update is true:
I created "My Blog Post" (id:1) that created "my-blog-post" slug.
I create another "My Blog Post" (id:2) that created "my-blog-post-1" slug.
I updated "My Blog Post" (id:2) that created "my-blog-post-2" slug.
I updated "My Blog Post" (id:2) that created "my-blog-post-3" slug.
I updated "My Blog Post" (id:2) that created "my-blog-post-4" slug.
I updated "My Blog Post" (id:2) that created "my-blog-post-5" slug.
...

This is the same problem I get.

Why do you do this? You are not supposed to call this manually on every update!
$recipe->resluggify();
This might be your problem since you are forcing slugify to update the slug on each update based on the title. If you want such behaviour you need to tell slugify to do this by setting on_update setting to true.

Excuse me for my bad English!
I have problem with on_update setting. I don't use resluggify() method manually.

do you have use_cache set to true? if yes try setting that to false and see if this solves your problem.

use_cache has default false value.


NotFound and MethodNotAllowed still doesn't work in some cases

Still in some cases it doesn't work ..I'll try to write examples soon

Please re-open with concrete examples.


Use updated converters on .designspace files

Begin to plumb actual unit types, established in #56, into system. Store a converter in StaticMetadata, serializing it in the standard user:design form (https://github.com/googlefonts/fontmake-rs/blob/main/resources/text/units.md#user--design).
Builds on https://github.com/googlefonts/fontmake-rs/pull/43. Step toward https://github.com/googlefonts/fontmake-rs/issues/22.
Clean copy of #47, this time against the units branch which hopefully addresses the core problems.

I think I like how individual units convert now, the conversion of sets of units could use some polish but it feels to me like it's moving in the right direction.

Per IM from dentist (dedication!), OK to merge if the min/max validation is added.


WIP: Update Apache Commons Pool to 2.4.3 and fix unit tests

Updated in pom.xml to pool 2.4.3. Started the tests in Maven command line, found which tests failed. Executed the tests in Eclipse, found which class was related to the failure. Then did a diff between both tags.
git diff POOL_2_4_2 POOL_2.4.3-RC1 -- ./src/main/java/org/apache/commons/pool2/impl/DefaultPooledObject.java

The short diff indicates that the failures started possibly due to the replacement of Exceptions in the DefaultPooledObject by a CallStack. The CallStack seems to not keep track of all the methods (due to a security context manager from what I could tell).

See discussion in the mailing list about recent changes before merging it. Added the WIP prefix to the pull request.

Closing a per discussion http://markmail.org/thread/hsg6qxsxofxiggmo#query:+page:1+mid:pmyjbzyjc26kbljs+state:results


Evaluate: Numbers cannot be recognized if there are signs in front of them (Basic)

Numbers cannot be recognized if there are signs in front of them
When calculate with numbers which has signs in front of, it returns an error: Uncaught TypeError TypeError: Cannot read properties of undefined (reading 'func')
-> const math = require('advanced-calculator');
undefined
-> math.evaluate("1")
1
-> math.evaluate("+1")
Uncaught TypeError TypeError: Cannot read properties of undefined (reading 'func')
    at eval (<project>/node_modules/advanced-calculator/src/Basic/shunting.js:260:14)
    at eval (<project>/node_modules/advanced-calculator/src/Basic/shunting.js:264:34)
    at evaluate (<project>/node_modules/advanced-calculator/src/Basic/index.js:11:15)
    at <anonymous> (repl:1:6)
-> math.evaluate("-1")
Uncaught TypeError TypeError: Cannot read properties of undefined (reading 'func')
    at eval (<project>/node_modules/advanced-calculator/src/Basic/shunting.js:260:14)
    at eval (<project>/node_modules/advanced-calculator/src/Basic/shunting.js:264:34)
    at evaluate (<project>/node_modules/advanced-calculator/src/Basic/index.js:11:15)
    at <anonymous> (repl:1:6)
-> math.evaluate("-1-1")
Uncaught TypeError TypeError: Cannot read properties of undefined (reading 'func')
    at eval (<project>/node_modules/advanced-calculator/src/Basic/shunting.js:260:14)
    at eval (<project>/node_modules/advanced-calculator/src/Basic/shunting.js:264:34)
    at eval (<project>/node_modules/advanced-calculator/src/Basic/shunting.js:264:34)
    at evaluate (<project>/node_modules/advanced-calculator/src/Basic/index.js:11:15)
    at <anonymous> (repl:1:6)
-> math.evaluate("(-1)-1")
Uncaught TypeError TypeError: Cannot read properties of undefined (reading 'func')
    at eval (<project>/node_modules/advanced-calculator/src/Basic/shunting.js:260:14)
    at eval (<project>/node_modules/advanced-calculator/src/Basic/shunting.js:264:34)
    at eval (<project>/node_modules/advanced-calculator/src/Basic/shunting.js:264:34)
    at evaluate (<project>/node_modules/advanced-calculator/src/Basic/index.js:11:15)
    at <anonymous> (repl:1:6)

Thanks for reporting this. It has been fixed in the PR below:
https://github.com/maan-patel/advanced-calculator/pull/3


Migrate to CircleCI 2.0

My reason:
https://circleci.com/blog/sunsetting-1-0/
Additional information:
I'll explain step by step what we are doing in a nutshell on circleci.ylm

YARN_VERSION: 1.6.0
verdacciobot user will perform all task
Testing with node 6, 8, 9 and 10
Default node should be 9
yarn run pre:ci first step before running anything else
- yarn run test:e2e should be run once using default Node
Publish docusaurus after (master branch) is being successful (right now it publish even from branches and that is wrong)
Deployment: This is really important

After a new tag is being detected we should deploy on npmjs /(v)?[0-9]+(\.[0-9]+)*/
l10n_master master branch should be ignored for builds (used by crowdin and it is nog being ignored currently).
gh-pages is being used by docusaurus and also should be ignored.

Upgrade

We should keep the same configuration
Fix what is not working properly
Deployment is essential
Wish list:

Use of artifacts ( to measure the size of the tarball in each PR)

Any another improvement that improves the workflow and maintainability

Testing

We should test in a separated repository (we cannot publish it accidentally on npmjs)

is that issue up for grabs or anyone is already working on it ?

@pgrzesik Yes. this issue is up and you can start to working on it. :)

We need to define first what we are covering so the migration is clean.

@pgrzesik wanna work on it?

@juanpicado did you think on moving to Travis-CI or prefer working on CircleCI? It would be a nice. I can study the upgrade before deadline or move to other alternative like Travis (I use several projects and understand something about how to build with it) :wink:

That would be cool  @sergiohgz ! The next week I'll add more detail about what we are doing now and what we need in 2.0 aside of migration.

@juanpicado great, this week i can make a PR with the changes. It's pretty cool how workflows work (better than Jenkins Pipelines)

@sergiohgz update, please let me know if is clear or you need more information. You can find me around https://gitter.im/verdaccio/ as well.

Let's keep it open while I test a while.

Impressive job @sergiohgz  🥇 builds now are much faster than the previous configuration.

Really nice to know an opinion about it. Caching across builders is so great, so I will use CircleCI in every project I can :smile:

Sorry, now I have bad news, something went wrong with the coverage.

Fixed the coverage 🚀 !! ... I'll test in the next days the deployment. Then we can close this ticket.

@sergiohgz deployment to npm did not work  :-(
https://circleci.com/workflow-run/25119d3b-2931-49fe-b8e5-de29e1e631f1

@juanpicado I am going to test that flow :wink:

Reviewing that, I found two problems:

Workflow was working only when pushing on branches, but not for tags
This weird: link. Basically, it says that builds triggered by tags will not be shown in workflows

The test I have done is trigger a release (failed because I haven't power to publish in registry) in my branch and examine the process of publishing:

First weird thing: When you run yarn release and perform a git push --follow-tags, it launches two execution, one for branch push event and other for tag push event
Second weird thing (explained in CircleCI discussion forum): You won't see the tag push triggered execution on Workflows view, but you can go to Builds view and look for that build that has no commit message

It looks better with an illustrated example:

First I push commit on branch but not the tag. This means that publish_package step is not running on branch push events:

After a while, I push the tag to see if the publish_package step will run. Here we can see that it was executed (but I cancelled it before because I have no power to publish on registry):

A final image with the complete test is a great example:

The example is not complete without an image of the Workflows view, where we can see the branch push execution:

This doesn't means that you cannot push tag and
I am going to remove that test from branch and create a PR :wink:

My branch has published a new website, I think the ignore_branches does not work proprely
 - publish_gh_pages:
          requires:
            - test_node6
            - test_node8
            - test_node9
            - test_node10
            - test_e2e
          <<: *ignore_branches

Let me know the branch name and, if you remember, the build number in CircleCI, to see in which conditions it executed. Thanks

:-( it fails ... but from local works fine. I dunno whether is our fault. I just stopped to work for no reason.
https://codecov.io/gh/verdaccio/verdaccio/commits

@sergiohgz fixed the coverage. We just have pending the publish on npm issue.

@juanpicado I have just tried to publish within my CircleCI and my npmjs account and it works well, I think it should work :wink:

@sergiohgz @juanpicado Great work here 👏


[Feature Request]: Option to disable "Full Screen" button

Choose the extension for feature request.
Full Screen Mode
📝 Provide a description of the new feature
I only ever use TV Mode on my other monitor, I never use the "full screen" mode itself. (I occasionally use the stock Spotify one, it looks better on my vertical monitor; the lyrics on-screen can't be beat though!)
So an option to hide the icon on the lower right would be awesome, since I get it confused with the stock fullscreen button. I love that you can hide that one but I need the opposite haha.
➕ Additional Information
No response

Perfect! And the settings and everything look so nice <3


fix: Inconsistent new lines in worksheets

fix https://github.com/nissl-lab/npoi/issues/1401

LGTM


Update README.md

Purpose

...

Does this introduce a breaking change?

[ ] Yes
[ ] No

Pull Request Type
What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

How to Test

Get the code

git clone [repo-address]
cd [repo-name]
git checkout [branch-name]
npm install

Test the code

What to Check
Verify that the following are valid

...

Other Information

@zhenlan @amerjusupovic Could you approve it again?

Approved and removed the requirement for reapproval for the branch.


Suggestion for txt2al: Show the related Line No. of that error

It would be nice if I could see the related Line No. in the Delta-File of a txt2al error (see #2369 and #2370). So I could at least see were the problem is coming from.

Hi @DanielGoehler, the txt2al returns warnings when facing unsupported features. The two issues you mention seem to be bugs from the converter that we have to fix, so I am not sure how knowing the lines would help you here.

@qutreson I just want to report at problem as generic example and not the hole Delta-File(, which contains parts of our IP.)


Release 1.2.2

Coverage increased (+0.007%) to 32.666% when pulling 74c151661bf33114ce645454455ee0f4def17973 on release_1.2.2 into 3e794ee01878df1eeedb1a199d33feadac83e30f on stable_1.2.


[Feature Request] Download debug vc dlls

Firstly thanks for this project, it solves the exact issue I had, x86_64-pc-windows-gnu has a different c++ exception handling abi to x86_64-pc-windows-msvc.
I am actively developing for a windows target, so I want to be able to debug my code. Downloading with --include-debug-libs --include-debug-symbols makes compiling a debug build work perfectly. The issue is, this build links against vcruntime140d.dll + related, dlls which wine obviously doesn't have. It would be nice if there was an option to also download these dlls to a folder somewhere, so you don't need to grab them from a full VS install. I don't think there's any reasonable generic way to install them, better to just leave copying them to the user.

I'd accept a PR for this, but won't do one myself, at least in the foreseeable future. I think the easiest would just be a new option --debug-dylibs <path> that would just dump the debug DLLs into the specified path, separate from wherever the rest of the contents are being placed, since this project is primarily focused on getting the headers/import libs needed for compilation, not runtime.


Googleless options?

Are there plans for non-Google authentication mechanisms for those of us who don't have a Google account?

Yeah, definitely. At least, the old good username+password authentication would be nice to have. Or, maybe auth via GitHub is more appropriate, since the primary target audience of Geekmarks is probably tech people. What's your opinion?

Since most of them rely on OAuth authentication, it should be easy to integrate.

Any of those other should be fine. Pure email/password is always nice for us picky purists and there are probably a horde of libraries already ready for that.
2FA is always a nice add-on feature to grant a bit more security (again, tons of ready solutions for that).
https://github.com/search?q=TOTP

@dimonomid are you actively maintaining geekmarks? Also, did Github login option get added to geekmarks or is this issue is still open?

I'm willing to fix bugs, but I don't have time to work on features. PRs are welcome though.


Windows 10 Soundbar Configuration

Description of the new feature / enhancement
That the user is able to manage the size of the bar, and the amount of time it is shown.

Scenario when this would be used?
It is used when the user wants to modify its behaviour, and then it's applied to the sound bar, every time the music is paused, or the volume is changed
Supporting information
I hate that I have to wait 5 seconds for that stuff to go away

Please file feedback for this in the Feedback Hub (Link only opens on Windows)


[HOLD][Tracking Issue] Implement HTML -> Markdown conversions in Expensimark

Problem
Expensify.cash uses the Expensimark library in the expensify-common repo to convert markdown to HTML. That way, a user can compose a message using markdown, and it can be displayed + stored using HTML. However, when a user later goes to edit the message, we currently have no way to take the HTML and display it to them as markdown. We also don't want to store the original markdown in our database.
Why this is important
Editing comments is an important usability feature in Expensify.cash. However, we need to make sure that the edit-comment experience is as similar as possible to the original message composition experience.
Solution
Implement bi-directional HTML <---> Markdown conversion in Expensimark for all of the elements we currently support for Markdown ---> HTML conversion:

[ ] Code fence
[ ] Inline code blocks
[ ] Email links
[ ] Hyperlinks
[ ] Bold
[ ] Italics
[ ] Strikethrough
[ ] BlockQuotes
[ ] Newline /

HOLDing this on a POC PR over here. We might need to do an internal PR to update our other repos that also use Expensimark, depending on how we change the Expensimark interface.

@roryabraham I was just thinking about this–and sorry if this has already been considered–but why don't we just store both the HTML and Markdown versions of the report comment, and switch between the two for editing/copying report comments?

@roryabraham I was just thinking about this–and sorry if this has already been considered–but why don't we just store both the HTML and markdown versions of the report comment, and switch between the two for editing/copying report comments?
To minimize space, we can only store the HTML if the markdown and the HTML are identical (and default to it if the markdown version doesn't exist).
Feel like this would be a whole lot simpler than doing a bunch of string manipulation on HTML to make it look like markdown–we'd probably also avoid a bunch of bugs this way.

@jasperhuangg has raised a very good point. But I think it in another way. I would say that we should save the MD as I believe converting HTML to markdown will loose original MD markup which user entered.
And we can always convert the MD to HTML uniformly.

I don't necessarily disagree @jasperhuangg, but this was discussed in slack here. If you'd like to reopen the discussion, I think Slack would be the best place for that so it gets better visibility. The reason we didn't go with that solution is that it will nearly double the amount of data we need to store in our largest table.
But @parasharrajat has introduced an interesting solution I don't believe was discussed before – only store the plain-text (markdown) version of a comment, then convert it to HTML only when needed.

Raised the proposal to store only markdown in this slack thread.

Moving forward with the original plan, I'm taking this off hold.

For reference, this PR was merged as a POC, and it's being integrated into E.cash here.

@Jag96 We can mark finished tags on the list.

Marked off the finished tags, left
 unchecked and linked the bug we found

@Jag96 Let's open another issue for other tags. I see many complaints about it.

Make hyperlinks editable in chat comments
Action Performed

Send this a link using the  hyperlink markdown to a conversation
Click on edit the comment and save without making any changes

Expected result
Hyperlink markdown should be visible when editing the comment
Actual result
Hyperlink markdown is removed when editing the comment and then it's saved as plain text.
Notes/images/Video
https://user-images.githubusercontent.com/44479856/126387580-547a11bd-0f95-4484-ac31-b85118b164fb.mp4

From @joaniew https://expensify.slack.com/archives/C01GTK53T8Q/p1626810543044700

When I edit a chat that has a link to it already, the link turns into plain text and removes the hyper link, and I can’t edit it again to make it a link

@parasharrajat it sounds like your proposal for https://github.com/Expensify/App/issues/4169 will solve Hyperlink markdown is removed when editing the comment and then it's saved as plain text. right? If so, I'd like to hold off on adding the other HTML->MD conversions until the ones we currently support are behaving the way we expect them to.

Ok sounds good, I'll make new issues then for each of the items in the checklist

@Jag96 Links and emails links can go into one. There will be one regex for both.

Quote support implemented here, integrated into E.cash here
Inline code blocks added here, not yet integrated into E.cash, PR is here
We've had some proposals but no movement yet on hyperlinks and email links.

Looks to me like everything except https://github.com/Expensify/App/issues/4188 is done or almost done

That is also done.

Looks like all the issues on our checklist are done here. I think I am going to ask Applause to do some extra QA here before we close this out for good.

Additional QA steps I would like to have done:

Create a message with each of the markdown features in the issue description above.
Attempt to edit the message. Verify that the markdown you entered when creating the message reappears in the edit box.
Perform additional exploratory QA around editing messages containing markdown in New Expensify. Try to break it and let us know how it goes. 🙃

QA requested in Slack


Snapshot restoration issues

When performing an upgrade following these instructions, at the step where I am restoring from my  full snapshot, I am having issues with it restoring everything.  It is failing to restore my customize, automation, iframe, and lights yaml files. It also fails to restore my nest.conf file. I can add them back manually and keep on going but it is strange that it is not putting everything back. I am coming from a 32 bit RP3B upgrading to a 32 bit RP3B+.
Thanks for all your hard work!

I haven't had an issue restoring Home Assistant or any items within /config or /share, but it failed to restore any addons which were not from core or community repos. Adding the missing repos manually to the new install allowed the snapshot to restore the missing addons. So somehow the repositories are not being restored.

@cogneato interesting, need look into this issue.
@Billabongodysee. Could be that you come from raspbian? It can only restore if the permission of files are correctly. Haspbian/raspbian user run in a context of a other user. If you edit your config files without a hass.io add-on, they will be fails. I think that I need reset the user permission before we do a snapshot to avoid this issue.

/move home-assistant/hassio


Provide a command to dump info to help report an issue/bug

If you run Help: Report Issue you get a pop-up window filled with system details that get used to pre-populate an issue. It would be great if we could do something similar for the details we have in our issue template by popping open a new tab with Markdown-formatted details.

We should use the full power of GitHub-flavoured Markdown as well, e.g. see https://github.com/Microsoft/vscode-python/issues/1125 and it's collapsible sections.

We might want to keep an eye out on the following issue https://github.com/Microsoft/vscode/issues/45673

Collection of user env may not be appropriate since we need to observe data collection and retention policies. 'Report issue' command may be suppressed by corporate IT policies and we can't bypass that.

This is to generate a Markdown file that people can copy and paste into GitHub, so there's no IT policy that we would be going against unless a web browser is against policy. 😉

@rchiodo /cc (found it)

We're going to close this for now. We'll revisit with a new issue in the future if needed.

Not sure why this got a "data science" label (because of the "-review-" label?), but we actually hope to get to this some day.

Not sure why this got a "data science" label (because of the "-review-" label?), but we actually hope to get to this some day.

I was informed these would get closed. I added ds flag as we need this feature and xteam to benefit core team.

Look for additional suggestions here: https://github.com/microsoft/vscode-python/issues/12109#issue-629544027

Completed in https://github.com/microsoft/vscode-python/pull/17568


add ndmctsgh.txt

@chlee0626 The organization seems to be not an educational one. It looks like a military government organization. Due to security concerns, it even blocks all connections to its web server from outside of some specific area. The website blocks even connections from Taiwan.
Therefore, I cannot add it to this repository for at least two reasons:

It does not qualify, because it's not a university or a college;
It's impossible to verify the information about the organization, because its website blocks all connections.

If you are a student in this organization, please apply for free educational license with an official document which proves that you are a student. You can do it using "OFFICIAL DOCUMENT" tab at https://www.jetbrains.com/shop/eform/students.
I have to close the request without merging.
Thank you for your understanding!

Dear Philip,
National Defense Medical Center (NDMC) is a military medical college,
which is composed of nearly 200 professors and 1700 students. Our
school is authorized by Ministry of Education and Taiwan Medical
Accreditation Council (TMAC) to provide medical education for
students.  With the rise of machine learning, we focus on the
application of data science and technology in the medical field. We
sincerely want to join this program and learn programming with python
Thank you for your understanding!
Best regard,
Lee
Philip Torchinsky @.***> 於 2023年2月2日 週四 下午3:05寫道：

@chlee0626 The organization seems to be not an educational one. It looks like a military government organization. Due to security concerns, it even blocks all connections to its web server from outside of some specific area. The website blocks even connections from Taiwan.
Therefore, I cannot add it to this repository for at least two reasons:
It does not qualify, because it's not a university or a college;
It's impossible to verify the information about the organization, because its website blocks all connections.
If you are a student in this organization, please apply for free educational license with an official document which proves that you are a student. You can do it using "OFFICIAL DOCUMENT" tab at https://www.jetbrains.com/shop/eform/students.
I have to close the request without merging.
Thank you for your understanding!
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>


Equals validator

Created the new equals validator.
Main use case: password checking.
If this PR is accepted, I shall work on some better docs. I have already created a new set of examples here: https://codepen.io/luca-vercelli/pen/KKzMJvR.

@luca-vercelli, Thank you. I'd recommend using document.querySelector (instead of document.getElementById) and passing the whole thing to it. This will make it versatile and not restricted to id only. e.g.
_('equals', { fn: (val, other) => { let otherField = document.querySelector(other); return (!val && !otherField.value) || (otherField.value === val);} });

Also, could you please follow the existing pattern of writing in one line (whenever possible). This make it easier to understand the already available validators and where it's defined.
Another thing, I think adding the validator name to ALLOWED_ATTRIBUTES is not necessary. This is for built-in attributes only. Could you please check? I think it needs a better naming though.

@sha256 I modified the code as requested. I did not rename the validator, as I have no other good name for it.

Merged. Thank you. I was talking about renaming ALLOWED_ATTRIBUTES to something more relevant like ALLOWED_HTML5_ATTRIBUTES or something like that. I'll do it later :)


Clear inspector and weird bug

Clear inspector on store open

Weird bug where grid overlay open/close very inconsistently when store opens

Let me know if I should split these into two PRs


An extra 'Tables' node displays under one Blob account

Storage Explorer Version: 1.4.0
Platform/OS Version: Windows 10/MacOS High Sierra/ Linux Ubuntu 16.04
Architecture: ia32/x64
Build Number: 20180724.5
Commit: 2605f1c9
Regression From: Previous release 1.3.1(20180711.1)
Steps to Reproduce:

Expand one Blob account then check the children nodes.

Expected Experience:
No 'Tables' node displays under the account.
Actual Experience:
An extra 'Tables' node displays under the account.

This is by design. Metrics can be collected for blob accounts and stored in tables. The Tables node here lets users view the account metrics.


Remove sass dependency

This closes #1064
Given that Ruby sass is now deprecated and developers are suggested to move to sassc, we should stop specifying sass as a runtime dependency.
Also, since it seems like we required sass only just to add to load_paths, it should be safe to silently fail if it is not available.
Note that there's some caveat about this patch: sassc (or, precisely, sassc-ruby) no longer has load_paths setting at the class level. User has to set load_paths themselves when they create SassC::Engine instance. This probably will affect people who are using sass in ruby but without Rails environment. I currently see no mention of this specific usage the README.md, so I think we shold be fine. (People who uses Rails and Asset Pipeline or Webpack should see no problem as they Railtie continues to work as expect).

Quick update here: We spent a bit of time testing this out today, and while it’s fine in Rails, we are indeed hitting the caveat mentioned above. Outside Rails—for example in a Middleman app—these changes create a Sass import error, as Bourbon cannot be found.
Middleman 4.3 has moved off Ruby Sass and uses SassC, but without it supporting load_paths, I'm unsure how to set Bourbon up for those situations. I'm hesitant to merge this until we have a better grasp on that front.
I was using our apprentice.thoughtbot.com repo as a test ground for this, in case anyone wants to try it.

Thanks @sikachu! This has been merged as https://github.com/thoughtbot/bourbon/commit/9bedac008832cb779ded2f9298f12a5ebae9e88a.


How to updates location.hash but NOT resolve URL

as far as I can tell, the only way to do this is the following:
window.history.pushState({"page":url},"", url);
Any plans on creating a short cut for this one? :)

When would that be useful?


Bugfix variable reference before assignment

Prevent non-valid image paths passed to ImageReward.score() resulting in a local variable pil_image referenced before assignment error.

Cool! Thanks a lot!


Add Manifest to compiled project

<IP_ADDRESS>
Python: 3.7.7 (tags/v3.7.7:d7c567b08f, Mar 10 2020, 09:44:33) [MSC v.1900 32 bit (Intel)]
Executable: C:\Users\Moein\AppData\Local\Programs\Python\Python37-32\python.exe
OS: Windows
Arch: x86

how to add manifest run project as administrator in windows?

 Can you point to an example of those?

@kayhayen in pyinstaller have switch --uac-admin and add manifest for run as administrator exe file , can you add this switch in nuitka?

Nuitka has gained the option --windows-uac-admin and --windows-uac-uiaccess on the factory branch. The change the manifests as needed.

Released as part of 0.6.10


enchancements redisson-quarkus-20 native mode and config

created enhancements for running the redisson extension in quarkus and especially native mode.

got rid of --allow-incomplete-classpath
made bean runtime configureable
removed some of offending spring dependencies from compilation tree (moved to spring specific impl of TaskInjector)
added testcontainers to easily start up a redis instance (if a docker environment is available)
upgraded to quarkus 2.0.3.Final
use already existing netty extension
changed only redisson-quarkus-20

Thank you for proposed changes.
According to your descriptions, it covers 7 tasks. Please split it into PR per task.

Thank you for proposed changes.
According to your descriptions, it covers 7 tasks. Please split it into PR per task.

Hi,
whats the point in doing that? Anyways those changes are dependent for example if I just create a PR to remove --allow-incomplete-classpath it would be meaningless since the native image would not work anymore.
The only things which could make sense but I don't see why it would make a practical difference would be splitting.

upgraded to quarkus 2.0.3.Final
made bean runtime configureable
make native image build possible without --allow-incomplete-classpath

whats the point in doing that?

To understand which changes to related to task.

any update?

@ineednousername
Three tasks are enough.

@mrniko
sorry for not reacting here.
I will get back to this as soon as I have some time to work on my proposed changes.

That's correct. Also please include only changed lines.

native image without --allow-incomplete-classpath --> most likely also relates to solve https://github.com/redisson/redisson/issues/3774

Exactly 😊

* native image without --allow-incomplete-classpath

https://github.com/redisson/redisson/pull/3782


Format

Current coverage is 43.68%

Merging #59 into master will increase coverage by +0.16% as of 27ef2c2

@@            master     #59   diff @@
======================================
  Files           43      43
  Stmts         1089    1085     -4
  Branches       251     251
  Methods          0       0
======================================
  Hit            474     474
  Partial         17      17
+ Missed         598     594     -4

Review entire Coverage Diff as of 27ef2c2

Powered by Codecov. Updated on successful CI builds.


Access denied windows server

Good morning, when I want to log in with any user it gives me access denied. rpwrap is all green and I connect to the server but the only user that lets me access is the administrator, any other user does not let me, I already deleted the user and created it again, I also checked the permissions and everything is ok. I attached screenshots
Archivos.zip

Technically it's duplicate of #2080 / #2224 / #2227 => #391 (which has been addressed in the installer).
Feel free to use ProcMon or API Monitor to check where the privileges are missing.

[10.0.17763.4252]
SingleUserPatch.x64=1
SingleUserOffset.x64=17362
SingleUserCode.x64=mov_eax_1_nop_1
DefPolicyPatch.x64=1
DefPolicyOffset.x64=1FF75
DefPolicyCode.x64=CDefPolicy_Query_eax_rcx
LocalOnlyPatch.x64=1
LocalOnlyOffset.x64=8A321
LocalOnlyCode.x64=jmpshort
SLInitHook.x64=1
SLInitOffset.x64=2C06C
SLInitFunc.x64=New_CSLQuery_Initialize
[10.0.17763.4252-SLInit]
bServerSku.x64=114D20
bRemoteConnAllowed.x64=114D30
bFUSEnabled.x64=114D3C
bAppServerAllowed.x64=114D2C
bMultimonAllowed.x64=114D34
lMaxUserSessions.x64=114D24
ulMaxDebugSessions.x64=114D38
bInitialized.x64=114D1C


Bump pre-commit black version to fix pre-commit

See https://github.com/psf/black/issues/2964 for upstream issue

I'm going to self-merge since this is a trivial change that fixes CI, so other PRs can be worked on.


chore: upgrade charts

updated chart jxgh/jxboot-helmfile-resources from 1.1.111 to 1.1.112

/close


Remove GoFish instructions to install 'flux'

It's been archived.
Fixes: #851
Signed-off-by: Daniel Holbach<EMAIL_ADDRESS>

Thanks @kingdonb for noticing this!

https://deploy-preview-852--fluxcd.netlify.app/docs/installation/
https://deploy-preview-852--fluxcd.netlify.app/docs/cmd/


Barnabas-Agile-Development week 4

my fork of module repository

Learning Objectives

Week 4
I Need Help With:
nothing I could not find on Google.
What went well?
collaborating with team mates.
What went less well?
having everyone at the same time
a bit confused with planning
Lessons Learned
working with people
Sunday Prep Work
in the process

Week 5

[ ] I have pushed my progress to my fork of exercises repo

I Need Help With:
nothing for the moment
What went well?
collaborating more with the teem
What went less well?
time  management  to finish the group projects
Lessons Learned
better gasp on css desponsivness
Sunday Prep Work
on it!


Writes file name of source maps, not the JS files themselves

Command I'm running:
NODE_ENV=development webpack --progress --colors --watch -d

Webpack output:
Hash: 9b913c8594ce98e06b21
Version: webpack 1.8.5
Time: 2638ms
                              Asset     Size  Chunks             Chunk Names
    desktop.js?9b913c8594ce98e06b21  75.8 kB       0  [emitted]  desktop
     mobile.js?9b913c8594ce98e06b21    62 kB       1  [emitted]  mobile
desktop.js.map?9b913c8594ce98e06b21   122 kB       0  [emitted]  desktop
 mobile.js.map?9b913c8594ce98e06b21  95.8 kB       1  [emitted]  mobile

webpack-assets.json:
{"desktop":"desktop.js.map?9b913c8594ce98e06b21","mobile":"mobile.js.map?9b913c8594ce98e06b21"}

If I disable outputting of source maps, it works fine.

Pushed v0.3.0 to NPM which I think resolves the issue, according to my specs anyway, could you have a try and let me know?

Yeah it seems fine now, thanks!


[BUG] FlashAttention安装成功但使用报错

是否已有关于该错误的issue？ | Is there an existing issue for this?

[X] 我已经搜索过已有的issues | I have searched the existing issues

当前行为 | Current Behavior
在Linux上配置环境，在未安装FlashAttention的时候一切正常，按照教程安装FlashAttention没有报错，但运行模型推理的时候报错RuntimeError: Expected is_sm90 || is_sm8x || is_sm75 to be true, but got false. (Could this error message be improved? If so, please report an enhancement request to PyTorch.)
期望行为 | Expected Behavior
No response
复现方法 | Steps To Reproduce
No response
运行环境 | Environment
- OS:
- Python:
- Transformers:
- PyTorch:
- CUDA (`python -c 'import torch; print(torch.version.cuda)'`):

备注 | Anything else?
No response

transformers的版本是 Version: 4.31.0

I have met the same problem. Have you solved it?

I have met the same problem. Have you solved it?

not yet

@Zplusdragon @ryanhe919 Please check whether your GPU supports FlashAttention, only Turing (sm75), Ampere (sm80, 86, 89) and Hopper (sm90) are supported.

 @Zplusdragon @ryanhe919 The GPU types of Turing, Ampere and Hopper frameworks.

thx a lot, im using v100 and look like it's not suitable for flashatt


lo and/or /32 not allowed

I have pimd 2.3.1, kernel 4.3.5 on debian amd64.
To get routing in the right direction, I have a route like:
root@host001# ip -d route show <IP_ADDRESS>/4
unicast <IP_ADDRESS>/4  proto boot  scope global
    nexthop via <IP_ADDRESS>  dev enp3s0 weight 1
    nexthop via <IP_ADDRESS>  dev enp4s0 weight 1

I noticed via netstat -g, that a multicast source will register against a specific interface.  If that interface goes down, the source won't move to another interface.
So try a different a form of resiliency, independent of the exit interface, I looked at alternate options.
At LCM, they show a route to the loopback.  Which makes sense.  I would like my RPs on the loopback, much like what can be done on Cisco devices.  I can add the following route, and it shows in the routing table.
ip route add <IP_ADDRESS>/4 nexthop via <IP_ADDRESS>

I can do the following to get multicast on the loopback:
ip link set lo multicast on

I even have an additional ip address on the loopback, and confirms multicast is set:
root@host001# ip addr show dev lo
1: lo: <LOOPBACK,MULTICAST,UP,LOWER_UP> mtu 65536 qdisc noqueue state UNKNOWN group  default
    link/loopback 00:00:00:00:00:00 brd 00:00:00:00:00:00
    inet <IP_ADDRESS>/8 scope host lo
       valid_lft forever preferred_lft forever
    inet <IP_ADDRESS>/32 brd <IP_ADDRESS> scope global lo
       valid_lft forever preferred_lft forever
    inet6 ::1/128 scope host
       valid_lft forever preferred_lft forever

I have phyint in pimd.conf:
phyint <IP_ADDRESS> enable

Pimd seems to pick up physical interfaces, but I can't seem to get it to recognize the lo device.  The following debug doesn't indicate why it doesn't pick up lo.  lo won't show up in the vif table.
pimd -f --debug=routes,peers,pim_routes,interface

Also, I notice that I am unable to use the 'rp-candidate local-addr' with an interface with a /32 address.  So that eliminates the lo as a suitable source of an rp location.  Is that by design? It seems possible in cisco land though.
Is this a pimd or a kernel issue?  Bug / feature?
Are there other ways to obtain some resiliency on the exit interfaces?
Thanx.

Hi @rburkholder :smiley:
First, traditional Cisco gear is quite different from Linux, so using loopback as is done there does not always apply. Even though some concepts used by them can be really handy.
Now, you figured out that you need to set the MULTICAST flag on loopback, so that's a start. But pimd (currently) does not allow /32's ... yeah, I don't think it's needed but there's a lot of legacy code in there I haven't dared to touch yet.  The configure script supports a --disable-masklen-check flag that you can try out, but I don't think you really need this anyway, see further down.
Now, when using a multicast routing daemon like pimd, mrouted, or smcroute you don't need to bother with setting up unicast routes to <IP_ADDRESS>/4. The multicast routers have their own routing tables that work with VIFs, virtual interfaces that map on top of physical interfaces (phyints).  With the former two multicast routes are installed in the kernel dynamically and with the latter smcroute routes are installed statically.  The former two are therefore more suitable for resilient setups.
However, I'm afraid I don't understand fully what it is you want to do or how much experience you have with PIM.  But in most cases you just need to fire up pimd with the default pimd.conf as argument (see -h for syntax) and you're off, basically.
For a crash course you need:

A multicast sender which is connected to your router, if that network is smarter than an unmanaged switched network you must verify that this network can learn of your routing daemon and direct all multicast towards your router.
The multicast sender must send multicast with a TTL >1 ... multicast is usually treated as broadcast and always default to TTL 1 to prevent "broadcast" from being routed.
On the receiver side you either need another PIM router, or a receiver that is capable of responding to IGMP Queries with an IGMP Join for the multicast group it wants to receive.  The join is (simplified) what triggers pimd to dynamically install a multicast route into the kernel.

Use the tools pimd -r, cat /proc/net/ip_mr_vif, and cat /proc/net/ip_mr_cacheto see what your PIM daemon does and what it installs in the kernel.  The latter can be parsed by a regular human using theip mroute` tool.

Thank you for your response.
I am just getting started with pimd, so I am doing a lot of 'brute force and ignorance' style of testing to get something working.  I do have something working now. An outline of the configuration can be found at Linux VXLAN with pimd, Quagga, and openvswitch.
I am going to have to spend some quality time and watch traces to build up my comfort with the group registrations.
For the routing, as outlined in my blog item, I have a default route going to a management interface, rather than towards the core.  I need to fix that, and then maybe the multicast will work properly without the specific route.  The multicast group woudn't work without that multicast route in place.
I also used an openvswitch vlan to act as a residence for the RP on each of the two core devices.  I will need to tune how groups are associated with each RP, and may obtain a bit of load balancing in the process.
Thank you for pointing out the compile time flag for the mask checking.  I will keep that in mind for future testing.
In the end, I was able to build a test rig to simulate your 'crash course'.  I was pleased with the results.  Things worked well.
Thanx for pointing out the mapping of 'ip mroute' and 'cat /proc/net/ip_mr_cache'.

No problem, good luck mate! :smiley:


Need to have p method

We have puts  and print, but we don't have  p method.

From the Ruby docs:

"For each object, directly writes obj.inspect followed by a newline to the program's standard output."

https://devdocs.io/ruby~2.6/kernel#method-i-p
MRI implementation:
https://github.com/ruby/ruby/blob/e51b6aa81dcad4f07021fbd986050e1995770855/io.c#L7849
Hope that helps :)

Hello! I took a stab at this and would love feedback.


Running warn

Hi,
I used the RAZOR 9DOF IMU.
I added the IMU data in backpack_2d, but no matter how I placed the IMU in the direction（z or x or y-axis aligned wit word),and i  didn't move it ，I always get the Max inclination exceeded 20 warning:
[ WARN] [1479396992.674407112]: W1117 23:36:32.000000 80945 local_trajectory_builder.cc:241] Max inclination exceeded: 20.044 > 20
[ INFO] [1479397001.974362521]: I1117 23:36:41.000000 80945 submaps.cc:176] Added submap 2
[ INFO] [1479397001.984268399]: I1117 23:36:41.000000 80972 sparse_pose_graph.cc:258] We caught up. Hooray!
[ INFO] [1479397006.629708952]: I1117 23:36:46.000000 80945 collated_trajectory_builder.cc:79] imu rate: 48.76 Hz 2.05e-02 s +/- 1.94e-03 s (pulsed at 99.95% real time)
[ INFO] [1479397006.629845005]: I1117 23:36:46.000000 80945 collated_trajectory_builder.cc:79] scan rate: 9.96 Hz 1.00e-01 s +/- 4.70e-04 s (pulsed at 100.00% real time)
[ WARN] [1479397013.148004198]: W1117 23:36:53.000000 80945 local_trajectory_builder.cc:241] Max inclination exceeded: 46.4567 > 20

This is urdf:
<robot name="cartographer_backpack_2d">
<link name="base_link"/>
<link name="base_imu_link"/>
<link name="laser"/>
<joint name="joint_base_imu_link" type="fixed">
    <parent link="base_link" />
    <child link="base_imu_link" />
    <origin xyz="0 0 0" rpy="0 0 0" />
</joint>
<joint name="laser_joint" type="fixed">
    <parent link="base_link" />
    <child link="laser" />
    <origin xyz="0 0 0" rpy="0 0 0" />
</joint>
</robot>

This is lau:
include "map_builder.lua"
options = {
  map_builder = MAP_BUILDER,
  sensor_bridge = {
    horizontal_laser_min_range = 0.,
    horizontal_laser_max_range = 6.,
    horizontal_laser_missing_echo_ray_length = 5,
    constant_odometry_translational_variance = 0.,
    constant_odometry_rotational_variance = 0.,
  },
  map_frame = "map",
  tracking_frame = "base_imu_link",
  published_frame = "base_link",
  odom_frame = "odom",
  provide_odom_frame = true,
  use_odometry_data = false,
  use_horizontal_laser = true,
  use_horizontal_multi_echo_laser = false,
  num_lasers_3d = 0,
  lookup_transform_timeout_sec = 0.2,
  submap_publish_period_sec = 0.3,
  pose_publish_period_sec = 5e-3,
}
MAP_BUILDER.use_trajectory_builder_2d = true
TRAJECTORY_BUILDER_2D.use_imu_data = true
TRAJECTORY_BUILDER_2D.use_online_correlative_scan_matching = true
SPARSE_POSE_GRAPH.optimization_problem.huber_scale = 1e2

This is launch:
<param name="robot_description"
  textfile="$(find cartographer_ros)/urdf/backpack_2d.urdf" />
 <node name="robot_state_publisher" pkg="robot_state_publisher"
 type="robot_state_publisher" />

<node name="cartographer_node" pkg="cartographer_ros"
    type="cartographer_node" args="
        -configuration_directory $(find cartographer_ros)/configuration_files
        -configuration_basename backpack_2d.lua"
    output="screen">
  <remap from="scan" to="/laser_scan" />
  <remap from="imu" to="/imu/data" />
</node>
   <node name="rviz" pkg="rviz" type="rviz" required="true"
    args="-d $(find cartographer_ros)/configuration_files/demo_2d.rviz" />
<node name="playbag" pkg="rosbag" type="play"
    args="--clock $(arg bag_filename)" />
</launch>

Can you provide a bag as well? Try enabling the IMU visualization in rviz. If your IMU is well aligned, the acceleration vector should be point straight up from the floor.

@carl1218 I also ran into this problem with my turtlebot 2 robot, and the robot model is inclined also in rviz.
I temporally solved this problem by disabling the imu in the configuration file, but I am not sure if it will affect the mapping

@damonkohler
hi，
The IMU is  positioned horizontally，and dind not move it,as shown in the picture.
This picture is IMU raw date:

This picture is the  Robot Initial stance:

IMU date look like OK，why always get the Max inclination exceeded 20 warning？

Sorry for the delay.
The warning happens when the IMU estimate of orientation deviates more than 20 degrees from vertical. This can be caused by a few things:

Your orientation is more than 20 degrees from vertical.
Your IMU link in the URDF doesn't match reality and makes the system think your orientation is more than 20 degrees from vertical.
Your IMU data isn't great and the orientation tracking is failing.

In the last case, you can try tuning TRAJECTORY_BUILDER_2D.imu_gravity_time_constant. Increasing it will trust the gyros more and the accelerometers less. Decreasing it will trust the accelerometers more and the gyros less. The perfect compromise will give you the best possible estimate of gravity for a given IMU.

I meet this issue too on the kobuki mobile base's IMU data.I check the raw_data through rostopic echo and  find consistent exsiting errors of the x,y channels of the gyroscope.But on the other hand,there is no corresponding data on the x,y channel of the accelerometers for the exponential moving averaging algorithm to fix these errors.
So we solve this problem by just block these two channels of the gyroscopes since they are unused by the robot moving in the flat world.

@generalvisionyyt
Did you solve this problem?
How did you block X and Y channels of the gyroscopes?
If we  block these two X,Y date of the gyroscopes , Whether to affect the use effect？


Append timezone to UTC-forced dates

Appends GMT to all date strings that have been forced to UTC+00.
We also now force GMT for any time that is not 11.59pm, not just for 12.59am times. This should probably be reviewed if we start storing non-naive datestamps in the DB.
Ticket: https://trello.com/c/8CJAv25W/714-show-time-opportunity-closes

Approved but would be nice to add a comment to the utc filter as discussed.

Hmm think this should have been a major version bump - it broke the supplier frontend's tests and I could only find out how/where by doing a bisect.


radioGroup  toolTipMetadata field name has different case then other tabs (version 2.1)

radioGroup definition contains a field name "tooltipMetadata".  All other tabs have this field name "toolTipMetadata".

Thanks for catching this. I have filed core ticket EC-1616.
Unfortunately, since fixing this error while still maintaining backwards compatibility with existing v2.1 will be a bit tricky, the problem may persist until the next rev of the API itself.


Update who sees project Settings page

Purpose
Update project header template with correct permissions for who sees the settings page
Changes
All non-registrations should be viewable to any user with read access.
All non-retracted registrations should be viewable to any user with
admin access.
Side Effects
n/a

@HarryRybacki I can see the settings page for one of my components on Prod, even though all the page says is "This component is a registration of this component; the content of the component has been frozen and cannot be edited." Is it expected that I'm able to see the settings page?

@NatashaRichter I don't think you should ever be able to see the settings page while in a registration. Please add me as a contrib (if you can) and link me to the project/comp.

@HarryRybacki Added you to the component as an admin contributor (https://osf.io/65tkw/). If you click into the registration of the component, you'll see that a settings page exists. I would expect to only see a settings page on a parent registration to retract registration. However, since you can't retract a component's registration, it's odd that I'm seeing the settings page.

The registration is apparently 'private' as well @NatashaRichter


flet/labels indent incorrectly

flet and labels currently indent like this:
(flet ((foo (a)
            (1+ a)))
  (foo 10))

but the function bodies should indent only two spaces:
(flet ((foo (a)
         (1+ a)))
  (foo 10))

This is an edge case that I had to hack around in my previous lisp indent script, so vlime probably needs to do something similar.

Thanks. I'm aware of this problem, but couldn't get around to fix it yet. There are other glitches too, such as method clauses in DEFGENERIC, and condition clauses in HANDLER-CASE. I'd like to tackle them in a unified/generic-enough way if possible.
The indentation code in Vlime is rather simple, in that it only checks the operator for the innermost expression. To make it aware of its surroundings, a little refactoring may be necessary.
P.S.: Do you mind sharing your solution to this edge case?

P.S.: Do you mind sharing your solution to this edge case?

Oh god.  Well, previously I was using a modified version of this thing, which maintains a stack of its current position as it reads forms.  I hacked in a special case there for to check if we're past the 2nd form in the list and the thing 2 levels up the stack was a flet or labels and use the proper indentation if so.  It's awful: https://bitbucket.org/sjl/dotfiles/commits/ab4a7458c22c748a0b1a4eb1b0ad60e39e101f6f

Thank you! Actually the indent code in Vlime is implemented by following the indent rules summarized by the author of scmindent (https://ds26gte.github.io/scmindent/index.html), so your patch may be quite relevant.
And it seems to me that the only way to fix this is to parse more surrounding Lisp code.

As of commit a1ee9f7, FLET/LABELS/MACROLET should indent properly. Please try it out.

Looks like it works now, thanks!


Want to Fix star alignment in testimonials on small devices

###Fix star alignment in testimonials on small devices
An issue where the star ratings in the testimonials section were displaying vertically on small devices.

Additional Context
Please assign to me i will change it to where the stars were displaying horizontally on small devices.
and also give the tags with hacktoberfest and gccos
Proposed Solution (optional)
I will chage it to like this please assign to me
in your website:

After i modify:

Checklist

[X] I have checked the existing issues
[X] I have read the Contributing Guidelines
[X] I am willing to work on this issue (optional)

@Manojkumar2806 sure go ahead
Make sure to join the discord server for further communication
Don't forget to star the repository
Complete the task in next 2 days and kindly mention your issue # in your pr like this:
Closes #IssueNumber
Note: the level of the issue may change based on the quality of the PR code

In this #619 I have already modified the code, and it now works properly. The ratings are aligned horizontally on small devices as intended. Please once check the code sir.

Initially, In this #619 I did not create an issue and directly cloned the code to make a pull request. I apologize for that. But i modify the code. Now it the stars aligned properly in the small devices. Please, once check the code sir.


Disclaimer saving state

Summary
Have redux variable for the user answer for disclaimer, which will store values: 'agree', 'not agree' (true, false?). In the future, we'll store it in the database inside the user table. So that way we'll know if the user agreed to the disclaimer.
Motivation
Why are we doing this?
We don't want the disclaimer to pop up every time and it won't go away because buttons currently don't work.
What is the expected outcome?
Disclaimer state will save the user answer and store it redux. And depends on the answer it will send you back or send it to the main tab.
Describe alternatives you've considered
Have maybe a user profile redux state, that will have a property with a disclaimer answer (initially could be null). Then send it to the disclaimer component and component will change the answer and send to redux, and then to the backend where it will be stored, so the next we won't ask a user to agree or disagree.

I can take this one on


monitoring training and cross-validation loss values

Hello,
Many thanks for providing your code for free.
I am training the model on my own dataset. I wonder how can I monitor the loss values for training and cross-validation?
Mohammad

Because TuSimple dataset has only about 3000 frames for training,  the code evaluated the performance about test set every epoch.

您好，非常感谢您免费提供您的代码。我正在自己的数据集上训练模型。我想知道如何监控训练和交叉验证的损失值？ 穆罕默德

Hello, I also want to see the loss function change during training, have you succeeded, can I communicate with you


[$250] Clicking back button takes user to wrong page from avatar preview page

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

Follow up to https://github.com/Expensify/App/issues/27856
Version Number:
Reproducible in staging?:
Reproducible in production?:
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: @aimane-chnaif
Slack conversation: v1.4.36-0
Action Performed:

Go to settings > profile
Click on avatar and add a photo if you don't have any
Click on the avatar again
Click on View photo
Click on the system/browser back button

Expected Result:
App should take user back to profile page
Actual Result:
App takes user to settings page
Workaround:
Unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

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

Screenshots/Videos
Add any screenshot/video evidence

View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~014d2a5e3d9548d625
Upwork Job ID:<PHONE_NUMBER>498659328
Last Price Increase: 2024-02-05
Automatic offers:

jjcoffee | Reviewer | 28142297
mkhutornyi | Contributor | 28142298

Simple change clearly given since this was a result of partial revert of a code.

@mkhutornyi can you raise a PR please

PR merged
@garrettmknight as a summary, this will be $250 to @mkhutornyi if no regression arises. No C+ on the PR

@adamgrzybowski just keeping you on loop as this might be related to the back navigation issue you're working on.

This one should be fixed

This was coming from the Ideal Nav big PR, no need to really post on there I believe as we are aware of some of the goBack issues.
@garrettmknight the payment summary is $250 to @mkhutornyi and $250 to @aimane-chnaif

Cool, will whip up payment tomorrow on payday!

Summary of payments:

Contributor: @mkhutornyi $250 paid via Upwork
Reviewer: @aimane-chnaif $250

Dropping to weekly for @aimane-chnaif to request payment.

Let me know when you accept and I'll pay!

Paid, closing


Release

:tada: This PR is included in version 1.7.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.7.2 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


When Clicking pagination, instead of loading next page, it adds rows to current page rows

Describe the bug
I have 3 pages of rows with pagination at the bottom. When I click the 2 for the second page, instead of loading the next 5 rows, it adds those 5 to the first page's 5. This seems random. It works in many screen sizes, but breaks in others.
Screenshots
If applicable, add screenshots to help explain your problem.

I apologize, bad code on my part. I had duplicate keys on table in react so it was mistaking the second table for the main table, which loads 10 at a time, not 5.


Lpfg 1067 Delete Files

Deletes files when file/mp4/elearning modules are deleted or updated with new files.
Delete and update module logic moved from CourseController.java to ModuleService.java
Added hasFiledChanged, deleteFile, deleteModule and updateModule functions to ModuleService.java
Added delete and deleteDirectory functions to FileUploadService.java and AzureUploadClient.java
-> Delete directory recursively deletes all of the files in a directory, as it appears directories don't
technically exist in blob storage

I'd be interested to understand

-> Delete directory recursively deletes all of the files in a directory, as it appears directories don't
technically exist in blob storage

replaced by https://github.com/Civil-Service-Human-Resources/lpg-learning-catalogue/pull/70


Shipping broken, doesn't work

Checklist

[X] I read the troubleshooting guide before raising this issue
[X] I made sure that the issue I am raising doesn't already exist

Current bug behaviour
I follow the exact instructions and just get this
══╡ EXCEPTION CAUGHT BY WIDGETS LIBRARY ╞═══════════════════════════════════════════════════════════
The following assertion was thrown while applying parent data.:
Incorrect use of ParentDataWidget.
The ParentDataWidget Flexible(flex: 1) wants to apply ParentData of type FlexParentData to a
RenderObject, which has been set up to accept ParentData of incompatible type ParentData.
Usually, this means that the Flexible widget has the wrong ancestor RenderObjectWidget. Typically,
Flexible widgets are placed directly inside Flex widgets.
The offending Flexible is currently placed inside a SizedBox widget.
The ownership chain for the RenderObject that received the incompatible parent data was:
  RichText ← Text ← Flexible ← StartText ← SizedBox ← Column ← Row ← Center ← Column ← Padding ← ⋯

When the exception was thrown, this was the stack:
#0      RenderObjectElement._updateParentData.<anonymous closure> (package:flutter/src/widgets/framework.dart:6230:11)
#1      RenderObjectElement._updateParentData (package:flutter/src/widgets/framework.dart:6247:6)
#2      RenderObjectElement.attachRenderObject (package:flutter/src/widgets/framework.dart:6270:7)
#3      RenderObjectElement.mount (package:flutter/src/widgets/framework.dart:6154:5)
#4      MultiChildRenderObjectElement.mount (package:flutter/src/widgets/framework.dart:6576:11)
...     Normal element mounting (22 frames)
#26     Element.inflateWidget (package:flutter/src/widgets/framework.dart:4182:16)
#27     MultiChildRenderObjectElement.inflateWidget (package:flutter/src/widgets/framework.dart:6569:36)
#28     MultiChildRenderObjectElement.mount (package:flutter/src/widgets/framework.dart:6581:32)
#29     Element.inflateWidget (package:flutter/src/widgets/framework.dart:4182:16)
#30     MultiChildRenderObjectElement.inflateWidget (package:flutter/src/widgets/framework.dart:6569:36)
#31     MultiChildRenderObjectElement.mount (package:flutter/src/widgets/framework.dart:6581:32)
...     Normal element mounting (4 frames)
#35     Element.inflateWidget (package:flutter/src/widgets/framework.dart:4182:16)
#36     MultiChildRenderObjectElement.inflateWidget (package:flutter/src/widgets/framework.dart:6569:36)
#37     MultiChildRenderObjectElement.mount (package:flutter/src/widgets/framework.dart:6581:32)
...     Normal element mounting (37 frames)
#74     Element.inflateWidget (package:flutter/src/widgets/framework.dart:4182:16)
#75     MultiChildRenderObjectElement.inflateWidget (package:flutter/src/widgets/framework.dart:6569:36)
#76     MultiChildRenderObjectElement.mount (package:flutter/src/widgets/framework.dart:6581:32)
...     Normal element mounting (330 frames)
#406    Element.inflateWidget (package:flutter/src/widgets/framework.dart:4182:16)
#407    MultiChildRenderObjectElement.inflateWidget (package:flutter/src/widgets/framework.dart:6569:36)
#408    MultiChildRenderObjectElement.mount (package:flutter/src/widgets/framework.dart:6581:32)
...     Normal element mounting (452 frames)
#860    Element.inflateWidget (package:flutter/src/widgets/framework.dart:4182:16)
#861    Element.updateChild (package:flutter/src/widgets/framework.dart:3707:18)
#862    RenderObjectToWidgetElement._rebuild (package:flutter/src/widgets/binding.dart:1253:16)
#863    RenderObjectToWidgetElement.mount (package:flutter/src/widgets/binding.dart:1222:5)
#864    RenderObjectToWidgetAdapter.attachToRenderTree.<anonymous closure> (package:flutter/src/widgets/binding.dart:1169:18)
#865    BuildOwner.buildScope (package:flutter/src/widgets/framework.dart:2719:19)
#866    RenderObjectToWidgetAdapter.attachToRenderTree (package:flutter/src/widgets/binding.dart:1168:13)
#867    WidgetsBinding.attachRootWidget (package:flutter/src/widgets/binding.dart:1001:7)
#868    WidgetsBinding.scheduleAttachRootWidget.<anonymous closure> (package:flutter/src/widgets/binding.dart:981:7)
#872    _RawReceivePort._handleMessage (dart:isolate-patch/isolate_patch.dart:189:12)
(elided 3 frames from class _Timer and dart:async-patch)

Expected behaviour
For it to play audio
Steps to reproduce

Execute flutter run on the code sample
...
...

Code sample

Code sample
void main() {
}

Affected platforms
Windows
Platform details
Windows 11
AudioPlayers Version
.
Build mode
No response
Audio Files/URLs/Sources
No response
Screenshots
No response
Logs

my relevant logs

Full Logs

my full logs or a link to a gist

Flutter doctor:
Output of: flutter doctor -v

Related issues / more information
No response
Working on PR
no way

Hey there. I don't see anything related to AudioPlayers. Which instructions did you use? What is the code base?


Can I programmatically install test-bulter apk onto an emulator?

Our CI setup uses firebase labs, I am wondering how do I install this apk on each of those emulators.

It's not possible. I just discussed this with the Firebase test lab team on Slack.

https://firebase-community.appspot.com/ join channel #test-lab


Ignore vendor/ directory for JavaScript by default

This can still be overriden if the user provides their own .jshintignore file.

Related to https://github.com/thoughtbot/hound/pull/649

@gylaz, is there a reason not to use a default .jshintignore instead of doing this in code, the same way we use other default configs?

:+1:

@salbertson I can see that being useful for consistency. Good idea. Done in 3c39135.

Nice, looks good. I have one question. :+1:


Describe setup process in documentation

I added a new documentation page about the setup wizard. Please add and correct material to this page as you see fit!
This page incorporates material from iiab/iiab#1573.

Codecov Report

Merging #6349 into master will increase coverage by 0.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #6349      +/-   ##
==========================================
+ Coverage   38.87%   38.88%   +0.01%
==========================================
  Files         363      363
  Lines       51210    51210
==========================================
+ Hits        19908    19915       +7
+ Misses      28434    28427       -7
  Partials     2868     2868

Impacted Files
Coverage Δ

models/repo_list.go
66.84% <0%> (-1.06%)
:arrow_down:

routers/repo/view.go
42.07% <0%> (+0.99%)
:arrow_up:

modules/process/manager.go
81.15% <0%> (+4.34%)
:arrow_up:

models/unit.go
14.28% <0%> (+14.28%)
:arrow_up:

Continue to review full report at Codecov.

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

I just added screenshots of the setup process. What is the proper location for these images? Would it be in docs/static?

@techknowlogick @kolaente Is this too much detail?
Also, I've only had to use one screenshot so far. If we don't use the others, I might delete them from the branch before we merge.

Is this still a WIP?

Looking at email and third-party sections, I would assume so.

@zeripath @0x5c Email is done. Third-party still needs to be worked on.

Email is one of those things that does need to be explained, as, contrary to database servers, someone needing email notifications to work will have but that one choice. (You don't need a database server to run Gitea, and will only see the benefits if you [already] have a server for other services, which also means that you most probably know what the server address, port, and db name should be.)
Good documentation is consistent, and that also goes with the extent to which it explains concepts, functions, and options. For this reason, I raise the question:
What is the goal here?
Is it to produce a "beginner's guide", or a blanket coverage of the Setup page's options?
Knowing the answer will help avoid creating a mix of the two, and thus, maintain consistency.

@0x5c I want the documentation to be comprehensive - that is, summarize all of the options shown on the setup page while duplicating the config cheatsheet as little as possible. I also want it to be beginner-friendly so that someone with a reasonable level of prior sysadmin knowledge can understand it.

That still does not answer the question, but ignoring that;

Any documentation about the setup page should not rely on documentation about the configuration file, and even less for a non-authoritative one (the cheat-sheet is not maintained to the extent that the app.ini sample is [1]). The setup page is not the configuration file.
The currently proposed document is not consistent.
A beginner's guide and extensive documentation are mutually-exclusive concepts: one targets the elements that are required to begin/"get started", while the other targets clear, concise, and complete documentation. One has to be chosen.

The current Gitea documentation is none of clear, concise, and complete, but let's start breaking the trend instead of making the matters worse.

[1] - [Config Cheat Sheet](https://docs.gitea.io/en-us/config-cheat-sheet/), ¶3:

The defaults provided here are best-effort (not built automatically). They are accurately recorded in app.ini.sample (s/master/<tag|release>).


Delayed response

any thoughts on adding the ability to use the delayed response response_url ?
https://api.slack.com/slash-commands

Currently you can access response_url from kwargs. After doing whatever you need, just send a POST request to the response_url with requests. You can also get a json data with slack.response() which could be used as the data to send the request.

cool @VeryCB! i'll take a look once pythonanywhere.com's dashboards START WORKING AGAIN!!!


Add more events to Crashlytics logging

Currently only logging:
login
postAvatar
loadCampaign
postSignup
postReportback
logout
Other events that could potentially help diagnose when things go wrong (like #925, #934)

[ ] Image picker events (menu presented, library/camera selected, image selected)
[x] Tab selected
[x] Loading user ID (helps us see the entire trail of screens visited)
[x] Modals presented / dismissed

Will close this issue and create separate for Image PIcker GAI + Crashlytics logging (per #951)


Claims to be listening for redis events, but not doing so

I'm using laravel-echo-server with Redis. Broadcast events aren't being picked up by laravel-echo-server - is ideas?
This is being run within a docker network, hence the slightly odd hostnames.
laravel-echo-server.json
{
	"authHost": "php",
	"authEndpoint": "/broadcasting/auth",
	"clients": [
		{
			"appId": "xxxxxxxxxxxxxxxx",
			"key": "xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx"
		}
	],
	"database": "redis",
	"databaseConfig": {
		"redis": {
			"port": "6379",
			"host": "redis"
		}
	},
	"devMode": true,
	"host": null,
	"port": "6001",
	"protocol": "http",
	"socketio": {},
	"sslCertPath": "",
	"sslKeyPath": "",
	"sslCertChainPath": "",
	"sslPassphrase": ""
}

Queue log
Horizon started successfully
[2017-09-11 10:35:22] local.INFO: Broadcasting [MyNamespace\MyEvent] on channels [testChannel] with payload: [] [] []

Laravel Echo Server Logs
L A R A V E L  E C H O  S E R V E R

version 1.3.1

⚠ Starting server in DEV mode...

✔  Running at localhost on port 6001
✔  Channels are ready.
✔  Listening for http events...
✔  Listening for redis events...

Server ready!

[10:33:17] - zwec8Ey7iqlmib4nAAAB joined channel: testChannel

@flxsource  what do you get within larave echo server logs when you fire an event. does it logged there ?

@anouarabdsslm No, the echo server logs remained empty.
Have now solved this - it turns out that environment variables weren't being transmitted correctly into the docker container, so Laravel was still using the log driver.
Apologies for wasting your time!

@SammyLee666 I was a while ago, so my memory is a little hazy, but I seem to recall using a .env within the container instead of relying on the environment variables set via docker-compose.

Thank you very much for your reply. I have already solved it. Thanks again


When Change Schema Database Close Window

In the version 4.1.2.
When Change schema database, the window is minimized

Duplicate of #1860


Unused timeout argument in SysCmd Initialization

Description:
The timeout variable is not being passed when constructing an instance of SysCmd. This should be required for ssh or scp commands, such as when copying resources during the diagnostic tool.
Files:
Initialization of timeout
https://github.com/NVIDIA/spark-rapids-tools/blob/24be442afcae256c695143ebcfae6b02e77c57d4/user_tools/src/spark_rapids_pytools/cloud_api/sp_types.py#L333-L338
Construction of SysCmd
https://github.com/NVIDIA/spark-rapids-tools/blob/24be442afcae256c695143ebcfae6b02e77c57d4/user_tools/src/spark_rapids_pytools/cloud_api/sp_types.py#L503-L511

@parthosa what is the resolution of this issue? Are you suggesting that we revisit all call sites to make sure that the timeout is set?
If so, how do we define a suitable timeout? I think that we should not set timeout copying results/eventlogs because we cannot define a timeout that fits all job sizes.

There are two issues here:

Currently, all classes inheriting from CMDDriverBase have their timeout value set to 0 by default. We should pass this value when calling SysCmd() in run_sys_cmd().
For specific cases like ssh and scp, we should add a timeout variable in run_sys_cmd() function. This would override the class-scoped timeout variable for these operations.


This is too cool and awesome to exist

I am almost 2 hours deep in your site ad loved every minute .
keep it up < 3

Glad you like it. It is a labor of love but I know the right people will appreciate it.


There was an error while trying to load the gem 't_t'. (Bundler::GemRequireError)

Hi,
i would really like to get hands on your gem which looks quite useful, but can't load it somehow.
I am getting a mentioned error. This is a bit more:

There was an error while trying to load the gem 't_t'. (Bundler::GemRequireError)
/home/XXX/.rvm/gems/ruby-2.2.2@informator/gems/bundler-1.11.2/lib/bundler/runtime.rb:80:in rescue in block (2 levels) in require'   /home/XXX/.rvm/gems/ruby-2.2.2@informator/gems/bundler-1.11.2/lib/bundler/runtime.rb:76:in block (2 levels) in require'
/home/XXX/.rvm/gems/ruby-2.2.2@informator/gems/bundler-1.11.2/lib/bundler/runtime.rb:72:in each'   /home/XXX/.rvm/gems/ruby-2.2.2@informator/gems/bundler-1.11.2/lib/bundler/runtime.rb:72:in block in require'
/home/XXX/.rvm/gems/ruby-2.2.2@informator/gems/bundler-1.11.2/lib/bundler/runtime.rb:61:in each'   /home/XXX/.rvm/gems/ruby-2.2.2@informator/gems/bundler-1.11.2/lib/bundler/runtime.rb:61:in require'
/home/XXX/.rvm/gems/ruby-2.2.2@informator/gems/bundler-1.11.2/lib/bundler.rb:99:in require'   /home/XXX/projects/development/informator/config/application.rb:7:in <top (required)>'
/home/XXX/projects/development/informator/config/environment.rb:2:in require'   /home/XXX/projects/development/informator/config/environment.rb:2:in <top (required)>'
config.ru:3:in require'   config.ru:3:in block in '
/home/XXX/.rvm/gems/ruby-2.2.2@informator/gems/rack-1.6.4/lib/rack/builder.rb:55:in instance_eval'   /home/XXX/.rvm/gems/ruby-2.2.2@informator/gems/rack-1.6.4/lib/rack/builder.rb:55:in initialize'
config.ru:1:in new'   config.ru:1:in '
/home/XXX/.rvm/gems/ruby-2.1.2/gems/passenger-5.0.10/helper-scripts/rack-preloader.rb:107:in eval'   /home/XXX/.rvm/gems/ruby-2.1.2/gems/passenger-5.0.10/helper-scripts/rack-preloader.rb:107:in preload_app'
/home/XXX/.rvm/gems/ruby-2.1.2/gems/passenger-5.0.10/helper-scripts/rack-preloader.rb:153:in <module:App>'   /home/XXX/.rvm/gems/ruby-2.1.2/gems/passenger-5.0.10/helper-scripts/rack-preloader.rb:29:in module:PhusionPassenger'
/home/XXX/.rvm/gems/ruby-2.1.2/gems/passenger-5.0.10/helper-scripts/rack-preloader.rb:28:in `'

It looks like you didn't run bundle before restarting the passenger

I did it several times actually, also tried moving gem up and down the Gemfile (maybe some loading order issue), but no luck. It just doesn't load with this gem.

Yes, there was a bug related to activerecord integration. Version 1.0.1 fixes it


Revert "Updates sonatype host."

Reverts dropbox/differ#6
We moved our host to s01 https://issues.sonatype.org/browse/OSSRH-78525

Already updated this elsewhere. Thanks.