Fix several NPEs in ReplicaThread

After enabling Nonblocking network client we found some NPEs in ReplicaThread.

In filterTimedoutRequests method, when the group doesn't exist, we try to call group.getId() and that's NPE.
In handleReplicaMetadataResponse method, when the response has an error, we try to process it anyway and that's NPE.
In handleGetResponse method, when the response has an error, we try to process it anyway and that's NPE.

This PR fixed all of them

Codecov Report
Base: 68.71% // Head: 14.60% // Decreases project coverage by -54.11% :warning:

Coverage data is based on head (56ec0c0) compared to base (28f9312).
Patch coverage: 0.00% of modified lines in pull request are covered.

Additional details and impacted files
@@              Coverage Diff              @@
##             master    #2345       +/-   ##
=============================================
- Coverage     68.71%   14.60%   -54.11%
+ Complexity    10025     2366     -7659
=============================================
  Files           765      765
  Lines         59122    59139       +17
  Branches       7247     7251        +4
=============================================
- Hits          40624     8639    -31985
- Misses        16139    49987    +33848
+ Partials       2359      513     -1846

Impacted Files
Coverage Δ

...va/com/github/ambry/replication/ReplicaThread.java
0.00% <0.00%> (-79.86%)
:arrow_down:

.../src/main/java/com/github/ambry/router/Router.java
0.00% <0.00%> (-100.00%)
:arrow_down:

.../src/main/java/com/github/ambry/store/Message.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...rc/main/java/com/github/ambry/named/PutResult.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...rc/main/java/com/github/ambry/quota/QuotaMode.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...rc/main/java/com/github/ambry/quota/QuotaName.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...rc/main/java/com/github/ambry/rest/RestMethod.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...c/main/java/com/github/ambry/commons/Notifier.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...c/main/java/com/github/ambry/config/KMSConfig.java
0.00% <0.00%> (-100.00%)
:arrow_down:

...c/main/java/com/github/ambry/config/SSLConfig.java
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 578 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Improve autocomplete

Pull Request Test Coverage Report for Build 715

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.02%) to 82.419%

Files with Coverage Reduction
New Missed Lines
%

src/main/java/seedu/planner/logic/commands/EditLimitCommand.java
1
0.0%

Totals

Change from base Build 687:
-0.02%

Covered Lines:
2944

Relevant Lines:
3572

💛  - Coveralls


Revert zephyr library ifdef

This PR reverts parts of #6835 to resolve #7565

Codecov Report

Merging #7567 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #7567   +/-   ##
=======================================
  Coverage   55.25%   55.25%
=======================================
  Files         468      468
  Lines       51673    51673
  Branches     9893     9893
=======================================
  Hits        28553    28553
  Misses      19207    19207
  Partials     3913     3913

Continue to review full report at Codecov.

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

I'm a little confused here.
It appears we are reverting @AdithyaBaglody patch series which defines ZEPHYR_SUPERVISOR for core kernel code.
Have you talked to Adithya about this? Do we understand a workable way forward?


[EPIC] V1.0.0

EPIC issue for the remaining v1 tasks

Will save the "prio:high" label for the tasks since they may vary in priority.


BUG: Content editable react component

[x] I am reporting a bug

Expected Behavior
when I click on the name of a mark in the visualization sidebar, I can update the name, but there is the following error in the console:
warning.js:45 Warning: A component is `contentEditable` and contains `children` managed by React. It is now your responsibility to guarantee that none of those nodes are unexpectedly modified or duplicated. This is probably not intentional.

Reading related to the error itself: https://github.com/facebook/react/issues/2477 (and by extension https://medium.com/medium-eng/why-contenteditable-is-terrible-122d8a40e480#.aq2xacp42 ) -- we've made the choices we did for a reason, but contenteditable is basically always going to shout at us in the console. The main purpose of this ticket then becomes, "make it so that the change persists throughout the app."


consumes support for PATCH

Trying to get our Swagger docs to pass some validations and noticed that for some reason the consumes setting was not supported for PATCH methods. This should fix the issue.

Coverage increased (+1.7%) to 99.644% when pulling 1e535b7328c545c8d6035f248ac680cfd3db3b19 on anakinj:support-consumes-on-patch into 3a93218bafdc89970336ecd6a26ee2defad5fb93 on ruby-grape:master.

Coverage increased (+1.7%) to 99.644% when pulling 2379f9263cb004dd34bd7eb5e8deb1425751c481 on anakinj:support-consumes-on-patch into 3a93218bafdc89970336ecd6a26ee2defad5fb93 on ruby-grape:master.

Coverage increased (+1.7%) to 99.644% when pulling 2379f9263cb004dd34bd7eb5e8deb1425751c481 on anakinj:support-consumes-on-patch into 3a93218bafdc89970336ecd6a26ee2defad5fb93 on ruby-grape:master.

Coverage increased (+1.7%) to 99.644% when pulling 00a1ebac22a15adbd8d8047607328933259ee7aa on anakinj:support-consumes-on-patch into 3a93218bafdc89970336ecd6a26ee2defad5fb93 on ruby-grape:master.

Coverage increased (+1.7%) to 99.644% when pulling 00a1ebac22a15adbd8d8047607328933259ee7aa on anakinj:support-consumes-on-patch into 3a93218bafdc89970336ecd6a26ee2defad5fb93 on ruby-grape:master.

thanks @anakinj


Add DigitalOcean provider

https://developer.hashicorp.com/packer/plugins/builders/digitalocean

Done during the integration team hackthon


Resolves #3171 - enable ksql tables, streams sorting

[ ] Breaking change? (if so, please describe the impact and migration path for existing application instances)

What changes did you make? (Give an overview)
Enable sorting in the "KSQL DB" section for Tables and Streams lists
Is there anything you'd like reviewers to focus on?
How Has This Been Tested? (put an "x" (case-sensitive!) next to an item)

[ ] No need to
[x] Manually (please, describe, if necessary)
[ ] Unit checks
[ ] Integration checks
[ ] Covered by existing automation

Checklist (put an "x" (case-sensitive!) next to all the items, otherwise the build will fail)

[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas
[x] I have made corresponding changes to the documentation (e.g. ENVIRONMENT VARIABLES)
[x] My changes generate no new warnings (e.g. Sonar is happy)
[x] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[x] Any dependent changes have been merged

Check out Contributing and Code of Conduct
A picture of a cute animal (not mandatory but encouraged)

@Haarolean looks like it's been forgotten 🤔

@michal-cesek have you fixed this or is it still in progress? Tried to build it and there is no sorting at KSQL at all.

@BulatKha it's still in progress

@BulatKha, it's done. I observed a similar bug with the pagination, so the same fix was applied to the pagination change handler.

@Haarolean @michal-cesek looks ok now

@michal-cesek thank you


Wrong fallback.

I noticed that DXVK is also active somehow on D3D9 games to, without taking part on them. But if DXVK fails somehow, it doesn't pass the torch to Gallium-Nine.

No.

DXVK doesn't affect D3D9 games. They don't even load the DLLs.

Okay, so the problem is not that "DXVK uses the wrong fallback", but that it doesn't catch one of its own exceptions when instance creation fails for whatever reason.
Please don't make random assumptions like that without posting any log files etc.

D3D9
mars@192:~/Games/Life_is_Strange/Binaries/Win32> DRI_PRIME=1 thread_submit=true wine LifeIsStrange.exe
000b:fixme:winediag:start_process Wine Staging 3.21 is a testing version containing experimental patches.
000b:fixme:winediag:start_process Please mention your exact version when filing bug reports on winehq.org.
0009:fixme:ver:GetCurrentPackageId (0x1d1e024 (nil)): stub
0009:fixme:gameux:GameExplorerImpl_VerifyAccess (0x1673a0, L"Z:\home\mars\Games\Life_is_Strange\Binaries\Win32\LifeIsStrange.exe", 0x1d1f5d0)
0009:fixme:module:load_dll Loader redirect from L"d3d9.dll" to L"d3d9-nine.dll"
ATTENTION: default value of option thread_submit overridden by environment.
0009:fixme:d3d9nine:d3dadapter9_new
Native Direct3D 9 is active.
For more information visit https://wiki.ixit.cz/d3d9
mesa: for the -simplifycfg-sink-common option: may only occur zero or one times!
0009:fixme:dxgi:DXGID3D10CreateDevice Ignoring flags 0x1.
0009:fixme:nvapi:unimplemented_stub function 0xbe7692ec is unimplemented!
0009:fixme:nvapi:unimplemented_stub function 0x6c2d048c is unimplemented!
ATTENTION: default value of option thread_submit overridden by environment.
0009:fixme:d3d9nine:d3dadapter9_new
Native Direct3D 9 is active.
For more information visit https://wiki.ixit.cz/d3d9
0009:fixme:msvcrt:type_info_name_internal_method type_info_node parameter ignored
0009:fixme:system:SystemParametersInfoW Unimplemented action: 59 (SPI_SETSTICKYKEYS)
0009:fixme:system:SystemParametersInfoW Unimplemented action: 53 (SPI_SETTOGGLEKEYS)
0009:fixme:system:SystemParametersInfoW Unimplemented action: 51 (SPI_SETFILTERKEYS)
0009:fixme:wbemprox:client_security_SetBlanket 0xef25a938, 0x18d2c0, 10, 0, (null), 3, 3, (nil), 0x00000000
0009:fixme:wbemprox:client_security_Release 0xef25a938
0009:fixme:wbemprox:enum_class_object_Next timeout not supported
0009:fixme:d3d9nine:DRI3PresentGroup_GetMultiheadCount (0x168eb0), stub!
0009:fixme:d3d9nine:DRI3PresentGroup_GetMultiheadCount (0x168eb0), stub!
0009:fixme:d3d9nine:D3DPERF_SetOptions (0x1) : stub
0009:fixme:imm:ImmReleaseContext (0x7004a, 0x1adb60): stub

btw why are you playing LiS over wine if the game have a great linux port?

There wasn't a Linux Port when I did get the game.

But now it has ( actually they ported it in 2016) and you can grab that game only on Steam.
But you still use it on Wine? Doesn't make sense with only one exception...


Issues with auto-complete and company support in merlin when used with flycheck-ocaml

It's not clear if this is a merlin bug or a flycheck-ocaml bug (flycheck/flycheck-ocaml#3), so I'm filing an issue here in the hope the root cause can be found.
flycheck-ocaml breaks if company-mode or auto-complete-mode (or both) are enabled. Strange parts of the buffer will get highlighted as errors; in particular, messages like these are common:
Merlin was already processing ((tell start at ((assoc) (line . 35) (col . 31)))) while ((tell start at ((assoc) (line . 35) (col . 31)))) was attempted
With auto-completion-mode specifically, Emacs will lock up after showing a completion listing.
I'm seeing this on Emacs 24.5.1 with the latest flycheck, merlin, company, and auto-complete.

Merlin doesn't expects to synchronous commands to happen concurrently (in this case, completion and error-checking), we had a lot of difficulty to find a convenient api for emacs, the situation is still not satisfying.
I think error-checking could be made to use the asynchronous interface. Who is maintaining the flycheck integration of merlin? The fix shouldn't take long, but I am not a flycheck user so I won't be able to test.

I have no familiarity at all with merlin's guts, but I'm willing to offer any help I can in tracking down the issue and testing changes.

@lunaryorn updated flycheck-ocaml code. Could you test on your side? I am not 100% confident about my code ;).

@drvink I have patched versions of merlin.el and flycheck-ocaml.el, could you try with those files in your local installation and tell if that fixes the bug?

See here https://gist.github.com/def-lkb/d3cc4907b81f15cc26d7

OK, let me give it a try.

Backtrace with the new files
JFYI, I had to change the version requirement for flycheck-ocaml.el to use merlin 1.4 to prevent package-install-from-buffer from fetching the ELPA one (since the version number of merlin.el in your gist is 1.4)

Wow! You are using completion-at-point, not the merlin-company-backend.

Look at https://github.com/the-lambda-church/merlin/wiki/emacs-from-scratch#company-mode for the setup.

Oh dear. Let me try that and get back to you.

NP, it might fail as well, but if that's the case it will be easier to debug this way :)

Backtrace with the correct config :)
Sorry about that!

Ok, something went wrong with the version of merlin.el I shared. Some parts of the code look 6 months old?!, its like I mixed different version of the same file.
Sorry for the inconvenience, I'll try to fix that.

This version of the gist looks saner https://gist.github.com/def-lkb/d3cc4907b81f15cc26d7

Backtrace using those files

@drvink: do you have some kind of instant messager (or IRC) so that we can have more interactive discussion? That would help solving this quickly

IRC would work; where should I connect?

On freenode, I am def`, you can also find me on #ocaml.

This issue should be fixed. Feel free to reopen/comment if you still have problems.

Seems like there are still some issues--Emacs will occasionally lock up. auto-complete-mode is fine, but company-mode isn't. Furthermore, there are frequent pauses if I stop typing momentarily and then start again. flycheck also gets desynchronized from the buffer--I'll get nonsensical error messages (like it claiming something is a string when it's an int).

I spoke too soon--I get a very occasional hard lockup even with company-mode off and only auto-complete-mode and flycheck-mode on. @def-lkb Let me know if there's any way I can provide more information.

Where you able to produce a stack trace of those lockups?

@def-lkb I'll try to get something from within Emacs when the lockup happens, but if I can't, I can get Emacs to dump core, and you might be able to do something with that, since Emacs includes gdb scripts which will give a complete stack trace (including the calls into Lisp from C and vice versa).


A Flutter example for sfu-ws.

Description
Flutter example for sfu-ws.
Create a Publisher/Subscriber mobile app using flutter.
Reference issue

Pull Request Test Coverage Report for Build 2685

0 of 0   changed or added relevant lines in 0 files are covered.
3 unchanged lines in 1 file lost coverage.
Overall coverage decreased (-0.08%) to 79.836%

Files with Coverage Reduction
New Missed Lines
%

sctptransport.go
3
76.97%

Totals

Change from base Build 2684:
-0.08%

Covered Lines:
3124

Relevant Lines:
3913

💛  - Coveralls

Fantastic work @cloudwebrtc
So, I think we are going to create a new repo called webrtc-example-applications you can read about it here
I want to put this, janus, GStreamer and websocket examples in there. As soon as we all come to a decision on the issue we can move forward.
I am so excited to try this! really nice work.

Close this PR and wait for new repo. ;)


b2ManifoldPoint point[].id.key changed to 65536

line 150 in the file 'b2CollideEdgeAndCircle', right after the initial 'id.key = 0'
149       manifold->points[0].id.key = 0;
150	      manifold->points[0].id.cf = cf;
The initialized key value changed to 65536 after line 150. I am not sure why doing that. Could that be a potential bug? Since the whole project keep id.key with 0.

I haven't notice that b2ContactID is union. key value is changed by assigning contact feature.


Add props table for each component

Description of Issue
Add props table for each component. We have a props table for each component on our functional testing site. However, props table does not currently work on terra-ui.com because we do not distribute our src directory(react-docgen only works with our src directory for generating the table).

Relates to: https://github.com/cerner/terra-core/issues/462

resolved in #38

resolved in #38


Add is_send_allowed to assistant.threads.setStatus arguments

Summary
This pull request adds a new argument to https://api.slack.com/methods/assistant.threads.setStatus
Category

[x] slack_sdk.web.WebClient (sync/async) (Web API client)
[ ] slack_sdk.webhook.WebhookClient (sync/async) (Incoming Webhook, response_url sender)
[ ] slack_sdk.socket_mode (Socket Mode client)
[ ] slack_sdk.signature (Request Signature Verifier)
[ ] slack_sdk.oauth (OAuth Flow Utilities)
[ ] slack_sdk.models (UI component builders)
[ ] slack_sdk.scim (SCIM API client)
[ ] slack_sdk.audit_logs (Audit Logs API client)
[ ] slack_sdk.rtm_v2 (RTM client)
[ ] /docs (Documents)
[ ] /tutorial (PythOnBoardingBot tutorial)
[ ] tests/integration_tests (Automated tests for this library)

Requirements

[x] I've read and understood the Contributing Guidelines and have done my best effort to follow them.
[x] I've read and agree to the Code of Conduct.
[x] I've run python3 -m venv .venv && source .venv/bin/activate && ./scripts/run_validation.sh after making the changes.

Converted this PR to draft state; We will hold off merging it untill the server is ready


[Backdrop] Render children inside div

[x] I have followed (at least) the PR section of the contributing guide.

fixes #17084

@dominictwlee Thanks


add test: select count(a, b) with null values

What problem does this PR solve?
close https://github.com/pingcap/tispark/issues/1754
issue fixed in https://github.com/pingcap/tispark/pull/1840
What is changed and how it works?
add some tests
Check List
Tests

Unit test
Integration test
Manual test (add detailed scripts or steps below)
No code

Code changes

Has exported function/method change
Has exported variable/fields change
Has interface methods change
Has persistent data change

Side effects

Possible performance regression
Increased code complexity
Breaking backward compatibility

Related changes

Need to cherry-pick to the release branch
Need to update the documentation
Need to update the tidb-ansible repository
Need to be included in the release note

/run-all-tests tikv=v4.0.5 tidb=v4.0.5 pd=v4.0.5

/run-all-tests tikv=v4.0.5 tidb=v4.0.5 pd=v4.0.5


Add Karaf feature + align dependencies + uncomment fixed examtest

Hi,
I wanted to start a small project with Karaf and found your presentation and project when looking for a recent paxexam configuration.
I had to fix the part I was interested in, I thought that maybe you are interested in integrating it.

Many thanks .. that is great. I did not yet have the time to fix that part.


[MODULES-446] Fixing tests - ensure proper test cleanups

Previous version of the test was causing 'maven clean' failures. Detailed message:
[INFO] --- clean:3.3.2:clean (default-clean) @ jboss-modules ---
[INFO] Deleting /home/opalka/git/redhat/PUBLIC/jboss-modules/target
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  1.334 s
[INFO] Finished at: 2024-02-21T18:33:45+01:00
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal org.apache.maven.plugins:maven-clean-plugin:3.3.2:clean (default-clean) on project jboss-modules: Failed to clean project: Failed to delete /home/opalka/git/redhat/PUBLIC/jboss-modules/target/junit8091973050755464239/.overlays -> [Help 1]

@dmlloyd please review


Update quads.scad

Seed RNG for repeatability
Add rotation around Z

Sorry, bad comment, rejected!


Singularity module now loads hidden modules.

The generated singularity module file now loads dependency modules, although no explicit autoload was set.
load(".cryptsetup/2.3.5-nmbjwez")
load("go/1.17.2")
load(".libgpg-error/1.46-5vso6hb")
load(".libseccomp/2.3.3-6m45xkp")
load(".shadow/4.8.1-4nbhlhp")
load(".squashfs/4.5.1-4dn2dwq")
load(".util-linux-uuid/2.38.1-mvepyq3")

Need to check whether this is true also for other packages.
In addition, the scripts/create_custom_singularity_modules.sh, when creating the no-mpi module, removes also parts of the module file that check for directory permissions and enforce the use of versions.

I suspect there is an error in the following sed command:
-e '/MPI + Singularity configuration/,/singularity_ld_path *= *singularity_ld_path/d' \

It references singularity_ld_path but I do not see that string anywhere in the original module file.

Also, we should mount /scratch and /software also in the no-mpi version.


Weird behavior importing perf data

I just used speedscope to open a perf recording.  The results are not very helpful:

The file starts with a few blocks like these that are apparently of the perf command itself:
perf 18021   663.145048:          1 cycles:
        ffffffffa5e76a8a [unknown] ([unknown])
        ffffffffa5e0d66d [unknown] ([unknown])
        ffffffffa5e0d6b0 [unknown] ([unknown])
        ffffffffa5e078dc [unknown] ([unknown])
        ffffffffa600a926 [unknown] ([unknown])
        ffffffffa6015302 [unknown] ([unknown])
        ffffffffa60e77f2 [unknown] ([unknown])
        ffffffffa615beaf [unknown] ([unknown])
        ffffffffa60e5601 [unknown] ([unknown])
        ffffffffa60e6856 [unknown] ([unknown])
        ffffffffa60e6c09 [unknown] ([unknown])
        ffffffffa5e04207 [unknown] ([unknown])
        ffffffffa6a0008c [unknown] ([unknown])
            7f91ec896ae7 [unknown] ([unknown])

Then the file switches to snapshots of my program (driver):
driver 18021   663.145133:     209655 cycles:
        ffffffffa606e8c5 [unknown] ([unknown])
        ffffffffa60e4fc3 [unknown] ([unknown])
        ffffffffa615bf09 [unknown] ([unknown])
        ffffffffa60e5601 [unknown] ([unknown])
        ffffffffa60e6856 [unknown] ([unknown])
        ffffffffa60e6c09 [unknown] ([unknown])
        ffffffffa5e04207 [unknown] ([unknown])
        ffffffffa6a0008c [unknown] ([unknown])
            7f91ec896ae7 [unknown] ([unknown])

driver 18021   663.145375:    1332411 cycles:
            7f9253f90d4a strlen+0x2a (/lib/x86_64-linux-gnu/ld-2.27.so)

If I manually remove the perf blocks, the file imports as expected:

At the top center of the the page there is a menu that allows you to select which thread to show. In the first screenshot it shows two threads. That single perf sample and likely your own process. In the second screenshot it only lists your process. As for why that perf block exists in the first place I'm not sure. I have never seen it locally. I would guess it is a bug either in your kernel or perf.

Thanks, didn't notice that I could select the thread.
I agree this sort of seems like a perf bug, but I haven't really used it before.


chomp and the number zero

Chomp (MODERN.EXE) will happily accept the number zero for players, rows, and columns.
With zero players, the game seems to run with an unlimited number of players (got up to player 22 before the game ended.)
With zero rows or zero columns, the board is blank and the game unplayable.

Many of these games have only minimal input-checking in their original
form, and thus far I've paid little attention to making the colorized
versions bulletproof.  However, I've updated Chomp with checks for 0
players, rows, and columns (and updated the version up on the Internet
Archive as well).
Thanks for the report!
On Mon, Aug 17, 2020 at 11:29 AM jpcompton<EMAIL_ADDRESS>wrote:

Chomp (MODERN.EXE) will happily accept the number zero for players, rows,
and columns.
With zero players, the game seems to run with an unlimited number of
players (got up to player 22 before the game ended.)
With zero rows or zero columns, the board is blank and the game unplayable.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/chaosotter/basic-games/issues/1, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEDKGUWXLLMSH6BSUDAFEJTSBFZIZANCNFSM4QCAGEPQ
.


functions createTag fails when the full file path where the code is has a space character

If the full file path where the code is located has a space character, i.e. /home/javi/functions/analyze Image/ the functions createTag command
appwrite functions createTag \
    --functionId=60dcbc0b83abc \
    --command='python main.py' \
    --code='./deploy'

fails with this error.
docker: invalid reference format.
See 'docker run --help'.

Otherwise, /home/javi/functions/analyzeImage/, it works fine.

Appwrite CLI 0.10.0
Docker version 20.10.6-ce, build 8728dd246c3a
Linux linux-openSUSE 5.12.13-1-default

@JaviBonilla  Thank you for raising this :smiley:
https://github.com/appwrite/sdk-generator/pull/228 Should fix it :+1:

@christyjacob4 Great! Thanks.
There is another kind of issue, when the size of the code.tar.gz is greater than the given in _APP_STORAGE_LIMITenvironment variable, the deployment fails and this warning (which doesn`t clearly point to the issue) is given.
Warning: Invalid argument supplied for foreach() in /usr/local/code/src/Parser.php on line 47

Could you try to replicate that?

@JaviBonilla  Yes , I was able to replicate it and this PR https://github.com/appwrite/sdk-generator/pull/229  has a fix for that as well :smile:
Thanks for helping with the debugging

Thanks @christyjacob4

This fix is now available in the latest CLI release.


Upgrading the terraform-google-pubsub module to release 4.0.0

Provides access to message_retention_duration on topics and exactly once delivery on
subscriptions.
The release for terraform-google-pubsub is published here: https://github.com/terraform-google-modules/terraform-google-pubsub/releases/tag/v4.0.0

/gcbrun

/gcbrun

/gcbrun

/gcbrun

/gcbrun


Added prince Logging

Added log options in build.xml to avoud Log Entries in
index-id-to-page.js

I have commented out the log options because moving the output in a log hides the error messages in the oXygen console. I expect you have to fork the project and make your changes on a branch.


cargo doc: Add --publish-dir option

Resolves #10089
The PR adds a new option to cargo doc/cargo rustdoc: --publish-dir
When using this new option, the generated documentation will be copied to the specified publish directory.
This can be used to publish documentation straight into, for example, a directory that is being served up by a local web server already.

r? @ehuss
(rust-highfive has picked a reviewer for you, use r? to override)

Thanks for the PR!  The team talked about this a bit, and we had a few comments:

What is the use case for this?  I would assume that if you are packaging or otherwise collecting the build results somewhere, that you have an outer build system or script that can do the same task of copying the files.
We were thinking this seems essentially the same as --out-dir. We were thinking that it would probably be best to avoid adding too many different options, so perhaps this should just be the same as --out-dir?

Related to that, unfortunately --out-dir is not really on a path to ever being stabilized. We're happy to include cargo doc with --out-dir, this is just a heads-up that it's not clear how it will move forward.

I can chime into this from my perspective.
Yes, this is exactly like --out-dir, just implemented specifically for doc. I agree that consolidating the two makes sense. I don't know that the current implementation of --out-dir allows setting it just for [doc], but if it did, it'd at least solve my use-case.
As for what that use-case is, for me at least the hope is to encode information about where other tools look for documentation directly into Cargo's configuration so that users can just run cargo doc and the documentation is automatically included in whatever those other tools may be (which suggests >1 out-dirs should be specifiable, but that's a separate point). Concretely, in my case, I have an external build system that links and exports any documentation present in a particular subdirectory of a folder named build, and I would like users using Cargo to be able to just use cargo doc, and still have their docs published to the external build system as well. out-dir would do that wonderfully.

I've tagged this as waiting-on-author now as the consensus seems to be that if we are to implement a feature such as this it would be done with --out-dir

Closing per the conversation above about not wishing to add a new flag for this.


Support netcoreapp3.0

Hello, I think it's time to add netcoreapp3.0 to TargetFrameworks :)

I mean publish netcoreapp build at nuget.


Task.Snmp.Update.Lookup get's duplicate key

Issue: https://github.com/RackHD/RackHD/issues/290
This is the 1.2 level fix (hack) for this that basically avoids the problem case outlined in the odr, nothing more.
The change sets up an array of found-macs for this run, and simply skips ones that have already been seen.
@RackHD/corecommitters @johren @derrickostertag

👍

Fixed tabs and squashed....


Docker create in not saving command which has multiple spaces

Output of docker version:
Docker Version:
Client:
Version:      1.12.0-dev
API version:  1.24
Go version:   go1.6.2
Git commit:   3d80884
Built:        Mon Jun  6 19:12:50 2016
OS/Arch:      windows/amd64
Server:
Version:      1.12.0-dev
API version:  1.24
Go version:   go1.6.2
Git commit:   3d80884
Built:        Mon Jun  6 19:12:50 2016
OS/Arch:      windows/amd64
Steps to reproduce the issue:
1.Docker create –name=test nanoserver ping <IP_ADDRESS> –t
2. Docker ps
3. The -t flag is missing.
CONTAINER ID        IMAGE               COMMAND                  CREATED             STATUS  PORTS               NAMES
27b378e74c25        nanoserver  "ping <IP_ADDRESS>"        13 seconds ago      Created                      test

This regression looks related to the Cobra CLI parsing. This worked before that PR was merged (can't find PR# from phone)

cc @dnephin

also cc @vdemeester

@justincormack should be fixed here #23338 :angel:


Error:  Operation is not implemented, or supported, or enabled.

Describe the bug
Getting below exception
-error: Kreait\Firebase\Exception\Messaging\MessagingError {#2996
      #message: "Operation is not implemented, or supported, or enabled."
      #code: 501
      #file: "/var/www/doctime/core/vendor/kreait/firebase-php/src/Firebase/Exception/Messaging/MessagingError.php"
      #line: 22
      #errors: array:1 [
        "error" => array:3 [
          "code" => 501
          "message" => "Operation is not implemented, or supported, or enabled."
          "status" => "UNIMPLEMENTED"
        ]
      ]

### Installed packages

```json
"kreait/laravel-firebase": "^4.1"

PHP version and extensions
8.1

Usages:
$message = CloudMessage::new()
                ->withAndroidConfig($androidConfig)
                ->withApnsConfig($apnsConfig)
                ->withWebPushConfig($webPushConfig);

$cloudMessaging = app('firebase.messaging');
$cloudMessaging->sendMulticast($message, $tokens);

Additional information
No response

update your laravel-firebase version to 5.9.1

What @beratallahverdi said - version 4.x isn't supported anymore. 5.x uses a newer version of the SDK which uses the currently supported FCM API endpoints.

update your laravel-firebase version to 5.9.1

Just for more information. Can you tell specific version which fix this issue of (laravel-firebase)[https://github.com/kreait/laravel-firebase] and (firebase-php)[https://github.com/kreait/firebase-php] or the root cause of this error 😄 . Thanks


BUG: NPSP Data Importer fails to process if Data Import custom settings are saved for the first time.

I get an upsert error.  need to handle this case and do an insert instead?  I guess maybe custom settings don't handle upserts the same as other objects?
This is the error I get with the managed package.  I can't seem to reproduce in my DE Org.
Upsert failed. First exception on row 0 with id a0ei0000005M6rFAAS; first error: INVALID_CROSS_REFERENCE_KEY, invalid cross reference id: []

I think I may have come across this in my sandbox org...? After searching, finally found this issue from https://github.com/SalesforceFoundation/Cumulus/blob/dev/src/classes/BDI_DataImport_API.cls#L79. I'm don't think the hack is working in my case. I set the initial default Data Import Settings directly through the setup menu, and the importer worked after that. Deleting the custom settings caused it to start failing again.

#1457 just went in a few days ago and hasn't been push upgraded yet.  version 3.36


Homematic signalgen

Description:
This pull request adds the homematic signal generator as notify component.
Related issue (if applicable): n/a
Pull request in home-assistant.io with documentation (if applicable): home-assistant/home-assistant.io#6379
Example entry for configuration.yaml (if applicable):
notify:
  - name: my_hm
    platform: homematic_signalgen
    address: NEQXXXXXXX
    value: "1,1,108000,8"

Checklist:

[x] The code change is tested and works locally.
[x] Local tests pass with tox. Your PR cannot be merged unless tests pass

If user exposed functionality or configuration variables are added/changed:

[x] Documentation added/updated in home-assistant.io

If the code communicates with devices, web services, or third-party tools:

[x] New dependencies have been added to the REQUIREMENTS variable (example).
[x] New dependencies are only imported inside functions that use them (example).
[x] New or updated dependencies have been added to requirements_all.txt by running script/gen_requirements_all.py.
[x] New files were added to .coveragerc.

If the code does not interact with devices:

[x] Tests have been added to verify that the new code works.

@pvizeli: I need to configure which MP3 track on the device to play in case of a notification. How does discovery work in this case? How do you mean that?

https://github.com/home-assistant/home-assistant/blob/dev/homeassistant/components/homematic/__init__.py#L444-L463

@pvizeli: Just to explain my intentions: This pull request aims to provide support for Homematic's signal generator "HM-OU-CFM-TW" as notification platform. The signal generator is already supported by the pyhomematic library: https://github.com/danielperna84/pyhomematic/blob/master/pyhomematic/devicetypes/actors.py#L461 and is discovered automatically. But like for the Kodi notification component, I would like to play a mp3 file stored on the Homematic signal generator (or let the LEDs blink) for example in case the washing machine is ready or the front door opens when the alarm mode is enabled. For Kodi there are also two "components": One for notifications and one to control the device itself. The implementation to control the signal generator is already available. This one provides support for notifications by invoking the already existing homematic service. Hence, it is not a complete new platform.
Beyond that I decided to name the notification homematic_signalgen because it only addresses the signal generator. This eases its ussage as you can use it without having to know the channel and the datapoint param. Hence, you can have in the future more than one "notification component" for homematic. On the other hand, it would be also possible to implement a "generic" homematic notification, but in that case you would have to know how to address the signal generator and you could also send notifications to your covers or switches, which would not make much sense.
Let me know what you think.

Isn't it possible to play MP3s by using the homematic.set_device_value service? It's intended for those cases where a dedicated platform for just a single device would be needed. There are other HomeMatic devices which in theory would deserve their own platform as well. But as long as using the mentioned service as a workaround does the job, I don't think dedicated platforms are necessary.
Apart from that: I have updated pyhomematic to 0.1.50, so that would need to be changed in the requirements. And I wonder how tests pass. Your tests don't initialize the virtual CCU. So setting up the component in the tests should actually fail. 🤔

Kodi is bad. You can use group notify to set fix value and make notify.homeassistant variable over data.

@pvizeli OK. I have renamed the notify component to notify.homematic and made it generic. You can now pass also channel and param:
notify:
  - name: my_hm
    platform: homematic
    address: NEQXXXXXXX
    channel: 2
    param: "SUBMIT"
    value: "1,1,108000,8"

Let me know what you think.
The group notification only allows to call other notifications from the notify domain: https://github.com/home-assistant/home-assistant/blob/dev/homeassistant/components/notify/group.py#L68. Maybe it makes sense to allow calling "normal" services, too?

The code still has references to the Signal generator, which isn't correct anymore now that you can specify the param and possibly can control all kinds of devices. So that's a cosmetic improvement that should be done

@danielperna84 OK. I have removed the terms signal generator from the sources. :wink:

@siom79 you can use data to pass additional parameters and targets for device target.
With groups, you can create a simple notify.homematic_mydoor and pass all data to this group and after that you can call this with a simple service call to this service. It allow also to have a service per mp3 like you want have but it is a very flexible solution and every body can build his own stuff around this.

@pvizeli: The final goal is to use an alert together with a notification like in the following example:
alert:
  washing_machine:
    name: Washing machine ready
    done_message: Washing machine done
    entity_id: binary_sensor.washing_machine_ready
    state: 'on'
    repeat: 10
    can_acknowledge: True
    notifiers:
      - my_hm_signalgen

notify:
  - platform: homematic
    name: my_hm_signalgen
    address: NEQXXXXXXX
    channel: 2
    param: "SUBMIT"
    value: "1,1,108000,8"

Please note that the example above already uses this pull request and does not work without it.
But you say that I can call any service using the group notification. But how?
The following example causes an "Unable to find service notify/homematic" error:
alert:
  washing_machine:
    name: Washing machine ready
    done_message: Washing machine done
    entity_id: binary_sensor.washing_machine_ready
    state: 'on'
    repeat: 10
    can_acknowledge: True
    notifiers:
      - my_hm

notify:
  - platform: group
    name: my_hm
    services:
      - service: homematic
        data:
          data:
            address: NEQ12341234
            channel: 2
            param: SUBMIT
            value: "1,1,108000,3"

It's because you can invoke from the alert component and the group notification component only services in the notify domain.


Added tackle-windup-integration enhancement

Application analysis is one of the core use cases for the Tackle toolkit. This is expected to be achieved by leveraging the Windup project, developed by Red Hat as the upstream for the Migration Toolkit for Applications product. The expected outcome of this enhancement is to have a seamless integration of Windup within the Application Inventory user experience in the same fashion as we currently have with the Pathfinder tool.

AC006 replace - If no packages are specified, all packages must be included in the analysis. With - If no packages are specified, then all packages identified as 'application packages' will be analyzed.

AC003 - As an Architect/Migrator I want to be able to use Git repositories or as input for an analysis.
As an Architect/Migrator I want to be able to use Git, SVN and Maven repositories as input for an analysis.

Generic query - do we need to make a statement about the proxy considerations for accessing data within the repositories?


Use same schedule param file for both arms

Fixes #54. The ft related params were missing for the right arm yaml file. This PR adds the necessary parameters and also adds a script for starting the force_torque controller.

Tested it on herb0. I am able to load all the controllers and also able to see f/t values for right arm being published.


Merging language_data with langcodes

I am starting the discussion here in order to allow people to contribute to the discussion. Since I entered the project while it was already existing for a long time, I am not aware of any original reasoning why language_data is distributed as its own package. I am wondering if that's actually useful and I am therefore considering to merge the two projects into one and the two packages into one, too.
From my perspective the two projects are pretty interconnected. You cannot run the tests in language_data without having langcodes installed and vice versa. Also, you'd have to run both test suites anyways in order to achieve a high coverage, or to phrase it differently: It might be the case that changes in language_data break test in langcodes, but we can't really tell because the changes in language_data don't trigger another test run in langcodes.
For the users I am not sure if there is much value in using only one the packages. I never used one of them isolated from the other, so here I'd be interested in input. Are you using only one of the packages? Whats your use case? Would it hurt to just install one langcodes package which includes the data? Let me know!
 I will let some months pass to allow everyone to leave their comment.

2 ways to find out who uses your library, other than waiting a few years for your users to organically visit this post:
Search github for requirements.txt or pyproject.toml or Pipfile for the library name e.g.
https://github.com/search?q=path%3Arequirements.txt language-data&type=code
Libraries IO - looks like they are only aware of 1 published user (and that might be a mistake)
https://libraries.io/pypi/language-data
https://libraries.io/pypi/langcodes
Anyhow, changing namespaces is a breaking change, if you merge the libraries and change the namespace it won't break anything but no one gets any benefit until they start referencing the new names. I'd chalk this up to a regrettable day 1 mistake and not change it.
If you merge them anyhow, I'd suggest checking how they behave when you install a merged library with the stand alone, i.e. will the one overwrite the folder of the other in the venv folder.

Search github for requirements.txt or pyproject.toml or Pipfile for the library name e.g.

Thank you for the idea! But even your linked search shows >250 projects, I don't think I have the capacity to reach out to all of them :)

Anyhow, changing namespaces is a breaking change

I wasn't clear enough here, I don't want to break the namespace but integrate language_data into langcodes.

If you merge them anyhow, I'd suggest checking how they behave when you install a merged library with the stand alone, i.e. will the one overwrite the folder of the other in the venv folder.

If I do this, some proper testing of the installations will be on my todo list, thank you for the hint. Thats a good think to watch out for.


How to build releases and how to edit the book?

What toolchain do you use to edit these files?

I'd like to contribute but I obviously have no idea what tool this was written in

It's written in mark down using Jupyter Notebook. Install Git on your machine, then clone the repo. You'll then need to make a branch, make your changes and then submit a pull request. Someone will review your changes and merge your pull request. Hope this helps.

I'd like to bump this issue - there should be a contributing.md that shows you how to actually contribute to this repo. With a clear set of instructions for how to set it up.
I'm happy to create that, if others agree?

Beginners contribution guide

thats great, it needs to be linked in the read me though. unless Im missing it?

Beginners contribution guide.md
I think this is what you were referring to? Apology if I'm mistaken.

ah its hidden away.


rdapConformance - Default Value

Within Section 1.3 of the RDAP Response Profile it states:
"For clarity, conformance to the current document MUST be noted with a value of "icann_rdap_response_profile_0""
The default for RedDog responses seems to be "rdap_level_0".   Shouldn't this be changed to "icann_rdap_response_profile_0"?
OR is it a case that I should add "add_custom_conformance" within configuration.properties (which I then believe would show both "rdap_level_0" and "icann_rdap_response_profile_0")

From RFC 7483 Section 4.1

The data structure named "rdapConformance" is an array of strings,
each providing a hint as to the specifications used in the
construction of the response.

rdap_level_0 means that the response is conformance with the RFC7483.
If you want to add more conformances, you add it in the "add_custom_conformance" property configuration.
So, you second thought is correct.

Thanks for the super quick response.  You guys are awesome!
(and apologies in advance I may have a few more questions for you)


Tidy up deps

This was causing some problems with incompatible versions of dependencies.
Signed-off-by: Jose Fuentes<EMAIL_ADDRESS>

/lgtm


Improved typing when using Pydantic models

What behavior of the library made you think about the improvement?
generator = outlines.generate.json(model, MyModel)
answer = generator(prompt)

Answer is now type hinted as str | int | ... but we know it should be MyModel.
When I explicitly define answer: MyModel = generator(prompt) it also says the types do not match.
How would you like it to behave?
SequenceGenerator could be generic over the provided pydantic model when using outlines.generate.json, so the return value of __call__ would be the right data structure.
This would probably be harder to do for other outlines.generate methods, but for pydantic it would be very nice.

I created a hacky example that roughly does what I mean:
T = typing.TypeVar("T")

class GenericSequenceGenerator(SequenceGenerator, typing.Generic[T]):
    def __call__(self, *args, **kwargs) -> T:
        return typing.cast(
            T,
            super().__call__(*args, **kwargs)
        )

def outlines_generate_json(model, klass: typing.Type[T]) -> GenericSequenceGenerator[T]:
    return typing.cast(
        T,
        outlines.generate.json(model, klass)
    )


release tool for lemming and operator

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.2%

Covered Lines:
3381

Relevant Lines:
5253

💛  - Coveralls

Another way to deal with having two projects in one is to use Git submodules: https://git-scm.com/book/en/v2/Git-Tools-Submodules
However I'm not convinced ours is the right use case, so this change overall looks good to me.

that would require two repos, which i wanted to avoid. the operator won't be changing often, so i think the slight burden of the two go.mod is ok.


dar: 2.5.16 -> 2.5.17

Motivation for this change
UYpdate package to ver 2.5.17
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

Already in the pipe here: #47817


Update test_frequency_sane_fmap integration test

There is a race condition where if the controller's first sample of
the region is prior to the hint change due to the entry into a network
sub-region then the report includes the map for the region otherwise it
does not.
Update the test to supress the KeyError when learning doesn't occur
due to the race.
Fixes #1155

Change-Id: Iee50d47c3b1b740f6cf4c7f328104e7ef80770b3
Signed-off-by: Brandon Baker<EMAIL_ADDRESS>Summary of change.

Details of change; please include all interfaces added, removed, or modified.
Fixes #XXX from github issues.

This seems like a bug, not a test issue.  Is it not possible to fix the bug?


Configure default automountServiceAccountToken in pods and service accounts

Should be set to false.
Additional resources

https://hub.datree.io/built-in-rules/prevent-service-account-token-auto-mount

I see this is already implemented, 🚀

Pods
ServiceAccounts


[MOB-6878] iOS IterableEmbeddedView (OOTB)

🔹 Jira Ticket(s)

MOB-6878

✏ Description
This PR implements the IterableEmbeddedView, which is an out of the box view for embedded messaging. The SDK implementation can be used with the corresponding Fiterable OOTB PR to see how the IterableEmbeddedView is used in a client app.
Merging into anelson/6734-update-messages-with-placements which is currently awaiting PR into embedded-messaging branch.

NB: This PR actually implements IterableBannerView.xib, but is renamed in a subsequent PR to IterableEmbeddedView.xib.

I fixed these errors in the subsequent url/action handlers PR.


Fix collins-notify for ruby 2.2.0

@maddalab @Primer42 @roymarantz @defect This fixes the #respond_to? invocations to actually find protected methods, which is preventing collins-notify from working on newer rubies. We ask respond_to to search all methods, not just public ones.
[0] http://ruby-doc.org/core-1.9.3/Object.html#method-i-respond_to-3F
[1] http://ruby-doc.org/core-2.2.2/Object.html#method-i-respond_to-3F
I have tested this and confirmed functionality in both 2.2.0 and 1.9.2.

lgtm

:+1:


Controller: change the ""Get leases 200"" and "Update leases 200" messages from "info" to "debug"

Summary
Currently workflow controller writes the "Get leases 200"  and  "Update leases 200" messages every 5 seconds to the job log.
Unless there is an error, users are not concerned with the success "200" messages, but they pollute the log and make it harder to find really important messages. This can be solved by changing the Lease-related non-error messages from "info" to "debug".
Use Cases
Analyzing job log of the workflow controller

Message from the maintainers:
Love this enhancement proposal? Give it a 👍. We prioritise the proposals with the most 👍.

Already changed in https://github.com/argoproj/argo-workflows/pull/10938

Hi @terrytangyuan , the messages are still shown as "info" in the v.3.4.8:
time="2023-06-26T13:40:42.577Z" level=info msg="Get leases 200"
time="2023-06-26T13:40:42.586Z" level=info msg="Update leases 200"
time="2023-06-26T13:40:47.593Z" level=info msg="Get leases 200"
time="2023-06-26T13:40:47.602Z" level=info msg="Update leases 200"
time="2023-06-26T13:40:52.610Z" level=info msg="Get leases 200"
time="2023-06-26T13:40:52.619Z" level=info msg="Update leases 200"
time="2023-06-26T13:40:54.162Z" level=info msg="List workflows 200"
time="2023-06-26T13:40:57.627Z" level=info msg="Get leases 200"
time="2023-06-26T13:40:57.637Z" level=info msg="Update leases 200"
time="2023-06-26T13:41:02.644Z" level=info msg="Get leases 200"
time="2023-06-26T13:41:02.688Z" level=info msg="Update leases 200"

Could it be that there other places to change the level apart from what you have done in April?

It's not part of the release yet

No schedule yet. Depends on whether we have many critical fixes.


Test fails on numpy  1.9.1

Hi I have some problem running the test.py:
import icoshift
import numpy as np

test = np.array([
[0, 0, 0, 2, 3, 2, 0, 0, 2, 1, 2, np.nan, 3, 4, 2, 3, 2, 0, 2, 3, 2],
[0, 0, 2, 3, 2, 0, 0, 2, 3, 1, 2, np.nan, 3, 4, 3, 2, 0, 2, 3, 2, 0],
[0, 2, 3, 2, 0, 0, 2, 3, 2, 1, 2, np.nan, 3, 4, 2, 0, 2, 3, 2, 0, 1],
])
xCS, ints, ind, target = icoshift.icoshift('average', test)
print '********** PASS 1: average **********'

test = np.array([
[0, 0, 0, 2, 3, 2, 0, 0, 2, 3, 2, np.nan, 0, 0, 2, 3, 2, 0, 2, 3, 2],
[0, 0, 2, 3, 2, 0, 0, 2, 3, 2, 0, np.nan, 0, 2, 3, 2, 0, 2, 3, 2, 0],
[0, 2, 3, 2, 0, 0, 2, 3, 2, 0, 0, np.nan, 2, 3, 2, 0, 2, 3, 2, 0, 1],
])

xCS, ints, ind, target = icoshift.icoshift('median', test)
print '********** PASS 2: median **********'

I get:
ValueError                                Traceback (most recent call last)
<ipython-input-3-729c4e491a6c> in <module>()
      7 [0, 2, 3, 2, 0, 0, 2, 3, 2, 1, 2, np.nan, 3, 4, 2, 0, 2, 3, 2, 0, 1],
      8 ])
----> 9 xCS, ints, ind, target = icoshift.icoshift('average', test)
     10 print '********** PASS 1: average **********'
     11

/gcdata/gcproj/Luca/noah/lib/python2.7/site-packages/icoshift/icoshift.pyc in icoshift(xt, xp, inter, n, scale, coshift_preprocessing, coshift_preprocessing_max_shift, fill_with_previous, average2_multiplier)
    386         else:
    387             intern_, flag_nan = remove_nan(
--> 388                 ref(inter), cat(0, xt, xp), select, flags=True)
    389             intern_ = vec(intern_.T).T
    390

/gcdata/gcproj/Luca/noah/lib/python2.7/site-packages/icoshift/icoshift.pyc in remove_nan(b, signal, select, flags)
    994
    995             c.resize(d.shape)
--> 996             c[count:count + max(a.shape) + 1] = d
    997
    998             count = count + max(a.shape)

ValueError: could not broadcast input array from shape (4) into shape (3)

Any idea on how to fix it?

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

The test fails  for me too but with a different error.
Any Idea why?
import icoshift
import numpy as np

test = np.array([
[0, 0, 0, 2, 3, 2, 0, 0, 2, 1, 2, np.nan, 3, 4, 2, 3, 2, 0, 2, 3, 2],
[0, 0, 2, 3, 2, 0, 0, 2, 3, 1, 2, np.nan, 3, 4, 3, 2, 0, 2, 3, 2, 0],
[0, 2, 3, 2, 0, 0, 2, 3, 2, 1, 2, np.nan, 3, 4, 2, 0, 2, 3, 2, 0, 1],
])
xCS, ints, ind, target = icoshift.icoshift('average', test)
print '********** PASS 1: average **********'

test = np.array([
[0, 0, 0, 2, 3, 2, 0, 0, 2, 3, 2, np.nan, 0, 0, 2, 3, 2, 0, 2, 3, 2],
[0, 0, 2, 3, 2, 0, 0, 2, 3, 2, 0, np.nan, 0, 2, 3, 2, 0, 2, 3, 2, 0],
[0, 2, 3, 2, 0, 0, 2, 3, 2, 0, 0, np.nan, 2, 3, 2, 0, 2, 3, 2, 0, 1],
])

xCS, ints, ind, target = icoshift.icoshift('median', test)
print '********** PASS 2: median **********'

gives
C:\Anaconda2\lib\site-packages\scipy\stats\stats.py:316: RuntimeWarning: invalid value encountered in true_divide
  return np.mean(x, axis) / factor
---------------------------------------------------------------------------
IndexError                                Traceback (most recent call last)
<ipython-input-1-729c4e491a6c> in <module>()
      7 [0, 2, 3, 2, 0, 0, 2, 3, 2, 1, 2, np.nan, 3, 4, 2, 0, 2, 3, 2, 0, 1],
      8 ])
----> 9 xCS, ints, ind, target = icoshift.icoshift('average', test)
     10 print '********** PASS 1: average **********'
     11

C:\Anaconda2\lib\site-packages\icoshift\icoshift.pyc in icoshift(xt, xp, inter, n, scale, coshift_preprocessing, coshift_preprocessing_max_shift, fill_with_previous, average2_multiplier)
    386         else:
    387             intern_, flag_nan = remove_nan(
--> 388                 ref(inter), cat(0, xt, xp), select, flags=True)
    389             intern_ = vec(intern_.T).T
    390

C:\Anaconda2\lib\site-packages\icoshift\icoshift.pyc in remove_nan(b, signal, select, flags)
    980
    981             if numpy.any(~in_[0]):
--> 982                 a = cat(1, numpy.array([0]), a)
    983
    984             else:

C:\Anaconda2\lib\site-packages\icoshift\icoshift.pyc in cat(dim, *args)
     23
     24 def cat(dim, *args):
---> 25     return numpy.concatenate([r for r in args if r.shape[0] > 0], axis=dim)
     26
     27

IndexError: axis 1 out of bounds [0, 1)

I see! removing the np.nan fixed the example
My own spectra(without 'nan') does not work though.

Is it safe to assume that this project is no longer maintained?

This is exactly why the versions of the required packages should be listed (install within virtualenv + include pip freeze in the repo) . Apparently NumPy/SciPy have had some breaking changes which is why this library no longer works with the up-to-date versions. For instance, nanmean has been deprecated and removed from numpy. Does anyone know what scipy/numpy version combination has worked with this library?

Hi @amergin, would you be interested in collaborating on a rewrite of this?

PRs that fix any issues are most welcome.
I'm no longer using the library myself (no longer working with NMR) so
another maintainer would also be very welcome.
There are plenty of improvements to make, not least adding tests to catch
these issues quicker.
On 18 Sep 2017 13:32, "KeironO"<EMAIL_ADDRESS>wrote:

Hi @amergin https://github.com/amergin, would you be interested in
collaborating on a rewrite of this?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/mfitzp/icoshift/issues/2#issuecomment-330193536, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAHtH04_WiK39VDx6Ty_1cRm-sU-9mQ4ks5sjlTOgaJpZM4DJH-W
.

@mfitzp Dim problem.
Disgwyl cais gwthio yn weddol fuan!

@KeironO Unfortunately I don't have the time to get into developing this.

 Github test


feat: improve typing for to_pandas / to_numpy / to_arrow

What type of PR is this? (check all applicable)

[ ] 💾 Refactor
[ ] ✨ Feature
[ ] 🐛 Bug Fix
[ ] 🔧 Optimization
[ ] 📝 Documentation
[ ] ✅ Test
[ ] 🐳 Other

Related issues

Related issue #
Closes #

Checklist

[ ] Code follows style guide (ruff)
[ ] Tests added
[ ] Documented the changes

If you have comments or can explain your changes, please do so below.

merging for now so I can use # ignore-banned-import in the other pr


"Show All Bookmarks" is confusing

I'm not sure if an issue is appropriate here, sorry if I'm creating clutter.
I expected the "Show All Bookmarks" command to show me bookmarks, i.e. popup some kind of window that would display the bookmarks. I did not realize at all that it was setting the visibility level for the Goto command. It didn't even occur to me, I just thought the command was broken.
I might be alone in getting confused, but I suspect others might be as well. Renaming the command to something like "Set Visibility - All Bookmarks" might help.
I'd be happy to make a PR if you wanted one.

I agree. I actually think it's just wrong. Probably it should be smtg like "show all bookmarks in goto cmd"


installer: build central syslogging

important for distributed installs.
define one backend server as logserver and additionally log everything there (also keep local logging)
move from syslog to syslog-ng?

duplicate to #1077


Remove usage of Dataset data_types

Remove the unused uniqueDataType constant:
https://github.com/x-atlas-consortia/data-ingest-board/blob/819bcc35b881ec9748acb31676636e9dac3b1ed3/src/components/DataTable/DatasetTable.jsx#L40-L50

@maxsibilla , If @libpitt is working in the data ingest board, I'll ask her to take care of this.  I think it is just dead code that needs to be removed.


Make sure the document processed in background.js has a title element.

This addresses issue #310: which describes a download failure for a specific website, because pageTitle is empty.
The proposed fix here makes sure that the HTML document processed in background.js has a proper <title> tag, so that pageTitle can be populated.
Note: Despite a missing <title> tag browsers manage to provide a useful window title. Hence the  best place to fix this issue seems to be  contentScript.js because it is run in the context of the original HTML page

I think this PR can be adapted in the obsidian feature too.


Add support for debian armel

If you haven't seen already, the ASP.NET folks were very excited about dnx running on LEGO MINDSOTRMS EV3.
I'm guessing that it is probably not worth the time it would take to implement, as there aren't really any other practical applications that I can think of but I am wondering just exactly what it would take to get AOT cross compiling working for Debian armel architecture.
I understand that "arm" here in corert means the same as debian armhf (armv7+vfp more or less). Debian armel is armv4t with no floating point.
@schellap, I saw your comment about arm64. Perhaps you could say something about this architecture as well while you are thinking about it?

This is the kind of scenario we are targeting with the C++ code generation backend. In that compilation mode we're not compiling into native object files (*.o on Unix or *.obj on Windows), but into a standard *.cpp file you can compile with the platform C++ compiler. So if LEGO MINDSTORMS has a C++ compiler (ideally CLang because that's what we test on Unix), you might be able to get a "Hello World!" working right now.
CoreRT is not in shape to run much more than Hello World at this point.
If you feel up for the challenge, you can follow the instructions here to generate the C++ file from a C# Hello World. Then you'll need to figure out how to crosscompile the runtime under /dotnet/corert/src/Native with the generated C++ file and link it together.

Thanks for the info. I didn't realize that this is how it works - I guess I assumed that it was like AOT in mono. This sounds much more achievable now. I will definitely give it a try.

@dlech corert can aot too without producing c++ files at all but instead of that producing c++ is also supported.


How to apply custom mask for cel-phone type?

I want to do something like this:
<TextInputMask
            type={'cel-phone'}
            placeholder="Enter your mobile number"
            placeholderTextColor="#cccbd0"
            style={styles.numberInput}
            options={{
              mask: '(999) 999-9999'
            }}
/>

But it doesn't work that way, so I did this instead:
<TextInputMask
            type={'custom'}
            placeholder="Enter your mobile number"
            placeholderTextColor="#cccbd0"
            style={styles.numberInput}
            options={{
              mask: '(999) 999-9999'
            }}
/>

Seems to be ok but it opens alphabetical keyboard. Is it possible to use type="cel-phone" with a custom mask?

Hi!
I fix it at 1.5.2.
You now can use keyboardType="numeric" for example.
Thanks for reporting =).

Thank you! Great job.


Destroy method

I'm dealing with a hidden element that I want to initialize stickUp on when I show it, then destroy when it hides (otherwise it doesn't seem to work), but I don't see anything in your documentation about a method to destroy/cancel/stop stickyness of an element. Can you suggest something or add it? Thanks.

Removing the dom element it was attached to, adding it back, and re-initializing doesn't work. Which makes me think you're doing things globally. Ugh.


Add troubleshooting sections and further guidelines to documentation sections

Description
As outcome of several faced complexities, add a troubleshooting section within unit tests and Storybook section in the DevKit and extend the existing guidelines.
Context
Unit tests

Selecting elements and use of data-testid over classes as separation of concerns
Namespaced states mocks
Vm linting issues
Testing elements appended to the document or outside the component (vue-select example handled case)
Test object get/set
Directives
Mapped computed properties to the state or getters
Slots
Dispatched actions with mocked state
Debounce  and computed/method cases with Vue
Case sensitive import in unit tests
Debugging setup
Debug views without components with stubs or shalloMount()
Testing views where is required to input value on select and checkbox inputs components
Mock imports
Testing complex views, e.g. cluster config editor

Storybook

Migrating i18n text in @rancher/components
Require jsdoc comments
Define default controls using Vue2 MDX, by default are all set automatically
Action binding to the existing examples, currently never triggered in any story
Example with Vuex integration, by default we have i18n mandatory
Example with parent component holding data
Global and component styling (SCSS)
<nuxt-link> integration (e.g. DetailTop)

Setup

Build issues in the monorepo (episode with rancher/icons)

Typescript

Extending JS classes
Extending JS mixins

Generic

Some global styling are applied also to other packages, e.g. the case with the headers and Harvester

@cnotv A couple of items more for the list

Ghost errors (yarn test:ci vs yarn test -> one showing errors the other doesn't):
jest tests/unit --clearCache

Problem with method not being called/reachable:
<button type="button" @click="someMethod" /> should be <button type="button" @click="someMethod()" />. If you need to get the event details, pass the arg $event like:
<button type="button" @click="someMethod($event)" />

This needs to be broken into smaller chunks.


Clang importer: Fix the 'self' type of imported ObjC generic members.

A variable was called selfInterfaceType even though it was inconsistently set to a contextual Self archetype for protocols, and an interface type for nominal types, and the one place we used the type expected to work with contextual types. Fixes rdar://problem/26396895.

@swift-ci Please test and merge

Swiftpm test failure looks unrelated.


I can't compile this plugin for UE5

I think something big changed inside Unreal Engine source because I cannot compile doing simple adjustments like changing the project version to "5.0.0" or adjusting SWidget function

Hi, try latest, should be fixed in this commit: https://github.com/Acren/RichTextBlockInlineDecorator/commit/8cba47e615901b3417e579d0398aee0ef793b454


No accessories in Eve for P1

Since the last 2 days the P1 accessory keeps disappearing in the Eve app. After a fresh restart the accessory is visible but disappears within a few hours. The log shows that P1 is recording the values. Any idea wha5 goes wrong?

Does homebridge crash and restart?  Homebridge-p1 won’t block the startup of homebridge if it cannot find the smart meter within the timeout period.  You might set a longer timeout in config.json.  See https://github.com/ebaauw/homebridge-p1#configuration.

I have changed the timeout but the P1 accessory keeps disappearing after several minutes. Only a reboot will solve the problem. Idea where I can start to find the problem?
It's still losing the data:
[7/2/2019, 8:50:53 AM] [P1] Electricity: set homekit total consumption from 25521.625 kWh to 25521.627 kWh
[7/2/2019, 8:50:53 AM] [P1] Electricity: set homekit total consumption normal from 14766.946 kWh to 14766.948 kWh
[7/2/2019, 8:50:53 AM] [P1] Electricity: set homekit current consumption from 779 W to 781 W

Several minutes after what?  Does homebridge restart before the accessory disappears?  Reboot of what?  On one iOS device or on multiple?
Does this still happen when you run homebridge-p1 as only plugin?

I do not see anything in the log why the accessory is disappearing. It is just disappearing after the reboot. So there is no event in the log which I can relate.
Homebridge is also no restarting before the accessory disappears. I have checked that with the uptime of homebridge.
The accessory will appear when I restart Homebridge or a reboot of the RaspberryPI. The disappearing  happens on all iOS devices (iPad and iPhone).
I will try tonight to disable all the other plugins, see what's happening.

Started the search for the disappearing of the accessories. During the homebridge startup I see these following warnings. What does this mean?

That NodeJS fired the heartbeat timer too late.  I wouldn’t worry if this happens once or twice shortly after starting homebridge, but almost 3 seconds (2753 ms) is a long time.  Could be an indication that a plugin is doing synchronous I/O, or otherwise blocking NodeJS.  Wouldn’t explain the disappearing accessories, though.

I removed the home and reset the home app. Now things are back to normal. Strange. Thanks for the support.


Enable glyphed hunter's mark while not having it yourself

As far as I can tell there's no way to sim as if another hunter is playing with both specced improved hunter's mark and also using glyph of hunter's mark. If there is I apologize and please ignore this.

The Hunter's Mark option under 'Debuffs' controls the debuff from another hunter. The option in the 'Player' section controls your own hunters mark (it doesn't have separate values for talent/glyph because those are controlled by your own talent/glyph settings).


Convert image into Sketch

I want to add a script that convert a normal image into sketch using opencv

Type of issue

[*] Feature (New Script)
[ ] Bug
[ ] Documentation

Checklist:

[*] I have read the project guidelines.
[*] I have checked all the existing projects, before submitting a new project issue.
[*] I have checked previous issues to avoid duplicates.
[*] This issue will be meaningful for the project.

@naazneen thanks for showing interest in our repo but please refer #282

You can create this issue in Python_and_the_Web Repo of this organization


Suggested edits to Lift Test Calibration notebook

I have some recommended edits for the Lift Test Calibration notebook.
1. Under the 'Requirements' section we have

Today, we won’t be discussing how to conduct various tests, but instead, we will focus on their utilization. If you wish to acquire knowledge on how to generate results that are compatible with your MMM models, you can check out CausalPy for conducting experiments.

I think it makes sense for us to build an example like that into CausalPy. There is already an issue (in pymc-marketing) to add an example using CausalPy: https://github.com/pymc-labs/pymc-marketing/issues/255. I don't really mind, but I think now that it could be a good idea to base this example in CausalPy.
So I'm just focussing the spotlight on https://github.com/pymc-labs/pymc-marketing/issues/255 again I guess. Happy to talk about how to do that. I'm happy to implement it.
2. Increase clarity around saturation curves
Under the "Indistinguishable Parameter Estimates" heading we have this:

And just below in the "Unobserved Saturation Curves" section we have more saturation plots

It was hard to tell what the differences were between these, so I think we need more explanation. I think the first sentence of this section does attempt to do that, but I think we need more elaboration on these being 'theoretical', hence we can see the whole continuous curve as a function of channel spend. Are they mean estimates based on a model fit, or are they the 'true' curves from our generated data?
And we can perhaps elaborate back up in the first plot that these are 'empirical'. I'm not 100% sure about this, but as far as I understand the terminology of 'direct response curves' means that these are plotted from empirically observed data? Maybe draw from some the docstring explanation of plot_direct_contribution_curves
It might even be pretty cool to directly plot the theoretical and the empirical over the top of each other. It might be easier to visually parse that the curves are pretty different, and to explain why.
In fact, we do this later on in the "Add lift tests to model" section...

Which is a great plot. But can we get some description of what the points (rug) are about? Do they represent x + delta_x? Needs to be made explicit and clear.
And I'm actually wondering: Would it be better to replace the points (direct response curves) with model estimated response curves with HDI's.
3. Increase clarity about lift tests
I know this notebook focusses on adding in data from lift tests to calibrate/refine the MMM parameter estimates, but I think a bit more explanation of the following could be a good idea:

An experiment characterized in this way can be viewed as two points on the saturation curve for the channel.

I also think the functionally important parts of a lift test could be made more clear by explaining (either in text or a docstring of the custom create_lift_test_from_actual_curve function). From what I can tell it's something like:

delta_y is the change in contribution going from a spend of x to x + delta_x.

So, functionally a lift test (or the mysterious output of CausalPy applied to lift test results) is an estimate of the causal impact of the contribution of a channel when it's spend x was changed to x + delta_x. And we can link to a CausalPy example that we put together on this to talk about how we practically get that estimate.
I think the plot of lift test triangles is almost genius level, but needs a bit more explanation. Maybe this could be a 2 panel plot. The upper panel could show the triangles on top of the saturation function, and the lower panel (i.e. the current plot) could remain the same.
But I think it could be a good idea to demonstrate an example of a lift test where the spend (delta_x) goes down. So we'd have a triangle pointing downwards as the contribution will decrease. Or does this mess with the Gamma prior over lift_measurements? If there is a limitation that lift test data has to involve increases in media spend, then that needs to be clearly mentioned.
We could maybe plot df_lift_test within an admonition box and add some text to expand out on "This is what you need from your lift tests in order to use it with pymc-marketing. Here is the definition of the columns. Does the delta_y value have to be estimated channel contribution change, or can it be estimated total change in outcome variable? If it's the former then I think estimating this would require an MMM and be outside the scope of CausalPy? But if it's the latter, then I guess there are a number of ways to do that in CausalPy, including synthetic control.
4. Increased clarity about influence of lift tests
The forest plots showing the influence of the lift tests are great. But can we add some more description to pre-digest this for the reader. For example, the first set of revised parameter estimates are "only a bit better". So we should flag this up explicitly, saying that more lift tests will help you more, and here's an example of doing that.
The conclusion does an excellent job of doing that:

... can be used in order to incorporate experiments into our model likelihood and nudge the model parameters closer to the actuals in this example.

but I think it is beneficial to 'hold-hold' the reader a bit more by giving interpretations of the forest plots as they appear.

Want to make the edits?

Sure. But will need some input on some of the question marks I've got. Would appreciate any pointers on parts of the issue that are overt questions/confusions 👍🏻

@drbenvincent
The article here could help with it.

functionally a lift test (or the mysterious output of CausalPy applied to lift test results) is an estimate of the causal impact of the contribution of a channel when it's spend x was changed to x + delta_x. And we can link to a CausalPy example that we put together on this to talk about how we practically get that estimate.

Great! Could you DM me the website preview render link? Can't seem to find it.


Add support to useServerSeoMeta

Looks like useServerSeoMeta is not supported, right?
https://nuxt.com/docs/api/composables/use-server-seo-meta
We can't see it on the Open Graph tab.
Thanks!

@arashsheyda


docs: Add dagger install constraint

I'm not sure if I have captured this constraint correctly. Can you please review and add suggestions @marcosnils @helderco ?
Closes #8373


[Discussion]: Workaround for error on 'time' modules

Script may fail with following error module 'time' has no attribute 'clock'

Lazy workaround is to open the script and add the following just underneath the imports
time.clock = time.time

Not sure if this was only my experience, but wanted to document the quick fix in case any one else struggles

Thanks for this workaround. I had the same issue and this worked for me.

Worked here too. Thanks

This will work
sudo pip3 uninstall PyCrypto
sudo pip3 install -U PyCryptodome

Can confirm the ^above^ works too. Thanks @sherlocksecurity

The other workaround for this is to use python 3.7!

My preferred option is to use an update Python3 version along with pycryptodome module instead of pycrypto :)

Another thread for the discussion: https://github.com/fullhunt/log4j-scan/issues/17