[No QA] Upgrade React Native to 0.66

Details
This pull request upgrades the following packages:

react-native: upgraded from 0.64.1 to 0.66.4
react-native-reanimated: upgraded from 2.3.0-alpha.1 to 2.3.1

Upgraded because it fixes https://github.com/software-mansion/react-native-reanimated/issues/2456

metro-react-native-babel-preset: upgraded from ^0.61.1 to ^0.66.2

Did this as it was recommended by the React Native Upgrade Helper

react-native-config: upgraded from 1.4.0 to 1.4.5

Was necessary to get Android to build

react-native-plaid-link-sdk: upgraded from 7.1.0 to 7.2.0

Was necessary to get Android to build

And adds the react-native-clean-project as a dev dependency, along with a useful clean script. More details on all that does here. Note that we're using an alpha version because it fixed this issue for me.
Note: The React Native Upgrade Helper was used to guide some of these changes.
Note: We should follow-up with an issue to upgrade react-native-modal because it's throwing errors due to the deprecated removeEventListener function. This has been fixed in their latest version.
Note: We should also follow-up with an issue to upgrade urbanairship-react-native, because it's throwing warnings like new NativeEventEmitter() was called with a non-null argument without the required removeListeners method
Fixed Issues
Coming from https://github.com/Expensify/App/pull/6898#issuecomment-1002236004 and https://github.com/Expensify/App/issues/6874#issuecomment-1000493831
Tests

(optional) Run npm run clean to clean the project
Install node modules and pods: npm i && cd ios && pod install && cd ..
Run the app
Run various basic regression tests to verify that the app is working as expected.

QA Steps
None, just regression testing.
Tested On

[x] Web
[x] Mobile Web
[x] Desktop
[x] iOS
[x] Android

Screenshots
Web

Mobile Web

Desktop

iOS

Android

This is the correct time to upgrade the RN-web as well. We will fix a couple of issues along with this.

I was thinking we would upgrade RN first, then RN-web in a separate PR. Do you think we should do both in a single PR?

It actually looks like we're pointing at our fork, which is several commits ahead of the upstream react-native-web. So I'm not sure there's anything to upgrade there.

Ok. Sounds great. Even if we had to. It should definitely be a separate PR.

I got Android working, but I'm seeing the following warning:

So we'll probably need to follow-up and update UrbanAirship. Going to want to do this in another separate PR.

I'm getting an infinite spinner trying to open the plaid modal on Android. I wonder if I broke it or if I'm just holding something wrong for testing Plaid on dev.

Yeah, and also refactor out the app to use .remove instead of removeEventListners

Yeah, and also refactor our app to use .remove instead of removeEventListners

I think this PR does that. Some of our dependencies will need to be upgraded for all these warnings to go away.

Ahh, got Plaid working on Android. I needed a second ngrok tunnel for the web-secure API.

Taking this out of draft

Tagging in @Julesssss for.a second review.

Regarding the Android build failure, @Julesssss did you try running npm run clean?

Or at minimum, rm -rf node_modules && npm i (because it looks like the error is coming from a node module which may have been cached from an out-of-date version

I tried npm run clean, but didn't completely clear node_modules. Will give that a try tomorrow.

Or at minimum, rm -rf node_modules && npm i (because it looks like the error is coming from a node module which may have been cached from an out-of-date version

Wiping node_modules resolved the Android terminal issue. I struggled to build iOS from the terminal, but I'm pretty sure it's an issue with my local env as I was able to build from XCode. Merging!

Actually, while I can build to iOS, I have been unable to fully verify iOS.
When building to a physical device I get a connection error, and with simulator, I get an immidiate dyld crash (seems to be related to the RCTTextHeaders library).

@Julesssss since you said "Merging!" I assumed you had just forgotten to merge this 😬

Bad timing on my part...

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

Yeah... my fault I started debugging and hadn't yet send the above message

Have you been able to verify iOS? If so I'm less worried as it seems to be issues on my end only.

Well it's not deployed yet. I had no issues building to a simulator, but I admittedly did not test with a physical device. Let me try that next

If you can log in with a physical device and simulator I think we're fine. I don't want to spend too much longer debugging this but will spend a little longer on this.

@roryabraham okay, I finally got it to work on an iOS simulator. Not exactly sure why, but after retargeting iOS 12 in XCode and a bunch of other things it works. It must have been something on my side.

Thanks for working through it @Julesssss. I'm still going to try and run the app on a physical device

Yeah, everything worked on a physical device too. As stated in Slack, I've had to build from XCode once and then everything starts working normally from the CLI

🚀 Deployed to staging by @roryabraham in version: 1.1.26-2 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to staging by @roryabraham in version: 1.1.27-0 🚀

platform
result

🤖 android 🤖
failure ❌

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
failure ❌

🕸 web 🕸
success ✅

🚀 Deployed to production by @roryabraham in version: 1.1.27-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅


Cannot read property 'length' of null

Challenge http://www.freecodecamp.com/challenges/waypoint-wrap-an-anchor-element-within-a-paragraph has an issue. Please describe how to reproduce it, and include links to screen shots if possible.

you've left some of the html out of the screen shot so it's hard to resolve this issue but I'm 100% confident that this is not a bug but rather that you've left something out
e.g. if you have $('a').length in the challenge tests and you have no a tags the $('a') is null/undefined.
So it saying "Cannot get length of null" is correct

@benmcmahon100 is right in how this operates and that the waypoint should be completable. This is a sort of bug with how the test cases work though; because the error message replaces the objective, it's necessary to reload the page to see what you need to accomplish.

I updated the screen. Is it too small to see?

You needclick here for
 cat photos

Thank you all !


Dictation canceled

On iPhone/iOS (15.1), when entering data in dictation mode, as soon as the cursor wraps to a new line, dictation mode is cancelled.  You then have to restart dictation for every new line.

I dug into this and it doesn't look like there is a way to preserve the keyboard dictation state between different text fields. I'll file a bug report with Apple about this. Hopefully they will add something in a future release.

Hello Maurice,
I thought I would reply to sort of tweak yours and perhaps my understanding of the issue. My concern is not with the dictation remaining active as I go to a new item rather that dictation seems to end as soon as a sentence automatically wraps to the next line.
In the case of an iPhone this might be after only four words or a few more words, in the case of my iPad.
I don’t believe this is an apple iOS related issue as I do not have this problem with any other application including email, text messages, notes, messenger, etc.
Regards,
Rachel.
Sent from my iPhone

On Jan 22, 2022, at 2:51 PM, Maurice Parker @.***> wrote:

I dug into this and it doesn't look like there is a way to preserve the keyboard dictation state between different text fields. I'll file a bug report with Apple about this. Hopefully they will add something in a future release.
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.

Sorry for misunderstanding you. That is definitely a bug in the 1.x series and if you are running the 2.0 beta using iOS 14. If you use iOS 15 and use the new Beta 2.0 version this no longer happens.
Here is a blog post if you want to try out the new Beta 2.0 version. It would be helpful if you could verify for me that this is no longer an issue when using iOS 15 and on Beta 2.0.
https://zavala.vincode.io/2021/12/09/2.0_Beta_1.html

That worked. Thank you.
Rachel.
Sent from my iPhone

On Jan 24, 2022, at 7:58 PM, Maurice Parker @.***> wrote:

Sorry for misunderstanding you. That is definitely a bug in the 1.x series and if you are running the 2.0 beta using iOS 14. If you use iOS 15 and use the new Beta 2.0 version this no longer happens.
Here is a blog post if you want to try out the new Beta 2.0 version. It would be helpful if you could verify for me that this is no longer an issue when using iOS 15 and on Beta 2.0.
https://zavala.vincode.io/2021/12/09/2.0_Beta_1.html
—
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.
You are receiving this because you authored the thread.


Add TSP Function

Created new function for traveling salesman problem

We also absolutely need unittests and this function needs to be added to the documentation

looks good to me!


Rearrange submodules

Move parallel and sequential to lib/control.js
Move firstOf out of lib/control.js

Landed in c6eaf9a and e3775e9


New style CELERY_REDIS_* configuration should take precedence over old st

New style CELERY_REDIS_* configuration should take precedence over old style REDIS_* configuration.

Thanks! This definitely makes sense!


[BUG] TypeError: init() got an unexpected keyword argument 'archivefile'

🐛 Bug

To Reproduce
Steps to reproduce the behavior:

Run the Tutorial on Google Colab
Build an environment and try to print it

(https://github.com/facebookresearch/minihack/tree/main/minihack/tiles)/rendering.py
Executed line: render_des_file(des_file, n_images=1, full_screen=False)
Error message: TypeError: init() got an unexpected keyword argument 'archivefile'
Expected behavior

Normally it should print out a picture of the environment
Environment

MiniHack Version (e.g., 1.0): 0.1.5
NLE Version (e.g. 1.0): 0.9.0
PyTorch Version (e.g., 1.0): 0.23.0
OS (e.g., Linux): Mac OS
Python version: 3.9

Temporary solution:
I redefined the function and deleted the "archivefile=None" from the function, then it worked.

Hi @EmmaHongW. Thanks for pointing this out. Indeed, the archivefile parameter has been removed in NLE==0.9.0 version and needs to be removed here.
Given that you have already come up with the solution, would you want to submit a PR for resolving the issue?

Resolved in this commit.


Auto-refresh UserInfoPopup recent messages

Pull request checklist:

[x] CHANGELOG.md was updated, if applicable

Description
Automatically add new messages to UserInfoPopup dialogs as they are received without having to press the refresh button. Clicking refresh will load the latest messages again from the channel.

Video examples
Example 1
Example 2

Messages in usercards are duplicated, probably because I have the same channel open in multiple splits. Please fix that :slightly_smiling_face:

Looks good to me 👍 Thank you for your contribution!

@fourtf I was doing some testing and found that this->setAttribute(Qt::WA_DeleteOnClose); needs to be added in the constructor otherwise the window won't be deleted if closed by the x button.

@dnsge I'll push a fix in a sec


Module underscore not found in latest Version

./node_modules/react-bootstrap-table-next/lib/src/utils.js
Module not found: Can't resolve 'underscore' in '/Users/''/Documents/Repos/react-software-manager/ClientApp/node_modules/react-bootstrap-table-next/lib/src'

Ever since updating to 1.0.1 I can't compile my project because of 'underscore' not being found.

@LiquidSean  please upgrade to<EMAIL_ADDRESS>again, I think it will fix this issue.


Is this possible to configure ChartJS to work like Google Charts to display both absolute values and percentage of total and they display the value outside the pie so that it is more visible?

I was considering using Google Chart API for a pie chart since they provide an option to display both absolute values and percentage of total and they display the value outside the pie. Is this possible to configure ChartJS to work like Google Charts to display  both absolute values and percentage of total and they display the value outside the pie so that it is more visible?
Does it require development or it is possible to achieve with configuration only?

I guess I could try to use tooltip callback to display percentage https://stackoverflow.com/questions/61953064/laravel-how-to-display-both-count-and-percentage-in-chartjs-pie-chart but how to display the value outside of the pie so it becomes visually distinguishable?

Hi @Patlatus !
Yep you nailed it, you will have to calculate them on your own, ChartJs will not does that for you.
As explained in the link you found, you'll need to use the tooltip label callback.
We are exposing this callback so you can use LWCC to do that


Implement filter config for generate-filter

It should be possible to pass a configuration file to generate-filter which contains:

Excluded collections
Excluded field names
Field names which specify additional fields and are not searchable (making https://github.com/OpenSlides/openslides-search-service/blob/main/cmd/generate-filter/main.go#L64 unnecessary)
Per collection excluded fields

We should discuss if it makes sense to keep maintaining the auto generation of the search.yml as it seems to be more efficient to maintain the config manually.


chore(dev): fix uid/gid mapping for non-vscode devs

I still see these errors when I build a new container with run -r:
 => ERROR [ 4/28] RUN groupmod --gid 20 vscode     && usermod --uid 501 --gid 20 vscode     && chown -R 501:20 /home/vscode                          0.3s
------
 > [ 4/28] RUN groupmod --gid 20 vscode     && usermod --uid 501 --gid 20 vscode     && chown -R 501:20 /home/vscode:
#0 0.224 groupmod: GID '20' already exists
------

and later on
Compiling native node packages...
➤ YN0000: ┌ Resolution step
➤ YN0060: │ @parcel/optimizer-image@npm:2.5.0 provides @parcel/core (p53bfc) with version 2.4.1, which doesn't satisfy what @parcel/workers requests
➤ YN0060: │ @parcel/types@npm:2.5.0 provides @parcel/core (p8f883) with version 2.4.1, which doesn't satisfy what @parcel/fs requests
➤ YN0060: │ @parcel/types@npm:2.5.0 provides @parcel/core (p9e6d4) with version 2.4.1, which doesn't satisfy what @parcel/workers requests
➤ YN0060: │ @parcel/types@npm:2.5.0 provides @parcel/core (p218a2) with version 2.4.1, which doesn't satisfy what @parcel/cache requests
➤ YN0060: │ @parcel/types@npm:2.5.0 provides @parcel/core (p17919) with version 2.4.1, which doesn't satisfy what @parcel/package-manager requests
➤ YN0000: │ Some peer dependencies are incorrectly met; run yarn explain peer-requirements <hash> for details, where <hash> is the six-letter p-prefixed code
➤ YN0000: └ Completed
➤ YN0000: ┌ Fetch step
➤ YN0000: └ Completed in 1s 64ms
➤ YN0000: ┌ Link step
➤ YN0000: │ ESM support for PnP uses the experimental loader API and is therefore experimental
➤ YN0007: │ lmdb@npm:2.2.4 must be built because it never has been before or the last one failed
➤ YN0007: │ msgpackr-extract@npm:1.1.4 must be built because it never has been before or the last one failed
➤ YN0007: │ @parcel/watcher@npm:2.0.5 must be built because it never has been before or the last one failed
➤ YN0007: │ cypress@npm:9.6.1 must be built because it never has been before or the last one failed
➤ YN0000: └ Completed in 23s 94ms
➤ YN0000: Done with warnings in 24s 394ms

and then finally
rm: cannot remove '/usr/local/share/nvm/current': Permission denied

@larseggert I'm looking into the first one (groupmod error).
For the yarn warnings, they can be ignored. Some parcel packages are simply using different versions of the same dependency and yarn is warning us about it. It doesn't prevent the packages from installing and being used.
As for the last one, that error is coming from bash / zsh directly (running zsh directly outputs the same thing). It seems to come from some nvm plugin being loaded with the shell... Looking into it.

The groupmod error should be silenced now (group already exists and mapped) and allow the build to continue.
I wasn't able to figure out where exactly is the nvm command is triggered (happens when calling zsh, so there's a hook somewhere, but it's not in .zshrc...). That error doesn't affect anything though, it's a superfluous command to set the current node version (which is already set).

I think I did. Maybe merge this and see if issues remain?

I think it would be good if changes to the dev container configs triggered a separate kind of CI test if possible - one that would be slow, of course, because it should build the affected container(s) from scratch and exercise them to the point that at least manage.py check succeeds.

The dev container is now leaking permission changes out onto the host, which is really not ok for many reasons, but just to point at one:
root@crawl:~/workspace/datatracker# ls -ld .
drwxr-xr-x 21 1000 1000 4096 May 19 13:40 .
root@crawl:~/workspace/datatracker# git status
fatal: unsafe repository ('/root/workspace/datatracker' is owned by someone else)
To add an exception for this directory, call:

	git config --global --add safe.directory /root/workspace/datatracker

The permissions change leak happens when running as root on the host, which no dev should be in theory?
It happens because of this line:
https://github.com/ietf-tools/datatracker/blob/fix-uidgid-devcontainer/docker/scripts/app-init.sh#L11
which I guess could technically be removed if we assume the host user id and container user id are the same...


FORTECH-87: Item's bottom border has a slight angle at the extremities

Added fix for Item's bottom-border and RadioItem's checked color
Enact-DCO-1.0-Signed-off-by: Adrian Cocoara<EMAIL_ADDRESS>

This is no longer necessary based on my changes (refactoring) of #268.
For future reference, border supports shorthand assignments when setting the rules individually. This was resolved in the other ticket by setting @agate-item-border-width: 0 0 3px 0; and not adding new variables or rules.

I didn't realize there was also apparently RadioItem color assignments in this ticket to fix the Item border.


[Proposal]: Adding Transformations for Loading and Preprocessing Datasets

Currently, Autolabel only supports using the data available in a CSV/JSON file that is explicitly provided by the user. However, we want to provide the ability to preprocess data which may be present in various forms -

Fetching information from web searches or webpages
Reading different document formats (PDFs, Word docs, etc) from file inputs
Reading from cloud / DB storage

We want to provide the user to do this in Autolabel directly using a new construct called Transforms. These will be similar to a 'Map' function on a dataset. The interface for a transform would look like this -
class Transform(ABC):
     def __init__(self, config):
         self.config = config

    @abstractmethod
    def name(self):
         return "transform_name"

    @abstractmethod
    def transform(self, row: Dict[str, Any]) -> Dict[str, Any]:
        """The transform would take in the row with all input columns and output consists of a mapping between column names and their values. Here we just return the output columns to avoid copying data."""

These transformations will be defined in the config by adding a new key called transforms:
config = {
     'dataset': ...,
     'task_name': ...,
     'transforms' : [
           {
               'name': 'tokenize',
               'params': {...},
               'input_column': '{col1}/{col2}',
               'output_columns': ['col3', 'col4'],
         },
        {
               'name': 'padding',
               'params': {...},
               'input_column': '',
              'output_columns': ['col5'],
       }
   ],
  'prompt': ...
}

Every transform can specified by using its name. Any params needed for the transform can be passed into the params which will be passed into the transform while initializing it. The input column can be a string combination of columns sent into the transform. We allow a list of multiple output columns in keeping with the multiple output columns supported in the transform signature.
If a user wants to define their own custom transformation, they just need to implement transform interface with their own custom transformation logic. The way for autolabel to recognize a custom transformation would be through register_transformation, which registers all transformations in the local registry to use with the agent.
Datasets
With this change, we will expose the AutolabelDataset class to the user. Now, a user will initialize their dataset using the config. However, every agent function will now accept AutolabelDataset and return AutolabelDataset.
agent.plan(dataset: AutolabelDataset, ...) -> None
agent.run(dataset: AutolabelDataset, ...) -> AutolabelDataset, ...

In order to run a transformation, the user will need to use agent.transform function. I.e
ds = agent.transform(ds)
df = agent.run(ds)

@yadavsahil197 @rajasbansal are working on this


refactor: tests/integration/310.command.dev.exec.test.cjs

replace ava with vitest
move and rename test file

🎉 Thanks for submitting a pull request! 🎉
Summary
Convert test file to ESM and replace ava with vitest
Related to #5698

For us to review and ship your PR efficiently, please perform the following steps:

[ ] Open a bug/issue before writing your code 🧑💻. This ensures we can discuss the changes and get feedback from everyone that should be involved.
[ ] Read the contribution guidelines 📖. This ensures your code follows our style guide and
passes our tests.
[x] Update or add tests (if any source code was changed or added) 🧪
[ ] Update or add documentation (if features were changed or added) 📝
[ ] Make sure the status checks below are successful ✅

A picture of a cute animal (not mandatory, but encouraged)

@hereje are they ready to review?

@hereje are they ready to review?

Yes, I'll change it to Ready to review.


How can I get the pre-computed proposals for the Fast R-CNN?

How should I generate the pickle file of proposals to feed it into Fast R-CNN?
Thanks!

You can just run the testing script to test an RPN model and obtain the proposals.

Thank you so much!

@hellock Hello, I want to train a Fast RCNN. I have trained a RPN using mmdetection and I have got the .pth file of RPN. What should I do next?
Thanks a lot.


Add OCaml PoS testing app

Discussion
Implements https://www.notion.so/minaprotocol/PoS-Selection-Algorithm-Testing-OCaml-Archive-Node-API-fc6fbbbeed1c4fe6a433fe67ba4176b7 for the OCaml client
:link:  Related to https://github.com/o1-labs/Archive-Node-API/issues/82
Just for context: The purpose of this app is to address this comment in the Archive Node API Repo: https://github.com/o1-labs/Archive-Node-API/pull/80#issuecomment-1535524275
The Archive Node API implements block selection logic to return the best tip from the Archive Node. To make sure the TypeScript implementation corresponds with the OCaml one, we write a simple CLI tool in OCaml that uses the same selection algorithm and processes a list of precomputed blocks. The output of this CLI will be all the JSON files that make up the current best chain after running the selection algorithm. This output will be compared with the TypeScript implementation to check if every block matches.

!ci-build-me

!ci-build-me

!ci-build-me


Training the model

When trying to run the command for training, the code wasn't able to find the files train.csharp_monolingual.pth in the given data path.

When trying to run the command for training, the code wasn't able to find the files train.csharp_monolingual.pth in the given data path.

Had renamed the files generated from preprocessing(as train.csharp_monolingual.0.pth) to required name and that error was resolved.

Getting this error now
Loading miniconda version 22.11.1-1
Traceback (most recent call last):
File "codegen_sources/model/train.py", line 854, in
main(params)
File "codegen_sources/model/train.py", line 688, in main
init_distributed_mode(params)
File "/work/pi_adrozdov_umass_edu/akshatchhabr_umass_edu/TransLRPL/CodeGen/codegen_sources/model/src/slurm.py", line 95, in init_distributed_mode
params.world_size = int(os.environ["SLURM_NTASKS"])
File "/home/vbojanapally_umass_edu/.conda/envs/transCoder_env/lib/python3.7/os.py", line 679, in getitem
raise KeyError(key) from None
KeyError: 'SLURM_NTASKS'

Getting this error now Loading miniconda version 22.11.1-1 Traceback (most recent call last): File "codegen_sources/model/train.py", line 854, in main(params) File "codegen_sources/model/train.py", line 688, in main init_distributed_mode(params) File "/work/pi_adrozdov_umass_edu/akshatchhabr_umass_edu/TransLRPL/CodeGen/codegen_sources/model/src/slurm.py", line 95, in init_distributed_mode params.world_size = int(os.environ["SLURM_NTASKS"]) File "/home/vbojanapally_umass_edu/.conda/envs/transCoder_env/lib/python3.7/os.py", line 679, in getitem raise KeyError(key) from None KeyError: 'SLURM_NTASKS'

Following this(https://stackoverflow.com/questions/66879939/i-got-no-output-using-echo-slurm-ntasks), configured the no.of tasks while launching the batch job


Add documentation for Windows Server 2022

Please update page with requirements for Windows Server 2022.

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

ID: 418dc0a0-0312-9d6f-b2dc-2eb4841285ba
Version Independent ID: 600b0469-3c96-5737-c39b-2ce9021b0f5a
Content: Windows Server 2016 Functional Levels
Content Source: WindowsServerDocs/identity/ad-ds/active-directory-functional-levels.md
Product: windows-server
Technology: ad-ds
GitHub Login: @iainfoulds
Microsoft Alias: daveba

#assign:dknappettmsft

This article has been updated for Windows Server 2022 and the changes should be visible later today.
#please-close


Googling for most questions returns v4.0 docs

Almost everything I search for relating to Bootstrap is returning content from v4.0.
Those c4 pages show v4.1 as the latest but even if v4.5 was listed and I change to it, I lose the page that I'm on and I have to do the search agian.
It is really frustrating.

Almost everything I search for relating to Bootstrap is returning content from v4.0.
Yes, I have noticed this too. I had to bookmark the correct version link.

Those c4 pages show v4.1 as the latest but even if v4.5 was listed and I change to it, I lose the page that I'm on and I have to do the search again.
The version dropdown links are linked to the first page of each version. It should have grabbed the current user page link and then switch between versions.

Known issue, and we have plenty of issues about this. We still haven't found a proper fix.

@XhmikosR
Can you use robots.txt to block Google from indexing those older versions?
Can you update the Algilia search index to include multiple versions or indexes and then prioritize the latest version?
Losing my results when you change versions isn't a huge issue if you're getting the proper version in the first place.
I would de-index all V4 docs except for the latest.
I typically search for Bootstrap 4 _____ or Bootstrap 5 ____
But I come across this issue multiple times a week.

Opened #31980 to track version linking across docs pages.


Axis title for Non-linear SVM/ Feature mapping broken

\lecture_cim2\2020\09-nonlinear-svm\slides-1-featuregen.Rnw slide 3

in slides-nonlinsvm-featuregen.pdf slide 3


Remove fish and zsh items and link to corresponding awesome list.

Remove fish items and link to awesome-fish
Remove zsh items and link to awesome-zsh-plugins

I manually checked whether each of the existing zsh items was or not in the awesome-zsh-plugins and every single one was there. :+1:

Remove the same items from the Chinese version of the README. Did not add anything there.

@alebcay
@sgoings
@xuxiaodong

Fixed Travis :smile:

Nice! Thanks also for fixing Travis. I also just now noticed that I had titled the main section "Awesome Bash", which I will probably get around to changing soon to emphasize the new focus.


feature clone

hey dev team,
can i request feature clone for vm instance, I'll appreciate that
Thanks

I agree

Duplicate of #654


Design: Homepage "Get started" buttons should be three columns

Please make get started buttons 3 columns. For example, if there are 6 buttons then it should be two rows of 3 columns. Currently, it's 4 columns of buttons.
Current:

Desired:

This is updated on cannabis.ca.gov, desktop.

Assigning to @timothyshambra, headless cannabis and drought both need this update.
Source css (in cannabis wordpress site) - width updated to 376px.
If you make any improvements, please ping me & I will add back to our cannabis WP code.
.cagov-grid, .cagov-grid .block-editor-block-list__layout {
  display: grid;
  grid-gap: 1.5rem;
}
/* editor override */
.cagov-grid .block-editor-block-list__block {
  margin: 0;
}
@supports (width: min(250px, 100%)) {
  .cagov-grid, .cagov-grid .block-editor-block-list__layout {
    grid-template-columns: repeat(auto-fit, minmax(min(376px, 100%), 1fr));
  }
}


Crypto/Rsa.VerifyPKCS1v15 - minor bug non security related

Please answer these questions before submitting your issue. Thanks!

What version of Go are you using (go version)?

1.6.1

What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="linux"
GOOS="linux"
GOPATH="/usr/local/google/home/******/work"
GORACE=""
GOROOT="/usr/lib/google-golang"
GOTOOLDIR="/usr/lib/google-golang/pkg/tool/linux_amd64"
GO15VENDOREXPERIMENT="1"
CC="gcc"
GOGCCFLAGS="-fPIC -m64 -pthread -fmessage-length=0 -gno-record-gcc-switches -fdebug-prefix-map=/tmp/go-build413535140=/tmp/go-build"
CXX="g++"
CGO_ENABLED="1"

What did you do?
If possible, provide a recipe for reproducing the error.
A complete runnable program is good.
A link on play.golang.org is best.

The following should result in an error, but do not. I'm assuming crypto.Hash points to md4?
rsa.VerifyPKCS1v15(public_key, crypto.Hash, hashed[:], sig[:])`
rsa.SignPKCS1v15(rand.Reader, priv, crypto.Hash, hashed)`

However it works, causing obvious errors later on when you wonder why a signature from openssl does not validate.
Here's an example of good api calls to the rsa sign & verify functions:
rsa.SignPKCS1v15(rand.Reader, priv, crypto.SHA512, hashed)
rsa.VerifyPKCS1v15(&priv.PublicKey, crypto.SHA512, hashed, sig)

What did you expect to see?
I expected an error to be returned when specfying crypto.Hash as the hash algorithm used in the making of the digest.

What did you see instead?
No error, and it successfully signed and validated using crypto.Hash.  I'm not sure which hash algorithm it validated against though.

I think something is wrong with your Go, because that doesn't even compile.
crypto.Hash is a type not a variable, it would give you an error prog.go:26: type crypto.Hash is not an expression.
https://play.golang.org/p/IG4bnQW60q

I think something is wrong with your Go, because that doesn't even compile.
crypto.Hash is a type not a variable, it would give you an error prog.go:26: type crypto.Hash is not an expression.
https://play.golang.org/p/IG4bnQW60q

Here's a working example of the problem:
https://play.golang.org/p/QAEs9MWgIZ
Not sure if this is working as intenteded behavior, but caused me trouble when debugging.  As I forgot to set the h variable to crypto.SHA512.

This is documented:
https://golang.org/pkg/crypto/rsa/#SignPKCS1v15 says:
"If hash is zero, hashed is signed directly. This isn't advisable except for interoperability."
https://golang.org/pkg/crypto/rsa/#VerifyPKCS1v15 says:
"If hash is zero then hashed is used directly."
This appears to be working as documented.


Update main.py

ok i forgot this one

snake-charmer set project version from 1.5.2 to 1.5.3. If you do not want to release a new version with this PR remove the release label from this PR


Too Many Arguments

Everytime i tryo to type /pex or /pex user, i get the same error saying too many arguments. No matter what i type either through console or in game, if i type /pex at all, the errors gets displayed
05.05 18:33:40 [Server] Server thread/INFO Usage: /pex [debug|reload|import|ranking|[--context|--contexts|-c] [-transient] subject permission-default|delete|options|info|permission|parents|[-transient] subject-type list [filter]]
05.05 18:33:40 [Server] Server thread/INFO Too many arguments!
YoItzSnuk ran command: pex user Yoniez add *
Please Help, I Am Running
PermissionsEx [Jenkins] #147 on Spongeforge-1.10.2-2254-5.2.0-BETA-2272.jar
Can someone tell me a build of Pex and a sponge version for 1.10.2 that works?

Unfortunately PermissionsEx is currently unmaintained and no further updates are planned. Because of this, all pull requests are being closed. If you are looking for an actively maintained permissions plugin, take a look at LuckPerms. It is compatible with major server platforms and can import data from PEX.


Git status hover reversed in codespace

What theme are you using? I have seen this one before but it was theme related?

You are correct it is the theme. It appears Github Light Default has this issue and that is the default codespaces theme. I wonder who we can get to fix that. @daviddossett

@adrianmg do you know who owns the GH VS Code themes and could help out here?

I believe @simurai and @langermank may be able to help on this one!

@lramos15 @daviddossett I think this got resolved by now. I didn't make any changes, so probably was fixed/changed in VS Code.


fix acs kube get-credentials ssh-key loading

When using acs create, the CLI assumes the use of ~/.ssh/id_rsa if not otherwise specified. In acs kubernetes get-credentials, it only tries to load the specified --ssh-key-file or what ever is in the ssh agent at the time. This will fail if ~/.ssh/id_rsa is not added to the agent or not specified on the command line.
This pull request will load ~/.ssh/id_rsa if no keys are found in the ssh agent and no key is specified via --ssh-key-file. It will also provide a better message to the user in the case no keys are found.
Fixes #2403 #3605

This checklist is used to make sure that common guidelines for a pull request are followed.
General Guidelines

[x] The PR has modified HISTORY.rst with an appropriate description of the change (see Modifying change log).

Command Guidelines

[x] Each command and parameter has a meaningful description.
[x] Each new command has a test.

(see Authoring Command Modules)

Codecov Report

Merging #3612 into master will decrease coverage by 0.01%.
The diff coverage is 16.66%.

@@            Coverage Diff             @@
##           master    #3612      +/-   ##
==========================================
- Coverage   71.56%   71.55%   -0.02%
==========================================
  Files         420      420
  Lines       26149    26157       +8
  Branches     3958     3959       +1
==========================================
+ Hits        18714    18716       +2
- Misses       6278     6284       +6
  Partials     1157     1157

Impacted Files
Coverage Δ

...re-cli-acs/azure/cli/command_modules/acs/custom.py
42.91% <0%> (ø)
:arrow_up:

...li-acs/azure/cli/command_modules/acs/acs_client.py
46.72% <17.64%> (-1.53%)
:arrow_down:

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 1562a22...dca259e. Read the comment docs.

@brendanburns to review

+1. Please.

LGTM, thanks.
The test problem looks like a flake.  @derekbekoe is there a way to re-run the tests?

@brendandburns I've restarted CI.
The error might be related to https://github.com/Azure/azure-cli/issues/3617.

@brendandburns I've restarted CI.
The error might be related to https://github.com/Azure/azure-cli/issues/3617.

@derekbekoe looks like it has passed after restarting the CI, but now needs a review.


Screen reader does not read the button name in a popup properly

We have a button in a popup. The screen reader reads the button as table + button name as opposed to button name
Attached video for reference:
https://user-images.githubusercontent.com/63236940/146760156-d4ae6970-123e-40a6-8738-988545084b0b.mov

We are looking into this issue. We will update you.

We are able to repro this. We have raised a bug for the same.

We are able to repro this. We have raised a bug for the same.

Hi Team,
Any update on the same?
Thanks,
Anjali

Currently we do not have any update to share, we will keep you posted if there are any updates.

This is an iframe-based task module and you fully control the HTML displayed inside the page. Please explain why you think this is a Teams issue, as it is not.
It looks like you have your button embedded in a "table" display element, and Chrome is announcing the navigation into the table. Note that Narrator doesn't announce "table" when you navigate backwards to the button (from the "troubleshooting" link) since the user is still in the table.


Regenerate Dart plugin registrant after a hot restart

The Dart plugin registrant is generated once the app is initially compiled.
There's a case where a user may start a flutter run session, then change the pubspec.yaml in the middle, and then hot restart the app.
With the changes from https://github.com/flutter/flutter/pull/74469, these steps will yield into an error.
In the case the plugins were removed from pubpsec.yaml:
Error: Could not resolve the package 'url_launcher_linux' in 'package:url_launcher_linux/url_launcher_linux.dart'.
Error: Could not resolve the package 'url_launcher_macos' in 'package:url_launcher_macos/url_launcher_macos.dart'.
.dart_tool/flutter_build/generated_main.dart:9:8: Error: Not found: 'package:url_launcher_linux/url_launcher_linux.dart'
import 'package:url_launcher_linux/url_launcher_linux.dart';
       ^
.dart_tool/flutter_build/generated_main.dart:10:8: Error: Not found: 'package:url_launcher_macos/url_launcher_macos.dart'
import 'package:url_launcher_macos/url_launcher_macos.dart';
       ^
.dart_tool/flutter_build/generated_main.dart:15:7: Error: Getter not found: 'UrlLauncherPlugin'.
      UrlLauncherPlugin.registerWith();
      ^^^^^^^^^^^^^^^^^
.dart_tool/flutter_build/generated_main.dart:17:7: Error: Getter not found: 'UrlLauncherPlugin'.
      UrlLauncherPlugin.registerWith();
      ^^^^^^^^^^^^^^^^^
Performing hot restart...                                              ⣾Error: Could not resolve the package 'url_launcher_linux' in 'package:url_launcher_linux/url_launcher_linux.dart'.
Error: Could not resolve the package 'url_launcher_macos' in 'package:url_launcher_macos/url_launcher_macos.dart'.
.dart_tool/flutter_build/generated_main.dart:9:8: Error: Not found: 'package:url_launcher_linux/url_launcher_linux.dart'
import 'package:url_launcher_linux/url_launcher_linux.dart';
       ^
.dart_tool/flutter_build/generated_main.dart:10:8: Error: Not found: 'package:url_launcher_macos/url_launcher_macos.dart'
import 'package:url_launcher_macos/url_launcher_macos.dart';
       ^
.dart_tool/flutter_build/generated_main.dart:15:7: Error: Getter not found: 'UrlLauncherPlugin'.
      UrlLauncherPlugin.registerWith();
      ^^^^^^^^^^^^^^^^^
.dart_tool/flutter_build/generated_main.dart:17:7: Error: Getter not found: 'UrlLauncherPlugin'.
      UrlLauncherPlugin.registerWith();

Either hot reload or hot restart could require regeneration - but only if the pubspec.yaml has changed. See the localizations code for some examples of how to wire this up https://github.com/flutter/flutter/blob/master/packages/flutter_tools/lib/src/resident_runner.dart#L912

@blasten The beta branch cut is coming up very soon. What is the status of this?

@cyanglaz offered help. Thanks!  I will be OOO starting next week.
From a very quick look, it would need to call this function from here (Hot restart) if the pubpsec.yaml has changed since last time it was invoked.

I'm confused about the expected result. Just want to confirm:
If I remove some deps (say url_launcher), and do a hot reload/restart, I should see the error in the main description of this issue, correct?
So what is the issue? Is the issue about error not showing?

If I remove some deps (say url_launcher), and do a hot reload/restart, I should see the error in the main description of this issue, correct?

It should be possible to remove a plugin from the pubspec and hot reload or restart without a compilation error.

I just tried with removing url_launcher in the middle of an app session, and run a hot restart, it seems fine, no error is generated.
https://user-images.githubusercontent.com/3756895/110398294-6c5ccb80-8028-11eb-9859-6fdfe206bb5d.mov

If you removed url_launcher from your pubspec yaml in an example app that was using url_launcher you should get a compilation error - did pub get run and remove url_launcher from the package_config.json? You would also need to remove the usage from the app itself.

I removed the url_launcher when the app is running, then hot restarted.
The hot start did not run pub get, which leads to no error.

The tool doesn't run pub for you on hot restart/reload

Fixed via https://github.com/flutter/flutter/pull/79669


fix add checks for existence and successful upload of data files in hive-metastore.sh

…hive-metastore.sh
What problem does this PR solve?
Issue Number: close #xxx
Related PR: #xxx
Problem Summary:
Release note
None
Check List (For Author)

Test

[ ] Regression test
[ ] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[ ] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No code files have been changed.
[ ] Other reason

Behavior changed:

[ ] No.
[ ] Yes.

Does this need documentation?

[ ] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

run external

skip buildall


controller: test: relax condition

since e8432043 we have e2e test coverage, so we can relax this condition and tolerate the case on which we miss daemonsets.

/cherry-pick release-4.14


Code block does not use line breaks

I'm using redcarpet in a rails app. In my markdown I have created code blocks using three backticks. When I display the markdown with redcarpet there are no line breaks in my code blocks. All of the code block is pushed onto one running line. Is there an option to maintain the line breaks in code blocks with redcarpet?
markdown = Redcarpet::Markdown.new(Redcarpet::Render::HTML, prettify: true)
markdown.render(@post.body).html_safe

Were you able to solve this?

For me using 3 back-ticks code generates  block which is supposed to be used for one-liners. For multiline code <code> should be wrapped by <pre>, where newlines /n will be displayed correctly unless they are escaped with additional backslash //n.

Looks like, setting fenced_code_blocks: true solves this issue.
Redcarpet::Markdown.new(Redcarpet::Render::HTML, fenced_code_blocks: true)


wget doesn't exist on all systems

I use macOS Catalina 10.15.7, and wget isn't a command. As far as I know, curl is universal/more common, so maybe migrate to curl?
Error:
dillonbarnes at Downstairs-iMac in ~/Sauce on Master
± make setup
mkdir -p Compiler/src/
wget --continue "https://ftp.gnu.org/gnu/binutils/binutils-2.37.tar.xz" -O Compiler/src/binutils-2.37.tar.xz
/bin/sh: wget: command not found
make: *** [Compiler/src/binutils-2.37.tar.xz] Error 127

Or, you could add a note in README.md saying for users on macOS, they need to install wget via homebrew: brew install wget

oh uhmmm ok thanks. i'm not running mac so i couldn't know that sorry.
I'll see about converting that into a curl command instead.

switched to curl.
looking into testing clang to see if that compiles my project. if it does then this fix was pointless.


Azure Service Bus Component: Metadata

https://github.com/dapr/components-contrib/blob/801297c724e9d55d55b3288b9b123c000a231df6/bindings/azure/servicebusqueues/servicebusqueues.go#L258
If the Service Bus integration is meant to be interoperable with message senders that aren't built with DAPR and possibly including such that talk to Service Bus directly using a generic AMQP client, which ought to be a goal, it needs to surface all potentially relevant metadata to the consumer, which includes all application properties and surely all of the fixed AMQP message properties. There's cherry-picking of three built-in properties here, notably excluding the "content-type" that's key to understand what "Body" is and the application properties are completely dropped.

This is indeed what we are also facing. Our downstream service publishes messages with ApplicationProperties but we have no way to get these using the service bus queue dapr binding (that I am aware of). So I think this issue is stil relevant and not stale.

Yea, how do you deal with OpenTelemetry if you've binned ApplicationProperties? I call shenanigans.
Please open up access to System or ApplicationProperties, if not through headers, invent a payload that you can opt to receive from the sidecar that contains all the things you need to properly handle tracing and other application properties. It's bad enough you skated over Baggage that could have been used to forward important dimensions for the telemetry that could help enormously with observabililty across the estate. Binning Application Properties as well is a last straw.


Methods with lazy initial param don’t end up in ops.

@typeclass trait Merge[F[_]] {
  def merge[A, B](fa: => F[A], fb: => F[B]): Option[F[(A, B)]]
}

doesn’t have a merge method in the ops class. But if I remove the => before F[A], it works as I’d expect.

The implicit def that's generated requires that F[A] is already evaluated, as well as the Ops itself since self isn't put as lazy on there.
Just out of curiosity what do you expect to be evaluated when you call fa merge fb if a fix is applied?

I would expect fa to be evaluated – I currently manually define the Ops class in this case, so that it looks like
def merge[B](fb: => F[B]): Option[F[(A, B)]] = typeclassInstance.merge(self, fb)

so, self must already be evaluated to a value of type F[A].


pull in owncloudsql fixes from reva

pull in https://github.com/cs3org/reva/pull/1990

{"level":"error","service":"storage","error":"listen tcp <IP_ADDRESS>:9216: bind: address already in use","time":"2021-08-13T12:18:51.270158554Z","message":"error getting sockets"} 🤔


Nx no longer uses workspace.json files and does not build

Which area this feature is related to?
/kind bug

Which area this bug is related to?

/area registry-viewer
/area landing-page
What versions of software are you using?
Node.js project

Operating System and version: N/A
Node.js version: 18
Yarn version: 1.22.19
Project.json: @nx/node
Bug Summary
Describe the bug:

Nx build no longer builds any projects due to the newer version of Nx using different project.json files to define the multiple projects. Without these files, all projects are ignored during yarn nx affected --target=build.
To Reproduce:

Run yarn nx affected --target=build.
Expected behavior

yarn nx affected --target=build should trigger building the Nx projects.
Any logs, error output, screenshots etc? Provide the devfile that sees this bug, if applicable

Output of yarn nx affected --target=build --parallel=3:
yarn run v1.22.19
$ /home/runner/work/devfile-web/devfile-web/node_modules/.bin/nx affected --target=build --parallel=3

 >  NX   workspace.json is ignored

   Nx no longer reads configuration from workspace.json.
   Run "nx g @nrwl/workspace:fix-configuration" to split workspace.json into individual project.json files.

 >  NX   No explicit --base argument provided, but found environment variable NX_BASE so using its value as the affected base: a0bc065e4c9b3ae9b93a5fa2b0e5102abdbc3243

 >  NX   No explicit --head argument provided, but found environment variable NX_HEAD so using its value as the affected head: f980d517bfb02f8d27455c0e8b9c88bdab6d4c90

 >  NX   No projects with target build for 0 projects were run

 >  NX   Successfully ran target build for 0 projects

   View logs and run details at https://cloud.nx.app/runs/gqnh2u7gpj

Done in 0.93s.

Additional context

Any workaround?

Revert version of Nx and dependencies to a previous state to restore usage of workspace.json.
Suggestion on how to fix the bug

Investigate usage of project.json and convert current workspace.json file(s) into multiple project.json files.

Explains why checks are now passing: https://github.com/devfile/api/issues/1464#issuecomment-2033079380
See #1464 for details.

I believe we are already using project.json and the workspace.json is just left there, the fix to this possibly is to just remove workspace.json?
Core: https://github.com/devfile/devfile-web/blob/main/libs/core/project.json
Docs: https://github.com/devfile/devfile-web/blob/main/libs/docs/project.json
Landing Page E2E: https://github.com/devfile/devfile-web/blob/main/apps/landing-page-e2e/project.json
Landing Page: https://github.com/devfile/devfile-web/blob/main/apps/landing-page/project.json
Registry-Viewer E2E: https://github.com/devfile/devfile-web/blob/main/apps/registry-viewer-e2e/project.json
Registry-Viewer: https://github.com/devfile/devfile-web/blob/main/apps/registry-viewer/project.json
fyi @michael-valdron

I believe we are already using project.json and the workspace.json is just left there, the fix to this possibly is to just remove workspace.json?
Core: https://github.com/devfile/devfile-web/blob/main/libs/core/project.json Docs: https://github.com/devfile/devfile-web/blob/main/libs/docs/project.json Landing Page E2E: https://github.com/devfile/devfile-web/blob/main/apps/landing-page-e2e/project.json Landing Page: https://github.com/devfile/devfile-web/blob/main/apps/landing-page/project.json Registry-Viewer E2E: https://github.com/devfile/devfile-web/blob/main/apps/registry-viewer-e2e/project.json Registry-Viewer: https://github.com/devfile/devfile-web/blob/main/apps/registry-viewer/project.json
fyi @michael-valdron

@Jdubrick This might need further investigation, though the project.json files are there the project builds are still being ignored. This is possibly related to the workspace.json that is still present that pointed to each project, it might be a possible file change or command change that would fix this process to detect the inner projects.

I believe we are already using project.json and the workspace.json is just left there, the fix to this possibly is to just remove workspace.json?
Core: https://github.com/devfile/devfile-web/blob/main/libs/core/project.json Docs: https://github.com/devfile/devfile-web/blob/main/libs/docs/project.json Landing Page E2E: https://github.com/devfile/devfile-web/blob/main/apps/landing-page-e2e/project.json Landing Page: https://github.com/devfile/devfile-web/blob/main/apps/landing-page/project.json Registry-Viewer E2E: https://github.com/devfile/devfile-web/blob/main/apps/registry-viewer-e2e/project.json Registry-Viewer: https://github.com/devfile/devfile-web/blob/main/apps/registry-viewer/project.json
fyi @michael-valdron

@Jdubrick This might need further investigation, though the project.json files are there the project builds are still being ignored. This is possibly related to the workspace.json that is still present that pointed to each project, it might be a possible file change or command change that would fix this process to detect the inner projects.

This may be related to the nx cache also, its possible the env variable for skipping it isn't working so it skips the tests, but I agree investigation should be done.

I know that the command in the issue will output a project build if something in the project was changed, else, it will have 0 builds

I know that the command in the issue will output a project build if something in the project was changed, else, it will have 0 builds

Yeah you correct, this would lower the priority and scope of this report as its just warnings.

Nx build no longer builds any projects due to the newer version of Nx using different project.json files to define the multiple projects. Without these files, all projects are ignored during yarn nx affected --target=build.

Revising to:

Nx build produces warnings due to the newer version of Nx using different project.json files to define the multiple projects.

yarn nx affected --target=build should trigger building the Nx projects.

Revising to:

yarn nx affected --target=build should not produce warnings about workspace.json.

Revert version of Nx and dependencies to a previous state to restore usage of workspace.json.

Revising to:

N/A


Add command to import adb public key

I've rebased #4 on the current master, simple to merge now.

Nice one @mcfedr! Thanks!

@Malinskiy do you intend to take a new release?

@mcfedr released 0.3.0 :)


Add jump-if-not-zero instruction.

Fixes #305

Thanks! Looks good to me. I'll let @Voxelot make the final approval.


Provide better error messages if the source library is missing.

This should help developers building this functionality out that may not
realize they need to source the linting library separately.
I ran into this after having it fail silently before I realized I needed the external library.

Thanks, merged as 03d99e5


Make OAuth authorization/authentication optional (let's switch it all off!)

Implement a switch, that disables auth completely reverting it to state equivalent to that before auth implementation. May be useful for other potential users of TESK installing it and testing installation and/or unwilling to integrate with Elixir and for us for test purposes and/or in case Elixir dies.
Things to consider:

switch off authN/Z - possibly by spring profile
make saving user/group data and constructing user/group-aware label selectors optional (possibly move it to some component, too)

Yes, I think this is the functionality we expect. Verified yesterday in the DEMO deployment.


fix: fix property name error

The pinchDeltaPrecision in the comments should be pinchPrecision. There is no pinchDeltaPrecision property.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Snapshot stored with reference name:
refs/pull/14854/merge
Test environment:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14854/merge/index.html
To test a playground add it to the URL, for example:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14854/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/14854/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/14854/merge
https://gui.babylonjs.com/?snapshot=refs/pull/14854/merge
https://nme.babylonjs.com/?snapshot=refs/pull/14854/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/14854/merge#BCU1XR#0

WebGL2 visualization test reporter:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14854/merge/testResults/webgl2playwright/index.html

Visualization tests for WebGPU (Experimental)
Important - these might fail sporadically. This is an optional test.
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14854/merge/testResults/webgpuplaywright/index.html


runtime/syntax/gitconfig.vim: do not change iskeyword

Ref: https://github.com/vim/vim/issues/2810#issuecomment-382127299
/cc @tpope (assuming that you prefer this over git-send-email)

Digits should be allowed.

Thanks, amended.

Codecov Report

Merging #2814 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #2814      +/-   ##
==========================================
+ Coverage   75.42%   75.43%   +<.01%
==========================================
  Files          92       92
  Lines      134583   134583
==========================================
+ Hits       101513   101517       +4
+ Misses      33070    33066       -4

Impacted Files
Coverage Δ

src/if_xcmdsrv.c
84.35% <0%> (-0.36%)
:arrow_down:

src/ex_cmds.c
79.46% <0%> (-0.23%)
:arrow_down:

src/os_unix.c
54.96% <0%> (-0.14%)
:arrow_down:

src/gui_gtk_x11.c
47.68% <0%> (-0.1%)
:arrow_down:

src/undo.c
82.51% <0%> (-0.07%)
:arrow_down:

src/gui.c
47.78% <0%> (-0.06%)
:arrow_down:

src/channel.c
83.13% <0%> (+0.08%)
:arrow_up:

src/ex_cmds2.c
78.66% <0%> (+0.09%)
:arrow_up:

src/term.c
60.57% <0%> (+0.15%)
:arrow_up:

src/window.c
81.99% <0%> (+0.16%)
:arrow_up:

... and 3 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 e87edf3...6e19040. Read the comment docs.

Alternatively, instead of using :setlocal iskeyword you can use :syntax iskeyword. That has been added specifically for changing the 'iskeyword' setting only for the syntax files without affecting other things in Vim. That means you would not have to change your syntax regular expressions. See :h :syn-iskeyword

I think this has been merged by the upstream @tpope
So I am closing this.
@tpope  you might want to send an updated version to Bram for distribution with Vim 8.1

Unfortunately Vim 8.1 already dropped before your comment. I'll aim to avoid waiting till the last minute with Vim 8.2.

Unfortunately Vim 8.1 already dropped before your comment. I'll aim to avoid waiting till the last minute with Vim 8.2.

@tpope You can always send updates to Bram whenever you think they should be included. Bram updates runtime files every once in a while and not just before a release.


[Feature Request] Undo

It would be really nice to be able to Undo the last Action taken:

in the tree
crafting an item
selecting an item

you can partially do it using ctrl+z

i feel stupid now, im pretty sure i tried that and have it not work :O
on another note ctrl+shift+z for redo would be nice, since z and y arent near each other on every kb-layout
can be closed i guess

CTRL+Y already does REDO.
CTRL+Z and CTRL+Y are the combinations for those commands since Windows95 and being known for that. So changing those shortcuts will cause more trouble than anything else and Z&Y are apart on all keyboardlayouts I know.

i didn't say you should replace ctrl-y...
also that's the thing not everyone is a Windows-Person
kinda unrelated but windows isn't exactly known for its great ux :-*


leaflet MarkerCluster plug in documentation

Hi,
could you kindly provide me the intuitively description of the marker cluster algorithm? does it rely on a well-known algorithm? I am trying to describe it in my thesis work. Thank you

It uses naive greedy clustering.
Whenever a marker, started at the bottom zoom level and going up we look for an existing marker or cluster that is within the cluster distance.
If one is found the marker is added to that cluster (or forms a new cluster with that marker) and inherits the parent clusters of the already existing clusters above.
Probably reading the _addLayer code is the right place to look, it is called to add every marker to the ClusterGroup.
https://github.com/Leaflet/Leaflet.markercluster/blob/master/src/MarkerClusterGroup.js#L770
Hope that helps!


Timer start bypass cheat

Hello :)
As of writing, the game as it hosted here has a bug where shift+tabbing at the start of every level will not start the timer.
This happens on Firefox 132.0b9 and also Chrome.
I suppose one way of fixing this from a js perspective could be to add an event listener for the user pressing tab or shift+tab.
If there's any other issue details I could provide lmk.

I’m okay with that :) The point of the game is to teach about keyboard focus. If you know enough that you can cheat your way through the game, that's great! Plus, I don't think there's any efficient way to detect this. (Listening for keyboard shortcuts can differ for every browser as it's customizable. And listening for a previous element focus would still count as cheating anyway I guess?) Thanks anyway for your interest and reporting this.


How to i make the script go into learn automatically?

hello! the script is working fine so far but i cant seem to figure out how i can do the learn lessons automatically. it only goes to stories. please reach back very soon.

If you want to do lessons and not stories:
The "-l" flag. I.E.
py auto-lingo.py -l
Warning learn wasn't working when I used it on Japanese
If you want it to do stories elaborate.


Unable to invoke focus on touchable ref

Description
The focus() function throws an error (No command found with name "focus") when invoked on a touchable ref.
Tested on "Apple TV" simulator.

React Native version:
System:
    OS: macOS 11.2
    CPU: (8) x64 Intel(R) Core(TM) i5-8257U CPU @ 1.40GHz
    Memory: 498.71 MB / 16.00 GB
    Shell: 5.8 - /bin/zsh
  Binaries:
    Node: 12.19.0 - /usr/local/bin/node
    Yarn: 1.22.10 - /usr/local/bin/yarn
    npm: 7.0.3 - /usr/local/bin/npm
    Watchman: 4.9.0 - /usr/local/bin/watchman
  Managers:
    CocoaPods: 1.10.0 - /usr/local/bin/pod
  SDKs:
    iOS SDK:
      Platforms: iOS 14.4, DriverKit 20.2, macOS 11.1, tvOS 14.3, watchOS 7.2
    Android SDK: Not Found
  IDEs:
    Android Studio: Not Found
    Xcode: 12.4/12D4e - /usr/bin/xcodebuild
  Languages:
    Java: Not Found
    Python: 2.7.16 - /usr/bin/python
  npmPackages:
    @react-native-community/cli: Not Found
    react: 16.13.1 => 16.13.1
    react-native: Not Found
    react-native-macos: Not Found
    react-native-tvos:  0.63.4-0
  npmGlobalPackages:
    *react-native*: Not Found

Steps To Reproduce
Use the reproducible demo below

Press Button 1

Expected Results
Button 2is focused
Snack, code example, screenshot, or link to a repository:
https://github.com/ivodaveiga/react-native-tvos-focus-test

The Touchable components do not have a focus() method -- that is a TextInput method.  If you want to programmatically change focus, you should set the hasTVPreferredFocus prop on the component you want to focus.

Thanks. Shouldn't that be reflected in typescript as it identifies the focus() function as a valid function inherited from the interface NativeMethods?
(I've updated the example repository to use typescript)

@ivodaveiga I do see that the focus() method is there in the types -- maybe that is a bug in in @types/react-native.  Checking the official docs at https://reactnative.dev, it does not appear that the method exists in any core components except TextInput.


proper way to handle import and typings with lodash/fp

Hi,
in an es2015 node / typescript environment:

import * as _ from 'lodash/fp'

Says it can't find the module.
And the require version looses all the typings.
So what's the proper way to use lodash/fp with proper import and typings ?

This is not the lodash repo, nor is this a bug.

Sorry, please delete this.

@Brakkar You can head on over to the Lodash gitter chat for help.


skip_1_if_same_as_2 refactors?

Originally did not bother because premature abstraction is worse than duplication, but thinking about #2 and the possibility of the "possible duplicate errno names" pair list growing, I'm thinking we probably want to take the duplicate | skip_1_if_same_as_2 E1 E2 lines and move them into a function called handle_possible_duplicates.
Also, the name skip_1_if_same_as_2 is... decent, all things considered, but I keep feeling bothered by the fact that it does not actually skip anything - it adds logic to conditionally skip if needed. The name add_checks_to_skip_1_if_same_as_2 comes to mind, but feels possibly too long. My benchmark for that is: assuming a 79 character max line width, and eight spaces for indentation, can we fit the function name and both errno names on the same line? Before I added z/OS, the longest errno name was 15 characters, and that was already pushing it with a function name that long.
One key thing about the name though is that I find the numeric place holders in the name extremely valuable: they tell explicitly communicate which name will be thrown out, and this is very important because it actually matters: for example, on systems where EWOULDBLOCK is the same as EAGAIN, it would be wrong and misleading to return EWOULDBLOCK as the errno name.

Relevant thought: whenever I want to start writing errno name deduplication pairs, I am always tempted to write EAGAIN EWOULDBLOCK, never EWOULDBLOCK EAGAIN.
And this does not feel like an alphabetical ordering - this feels like a [primary thing that is kept] [secondary thing that is dropped if needed] ordering.
So maybe if I just found a word or phrase that would hint at this natural ordering reliably enough, that might suggest a much better name for this function.

Also instead of just moving all the skip_1_if_same_as_2 lines into one handle_possible_duplicates line, could move the wrap_in_ifdef_blocks with them too, and name the function add_preprocessor_{checks,conditionals,something}.

There was a brief moment where I was hoping I could reason my way into thinking "we don't need any duplicate protection at all" and then none of this would matter because I could just do the ultimate code cleanup of deletion.
But then I remembered that at least three pairs of errno names are actually confirmed duplicates in real popular systems like Linux today. So that's out the window.

Also maybe add_duplication_guards is perhaps better than handle_possible_duplicates.
My initial thought against was that it kinda leaks implementation choice (how specifically to handle the duplicates) into the name. But these names are in a script which explicitly generates C code, so that's probably a helpful/informative leak, an appropriate level of transparency and relevant self-description rather than breaking a justified abstraction.

Everything I had in mind with this issue has been done over the course of commits d9973a2e7d6e664d93c7c356897f7388557c089c, f1caf3621da9fbf4f63638a97a2abf0ed26c6c4d, and a770493f41639355baafe451e23ad745568576f1.
Still not sure if I love handle_possible_alias as the name for the function that adds the #if guard against aliased errno, but this seems better.


Client Allocation Request for: wujia

Client Allocation Questions
Core Information

Name: wujia
Website / Social Media: http://www.hzfdzs.com/
Region: Greater China Region
DataCap Requested: 50TiB
Addresses to be Notarized: f17subptxqumxfdt27r64amsvkehh5bwxtd2aethi
Notary Requested: fireflyHZ

@sjxqqq Please subscribe to notifications for this Issue to be aware of updates. Notaries may request additional information on the Issue.

Hi @fireflyHZ, we have used up the datacap you allocated last time #2093. Hope we can get your support this time too. Many thanks!

@sjxqqq I am very happy to support you, But, I cheked your datacap allocated list

That you allocated all datacap to single SP. Can you explain it ？

Yes, because the last time the quota was only 2T, so I was test-oriented. This time I plan to choose at least 4 SPs. And select SPs in different regions. Hope to get your support

I will support you this time. And you should select at lest 4 SPs to store your data.

please check: https://filfox.info/zh/message/bafy2bzacebrejuzutdo5k6dwdubgnyqxw2lyrp4q72e3dzmy3tjhwi6fchoi6


Update dependencies for eclipse.

Codecov Report

Merging #17 into master will decrease coverage by 0.46%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master      #17      +/-   ##
==========================================
- Coverage   26.22%   25.76%   -0.47%
==========================================
  Files          16       16
  Lines         732      749      +17
  Branches       92       96       +4
==========================================
+ Hits          192      193       +1
- Misses        527      545      +18
+ Partials       13       11       -2

Impacted Files
Coverage Δ

.../main/java/org/oscm/app/shell/business/Script.java
51.78% <0%> (-5.91%)
:arrow_down:

.../main/java/org/oscm/app/shell/ShellController.java
40.1% <0%> (-1.57%)
:arrow_down:

src/main/java/org/oscm/app/shell/Initializer.java
0% <0%> (ø)
:arrow_up:

...ava/org/oscm/app/shell/business/api/ShellPool.java
0% <0%> (ø)
:arrow_up:

...ava/org/oscm/app/shell/ui/customtab/ShellBean.java
0% <0%> (ø)
:arrow_up:

...in/java/org/oscm/app/shell/business/api/Shell.java
0% <0%> (ø)
:arrow_up:


Add version specification in setup.py; Making Clipper_admin python 2-3 compatible.

This PR can contribute to 0.3.0 release of clipper such that it will
be py2 and py3 compatible. Issue #138 can be closed once this
PR is merged.
The fix is simple: just add version specification for subprocess32.
The file setup.cfg is added for easier wheel generation. If we don't
have this file, you have to manually add --universal flag to generate
the wheel such that the wheel is py2.py3 compatible.
Modify version in setup.py; add setup.cfg
This can be last step towards python2 and python3 compatibility.
The issue with running clipper_admin on python3 is not the code
itself, but the installation process. This change fixes the
pip install process.

Can one of the admins verify this patch?

jenkins ok to test

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/769/
Test FAILed.

@dcrankshaw Do you think this error has anything to do with the change in setup.py?
======================================================================
ERROR: test_link_not_registered_model_to_app_fails (__main__.ClipperManagerTestCaseShort)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "../integration-tests/clipper_admin_tests.py", line 49, in setUp
    cleanup=True, start_clipper=True)
  File "/clipper/integration-tests/test_utils.py", line 74, in create_docker_connection
    cl.start_clipper()
  File "/clipper/clipper_admin/clipper_admin/clipper_admin.py", line 101, in start_clipper
    cache_size)
  File "/clipper/clipper_admin/clipper_admin/docker/docker_container_manager.py", line 125, in start_clipper
    **self.extra_container_kwargs)
  File "/opt/conda/lib/python2.7/site-packages/docker/models/containers.py", line 714, in run
    container.start()
  File "/opt/conda/lib/python2.7/site-packages/docker/models/containers.py", line 334, in start
    return self.client.api.start(self.id, **kwargs)
  File "/opt/conda/lib/python2.7/site-packages/docker/utils/decorators.py", line 19, in wrapped
    return f(self, resource_id, *args, **kwargs)
  File "/opt/conda/lib/python2.7/site-packages/docker/api/container.py", line 1081, in start
    self._raise_for_status(res)
  File "/opt/conda/lib/python2.7/site-packages/docker/api/client.py", line 222, in _raise_for_status
    raise create_api_error_from_http_exception(e)
  File "/opt/conda/lib/python2.7/site-packages/docker/errors.py", line 31, in create_api_error_from_http_exception
    raise cls(e, response=response, explanation=explanation)
APIError: 500 Server Error: Internal Server Error ("driver failed programming external connectivity on endpoint mgmt_frontend-3344 (18cf116564341868a7e0a7dc955bb40e79b032948a9d4a5587fe5aa150302023): Error starting userland proxy: listen tcp <IP_ADDRESS>:37600: bind: address already in use")

======================================================================
ERROR: test_stop_models (__main__.ClipperManagerTestCaseShort)
----------------------------------------------------------------------
Traceback (most recent call last):
  File "../integration-tests/clipper_admin_tests.py", line 49, in setUp
    cleanup=True, start_clipper=True)
  File "/clipper/integration-tests/test_utils.py", line 74, in create_docker_connection
    cl.start_clipper()
  File "/clipper/clipper_admin/clipper_admin/clipper_admin.py", line 101, in start_clipper
    cache_size)
  File "/clipper/clipper_admin/clipper_admin/docker/docker_container_manager.py", line 125, in start_clipper
    **self.extra_container_kwargs)
  File "/opt/conda/lib/python2.7/site-packages/docker/models/containers.py", line 714, in run
    container.start()
  File "/opt/conda/lib/python2.7/site-packages/docker/models/containers.py", line 334, in start
    return self.client.api.start(self.id, **kwargs)
  File "/opt/conda/lib/python2.7/site-packages/docker/utils/decorators.py", line 19, in wrapped
    return f(self, resource_id, *args, **kwargs)
  File "/opt/conda/lib/python2.7/site-packages/docker/api/container.py", line 1081, in start
    self._raise_for_status(res)
  File "/opt/conda/lib/python2.7/site-packages/docker/api/client.py", line 222, in _raise_for_status
    raise create_api_error_from_http_exception(e)
  File "/opt/conda/lib/python2.7/site-packages/docker/errors.py", line 31, in create_api_error_from_http_exception
    raise cls(e, response=response, explanation=explanation)
APIError: 500 Server Error: Internal Server Error ("driver failed programming external connectivity on endpoint mgmt_frontend-47101 (df5143a5afce2d6deb82bffaf6579526038c5d90024400e2eadcd95ce6bbfa21): Error starting userland proxy: listen tcp <IP_ADDRESS>:37042: bind: address already in use")

I don't think so.

jenkins test this please

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/775/
Test FAILed.

This is great! I was worried this would be more complicated. However, we should make sure that we run the integration tests under both Python 2 and Python 3 now. There are a few options for the best way to do this given our testing infrastructure, so consider this a TODO item on my end to look into.

I just realized we have some more work to do here. We need to also implement Python 3 model containers for the model deployers (which means a Python 3 RPC implementation). And then the model deployers need to detect whether they are in Python 2 or 3 and use the corresponding base image. None of this should be overly difficult, but it's slightly involved.

jenkins test this please

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Clipper-PRB/777/
Test FAILed.


Skipped image optimization when input image is smaller byte size than optimized one

closes #10144

When the input image is well optimized and has a smaller byte size than the processed one it's still being used
Bumped sharp version to have access to size property

@kirrg001 would be great if you could run a quick test on OSX/Windows for this branch. Only checked on Linux

@kirrg001 would be great if you could run a quick test on OSX/Windows for this branch. Only checked on Linux

Will do later :)

@kirrg001 would be great if you could run a quick test on OSX/Windows for this branch. Only checked on Linux

Looks like it's working 👍
I checked out your branch, ran yarn, sharp was reinstalled/rebuild/downloaded and then i uploaded an image.

Hm i switched back to master, ran yarn and got for upload API test
    success cases
dyld: lazy symbol binding failed: Symbol not found: __ZN4vips6VImage7pngsaveEPcPNS_7VOptionE
  Referenced from: /Users/kirrg001/dev/ghost/ghost-workspace2/node_modules/sharp/build/Release/sharp.node
  Expected in: /Users/kirrg001/dev/ghost/ghost-workspace2/node_modules/sharp/build/Release/../../vendor/lib/libvips-cpp.42.dylib

dyld: Symbol not found: __ZN4vips6VImage7pngsaveEPcPNS_7VOptionE
  Referenced from: /Users/kirrg001/dev/ghost/ghost-workspace2/node_modules/sharp/build/Release/sharp.node
  Expected in: /Users/kirrg001/dev/ghost/ghost-workspace2/node_modules/sharp/build/Release/../../vendor/lib/libvips-cpp.42.dyli

Maybe a local problem, but wanted to share.

Skipped image optimization when input image is smaller byte size than optimized one

Commit message probably needs an emoji.
e.g. 🐛  Fixed image optimisation for input image being smaller than optimized one

The version bump does seem to install properly even when having old version of libvips cached by npm cache. It just takes a bit more time to download new binaries. As for changes in the library itself, minor version changes didn't affect us. Merging! :tada:


fix typo in vitest docs

The docs would really like some love.

Thanks! Yes I know 😅 that’s on the list and we want to restructure them a little.


[CSApply] ExprRewritter has to make sure that tuple elements are of…

… expected type
Problem exposed by SE-0230 - Flatten nested optionals resulting from 'try?'
Example:
protocol P : class {}

class F<T> {
  func wait() throws -> T { fatalError() }
}

func bar(_ a: F<P>, _ b: F<P>) throws {
  _ = (try? a.wait()) === (try? b.wait())
}

Here result of try? {a, b}.wait() used to be inferred to P because
of OptionalObject constraint connecting type of {a, b}.wait() with
try?. Since OptionalObject has been replaced with Conversion
constraint to avoid introducing new level of optionality it creates
a side-effect of inferring type of try? to be AnyObject? which
comes from parameter-to-argument conversion.
Resolves: rdar://problem/46742002

@swift-ci please test

@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - ea22115dc8c2cf0935aa61b1efc93a8e37a6e16b

Build failed
Swift Test Linux Platform
Git Sha - ea22115dc8c2cf0935aa61b1efc93a8e37a6e16b


Disable unsupported chained comparisons, related test case fixes

Closes #810

Verified this locally. We should get Travis CI set up for backends other than Impala (which is difficult to boot up on Travis)


v0.42.x with iavl fast node

What is the purpose of the change
This PR properly updates osmosis v0.42.9-osmo-v3.
It avoids the previous race condition by using pebble instead of goleveldb.
Since we need to look at changes like this as coming from the beginning of the chain, it may make sense to make pebble the default from genesis, and then advance through the 13 versions of Osmosis.
If a small amount of additional tinkering shows that the race condition doesn't resolve then I think it makes sense to disable the additional test.
--- FAIL: TestCLIQueryConn (10.53s)
Conclusion
Because the upstream cosmos-sdk ran tests conditionally, and the needed files didn't change, the harsh, bare reality is that every sdk 42.x fails this race condtion:
git clone https://github.com/cosmos/cosmos-sdk --branch release/v0.42.x
cd cosmos-sdk
make test-all

will replicate this race condition.  Osmosis has corrected the issues with CI, so the issues are visible on the osmosis-labs/cosmos-sdk repository, but are not visible in the cosmos-sdk repository.
Further evidence of this can be seen here:

https://github.com/cosmos/cosmos-sdk/issues/10446
https://github.com/cosmos/cosmos-sdk/pull/13186
https://github.com/cosmos/cosmos-sdk/pull/13185
https://github.com/cosmos/cosmos-sdk/pull/14268
https://github.com/cosmos/gaia/issues/1449
https://github.com/cosmos/ibc-go/pull/1418

what to do?
I think that in the readme file of sdk v0.42.x in both osmosis and upstream, we should note this issue and proceed.
Brief Changelog

use the modern standard for linting and formatting
use iavl v0.19.4
tests have been added, mainly error checks
bump tm-db to the tip of its master branch as found at tendermint/tm-db

Testing and Verifying
This PR must be manually verified by syncing osmosis as an archive node from genesis to tip.
Documentation and Release Note

Does this pull request introduce a new feature or user-facing behavior changes? (yes / no)
Is a relevant changelog entry added to the Unreleased section in CHANGELOG.md? (yes / no)
How is the feature or change documented? (not applicable   /   specification (x/<module>/spec/)  /  Osmosis docs repo   /   not documented)

Today I spoke with @ValarDragon about this PR, which is part of this issue:

https://github.com/osmosis-labs/osmosis/issues/695

as well as this issue:

https://github.com/osmosis-labs/osmosis/issues/3518

and relates to this issue in the cosmos-sdk:
https://github.com/cosmos/cosmos-sdk/issues/14426
As well as helping to address these issues in iavl:

https://github.com/cosmos/iavl/issues/647
https://github.com/cosmos/iavl/issues/637

He and I figured that we should:

disable the test with the race condtion
make a pr to upstream sdk with the deprecation warning readme, as well as the pebble and fast node support

Codecov Report

Merging #405 (ef063fd) into v0.42.9-osmo-v3 (21486da) will decrease coverage by 0.27%.
The diff coverage is 67.38%.

:exclamation: Current head ef063fd differs from pull request most recent head e832824. Consider uploading reports for the commit e832824 to get more accurate results

@@                 Coverage Diff                 @@
##           v0.42.9-osmo-v3     #405      +/-   ##
===================================================
- Coverage            63.06%   62.80%   -0.27%
===================================================
  Files                  623      624       +1
  Lines                41528    41514      -14
===================================================
- Hits                 26189    26071     -118
- Misses               12934    13049     +115
+ Partials              2405     2394      -11

Impacted Files
Coverage Δ

baseapp/baseapp.go
75.75% <ø> (ø)

baseapp/msg_service_router.go
90.90% <ø> (ø)

client/cmd.go
61.29% <ø> (+2.37%)
:arrow_up:

client/context.go
62.67% <0.00%> (ø)

client/flags/flags.go
20.75% <0.00%> (ø)

client/keys/migrate.go
21.33% <0.00%> (-37.34%)
:arrow_down:

client/tx/legacy.go
0.00% <ø> (ø)

codec/amino_codec.go
100.00% <ø> (ø)

codec/proto_codec.go
77.77% <ø> (ø)

codec/types/any.go
67.74% <ø> (ø)

... and 208 more


CONTRIBUTING.md typo correction

The CONTRIBUTING.md file currently has a typo on line 62.
📝 Provide a description of requested docs changes
CONTRIBUTING.md
Line 62, Col 141
I would be willing to contribute to fix this typo.
What is the purpose and what should be changed?
To correct a typo in this file.
In line 62 the document states "that the issue is our to solve" however, to fix this typo, a "s" should be appended onto the end of "our" to make it "that the issue is ours to solve".
Even more in depth explanation can be found below...
On line 62, what currently states * **If you intend to implement the fix/feature yourself then say so!** If you do not indicate otherwise we will assume that the issue is our to solve, or may label the issue as Help-Wanted.
Could be corrected to * **If you intend to implement the fix/feature yourself then say so!** If you do not indicate otherwise we will assume that the issue is ours to solve, or may label the issue as Help-Wanted.

Is "ours" to "our" the only change?

Yes, maybe too small to be worth while?

@bentdoug small PRs don't matter. You can give us a PR

Oh, my apologies as I was unaware of that. So I should skip creating an issue and simply send a PR?
Thanks so much for the clarification.

Oh, my apologies as I was unaware of that. So I should skip creating an issue and simply send a PR?
Thanks so much for the clarification.

No. You should have an issue for a PR. But it doesn't matter if its small or big

Okay got it thanks. I'll send the PR now.


How to edit already existing Repos?

Is there a way to edit already existing repositories?
If so what's the command for that?

In the process of doing it, it will take some time. but the idea is very clear

https://docs.deepwisdom.ai/main/en/guide/in_depth_guides/incremental_development.html


Replace icon with solidus_icon

In solidus 2.3 iconhelper has been deprecated to solidus_icon
Ref https://github.com/solidusio/solidus/commit/62a37ccd358e416d8d6a4a5165a31593e184ac34
PR: https://github.com/nebulab/rubocop-solidus/pull/17
solidus minimum version: 2.3

icon is too much generic to deprecate. We agree that we can remove it for now and re-evaluate if it is worth it to reintroduce it, when we'll have a real project to migrate.

Moved to ready to be removed. Then we can move it to backlog

This COP could be related to #37


Add Ripper for nsfwalbum.com

Category
This change is exactly one of the following (please change [ ] to [x]) to indicate which:

[ ] a bug fix (Fix #...)
[x] a new Ripper
[ ] a refactoring
[ ] a style change/fix
[ ] a new feature

Description
I've gone ahead and added a ripper for nsfwalbum.com.
They host their images on many different image hosting sites so you must replace part of the image urls to get the full image instead of the thumbnails.
There may be more hosts that I did not find, they can be added in later very easily.
Testing
Required verification:

[ ] I've verified that there are no regressions in mvn test (there are no new failures or errors).
[x] I've verified that this change works as intended.

[x] Downloads all relevant content.
[x] Downloads content from multiple pages (as necessary or appropriate).
[x] Saves content at reasonable file names (e.g. page titles or content IDs) to help easily browse downloaded content.

[x] I've verified that this change did not break existing functionality (especially in the Ripper I modified).

Optional but recommended:

[ ] I've added a unit test to cover my change.

NOTES:
mvn test failed, i think it may because of how my maven is setup, it doesnt reference any files of ripme in the stacktrace, i can post the error if that is helpful.
I don't have any experience on how to do unit testing, I am eager to learn if someone wants to help and give me a quick run down.

It says the checks were not successful, this is my first pull request, can someone help me out here? From what I can gather the test failures aren't related to my contribution.


Add user-facing documentation

Added ABOUT.md and LEARNING.md.

Looks good! but, I feel kinda confused, sometimes you use ML when I think you mean SML.

Hm, maybe it's possible to merge this one since adds real value?

I believe that some changes were requested but the original author never followed up. I'll update it and try to get this in sometime today to help out with https://github.com/exercism/meta/issues/85


Doesn't recognize when stream goes live, but force recording works fine

I am having an issue where it doesn't automatically record when the stream goes live, but if I go and click force record then it finds the stream and records fine. Looking in the logs, there aren't any errors around the time the stream began. In fact there is nothing until I manually start the recording
2022-05-09.log

seems like your subscriptions don't apply, check your external app url

Do I need to be subscribed to the channel for it to auto record, or am I misunderstanding what you mean by subscriptions? Also dumb question, but what exactly do I need for the external app url? I was having trouble finding information on what it needs to be.

Not the payment related subscription, no.

This, under the settings. The help text should explain it. You can validate it with the button below the save button.

It needs to be the publically accessible URL of your webserver, using a domain name with https (let's encrypt works)


Dijkstra's Algorithm

Description 📜
This is a brief documentation on Dijkstra's Algorithm. I've tried to explain the algorithm in simple language through pseudo code and also the actual cpp code.  #6785

Type of change 📝

[x] Doc#umentation (Content Creation in the form of codes or tutorials)

Domain of Contribution 📊

[x] C/#CPP

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] My changes generates no new warnings.
[x] I'm GWOC'21 contributor

Screenshots / Gif (Optional) 📸

@kashishahuja2002  I've updated the file as suggested , pls check .

@kashishahuja2002 Done!

Then, kindly merge it.

Then, kindly merge it.

For that you need to tag the supervisor of your batch, mentors don't have that access.

@Suvraneel pls merge this PR.

I haven't added any reference links as this code is from my personal notes and my personal understanding.

@prishit55  Awesomeeeee ! ❤ :octocat:   LGTM 💯  🚀


[site]: UIP update to 2.1.0-beta.2 (UIP 2 April) + ESL Toggleable examples migration

Note: PR does not contain anything except site content updates and site UIP dependency update
According to the agreement with @exadel-inc/esl-core-maintainers merging without explicit review to start live testing.
PR still available to review postfactum

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:

:tada: This PR is included in version 5.0.0-beta.12 :tada:
The release is available on:

npm package (@beta dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


table cache issues inside trim_cube etc.

Just noting here that attempting batch use of some of the cube routines I was running into table cache issues. The workaround in trim_cube is here but I want to dig into this more before committing:
I added the 'rm -rf' , deg_im = , and deg_im.done()
# Figure out the extent of the image inside the cube

myia = au.createCasaTool(casaStuff.iatool)
myia.open(outfile + '.temp')
os.system('rm -rf '+outfile + '.temp_deg')
deg_im = myia.adddegaxes(outfile=outfile + '.temp_deg', stokes='I', overwrite=True)
deg_im.done()
myia.close()

Noting that I hit this issue (though it took me awhile to identify where it was coming from). The fix here worked for me.


how to change the code templates

when i created a new file by nodeclipse wizard, the new file followed the template like this
/**

http://usejsdoc.org/
*/

i wanted to change that , but i didnt know where it is from preferences

They are bundles with Nodeclipse main plugin, sources are at https://github.com/Nodeclipse/nodeclipse-1/tree/master/org.nodeclipse.ui/templates


nowait option to wait for async task result

With 0.3.5, yield gen.Task(taskxxx.apply_async) is blocked forever without result.
    @gen.coroutine
    def get(self):
        # this will be blocked forever
        yield gen.Task(hello.task_hello.apply_async, args=['world'])
        ...
        ...

It's because nowait flag is alway True for channel. So, I added option to it with 'TCELERY_RESULT_NOWAIT' as celery config.
Thanks,
Sangwon Lee

Merged, thanks! Please add your name to the list of contributors.


Remove EventSignups Table

Further to issue #919 and PR #1208 we believe we can and should remove the EventSignups table.
There is no longer the concept of a direct event volunteer signup. All signups occur against tasks. Therefore, maintaining this table is extra work and risks becoming out of sync with the true signups. By removing this table we can instead enumerate the signups where required by reviewing signups for all associated tasks.
This was kept as a separate block of work since it directly affects areas such as notifications and a few other key areas of the application that refer to this table.

@tonysurma Please assign to me. I have some stashed changes that get me started on the path to removing this.

Made some good progress during lunch earlier. Have mostly ripped this out and functionally most things are working well.
I have some tests to fix-up and one oddity with notifications that may/may not be related to this work that I'll take a look into also.


Modified addRoute procedure

It is impossible to set priority through the addRoutes method because it does not pass the route priority property and the $priority parameter will always be null

Can you add a unit test when you fix a use case?
@DASPRiD should send a PR to solve this problem with unit tests.

This is fixed in my latest pull request already.


Add a spec for String#-@ re-use when called on an already frozen string

Since 2.6 2.8, if you try to dedup a frozen string, and that string no yet exist in the fstring table, MRI will directly intern that object and return self.
Very often when you are deduplicating strings, you are trying to be efficient in both memory retention and memory allocation.
So I frequently rely on this behavior.
While implementing a jruby extension (in msgpack-ruby and json) I noticed it would always duplicate the provided string, even when it was eligible for interning.
cc @eregon maybe I can pick your interest on this?

Thanks!

I just realized, this is basically impossible to implement on TruffleRuby.
So better always check for that behavior and not rely on a specific Ruby version having it.
The reason is interned strings use the Java class ImmutableRubyString and can be shared between multiple Ruby contexts in the same process.
OTOH, non-interned strings use the Java class RubyString and cannot be shared between multiple Ruby contexts in the same process (since they can be mutable, and as a fun fact str.freeze must mutate the string as it must return self).
In terms of cost it's fine as it just means an extra ImmutableRubyString allocation, but the String contents itself is held in a Rope, which is immutable and is either reused or its byte[] is reused.
-str.freeze still works fine though, just the .freeze useless on TruffleRuby.
Not sure what's better between adding a comment about this on this spec, or removing it and ensuring there is a corresponding test in CRuby. It seems too much of an implementation detail in practice to be expected behavior of Ruby implementations.

Not sure what's better between adding a comment about this on this spec, or removing it and ensuring there is a corresponding test in CRuby.

Up to you, but it would be best if either a spec or a test was kept to make sure MRI keep that behaviour, has it can make a fairly big difference when deserializing large datasets.


Error when adding to ballot from single project page

When clicking Add To Ballot when clicked in to a single project the following error occurs.  Doesn't happen when adding from projects grid or horizontal cards.

Working on this

This has been fixed

Fixed


permission issues found  gmod server

So ive created everything, i've uploaded everything and whenever i try to start the gmod server, under the garrysmod username, it keeps saying perrmission denied. User does not have access to filse () then it shows every file in garrysmod.  This happened after i started server once and stopped it and a tutorial said to put sv.db and data folder to 755

Protip if you did crap with permissions :
chown -R * username:username
Google about chown if needed.

Thank you so much, I'm sorry to waste your time, about this.

No problem about it. If i had no time at all i wouldn't answer. :p
You'd rather be sorry to use github instead of the steam group for this kind of issue ! :p

wait, what is this supposed to do ?
chown -R * username:username
and what am i supposed to do ?

man chown

chown - change file owner and group
change owner and group recursively e.g
chown -R * username:group

chown -R /home/csgoserver csgoserver:csgoserver

For the username:group do I put the main core username and the garrysmod group like root:garrysmod?
or do i leave it as it is, and same with the csgoserver part, do i just put gmodserver on both parts?Sorry, I'm kinda new to this

Any user has a group crated with it. In your case, you cant files to belong to your user and group. Certainly not to root.
Does it answer your question ?

This all happened after a tutorial said to put sv.db and and data folder to 755.

Then sv.db should have been the only file affected.
By the way, sv.db is not something you'd like to share or take from elsewere, it contains a lot (if not every in some cases) of data from your personal gmod server. So hopefully you just put a backup or your old sv.db or you could get rekt.

Yeah, I know about the data, in it
here's the link to the tutorial that said to change the file permissions to 755

http://steamcommunity.com/sharedfiles/filedetails/?id=555696186

When a tutorial advises a VPS for a game server, run away. lol
"If your server is going to be for around 42-64 people then I recommend going with:
2 high spec intel xeon V-Core (3.2GHz +)"
Lol, 2 CPU for a monothreaded process.
"you may notice that nothing saves (if you have already launched the server).

However, if your FTP program is correctly set, changing permissions won't affect ownership.
Didn't read a lot from this tutorial, but it seems like it contains a lot of bullshit. I wouldn't learn from that.
For linux system admin, search better tutorials. For Instructions about LGSM, they are given on gameservermanagers.com's game page, and we got a pretty well ellaborated wiki here to learn the basics and go further : https://github.com/dgibbs64/linuxgsm/wiki
For gmod specific stuff, such as mounting css content to your server, then you can use LGSM too, just make another user, i usually do an "srcds" user with a subfolder by game content i wanna mount. Then you just edit mount.cfg accordingly. Pretty straight forward.
To conclude, there is not any point where you gotta change chmod or chown if you do things properly.

Just login as root and apply the chown command we told several times. Permissions issue will disappear.

Anyways, in order to be a good admin, i'd suggest to get some Linux knowlege, try stuff, fuckup and reinstall a test machine, before getting into something more serious.

Im still a bit confused on  the chown commands. Do i just copy paste?  the new username i created for the gmod server is called garrysmod, so do i just replace
chown -R /home/gmodserver gmodserver:gmodserver
with  chown -R /home/garrysmod  garrysmod:garrysmod?

That's it. ;)

This is one of the very basics of Linux, that's cool you learned it, but you should already know that if you plan making servers. As i said, really consider experimenting and learning a bit more before starting something serious. You don't need months, you just need to try a lot of stuff quite a few times.
I guess we can close the issue. Please, use the Steam Group if you got more questions, I and others will be glad to help as long as you're kind, clear, and provide enough information for us to help you.
http://steamcommunity.com/groups/linuxgsm/discussions/0/
And always remember that google is your best friend !
Have a great Linux time ;)