Clarity on quick button options

On the app it says "Press during 0.5s to  apply calc" a better prompt would be something like: "Press and hold for 0.5s to apply".
This is just for clarity and improvement.

Thanks to your suggestion. That message will change next minor update  x~x

Update that string on 1.7.1


feat: add method to query supported RF regions

With the upcoming support for EU Long Range, the list of supported regions can differ between different 700/800 series controllers.
This PR adds a method getSupportedRFRegions, which can be used to dynamically determine which regions to show to a user.

@robertsLando Z-Wave JS UI should use this, like we did with Zniffer.

@zwave-js-bot automerge

like we did with Zniffer.

What did we do for Zniffer? I actually parse them statically:
https://github.com/zwave-js/zwave-js-ui/blob/f386c4f94c4727578919c2277a81f47b62ed6155/src/lib/items.js#L4-L10
https://github.com/zwave-js/zwave-js-ui/blob/f386c4f94c4727578919c2277a81f47b62ed6155/src/lib/items.js#L12-L18

I thought we talked about that. Zniffer should definitely have a dynamic frequency selection, because they are totally different for 500 series based Zniffers:


two directories for looped Thai

fonts/ in this repo contains both NotoLoopedThai/ and NotoSansThaiLooped/.... I think the files in them are identical other than the name change

Sans Thai Looped is the correct one. ("Noto Style Script [Variant]") I'll remove the other.

thanku :3

Noto Looped Thai is still showing up in index.html. Stripping Noto Looped Thai out of state.json and noto.json and rerunning build-site.py fixes this, but rerunning gather.py puts Noto Looped Thai back into those json files, so I'm guessing the issue is https://github.com/notofonts/thai still has a Noto Looped Thai tag.


wip: extract api in different project so that it can be reused

Obsolete, will open a new one using halkyon.io/api instead.


Run EncyclopeDia in Skyline

Still some WIP but it's testable. I added a new "EncyclopeDia Search" option under Peptide Search in File/Import.

added EncyclopeDiaSearchDlg and EncyclopeDiaHelpers for running its search engine inside Skyline

refactored Prosit model code so it could be used without the Skyline document model
refactored DDASearchControl as a base class (SearchControl) for EncyclopeDiaSearchControl and DDASearchControl
fixed max_occurs for peptide_integration Skyline_Current.xsd
added hard-link utility functions in UtilIO because EncyclopeDia requires input files to be in the same directory and tries to process all files in that directory even when only a subset is desired; hard-links are of course less space-wasting than actually copying the data
updated C# version in Shared/Common to 8.0
added SqliteOperations.DumpTable() for quick testing of SQLite file contents
changed ProcessRunner to take an optional outputAndExitCodeAreGood delegate for detecting errors in processes which don't report exit codes properly (for example, EncyclopeDia!)
added new option in KeyValueGridDlg for setting additional parameters from a list of valid values (which translates to a drop-down style ComboBox)

@brendanx67 @nickshulman
I'm thinking about adding a SHA-1 hash column to SpectrumSourceFiles in BiblioSpec so we can confidently skip regenerating a blib if the input file(s) and settings haven't changed. This would be particularly useful for this PR for unit and functional testing of the Prosit stuff so I can cache the Prosit output and not have to call Prosit if the input file (in this case a CSV file) hasn't changed. Especially for the perf test we don't want to actually run that on Prosit every night for every PR. But maybe do it once a week on master.
If you don't think that's a good idea I can implement an alternative like just assuming the file hasn't changed if it exists during testing (similar to how SimpleFileDownloader changes its behavior when functional testing).

Yeah. Never put a decimal point in quotes in a test.
On Tue, Jun 20, 2023 at 12:26 PM nickshulman @.***>
wrote:

@.**** commented on this pull request.
In pwiz_tools/Skyline/TestFunctional/EncyclopeDiaSearchTest.cs
https://github.com/ProteoWizard/pwiz/pull/2386#discussion_r1235513223:

           File.Delete(TestFilesDir.GetTestPath("pan_human_library_690to705-z3_nce33-prosit-5950B898E945AE52AD86D9CE06220EE.blib"));

       RunUI(searchDlg.NextPage); // now on Prosit settings

       Settings.Default.PrositIntensityModel = PrositIntensityModel.Models.First();

       Settings.Default.PrositRetentionTimeModel = PrositRetentionTimeModel.Models.First();

       RunUI(() =>

       {

           searchDlg.DefaultCharge = 3;

           searchDlg.DefaultNCE = 33;

           searchDlg.MinCharge = 2;

           searchDlg.MaxCharge = 3;

           searchDlg.MinMz = 690;

           searchDlg.MaxMz = 705;

           searchDlg.ImportFastaControl.MaxMissedCleavages = 2;

           searchDlg.SetAdditionalSetting("PercolatorTrainingFDR", "0.15");

This line seems to fail in foreign languages.
I'm thinking maybe it should be:
searchDlg.SetAdditionalSetting("PercolatorTrainingFDR",
0.15.ToString(CultureInfo.CurrentCulture));
or maybe "SetAdditionalSetting" should be changed to take an object
instead of a string.
—
Reply to this email directly, view it on GitHub
https://github.com/ProteoWizard/pwiz/pull/2386#pullrequestreview-1488544309,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACYBWUGIHKQLV2DQQ2YHFHTXMHFJZANCNFSM6AAAAAASCYPKW4
.
You are receiving this because you were mentioned.Message ID:
@.***>

Let me know if your testing on macs2 works and I'll squash it in.

I got a failure in both TestEncyclopeDiaSearchTutorial and TestEncyclopeDiaSearchTutorialFullDataset:
Microsoft.VisualStudio.TestTools.UnitTesting.AssertFailedException: Assert.Fail failed. Microsoft.VisualStudio.TestTools.UnitTesting.AssertFailedException: Assert.Fail failed. Expected target counts <557, 5734, 5734, 40799> do not match actual <557, 5733, 5733, 40777>.
   at Microsoft.VisualStudio.TestTools.UnitTesting.Assert.HandleFailure(String assertionName, String message)
   at Microsoft.VisualStudio.TestTools.UnitTesting.Assert.Fail(String message, Object[] parameters)
   at TestPerf.EncyclopeDiaSearchTutorialTest.ValidateTargets(Int32[]& targetCounts, Int32 proteinCount, Int32 peptideCount, Int32 precursorCount, Int32 transitionCount, String propName) in D:\Users\nicksh\git_d\sky_impenc2\pwiz_tools\Skyline\TestPerf\EncyclopeDiaSearchTutorialTest.cs:line 480
   at TestPerf.EncyclopeDiaSearchTutorialTest.<>c__DisplayClass10_0.<DoTest>b__12() in D:\Users\nicksh\git_d\sky_impenc2\pwiz_tools\Skyline\TestPerf\EncyclopeDiaSearchTutorialTest.cs:line 336
   at pwiz.SkylineTestUtil.AbstractFunctionalTest.<>c__DisplayClass30_0.<RunUI>b__0() in D:\Users\nicksh\git_d\sky_impenc2\pwiz_tools\Skyline\TestUtil\TestFunctional.cs:line 300
   at System.Windows.Forms.Control.MarshaledInvoke(Control caller, Delegate method, Object[] args, Boolean synchronous)
   at System.Windows.Forms.Control.Invoke(Delegate method, Object[] args)
   at TestPerf.EncyclopeDiaSearchTutorialTest.DoTest() in D:\Users\nicksh\git_d\sky_impenc2\pwiz_tools\Skyline\TestPerf\EncyclopeDiaSearchTutorialTest.cs:line 338
   at pwiz.SkylineTestUtil.AbstractFunctionalTest.RunTest() in D:\Users\nicksh\git_d\sky_impenc2\pwiz_tools\Skyline\TestUtil\TestFunctional.cs:line 1879
   at pwiz.SkylineTestUtil.AbstractFunctionalTest.WaitForSkyline() in D:\Users\nicksh\git_d\sky_impenc2\pwiz_tools\Skyline\TestUtil\TestFunctional.cs:line 1824
------------------------------------------------------

OK good catch. TC perftests aren't running with the Prosit production config. I'll fix that and the tests.


feat(wasm): bump the spin shim the latest version from SpinKube

What type of PR is this?
/kind feature

What this PR does / why we need it:
Unlike #4695 , this PR does not remove the existing Wasm shims, including Slight and WWS. Instead, it only bumps the spin shim to the latest version v0.15.1 released from the SpinKube organization (instead of Deislabs). It will replace #4695
Which issue(s) this PR fixes:

Fixes #
Requirements:

[ ] uses conventional commit messages

[ ] includes documentation
[ ] adds unit tests
[ ] tested upgrade from previous version

Special notes for your reviewer:
Release note:
none

@cameronmeissner Could you please take a look at this PR? 🙏

Pull Request Test Coverage Report for Build<PHONE_NUMBER>1
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 71.005%

Totals

Change from base Build<PHONE_NUMBER>5:
0.0%

Covered Lines:
2564

Relevant Lines:
3611

💛  - Coveralls

@cameronmeissner I don't think I have access to update this branch, and @Mossaka is out on holiday. I have authored a test which you can see here: https://github.com/Azure/AgentBaker/compare/jiazho/bump-spinkube...devigned:bump-spinkube?expand=1.
I'm happy to provide the test as a follow on PR if we were able to get this PR through. If this PR isn't going to make it, I'm happy to open a new PR with this PR's content + the test. Which would you prefer?

@cameronmeissner I noticed that the AgentBaker E2E tests failed for Wasm in the CI. I tried to run the e2e tests on local, but I got
--- FAIL: Test_azurelinuxv2 (4.05s)
panic: failed to get RG "abe2e-westus3": HEAD https://management.azure.com/subscriptions/8ecadfc9-d1a3-4ea4-b844-0d9f87e4d7c8/resourcegroups/abe2e-westus3
        --------------------------------------------------------------------------------
        RESPONSE 403: 403 Forbidden
        ERROR CODE UNAVAILABLE
        --------------------------------------------------------------------------------
        Response contained no body
        --------------------------------------------------------------------------------
         [recovered]
        panic: failed to get RG "abe2e-westus3": HEAD https://management.azure.com/subscriptions/8ecadfc9-d1a3-4ea4-b844-0d9f87e4d7c8/resourcegroups/abe2e-westus3
        --------------------------------------------------------------------------------
        RESPONSE 403: 403 Forbidden
        ERROR CODE UNAVAILABLE
        --------------------------------------------------------------------------------
        Response contained no body
        --------------------------------------------------------------------------------

I think that's because my azure account does not have the necessary permissions. Could there be something I didn't set up on my devbox properly? Do you have any instructions on how to run E2E tests locally, including setting up the environment and authenticate?

I might have missed something but where can I see the VHD content tests failures? I was expecting the Test Scan and Cleanup step in AKS Linux VHD Build pipeline would fail for the "always fail" test I committed, but it's not. @cameronmeissner

@ganeshkumarashok I realized that self-contained/bootstrap_install.sh also defines a downloadContainerdWasmShims() and it's being used in self-contained/bootstrap_main.sh. Should I update this one as well? What's the usage of it?

Xinhe just informed me that the SpinKube shim v0.15.1 was pushed to MCR. I've updated the PR (in cse_install.sh) to reflect that change. I've also done a bit refactoring to make the code more readable. Please give it a review if you are free.
@cameronmeissner, @AlisonB319

It looks like all the tests, including the e2e Wasm validators, were passing, except 1804 builds. I've update the self-containerd to have the mirrored logic from cse_install.


Update google-cloud-bigtable to 2.41.0

About this PR
📦 Updates com.google.cloud:google-cloud-bigtable from 2.5.3 to 2.41.0
📜 GitHub Release Notes - Version Diff
Usage
✅ Please merge!
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

⚙ Adjust future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.google.cloud", artifactId = "google-cloud-bigtable" } ]

Or, add this to slow down future updates of this dependency:
dependencyOverrides = [{
  pullRequests = { frequency = "30 days" },
  dependency = { groupId = "com.google.cloud", artifactId = "google-cloud-bigtable" }
}]

labels: library-update, early-semver-minor, semver-spec-minor, commit-count:1

Superseded by #331.


feat(invitation): added invitation id and api for invitation details

Description

Added Invitation Id in response of ExecuteInvitation api i.e POST: api/administration/invitation
Created a new api to get the application id and company id by using invitation id i.e
GET: api/administration/invitation/{invitationId}

Why
In automation and CRM for self registration (Hubspot) there was no reference id to track the Invitation .
So now by this invitation Id they can call the newly created api to get the Application id and Company Id
Issue
#834
Checklist
Please delete options that are not relevant.

[x] I have followed the contributing guidelines
[x] I have performed a self-review of my own code
[x] I have successfully tested my changes locally
[x] I have added tests that prove my changes work
[x] I have checked that new and existing tests pass locally with my changes
[x] I have commented my code, particularly in hard-to-understand areas

@dhiren-singh-007 I think we can close this PR because of the merge of: #963 correct?


Certain color aliases are missing in the UdonConsole

So far I've noticed at least silver and magenta aren't being color converted properly.

Reference: https://docs.unity3d.com/2018.3/Documentation/Manual/StyledText.html

UdonConsole was recently (#16) migrated to use TextMeshProUGUI instead of native Unity UI Text components, and not all rich color tags are supported anymore.
Wiki page: https://github.com/Varneon/VUdon-Logger/wiki/UdonConsole:-Supported-Rich-Text-Color-Tags


Update to Twilight 0.11

The biggest change would be permissions v2, which I sadly couldn't figure out

New version 0.11.0 has just been released.


Create c2lqd.txt

@ndcung Please provide us with:

the school official website URL
the school street address, including city and country
a proof, which shows that the school recognizes the domain you are submitting as an official email domain for the students.

You can provide us with any of the following as the proof:

A copy of instruction (a hard-copy or electronic one) given to you by the school that tells you to use an email address in this email domain
An URL of a page at the official school website that says that students use this email domain
A reference to a recognized journal where an article was published by someone from the school, with explicit affiliation to the school, and e-mail address from the email domain
A WHOIS data that link the email domain to the school somehow
A copy of official letter from the school representative saying that the school provides e-mail addresses in this domain to the students
Anything else that can prove this fact in similar way, i.e. can be verified independently.

@ndcung You might miss my previous comment. Please react.

@ndcung I am sorry, this request remains unanswered for a while. I have to close it without merging. Please reopen and comment if you have an update regarding the aforementioned issue. Thank you for your understanding!


cleanup DrawerLayoutAndroid

Summary
This PR removes Java reflection use in ReactDrawerLayoutManager.java because we all use AndroidX and setDrawerElevation is available. Also adds NonNull annotations
Changelog
[Android] [Changed] - cleanup DrawerLayoutAndroid
Test Plan
RNTester is working as expected.

cc @makovkastar since you have a diff changing this file too


Fix logic around srv records at other code path

This fixes the code path for when SRV result is immediate

I did verify this fix in production.  Thanks for the review!

Cherry-picked to v10.0.x


Don't add loop devices with osd_auto_discovery

Discovered this as a problem when deploying openstack-ansible,
during container creation, /var/lib/machines.raw gets mounted,
which leads to a loop0 device with sectors != "0"
Signed-off-by: Erik Berg<EMAIL_ADDRESS>

Closing this in favor of #3595


Create CODE_OF_CONDUCT.md

code of conduct added, please add if suitable.

Thanks @zaid-syed
all the issues are handled from our main repo amplication/amplication so I am not sure if we need the file in this repo.


:bomb: Bug: There is an error when destroy the devstream

What Happened?
Background:
deploy the devstream according to the quick start
env:
windows WSL ubuntu 18.04
There is an error when I try to destroy the devstream by following the quick start step by step, and the previous step is successful.
`[going@dev devstream-quickstart]$ dtm destroy -f quickstart.yaml
2022-06-28 20:05:42 ℹ [INFO]  Destroy started.
2022-06-28 20:05:42 ℹ [INFO]  Got Backend from config: local
2022-06-28 20:05:42 ℹ [INFO]  Using local backend. State file: devstream.state.
2022-06-28 20:05:42 ℹ [INFO]  Tool (githubactions-golang/default) will be deleted.
2022-06-28 20:05:42 ℹ [INFO]  Tool (github-repo-scaffolding-golang/default) will be deleted.
Continue? [y/n]
Enter a value (Default is n): y
2022-06-28 20:05:44 ℹ [INFO]  Start executing the plan.
2022-06-28 20:05:44 ℹ [INFO]  Changes count: 2.
2022-06-28 20:05:44 ℹ [INFO]  -------------------- [  Processing progress: 1/2.  ] --------------------
2022-06-28 20:05:44 ℹ [INFO]  Processing: (githubactions-golang/default) -> Delete ...
2022-06-28 20:05:46 ℹ [INFO]  Deleting GitHub Actions workflow pr-builder.yml ...
2022-06-28 20:05:46 ℹ [INFO]  -------------------- [  Processing aborted.  ] --------------------
2022-06-28 20:05:46 ℹ [INFO]  githubactions-golang/default-Delete -> DELETE https://api.github.com/repos/jxs1211/go-webapp-devstream-demo/contents/.github/workflows/pr-builder.yml: 404 Not Found []
2022-06-28 20:05:46 !! [ERROR]  Destroy failed => some error(s) occurred during plugins destroy process.
[going@dev devstream-quickstart]$
`
How to Reproduce?
exec the destroy cmd after apply action is successful
Anything else
some detail about my deployment:
`
[going@dev devstream-quickstart]$ tree
.
├── devstream.state
├── quickstart.yaml
└── quickstart.yaml.bak
0 directories, 3 files
[going@dev devstream-quickstart]$ cat quickstart.yaml
core config
varFile: "" # If not empty, use the specified external variables config file
toolFile: "" # If not empty, use the specified external tools config file
state: # state config, backend can be local or s3
backend: local
options:
stateFile: devstream.state

plugins config
tools:

name: github-repo-scaffolding-golang
instanceID: default
options:
owner: jxs1211
repo: go-webapp-devstream-demo
branch: main
image_repo: jxs1211/go-webapp-devstream-demo
name: githubactions-golang
instanceID: default
dependsOn: ["github-repo-scaffolding-golang.default"]
options:
owner: jxs1211
repo: go-webapp-devstream-demo
language:
name: go
version: "1.18"
branch: main
build:
enable: True
test:
enable: True
coverage:
enable: True
docker:
enable: False
[going@dev devstream-quickstart]$
`

DevStream Version
latest

I look at the code, I suspect is in workflow_helper.go file generateGitHubWorkflowFileByName() method have some problem in here, cause I couldn't find the corresponding GitHub plugin cause error when the deleteFile, but I'm not sure if my guess is correct, because I can't debug locally

@hxcGit please take a look.
@jxs1211 thanks for raising this issue!

I'm not certain whether this is a reproducible problem. I didn't encounter this error in my environment: mac-arm64 , devstream: v0.7.0. Could you provide more information?

So what's the problem

@azhsmesos  Can you reproduce this problem? It works well in my machine.

I also met this error when I run "./dtm delete -f quickstart.yaml" on SUSE
2022-07-15 04:13:27 ℹ [INFO]  -------------------- [  Processing aborted.  ] --------------------
2022-07-15 04:13:27 ℹ [INFO]  githubactions-golang/default-Delete -> DELETE https://api.github.com/repos/calvinxu/go-webapp-devstream-demo/contents/.github/workflows/pr-builder.yml: 404 Not Found [].
2022-07-15 04:13:27 !! [ERROR]  Delete error: some error(s) occurred during plugins delete process.
-- github.com/devstream-io/devstream/cmd/devstream/delete.go 26
--<EMAIL_ADDRESS>860
--<EMAIL_ADDRESS>974
--<EMAIL_ADDRESS>902
-- github.com/devstream-io/devstream/cmd/devstream/main.go 100

@calvinxu  Many thanks to your feedback! This is caused by calvinxu/go-webapp-devstream-demo/contents/.github/workflows/pr-builder.yml not being found. Please check whether you delete this repo or this file manually before you use dtm delete. If not, please check whether this problem is reproducible and offer more information, such as os, dtm version and so on.

@jxs1211 You are missing the delete repo/delete repo content permission for GITHUB_TOKEN.
We recently encountered a similar issue #882, and after one-on-one communication on WeChat, we found that the problem lies in incomplete GITHUB_TOKEN permissions.
The description of token permissions in our quickstart doc only has an external link, and we will improve the quickstart doc. Thank you for your hard work and follow up.❤


[DOC] : ENHANCE CODE OF CONDUCT.md file

Description:
We are proposing enhancements to our project's Code of Conduct to foster a more inclusive and respectful community environment. These enhancements aim to clarify expectations, provide guidance on appropriate behavior, and ensure that all contributors feel safe and valued.
##we want to make it much more informative and error free document.
Objectives:
Clarification: Review and clarify existing language to eliminate ambiguity and ensure understanding.
Expansion: Incorporate additional guidelines to address emerging issues and promote inclusivity.
Accessibility: Ensure that the Code of Conduct is easily accessible and understandable for all contributors, including newcomers.
Enforcement: Strengthen enforcement mechanisms to address violations promptly and fairly.
Community Involvement: Seek input from the community to ensure that the Code of Conduct reflects diverse perspectives and needs.
Approach:
Review
Research
Collaboration
Drafting
Review and Approval
Expected Outcome:
An updated Code of Conduct that provides clear guidelines for expected behavior, fosters a welcoming and inclusive community environment, and strengthens mechanisms for enforcing the Code of Conduct effectively.
Thank You.

Kindly , assign and label its level for this issue.(GSSOC and level).

Thanks @anglerfishlyy but this area we are not working
You can pick another problem like creating page or somefixing bugs !


Use API to get issue map of aliases to issue keys

It should be possible to use /rest/api/latest/issue/createmeta to look up the data needed for the issues map for a given project. This would prevent the need to manually look up and hard code the values in the~/.track.yaml config file.

I am not sure I understand what you mean. The aliases are purely meant as an easy, personal mnemonic for issues. Even if they were shared in a project, I would not see how to use the resource you mentioned. Then again, I am not so familiar with JIRAs API.


add //%flag to hide entry from TS autocompletion

Some methods are only for blocks really, especially ones using enums

//%hidden flag added


Code Cleanup: Organising folder structure into frontend, backend and README

Is your feature request related to a problem? Please describe.
The README.md seems to indicate that the folder structure is as follows
PassOP/
├── backend/ # Backend Node.js application
│   ├── server.js # Main server file
│   ├── .env # Environment configuration (not included in repo)
│   ├── package.json # Backend dependencies
│   └── ... # Other backend files
├── frontend/ # React frontend application (using Vite)
│   ├── src/ # React components and files
│   ├── public/ # Public static files
│   ├── package.json # Frontend dependencies
│   └── vite.config.js # Vite configuration file
├── README.md # This file
└── ...
Whereas, currently that is not the case. This caused me a lot of confusion initially, and I'd imagine it is the same for many people.
Describe the solution you'd like
I would like to first organise the files into frotnend and backend folder, and then update the README to match this.
Describe alternatives you've considered
We could simply change the README to fit the current structure,, but the current structure is not the most widely seen uniform seperation of concerns of frontend and backend, and so it is not ideal.
Additional context
Will add any that is needed by admin. As of now, seems self explanatory.

@jinx-vi-0 Do you mind setting up relevant labels and assigning it to me? Although I have some experience in dev, It's always been solo projects, never really open source. Looking forward to do my best here!

Okay, sounds good! Just make sure to check every endpoint and configuration to ensure everything works perfectly. Also, please update the README to reflect any changes you make. Looking forward to your contributions!

Will do!
In the meantime, wanted to ask if .env being in backend folder is required. Isn't it supposed to be in gitignore?
Also, as a follow up, isn't there supposed to be a single gitingore? Why does there seem to be 2?

Yeah, add the .env to gitignore and create an example.env to help contributors understand the .env structure.

@jinx-vi-0 is there any issue with the export option? It doesn't seem to work. I reverted all my changes, but it still seems to not respond?

@jinx-vi-0
Also... noted a major issue in backend.
The download doesn't happen cause of the following reason. In server.js,

This expects id as a parameter. Which means anything after "http://localhost:3000/" will be seen as the id parameter. Hence when we try to export passwords, it is unable to reach the handler:

Leading to error.
Check it out yourself. Once you comment out the /:id handler, the export option works. This is a major backend flaw. Would you like me to make an issue for the same?

@pranavajith create an issue. I'll assign it to you.

Hi, @pranavajith what's the progress status on this issue?

Marked as Up-To-Grab since there's no update yet.

@jinx-vi-0 Could you please assign this to me? I’ve been exploring open source projects, and I’ll do my best to contribute and resolve it.


otp: break user flow to have its own supervisor

Properly add a gen_server for the distpacher and handle_user process.
Each user process is managed by a user_sup supervisor.

New Arch

Testing

db-reset
server
client

Also, the image of this PR's description should be in some docs folder somewhere.


MSID: 98612 Version: 1 DOI: 10.1101/2<IP_ADDRESS>4949

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

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

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

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

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

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

Useful links:

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

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

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

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

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

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

[ ] Kettle is on!

warn: [mixed-citation-other-publication-flag] mixed-citation in reference (id=c60) has a publication-type='other'. Is that correct?
warn: [title-lower-case] Content of  element is entirely in lower-case case: Is that correct? 'aqp1a.1 and aqp8a.1 are differentially expressed by tip and stalk cells during sprouting angiogenesis'


Fix NuGetFeed variable check

Follow up fix for #388. We added a queue time variable NuGetFeed in that PR, but when the variable is not set, it's passing raw the string $(NuGetFeed) to publishVstsFeed instead of null, which breaks MUX-CI-Experiments pipeline since we don't set NuGetFeed there. Create a PR for the fix to unblock people from using the experiments pipeline.

Maybe it would be better if the MUX-CreateNugetPackage-Job.yml defended against that value being empty?

Maybe it would be better if the MUX-CreateNugetPackage-Job.yml defended against that value being empty?

I think

Maybe it would be better if the MUX-CreateNugetPackage-Job.yml defended against that value being empty?

I think the value being passed to publishVstsFeed parameter is string$(NugetFeed), I've tried ${{if parameters.publishVstsFeed}} and ${{if ne(parameters.publishVstsFeed,'') }}, they are both false, those checks didn't expand the variable for some reason. It only gets expanded when you actually use it, ${{parameters.publishVstsFeed}}.

Oh interesting. ok, this change is good then.


Create output folder if it doesn't exist

Currently app throws an error when output directory does not exist or it's not writable. If doesn't exist, app should create the folder on behalf of the user.

:tada: This issue has been resolved in version 1.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Uncaught Error: Undefined class constant 'XML_PATH_DISABLE_CUSTOMER_REGISTRATION'

Hello Bodak,
I tried your module in Magento 2.3.4 but it throws an error
[error] 27008#27008: *16657 FastCGI sent in stderr: "PHP message: PHP Fatal error:  Uncaught Error: Undefined class constant 'XML_PATH_DISABLE_CUSTOMER_REGISTRATION' in /var/www/html/Magento/app/code/Bodak/DisableRegistration/Plugin/RegistrationPlugin.php:54
In order to fix the issue, I updated line 32 and changed the constant to "XML_PATH_DISABLE_CUSTOMER_REGISTRATION".

Thanks!

@mazestricks ugh, thanks. Fixed.


Blank Dashboard and Discover Screen

@emertoglu commented on Thu Jan 12 2023
My scenario is as follows.
I created 2 roles[role1, role2] , 2 users[user1, user2] and 2 tenants[tenant1, tenant2] and I mapped roles to users.
I loaded the data with two different index names[index_u1-, index_u2-]
user1 index_u1-* and user2 index_u2-* have read , write manage index permissions
when i sign in with a user name and password, i see a blank screen. The Dashboard and Discover screens are blank and I can't create anything.
if I map the role of kibana_user to users, users can create dashboards, but this time they can see someone else's dashboards (they can't see the data because the indexes are different)
if I map all_access role to users, users can create dashboards, but this time they can see someone else's dashboards and data (although the indexes are different)
how can i fix the problem?

@minalsha commented on Fri Jan 13 2023
@seanneumann  can this issue be looked upon by you?

Thanks @minalsha
It looks about setup and usage by using security feature
@opensearch-project/transfer-request would you help to route to https://github.com/opensearch-project/security

@seraphjiang Thanks for mentioning the transfer team, I don't have permissions to transfer issues in this repo.  I suspect that transfer team needs to be added to the security groups and I think that the admin team might need their permissions added too - could you help make that happen?

Transfer issue required write permission for both repo, so manually create a issue and copy the content to security repo. https://github.com/opensearch-project/security/issues/2408


"genesyscloud_tf_export" to generate Routing Queue using Bullseye_rings issue

I don't know if this Repo has a specific issue format to follow, so I apologies in advance.
Hi,
I've noticed an issue with the "genesyscloud_tf_export" function when it generates an existing Routing queue using the bullseye_rings routing method.
If the queue has a Group assigned to the last Ring (which is not really a ring but "Route to all agents in queue, based on selected evaluation method." option), the routing queue terraform resource won't have the group in the generated output of the tf_export run.
Ex.:
A test queue using the bullseye rings:

Groups assigned to the last ring (a.k.a the "Route to all agents in queue, based on selected evaluation method." option):

This is the generated output of this tf_export:
resource "genesyscloud_tf_export" "export" {
  directory          = "./queue"
  export_as_hcl      = true
  log_permission_errors = true
  split_files_by_resource = true
  include_state_file = true
  enable_dependency_resolution = false
  include_filter_resources = [
     "genesyscloud_routing_queue::^test_xxxxx_group$",
   ]
}

Result (notice that there's no groups in this output whatsoever) :
resource "genesyscloud_routing_queue" "test_xxxxx_group" {
  calling_party_number    = "+undefined"
  enable_audio_monitoring = false
  media_settings_email {
    enable_auto_answer        = false
    enable_auto_dial_and_end  = false
    service_level_duration_ms = 86400000
    service_level_percentage  = 0.8
    alerting_timeout_sec      = 300
  }
  name                             = "test_xxxxx_group"
  suppress_in_queue_call_recording = true
  acw_timeout_ms                   = 8000
  acw_wrapup_prompt                = "MANDATORY_TIMEOUT"
  bullseye_rings {
    expansion_timeout_seconds = 2
  }
  bullseye_rings {
    expansion_timeout_seconds = 4
  }
  division_id = "56f815ff-dd0e-439c-a34b-425cef6b1706"
  media_settings_callback {
    enable_auto_dial_and_end  = false
    service_level_duration_ms = 20000
    service_level_percentage  = 0.8
    alerting_timeout_sec      = 30
    auto_dial_delay_seconds   = 300
    auto_end_delay_seconds    = 300
    enable_auto_answer        = false
  }
  media_settings_chat {
    enable_auto_answer        = false
    enable_auto_dial_and_end  = false
    service_level_duration_ms = 20000
    service_level_percentage  = 0.8
    alerting_timeout_sec      = 30
  }
  media_settings_message {
    enable_auto_answer        = false
    enable_auto_dial_and_end  = false
    service_level_duration_ms = 20000
    service_level_percentage  = 0.8
    alerting_timeout_sec      = 30
  }
  description              = "this is a test"
  enable_manual_assignment = false
  enable_transcription     = true
  media_settings_call {
    enable_auto_answer        = false
    enable_auto_dial_and_end  = false
    service_level_duration_ms = 20000
    service_level_percentage  = 0.8
    alerting_timeout_sec      = 8
  }
  scoring_method          = "PriorityOnly"
  skill_evaluation_method = "ALL"
  auto_answer_only        = false
  calling_party_name      = "Security"
}

However, if the group is assigned to the 2nd ring rather than the last one (3rd), it works just fine when it comes to the tf_export output of the queue.
  bullseye_rings {
    expansion_timeout_seconds = 2
  }
  bullseye_rings {
    expansion_timeout_seconds = 4
    member_groups {
      member_group_id   = "589e24a2-bcf9-4769-ba32-221d8a3b2ea2"
      member_group_type = "GROUP"
    }
  }

Is this a bug?
Thank you!

Hey Tony,
Thanks for the detailed writeup.  I will have someone on our team and research.
Thanks,
John


Dummy PR to debug #31117, please ignore.

This is a dummy PR, not intended to be merged. It contains some extra debug printing, to be executed by the Jenkins builds, intended to discover why the bug described at https://code.djangoproject.com/ticket/31117 does not occur inside Jenkins.

@felixxm, Could you perhaps reopen this PR? I am still pushing some new commits to this for testing, and when it is closed Jenkins won't build from my branch :-)
I think I've figured this out, so I just need two or three more builds to confirm.

You can run exactly the same tests locally. I don't see any reason to debug via Jenkins.

The problem I'm trying to debug occurs locally, but not on Jenkins. I'm trying to figure out why. See https://code.djangoproject.com/ticket/31117


How to resolve Auto IAM AuthN Configuration Error?

Question
A few of my users are on Macs, and they can't seem to connect to our Cloud SQL Postgresql instance using the Auth Proxy, even though it's working fine for me on Windows (and for at least one other user on an M1 Mac). We're following the instructions on this page, but connecting in a db manager using IAM credentials fails (we've tried several). The proxy indicates that it's running, but when we try to open the connection it shows that it's opening and then immediately closing connections on a few random ports, and the db manager (dbeaver CE, pgAdmin 4) says "FATAL: Cloud SQL IAM user authentication failed for user ". We've got Cloud CLI up, running, and authenticated; I've triple checked that user permissions were correct both in IAM and the SQL instance.
What are we missing?
I'll note by the way that this version of the proxy does not have the command line option "-enable_iam_login". Instead, it seems to require to use "--gcloud-auth".
Code
No response
Additional Details
No response

Can you show me how you're invoking the proxy on a machine that works and how you're doing it on a machine that doesn't?
If you're logging in using Auto IAM AuthN, you'll need to:

Make sure the credentials used by the Proxy match those of the IAM user in the database
Use either Application Default Credentials for a user account, or a credentials file for a service account
Launch the proxy like so:

# with a credentials file
cloud-sql-proxy --auto-iam-authn <INSTANCE_CONNECTION_NAME> --credentials-file /path/to/key.json

# with a user's ADC
cloud-sql-proxy --auto-iam-authn <INSTANCE_CONNECTION_NAME>

Thanks for getting back to me so quickly. Unfortunately, I didn't record anything and I don't have a Mac around to test it on. The people who do have their own stuff to be working on so I can't keep pestering them. I'll just recount the steps to the best of my memory, I hope that suffices.
I should note though, that I only found out about the version bump after I posted the question. It would be nice if that were mentioned in the google cloud documentation, that would've saved us some confusion. The person who had it working on a Mac M1 was using a V1.xx version that he downloaded just before the page was changed.
What we did

installed the Cloud SDK / CLI
gcloud init
gcloud auth login
downloaded the Auth Proxy from here and followed the steps appropriate for the relevant OS
reboot
what works:

all OSes: V1.xx: cloud_sql_proxy -enable_iam_login -instances=<INSTANCE_CONNECTION_NAME>=tcp:<PORT>
only on Windows: V2.xx: cloud_sql_proxy --port <PORT> <INSTANCE_CONNECTION_NAME> --auto-iam-authn

what does work on an Intel Mac:

V2.xx: cloud_sql_proxy --port <PORT> <INSTANCE_CONNECTION_NAME> --gcloud-auth
and then logging into the database with a SQL account rather than the IAM account.

I should note that we haven't tested V2 on an M1 Mac yet.

The workaround that I've landed on for now is to have people grab the latest V1.xx release from the Github releases page. It requires a bit more handholding from me, but it doesn't affect so many users that it's an immediate problem.

Hi @BartStikkers, thanks for the additional details! 😄
I believe the issue you are facing is a result of a bug in our documentation on the Connect using the Cloud SQL Auth Proxy that has to do with setting the credentials used by the proxy.
TLDR
You should use gcloud auth application-default login to set credentials to be used by the proxy (and user to be used for Auto IAM AuthN) instead of gcloud auth login.

gcloud auth login is to set credentials for a user to be able to perform gcloud commands
gcloud auth application-default login is to set credentials for a user to be used by application developers using client libraries (such as the Cloud SQL Proxy).

Step in the docs where bug is:

4. Credentials from an authenticated gcloud CLI client

This step prompts users to run gcloud auth login to set credentials to be used by the Auth Proxy. However, this command only sets the user credentials to be used by gcloud commands and does not set credentials to be used by applications (or client libraries such as the proxy).
The Cloud SQL Proxy sources credentials from the environment's Application Default Credentials and the proper command to run instead of gcloud auth login is:
gcloud auth application-default login

Let me know if running the new command resolves your problems! 😄 I have filed a bug against our docs to have the credentials steps updated to showcase the proper command.
Have a great day!

Okay, thanks, that makes sense. It's the end of the day here now though, so I'll revisit this with my team on Monday. I'm only left wondering why the new version works on my Windows machine without having run gcloud auth application-default login. Quite possibly, I've set ADC for some other reason at some point, or do they get set behind the scenes when one runs the old version?

@BartStikkers You are most likely correct in that you probably set ADC credentials in the past and your environment is still picking them up to be used by the proxy.
You can test this theory by running gcloud auth application-default revoke on your machine and then try invoking the Cloud SQL proxy and see if it begins to fail.
If it does then you must of had ADC credentials set in your environment previously.

@BartStikkers The step in the docs has been fixed to showcase the proper gcloud auth application-default login command. Thanks for bringing this to our attention and helping make the Cloud SQL Proxy experience a better one 😄
I am going to close this as I believe this command was the root cause of your issues. If you are still facing issues feel free to re-open this and I will take another look.
Have a wonderful day!


Set up your Kanban board

Project Link
Click Here
#Number of People Working in the team

Two

Hi @bhskr44 and @Alabi12,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
You should provide the link to the kanban board, not your repository
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.

Status: Required Changes ♻♻♻
Hi Team,
Good job so far!
There are some issues that you still need to work on to go to the next project but you are almost there!
Highlights
✅ You used the template given 👍
Required Changes ♻
Check the comments under the review.

[ ] It's required that for a group of 2 students for this project, you should remove the tasks for group C and have only 15 tasks remaining in the project. Currently, you shared the tasks meant for the group-C student but it's required you have only 15 tasks remaining and the ones for group C should be removed. Basically, you are not to do those tasks, except if you guys have the time and can do that but for the Kanban board, kindly remove them 👍
[ ] Also, please add two assignees for tasks that are marked as group tasks. 👍

Optional suggestions
Every comment with the [OPTIONAL] prefix is not crucial enough to stop the approval of this PR. However, I strongly recommend you take them into account as they can make your code better.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Hi team,
STATUS: APPROVED ✔ ✔
Your project is complete!  There is nothing else to say other than... it's time to merge it :shipit:
Congratulations and well done and all the best with your project! 🎉

Highlights

All cards have been assigned properly 👍 👍
Kanban board is setup correctly 👏

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


'Shift + Click' or 'Command + Click' won't work when using Karabiner-Elements

I recently updated my Mac OS X to macOS Sierra 10.12. So I installed Karabiner-Elements instead of Karabiner and Seil, but I couldn't use shift + click or command + click after lauching Karabiner-Elements.
My ~/.karabiner.d/configuration/karabiner.json file is like below.
{
    "profiles": [
        {
            "name": "Default profile",
            "selected": true,
            "simple_modifications": {
                "caps_lock": "left_command"
            }
        }
    ]
}

Now I removed Karabiner-Elements.app but It still won't work, so I also removed /Library/Application\ Support/org.pqrs/Karabiner-Elements then shift + click worked properly.

ack, same problem here. Makes, for example, selecting multiple files in Finder, quite impossible.

Should have been fixed by https://github.com/tekezo/Karabiner-Elements/issues/5, but I saw this issue again on 0.90.40.

i'm experiencing the same thing. shift-click does not work as expected in adobe illustrator, for example: the shift key is essentially ignored when trying to constrain drawing or toggle selection.
here's my configuration file:
{
    "profiles": [
        {
            "name": "Default profile",
            "selected": true,
            "simple_modifications": {
                "right_option": "fn"
            }
        }
    ]
}

I just upgraded to Karabiner Elements 0.90.44, shit-click and cmd-click worked again.

Looks like #5 is still not fully fixed. Randomly, after many hours of work, it sometimes stops working. Turning Karabiner Elements off and on again fixes it for some time, but it breaks again after a couple of hours.

I'm still seeing this on 0.90.46, particularly when trying to select multiple files in Finder, i.e. it works for a while but then at some point stops working. Quitting the app works as mentioned.

Having this issue.
100% reproducable in Chrome/Safari when trying to open link in a new tab with CMD+click


Failed to connect - another one.

Trying a first installation, from HACS.
But every time I try to add the integration:

log:

Already tried various times to delete the frigate's folder, from custom_components. Or simply uninstalling it from HACS and installing again after a reboot; but always with the same result.
I even changed the Synology DSM ports (5000,5001) in case it could create some conflites, but no luck.
Any ideas?
Frigate stats
Not able to connect - ver: 3.00
Operating system
Home Assistant 2022.10.5
Supervisor 2022.10.0
Operating System 9.2
Install method
HACS
Hardware
CPU - NUC Intel 11th (no coral)

Ugh... forget it.
I was thinking about Docker, while trying to install it as a add-on, within HA.
Somewhat in my mind I was connecting the custom component in Hacs as a Add-on, instead of way to communicate with another platform.
I already sorted it out.


[Bug Report] Can't load a page when I specified "height" of table attribute with nuxt.js of Universal mode

Element UI version
2.2.0
OS/Browsers version
macOS/Chrome 64.0.3282.167 and Safari 11.0.3
Vue version
2.5.13
Reproduction Link
https://github.com/odanado/nuxt-element-ui-table-error
Steps to reproduce
vue init nuxt-community/starter-template nuxt-element-ui-table-error
cd nuxt-element-ui-table-error
yarn && yarn add element-ui
fix files (https://github.com/odanado/nuxt-element-ui-table-error/commit/0127c5b5e70d69519cc7753bf04d6d00f1b68398)
yarn run dev # ng
yarn run dev --spa # ok, can loading page
What is Expected?
Show page
What is actually happening?
Page don't load.
The status of some js files is pending in a network tab of Developer Tool.


OLH-1551 Fix DfE service card links.

Proposed changes

Update links for signed in state of DfE services.
What changed
New links for signed in state provided by DfE.

Why did it change
Previous links were incorrect.

Related links

Checklists

Environment variables or secrets

[x] No environment variables or secrets were added or changed

Testing

How to review

Do not merge until new links are confirmed as working correctly.


Can't add same schema definition after removal and publish

Describe the bug
For removing primary key and add with new primary key I removed mentioned schema
type MasterTechSegment @model(timestamps:null) {
	techSegId: ID!
	techSegCode: String
	techSegDesc: String
}

and done amplify publish. After that I added the same schema with slight change to add new primary key as below
type MasterTechSegment @model(timestamps:null) @key(fields: ["techSegId"]) {
	techSegId: ID!
	techSegCode: String
	techSegDesc: String
}

and done amplify publish. But this time it ended up with following error
Resource Name: MasterTechSegment-e2owpdgtivelrlwbcw7hplm7jq (AWS::DynamoDB::Table)
Event Type: create
Reason: Resource creation cancelled

Resource Name: MasterTechSegmentIAMRole (AWS::IAM::Role)
Event Type: create
Reason: MasterTechSegmd853ef-role-e2owpdgtivelrlwbcw7hplm7jq-dev already exists in stack arn:aws:cloudformation:us-east-1:199522889564:stack/amplify-spad-dev-221813-apispad-B7FRPJ5LNHM7-MasterTechSegment-BKVYTVKC4J9/48ae4700-9f81-11ea-9171-0ed24b955b13

Please advise me a solution to move forward
Amplify CLI Version
4.29.2

@meetrij Did you remove the @model from the schema -> do an amplify push -> and re-add the @model back and then get the error on amplify->push or did you remove the DDB table directly from the AWS Console?

@kaustavghosh06 I didn't remove the DDB table directly from the AWS console, just removed below mentioned schema type and done amplify publish
type MasterTechSegment @model(timestamps:null) {
	techSegId: ID!
	techSegCode: String
	techSegDesc: String
}

@meetrij When you removed the @model was there a function or other resource in your project with a dependency on the DynamoDB table associated with the model? If so, there is a known issue where the child stack defining the table gets orphaned when removing the @model which could cause the behavior you're seeing if you add the model back later. See https://github.com/aws-amplify/amplify-cli/issues/5118#issuecomment-770120224 for more details

Closing due to lack of further communication and response.


Search doesn't work in https://googlecontainertools.github.io/

For example, https://googlecontainertools.github.io/kpt/search/?q=kpt doesn't return anything.

Thanks Janet, I'll either get it working or remove the search bar altogether.


Rebuild bot implementation - Solves #237 & #246

I rebuilt the bot implementation so that it aligns more with the official botbuilder samples and it allows devs to extend it with their dialogs quite easily.
Additionally, we can use this new template to add SSO functionality in the future as well, with little effort.
It should solve #237  and #246

Let me work on that asap so that linting issues are resolved

Linting errors should be all fixed now, so we are good to go I guess


cmd/compile: better error message when generics are used with -lang=go1.17

Currently, when compiling with -G=3 -lang=go1.17, we run the types2 typechecker, but parse without the AllowGenerics flag. The result is when users try to write generic code, they'll get syntax error messages like unexpected [, expecting (, which seems to be a recurring source of confusion and leaves users (incl. expert Go users) uncertain of how to proceed.
I think we should probably instead always parse with AllowGenerics (at least when we're going to use types2), and have types2 report the usual allowVersion error messages if users try to use generics with go1.17 or earlier.
/cc @griesemer @findleyr

Agreed. This makes sense now that we're on 1.18. I will take care of this (but I'm ooo until next Monday).


Google Chrome History Plugin

History data can be found at %AppData%\Local\Google\Chrome\User Data\Default\History.
and this is a SQLite database file.

Went to implement this yesterday evening, ran into two issues:

SQLite database is locked constantly, but can be safely (and quickly) copied before opening.
The sqlite package I was using did not compile correctly for running inside of electron, and I didn't have time to get electron-rebuild going.

Less than helpful than submitting a full plugin, but figured those points may be useful if someone else wants to run at the problem. Can share queries/skeleton code I used if someone would like to have a go at it.

The bookmarks are stored in a json file, which might be easier to use. You can see the alfred chrome bookmarks for reference on this.
The alfred plugin that does chrome history copies the db file for 60 seconds, although I think it could be closer to 30+ minutes depending on your needs, since this copy operation is the slowest part of the all this. I couldn't find the source on github, but the workflow is on github and I added the relevant source to a gist.

I think caching the db file for a (configurable!) short amount of time would be good without impacting search performance. Copying, querying, searching (with fuzzy package) happens on my windows machine in less than 100ms on average. The sqlite native component is the only thing that bothers me (I'm not a node guy!).


Improve the look of top navigation bar in accordance to overall design of website

Describe the feature
As shown in screenshot, the navbar needs abit redesign according to overall look and feel of the site. Kindly assign me this task under GSSOC 24 as I have a good experience in UI designs and frontend.
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Can you please assign me this issue i can work on this..!


Unexplained permissions on macos binary.

Hi @jmorganca, why does this app need Bluetooth, Camera, and Mic permission? All of this permission is listed on the info. list of the app. This app has quite minimalistic functions and runs in a CLI, I couldn't think of any necessary requirements for these permissions, please explain or if possible remove these permissions.

Hi @Aniruddha200, the macOS app is a standard electron app, and is definitely not using any of these permissions. I will make sure they get removed from the generated Info.plist file


May I ask how to process the las data?

Hi,
what an amazing work you do!!! I really appreciate your work!!
I have some questions for you, can you give me some advice? :)

What version of las you use? 1.2 or 1.4?
How can I annotate the las? Do I need to prepare an annotation file, or can I directly modify a field attribute of the point cloud?
Can you provide some demo data? Used to test the effectiveness of algorithms or models that you have trained well.

Thank you very much for following my issue. If you can provide assistance, I would be very grateful and grateful!!!
Have a nice day~

Wait, I found the data link!!!
datasets
Does this work use this dataset?

However, I use the link to download the datasets, it's cannot be work :(


Debug build issue

compiled in debug mode
--
Depth: 1 N_G8-F6 score -8 (0.02 s)
Depth: 2 N_G8-F6 score -4 (0.02 s)
Depth: 3 N_G8-F6 score -19 (0.02 s)
Depth: 4 N_G8-F6 score -5 (0.04 s)
Depth: 5   E7-E6 score -21 (0.36 s)
thread 'main' panicked at src/engine.rs:1990:13:
assertion failed: v_depth_inc + sdi[el.sf.abs() as usize] + ddi[el.df.abs() as usize] < 8

The assert is too restrictive, will be fixed soon.


Icon improvement

Problem/Motivation

All the addons have color icons, and when the addon isn't running the icon is black and white, as the chrony addon icon is black and white we need to open the addon page to know if is running or stopped

Expected behavior

Color icon

Actual behavior

(What actually happened)

Icon black/white

(How can someone else make/see it happen)

Start and stop the addon, the icon is always the same

Please feel free to issue a PR with an updated icon.

I'm working on the colored version of the Chrony icon and logo, I will issue a PR after I'm done so you can tell me if you like it
(I'm not an artist, I'm just making the icon colored and adding the name of the addon in the logo)


Question - Why do I constantly get "Input buffer contains unsupported image format"

I might be doing something wrong but I keep getting the error above. I've reviewed the docs for input stream as well as #220, followed all suggestions and still can't get this to work. My backend code is below:
//image-processor.js
import Sharp from 'sharp';
import Hoek from 'hoek';

class ImageProcessor {
    constructor(imageBuffer) {
        Hoek.assert(imageBuffer, 'You need to provide a buffer.');
        Hoek.assert(Buffer.isBuffer(imageBuffer), 'Invalid buffer.');
        this._transformer = Sharp(imageBuffer);
        //console.log(Sharp.format);
    }

    resize(width, height) {
        Hoek.assert(Hoek.isInteger(width), 'Width needs to be an integer.');
        Hoek.assert(Hoek.isInteger(height), 'Height needs to be an integer.');

        return this._transformer.resize(width, height).toBuffer();
    }
}

module.exports = exports = ImageProcessor;

   //model.js
      if (photo) {
              const processor = new ImageProcessor(photo);

              processor.resize(750, 750).then((outputBuffer) => {

                  model.set('photo', outputBuffer);

                  resolve();
             }).catch((err) => {

                  reject(err);
              });
          }

My javascript client sends the "photo" parameter as a base64 encoded string like below:
{
  photo: "
"ÿØÿàJFIF``ÿáDExifMM*&z ²1&¾2äiøên¬êPhoto Editor 10.0.10011.16384"
}

Any help would be very appreciated. Thanks.

Perhaps photo is a Buffer containing a base64 encoded string?
const photoBufferConvertedFromBase64 = Buffer.from(photo.toString(), 'base64');
const processor = new ImageProcessor(photoBufferConvertedFromBase64);
...
```

Hi @lovell, thanks for your prompt response. Yes, it is. Made changes as per your example above but am still getting the same issue. It does look like a buffer to base64 encoding problem but can't find where .... I am using hapi.js and Joi for validating the my payload so wonder if that's where the issue is ....

Hi @lovell, found out what the errors were:

Client side app sending uploaded file as readAsBinaryString instead of readAsDataURL
Server side app failing to remove "data:image/png;base64," part of the base64 encoded string.

Cheers!

Hi @lovell, found out what the errors were:

Client side app sending uploaded file as readAsBinaryString instead of readAsDataURL
Server side app failing to remove "data:image/png;base64," part of the base64 encoded string.

Cheers!

I've been looking for hours and this fixed my issue! It should be written somewhere that the data:image/png;base64, bit has to be removed before being sent to the Buffer.


[Draft]: Whether the operator scope should be restricted to namespaces

Desc
Operator is managed and installed by OLM. According to the OLM specification, when an OLM installation mode is used to install operator, the rights of operator are restricted by OLM.
For example, if the installation mode is selected as the owner namespace, or a namespace, then the operator should only manage CR resources for the corresponding namespace. However, this situation violates the functional responsibilities of the operator itself. The operator itself should listen for more content to meet the needs of the user.
Q&A
Based on the above, several issues need to be discussed:

[ ] What are the installation modes supported by Operator? Whether to support limited namespaces
[ ] If limited namespaces are supported, check whether the visible scope of CR is the same as that of OLM installation when operator manages CR resources
[ ] Considering the particularity of secret, can we limit the namespace to avoid the leakage of confidential information

There is a good article on this subject that details some typical scenarios, best practices, and recommendations.
https://medium.com/@nathanpbrophy/olm-and-operator-permissions-declassified-54cd1465f2b4
It recommends a pattern for Operators to consume permissions on a cluster, and how a user can determine what permissions an Operator can perform.
Cluster Admins:

Define an Operator Group with a Service Account that limits permissions to only those the Operator needs
Install Operators in the minimum scope required to perform the task

Operator Developers:

Defer as many permissions to the Operator as possible (limit the number of permissions the Operand i.e., application, needs)
Do not put Operand permissions or bindings to permissions in the Operator bundle
Limit the amount of cluster scoped permissions used by the Operator or Operand

This article mentions that there are some valid use cases for injecting Operand permissions into the bundle of the Operator, but those use cases are highly specialized. In general, injecting Operand permissions into the bundle for the Operator is an anti-practice, because the Operand is not generally an extension of the control-plane. It is recommended to minimize the amount of Kubernetes permissions an Operand needs and defer the interaction with the control-plane to the Operator.


Minor correction to usage docs

The current docs don't properly style the one code excerpt, and
have incorrect heading markdown.
I thought they were small enough to fit in one commit/pull.
Doc section with incorrect styling

Brilliant. Thanks. 👍


⚠ Bofrost.at has degraded performance

In 3a64d89, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2831 ms

Resolved: Bofrost.at performance has improved in f6fcda3.


Add 404 response to Create a Repository Dispatch docs

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
Create a repository dispatch event
What part(s) of the article would you like to see updated?
The responses listed are:
HTTP response status codes for "Create a repository dispatch event"

Status code
Description

204
No Content

422
Validation failed, or the endpoint has been spammed.

But calling the endpoint today, I got a 404:
Response body

{
  "message": "Not Found",
  "documentation_url": "https://docs.github.com/rest/repos/repos#create-a-repository-dispatch-event"
}

So, I'd love to see a proper explanation of what is not found — I suppose the event I called was not found, but that's unclear.
Additional information
No response

Thanks for opening this issue. A GitHub docs team member should be by to give feedback soon. In the meantime, please check out the contributing guidelines.

Error 404 found at Lloyd's of London and Vodacom apps

@angelod1as Thanks so much for opening an issue! I'll triage this for the team to take a look :eyes:

@cmwilson21 what's funny is that I found the error: the User-Agent was invalid. It missed a hyphen. E.g., User-Agent: invalid name instead of User-Agent: valid-name. So, the 404 error is even more cryptical.

For those who end up here searching for answers: this may as well mean that you have Fine-grained access token generated, with no "Read and write" access to repository's "Contents". I don't know why it is required to trigger dispatch, but it seems to be a requirement.
If the token exists, is valid, even has access to this repository but does not have write permission to Contents, it will return 404.


Bluetooth: controller: Fix ticker collision update race

In the case where a BLE connection ticker node collides and does not
trigger a callback to ULL, and at the same time the same node receives a
ticker update request, the update causes the node to be scheduled for
execution a second time. This has the consequence of event_counter going
out of sync, and BLE connection is ultimately lost.
The solution is to mark collsions and re-scheduled nodes as expired, and
use the increased lazy count to identify its state instead of req/ack.
Signed-off-by: Morten Priess<EMAIL_ADDRESS>

@cvinayak, this is my proposal for fixing the issue I mentioned on email. All our tests pass locally with 2.2.1 branch, but on master, the advx test is not happy. Does the error message make sense to you? It seems to say the test passed, but result is failed anyway.

In the case where a BLE connection ticker node collides and does not
trigger a callback to ULL, and at the same time the same node receives a
ticker update request

I am trying to narrow down on such a scenario, is this the case when slave connection is using slave latency, and there is a race between ticker_update invoked at the same time as the slave connection event collides with another state/role?

In the case where a BLE connection ticker node collides and does not
trigger a callback to ULL, and at the same time the same node receives a
ticker update request

I am trying to narrow down on such a scenario, is this the case when slave connection is using slave latency, and there is a race between ticker_update invoked at the same time as the slave connection event collides with another state/role?

This specific scenario is without slave latency. But yes, the slave connection collides with ADV and does not get to execute. However, it does increment it's lazy count in ticker_worker. Next ticker_job execution then sees both an update and the "pseudo-expired" slave node, and the update thinks the slave node is no expired, and put's it pending (again) immediately.
The problem is in ticker_job_list_manage, which doesn't take the "else" path, because state is 1 where it should be 2.

I will trying looking into the bsim_bt test failure this week.

@mtpr-ot Feel to clean up my commit, may be have a inline functions for handling ticker state, but can be a different PR.

@mtpr-ot Feel free to clean up my commit, may be have inline functions for handling ticker state, but can be a different PR.

I think it's fine - good catch. Thanks for your help.


Use language strings from language i18n file

As Waifusion expands, we're needing to add support for other languages, namely Japanese, so we can expand our worldwide appeal. There are contributors ready to add translations, so I've created an initial i18n file at src/i18n/strings.js which exports a {[language_code: string]: {[string_name: string]: string}} object to map languages to their respective strings.
With this, we'll also need to add a new function or hook to make it easy to use the current language strings that the user has selected from the file. In a perfect world, we should just be able to include {lang.WAIFU_NAME_PREFIX}, for example, in a component and it should work. Open to suggestions on what people think the best way of going about this is.

To target a more global audience JSON files may be better since they are supported across more translation platforms. One of them being Crowdin or if you prefer a self-hosted solution there's also Weblate.
As for something to interpret the files, there's an internationalization-framework called i18next that also offers a React library called react-i18next.


Completed - In-class exercise CSS flex

Hello @sevresbabylone @cbas,
here's our work. :) Inspired by: https://news.ycombinator.com/news
Thanks!
Cheers,
Min and Jeremly (@wytanj)

:sparkles: Great! :sparkles:
Feedback:

Avoid non-semantic elements like #breaker and #breakerline and use paddings/margins/borders instead.
Don't make the | pipes part of the <a> content. Place them in between, or, better yet, inject them using CSS using a border or the content property on a :before or :after pseudo-element.
Use an <ol> to add numbering to the list.
Use semantic, meaningful, descriptive names for classes instead of container1.

Thanks alot! :smile:
:chocolate_bar:


[Question] How can I get my own debug logs?

Just starting with Goose and working on a simple login scenario, following the examples. I'm running into a tokio panic when trying to user.get_session_data_unchecked, but I'm unsure why.
I'd like to add some debug prints or logs, but it's not clear to me how to actually get that output with Goose. Could someone give me some pointers?

First, I recommend not using the _unchecked variation as that will panic if something is not right. Possibly you can look at the backtrack for full details.
Logging can be read about here:
https://book.goose.rs/logging/overview.html
Specifically you’d be able to use the debug log:
https://book.goose.rs/logging/debug.html
Development specific docs are here:
https://docs.rs/goose/latest/goose/logger/index.html#writing-debug-logs

Perfect, thanks!


Removing duplicate method definition (#514)

This resolved the issue #514 when building the freestanding binary.

Hi, thanks for the PR! I didn't intend that all three entry points should be used in the same file. If you want to do that you should use conditional compilation.
I see how the post is a bit confusing in this regard. I opened #516 to clarify it.

No worries. I was following the example manually (only typed in what was needed for my OS) and when I got to the summary section, I wasn't sure if it was intended or not so I solved for the obvious error.
Thank you for the great tutorial! 🙇


build: phony: Enhancement

added CesiumOS ASCII art
added "cesium" under phony to support "make/mka cesium"

Fine merged


remove the '' escape character in query result, fixed #58

Unfortunately this is currently breaking 30 out of 115 integration tests so I can't let it go into production. As per #49 escaping of whitespace shouldn't be happening anyway so the fix might not be necessary in the and at all.

Work done for #49 makes this fix unnecessary so I'll close it down, hope you don't mind. And thanks a lot for the effort. :)


Added rolling category to updater

@pgaskin Did you have time to look at this?

I won't have time until next week (I've been busy with my username change and some of my other projects).

@pgaskin When will you have time to look at this? More than a month ago you suggested that you will look at is a week later...

Thanks a lot for reviewing this! Could you please also look at my other updater PRs?


[WIP] Add the new function to update the ContractPublisher

Fixes #3351
Proposed Changes

Add the new class function updateContractPublisher

Release Note

Docs

Closed as this function is implemented in #3325


[Bug]: The response from the Appsmith S3 plugin only returns the content of the files, but not their headers.

Is there an existing issue for this?

[X] I have searched the existing issues

Description
A user is successfully retrieving their data from S3, but wants to display the last modified date of the file. Currently, they are unable to obtain it because Appsmith does not return this header.
Attach the Amazon documentation regarding this matter:https://docs.aws.amazon.com/AmazonS3/latest/API/API_GetObject.html#API_GetObject_ResponseSyntax
Steps To Reproduce
`

Connect to your S3 bucket.
Use the read file function.
Observe that Appsmith only returns the file data and does not return any other header.

Public Sample App
No response
Environment
Production
Issue video log
No response
Version
Appsmith v1.9.15-SNAPSHOT
 Front conversations

Hi Team,
As you can show the data of a stored file in S3, it would also be very useful to show at least the last modified date of the file.
Otherwise, users doesn't know which version of the file are we showing up.
Thanks!
Best regards,


Proper NativeAnimated node invalidation on Android

This diff attempts to fix a number of Android native animation bugs related to incomplete node invalidation, e.g. https://github.com/facebook/react-native/pull/10657#issuecomment-258297467.
For full correctness, we should mark any node as needing update when it is:

created
updated (value nodes)
attached to new parents
detached from old parents
attached to a view (prop nodes)

cc/ @janicduplessis

Tested it and it works well, it fixed an issue on android where sometimes there was a white screen after poping a route on android with NavigationExperimental + native animations.

@janicduplessis Not sure if it has something to do with this particular PR, but is there any chance that the #8968 have the same root cause? Like not proper node invalidation after the node is removed from the array of siblings with the different zIndexes?
I'm struggling with the same problem with white screen when transitioning between routes, but using react-router v4 and react-motion for animation. When the fading-out component should became null at the end of animation, I see the white screen instead. However during animation itself zIndexes behave as expected, I can see the "new" page floating above and the "old" fading-out page stays on background. So it happens exactly when the old one becomes null (probably not fully detached).

This only affects native driven animations (when you start an animation with useNativeDriver=true) so unless you are using that it doesn't seems related.

@facebook-github-bot shipit


how to use without Docker

Hello sir
How can i use this application without Docker on ubuntu please.Is there any init.d script too.How many  tunnels at a time can be used sir.

sish is fairly lightweight, so any number of tunnels can be used as long as resources are permitted. Without docker, you need to build the project manually with the go toolchain. There aren't any prebuilt binaries, but I can look at potentially creating them through CI. As for the init.d script, you're mainly on your own. Here's some decent documentation on this. Note that you can use systemd with most recent linux distros.

Thank you so much.I have build successfully


Use fallback proof courier service if transfer attempt has failed beyond backoff procedure cutoff

Asset send transfer proofs are transferred via a proof courier service. The address of the courier service is customizable. A fallback courier service address is available if a custom address is not specified. This fallback address points to a hashmail protocol service (called Aperture) that LL hosts.
Consider the case where a custom courier address has been specified (not the fallback address). If the sending node has failed to transfer the proof to the courier (failed beyond some backoff procedure cutoff), then the sending node should switch to using using a proof courier specified by the default fallback address. Similarly for the receiving node, if it fails to retrieve the proof from the courier service (failed beyond some backoff procedure cutoff) then it can just assume that the sending node switched to using the fallback proof courier.

CMIIW this has been satisfied by an old universe resync PR


Error When Building Windows HOST=x86_64-w64-mingw32

Can you reliably reproduce the issue? Yes
If so, please list the steps to reproduce below:

While following the windows build guide using bash in windows 10 running everything goes smoothly until it try to make: make HOST=x86_64-w64-mingw32 the make fails, also happens on ubuntu builds
Error:
Building native_nsis...
/bin/sh: 1: scons: not found
funcs.mk:242: recipe for target '/home/bige/ion/depends/work/build/x86_64-w64-mingw32/native_nsis/3.01-e4d86e28d2b/./.stamp_built' failed
make: *** [/home/bige/ion/depends/work/build/x86_64-w64-mingw32/native_nsis/3.01-e4d86e28d2b/./.stamp_built] Error 127

Expected behaviour
Should continue to make the file
Actual behaviour
Error:
Building native_nsis...
/bin/sh: 1: scons: not found
funcs.mk:242: recipe for target '/home/bige/ion/depends/work/build/x86_64-w64-mingw32/native_nsis/3.01-e4d86e28d2b/./.stamp_built' failed
make: *** [/home/bige/ion/depends/work/build/x86_64-w64-mingw32/native_nsis/3.01-e4d86e28d2b/./.stamp_built] Error 127
Screenshots.

Machine specs:

OS: Window 10 Ubuntu bash
CPU: FX 8370
RAM: 16GB
Disk size: 512Gb
Disk Type (HD/SDD): SSD

Excellent, thanks!
NSIS is the installer creator , which creates Windows installers.
We added it to the depends system because the default version provided by eg Ubuntu doesn't include shipper support for Unicode characters.
Compiling NSIS requires scons, but that's not installed by default.
scons should be added to the build instructions in doc/build-windows.md

@jpass022: I think you were also looking at the flow of the build documents? If so, could you please check that the documents you were looking at also include 'scons' as a dependency for cross-compiling the Windows version?
Thanks!
@Bige086: it should work now. Thanks again for noticing. (Please double check the install destination folder exists before you try to install, the folder is not automatically created).
I'm closing the issue.


Write README.rst

We need an intro README!
Include an introduction and some basic examples of functionality. The README should be a very quick introduction to the package to give users within a couple minutes a good idea of what functionality the package provides. Pretty pictures and animations are always good! Then at the end of the README the user is referred to the Jupyter notebook tutorial for more details.

Started in 5ec1366dfef1cd324e55fcf197b20065bd5a4366 in bdf-implementation Should add photos, animations of usage. A good animation would maybe be one like in the README of lightkurve which highlights how easy the package is to use.

Your ideas all sound good, @anthonyozerov !

I recommend adding a couple pretty figures to the README.rst to draw in the user. Perhaps a good figure of a globe scatter plot, a time histogram and a bolide light curve. Some eye candy to attract a user.

Just added a preview gif demonstrating some plotting in 50e6d309a882dcc95fbdd29578fbeaa63b66a87b.

Your preview gif looks great! I recommend slowing down the speed. The commands scroll off screen very quickly and is difficult to read what you typed in time before it's gone.


Automator: update<EMAIL_ADDRESS>in<EMAIL_ADDRESS>

Generated by Automator - 2021-04-07T15:12:28+00:00

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio_release-1.9
57855764cad081cb8b60ea762feee2a04499c88a
link
/test gencheck_istio_release-1.9

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@istio-testing: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

gencheck_istio_release-1.9
57855764cad081cb8b60ea762feee2a04499c88a
link
/test gencheck_istio_release-1.9

integ-security-multicluster-tests_istio_release-1.9
57855764cad081cb8b60ea762feee2a04499c88a
link
/test integ-security-multicluster-tests_istio_release-1.9

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@istio-testing: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-security-multicluster-tests_istio_release-1.9
57855764cad081cb8b60ea762feee2a04499c88a
link
/test integ-security-multicluster-tests_istio_release-1.9

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Download CV does not work

a 404 error is raised, which could be because it genuinely cant find the google drive, or because it does not have access to read

[FIXED] - had to make the google drive pdf a native file - this was done by screenshotting cv and adding to google docs - also using .env to hide API key


Add event object to all props related to "click"

Hello @AllenFang,
Related to https://github.com/AllenFang/react-bootstrap-table/issues/1524, I suggest to at least add the event object to props.onRowClick and props.onExpandRow. This should allow propagation handling.
I have a similar corner case where I'm expanding a row from an "inside" icon and my workaround is to check if the cellIndex is NaN (that means the target is the icon) and return immediately the function that handle the click event.
I can work on this in the weekend, if you're accepting this.

Markdown of an expandable row

@Robertmw sure PR is welcome, let me know if any problems, I can help on it. thanks

Fixed  on 4.2.0, thanks @Robertmw your PR 👍


Documentation on functions

Expected Results
Nevergrad includes a pretty comprehensive list of implemented functions to test algorithms on. However, is it possible to include some documentation to the functions implemented inside corefuncs.py regarding the nature of these functions (multimodal, monotonous etc.)? If not some detailed documentation of the function properties, at least a link to some white paper/publication from where the functions are sourced (any of the past CECs? Since nevergrad is organising a competition in CEC19)?
Relevant Code
from nevergrad.functions.corefunc import registry as func_registry
print(sorted(func_registry.keys()))

@teytaud will try to provide more information.
However, what would be the best way to document this? I would go for docstrings, but it may not be that easy to access?

What is the long-term plan for generating documentation for nevergrad? Docstrings that compile using Sphinx as usual? For now, since most of the Nevergrad user community is typically comfortable with diving deep into source codes, plain docstrings should be fine I presume.

Pushed. Criticisms welcome :-)

Concerning documentation, on the long run, sphynx generated doc is the aim indeed. I am happy with delaying this a bit for now though :D

Pushed. Criticisms welcome :-)

@teytaud Is it possible to find the source of these functions in literature? Were these part of some other benchmarking suite/competition/publication? In particular deceptiveillcond, deceptivemultimodal and deceptivepath. Would be good to see how they were arrived at and also to cite the original source in future publications.

The deceptive functions are new. They are introduced in the paper "Openly Revisiting Derivative-Free Optimization" this year at Gecco'19, Prag. There is a competition related to it in https://github.com/facebookresearch/nevergrad/blob/cec2019/docs/CEC2019.md .

+I add in my TODO-list the need for more detailed doc.

@teytaud As per the definition in corefuncs.py, deceptivepath and deceptivemultimodal are essentially the same function? Is this how it is intended to be?


Nuget Restore Improvements

Currently we only run nuget restore for a C# function when the host is up and running and we see a change to project.json. We also need to support scenarios where the project.json is modified (or created) when the host is NOT running. When the host starts, we should be able to discover these modifications and run restore.
We can use an approach like David suggested:
"Yes, we will find a way to make this work when deploying via Kudu API. Fabio, I think we could just keep track of the timestamp of project.json (under somewhere data folder). When host starts, any time we see the current timestamp is different from what we saved, we should run nuget."

It'd be good if the implementation of this could be generalized so it would also allow us to do the same for Node NPM when we enable that.
Perhaps we write a single timestamp somewhere capturing the last time the host has successfully started. Then the various language invokers can check this on startup, compare with project.json / package.json timestamp and run as needed. That will allow them to easily discover file changes that have occurred while the host was sleeping.

Bringing the comment from an internal email thread to the issue; for NuGet, instead of keeping track of the timestamp and restoring based on that, since we really only rely on the lock file that results from a package restore, we can identify the need to perform a restore if the following conditions are true:

We have a project.json file, but no project.lock.json
We have a project.json file and a project.lock.json but the lock file is outdated or doesn't match the project.json

Also, we need to make sure we don't do it simultaneously from multiple instances, which could cause corruption.

Yes, the multi-instance issue has come up in the context of function logs as well https://github.com/Azure/azure-webjobs-sdk-script/issues/164. The same mechanism of identifying a "primary" instance for things like recompilation logging, nuget restore, etc. will be needed.

Can we also support nuget resolution every time the host comes up to fetch the newer version packages if any?
It will be useful in case of bug fixes instead of asking customers to upgrade every time we roll out a fix/update. If the customer don't want that he can just turn off auto updates (with a flag).

Seems related to https://github.com/Azure/azure-webjobs-sdk-script/issues/254 which I'd like to consider for rc:should-have


Spring Boot - Loggers

View and configure loggers settings for a Spring Boot application.
https://github.com/hawtio/hawtio-integration/tree/main/plugins/spring-boot/loggers

fixed by: https://github.com/hawtio/hawtio-next/pull/669