Fabula Ultima: Small Changes to the sheet.

Submission Checklist
Required

[x] The pull request title clearly contains the name of the sheet I am editing.
[x] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[x] The pull request makes changes to files in only one sub-folder.
[x] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: < Fabula Ultima >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: < Rooster Games > (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: < Fabula Ultima > (i.e. Dungeons & Dragons, FATE)

[x] I have followed the Character Sheets Standards when building this sheet.

[ ] This sheet has been made on behalf of, or by, the game's publisher.

[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >

[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description
Feel free to join the following game and check/test the new sheet by yourself (creating a character is enabled as player):
https://app.roll20.net/join/16031048/cwvlkA

[x] Added repeating field for Zero Power basic description and effects. Trigger and Clock are fixed as per FU rules.
[x] Added simulated optgroup to separate Bond pairs as described by FU rules, to make it easier for people to know which is the opposing for each Bond emotion.

Character Sheet Info Roll20 Internal Use only.


JMS for Service Bus standard AMQP session timeout

hi Team,
We have a issue with JMS AMQP session timeout. And it make the system cannot get the message for a while.
the error message is :
Aug 01, 2020 11:25:50 AM org.springframework.jms.connection.SingleConnectionFactory onException
WARNING: Encountered a JMSException - resetting the underlying JMS Connection
org.apache.qpid.amqp_1_0.jms.MessageConsumerException: Force detach the link because the session is remotely ended.
at org.apache.qpid.amqp_1_0.jms.impl.MessageConsumerImpl$1.run(MessageConsumerImpl.java:143)
at java.lang.Thread.run(Thread.java:745)
Aug 01, 2020 11:25:50 AM org.springframework.jms.listener.DefaultMessageListenerContainer handleListenerSetupFailure
WARNING: Setup of JMS message listener invoker failed for destination 'org.apache.qpid.amqp_1_0.jms.impl.QueueImpl@6c292189' - trying to recover. Cause: Force detach the link because the session is remotely ended
We checked the document from https://docs.microsoft.com/en-us/azure/developer/java/spring-framework/configure-spring-boot-starter-java-app-with-azure-service-bus
it only have
spring.jms.servicebus.idle-timeout | Specify the idle timeout in milliseconds. The recommended value for this tutorial is 1800000.
In addition, for this document https://docs.microsoft.com/en-us/azure/service-bus-messaging/service-bus-java-how-to-use-jms-api-amqp also don't have the session error information.
I checked online, there have one person said:
The JMS Session instance can not be used concurrently by multiple threads, so the best advise is to create the JMS Session on each thread you are using and close the jms session when you have finished that unit of work (sending a message or a collection of messages in a transaction)
Is that correct? for this issue how we can do anything to prevent it.
Thank you

We've deprecated this repo and migrated to azure-sdk-for-java repo, so I created this issue there. Closing this one.

Hi Xiaolu,
Thank you so much for this email.
I’m Scarlett from Azure PaaS Dev team. Actually this was a customer’s issue, I’m behalf of him to create this issue in your Guithub.
I tried to find the document for JMS session timeout issue, but without luck. Could you please send me any relative documents then I can explain to my customer.
We are appreciated for your reply.
Best regards,
Scarlett Liu
Support Engineer
Azure- PaaS- Developer POD
Office: +86 (21) 52639029
Working Hours:  M-F 9:00am – 6:00pm UTC+8
Need help outside of my working hours?
Locate an engineer /azurebu<EMAIL_ADDRESS>Manager: Anita Yang / anitay<EMAIL_ADDRESS>[Image result for microsoft logo]
From: Xiaolu Dai<EMAIL_ADDRESS>Sent: Friday, August 14, 2020 2:43 PM
To: microsoft/azure-spring-boot<EMAIL_ADDRESS>Cc: Scarlett Liu<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [microsoft/azure-spring-boot] JMS for Service Bus standard AMQP session timeout (#930)
We've deprecated this repo and migrated to azure-sdk-for-java https://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-sdk-for-java&data=02|01|yuchen.liu%40microsoft.com|2a81b15f358649ade4aa08d8401d486d|72f988bf86f141af91ab2d7cd011db47|1|0|637329841804888871&sdata=%2FH9yThiLn36BwL9YoEQu%2FX%2Bei200wxAYAnoBb%2By8Eo4%3D&reserved=0 repo, so I created this issue therehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FAzure%2Fazure-sdk-for-java%2Fissues%2F14100&data=02|01|yuchen.liu%40microsoft.com|2a81b15f358649ade4aa08d8401d486d|72f988bf86f141af91ab2d7cd011db47|1|0|637329841804888871&sdata=pY5ERZxpxp3T9w%2FbMXuTJcwbYY0WymgFi7A2SBeazwA%3D&reserved=0. Closing this one.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fazure-spring-boot%2Fissues%2F930%23issuecomment-673916044&data=02|01|yuchen.liu%40microsoft.com|2a81b15f358649ade4aa08d8401d486d|72f988bf86f141af91ab2d7cd011db47|1|0|637329841804898863&sdata=kQAFq%2B%2B5eTzzvAOWKcTekSZR%2B%2BONc1nc%2B4IY3Wkc%2Fb0%3D&reserved=0, or unsubscribehttps://nam06.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAQTFUOJUEUO7OJGF5EEWGFDSATMHDANCNFSM4P56XBNA&data=02|01|yuchen.liu%40microsoft.com|2a81b15f358649ade4aa08d8401d486d|72f988bf86f141af91ab2d7cd011db47|1|0|637329841804898863&sdata=W%2BNO1ZRxipB5Q5J73WxMzHbakJiUZXfwhzmt0%2BRYB9g%3D&reserved=0.


Zhytkovd

I finished "project" and almost finished questions. I will submit the rest of the questions until Wednesday.

You have two versions of the hw03_questions.md file included in the PR. Please remove the one that is NOT inside the Answers folder.


Updated text regarding Candidate Gathering

Updated text regarding Candidate Gathering

Learn Build status updates of commit a70b500:
:bulb: Validation status: suggestions

File
Status
Preview URL
Details

Teams/direct-routing-protocols-media.md
:bulb:Suggestion

Details

Teams/direct-routing-protocols-media.md

Line 21, Column 1: [Suggestion: duplicate-h1s - See documentation] H1 'Overview' is duplicated with other articles: 'dial-plans-routing-overview.md(26,1), direct-routing-protocols-media.md(21,1)'. First level headings must be unique within a docset.

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


underpromotion moves for black and whiter should be different

https://github.com/sotetsuk/pgx/blob/main/pgx/_src/chess_utils.py#L14
I feel that here should also include a case for the white player. Now there is only a condition for the black player
if from_ % 8 == 6:
    to = from_ + jnp.int8([+1, +9, -7])[dir_]

That means we should add an additional case
if from_ % 8 == 1:
    to = from_ + jnp.int8([-1, -9, +7])[dir_]

Thank you for your comments! Could you provide an example of wrong move case?

We really appreciate if you could kindly give an bug example like this:
https://github.com/sotetsuk/pgx/blob/main/tests/test_chess.py#L659

Note that, the board flips every turn. So I suppose our chess program appropriately deal with the black pawn moves.


extras/courses: Add Introduction to Computational Thinking by MIT

As noted here #912 adding this course to The Math section

Thanks!


[BUG] Sorry Cypress doesn't show E2E test run videos

Describe the bug
Videos of E2E runs are not available to view
/instance/60b5e400-7c30-400a-a233-a478d7dcb860/test/r11
To Reproduce

Create a PR / Trigger a e2e run
Click on a spec
Click on "Recorded Video" button under Tests
e.g. /instance/6bb3b834-05ef-4b93-880f-87baa5d2517e/others/RECORDED_VIDEO

Result
Videos are not uploaded and available
Expected Result
Test run videos should be available to view
Additional context
Could be a permission issue?

CICD logs show

Dashboard github actions runs shows this error even after successful e2e test run
https://github.com/rancher/dashboard/actions/runs/4994198048/jobs/8944416957
Failed Uploading (1/1) /home/runner/work/dashboard/dashboard/cypress/videos/setup/rancher-setup.spec.ts.mp4

Closed as duplicated https://github.com/rancher/dashboard/issues/8904


Error about varying image sizes in fully supervised model training

I am on the main branch and using the dev pip install.
Running the training hydra script as usual and get the following error:

From what I understood, this error should not occur when using a fully supervised model (without pca or temporal losses).
This is related to the training of the Topvieemouse5k dataset, which has varying sizes: https://github.com/danbider/lightning-pose/issues/158
Btw, this error does not occur on the dynamic_crop branch.

I am a bit confused, as I was able to run this on the main brain a few days ago without getting this error. The only thing that changed was that I pulled the recent changes to the repo. So the error must have been quite recently, but I might be mistaken.
I switched to the dynamic_crop branch since it looks like this dependency on image size was merged there: https://github.com/danbider/lightning-pose/pull/117

Oh yes, my apologies - ok I recommend switching to the unsuper-multiview branch as this (1) has the ability to train on multiple image sizes; and (2) does not run the validation data through the augmentation pipeline.
This branch is now PR #187 which will be merged into main in the next day or so!


Design the settings screen.

Please follow this thread for the desription of the issue.

@ganeshpatro321 Can I take this issue?

Yes, follow the discussion thread and go with it. @Jayashree-panda


adding EPD Resources

Authored-by: csells<EMAIL_ADDRESS>

Trying #8794 to work out why build is blocked on that PR, guessing it is because the PR is in a clone of the repo?


line chart

This is great! Any ideas on how to implement a line chart?

Hey!
Have a look at the SomeCustomChart example - you can take a similar approach to bring a line chart in, such as this simple implementation from Mike Bostock.

I am going to assume that this answered your question - let me know if there is anything else I can help you with :)


General Issue: changes needed due to testthat3e: derived_merge, user_helpers, user_utils

Background Information
Once we introduce testthat3e (see https://github.com/pharmaverse/admiral/pull/1538), messages are thrown during testing which did not happen before. We need changes made to some tests as testthat3e is introduced so that the message will not be thrown. There were also small changes made due to deprecated functions.
The tests needing changing (test-derive_param_wbc_abs.R, test-derive_merged.R, test-user_helpers.R, test-user_utils.R) and the introduction of testthat3e are all done in this single issue.
From testthat documentation:

Definition of Done
We will be able to use testthat3e without having unimportant messages shown in the test results and have all test pass.

Changes were made to:

DESCRIPTION
tests/testthat
test-derive_date_vars
test-derived_merge_vars
test-derived_param_computed
test-derived_param_wbc_abs
test-user_helpers
test-user_utils


Fix viewschedule bugs

Codecov Report

Merging #190 (3d3ee83) into master (dd79661) will increase coverage by 1.19%.
The diff coverage is 70.27%.

@@             Coverage Diff              @@
##             master     #190      +/-   ##
============================================
+ Coverage     65.81%   67.01%   +1.19%
- Complexity      668      694      +26
============================================
  Files           107      107
  Lines          2396     2419      +23
  Branches        321      326       +5
============================================
+ Hits           1577     1621      +44
+ Misses          694      672      -22
- Partials        125      126       +1

Impacted Files
Coverage Δ

...in/java/seedu/address/model/schedule/Schedule.java
33.33% <0.00%> (-0.82%)
:arrow_down:

.../main/java/seedu/address/model/schedule/Event.java
64.23% <78.78%> (+23.15%)
:arrow_up:

Continue to review full report at Codecov.

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

Looks good to me

Looks good


Build fails on Windows

What's going wrong?
I attempted to run the tests on Windows, and the following is the output:
$ go test ./... --count=1
# github.com/pilosa/pilosa/syswrap
syswrap\mmap.go:53:14: undefined: syscall.Mmap
syswrap\mmap.go:63:8: undefined: syscall.Munmap
FAIL    github.com/pilosa/pilosa [build failed]
FAIL    github.com/pilosa/pilosa/cmd [build failed]
FAIL    github.com/pilosa/pilosa/ctl [build failed]
FAIL    github.com/pilosa/pilosa/gopsutil [build failed]
FAIL    github.com/pilosa/pilosa/http [build failed]
FAIL    github.com/pilosa/pilosa/inmem [build failed]
FAIL    github.com/pilosa/pilosa/internal/clustertests [build failed]
ok      github.com/pilosa/pilosa/internal/test  0.268s
FAIL    github.com/pilosa/pilosa/pql [build failed]
ok      github.com/pilosa/pilosa/prometheus     0.314s
FAIL    github.com/pilosa/pilosa/roaring [build failed]
FAIL    github.com/pilosa/pilosa/server [build failed]
FAIL    github.com/pilosa/pilosa/stats [build failed]
FAIL    github.com/pilosa/pilosa/statsd [build failed]
FAIL    github.com/pilosa/pilosa/test [build failed]

Steps to reproduce the behavior
Navigate to the base folder and run go test ./... on a Windows machine. --count=1 is not required for the reproduction of the issue.
Information about your environment (OS/architecture, CPU, RAM, cluster/solo, configuration, etc.)
Windows 10 Pro 64-bit
go version go1.12.1 windows/amd64
Additional Information
It looks like the syscall.Mmap and syscall.Munmap calls have been deprecated, and are Unix-specific calls. They're available on both Linux and MacOS platforms:
https://golang.org/pkg/syscall/?GOOS=linux#Mmap
https://golang.org/pkg/syscall/?GOOS=darwin#Mmap
But the function cannot be found on Windows:
https://golang.org/pkg/syscall/?GOOS=windows

@Hydrocharged Thank you for the report. Pilosa depends heavily on memory-mapped files, which are unsupported on Windows, and there are no plans to support Windows natively.
For development in Windows, I recommend trying to run either in Docker or in a virtual machine (ie. Virtualbox). We have published a guide for running Pilosa in Docker for Windows here: https://www.pilosa.com/docs/latest/installation/#windows
Please file a separate ticket if you encounter problems running Pilosa in Docker.


Cannot export an read-only Shareable link

Steps to reproduce:

Create some drawing using the tools, and import a few icons from screenshot png image.
Click Export -> Export to Link
Shows "Couldn't create shareable link." error

Works for me. Is it possible your drawing it too large?

The issue is I was in the company version of Excalidraw and it has certificate issue.

Do you mean Excalidraw Plus? There's no "export to link" feature. You can enable link sharing, but that works slightly differently (you certainly won't be getting errors like you posted above), and isn't in the export dialog. Can you clarify what you were using?
Also, what certificate issue do you mean?
Thanks!


Best-download have backward compatibility issue

https://github.com/EleutherAI/best-download/commit/d0b8cc87cff520fa547283e1acc350267f2d72fd broke backward compatibility for best_download.
Instead:

76dc60938b43ec471149640e7ed954f3229fbc3e forced lm_eval to install only version <0.0.8.
c65412e530c1780ec7b2b970373e9e83f796ab20 allows lm_eval to use version >=0.0.8 by specifying argument names and removing version constraint in setup.py

I think this fixed partially #249 (I think that was the cause as the directory would be added, but dataset were downloaded in the current directory instead of the data one). However there's an issue that one has to manually remove the data cache to redownload correctly the dataset. (btw this isn't the fix we used, we just reverted back to 0.0.7 instead.
cc @leogao2

Looks good to me! We do want to get 0.0.8 working eventually because of the fallback feature (which would be really useful for situations in the future similar to the TE downtime)


add GitHub actions

PR #3 included the first set of changes for disabling Travis CI, but the Publish MkDocs static HTML workflow still needs to be added by someone with admin privileges

Ok I see it.. thanks Brian

done


Add feature flag for vendored OpenSSL

The native-tls and tokio-native-tls crates have a feature vendored, linking a vendored OpenSSL statically to the binary on Linux systems. We'd like to have a way to enable this feature from mysql_async for certain builds of ours.
This PR adds a new feature flag vendored-native-tls-tls, that enables vendoring and the use of native-tls-tls feature.

@pimeys, hi.
In general I don't like the idea of putting dependencies features into crate's own feature set – crates dependencies features are not our problem, I believe, or we'll end up with a huge set of unrelated features.
Please consider enabling the vendored feature yourself.
Related issues: #118, #143, blackbeam/rust-mysql-simple#238

Yup. We can do that too, we just have to be careful if somewhere we get a different major version of tokio-native-tls that does not use this feature. I advised people reviewing code to be diligent about that from now on.


Open with chunking

open_xarray_dataset now checks if the uncompressed data would be too large to fit in memory with default dask chunking (lat, lon, 1) and if that's the case, smaller chunks are calculated and applied to the dataset to open.

Codecov Report

Merging #129 into master will increase coverage by 0.05%.

@@            Coverage Diff             @@
##           master     #129      +/-   ##
==========================================
+ Coverage   70.78%   70.83%   +0.05%
==========================================
  Files          62       62
  Lines        7122     7133      +11
==========================================
+ Hits         5041     5053      +12
+ Misses       2081     2080       -1

Impacted Files
Coverage Δ

cate/core/workspace.py
74.39% <100%> (-0.52%)
:x:

cate/util/misc.py
94.26% <100%> (+0.51%)
:white_check_mark:

cate/ops/harmonize.py
100% <100%> (ø)
:white_check_mark:

cate/core/ds.py
72.22% <95%> (+3.47%)
:white_check_mark:

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 1e38b5b...5b22042. Read the comment docs.

Closes #129
See also #128

Added a comment on why it works the way it does, added integer division where appropriate!


matchRequestsBy functions should have the entire request context at their disposal

Description
I use many matchRequestsBy custom functions to unify different requests across test domains, users, tokens and proxies. Specifically with url segments functions (but also with other functions), I often find myself needing the entire context of the request in order to effectively unify the relevant request field.
For example, we have a jump-host proxy for testing from outside our VPM, that requires prefixing the URL pathname with some fragment specifying the target environment. However, to determine if I should actually remove that fragment from the pathname in order to unify the request ID, I need to have a look at the hostname of the request (and vice-versa, and unifying the hostname requires me to have the pathname data to understand where the jump host leads me).
It would be immensely useful if each matchRequestsBy function would receive two parameters - the relevant value to be mutated (e.g. the pathname) and an optional request object in its entirety, containing all the properties and values relevant for the matching mechanism.
Using the two-parameters approach wouldn't even break the existing API (current functions will still adhere the function signature and work as designed).
Thanks for a great and useful tool!

Makes sense to me. Should be a very simple add.


Set zero stage to 0 in autotuning when prescale gradients is used

In autotuning, the profiling run uses zero stage 3 which currently does not support prescale gradients. Thus this PR does this: when prescale_gradients is true, the autotuning profiling run uses zero stage 0 (ZeRO disabled).

Closing as stale for now.


How to Remove Layout Options?

Guys how do remove Layout Option Settings?
Thank You in Advance, for those who can help me ..

delete


onChange event does not work.

Anyone face this problem, with simple  code as present at Readme onChangeEvent does not trigger
onChange event.
  handleChange = (event, value, maskedValue) => {
this.setState({ amount: value })
}
render() {
const { amount } = this.state
return (
  <CurrencyInput value={amount} onChange={this.handleChange} />
)

}
`

I am seeing the same issue. Can someone please take a look at this?

onChangeEvent is the new event to use. But that's in the latest version, which doesn't work in safari. So if you downgrade to 1.2.6, you can get this working in safari-- but then you need to use onChange.


An error occurred while trying to save

I have this issue all the time. Sometimes I have to click the save button at least 5 times until it saves my updates. In this example I made over 290 changes and it will not save! It really freaks me out, because I already trying it for an hour ...

Whenever I get this message, it's an actual connectivity issue with one of the 3 api.openstreetmap.org servers.
If you are ever stuck in iD and it won't upload, you can try opening a new browser tab and visiting http://api.openstreetmap.org/api/0.6/capabilities to see if the api is actually available.
You can also check: http://wiki.openstreetmap.org/wiki/Platform_Status to see if there is an outage.
Sometimes you can get even more info by running it through curl with verbose option:
curl -v http://api.openstreetmap.org/api/0.6/capabilities
(e.g. This happened to me once:  I had been editing on a network with ipv6, then closed laptop, and woke it back up on a network with no ipv6 route to the dns cached api server).
Closing this issue, as it's not really something that we can do much about from the iD side, but hope some of this info might be helpful..

I think we could give a more descriptive message when this is encountered

Yes, in the next version the errors screen will have an expandable section so we can show more details.

Thanks Guys


deployment issues with latest master

plasma-deployer_1  | Compiling your contracts...
plasma-deployer_1  | ===========================
plasma-deployer_1  | > Everything is up to date, there is nothing to compile.
plasma-deployer_1  |
plasma-deployer_1  | Unknown network "infura". See your Truffle configuration file for available networks.
plasma-deployer_1  | Truffle v5.0.38 (core: 5.0.38)
plasma-deployer_1  | Node v8.16.2
plasma-deployer_1  | cat: can't open 'outputs.json': No such file or directory
plasma-deployer_1  | npx: installed 237 in 18.891s
plasma-deployer_1  |
plasma-deployer_1  |   \{^_^}/ hi!
plasma-deployer_1  |
plasma-deployer_1  |   Loading db.json
plasma-deployer_1  | SyntaxError: Malformed JSON in file: db.json
plasma-deployer_1  | Unexpected end of JSON input
plasma-deployer_1  |     at FileAsync.parse [as deserialize] (<anonymous>)
plasma-deployer_1  |     at /root/.npm/_npx/31/lib/node_modules/json-server/node_modules/lowdb/adapters/FileAsync.js:41:35
plasma-deployer_1  |     at <anonymous>
plasma-deployer_1  | fetch http://dl-cdn.alpinelinux.org/alpine/v3.9/main/x86_64/APKINDEX.tar.gz
plasma-deployer_1  | fetch http://dl-cdn.alpinelinux.org/alpine/v3.9/community/x86_64/APKINDEX.tar.gz
plasma-deployer_1  | OK: 247 MiB in 53 packages

I think this is what broke deployments:
https://github.com/omisego/plasma-contracts/commit/1608296ef56ab98d857321a687c10dcdd86d72d3

Looks like infura was changed to remote.
Elixir-omg docker-compose.yml needs to be updated to reflect this change:
npx truffle migrate --network remote

Unknown network "infura" is definitely my PR. I have this fix ready for elixir-omg in https://github.com/omisego/elixir-omg/pull/1100 but not sure if deployments need other changes too...

https://github.com/omisego/elixir-omg/pull/1100 is now merged. I'm testing docker-compose up on my machine now.

Works now after the merge, but I don't have permission to close this ticket.

Works.


made 8ball cmd

Made a 8ball command - followed 8-PEP guidelines, added a description to Russian roulette

I mean you can do the randomisation of the list in the embed but ok


Create z-index table in documentation/wiki

Issues like #4120 lead me to believe we should have a table of all of the z-indices used in our components, and the component parts inside them, as part of our documentation or wiki.
This would serve as a useful reference to developers trying to ensure their custom UI bits stay at the right height. We could also run it by the Material Design folks for validation to make sure we're doing it right.
@Garbee @surma What do you think?

SGTM. We can wiki for now to get things going and roll into the on-site docs with V2. The wiki overall should be deprecated since others can't contribute to it.

SGTM2. I agree with @Garbee on wiki'ing this for now. Did one of you want to take this or do you want us to see if Dave has cycles to help?


Fixed spelling

Summary
Fixed spelling in the sentence.
Fixes #Issue_Number (if available)

And please let me know if you need any help figuring out how to do this. One option would be for you to reset --hard your local fork with upstream and then do the new commit and git push -f.

Yes Maira that would be great. I trying to clean up this. Okay let me try the solution you provided.

@ThePiranha When working with git locally, this is usually my process for starting new work (variations on this method are possible of course):

Follow the steps to have a local clone, including setting up upstream as indicated here
git status (to make sure that are no unstaged changes)
git checkout master
git fetch upstream
git rebase upstream/master
git checkout -b working-branch-name

Then for the rest of the workflow, I use VS Code since we have some nice extensions for it:
https://docs.microsoft.com/en-us/contribute/get-started-setup-tools#visual-studio-code
For quick edits, I just use the GitHub UI: https://docs.microsoft.com/en-us/contribute/#quick-edits-to-existing-documents
I hope this helps! And thanks for contributing!

@ThePiranha When working with git locally, this is usually my process for starting new work (variations on this method are possible of course):

Follow the steps to have a local clone, including setting up upstream as indicated here
git status (to make sure that are no unstaged changes)
git checkout master
git fetch upstream
git rebase upstream/master
git push
git checkout -b working-branch-name

Then for the rest of the workflow, I use VS Code since we have some nice extensions for it:
https://docs.microsoft.com/en-us/contribute/get-started-setup-tools#visual-studio-code
For quick edits, I just use the GitHub UI: https://docs.microsoft.com/en-us/contribute/#quick-edits-to-existing-documents
I hope this helps! And thanks for contributing!

Thanks @mairaw I will follow the steps.


fix(deps): update github.com/chromedp/cdproto digest to eeb3d13

This PR contains the following updates:

Package
Type
Update
Change

github.com/chromedp/cdproto
require
digest
6d0b6a3 -> eeb3d13

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

⚠ Artifact update problem
Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.
♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

any of the package files in this branch needs updating, or
the branch becomes conflicted, or
you click the rebase/retry checkbox if found above, or
you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:
File name: undefined
Command failed: nix --extra-experimental-features nix-command --extra-experimental-features flakes --option accept-flake-config true run .#gomod2nix
warning: '/nix/var/nix' does not exist, so Nix will use '/home/ubuntu/.local/share/nix/root' as a chroot store
warning: Git tree '/tmp/renovate/repos/github/msfjarvis/gphotos-cdp' is dirty
copying path '/nix/store/97j7mjdvbxb59jnkr97qmhlkk97mnly8-source' from 'https://nix-community.cachix.org'...
copying path '/nix/store/1h1ss4ggf1ijl9mmi57s327sp6jpm8sw-source' from 'https://cache.garnix.io'...
copying path '/nix/store/fpr5fsl88azrdsbzjpfq2hk307bbr7cm-source' from 'https://cache.nixos.org'...
these 10 paths will be fetched (46.56 MiB download, 248.12 MiB unpacked):
  /nix/store/087167dfxal194pm54cmcbbxsfy3cjgn-bash-5.2p26
  /nix/store/8mc30d49ghc8m5z96yz39srlhg5s9sjj-glibc-2.38-44
  /nix/store/hkzgcfj86jzzpayghzvy09xpvjg4xmac-go-1.21.7
  /nix/store/df2j8zxx6g5gnzmvy3brfvgvhxfjrc12-gomod2nix-dev
  /nix/store/0qwpih2kjj1vgb5imcgdqybsr2dr92aa-iana-etc-20231227
  /nix/store/rny9bdwrs6fkgkq678ixfg1lv3sa18ql-libidn2-2.3.7
  /nix/store/y6b21pzdhxgpj59ia02xfcq3lah13n1q-libunistring-1.1
  /nix/store/0q0krya2j3slr91w5n3acl3673z82j75-mailcap-2.1.53
  /nix/store/lb1z8nckdwx4kmwy19znh1bl1p63wxhv-tzdata-2024a
  /nix/store/hihnkpyd5av509x6cz2i5zhf04j57fyh-xgcc-13.2.0-libgcc
copying path '/nix/store/0qwpih2kjj1vgb5imcgdqybsr2dr92aa-iana-etc-20231227' from 'https://cache.nixos.org'...
copying path '/nix/store/0q0krya2j3slr91w5n3acl3673z82j75-mailcap-2.1.53' from 'https://cache.nixos.org'...
copying path '/nix/store/lb1z8nckdwx4kmwy19znh1bl1p63wxhv-tzdata-2024a' from 'https://cache.nixos.org'...
copying path '/nix/store/hihnkpyd5av509x6cz2i5zhf04j57fyh-xgcc-13.2.0-libgcc' from 'https://cache.nixos.org'...
copying path '/nix/store/y6b21pzdhxgpj59ia02xfcq3lah13n1q-libunistring-1.1' from 'https://cache.nixos.org'...
copying path '/nix/store/rny9bdwrs6fkgkq678ixfg1lv3sa18ql-libidn2-2.3.7' from 'https://cache.nixos.org'...
copying path '/nix/store/8mc30d49ghc8m5z96yz39srlhg5s9sjj-glibc-2.38-44' from 'https://cache.nixos.org'...
copying path '/nix/store/087167dfxal194pm54cmcbbxsfy3cjgn-bash-5.2p26' from 'https://cache.nixos.org'...
copying path '/nix/store/hkzgcfj86jzzpayghzvy09xpvjg4xmac-go-1.21.7' from 'https://cache.nixos.org'...
copying path '/nix/store/df2j8zxx6g5gnzmvy3brfvgvhxfjrc12-gomod2nix-dev' from 'https://cache.garnix.io'...
error: setting up a private mount namespace: Operation not permitted


Repo server should either be more forgiving in terms of the contract address in the URL or use EIP55 (or all lowercase)

The current implementation seems to just read the addresses from the TokenScript file and expects the address to be provided in the URL to match exactly. This makes it impossible for the TokenScript client to construct a URL that will predictably fetch the file correctly.
Given the TokenScript for FIFA:
<ts:address network="3">0xd8e5f58de3933e1e35f9c65eb72cb188674624f3</ts:address>

$ curl -I https://repo.tokenscript.org/2019/05/0xd8e5f58de3933e1e35f9c65eb72cb188674624f3
> 200

$ curl -I https://repo.tokenscript.org/2019/05/0xD8e5F58DE3933E1E35f9c65eb72cb188674624F3
>404

On the other hand, with EntryToken:
<ts:address network="3">0xFB82A5a2922A249f32222316b9D1F5cbD3838678</ts:address>

We have:
$ curl -I https://repo.tokenscript.org/2019/05/0xfb82a5a2922a249f32222316b9d1f5cbd3838678
> 404

$ curl -I https://repo.tokenscript.org/2019/05/0xFB82A5a2922A249f32222316b9D1F5cbD3838678
> 200

We should either:
A) Expect to be liberal about casing in the URL, or
B) Enforce all lowercase, or
C) Enforce EIP55

Will handle it. To save you a bit of time in the future, I can understand the issue if you just say the title.
Since it's not TokenScript's task to make sure the address actually exists, I'll just implement case-ignore, close this one, file a new one about EIP55 to work on perfecting it in the future.


Migrate to opm after next Openresty release

done! opm install sumory/lor is supported after v0.2.2


app not working on iOS 14 developer beta

Describe the bug

Once “cancelled” the NEXT buttons also stopped working, see video:
https://youtu.be/5IObvfRpxIs
Steps to reproduce the issue

Download iOS 14 Profile
Start Corona Warn App

Technical details

iOS Version: iOS 14
Device: iPhone XR

https://www.coronawarn.app/en/faq/#ios_14_beta

overlooked that, my bad!


added syntaxsql/sql tags

@icoric : Thanks for your contribution! The author(s) have been notified to review your proposed change.


Phil drafting site

Making changes so that the github.io site is set up as a drafting site.

@kcoyle I double checked this to make sure I had started from the right version. It does look OK. If you merge it we will be able to see what it looks like on the github.io site.
I moved the spec doc to the /drafts folder from the index page and put text on the index page to indicate where drafts and published versions could be found.
I updated the json-ld context file to use the correct name space uri and our preferred prefix, and added notes to warn people that it is a draft.


Remote desktop - \.\DisplayXX

Greetings, thank you for your contribution this driver is great! However with that said I am encountering an issue with my current setup.
Currently the driver assigns a random Display Number when a Virtual Display is created, the problem I am having is I would like moonlight to always default to this Virtual display so I have manually set the display within sunshine and this works great. However when I connect to my computer via remote desktop it seems as if the Virtual display is assigned  a new display number. Which unfortunately means that I have to reset the chosen display within sunshine which is unfavorable.
Would it be / is it possible to hard set the display number within the driver so that when connecting via remote desktop I do not have to manually set the display to the new display number assigned?
For reference by display numerous I am referring to the \.\DisplayXX name value.

Just a quick update I did find this tool https://github.com/WeebLabs/Virtual-Monitor-Helper/releases which should fit my needs in this situation I'll give it a shot and see if it fixes the issue that I am having.

I'm also experiencing this, upon restarting. I've tried using the project mentioned by @Death666Machine but it doesnt seem to be detecting the drivers as it isn't updating the display name within the sunshine config file. There's no logs that I can see so I'm unsure how far the Virtual Monitor Helper is getting anyways.

@SavageTheUnicorn this is the solution I have found https://github.com/Nonary/MonitorSwapAutomation basically I have it setup to swap the primary monitor to be the virtual display driver 😉 hope this helps!

@SavageTheUnicorn this is the solution I have found https://github.com/Nonary/MonitorSwapAutomation basically I have it setup to swap the primary monitor to be the virtual display driver 😉 hope this helps!

@Death666Machine Do you have the virtual display set to extend, duplicate, or show only on 1? I assume the main monitor is the primary in windows settings and sunshine config is left blank?
I saw this project yesterday but I was a bit skeptical of it providing valuable results so I'll look into it again now. Thanks! 😊

In my current configuration I have mine setup to extend. I only have my main monitor active until a connection to sunshine is made, then the virtual display is activated, set as primary.  then I move the position of my primary monitor to the right of virtual display monitor. When connection is terminated the virtual display monitor gets deactivated, my monitor is set to primary, and moved mack to the left of the virtual Display monitor. In my sunshine configuration I have it set to primary monitor instead of setting a display name. This way it doesn't matter what the name of the display is, it just chooses the primary monitor at the time of connection. Hope this is understandable and makes sense 😁 I've had absolutely 0 issues with this setup.

@Death666Machine this works to swap the monitor, but the functionality I'm looking for is different. I'm attempting to stream ONLY the second virtual display adapter while retaining function on my host pc. I can use DisplayFusion to allow simultaneous mouse and controller input. My only issue is the randomizing display name, I can easily set it every session (disregarding the fact i cant really see my main screen while streaming the second monitor to move the apps and lock input)

This is exactly how my setup works aswell, the only difference being is that I set the virtual display monitor to be disabled so I don't run my mouse off screen when not streaming but thats purely preference as you can adjust the script to do exactly what you are looking for. 😁 when streaming you just have to move apps and lock input as you say. I haven't found a way to simply that process as the primary monitor gets switched to be the virtual display monitor during streaming.

Ahh. I'm lacking knowledge of how to reliably get the result I want so I'm just resorting to leaving it as is, but using TeamViewer to view the main desktop remotely when needed for changing the display name. I've got a batch script set to run at boot that will autorun the dxgi-info.exe, so I just have to scroll up and read it and enter it into the web page that also is set to auto start. At that point sunshine can be restarted to allow functionality between both desktops (the mouse running off doesn't bother me, DisplayFusion also allows locking the mouse to a seperate window as well as allowing "unfocused" controller inputs to games, but I'm unsure about entire displays). It's just a huge pain caused by specifically the driver having a randomized display name upon reboots. If this could be set to a static number somehow it would solve everything.


More specifics from docs/others for camel case variables

removing loglevel setting until such time as we can pull it from the underlying operator fully as a future improvement

Thanks, LogLevel was there mainly for debug on the initial tests, happy to remove it!


Install and configure vue-i18n for internationalization support

We can refer to the official documentation.

Fixed in #880


Saving user's preferences

Saving and loading user settings locally using SharedPreferences
https://developer.android.com/training/data-storage/shared-preferences

Also functions for

Saving user's unique ID (UUID)
Getting said ID

Share location to other devices (true/false)

   Location sharing means if the user is seen on a map to other users. This does not mean disabling the GPS sensor on the device.


Markdown is broken when read from GitHub viewer

I write markdown headers on VS Code and they are visualized correctly on the VS Code Markdown Previewer. However, GitHub Markdown Viewer doesn't recognize them as headers. I always need to fix that error on GitHub web interface.
I suspect that there is an invisible character on VS Code different to the GitHub one when I click "space bar" between the hashtags and header title. (https://github.com/jorge-sanz/JUnitExample/commit/186ad2450abcec89ec4368b57a77e8f3eebc8ce9)

VSCode Version: 1.16.1
OS Version: macOS High Sierra

I discovered my error. For the space between the hashtags and the title I click space + ALT instead of only space because of the fast typing. And that's the reason any markdown previewer doesn't recognize them as headers.


[stable/nifi] Adding Nifi chart into the helm repository (New chart)

Signed-off-by: AlexsJones<EMAIL_ADDRESS>
Is this a new chart
YES

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

What this PR does / why we need it:
This adds a simple kubernetes statefulset for Nifi as orchestrated by Helm.
Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)
N/A
Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md
[x] Title of the PR starts with chart name (e.g. [stable/chart])

/ok-to-test


Test case <IP_ADDRESS>: Unable to import an application by the application owner

Test case <IP_ADDRESS>:
Should support Export and Import (migrating) an Application preserving the application owner.
Result:
When try to import an application created by admin, cannot import.

Related issues https://github.com/wso2/product-apim/issues/6044, https://github.com/wso2/product-apim/issues/6143

Can reproduce the issue during both update and import applications


[WIP]Make a common template for /api pages

This PR removes the need for multiple .md files and makes a common template to generate /api pages from the OpenAPI data. This is currently a WIP.
Work plan:

[x] Add title of API pages as summary parameter in YAML.
[x] Write MarkdownProcessor to get title from summary parameter.
[x] Refactor titles of existing /api files to use markdowngenerator.
[x] Fix HTML titles for the /api pages.
[x] Add the template .md file.
[x] Refactor code to use the template .md file.
[x] Remove files following the template(CHECKED VIA SCRIPT)
[x] Proper rendering of JS examples.
[x] Proper rendering of 404 responses.
[x] Tackle specific curl options
[x] Tackle specific configurations(like admin).
[ ] Migrate stray descriptions.

Testing plan:
GIFs or screenshots:

Thanks for the review @timabbott ! I have addressed the comments, and am fixing the currently failing tests. The PR should be ready for another review by tomorrow.

This is great @MSurfer20!  I reviewed the first half of the PR and merged it with the following changes.  (Mainly a few different editorial decisions for what to call endpoints with non-standard names).
I'll post a batch of comments next on the remaining changes.
diff --git a/templates/zerver/api/changelog.md b/templates/zerver/api/changelog.md
index c461276050..0f591a065a 100644
--- a/templates/zerver/api/changelog.md
+++ b/templates/zerver/api/changelog.md
@@ -162,14 +162,14 @@ field with an integer field `invite_to_realm_policy`.

 **Feature level 49**

-* Added new [`POST /realm/playground`](/api/add-realm-playground) and
-  [`DELETE /realm/playground/{playground_id}`](/api/remove-realm-playground)
-  endpoints for realm playgrounds.
+* Added new [`POST /realm/playground`](/api/add-code-playground) and
+  [`DELETE /realm/playground/{playground_id}`](/api/remove-code-playground)
+  endpoints for code playgrounds.
 * [`GET /events`](/api/get-events): A new `realm_playgrounds` events
-  is sent when changes are made to a set of configured playgrounds for
+  is sent when changes are made to a set of configured code playgrounds for
   an organization.
 * [`POST /register`](/api/register-queue): Added a new `realm_playgrounds`
-  field, which is required to fetch the set of configured playgrounds for
+  field, which is required to fetch the set of configured code playgrounds for
   an organization.

 **Feature level 48**
diff --git a/templates/zerver/api/check-narrow-matches.md b/templates/zerver/api/check-messages-match-narrow.md
similarity index 100%
rename from templates/zerver/api/check-narrow-matches.md
rename to templates/zerver/api/check-messages-match-narrow.md
diff --git a/templates/zerver/help/include/rest-endpoints.md b/templates/zerver/help/include/rest-endpoints.md
index f630c8f9d7..14f161df2f 100644
--- a/templates/zerver/help/include/rest-endpoints.md
+++ b/templates/zerver/help/include/rest-endpoints.md
@@ -10,7 +10,7 @@
 * [Remove an emoji reaction](/api/remove-reaction)
 * [Render a message](/api/render-message)
 * [Get a message's raw Markdown](/api/get-raw-message)
-* [Check messages match narrow](/api/check-messages-match-narrow)
+* [Check if messages match narrow](/api/check-messages-match-narrow)
 * [Get a message's edit history](/api/get-message-history)
 * [Update personal message flags](/api/update-message-flags)
 * [Mark messages as read in bulk](/api/mark-all-as-read)
@@ -61,8 +61,8 @@
 * [Add a linkifier](/api/add-linkifier)
 * [Update a linkifier](/api/update-linkifier)
 * [Remove a linkifier](/api/remove-linkifier)
-* [Add a playground](/api/add-realm-playground)
-* [Remove a playground](/api/remove-realm-playground)
+* [Add a code playground](/api/add-code-playground)
+* [Remove a code playground](/api/remove-code-playground)
 * [Get all custom emoji](/api/get-custom-emoji)
 * [Upload custom emoji](/api/upload-custom-emoji)
 * [Get all custom profile fields](/api/get-custom-profile-fields)
diff --git a/tools/linter_lib/custom_check.py b/tools/linter_lib/custom_check.py
index 89b46cc1cb..3b48a965c3 100644
--- a/tools/linter_lib/custom_check.py
+++ b/tools/linter_lib/custom_check.py
@@ -884,10 +884,7 @@ help_markdown_rules = RuleList(
         {
             "pattern": r"\b[rR]ealm[s]?\b",
             "include_only": {"templates/zerver/help/"},
-            "exclude": {
-                "templates/zerver/help/change-organization-url.md",
-                "templates/zerver/help/include/rest-endpoints.md",
-            },
+            "exclude": {"templates/zerver/help/change-organization-url.md"},
             "good_lines": ["Organization", "deactivate_realm", "realm_filter"],
             "bad_lines": ["Users are in a realm", "Realm is the best model"],
             "description": "Realms are referred to as Organizations in user-facing docs.",
diff --git a/zerver/openapi/openapi.py b/zerver/openapi/openapi.py
index 951d5651d6..f2052bee02 100644
--- a/zerver/openapi/openapi.py
+++ b/zerver/openapi/openapi.py
@@ -224,7 +224,7 @@ def get_endpoint_from_operationid(operationid: str) -> Tuple[str, str]:
             operationId = openapi_spec.openapi()["paths"][endpoint][method].get("operationId")
             if operationId == operationid:
                 return (endpoint, method)
-    raise AssertionError("No such page exists in OpenApi data.")
+    raise AssertionError("No such page exists in OpenAPI data.")

 def get_openapi_paths() -> Set[str]:
diff --git a/zerver/openapi/zulip.yaml b/zerver/openapi/zulip.yaml
index dae8854340..f14a638bb9 100644
--- a/zerver/openapi/zulip.yaml
+++ b/zerver/openapi/zulip.yaml
@@ -7040,8 +7040,8 @@ paths:
                 $ref: "#/components/schemas/JsonSuccess"
   /realm/playgrounds:
     post:
-      operationId: add_realm_playground
-      summary: Add realm playground
+      operationId: add_code_playground
+      summary: Remove a code playground
       tags: ["server_and_organizations"]
       description: |
         Configure [code playgrounds](/help/code-blocks#code-playgrounds) for the organization.
@@ -7097,8 +7097,8 @@ paths:
                     example: {"id": 1, "result": "success", "msg": ""}
   /realm/playgrounds/{playground_id}:
     delete:
-      operationId: remove_realm_playground
-      summary: Remove realm playground
+      operationId: remove_code_playground
+      summary: Remove a code playground
       tags: ["server_and_organizations"]
       description: |
         Remove a [code playground](/help/code-blocks#code-playgrounds) previously

One other small thought -- you should adjust the commit order so that each commit works.  E.g. move the commit to change discovery tests earlier in the branch, since that one will work both before and after deleting the files.  That makes it easier to safely incrementally merge things.

This is great @MSurfer20! I reviewed the first half of the PR and merged it with the following changes. (Mainly a few different editorial decisions for what to call endpoints with non-standard names).
I'll post a batch of comments next on the remaining changes.
diff --git a/templates/zerver/api/changelog.md b/templates/zerver/api/changelog.md
index c461276050..0f591a065a 100644
--- a/templates/zerver/api/changelog.md
+++ b/templates/zerver/api/changelog.md
@@ -162,14 +162,14 @@ field with an integer field `invite_to_realm_policy`.

 **Feature level 49**

-* Added new [`POST /realm/playground`](/api/add-realm-playground) and
-  [`DELETE /realm/playground/{playground_id}`](/api/remove-realm-playground)
-  endpoints for realm playgrounds.
+* Added new [`POST /realm/playground`](/api/add-code-playground) and
+  [`DELETE /realm/playground/{playground_id}`](/api/remove-code-playground)
+  endpoints for code playgrounds.
 * [`GET /events`](/api/get-events): A new `realm_playgrounds` events
-  is sent when changes are made to a set of configured playgrounds for
+  is sent when changes are made to a set of configured code playgrounds for
   an organization.
 * [`POST /register`](/api/register-queue): Added a new `realm_playgrounds`
-  field, which is required to fetch the set of configured playgrounds for
+  field, which is required to fetch the set of configured code playgrounds for
   an organization.

 **Feature level 48**
diff --git a/templates/zerver/api/check-narrow-matches.md b/templates/zerver/api/check-messages-match-narrow.md
similarity index 100%
rename from templates/zerver/api/check-narrow-matches.md
rename to templates/zerver/api/check-messages-match-narrow.md
diff --git a/templates/zerver/help/include/rest-endpoints.md b/templates/zerver/help/include/rest-endpoints.md
index f630c8f9d7..14f161df2f 100644
--- a/templates/zerver/help/include/rest-endpoints.md
+++ b/templates/zerver/help/include/rest-endpoints.md
@@ -10,7 +10,7 @@
 * [Remove an emoji reaction](/api/remove-reaction)
 * [Render a message](/api/render-message)
 * [Get a message's raw Markdown](/api/get-raw-message)
-* [Check messages match narrow](/api/check-messages-match-narrow)
+* [Check if messages match narrow](/api/check-messages-match-narrow)
 * [Get a message's edit history](/api/get-message-history)
 * [Update personal message flags](/api/update-message-flags)
 * [Mark messages as read in bulk](/api/mark-all-as-read)
@@ -61,8 +61,8 @@
 * [Add a linkifier](/api/add-linkifier)
 * [Update a linkifier](/api/update-linkifier)
 * [Remove a linkifier](/api/remove-linkifier)
-* [Add a playground](/api/add-realm-playground)
-* [Remove a playground](/api/remove-realm-playground)
+* [Add a code playground](/api/add-code-playground)
+* [Remove a code playground](/api/remove-code-playground)
 * [Get all custom emoji](/api/get-custom-emoji)
 * [Upload custom emoji](/api/upload-custom-emoji)
 * [Get all custom profile fields](/api/get-custom-profile-fields)
diff --git a/tools/linter_lib/custom_check.py b/tools/linter_lib/custom_check.py
index 89b46cc1cb..3b48a965c3 100644
--- a/tools/linter_lib/custom_check.py
+++ b/tools/linter_lib/custom_check.py
@@ -884,10 +884,7 @@ help_markdown_rules = RuleList(
         {
             "pattern": r"\b[rR]ealm[s]?\b",
             "include_only": {"templates/zerver/help/"},
-            "exclude": {
-                "templates/zerver/help/change-organization-url.md",
-                "templates/zerver/help/include/rest-endpoints.md",
-            },
+            "exclude": {"templates/zerver/help/change-organization-url.md"},
             "good_lines": ["Organization", "deactivate_realm", "realm_filter"],
             "bad_lines": ["Users are in a realm", "Realm is the best model"],
             "description": "Realms are referred to as Organizations in user-facing docs.",
diff --git a/zerver/openapi/openapi.py b/zerver/openapi/openapi.py
index 951d5651d6..f2052bee02 100644
--- a/zerver/openapi/openapi.py
+++ b/zerver/openapi/openapi.py
@@ -224,7 +224,7 @@ def get_endpoint_from_operationid(operationid: str) -> Tuple[str, str]:
             operationId = openapi_spec.openapi()["paths"][endpoint][method].get("operationId")
             if operationId == operationid:
                 return (endpoint, method)
-    raise AssertionError("No such page exists in OpenApi data.")
+    raise AssertionError("No such page exists in OpenAPI data.")

 def get_openapi_paths() -> Set[str]:
diff --git a/zerver/openapi/zulip.yaml b/zerver/openapi/zulip.yaml
index dae8854340..f14a638bb9 100644
--- a/zerver/openapi/zulip.yaml
+++ b/zerver/openapi/zulip.yaml
@@ -7040,8 +7040,8 @@ paths:
                 $ref: "#/components/schemas/JsonSuccess"
   /realm/playgrounds:
     post:
-      operationId: add_realm_playground
-      summary: Add realm playground
+      operationId: add_code_playground
+      summary: Remove a code playground
       tags: ["server_and_organizations"]
       description: |
         Configure [code playgrounds](/help/code-blocks#code-playgrounds) for the organization.
@@ -7097,8 +7097,8 @@ paths:
                     example: {"id": 1, "result": "success", "msg": ""}
   /realm/playgrounds/{playground_id}:
     delete:
-      operationId: remove_realm_playground
-      summary: Remove realm playground
+      operationId: remove_code_playground
+      summary: Remove a code playground
       tags: ["server_and_organizations"]
       description: |
         Remove a [code playground](/help/code-blocks#code-playgrounds) previously

Thanks for the merge @timabbott ! In this change, I notice that there was a typo of 'Add' being written as 'Remove' for the add_code_playground summary. Opened https://github.com/zulip/zulip/pull/18625/ to fix it :)

@timabbott I have replied to most of the comments above. Further, the deleted api pages currently are the ones where there is no diff between the original page and page after deleting. However, this removed some pages that earlier i thought would be exact matches--so I'll have to look more into this.

Attaching the diff of all the files. Of these, only update_display_settings and update_notification_settings need to be looked at, since their return values weren't displayed earlier, but now they appear due to the template. Others are just improvements over what the templates displayed.
old-check-messages-match-narrow new-check-messages-match-narrow
262c262
<
Arguments

---
>
Parameters

old-check-messages-match-narrow new-check-messages-match-narrow

old-fetch-api-key new-fetch-api-key
280c280
<
Example response

---
>
Example response

old-fetch-api-key new-fetch-api-key

old-get-own-user new-get-own-user
263c263
<
This endpoint takes no parameters.

---
>
This endpoint does not accept any parameters.

old-get-own-user new-get-own-user

old-get-subscription-status new-get-subscription-status
269a270,273
>
Return values

>

>
is_subscribed: boolean Whether the user is subscribed to the stream.

>

old-get-subscription-status new-get-subscription-status

old-unsubscribe new-unsubscribe
310a311
>
Response

old-unsubscribe new-unsubscribe

old-update-display-settings new-update-display-settings
409a410,458
>

>

>
twenty_four_hour_time: boolean The setting for twenty_four_hour_time, if it was changed in this request.

>

>

>
dense_mode: boolean The setting for dense_mode, if it was changed in this request.
>     This setting is however reserved for future, and can not be
>     modified in production environment.

>

>

>
starred_message_counts: boolean The setting for starred_message_counts, if it was changed
>     in this request.

>

>

>
fluid_layout_width: boolean The setting for fluid_layout_width, if it was changed
>     in this request.

>

>

>
high_contrast_mode: boolean The setting for high_contrast_mode, if it was changed in
>     this request.
>     This setting is however reserved for future, and can not be
>     modified in production environment.

>

>

>
color_scheme: integer The numerical key corresponding to new color_scheme if it was changed in this request.

>

>

>
translate_emoticons: boolean The setting for translate_emoticons, if it was changed in this request.

>

>

>
default_language: string The language code corresponding to new default_language if it was changed in this request.

>

>

>
default_view: string The new setting for default_view, if it was changed in this request.

>

>

>
left_side_userlist: boolean The setting for left_side_userlist, if it was changed in this request.

>

>

>
emojiset: string The string identifier corresponding to new emojiset if it was changed in this request.

>

>

>
demote_inactive_streams: integer The numerical key corresponding to new demote_inactive_streams setting if it was changed in this request.

>

>

>
timezone: string The setting for timezone, if it was changed in this request.

>

>

>
Return values

old-update-display-settings new-update-display-settings

old-update-notification-settings new-update-notification-settings
436a437,482
>

>

>
enable_desktop_notifications: boolean The setting for enable_desktop_notifications, if it was changed in
>     this request.

>

>

>
enable_digest_emails: boolean The setting for enable_digest_emails, if it was changed in this
>     request.

>

>

>
enable_marketing_emails: boolean The setting for enable_marketing_emails, if it was changed in this
>     request.

>

>

>
enable_offline_email_notifications: boolean The setting for enable_offline_email_notifications, if it was changed
>     in this request.

>

>

>
enable_offline_push_notifications: boolean The setting for enable_offline_push_notifications, if it was changed
>     in this request.

>

>

>
enable_online_push_notifications: boolean The setting for enable_online_push_notifications, if it was changed in
>     this request.

>

>

>
enable_sounds: boolean The setting for enable_sounds, if it was changed in this request.

>

>

>
enable_stream_email_notifications: boolean The setting for enable_stream_email_notifications, if it was changed
>     in this request.

>

>

>
enable_stream_push_notifications: boolean The setting for enable_stream_push_notifications, if it was changed in
>     this request.

>

>

>
enable_stream_audible_notifications: boolean The setting for enable_stream_audible_notifications, if it was changed
>     in this request.

>

>

>
message_content_in_email_notifications: boolean The setting for message_content_in_email_notifications, if it was
>     changed in this request.

>

>

>
Return values

old-update-notification-settings new-update-notification-settings

old-upload-file new-upload-file
261a262
>
This endpoint does not accept any parameters.

old-upload-file new-upload-file

Also, talking about the files which i couldn't remove as of now(I would appreciate feedback about how I should proceed with them):

construct-narrow: This one is all text, and doesn't remotely follow the template.
create-stream:  This one is all text, and doesn't remotely follow the template.
get-events: This one contains import sys in its python example, due to which it is currently hardcoded in the template.
mark-all-as-read: This one contains 3 endpoints documentation. We can split it to make 3 different doc pages, but these do seem to fit better on a single page.
real-time-events: This one contains import sys in its python example, due to which it is currently hardcoded in the template.
register-queue: ?(I need to cross-check this one).
send-message: Its docs also contain zulip-send and bash examples, and so it can't be fit into the template.

@timabbott Managed to delete 58 files :)

I merged 4 early commits as the series ending with e892a02bb8e1f06fa728c70f2fe37d7ec069376c.
I agree that the output looks great; essentially all the changes are bug fixes :).  Posted a comment on the Python commit, but I don't think it's a blocker for merging (since we can always fix that forward).
I think I agree with your thoughts on which of these can be readily deleted; we should fix the import sys thing and let the rest be.

@timabbott Apologies for the unnoticed bug.... Didn't realise it since all the tests had passed in the end.

Merged after the small update above and also squashing the "add template" and "use template" commits.  Huge thanks for doing this giant migration @MSurfer20!
A few notes on near-term follow-ups:

We should probably switch to consistently using endpoint_path as the local variable name for e.g. /messages/{message_id} URL paths; endpoint_name is confusing for something with /s in it.  It's still a little mixed right now.
We should do an update of the "writing API documentation" docs, since there's a lot of things one no longer needs to do.  We can close https://github.com/zulip/zulip/issues/12571 after doing so.
It's worth noting that /api/get_messsages now works just as well as /api/get-messages.  That's probably a good thing; I remember someone wanting that feature.  I'll post about it in #documentation.


Sea Animals Detection

Pull Request for DL-Simplified 💡
Issue Title : Sea Animals Detection

Info about the related issue (Aim of the project) :
Name: Kunal Agrawal
GitHub ID: kunalagra
Email ID<EMAIL_ADDRESS>Idenitfy yourself: SSoC'22

Closes: #109
Describe the add-ons or changes you've made 📃
Give a clear description of what have you added or modifications made
Type of change ☑
What sort of change have you made:

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Code style update (formatting, local variables)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested? ⚙
Model Accuracy tested & used on test dataset
Checklist: ☑

[x] My code follows the guidelines of this project.
[x] I have performed a self-review of my own code.
[x] I have commented my code, particularly wherever it was hard to understand.
[x] I have made corresponding changes to the documentation.
[x] My changes generate no new warnings.
[x] I have added things that prove my fix is effective or that my feature works.
[x] Any dependent changes have been merged and published in downstream modules.

Done. @abhisheks008
It seems to have been skipped out cause for some weird reasons.
Thanks for pointing out.


gcc: fix missing if-else in kalman

Description
Fixes a deviation from the GCC rfc spec.
Reference issue
Fixes #286

I'm not sure what's going on with the linter - it doesn't seem to mention the offenders. I tried to fix some locations where newline was added but the linter still isn't passing.


Make Stats::frameRead and Stats::frameWritten accept FrameType …

…instead of std::string
@lexs

LGTM, letting @lehecka merge

I am not sure if FrameType enum is part of the public API. Can you verify that?

How do you mean public api?
It's part of the spec as of https://github.com/ReactiveSocket/reactivesocket/blob/master/Protocol.md#frame-types (although,  although looking at the protocol definition, there's a discrepancy between our definitions and the official protocol spec - REQUEST_SUB is missing from the latter)

public API of the cpp library, not the spec.
I am not sure if the application code has access to the internal enum type.

Frame.h is pulled in via reactivesocket-plugins.


Support Sparse Tensors in py_function

TensorFlow version (you are using): 2.0.0b0
Are you willing to contribute it (Yes/No): No

Describe the feature and the current behavior/state.
Currently, sparse tensors don't seem to be supported as inputs to py_function. Attempting to pass one results in an error like "TypeError: Tensors in list passed to 'input' of 'EagerPyFunc' Op have types [<NOT CONVERTIBLE TO TENSOR>] that are invalid.". Example:
def dataset_map_sparse_test():
    input_data_sparse = tf.SparseTensor(indices=[[0, 0], [1, 2]], values=[1, 2], dense_shape=[3, 4])
    input_data_dense = tf.sparse.to_dense(input_data_sparse)

    ds_dense = tf.data.Dataset.from_tensor_slices(input_data_dense)
    ds_sparse = tf.data.Dataset.from_tensor_slices(input_data_sparse)

    def inner_fn(*input):
        return input

    def outer_fn(*input_data):
        return tf.py_function(
            inner_fn,
            input_data,
            (tf.int32,))

    # this works
    ds_dense_mapped = ds_dense.map(outer_fn)
    print(next(iter(ds_dense_mapped)))

    # this results in the error specified above
    ds_sparse_mapped = ds_sparse.map(outer_fn)
    print(next(iter(ds_sparse_mapped)))

Will this change the current api? How?
Yes. The inp parameter of py_function currently accepts a list of Tensor objects; this change would broaden it to accept either Tensors or SparseTensors. (Possibly _TensorLikes?)
Who will benefit with this feature?
Anyone wishing to pass sparse tensors to a py_function-wrapped function. There are situations where this wrapping is necessary; for example, functions passed to Dataset.map cannot perform certain operations unless the function is wrapped using py_function.
Any Other info.

I think now that we have compositetensor and typespec it should be straightforward to implement sparsetensor support in py_function by treating all composites generically.
I don't have the bandwidth to do this now, so leaving as contributions welcome.

@alextp I would like to work on this, can I please go ahead?

Would "all composites" include ragged tensors, or should I create a separate feature request for that? Thanks!

Composites include ragged tensors and a lot of other things as well.

@pradyumna1 please go ahead!

I notice that someone had already created issue #26453 (Allow py_function to support functions that return RaggedTensor). Implementing this feature using composite tensors would also satisfy that issue.

I made changes in file op_def_library.py so that CompositeTensor is now allowed. However, there are some missing features of CompositeTensor that does not allow adding the node to the computation graph, for example ._as_tf_output(), which is used in function _create_c_op. That function only appears in class Tensor. Could you advise on this @alextp?

@minhtriet, in a similar feature request (#27679), @edloper provided a helpful workaround in the form of a wrapper around py_function, which he suggested could be integrated into _internal_py_func. Does that help?

Thank you @novog, I could not read work on it right now, would get back to you in a couple of days

So, I updated tf.py_function in my fork
Rerunning the code, the error is
2019-12-14 21:38:52.367270: I tensorflow/core/common_runtime/process_util.cc:115] Creating new thread pool with default inter op setting: 4. Tune using inter_op_parallelism_threads for best performance.
(<tf.Tensor: id=20, shape=(4,), dtype=int32, numpy=array([1, 0, 0, 0])>,)
2019-12-14 21:38:53.274875: W tensorflow/core/framework/op_kernel.cc:1622] OP_REQUIRES failed at iterator_ops.cc:929 : Invalid argument: {{function_node __inference_Dataset_map_outer_fn_28}} pyfunc_1 returns 3 values, but expects to see 1 values.
	 [[{{node PyFuncStateless}}]]
Traceback (most recent call last):
  File "C:/Users/cool/Code/tf_contrib_conda/test.py", line 28, in <module>
    dataset_map_sparse_test()
  File "C:/Users/cool/Code/tf_contrib_conda/test.py", line 26, in dataset_map_sparse_test
    print(next(iter(ds_sparse_mapped)))
  File "C:\Users\cool\Miniconda3\envs\tf_contrib_conda\lib\site-packages\tensorflow_core\python\data\ops\iterator_ops.py", line 622, in __next__
    return self.next()
  File "C:\Users\cool\Miniconda3\envs\tf_contrib_conda\lib\site-packages\tensorflow_core\python\data\ops\iterator_ops.py", line 666, in next
    return self._next_internal()
  File "C:\Users\cool\Miniconda3\envs\tf_contrib_conda\lib\site-packages\tensorflow_core\python\data\ops\iterator_ops.py", line 651, in _next_internal
    output_shapes=self._flat_output_shapes)
  File "C:\Users\cool\Miniconda3\envs\tf_contrib_conda\lib\site-packages\tensorflow_core\python\ops\gen_dataset_ops.py", line 2672, in iterator_get_next_sync
    _six.raise_from(_core._status_to_exception(e.code, message), None)
  File "<string>", line 3, in raise_from
tensorflow.python.framework.errors_impl.InvalidArgumentError: {{function_node __inference_Dataset_map_outer_fn_28}} pyfunc_1 returns 3 values, but expects to see 1 values.
	 [[{{node PyFuncStateless}}]] [Op:IteratorGetNextSync]

My guess is that the creation of sparsed tensor works, but somehow next() does not work because pyfunc_1 returned 3 values, which I could not debug. Could someone give me a pointer?

This issue likely happens because the sparsetensor isn't being unpacked into its 3 component values.

Hi @alextp, thank you for the reply. Right now I am using tensorflow_core.python.util.nest.pack_sequence_as to produce the output (here), but somehow _is_composite_tensor returns False, which I do not understand why
The call stack is
_sequence_like, nest.py:149
pack_sequence_as, nest.py:471
_get_defun_inputs, func_graph.py:1160
_get_defun_inputs_from_args, func_graph.py:1062
func_graph_from_py_func, func_graph.py:837
_create_graph_function, function.py:2041
_maybe_define_function, function.py:2150
_get_concrete_function_internal_garbage_collected, function.py:1848
_get_concrete_function_internal, function.py:1854
__init__, dataset_ops.py:2695
__init__, dataset_ops.py:3416
map, dataset_ops.py:1211
dataset_map_sparse_test, test.py:25          << This is the test from OP
<module>, test.py:28

That doesn't make sense to me because SparseTensor is a composite: https://github.com/tensorflow/tensorflow/blob/bb45024ae9d3df0127d1c1056b08f25e60ba601c/tensorflow/python/framework/sparse_tensor.py#L48
So it's likely something else that is going on here.


[CTL] Add missing theme properties for outlines, fix underline scaling, and RTL cell padding.

Adds some missing theme constants and drawing code for outlines.
Fixes underline size and position scaling (16.6 font units → pixels).
Fixes RichTextLabel table cell padding.

For the first commit, I wonder if it would be worth adding the font-related theme constants to Control directly instead of to each specific Control that can have text? It would then be without effect in non-text controls but would avoid duplication.
Another option being to insert a TextControl intermediate class in the inheritance tree for controls that handle text.
Just food for thought, I'm not saying that this would be a good option/better than what is done here.

Thanks!


Fix bug from #11765

…1369296362
Fixes #12051
Status
not-tested
Description
Fixes leftover misspelled keyword from #11765 , noted in https://github.com/dmwm/WMCore/pull/11765#discussion_r1369296362
Is it backward compatible (if not, which system it affects?)
MAYBE
Related PRs
External dependencies / deployment changes

test this please

Hi @amaltaro . The Pylint warnings don't seem to be blocking this PR, so was there some transient build issue? Clicking on Details  for "default" check above I only see a yellow ! for the unit tests, with one more stable test failing for another of these 3 PR's, but not for this one? Or should I go through and fix some of the code warnings (necessary at least in the other PR's with failures).
Or rebase on top of your newly merged PR?

This PR is good to go and there is nothing to be updated. About the other 2, we can follow them up in the relevant place.
Just for the sake of development workflow, I am triggering a review request (top right side) such that it can get an approval, before getting merged. Thanks again, Fredrik!


LR-217 Modal content size updates (Slight API change)

Changes

Add a size={2} option to allow vertical size increase of Modal up to screen size minus margin
Update the fullWidth option to allow horizontal size increase of Modal up to screen size minus margin

Purpose
PHC and other products have fairly complicated UI's that require more flexibility than two Modal sizes.  The existing solution is to use the full-screen option.  However, there are a couple reasons the full-screen option is not adequate for some use cases.

Overkill.  The largest non-full-screen option is still fairly small.  When a UI requires Modal only slightly larger than this, it must use the full-screen option.  The result can be a lot of extra whitespace and a suboptimal user experience.
Loss of context.  The full-screen Modal option isn't great for sub-tasks because it is more like navigating to a new page.  While editing the many options of a Survey question for example, there is no visual reminder of the main task because the whole screen is hidden.  This is compounded when you have a full-screen Modal opening another full-screen Modal.

Currently the width and height of the Modal are controlled by the values of fullWidth and size.

fullWidth={false} causes the model width to be set to a hard 480px
fullWidth={true} causes the model width to be set to a hard 600px
size={0} causes the model height to be able to expand with the content up to 256px
size={1} causes the model height to be able to expand with the content up to 480px

Content larger than the above values becomes scrollable within the Modal. This is sometimes okay for vertical scrolling but horizontal scrolling is almost always suboptimal.
BEFORE
https://user-images.githubusercontent.com/220893/114229770-489de580-9946-11eb-8288-7e7066c7fdff.mp4
Non-breaking change
The first change this PR proposes is an additional size value of 2.  When set to size={2} the height is allowed to expand with the content up to the size of the screen minus a significant margin of 3rem (48px) on the top and bottom.   This won't affect any existing apps.
Breaking change
The second change is with fullWidth={true}.  Previously this produced a hard width of 600px.  Now the width starts at a minimum of 600px, but allows the content to expand the Modal up to the width of the screen minus a margin of 3rem (48px) on the left and right.
Technically a breaking change in the API, but in practice, I'm unaware of any use of the Modal that relies on horizontal scrolling of it's content.  As mentioned, a slightly wider Modal is almost always better than horizontal scrolling.
Near full-screen
If required by the design, the PR allows for the creation of near full screen Modals that allow for a lot of UI real estate, yet retain a significant border of 48px, enough to provide a visual reminder of the main task a user is involved with.
AFTER
https://user-images.githubusercontent.com/220893/114233353-265a9680-994b-11eb-8e45-9bc0a2c59b12.mp4
Consistency
We don't want a Modal that only expands to whatever size it's content happens to be. This could produce an app with an handful of randomly sized Modals over time.  This PR attempts to keep existing defaults and several good presized options, while at the same time providing the ability create a new size.
When a specific design calls for a bunch of Modals opening at 400x750 to accommodate the UI, this makes that possible.  It is easy to limit the size of the content itself, allowing the Modal to constantly enclose itself around that content.
PHC BEFORE:
https://user-images.githubusercontent.com/220893/114750526-131c4200-9d22-11eb-8183-4cf16e1526d5.mp4
PHC AFTER
https://user-images.githubusercontent.com/220893/114750685-41018680-9d22-11eb-97ab-de59fa0ef1ea.mp4

I will update the documentation / README in the story if we think this API change is good.

Sounds like the discussion went well? Please don't merge with my approval alone, just wanting to show support as this LGTM!

Yes, I was being a bit conservative in trying not to make breaking changes.  We have come a long way since this was originally created and want to move forward with prop changes that make more sense.

I'm going to close this for now since I don't have time to work on it.  Will reopen when I get back to it.


has_many_through select not working properly

Using the same example as the documentation where we have 3 tables countries, users and posts
countries
    id - integer
    name - string

users
    id - integer
    country_id - integer
    name - string

posts
    id - integer
    user_id - integer
    title - string

And the same model as the documentation
from orator.orm import has_many_through

class Country(Model):

    @has_many_through(User)
    def posts(self):
        return Post

If we make a query like this
Country.with('posts').find(1).to_json()

The result will be a json with the country and post like this
{
    id: 1,
    name: foo,
    posts: [
        {
            id: foo,
            user_id: foo,
            title: foo
        }
    ]
}

For this to happend without bringing the users table columns has_many_through has this two functions that returns the columns to select living out all the parent table columns.
The problem with the get function is that add_select overriding our self._query selects or get(columns=[]) because the get is on a Post model query, so we don't have access to filter the results.
    def _get_select_columns(self, columns=None):
        """
        Set the select clause for the relation query.

        :param columns: The columns
        :type columns: list

        :rtype: list
        """
        if columns == ["*"] or columns is None:
            columns = ["%s.*" % self._related.get_table()]
        return columns + ["%s.%s" % (self._parent.get_table(), self._first_key)]

    def get(self, columns=None):
        """
        Execute the query as a "select" statement.

        :type columns: list

        :rtype: orator.Collection
        """
        if columns is None:
            columns = ["*"]

        select = self._get_select_columns(columns)

        models = self._query.add_select(*select).get_models()

        if len(models) > 0:
            models = self._query.eager_load_relations(models)

        return self._related.new_collection(models)

I have solve the problem on this pull request let me know what i need to change for this to be approved or if theres another way to do it

@alfonsocv12 have you looked into Masonite ORM?
It's basically an almost drop in replacement of Orator. https://github.com/MasoniteFramework/orm
Orator will not be receiving updated for the foreseeable future and I don't want you to waste too much time on PRs that won't get merged

Excelent, Thank you @josephmancuso

@josephmancuso You remove has_many_through from masonite orm because its better to use belongs_to_many, btw I can add the functionality if you want

Sure! Let me know if you have questions. It's useful to read the ORM white paper which explains how all the pieces of the ORM are built and how they relate with each other


Cannot find module 'slscrypt'

When attempting to decrypt I get the error in the title
const slscrypt = require('slscrypt');

slscrypt.get($SECRET_NAME);

I can temporarily fix it by using the node_modules path in the require()
const slscrypt =  require('./node_modules/serverless-crypt/dists/slscrypt');

Any update on fixing this defect?

I ended up pulling the function out of the npm files and put it in one of my own

any update on this issue ?

What is your package section of serverless.yml. If it had broad wildcard excludes, that was fixed in #13
which is in the v0.0.4 release. Tho I never tested that against nodejs, only python.


Move to Java 11

Java 8 is currently still LTS but e.g. Oracle's commercial distribution has been discontinued and the latest LTS version is 11. While e.g. AdoptOpenJDK plans to backport fixed until 2023, new features are being added to the language and it would be nice to be able to use them.
Move to Java 11 in the build configuration to allow using Java 9-11 features.

Would that mean that using Java 1.8 would not work anymore to start the DITA OT? Because we still have Oxygen installation kits with Java 1.8 bundled.

Yes, the minimum version would be 11. Note that this is for DITA-OT 4.0. There has been no discussion or decision on how many 3.x versions with Java 8 we will release.

I do not know much about the release timeline. What's the next release of the DITA OT? Will DITA OT 3.4 be released at the DITA OT Day? Or do you plan to release 4.0 there?
Because if you plan 4.0 for next year we'll probably also switch next year to Java 11 or newer (13).
We plan to release Oxygen 22 next year in January and somehow I would want it to have a bundled DITA OT which still works with Java 1.8.

Java 14 was released 2020-03-17.


Add visualization of the unit cell structure to the Material class

Simplest solution is to use 3d scatter plot in matplotlib:

use different colors for different atoms
add labels to atoms
scale size of atoms based on atomic radius
scale size of the unit cell by lattice parameters
draw the unit cell box (wireframe)

Addressed by 6fb2f3dc50dfc3228185abcac3994cacbac375b5


x/build: move off of google.golang.org/cloud

Builder is still importing google.golang.org/cloud packages, need to switch to the new cloud.google.com/go and its new pattern of client construction.
/cc @bradfitz
(I may have time to get this done later today, will assign it to me if I start.)

Also /cc @broady.

Addressed in https://go-review.googlesource.com/#/c/25464/
30d377d8a375e0c3a3b8ec0226957374e4d18edc


updates for stdlib 9.x.x

is_domain_name and all validate_* functions are deprecated, these can be replaced types.

@xepa thanks for you commit.
do you mind fixing is_hash inside manifests/log.pp (line 14) ?


VisIt menus have no icons when running on iSRD under GNOME.

The iSRD Linux image was recently updated so I installed a binary that we made on the open network. This was so we could have a linuxx86_64 binary since linuxintel is not the right platform anymore. VisIt seems to run okay and seems nice and fast. I was able to remove the "style windows" hack from the customlauncher. I did notice that menus are not displaying icons. This affects the VisIt main menus, plot and operator menus, as well as inwindow menus like the color table swatches in the Pseudocolor plot's color table widget. The lack of icons might be due to the binary I installed. Alternatively, maybe the lack of icons was due to a global GNOME setting.
-----------------------REDMINE MIGRATION-----------------------
This ticket was migrated from Redmine. As such, not all
information was able to be captured in the transition. Below is
a complete record of the original redmine ticket.
Ticket number: 1471
Status: Pending
Project: VisIt
Tracker: Bug
Priority: Normal
Subject: VisIt menus have no icons when running on iSRD under GNOME.
Assigned to: -
Category: -
Target version: -
Author: Brad Whitlock
Start: 05/22/2013
Due date:
% Done: 0%
Estimated time:
Created: 05/22/2013 03:18 pm
Updated: 05/29/2013 12:39 pm
Likelihood: 3 - Occasional
Severity: 2 - Minor Irritation
Found in version: 2.6.2
Impact:
Expected Use:
OS: All
Support Group: DOE/ASC
Description:
The iSRD Linux image was recently updated so I installed a binary that we made on the open network. This was so we could have a linuxx86_64 binary since linuxintel is not the right platform anymore. VisIt seems to run okay and seems nice and fast. I was able to remove the "style windows" hack from the customlauncher. I did notice that menus are not displaying icons. This affects the VisIt main menus, plot and operator menus, as well as inwindow menus like the color table swatches in the Pseudocolor plot's color table widget. The lack of icons might be due to the binary I installed. Alternatively, maybe the lack of icons was due to a global GNOME setting.
Comments:

Outdated.


empty message when passing options to the pnotify() helper

Messages are not displaying when passing options to the pnotify() helper.  For the messages that work, the message is in the envelope object twice.  As the message and as an option.   For the message that doesn't work when passing an options array, there is no message in the options and it has a 'pnotify' key.
This works:
pnotify('No options','success');
This does not work:
pnotify('Options','success',["delay" => 1000]);
Result:
Flasher.getInstance().render({"envelopes":[{"notification":{"type":"success","message":"No options message","options":{"text":"No options message"}},"handler":"pnotify","created_at":"2021-08-21 14:02:41","priority":0},{"notification":{"type":"success","message":"Options message","options":{"delay":1000}},"handler":"pnotify","created_at":"2021-08-21 14:02:41","priority":0}],"scripts":[],"styles":[],"options":{"pnotify":{"delay":1000}}});

Hello @jamesj2 can you please upgrade to v0.4.10 and check if it works ?

Updated and tested.  It's working as expected.  Thanks!


Segfault in "sctp_find_alternate_net" (timer thread)

Hi,
Another segfault, this time in timer thread
usrsctp: a17109528c75d01f6372d5c30851a639684c6e99
Program terminated with signal SIGSEGV, Segmentation fault.
#0  0x000055d5bb68bf42 in sctp_find_alternate_net (stcb=0x7f46880dcdc0, net=0x7f46880e7b40, mode=0)
    at /home/user/cpp/usrsctp_lib-cpp/src/usrsctp_lib/usrsctplib/netinet/sctp_timer.c:350
350                     if (alt->ro.ro_rt == NULL) {
[Current thread is 1 (Thread 0x7f46a4e77700 (LWP 5349))]

(gdb) bt full
#0  0x000055d5bb68bf42 in sctp_find_alternate_net (stcb=0x7f46880dcdc0, net=0x7f46880e7b40, mode=0) at /home/user/cpp/usrsctp_lib-cpp/src/usrsctp_lib/usrsctplib/netinet/sctp_timer.c:350
        alt = 0x362d746e65696c63
        mnet = 0x7f46880e5ce0
        min_errors_net = 0x0
        max_cwnd_net = 0x0
        once = 0
        min_errors = -1
        max_cwnd = 0
#1  0x000055d5bb68da55 in sctp_t3rxt_timer (inp=0x7f4688083a30, stcb=0x7f46880dcdc0, net=0x7f46880e7b40) at /home/user/cpp/usrsctp_lib-cpp/src/usrsctp_lib/usrsctplib/netinet/sctp_timer.c:906
        alt = 0x0
        win_probe = 0
        num_mk = 21973
        num_abandoned = -1139085240
#2  0x000055d5bb6a8e1d in sctp_timeout_handler (t=0x7f46880e7c70) at /home/user/cpp/usrsctp_lib-cpp/src/usrsctp_lib/usrsctplib/netinet/sctputil.c:1893
        tv = {tv_sec = 0, tv_usec = 0}
        inp = 0x7f4688083a30
        stcb = 0x7f46880dcdc0
        net = 0x7f46880e7b40
        tmr = 0x7f46880e7c70
        op_err = 0x0
        upcall_socket = 0x55d5bc10fb80
        type = 1
        i = 0
        secret = 0
        did_output = true
        released_asoc_reference = true
#3  0x000055d5bb64b9dd in sctp_handle_tick (elapsed_ticks=10) at /home/user/cpp/usrsctp_lib-cpp/src/usrsctp_lib/usrsctplib/netinet/sctp_callout.c:172
        c = 0x7f46880e7c70
        c_func = 0x55d5bb6a8adc <sctp_timeout_handler>
        c_arg = 0x7f46880e7c70
#4  0x000055d5bb64baa7 in user_sctp_timer_iterate (arg=0x0) at /home/user/cpp/usrsctp_lib-cpp/src/usrsctp_lib/usrsctplib/netinet/sctp_callout.c:214
        amount = {tv_sec = 0, tv_nsec = 10000000}
        remaining = {tv_sec = 0, tv_nsec = 10000000}
#5  0x00007f46abdd2fa3 in start_thread (arg=<optimized out>) at pthread_create.c:486
        ret = <optimized out>
        pd = <optimized out>
        now = <optimized out>
        unwind_buf = {cancel_jmp_buf = {{jmp_buf = {139941391070976, -1869884875519692240,<PHONE_NUMBER>35230,<PHONE_NUMBER>35231,<PHONE_NUMBER>70976, 0,<PHONE_NUMBER>945334832,<PHONE_NUMBER>796337712}, mask_was_saved = 0}}, priv = {pad = {0x0, 0x0, 0x0, 0x0}, data = {prev = 0x0,
              cleanup = 0x0, canceltype = 0}}}
        not_first_call = <optimized out>
#6  0x00007f46aae9d4cf in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:95
No locals.

(gdb) l -
335             for (;;) {
336                     alt = TAILQ_NEXT(mnet, sctp_next);
337                     if (alt == NULL) {
338                             once++;
339                             if (once > 1) {
340                                     break;
341                             }
342                             alt = TAILQ_FIRST(&stcb->asoc.nets);
343                             if (alt == NULL) {
344                                     return (NULL);
(gdb) l
345                             }
346                     }
347     #if defined(__FreeBSD__) && !defined(__Userspace__)
348                     if (alt->ro.ro_nh == NULL) {
349     #else
350                     if (alt->ro.ro_rt == NULL) {
351     #endif
352                             if (alt->ro._s_addr) {
353                                     sctp_free_ifa(alt->ro._s_addr);
354                                     alt->ro._s_addr = NULL;

(gdb) p alt
$1 = (struct sctp_nets *) 0x362d746e65696c63
(gdb) p *alt
Cannot access memory at address 0x362d746e65696c63

I think this may be related to 1st point in https://github.com/sctplab/usrsctp/issues/553
Valgrind snip from mentioned issue:
Thread 7 SCTP timer:
Invalid read of size 8
   at 0x18E1B4: sctp_find_alternate_net (sctp_timer.c:350)
   by 0x18FCC6: sctp_t3rxt_timer (sctp_timer.c:906)
   by 0x1AB08E: sctp_timeout_handler (sctputil.c:1893)
   by 0x14DC4E: sctp_handle_tick (sctp_callout.c:172)
   by 0x14DD18: user_sctp_timer_iterate (sctp_callout.c:214)
   by 0x4863FA2: start_thread (pthread_create.c:486)
   by 0x55BE4CE: clone (clone.S:95)
 Address 0xad231e0 is 208 bytes inside a block of size 632 free'd
   at 0x48369AB: free (vg_replace_malloc.c:530)
   by 0x185E6B: sctp_remove_net (sctp_pcb.c:5193)
   by 0x185EEA: sctp_del_remote_addr (sctp_pcb.c:5227)
   by 0x1BDF64: sctp_process_asconf_delete_ip (sctp_asconf.c:437)
   by 0x1BEE39: sctp_handle_asconf (sctp_asconf.c:759)
   by 0x15B32B: sctp_process_control (sctp_input.c:5512)
   by 0x15C3AC: sctp_common_input_processing (sctp_input.c:5933)
   by 0x1BB230: recv_function_raw (user_recv_thread.c:427)
   by 0x4863FA2: start_thread (pthread_create.c:486)
   by 0x55BE4CE: clone (clone.S:95)
 Block was alloc'd at
   at 0x483577F: malloc (vg_replace_malloc.c:299)
   by 0x184161: sctp_add_remote_addr (sctp_pcb.c:4440)
   by 0x18A45E: sctp_load_addresses_from_init (sctp_pcb.c:7156)
   by 0x153825: sctp_process_cookie_new (sctp_input.c:2322)
   by 0x15483C: sctp_handle_cookie_echo (sctp_input.c:2888)
   by 0x15AE8D: sctp_process_control (sctp_input.c:5383)
   by 0x15C3AC: sctp_common_input_processing (sctp_input.c:5933)
   by 0x1BB230: recv_function_raw (user_recv_thread.c:427)
   by 0x4863FA2: start_thread (pthread_create.c:486)
   by 0x55BE4CE: clone (clone.S:95)

I'm able to reproduce the use using a syzkaller reproducer found here: Fatal trap 12: page fault in sctp_find_alternate_net.


jax.lax.scan for recurrent models on very long sequences: can it be done?

I'm curious if jax.lax.scan can reliably deliver high speeds on GPU/TPU for long sequences, in the range of 10k+ samples. For example, multiple seconds of audio will be 30k+ samples. I want to know if we can expect recurrent-ish models written in JAX to be as fast or faster than CUDA RNNs.
I've been finding that hardware acceleration with my scan-based code either causes a slowdown (GPU) or makes a very minor difference in speed (TPU). For my training loop I got these results:

Device
Speed

CPU
3.4 it/s

GPU
0.8 it/s

TPU
3.6 it/s

These functions represent the bulk of what my code is doing — I'd be curious to know if there's something more efficient than dynamic slicing and dynamic updates.
sample_length = 10_000
receptive_field = 50

def recurrent_forward(args, t):
  [nkey, memory, params] = args
  memory_slice = jax.lax.dynamic_slice(memory, [t], [receptive_field])
  x = model.apply({'params': params}, memory_slice, t)
  memory = jax.lax.dynamic_update_index_in_dim(memory, x, t, 0)
  return [nkey, memory, params]

def unroll_recurrent_forward(nkey, memory, params):
  [nkey, memory, params], _ = jax.lax.scan(recurrent_forward, [nkey, memory, params],
                                           np.arange(sample_length))
  return memory

@jekbradbury

@jekbradbury

Some more info:

I found that removing a call to jax.random.split inside each recurrent step actually increased the speed 50%
I reimplemented my model in TensorFlow and modeled it with tf.function and tf.while_loop, and it is slower than my model in JAX. This makes me think that custom recurrent loops are just hard to write in general?

Related issue: https://github.com/google/jax/issues/2491

The code snippet you posted isn't self-contained enough; we'd need either your model.apply code (presumably a Flax neural net) or a representative replacement for it.

@noahtren thanks for raising this. Let's try to sort out what's going on!
The most helpful thing would be if you could grab a profile and share it. Does that sound feasible?
(A priori, the slicing and updating you're doing looks like the right way to write things!)

The code snippet you posted isn't self-contained enough; we'd need either your model.apply code (presumably a Flax neural net) or a representative replacement for it.

Here's a gist of the full forward pass. I refactored the earlier code, now using Flax's lifted transformations (nn.scan and nn.remat, which saves a lot of memory,) but speed performance is roughly the same as before.
You'll see that I'm doing a couple dynamic slices and one dynamic update for each iteration. Besides that, the ops are mainly MLPs and reshaping/reduce operations. The latent_code is provided by another convolutional model that runs very quickly. Interestingly, increasing num_modules (and thus the size of some MLP layers) does slow this down quite a bit, which surprised me because I imagined that it would only effect memory usage, and not speed.

@noahtren thanks for raising this. Let's try to sort out what's going on!
The most helpful thing would be if you could grab a profile and share it. Does that sound feasible?
(A priori, the slicing and updating you're doing looks like the right way to write things!)

Thanks! I can profile this for sure, what's the easiest way to share a profile? (I could share a path on a public cloud bucket with my TensorBoard logs, which are <1GB.)


Unit tests exist for Windows specific functionality

Is this intended to cover more than what is described here? https://github.com/cyberark/conjur-puppet/issues/28

Nope :smile:.  I'm going to mark this as duplicate.


sidecar: change restore.sh to go codes #292

What type of PR is this?

/enhancement
Which issue(s) this PR fixes?

Fixes #292
What this PR does?
Summary:
Special notes for your reviewer?

lgtm


Throwing generic error messages from MIMOTO and handiling that in INJI

Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.

@Sujithbn Please share the list of error codes and messages that can be used for reference.

@MonobikashDas: Could you please share the list of error codes. Swati can provide the list of error messages.


Fix parallel build

Probably needs to be closed without merging.  The companion PR #625 breaks all Cygwin-based builds.  PR #631 reverts it (All Cygwin users are dead in the water until that change is reverted).

Probably needs to be closed without merging. The companion PR #625 breaks all Cygwin-based builds. PR #631 reverts it (All Cygwin users are dead in the water until that change is reverted).

@patacongo but the inclusion sequence issue doesn't exist in apps folder, since INCDIR here is defined after $(TOPDIR)/Make.defs is fully included.

Don't merge it, I will update with the better method.


initial dialog

The initial dialog is confusing. It asks for the session name and then says "e.g. your_name". Why are you asking for the session name? Can't you get that from the file? And it looks like you are asking for my name (Ben).

The session name can be the user name or any other name the user wants to associate with the item he/she will include and save. This is related to #15

You can, of course, just press enter and leave it blank

@bendichter can I close this?


[SR-538][Stdlib][3.0] Updated constraints and docs for popFirst/popLast. Added tests.

3.0 piece of SR-538. When/if this is accepted, I can backport the changes to 2.2.

Thank you!
Could you add the tests to stdlib/private/StdlibUnittest/CheckCollection.swift instead?  This way, they will get run on all collections, not just arrays, and it will check that the implementation does indeed work on a strict collection implementation that has minimal guarantees.
popFirst() tests will go into addForwardCollectionTests.  You can model them after removeFirst() tests (and even reuse the same data set + add a test for an empty collection).
popLast() tests will go into addBidirectionalCollectionTests().  These would be similar to removeLast() tests (again, you can reuse the same dataset).

I moved the tests as suggested and rebased against latest master.

Thank you!  Tested on OS X, tests pass.  Merging.

When/if this is accepted, I can backport the changes to 2.2.

This would be awesome, thank you!

Backport in #1086


[PLAT-11248] Add bundle install to pipeline file where needed

Goal
Install required gems on ci steps where they are needed.
Changeset
Add bundle install to ci steps where needed.
Testing
Covered by CI

I don't think this is needed anymore as we now test against the v48 to v50 versions of Expo.  Perhaps there are in-flight branches of bugsnag-js that just ned updating now?

I don't think this is needed anymore as we now test against the v48 to v50 versions of Expo. Perhaps there are in-flight branches of bugsnag-js that just ned updating now?

I may have just done it out of habit. I'll check if it is still needed

I don't think this is needed anymore as we now test against the v48 to v50 versions of Expo. Perhaps there are in-flight branches of bugsnag-js that just ned updating now?

@twometresteve it looks like nightly runs are still run against v47 so we should probably update it so that it doesn't break anything


Cirrus: imgprune fixes

One suggestion from Ed
One un-workaround (issue fixed in Cirrus-CI since 78b0773f7d544e4fd28fc786f9a754e6352f8307

Working example run: https://cirrus-ci.com/task/4881090025619456

(rebase & force-push)

/approve

/lgtm

/retest


Granular freeze

Initial attempt at this

Closing as stale. We made some progres on this topic after https://github.com/opencollective/opencollective/issues/3593