[1.20.1] [SMP] items filter not working

Is there an existing issue for this?

[X] I did not find any existing issues.

Current Behavior
items filter not working
Expected Behavior
No response
How to Test

place 2 chest and item conduit
setup item filter (add any items in white list)
insert filter into conduit (insert section)
add in chest some items
see that all items moving by conduit

Local Environment
- **Operating System**:
- **Java version**:
- **Minecraft Version**:
- **Version**:
- **Other Mods**: Applied Energistics 2, Mekanism...
EnderIO-1.20.1-6.1.2-beta-all
1.20.1-forge-47.3.1
all other was temp. removed for test

Any Additional Information?
No response

only insert, extract filtering working

only insert, extract filtering working

Thanks for the additional context, I was able to identify the problem. Someone will fix this when they have a second and the fix will be released later today. Thanks for reporting!

fixed on 1.20.1, needs to be ported to 1.21, potentially in #715

Compiled from lts/1.20.1, checked, - insert filtering worked.
Thanks for the wonderful mod.

fixed on 1.20.1, needs to be ported to 1.21, potentially in #715

Seems this is the wrong issue I posted on, this is fixed in both 1.20. and 1.21


BREAKING CHANGE: Adl object refactor

Code gen for this PR: https://github.com/Azure/azure-rest-api-specs/pull/1452 (which is below for reference)

Refactor objects for account and for job submission

This does the following:

Create an explicit set of parameters for job submission. NOTE: Only
U-SQL is supported in this change, no hive jobs are supported at
present.

Create a "Basic" jobInformation that is returned for LIST calls

Setup inheritance for GET jobs

Create an inheritance structure for GET and LIST ADLS and ADLA
accounts.

This also follows the Basic -> Object inheritance pattern.

Fix examples and job_build

Missed some of the example changes

Missed Job_Build, which should have its own parameters object as well.

Moving accountId into the right place in the example.

Updates to include name for buildjob

Description

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

[ ] I have read the contribution guidelines.
[ ] The pull request does not introduce breaking changes.

General Guidelines

[ ] Title of the pull request is clear and informative.
[ ] There are a small number of commits, each of which have an informative message. This means that previously merged commits do not appear in the history of the PR. For more information on cleaning up the commits in your PR, see this page.

Testing Guidelines

[ ] Pull request includes test coverage for the included changes.

SDK Generation Guidelines

[ ] If an SDK is being regenerated based on a new swagger spec, a link to the pull request containing these swagger spec changes has been included above.
[ ] The generate.cmd file for the SDK has been updated with the version of AutoRest, as well as the commitid of your swagger spec or link to the swagger spec, used to generate the code.
[ ] The *.csproj and AssemblyInfo.cs files have been updated with the new version of the SDK.

@ro-joowan,
Thanks for your contribution as a Microsoft full-time employee or intern. You do not need to sign a CLA.
Thanks,
Microsoft Pull Request Bot

@azuresdkci test this please

@ro-joowan test this please


Set CSS value during complete and have queue continue with that value

Queued animations seem to ignore values set in the complete function of prior animation:
http://jsfiddle.net/65xCP/173/
It only works if next animation is nested in the complete.

That's exactly how it's supposed to work, read up on forcefeeding.


Save The Foxy

Hi My name is Harshita Singh
In the game "Save The Foxy" we need to save the fox by dodging the objects that are coming to hit the fox, and by doing this, our fox is saved, if our fox gets hit by the object we will lose 1 of our live. We have 3 lives.
We play this game by pressing button A to move left and D to move right.
I have used  Sprig Editor and the tutorial section to make this game.
Thanks.

@HarshitaSingh18 I am closing this PR. You are free to reopen this PR yourself (or reach out to me directly!) and continue advocating for this code landing, but it seems that it hasn't been acted upon for about 3 months.


chore(project): migrate to pnpm

Noticeable Changes:

Many dev scripts are updated (to replace "yarn run xxx" with "pnpm"), including github actions. These scripts may fail and need more investigating. @BlackHole1 Please help reviewing this.
Common devDependencies are hoisted to the root package.json instead of having several copies at each package, because pnpm will create different symlinks to them if they occured in difference place.
ESLint is upgraded to 8 along with some eslint plugins. The eslint vite plugin was removed due to parsing errors. @BlackHole1 Please help reviewing this change and maybe fix it in the future.
I tried running/building storybook locally and it works. @crimx Please help reviewing this change.

ready merge


[exp] Firestore rxfire function names match Firebase SDK names

In v9 of the Firebase SDK, firebase/firestore exports doc and collection as functions. It may be confusing for developers to have to always remember to be careful with their imports. For example, they'd have to do:
import { doc } from "firebase/firestore";
import { doc as doc$ } from "rxfire/firestore";

const colorDoc = doc(firestoreInstance, "colors", "blue");
doc$.subscribe(colorDoc, () => {
  /* ... */
});

Maybe we should change the names of rxfire's doc and collection? Maybe docSnapshots and querySnapshots?

I like the doc$ function alias or a getDoc$ function alias. Let's start with updating the current firestore examples to eliminate the name errors.


Search docs: still warns about ES aliasing, but referenced bug appears fixed

The docs for ATOMIC_REBUILD say:

Warning
This option may not work on Elasticsearch version 5.4 and above, due to a bug in the handling of aliases affecting these releases.

But https://github.com/elastic/elasticsearch/pull/25043 appears to fix that issue, and the fix looks to be in >=5.5 - see https://github.com/elastic/elasticsearch/pull/25043#issuecomment-306594733.
So I think that this warning is wrong, and that the issue may only affect version 5.4, and that the warning may be incorrectly guiding people away from using ATOMIC_REBUILD with recent versions of Elasticsearch.

Hi @plumdog,
The current wording was put in place after #3985, where a user reported the bug reappearing in 5.6.3. If you can confirm that ATOMIC_REBUILD does indeed work with the latest versions of each major release (and, ideally, verify that you're seeing the same failures on 5.4.x and 5.6.3 so that we know we're dealing with a consistently reproducible issue) then we'll be happy to update this note.

I'm afraid I don't have anything based on experimental evidence right now, and was just working from theory.
However, it looks to me like a reproducer might look like:

a docker-compose.yaml, with Django and ES (and Postgres)
a super simple Wagtail instance, configured to use ES, and with ATOMIC_REBUILD set to True.

then play with the version number for the ES docker image and see if rebuild-index is happy or not.
Is there a public codebase somewhere that would provide me with the simplest Wagtail that is sufficiently complicated to have some to index in search?

Bakerydemo should be sufficient to test search against, I think.
Given that we have no data to indicate that this has been fixed since ES 5.6.3, I'll close this for now, but happy to reopen if there's confirmation that it does work now.

Yes, agreed. I'll report back here if I can reproduce something useful.

@gasman Progress, ish.
I made https://github.com/plumdog/wagtail-es-atomic-index which I think provides a useful test-bed for this. I tested against several ES versions, and found that alias bug really does appear to be fixed in >= 5.5. However, I was not able to make update_index break with any version of ES.
Is there anywhere that might give me a hint about how to induce update_index to fail due to the ES bug?

@gasman So I can't make update_index fail.
But.
Comparing the code for the ES search backend between current master and https://github.com/wagtail/wagtail/pull/3637, I spotted a change that I think might explain why:

time of the PR that avoids 5.4 in tests, https://github.com/wagtail/wagtail/blob/0029291685dbc86eec8fec106d94e0a83569c37d/wagtail/wagtailsearch/backends/elasticsearch.py#L512-L513
master equivalent code, https://github.com/wagtail/wagtail/blob/master/wagtail/search/backends/elasticsearch2.py#L806-L807

This change was introduced in https://github.com/wagtail/wagtail/pull/4128 which fixed the usage of the ES client (because positional arguments changed order in ES Python client 1.7.0).

So I think that anyone using Wagtail without the #4128 fix, which is to say anything not >=2 or >=1.13.2 or >= 1.12.4 and using ES Python client >=1.7.0 would have had broken ATOMIC_REBUILD.
That would explain #3985 (given as "Wagtail Version 1.13", but opened Oct 31st 2017, and fix didn't go into 1.13 until Dec 14th 2017, so some unfixed 1.13 version), and "Elasticsearch PY Version: 5.4.0", so would have had the broken behaviour that #4128 fixed.
Then as for why as of #3637 (June 7th 2017) you saw misbehaving ATOMIC_REBUILD behaviour, I wonder whether it was because this https://github.com/wagtail/wagtail/blob/609d38f17306bbe153184c850f0ecbbaa96f52d7/wagtail/wagtailsearch/tests/test_elasticsearch5_backend.py#L1021-L1023 would have always failed on ES 5.4, because the bug means this always returns True.
Against my test bed, using ES 5.4.2:
>>> from elasticsearch import Elasticsearch
>>> es = Elasticsearch(hosts=['elasticsearch'])
>>> some_index = es.cat.indices(format='json')[0]['index']
>>> print(some_index)
wagtail__wagtaildocs_document_ezdcnhp
>>> es.indices.exists_alias(name='doesnotexist', index=some_index)
True

which is lies, and would break that test. But on ES 5.5.2:
>>> from elasticsearch import Elasticsearch
>>> es = Elasticsearch(hosts=['elasticsearch'])
>>> some_index = es.cat.indices(format='json')[0]['index']
>>> print(some_index)
wagtail__wagtaildocs_document_rbndbkt
>>> es.indices.exists_alias(name='doesnotexist', index=some_index)
False

So I think that the tests failed because the assertions about whether or not aliases existed were getting lies from ES because of the ES 5.4 bug.

Which is all an extremely long-winded way of saying, if you are using:

a Wagtail released after Feb 2018 (>=2, >=1.13.2, >= 1.12.4)
an Elasticsearch released after July 2017 (>= 5.5)

then ATOMIC_REBUILD will work fine. And in fact, I think that a recent Wagtail on ES 5.4 might actually cope OK (at least, I couldn't break it). But given it definitely does use the "check if alias for index exists" that will be getting lies from ES, so I think it is probably safest to say "ES 5.4 not supported for ATOMIC_REBUILD".
So I think that means - thanks to #4128 - the issue really is at worst confined to ES 5.4, and the warning should say as much.

Thanks for the detective work! That sounds like a very plausible explanation of the problems seen on 5.6.3 in #3985 - will go ahead and update the note now.


bufio.go : Fix io.Copy will lost some data

I submitted a IssueIssue:31957 for this bug.
Wen I use io.Copy to copy a file, I find, new file's size less than old file's size
I uesd many files to tried many times, find a rule :
if   old file size  % 4K == 0
No problem
if   old file size  % 4K > 0
new file can be lost  some data
The lost data is old file's end data
The lost data's size is old file's size % 4K
(Code at the bottom)
So I tracking code, I find
in bufio.go, some codes write counter
bufio.io: line 739
When if b.Available() == 0
Because file's size % 4k == 0
So not use Flush
When if b.Available() > 0
Because file's size % 4k > 0
So must to be  Flush
code:
`
oldfile := "E:/aa.txt"		// size: 6K
newfile := "E:/bb.txt"		// When program done,  this file size: 4K
FileOld,_ := os.Open(oldfile)
reader := bufio.NewReader(FileOld)

FileNew,_ := os.OpenFile(newfile, os.O_WRONLY | os.O_CREATE | os.O_TRUNC, 0666)
writer := bufio.NewWriter(FileNew)
io.Copy(writer, reader)

// When the program has been executed, new file size is 4K , but old file size is 6K

`

I signed it!

I submitted a Issue for this bug
Issue:31957
https://github.com/golang/go/issues/31957

@suliang

@openset Yeah,Iknow ,
But I don't think I need call write.Flush() by myself
Because io.Copy is a packaged function
tracking code , You will find, in io.Copy ,is already contains Flush method

@suliang can use condition  b.n > 0 ?
	if err == io.EOF {
		// If we filled the buffer exactly, flush preemptively.
		if b.n > 0 {
			err = b.Flush()
		} else {
			err = nil
		}
	}

@gopherbot So,
You mean is : Every time， I need to call Writer.Flush() after When I call io.Copy() ？

Yes, as the documentation for bufio.Writer says, you must call Flush after writing all your data.


Add phone to order shipping address model

You can enter a phone number for you shipping address but it won't be persisted since the column is missing in the model/database.

Questions
Answers

BC breaks?
no

Tests pass?
yes

Related tickets?
no

How to test?
Enter a phone number in your shipping address. After placing the order the phone number is saved in the s_order_shippingaddress table.

hey @FiveDigital
thanks for your PR. please have a look at the tests. some of them are failing due to your changes

The CI says:

SQLSTATE[42S22]: Column not found: 1054 Unknown column 's11_.phone' in 'field list'

I have added the column in my migration. Perhaps the migration is not applied? Do I have to do more than just add the migration file?

the latest migration number is 809. so you have to change it, to at least 810
otherwise the migration number is duplicated and will have no affect

Okay that fixed the problem :) I don't know why I chose the wrong migration number.

@mitelg All green now 👍

hey @FiveDigital
after talking with my colleagues, we think there are still missing some things. a new field must be nullable and the new field will not show up in the backend. after you have changed your PR we could schedule it.

Done 👍

Done 👍

Hello,
thank you for creating this pull request.
I have opened an issue on our Issue Tracker for you. See the issue link: https://issues.shopware.com/issues/SW-17303.
Please use this issue to track the state of your pull request.

oh, the tests are failing :see_no_evil:

Ah new migration was added in the meantime so I have renamed my new one and rebased the PR.

@mitelg Will this be merged in 5.2 or should I rebase/recreate this PR for 5.3?

@FiveDigital I don't think we are going to merge this in 5.2.
If you like, you can do a rebase. otherwise we will cherry pick your commit :wink:

I have created a new PR for 5.3 https://github.com/shopware/shopware/pull/1172


gccgo compiled app fails with: fatal error: schedule: spinning with local work

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

go version go1.14.6 gccgo (GCC) 10.4.0 solaris/sparc64

Does this issue reproduce with the latest release?
Don't know.
What did you do?

gccgo built application sometimes ends with:
fatal error: schedule: spinning with local work

https://github.com/golang/go/blob/master/src/runtime/proc.go#L3338
Any comment or suggestion how to debug it is welcome!

That clearly shouldn't happen.  It's worth trying a newer version (I believe the current version is 12.2).
I'm not sure what to suggest for debugging this.  You could start by editing the code to print out information about the goroutines on the local run queue.  Then try to figure out how they got there.
I really don't know what is going on here, sorry.  Is it possible to share code that can be used to reproduce the problem?

I don't have myself access to problematic app. I was just told that the problem appears about once per two days.


Standalone web display of BiblioPixel projects.

It must be possible to display a BiblioPixel project on the web without having the user run BP on their own.
This is a collaborative project between BiblioPixel and SimPixel...

This exists: https://github.com/ManiacalLabs/SimPixel/issues/20


Fix mediaengine negotiation when SDPFmtpLine doesn't match

Description
When the remote peer supports a codec with a SDPFmtpLine that local MediaEngine doesn't support, it blindly adds remote offered codec into the negotiated list.
This causes issues Pion peer to accept a codec that it cannot support.. For example, say we supported H264 baseline:
		{
			RTPCodecCapability: webrtc.RTPCodecCapability{MimeType: webrtc.MimeTypeH264, ClockRate: 90000, SDPFmtpLine: "level-asymmetry-allowed=1;packetization-mode=1;profile-level-id=42e01f", RTCPFeedback: videoRTCPFeedback},
			PayloadType:        102,
		}

but remote SDP included the following
a=rtpmap:96 H264/90000
a=rtcp-fb:96 goog-remb
a=rtcp-fb:96 transport-cc
a=rtcp-fb:96 ccm fir
a=rtcp-fb:96 nack
a=rtcp-fb:96 nack pli
a=fmtp:96 level-asymmetry-allowed=1;packetization-mode=1;profile-level-id=640c1f
a=rtpmap:98 H264/90000
a=rtcp-fb:98 goog-remb
a=rtcp-fb:98 transport-cc
a=rtcp-fb:98 ccm fir
a=rtcp-fb:98 nack
a=rtcp-fb:98 nack pli
a=fmtp:98 level-asymmetry-allowed=1;packetization-mode=1;profile-level-id=42e01f

The current negotiated output would include both offered codecs, even though only the latter is supported
Solution
The negotiation process should be using partial matches if exact matches cannot be found. When exact matches are present, they should be used exclusively.

Hey @davidzhao
Yes we would fix this, thank you so much for taking this on! We have two cases I am worried about. I need to look into the answer.

Should we match if locally we don't have fmtp value, and the remote does?
Should we match if locally we have a fmtp value, and the remote doesn't?

Would you be interested in being added to the Pion org? I would love to have you stay involved. These are good/deep fixes and hard to find RTC devs to help :)

Should we match if locally we don't have fmtp value, and the remote does?
Should we match if locally we have a fmtp value, and the remote doesn't?

Sure, let me know. I think the existing fallback behavior is that we would match in both of these cases. I'll wait to hear from you whether we should or not.

Would you be interested in being added to the Pion org? I would love to have you stay involved. These are good/deep fixes and hard to find RTC devs to help :)

Sure, I'd like that. Using Pion for our product and it's been fun to dig in. Would love to contribute more moving forward.

So I think this will cover what you need, and not break users running older WebRTC clients.
Match (MimeType and fmtp)
webrtc.RTPCodecCapability{MimeType: webrtc.MimeTypeH264, SDPFmtpLine: "level-asymmetry-allowed=1;packetization-mode=1;profile-level-id=42e01f"}

a=rtpmap:96 H264/90000
a=fmtp:96 level-asymmetry-allowed=1;packetization-mode=1;profile-level-id=640c1f

No Match (MimeType and fmtp)
webrtc.RTPCodecCapability{MimeType: webrtc.MimeTypeH264, SDPFmtpLine: "level-asymmetry-allowed=1;packetization-mode=1;profile-level-id=42e01f"}

a=rtpmap:96 H264/90000
a=fmtp:96 level-asymmetry-allowed=1;packetization-mode=1;profile-level-id=ffffff

One Match (MimeType)
webrtc.RTPCodecCapability{MimeType: webrtc.MimeTypeH264, SDPFmtpLine: "level-asymmetry-allowed=1;packetization-mode=1;profile-level-id=42e01f"}

a=rtpmap:96 H264/90000

One Match (MimeType)
webrtc.RTPCodecCapability{MimeType: webrtc.MimeTypeH264, SDPFmtpLine: ""}

a=rtpmap:96 H264/90000
a=fmtp:96 level-asymmetry-allowed=1;packetization-mode=1;profile-level-id=640c1f

Are you saying that require exact fmtp matches IF both mediaengine and media description contain FMTP, but otherwise match only MimeType? In that case it'd be easier to enforce these requirements instead of the fuzzy matching logic in my current PR.
Your first example actually would not match under that definition, I'm assuming that's a typo?
Match (MimeType and fmtp)
webrtc.RTPCodecCapability{MimeType: webrtc.MimeTypeH264, SDPFmtpLine: "level-asymmetry-allowed=1;packetization-mode=1;profile-level-id=42e01f"}

a=rtpmap:96 H264/90000
a=fmtp:96 level-asymmetry-allowed=1;packetization-mode=1;profile-level-id=640c1f

Are you saying that require exact fmtp matches IF both mediaengine and media description contain FMTP, but otherwise match only MimeType? In that case it'd be easier to enforce these requirements instead of the fuzzy matching logic in my current PR.
Your first example actually would not match under that definition, I'm assuming that's a typo?
Match (MimeType and fmtp)
webrtc.RTPCodecCapability{MimeType: webrtc.MimeTypeH264, SDPFmtpLine: "level-asymmetry-allowed=1;packetization-mode=1;profile-level-id=42e01f"}

a=rtpmap:96 H264/90000
a=fmtp:96 level-asymmetry-allowed=1;packetization-mode=1;profile-level-id=640c1f

Oh yea that was a typo! Sorry about that, makes these conversations way harder when I can't type basic stuff :)

@Sean-Der what do you think of this approach? It should satisfy the above conditions.
I've kept the prefer-exact-match scenario because it's possible (tho unlikely) one codec could've set fmtpline while the next one didn't. We would still prefer to use the one with it set.

Merged! e5c8c659caa915041dd6dbcaacbb85fe28733820
Fantastic work @davidzhao, I hope you stay involved :)
Feel free to tag a new version if you want! git tag v3.0.12 && git push --tags or you can use the GitHub web ui.

Thanks Sean!


Can not logout

Library

[x ]<EMAIL_ADDRESS>
Framework
Vanilla JS
Description
When you use the logout method it calls the Azure B2C logout endpoint, but it doesn't set the client id as query parameter. You get redirected to your application but it doesn't log you out.
Security
Is this issue security related?
Yes
Regression
Did this behavior work before?
No
Configuration
MSAL Example Configuration
Reproduction steps

Download the MSAL example
Try to logout

Expected behavior
Logout from the service
Browsers
No

This appears to be a duplicate of #1235, closing.


Changing "vm.max_map_count" sysctl conf in a node does not work

To launch an ELK stack, the "vm.max_map_count" (/etc/sysctl.conf) parameter must be set to at least "262144" as mentionned in the documentation.
But the command to change it does not seem to work in PWD nodes :
[node1] (local<EMAIL_ADDRESS>~/swarm-elk
$ sysctl -w vm.max_map_count=262144

[node1] (local<EMAIL_ADDRESS>~/swarm-elk
$ sysctl vm.max_map_count
vm.max_map_count = 65530

Without this parameter set to 262144, elastic container does not start and logs the following error :
ERROR: [1] bootstrap checks failed
myself_elasticsearch.1.zbqincb3vovv@node1    | [1]: max virtual memory areas vm.max_map_count [65530] is too low, increaseto at least [262144]

I think it must have been worked before because some tutorial indicates so. For example : http://collabnix.com/test-drive-elasticsearch-logstash-kibana-on-pwd-platform-running-docker-17-06-ce-swarm-mode-in-5-minutes/
Is it a regression or it this another way to change this option ?
Thank you for your help

hey @yogeek thx for reporting the issue. This is probably a regression. Just changed that flag in the PWD sessions.

Great ! Thank you for the fix and the quick answer.
And thanks for this awesome project 👍


Drupal core upgrade to v7.87 [1h]

We have 3 new core releases:

v7.86 is a security upgrade, so we need to perform it ASAP:

Core version updated on live:

This is done. Closing..


Remove reductant code in exemples

Update readme to remove the code of the initial exemple that is repeated everywhere

All good, looks better like that!


Provide a script to run on Linux

It appears that only a windows batch file is provided.  Please provide a way to run this on Linux.

Currently I'm not working in the Linux PC, If you have can you write a shell script similar to the batch file. we do accept pull requests.

you can use it as below
#!/bin/bash
java -cp "lib/*"  org.audit4j.microservice.Main


Sense and scale flow sensors (2x)

Read flow sensor voltages
Apply calibration data to scale measured voltages into SI units

I worked on this (pending some breakages with platformIO that I'm in process of fixing). Is there something I'm missing or should this be considered closed?
PR #74

I think this is done.  @verityRF are you interested in fixing https://github.com/RespiraWorks/VentilatorSoftware/issues/101?  I think this is a simple matter of plugging these two things together, but I am not 100% sure.


Allow rails 5.1 and 5.2

In addition to change the rails version range to < 5.3 the PR adds appraisal gemfiles for rails 5.1 and 5.2 and updates travis.yml to use them.

cc @barelyknown

@barelyknown can we activate Travis to run the tests?


Cache the GenTxId in GenTx so that querying it is cheap

Closes #780.
The docstring of txId says it should be cheap to get the GenTxId of a
GenTx. This was not the case as the transaction was being hashed each time,
both for Mock and Byron blocks. Solve this by caching the hash.

Force-pushed a fix for both comments


MPU6050 clash detection works but no clue why...?

Hi neskweek,
Yesterday I've been experimenting again with the clash detection, because it still seems to be not as responsive as it should be. I looked at the code and also read the MPU6050 Manual, all about motion interrupts.
It seems there are only 4 parameters playing a roles:
These settings change the full scale range of the accelero/gyro (not yet in LSOS...). These should change the sensitivity of the saber because an LSB is scaled with full range.
// set the fll scale range of the gyro- and accelerometer respectively
mpu.setFullScaleGyroRange(0); //0: 250deg/s | 1: 500deg/s | 2: 1000deg/s | 3: 2000deg/s
mpu.setFullScaleAccelRange(0); //0: 2g | 1: 4g | 2: 8g | 3: 16g
Then there are the settings for the motion interrupt. The first one sets the threshold, but no idea how...I guess the number means LSB. But it's strange, with a setting of 10, we should hear clash triggering all the time even if we just shudder.
mpu.setMotionDetectionThreshold(10); // 1mg/LSB
mpu.setMotionDetectionDuration(1); // number of consecutive samples above threshold to trigger int
I experimented with following things (using 2 sabers both with DIYino):
set mpu.setFullScaleAccelRange(0) and mpu.setFullScaleGyroRange(0) once to 0 and once to 3 (2 extreme values) -> no change in the clash detection, no tangible effect on other motion detection
play with mpu.setMotionDetectionDuration(1) -> no change in clash detection
Funny thing is, my LED string saber is no (without any change) very responsive, even small taps will trigger clash (the right way!), while my new neopixel saber (using also DIYino) sometimes even fail to detect quite hard hits...I re-calibrated both MPUs and also tried with a sketch what they read, they read and are calibrated both perfectly.
So I feel we still do not fully understand how this motion interrupt feature works...although clash is by far the most important feature of a saber (besides the LEDs).
It is more a fundamental topic, not an immediate bug, but I did not find a better place to report/discuss.
Cheers: Andras

In fact it's more trickier than you might think :+1:
mpu.setIntMotionEnabled(true);
mpu.setMotionDetectionThreshold(10); // 1mg/LSB
mpu.setMotionDetectionDuration(1); // number of consecutive samples above threshold to trigger int
are the parameter involved in this "clash" detection.
The thing is that, on a normal use case this system is meant to detect a movement and generate an interrupt. In this config if the system get a movement superior to the treshold on 1 sample it trigers the interrupt => Clash.
The treshold is "so high" for a set of samples so small here, that only a clash triggers it.
For your Neopixel hilt you might just want to lower the treshold or In this build your MPU is maybe moving to much inside the hilt not allowing it to take a hit :p  (just a supposition ;) ).

:) we think quite similar, my first assumption was also that the DIYino is moving around in the hilt and therefore is not rigid enough for a dependable clash detection. But looking at the inside of the hilt I cannot confirm this...It cannot move, there is simply no space left... I can also lover the threshold, of course with 1 even small movements can trigger a clash, with 5+ no change, but what bugs me is that with the 2g full-scale setting you have 2g/32768=0.06mg/LSB -> a threshold of 10 corresponds to 0.6mg, which is less than a twich of a finger... I simply do not undrstand the scaling of this threshold, but the fact that it also does not change behaviour with the full-scale change means it is somehow indenpenent of the full-scale range and resoultion...but then what does it mean? If I increase it to 1000 (1g=16000), I get no more clash at all!
It is also interesting, that even disabling motion_int will not stop the interrupt to be latched into the status register. I can disable all interrupt sources but clash will still work.

So when You just twich the card with your finger it clashes ?
Looking at the photo of your setup on your bucket, maybe cables secures it but it might also react like springs or foam, and absorbing the shock.
for motion int, beware: on previous versions of LSOS I didn't saw there was twice this instruction. Maybe you still have those 2.
I maybe have another solution.
I was saving this for a future email but hey ... if it works better on this build. Keep in mind that Ionly starte"d to experiment that this afternoon. Treshold maybe needs tweaking (augmenting) :
add/modify those lines :
mpu->setIntZeroMotionEnabled(true);
mpu->setZeroMotionDetectionThreshold(100); // 2mg/LSB
mpu->setZeroMotionDetectionDuration(1); // number of consecutive samples under threshold to
trigger int```

and replace this :
`mpuIntStatus > 60 and mpuIntStatus < 70`
by this :
`mpuIntStatus > 30 and mpuIntStatus < 40`

if it works, I'll explain that in this future mail ^_^,

Should be corrected in V1.2.1


POST_disable_authy action does not clear the session["#{resource_name}_authy_token_checked"] variable

I have parts of my layout views that are dependent on 2FA having been passed (which can be checked via the value of session["#{resource_name}_authy_token_checked"], which is set via record_authy_authentication in helpers.rb).
When I logged in as a User and verified with 2FA, then clicked a button that would POST to the disable_authy_path for my resource, it did not reset the session variable (session[:user_authy_token_checked] in my case). I checked the code for the POST_disable_authy action in devise_authy_controller.rb to verify and saw that there is no logic to clear this.
I will work on submitting a PR for a fix in the next day or so.

This library is no longer actively maintained. The Authy API has been replaced with the Twilio Verify API. Twilio will support the Authy API through November 1, 2022 for SMS/Voice. After this date, we’ll start to deprecate the service for SMS/Voice. Any requests sent to the API after May 1, 2023, will automatically receive an error.  Push and TOTP will continue to be supported through July 2023.
Learn more about migrating from Authy to Verify.
Please visit the Twilio Docs for:

Verify + Ruby (Rails) quickstart
Twilio Ruby helper library
Verify API reference
Coming soon: Look out for a new Devise plugin to use Twilio Verify with Devise

Please direct any questions to Twilio Support. Thank you!


search by PMID

Description

[x] WHY - What is the problem this feature would answer?

Users wants to see the genes and disease involved by a single paper, or all the evidence that is associated with the same piece of literature.

[ ] HOW are we going to solve the problem?

Library API

[ ] WHAT would this look like?

Screenshots and mock-ups are great

Progress Tracker
Get to a prototype at branch.targetvalidation.org:

Write code + tests + docs. Isolate the work in feature branches.
Use as many PRs as you need.
Write tests in the same or different PRs, as is convenient for you.
As each PR is merged, add a comment to this issue referencing the PRs.

[ ] Feature tag Code should have a feature tag to be disabled by default.
[ ] Minimal functional testing
[ ] Minimal docs
[ ] if on webapp, test a deployment on netlify

Push to staging

[ ] merged code and pushed to staging.targetvalidation.org
PULL request should be reviewed by 1-2 people at least
[ ] at least 1 week internal testing?
[ ] received some feedback/input from users (eg. pharma scientist)

Released

[ ] in the release notes?
[ ] documentation updated?
[ ] blog post?
[ ] users that requested it originally informed?

Make sure you can measure impact

As dev team we want to know if a new feature are succesful.
To do that we need to define metrics to assess the impact of each new feature we introduce to the platform

[ ] metric identified
[ ] identify the one metric that matters (OMTM)

As on 22Aug2018 checks:
can find PMID in library but no abstract.
Search in LINK is very slow to load.
In OT, cannot search for PMID directly via search bar but in bibliography, abstract is presented with annotations as expected.

To be considered under pipeline refactor high level topic


[Suggestion] Remove too expensive message on anvils

Can you make it so that we don't get that too expensive message on anvils?

Quoting myself from the CurseForge page:

This is (at least currently) not planned because there's a lot of mods that already do this, e.g. NoExpensive for Fabric, which is tested compatible - not sure about Forge but there has to be one too.

There's also Anvil Fix for 1.16, a 1.18 port called Anvil Fix Binchibubil and another mod called Forging Beyond for 1.17-1.18 which should all be compatible with Tax Free Levels.

For Forge 1.18, Xp Is No Object seems to be a good one that surely is compatible too.

yeah, the problem with most of those other mods that do a similar thing, is that they are just normally updated for a very short time, so e.g. 1.19 comes around and now the mod and the feature is dead because it was never updated...
So I prefer to stick with mods that have passed the test of time... test of versions... you know what I mean XD.

Anvil Fix with 300k downloads did pass the test of time, I just saw it now has official 1.18 versions too.
The thing about adding the feature is that it needs to be configurable, which I usually do by adding Cloth Config as a dependency, but adding a huge dependency for a single setting isn't ideal to say the least.
I'd need to release at least 6 new version for Fabric and Forge 1.16-1.18 too, hence why I'm holding off until there's a more substantial reason.
When 1.19 comes around, I might add it as a config file exclusive option (so it won't need Cloth Config).

Actually, adding it as a setting would even require server -> client config synchronization...
Yeah, I'm definitely leaving that for the future. 😄

well the guy from xp is no object said, it is only 10 lines of code, shouldn't be that complicated right? XD

Yes, but I don't want to have the feature enabled by default (because it's not the focus of this mod) and that's what makes everything so complicated.

ohhh, I see, ok, hopefully you can change your mind in the future!, tx!

Finally came around to it.
1.4 removes "Too Expensive!" by default, it's configurable and the server -> client sync is there.


Bug/Security: usage of 'eval' function is blocks pdf rendering when CSP is enabled

PDF cannot be previewed due to the usage of eval function when enabling Content Security Policy with script-src 'self'.
In order to overcome this issue unsafe-eval has to be added, but this would only increase security vulnerabilities.
vue-pdf-embed - 1.1.2
vue - 3.2.31
Browser - Firefox/Chrome

can confirm!

Hi @Diana5,
The eval function is used for the worker loading. I'm not sure if there are other options to load it, but I'll check.

hi,
Any news about this issue ? we meet the same in our project.

No easy solution so far, but I'm planning to add more builds in the next minor release, so that the lib can be imported without the worker

thanks for the answer.
We previously used the lib pdfvuer and we had no issue like that (but it has other issue). it use worker's too, but the way is it loaded is different. Maybe it's an idea.

Any news on this issue?

In v2.0.0 there is an additional build (index.essential.mjs) exported which doesn't load the worker, but exposes GlobalWorkerOptions instead. Hopefully, using it along with the worker loaded from PDF.js would be a solution.
Please check a note about it in README


union strange result

ddedAllowedConns: protocol: ICMP icmp-type: 135-136, prev allowed conns: protocol: ICMP
reunion result of the two above:
new allowed conns: protocol: ICMP icmp-type: 0-16,135-136

the issue is tha max value of  icmp-type is defined  at the model to be 16, the max value of icmp-code is 5.
these max values are based on https://datatracker.ietf.org/doc/html/rfc792.
on the other hand, these parameters has 8bits, and and defined to be 0-255 here:
https://vdc-download.vmware.com/vmwb-repository/dcr-public/ce4128ae-8334-4f91-871b-ecce254cf69e/488f1280-204c-441d-8520-8279ac33d54b/api_includes/types_ICMPTypeServiceEntry.html
in practice, at the services I get from the nsx api, the values of icmp-code is 0, (or missing), the values of icmp-type is 0-143

@adisos @zivnevo @kyorav

Thanks @haim-kermany . Can you push the branch with the test that demonstrates this issue?

created a pr with the test only.

Pushed to this branch a fix for this issue.


Toolbar doesn't respect "itemComponent" for items with "locateInMenu": "always"

Issue Type

bug report
Steps to Reproduce

Init Toolbar with
items ={ [          {            "locateInMenu": "always",            "location": "before",            "widget": 'button',            "options": {              "text": "Contacts"            }          } ]}
and
 ItemComponent ={() => {        return (
Expected
);      }}
Open menu in toolbar.

Current Behavior
ReactToolbar ignore ItemComponent and create button widget in menu.
Expected Behavior
ReactToolbar respect ItemComponent and render in
Expected
 menu.
Demo

https://stackblitz.com/edit/devextreme-react-rgobqk?file=index.js
Environment Details

devextreme 18.1.1-pre-18093
devextreme-react 18.1.1-alpha.4
react 16.3.0

Hi,
It seems that the problem is not the React integration issue, see [this example]((https://stackblitz.com/edit/devextreme-mwaggk?file=index.js).
DevExtreme React *Component and *Render syntax is a "mirror" of the corresponding *Template syntax in the DevExtreme API.
Please feel free to file a ticket in the DevExpress Support Center.


🐛 Never goes to sleep

branch
stable
Architecture
x86_64
Description
My framework laptop battery runs out from time to time, because manjaro sway does not automatically go to sleep. https://github.com/manjaro-sway/manjaro-sway/issues/261 seems to indicate otherwise, but I am not seeing that.
When I access my laptop after a while of inactivity, it is only blurred and locked, not sleeping.
Reproduction steps
leave it alone

Screenshots
No response
Logs
No response

Just confirmed today. Heard the Stretchly bell regularly.

I tested the fix and think it worked. Let me know if this didn't fix it.


Conflict WIth AutoComplPop plugin

When using multiple cursors in insert mode, I'm seeing lots of errors with AutoComplPop plugin.
This plugin provides command to enable/disable itself: :AcpEnable and AcpDisable.
Is there a way to hook a custom command into vim-multiple-cursors that gets triggered on any transition into or out of insert mode?  If not, how could I hack around this?

Thanks @kristijanhusak, still fixes it in 2016


Potential class prototype

Description
Minimal prototype of a Potential class mirroring gmso.Potential but built off of Pydantic.
Todos
Notable points that this PR has either accomplished or will accomplish.

[ ] Think about more complicated and interesting cases that will break this
[ ] Think about interoperability with gmso.Potential
[ ] More tests
[ ] Importing from analogous openforcefield.typing.engines.smirnoff.parameters objects
[ ] Exporting to analogous openforcefield.typing.engines.smirnoff.parameters objects

Questions

[ ] Should this exist alongside gmso.Potential or should it be designed as a class to be inherited/constructed?

Status

[ ] Ready to go

Rough around the edges but (if we want to) it should be feasible to digest a parameter within a SMIRNOFF force field an instantiate it here, based on this structure
$ ipython
from evaluator.utils.openmm import openmm_quantity_to_pintPython 3.7.6 | packaged by conda-forge | (default, Mar 23 2020, 23:03:20)
Type 'copyright', 'credits' or 'license' for more information
IPython 7.14.0 -- An enhanced Interactive Python. Type '?' for help.

In [1]: from evaluator.utils.openmm import openmm_quantity_to_pint

In [2]: from openforcefield.typing.engines.smirnoff import ForceField
Warning: Unable to load toolkit 'OpenEye Toolkit'. The Open Force Field Toolkit does not require the OpenEye Toolkits, and can use RDKit/AmberTools instead. However, if you have a valid license for the OpenEye Toolkits, consider installing them for faster performance and additional file format support: https://docs.eyesopen.com/toolkits/python/quickstart-python/linuxosx.html OpenEye offers free Toolkit licenses for academics: https://www.eyesopen.com/academic-licensing
_ColormakerRegistry()

In [3]: from system.potential import AnalyticalPotential

In [4]: ff = ForceField('openff-1.1.0.offxml')

In [5]: parsley_bond = ff.get_parameter_handler('Bonds').get_parameter({'id': 'b1'})[0]

In [6]: bond_potential = AnalyticalPotential(name='MyBond', expression='0.5*k*(l-l_0)', parameters={'k': openmm_quantity_to_pint(parsley
   ...: _bond.k), 'l_0': openmm_quantity_to_pint(parsley_bond.length)}, independent_variables={'l_0'})

In [7]: bond_potential.parameters
Out[7]:
{'k':<PHONE_NUMBER>218 <Unit('kilocalorie / angstrom ** 2 / mole')>,
 'l_0': 1.520564290937 <Unit('angstrom')>}

Codecov Report

Merging #2 into master will increase coverage by 12.72%.
The diff coverage is 100.00%.

This was merged into #9 and should have been closed automatically ....


Clarify timezone for Lambda recurring schedule

Note: If your question is regarding the AWS Amplify Console service, please log it in the
AWS Amplify Console repository

Which Category is your question related to?
function
Amplify CLI Version
5.3.0
What AWS Services are you utilizing?
Lambda
Provide additional details e.g. code snippets
When I setup a lambda on recurring schedule, the default time amplify cli suggest is current time on my local machine. So I assume then I am setting the lambda's schedule based on my timezone and amplify cli will convert to UTC. For example I set my lambda to run 7:11am MST each day. But then checking the deployed lambda I see the time is not converted to UTC: cron(11 7 * * ? *).
Amplify CLI should be super clear what timezone I'm inputting the time in. It seems to imply it's my timezone because it picked the current time, but turns out it's current time in UTC which is not my current local time.

@hisham thanks for the heads up I mark it as a bug because it is an easy DX change in the prompts, the time given is in UTC as you also figured since that's how Lambda works: https://docs.aws.amazon.com/lambda/latest/dg/services-cloudwatchevents-expressions.html


Mobile site: "join" button doesn't work

@2xAST can you try again - wonder if you might have been testing an old version before this PR was merged?
I've just tried on safari and brave and it's working on both.

@gpxl-dev yes that was the case :) works perfectly now


点击签到报错

2022-11-29 14:29:40 DEBUG: 发现“xxx邀请您使用签到”
2022-11-29 14:29:46 ERROR: 未找到签到窗口！
Exception in thread signin:
Traceback (most recent call last):
File "threading.py", line 1038, in bootstrap_inner
File "threading.py", line 975, in run
File "main.py", line 243, in task_signin
File "main.py", line 118, in clickSigninButton
File "pyautogui_init.py", line 598, in wrapper
File "pyautogui_init_.py", line 983, in click
File "pyautogui_init_.py", line 1509, in mouseMoveDrag
File "pyautogui_init.py", line 1722, in failSafeCheck
pyautogui.FailSafeException: PyAutoGUI fail-safe triggered from mouse moving to a corner of the screen. To disable this fail-safe, set pyautogui.FAILSAFE to False. DISABLING FAIL-SAFE IS NOT RECOMMENDED.

腾讯会议最近更新了一次，脚本暂时失效了。等我修复吧。

刚刚发布了新版本v1.0.1，修复了自动签到功能。


python311Packages.bimmer-connected: 0.13.3 -> 0.13.5

Diff: https://github.com/bimmerconnected/bimmer_connected/compare/refs/tags/0.13.3...0.13.5
Changelog: https://github.com/bimmerconnected/bimmer_connected/releases/tag/0.13.5
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

[ ] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 232677 run on x86_64-linux 1

5 packages built:

home-assistant-component-tests.bmw_connected_drive
python310Packages.bimmer-connected
python310Packages.bimmer-connected.dist
python311Packages.bimmer-connected
python311Packages.bimmer-connected.dist


Attribute names (incorrectly) cannot start with a Unicode character

Hi!
While parsing the XML file, If the first character of unquoted Attribute name is not Ansi-127, it is replaced with 1-byte code and is not converted to Utf properly. The rest characters are OK. Element names are OK too.
The effect exists from early mxml versions up to now.
Could you please replace just 2 lines in minixml-3.1 -> mxml-file.c -> mxml_parse_element:
name[0] = ch;         (line 2191)
ptr     = name + 1;  (line 2192)
with something like that:
ptr = name;
if (mxml_add_char(ch, &ptr, &name, &namesize))
goto error;

Will incorporate this fix into the next Mini-XML release.

Thanks a lot
George Razutov
----- Original Message -----
From: Michael R Sweet
To: michaelrsweet/mxml
Cc: Yurgis57 ; Author
Sent: Friday, June 12, 2020 2:15 PM
Subject: Re: [michaelrsweet/mxml] Attribute names (incorrectly) cannot start with a Unicode character (#267)
Will incorporate this fix into the next Mini-XML release.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

[master dd55272] Fix handling of elements that start with a Unicode character (Issue #267)


bug: admonition class names don't remove spaces (e.g. "See Also")

Describe the bug
context
   ```{seealso}
      seealso
   `` `

expectation
<aside class="admonition seealso">
  <header class="admonition-title">See Also</header>
  <p>seealso</p>
</aside>

bug
<aside class="admonition see also">
  <header class="admonition-title">See Also</header>
  <p>seealso</p>
</aside>

solution
admonitions.ts#L31 should read:
if (this.title) adToken.attrJoin("class", this.title.replace(/ /g, '').toLowerCase())

Personally, I would prefer it to read this.title.replace(/ +/g, '-') – converting admonition title spaces into kebab-case, as this would make the resulting CSS styling nicer to read. Doing so would mean updating the current CSS rules for seealso however, and so might be a larger change.
Reproduce the bug
   ```{seealso}
      seealso
   `` `

List your environment
No response

Yep thanks, I would note that sphinx adds it as seealso: https://github.com/sphinx-doc/sphinx/blob/e54b63d3eead95ef3fac1906deae54b6972573e4/sphinx/writers/html.py#L289, so I would want to stick to that

This is fixed in #22! Thanks for the report @Pugio.

This has been merged into main, but not yet released.

Released in 0.1.2:
https://github.com/executablebooks/markdown-it-docutils/releases/tag/v0.1.2


SQL Queries wrong type

Hi,
I tried to use SQL queries in a DAO but in those queries I used a WHERE condition using DateTime parameter (BETWEEN start and end).
When running the build step, moor is generating both queries with parameters as double instead of DateTime. Maybe I'm doing something wrong.
The transactions table
@UseMoor(tables: [Transactions], daos: [TransactionDao])
class Transactions extends Table {
  IntColumn get id => integer().autoIncrement()();
  TextColumn get title => text().withLength(min: 1, max: 32)();
  RealColumn get amount => real()();
  DateTimeColumn get date => dateTime()();
  BoolColumn get isEssential => boolean().nullable()();
  BoolColumn get isRecurrent => boolean().nullable()();
  IntColumn get categoryId => integer()();
}

The SQL queries
@UseDao(
  tables: [Transactions],
  queries: {
    'dayIncomes':
        'SELECT sum(t.amount) FROM transactions t WHERE t.date BETWEEN :start and :end AND t.amount >= 0 GROUP BY date(t.date);',
    'dayExpenses':
        'SELECT sum(t.amount) FROM transactions t WHERE t.date BETWEEN :start and :end AND t.amount <= 0 GROUP BY date(t.date);'
  },
)

The generated methods
mixin _$TransactionDaoMixin on DatabaseAccessor<FibDatabase> {
  Selectable<String> dayIncomes(double start, double end) {
    return customSelect(
        'SELECT sum(t.amount) FROM transactions t WHERE t.date BETWEEN :start and :end AND t.amount >= 0 GROUP BY date(t.date);',
        variables: [Variable.withReal(start), Variable.withReal(end)],
        readsFrom: {}).map((QueryRow row) => row.readString('sum(t.amount)'));
  }

  Selectable<String> dayExpenses(double start, double end) {
    return customSelect(
        'SELECT sum(t.amount) FROM transactions t WHERE t.date BETWEEN :start and :end AND t.amount <= 0 GROUP BY date(t.date);',
        variables: [Variable.withReal(start), Variable.withReal(end)],
        readsFrom: {}).map((QueryRow row) => row.readString('sum(t.amount)'));
  }
}

The issue is that I want to pass DateTime parameters and not double (double start, double end).
What I want to do is select all transactions between 2 dates, group them by day and get the sum of all their amount.
I tried with moor Dart API but without success, that why I wanted to achieve it using SQL.
Thank you for your time :)

Thanks for the report! However, I can't reproduce this locally. I setup a new Dart project depending on moor and moor_generator versions ^3.3.0. I'm using the following content:
import 'package:moor/moor.dart';
part 'db.g.dart';

class Transactions extends Table {
  IntColumn get id => integer().autoIncrement()();
  TextColumn get title => text().withLength(min: 1, max: 32)();
  RealColumn get amount => real()();
  DateTimeColumn get date => dateTime()();
  BoolColumn get isEssential => boolean().nullable()();
  BoolColumn get isRecurrent => boolean().nullable()();
  IntColumn get categoryId => integer()();
}
@UseMoor(
  tables: [Transactions],
  queries: {
    'dayIncomes':
        'SELECT sum(t.amount) FROM transactions t WHERE t.date BETWEEN :start and :end AND t.amount >= 0 GROUP BY date(t.date);',
    'dayExpenses':
        'SELECT sum(t.amount) FROM transactions t WHERE t.date BETWEEN :start and :end AND t.amount <= 0 GROUP BY date(t.date);'
  },
)
class Database extends _$Database {
  Database(QueryExecutor e) : super(e);

  @override
  int get schemaVersion => throw UnimplementedError();
}

This generates a dayExpenses(DateTime start, DateTime end) and dayIncomes method, as I'd expect. Can you check that you're using the latest moor version too?

I tried with moor Dart API but without success

This should work:
Selectable<double> dayIncomes() {
  final sum = transactions.amount.sum();

  final query = selectOnly(transactions)
    ..addColumns([sum])
    ..where(transactions.amount.isBiggerOrEqualValue(0))
    ..groupBy([transactions.date.date]);

  return query.map((row) => row.read(sum));
}

For now, you'll also need the following extension (but the function will be builtin in the next moor version):
extension on Expression<DateTime> {
  Expression<String> get date {
    return FunctionCallExpression(
      'DATE',
      [this, const Constant<String>('unixepoch')],
    );
  }
}

@simolus3 I was not using the last version (3.1.0 instead of 3.3.0), I updated it but I run into the same issue.
By looking at you test, I saw that your queries were at the database level, with @UseMoor while on my side I was putting the queries in the @UseDao, at DAO level.
I switch my queries to the database level and it worked fine like that.
Can you check on your side that there is not an issue when using the queries at DAO level?

I'm now using
import 'package:moor/moor.dart';

part 'database.g.dart';

class Transactions extends Table {
  IntColumn get id => integer().autoIncrement()();
  TextColumn get title => text().withLength(min: 1, max: 32)();
  RealColumn get amount => real()();
  DateTimeColumn get date => dateTime()();
  BoolColumn get isEssential => boolean().nullable()();
  BoolColumn get isRecurrent => boolean().nullable()();
  IntColumn get categoryId => integer()();
}

@UseMoor(
  tables: [Transactions],
  daos: [SomeDao],
)
class Database extends _$Database {
  Database(QueryExecutor e) : super(e);

  @override
  int get schemaVersion => throw UnimplementedError();
}

@UseDao(
  tables: [Transactions],
  queries: {
    'dayIncomes':
        'SELECT sum(t.amount) FROM transactions t WHERE t.date BETWEEN :start and :end AND t.amount >= 0 GROUP BY date(t.date);',
    'dayExpenses':
        'SELECT sum(t.amount) FROM transactions t WHERE t.date BETWEEN :start and :end AND t.amount <= 0 GROUP BY date(t.date);'
  },
)
class SomeDao extends DatabaseAccessor<Database> with _$SomeDaoMixin {
  SomeDao(Database attachedDatabase) : super(attachedDatabase);
}

Still, no problems.
I saw that you had the @UseMoor annotation on a table class, but you need to use another class for your database. Also, make sure you're upgrading moor_generator too! Maybe flutter pub run build_runner clean helps as well.

Closing this since I can't reproduce this and it seems to work for me. If you still run into this problem (and have a good way to reproduce this), please let me know.


Show planting site map in popup

To avoid cluttering the admin UI, the inline map on the planting site page had a
fixed height, which made it awkward to view sites with tall thin shapes. Move
the map to a popup window that can be resized as the user sees fit.

Current dependencies on/for this PR:

main

PR #1123

PR #1124   👈

This comment was auto-generated by Graphite.


Merge changes from alpha into master branch

Also update README for v2.0.0 https://github.com/jubilee-works/timetree-sdk-js/pull/275/commits/5d06a23c6099ce3ea6c143c4c7a45b157bae307d

@fujikky Thank you for suggestions! Updated.

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

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Doesn't work with Plex for Kodi

It's working fine for me in the Plex Android app but when using Plex for Kodi I get the following error:
2019-12-20 18:12:35,425 :: bw_plex :: DEBUG :: plex.py:1188 :: Found Futurama.s06e02 theme start 00:06, theme end 00:13, ffmpeg_end 07:00 progress 00:02 fake_progress 00:12 best_time 00:13 credits_start 00:00 credits_end 00:00
2019-12-20 18:12:35,426 :: bw_plex :: DEBUG :: plex.py:1229 :: Futurama.s06e02 is in the correct time range theme_end
2019-12-20 18:12:35,426 :: bw_plex :: DEBUG :: plex.py:1157 :: Called jump with Futurama.s06e02 42 13.862312925170068 None
2019-12-20 18:12:35,428 :: bw_plex :: INFO :: plex.py:931 :: Called client_action with 13.862312925170068 00:13 42 None
2019-12-20 18:12:35,529 :: bw_plex :: INFO :: plex.py:964 :: sessionkey 42 media_sessionkey 42
2019-12-20 18:12:35,530 :: bw_plex :: INFO :: plex.py:968 :: client Kodi Add-on (localhost) 2.911
2019-12-20 18:12:35,530 :: bw_plex :: INFO :: plex.py:995 :: Checking if we cant find the correct client
2019-12-20 18:12:35,531 :: bw_plex :: INFO :: plex.py:997 :: 82d73289ca7e5af2-com-plexapp-android 8c423f5e-2a31-47bb-8f41-5d1048b03878
2019-12-20 18:12:35,531 :: bw_plex :: INFO :: plex.py:997 :: c0bf744e04b03449-com-plexapp-android 8c423f5e-2a31-47bb-8f41-5d1048b03878
2019-12-20 18:12:35,531 :: bw_plex :: INFO :: plex.py:997 :: o15p7jep2xrhnryqzmnl7iom 8c423f5e-2a31-47bb-8f41-5d1048b03878
2019-12-20 18:12:35,532 :: bw_plex :: DEBUG :: plex.py:1006 :: We couldnt match the client. Trying a hail marry.
2019-12-20 18:12:35,532 :: bw_plex :: DEBUG :: plex.py:1019 :: Client hasnt a _baseurl or is a browser, enabling proxyThroughServer
2019-12-20 18:12:35,533 :: bw_plex :: DEBUG :: plex.py:1040 :: calculated_offset 13 0.013
2019-12-20 18:12:35,619 :: bw_plex :: ERROR :: plex.py:89 :: There was an exception in client_action
Traceback (most recent call last):
  File "/app/bw_plex/bw_plex/plex.py", line 84, in inner
    return func(*args)
  File "/app/bw_plex/bw_plex/plex.py", line 1043, in client_action
    proxy_on_fail(correct_client.seekTo(calculated_offset * 1000))
  File "/usr/local/lib/python3.6/dist-packages/plexapi/client.py", line 344, in seekTo
    self.sendCommand('playback/seekTo', offset=offset, type=mtype)
  File "/usr/local/lib/python3.6/dist-packages/plexapi/client.py", line 204, in sendCommand
    return query(key, headers=headers)
  File "/usr/local/lib/python3.6/dist-packages/plexapi/server.py", line 381, in query
    raise BadRequest('(%s) %s; %s %s' % (response.status_code, codename, response.url, errtext))
plexapi.exceptions.BadRequest: (500) internal_server_error; http://<IP_ADDRESS>:32400/player/playback/seekTo?commandID=1&offset=13000&type=video <?xml version='1.0' encoding='utf-8'?> <Response code="2000" status="FrameworkException: Unable to find player with identifier 8c423f5e-2a31-47bb-8f41-5d1048b03878">   <Traceback>Traceback (most recent call last):   File "/volume1/@appstore/Plex Media Server/Resources/Plug-ins-349e9837e/Framework.bundle/Contents/Resources/Versions/2/Python/Framework/components/runtime.py", line 843, in handle_request     result = f(**d)   File "/volume1/@appstore/Plex Media Server/Resources/Plug-ins-349e9837e/System.bundle/Contents/Code/playerservice.py", line 50, in process_remote_command     raise FrameworkException('Unable to find player with identifier %s' % identifier) FrameworkException: Unable to find player with identifier 8c423f5e-2a31-47bb-8f41-5d1048b03878 </Traceback> </Response>```

Thanks for the report!
I dont think plexapi works with this client. Its easy to test if you do Plexserver().clients() the client needs to be listed there for it to work. In this case, we dont have url for the kodi addon client and (_baseurl in the errors msg) so we try to relay the command to the server but even that fails.


Watch for logs within subdirectories.

Signed-off-by: Saman Alvi<EMAIL_ADDRESS>

This'll emit all logs recursively with the tag determined by the first subdirectory. Does that sound OK? That's probably why the negative-case test was there; we just didn't decide since we had no use case for it.
That is, the following log files:
atc/foo.log
atc/xxx/bar.log
baggageclaim/baz.log
baggageclaim/buzz.log

will be emitted with the following tags: atc, atc, baggageclaim, baggageclaim.

@vito We'll modify it to include all the directories so it's similar to:
atc/foo.log
atc/xxx/bar.log
baggageclaim/bar/baz/baz.log

will be emitted with the following tags: atc, atc/xxx, baggageclaim/bar/baz

i would prefer to include filename as well. that way its clear if something is being logged to migrate.log vs stdout.log for example.
Sent from my iPhone

On Oct 31, 2016, at 1:05 PM, Andrew Su<EMAIL_ADDRESS>wrote:
@vito We'll modify it to include all the directories so it's similar to:
atc/foo.log
atc/xxx/bar.log
baggageclaim/bar/baz/baz.log
will be emitted with the following tags: atc, atc/xxx, baggageclaim/bar/baz
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@vito The TAG already includes the full path being sent to syslog. We updated to include the filename as part of the tag.
cc: @cppforlife

Actually I think this was correct before. The tag can only contain alphanumeric characters, and is supposed to just be the name of the program that produced the output.
Excerpt from RFC 3164:
   The MSG part has two fields known as the TAG field and the CONTENT
   field.  The value in the TAG field will be the name of the program or
   process that generated the message.  The CONTENT contains the details
   of the message.  This has traditionally been a freeform message that
   gives some detailed information of the event.  The TAG is a string of
   ABNF alphanumeric characters that MUST NOT exceed 32 characters.  Any
   non-alphanumeric character will terminate the TAG field and will be
   assumed to be the starting character of the CONTENT field.  Most
   commonly, the first character of the CONTENT field that signifies the
   conclusion of the TAG field has been seen to be the left square
   bracket character ("["), a colon character (":"), or a space
   character.  This is explained in more detail in Section 5.3.

@xtreme-andrew-su @pivotal-saman-alvi all right, looks like RFC wins this time. tag should remain what it is (top level directory). we'll have a separate story to add additional metadata later.

@vito updated
cc @cppforlife

thanks


Investigate scrolling issues on mobile iOS platform

Problem occurs in previous versions of iOS

Fixed in iOS version 12.2


Support for Ruby > 2.4.2

Hello,
we are deploying admin-ui as a  CF application and, therefore, we are using the ruby-buildpack to deploy and run admin-ui.
Starting from v1.7.18, supports for Ruby 2.4.2 was removed, meaning that we can no longer update the Ruby buildpack.
Do you have any plan to support a more recent Ruby version (eg 2.4.4)?
Thanks

@paolostivanin Right now I am intentionally using the same level of ruby as the cloud controller: https://github.com/cloudfoundry/cloud_controller_ng/blob/master/.ruby-version

Ok, I understand. Just for curiosity, do you plan to keep it that way or are you also planning to provide newer version?

@paolostivanin I plan on keeping the ruby level in sync with the cloud controller.

@paolostivanin Perhaps this issue on the cloud controller will result in some movement there: https://github.com/cloudfoundry/cloud_controller_ng/issues/1176


[Nox.Generator] Get Entity Translations

relates to #907
depends on #936
depends on #957

End Point to get all translations for a Entity by Id
Tasks

Create a new template for adding the translations end points (see how the controllers templates are organized), generating a partial class in Presentation.Api.Odata.<EntityPluralName>Controller.Translations.g.cs
Add the following End Point:  GET /api/<entity>/{key}/<Entity>Localized/
Add the following query Application.Queries.Get<Entity>ByIdTranslations in a new template
Returns IQueryable<<Entity>LocalizedDto>
EnableQuery for OData
Test the end point with Odata Select and filter for example
Register <Entity>LocalizedDto> in Odata Model
Check OData Metadata

https://regusit.atlassian.net/browse/MP-9639


NavigationView top whitespace

Updating doc with new API and theme resource that will be available in WinUI 2.2 stable release. Currently, the code changes are included in latest prerelease version 2.2.19073101.
The code changes give app developer more control over the whitespace baked into the top parts of NavigationView. Feature request is tracked using Ability to turn off "TopPadding" in NavigationView.

#sign-off

Hi @YuliKl - can you resubmit your changes/commits through the private repo (windows-uwp-pr)?  there are some blocking issues that are best discussed in that repo. In addition, as Microsoft PM, you should be working in the private repos. I've included some info about that below:

Microsoft authors and contributors need to make content contributions through the private repository so the content can be staged and validated by the current validation rules.
After you review staging and fix all validation issues, you can tag an author for review or add the sign-off comment to let PR reviewers know that the updates are ready to be merged.
Read about authors working in the public repo for more information.
If you need help getting set up to work in the private repo, see How to get to the private repo. You can also take a look at the GitHub account setup page for additional details on making sure you're set up.

We'll close this PR. (Do let us know if you'd like help moving the commits to the private repo - after you've gained access.) Thanks.


Playlists sample can not be delpoyed to phone emulator

Can I use playlist in a UWP project ?
I noticed that it set  TargetDeviceFamily Name="Windows.Desktop" , do I have to set this to only run on desktop when I use the desktop extension?
Can I check this api contract at runtime?

Sorry, for Win10 this is a desktop only API. There is an example here for how to check API contracts at runtime:
https://blogs.windows.com/buildingapps/2015/09/15/dynamically-detecting-features-with-api-contracts-10-by-10/
However, if you have your own playlist serialization mechanism you can still use MediaPlaybackList at runtime to play it.


update Falcon IDs

Updates Falcon (O)IDs after https://github.com/open-quantum-safe/liboqs/pull/1395 landed.

@bhess Are the qsc_encodings OID-dependent? Or asked another way: Does this PR effectively disable the QSC encoding library? In that case, oqsprovider would fail to operate as desired at the IETF hackathon. I hope @xvzcf could cope regardless (e.g., by reverting this PR).

Yes the qsc_encodings lookup is OID-dependent, the tests were (silently) failing because of the changed OID (fallback to no encoding). I updated the encoder library with the new OID, it should work again after #131.


Postal feature precision is too high

Describe the bug
A/POST -- an administrative feature for postal code boundary is precision 50000 meters.  It should be 5000.
To Reproduce
run tests for postal coding.
Expected behavior
postal code should have a lower precision error than +/- 50KM.

addressed in v3.5.9


Fatal error: Uncaught Exception: Failed to init conversation

Hi, how to fix the error? maximerenou/bing-ai/src/Chat/Conversation.php on line 87
Fatal error: Uncaught Exception: Failed to init conversation

Hello,
Could you please give a code sample? I cannot reproduce this error, and examples/chat.php is still working fine.
You may want to check that your $cookie is valid, using checkCookie() method.
Hope it helps

Thanks for the answer, it turns out you need an account with another country.

Thanks for the feedback!


fix: fix builds on darwin for PHP <= 7.3

Context: https://github.com/fossar/nix-phps/pull/201#discussion_r1087243962
Related PR: https://github.com/fossar/nix-phps/pull/201

It's failing... I don't get it.


(opera) Doesn't do silent uninstall

Expected Behavior
Expecting the package to silently uninstall opera, currently we're relying on the auto uninstaller which is unable to detect the correct switches.
Current Behavior
The uninstall dialog is popping up.
Possible Solution
Figure out which switches needs to be used for uninstallation.
Steps to Reproduce (for bugs)

Install the opera package
Try uninstalling it

Your Environment

Package Version used: 53.0.2907.88
Operating System and version: Windows Server 2008 R2 Standard
Chocolatey version: 0.10.11
Install/uninstall gist: Not needed

I'll try to address it today.

WOuld be great if you could, I haven't yet looked into what switches is available for the uninstaller.


Issue 1266

fix golint errors
Fixes #1266
main.go:578:9: if block ends with a return statement, so drop this else
and outdent its block
main.go:586:10: if block ends with a return statement, so drop this
else and outdent its block
main.go:630:10: if block ends with a return statement, so drop this
else and outdent its block
main.go:736:10: should omit 2nd value from range; this loop is
equivalent to for k := range  …
Signed-off-by: Christopher Ferris<EMAIL_ADDRESS>

closing in deference to #1281


Add a widget to show logging messages in map

This Widget displays the given last logs to the Map.

@pauldendulk Not sure, if this is the way you want to have it. Perhaps you could look into it and suggest, what you want to have changed.

So the question is shouldn't be this widget be added by default to all samples so that Logs are always displayed? I don't have a strong opinion on that only asking for your opinion on that.

I think it should be part of all samples and we should remove the existing ones. I don't want to add it explicitly to all samples. One option is to add this Widget to the Map by default. It would be nice to keep the default Map super clean though, and add no widgets at all. So, I am not sure about that. Another option is to introduce a MapBuilder which we would then use in all of our samples. So, currently all our samples have a var map = new Map(); line. This should be replaced with var map = MapBuilder.Create();. The MapBuilder could have several overloads for different scenarios. For instance  var map = MapBuilder.CreateDiagnostics(); to turn on everything that helps for debugging.

Change the most what you expect. Some parts are not possible:

LimitedQueue isn't possible, because it is in namespace of WPF.
Cleared in the sample by clicking on the widget.
Not possible to make the widget "full screen", because we don't know the correct size. Isn't stored in Map, but in MapControl.
Didn't add it automatically to the Map. Should be discussed.
Printing is difficult. If the widget is at the top, it would be nice to have the newest at the top, older going down. If you have the widget at the bottom, it would be the best to have the newest at the bottom, older going down. But I would say, this is too much for such a small widget. Or should I add a flag for orientation?

LimitedQueue isn't possible, because it is in namespace of WPF.

Fine to use ConcurrentQueue. We could move LimitedQueue to Mapsui core, but perhaps we there are better options nowadays.

Cleared in the sample by clicking on the widget.

For the sample that is a good improvement.

Not possible to make the widget "full screen", because we don't know the correct size. Isn't stored in Map, but in MapControl.

In other widgets the Draw method which takes the Viewport is used for that. It is a bit of a hack, but since this is already used in other widgets it will do for now. Could also changed after this PR.

Didn't add it automatically to the Map. Should be discussed.

I think it is very important to have this on by default in debug but we need to think about the best way to do this. I was thinking about a MapBuilder https://github.com/Mapsui/Mapsui/issues/2328. Also, I think enabled should have three states: On, Off, DebugOnly (which should be default).

Printing is difficult. If the widget is at the top, it would be nice to have the newest at the top, older going down. If you have the widget at the bottom, it would be the best to have the newest at the bottom, older going down. But I would say, this is too much for such a small widget. Or should I add a flag for orientation?

A nice to have, not needed.
I think we could merge this. If you think so too then remove the WIP.

@charlenni You did something that is new to me. In GitLab there is a Draft checkbox, but in GitHub I had to use the WIP plugin. But now this PR is in Draft mode and the WIP is removed. So, apparently GitHub now added this feature. How did you put it in Draft?

oh, it is this:

oh, it is here:

After you've put it in Draft you need to click the 'ready for review' button.

Oh, I only removed the WIP in the headline. Sorry. Try to do it next time correct.

Oh, I only removed the WIP in the headline. Sorry. Try to do it next time correct.

Just trying to figure out how this works. Perhaps we could work with the 'convert to draft' button instead of the WIP tag and plugin.


Research on project, the graph

https://thegraph.com

GraphQL and https://hasura.io/

This is the research topic for subscan, for Darwinia and Evolution.Land, they are already adpting the graph or similar subquery. https://www.subquery.network/


Implement Hypervisor field on instances

Problem: Instances with QEMU hypervisor cannot be created.
Solution: Implement hypervisor field into create_instance method.

Nice, can we add a unit test to prevent regressions ?

Tests are failing due to isort, can you fix that ?

And now black is failing ^^'

And now black is failing ^^'

I already passed black on whole src folder, but in this project also we check the tests and the examples folder. Already fixed that issues.

Tests fail due to:
>       assert instance_message.content.hypervisor == HypervisorType.firecracker
E       AttributeError: 'InstanceContent' object has no attribute 'hypervisor'


1.62.0 version does not show plot in the center of the plot viewer canvas

The plot viewer does not show the plot in the center of the canvas. I need to zoom out a lot to bring the plot back in the center.

Duplicate of https://github.com/microsoft/vscode-jupyter/issues/7523


"Architecting and Customizing Database Solutions for Enhancing Core Functionalities and Backend Performance Evaluation"

The critical role of properly architected data infrastructure and the selection of specific data technologies and models cannot be overstated in the development and performance optimization of Large Language Models (LLMs) and GPT-based projects. These foundational elements are pivotal for enhancing core functionalities, achieving unprecedented process acceleration—potentially up to 1000X—and managing vast contextual volumes. Such infrastructure underpins the strategic long-term objectives of GPT-based agents, enabling them to navigate and manipulate extensive data landscapes efficiently.
Research and developments in the integration of database systems with LLMs underscore this significance. For instance, "DB-GPT: Empowering Database Interactions with Private Large Language Models" discusses optimizing database interactions through adaptive contextual learning techniques, which significantly enhance LLM performance in contextual information management. This research highlights the necessity of a robust data architecture for efficient knowledge construction and retrieval [❞].
Similarly, the exploration of LangChain's integration with GPT and database technologies reveals the transformative potential of facilitating natural language interactions with databases. By translating user requests into SQL queries, LangChain demonstrates the power of merging LLMs with database technologies, making database interactions more accessible and efficient for users without SQL expertise. This advancement is a testament to the flexibility and efficiency achievable through the strategic customization of data models and technologies [❞].
These examples illustrate the indispensable need for selecting and customizing specific data technologies and models to support the primary functionalities and performance optimization in projects involving LLMs and GPT-based agents. The correct choice and customization of these technologies not only facilitate accelerated processes but also pave the way for expanded capabilities and more ambitious long-term goals for the project.
For further insights and details, you can refer to the full articles:

DB-GPT: Empowering Database Interactions with Private Large Language Models
Explore LangChain, GPT, and Database Integration

Also this IS just my first brainstorming for DB architechture for OpenDevin.
@neubig what do you think, could i contribute in this field?

@mslmr369 thanks for discussion! If you have a specific proposal on a technology we should adopt, feel free to open a new issue. Otherwise let's move this discussion to Slack/Discord


Added method which can request a coffee server for JSON response

Changes at vizceral end for it to query the server for new data

@wescarr :  Brought all the changes here as discussed this morning.

@ranjanmanish Are you then going to delete narrows-tracing and rename this repo?

Yup.


Fixed #9475 -- allow add(), create(), etc for m2m with through

Very promising!
Do we have a test where the through table requires values and through_defaults is empty?

Ok I added some docs and added a test for a case where the through table requires a value.


Dark Mode Functionality added

Hi ,
I've added the dark mode functionality in the app. It was quite uneasy task to add the same because dark mode feature on the UI for overall app was not an good option. So I've created the same for main data-weather-component. Please check the same!
Attaching the screenshot for your reference!:-
Light Mode:-

Dark Mode:-

Please check the same!

I think you haven't pulled the last changes to your local repository and you are working on some older version which is different from master branch which is creating conflicts for reference its {min_temp} which was fixed is the last version but still in your code its {mintemp}. so would request you to make changes accordingly so that code doesn't breaks

you can check above netlify deploy preview its not showing minimum temperature value where as in master it is fixed

Hi Please find the updated one with resolved bug.

great work, merging the pr


Add comments to the Left Out article

I thought the Try Ruby website was a good concise intro to Ruby's syntax. But I thought the additional article "What I’ve left out" was missing comments! I think this absolutely needs to be included.

Excellent suggestion, thanks. Maybe it would be useful to mention that a # comment can be after a statement. For example:
x = 2 # Lets start with two

And maybe add that =begin / =end can't have any whitespace to the left? Personally I never use this style of commenting.

Good idea.
Right. And neither do I. I usually work in an editor and use the feature of commenting multiple lines by using a lot of single line comments.
I still think it's good to mention the syntax for multiple line comments as it's a common feature of most (all?) languages.

Perfect, thanks.


Build multi-arch docker images for kuberhealthy(amd64/arm64)

Addresses: https://github.com/kuberhealthy/kuberhealthy/issues/166

Hey @integrii, I've added support to build kuberhealthy's docker images to support arm64 architecture by publishing multi-arch images using docker buildx Would really appreciate if you could given an initial review to get a sense if I'm going in the right direction. Thank you!


Support for Adafruit-PWM-Servo-Driver-Library

I'm trying to add support for the Adafruit-PWM-Servo-Driver-Library (https://learn.adafruit.com/16-channel-pwm-servo-driver/overview)
Unfortunately I'm not as successful as I hoped.
I have experience in web programming and java but not so much in python or c.
I suspect the problem is, that the Adafruit-PWM-Servo-Driver-Library is statically initialized, and not with the normal "new", but it could very well be something totally different...
Any help is much appreciated :-)
Error Message

PWMServoClass.cpp: In member function 'virtual void nanpy::PWMServoClass::elaborate(nanpy::MethodDescriptor*)':
PWMServoClass.cpp:21:21: error: no matching function for call to 'nanpy::SlimArray<PWMServo*>::insert(Adafruit_PWMServoDriver&)'
v.insert(pwm);
^
PWMServoClass.cpp:21:21: note: candidates are:
In file included from BaseClass.h:5:0,
from PWMServoClass.h:4,
from PWMServoClass.cpp:7:
SlimArray.h:29:18: note: void nanpy::SlimArray::insert(T) [with T = PWMServo*]
void insert(T el) {
^
SlimArray.h:29:18: note:   no known conversion for argument 1 from 'Adafruit_PWMServoDriver' to 'PWMServo*'
SlimArray.h:54:18: note: void nanpy::SlimArray::insert(int, T) [with T = PWMServo*]
void insert(int pos, T el) {
^
SlimArray.h:54:18: note:   candidate expects 2 arguments, 1 provided

PWMServoClass.h
#ifndef PWMSERVO_CLASS
#define PWMSERVO_CLASS

#include "BaseClass.h"
#include "MethodDescriptor.h"

class PWMServo;

namespace nanpy {
    class PWMServoClass: public ObjectsManager<PWMServo> {
        public:
            void elaborate( nanpy::MethodDescriptor* m );
            const char* get_firmware_id();
    };
}

#endif

PWMServoClass.cpp
#include "cfg.h"

#if USE_PWMServo

#include <Arduino.h>
#include <Adafruit_PWMServoDriver.h>
#include "PWMServoClass.h"
#include <stdlib.h>

const char* nanpy::PWMServoClass::get_firmware_id()
{
    return "PWMServoDriver";
}

void nanpy::PWMServoClass::elaborate( MethodDescriptor* m ) {
    ObjectsManager<PWMServo>::elaborate(m);

    if (strcmp(m->getName(),"new") == 0) {
        Adafruit_PWMServoDriver pwm = Adafruit_PWMServoDriver();
        pwm1.begin();
        v.insert(pwm);
        m->returns(v.getLastIndex());
    }

    if (strcmp(m->getName(), "setPWMFreq") == 0) {
        //m->returns(v[m->getObjectId()].setPWMFreq(m->getFloat(0)));
    }

    if (strcmp(m->getName(), "setPWM") == 0) {
        //m->returns(v[m->getObjectId()].portRead(m->getInt(0), m->getInt(1), m->getInt(2)));
    }
};

#endif

Hi @balzreber did you set to 1 USE_PWMServo in cfg.h? Did you include Adafruit library in the default libraries? Which version of firmware are you using? I suggest to download the latest firmware from pypi

Hi @astagi thank's for your quick answer.
Jep, in cfg.h file I have
#define USE_PWMServo                                1

further, here are all related code lines in Nanpy file (witch I don't included in the previous post).
#if USE_PWMServo
#include <Adafruit_PWMServoDriver.h>
#endif

#include "PWMServoClass.h"

REGISTER_CLASS_CONDITIONAL(PWMServoClass, USE_PWMServo);

I'm not sure witch firmware I was using.
I downloaded the latest Version (9.4) from pypi as you suggested, but still, I get the same Error message...
Do you think the rest of the code, as you see it in my last post, is correct???
Thanks a lot, for your help :-)

I tried with another Adafruit Product. The TLC5947 LED Controller (https://learn.adafruit.com/tlc5947-tlc59711-pwm-led-driver-breakout/overview).
But I get exactly the same error message.
I'm not sure if there is a very basic C Coding error somewhere in my files.
You can see the whole code of the PWMServo example in my previous posts, but If it would help to have the whole package for testing, I gladly send it on request.

yes please @balzreber send me your files I'll try to compile them directly and see what happens, you'll find my email on my profile (https://github.com/astagi)

@astagi thank you very much. I have sent you the whole shabang ;-)

hey @balzreber I received your mail, thanks! I'll try to make it compile during the weekend!

hey @balzreber here's the link of my Arduino folder: https://www.dropbox.com/s/vasuvh5obgqcxmx/Arduino_TLC5947.zip?dl=0
It now works! You made some mistakes using Nanpy (I'm sorry for that, it's not well documented :P), I'll list them below:

You forgot to use pointers, anytime you call "new" please create a new instance and pass it to the SlimArray
if(strcmp(m->getName(), "new") == 0) {
Adafruit_TLC5947* tlc = new Adafruit_TLC5947(1, 6, 7, 5);
tlc->begin();
v.insert(tlc);
m->returns(v.getLastIndex());
}

The ObjectManager needs to be init with the same type of the object used in the library, in this case Adafruit_TLC5947
class TLC5947Class: public ObjectsManager<Adafruit_TLC5947>

You need to import Adafruit_TLC5947 in Nanpy.ino:
#if USE_TLC5947
#include <Adafruit_TLC5947.h>
#endif

Hope it helps! If you want you may open a PR when it's ready and close this bug if this fixes your problem :)

@astagi Brilliant!!! It works!!!
As I sed, id could very well be some basic stuf, I don't understand :-)
I made PR for both Nanpy and Nanpy-Firmware. Although I build the Nanpy-Firmware on the latest release on pypi as suggested. So I'm not sure if the Firmware works with the Git Release.
Thank you very much astagi for your kind help!


FR: (beta) allow setting of the Use common prefix completion of suggestion hotkey

This might already on the map but I love this feature and would love to use it more, but tab is currently the key I use to cycle between options, and enter to select.
I would love to cycle with a left hand key and select with a right hand, which is not currently possible (without using cmd keys, I don't have CMD on the right side of my keyboard)

Hi, @brimwats :)
Your issue looks to be duplicated with #156 (comment). If so, let me close this issue.
Best regards.

agreed! closing


Unity2018 support

Please downgrade to 2018 coz it's LTS version.

I have upgraded the version to unity 2019.2.0f1 half month ago, this project is mainly used to test the latest features, so I will upgrade often.

at least offer a 2018 project coz 2019 is troublesome.

I remember that only a few files were changed when upgrading.
So if you need, you can try to downgrade.

tried few times, dependency contains a lotta errors.
might be the unmatched version during downgrading.
anyway, thx for the response.

you are welcome, thanks for your attention.


question: maintaining the starter kit in a production project

I'm just getting started with the full react/redux stack and the starter kit seems to have considered everything I can think of for building and maintaining a production application.
Here are my only concerns:

I hate copy and paste.
There is a not so insignificant amount of code/thoughts/fixes in the current kit that I don't want to lose
I like to update automatically if possible/stay as up-to-date as possible

I understand much of the changes are isolated, but nonetheless there are the important files that I'll want to keep up to date.
I have considered two options, I'd like to hear from others:
1, I ran across https://github.com/andrewmwhite/git-subtree.  Has anyone used something like this to maintain the starter-kit within their project?
2. write a script to clone and copy key files into my project
Any thoughts/strategies/ideas would be appreciated.

Hey,
Until I can type out a more thoughtful reply, these two issues may be of interest to you:

https://github.com/davezuko/react-redux-starter-kit/issues/692
https://github.com/davezuko/react-redux-starter-kit/issues/336

I think there might be a couple of other issues that I can't find right now, but at least the first one touches on this slightly. I do have plans to make this easier, but much of it revolves around currently-private work that is yet-to-be-open-sourced.

Another possibility is to fork the project, remove the example code and add your code. Every once in a while, merge this project into your own.
I'm doing this for another similar starter kit. I've done one merge, three months after the initial fork. The merge was tricky, but manageable. My changes were mostly additions of application functionality, not changes to the framework.
I bet your experience would be similar for a project based on this one.

Closing as as-resolved-as-can-be.


test

This is just for testing the Issue

It is working, thank you. You should receive now a notification. Please, remember to use markdown for styling the code:
import numpy as np


 tags make keyboard keys more identifiable.

Didn't know you can do that. Thank you!

No problem. :+1:


[Bug] removeItem is actually removing all items

Describe the bug
I just try to remove one item for a given key but after running removeItem method all other item keys are removed. This is not happening immediately, only after the app is restarted.
To Reproduce
import {MMKVLoader} from 'react-native-mmkv-storage';

const settings = new MMKVLoader().withInstanceID('settings').initialize();

settings.setString('oneKey', 'stringValue');

settings.setInt('anotherKey', 1);

settings.removeItem('anotherKey');

To Reproduce
Steps to reproduce the behavior:

run settings.removeItem('anotherKey')
remove the app from background / kill the app
open the app and check other items, they are just gone..

Expected behavior
Only one item should be removed
Platform Information:

OS: Android Emulator
React Native Version 0.68
Library Version 0.7.3

Additional context
If I don't run removeItem everything is working as expected.

@mydesweb Let me check

@ammarahm-ed I don't use any debugger, just running my code in DEV mode on Android device & Emulator, I manage to reproduce this issue each time..

@mydesweb I have tested on android emulator and a real device. I am doing same as what you told me and I cannot reproduce this.
On clicking a button in app:
  console.log(storage.getString('oneKey'), storage.getInt('anotherKey')); // Check values.
        storage.setString('oneKey', 'stringValue');

        storage.setInt('anotherKey', 1);

        storage.removeItem('anotherKey');

oneKey is always there on restart/relaunch when i click the button.

@mydesweb So you are saying that after removing a value from storage and restarting the app, all useMMKVStorage hooks return default/null values?

@mydesweb An example would be helpful.

@mydesweb I think I found the bug. Will fix this asap

All fixed in v0.7.4


Queries made with more than 3 cities

I noticed some queries were made with more than 3 cities, thus websites discarding those cities.
Conf : http://paste.awesom.eu/EU2A
Example : http://htx.fr/images/2017-24-10_20-03-11.png

Ok, so this is a bug from logicimmo module for Weboob, which does not support well the case where multiple cities have the same postal code.
For instance, for postal code 92140, it returns
[u'TOUTES-COMMUNES-92140', u'CLAMART-92140', u'LE-PETIT-CLAMART-92140']

which counts for three cities. It should only be using the first TOUTES-COMMUNES-92140 item though.

Actually, I think it would be better to fix it in Flatisfy directly. I need a bit of reflexion on this though :/

Reopening and keeping this issue open as there might still be a bug with logicimmo. I could not test it yet.
Problem is that the latest fix will add all cities, which will include duplicates (as TOUTES-COMMUNES is a duplicate with the other two entries). I don't think this should be an issue (at most, there will be some extra deduplication job for Flatisfy), but I should check it.

Completely fixed in latest commit.


Make colors scheme follow actual indigo

Tweaks color scheme to indigo-based. Favicon wasn't updated as the difference is negligible and it's changed again after actual branding

Is this ready for merge or do you want to tweak some more?

It's ready for merge


Can we trigger Depfu based on minor and not patch?

Depfu generates to many unesesary PRs.

If you hit merge, then you also have to rebase other other packages, due to yarn.lock.

This takes to time, since you have to rebase locally.

@tombruijn
I don't have access to Depfu. Is this something we can fix through configuration?
Also, is there a way not to have to rebase all the time?

@kouliavtsev I don't see a way to configure it to only update to minor versions. You'd miss any bug fixes in that release, so I understand why it's not possible.
If the number of updates is too much, we can instead group updates together on a specific day like so:

We can also enable auto merging for every (patch) update:

I don't see a way to configure it to only update to minor versions. You'd miss any bug fixes in that release, so I understand why it's not possible.

Ah, bummer that we can't configure this, yet I can understand the reason for this.

We can also enable auto merging for every (patch) update:

Is it possible to prevent auto-merge for the majors? For example react-medium-image-zoom has a breaking change.

Is it possible to prevent auto-merge for the majors? For example react-medium-image-zoom has a breaking change.

Yes, I can configure it like this:

If the build fails, it won't auto merge.

@tombruijn
This is great! Let's implement this one then. :)

Okay, it's configured. Let me know if requires changes. Should the dev dependencies have the same auto merge rule?

@tombruijn
Ty! 🙏
Yeah, makes sense to implement them in the same way.

Done! Shall we close this issue then @kouliavtsev ?

@tombruijn
Cool! Let's closed this one.


Add plugin overriding heavyweight operators with Wasm implementations

Note this can go in a separate repo too, please create one that I can send this too if that's better. It wasn't clear to me if we plan on having non-vendor plugins here and vendor ones separate.
The README provides more detail on what this is about.
Notice the orders of magnitude improvement using the plugin for larger payload sizes (even just 1K has a good improvement). It's mostly from wasilibs/go-re2 which performs way better, but pm and injection have some smaller impacts too. You can find more broken down analysis in the individual wasilibs repos.
BTW, we already use these in coraza-proxy-wasm where the difference is not just this speed but the ability to have functioning performance at all
wasi:
BenchmarkWAF/FTW-10         	       1<PHONE_NUMBER>3 ns/op
BenchmarkWAF/POST/1
BenchmarkWAF/POST/1-10      	     766	   1600783 ns/op
BenchmarkWAF/POST/1000
BenchmarkWAF/POST/1000-10   	     490	   2470757 ns/op
BenchmarkWAF/POST/10000
BenchmarkWAF/POST/10000-10  	     148	   8084505 ns/op
BenchmarkWAF/POST/100000
BenchmarkWAF/POST/100000-10 	      18	  65337887 ns/op

default
BenchmarkWAF/FTW-10         	       1<PHONE_NUMBER>7 ns/op
BenchmarkWAF/POST/1
BenchmarkWAF/POST/1-10      	     841	   1409447 ns/op
BenchmarkWAF/POST/1000
BenchmarkWAF/POST/1000-10   	     133	   9006786 ns/op
BenchmarkWAF/POST/10000
BenchmarkWAF/POST/10000-10  	      14	  82525086 ns/op
BenchmarkWAF/POST/100000
BenchmarkWAF/POST/100000-10 	       2	 822811958 ns/op

Codecov Report
Base: 77.56% // Head: 77.56% // No change to project coverage :thumbsup:

Coverage data is based on head (e0b41fc) compared to base (bb55a14).
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@           Coverage Diff           @@
##           v3/dev     #588   +/-   ##
=======================================
  Coverage   77.56%   77.56%
=======================================
  Files         145      145
  Lines        7404     7404
=======================================
  Hits         5743     5743
  Misses       1419     1419
  Partials      242      242

Flag
Coverage Δ

default
73.07% <ø> (ø)

examples
28.01% <ø> (ø)

ftw
54.85% <ø> (ø)

ftw-multiphase
54.95% <ø> (ø)

tinygo
71.08% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out 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.

This is great. I would just move it to something like coraza-wasilibs. Does that make sense @anuraaga @jptosso @fzipi ?

@jcchavezs That sounds fine to me will send a PR if you can set up the repo

Just created the repo https://github.com/corazawaf/coraza-wasilibs

Thanks - added them
https://github.com/corazawaf/coraza-wasilibs