fix:  doesn't unlock the body on unmount

Description

As described in https://github.com/streamich/react-use/issues/858 the useLockBodyScroll hook, is not unlocking the body on unmount.
Fix: Added an effect for observing component unmount. This effect will unlock the body when the component is unmounted. Also due to the changes, there was duplicate code for unlocking functionality. So I have separated the lock and unlock functionalities into methods so that it can be reused.
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as before)

Checklist

[x] Read the Contributing Guide
[x] Perform a code self-review
[ ] Comment the code, particularly in hard-to-understand areas
[ ] Add documentation
[ ] Add hook's story at Storybook
[ ] Cover changes with tests
[x] Ensure the test suite passes (yarn test)
[ ] Provide 100% tests coverage
[x] Make sure code lints (yarn lint). Fix it with yarn lint:fix in case of failure.
[x] Make sure types are fine (yarn lint:types).

@streamich can we merge this one?

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


💡[FEATURE]: Adding Twitter Authentication

Is your feature request related to a problem? Please describe.
@Abidsyed25  I want to add Twitter Authentication using Firebase. Kindly assign me this issue.
Describe the solution you'd like
N.A
Describe alternatives you've considered
N.A
Additional context
N.A

@Abidsyed25 Sir kindly pls tell whether I shoud start working on this issue or not?


[Bug] create repository failed: invalid backend (s3)

Describe the bug
I have tried to use the image in a simple k8s cronjob.
Additional context
Add any other context about the problem here.
Logs
If applicable, add logs to help explain your problem.
  In my local setup it looks fine with the same ENV vars:
➜  samba git:(immediate-deployment) ✗ export RESTIC_REPOSITORY="s3:https://s3.amazonaws.com/example-k8s-usb-restic-backup-ie"
➜  samba git:(immediate-deployment) ✗ export RESTIC_PASSWORD=xxxxxxxxxxx
➜  samba git:(immediate-deployment) ✗ export AWS_ACCESS_KEY_ID=yyyyyyyyyyyyy
➜  samba git:(immediate-deployment) ✗ export AWS_SECRET_ACCESS_KEY=zzzzzzzzzzzzzzzzz
➜  samba git:(immediate-deployment) ✗ restic snapshots
Fatal: unable to open config file: Stat: The specified key does not exist.
Is there a repository at the following location?
s3:https://s3.amazonaws.com/example-k8s-usb-restic-backup-ie
➜  samba git:(immediate-deployment) ✗ restic init
created restic repository d33c655594 at s3:https://s3.amazonaws.com/example-k8s-usb-restic-backup-ie

Please note that knowledge of your password is required to access
the repository. Losing your password means that your data is
irrecoverably lost.
➜  samba git:(immediate-deployment) ✗ restic snapshots
repository d33c6555 opened successfully, password is correct
created new cache in /home/phil/.cache/restic

To Reproduce
Steps to reproduce the behavior:

create cronjob like this

apiVersion: batch/v1
kind: CronJob
metadata:
  name: backup-restic
  namespace: samba
spec:
  #schedule: "@daily"
  schedule: "* * * * *"
  jobTemplate:
    spec:
      template:
        spec:
          volumes:
          - name: data-private
            persistentVolumeClaim:
              claimName: local-pv-samba-share-private
          containers:
          - name: backup
            image: ghcr.io/k8up-io/wrestic:v0.3.3
            imagePullPolicy: IfNotPresent
            env:
            - name: KEEP_LAST
              value: "10"
            - name: KEEP_DAILY
              value: "7"
            - name: KEEP_WEEKLY
              value: "4"
            - name: KEEP_MONTHLY
              value: "6"
            - name: KEEP_YEARLY
              value: "3"
            - name: BACKUP_DIR
              value: "/data-private"
            envFrom:
            - secretRef:
                name: backup-secret
            volumeMounts:
            - name: data-private
              mountPath: "/data-private"
          restartPolicy: OnFailure

create secret like this

apiVersion: v1
kind: Secret
metadata:
  name: backup-secret
  namespace: samba
type: Opaque
data:
  AWS_ACCESS_KEY_ID: xxxxxxxxxxxxxxx=
  AWS_SECRET_ACCESS_KEY: yyyyyyyyyyyyyy==
  RESTIC_PASSWORD: zzzzzzzzzzzzzzzzzz=
  RESTIC_REPOSITORY: xxxxxxxxxxxxxxxxxxxxxxxxxxxx==

Environment (please complete the following information):

Image Version: v0.3.3
K8s Version: v1.23
K8s Distribution: k3s

Hi @runningman84
Thanks for your interest in wrestic/K8up. Please excuse us, we didn't properly mark this repository as unmaintained.
wrestic was a separate part of K8up, the backup operator. However, the code has been integrated into K8up since version 2.0 and since then, wrestic is no longer in development. We'll keep this repository since there are still K8up versions <2.0 deployed around.
We no longer support issues opened in this repository. Are you using K8up? If so, please open an issue in the main K8up repository instead (https://github.com/k8up-io/k8up).
Thanks for your understanding.

Hi @runningman84
I'm closing here, so we can properly archive the repository.


Rev package dependencies up.

This is primarily needed to pick up updated petitparser and xml packages so flutter test --preview-dart-2 works.

Dup of https://github.com/flutter/flutter/pull/15376


Pod uses the deprecated version of Flurry

Hi! Thanks for a great analytics library - it's very fun and easy to use!
It seems that you're using a deprecated version of Flurry SDK:
My Podfile:
pod 'ARAnalytics', '~> 3.8', :subspecs => ["Flurry"]

Console output:
Using ARAnalytics (3.8.0)
Using FlurrySDK (6.7.0)
...
Using FlurrySDK (6.7.0) [!] FlurrySDK has been deprecated in favor of Flurry-iOS-SDK

Am I doing something wrong?

You might need to do a pod update ARAnalyicsor ARANalytics/Flurry - we now point to the new Flurry library: https://github.com/orta/ARAnalytics/blob/master/ARAnalytics.podspec#L17

ah it's not been shipped yet, and I'm flying most of today, you'll need to use :head to use the latest master

@orta Thanks a lot for your help!

I'm trying to get this to switch to Flurry-iOS-SDK, but it refuses to do so. I've been using ARAnalytics for a while, and nothing I try will make it switch away from FlurrySDK.
The Podfile says:
pod 'ARAnalytics', :head, :subspecs => ["Flurry"]
pod install, pod update, pod update ARAnalytics, pod update ARAnalytics/Flurry, all have the same result - FlurrySDK stays. Deleting all the pods and running pod install, Flurry SDK comes back.. I tried cleaning the cache, it still comes back.
I'm on CocoaPods 0.39.0.

Hrm, you could try deleting the ARAnalyitcs / FlurrySDK line in your podfile.lock which should re-trigger a force lookup.
Other option is to try pod update ARAnalytics FlurrySDK I'm not too sure how to trigger the resolver to ignore the dependency on Flurry and allow updating to another.

No luck. I even tried completely removing CocoaPods from the project and doing a fresh init/install, no luck.
I tried making a brand new test project. Did pod init, and added this to the Podfile:
pod 'ARAnalytics', :head, :subspecs => ["Flurry"]
And pod install does:
Updating local specs repositories
Analyzing dependencies
Downloading dependencies
Installing ARAnalytics (HEAD based on 3.8.0)
Installing FlurrySDK (6.7.0)
Generating Pods project
Integrating client project
[!] Please close any current Xcode sessions and use Test.xcworkspace for this project from now on.
Sending stats
Pod installation complete! There is 1 dependency from the Podfile and 2 total pods installed.
[!] FlurrySDK has been deprecated in favor of Flurry-iOS-SDK

Ah! OK
So, this is my fault, sorry! So :head uses the trunk podspec, and the latest commit. I've pushed a release anyway, but I should have been recommending just using :git =>"https://github.com/orta/ARAnalytics.git"

That worked, thanks! I'll switch it back to using the specs repo again.


using taskscheduler with a user without a profile loaded

Is this possible or a bug that can be fixed?
I think what happens with an administrator logon without a profile (and the reghive not being present) is that the com object lookup isn't performed.
System.Runtime.InteropServices.COMException (0x800703FA): Retrieving the COM class factory for component with CLSID {0F87369F-A4E5-4CFC-BD3E-73E6154572DD} failed due to the following error: 800703fa Illegal operation attempted on a registry key that has been marked for deletion. (Exception from HRESULT: 0x800703FA).
Thank you!

You are correct. If you specify a user in either task registration or for a logon trigger, and that user does not have a local registry hive, you will get error like the one you're seeing. There is no work-around. :(


docs: How to refresh/revoke a token

As suggested by #261, we should document how to refresh the token and revoke a token in the README.

Is there any update on this? I am seeing a token being expired (getting 401 response from the Drive api) pretty quickly (perhaps within a couple of hours). Is there any way to automatically refresh it?

Making authenticated requests: https://github.com/google/google-api-nodejs-client#making-authenticated-requests

If you provide a refresh_token and expiry_date (milliseconds since the Unix Epoch) and the access_token has expired, the access_token will be automatically refreshed and the request is replayed

Manually refreshing an access token: https://github.com/google/google-api-nodejs-client#manually-refreshing-access-token

@jmdobry what is the logic behind expiry_date? does it request with Google how long the token should be valid for? If so, is it my responsibility to refresh it manually before that date ends, or does the library automatically refresh it in the background somehow, at some time?

expiry_date defines when the token expires.
If you provide a refresh_token and expiry_date the library will automatically refresh the access token if it is expired.

I am not aware of any limit, nor could I find any evidence of one. However, you should probably code your app to assume tokens are rather short-lived.

@jmdobry I tried adding the expiry_date property in the credentials object for each API call for 15 days out (new Date()).getTime() + (1000 * 60 * 60 * 24 * 15). However, it seems like the auth token is still expired way before that (within the same day maybe). Is there a way I could debug this?
When will the token be refreshed automatically by the library?

@ryanseys if I provide the expiry_date and the access_token is expired, how do I receive the updated accessToken? Because I'm guessing I would like to store that one istead of the old expired one in my db?
@tnguyen14 I believe I have read in the docs that the token expires in one hour.

@Lewenhaupt thanks. if you have a reference to that doc, I'd love to take a look. It would also be good to know if there's any good way to renew this token in the background if possible.

Hi, is there an update to this? I am having the same issue as @Lewenhaupt and had also think of the solution to store the updated access token instead of the expired one in the db - is there any way I can do this?


chore: Rename ScpCf prefixed classes

SAP/cloud-sdk-java-backlog#346.
This PR renames the classes prefixed with ScpCf.... All of them where in the context of the ScpCfDestinationLoader and got renamed to DestinationService....

I think we should also rename the Maven module

You really want to put up with dependency conflicts on customer side accidentally mixing v4+v5?

I think we should also rename the Maven module

You really want to put up with dependency conflicts on customer side accidentally mixing v4+v5?

Yeah it is a tradeoff.. This is again a module that uses class loading so having it on the class path twice would actually be bad. We could explicitly check if the ScpCfDestinationLoader class exists and throw if it does, not sure where to add this check though..
On the other hand, it would also be some migration effort for customers. A quick search reveals around 200 usages in total. So maybe not worth it? But if we already rename the core classes then renaming the module accordingly kinda makes sense..
How about this: We create a new module connectivity-destination-service but also keep the old module, make it empty and simply reference the new module as dependency. Then we dodge the version conflict and customers can migrate slowly if they like.

Okay, I will have a look into the 2 module solution, sounds good to me!

For reference here as well:

Current usages of ScpCfDestinationLoader:
github.wdf.sap.corp: 182
github.tools.sap: 176
github.com: 14

@MatKuhr : Do we want to wait for the meeting on monday to merge this, or get it done now and "fix" (whatever we decide on) after the meeting?

I think we can merge now, for the meeting on Tuesday I'd rather discuss the "aggregator" module strategy and only revert this change if we come to drastically different conclusions there.


Add honeypot field

Resolves #12

Hey, @smbkr thanks again. I'm just holding off merging stuff for today since the demo is live. I'll merge your PRs tomorrow, if that's okay with you :)

@smbkr I'll cherry pick this commit into my mega PR too (author information will be unchanged), if that's okay with you ^^

Cherry-picked and merged here: https://github.com/adtac/commento/pull/35
Of course, he commit author information is unchanged.
Thanks!


Currency and number formatting in hugo

Im not sure on how to do this in hugo and hence this could be  treated as a question
Say i have front matter for a "product" like this
+++
name = "Awesome android phone"
price = "13500"
priceunit = "INR"
+++

I want the price info to be displayed something like this
  13,500 ₹

Note the Currency display and comma separation
How do i do this? Im using the latest version 0.17

I think this is more of a question for the discussion forums: https://discuss.gohugo.io/

To give you some direction: Take a look at the variables documentation, especially Page Params at https://gohugo.io/templates/variables#page-params
Once you got the price with {{ .Params.price }} inside your template you can using Hugo (or Go)  functions or a custom shortcode (https://gohugo.io/extras/shortcodes) to format it as you like.

Hi @kevingimbel Thanks for the guidance! Im sure i will be able to do this using some printf for commas and a map with currency code and currency symbol but i am looking out for an "out of the box solution" which would be more maintainable. Hence i am opening a thread is hugo forum.
https://discuss.gohugo.io/t/currency-and-number-formatting-in-hugo/4838

Reopening the issue. Because i feel locale based currency symbol display and price formatting is an important feature.

Agreed. I'm pretty happy with the solution @moorereason put together. But there are limitations given its simplicity. A couple of issues I'm running up against trying to do just simple stuff with currency formatting include:

Cannot typecast currency strings to longs or BigEndian
Not able to format the currency symbol (฿ anyone?)
Not able to convert between currencies given a rate

It's arguable much of this stuff should be handled outside the generator. But as a presentation layer some additional affordances for handling currencies would be apt for comparison shopping sites and e-commerce applications.
For some examples of how to convert and format money in Jekyll, may I present https://github.com/jhabdas/jekyll-money


How do I add the custom admin page to the permission?

Laravel Version: v5.8
Voyager Version: v1.2
PHP Version: 7.3
Help me. I've been searching for days, not knowing what to do with it.
How do I add new views or actions to the permissions in the figure below that has nothing to do with a BREAD view?

The answer has already been answered on Slack, and, as the issue templates suggest, questions shouldn't be asked in issues.

Slack已经回答了答案，并且正如问题模板所示，问题不应该在问题中提出。

I searched Slack for a long time, but couldn't find the answer. Can you tell me the link to the answer?


if you want.

xml like.

totally didn't steal hscript from Wednesdays infidelity :trollface:

totally didn't steal hscript from Wednesdays infidelity :trollface:

HEY HEY BRO THIS EVENTS ARE UNHARDCODED this way easier events but yes XDDDDDD

totally didn't steal hscript from Wednesdays infidelity :trollface:

Fatherless act, but seems good to have hscript support in this abandoned fork.

totally didn't steal hscript from Wednesdays infidelity :trollface:

Fatherless act, but looks good to have hscript support in this abandoned fork.

wait what?

totally didn't steal hscript from Wednesdays infidelity :trollface:

Fatherless act, but looks good to have hscript support in this abandoned fork.

mf just missclicked and closed the pr.


error while installing neon on centos6.4

My system is CentOS6.4, and I have installed python2.7 to replace 2.6 version.  Version 2.7 is now in use, but I encountered a problem that it still use the old version while installing neon. It has no effect by setting PYTHONHOME or PYTHONPATH. How can I deal with the problem.

I have solved the problem by reinstalling numpy and PyYAML. Now everything goes well.


What copyright is on the csmabF5.gif that is linked in this project?

Hi!
I am interested to use the csmabF5.gif that is linked (in /www/loading.html) in this project.
What is the copyright on this gif?
Thanks!
Frederick
http://i.imgur.com/csmabF5.gif

@frederickjh I googled it at some point as "loading animation flask". It still shows up in the Google search results for images with no copyright mention anywhere.
You can do your due diligence by mentioning the source of the image you are using.

Thanks @lmakarov!
The most I can find is that it was posted on Imgur May 13, 2105 by a Mark Bennett. Who also posted it on giphy.


Usability check for the XmlData library

Description:
$Subject

The following improvement has in this module when checking a usability check.

https://github.com/ballerina-platform/ballerina-standard-library/issues/2522
https://github.com/ballerina-platform/ballerina-standard-library/issues/2513
https://github.com/ballerina-platform/ballerina-standard-library/issues/2511


Update entrypoint.sh

fixed false positives when a local id in interlocking.exml is a substring of an entry in LOCAL_IDS

I have committed an commented version.
A simpler fix would be:
Line 7 original file
LOCAL_IDS=",$LOCAL_IDS,"
Line 38 original file
if [[ " ${LOCAL_IDS[*]} " =~ ",$name," ||  -z "${LOCAL_IDS}" ]]
This way we can also be sure that the whole element is compared and not just a substring.


Appstudio update mvp-test-component

Pipelines as Code configuration proposal

Pipelines as Code CI/mvp-test-component-on-pull-request has failed.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
15 seconds
clone-repository

---
---
build-container

Pipelines as Code CI/mvp-test-component-on-pull-request has failed.

StatusDurationName

✅ Succeeded
6 seconds
init

✅ Succeeded
15 seconds
clone-repository

---
---
build-container

Pipelines as Code CI/mvp-test-component-on-pull-request has successfully validated your commit.


Error in Postprocess.py file

Hello team,
First of all I highly appreciate the work that you are doing for the open source community by providing such robust and highly designed models and I am very thankful for this.
Coming to my issue I was trying to form csv file using the post process function after doing OCR to the table and supplying the bounding boxes of text as well as the text itself. I passed the tokens to the function "objects_to_structures(objects, tokens, str_class_thresholds)" and from thereon it went to the post process file in which i encountered a key error stated as follows -
KeyError: 'span_num'
This error comes in line 332 which is as follows -
spans_copy.sort(key=lambda span: span['span_num'])
the error comes in the lambda function and i am having a hard time to understand this issue.
Can you please help me to overcome this please.
Thankyou in advance.

Tagging just for the visibility of post.
@bsmock

Hi,
That code was written with PyMuPDF in mind, which is a Python library that can be used to extract words from digital-born PDF documents. Every word extracted by PyMuPDF has a block_num, line_num, and span_num property.
The code in postprocess.py sorts the words by these fields in order to put the words in each table cell into reading order.
If your text is already in reading order, the simplest thing you can do to fix the issue is to comment out or remove the 3 lines of code that do the sorting.
Otherwise, for each word you can set the 'span_num' field to have a number (0, 1, 2...) that indicates its sequential reading order. If you do that, 'line_num' and 'block_num' can all be 0.
Best,
Brandon


Getting "Expected to have result"error after completion of a step

Below is my test.But am getting the error ""Expected to have result"" just after typing in username.Can you help me here.
test.only("executes query", async ({ page }) => {
await page.goto("https://www.saucedemo.com/");
const headerText="Playwright";
await auto(`type "${headerText}" in the username`, { page, test },{model: 'gpt-3.5-turbo-1106', debug: true});
await auto(`type "Hello" in the password`, { page, test },{model: 'gpt-3.5-turbo-1106', debug: true});
await page.pause();

});

Here the same.

Same issues here

same here

I bumped into this today while looking to try out auto-playwright on the cheap with GPT-3.5. I was able to get around the issue as follows:
in node_modules/auto-playwright/dist/completeTask.js at line 23 there's an if statement. The GPT-3.5 final role === "assistant" response message doesn't appear to contain a value that satisfies this predicate:
message.function_call?.name.startsWith("result") . It also doesn't contain a message.function_call value.
I updated the if as follows:
        if (message.role === "assistant" &&
            (message.function_call?.name.startsWith("result"))
            || message.content?.indexOf(" successfully ") > -1) {
            lastFunctionResult = JSON.parse(message.function_call?.arguments?? 'true');
        }

GPT-3.5 has been returning a message like "I have successfully ..." or "... as been completed successfully...". I'm not sure why it isn't the same every time, but I've seen a couple variations. In my cursory testing this worked, but YMMV. I'm not sure how it will hold up to more robust situations.


How to write a wrapper for an existing JS library?

I'd like to create a wrapper for an existing JS lib (in my case it's leaflet) but I couldn't find any documentation about how to do this.
Do I have to create e.g. a leaflet_wrapper.js (like the example @ rustwasm #importing-non-browser-js) and then create the actual leaflet.rs that is linked to the wrapper?

:+1:
I am in a similar situation. I currently do what you suggest: have a wrapper.js and a wrapper.rs. It is very nice to be able to wrap an .rs directly around the JS library. I hesitate to ask because rustwasm focus on using wasm in a JS project rather than JS in a wasm project.

I hesitate to ask because rustwasm focus on using wasm in a JS project rather than JS in a wasm project.

Please don't hesitate! It's true that for the MVP we focused on using Rust within JS projects, but there are absolutely plans to support using JS in Rust:
https://github.com/rustwasm/rfcs/pull/7
https://github.com/rustwasm/rfcs/pull/6
https://github.com/rustwasm/rfcs/pull/8

I currently do what you suggest: have a wrapper.js and a wrapper.rs.

:+1:
Could you show me a working example with all your build steps?

Could you show me a working example with all your build steps?

You don't mention whether you want nodejs or not!?? If you are happy with it, I think the example you linked in your first comment should be enough (but not sure, :smile:, because I never try it myself).

If you want to avoid nodejs, then I am sorry because I don't have a ready-to-use-build-steps list now! If you want to find the solution yourself, you can go through source code of one of two tools:

wasm-bindgen-test-runner - this is where I learn to run a front-end web app that allow non-browser-js without the need of a bundler.
simi-cli - I implement what I learn from wasm-bindgen-test-runner in this tool, and I use it - this is the reason why I don't have a ready-to-use-build-steps list available for you right now! (warning: it is a mess now, I am not happy with it but still too lazy to reorganize it).

From what I can recall, these are what involve: wasm-bindgen --browser, wasm2es6js, and a custom JS snippet like this
If I can find enough free time in next few days, I will try to make a working example with a build.sh in case you still want it.

I forgot about file name's extension. wasm-bindgen will generate a package_name.js that contains a
import * as wasm from './package_name_bg'; // Without extension

From what I understand, bundlers like webpack think it refer to the package_name_bg.wasm (also generated by wasm-bindgen). But if you process with wasm2es6js, then it turn out that line refer to the package_name_bg.js file generated by wasm2es6js. If you leave the mentioned line intact, your server do a trick like the wasm-bindgen-test-runner do here. In simi-cli I choose to append a .js to the filename.
So, there is no easy build-steps to make it works without webpack. (Unless you know how to search and replace text from the terminal).

Thanks for the report, and this is definitely something we want to support! The RFC's linked by @Pauan have all our future plans, so I'm gonna close this in favor of those.


DayPickerRangeController CalendarDay--hovered-span class remains on previously selected date

In the DayPickerRangeController after changing the date range, the previous selected date has the className CalendarDay--hovered-span which leaves it coloured.
This can be reproduced in the react-storybook examples.
I would be more than happy to help in the issue with some guidance!
Thanks for the awesome work!!!

Hi @gatzjames! I think the fix is in https://github.com/airbnb/react-dates/pull/523 which has been merged but not released yet! I will do a release right now. :)

Should be fixed in v12.1.1!


Added partial_sort in algorithm

Thanks for your contribution but it looks like this method has already been contributed before.


Integration Test - Release - 2021/11/02 12:07:54

Auto create for integration test. IssueCommandFlow

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

Docs Build status updates of commit 9460f20:
:warning: Validation status: warnings

File
Status
Preview URL
Details

csidevdocs-public/IntegrationTest/BuildIntegrationTestFiles/ChangeSentToAuthor.md
:warning:Warning

Details

csidevdocs-public/IntegrationTest/BuildIntegrationTestFiles/ChangeSentToAuthor.md

Line 4, Column 1: [Warning: ms-prod-and-service - See documentation] Only one of the following attributes can exist: 'ms.prod', 'ms.service'. Use ms.prod for on-premise products, or ms.service for cloud services.
Line 4, Column 8: [Warning: duplicate-titles - See documentation] Title 'VS Code Authoring Extension' is duplicated in: 'Demo/demo.md(4,8), Demo/v-caxian/AuthorNotification.md(4,8), Demo/v-caxian/auto-merge.md(4,8), Demo/v-caxian/auto-merger.md(4,8), IntegrationTest/BuildIntegrationTestFiles/ChangeSentToAuthor.md(4,8)'. Titles must be unique within a docset.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
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 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 docs.microsoft.com contributor guidesPost your question in the Docs support channel


Apex Chart Service does not work in Blazor WASM

Description:
Encountering a type resolution error when adding builder.Services.AddApexCharts() to the Blazor WASM program file in .NET 8.
Steps to Reproduce:

Add builder.Services.AddApexCharts() to the Blazor WASM program file
Run the application

Specs:

.NET 8
Blazor WASM
Blazor-ApexCharts v4.0.0

Error Message:
`ManagedError: Could not resolve type with token 010000b2 from typeref (expected class 'ApexCharts.ApexChartsExtensions' in assembly 'Blazor-ApexCharts, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=null')
at Jn (c:\Users\admin1\source\repos\website\website\website\wwwroot\_framework\https:\raw.githubusercontent.com\dotnet\runtime\81cabf2857a01351e5ab578947c7403a5b128ad1\src\mono\wasm\runtime\marshal-to-js.ts:349:18)
at kr(c:\Users\admin1\source\repos\website\website\website\wwwroot\_framework\https:\raw.githubusercontent.com\dotnet\runtime\81cabf2857a01351e5ab578947c7403a5b128ad1\src\mono\wasm\runtime\invoke-cs.ts:277:19)
at l.javaScriptExports.call_entry_point(c: \Users\admin1\source\repos\website\website\website\wwwroot\_framework\https:\raw.githubusercontent.com\dotnet\runtime\81cabf2857a01351e5ab578947c7403a5b128ad1\src\mono\wasm\runtime\managed - exports.ts:60:13)
at Object.Fc[as runMain](c: \Users\admin1\source\repos\website\website\website\wwwroot\_framework\https:\raw.githubusercontent.com\dotnet\runtime\81cabf2857a01351e5ab578947c7403a5b128ad1\src\mono\wasm\runtime\run.ts:44:45)
at Object.callEntryPoint(:1:156271)
at ti(https://localhost:7204/_framework/blazor.web.js:1:163775) {superStack: {…}, managed_stack: '   at System.Runtime.CompilerServices.Asy…t:7204/_framework/blazor.web.js:1:163775)', stack: <accessor>, message: 'Could not resolve type with token 010000b2 f….0.0, Culture=neutral, PublicKeyToken=null')', Symbol(wasm js_owned_gc_handle): 38}`

Hi,
Do you have AOT enabled?

Hi,
No, I do not have AOT enabled on my project.
I would also like to give more context on the issue. I upgraded directly from Blazor-ApexCharts 3.4.0 -> 4.0.0 with no other changes to my project. Even without adding builder.Services.AddApexCharts() the package errors out when navigating to a page with an Apex Chart.
Could it be that there were breaking changes in v4.0.0 that I may have missed?

Hi,
Just to be clear this happens in debug mode, right?
Have you tried to clean your solution?

Hi,
Yes, this was occurring in debug mode.
Cleaning the solution was all I needed to do. Sorry about that. Thanks for the help!


Testing for API

Needs to be tested with mocks.

@MetaRossi
Identified a need for mock testing to ensure the robustness and reliability of our API interactions, particularly for environments where direct calls to live endpoints are impractical or risky. Recommending the integration of Apidog for its advanced mocking capabilities. Apidog allows for the creation of detailed mock responses, enabling us to simulate various scenarios and responses without hitting the actual services. This approach will aid in comprehensive testing across different cases and conditions.


I get a heap corruption error running test.cpp ( VS2015 )

Upon executing testReturnValues()  I get a heap corruption error on the VM::reallocateFn line of this function
Value::~Value() {
if (string_) {
VM::reallocateFn(string_, 0u);
}
}

I fixed an off-by-one error in the wrenpp::Value constructor for strings, which caused a crash in the test you mentioned.
Although I think the off-by-one error was causing your crash, I wasn't able to replicate the crash in the destructor -- what compiler were you using?

I am using VC2015
On Fri, Sep 16, 2016 at 11:09 AM, Johann Muszynski<EMAIL_ADDRESS>
wrote:

I fixed an off-by-one error in the wrenpp::Value constructor for strings,
which caused a crash in the test you mentioned.
Although I think the off-by-one error was causing your crash, I wasn't
able to replicate the crash in the destructor -- what compiler were you
using?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Nelarius/wrenpp/issues/8#issuecomment-247654913, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AHXmivqQjS4vchdXzZeivSwCu6NoxUWiks5qqs0tgaJpZM4J7y02
.

--
John Lagerquist
Chief Engineer
RallyTronics LLC
801-866-5981

Yeah, looked at the title of the issue again right after posting 😅

I'll close this since it's been a while. Feel free to reopen if necessary!


Added GnomeSort algorithm

added gnome sort algorithm, with added unit tests and updated DIRECTORY.md

Hi, i have just rebase my branch, could you help me check if is ok to merge now?


Add link to Monnaie Libre Occitanie

Please add a link for Monnaie Libre Occitanie for the flash conference:
index.html line 517 : change to
Principe et installation de la Brique Internet & monnaie libre
Thanks !

Oops sorry I was redirected to another repo. Noob here. Sorry about that :-(
Can I delete this issue somehow ?

Thanks @p3x-robot at least I learned something \o/


lets delegate to the KubernetesClient and let it figure out where the master is via env vars, service discovery or the ~/.kube/config file

lets delegate to the KubernetesClient and let it figure out where the master is via env vars, service discovery or the ~/.kube/config file

Refer to this link for build results (access rights to CI server needed):
https://fabric8-ci.fusesource.com/job/fabric8-quickstarts-master-pullreq/575/
Test PASSed.

Think we should delete the autoscaler tbh

On 22 Jun 2015, at 07:40, James Strachan<EMAIL_ADDRESS>wrote:
lets delegate to the KubernetesClient and let it figure out where the master is via env vars, service discovery or the ~/.kube/config file
You can view, comment on, or merge this pull request online at:
https://github.com/fabric8io/quickstarts/pull/525
Commit Summary
lets delegate to the KubernetesClient and let it figure out where the master is via env vars, service discovery or the ~/.kube/config file
File Changes
M apps/fabric8mq-autoscaler/src/main/java/io/fabric8/mq/autoscaler/MQAutoScaler.java (6)
Patch Links:
https://github.com/fabric8io/quickstarts/pull/525.patch
https://github.com/fabric8io/quickstarts/pull/525.diff
—
Reply to this email directly or view it on GitHub.

Wait - fusesource-ci has become sentient?! Wow! Why do you think we should delete it? Not useful?


Accessibility and parameter hints

VS Code Version: 1.64.0
OS Version: Windows 10 home 21h1
nvda version: 2021.3.1

Problems brief description
parameter hints(below called hints) is not always reported by nvda screen reader
Steps to reproduce
Say I have the following python script:
def func(str_=None, happy=True):
pass
for ease of illustration, below I will use x to mark the position of cursor,
ctrl+shift+space will be expressed as hint hotkey
func(x)

if I press the hint hotkey the first time, the nvda will report 'str_=, hint'
if I press the hotkey the second time, nvda do not report the hint
I try moving the cursor out of the bracket and moving into the bracket again, i.e. funcx() -> func(x) the hint hotkey do report the hint again

I try entering the first Parameter:
func(str_='hi',x)

if I press the hotkey at the above position, nvda do not report any hint

Please advice me if I am doing anything wrong, thanks.

@sine-w you are not doing anything wrong. Parameter hints use aria alert (live region) to announce its content. This is not a good approach and ideally we would do the same as the suggest widget does (move focus into the parameter hints and use activedescendent to make screen readers aware of the editor - parameter hints relationship).
That though has a problem, when the parameter hints get closed, the editor title and content would get re-read. So we would first have to tackle https://github.com/microsoft/vscode/issues/89098#issuecomment-1031505427 before we do the activedescendant work.
@jrieken @mjbvz who owns the parameter hints widget?
Here's a sketch of a PR that I did some time back which changes the paremeter hints to do exactly what I mention above - to use activedescendant https://github.com/microsoft/vscode/pull/93606/files

Let's close as dup of https://github.com/microsoft/vscode/issues/90874
And I will move my comment to that issue.


EPERM issue updating

Almost every other version from atom-beautify gives me issues while trying to update. I can't seem to find a clear reason or clear steps to reproduce, but you might have more inside then me.
The following error is shown when pressing the update button when a new version of atom-beautify is released:

Updating to<EMAIL_ADDRESS>failed.Hide output…
EPERM, unlink 'C:\Users\jerone.atom\packages\atom-beautify\styles'

Navigating to that folder gives me access denied in Windows. When I close Atom, I have access to that folder again.
I don't have any tabs open or any other processes running.
Atom 0.196.0
Windows 7 64-bits
Feel free to close this issue as it's probably not related to atom-beautify, it's just that it only happens to your package. Great package btw!

I have no idea why this would be occurring. You're right, it is likely not an Atom Beautify issue. Did you find a way to resolve this?

Did you find a way to resolve this?

Sadly no. Maybe it's fixed in Atom. I'll close this issue until I or someone else has more info.


Support for App & Environment resources

This is awesome 👍. I'm so excited to start using this.

@ejholmes this good to merge?

Implemented this with stacker here: https://github.com/remind101/stacker_blueprints/pull/24
The Custom::EmpireAppEnvironment is harder to implement cleanly with the way stacker currently handles parameters.
Ideally we want something like:
  - name: acmeAppEnv
    class_path: stacker_blueprints.empire.app.Environment
    parameters:
      ServiceToken: empireDaemon::CustomResourcesTopic
      AppId: acmeApp::AppId
      Variables:
        - Name: FOO
          Value: BAR
        - Name: TEST
          Value: HERE
        - Name: DATABASE_URL
          Value: db::DatabaseURL

but that syntax doesn't work with stacker PARAMETERS and it doesn't work with LOCAL_PARAMETERS because they don't resolve outputs like db::DatabaseURL. I'm going to add support for LOCAL_PARAMETERS resolving outputs so we can support that.

👍


Cleanup and add example

Don't be too scared of the diff, I mainly moved things into sub files (and improved a bit tests).

Updated.

Thanks ! :)


SETSUYAK

Charl says it is a Japanese word for 'CONSERVE' that is used by Palauans. Did we spell it right? Maybe SETZUIAK? Can anyone else confirm this please? Suggest how it should be spelled?

Setsuyaku actual Japanese word ‘u’ is silent.
On Wed, Mar 23, 2022 at 2:30 PM John Bent @.***> wrote:

Charl says it is a Japanese word for 'CONSERVE' that is used by Palauans.
Did we spell it right? Maybe SETZUIAK? Can anyone else confirm this please?
Suggest how it should be spelled?
—
Reply to this email directly, view it on GitHub
https://github.com/tekinged/missing/issues/12, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF2DJJG4ZFYSMRWKYRPLM6DVBOEPZANCNFSM5RPHUSIA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

Japanese dictionary search for "setsuyaku"
|
|
|  |
Japanese dictionary search for "setsuyaku"
3 words found. See the full details ...
|
|
|
On Wednesday, March 23, 2022, 03:23:05 PM PDT, Gnalu<EMAIL_ADDRESS>wrote:

Setsuyaku actual Japanese word ‘u’ is silent.
On Wed, Mar 23, 2022 at 2:30 PM John Bent @.***> wrote:

Charl says it is a Japanese word for 'CONSERVE' that is used by Palauans.
Did we spell it right? Maybe SETZUIAK? Can anyone else confirm this please?
Suggest how it should be spelled?
—
Reply to this email directly, view it on GitHub
https://github.com/tekinged/missing/issues/12, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF2DJJG4ZFYSMRWKYRPLM6DVBOEPZANCNFSM5RPHUSIA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>

I write it "setsuiak" just b/c Palauan doesn't have "y"
On Wed, Mar 23, 2022, 4:48 PM JimGeselbracht @.***>
wrote:

Japanese dictionary search for "setsuyaku"
|
|
| |
Japanese dictionary search for "setsuyaku"
3 words found. See the full details ...
|
|
|
On Wednesday, March 23, 2022, 03:23:05 PM PDT, Gnalu @.***> wrote:
Setsuyaku actual Japanese word ‘u’ is silent.
On Wed, Mar 23, 2022 at 2:30 PM John Bent @.***> wrote:

Charl says it is a Japanese word for 'CONSERVE' that is used by Palauans.
Did we spell it right? Maybe SETZUIAK? Can anyone else confirm this
please?
Suggest how it should be spelled?
—
Reply to this email directly, view it on GitHub
https://github.com/tekinged/missing/issues/12, or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AF2DJJG4ZFYSMRWKYRPLM6DVBOEPZANCNFSM5RPHUSIA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>
—
Reply to this email directly, view it on GitHub
https://github.com/tekinged/missing/issues/12#issuecomment-1076895499,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AXYSPXWSKEINQSYMX5IU77LVBONVNANCNFSM5RPHUSIA
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Option to revert change to Combined Feed Behavior

Hi! I know this is a strange request maybe and I apologize for my rather long-winded style of communication.
The recent change to combined feed really makes Zulip much harder for me to use. I searched the issues and can't find anyone else mentioning this and can't track down the PR that resulted in the change to point it out directly, I apologize.
To explain: I'm autistic and have additional movement and cognitive impairment. I found the combined feed view and the ability to switch between conversations and easily return to a neutral view an effective way to keep track of important conversations among my family and friends, as I have managed to get most of them to switch to using Zulip.
Recently, the Zulip app forces me to focus on a conversation when I reply to it in combined feed and does not show incoming notifications on the fly as they arrive, unless I manually back out of the conversation to return to the feed. I feel like this change makes the feed view significantly less usable, and I think it is the loss of one of this project's best features.
Can an option be implemented to use the old behavior? I know, as developers, there's always a good reason for changes like this, so I am not coming and saying "please change this back I don't like it" but the requirement of having to remember to use the backspace function and having to make my window wider has resulted in me missing important messages from people I love.
Thank you for reading this message.
-Ruby

@crsongbird thanks for the feedback! This change was definitely one where we were pretty confident that the new behavior would be preferred by most users, but weren't sure whether or not there would be some users who preferred the old workflow, so it's very helpful to hear from someone who did.
I've started a conversation in the development community here: https://chat.zulip.org/#narrow/stream/101-design/topic/jumping.20to.20sent.20message.20conversation.20.28experiment.29.20.2329186/near/1833691

I do appreciate the way Mr. Michael Davis phrased the request. I have to say, at first, I felt pretty similarly and had to soften my phrasing. You guys rock.
The proposal to have a dropdown of always focus / never focus in user options is good enough for me tbh. I don't personally see myself needing to set it on a per-stream basis, or needing to change the default for users serverwide on the server I admin.

Closing in favor of a specific plan in #30600. Thanks again for the report @crsongbird !

@crsongbird just to follow up, that setting is now merged, so you should be able to configure the old behavior again on Zulip Cloud within the next week or so. Thanks again for the detailed and specific feedback, it was very helpful!


@twind/next doesn't load custom twind.config.js

Hi! Thank you for the library, i don't know if i'm missing something or what but if i add a custom config for twind the styles doesn't work anymore https://codesandbox.io/s/twind-nextjs-forked-l5r91?file=/twind.config.js

TL;DR replace @twind/next/app with @twind/next/shim/app and @twind/next/document with @twind/next/shim/document
You are using the className attribute for styling and therefore need the shim modules: https://github.com/tw-in-js/use-twind-with/tree/main/examples/next-with-shim
See the this (forked) sandbox
In twind you can either do:
import { tw } from 'twind'

<main className={tw`h-screen bg-purple-400 flex items-center justify-center`}>
  <h1 className={tw`font-bold text(center 5xl white sm:gray-800 md:pink-700)`}>
    This is Twind!

Or use the shim:

This is Twind!

OMG, i think im lost today! Thank you for the clarification, and if i want to use it mixed i can't right? On with shim or without?

Nope. You can use both! No problem.

Thank you!!!


Issue :- #745 [Solved] :-   Added New Scheduling Algorithms Directory

Issue #745
Adding New Complete Scheduling ALgorithms Directory
Name:
Priority Scheduling Algorithm
About:
Propose the Priority Scheduling algorithm to be added to the repository. This scheduling method selects processes based on their priority, allowing for efficient handling of critical tasks. It can be preemptive or non-preemptive, depending on the implementation.
Name:
Round Robin Scheduling Algorithm
About:
Propose the Round Robin (RR) scheduling algorithm to be added to the repository. This preemptive algorithm assigns a fixed time slice to each process, allowing for fair time-sharing among processes in a cyclic manner. It is well-suited for time-sharing systems.

I have added scheduling algorithms in a very organized way and there are 3 algorithms present as scheduling algorithm are very important and has apllications like Graph colouring and more so in next pull request i will commit all other importatn one's there.


zoom for individual features in map.features.plot.batch

The description for the map.feature.plot.batch.features property says "an array of map.feature.plot payloads minus the messageId property".  What about the map.feature.plot.zoom property?  This seems like it should be ignored at the feature level as well.

Added text "...minus the messageId and zoom properties" will be committed in RC5


Placeholder text  is not visible properly in iOS 11, Any suggestion please let me know.

which pod version you using also which lib objc or swift??
Sent from my iPhone

On 22-Nov-2017, at 12:55 PM, nareshGoud413<EMAIL_ADDRESS>wrote:
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Swift
On Nov 22, 2017 1:47 PM, "Er Abhishek Chandnai"<EMAIL_ADDRESS>wrote:

which pod version you using also which lib objc or swift??
Sent from my iPhone

On 22-Nov-2017, at 12:55 PM, nareshGoud413<EMAIL_ADDRESS>wrote:
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ErAbhishekChandani/ACFloatingTextfield/issues/48#issuecomment-346275318,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Acl8wgasojhbNkf6fbGvv_zGgk-d4cFNks5s49iIgaJpZM4Qm9qL
.

I'm not using any pods I'm just manually added swift classes.
On Nov 22, 2017 1:47 PM, "Er Abhishek Chandnai"<EMAIL_ADDRESS>wrote:

which pod version you using also which lib objc or swift??
Sent from my iPhone

On 22-Nov-2017, at 12:55 PM, nareshGoud413<EMAIL_ADDRESS>wrote:
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ErAbhishekChandani/ACFloatingTextfield/issues/48#issuecomment-346275318,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Acl8wgasojhbNkf6fbGvv_zGgk-d4cFNks5s49iIgaJpZM4Qm9qL
.

thn please make sure you have downloaded the latest files. We have resolved the iOS 11 issues already if still facing issue please provide the code sample to reproduce it so that I can reproduce the issues thanks.
Sent from my iPhone

On 22-Nov-2017, at 5:12 PM, nareshGoud413<EMAIL_ADDRESS>wrote:
I'm not using any pods I'm just manually added swift classes.
On Nov 22, 2017 1:47 PM, "Er Abhishek Chandnai"<EMAIL_ADDRESS>wrote:

which pod version you using also which lib objc or swift??
Sent from my iPhone

On 22-Nov-2017, at 12:55 PM, nareshGoud413<EMAIL_ADDRESS>wrote:
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/ErAbhishekChandani/ACFloatingTextfield/issues/48#issuecomment-346275318,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Acl8wgasojhbNkf6fbGvv_zGgk-d4cFNks5s49iIgaJpZM4Qm9qL
.

—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

Thanks for responding, now its working fine I took latest file.

glad to know thanks
On Wed, 22 Nov 2017 at 5:36 PM, nareshGoud413<EMAIL_ADDRESS>wrote:

Thanks for responding, now its working fine I took latest file.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/ErAbhishekChandani/ACFloatingTextfield/issues/48#issuecomment-346331704,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ANMZ173jufGbh9av3aI_Vpc_3GCoxU81ks5s5A5VgaJpZM4Qm9qL
.

--
Thanks & Regards
Er. Abhishek Chandani


Changes required for ORDER_PAYMENT network message

Reopening of #92 as a local branch. This also adds the coin-specific ExtractPubkeyAddrs functions which was missing from the prior PR.

Pull Request Test Coverage Report for Build 456

0 of 79 (0.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.8%) to 53.565%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

service/wallet_service.go
0
4
0.0%

litecoin/wallet.go
0
23
0.0%

zcash/wallet.go
0
23
0.0%

bitcoin/wallet.go
0
29
0.0%

Totals

Change from base Build 436:
-0.8%

Covered Lines:
2757

Relevant Lines:
5147

💛  - Coveralls


"FREE pdf book link"

The "FREE pdf book" download link is not working.

Fixed!
Thanks for pointing that out!


Fix bug for aggregation + distinct SQL

Fixes #2534.

Pull Request Test Coverage Report for Build 8437

17 of 47 (36.17%)  changed or added relevant lines in 7 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.07%) to 60.661%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

sharding-core/sharding-core-rewrite/src/main/java/org/apache/shardingsphere/core/rewrite/token/generator/SelectItemsTokenGenerator.java
1
2
50.0%

sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/constant/DerivedColumn.java
4
8
50.0%

sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/sql/segment/dml/SelectItemsSegment.java
0
6
0.0%

sharding-core/sharding-core-parse/sharding-core-parse-common/src/main/java/org/apache/shardingsphere/core/parse/optimizer/select/SelectOptimizer.java
0
19
0.0%

Totals

Change from base Build 8434:
-0.07%

Covered Lines:
8256

Relevant Lines:
13610

💛  - Coveralls


Use of uninitialized value $value in addition

Hello,
I use the template to monitor Ngnix and get this error
Use of uninitialized value $value in addition (+) at /etc/zabbix/zabbix-nginx-stats/zabbix-nginx-stats.pl line 146, <> line 1.
Thank you for your help

Hi,
i have same problem :)
i can try to explore this error/warning.


deterministic-uname: init

for reproducibility
based on https://github.com/NixOS/nixpkgs/pull/64258
Closes https://github.com/NixOS/nixpkgs/issues/140175 and probably more
Description of changes

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
23.05 Release Notes (or backporting 22.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[ ] Fits CONTRIBUTING.md.

sidenote
coreutils uname has basic supports for env vars but only on darwin since https://github.com/coreutils/coreutils/commit/e3d1ff04370b3e090e4cfeb8f6cc4a63590a516f

https://github.com/NixOS/nixpkgs/pull/210569

This seems to have broken the tarball job:
Nixpkgs is not allowed to use <nixpkgs> to refer to itself.
The offending files: /nix/store/79pm0hmza6jwi2swa62kzlh549ivy9jh-source/pkgs/build-support/deterministic-uname/default.nix

Would be awesome if this could be used in low-level packages (perl for example, adding deterministic-uname to nativeBuildInputs triggers infinite recursion) to build NixOS for armv7l-linux on aarch64 machines without patching the kernel.

for that we need to get rid of the coreutils and getopt dependencies or do something with bootstrapTools


Exception thrown in IE when dragging over an svg

I'm debugging an exception that seems to be specific to IE (only tested in IE11). The exception, thrown here, is "Unable to get property 'insertBefore' of undefined or null reference". In looking into this, I've discovered that parentElement (called from findDropTarget) is undefined in IE11 when target is an svg.
I'd be happy to open a PR for this, but I'm unsure whether it's more correct to update this line to return target || null, or if it would be better to use parentNode over parentElement (which seems to return the desired element/node). I'm thinking the latter, but that begs the question: should all instances of parentElement be replaced with parentNode?

Nice catch.
I hadn't thought of that.
We should use parentNode in there for sure.
I can't think of a reason not to switch to parentNode everywhere.

Okay, cool. I'll open a PR shortly.


Citr support

Adding new commands for citr:

cf start-restore SERVICE_INSTANCE_NAME --timestamp TIME_STAMP --from-instance SOURCE_INSTANCE_NAME
cf start-restore SERVICE_INSTANCE_NAME --timestamp TIME_STAMP --from-instance SOURCE_INSTANCE_NAME --deleted
cf start-restore SERVICE_INSTANCE_NAME --timestamp TIME_STAMP --from-instance-id SOURCE_INSTANCE_GUID

@sauravmndl / @pritishmishra : can you review this and merge ?

There are few improvements required in

In help  of commands
Messaging
Few cosmetic changes on restore command like instead of --from-instance, --source or --source-instance like that.


Create Cybersecurity: 3 Computer Viruses? #1671.md

Description 📜
Explanation of What are Computer Viruses? How it spreads, its types and ways to protect against computer virus.

Type of change 📝

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

Domain of Contribution 📊

[x] Cyber #Security

Checklist ✅

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

Screenshots / Gif (Optional) 📸

Please review it @Anant1711 .

Re-review it please @Anant1711


What type of update are you requesting?
A new version of an existing package
Current Package Identifier
Mikrotik.Winbox
Package Version
4.0beta9
Please describe the changes you would like to see
There is a new beta release (4.0beta9) available. Download Page Url: https://mikrotik.com/download (https://download.mikrotik.com/routeros/winbox/4.0beta9/WinBox_Windows.zip)
Make sure to add the app using a Mikrotik.Winbox.Beta namespace.

Since https://github.com/microsoft/winget-pkgs/pull/183062 is merged, I'll assume this PR is complete. If this is in error, please @ me and I'll reopen.


Adds changes recommended from periodic review

Periodic metric review of older metrics from @Preshh0 and @oma131 resulted in the following changes:

updating language throughout the metric, in most sections
rewording some of the survey questions
adding date of last review

Thank you for your hard work on these!
Signed-off-by: Elizabeth Barron<EMAIL_ADDRESS>

Thank you!


Invalidate macro source when its dependency changes

Invalidate macro classes that transitively depend on any of the recompiled classes
The macro expansion tree can depend on the behavioural change of any upstream code change,
not just API changes, so correctness requires aggressive recompilation of downstream classes.
Technically the macro doesn't need to be recompiled - it's the classes downstream of the macro
that need to be recompiled, so that the macros can be re-expanded.  But recompiling is the most
straightforward way to signal any classes downstream of the macro that they need to recompile.
Also, note, that this solution only works for behavioural changes in sources within the same
subproject as the macro.  Changes in behaviour in upstream subprojects don't cause downstream
macro classes to recompile - because downstream projects only have visibility of the upstream
API, and if it changed, which is insufficient, and upstream projects have no other way than
their API to signal to downstream.

@Friendseeker you've been doing invalidation stuff recently (which is noticed and much appreciated!) — perhaps you'd like to do a little reviewing on this one?

@Friendseeker you've been doing invalidation stuff recently (which is noticed and much appreciated!) — perhaps you'd like to do a little reviewing on this one?

Thank you for considering me for this review; I really appreciate it! I must admit, my experience with Scala macros is quite limited. However, I'm willing to take a closer look at the PR and provide my comments where I can.

@dwijnand I've reviewed the changes in your PR and understand the modifications you've made. However, to get a complete picture, could you please provide some insight into the reasoning behind this approach? It appears to be a quick fix, and I'm curious if there were specific constraints or considerations that led to this solution.

I went with the first fix that worked, not being able to tell where I should look to slot this fix in.  In particular I ended up moving my change to "initial changes" from detectAPIChanges and invalidateAfterInternalCompilation where I was originally looking.  I'm open to suggestions and ideas.  Meanwhile I'll study your transitive counter-example.

We can either address the edge case in this PR, or merge this PR first and create a new PR to address this edge case. I think either is fine.

I think we should defer this part, perhaps to never.
The first reason is because I think it's common practice in projects to have a subproject dedicated for a macro, and that subproject doesn't have any dependencies.  And the reason for that (IIRC and if things haven't changed) because declaring a macro dependency only puts the macro jar on the compilation classpath so it's present during macro expansion, without any of it (potential) dependencies - thus only scala-library, scala-reflect and the jdk of course.  Now technically a project could develop that jar as separate subprojects and then assemble it into one jar, and then could suffer this problem, but... perhaps that reality doesn't exist and even if so, it's "only" incremental builds.
The second reason is I can't find a good way to implement that.  You mentioned that to address this you need to traverse external dependencies.  That won't work.  The intra-subproject solution we have here takes the "recompiledClasses" (which might or might not have API changes), e.g. lib.InternalApi, and deduces if there are any macro classes, e.g. lib.Macro, that (transitively) depend on any of those recompiled classes.  And if there are, the source files associated for those macro classes are recompiled.  If the macro is in a downstream project, we don't have the information locally.  If, instead, the current project is the one that contains the macro and the code that changes are upstream, e.g. util.InternalApi, then we don't even have visibility on util.InternalApi because lib.Macro only has visibility of util.Data the public API, and that API didn't change.
I'll detail that in the PR description and in the code.

We can either address the edge case in this PR, or merge this PR first and create a new PR to address this edge case. I think either is fine.

I think we should defer this part, perhaps to never.
The first reason is because I think it's common practice in projects to have a subproject dedicated for a macro, and that subproject doesn't have any dependencies.  And the reason for that (IIRC and if things haven't changed) because declaring a macro dependency only puts the macro jar on the compilation classpath so it's present during macro expansion, without any of it (potential) dependencies - thus only scala-library, scala-reflect and the jdk of course.  Now technically a project could develop that jar as separate subprojects and then assemble it into one jar, and then could suffer this problem, but... perhaps that reality doesn't exist and even if so, it's "only" incremental builds.
The second reason is I can't find a good way to implement that.  You mentioned that to address this you need to traverse external dependencies.  That won't work.  The intra-subproject solution we have here takes the "recompiledClasses" (which might or might not have API changes), e.g. lib.InternalApi, and deduces if there are any macro classes, e.g. lib.Macro, that (transitively) depend on any of those recompiled classes.  And if there are, the source files associated for those macro classes are recompiled.  If the macro is in a downstream project, we don't have the information locally.  If, instead, the current project is the one that contains the macro and the code that changes are upstream, e.g. util.InternalApi, then we don't even have visibility on util.InternalApi because lib.Macro only has visibility of util.Data the public API, and that API didn't change.
I'll detail that in the PR description and in the code.

Your reasoning makes a lot of sense. I agree with deferring.


where can i find the Class which name is fcp.client.tasks.Task？

where can i find the Class which name is fcp.client.tasks.Task？

I'm sorry, I don't believe that class is currently a part of our open-source code base.


how to render icon or image horizontally with text label?

_renderIcon = ({ route }) => (

);
TabBar labelStyle={{color:'black'}} {...props} renderIcon={this._renderIcon} style={{backgroundColor:'white'}} indicatorStyle={{backgroundColor:'#f56231'}} />;
this works in vertical direction

<TabBar tabStyle={{flexDirection: 'row'}}/>
that works for me

omg thankyou so much.......

Hey, I just released a new alpha 2.0.0-alpha.0 of the library. It's rewritten using react-native-gesture-handler and react-native-reanimated addresses a many platform specific bugs and performance problems. The documentation is updated as well.
Please try the new version and see if it addresses your issue. If not, please open a new issue following the issue template.


fix: default ttl set to 72 hours on reset

fixes #227
replaced the 3 hours ttl with 72 hours (default ttl) on reset

Thank you :)


Counting Sort Added

Hi,
This PR to add the counting sort algorithm
Assigned with issue #14
I hope this helps other developers
Thanks

Looks good to me :)

Thanks :)

Looks good to me :)

Thanks :)


ErrorException: socket_read(): unable to read from socket [4]: Interrupted system call

I have got some errors:
ErrorException: socket_read(): unable to read from socket [4]: Interrupted system call - (stripped)/vendor/stefanak-michal/bolt/Socket.php:136

#1  (stripped)/vendor/stefanak-michal/bolt/Socket.php(136): socket_read(Resource id #15, 2, 2)
#2  (stripped)/vendor/stefanak-michal/bolt/Socket.php(104): Bolt\Socket::readBuffer(2)
#3 (stripped)/vendor/stefanak-michal/bolt/protocol/V4.php(41): Bolt\Socket::read(Object(Bolt\PackStream\v1\Unpacker))
#4  (stripped)/vendor/stefanak-michal/bolt/protocol/V4.php(21): Bolt\protocol\V4->pull()
#5  (stripped)/vendor/stefanak-michal/bolt/Bolt.php(240): Bolt\protocol\V4->pullAll()
#6  (stripped)/Helper/DB.php(27): Bolt\Bolt->pull()

Hi.
I like to know a little more informations.

Where is running neo4j instance? (same pc as php application or different machine)
What platform do you have? (linux, win, macos)
What version of Neo4j do you have?
How it looks your cypher query you call in run() before pull()?

What version of Neo4j do you have?
^1.1
How it looks your cypher query you call in run() before pull()?
This query i've used:
MATCH (u:User)-[:FOLLOWED]->(t:Tag)<-[:BELONG_TO]->(th)
WHERE u.user_id = %1$d AND NOT EXISTS((u)-[:READ]->(th))
WITH u.user_id AS user_id, th.thread_id AS thread_id,
                (0.5*th.like_count + 0.3*th.comment_count + 0.005*th.view_count
                    + duration.inDays(datetime(),datetime(th.published_date)).days*100) AS point
ORDER BY point desc
RETURN DISTINCT thread_id
LIMIT %2$d

Can I ask, what is this %1$d? From Neo4j version 4 parameters are $name. Do you have specific own parser for queries?
https://neo4j.com/docs/cypher-manual/current/syntax/parameters/
If I change your query parameters to correct format and call it like this:
$bolt = new \Bolt\Bolt();

if (!$bolt->init('MyClient/1.0', $user, $password)) {
    throw new Exception('Wrong login');
}

$bolt->run('MATCH (u:User)-[:FOLLOWED]->(t:Tag)<-[:BELONG_TO]->(th)
  WHERE u.user_id = $id AND NOT EXISTS((u)-[:READ]->(th))
  WITH u.user_id AS user_id, th.thread_id AS thread_id,
                (0.5*th.like_count + 0.3*th.comment_count + 0.005*th.view_count
                    + duration.inDays(datetime(),datetime(th.published_date)).days*100) AS point
  ORDER BY point desc
  RETURN DISTINCT thread_id
  LIMIT $limit', ['id' => 1, 'limit' => 2]);
var_dump($bolt->pull());

it returns (xdebug on), which is correct because last response entry is statistics:
D:\Programovanie - www\Bolt\index.php:37:
array (size=1)
  0 =>
    array (size=4)
      'bookmark' => string 'FB:kcwQCSYQnT3QRKyGhkGXdnJGHxyQ' (length=31)
      'type' => string 'r' (length=1)
      't_last' => int 0
      'db' => string 'neo4j' (length=5)

I performed this code on Neo4j 4.1.1 and php 7.1.13
Also there can be a problem with new format of parameters, which is double quote string in PHP. Because in double quoted string PHP is parsing variables with dollar sign.

It's template then i fill params by using sprintf function.

Okay, I tried that with sprintf and it works. Doesn't seem problem is with query. This is first query you run or there is others before?
You can try turn on hex debug with \Bolt\Bolt::$debug = true; and post it here.

I only run that query then pull. It's not 100% occurred that bug. I see some errors in prod not have issue on local dev.

I read something about that socket error and maybe you can try disable socket blocking, comment line 44 in Socket.php socket_set_block(Socket::$socket);

Is there anyway to check if that has been disabled?

https://github.com/stefanak-michal/Bolt/blob/master/Socket.php#L44
I would suggest to check if we are successfully block.
if (socket_set_block(Socket::$socket) === false) {
    // throw some error due cannot block socket
}

I don't understand. Did you discover the blocking operation wasn't successful at the exception situation?

At this moment. No.

I made some small upgrades to Socket.php. Catch error on set block mode how you suggested, also catch error on socket_read.
https://github.com/stefanak-michal/Bolt/commit/b813f198b3bfc7ff81625c615d9015d30bd70be4

If it happen only on production, how big is workload on it?

It's still error.

By how you describe the problem it doesn't look like problem with this library, but it can be problem with your environment configuration. I'm just trying to help you identify your problem, I don't think I can solve it.

@stefanak-michal i think so. I will update issue status when our server has some changes for this.


Using SelectedIndex on Combobox does not show selected item

Describe the bug
I populate a ComboBox at runtime and once I have added all my items, I set the SelectedIndex property of the Combobox via an ObservableProperty. However, the combox remains blank and does not show the selected item.
To Reproduce
This is my very simple test view:
	<StackPanel Orientation="Vertical" Margin="8">
		<ComboBox Grid.Row="2" Grid.Column="0" ItemsSource="{Binding Items}" SelectedIndex="{Binding SelectedItemIndex}" Width="300"/>
		<TextBlock Grid.Row="3" Text="{Binding SelectedItemText}"/>
		<Button Grid.Row="5" Command="{Binding Populate}">
			Populate Items
		</Button>
	</StackPanel>

And the view model that is attached to this view:
public partial class MainViewModel : ViewModelBase
{
    [ObservableProperty]
    private ObservableCollection<string> items = new ObservableCollection<string>();

    [ObservableProperty]
    private int selectedItemIndex;

    [ObservableProperty]
    private string selectedItemText;

    [RelayCommand]
    public void Populate()
    {
        Items.Clear();

        for ( int i = 1 ; i < 11 ; i++ )
        {
            Items.Add( $"Item {i}" );
        }

        SelectedItemIndex = 0;
    }

    public MainViewModel()
    {
        SelectedItemText = "NO SELECTION";
    }

    partial void OnSelectedItemIndexChanged( int value )
    {
        SelectedItemText = Items[ value ];
    }
}

Expected behavior
I would expect to see the first item text to appear in the Combobox rather than it remaining blank as shown below.
Screenshots

Environment

Windows 11 x64
Avalonia 11.0.4

Andy

Okay, it seems that I need to use the SelectedItem to cause the selected item to appear.
I would say that this is a bug - specifying SelectedIndex should also show the selected item.
Andy

okay that's not an actual bug. SelectedIndex is 0 by default. So if you set it to 0 again, the UI does not get notified as nothing has changed for the ViewModel. Either set SelectedIndex=-1 or RaisePropertyChanged(nameof(SelectedIndex)) in populate. See:
public partial class MainViewModel : ViewModelBase
{
    [ObservableProperty]
    private ObservableCollection<string> items = new ObservableCollection<string>();

    [ObservableProperty]
!   private int selectedItemIndex = -1;

    [ObservableProperty]
    private string selectedItemText;

    [RelayCommand]
    public void Populate()
    {
        Items.Clear();

        for ( int i = 1 ; i < 11 ; i++ )
        {
            Items.Add( $"Item {i}" );
        }

        SelectedItemIndex = 0;
    }

    public MainViewModel()
    {
        SelectedItemText = "NO SELECTION";
    }

    partial void OnSelectedItemIndexChanged( int value )
    {
        SelectedItemText = Items[ value ];
    }
}

NOTE
I'll make a discussion from it. Please ask questions not as an issue but as a Q&A or on Telegram if you're new to all of this. We can still make an issue of it if it turns out it acutally is an issue. Thx. 🙏


Call to undefined method App\Models\Dir::factory()

Laravel Version: 8.2.0

Description:
I am currently making a test. However, everytime I run it I get the following error:   Call to undefined method App\Models\Dir:factory() I do not know why I get this. I also am doing use App/Models/Dir but I still get this error. How can I solve this?
Error:
BadMethodCallException Call to undefined method App\Models\Dir::factory()

This seems like a question for the forums since this is support. Does your App\Models\Dir Model use the HasFactory trait?

Hi, yes it does.
use Illuminate\Database\Eloquent\Factories\HasFactory;

Just having that at the top of the file does not mean that your model is using that trait. Is there a use HasFactory; statement inside the class definition?

No, there is now a use HasFactory at the top of of class dir. I know have the following error:   Class 'Database\Factories\AssortmentFactory' not found

This is support, not a bug. You can take this to one of the forums like laracasts or laravel.io and ask.

I solved it. Thanks for the help


app closed launch function??

hello I wanted to know if it was possible to launch a function of timing and make it run even if the app is closed ?
there is a plugin as it does wotzapp that remains always open to speak!!
thank you

To launch it where? If the app is closed you don't have access to the webview. Could you elaborate?

Hello,
I have probably a similar question.
Is it possible to launch the app if it is closed on receiving a notification? I tried the high with notifications with high priority - unfortunately, it does not help.

If the app is not running (the plugin's webview is null) it caches the notification payload data see here
When the user taps on the notification that launches the app and when you register the notifications callback you are going to receive the cached payload. Is that not working for you?

Nicolás, thanks for your quick response.
Sure, it's working. However, in our case, we need to get the app running immediately without a notification (no 'title' or 'body' to specify in the payload) - say, for an incoming call, which is supposed to fire once the app gets loaded. I thought it's a default behaviour for all high priority notifications, but it seems that I was wrong.

I think I see what you mean, but I don't know if that is possible check this link about content-available flag.
I'm not sure if you can put the app in foreground.
There is some work being done on android to emulate that behavior aswell this issue

From this changes, I see that the behaviour with "content-available: 1" is the same as now with no "title" and "body" keys. Unfortunately, it does not instantly launch the application if it is closed - the notification gets delivered to JavaScript only on the next app run.

Yes exactly, I don't know if it is possible to launch it without user interaction.

Well, Skype somehow does that unless force closed. Now I'm trying to figure out how.

True, keep us updated with your findings ;)

Works! :)
What I did is added a new parameter, "force_launch". If received a message with "force_launch": 1, it restarts the PushHandlerActivity, onCreate method of which restarts the MainActivity. This makes the app start if it was closed or move to foreground if it was hidden. I can push the changes if you need them.

Please do so @EternallLight, then some work will need to be done on the other platforms but it is a good start to begin with! Thanks.
@macdonst do you agree?

I've made a fork and committed my changes there. Not sure what branch I have to make a pull request for. Since it's done only for one platform, it might be a good idea to create a separate branch for this, wait until another platforms will be ready, and merge it to the next release branch.

@EternallLight please just submit a PR for it. Sometimes what is possible for one platform is not possible for another, no sense depriving folks of the new functionality.

@macdonst I've just submited a pull request, please check it out.


Filter non-TCF systems from datamap (fides)

part of PROD-1399
done alongside https://github.com/ethyca/fidesplus/pull/1241
Description Of Changes
Provides some basic utility functions to allow us to easily retrieve only non-TCF systems (i.e. systems that are neither GVL nor AC vendor-based) from our DB.
This is just a starting point, we may want to refactor these utilities if/when they're needed in other places. For now, we need them in fidesplus to complete https://ethyca.atlassian.net/browse/PROD-1399.
This PR also includes the FE change necessary to display the requested notice on the datamap page.
Code Changes

[x] utility functions to support filtering out GVL and AC systems from queries that retrieve System records

[x]  some minor reorg of TCF utility code to support the above

[ ] TODO: FE updates to display requested notice on datamap pages if TCF is enabled

Steps to Confirm
see fidesplus PR for testing steps
Pre-Merge Checklist

[ ] All CI Pipelines Succeeded
Documentation:

[ ] documentation complete, PR opened in fidesdocs
[ ] documentation issue created in fidesdocs

[ ] Issue Requirements are Met
[ ] Relevant Follow-Up Issues Created
[ ] Update CHANGELOG.md
[ ] For API changes, the Postman collection has been updated

UAT  -  https://www.loom.com/share/cb1c8deec56246efa3b0f6dc3c958608

awesome thank you @Kelsey-Ethyca! i'll give it one last run through locally after all the changes then i'll plan to merge and cherry pick 👍


Notes on fit calcium imaging demo

This is a great addition, and impressive that you added it so quickly!
I would suggest adding details about the fluorophore (e.g., gcamp6f?) and its kinetics, so we have a better sense of what we are recording here. When we load the calcium traces, I would probably add a few plots of the raw traces just to give people a sense of what the raw data look like (about half of users will have strong intuitions for ephys, but not optical phys, so I think just giving them a view of the slow time course etc will be useful).
Generally, this notebook marks a pretty major shift in the GLM theory from the LNP backbone we have been following until now. I guess it's now an LNG model :man_shrugging:  Previous notebooks take quite a bit of time to explain conceptual transitions additions. I think this deserves a bit of space. E.g. currently there is:

Since the transients are non-negative, we will use a Gamma distribution from nemos with a soft-plus non linearity.

I think way more is going to be needed to massage intuitions about what exactly is happening in the shift from Poisson to Gamma.  For example, something like:

Everything up to this is still the same, but we are adding a new noise/observation model.
Need continuous distribution because we are no longer modeling a count process.
Previously we interpreted the output of the linking function as mean firing rate now we interpret it as mean calcium signal and we use it to generate a noisy signal on top of that. We can't use Gaussian because Gaussian has negative values.

Something like that -- that's just me spitballing not finished or good. :laughing:
I think it would help to plot, like in the intro to GLM with LNP case, the output of the rate estimate, and then the output of the Gamma observation model, so people can visualize how the operations work with these continuous signals.
We should probably add R^2 for the linear regression vs GLM, it's hard to compare them by just looking.
In general, while this is promising, it would help to have a discussion of why the results are so far off sometimes (there are clear false positive and false negative calcium signals). I'm not sure what needs to be added to it to improve accuracy? Gamma params need some further constraints? Coupling params needed? Maybe it's better for high SNR units (it's not too hard to calculate SNR from Ca  traces -- I wonder if it would be better for 2p data which has better SNR).
Anyway those are my initial thoughts this seems very cool and an impressive addition to nemos! It is concerning that the results are so similar to linear regression. A lot to chew on here.
Sorry I'm getting tired but I can tweak it with a PR in some of the ways above, and put in little notes where I would need help or am unsure of things.

Hei Eric, this is great feedback.  I wonder if you want to open a PR improving the flow of the notebook with your suggestions, specifically spelling out the differences between the Poisson and the Gamma model.
For the R^2 values, we should keep in mind that the linear regression is minimizing directly the mean squared error, so in the training set it may have higher scores, but also, I never did that, so we should try and see.
I agree that the linear regression is similar to the glm, this is not entirely unexpected. The advantages of the gamma are the positiveness. If one wants to simulate an firing rate, enforcing positivity is what you want. If the scale is held constant (which we assume), the Gamma, has a similar property to the Poisson model, i.e. the relation between the mean and the variance is linear.
This assumption is generally true for the spike count variability, does it hold also for calcium traces?
If so, we can come up with an example in which the assumption is important to estimate well the parameters, and show that a linear regression could may perform poorly

Thanks Edoardo -- let me start a PR this weekend and start with some of the easier changes and some of the tougher ones I'll at least put some placeholders in where I might need some help moving them forward.

Sounds good! 👍


Adding white and black colors

Trying to add additional white and black colors to the picker and failing. Any ideas on how to implement?
Thanks

I Second this. Need option to add more colors

@iSaadSalman @jay-cohen You can already pick B/W color by dragging outside of the control.
Is this what you are looking for?

More of a visual representation. Ideally there should be a bool method which shows white and black above and below the color gradient. Maybe gesture detection allowing it to go straight to black/white if they are over it rather than the current way.

@jay-cohen @iSaadSalman @wassup- This will be added as part of a larger refactor in the coming weeks. Unfortunately in the current implementation, there's no single source of truth of which colors are used (it shows all HSB colors with hues between [0.0, 1.0], saturation 1, brightness 1).

@gizmosachin  Can you recommend a manual solution until you the refactor you mentioned?
Thanks :)

@iSaadSalman You can modify commonInit to change which colors appear in the slider and updateForTouch to respond correctly when a touch moves within the slider, though it may be easier to add manual buttons for black and white to your app outside the ColorSlider.

@gizmosachin  Thanks for reply :)

@TheSwiftyCoder @iSaadSalman @wassup- There's an in-progress version of the refactor mentioned above that's available on this branch. There are a few breaking changes from the current version of ColorSlider, but the README and demo app have been updated to match. Ready for use now, should be merged in the next two weeks.

This has been merged in v4.0.

Perfect. Thanks @gizmosachin