Inconsistent LRU: The command returns encoded 'OK' message instead of returning the  least recently used keys

Steps to reproduce

Connect to diceDB
launch dicedb-cli
Enter command : SET k1 v1
Enter command: LRU

Expected output
The output expected is the list of keys that are least recently used ( according to the Dice DB docs ).
  1.) k1

Observed output
The output observed it the 'OK' message.
LRU command eval function
  /* /internals/eval/eval.go
    evalLRU deletes all the keys from the LRU
    returns encoded RESP OK
 */
 func evalLRU(args []string, store *dstore.Store) []byte {
	dstore.EvictAllkeysLRUOrLFU(store)
	return clientio.RespOK
 }

We can have EvictAllKeysLRUorLFU to return the list of keys and send the list to the CLI. Also the doc doesn't mention if the intended usecase is to remove all the LRU keys or just return the list to the clients for their information. @AshwinKul28 @JyotinderSingh please let me know your thoughts on this.

@arpitbbhayani can I take this up... it will be a good start as a contributor


Unable to run webdriver-manager-update

Hi there!
Getting the below error when running webdriver-manager update...
C:\Users\pctr294290>webdriver-manager update
events.js:160
throw er; // Unhandled 'error' event
^
Error: self signed certificate in certificate chain
at Error (native)
at TLSSocket. (_tls_wrap.js:1092:38)
at emitNone (events.js:86:13)
at TLSSocket.emit (events.js:185:7)
at TLSSocket._finishInit (_tls_wrap.js:610:8)
at TLSWrap.ssl.onhandshakedone (_tls_wrap.js:440:38)
Please help.
Bug report

Node Version: ``
Protractor Version: ``
Angular Version: ``
Browser(s): ``
Operating System and Version ``
Your protractor configuration file
A relevant example test
Output from running the test
Steps to reproduce the bug
The URL you are running your tests against (if relevant)

Feature Request

Reasons for adopting new feature
Is this a breaking change? (How will this affect existing functionality)
Hi there!

Getting the below error when running webdriver-manager update...
C:\Users\pctr294290>webdriver-manager update
events.js:160
throw er; // Unhandled 'error' event
^
Error: self signed certificate in certificate chain
at Error (native)
at TLSSocket. (_tls_wrap.js:1092:38)
at emitNone (events.js:86:13)
at TLSSocket.emit (events.js:185:7)
at TLSSocket._finishInit (_tls_wrap.js:610:8)
at TLSWrap.ssl.onhandshakedone (_tls_wrap.js:440:38)
Please help.
Bug report

Node Version: ``
Protractor Version: ``
Angular Version: ``
Browser(s): ``
Operating System and Version ``
Your protractor configuration file
A relevant example test
Output from running the test
Steps to reproduce the bug
The URL you are running your tests against (if relevant)

Feature Request

Reasons for adopting new feature
Is this a breaking change? (How will this affect existing functionality)

Node Version: 6.10.2 Protractor Version: 5.1.1
Operating System and Version `` windows 7

I faced the same issue and you can run the following as workaround:
npm cache clean
webdriver-manager update --ignore_ssl

This is typically due to a proxy or other networking layer that is sitting between you and the servers hosting the browser binaries. @bobbai's solution should work but the underlying issue is a networking one.
I'm going to go ahead and close this as it isn't an issue with webdriver-manager itself.

@JimTheMan Same here, I can't do webdriver-manager update, but I tried to copied all the files needed, and 'webdriver-manager start' does get run; then when I tried to run the first Protractor test (copied from the tutorial page), it doesn't get the page displayed on the chrome browser. Anyone can provide any help, it will be highly appreciated!!

@ashishbansali Thank you so much, I tried that. And received an unhandled error. Do you have any guess?
events.js:167
throw er; // Unhandled 'error' event
^
Error: getaddrinfo ENOTFOUND userID userID:443

Thank you so much!! I still have the same error, see below.  Do you know if this is caused by the proxy may be not correct?
Sent from my iPhone

On Oct 24, 2018, at 10:27 AM, ashishbansali<EMAIL_ADDRESS>wrote:
Ensure you follow below steps:
1.Install Node JS and acquire read/write access to the corresponding folder.
2.Navigate to NPM folder and execute below commands via cmd
npm config set proxy
npm config set registry http://registry.npmjs.org/
npm config set http_proxy=
npm config set https_proxy=
3.Install protractor
npm install i protractor
4.webdriver-manager clean
5.webdriver-manager shutdown
6.webdriver-manager update --proxy
7.webdriver-manager start --proxy
This should work just fine.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

@ashishbansali Sorry to bug you, here is the response after running 'webdriver-manager update --proxy --ignore_ssl', the screen capture I sent you was not displayed, so I attached it here as a file.
2018-10-26_13-24-47.pdf

I have added proxy and proxy sees traffic to the  registry http://registry.npmjs.org/. But there are traffic to other IPs such as 192.30.1254.16:443, could the code be updated to going to the registry place for all traffic?  @ashishbansali , do you know if this is a known issue to people who work behind the corporate firewall?


Maintenance

Just some dependency updates.

Testing with different random settings and permalink on windows Firefox and Chrome worked flawlessly.

Thanks @jimmyhenne !


add a configurable chunksize whenever loading

We have seen in pycytominer that using a chunksize parameter on pandas.read_sql averts runaway memory consumption errors; we are hoping doing so here for read_csv has the same effect!
A new read_csvs_with_chunksize function is added her to io_utils, and all other pd.read_csv calls are replaced with it.  Right now we are not actually configuring what size chunks to use anywhere, but I've left it configurable in case in future user testing we find it works better with different sizes for different calls.
I've also explicitly not done any error handling other than raising, simply because in various places in the script sometimes even the same error class is sometimes breaking for the recipe and sometimes not, and I thought we should continue to just support the logic where it currently is.

@gwaygenomics your requested changes are made.  It would be great to get you or @MerajRamezani or @mlozada21 or anyone else who runs the recipe to test this to see if things are sad before we pull, because I currently have not been able to test this.
RE: Always using it- I tested reading on a 2080 row by 1023 column CSV; reading with no chunk size at all took 0.402s, with a chunk size of 3000 (aka bigger than the file) it took 0.404s, a size of 1000 it took ~0.5s, with a size of 100 it took 1.3 seconds. So with the proposed default of 10K chunksize, we should see no performance hit at all on smallish files, it should just be a "guardrail" for really big files.  I think we could probably safely drop it down smaller than 10K as the default, but that's the sort of thing I think would be useful to do some testing with.

For funsies, timeit of using this on a 47K row vs 1K column CSV
no chunking - 7.94 s ± 222 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)
chunksize 100 - 32 s ± 1.08 s per loop (mean ± std. dev. of 7 runs, 1 loop each)
chunksize 1K - 10.3 s ± 360 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)
chunksize 10K - 8.57 s ± 554 ms per loop (mean ± std. dev. of 7 runs, 1 loop each)

great to see this enhancement. I won't be able to play around with this until next week (will need to play catch up after my vacation ends) but I'll be happy to test it out.
But @MerajRamezani maybe you'll get to this before me? It should be a fairly simple test - try running CPXXX well level profiles (we don't talk about numbers in this public repo!) but go into the recipe, git pull, and git checkout chunksize (the branch @bethac07 is editing - then try rerunning one of the profiling steps, maybe aggregation)
Also, @bethac07 for that timeit test, can you add a code block of what you did to generate those numbers?

@gwaygenomics @bethac07 I can give it a try and update you on how it goes by the end of the week.

Yup, good catch.
RE: timeit , within jupyter -
Cell1 - configuration
import pandas as pd
filename = 'path/to/large/file.csv'
chunksize = 100

Cell 2- test without chunksize
%%timeit
df = pandas.read_csv(filename)
```

Cell 3- test with chunksize (which can be configured in Cell 1
```
%%timeit
with pandas.read_csv(filename,chunksize=chunksize) as reader:
    dflist=[]
    for chunk in reader:
        dflist.append(chunk)
    df = pandas.concat(dflist)
```

@bethac07 @gwaygenomics I tried to run the updated branch but I am getting this error:
Traceback (most recent call last):
File "/home/ubuntu/miniconda3/envs/pooled-cell-painting/lib/python3.7/site-packages/pandas/core/indexes/base.py", line 3081, in get_loc
return self._engine.get_loc(casted_key)
File "pandas/_libs/index.pyx", line 70, in pandas._libs.index.IndexEngine.get_loc
File "pandas/_libs/index.pyx", line 101, in pandas._libs.index.IndexEngine.get_loc
File "pandas/_libs/hashtable_class_helper.pxi", line 4554, in pandas._libs.hashtable.PyObjectHashTable.get_item
File "pandas/_libs/hashtable_class_helper.pxi", line 4562, in pandas._libs.hashtable.PyObjectHashTable.get_item
KeyError: 'level_3'
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "recipe/0.preprocess-sites/4.image-and-segmentation-qc.py", line 511, in
cp_sat_df[["cat", "type", "Ch"]] = cp_sat_df["level_3"].str.split(
File "/home/ubuntu/miniconda3/envs/pooled-cell-painting/lib/python3.7/site-packages/pandas/core/frame.py", line 3024, in getitem
indexer = self.columns.get_loc(key)
File "/home/ubuntu/miniconda3/envs/pooled-cell-painting/lib/python3.7/site-packages/pandas/core/indexes/base.py", line 3083, in get_loc
raise KeyError(key) from err
KeyError: 'level_3'
Config file set to perform=False, not performing step: profile--single_cell
Traceback (most recent call last):
File "recipe/1.generate-profiles/1.aggregate.py", line 14, in
from io_utils import read_csvs_with_chunksize
ModuleNotFoundError: No module named 'io_utils'
Traceback (most recent call last):
File "recipe/1.generate-profiles/2.normalize.py", line 13, in
from io_utils import read_csvs_with_chunksize
ModuleNotFoundError: No module named 'io_utils'
Traceback (most recent call last):
File "recipe/1.generate-profiles/3.feature-select.py", line 13, in
from io_utils import read_csvs_with_chunksize
ModuleNotFoundError: No module named 'io_utils'

I think I see the issue and pushed a patch, can you pull and try again?
Thanks!
On Wed, Jul 14, 2021 at 3:23 PM MerajRamezani @.***>
wrote:

@bethac07 https://github.com/bethac07 @gwaygenomics
https://github.com/gwaygenomics I tried to run the updated branch but I
am getting this error:
Traceback (most recent call last):
File
"/home/ubuntu/miniconda3/envs/pooled-cell-painting/lib/python3.7/site-packages/pandas/core/indexes/base.py",
line 3081, in get_loc
return self._engine.get_loc(casted_key)
File "pandas/_libs/index.pyx", line 70, in
pandas._libs.index.IndexEngine.get_loc
File "pandas/_libs/index.pyx", line 101, in
pandas._libs.index.IndexEngine.get_loc
File "pandas/_libs/hashtable_class_helper.pxi", line 4554, in
pandas._libs.hashtable.PyObjectHashTable.get_item
File "pandas/_libs/hashtable_class_helper.pxi", line 4562, in
pandas._libs.hashtable.PyObjectHashTable.get_item
KeyError: 'level_3'
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "recipe/0.preprocess-sites/4.image-and-segmentation-qc.py", line
511, in
cp_sat_df[["cat", "type", "Ch"]] = cp_sat_df["level_3"].str.split(
File
"/home/ubuntu/miniconda3/envs/pooled-cell-painting/lib/python3.7/site-packages/pandas/core/frame.py",
line 3024, in getitem
indexer = self.columns.get_loc(key)
File
"/home/ubuntu/miniconda3/envs/pooled-cell-painting/lib/python3.7/site-packages/pandas/core/indexes/base.py",
line 3083, in get_loc
raise KeyError(key) from err
KeyError: 'level_3'
Config file set to perform=False, not performing step: profile--single_cell
Traceback (most recent call last):
File "recipe/1.generate-profiles/1.aggregate.py", line 14, in
from io_utils import read_csvs_with_chunksize
ModuleNotFoundError: No module named 'io_utils'
Traceback (most recent call last):
File "recipe/1.generate-profiles/2.normalize.py", line 13, in
from io_utils import read_csvs_with_chunksize
ModuleNotFoundError: No module named 'io_utils'
Traceback (most recent call last):
File "recipe/1.generate-profiles/3.feature-select.py", line 13, in
from io_utils import read_csvs_with_chunksize
ModuleNotFoundError: No module named 'io_utils'
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/broadinstitute/pooled-cell-painting-profiling-recipe/pull/78#issuecomment-880148783,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABTI724AVDFU2EG57CF6733TXXP23ANCNFSM5ADD5AUA
.

--
Beth Cimini, PhD
Senior Group Leader & CZI Imaging Scientist
Imaging Platform, Broad Institute
415 Main St Room 5011
Cambridge, MA 02142
Current office number-<PHONE_NUMBER>
Pronouns - She/her/hers
I will sometimes send or respond to emails outside of my local office
hours, but I never expect responses outside of your local office hours.

Great, I have started a new test and it seems to be running smoothly for now. I am going to update you as soon as I have the results.

@bethac07 @gwaygenomics The new code failed to aggregate profiles on a m5.4xlarge machine but ran without an issue on a m5.24xlarge machine. I can test it on another machine as well if relevant(let me know a specific one!). Here is the log and where it failed:
Config file set to perform=False, not performing step: preprocess--prefilter
Config file set to perform=False, not performing step: preprocess--process-spots
Config file set to perform=False, not performing step: preprocess--process-cells
Config file set to perform=False, not performing step: preprocess--summarize-cells
/home/ubuntu/efs/2018_11_20_Periscope_Calico/workspace/software/Meraj_analysis/CP228W/recipe/scripts/io_utils.py:11: UserWarning: data/0.site-qc/20210124_6W_CP228/figures/plate_layout_cells_count_per_well_CP228A.png exists, overwriting
/home/ubuntu/efs/2018_11_20_Periscope_Calico/workspace/software/Meraj_analysis/CP228W/recipe/scripts/io_utils.py:11: UserWarning: data/0.site-qc/20210124_6W_CP228/figures/plate_layout_cells_count_per_well_CP228B.png exists, overwriting
/home/ubuntu/efs/2018_11_20_Periscope_Calico/workspace/software/Meraj_analysis/CP228W/recipe/scripts/io_utils.py:11: UserWarning: data/0.site-qc/20210124_6W_CP228/figures/plate_layout_ratios_per_well_CP228A.png exists, overwriting
/home/ubuntu/efs/2018_11_20_Periscope_Calico/workspace/software/Meraj_analysis/CP228W/recipe/scripts/io_utils.py:11: UserWarning: data/0.site-qc/20210124_6W_CP228/figures/plate_layout_percent_empty_cells_per_well_CP228A.png exists, overwriting
/home/ubuntu/efs/2018_11_20_Periscope_Calico/workspace/software/Meraj_analysis/CP228W/recipe/scripts/io_utils.py:11: UserWarning: data/0.site-qc/20210124_6W_CP228/figures/plate_layout_ratios_per_well_CP228B.png exists, overwriting
/home/ubuntu/efs/2018_11_20_Periscope_Calico/workspace/software/Meraj_analysis/CP228W/recipe/scripts/io_utils.py:11: UserWarning: data/0.site-qc/20210124_6W_CP228/figures/plate_layout_percent_empty_cells_per_well_CP228B.png exists, overwriting
/home/ubuntu/efs/2018_11_20_Periscope_Calico/workspace/software/Meraj_analysis/CP228W/recipe/scripts/io_utils.py:11: UserWarning: data/0.site-qc/20210124_6W_CP228/figures/plate_layout_BC_to_CP_DAPI_correlation_per_well_CP228A.png exists, overwriting
/home/ubuntu/efs/2018_11_20_Periscope_Calico/workspace/software/Meraj_analysis/CP228W/recipe/scripts/io_utils.py:11: UserWarning: data/0.site-qc/20210124_6W_CP228/figures/plate_layout_BC_to_CP_DAPI_correlation_per_well_CP228B.png exists, overwriting
/home/ubuntu/efs/2018_11_20_Periscope_Calico/workspace/software/Meraj_analysis/CP228W/recipe/scripts/io_utils.py:11: UserWarning: data/0.site-qc/20210124_6W_CP228/figures/plate_layout_Cells_FinalThreshold_per_well_CP228A.png exists, overwriting
/home/ubuntu/efs/2018_11_20_Periscope_Calico/workspace/software/Meraj_analysis/CP228W/recipe/scripts/io_utils.py:11: UserWarning: data/0.site-qc/20210124_6W_CP228/figures/plate_layout_Cells_FinalThreshold_per_well_CP228B.png exists, overwriting
/home/ubuntu/efs/2018_11_20_Periscope_Calico/workspace/software/Meraj_analysis/CP228W/recipe/scripts/io_utils.py:11: UserWarning: data/0.site-qc/20210124_6W_CP228/figures/plate_layout_Nuclei_FinalThreshold_per_well_CP228A.png exists, overwriting
/home/ubuntu/efs/2018_11_20_Periscope_Calico/workspace/software/Meraj_analysis/CP228W/recipe/scripts/io_utils.py:11: UserWarning: data/0.site-qc/20210124_6W_CP228/figures/plate_layout_Nuclei_FinalThreshold_per_well_CP228B.png exists, overwriting
/home/ubuntu/efs/2018_11_20_Periscope_Calico/workspace/software/Meraj_analysis/CP228W/recipe/scripts/io_utils.py:11: UserWarning: data/0.site-qc/20210124_6W_CP228/figures/plate_layout_PercentConfluent_per_well_CP228A.png exists, overwriting
/home/ubuntu/efs/2018_11_20_Periscope_Calico/workspace/software/Meraj_analysis/CP228W/recipe/scripts/io_utils.py:11: UserWarning: data/0.site-qc/20210124_6W_CP228/figures/plate_layout_PercentConfluent_per_well_CP228B.png exists, overwriting
/home/ubuntu/efs/2018_11_20_Periscope_Calico/workspace/software/Meraj_analysis/CP228W/recipe/scripts/io_utils.py:11: UserWarning: data/0.site-qc/20210124_6W_CP228/results/sites_with_confluent_regions.csv exists, overwriting
/home/ubuntu/efs/2018_11_20_Periscope_Calico/workspace/software/Meraj_analysis/CP228W/recipe/scripts/io_utils.py:11: UserWarning: data/0.site-qc/20210124_6W_CP228/figures/PLLS_per_well_CP228A.png exists, overwriting
/home/ubuntu/miniconda3/envs/pooled-cell-painting/lib/python3.7/site-packages/plotnine/layer.py:401: PlotnineWarning: geom_text : Removed 360 rows containing missing values.
/home/ubuntu/efs/2018_11_20_Periscope_Calico/workspace/software/Meraj_analysis/CP228W/recipe/scripts/io_utils.py:11: UserWarning: data/0.site-qc/20210124_6W_CP228/figures/PLLS_per_well_CP228B.png exists, overwriting
/home/ubuntu/miniconda3/envs/pooled-cell-painting/lib/python3.7/site-packages/plotnine/layer.py:401: PlotnineWarning: geom_text : Removed 360 rows containing missing values.
Traceback (most recent call last):
File "/home/ubuntu/miniconda3/envs/pooled-cell-painting/lib/python3.7/site-packages/pandas/core/indexes/base.py", line 3081, in get_loc
return self._engine.get_loc(casted_key)
File "pandas/_libs/index.pyx", line 70, in pandas._libs.index.IndexEngine.get_loc
File "pandas/_libs/index.pyx", line 101, in pandas._libs.index.IndexEngine.get_loc
File "pandas/_libs/hashtable_class_helper.pxi", line 4554, in pandas._libs.hashtable.PyObjectHashTable.get_item
File "pandas/_libs/hashtable_class_helper.pxi", line 4562, in pandas._libs.hashtable.PyObjectHashTable.get_item
KeyError: 'level_3'
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
File "recipe/0.preprocess-sites/4.image-and-segmentation-qc.py", line 511, in
cp_sat_df[["cat", "type", "Ch"]] = cp_sat_df["level_3"].str.split(
File "/home/ubuntu/miniconda3/envs/pooled-cell-painting/lib/python3.7/site-packages/pandas/core/frame.py", line 3024, in getitem
indexer = self.columns.get_loc(key)
File "/home/ubuntu/miniconda3/envs/pooled-cell-painting/lib/python3.7/site-packages/pandas/core/indexes/base.py", line 3083, in get_loc
raise KeyError(key) from err
KeyError: 'level_3'
Config file set to perform=False, not performing step: profile--single_cell
Loading one single cell file: data/1.profiles/20210124_6W_CP228/single_cell/20210124_6W_CP228_single_cell_profiles_20210124_6W_CP228___ALLPLATES___perfectset1.csv.gz
Config file set to perform=False, not performing step: profile--normalize

Well, I'm glad it didn't make anything EXCESSIVELY WORSE, at minimum! Did you get any sense of if it seemed faster or slower than expected based on your experience doing the same steps with the master branch, or too hard to say?
I'm not sure what specifically that error might mean in this context.
We probably want to look into this more carefully- briefly, we can set up a machine with the CloudWatch Agent installed and configured to do memory profiling at a pretty tight interval (30 seconds?), then run successive tests on the same data set with this branch and on master.  I'm happy to help get such a machine set up some day next week if someone wants to put a meeting on my calendar, though i certainly need not be involved if someone else is motivated and wants to dig in further.

I don't think the update has any major side effects. It is hard to be quantitative about it but it definitely felt faster than before ( hard to be sure without testing it).
The traceback error happened on the other machine as well but it is not stopping the process so probably not very serious. Memory still seems to be the issue but the update might have improved the situation. More careful comparison can help to measure any possible improvements.
It is a good idea to get together with @gwaygenomics and setup the machine as you suggested to dig a little bit deeper. At least by figuring out the requirements we can optimize the type of machine setup for the task.

Thanks both - I've sent you calendar invites for tomorrow to discuss memory and alarm issues.

Also, I think we're good to merge this in, right @MerajRamezani - no errors on your side?

If we're going to memory profile this tomorrow, I would say let's wait for the results - I've got a strong suspicion that this will help, but I'd rather close unmerged if it turns out to hurt.

sounds good!

Per @MerajRamezani - branch (lower graph) knocks a couple of minutes off the processing time AND decreases memory usage by about 20% (42/54 ~= 0.8) compared to master (upper graph).  Not a slam dunk, but better than before!
We may or may not want to test this with a lower chunksize going forward, we may see even further gains, though likely not the only optimization we need to do.
Master:

Branch:

@bethac07 @MerajRamezani I'm about to weld the next data - good to merge this?


fix(RemoteRenderingView): add onMounted in setup component

:tada: This PR is included in version 8.1.1-rc.1 :tada:
The release is available on:

GitHub release
npm package (@next dist-tag)

Your semantic-release bot :package::rocket:


Adds a simple panel for loading custom science data

This is a minor quality of life improvement but it enables science data to be loaded using a GUI. You can click on "Environmental Configuration" panel when running ign gazebo empty_environment.sdf then it will open a file browser to load the correct data. Under the hood this will sens a message the to science data plugin to trigger a reload.

Closed in favor of #211


feat: add a gRPC & CLI query to get a validator's delegated voter address

Description
Adds a querier & cli command for getting a validator's delegated voter address
closes: #324

Author Checklist
All items are required. Please add a note to the item if the item is not applicable and
please add links to any relevant follow up issues.
I have...

[ ] included the correct type prefix in the PR title
[ ] added appropriate labels to the PR
[ ] added ! to the type prefix if API or client breaking change
[ ] targeted the correct branch (see PR Targeting)
[ ] provided a link to the relevant issue or specification
[ ] added a changelog entry to CHANGELOG.md
[ ] included comments for documenting Go code
[ ] updated the relevant documentation or specification
[ ] reviewed "Files changed" and left comments if necessary
[ ] confirmed all CI checks have passed

Reviewers Checklist
All items are required. Please add a note if the item is not applicable and please add
your handle next to the items reviewed if you only reviewed selected items.
I have...

[ ] confirmed the correct type prefix in the PR title
[ ] confirmed ! in the type prefix if API or client breaking change
[ ] confirmed all author checklist items have been addressed
[ ] reviewed state machine logic
[ ] reviewed API design and naming
[ ] reviewed documentation is accurate
[ ] reviewed tests and test coverage
[ ] manually tested (if applicable)

Codecov Report

Merging #325 (c37db5e) into main (5e6a480) will decrease coverage by 0.14%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #325      +/-   ##
==========================================
- Coverage   32.78%   32.63%   -0.15%
==========================================
  Files          74       74
  Lines        9258     9300      +42
==========================================
  Hits         3035     3035
- Misses       5930     5972      +42
  Partials      293      293

Impacted Files
Coverage Δ

x/oracle/keeper/grpc_query.go
51.03% <0.00%> (-5.89%)
:arrow_down:

x/oracle/types/query.pb.gw.go
0.00% <0.00%> (ø)


Usage as PNG image/source object on RN 0.14 not working!

As of 0.14 release, React Native changed the way of managing images in apps. Seems it causes using icons as png don't work.
I have tested on Android device and debugging returns icon object state as
{uri: file:///path/to/file/something.png}
maybe "file:///" causes this issue?!

It works as expected on iOS, but not on android. I've looked into the issue quickly and can't really understand why it doesn't work, the new asset system actually doesn't seem to bother with file:/// images, but maybe that's the whole problem. If you can do further investigations in the git history to see what commit broke this I'd be grateful!

@oblador thank you for your response, today without any changes the problem was solved! On android device i could get it to work as expected but the quality of icon images is not good in medium or big sizes, and i wonder to request an improvement to your plugin, if you can make usage as png image/source to work with require function we can also use the png files inside toolbarAndroid and other plugins, thanks

@knot123: it will work with other plugins and my intention is for it to work with ToolbarAndroid eventually and I don't really understand why it doesn't now since networked images support was recently added. But from what I understand it's discouraged by FB. The documentation says:

Although the Toolbar supports remote images for the logo, navigation and action icons, this should only be used in DEV mode where require('./some_icon.png') translates into a packager URL. In release mode you should always use a drawable resource for these icons. Using require('./some_icon.png') will do this automatically for you, so as long as you don't explicitly use e.g. {uri: 'http://...'}, you will be good.

About the quality, pixel ratio is respected and should give you crisp icons! Do you pass a size argument?

@oblador Is there any way we can get it to work with ToolbarAndroid?

Not that I know of unfortunately, but it something I'd like to support. Tried to research why not, because even remote images work, but couldn't figure it out. Might make a second try and maybe make a PR to RN core, but don't count on it.


NextJS support?

Prerequisites:

[x] This is question only ❓
[x] I am running the latest version
[x] I checked the documentation and found no answer
[x] I checked to make sure that this is not already asked
[x] I'm asking the question in the correct repository

Describe the question
I'm trying to get used/learn NextJS and wanna do that by creating a little app with user auths (and db backend is on the server). Just read the react and nextjs tutorials and did a
npm exec create-next-app ${APP_NAME}
npm install --save  react-auth-kit

Now I try to follow Add Login/Auth to your React app in 5 mins as well as the docs but it doesn't seem to work, e.g.:
Cannot find module './routes' or its corresponding type declarations.ts(2307)

Do you have any hints how to fix it? Can you add some instructions or hints to the documentation? Or providing an example, which works out of the box would be great!
Please provide the following information:

OS: Linux (Ubuntu 20.04)
Browser: Firefox 111
Nodejs-Version: v18.15.0

@LupuC thanx. IMHO the real problem is the chaotic shallow documentation of NextJS. If they would provide a "basic concepts" chapter, there would probably much less confusion. Right now it is like in old UNIX times: You need to know, what you are looking for (man $tool) ;-) .
Anyway, meanwhile after watching some youtube stuff and reading many docs I'm currently happy with:
import { useSession } from "next-auth/react";
import { useRouter } from "next/router";
import React, { PropsWithChildren, useContext } from "react";
import GlobalCtx from "./global_ctx";

// components/session.tsx

/**
 * Ensure there is a session running when pages beneath /app/ are requested.
 * If not, just show a sign in message and stop the flow.
 * @param props page properties. Uses the auto generated `props.children` to
 * 	render the JSX componentens inside the `<EnsureSession>...</EnsureSession>`
 *  tag if a session exists and the user has loged in. Otherwise props.children
 *  get ignored.
 * @returns the page to render.
 */
const DEBUG = true;

export default function EnsureSession(props: PropsWithChildren) {
	const session = useSession();
	const router = useRouter();
	const ctx = useContext(GlobalCtx);

	if (session.status === 'loading') {
		if (DEBUG)
			return (

Lade ...

			);
		return <></>
	}
	if (DEBUG) console.log('EnsureSession:\n', session);
	if ((session !== null && session.status === 'authenticated')
		|| (!router.pathname.startsWith('/app/') && router.pathname !== '/app'))
	{
		// We rely on userUpdate() that it does the right thing.
		ctx.userUpdate(session.data?.user);

		return (
			<>
				{props.children}
			</>
		);
	}

	router.push('/api/auth/signin');
	return <></>;
}

and
import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>
import type { AppProps } from 'next/app';
import { SessionProvider } from "next-auth/react";
import { StrictMode } from 'react';
import EnsureSession from '@/components/session';
import { GlobalCtxProvider } from '@/components/global_ctx';
import { SESSION_TIMEOUT } from '@/lib/utils';
import Layout from '@/components/layout';

// pages/_app.tsx

export default function App ({ Component, pageProps: { session, ...pageProps } }:
	AppProps)
{
	let s = process.env.APP_CONTEXT ?? '';
	s = s.trim();
	const APP_CONTEXT = s.startsWith('/') ? s.substring(1) : s;

	return (
		<StrictMode>
		<GlobalCtxProvider> {/* needs to sit here to survive session updates */}
			<SessionProvider session={session}
				refetchInterval={SESSION_TIMEOUT} basePath={APP_CONTEXT}>
				<EnsureSession>
					<Layout>
						<Component {...pageProps} />
					</Layout>
				</EnsureSession>
			</SessionProvider>
		</GlobalCtxProvider>
		</StrictMode>
	);
}

The solution seems to be: do not use routes.

I will create the next plugin by the end of January. OSS's contribution will be appreciated.

@darkmatter18 Hope to see Next.js support! By the way, the createStore with localstorage cannot be used in the SSR environment (it tries to access the localstorage but it does not exist in server environment)

Next JS support is added on V3.1.0, but for now it's only uses Client Side Rendering. Hope to see SSR in near future.


[Client] Add collectd into meta-comodoo

Add Collectd for metrics gathering from device.

The sample file used in #11 configuration, here: /etc/collectd.conf
Hostname    "producer-node1"
FQDNLookup   true
BaseDir     "/var/lib/collectd"
PIDFile     "/var/run/collectd.pid"
PluginDir   "/usr/lib64/collectd"
TypesDB     "/usr/share/collectd/types.db"

Interval     1
ReadThreads     5
WriteThreads    5

LoadPlugin logfile
<Plugin logfile>
        LogLevel info
        File STDOUT
        Timestamp true
        PrintSeverity false
</Plugin>

LoadPlugin csv
#LoadPlugin syslog

LoadPlugin cpu
LoadPlugin cpufreq
LoadPlugin cpusleep
LoadPlugin disk
LoadPlugin interface
LoadPlugin load
LoadPlugin lvm
LoadPlugin memory
LoadPlugin sensors
LoadPlugin statsd
LoadPlugin swap

<Plugin csv>
        DataDir "/var/lib/collectd/csv"
        StoreRates true
</Plugin>

#<Plugin "syslog">
#  LogLevel "info"
#  NotifyLevel "OKAY"
#</Plugin>

<Plugin disk>
        Disk "/^[hs]d[a-f][0-9]?$/"
        IgnoreSelected false
        UdevNameAttr "DEVNAME"
</Plugin>

Include "/etc/collectd.d"

Changes to complete this issue will be done in branch krogoth-syslog.

@javiroman The version of collectd provided by meta-openembedded is 5.5.0 and it does not include the plugin called cpusleepcpusleep. Is this a problem??

@indisoluble no problem with this collectd version. We'll disable this metrics.

In that case, with the changes I made in commit 8e2f626f07f2a9369fbb693c1b65aff8423d5c23, we are already generating a Yocto image with the following configuration for collectd: collectd.conf.
@javiroman Whenever you have a  moment, have a look and confirm if this configuration is alright. There are some minor differences:

Hostname    "comodoo-pos" instead of Hostname    "producer-node1"
FQDNLookup   false instead of FQDNLookup   true
PIDFile, PluginDir & TypesDB remain commented
LoadPlugin cpusleep not included

So far, it looks like this configuration is alright; @javiroman if you think there is a problem, please open a new issue and link it to this one.


Update README.md

Adding v20 in JSON Sheets List

@ShikariShambuJr , Thanks for the PR.
Can you also add v21, v22 and the vaccination data sheet to readme?

Done.Please Approve


add nttnue

@haipv2 First stage of verification passed, official domain is ntthnue.edu.vn as far as I verified. This is just an informational note. Pull request is still in the review. I appreciate your patience while waiting for the final decision.

@haipv2 Thank you for your patience while waiting for a review and processing your request. Pull request merged. Thank you!


Is plugin implementation on the TensorRT side necessary?

Hi,
First, thank you for this repo.
I understand that we need to implement custom layer on the onnx-tensorrt side inside builtin_op_importers.cpp in order tensorrt onnx parser (nvonnxparser) to be able to parse the custom layer. Shouldn't this be enough in order to parse the onnx model file and create a TensorRT inference engine?
I see that you implemented TensorRT plugin and even CUDA implementations for the grid sample op. I thought TensorRT plugins were necessary in order to parse unsupported layers by the uff or caffe parsers (maybe onnx also). But by just implementing custom layer inside builtin_op_importers.cpp and rebuilding onnx-tensorrt, nvonnxparser should be able to parse the model, shouldn't it?
Thanks in advance.

Within the scope of DEFINE_BUILTIN_OP_IMPORTER(GridSampler)
    const auto mPluginRegistry = getPluginRegistry();
    const auto pluginCreator
        = mPluginRegistry->getPluginCreator(pluginName.c_str(), pluginVersion.c_str());

It is used to find plugin with name "pluginName" and  version "pluginVersionin" in the registry.

Hi,
First, thank you for this repo.
I understand that we need to implement custom layer on the onnx-tensorrt side inside builtin_op_importers.cpp in order tensorrt onnx parser (nvonnxparser) to be able to parse the custom layer. Shouldn't this be enough in order to parse the onnx model file and create a TensorRT inference engine?
I see that you implemented TensorRT plugin and even CUDA implementations for the grid sample op. I thought TensorRT plugins were necessary in order to parse unsupported layers by the uff or caffe parsers (maybe onnx also). But by just implementing custom layer inside builtin_op_importers.cpp and rebuilding onnx-tensorrt, nvonnxparser should be able to parse the model, shouldn't it?
Thanks in advance.

Just as michael-peng commented, in DEFINE_BUILTIN_OP_IMPORTER(GridSampler), we instruct onnxparser to parse onnx op to tesnorrt layers. If the op is conv, you just translate onnx conv to tesnorrt conv which has native mapping in tensorrt, i.e., IConvolutionLayer. But if the op is GridSampler here which is not directly mapped to any tensorrt layer combination, then, we have to implement a plugin for tensorrt and add the plugin into the network. What you implement in builtin_op_importers.cpp is not the custom layer itself but the mapping (from onnx op to tensorrt op).

@TrojanXu Hi, I am also confused about the DEFINE_BUILTIN_OP_IMPORTER.
I am new at TensorRT. I read some plugin examples/blog and tensorrt develop guide.
Many project referred out the way to register plugin by REGISTER_TENSORRT_PLUGIN or DEFINE_BUILTIN_OP_IMPORTER.
And I could not find how to register plugin in the repo.
Dose the plugin is registered by LD_PRELOAD?
By the way, Could you provide some suggestion about how to implement the plugin in TensorRT or some overal blog/tutorial ?
Thanks!


my game region keep reseting to North America even i already choose japan it wont save

my game region keep reseting to North America even i already choose japan it wont save

already tick it. and using xiv quick launcher


fix: mandatory limit

Overview
Swagger cannot generate optional parameters request body
Changes

Mandatory limit on get all transactions (it's dangerous to leave it without, we should also limit the amount!)

Testing
@boolafish

Coverage remained the same at ?% when pulling 93943eec70b8849e092d1b41e69bb856668012cb on inomurko/fix_swagger_for_get_all_transactions into 862a6adfd436fbb1316be760ac6b72090127a115 on master.

Can you help me understand this :pr: a little better.
If I got it correctly API client generators we use don't handle optional parameters well?
Therefore to fix it, we're faking our documentation (API specs) - is it what this PR delivers?

Not really. This PR requires you to send a linit param in get all transactions because it’s a security issue to nit limit the response.

☝ absolutely. the backend should always enforce such things. client can ask for what it wants, but server validates/adjusts the constraints.
A minor thing, but IMHO, I hope we can improve our pagination handling, there is another for ticket for that: https://github.com/omisego/elixir-omg/issues/1215)
But specific to this ticket, I think it's not a good API user experience. The server validates and adjusts the limit. But there is no indication of this to the user. This can make them think there is a bug in our code or they did something wrong etc...
Returning a response with error message explaining that their pagination limit request exceeds the max limit of #### would be more useful and informative and less confusing to the API user.

I agree with @jrhite.
@unnawut I think the server should set a cap on the limit param
Anyhow, this is mostly getting aroung swagger bugs that can’t handle all optional params in requesys


fix(executor): GODEBUG=x509ignoreCN=0

To test this,lets merge it and make sure https://workflows.apps.argoproj.io/ can run coin flip

Next steps.

[ ] Wait for new image.
[ ] Run coinflip

@simster7 @sarabala1979 this appears to have fixed the issue on GCP.


Workaround that should fix #422 (liquid complaining for example on syntax highlighting blocks)

As mentioned in the issue #422, downgrading to version 2.2.2 of Liquid let pygment highlighting run ok.
Please notice that I don't have any real ruby experience, let alone rubygems: I've just dug to find the way to set max version for a dependency. I've tried this patch in my local installation and works, but I'm not aware of any possible side effect the max-version dependency could introduce.
Bye

Please consider my pull request #441 which fixes the problem without restricting people from using modern versions of Liquid.

agree with @kylev :) . Closing this pull request


reformatted *.go with gofmt -s -w

This is done to fix https://goreportcard.com warnnigs:
gofmt 33%
Gofmt formats Go programs. We run gofmt -s on your code, where -s is for
the "simplify" command
intel-device-plugins-for-kubernetes/cmd/fpga_plugin/fpga_plugin_test.go
Line 1: warning: file is not gofmted with -s (gofmt)
intel-device-plugins-for-kubernetes/internal/deviceplugin/deviceplugin_test.go
Line 1: warning: file is not gofmted with -s (gofmt)
intel-device-plugins-for-kubernetes/cmd/gpu_plugin/gpu_plugin_test.go
Line 1: warning: file is not gofmted with -s (gofmt)
intel-device-plugins-for-kubernetes/cmd/fpga_plugin/fpga_plugin.go
Line 1: warning: file is not gofmted with -s (gofmt)

will be replaced with new PR


Docs for supported databases / datastores are potentially confusing

The supported frameworks documentation might be confusing for people interested in knowing about compatibility with datastores for the following reasons:

The "instance details" section still lists ActiveRecord 2/3/4 as supported, which is incorrect
The existence of separate "databases" and "instance details" sections to outline datastore compatibility is confusing
The "instance details" section has a "gem databases" table which is confusing. The term "gem databases" is incorrect. Perhaps there was once an intention to list individual gems used for datastore client usage. The NoSQL stores Mongo and Redis are listed both in this table and in the one in the "databases" section. Given that there is only a single supported gem for each of these datastores, moving the info into the "databases" section would make the most sense.

https://docs.newrelic.com/docs/apm/agents/ruby-agent/getting-started/ruby-agent-requirements-supported-frameworks/

Instance details was updated with: https://github.com/newrelic/docs-website/pull/19444#event-15540735339
We should still consider bullet points 2 and 3 in the description.


Fix incorrect sourcemap destination

According to the documentation:

To write external source map files, pass a path relative to the destination to sourcemaps.write().

In WPGulp, however, there is this:
.pipe( sourcemaps.write ( styleDestination ) )
.pipe( lineec() ) // Consistent Line Endings for non UNIX systems.
.pipe( gulp.dest( styleDestination ) )

which, if styleDestination is set to './dist/', effectively creates this structure:
dist/
├── dist/
│   └── style.css.map
└── style.css

(notice the double dist/)
I've fixed this by setting the write destination to proper - current folder.

please ignore 11f761e and 2d5ff29, they are meant for another PR, but I messed up something here on GH, so now they're showing here...

Thanks for that. I think it's better if you create a new PR or clean this one up. Looking forward!

@ahmadawais will do. However, as you seem quite busy and I'd love to help developing this further, as this (WordPress + Gulp) is currently what I'm working with, would you consider adding me as a maintainer?

@selrond you couldn't have asked for it at a better time. Have you seen #61 I just announced the WPGulp 2.0 — hop over there and comment on your ideas and contribute. That's the best way for both of us to get to know each other and work together as things progress i.e. by adding you as a maintainer.
→ https://github.com/ahmadawais/WPGulp/issues/61

@ahmadawais done, ready to be merged

Merged! Thanks! 🤗


Create LongestPalinString

Hello @ritik001!
Could you please elaborate about the purpose of this PR?
@exercism/cpp could you please label as invalid for now?


Game freezes when closing the game immediately after it starts

Reproducible with Debug Menu v1.2.0 in Godot v4.2.1.stable.mono.official [b09f793f5].
System Information
Godot v4.2.1.stable.mono - Windows 10.0.22631 - Vulkan (Forward+) - dedicated NVIDIA GeForce RTX 3060 Laptop GPU (NVIDIA; <IP_ADDRESS>33) - 12th Gen Intel(R) Core(TM) i7-12700H (20 Threads)
Issue Description
Closing the game immediately after it starts causes the game to freeze when the "Debug Menu" addon is enabled.
Minimal reproduction project (MRP)
Test_Bug.zip

I can't reproduce this on my end with Godot 4.2.1 and Debug Menu 1.2.0 on Linux. Here it is working after being installed in an empty project:

I can't reproduce this on my end with Godot 4.2.1 and Debug Menu 1.2.0 on Linux. Here it is working after being installed in an empty project:

To reproduce the issue, I need to close the game right after launch, before the debug menu loads hardware and project info. It will then become unresponsive, as shown below:

Thanks for the report! Consolidating in https://github.com/godot-extended-libraries/godot-debug-menu/issues/22.


docs: add docs for file imports

Description

Closes #250
Adds docs for file imports using shortcuts file and solution

@AriPerkkio, I agree with your proposed structure! Long term we could even have the # Markdown support section describes everything that's possible but right now if it shows the most commonly used things and link to expressive-code that would be a great start :raised_hands:

Thanks for the review and clarification. I wasn't sure if I should write an entire markdown/mdx section so I decided to start with just file imports.


feat(compiler): output source range for compiler errors (#6338)

What kind of change does this PR introduce? (check at least one)

[ ] Bugfix
[x] Feature
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change? (check one)

[ ] Yes
[x] No

The PR fulfills these requirements:

[x] It's submitted to the dev branch for v2.x (or to a previous version branch), not the master branch
[x] When resolving a specific issue, it's referenced in the PR's title (e.g. fix #xxx[,#xxx], where "xxx" is the issue number)
[x] All tests are passing: https://github.com/vuejs/vue/blob/dev/.github/CONTRIBUTING.md#development-setup
[x] New/updated tests are included

If adding a new feature, the PR's description includes:

[x] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)

Other information:
Just like #6764, but will output the range data instead of writing it into the error message. Here's an example of how it works:
https://gist.github.com/gzzhanghao/363fac4a201306f5fccf903e35bc652a

This is great! Thanks a lot for the work. Since this is a big PR, it will take some time to properly review. But I hope we can include this in 2.6.

@HerringtonDarkholme I've just enclosed outputSourceRange code into conditional env block in https://github.com/vuejs/vue/pull/7127/commits/facacc1fefb0773778ae8eac20acabc82b602810
And here are the new bundle size from npm run build:
File                                          | Origin        | New
---
dist\vue.runtime.common.js                    | 220.81        | 220.82
dist\vue.common.js                            | 318.01        | 322.35
dist\vue.esm.js                               | 318.00        | 322.33
dist\vue.js                                   | 315.06        | 319.35
dist\vue.min.js                               | 91.54 (31.78) | 92.21 (32.00)
packages\vue-template-compiler\build.js       | 177.66        | 185.11
packages\vue-template-compiler\browser.js     | 260.55        | 267.90
packages\vue-server-renderer\build.js         | 261.09        | 265.02
packages\vue-server-renderer\basic.js         | 316.17        | 320.05

@gzzhanghao Thanks!  Size overhead seems to be acceptable! Thanks again!

@yyx990803 I need to tranfsorm start, end to be line number. I'm writing small tool to static analysis of code. I have something like:
const content = FileUtils.readFile(path);
const template = compiler.parseComponent(content).template.content;
const ast = compiler.compile(template, { outputSourceRange: true }).ast;

I'm searching for "something" in ast. When I find "something" I'd like to create output like f.e.:
/components/v-some-component.vue:11

where 11 is line number. Do you you think I should do it myself in my code or I can create issue to as about extending CompilerOptions ?


[time series table] visual improvements

@graceguo-supercat @michellethomas
cc @mistercrunch
This PR adds several visual improvements to the time series table 🚀

improves positioning of tooltip by updating to<EMAIL_ADDRESS>adds partial tooltip opacity also enabled by<EMAIL_ADDRESS>uses a more neutral #767676 for the sparkline color so it doesn't clash with the red/blue
adds ability to customize the format of the sparkline date
removes grey background from table rows on hover
font size increased from 12px to 14px
color encoding is made more apparent and normalized for area through cell coloring (vs just the number)

Before

After

Gif

Side note: I've been thinking that we could parameterize the font size. When there's a sparkline there's plenty of room to bring in bigger digits. Could also allow users to add emphasis on some of the columns.

wohoo! thanks 🎉


feat: separating out array and slice types in the AST

Description
Replacement for https://github.com/noir-lang/noir/pull/4232. Holding off on closing it to possibly port fixes from it, but wanted to start fresh because this approach is quite different.
Problem*
Resolves https://github.com/noir-lang/noir/issues/4220
Summary*
Additional Context
Documentation*
Check one:

[ ] No documentation needed.
[ ] Documentation included in this PR.
[ ] [Exceptional Case] Documentation to be submitted in a separate PR.

PR Checklist*

[ ] I have tested the changes locally.
[ ] I have formatted the changes with Prettier and/or cargo fmt on default settings.

Actually, just looked at the error message in that example more closely. It looks like somewhere the argument order for Type::Array was reversed

@jfecher What do we think about using as_slice() for slice literals rather than &[]. Do we think that syntax will be confusing for users with us not supporting immutable references?


Always in "initializing" status when setup tenant with openapi recipe

Hi,
I am new here and playing around this openapi sample. But Always in "initializing" status when setup tenant with openapi recipe

Or enable the OpenApi Module feature will cause the exist tenant into dead state no response.
Appreciate any advice. Thanks

Can you have a look at the logs?

Unlucky, there was no log generated for the action.

But I found that the yessql.db-journal file was flashing. Could it be the yessql initializing/data loading failure?

Yes, more likely the database did not close a transaction properly.

@Skrypt  found the issue


refactoring

some refactoring and code cleaning to have easier use without changing the code...
main features:

refactoring to package and scripts
the yolo3 package can be installed
using scripts with parameters
updating some structures

@qqwweee ^^


nixos/mastodon: updates could be handled better

Issue description
When using the Mastodon NixOS module, updates currently work by the mastodon-init-db oneshot service running database migrations, and each individual Mastodon service being restarted. However, in some cases this can require manually restarting specific services, and in all cases it will lead to downtime.
The manual upgrade process recommended by upstream is very different, and should not result in downtime. It generally consists of restarting mastodon-sidekiq and mastodon-streaming (the latter only necessary for some releases), then reloading mastodon-web (which in the upstream unit sends SIGUSR1). Some releases may have additional steps. When this applies, it's usually running database migrations via SKIP_POST_DEPLOYMENT_MIGRATIONS=true bundle exec rails db:migrate before restarting/reloading services, and running bundle exec rails db:migrate afterwards.
While it wouldn't necessarily always work, and would still do some unnecessary steps for minor releases, I think it should be feasible for the module to do a zero-downtime update. The general template would likely be:

Have a unit run SKIP_POST_DEPLOYMENT_MIGRATIONS=true bundle exec rails db:migrate before restarting/reloading anything
Keep the current restart behavior for mastodon-sidekiq and mastodon-streaming
Set systemd.services.mastodon-web so that systemctl reload mastodon-web sends SIGUSR1 like the upstream unit
Set systemd.services.mastodon-web.reloadTriggers so that it gets reloaded instead of restarted
Have bundle exec rails db:migrate run once all three services have been reloaded/restarted successfully

I'm not fully sure how to implement this, but I think it should be possible.
Steps to reproduce
Update the Mastodon package when using the NixOS module.
Technical details
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
 - system: `"aarch64-linux"`
 - host os: `Linux 5.15.76, NixOS, 22.05 (Quokka), 22.05.20220616.760ed64`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.9.1`
 - channels(root): `"nixos-22.05"`
 - nixpkgs: `/nix/store/1wiw1x4n29nzaj1sp7kqmwg1xnhskb3d-source`

Thinking about this more, one way to implement this might be to replace the current mastodon-init-db.service with an umbrella mastodon.service. Then, the other three services could be set to BindsTo and After mastodon.service so that they aren't newly started with an unmigrated database. For the proper behavior on updates, the three individual services could have restartIfChanged set to false, mastodon.service could have reloadTriggers set, and its reload script could orchestrate the update (run pre-deployment migrations, restart mastodon-sidekiq and mastodon-streaming, reload mastodon-web, and finally run post-deployment migrations).

Two issues are apparent with using SIGUSR1, but both are solvable:

SIGUSR1 is asynchronous, so it can't be known when the restart is completed. However, Puma (the web server implementation handling SIGUSR1) supports using a control socket instead: https://github.com/puma/puma#controlstatus-server
With Nix, the new Mastodon package will be at a different path. Puma's solution to this issue is that if the config file specifies the application directory (instead of having it default to the working directory) then that can be a symlink, and the symlink will be resolved at each restart: https://github.com/puma/puma/blob/master/docs/restart.md#additional-notes-1


Add Colombian holidays

Contributing a new country ?

[x] Have you checked to ensure there aren't other open Pull Requests for the same country?

Yes, i have checked and I know PR#55 exists for the same country, but it has problems with phpstan. I have rewritten the function that generated problems with types and now the phpstan analysis passes.
PR#55 can be closed in favour of this.

Create a new class in the Countries directory. It should extend the Country class.
Add a test for the new country in the tests directory.
Run the tests so a snapshot gets created.
Verify the result in the newly created snapshot is correct.

Done. https://github.com/spatie/holidays/pull/145

Could you help https://github.com/spatie/holidays/pull/55 with fixing it instead? Thanks!


Windows Builds cache misses

Bulk of the build time is Windows since it keeps finding the build cache that doesn't exist.

Heads up.
I think this has been fixed in actions/cache@v3.
Checkout MediaPipeUnityPlugin's Windows test CI.
It does manage to cache the Bazel folder properly.

Welp...

Seems like we're missing something. We'll have to investigate at one point.

Logged in https://github.com/actions/cache/issues/1109

Accidentally solved it in GH-16, whoops. Looks like the new v3 runners can cache them now.


Secondary Schools of Extremadura(Spain)

Added reference to secondary schools of Extremadura(Spain)

secundary school added

@miguelespino3 I am sorry for a delay with processing your request.
Pull request merged. Thank you!


popUntilRootX Equivalent

When I'm working with compose navigation, Is there an equivalent to the popUntilRootX method?
didn't find the way to pop back to root there is only
navController.popBackStackX()
Or I'm supposed to continue using the compose pop method
navController.popBackStack(              route = AppMainScreen,             inclusive = false    )

no it wont work, because this will try to pop the backstack from compose navigation, this needs to be added


feat(user-edit): Added composition of user edit - Front 3040

you forgot to add this new package as a dependency in bcl-twig-templates...
In any case, once we merge this, we will put everything in one single package, the limitation we had before is not anymore in place, so no reason why multiplicating the packages (even if they are private ones)


token 2022: add a test for mint with group and member pointer

This PR adds a test to ensure a mint can successfully be initialized with both
a GroupPointer and a GroupMemberPointer.
A mint may be both a member of a group, but also a group itself (or sub-group).

Current dependencies on/for this PR:

master

PR #5628

PR #5839
PR #5629

PR #5927   👈

This stack of pull requests is managed by Graphite.

Current dependencies on/for this PR:

master

PR #5628

PR #5839
PR #5629

PR #5927   👈

This stack of pull requests is managed by Graphite.

Merge activity

Nov 29, 12:50 PM: @buffalojoec started a stack merge that includes this pull request via Graphite.


Feature - Fields for both mainnet and testnet addresses in subgraph.yaml

Do you want to request a feature or report a bug?
feature
What is the current behavior?
When I work with subgraphs, I sometime switch back and forth between the testnet and mainnet contracts a few times throughout the day. This means I have to change the contract addresses of the subgraph.yaml. Sometimes I forget, and then I think I have a bug, only to realize I'm trying to grab events from kovan , while connected to an ethereum node.
Currently, I do have both addressed in comments to make my life easy:
  source:
    address: "0x4e0f2b97307ad60b741f993c052733acc1ea5811" # Kovan: 0x9662d6cae0e6914a388cb96c1c161cc4d12c3d7a Mainnet: 0x4e0f2b97307ad60b741f993c052733acc1ea5811

What is the expected behavior?
From a UX perspective it would be nice to have both fields . I've never seen contracts on multiple test networks, it is always one test suite on a chosen test net, and the mainnet smart contracts. So I suggest one field for mainnetAddress and one for testnetAddress, rather than having 4 fields for mainnet and the three testnets. See below:
  source:
    mainnetAddress: "0x4e0f2b97307ad60b741f993c052733acc1ea5811"
    testnetAddress: 0x9662d6cae0e6914a388cb96c1c161cc4d12c3d7

But, we gotta ask ourselves:

Is it worth our time and what version should we try to release for?
This causes the subgraph ID to be the same for a mainnet subgraph and a testnet subgraph. Is this desirable?

well, it turns out 0x uses mainnet, ropsten, and kovan, so I am wrong about only needing two addresses. I guess having fields for all four wouldnt hurt

now that we have a field to indicate that its mainnet, or kovan, or ropsten, we can close this.


ApiTest. "create a New Tus resource" method doesn't create resource

In ocis we use core method When user "Alice" creates a new TUS resource for the space "Project Jupiter" using the WebDAV API with these headers: which doesn't create resource
if you run test:
Scenario: upload a file bigger than the set quota to a project space
    Given user "Alice" has created a space "Project Jupiter" of type "project" with quota "20"
    When user "Alice" creates a new TUS resource for the space "Project Jupiter" using the WebDAV API with these headers:
      | Upload-Length   | 10                       |
      # dXBsb2FkLnR4dA== is the base64 encoded value of filename upload.txt
      | Upload-Metadata | filename dXBsb2FkLnR4dA== |
      | Tus-Resumable   | 1.0.0                     |
    Then the HTTP status code should be "201"
    And for user "Alice" the space "Project Jupiter" should contain these entries:
      | upload.txt |

test failed because file didn't create.
here we make dav request: https://github.com/owncloud/core/blob/master/tests/TestHelpers/WebDavHelper.php#L628
we are changing url as: https://localhost:9200/dav/spaces/uuid
but when we create file TUS we use: https://localhost:9200/remote.php/dav/spaces/uuid
My questions: should both requests create resource? if not why we use WebDAV API in creating TUS resource in tests. related: https://github.com/owncloud/ocis/issues/3050

I would expect the API behave the same in both cases
@micbar ?

@butonic ping

According to tus official documentation, we should be able to create files with an upload using creation-with-upload in the Tus-Extension header.
Request with creation-with-upload doesn't create file in UI
curl -ks -u admin:admin -XPOST https://host.docker.internal:9200/webdav/
-H "Tus-Resumable: 1.0.0"
-H "Upload-Length: 100"
-H "Content-Type: application/offset+octet-stream"
-H "Upload-Metadata: filename dGVzdC50eHQ="
-H "Tus-Extension: creation-with-upload"
-d "upload content"
-v | xmllint --format -

Response (same in new, old and spaces dav)
> Tus-Resumable: 1.0.0
> Upload-Length: 100
> Content-Type: application/offset+octet-stream
> Upload-Metadata: filename dGVzdC50eHQ=
> Tus-Extension: creation-with-upload
> Content-Length: 14
>
} [14 bytes data]
* upload completely sent off: 14 out of 14 bytes
{ [5 bytes data]
* TLSv1.3 (IN), TLS handshake, Newsession Ticket (4):
{ [130 bytes data]
* Mark bundle as not supporting multiuse
< HTTP/1.1 201 Created
< Access-Control-Allow-Headers: Tus-Resumable, Upload-Length, Upload-Metadata, If-Match
< Access-Control-Allow-Origin: *
< Access-Control-Expose-Headers: Tus-Resumable, Location
< Content-Length: 0
< Content-Security-Policy: default-src 'none';
< Date: Tue, 13 Dec 2022 07:48:51 GMT
< Location: https://host.docker.internal:9200/data/eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJhdWQiOiJyZXZhIiwiZXhwIjoxNjcxMDA0MTMxLCJpYXQiOjE2NzA5MTc3MzEsInRhcmdldCI6Imh0dHA6Ly9sb2NhbGhvc3Q6OTE1OC9kYXRhL3R1cy9hYmQyNWU1ZC04NTlkLTQxMGUtOWIzYy1iNDcwMTExMTYxMzIifQ.e_V1SE8hZTtL6Fs4OY70QpheDZh4blf33sLj3NoYI5k
< Tus-Extension: creation,creation-with-upload,checksum,expiration
< Tus-Resumable: 1.0.0
< Upload-Expires:<PHONE_NUMBER>
< Upload-Offset: 14
< X-Content-Type-Options: nosniff
< X-Download-Options: noopen
< X-Frame-Options: SAMEORIGIN
< X-Permitted-Cross-Domain-Policies: none
< X-Robots-Tag: none
< X-Xss-Protection: 1; mode=block
<
* Connection #0 to host host.docker.internal left intact
-:1: parser error : Document is empty

^

Currently, we have two types of steps:

low-level creation which uses makeDavResquest. This step doesn't create file while upload even after adding   | Tus-Extension   | creation-with-upload            |

    When user "Alice" creates a new TUS resource with content "uploaded content" on the WebDAV API with these headers:
      | Upload-Length   | 100                             |
      | Tus-Resumable   | 1.0.0                           |
      | Content-Type    | application/offset+octet-stream |
      #    dGVzdC50eHQ= is the base64 encode of test.txt
      | Upload-Metadata | filename dGVzdC50eHQ=           |
     #  | Tus-Extension   | creation-with-upload            | // didn't create file while upload

second which uses ankitpokhrel/tus-php package. This step creates files while upload

When user "Alice" creates file "textFile.txt" and uploads content "12345" in the same request using the TUS protocol on the WebDAV API

curl -vku admin:admin -X PROPFIND https://host.docker.internal:9200/remote.php/webdav/ -d "<?xml version='1.0' encoding='UTF-8'?> <d:propfind xmlns:d='DAV:'> <d:prop></d:prop> </d:propfind>" | xmllint --format -
<?xml version="1.0"?>
<d:multistatus xmlns:s="http://sabredav.org/ns" xmlns:d="DAV:" xmlns:oc="http://owncloud.org/ns">
  <d:response>
    <d:href>/remote.php/webdav/</d:href>
    <d:propstat>
      <d:prop>
        <oc:id>some-admin-user-id-0000-000000000000!some-admin-user-id-0000-000000000000</oc:id>
        <oc:fileid>some-admin-user-id-0000-000000000000!some-admin-user-id-0000-000000000000</oc:fileid>
        <oc:spaceid>some-admin-user-id-0000-000000000000</oc:spaceid>
        <oc:name>Admin</oc:name>
        <d:getetag>"2c4e60e10695f52bfc54d4ba228b9b9d"</d:getetag>
        <oc:permissions>RDNVCKZ</oc:permissions>
        <d:resourcetype>
          <d:collection/>
        </d:resourcetype>
        <oc:size>0</oc:size>
        <d:getlastmodified>Tue, 13 Dec 2022 10:22:50 GMT</d:getlastmodified>
        <oc:favorite>0</oc:favorite>
      </d:prop>
      <d:status>HTTP/1.1 200 OK</d:status>
    </d:propstat>
    <d:propstat>
      <d:prop>
        <oc:file-parent/>
      </d:prop>
      <d:status>HTTP/1.1 404 Not Found</d:status>
    </d:propstat>
  </d:response>
</d:multistatus>

@individual-it

@amrita-shrestha in https://github.com/owncloud/ocis/issues/5054#issuecomment-1347888585 the initial curl request uses an -H "Upload-Length: 100" header, but only sends 14 bytes. The response correctly returns a 201 created status with a location header and the upload-offset: 14.
 We could remove the location header when the file was uploaded completely, but I think that would violate the TUS spec.
In any case ... the client has to either send all bytes, or continue sending bytes to the returned location until the upload-length equals the upload-offset.
@ScharfViktor AFAICT the scenario triggers the same response:
Scenario: upload a file bigger than the set quota to a project space
    Given user "Alice" has created a space "Project Jupiter" of type "project" with quota "20"
    When user "Alice" creates a new TUS resource for the space "Project Jupiter" using the WebDAV API with these headers:
      | Upload-Length   | 10                       |
      # dXBsb2FkLnR4dA== is the base64 encoded value of filename upload.txt
      | Upload-Metadata | filename dXBsb2FkLnR4dA== |
      | Tus-Resumable   | 1.0.0                     |
    Then the HTTP status code should be "201"
    And for user "Alice" the space "Project Jupiter" should contain these entries:
      | upload.txt |

We are sendting an upload-length

The related PR has been merged. Closing this as completed.


Fauxpen source

You seem to refer to your creation as open source, but I believe that "open weights" would be a fairer asssement? Since you haven't open sourced the training data, for example.
Happy to be wrong!

Thanks for asking about this—I hate dubiously "open-source" licenses as much as anyone. We consider this release of hertz-dev to meet the OSI's Open Source Definition. All of our code and weights are released under the Apache 2.0 license, allowing free redistribution and derived works without discrimination against persons, groups, or fields of endeavor and without restriction to use with or without specific products. The most open-to-interpretation point is how the point on source code applies to weights:

The program must include source code, and must allow distribution in source code as well as compiled form. Where some form of a product is not distributed with source code, there must be a well-publicized means of obtaining the source code for no more than a reasonable reproduction cost, preferably downloading via the Internet without charge. The source code must be the preferred form in which a programmer would modify the program. Deliberately obfuscated source code is not allowed. Intermediate forms such as the output of a preprocessor or translator are not allowed.

The hertz-dev weights are "the preferred form in which a programmer would modify the program": base models are valuable because they can be fine-tuned to do specific tasks. On the other hand, the training data is less useful for extending and modifying hertz-dev. We're happy to provide a full codebase, model checkpoint, and architecture details to make this possible.


Fix Build Scan links when running on GitHub Enterprise Server

It is possible to use the GitHub actions with GitHub Enterprise Server. This means that any reference to github.com in Build Scan links will be incorrect. For example:
https://github.com/gradle/common-custom-user-data-gradle-plugin/blob/3908b28b83348c4485bf5d760690b852f4256dba/src/main/java/com/gradle/CustomBuildScanEnhancements.java#L187
Instead of hardcoding github.com we should key off of the GITHUB_SERVER_URL environment variable. According to the documentation, this environment variable is available on Enterprise Server, Enterprise Cloud, and Free.
This aligns with the GitHub documentation as well:

If you need to use a workflow run's URL from within a job, you can combine these environment variables: $GITHUB_SERVER_URL/$GITHUB_REPOSITORY/actions/runs/$GITHUB_RUN_ID

@tylerbertrand do you want to take on this issue and have Jerome review it? //cc @clayburn

Sure 👍


AU manifest_url field

LOCKSSOMatic doesn't have code to generate a manifest_url field for a an AU. The code that's there is just stubbed, pointing at example.com.

This was fixed a while ago, but the issue wasn't updated.


Wrong colours in Light app

Reproduction steps
When the light app is created, the button works fine, but the colours on the LED do not match the colour selected in the home app.
Bug prevalence
Always
GitHub hash of the SDK that was being used

Platform
esp32
Platform Version(s)
No response
Type
Test Improvement
Anything else?

@AchimPieters How do you control the light from? Which Home app is this?
Can you please use the chip-tool and try the commands for setting specific colors? Or, may be, you can mention the colors that you are seeing incorrectly and we will try them using the chip-tool and provide the commands. Just want to understand where is the actual problem.

@dhrishi It's solved here: https://github.com/Kristian8606/esp-matter/tree/main/examples/ws2812_led_strip

Thanks. We will take a look. cc @jadhavrohit924

Hi @AchimPieters, It will be helpful if you can specify which colors do not match the colour selected in the home app.


[Loc] update to sql-database-projects

This PR fixes #

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.02%) to 48.103%

Totals

Change from base Build<PHONE_NUMBER>:
0.02%

Covered Lines:
14886

Relevant Lines:
29369

💛  - Coveralls


Create a UML use-case diagram

Create a use-case diagram.
What functions will be available in your application?
You can think from the system perspective here – not only the backend, but also the frontend part.
What will the user do? What will be typical things that the website user will perform?
Create a UML use-case diagram for it.
Remember the syntax? The consumers of services are represented as "humans" while the use cases are
ovals, see https://www.lucidchart.com/pages/uml-use-case-diagram

UML use-case diagram created, all members contributed


Module keyword always report an error.

The module keyword always says it is expecting a ";" after the keyword

Not sure if you closed this intentionally or not but since one of the betas the TypeScript team changed the keyword to require.  So import mylib = require("somelib"); .

I noticed that after I filed it, so I closed it ASAP. I was using some old
documentation.
On Sat, Jun 13, 2015 at 3:49 PM, Steve Ognibene<EMAIL_ADDRESS>wrote:

Not sure if you closed this intentionally or not but since one of the
betas the TypeScript team changed the keyword to require. So import mylib =
require("somelib"); .
—
Reply to this email directly or view it on GitHub
https://github.com/TypeStrong/atom-typescript/issues/391#issuecomment-111758155
.


Test Different Scenarios

General

[ ] The login feels pretty slow, i do see some screens sometimes, which then are skiped because some informations already exist, e.g login hint, first to second screen
[x] Error messages are not human readable, its always a server error

Bugs

[x] Redirect to app doesn't work after user registration

[x] Reset Password click results in error

[x] When redirecting to the login with an org scope, and entering a user from the wrong organization, I get an error message which doesn't help me anything

[x] When a error occures, I can't do anything afterwards, continue button loads, and back doesn't work

[x] When sending a org scope and clicking on register new user, i loose the context of the org and be on the default org again

[ ] If i have enforced mfa, i can skip it and login without mfa

[ ] MFA setup doesn't show what i have configured on my organization, it always shows passkey

[x] if password hide is enabled, password reset is still shown on the password screen

[x] if "ignore unknown username" is enabled, i still get an error if the username could not be found

[ ] Per default the settings of the default organization should be triggered and not the instance settings

Improvements

[x] After registering a new user with username password I want to be authenticated directly, so i don't have to reenter username password again
[ ] If username password needs to entered again, I want the loginname to be prefilled
[ ] when i redirect to the login, and get the username screen, i have a back button, which doesn't make anything, I think the back button is useless in that case or should redirect back to the app
[ ] Should error message differ when user is locked? I always get password could not be verified
[ ] Default behaviour is not always the same, sometimes i am per default on the username screen and sometimes on the select account screen

Register User with Username Password
Register User with Passkey
Register User with External IPD

"If i have enforced mfa, i can skip it and login without mfa" and
"MFA setup doesn't show what i have configured on my organization, it always shows passkey"
I guess you ment the prompt to setup passkeys. this shows up, when passkeys are enabled in the login settings. I have changed the priority to setup MFA before hinting to setup passkeys now.

"Per default the settings of the default organization should be triggered and not the instance settings"
this can be implemented after: https://github.com/zitadel/zitadel/issues/8616

@peintnermax should we close this issue for now as i did a new test?
#199

i close it for now, let me know if we need to reopen


new approach: init should not be a method on Component

The current design of the Component trait expects users to initialize a ComponentBuilder by calling init() on their Model. This feels weird to me, since reading YourModel::init() looks like it creates an instance of YourModel, not ComponentBuilder. Also, having it be a trait method allows users to override it in their Component impl, which is never something they should do.
Instead, why not define a method ComponentBuilder::new() that constructs a new builder, with the component type specified as a type parameter? This makes it clearer that a component is being constructed, and type inference will usually allow the model type to be inferred anyways. Another benefit of moving the method is that it would free up the valuable name init to be used instead of init_parts.

I agree, that seems to be a more idiomatic way to handle this. Any thoughts from your side @mmstick?

Yeah that's fine

It may also be good if the component trait instead had a ::builder() method which calls this. That's usually the API that people expect with creating a builder.

Fixed by #142

I've been thinking about the builder more, and I think we could go a bit further: why not provide the builder methods on Controller itself? That makes the API a lot more discoverable. You want to create a Controller eventually, so you can start there.
IMO it would be great if we could make the invocation look like this:
val component_params = ...;

let controller: Controller<MyComponent> = Controller::builder(component_params)
  .transient_for(...)
  .connect_receiver(...);

This gives us fewer builder "steps" to expose. Instead of ComponentImpl -> ComponentBuilder -> Connector -> Controller, you have ControllerBuilder, which lets you call transient_for, update_root, etc. and then detach, forward, etc. all start the component internally and consume the builder, giving you a component.

I like your idea. Yet, the reason for allowing to launch later on is to help with ordering.
For example, you want to attach the root of your component to your view but also pass a widget from the view (for example a TabView for a TabBar of the component). With your approach you now have a conflict: You need the view to attach the component but you also need the view to initialize the component.
Maybe we can make it possible to skip the builder part if you have the init params ready from the start.

I see. Sounds like a good first step would be to add an example to the repo demonstrating TabView/TabBar usage so that we can be sure that is supported, and then we can experiment with simplifying the builder.


Issue with Tennis Optimizer

Has the tennis optimizer been implemented? I'm having trouble loading it into my script.

Please check your library version, tennis was added in version 2.5.0

I had to remove the older packages and then reinstall it.
Thanks!


[CSL-2089] shorten InfraConfiguration

This values were always the same in practice and I really don't see why
they should be configurable.
The Kademlia stuff could be configurable, as I don't think mismatches between peers would cause any trouble, but it doesn't need to be delivered by Data.Reflection.

You forgot to remove these values from lib/configuration.yaml.

Will do

I think it makes sense to rename infra/Pos/{Communication,DHT}/Configuration.hs to infra/Pos/{Communication,DHT}/Constants.hs. I think we had a convention that Configuration means values which are passed using reflection mechanism (and can be changed without recompilation), but hardcoded values are called Constants.

Yeah I agree, I'll make the change.

@gromakovsky Changes made.


Flydigi apex 2 mapping

Added mappings for one of my controllers, Flydigi Apex 2. This is compatible with wired/android mode where the most possible buttons are available. Other methods of connecting the controller might need different mappings.
I mostly just gave sensible names to the right stick and the X/Y buttons, which had the V and H inverted. The underbuttons and extra bumpers were given designations of PinkyLeft/Right, PaddleLeft/Right, and ActionL/R.
I was unable to find anything useful to do with the "wheel" input, a sliding block under some of the face buttons that can be used in a similar fashion to an extra stick. I left them as the already designated "Brake" and "Gas" as the output ranges from 0-1 for Left/Right and Up/Down and that really only describes pedals and throttles.

@zeddidragon Thanks for opening this PR!

I was unable to find anything useful to do with the "wheel" input, a sliding block under some of the face buttons that can be used in a similar fashion to an extra stick. I left them as the already designated "Brake" and "Gas" as the output ranges from 0-1 for Left/Right and Up/Down and that really only describes pedals and throttles.

If they can be moved either both up/down and left/right then they should be remapped to a signed input event.  If there are no suitable events in the Event enum, then feel free to append them.  I'm not sure which of the face buttons have it, but if it was on for example, ActionA, the x axis event might be named something like ActionAX(f64), and the y, ActionAY(f64).

If they can be moved either both up/down and left/right then they should be remapped to a signed input event. If there are no suitable events in the Event enum, then feel free to append them. I'm not sure which of the face buttons have it, but if it was on for example, ActionA, the x axis event might be named something like ActionAX(f64), and the y, ActionAY(f64).

Oh I see! It's the difference between a "Pressure" event and an "Axis". I had remapped the x-axis to "Wheel" and still saw unsigned output and chalked it up to third-party controller jank. Remapping to JoyZ and CamZ gives the expected result although a dedicated axis-pair would be nice.

The block houses the B, Y, and X buttons with all the official texts calling it the Wheel. I'll try to edit it in although all the events seem to follow some specific hex codes and I'm not sure how many place I have to touch for it to work.

I added it as "ActionWheel", and I think I found all the necessary places to add it. Incrementing Axs::Count felt iffy, so I found that by stripping the number the value is simply inferred.
haptic.rs example performs as expected now, I'm not sure what else would need to be checked for a correct implementation.

@zeddidragon If haptic.rs works as expected, then I think that's good enough.  This looks great, so I'm merging it.  Thanks again!

@zeddidragon Changes now published to crates.io in stick 0.12.4


Add term of use and trademark links

All changed packages have been documented.

You can try these changes at https://cadlplayground.z22.web.core.windows.net/prs/2949/
Check the website changes at https://tspwebsitepr.z22.web.core.windows.net/prs/2949/


touch option cannot be enabled after plugin init

Hi,
I used the slideout plugin on my project and i really like the smooth slide effect. On My Project, one requirement is, to only enable touch, when the menu is open. For that I wanted to disabled the touch option on init and later enable it with the method slideout.enableTouch();
That does not seem to work. Initializing the plugin with touch: false makes it being disabled forever. Using the method slideout.enableTouch() afterwards does not seem to do anything.
See example in JS fiddle: http://jsfiddle.net/mt7bp04p/
I made a workaround by enabling it on init, then calling the method slideout.disableTouch() right after initializing and that seems to work. Still it seems like a bug to me.
Thanks for your efforts.
Jan

It is a bug. After merging the destroy method (there is already a pull request for that) I'll decouple so the _initTouchEvents will have a counterpart called _destroyTouchEvents and fixing the bug. Thanks for reporting.

ok cool

Same problem here

#114


Validation fails for sandboxed Mach service

I'm trying to communicate with a bundled Mach service that is installed using SMAppService. Both the main .app and the Mach service are sandboxed.
Here's how I configure the XPCServer in the Mach service:
try XPCServer.forMachService(withCriteria: .forDaemon(withClientRequirement: .sameTeamIdentifier))

Unfortunately, this fails. The function validateThisProcessIsAnSMAppServiceDaemon() returns a .failure because the Sandbox prevents it from reading Library/LaunchDaemons/ in the main app's bundle.
Logged messages:
Sandbox: Some Daemon(58726) deny(1) file-read-data /Users/someuser/Library/Developer/Xcode/DerivedData/Some_App-liughliuhliuagsiudfasdfj/Build/Products/Debug/Some App.app/Contents/Library/LaunchDaemons

SecureXPC.XPCError.misconfiguredServer(description: "An SMAppService daemon must have a property list within its parent bundle\'s Contents/Library/LaunchDaemons /\ndirectory.\nParent bundle: file:///Users/someuser/Library/Developer/Xcode/DerivedData/Some_App-liughliuhliuagsiudfasdfj/Build/Products/Debug/Some%20App.app/")

I'd like keep the service sandboxed (XPC worked fine when using Apple's XPCSession).
Failed attempts:
I've tried to find a Sandbox exception entitlement that allows reading a bundle-relative path. There doesn't seem to be one.
Is there perhaps another solution or did I miss anything?

I've realized that simply creating a named Mach service is possible, so I'm doing just that:
try XPCServer.forMachService(withCriteria: .init(machServiceName: "com.somecompany.someapp.someservice", clientRequirement: .sameTeamIdentifier))

Thanks for this great library!


std::bind1st is removed in C++20

Hi,
In super_point.cpp line 280, std::bind1st is removed in C++20 .

How to make it work with C++20?

Thanks,

-Scott

std::bind1st(std::multiplies(), norm_inv)
==>>>
std::bind(std::multiplies(), norm_inv, -1)
Compiled.


Possible bug in labelled contraction

The following

should contract to

This works fine in the unlabelled version but it's invalid in the labelled version. There are no inverses here, so this should just work.

This is also invalid as an oriented contraction.

Hang on that means it has nothing to do with the labelled contraction code. Maybe the problem is that it's trying to annihilate the strands rather than braid them? What if you go into the slice and drag diagonally so it's definitely going to try to do the interchanger?

I thought so too, but even if you go inside the singular slice and drag one of them diagonally, it still doesn't work.

The error is triggered here where we propagate an expansion in a singular level to the whole diagram:
https://github.com/homotopy-io/homotopy-rs/blob/76953762548c131af8df764f18ecef0b7e8316a6/homotopy-core/src/contraction.rs#L267-L275
It's because the height is out of bounds, we probably need to adjust it after contraction.

Fixed by https://github.com/homotopy-io/homotopy-rs/commit/7a0e698bd940fbeefe9a1c1fbe48e83abe156019.


No module named 'builtin'

while I was trying to run on debian 10 the simple example which has been provided in pypi I encountered the following error:
File "/home/sh/spyder/temp.py", line 5, in <module>
  from pyqtconsole.console import PythonConsole
File "/home/sh/anaconda3/lib/python3.7/site-packages/pyqtconsole/console.py", line 10, in  <module>
from .interpreter import PythonInterpreter
File "/home/sh/anaconda3/lib/python3.7/site-packages/pyqtconsole/interpreter.py", line 14, in <module>
from __builtin__ import exit    # py2
ModuleNotFoundError: No module named '__builtin__'

then I search for solution like changing to:  __ builtins __,  __ builtin __, __ builtin __ . __  dict __
but still have the same problem.

The code behind it:
try:
    from builtins import exit       # py3
except ImportError:
    from __builtin__ import exit    # py2

I'm not sure why that wouldn't work. What happens if you run python3 -c "from builtins import exit"? What about python3 -c "import builtins; print(builtins.__file__)"?

first of all I have to mention that I'm using python37. The first command you mentioned works fine. but the second one returns the following error:
Traceback (most recent call last):
File "<string>", line 1, in <module>
AttributeError: module 'builtins' has no attribute '__file__'

by the way, I could easily run it on windows 10.

This is the webpage I was trying to find something useful:
https://www.programcreek.com/python/example/2117/code.InteractiveConsole

If python3 -c "from builtins import exit" works, you should never get to the line trying to import from __builtin__. Are you sure you're using the same Python for both? From your stacktrace above, you seem to be using Anaconda's Python.

yes I'm trying it with spyder. but the two code snippets you sent, I tried them on my ordinary terminal.
Again when I'm trying the code out of spyder (in terminal) I see the following message:
Traceback (most recent call last):
File "./n1.py", line 10, in <module>
console.eval_in_thread()
File "/home/sh/anaconda3/lib/python3.7/site-packages/pyqtconsole/console.py", line 582, in eval_in_thread
self.interpreter.exec_, Qt.ConnectionType.QueuedConnection)

AttributeError: type object 'ConnectionType' has no attribute 'QueuedConnection'

The last traceback appears to be the same issue as reported in #23 that appears for PyQt5<5.11. It has been fixed on master but not yet released.
However, this doesn't really explain the first traceback you showed. Can you reproduce and describe the exact circumstances under which you obtain the initially reported error?

yes I can definitely reproduce that error. As I mentioned earlier I am using Debian 10, and im trying to run the code on Spyder IDE, and I have python3.7. if you need any extra information which I didn't provide ask to me to give more information about my machine and system. my first comment is not the complete error message, so I post it again here:
runfile('/home/sh/spyder/temp03.py', wdir='/home/sh/spyder')
Traceback (most recent call last):
File "<ipython-input-3-d0dc2c800490>", line 1, in <module>
runfile('/home/sh/spyder/temp03.py', wdir='/home/sh/spyder')
File "/home/sh/anaconda3/lib/python3.7/site-packages/spyder_kernels/customize/spydercustomize.py", line 827, in runfile
execfile(filename, namespace)
File "/home/sh/anaconda3/lib/python3.7/site-packages/spyder_kernels/customize/spydercustomize.py", line 110, in execfile
exec(compile(f.read(), filename, 'exec'), namespace)
File "/home/sh/spyder/temp03.py", line 5, in <module>
from pyqtconsole.console import PythonConsole
File "/home/sh/anaconda3/lib/python3.7/site-packages/pyqtconsole/console.py", line 10, in <module>
from .interpreter import PythonInterpreter
File "/home/sh/anaconda3/lib/python3.7/site-packages/pyqtconsole/interpreter.py", line 14, in <module>
from __builtin__ import exit    # py2
ModuleNotFoundError: No module named '__builtin__'

Hey, thanks for the info.
I installed spyder and can reproduce. It seems that spyder deletes exit from the builtins:
>>> import builtins
>>> print('exit' in dir(builtins))
False

Not sure why they are doing this, probably so that it doesn't conflict with their custom exit.
Normally, I would consider this a misfeature of spyder that doesn't warrant a fix here, but it should be easy enough to do by simply defining a custom exit function that just raises SystemExit instead of importing it. I can do the patch/PR later or tomorrow.


invalid link

http://www.themethodology.net/2015/02/android-sdk-manager-setup-for.html?m=1
(latest link on the page) doesn't work

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

ID: e59c38b4-e25c-c5d8-55c5-a37c6513e7ac
Version Independent ID: 8e4c9da9-4358-c70d-fbc5-e7953023bd1c
Content: Which Android SDK packages should I install? - Xamarin
Content Source: docs/android/troubleshooting/questions/install-android-sdk-packages.md
Product: xamarin
Technology: xamarin-android
GitHub Login: @conceptdev
Microsoft Alias: crdun

Thanks for submitting an issue and we apologize for the delayed response. The requested updates haven't been made since the creation of the issue, and the timeline for resolution may vary based on resourcing, so the issue is being moved to an internal repo to better track it.


Move data to consumerdata

Cleanup the root directory to have a more easier to read repository. The main user of data are the consumer interfaces so it is safe to assume that these are consumerdata.

Codecov Report

Merging #115 into master will not change coverage.
The diff coverage is 81.96%.

@@          Coverage Diff           @@
##           master    #115   +/-   ##
======================================
  Coverage    69.8%   69.8%
======================================
  Files         101     101
  Lines        6389    6389
======================================
  Hits         4460    4460
  Misses       1695    1695
  Partials      234     234

Impacted Files
Coverage Δ

exporter/exporterwrapper/exporterwrapper.go
0% <0%> (ø)
:arrow_up:

...ccollector/app/sender/jaeger_thrift_http_sender.go
0% <0%> (ø)
:arrow_up:

receiver/vmmetricsreceiver/vm_metrics_collector.go
13.86% <0%> (ø)
:arrow_up:

...occollector/app/sender/jaeger_proto_grpc_sender.go
0% <0%> (ø)
:arrow_up:

...lector/app/sender/jaeger_thrift_tchannel_sender.go
0% <0%> (ø)
:arrow_up:

configv2/example_factories.go
55.33% <0%> (ø)
:arrow_up:

...anslator/trace/jaeger/jaegerthrift_to_protospan.go
94.47% <100%> (ø)
:arrow_up:

...iver/prometheusreceiver/internal/metricsbuilder.go
100% <100%> (ø)
:arrow_up:

...nal/collector/processor/queued/queued_processor.go
47.61% <100%> (ø)
:arrow_up:

exporter/exporterhelper/tracehelper.go
100% <100%> (ø)
:arrow_up:

... and 21 more

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 797d28b...79e5dc8. Read the comment docs.


Add Compare-DscParameterState to DnsRecordBase

Pull Request (PR) description

This Pull Request (PR) fixes the following issues

Task list

[ ] Added an entry to the change log under the Unreleased section of the
file CHANGELOG.md. Entry should say what was changed and how that
affects users (if applicable), and reference the issue being resolved
(if applicable).
[ ] Resource documentation added/updated in README.md.
[ ] Resource parameter descriptions added/updated in README.md, schema.mof
and comment-based help.
[ ] Comment-based help added/updated.
[ ] Localization strings added/updated in all localization files as appropriate.
[ ] Examples appropriately added/updated.
[ ] Unit tests added/updated. See DSC Community Testing Guidelines.
[ ] Integration tests added/updated (where possible). See DSC Community Testing Guidelines.
[ ] New/changed code adheres to DSC Community Style Guidelines.

This change is

Codecov Report

Merging #2 (191a979) into ClassRecords (92cbb2a) will increase coverage by 0%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           ClassRecords     #2   +/-   ##
===========================================
  Coverage            86%    86%
===========================================
  Files                16     16
  Lines              1106   1108    +2
===========================================
+ Hits                956    958    +2
  Misses              150    150

Impacted Files
Coverage Δ

source/xDnsServer.psm1
98% <0%> (+<1%)
:arrow_up:

@Sudman1 would be great to merge this, if you feel it is right. 🙂

@Sudman1 would be great to merge this, if you feel it is right. 🙂

It felt right. I'll study exactly how you implemented that and incorporate it going forward.


configure a file-target for each different thread

How can i group all the log messages from a specific thread into a single file-target?
Nlog in used inside a windows service where multiple threads are created to handle specific tasks.

With a filter or just use https://github.com/NLog/NLog/wiki/ThreadId-Layout-Renderer in the file name

I assume that your question is answered. If not, feel free to re-open.

Thanks! Is it possible to use a PostFilteringWrapper in combination with a filter on the threadid equals threadid from the exception?
I want to create an email target that will send an email when an exception is logged and it should contain the buffered log messages from all levels from the same threadid


I'd like to publish this to maven

Is that ok with you?
Here are a few questions:

Of resonable quality (you're a better judge of this than I am)
Do you mind if I publish it to maven?
Do you want me to push the changes back in, they'll include:

porting from ant to maven
checking library versions and pulling them from maven as well

And also changing some documentation to be compatible with javadoc's HTML format

Yeah you can do whatever you want with this code! Just Fork it and go ahaead.
I just migrated this lib from googlecode before they shut it down, and I wasn't planning to maintaining it here. I'm OK though if you prefer making PRs and start advancing this repository, but maybe do a search on github first to see if someone already did the things you want to do.
Happy coding!
--R

https://github.com/pwheimann/s2-geometry-library-java

Ah thanks!


Added links to the Metadata SDK reference docs

Fixes https://github.com/kubeflow/website/issues/1206

This change is

/assign @zhenghuiwang
@zhenghuiwang Please would you review this PR, which adds links in the Kubeflow docs pointing to the Metadata ref docs on readthedocs.io

Preview:

https://deploy-preview-1430--competent-brattain-de2d6d.netlify.com/docs/components/misc/metadata/
https://deploy-preview-1430--competent-brattain-de2d6d.netlify.com/docs/reference/metadata/

/lgtm
/approve

Thanks @sarahmaddox !

Thanks @zhenghuiwang !
/approve


TypeError: unsupported operand type(s) for +: 'int' and 'Value'

When I run the following command from micrograd_lecture_second_half_roughly.ipynb:
[(yout - ygt)**2 for ygt, yout in zip(ys, ypred)]
I get a clean output:
[Value(data=1.8688676392069992)), Value(data=0.37661915959598025)), Value(data=0.13611849326555403)), Value(data=1.69235533142263))]
But whenever I tried to enclose it in a sum() function:
sum((yout - ygt)**2 for ygt, yout in zip(ys, ypred))
It throws an error:
TypeError: unsupported operand type(s) for +: 'int' and 'Value'

nvm :)


Evaluate PostCSS-preset-env to see if IE11 support is feasible

From their site - https://preset-env.cssdb.org/ lets you convert modern CSS into something most browsers can understand, determining the polyfills you need based on your targeted browsers or runtime environments.
Evaluate whether or not we can include a recommendation to have downstream consumers use this (or is there a sample application we should provide with configuration samples to do this).  See if IE11 support is attainable using this as a build tool.

Example config/script
const presetEnv = require('postcss-preset-env');
const postcss = require('postcss');
const fs = require('fs');
const cssvariables = require('postcss-css-variables');
const path = require('path');

const css = fs.readFileSync(
  path.resolve(__dirname, './dist/patternfly.css'),
  'utf8'
);

postcss([
  presetEnv({
    stage: 0,
    features: {
      'custom-properties': false
    },
    autoprefixer: { grid: true }
  }),
  cssvariables()
])
  .process(css, {
    from: path.resolve(__dirname, './dist/patternfly.css'),
    to: path.resolve(__dirname, './dist/paternfly.old.css')
  })
  .then(result => {
    fs.writeFileSync(
      path.resolve(__dirname, './dist/paternfly.old.css'),
      result.css
    );
  });

this still has some issues with handling a few variables, but gets it 90%.  It fails when it has something like this
:root {
  --global-color: #fff;
  --global-color-hover: green;
}
.btn {
  --btn-color: var(--global-color);
  --btn-color-hover: var(--global-color-hover);
  color: var(--btn-color);
}
.btn:hover {
  color: var(--btn-color-hover); // this works
}
.btn.hover {
  color: var(--btn-color-hover); // this does not work
}


[Pyamqp GA] Stress Testing Bugs

[ ] Potential Bug: failing to decode empty frames - which would be intermittent. That should be repro-able by polling an otherwise idle connection for a number of minutes. And we should definitely test that - I doubt the empty frame decode logic was ever actually tested.

[ ] we received a SASL frame outside of the SASL negotiation loop, so it showed up in the main frame processing loop and was then ignored. Though this is just a log statement, so not sure how it would have caused issues itself. It also looks like the contents of the frame aren't valid with all those zeros.
- EventHubError('index out of bounds on dimension 1\nindex out of bounds on dimension 1)
- To repro: Catch an IndexError on decode and log the incoming frame

[ ] Sync Websocket -- CPU ConsumerClient is 0, Memory ProducerClient is linearly increasing, logs do not show

[ ] self._shutdown bug with ValueError not allowing for iteration via retries

Label prediction was below confidence level 0.6 for Model:ServiceLabels: 'Azure.Core:0.50518394,Storage:0.23801424,Event Hubs:0.06323718'