etcdserver: mvcc: database space exceeded on mirroring

Feb 15 02:57:31  vm3 systemd[1]: Starting etcd - highly-available key value store...
Feb 15 02:57:33  vm3 etcd[1200]: recognized and used environment variable ETCD_DATA_DIR=/var/lib/etcd/default
Feb 15 02:57:33  vm3 etcd[1200]: recognized and used environment variable ETCD_NAME= vm3
Feb 15 02:57:33  vm3 etcd[1200]: etcd Version: 3.2.0+git
Feb 15 02:57:33  vm3 etcd[1200]: Git SHA: b9bbfda
Feb 15 02:57:33  vm3 etcd[1200]: Go Version: go1.6.2
Feb 15 02:57:33  vm3 etcd[1200]: Go OS/Arch: linux/amd64
Feb 15 02:57:33  vm3 etcd[1200]: setting maximum number of CPUs to 2, total number of available CPUs is 2
Feb 15 02:57:33  vm3 etcd[1200]: the server is already initialized as member before, starting as etcd member...
Feb 15 02:57:33  vm3 etcd[1200]: advertising using detected default host "x.x.x.x"
Feb 15 02:57:33  vm3 etcd[1200]: listening for peers on http://localhost:2380
Feb 15 02:57:33  vm3 etcd[1200]: listening for client requests on localhost:2379
Feb 15 02:57:33  vm3 etcd[1200]: name =  vm3
Feb 15 02:57:33  vm3 etcd[1200]: data dir = /var/lib/etcd/default
Feb 15 02:57:33  vm3 etcd[1200]: member dir = /var/lib/etcd/default/member
Feb 15 02:57:33  vm3 etcd[1200]: heartbeat = 100ms
Feb 15 02:57:33  vm3 etcd[1200]: election = 1000ms
Feb 15 02:57:33  vm3 etcd[1200]: snapshot count = 100000
Feb 15 02:57:33  vm3 etcd[1200]: advertise client URLs = http://localhost:2379
Feb 15 02:57:34  vm3 etcd[1200]: restarting member eb1aa43632ee8c7e in cluster 30e15548eff6b6d7 at commit index 6
Feb 15 02:57:34  vm3 etcd[1200]: eb1aa43632ee8c7e became follower at term 3
Feb 15 02:57:34  vm3 etcd[1200]: newRaft eb1aa43632ee8c7e [peers: [], term: 3, commit: 6, applied: 0, lastindex: 6, lastterm: 3]
Feb 15 02:57:34  vm3 etcd[1200]: starting server... [version: 3.2.0+git, cluster version: to_be_decided]
Feb 15 02:57:34  vm3 etcd[1200]: added member eb1aa43632ee8c7e [http://x.x.x.x:2380] to cluster 30e15548eff6b6d7
Feb 15 02:57:34  vm3 etcd[1200]: set the initial cluster version to 3.2
Feb 15 02:57:34  vm3 etcd[1200]: enabled capabilities for version 3.2
Feb 15 02:57:34  vm3 etcd[1200]: eb1aa43632ee8c7e is starting a new election at term 3
Feb 15 02:57:34  vm3 etcd[1200]: eb1aa43632ee8c7e became candidate at term 4
Feb 15 02:57:34  vm3 etcd[1200]: eb1aa43632ee8c7e received MsgVoteResp from eb1aa43632ee8c7e at term 4
Feb 15 02:57:34  vm3 etcd[1200]: eb1aa43632ee8c7e became leader at term 4
Feb 15 02:57:34  vm3 etcd[1200]: raft.node: eb1aa43632ee8c7e elected leader eb1aa43632ee8c7e at term 4
Feb 15 02:57:34  vm3 etcd[1200]: published {Name: vm3 ClientURLs:[http://localhost:2379]} to cluster 30e15548eff6b6d7
Feb 15 02:57:34  vm3 etcd[1200]: ready to serve client requests
Feb 15 02:57:34  vm3 systemd[1]: Started etcd - highly-available key value store.
Feb 15 02:57:34  vm3 etcd[1200]: serving insecure client requests on <IP_ADDRESS>:2379, this is strongly discouraged!
Feb 21 00:42:17  vm3 systemd[1]: Stopping etcd - highly-available key value store...
Feb 21 00:42:17  vm3 etcd[1200]: received terminated signal, shutting down...
Feb 21 00:42:18  vm3 etcd[1200]: skipped leadership transfer for single member cluster
Feb 21 00:42:18  vm3 systemd[1]: Stopped etcd - highly-available key value store.
Feb 21 00:50:08  vm3 systemd[1]: Starting etcd - highly-available key value store...
Feb 21 00:50:08  vm3 etcd[45612]: recognized and used environment variable ETCD_DATA_DIR=/var/lib/etcd/default
Feb 21 00:50:08  vm3 etcd[45612]: recognized and used environment variable ETCD_NAME= vm3
Feb 21 00:50:08  vm3 etcd[45612]: etcd Version: 2.2.5
Feb 21 00:50:08  vm3 etcd[45612]: Git SHA: Not provided (use ./build instead of go build)
Feb 21 00:50:08  vm3 etcd[45612]: Go Version: go1.6.2
Before mirroring there was no issue with space, once mirroring happened it took around ( 3.5 GB) only for around 10 keys.

Can you please provide more details? A step by step reproduce would be good.

@AnjaniKr mvcc: database space exceeded on mirroring is from the store exceeding the space quota-- since etcd is MVCC it keeps a full key history; keys need to occasionally be compacted away or the database will grow indefinitely. Please see https://github.com/coreos/etcd/blob/master/Documentation/op-guide/maintenance.md#history-compaction and https://github.com/coreos/etcd/blob/master/Documentation/op-guide/maintenance.md#space-quota

1 > installed etcd using apt-get
2 > second, upgrade to etcd 3.x from source
# apt-get -y install git golang
3 > replaced binaries
4 > checked the version :
ETCDCTL_API=3 etcdctl version
etcdctl version: 3.2.0+git
API version: 3.2
5 > Added around 10 keys without mirroring (space allocation 62M)
6 > started mirroring with same set of keys ( space rose to 3.5GB)
7 > started giving error database space exceeded

@AnjaniKr how is etcdctl make-mirror being invoked? I tried:
bin/etcd --name infra1 --listen-client-urls http://<IP_ADDRESS>:2379 --advertise-client-urls http://<IP_ADDRESS>:2379 --listen-peer-urls http://<IP_ADDRESS>:12380 --initial-advertise-peer-urls http://<IP_ADDRESS>:12380 --initial-cluster-token etcd-cluster-1 --initial-cluster 'infra1=http://<IP_ADDRESS>:12380' --initial-cluster-state new &
bin/etcd --name infra2 --listen-client-urls http://<IP_ADDRESS>:22379 --advertise-client-urls http://<IP_ADDRESS>:22379 --listen-peer-urls http://<IP_ADDRESS>:22380 --initial-advertise-peer-urls http://<IP_ADDRESS>:22380 --initial-cluster-token etcd-cluster-2 --initial-cluster 'infra2=http://<IP_ADDRESS>:22380' --initial-cluster-state new &

for a in `seq 1 10`; do ETCDCTL_API=3 bin/etcdctl put $a $a; done

# mirror from infra1 to infra2
ETCDCTL_API=3 ./bin/etcdctl make-mirror --no-dest-prefix http://localhost:22379

This mirrors 10 keys, periodically printing 10 to indicate that it's only propagated 10 updates; I can't seem to reproduce the failure.

this is what we r doing for mirroring - $from=$HOST1  prefix=/realm/infra1 remote=$HOST2  &&  ssh root@$from "no_proxy=<IP_ADDRESS>,$from,$remote ETCDCTL_API=3 nohup etcdctl make-mirror --prefix $prefix --dest-prefix=$prefix $remote:2379 > /dev/null 2>&1 &"
Also we are doing bidirectional mirroring. Once mirroring starts it starts to print key from 0 , 10 ,...some big number(988674)

@AnjaniKr can you reproduce this with hardcoded values, without all the variable substitution?
I strongly suspect make-mirror is being configured to mirror a single cluster back to itself. If the source is the same as the destination then make-mirror will storing keys to the source which will generate new events from the source which will then write back to the source which will generate new events which will be written to the source and so on until storage is exhausted. This can happen if HOST2 is never set; $remote:2379 will substitute to :2379 which will be accepted by a listener on <IP_ADDRESS>:2379.

@heyitsanthony please find detailed steps :
I'm using two host for mirroring ( bidirectional mirroring)
on first host - etcdctl make-mirror --prefix /realm --dest-prefix=/realm 2nd hst:2379
on second host - etcdctl make-mirror --prefix /realm --dest-prefix=/realm 1st host ip:2379
these r the netstat -lntp status on each host :
tcp        0      0 x.x.x.x:7001        <IP_ADDRESS>:*               LISTEN      26560/etcd
tcp        0      0 <IP_ADDRESS>:4001          <IP_ADDRESS>:*               LISTEN      26560/etcd
tcp        0      0 x.x.x.x:4001        <IP_ADDRESS>:*               LISTEN      26560/etcd
tcp        0      0 <IP_ADDRESS>:2379          <IP_ADDRESS>:*               LISTEN      26560/etcd
tcp        0      0 x.x.x.x:2379        <IP_ADDRESS>:*               LISTEN      26560/etcd
tcp        0      0 x.x.x.x:2380        <IP_ADDRESS>:*               LISTEN      26560/etcd
One one way mirroring keys remain constant but as we start bidirectional then it starts increasing (301974) but only 10 keys are there

etcdctl make-mirror --prefix /realm --dest-prefix=/realm 2nd hst:2379
etcdctl make-mirror --prefix /realm --dest-prefix=/realm 1st host ip:2379

This won't work. A write to /realm/X on host1 willl write to /realm/X on host2. If host2 mirrors all writes on /realm back to host1, then /realm/X will be written back to host1, which will then mirrored to host2 then mirrored to host1, etc. There is a cycle; the prefixes must be distinct.

sorry but both realm are different ie /realm and /realm2
etcdctl make-mirror --prefix /realm2 --dest-prefix=/realm2 host1:2379
etcdctl make-mirror --prefix /realm --dest-prefix=/realm host2:2379

@heyitsanthony  within 5 minutes of time mirror count was - 752150 & 789445

@heyitsanthony thanks its working :) 👍

@AnjaniKr --prefix=/realm --dest-prefix=/realm may cause problems for a bi-direction setup since /realm prefixes /realm2; you may want to try /realm1 and /realm2 or /realm/ and /realm2/. If there's no traffic going into the cluster (aside from the mirroring) the count should be stable.

@heyitsanthony as per  ur comments if I need to set up bidirectional between host A and host B, this is how I need to do :
frm host1 :
etcdctl make-mirror --prefix /realm2 --dest-prefix=/realm2 host1:2379
(here prefix realm2 is the key that host1 is putting its data and --dest-prefix is what is being exposed to host2 to mirror right)
So, at host1 I can give both values to be same and at other host those will have different values (ex realm2)

@AnjaniKr
Running on host1, this will mirror from host1 to host2:
etcdctl make-mirror --prefix /realm1 --dest-prefix=/realm1 host2:2379

Running on host2, this will mirror from host2 to host1:
etcdctl make-mirror --prefix /realm2 --dest-prefix=/realm2 host1:2379

The main thing to avoid is a cycle where make-mirror is writing to the same destination path and cluster that it's reading from.

@AnjaniKr Can we close this issue now? I believe @heyitsanthony already helped with all your questions.

@xiang90 Yes thanks we can close the issue.


adding instructions for setting CORS headers when developing locally

Signed-off-by: Mark Cohen<EMAIL_ADDRESS>Description
This adds instructions to add settings to opensearch.yml so developers can run opensearch locally and also develop an application on the same host.
Issues Resolved
None
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

I think this is more of a developer guide item that should go into https://github.com/opensearch-project/OpenSearch-Dashboards/blob/main/DEVELOPER_GUIDE.md

I think this is a good point and maybe it needs to also be linked to in the DEVELOPER_GUIDE.md for Dashboards. However, it is not specific to Dashboards because the CORS headers need to be configured on the OpenSearch Server to allow specific (or all) clients to connect to it. That's why I think it belongs either in the main documentation or maybe in https://github.com/opensearch-project/OpenSearch/blob/main/DEVELOPER_GUIDE.md.

@macohen Is the audience the developer on OpenSearch core or OpenSearch Dashboards? Think of them as two separate products. This should go into the developer guide for the right audience.

@macohen Is the audience the developer on OpenSearch core or OpenSearch Dashboards? Think of them as two separate products. This should go into the developer guide for the right audience.

The audience is OpenSearch core because I came across the issue when running core locally and building an independent web app not in OpenSearch Dashboards.

@Naarcha-AWS suggested that this might be best in https://opensearch.org/docs/latest/opensearch/configuration/ because it's not docker specific and does apply to OpenSearch Server. I'll undo the original change and apply the new one this week.

Can you please update this PR after rebasing main on to your dev branch. I see 48 files changed with many unrelated commits which doesn't sound right. If your branch continues to say # commits ahead and the number is more than the number of commits you are PRing, something is very wrong.

@macohen: Let's close this PR and open up a new branch, that we avoid any potential conflicts between this branch and main. Feel free to email me at<EMAIL_ADDRESS>if you need any help.

Closing as per @Naarcha-AWS. Will create a fresh branch, re-apply the change, and open a new PR.


FIX ability to log in without MFA verification

During recent work to have MFA always enabled (#186) an oversight was made in testing
meaning that a user could log in without verifying their registered MFA methods, if the site
was set to have MFA as optional.

Merging with Robbie's approval.


Add IsInf() check to Vector3

Hmm not sure what happened here. I would take Add IsInf() check to Vector3 commit, but there is way more than that :p

woops.  yeah.  i'll make a better pr later when I have time.


Add Chrome steps

I guess it'll be a lot easier to just submit and install it from the Chrome addon store.


[SUGGESTION] Skill Rolls and Tracking

Hi,
Unless I missed something, for Skill rolls to be counted, they must be made from the Tracking tab and not from the Skills block of the character sheet.
Is there a way for a Skill roll, regardless of where it is made, to be counted in the tracking?

Your understanding is correct, yes.
The short answer to your question is yes, but it's not something I'm super willing to write into this module. It's not in active development stages anymore, just getting compatibility updates at this point.
The long answer to your question is that it depends on what you want to do with it. Do you want every roll of the chosen type to advance all activities that use that kind of roll? Do you want to be selective of which rolls count towards the progress of tracked activities?
The module is centered around the latter use case; if you need someone to roll a survival check and want it to count towards progress, just have them roll thee activity instead of rolling from the skills section of the sheet.
If you need them to roll a Survival check and want it to count towards all activities that use Survival, or if you want every Survival check to count towards all activities that use Survival, that can almost definitely be accomplished with a world script.

Hi,
The problem is that my players use HUDs a lot like Argon or Token HUD.
If they want to benefit from Tracking, this requires them to open their file and go through the Tracking tab. It's a bit restrictive but not at all insurmountable either ;)
In any case, thank you for your response.

Ah okay, I understand. I don't use TAH or Argon, but I do know it's possible to add custom content to those modules; both of them are built specifically to allow for other modules that enable additional functionality. I think getting my hands dirty with those is far beyond the scope of a project that's not in active development right now, but if I come back to modding Foundry in the future I'll try to remember this request. Even if the inconvenience isn't insurmountable, accessibility is important and definitely something to keep in mind.
Thanks for the suggestion; I really do appreciate hearing from people!


Update README.md

I think you need to specify the url which is the zip of the project no?

No, github repository url is enough after git+ prefix


Available languages

Hi,
Is there a place where we can check the pre-trained languages available for the transformer models (XLNet, RoBERTa, and XLM) up-to-date?
Thanks

Yes, they are available in the Hugging Face docs. I can post a link later if you need.

Can be these ones?

I was trying to find the languages for roBERTa, as I supposed all the BERT ones would be but I am not sure about this assumption...

I don't think Roberta has multilingual support as of yet. At least, I haven't been able to find anything of the sort.

Ok, let's see if any of these models cover more languages in the near future...


Make cepl:quit use shutdown-function for api-1

I might be doing something backwards with the versioned api shenanigans, but this seems to work.

Thanks @nja. I'll merge this next weekend as I'd like to take a good few hours to review the lifecycle code as I'm pretty sure I have some other bugs in there too.

branched, rebased, and merged. Thankyou @nja, this was spot on :)


Form required login

How to fix it?

Just like stated there, the form you tried to submit required a valid user with a valid Gmail account. Can't do much about that.


Deploy with a non-default stage prints "Unknonwn function"

Steps to reproduce
serverless deploy

then:
serverless deploy -s prod

Result
Serverless: Packaging service...
Serverless: Excluding development dependencies...
Serverless: CredentialsError: Missing credentials in config, if using AWS_CONFIG_FILE, set AWS_SDK_LOAD_CONFIG=1
      Maybe you should set AWS_ACCESS_KEY_ID/AWS_SECRET_ACCESS_KEY environment variables
Serverless: Unknonwn function "test-dev-api" found
Serverless: Function created
            api: test-prod-api
Serverless: Service deployed successfully

Expected
No "Unknonwn function "camera-api-dev-api" found" error (and obviously "Unknonwn" should read "Unknown").

Could you share your serverless.yml?
I also have such message Unknown function "xxx" found when deploying to test or prod function.
It's seems serverless reads all functions from account..


Remove unmaintained ansi_term crate

ansi_term is an unmaintained crate that we can remove.

Removing it does not break anything... I think this was not being used? @ionut-arm @gowthamsk-arm

PS: you can use cargo-udeps to check for unused dependencies!

Hi @hug-dev, Thanks a lot for the comment! Yes, we are (and were) using that. The thing that caught my attention was that we had this marked as an exception for cargo audit (please see https://github.com/parallaxsecond/parsec-tool/pull/107/files#diff-42d80c82c252835c55a95ccaecb4b8583244e79721e5dd62864e11e2dc89eb65R2) instead of just removing the crate, that's why I asked them to verify.


Support for Minecraft 1.13.2

Please add support for the following game:
Game Name: Minecraft Version 1.13.2
Methods:Already has support for versions around it, and there is a Forge version for 1.13.2 now.
https://github.com/antonpup/Aurora/pull/1213 can be updated
The new forge can be found here: https://files.minecraftforge.net/maven/net/minecraftforge/forge/index_1.13.2.html

I have a working 1.13 version working. I'll try and push it to the gitlab soon.
Please keep these issues asking for the mod to that repo, please. You can find a link to it in the Minecraft overview tab in Aurora.

I just uploaded the mod here https://gitlab.com/aurora-gsi-minecraft/mc1.13.2/-/tags


Adds unit test for pac handler

Signed-off-by: Shivam Mukhade<EMAIL_ADDRESS>
Changes

Submitter Checklist

[ ] ♽  Run make test lint before submitting a PR (ie: with pre-commit, no need to waste CPU cycle on CI
[ ] 📖 If you are adding a user facing feature or make a change of the behavior, please verify that you have documented it
[ ] 🧪 100% coverage is not a target but most of the time we would rather have a unit test if you make a code change.
[ ] 🎁 If that's something that is possible to do please ensure to check if we can add a e2e test.
[ ] 🔎 If there is a flakiness in the CI tests then don't necessary ignore it, better get the flakyness fixed before merging or if that's not possible there is a good reason to bypass it. (token rate limitation may be a good reason to skip).

ci failed in just one second 😄 super fast!

/retest

operator failed to create pipeline sa ...
we are going to need a 1.6.3 for openshift pipelines
flakiness 😞

/retest


Various OmniBLEParser updates and improvements

Refactored for more uniform date and time display for all cases
New optional -v command line option to enable all print options
Enable new pod disconnect/reconnect message printing with -v command line option
Enable message address and seq #'s printing with -v command line option
Disable new unacknowledged message error printing with -q command line option
Disable date printing with timestamps with -q command line option

Status
Success. LGTM.
Testing Overview
This feature is used almost exclusively by @itsmojo as part of his code development and testing.
Apply the modifications and perform minimal testing.
How to use OmniBLEParser:
In Xcode:

Select scheme OmniBLEParser, destination My Mac
Select edit scheme
Add argument: Run, tap + for Arguments Passed on Launch

Put full pathname, use quotes if there are any spaces, for file for which parsing is desired

Test with a variety of input files
All these tests were successful.
Used both Xcode 16.1 (for most) tests.
Confirmed Xcode 15.4 also works.

Loop Report from main
Loop Report from tidepool-merge
Trio 0.2.2 log.txt and log_prev.txt files
Loop 3.4.4 Xcode Debug log (Xcode 15.4) with metadata on a separate line
Loop 3.4.4 Xcode Debug log (Xcode 16.1)
Loop tidepool-merge Xcode Debug log (Xcode 16.1)


E0464 in nightly

The rust cookbook started seeing E0464 on nightly.  Has anyone else seen this?
https://github.com/rust-lang-nursery/rust-cookbook/issues/590

Now that we've updated to 2018 edition, nightly fails all of our tests.  I'm trying to get more information, but what I can tell is this:
"rustc" "/tmp/rust-skeptic.k9jtfsC2cudt/test.rs" "--verbose" "--crate-type=bin" "--edition=2018" "-L" "/home/travis/build/rust-lang-nursery/rust-cookbook/target/debug" "-L" "/home/travis/build/rust-lang-nursery/rust-cookbook/target/debug/deps" "--target" "x86_64-unknown-linux-gnu" "-o" "/tmp/rust-skeptic.k9jtfsC2cudt/out.exe"', /home/travis/.cargo/registry/src/github.com-1ecc6299db9ec823/skeptic-0.13.4/lib.rs:855:13

used to have a ton of --extern /path/to/crate_name-hash.rlib that are now suddenly missing.

Turns out the minimized example https://github.com/AndyGauge/Skeptic-issue-111 also experiences this problem.

running 1 test
error[E0464]: multiple matching crates for percent_encoding
--> /var/folders/89/62kc3cz5615566tm8t4kzgb00000gp/T/rust-skeptic.bG1bjPaq2TG5/test.rs:2:1
|
2 | extern crate percent_encoding;
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= note: candidates:
crate percent_encoding: /Users/andy/git/Skeptic-issue-111/target/debug/deps/libpercent_encoding-ffbafd40a9d3bbe4.rlib
crate percent_encoding: /Users/andy/git/Skeptic-issue-111/target/debug/deps/libpercent_encoding-2ae3b6f2cfc25b66.rlib
error[E0463]: can't find crate for percent_encoding
--> /var/folders/89/62kc3cz5615566tm8t4kzgb00000gp/T/rust-skeptic.bG1bjPaq2TG5/test.rs:2:1
|
2 | extern crate percent_encoding;
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ can't find crate
error: aborting due to 2 previous errors
For more information about this error, try rustc --explain E0463.
test readme_line_1 ... FAILED
failures:
---- readme_line_1 stdout ----
/Users/andy/git/Skeptic-issue-111/target/debug
thread 'readme_line_1' panicked at 'Command failed:
"rustc" "/var/folders/89/62kc3cz5615566tm8t4kzgb00000gp/T/rust-skeptic.bG1bjPaq2TG5/test.rs" "--verbose" "--crate-type=bin" "--edition=2018" "-L" "/Users/andy/git/Skeptic-issue-111/target/debug" "-L" "/Users/andy/git/Skeptic-issue-111/target/debug/deps" "--target" "x86_64-apple-darwin" "-o" "/var/folders/89/62kc3cz5615566tm8t4kzgb00000gp/T/rust-skeptic.bG1bjPaq2TG5/out.exe"', /Users/andy/git/rust-skeptic/src/skeptic/lib.rs:875:13
note: run with RUST_BACKTRACE=1 environment variable to display a backtrace
failures:
readme_line_1
test result: FAILED. 0 passed; 1 failed; 0 ignored; 0 measured; 0 filtered out
error: test failed, to rerun pass '--test skeptic'

OK, so I found a big difference in cargo between the two versions that I believe is causing this.  The json file found within the .fingerprint directory has been renamed in nightly.
In stable the file looks like this:
lib-cratename.json
In nightly:
lib-cratename-hash.json


Use of @ instance attributes in interactor: good or bad?

Is it a good or bad practice to use instance-level attributes in an Interactor? (Why would you want to? Our current wish was to memoize an expensive lookup. We also use private instance methods to make the code more understandable. Combining these two techniques results in this desire.)
Currently, an Interactor instance's lifespan does not extend past its call-site. In other words, it works like we expect. If that ever were to change such that the instances were re-used, though, things might become...interesting.
I can think of no reason at all why the lifespan of Interactor instances might be changed, but if using instance attributes is regarded as an acceptable practice, doing so somewhere in an example or test case would be a good way to make sure it's known to be acceptable.

@antifun
I don't think the use of instance variables in an interactor is necessarily a good practice or a bad practice;  it's simply a part of the Ruby language that is available to the programmer if needed. As such, I don't think that it is necessary to call out instance variables explicitly in an example/test case.


IOError: [Errno -9996] Invalid input device (no default output device)

I'm getting the following error with osx 10.11 python 2.7.10
$ soundmeter --collect --seconds 10 -v
Traceback (most recent call last):
  File "/usr/local/bin/soundmeter", line 9, in <module>
    load_entry_point('soundmeter==0.1.3', 'console_scripts', 'soundmeter')()
  File "/Library/Python/2.7/site-packages/soundmeter/meter.py", line 310, in main
    m = Meter(**kwargs)
  File "/Library/Python/2.7/site-packages/soundmeter/meter.py", line 66, in __init__
    frames_per_buffer=FRAMES_PER_BUFFER)
  File "/Library/Python/2.7/site-packages/pyaudio.py", line 750, in open
    stream = Stream(self, *args, **kwargs)
  File "/Library/Python/2.7/site-packages/pyaudio.py", line 441, in __init__
    self._stream = pa.open(**arguments)
IOError: [Errno -9996] Invalid input device (no default output device)

This sounds more like a pyaudio/portaudio error related to your audio hardware/driver. Since soundmeter is very high-level, which uses these library, it's difficult to tell what the problem is, especially when it's working on my side.
My suggestions are:

Try to plug in a microphone device to your computer and see if the error still exists
Check your system settings on sound and verify if input/output device is working correctly

I searched symptoms for possible solutions:

http://stackoverflow.com/questions/37063540/i-cant-run-a-simple-code-using-pyaudio-errno-9996-invalid-output-device-n
http://stackoverflow.com/questions/4672155/pyaudio-ioerror-no-default-input-device-available?rq=1
http://stackoverflow.com/questions/33382305/pyaudio-ioerror-errno-invalid-input-device-no-default-output-device-9996

You may want to take a look at these, and possibly use the following method to see if you have a default input device:
>>> import pyaudio
>>> pa = pyaudio.PyAudio()
>>> pa.get_default_input_device_info()

I run the function 'get_device_count' but got a 0.Is it the problem of my Operating System?

I am  a linux user and I am gettnig same issue How I can resolve it

I have been struggling with the same issue for over a month now. Mine is OSError: No Default Input Device Available, and I think it's peculiar to my system as the same code runs on other systems.

import pyaudio
pa = pyaudio.PyAudio()
pa.get_default_input_device_info()

Trying to check my default input and output devices with the above lines of codes throws the same error.

我在Linux上遇到了同样的问题，并通过首先安装ALSA开发库，重新构建PortAudio，然后重新安装PyAudio来解决了该问题。构建PortAudio时，的输出./configure将告诉您它具有哪些后端，
  目标...................... x86_64-unknown-linux-gnu
  C ++绑定.......否
  调试输出.......否

  ALSA ........................是
  ASIHPI ......................否

  OSS ....................是
  杰克........................否

我不了解OS X，但是如果您在此处发布配置，其他人可以帮忙吗？

Hello, my alsa display is installed successfully. But it still reported 9996 error. Could you tell me how to solve the problem。my system is ubuntu16.04

我在Linux上遇到了同样的问题，并通过首先安装ALSA开发库，重新构建PortAudio，然后重新安装PyAudio来解决了该问题。构建PortAudio时，的输出./configure将告诉您它具有哪些后端，
  目标...................... x86_64-unknown-linux-gnu
  C ++绑定.......否
  调试输出.......否

  ALSA ........................是
  ASIHPI ......................否

  OSS ....................是
  杰克........................否

我不了解OS X，但是如果您在此处发布配置，其他人可以帮忙吗？

Hello, my alsa display is installed successfully. But it still reported 9996 error. Could you tell me how to solve the problem。my system is ubuntu16.04

I got the same issue on Linux, and resolved it by first installing the ALSA dev library, re-building PortAudio, and then re-installing PyAudio. When you build PortAudio, the output from ./configure will tell you which backends it has available,
  Target ...................... x86_64-unknown-linux-gnu
  C++ bindings ................ no
  Debug output ................ no

  ALSA ........................ yes
  ASIHPI ...................... no

  OSS ......................... yes
  JACK ........................ no

I don't know about OS X, but maybe if you post your config here someone else can help?

Hello, my alsa display is installed successfully. But it still reported 9996 error. Could you tell me how to solve the problem。my system is ubuntu16.04

I got the same issue on Linux, and resolved it by first installing the ALSA dev library, re-building PortAudio, and then re-installing PyAudio. When you build PortAudio, the output from ./configure will tell you which backends it has available,
  Target ...................... x86_64-unknown-linux-gnu
  C++ bindings ................ no
  Debug output ................ no

  ALSA ........................ yes
  ASIHPI ...................... no

  OSS ......................... yes
  JACK ........................ no

I don't know about OS X, but maybe if you post your config here someone else can help?

Hello, my alsa display is installed successfully. But it still reported 9996 error. Could you tell me how to solve the problem。my system is ubuntu16.04

On redhat, before compile install portaudio, first run yum install alsa*

我也是遇到了这个问题 使用的是Pyaudio的Python程序。OSError: [Errno -9996] Invalid input device (no default output device)


feat: annotation delete modal, bulk delete and empty state

SUMMARY

added DatePicker and RangePicker to storybook
empty state for annotation CRUD list view
bulk delete action annotations list
annotation delete modal
add data-test on submenu buttons

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

DELETE MODAL:

BULK DELETE:

EMPTY STATE

TEST PLAN

unit test for annotation delete modal and bulk delete

ADDITIONAL INFORMATION

[ ] Has associated issue:
[x] Changes UI
[ ] Requires DB Migration.
[ ] Confirm DB Migration upgrade and downgrade tested.
[ ] Introduces new feature or API
[ ] Removes existing feature or API

Codecov Report

Merging #11540 into master will decrease coverage by 12.31%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##           master   #11540       +/-   ##
===========================================
- Coverage   66.61%   54.29%   -12.32%
===========================================
  Files         875      411      -464
  Lines       42061    14235    -27826
  Branches     3912     3636      -276
===========================================
- Hits        28019     7729    -20290
+ Misses      13940     6329     -7611
- Partials      102      177       +75

Flag
Coverage Δ

cypress
54.29% <0.00%> (-0.81%)
:arrow_down:

javascript
?

python
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

superset-frontend/src/components/Menu/SubMenu.tsx
75.00% <ø> (-25.00%)
:arrow_down:

...ntend/src/views/CRUD/annotation/AnnotationList.tsx
1.36% <0.00%> (-80.78%)
:arrow_down:

...tend/src/views/CRUD/annotation/AnnotationModal.tsx
4.34% <ø> (-62.59%)
:arrow_down:

...uperset-frontend/src/dashboard/util/dnd-reorder.js
0.00% <0.00%> (-100.00%)
:arrow_down:

...rset-frontend/src/dashboard/util/getEmptyLayout.js
0.00% <0.00%> (-100.00%)
:arrow_down:

...et-frontend/src/components/Menu/LanguagePicker.tsx
0.00% <0.00%> (-100.00%)
:arrow_down:

...dashboard/components/resizable/ResizableHandle.jsx
0.00% <0.00%> (-100.00%)
:arrow_down:

.../src/dashboard/util/getFilterScopeFromNodesTree.js
0.00% <0.00%> (-93.48%)
:arrow_down:

...src/dashboard/components/gridComponents/Header.jsx
10.52% <0.00%> (-86.85%)
:arrow_down:

...rc/dashboard/components/gridComponents/Divider.jsx
13.33% <0.00%> (-86.67%)
:arrow_down:

... and 702 more

Continue to review full report at Codecov.

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


SNNGRU forward part ValueError: Input size mismatch!

Hi team,
Great work first of all. I would like to rewrite you code at local from basic part, when i trying to re-run the TSSNNGRU model, i got an issue about the  size mismatc at the forward.
When i set the input is an 3-dimensions tensor[32, 14, 10]:=>[B, L, C];
after, the  encoder output is [32, 16, 10, 14]:=>[B, T, L, C] when i set the hidden is 16, whatever for the ConvEncoder or deltaEncoder;
however, the code in the github is 3-dimension is [B, L, H];
so, is there any problem for this part, or should i reshape the [B, T, L ,C] to [B, L, T*C] before to the GRUCell?
Thank's a lot in advance!

Thanks for your attention!
First, maybe you can reshape [B, T, L ,C] to [B, L, TC], or other sjhapes like [TB, L, C].
Second, you can try TSSNNGRU2D or ISNNGRU2d.


Remove/Add option for automatic trimming of text

I was trying to add right aligned text in something similar to the following:
  Item One: Some Value
  Item Two: Second
Item Three: Another

Instead of getting the above, I get
Item One: Some Value
    Item Two: Second
 Item Three: Another

As you can see the latter is a bit harder to read. Is there a reason for automatically trimming the string? If I want to have my text be
"Some string with extra padding       "

I would expect the extra space at the end. Thoughts?

Hmm I also  just realized this may be able to be solved with https://github.com/devongovett/pdfkit/issues/29 provided that you would be able to specify alignment of columns individually.

I just tried in the browser demo, PDFKit doesn't trim anything, but aligning the text with spaces only works correctly with "monospace" fonts (Courier for example)

@alafr The problem is space on the end of the text.

Removing the .replace call here let's me do what I want.

I guess this is wanted for multiline wrapped text. But a new simpler function without any automatism would be useful in many cases like yours.

I would still like to have an option to avoid trimming. For example "\n" at the beginning or at the end of a string should not be trimmed!


"Frumosu" is listed in the drop-down menu

Description:
"Frumosu" is listed in the drop-down menu alongside its corresponding county according to the law
Precondition:
The website is up an running.

Step 1
Type "Frumosu" in the search bar.

Expected results
The location is listed in the drop-down menu as "Comuna Frumosu, SUCEAVA".


Feature Request: Add flexible/smart search option

Currently the data-live-search searches the text string and returns results that match that string exactly.
Request allowing the search results to treat a space as a wildcard so multiple strings can be entered and the results would be the values that have each string present.
For example, consider the value "First_Second_Third"
With the current search, if "First Third" was entered there would be no results.
Request finding this value as a result of searching "First Third" or "Third First"

I was able to modify bootstrap-select.js as shown below to accomplish what I needed.  This starts on line 200.
        if (method === 'contains') {           //searchSuccess = string.indexOf(searchString) >= 0;           var searchWords = searchString.split(" ")           searchSuccess = searchWords.every(word => string.includes(word))


Make forwarded scheme handle comma separated schemes

When using HAProxy to add X-Forwarded-Proto via reqadd X-Forwarded-Proto ... and the header already exists, the value will be comma separated. This causes an assertion error to be thrown.
The real issue is with the HAProxy configuration as it should detect the presence of the header and behave accordingly. I'm not sure if this should be a concern of ring-ssl. Thoughts?

The draft spec for x-forwarded-proto suggests the header is supposed to inform a client which protocol is being used. The idea being you can modify your response to provide the correct protocol in links etc.
I'm not sure http, https makes sense in this context, and wonder how an application would behave with a confusing value like this.
Can HAProxy be configured to pass http or https based on which is actually being used?

A quick Google seems to suggest HAProxy can be configured to do the 'right' thing.

https://www.digitalocean.com/community/tutorials/how-to-implement-ssl-termination-with-haproxy-on-ubuntu-14-04
http://serverfault.com/questions/302282/how-can-i-use-haproxy-with-ssl-and-get-x-forwarded-for-headers-and-tell-php-that

This issue is 100% due to a poorly configured HAProxy.

I'm not sure http, https makes sense in this context, and wonder how an application would behave with a confusing value like this.

Perhaps the test is confusing - the real header value that broke the application was "https,https".
Originally HAProxy was configured correctly and the application worked fine. Someone made a change    (duplicating: reqadd X-Forwarded-Proto:\ https) that caused the app to throw an assertion error.
The fix is to configure HAProxy correctly but I fear that in the future, a similar change could bring down the application again. I decided to open a PR and get some input.
At any rate, I'm going to have to duplicate this middleware to prevent the errors in the future. I'm fine with closing this PR since you can't be expected to accommodate all possible incorrect header values.

https,https isnt a valid value according to the spec either…
A few ideas spring to mind:

Change the code to accept any valid scheme rather than just HTTP and HTTPS
Return a client error rather than server error when we can't parse the header
Expect clients to be well behaved (which is perhaps naive)

What do you think @weavejester?

In HTTP, all header values can be concatenated together with a comma, so my guess is that it is technically correct, but it doesn't make any sense. For security purposes the proxy should replace the whole header, since otherwise a HTTP client could make the server believe that it's actually being served over HTTPS.
Maybe we should have a more detailed error message, but I think this should throw an exception, because of the security implications.

I think the assertion did its job in alerting me to the issue. Thanks for the feedback.

Could reopening this issue be considered?
The Forwarded HTTP Extension spec considers a chain of proxy servers and specifies an order for the chain. Further, AWS ELB behavior appends to the chain of x-forwarded-* headers, and given the wide usage of that service, it might be deemed close to standard.
For context:
We encountered this while debugging issues with using Readme.io's ability to test API calls from the browser. Their system proxies the request through a chain, each of which appends the previous client to the x-forwarded-* headers, before arriving at our ELB, which appends the final proxy from Readme.io. By the time it reaches us, the header value is "https,https,http,http,http,http,http,http".
Bottom line: there are situations where you can't control all hops, and ring-ssl could be made flexible enough to handle those in a non-breaking manner.
A final note, with regard to:

For security purposes the proxy should replace the whole header, since otherwise a HTTP client could make the server believe that it's actually being served over HTTPS.

The opposite is also true: by replacing the whole header, a proxy could make a server believe the request is HTTPS end-to-end, when it may not be. Ultimately, any proxy outside your network (and maybe those inside?) can't be trusted.

Further, AWS ELB behavior appends to the chain of x-forwarded-* headers, and given the wide usage of that service, it might be deemed close to standard.

Do you have a reference for that? It seems reasonable to support the behavior if that's how AWS behaves, but I'd like a "paper trail" justifying the decision.

The opposite is also true: by replacing the whole header, a proxy could make a server believe the request is HTTPS end-to-end, when it may not be.

Sure, but if you have a proxy inside your network then that should take priority, and if you don't, then you shouldn't be using the middleware. By using the middleware you're saying: "I know a trustworthy proxy will set this value."

Regarding the paper trail, the AWS documentation doesn't specify the behavior of ELBs if x-forwarded-* headers are present on an incoming request, so I have reached out to support for clarification.

Finally got sufficient context on this; it depends on the ELB listener mode.
If configured as HTTP/HTTPS, the behavior is, from AWS:

"X-Forwarded-For” – ELB will append to existing value if this header existed in request header, otherwise it will be added
"X-Forwarded-Proto” – ELB will add this header. If this header existed, ALB will replace it.
"X-Forwarded-Port” – ELB will add this header. If this header existed, ALB will replace it.

However, if configured as TCP/SSL (e.g. w/ proxy protocol to support WebSockets), the behavior is to preserve the client headers and append-only. Quoting AWS:

When you use TCP (layer 4) for both front-end and back-end connections, the load balancer forwards the request to the back-end instances without modifying the headers. In that case, you should see the headers as they are sent by the client.

Unfortunately not, that was from the support response.


Improve comments

remove gendered language
fix some typos

Thank you for submitting a contribution to Apache Geode.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced in the commit message?

[ ] Has your PR been rebased against the latest commit within the target branch (typically develop)?

[ ] Is your initial contribution a single, squashed commit?

[ ] Does gradlew build run cleanly?

[ ] Have you written or updated unit tests to verify your changes?

[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and
submit an update to your PR as soon as possible. If you need help, please send an
email to<EMAIL_ADDRESS>

Requesting reviews from @karensmolermiller @joeymcallister
(since the changes are almost all comments)

StressNewTest failures were:
DistributedLockServiceDUnitTest > testFairness -- has a fix in https://github.com/apache/geode/pull/2393
StatisticsDistributedTest > testPubAndSubCustomStats -- has a ticket opened and added to the swat track of work https://issues.apache.org/jira/browse/GEODE-5682

@balesh2 Your commit messages should have a JIRA ID in them. Is there an open JIRA for this work?


Extract ETL content in upload-etl.sh

Completes "TODO: exec into the pod and replace the ETL."

(After that go ahead and merge!)

Sweet, LGTM! Could you leave a quick note on how it was tested?

Certainly! I used an existing backup pulled by the download-etl.sh script to run the import against my cluster. Before removing the current ETL directory, I tested if the tar provided contains the expected kc-etl-backup directory supplied by the download script and added a fail condition if that was not the case.
I also tested canceling the script unexpectedly (if the tar isn't correct or the user cancels while running), which resulted in a leftover kc-etl-backup. I added some cleanup for that directory as the script first execs into the pod to handle this edge case.


[rt.openssl.org #4613] openssl  RSA key: verify error 1.0.1t

Migrated from rt.openssl.org#4613 (status was 'open')
Requestors:

<EMAIL_ADDRESS>
From<EMAIL_ADDRESS>on 2016-07-11 12:10:27:
| |   | |
| |
|
Dear

I was trying to install openssl 1.0.1t on AIX5.3, and it report error when running "make test". Below is the error test.

CMS consistency test
        /usr/bin/perl cms-test.pl
CMS => PKCS#7 compatibility tests
signed content DER format, RSA key: verify error
make: 1254-004 The error code from the last command is 1.

Stop.
make: 1254-004 The error code from the last command is 2.

Could you please help to confirm whether it's a BUG, or Could you please to give me some advice about this error.

Thanks very much.

--
Gao Wenchao é«æè¶
*************************************************************
Phoneï¼13731461987
<EMAIL_ADDRESS>

|
| |
| |   | |

From<EMAIL_ADDRESS>on 2016-07-11 12:18:53:
Can you use a more recent version?  1.0.1 is end of life and only getting security fixes (and then only for the rest of the year).

From<EMAIL_ADDRESS>on 2016-07-24 20:38:52:
On Mon Jul 11 12:10:27 2016<EMAIL_ADDRESS>wrote:
>
> I was trying to install openssl 1.0.1t on AIX5.3, and it report error
> when running "make test". Below is the error test.
>
> CMS consistency test
> /usr/bin/perl cms-test.pl
> CMS => PKCS#7 compatibility tests
> signed content DER format, RSA key: verify error
> make: 1254-004 The error code from the last command is 1.
>
> Stop.
> make: 1254-004 The error code from the last command is 2.
>
> Could you please help to confirm whether it's a BUG, or Could you
> please to give me some advice about this error.
>

Other platforms don't give that error. It could be a compiler issuer. You
could try updating the compiler version or turing optimisation off to see if
you still get it.

Steve.
--
Dr Stephen N. Henson. OpenSSL project core developer.
Commercial tech support now available see: http://www.openssl.org

1.0.1 is no longer supported.


ClientModel: API Updates from BCL Feedback on KeyCredential

Also some misc tidy-up from last few PRs.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.Core


Minor fixes in the CSS file

Good job so far
Here are some suggestions in order to get it even better:
You are missing the markers that goes on the list of skills.
You have no :hover nor :focus effects.
My recommendation here is to solve those issues in order to comply with the requirements.

! [Good.. Bro] (https://giphy.com/gifs/funny-random-house-ZZiLDJ98R2GOY. gif]


How to make Laravel use Date instead of Carbon when pulling dates from database

Hi,
Larval uses Carbon to convert timestamps from the database (created_at, updated_at and custom ones) into a Carbon object. Is there a way to make Laravel use laravel-date instead?
thank you for your help

Sorry, I didn't see the "closed" tab. This question has been asked before

You can use Mutator possibility of Laravel Eloquent to use laravel-date.
In your models, set functions like this :
// Assuming using Jenssegers\Date\Date in your class

public function getCreatedAtAttribute($attr)
{
	if(!is_null($attr)) {
		return Date::parse($attr);
	}

	return null;
}

public function getUpdatedAtAttribute($attr)
{
	if(!is_null($attr)) {
		return Date::parse($attr);
	}

	return null;
}

// If your model is softDelete capable
public function getDeletedAtAttribute($attr)
{
	if(!is_null($attr)) {
		return Date::parse($attr);
	}

	return null;
}

@d3mueller Since you've found the answer maybe you could close this?

Where?

See #44.


Privnet error fix

When running neo-python with a wrong privnet chain database, the instructions how to remove it were wrong. Simple fix.

Coverage remained the same at 79.409% when pulling a2316ceae5c5b69748b50023e0101eea41c8e154 on privnet-error-fix into e34542b36547964c0742d050f5d21ea95d7a132e on development.


Update api GET /home to be replaced with /folders/personal

GET /home endpoint has been removed in favour of GET /folders/personal
See thread in sdk_maintainers for more info.
Update thread with code example once complete.

Also, can you send code samples to @kfansler to add to the API documentation?


API-Route price-detail throws Exception.

Used Shopware Version: <IP_ADDRESS>
Description:
When you call the API-Route /store-api/customized-products/price-detail it throws an Exception.
Error-Output:
{ "errors":
  [
    {
      "code":"0",
      "status":"500",
      "title":"Internal Server Error",
      "detail":"Controller Swag\\CustomizedProducts\\Storefront\\Page\\Product\\PriceDetail\\Route\\PriceDetailRoute::priceDetail requires that you provide a value for the $cartRuleIds argument. Either the argument is nullable and no null value has been provided, no default value has been provided or there is a non-optional argument after this one."
    }
  ]
}

Own Checks:
In file PriceDetailroute.php in directory SwagCustomizedProducts/src/Storefront/Page/Product/PriceDetail/Route the third parameter from method priceDetail is an array. When you call this route direct it comes to this exception. For fast repair we changed the third parameter as optional.
Changes:
public function priceDetail(Request $request, SalesChannelContext $context, array $cartRuleIds): PriceDetailResponse

public function priceDetail(Request $request, SalesChannelContext $context, array $cartRuleIds = []): PriceDetailResponse

[public] Automated response: This issue is linked to the internal issue https://shopware.atlassian.net/browse/CUS-643.
[created from CUS-643, comment 484968]


Accessing Invoked Child Machine State In React Typescript

Description
I have two machines, a parent machine and a child machine. The parent machine invokes the child machine on start and the child machine keeps running indefinitely between three states. How can I print out the child machine´s current value using react?
Expected Result
I expect to be able to subscribe to invoked child machines state transitions somehow so that I can update my react ui with the information.
Actual Result
Accessing the child machine does not return the invoked instance nor an interpreter, but instead some metadata.
Reproduction
See a full sample here, notice how the child machine state is rendered N/A even though the invoked child machine is running (see console output):

CodeSandbox Reproducible Example

Additional context
I've extracted this issue from a comment on 772. As far as I can tell #772, #664 , #938 may or may not be related issues.
This is the package.json versions:
    "@types/jest": "24.0.25",
    "@types/node": "13.1.6",
    "@types/react": "16.9.17",
    "@types/react-dom": "16.9.4",
    "@xstate/react": "1.0.0-rc.1",
    "react": "^16.12.0",
    "react-dom": "^16.12.0",
    "react-scripts": "3.3.0",
    "typescript": "3.7.4",
    "xstate": "^4.7.6"

I'm wondering though - how exactly do you use .children property? It would be highly informative for me to get some insight on how people use it.

In my case, I have an initial identify state that invokes a new machine that onDone sets a session to the parent context. Only then I transition. This child machine is plucked from rootService.children.get("identify") and injected through react props/hooks. This keeps both machines (and react trees) loosely coupled.
Hope this helps! Happy to hear about other ways to go about it.

The original issue is now fixed with the latest @xstate/react and xstate versions: https://codesandbox.io/s/great-hill-31eqd

It looks like children is strongly types as Record<string, ActorRef<T>>? Really loving the library y'all!


403 Error with $this->crud->enableDetailsRow()

When I add $this->crud->enableDetailsRow();, I get a following error.
When I see ShowDetailsRow.php, it seems that I need to give an access "details_row".
public function showDetailsRow($id)
    {
        $this->crud->hasAccessOrFail('details_row');
        $this->data['entry'] = $this->crud->getEntry($id);
        $this->data['crud'] = $this->crud;
        // load the view from /resources/views/vendor/backpack/crud/ if it exists, otherwise load the one in the package
        return view($this->crud->getDetailsRowView(), $this->data);
    }

How can I give an permission of "details_row" to users?

Ok, I added the following in the controller. Thanks.
    $this->crud->allowAccess('details_row');


Privatization of variables defined inside prange

My original question was whether or not a variable defined inside a prange is independent (privatized) across the loop, so that running it in parallel yields the expected result. For example, whether axes_hits is indeed branched between loops so that there are no cross-alterations.
@njit(parallel = True)
def get_axis_filter(line_indexes, column_indexes):
    n_solutions = line_indexes.shape[0]
    filter_ = np.full(solutions.shape[0], True)

    for i in prange(line_indexes.shape[0]):
        axes_hits = [0]*10
        for j in range(line_indexes.shape[1]):
            l_i = line_indexes[i,j]
            c_i = line_indexes[i,j]
            if l_i==c_i:
                axes_hits[l_i] +=  1
            else:
                axes_hits[l_i] += 1
                axes_hits[c_i] += 1
        if max(axes_hits) > 5:
            filter_[i] = False
    return filter_

My suggestion would be to make this more explicit in the docs. I suppose this might be implicit in the here, but perhaps it could be beneficial to refer this privatization process more explicitly, as for me it was not super clear.

This discussion spans Gitter, Discourse, and now Github. Please use your Discourse thread for continued discussion: https://numba.discourse.group/t/are-variables-defined-inside-prange-privatized/1706

Re-reading today, the approach / attitude with which I approached this and the tone of my response was ill-judged. I'm reopening the issue to track the update to the documentation, and I apologise for my words from yesterday.


[Bug]: TWAMM SinglePool LiquidityNet update formula is incorrect

Describe the bug
In function _advanceTimestampForSinglePoolSell, the liquidity is updated as:
params.pool.liquidity = params.zeroForOne
? params.pool.liquidity - uint128(liquidityNetAtTick)
: params.pool.liquidity + uint128(-liquidityNetAtTick);
The above is only correct for cases where zeroForOne == true && liquidityNetAtTick >0. It is incorrect for other cases.
For example,
Wrong case 1: assume

swap zeroForOne = false, & price go up/right-side,
cross from a lower to higher liquidity zone, (liquidityNetAtTick is positive),
the correct computation is: liquidity = liquidity + liquidityNetAtTick.

Wrong case 2:
zeroForOne == false && liquidityNetAtTick < 0.
in short, the correct formula also depends on liquidityNetAtTick>0 or not.
Expected Behavior

The correct way has for cases, shown in the picture attached.
we can bring down the 4 into 2 cases using

get liquidityNet Absolute value

check whether zeroForOne == (liquidityNetAtTick>0) and update as below:
         uint128 liqudityNetAbs =
             liquidityNetAtTick > 0 ? uint128(liquidityNetAtTick) : uint128(-liquidityNetAtTick);
         params.pool.liquidity = params.zeroForOne == (liquidityNetAtTick > 0)
             ? params.pool.liquidity - liqudityNetAbs
             : params.pool.liquidity + liqudityNetAbs;

To Reproduce
Have Added 3 test functions. the below --match-test will reproduce. and the new code solves it.
forge test --match-test testTWAMM_OrderFill_SingleSide -vv
[⠒] Compiling...
No files changed, compilation skipped
Running 3 tests for test/TWAMM.t.sol:TWAMMTest
[PASS] testTWAMM_OrderFill_SingleSide_OneForZero_Crosstick() (gas: 567406)
[PASS] testTWAMM_OrderFill_SingleSide_ZeroForOne_Crosstick() (gas: 600276)
[PASS] testTWAMM_OrderFill_SingleSide_ZeroForOne_NoCross() (gas: 509822)
Test result: ok. 3 passed; 0 failed; 0 skipped; finished in 12.63ms
Ran 1 test suites: 3 tests passed, 0 failed, 0 skipped (3 total tests)
Additional context
All for this issue.
but separately, the SingleSide TWAMM order fill has another issue on line below:
poolManager.lock(abi.encode(key, IPoolManager.SwapParams(zeroForOne, type(int256).max, sqrtPriceLimitX96)));
using of type(int256).max, may lead to insufficient balance error. I have also worked on this, and will use another issue to disscuss it.

where to get access to open PR?

more details can be found here: https://medium.com/@xben12/defi-decode-an-inside-out-explanation-of-uniswap-v4-hook-with-twamm-as-example-e9e962d95d7d

closing this issue for now, since example hooks do not live in the main branch anymore.
i've added a revisit tag as a reminder


Name disappearance and Macro button problems

So I can verify what others have been reporting.
I started having problems today after trying to add macros to a new character.  First, the names disappeared (which had been occurring already sporadically).  After that, I could not drag macros to my bars.   I tried will all my addons except Bartender turned off,   same issue.
I then deleted every trace of GSE,   reinstalled,  and reimported addons.  The same issue occurred.  First the names disappeared, and then by the third character,  the addon names were all gone, and I could not drag the icons.
Also...  the icon of the addons changed from the default ? icon that most authors use,  to a book-looking icon.
If I go back to the first character, the names are gone, but the icons are still the ? one.  Back to the third character, and nothing works.
Finally, I started all over and deleted it all again, and repeated, with the same results.

Hey TheSlickRock
This is a duplicate of #426 - pity that 2.3.02 didnt solve the issue I will keep looking and report progress over on that thread.


Support pre_exec in Proxy

Signed-off-by: CalvinNeo<EMAIL_ADDRESS>What problem does this PR solve?
Issue Number: ref #5170
Problem Summary:

Support pre_exec in Proxy
Filter CompactLog if a prelim flush fails
Always filter ComputeHash and VerifyHash

We also use a isolated Proxy branch here for easier rollback when failure.
What is changed and how it works?
Check List
Tests

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

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/run-all-tests

/run-all-tests

/run-all-tests

/cc @JaySon-Huang @lidezhu

/run-all-tests

/run-all-tests

Could we return an enum value {Done/Skip} instead of boolean for AdminObserver:: pre_exec_admin/QueryObserver:: pre_exec_query?
https://github.com/pingcap/tidb-engine-ext/compare/fbd8e6f209952d5bb79897d8bce17230acf23495...e83a37d2d8d8ae1778fe279c5f06a851f8c9e56a#diff-6342e87aaabe7cc62e18d32b39683f4db7c3566756769c6d2c296188d40bd805R90-R93

Could we return an enum value {Done/Skip} instead of boolean for AdminObserver:: pre_exec_admin/QueryObserver:: pre_exec_query?
<EMAIL_ADDRESS>
LGTM. However, we'd better use this solution for now.

/hold need update some mock and tests

/unhold

/merge

/hold seem not updated

/unhold

/hold

/unhold


chore: add shared workflow for processing nix flake files

This pull request is generated the workflow maintenance script.

Will regenerate this PR again. Closing.


"make check" is telling sensor's header is not in the correct format.

I was using the same sensor a couple of days ago and I was able to build and use this code with the sensor(O3D303) then. Now I am starting from scratch again but this time I am getting these errors while I am checking the built code with 'make check':

make check
[ 22%] Built target ifm3d_camera_shared
[ 37%] Built target ifm3d_framegrabber_shared
[ 44%] Built target ifm3d_image_shared
[ 51%] Built target gtest
[ 59%] Built target gtest_main
[ 70%] Built target ifm3d-image-tests
[==========] Running 10 tests from 1 test case.
[----------] Global test environment set-up.
[----------] 10 tests from Image
[ RUN      ] Image.MoveCtor
unknown file: Failure
C++ exception with description "Sensor: Header version mismatch while parsing data" thrown in the test body.
[  FAILED  ] Image.MoveCtor (183 ms)
[ RUN      ] Image.MoveAssignmentOperator
unknown file: Failure
C++ exception with description "Sensor: Header version mismatch while parsing data" thrown in the test body.
[  FAILED  ] Image.MoveAssignmentOperator (196 ms)
[ RUN      ] Image.CopyCtor
unknown file: Failure
C++ exception with description "Sensor: Header version mismatch while parsing data" thrown in the test body.
[  FAILED  ] Image.CopyCtor (200 ms)
[ RUN      ] Image.CopyAssignmentOperator
unknown file: Failure
C++ exception with description "Sensor: Header version mismatch while parsing data" thrown in the test body.
[  FAILED  ] Image.CopyAssignmentOperator (201 ms)
[ RUN      ] Image.References
unknown file: Failure
C++ exception with description "Sensor: Header version mismatch while parsing data" thrown in the test body.
[  FAILED  ] Image.References (199 ms)
[ RUN      ] Image.CloudMechanics
unknown file: Failure
C++ exception with description "Sensor: Header version mismatch while parsing data" thrown in the test body.
[  FAILED  ] Image.CloudMechanics (200 ms)
[ RUN      ] Image.XYZImage
unknown file: Failure
C++ exception with description "Sensor: Header version mismatch while parsing data" thrown in the test body.
[  FAILED  ] Image.XYZImage (200 ms)
[ RUN      ] Image.ComputeCartesian
unknown file: Failure
C++ exception with description "Sensor: Header version mismatch while parsing data" thrown in the test body.
[  FAILED  ] Image.ComputeCartesian (209 ms)
[ RUN      ] Image.TimeStamp
unknown file: Failure
C++ exception with description "Sensor: Header version mismatch while parsing data" thrown in the test body.
[  FAILED  ] Image.TimeStamp (1129 ms)
[ RUN      ] Image.IlluTemp
unknown file: Failure
C++ exception with description "Sensor: Header version mismatch while parsing data" thrown in the test body.
[  FAILED  ] Image.IlluTemp (197 ms)
[----------] 10 tests from Image (2914 ms total)

[----------] Global test environment tear-down
[==========] 10 tests from 1 test case ran. (2914 ms total)
[  PASSED  ] 0 tests.
[  FAILED  ] 10 tests, listed below:
[  FAILED  ] Image.MoveCtor
[  FAILED  ] Image.MoveAssignmentOperator
[  FAILED  ] Image.CopyCtor
[  FAILED  ] Image.CopyAssignmentOperator
[  FAILED  ] Image.References
[  FAILED  ] Image.CloudMechanics
[  FAILED  ] Image.XYZImage
[  FAILED  ] Image.ComputeCartesian
[  FAILED  ] Image.TimeStamp
[  FAILED  ] Image.IlluTemp

10 FAILED TESTS
modules/image/test/CMakeFiles/check_image.dir/build.make:57: recipe for target 'modules/image/test/CMakeFiles/check_image' failed
make[3]: *** [modules/image/test/CMakeFiles/check_image] Error 1
CMakeFiles/Makefile2:741: recipe for target 'modules/image/test/CMakeFiles/check_image.dir/all' failed
make[2]: *** [modules/image/test/CMakeFiles/check_image.dir/all] Error 2
CMakeFiles/Makefile2:76: recipe for target 'CMakeFiles/check.dir/rule' failed
make[1]: *** [CMakeFiles/check.dir/rule] Error 2
Makefile:184: recipe for target 'check' failed
make: *** [check] Error 2

Any Idea what is wrong here?

@pedghz what SW version do you have running on the camera?

@graugans Camera hardware info:

Connector: #!02_A300_001_02272689_008001175
Diagnose: #!02_D322_C32_02855660_008023267
Frontend: #!02_F342_C34_17_00001_008023607
Illumination: #!02_I300_001_02839215_008001175
MACAddress: 00:02:01:40:62:54
Mainboard: #!02_M381_003_02839906_008022954
MiraSerial: 0ef0-5909-00b1-0714
Article: O3D303 AD

Camera software info:

Algorithm_Version: 1.2.19
Calibration_Device: 00:02:01:40:62:54
Calibration_Version: 0.9.0
Diagnostic_Controller: 0.3.207
ELDK: GOLDENEYE_ELDK-2016-07-14_09-36-02-2e9219cce224332ecbe2f36934d2c3c7aa738c78
IFM_Software: 1.6.2114
Linux: Linux version 3.11.0-00002-gd1bbe0b (eswadmin@dettlx79) (gcc version 4.7.2 (GCC) ) #1 SMP Fri Jun 24 16:34:26 CEST 2016
Main_Application: 0.4.222

So the firmware version is 1.6.2114 and the device is O3D303 and te ifm3d version I am using is 0.11.0

I just updated the firmware to 1.23.1522 and that problem was solved


Additional Audio Indication

Additional sound for sound effects. This issue is to track the sounds required, not their implementation.

[x] Ding / Ping for new description in textual KIS tasks (could also be used in the judgement view upon change of task)
[x] Indication of task is ending soon (maybe a tick-tack sound)

done by @lucaro in de128343db935ca979e8f19bf30dbfabc5c1d2dc


JDK 13

RL doesn't work well with JDK 13, I get a bunch of weird artifacting (especially) whilst using the GPU plugin.

I think we ship Java 11 with the launcher, so I suggest using a version of java that we do support. Things can change between java versions and I guess that's what's going wrong for you.

This might be the same issue I'm having, I've described the issue here some time ago:
https://discordapp.com/channels/301497432909414422/442492468307427330/678674291696336906
I think it's caused by JDK 13 (and up) being incompatible with JOGL 2.3.2 though don't take my word for it, I'm trying to take a closer look at it at the moment.
jogamp.opengl.awt.Java2D calls getOGLUtilitiesIntField("PBUFFER");
however sun.java2d.opengl.OGLUtilities (in java.desktop) doesn't have a variable public static final int PBUFFER
java.lang.NoSuchFieldException: PBUFFER
	at java.base/java.lang.Class.getField(Class.java:2013) //public static final int sun.java2d.opengl.OGLUtilities.UNDEFINED
	at jogamp.opengl.awt.Java2D$2.run(Java2D.java:574)
	at jogamp.opengl.awt.Java2D$2.run(Java2D.java:569)
	at java.base/java.security.AccessController.doPrivileged(AccessController.java:312)
	at jogamp.opengl.awt.Java2D.getOGLUtilitiesIntField(Java2D.java:569)   //Parameter NAME is "UNDIFINED"
	at jogamp.opengl.awt.Java2D.<clinit>(Java2D.java:92)
	at java.base/java.lang.Class.forName0(Native Method)
	at java.base/java.lang.Class.forName(Class.java:340)
	at jogamp.nativewindow.jawt.JAWTUtil.<clinit>(JAWTUtil.java:345)
	at java.base/java.lang.Class.forName0(Native Method)
	at java.base/java.lang.Class.forName(Class.java:427)
	at com.jogamp.nativewindow.NativeWindowFactory$3.run(NativeWindowFactory.java:344)
	at com.jogamp.nativewindow.NativeWindowFactory$3.run(NativeWindowFactory.java:340)
	at java.base/java.security.AccessController.doPrivileged(AccessController.java:312)
	at com.jogamp.nativewindow.NativeWindowFactory.initSingleton(NativeWindowFactory.java:340)
	at com.jogamp.opengl.GLProfile.initProfilesForDefaultDevices(GLProfile.java:1721)
	at com.jogamp.opengl.GLProfile.access$000(GLProfile.java:80)
	at com.jogamp.opengl.GLProfile$1.run(GLProfile.java:230)
	at java.base/java.security.AccessController.doPrivileged(AccessController.java:312)
	at com.jogamp.opengl.GLProfile.initSingleton(GLProfile.java:216)
	at program.main(program.java:34)

We have recently replaced the GL library used by the GPU plugin. Given that change and that we are not able to reproduce this issue, we are closing this. Please comment if you still experience this issue.


[Q] LiteListDemoActivity handle item click with own listener

How do I disable opening full Google Maps when an item is clicked in LiteListDemoActivity.java?
I do not see any click listeners implemented in the example, I wonder from where click is getting triggering, and default Google Maps app gets opened
I want to have my own click listener which open Map with some GeoFencing stuff

Hi,
You can make your class implements OnMapClickListener, and set the on map click listener of each GoogleMap objects. Please have a look at the support page if you still have questions.


[Item] Resonator

Mid- somewhat late-game item. Used on carries who need mana/slows/illusions.
Crafted using Echo Saber, Yasha and a Manta Style Recipe.(Alternatively, a Resonator recipe could be used.)
Passive Effect: All attacks have a 20% chance to create an illusion of the enemy that is attacked. This illusion takes 400% damage but deals 25% of the target's damage. Illusion lasts for 4 seconds.
Active Effect: Shadowspawn :Costs 100 mana and creates an illusion of the player. This illusion is invulnerable and deals no damage. However, it provides vision and applies a 25% slow on attack. Illusion lasts for 10 seconds.
Stats: Provides considerable mana regen, armor, attack speed and attack damage.Also gives a ~10% increase to movement speed. Player also gains about +12 to all stats.
Can be disassembled, but it destroys the recipe. Cannot be upgraded further.
Note: All numbers are subject to change. They're just ballpark figures.

The numbers will certainly change a bit (I know they're fiction in your original text,  but I'm just saying FYI). Having such a passive effect can make PL lose his specialty  , but if we limit the amount of illusions somehow...or just have a low chance of spawning an Illusion - I like it.
You didn't say if the Active effect has any cooldown: does this skill work as long as you have enough mana , as long as the illusion is alive  or is there something else?
I like this item and imo would refer to it as a boss item (if it's power fits).

@VoidsKeeper Ah yes. Forgot about that. Edited the active to have a cooldown.
I don't think our old Cancer Lancer would lose his specialty, as he can almost instantaneously generate an illusion army. This item, at the most, would allow one or two illusions to be up (But the hero would need rather high attack speed to consistently have 1 illusion. Two would be quite hard to maintain.). Also, these illusions are pathetically weak and would serve as a slight boost to DPS or as damage absorbers.

I don't think an item can be disassembled if it has a recipe. Other than that, I like it.

i think i should have top cup of ilusions. like max 2-4 ilusions or 2 that don't take additonal damage. without it it would make cancer lancer from evry hero


work stuck in "depositing" state in H2 despite Argo showing accessioning completed

Split from #3071, which had 2 separate problems.
Peter wrote:

During the week of May 15, we saw this unexplained issues:

Work ID 6601, druid:kq565vk7202 was stuck in "depositing" in H2, even though accessioning completed successfully as shown in Argo, and the PURL page was available.  We investigated and found that the most likely situation was that the app/services/deposit_completer.rb never ran, which would have moved the object to the correct "deposited" state.  This class is run by the app/jobs/deposit_complete_job.rb, which is a worker that should have run based on a RabbitMQ message from the h2.deposit_complete queue.  This queue is added to by dor-workflow-service whenever a workflow step is changed, and the end-accession step should trigger this job.  For some reason it didn't in this case, even though we found evidence in the log file on workflow-service-prod that messages were being sent to RabbitMQ.  We didn't see any issues in the RabbitMQ console, and all apps showed connections as successful.

We added some extra logging to workflow-server-rails to hopefully help debug this issue in the future: https://github.com/sul-dlss/workflow-server-rails/pull/653

Naomi adds:
PR sul-dlss/workflow-server-rails/pull/653 was merged on May 17.
I believe we've seen this, or something like it, before and have had to manually reset the state.  Wondering if there are networking or rabbitMQ hiccups that can cause a signal to be missed??
Wonder further if it would be worth it to have a cron run once daily that checks for works in the "depositing" state and then looks up their workflow status to see if accessioning is completed?  (or maybe there's a DSA call that would answer this?)??   Just a thought.

Until we've seen this with some consistency, I'd argue against adding more working machinery. If we "second guess" this use of RabbitMQ, then we should create a similar backup for every use of RabbitMQ.

I agree with @justinlittman that we should close this and move on until we see this recur more frequently. I don't think there's anything worth doing here. We have better logging now at least.


added reddit to userRecon.js

Hi, thought I would help out by adding reddit account detection.

Thanks @jfroome


How to distinguish between invalid auth data and service unavailable with deepstream and webhook

Hello,
I'm using a webhoot to authenticate users https://deepstream.io/tutorials/core/auth/http-webhook/
My webhook server is working great, it authenticates users against a database.
In the last days, the database went down, my users in front end see a message saying they've provided wrong username or pass. which is totally wrong, they should see a message saying "service unavailable for the moment, please try again later".
My webhook can catch the db down error, but don't know how to report this for the front end and which status code should I send and how to retrieve it in frontend DS client.
Any pointer to docs, example is very appreciated.
Regards,

Hello,
thanks for the hints, I applied the config for my DS server.
In the logs of DS, I can see the error coming:
Jan 25 16:40:14 testsrv deepstream[395783]: INCOMING_CONNECTION | from undefined (<IP_ADDRESS>)
Jan 25 16:40:14 testsrv deepstream[395783]: AUTH_ERROR | http auth server error: {"error":{"errorno":1,"details":"db error"}}
Jan 25 16:40:14 testsrv deepstream[395783]: AUTH_UNSUCCESSFUL | invalid authentication data

But the AUTH_ERROR  is never reported  on the client, all what I get in the login function is:
success : false
clientData : { reason: 'INVALID_AUTHENTICATION_DETAILS' }
I'm using success true/false to know if the credentials are valid and show invalid login credentials on my front end in case of success: false
What I want to display in front end now is "service unavailable for the moment, please try again later"
But I don't know how to get the AUTH_ERROR to be able to display such message.
Regards,

Ok it seems to be a bug on the client side
here: https://github.com/deepstreamIO/deepstream.io-client-js/blob/4320d058e34886a5422c4b18bc239c2b62c8bc49/src/connection/connection.ts#L575
The message.data should be passed to onAuthUnsuccessfull in order to receive the server response. Instead it has the prewritten message that you get.
I'm kind of busy now, If you can try to solve it it would be great it should be something like redefining onAuthUnSuccessful to accept an argument message and pass the message.parsedData as argument so that value (if defined) is assigned to the reason variable.
Best

Thanks, I'll look into it.
On Fri, 26 Jan 2024 at 02:47 jaime echegaray zipper <
@.***> wrote:

Ok it seems to be a bug on the client side
here:
https://github.com/deepstreamIO/deepstream.io-client-js/blob/4320d058e34886a5422c4b18bc239c2b62c8bc49/src/connection/connection.ts#L575
The message.data should be passed to onAuthUnsuccessfull in order to
receive the server response. Instead it has the prewritten message that you
get.
I'm kind of busy now, If you can try to solve it it would be great it
should be something like redefining onAuthUnSuccessful to accept an
argument message and pass the message.parsedData as argument so that
value (if defined) is assigned to the reason variable.
Best
—
Reply to this email directly, view it on GitHub
https://github.com/deepstreamIO/deepstream.io/issues/1138#issuecomment-1911279603,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AATKH2SKV2IT7YX6BIC33GTYQMDMNAVCNFSM6AAAAABCHM2DSCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSMJRGI3TSNRQGM
.
You are receiving this because you authored the thread.Message ID:
@.***>

check client v7.0.4
reopen if problem persists on client repo please

server is not sending the right message

solved in server v7.0.9


Diamond Xenial Updates

Reviewers

[ ]  @esacteksab
[ ]  @jwitko

cc:
High level summary
Updated the diamond role to be compatible with Ubuntu 16.04 (Xenial).  Due to pypi issues I had to install via pip which required some task additions.

Need to figure out how to test both trusty and xenial https://github.com/saucelabs-ansible/diamond/blob/master/.travis.yml#L4

https://docs.travis-ci.com/user/multi-os/

LGTM.
Whats going on with these checks?

It's only running on trusty and 403ing due to pypi-install hitting http(vs https) I believe.  I could be wrong.  I looked at the pypi-install script and by default
    parser.add_argument('--allow-unsafe-download', action='store_true',
                      default=False,
                      help='allow unsafe downloads')

¯\_(O_o)_/¯

lol https://github.com/astraw/stdeb/issues/122

Still digging but.. https://github.com/travis-ci/travis-ci/issues/5821

Receiving the same error as trusty which means trusty is running, not xenial.
xmlrpclib.ProtocolError: <ProtocolError for pypi.python.org/pypi: 403 Must access using HTTPS instead of HTTP>

So to summarize about the failing tests, a normal trusty test isn't going to pass due to pypi not using https (link above with more details) and xenial tests won't pass because travis ci won't support xenial until possibly next year, (link above with more details).


Move markdown parsing to html to the backend

preprocess the blog posts on the server to enable storing metadata in the DB.
whenever a blog gets re-rendered, it'll invalidate the cache

should comments be fully dynamic or should posting a comment cause the page to refresh, and store the static page with comments?

https://github.com/wooorm/mdxjs-rs
https://github.com/badboy/mdbook-mermaid
https://github.com/sytsereitsma/mdbook-plantuml
https://github.com/joelcourtney/mdbook-kroki-preprocessor
https://github.com/yuzutech/kroki.io/blob/master/examples.html

https://svelte.dev/docs/svelte-compiler

pre-render most recent comments


chore: Revised readme file

Apparently, tests we not run again – are the actions smart enough to notice that no code was changed? That would be amazing :)

Yes, I skipped the tests and coverage for changes in *.md and the files in docs/ and examples/.
https://github.com/qiangyicheng/flory/blob/8f0dc67fb2b85086c1a4acd1902752f894e2b196/.github/workflows/python-package.yml#L9-L12


Returning "nan" in running train stage

@jimth001
I trained (sls_rnn.py) this model on GYAFC dataset and pre-trained embedding 300d but it returned "nan" in running train stage which is shown in log.txt below.
furthermore, returned Zero Blue score in running test stage. the output of the model was all "UNK".

log.txt

output:
informal.semd.result.txt

@jimth001
I trained (sls_rnn.py) this model on GYAFC dataset and pre-trained embedding 300d but it returned "nan" in running train stage which is shown in log.txt below. furthermore, returned Zero Blue score in running test stage. the output of the model was all "UNK".

log.txt

output: informal.semd.result.txt

Hi Mona, apologize for the late response, could you provide the input files to reproduce this issue?

@jimth001
Hi, I sent you the data by mail. please have a look. and if it is possible provide me your training data.

Do you solve this problem? Could you tell me the reason?


Fix formula on Debian

This a WIP PR.

@n-rodriguez Please tell me, when you're done. I'll test it on FreeBSD, Ubuntu and (if i find one) Debian.


Remove references to deprecated UIWebView

Apple now give warnings in App Store Connect if trying to submit with references to UIWebView.

LGTM 👍

@ricsantos this was merged on the wrong branch, so it's kind of broken the tree.  I'll restart the develop branch from current master when I do the next release.


Feedback from Nathan

Hi @Eung1,
First of all congratulations on putting together a very impressive research report. Your chosen topic is very interesting and you have structured the report very well to explain your model and address your research question. Likewise, the implementation of your code and the display of the findings is very clear and concise. I found it difficult to find anything I would recommend changing!
For my suggested amendments I have included two small changes:

I amended the labels on the IRF plots for Figures 5 and 7 to be more clear and presentable: fec3b9761a59a4c84937a1801ab62118ee5bf56b
I placed the code for the plotting of the IRFs into a function which can be called for subsequent plots rather than duplicating the plotting code: fec3b9761a59a4c84937a1801ab62118ee5bf56b

As for further developments, I would make the following suggestions:

[ ] You mention in your results for Case 2 that you expect that the results would be similar to the baseline model, however the IRF plots look quite different. It would be good to have some comments explaining why this might be. Do you suspect it might just be a bug in the code?
[ ] It would be good to include some brief economic intuition for your findings in Case 3 where you find that GDP and Credit respond less sensitively when Deposits are excluded.

Good luck for the rest of your project!

Hi @nathanh93,
Thanks for your comments and suggestions!
Good luck for the rest of your project as well!


[test] programming with flash tools that don't enumerate as a MSD

Hi @theotherjimmy and @bridadan , thanks again for all the great info yesterday!
It's looking like we can't buy those DIPdaps anymore, so can we revisit the idea of programming an mbed target via a flasher that does not enumerate as a mass storage device?  It would be used via the command line.  I know yesterday we were talking about how there would be an issue around target detection, any more thoughts on the direction we should head in trying to work around that?  Happy to back contribute whatever we come up with!

Have any idea's been discussed or proposed internally on how and when we can tackle this issue?

The professional  sector of the Mbed community very obviously needs this.
Related issues:
https://github.com/ARMmbed/mbed-os/issues/10009
https://github.com/ARMmbed/DAPLink/issues/577
https://github.com/ARMmbed/DAPLink/pull/559
https://github.com/ARMmbed/mbed-cli/issues/475

@maclobdell

We have j link working as a program and reset plugin, but I think it gets past mbed ls bc we still have daplink for serial over usb.  I think what is needed is a command line switch to skip MSD detection altogether.

Yes, that would help.
I've also setup a script that:

calls mbed test --compile to compile the greentea
calls mbedhtrun directly with the compiled binary

I can run my greentea tests this way, but off course the middle layer which is greentea itself is left out here and it would be truly great if we could have it parse the mbedhtrun output

Thank you for raising this issue. Please note we have updated our policies and
now only defects should be raised directly in GitHub. Going forward questions and
enhancements will be considered in our forums, https://forums.mbed.com/ . If this
issue is still relevant please re-raise it there.
This GitHub issue will now be closed.


Implement signing using SSH certificates

Fixes #80

Regarding

I've realized that the internal implementation of certificates in ssh-tpm-agent is a bit problematic and doesn't work the same way as ssh-agent so I suspect I need to do some larger changes.

Do you think the refactor would change the external behavior of this patch? If not, maybe you could merge the current changes with an alpha notice. Otherwise I'm perfectly fine just running a fork for now.
Also curious to hear what those differences from ssh-agent are? :-)

Do you think the refactor would change the external behavior of this patch? If not, maybe you could merge the current changes with an alpha notice. Otherwise I'm perfectly fine just running a fork for now.

I can merge my slight rewrite into master.

Also curious to hear what those differences from ssh-agent are? :-)

The implementation is done so that the SSHTPMKey struct contains both the public key and the certificate. It implies a 1-to-1 relationship between one public key, and one certificate. So multiple certificates for one key would not work very well. It also implies that the agent will do the wrong thing if you try to remove the key or the certificate from the keyring.
Effectively need to decouple this and ensure that the signer/key list contains both.


feat: Add the debugging ability for scanning Helm chart

Overview

Mapping struct is defined in. 'mappingnode.go'. Three mapping objects are used, the connection among them is that one MappingNode represents one line in one template, MappingNodes represents one template, FileMapping represents one file, and one file can contain many template.
GetMapping function in GetWorkLoads function is to call yqlib package to form the FileMapping, which is the most important part.
Adding one more output FileMapping for GetWorkLoads function and change all the related functions which call it.
After that, I connect the Filemappingwith sarif Printer. After connecting, I can use the mapping to generate correct location and fixed object for helm chart.
Additional Information
Kubescape can scan helm chart files, however, Helm does not give information about which output line is coming from which input line.
For kubescape, there is no backward connection to the original sources file after the chart was templated.
More than that, Kubescape cannot not produce correct fixes for Helm charts.
How to Test
I have created the unit tests for filemapping in helmchart.go and sarif report.
In addition, I created a Github-action repository where I change docker image to my branch image, so I could test and use my branch there.
Examples
Here is the result example, the report format is sarif.
report.sarif.txt
For the helm chart object, it has correct locations and recommended fix objects

@MMMMMMorty can you rebase?

@MMMMMMorty can you rebase?

Yes, I can do that.

@MMMMMMorty please notice the tests failed

@MMMMMMorty please notice the tests failed

Yes, I noticed that. I guess it is the part in unit test, I will change them.

This is a great contribution that we would like to merge to Kubescape.
Here are my two findings:

We are printing everything to the stdout. We should not do that.
I scanned the kubescape chart and I panicked:

panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x2 addr=0x18 pc=0x103530a44]

goroutine 1 [running]:
github.com/kubescape/kubescape/v3/core/cautils.writeNoteToMapping({0x140020c97c4?, 0x3d?}, 0x30?, {0x14002ff67b0, 0x8}, {0x14001dd7200?, 0x14001348428?}, 0x14001348578, 0x30?, 0x1, ...)
	/Users/davidwertenteil/armo/repos/kubescape/core/cautils/parseFile.go:128 +0x164
github.com/kubescape/kubescape/v3/core/cautils.writeNodes({0x14002754f10?, 0x2, 0x1040fa324?}, 0x4?, {0x14001dd7200, 0x3c}, 0x14002237f30?, 0x24?, 0x14002d7ae70?)
	/Users/davidwertenteil/armo/repos/kubescape/core/cautils/parseFile.go:105 +0xe8
github.com/kubescape/kubescape/v3/core/cautils.GetMapping({0x14001dd7200, 0x3c}, {0x14002237c00, 0x4c5})
	/Users/davidwertenteil/armo/repos/kubescape/core/cautils/parseFile.go:85 +0x330
github.com/kubescape/kubescape/v3/core/cautils.GetTemplateMapping(0x140017c45d0?, 0x0?)
	/Users/davidwertenteil/armo/repos/kubescape/core/cautils/helmchart.go:142 +0x84
github.com/kubescape/kubescape/v3/core/cautils.(*HelmChart).GetWorkloads(0x140017c45d0, 0x47?)
	/Users/davidwertenteil/armo/repos/kubescape/core/cautils/helmchart.go:71 +0x16c
github.com/kubescape/kubescape/v3/core/cautils.(*HelmChart).GetWorkloadsWithDefaultValues(...)
	/Users/davidwertenteil/armo/repos/kubescape/core/cautils/helmchart.go:51
github.com/kubescape/kubescape/v3/core/cautils.LoadResourcesFromHelmCharts({0x105dd96c0, 0x14002506b70}, {0x16f373768?, 0x44?})
	/Users/davidwertenteil/armo/repos/kubescape/core/cautils/fileutils.go:56 +0x1bc
github.com/kubescape/kubescape/v3/core/pkg/resourcehandler.getResourcesFromPath({0x105dd96c0, 0x14002506b70}, {0x16f373768, 0x2d})
	/Users/davidwertenteil/armo/repos/kubescape/core/pkg/resourcehandler/filesloader.go:282 +0x6fc
github.com/kubescape/kubescape/v3/core/pkg/resourcehandler.(*FileResourceHandler).GetResources(0x140012067d0?, {0x105dd96c0, 0x14002506b70}, 0x1400023f600, {0x0?, 0x0?}, 0x140013ee000)
	/Users/davidwertenteil/armo/repos/kubescape/core/pkg/resourcehandler/filesloader.go:53 +0x1a0
github.com/kubescape/kubescape/v3/core/pkg/resourcehandler.CollectResources({0x105dd96c0, 0x14002506b40}, {0x105dc20f8, 0x108806be0}, {0x1?, 0x0?, 0x140013ee000?}, 0x1400023f600, {0x105dc21b8, 0x14002213c08}, ...)
	/Users/davidwertenteil/armo/repos/kubescape/core/pkg/resourcehandler/handlerpullresources.go:29 +0x168
github.com/kubescape/kubescape/v3/core/core.(*Kubescape).Scan(0x0?, {0x105dd97a0?, 0x108806be0}, 0x140013ee000)
	/Users/davidwertenteil/armo/repos/kubescape/core/core/scan.go:166 +0x6c4
github.com/kubescape/kubescape/v3/cmd/scan.getFrameworkCmd.func2(0x140015c4700?, {0x14001274ae0, 0x2, 0x6})
	/Users/davidwertenteil/armo/repos/kubescape/cmd/scan/framework.go:120 +0x5f8
github.com/spf13/cobra.(*Command).execute(0x1400112c900, {0x14001274a80, 0x6, 0x6})
	/Users/davidwertenteil/go/pkg/mod/github.com/spf13/cobra@v1.8.0/command.go:983 +0x840
github.com/spf13/cobra.(*Command).ExecuteC(0x1400112c000)
	/Users/davidwertenteil/go/pkg/mod/github.com/spf13/cobra@v1.8.0/command.go:1115 +0x344
github.com/spf13/cobra.(*Command).Execute(...)
	/Users/davidwertenteil/go/pkg/mod/github.com/spf13/cobra@v1.8.0/command.go:1039
github.com/kubescape/kubescape/v3/cmd.Execute()
	/Users/davidwertenteil/armo/repos/kubescape/cmd/root.go:117 +0x38
main.main()
	/Users/davidwertenteil/armo/repos/kubescape/main.go:27 +0xbc

@dwertent For the printing problem, I will fix it soon. For the panic problem, I guess it mainly because I haven't updated this feature for a long time, I will fix it, too.

Thank you! You can reproduce the panic by running
kubescape scan https://github.com/kubescape/helm-charts


Remove python version upper constraint

Hi, this PR aims to remove the python upper version constraint <4.0 which is adding extra constraint for no reason and forces children project to adopt the unwanted poetry version notation or add this extra <4.0 constraint.
This would allow children project to avoid this useless warning

Note that the resolver complains only in poetry and not in rye, maybe rye knows python 4.0 is far from reality :hugs:

A bit niche, but I don't mind removing this constraint. Thanks! (And sorry for the late reply!)


[BUG] no action triggered on click for text upload/paste option

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to Home Page
Click on Click here if you rather upload text or try pasting it now.

Expected behavior
Clicking the text (button) should provide the option to either upload text or paste it directly.

Same issue on my end


Allow to stage selected lines only

When in diff mode ("d" key) it is currently possible to stage blocks of code by pressing "s" (see screen recording http://take.ms/sJbDz). But sometimes I only need to stage one or two lines. For this I have to use another git client that allows to do this, such as gitUp or Tower (see screen recording of gitUp http://take.ms/I2X6X).
Would be very cool if I could select few lines and hit "s" for this

This is also possible in GitSavvy Sublime plugin. But I still prefer SublimeGit because the other one is a bit glitchy

I actually think that this project is dead as the maintainer is not responding. See https://github.com/SublimeGit/SublimeGit/issues/257

Yeah, it seems so 😓


Xmonad: Fix ManageDocks bug

Issue description
This bug is present in the current version of Xmonad we ship:
https://wiki.haskell.org/Xmonad/Notable_changes_since_0.11#.2AURGENT.2A_Bug_in_0.12_ManageDocks
We should try and update the Xmonad version we install by default to include a fix for this bug, otherwise the out-of-the-box Xmonad user experience is frustrating.

XMonad 0.13 is shipped since 5d3bb96f622 -- does that close this issue?

The newer xmonad 0.13 is not even working for me:
k@z0 ~ 1 % xmonad --version
xmonad 0.13
k@z0 ~ 0 % xmonad --recompile
xmonad: ghc: runProcess: runInteractiveProcess: exec: does not exist (No such file or directory)

My bad. I forgot to add our patch to the upstream xmonad distribution. It's working now, though not in 16.09. Maybe we should backport this new version of xmonad xmonad-contrib (and presumably, xmonad-extras) to 16.09?  The only Haskell dependency that I had to update was X11.

I'm not sure it would qualify as non-intrusive bugfix-only update.

I think the best we can do is add xmonad_0_13_0 to the set of Haskell packages, then let people override xmonadPackages (or whatever it's called) to have this be the default version. In the next stable release, we can bump the default version to be 0.13.

@ocharles I like that compromise. I'll try and come up with a patch in the next few days.

Is this still relevant now that 17.09 is out?

release 17.03 has xmonad 0.13 (only).  Any older nixpkgs is certainly unsupported.


oauth getRedirectResult should only remove magic/oauth-related keys from url search params

✅ Prerequisites

[x] Did you perform a cursory search of open issues? Is this feature already requested elsewhere?
[x] Are you reporting to the correct repository (magic-sdk)?

✨ Feature Request
Looking at https://github.com/magiclabs/magic-js/blob/3d5e207a5754b920b446cd42e9858bda6357ecae/packages/%40magic-ext/oauth/src/index.ts#L40
I see that the sdk is removing the whole query string which I understand is meant for security reasons.  Our issue is we are developing a feature similar to a referral system, where we need some custom keys in the link for it to function.
Would it be possible to only remove the oauth related keys? I believe they would be 'state', 'scope', 'magic_oauth_request_id', 'magic_credential'.

Hi @rhyek ,
Thanks for this issue submission!
You can actually grab those parameters from the URL before calling getRedirectResult(). This way you can store the info needed, like your referral info, and then call getRedirectResult() which will clear the params entirely.
Example login call:
await magic.oauth.loginWithRedirect({
    provider,
    redirectURI: new URL(`/callback?referral=${generateRandomKey(9)}`, window.location.origin).href,
});

On your callback (redirectURL) page/component you can do the following in your useEffect or on page load:
const referralID = new URLSearchParams(props.location.search).get("referral");

// do something with the info from the URL like save to your DB
console.log("referral ID: ", referralID);

// now you can finish the social login flow
await magic.oauth.getRedirectResult();

tl;dr
Grab the needed info from the URL before calling getRedirectResult().


[CT-3439] [Feature] Support for postgres index extensions

Is this your first time submitting a feature request?

[X] I have read the expectations for open source contributors
[X] I have searched the existing issues, and I could not find an existing issue for this feature
[X] I am requesting a straightforward extension of existing dbt functionality, rather than a Big Idea better suited to a discussion

Describe the feature
Developers should easily be able to select indices from installed extensions -- e.g. I would like to use a gin index from pg_trgm.
Describe alternatives you've considered
This is the clearest workaround, but it's not obvious to new users and it's counterintuitive to do something different for this index than for other regular indices.
{{
    config(
        indexes=[
            {"columns": ["my_regular_column"], "type": "btree"},
        ],
        post_hook=[
            "create index my_extension_index on {{this}} using gin (my_extension_column gin_trgm_ops);"
        ]
    )
}}

Who will this benefit?
This will benefit users of Postgres who wish to add indices that are not supported out of the box.
Are you interested in contributing this feature?
Unfortunately I don't have the time.
Anything else?
No response

Thanks for proposing this @scottgigante !
Something we'll often say that we want dbt to "make easy things easy and hard things possible". Since this proposal involves  an optional postgres exension, we would put this in the "hard" category rather than the "easy" category. So our aspiration would be to make this at least possible (even if we can't / won't make it easy).
If we were to implement this, the syntax might look like this:
{{
    config(
        indexes=[
            {"columns": ["my_regular_column"], "type": "btree"},
            {"columns": ["my_1st_ext_column"], "type": "gin", "opclass": "gin_trgm_ops"},
            {"columns": ["my_2nd_ext_column"], "type": "gist", "opclass": "gist_trgm_ops"},
            {"columns": ["my_2nd_ext_column"], "type": "gist", "opclass": "gist_trgm_ops(siglen=32)"}
        ]
    )
}}

But since it is possible right now via post-hooks and not a priority for us to implement ourselves or accept an open source contribution, I'm going to close this as "not planned".
p.s.
Agreed that the workaround is not obvious to new users and it's counterintuitive to do something different for trigram-based indexes like gist (Generalized Search Tree) or gin (Generalized Inverted Index) from the pg_trgm extension module. So we should supplement the documentation for Postgres indexes with code examples similar to what you provided.
Here's the issue I've opened for that supplemental documentation: https://github.com/dbt-labs/docs.getdbt.com/issues/4568.

Thanks @dbeatty10, that explanation makes a lot of sense and the proposed docs addition solves 90% of the problem. Thank you!

Would love to get support for pg_bm25 in dbt as well! https://github.com/paradedb/paradedb


Migrate to Chrome Manifest V3

Chrome Manifest Version 2 has been deprecated, that is why we need to migrate Manifest Version 3:
https://developer.chrome.com/docs/extensions/develop/migrate

Implemented with 1.5.0


Add assurance_required key to validation errors

Required as part of this story/bugfix: https://www.pivotaltracker.com/story/show/102149718
All the different types of assurance array have "Service provider assertion" as the first option, so this should work OK.

Bit hacky but should work fine for now. :+1:

No more hacky than the other mappings from JSON schema error messages to our error keys...


Using an = in the boundary returns a malformed content-type error

Hello.
= should be considered a valid character to use for the boundary in a multipart/form-data request.
example content-type that can produce the problem multipart/form-data; boundary====1656457491496===
Seems like it's returning an error in parseContentType

Unquoted parameter values must consist only of token characters, which does not include '='.