unable to install typings

npm install @types/rethinkdbdash --save-d
throws a not found error, am I doing something wrong?

@jimjones26 until the long awaited https://github.com/Microsoft/types-publisher/issues/4 lands you need to reference these typings from GitHub (see https://github.com/types/rethinkdbdash/issues/1#issuecomment-308442936 for the explanation). For now just do:
npm install types/rethinkdbdash --save-dev

... which will reference it as a git dependency in your package.json.

Thank you for the quick reply! That worked


Get config file validation under test

This is fairly low-priority, since there's little in the way of custom handling being done, but at least a cursory attempt should be made to ensure that reading config files works as intended.

There is some basic config file validation with go-playground/validator.  Should mostly do what we want.
See:

https://github.com/retr0h/go-gilt/blob/main/pkg/config/types.go
https://github.com/retr0h/go-gilt/blob/main/pkg/config/schema_test.go


Pdf.js for Firefox missing

Issue description
Pdf.js for Firefox is missing.

This is not a well formed issue, you have neglected to include information such as which commit of nixpkgs you're using. Please see the issue template for more details.

You are right. Sorry. I also tried nixos-unstable-small and Firefox here knowns pdf.js.


SAFE-26567: WebSocket reconnect fix

Move TCP socket close from ClientWorker class to SocketImpl destructor
Remove unused clientTypes vector
Use connections mutex in ClientInformation class
(instead of WsHandlerFactoryMultiClient)
Do cleanup of ClientWorkers immediately

Reviewed with REr


fix: make sure midtrans-payment-page.js load with JQuery

This plugin unable to load JQuery, maybe conflict with some other plugin, make "data-cfasync" to true resolved the issue

Answered on: https://github.com/veritrans/SNAP-Woocommerce/issues/63#issuecomment-909884960
Also, due to: last time we check 'data-cfasync' => 'false', will be needed for compatibility with Cloudflare WP plugin.
we will close this for now. Thanks!


[Addition]: "Icy Topaz Hippo" , stolen miner, FRAUD

Hotspot b58 Addresses
11nEcgpABvs7Ndd88n2DEmCWBuuqHREKhjHvnQdDmNQvUPMMfVN
Discord Handle
double_d#6206
Reason(s)
Hello,
I ordered a few Nebra outdoor miners from Parley Labs and ended up selling a couple via eBay. Some guy by the name of Graham Zulauf stole one of the packages with an outdoor Nebra miner in it and refuses to return it. I have proof via iMessage.  link: https://imgur.com/a/4Cep4Ej
I'd like to report that Nebra miner (named Icy Topaz Hippo) located at 9842 Grandview Drive, La Mesa CA, 91941 and see if we can blacklist it from mining on the network...
I have attached our iMessage conversation and my order from Parley Labs for reference. He is now trying to threaten me by trying to report my own personal miner for no reason...
I would like to pursue this to the full extent of the law. Please let me know if you need any more information on this matter.
Here's the link again to the iMessage conversation where he admits guilt: https://imgur.com/a/4Cep4Ej
Thank you.
His information is:
Name: Graham J Zulauf
Street: 9842 Grandview Dr
City: La Mesa
State/province: CA
Zip code: 91941-5622
Country/region: United States
Phone: +1<PHONE_NUMBER>

Feel free to email<EMAIL_ADDRESS>with an the details as well @danielomid


Switch to using QuantumLib

Hi @rnrand, here's a pull request to switch to using QuantumLib. Based on my local testing, it should compile with Coq versions 8.12-8.15 and the most recent version of QuantumLib (see instructions in the README for installation).
The changes are mostly just deleting redundant files and renaming imports. One objectionable edit is that I deleted the following four files, which are not built by the current Makefile:

DBGenerator.v
Generator.v
HOASGenerator.v
TypesDBCircuit.v

If these files are important, then you should add a target for them in the Makefile.

The deleted files are for random testing, I believe, except for TypesDBCircuit.v which attempts to type de Bruijn circuits. I agree that they can be deleted – I think SQIR is a better place for random testing anyhow.


Can't start kbfs

OS: OS X
Keybase Version: keybase version 1.0.15-20160401130013+78dff87
I can't seem to start kbfs, keybase ctl restart hasn't helped.
Output of keybase status:
Username:      jonreem
Logged in:     yes

Device:
    name:      laptop
    ID:        6fcca53180e3f64881ad6ee5d6f34518
    status:    active

Session:       no session
    is valid:  yes
    keys:      locked
    lksec:     not loaded

KBFS:
    status:    not running
    version:   1.0.2-20160401130057+25510ca
    log:       /Users/reem/Library/Logs/keybase.kbfs.log

Service:
    status:    running
    version:   1.0.15-20160401130013+78dff87
    log:       /Users/reem/Library/Logs/keybase.service.log

Platform Information:
    OS:        darwin
    Runtime:   go1.6
    Arch:      amd64

Client:
    version:   1.0.15-20160401130013+78dff87

Desktop app:
    status:    running
    version:   1.0.15-20160401130013+78dff87
    log:       /Users/reem/Library/Logs/Keybase.app.log

Config path:   /Users/reem/Library/Application Support/Keybase/config.json
Default user:  jonreem
Other users:

command-line client:
    keybase status [pid: 15719, version: 1.0.15-20160401130013+78dff87]

desktop:
    /Applications/Keybase.app/Contents/Frameworks/Keybase Helper.app/Contents/MacOS/Keybase Helper --type=renderer --no-sandbox --lang=en-US --node-integration=true --num-raster-threads=4 --enable-zero-copy --enable-gpu-memory-buffer-compositor-resources --content-image-texture-target=3553,3553,3553,3553,3553,34037,3553,3553,3553,3553,34037,3553,34037,34037 --video-image-texture-target=34037 --channel=12348.1.186385196 [pid: 12354, version: 1.0.15-20160401130013+78dff87, description: Menubar]
    /Applications/Keybase.app/Contents/Frameworks/Keybase Helper.app/Contents/MacOS/Keybase Helper --type=renderer --no-sandbox --lang=en-US --node-integration=true --num-raster-threads=4 --enable-zero-copy --enable-gpu-memory-buffer-compositor-resources --content-image-texture-target=3553,3553,3553,3553,3553,34037,3553,3553,3553,3553,34037,3553,34037,34037 --video-image-texture-target=34037 --channel=12348.2.1751919681 [pid: 12355, version: 1.0.15-20160401130013+78dff87, description: Main Renderer]
    /Applications/Keybase.app/Contents/MacOS/Electron -psn_0_25102319 [pid: 12348, version: 1.0.15-20160401130013+78dff87, description: Main Thread]

Output of keybase launchd list:
keybase.service: Running (pid=14800) [Keybase]
keybase.kbfs: Not Running exit=2 [KBFS]

keybase log send id: ec6a9aea3bde5eb2c5ef791c

The log says:
2016/04/04 14:09:21 mount helper error: mount_kbfuse: failed to mount /keybase@/dev/kbfuse0: Operation not permitted

Does your OSX user have administrative privileges?

I was not a root user when running any start commands, should I sudo keybase ctl restart or sudo keybase launchd start keybase.kbfs?

You shouldn't need to be root.  I just meant admin privileges on your non-root account, e.g.: https://superuser.com/questions/114422/how-to-grant-admin-permissions-to-user-account-in-snow-leopard

Oh, in that case yes, I am an admin user.

And you've dragged the keybase dmg into the applications folder and launched it from within that folder?
(We're reaching the end of my ability to debug OSX issues, so if that's not it I'll need to get someone else to take a look...)

Yeah, I've done that as well, still not working.

Ok, @gabriel can you take a look please?

This seems like another kbfs might be running which has the mount already?
Is this installed or running as another user?
You could try doing:
keybase uninstall
pkill kbfs
keybase install

Just tried that, didn't change anything. mount doesn't list any custom mounts.

Note that it looks like keybase comes with its own version of fuse (kbfuse), and I also have osxfuse installed, if this could be causing a problem.

what does ls -al look like for /keybase (permissions)?

Yeah we have a separate KBFuse that we manage that can co-exist with OSXFuse, so that shouldn't be the issue...
Also, what's the output of:
kextstat | grep fuse

It didn't exist, and when I created it with sudo mkdir /keybase I get:
❯ ls -al /keybase
total 0
drwxr-xr-x   2 root  wheel    68 Apr  4 15:48 .
drwxr-xr-x  36 root  wheel  1292 Apr  4 15:48 ..

❯ kextstat | grep fuse
152    0 0xffffff7f832db000 0x16000    0x16000    com.github.osxfuse.filesystems.osxfusefs (2.8.3) 7DBFEA1C-C986-3A12-9419-C09D25A6F3EA <7 5 4 3 1>
249    0 0xffffff7f833d2000 0x1a000    0x1a000    com.github.kbfuse.filesystems.kbfuse (3.2.0) AF9EEF06-5277-351B-83D6-D9E65624AB90 <7 5 4 3 1>

Is KBFuse open source? I've worked with osxfuse before and have been pretty frustrated with the API so I'm curious to see what you all changed.

It doesn't exist? The log id from above seemed to think it did...
Can you rm /keybase and then restart the Keybase.app? The app will create this directory with the right permissions.
KBFuse is exactly same as OSXFuse but it's namespaced (or "branded") so we can control and enforce specific versions and not conflict with other apps.

That worked! Odd. Thanks for your help. Makes sense re: kbfuse.

I'm gonna make an issue to add some checks around this for the future. It should probably resolve/fix any permissions problems it encounters.
Thanks for helping debug!

I think this is the same issue I'm having on macOS Sierra. I'm getting
keybase.service: Running (pid=2407) [Keybase] keybase.kbfs: Not Running exit=2 [KBFS] keybase.updater: Running (pid=2422) [Updater]
after running keybase launchd list
How do I do your solution:

Can you rm /keybase and then restart the Keybase.app? The app will create this directory with the right permissions.

We are still testing the macOS sierra builds and waiting on golang fixes. There we some issues with golang which are still being resolved by that team.

Here is a build which supports macOS 10.12 (Sierra):
https://prerelease.keybase.io/darwin/Keybase-1.0.17-20160817191833%2Be83e8a5.dmg

Suggestions on how to get it up and running once I've installed it? Same error as before.

▶ ERROR keybase.kbfs is not running (exit status 2

Can you post a keybase log send ID?

279edf7db429d756207ea81c

Oh it's possible we need to reset permissions on the placeholder /keybase directory.
If you do:
keybase ctl stop
keybase uninstall
sudo rmdir /keybase
# Then re-open app.
open /Applications/Keybase.app/

If this doesn't fix, lmk and will dig further.
We have a patch coming soon to look for and fix permissions problems if something weird happened (or in your case an earlier Sierra incompatibility), to make this stuff a little more bulletproof.

Did all that. Same as before - doesn't work.

455bfe78989294c9f20b3f1c

From the logs it looks like somehow the old version of the app (helper tool) re-created the directory with the bad permissions before the new one ran.
Now that you have the more recent app with fixes, can we try one more time?
keybase ctl stop
keybase uninstall
sudo rmdir /keybase
open /Applications/Keybase.app/

Do I need to remove the UI app from my applications folder at all? I've done all of these steps again and still, the Keybase folder will not open.

New log ID ce5c9d5dafdeb39120d3f61c

Seems like a lot of helpers.

After uninstalling and physically removing the app, there were still helpers running. I managed to find and remove them as well. Did a full fresh restart. My other computer detected the new install and rekeyed my files. Still can't open them. New log ID f22503d0f108f0a5f8ae7d1c

The helpers are ok.
Sorry it's still not working! I'll have to dig into the logs a little deeper.

No apology needed. I know Keybase is in alpha and I'm running a beta OS, so I'm playing with fire here. 😄
I'm hope this just helps y'all with testing before Sierra goes live next month.

Yep this is super helpful.
Can you try running this version:
https://keybase.pub/gabrielh/Keybase-1.0.17-20160824235155%2B51f7870.dmg
It has some extra debugging in it.

Oh and can you do the following when installing:

Quit Keybase.app
Copy from dmg to /Applications
Run keybase ctl stop from command line
Then open /Applications/Keybase.app

▶ ERROR No service (plist) installed with label: keybase.kbfs
Log ID 64f78d8ad23953ae2728f41c

I feel like we're getting closer. I can actually open the Keybase folder now without getting the error message that I don't have permission. However, the folder is completely empty and never shows private or public folders.

I'm still a little baffled what's going on, the fuse mount is still erroring the same way and the error is especially cryptic... And I can't reproduce it either (on a machine and a vm so far).
If you ls -al / what does the keybase folder line look like:
It might look something like this:
drwx------     2 gabe  staff     68 Aug 25 14:25 keybase

What is the output of:
kextstat | grep fuse

It should look something like this:
 290    0 0xffffff7f83f8b000 0x19000    0x19000    com.github.kbfuse.filesystems.kbfuse (3.4.2) 3D1D2A40-206A-39DD-A1F6-5DE4AD1A42B2 <7 5 4 3 1>

And what is the output of mount -t kbfuse. It might not list anything.
And finally... when you start Keybase, do you see anything happen in the system.log.. You can view from Console.app (or looking at /private/var/log/system.log).

Oh yeah, and are you on the latest Sierra beta? (I think beta 7)... In About This Mac does it say Version 10.12 Beta (16A304a).

Yes, that's the version.
ls -al / returns nothing about Keybase

kextstat | grep fuse returns nothing

Console:
Aug 25 17:49:26 Lucs-MacBook-Pro keybase.Helper[30277]: Request: version({ 	}) Aug 25 17:49:26 --- last message repeated 3 times --- Aug 25 17:49:26 Lucs-MacBook-Pro com.apple.xpc.launchd[1] (keybase.updater): This service is defined to be constantly running and is inherently inefficient. Aug 25 17:49:28 Lucs-MacBook-Pro com.apple.xpc.launchd[1] (keybase.service): This service is defined to be constantly running and is inherently inefficient. Aug 25 17:49:29 Lucs-MacBook-Pro keybase.Helper[30277]: Request: remove({ 	    path = "/keybase"; 	}) Aug 25 17:49:29 --- last message repeated 1 time --- Aug 25 17:49:29 Lucs-MacBook-Pro keybase.Helper[30277]: Request: createDirectory({ 	    directory = "/keybase"; 	    excludeFromBackup = 1; 	    gid = 20; 	    permissions = 448; 	    uid = 501; 	}) Aug 25 17:49:30 --- last message repeated 1 time --- Aug 25 17:49:30 Lucs-MacBook-Pro com.apple.xpc.launchd[1] (keybase.kbfs): This service is defined to be constantly running and is inherently inefficient. Aug 25 17:49:31 Lucs-MacBook-Pro com.apple.xpc.launchd[1] (keybase.kbfs[44599]): Service exited with abnormal code: 2 Aug 25 17:49:31 Lucs-MacBook-Pro com.apple.xpc.launchd[1] (keybase.kbfs): Service only ran for 0 seconds. Pushing respawn out by 10 seconds. Aug 25 17:49:32 Lucs-MacBook-Pro Console[44553]: BUG in libdispatch client: kevent[EVFILT_MACHPORT] monitored resource vanished before the source cancel handler was invoked Aug 25 17:49:39 Lucs-MacBook-Pro com.apple.xpc.launchd[1] (com.apple.quicklook[44615]): Endpoint has been activated through legacy launch(3) APIs. Please switch to XPC or bootstrap_check_in(): com.apple.quicklook Aug 25 17:49:41 Lucs-MacBook-Pro com.apple.xpc.launchd[1] (keybase.kbfs[44616]): Service exited with abnormal code: 2 Aug 25 17:49:41 Lucs-MacBook-Pro com.apple.xpc.launchd[1] (keybase.kbfs): Service only ran for 0 seconds. Pushing respawn out by 10 seconds. Aug 25 17:49:41 Lucs-MacBook-Pro com.apple.xpc.launchd[1] (com.apple.imfoundation.IMRemoteURLConnectionAgent): Unknown key for integer: _DirtyJetsamMemoryLimit

ok, maybe it's the kext failing to load... what happens if you run this:
/Library/Filesystems/kbfuse.fs/Contents/Resources/load_kbfuse
echo $?

Then when you run kextstat | grep fuse does it show anything.
What is the output of (so I can see permissions):
ls -al /Library/Filesystems/kbfuse.fs/Contents/Resources/load_kbfuse

I did grep instead of grep fuse the first time. Here's the scoop:
Last login: Thu Aug 25 18:29:25 on ttys000 Lucs-MacBook-Pro:~ lucherlitz$ /Library/Filesystems/kbfuse.fs/Contents/Resources/load_kbfuse  Lucs-MacBook-Pro:~ lucherlitz$ echo $? 6 Lucs-MacBook-Pro:~ lucherlitz$ kextstat | grep usage: grep [-abcDEFGHhIiJLlmnOoqRSsUVvwxZ] [-A num] [-B num] [-C[num]] 	[-e pattern] [-f file] [--binary-files=value] [--color=when] 	[--context[=num]] [--directories=action] [--label] [--line-buffered] 	[--null] [pattern] [file ...] Lucs-MacBook-Pro:~ lucherlitz$ kextstat | grep fuse Lucs-MacBook-Pro:~ lucherlitz$ ls -al /Library/Filesystems/kbfuse.fs/Contents/Resources/load_kbfuse -rwsr-xr-x@ 1 root  wheel  26636 Apr 12 15:40 /Library/Filesystems/kbfuse.fs/Contents/Resources/load_kbfuse

http://recordit.co/oKcZoOWy91

ok, it's the kext failing to load and isn't directly a mount issue... we're getting closer.. I'll keep digging.

Sweet! Thank you!

On Aug 25, 2016, at 6:50 PM, Gabriel Handford<EMAIL_ADDRESS>wrote:
ok, it's the kext failing to load and isn't directly a mount issue... we're getting closer.. I'll keep digging.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

What happens if you run and install the latest osxfuse:
https://github.com/osxfuse/osxfuse/releases/download/osxfuse-3.4.2/osxfuse-3.4.2.dmg
(kbfuse is a renamed version of that, so we don't conflict with existing fuse installs, if people rely on specific versions of fuse)
After installing, what happens if you do:
/Library/Filesystems/osxfuse.fs/Contents/Resources/load_osxfuse
echo $?

And then whats the output of kextstat | grep fuse again?

Ok, getting closer.. what is the output of:
ls -ld $(find /Library/Filesystems/kbfuse.fs/)

And the output of:
codesign --verify --verbose=4 /Library/Filesystems/kbfuse.fs

Terminal Saved Output.txt

ok, I think I see the problem... one last thing (I think):
cat /Library/Filesystems/kbfuse.fs//Contents/Info.plist

Terminal Saved Output 2.txt

ok, I know what the issue is... I'll have an fix soon. Thanks a ton for your help!

Ok, this build should work:
https://keybase.pub/gabrielh/Keybase-1.0.17-20160826020144%2B61c0667.dmg

You, sir, are a genius! It works!

Well, I now have two keybase folders, seemingly identical. One on my desktop, one at the top level.  Can't remove either. but, I can add files, so that's working!

Hooray, that fixed it for me also.

Ok, this build should work:
https://keybase.pub/gabrielh/Keybase-1.0.17-20160826020144%2B61c0667.dmg

@gabriel Is there a more recent fix I can grab?

Our latest release now works on Sierra: https://keybase.io/docs/the_app/install_macos

@gabriel If by latest release you mean 1.0.17, that's what I'm running:

@zbeat No, there's a newer version available now.  The version string is 1.0.17-20160930202009+0229d49, you can get it from an automatic update (prompted hourly) or from https://keybase.io/docs/the_app/install_macos.

@cjb And magically a popup appears 1 hour later:

👍


cannot call 'pyInitialize'

what could be the cause of this problem (I ma running py3.10):
2022.11.22 17:25:01.581	PythonDLL_Example (AUDJPY,D1)	cannot call 'pyInitialize', 'Roffild\PythonDLL\x64\Release\PythonDLL.dll' is not loaded

yes it's the version, so I recompiled with 3.10 and all done


Will not start, failed to create database

Describe the bug
On a new Archlinux install with onagre 1.1.1-1 from the AUR, it will not launch stating it cannot find the database
Config
No config done, straight out of the "box"
To Reproduce
On a fresh install of Archlinux, use Paru to install the latest version of Onagre, and from a terminal prompt try and run
Expected behavior
It runs
Screenshots
2024-12-30T15:50:59.085519Z  INFO onagre: Starting onagre
2024-12-30T15:50:59.085576Z ERROR onagre::app::style: Failed to parse theme "/home/kihaji/.config/onagre/theme.scss": Failed to open config file
2024-12-30T15:50:59.085581Z  WARN onagre::app::style: Failing back to default theme
thread 'main' panicked at src/db/mod.rs:34:55:
failed to create database: Storage(Io(Os { code: 2, kind: NotFound, message: "No such file or directory" }))
stack backtrace:
0:     0x5a7570f41437 -
1:     0x5a75709dc773 -
2:     0x5a7570f06df2 -
3:     0x5a7570f42b12 -
4:     0x5a7570f4285b -
5:     0x5a7570f4312c -
6:     0x5a7570f42be8 -
7:     0x5a7570f42b49 -
8:     0x5a7570f42b3c -
9:     0x5a7570787bdf -
10:     0x5a7570787fb5 -
11:     0x5a75708f4aba -
12:     0x5a75707e3f46 -
13:     0x5a757090e1ef -
14:     0x5a757092bf66 -
15:     0x5a75708372a3 -
16:     0x5a75709442fa -
17:     0x7ad1ea551e08 -
18:     0x7ad1ea551ecc - __libc_start_main
19:     0x5a75707c9b25 -
20:                0x0 -
Window manager (please complete the following information):

Window manager:  Hyprland

Additional context
I'm pretty sure I'm an idiot and either do not or have a bad environment variable, so once I figure out what the path should be I can fix it, but my Rust is rusty, so it's going to take a second. But at a minimum the error should print out where it is trying to find the database to make it easier to debug.

As a quick update, still the same issue either with or without XDG_DATA_DIRS set

you need either $XDG_DATA_HOME or $HOME/.local/share

Yeah, it started to work after I installed Brave, which created the .local/share.
If I am reading the XDG spec correctly, $XDG_DATA_HOME defaults to .local/share when it is empty, so to me that means the expected behavior of Onagre on install is if it is unset, and that directory does not exist, it should create it, or at least ask the user if they want to create it/tell them they need to create it.
It's not a huge thing, but a good Quality of Life thing


Come up with software design of how to process the persisted coin data

Thinking to use postgres instead of mongo

**TEMP PLAN:
plan is to run this docker img in AWS and utilize some amazon db functionality to store data as a table:
COIN | COUNT | TIMESTAMP
this way i can create API to access said table quite easily

storing in mongo DB


[$1000] Connect back account: Phone number format in placeholder is different in English and Spanish

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Open the app
Open Workspaces
Open any workspace
Open connect bank account
Click on connect manually
Fill in details in step 1, eg details: routing number= 210021002 , acc no = 1234
Check the placeholder for phone number field in step 2
Change the language to spanish (steps: settings->preferences->Language)
Repeat step 1 to 7 and observe placeholder for phone number field in step 2

Expected Result:
Placeholder for phone number in spanish should either be kept same as there is no words in english placeholder [placeholder in english: (xxx)xxx-xxxx] or placeholder in spanish should provide more information about phone number format as it does in english (Also we are telling users to add prefix + number in spanish but that field does not accept prefix)
Actual Result:
Placeholder in spanish asks user to provide prefix + number even though we don't support prefix in that field and it also does not specify the number format as it does in english
Workaround:
unknown
Platforms:

Which of our officially supported platforms is this issue occurring on?

[x] Android / native
[x] Android / Chrome
[x] iOS / native
[x] iOS / Safari
[x] MacOS / Chrome / Safari
[x] MacOS / Desktop

Version Number: 1.2.88-0
**Reproducible in staging?:**y
Reproducible in production?: y
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos:
https://user-images.githubusercontent.com/43996225/227048328-b0224f5d-81f8-4717-b3cf-d35eabc6791c.mp4
https://user-images.githubusercontent.com/43996225/227048589-e6684434-b391-4f2b-8cfa-1b674f9f4e81.mp4
Expensify/Expensify Issue URL:
Issue reported by: @dhanashree-sawant
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1679503411239369
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0153e7a1a0f24ef81a
Upwork Job ID:<PHONE_NUMBER>291146752
Last Price Increase: 2023-03-28

Bug0 Triage Checklist (Main S/O)

[ ] This "bug" occurs on a supported platform (ensure Platforms in OP are ✅)
[ ] This bug is not a duplicate report (check E/App issues and #expensify-bugs)

If it is, comment with a link to the original report, close the issue and add any novel details to the original issue instead

[ ] This bug is reproducible using the reproduction steps in the OP. S/O

If the reproduction steps are clear and you're unable to reproduce the bug, check with the reporter and QA first, then close the issue.
If the reproduction steps aren't clear and you determine the correct steps, please update the OP.

[ ] This issue is filled out as thoroughly and clearly as possible

Pay special attention to the title, results, platforms where the bug occurs, and if the bug happens on staging/production.

[ ] I have reviewed and subscribed to the linked Slack conversation to ensure Slack/Github stay in sync

Job added to Upwork: https://www.upwork.com/jobs/~0153e7a1a0f24ef81a

Current assignee @garrettmknight is eligible for the External assigner, not assigning anyone new.

Triggered auto assignment to Contributor-plus team member for initial proposal review - @parasharrajat (External)

Proposal
Please re-state the problem that we are trying to solve in this issue.
Connect back account: Phone number format in placeholder is different in English and Spanish
What is the root cause of that problem?
Within es.js value for phoneNumberPlaceholder  is set wrong as shown below, so this is the root cause.
https://github.com/Expensify/App/blob/f79a0bade071683ffd921573f4df118191a2b5d5/src/languages/es.js#L35
In english it is correct format.
https://github.com/Expensify/App/blob/f79a0bade071683ffd921573f4df118191a2b5d5/src/languages/en.js#L36
What changes do you think we should make in order to solve the problem?
Within es.js we  have to update phoneNumberPlaceholder  value as shown below, It will solve the problem.
common: {
  ...
  phoneNumberPlaceholder: '(xxx)xxx-xxxx',
  ...
}

What alternative solutions did you explore? (Optional)
None

Triggered auto assignment to @puneetlath (External), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

Proposal
Please re-state the problem that we are trying to solve in this issue.
Connect back account: Phone number format in placeholder is different in English and Spanish
What is the root cause of that problem?
This issue came from this PR
we set placeholder of phone number format different for 2 languages
English: https://github.com/Expensify/App/blob/fb90cd85c60b48d83c7b09aa911900600b23224e/src/languages/en.js#L36
Spainese: https://github.com/Expensify/App/blob/fb90cd85c60b48d83c7b09aa911900600b23224e/src/languages/es.js#L35
What changes do you think we should make in order to solve the problem?
We can update the phoneNumberPlaceholder field in the es file into (xxx)xxx-xxxx like in the en file or update the phoneNumberPlaceholder field in the en file into (prefix) + (number)
What alternative solutions did you explore? (Optional)
NA

Open for business!

📣 @daraksha-dk You have been assigned to this job by @garrettmknight!
Please apply to this job in Upwork and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑💻
Keep in mind: Code of Conduct | Contributing 📖

@puneetlath, @garrettmknight, @parasharrajat, @daraksha-dk Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

Huh, looks like it was deployed to production this week, but Melv missed it.

Still awaiting payment - will review tomo

@garrettmknight looks like this one is ready to be paid.

@parasharrajat offer out to you.
@dhanashree-sawant @daraksha-dk you've been paid!

@parasharrajat is paid - closing.


Request setGlobal not works

I am trying to perform a test where I pass some data via post, but when using the setGlobal function it does not enter the data to the request, it has tried to debug with $request->getBody() and indeed it does not return anything to me
public function testStoreInvalidData(){
        helper('text');
        $request = Services::request();
        $request->setMethod('post');
	$request->setGlobal('post',[
            'concept' => 'testcoupon',
            'type' => 'FS',
            'code' => random_string('alnum', 8),
            'amount' => random_string('numeric',3),
            'usage_limit' => 99999,
            'expiration' => date('Y-m-d H:i:s')
        ]);

	$result = $this->withRequest($request)
         		    ->controller(Coupons::class)
                            ->execute('store');

        $this->assertEquals(400, $result->response()->getStatusCode());
}

the test should fail because all the data is correct.
greetings!

I've been digging into the code and it works correctly, the problem happens I'm applying the validations with the $validation->withRequest($this->request->run()) method and the $request->setGlobal('method post ', [...])only adds the data to the $_POST method not to $_REQUEST, this causes the withRequest method to have no data to validate.
It works perfectly if we handle in the test setGlobal('request', [...]) otherwise if we want to use post in the controller we should validate data with $validation->run($this->request->getPost())
I leave this information in case someone gets confused at any time.
regards!


Dataset Conversion

Is there a script to convert this dataset to COCO?

We have not developed a script to convert into COCO. You might want to look at something like https://github.com/yukkyo/voc2coco/blob/master/voc2coco.py and modify as necessary for the conversion.


Delete database instance if one exists already

Start trying to any leftover database instances before creating a new one.  If the script failed before and didn't delete its old database instance, then it would be impossible to run the migration again without manually deleting it.

Also upgrades bcrypt to get rid of a warning message.

Did we ever work out why it sometimes fails to clean up instances in the first place?
This looks like it will do the job, but it might be better to ensure that old ones aren't left about at the end of a run rather than having to tidy up at the start of a subsequent run?

Yeah, I was thinking that too, but it felt like an odd thing to try and do.  Basically, the failures I've seen have been odd exceptions being thrown at different parts of the process, so it's hard to know where to expect them to come from.
Having said that, maybe we could surround everything in the migratedata method with a big try/finally:, where we do all the cleanup.
Do you think that's a better approach, Kev?

I'd like to hear from @allait on this, but to me it feels nicer to tidy up at the end of a run rather than the beginning of a new one.

Alexey agrees with you, by the way.  But now we're working on saving the database dump somewhere else so that we can give developers access to it.


LG Webos change channel and programs information

Description:
Update of webos component to show information about current channel, program, program duration, app. Added option to change channel.
Example service call
service: media_player.play_media
entity_id: media_player.living_room_tv
data:
  media_content_id: 7_165_52_0_3_8_213
  media_content_type: channel

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

[X] Local tests with tox run successfully. Your PR cannot be merged unless tests pass

Closing this as it is being continued in another PR.


Add zsh completion for 'docker run --storage-opt size='

Ref: #23780
@albers thx for the ping
@vincentbernat
Signed-off-by: Steve Durrheimer<EMAIL_ADDRESS>

LGTM

LGTM 🐮


Adding nuclio component doc

Adding nuclio component documentation and examples

This change is

/assign @jlewi

I wonder if we shouldn't create a separate section named something like "Integrating with other systems" or use cases?
We're starting to have more docs aimed at explaining how Kubeflow can connect with other systems and it might be worth distinguishing between docs describing a particular application and docs explaining how applications that play together.
This doc is really about how to connect Kubeflow Jupyter and (maybe later pipelines) with nuclio.
So this document isn't really describing an independent component or application. The applications would be

jupyter
pipelines
nuclio

The main docs for the latter live elsewhere.
/cc @sarahmaddox

I wonder if we shouldn't create a separate section named something like "Integrating with other systems" or use cases?
We're starting to have more docs aimed at explaining how Kubeflow can connect with other systems and it might be worth distinguishing between docs describing a particular application and docs explaining how applications that play together.
This doc is really about how to connect Kubeflow Jupyter and (maybe later pipelines) with nuclio.
So this document isn't really describing an independent component or application. The applications would be

jupyter
pipelines
nuclio

The main docs for the latter live elsewhere.
/cc @sarahmaddox

right, i do assume that each app has its own web site w details about it and its usage, so this section is sort of an overview + how it plugs into the overall story.
BTW i added example of how it integrate w the nuclio new pipelines components + a link to nuclio pipelines md doc.

/lgtm
/approve


Dedicated server load game corrupted

Every time I load game in dedicated locations fail to load - all missions say unknown and map in debug is empty.
No errors in log.
Really not sure what can be going wrong.

Hey @SomethingSimpl , I have set up github for my own version. We could share some ideas ,and I could add you as a contributor. Contact me at discord: Kortonki#7057


Circleci for endpoint

This PR adds a circleCI step for extract_endpoint 🎉  I think with this step the folder should be a good starting point for @sastels to start adding his code
To-do: An extract_endpoint_integration step will eventually need to be added but I'm not sure what that looks like yet

 Build endpoint to receive extracted data


Leave default as None in mixed_precision for launch command

The default in mixed_precision for the launch command should be None, otherwise the config value is not used. Thanks @nbroad1881 for reporting the bug!

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


Prom oom

Type of PR

[x] Documentation changes
[x] Code changes
[ ] Test changes
[ ] CI-CD changes
[ ] GitHub Template changes

PR Checklist

[x] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.
[x] My code follows the code style of this project.
[ ] I ran lint checks locally prior to submission.
[ ] Have you checked to ensure there aren't other open Pull Requests for the same update/change?

Purpose of PR
Modify scrape targets to only include NGSA and LR apps, since these metrics are currently used in Grafana
Does this introduce a breaking change

[ ] YES
[x] NO

Issues Closed or Referenced

Closes https://github.com/retaildevcrews/wcnp/issues/686

I noticed the 3d got removed, any thoughts of keeping it to maybe t 7 days? This is to prevent failures just in case ngsa metrics grow quickly with all the LR testing we are doing.

I noticed the 3d got removed, any thoughts of keeping it to maybe  7 days? To prevent failures just in case ngsa metrics grow quickly with all the LR testing we are doing.


Issue with awareness channel implementation (getting unexpected chars/buffer from decoder)

hi there, as a follow up on my work with yjs websocket lib I have referred this implementation.
I have observed that my update buffer in handler consists of unexpected tokens
{�ӓ�s{"name":"Shivam","color":"rgba(224, 17, 117, 1)","alphaColor":"rgba(224, 17, 117, 0.2)","anchor":null,"focus":null}
    at Object.create (/code/node_modules/lib0/dist/error-8582d695.cjs:14:21)
    at Object.<anonymous> (/code/node_modules/lib0/dist/buffer-c2f560d5.cjs:927:41)
    at Module._compile (node:internal/modules/cjs/loader:1191:14)
    at Object.Module._extensions..js (node:internal/modules/cjs/loader:1245:10)
    at Module.load (node:internal/modules/cjs/loader:1069:32)
    at Function.Module._load (node:internal/modules/cjs/loader:904:12)
    at Module.require (node:internal/modules/cjs/loader:1093:19)
    at require (node:internal/modules/cjs/helpers:108:18)
    at Object.<anonymous> (/code/node_modules/lib0/dist/encoding.cjs:5:16)
    at Module._compile (node:internal/modules/cjs/loader:1191:14)

Also I have observed this happens when this line is called: https://github.com/kapv89/yjs-scalable-ws-backend/blob/main/src/setupWSConnection.ts#L139
My thoughts, would be as I am testing on local my published content is directly subsribed and some OOM or buffer noise is there.

hi .. trying to identify the issue ... are you running the crdt websocket server in a horizontally scaled manner such that there are multiple instances of the server runnning?

hi @kapv89 I was able to fix this issue so will close this.
But I have few issues on hand on which I would like to discuss with you regarding scaling this yjs websocket extension


Handle ErrorFolderExists for contact restore folders

Handle 409, ErrorFolderExists error in restore path while creating destination contact folder. We need to fix this for all m365 services. Starting with contacts to get a buy in on the fix. Will add integration tests + similar fixes for rest of exchange, onedrive, etc. in later PRs.
Context:
CreateContactFolder() may fail with ErrorFolderExists under certain error conditions. For e.g. consider below scenario.

CreateContactFolder() does a POST to graph to create restore destination folder but this fails with 5xx.
It's possible that step 1 may have left some dirty state in graph. For e.g. it's possible that the destination folder in step 1 was actually created, but 5xx was returned due to other reasons.
So when we reattempt POST in such a scenario, we sometimes observe ErrorFolderExists error .
Corso should be resilient to such errors. To fix this, when we encounter such an error, we will do a GET to fetch the restore destination folder and add it to folder cache.

Does this PR need a docs update or release note?

[ ] :white_check_mark: Yes, it's included
[ ] :clock1: Yes, but in a later PR
[x] :no_entry: No

Type of change

[ ] :sunflower: Feature
[x] :bug: Bugfix
[ ] :world_map: Documentation
[ ] :robot: Supportability/Tests
[ ] :computer: CI/Deployment
[ ] :broom: Tech Debt/Cleanup

Issue(s)

#

Test Plan

[x] :muscle: Manual
[ ] :zap: Unit test
[ ] :green_heart: E2E

not needed. closing.


How to start manual backup after to create policy

Hi, how to start manual backup after to create policy? in the API documentation I can not find the procedure.

Is this really still a valid PR? Was it ever a valid PR? With over 100 files changes over two years, I think this PR is way too big to accept anymore. If there's content from it that's still desired, then please clean up the branch and re-open the PR or submit a new one.


Offline Heartbeat fails

Expected behavior (what should have happened):
$ go test -tags=integration ./main_test.go
ok  	command-line-arguments	1.040s

Actual behavior (what went wrong):
$ go test -tags=integration ./main_test.go
...
./build/wakatime-cli-linux-amd64 --log-file /tmp/TestSendHeartbeats_ExtraHeartbeats3249327417/002/1660507848 --api-url http://<IP_ADDRESS>:35321 --key 00000000-0000-4000-8000-000000000000 --config /tmp/TestSendHeartbeats_ExtraHeartbeats3249327417/001/wakatime.cfg244993821 --entity testdata/main.go --extra-heartbeats true --sync-offline-activity none --cursorpos 12 --offline-queue-file /tmp/TestSendHeartbeats_ExtraHeartbeats3249327417/001/2635607406 --lineno 42 --lines-in-file 100 --time<PHONE_NUMBER> --hide-branch-names .* --write --verbose
...
--- FAIL: TestSendHeartbeats_ExtraHeartbeats (0.12s)
    main_test.go:264:
        	Error Trace:	main_test.go:264
        	Error:      	Not equal:
        	           	expected: 2
        	           	actual  : 1
        	Test:       	TestSendHeartbeats_ExtraHeartbeats
...
FAIL
FAIL	command-line-arguments	1.026s
FAIL

Environment:

OS: linux
Platform: amd64
Details: Arch Linux x86_64, Go 1.18.2, GCC 12.1.0

Logs:
wakatime.log

Our pipeline runs integration tests for macOS, Windows and Linux and it hasn't been failing. Have you tried running more than once? Also try running make test-integration.

TestSendHeartbeats_ExtraHeartbeats is one of the flakey tests that sometimes fails and we're not sure why.

I'm shipping this for Arch Linux and running your Makefile directly has some side-effect which I'd like to avoid. This is why I'm running it manually to reproduce the build and test configs for Arch. But I may be doing something wrong entirely. You can see the build() and check() functions which compile from source and then test the binary here:
build() {
  cd "${srcdir}/${pkgname}-cli"

  mkdir -p build  # create build dir

  export CGO_CPPFLAGS="${CPPFLAGS}"
  export CGO_CFLAGS="${CFLAGS}"
  export CGO_CXXFLAGS="${CXXFLAGS}"
  export CGO_LDFLAGS="${LDFLAGS}"

  local _date="$(date -u '+%Y-%m-%dT%H:%M:%S %Z')"
  local _commit="$(git rev-parse --short HEAD)"

  CGO_ENABLED="0" go build -o build/"${_binname}" \
    -trimpath \
    -buildmode=pie \
    -mod=readonly \
    -modcacherw \
    -ldflags "-linkmode external -extldflags '${LDFLAGS}' -X '${_repo}/pkg/version.OS=linux' -X '${_repo}/pkg/version.Arch=amd64' -X '${_repo}/pkg/version.BuildDate=${_date}' -X '${_repo}/pkg/version.Commit=${_commit}' -X '${_repo}/pkg/version.Version=${pkgver}'"
}

check () {
  cd "${srcdir}/${pkgname}-cli"
  echo "wakatime-cli" > ./.wakatime-project
  # some weird test assumptions fail
  sed \
    -e "s/<local-build>/${pkgver}/" \
    -e "s/Equal(t, 2, offlineCount)/Equal(t, 1, offlineCount)/" \
    -i ./main_test.go
  CGO_ENABLED="0" go test -tags=integration ./main_test.go
}

Something that has always been a problem is for the test is that for some reason it always thinks it's a local build and seems to ignore me setting the version number and also the heartbeat count. Any tips or help will be appreciated. Thanks.

If it helps you, I can say that for version 1.42.2 the heartbeat test worked correctly.


Add tests for FastNoiseLite/NoiseTexture

This PR implements a test suite for the noise module while fixing some minor bugs (primarily with seamless noise generation).
Detailed changes:

Implement a test suite for FastNoiseLite and NoiseTexture2D

There are some macros and utility function which might be worth to be extracted in test_utils.h or test_macros.h (and generalized), but because they are currently just needed by these tests I left them in test_fastnoise_lite.h

Stabilize seamless noise texture generation

Fix artifacts with small resolutions combined with a small blend skirt value, e.g. 1x32, 16x16 pixels
Fix #61044
Remove the provisional restriction of 0.05 for the blend skirt (now it can be set as low as 0)

There are still some small structural quirks in the noise module (e.g. the private method _generate_texture shouldn't be exposed and rather replaced with another solution, because right know it just returns the generated image without setting it internally so it is generated twice). But that's for another PR.

-ADD_PROPERTY(PropertyInfo(Variant::FLOAT, "seamless_blend_skirt", PROPERTY_HINT_RANGE, "0.05,1,0.001"), "set_seamless_blend_skirt", "get_seamless_blend_skirt");
+ADD_PROPERTY(PropertyInfo(Variant::FLOAT, "seamless_blend_skirt", PROPERTY_HINT_RANGE, "0,1,0.001"), "set_seamless_blend_skirt", "get_seamless_blend_skirt");

Did you verify that values between 0 and 0.05 do something and provide a useful range rather than dead space on the slider? This looks like something I would have cut off if it was useless. But it was a long time ago.

Thanks!


Add support for local timezone on ISO8601 output

To be able to provide a appropriate time representation the fix
conversion to UTC is removed. time.Time holds the information about the timezone corresponding to the actual object and this timezone should be used for jsonapi representation.
Also the time.RFC3339 format is used instead of the custom format constant.

I signed it!

@aren55555 yeah, the main goal of this PR is to allow a correct serialization/deserialization of the timezone information.
According to the date formatting:
I've tried to modify the iso8601TimeFormat constant to allow the serialization/deserialization of the timezone information and the updated constant defines the exact same format as time.RFC3339. Therefore I didn't see the need for a custom constant any more.
But if you favour the custom constant, it is absolutely no problem to use an updated version of it:
iso8601TimeFormat = "2006-01-02T15:04:05Z07:00"
What do you think?

I'm pretty sure time.Time implements the json.Marshaler and json.Unmarshaler interfaces with time.RFC3339... meaning if you just remove all this it should just end up in that format.

@aren55555 Parsing the time.Time values is done manually by calling time.Parse (needs layout information).
In the response part I tried to remove the Format call but then my added test case breaks. So I can confirm that the code in this PR is required to implement the expected behavior.

I ran into the need for this as well, forcing times to UTC was unexpected.


angular file upload as ng-include in another module

HI
I have angular-file-upload in a module by itself
then I include it in other modules using ng-include like this

my upload.hrml looks like this

            {{uploadMessage}}

File Drag/Drop is not supported for this browser

when I drop any files nothing gets dropped, and invalidFiles gives me
TypeError: $parse(...).assign is not a function
Any suggestions?

create a jsfiddle

https://plnkr.co/edit/W9Mct0dYulmI1FlXKfIu

That's a $scope.$watch() issue. If you add double curly bracket in your html for files and invalidFiles you will see that they have the expected values.
You can post it on SO for help since it is a general angular scope question.

these are models
I don't think I should add double curly for them
Thanks anyway

This is what i meant: https://plnkr.co/edit/lMlcn8soJ4n6YoOGnFDs?p=preview

Got you
now if you try to drop a JPG file, it shows the files empty array and the invalidFiles empty also.
so why do you think it is not picking it up?
Thanks for your help

files isn't empty. it is an object it just doesn't print it's properties, try files[0].name


Reverse interactive mode test in __run_stack

Walter said;

you do not want to find value blocks in non-interactive mode

See also
https://github.com/sugarlabs/activity-turtle-confusion/pull/4/commits/ebd529bcb4632fe0a6a8191f9587e1dab892d3e6
https://github.com/sugarlabs/activity-turtle-confusion/pull/4
Reported-by: Yash Agrawal<EMAIL_ADDRESS>Signed-off-by: James Cameron<EMAIL_ADDRESS>

@walterbender, please review?  I'm holding this in my branch of Confusion and it keeps coming up as I compare with @yashagrawal3's branch of Confusion.

I trigger the above error by running in slow mode (snail) with the blocks visible.

'k, thanks.  I've no idea.  I'll pull back.


[bug]: Outpainting dont work

Is there an existing issue for this?

[X] I have searched the existing issues

OS
Windows
GPU
amd
VRAM
16gb
What version did you experience this issue on?
2.3.2
What happened?
When I try to expand the image it draws me some random patterns
Screenshots

Additional context
No response
Contact Details
No response

What I see:

CFG may be too low
Seed is locked
I don't see prompt. Errors in prompt syntax may cause this.


Improve lua report generation

One more thing:

[ ] Use engine(lua) instead of report for the commit prefixes.

Looks great! As soon as the check completes, I'll merge.


[WIP] Extensible admission controller

The text is not readable yet. I opened this PR just to reserve a slot for us in 1.7 docs.
cc @smarterclayton @lavalamp @whitlockjc

This change is

@caesarxuchao You will also need to add an entry for this doc in the Reference table of contents, /_data/reference.yml, probably after - docs/admin/admission-controllers.md (line 14).
Ping me when this is ready for review. Cheers

@caesarxuchao Do you have an ETA for when this will be ready for review?

@abiogenesis-now Can you take a look at this? Thx

FYI, all feedback must be addressed and LGTMs given by EOD Tue, June 27th so that this can be merged for the 1.7 release on June 28th.

@caesarxuchao are the folks cc'ed in your descriptions going to be the tech reviewers?

@caesarxuchao BTW, you still need to add an entry for this doc in the Reference table of contents, /_data/reference.yml.

Somehow i can't assign @whitlockjc as reviewer.

@chenopis i'll fix the enty.

I'm not a committer.  :)  But I will gladly review.  As for the webhook stuff, I just got to a point where I can put some time into that if it's not too late.

@whitlockjc Yeah, if you can get that in by the end of day today, that would be great. Also, this still needs a tech review. Thx

@abiogenesis-now Were your issues addressed? Is this ready for a docs lgtm?

taking a look now

@caesarxuchao are there meant to be TODOs in the documentation?

are there meant to be TODOs in the documentation?

No, they should be all filled out.

Did anyone push the changes up to GitHub? I haven't seen any commits since yesterday.

@caesarxuchao What is the status of this PR? It doesn't look complete yet. Also, I think it needs to be broken up into separate Concept and Task docs.

@chenopis sorry for the delay. I talked with @lavalamp and he's ok with iterating the docs after the release, so the doc team can review it now.
I'll add the reference entry.
cc @whitlockjc @smarterclayton

@caesarxuchao Ok, no problem. Thanks!

I'm going to merge this and make some changes in a follow-up PR.


Fix incorrect error assumption in sendMessage

Hello. If I use api.sendMessage without the last argument (reply message id), I get MessageID should be of type string and not String. So I fixed it in this PR.

Thank you for the fix and sorry for that bug i made.
Verified that it works.
@Schmavery

thanks


fix inputs hydration error & update EtherInput to take in .number values

Demo :
https://github.com/scaffold-eth/se-2-challenges/assets/80153681/17b0fbb2-dd2e-41e6-837e-77a9f49e8ad1
Fixes #44

My Contract solution if anyone wants refer :
        pragma solidity >=0.8.0 <0.9.0;
        // SPDX-License-Identifier: MIT

import<EMAIL_ADDRESS>import<EMAIL_ADDRESS>
/**
 * @title DEX
 * @author stevepham.eth and m00npapi.eth
 * @notice this is a single token pair reserves DEX, ref: "Scaffold-ETH Challenge 4" as per https://speedrunethereum.com/, README.md and full branch (front-end) made with lots of inspiration from pre-existing example repos in scaffold-eth organization.
 */
contract DEX {
    /* ========== GLOBAL VARIABLES ========== */

    uint256 public totalLiquidity; //total amount of liquidity provider tokens (LPTs) minted (NOTE: that LPT "price" is tied to the ratio, and thus price of the assets within this AMM)
    mapping(address => uint256) public liquidity; //liquidity of each depositor
    using SafeMath for uint256; //outlines use of SafeMath for uint256 variables
    IERC20 token; //instantiates the imported contract

    /* ========== EVENTS ========== */

    /**
     * @notice Emitted when ethToToken() swap transacted
     */
    event EthToTokenSwap(
        address swapper,
        string txDetails,
        uint256 ethInput,
        uint256 tokenOutput
    );

    /**
     * @notice Emitted when tokenToEth() swap transacted
     */
    event TokenToEthSwap(
        address swapper,
        string txDetails,
        uint256 tokensInput,
        uint256 ethOutput
    );

    /**
     * @notice Emitted when liquidity provided to DEX and mints LPTs.
     */
    event LiquidityProvided(
        address liquidityProvider,
        uint256 tokensInput,
        uint256 ethInput,
        uint256 liquidityMinted
    );

    /**
     * @notice Emitted when liquidity removed from DEX and decreases LPT count within DEX.
     */
    event LiquidityRemoved(
        address liquidityRemover,
        uint256 tokensOutput,
        uint256 ethOutput,
        uint256 liquidityWithdrawn
    );

    /* ========== CONSTRUCTOR ========== */

    constructor(address token_addr) public {
        token = IERC20(token_addr); //specifies the token address that will hook into the interface and be used through the variable 'token'
    }

    /* ========== MUTATIVE FUNCTIONS ========== */

    /**
     * @notice initializes amount of tokens that will be transferred to the DEX itself from the erc20 contract mintee (and only them based on how Balloons.sol is written). Loads contract up with both ETH and Balloons.
     * @param tokens amount to be transferred to DEX
     * @return totalLiquidity is the number of LPTs minting as a result of deposits made to DEX contract
     * NOTE: since ratio is 1:1, this is fine to initialize the totalLiquidity (wrt to balloons) as equal to eth balance of contract.
     */
    function init(uint256 tokens) public payable returns (uint256) {
        require(totalLiquidity == 0, "DEX: init - already has liquidity");
        totalLiquidity = address(this).balance;
        liquidity[msg.sender] = totalLiquidity;
        require(
            token.transferFrom(msg.sender, address(this), tokens),
            "DEX: init - transfer did not transact"
        );
        return totalLiquidity;
    }

    /**
     * @notice returns yOutput, or yDelta for xInput (or xDelta)
     */
    function price(
        uint256 xInput,
        uint256 xReserves,
        uint256 yReserves
    ) public view returns (uint256 yOutput) {
        uint256 xInputWithFee = xInput.mul(997);
        uint256 numerator = xInputWithFee.mul(yReserves);
        uint256 denominator = (xReserves.mul(1000)).add(xInputWithFee);
        return (numerator / denominator);
    }

    /**
     * @notice returns liquidity for a user. Note this is notneeded typically due to the `liquidity()` mapping variable being public and having a getter as a result. This is left though as it is used within the front end code (App.jsx).
     */
    function getLiquidity(address lp) public view returns (uint256) {
        return liquidity[lp];
    }

    /**
     * @notice sends Ether to DEX in exchange for $BAL
     */
    function ethToToken() public payable returns (uint256 tokenOutput) {
        require(msg.value > 0, "cannot swap 0 ETH");
        uint256 ethReserve = address(this).balance.sub(msg.value);
        uint256 token_reserve = token.balanceOf(address(this));
        uint256 tokenOutput = price(msg.value, ethReserve, token_reserve);

        require(
            token.transfer(msg.sender, tokenOutput),
            "ethToToken(): reverted swap."
        );
        emit EthToTokenSwap(
            msg.sender,
            "Eth to Balloons",
            msg.value,
            tokenOutput
        );
        return tokenOutput;
    }

    /**
     * @notice sends $BAL tokens to DEX in exchange for Ether
     */
    function tokenToEth(uint256 tokenInput) public returns (uint256 ethOutput) {
        require(tokenInput > 0, "cannot swap 0 tokens");
        uint256 token_reserve = token.balanceOf(address(this));
        uint256 ethOutput = price(
            tokenInput,
            token_reserve,
            address(this).balance
        );
        require(
            token.transferFrom(msg.sender, address(this), tokenInput),
            "tokenToEth(): reverted swap."
        );
        (bool sent, ) = msg.sender.call{value: ethOutput}("");
        require(sent, "tokenToEth: revert in transferring eth to you!");
        emit TokenToEthSwap(
            msg.sender,
            "Balloons to ETH",
            ethOutput,
            tokenInput
        );
        return ethOutput;
    }

    /**
     * @notice allows deposits of $BAL and $ETH to liquidity pool
     * NOTE: parameter is the msg.value sent with this function call. That amount is used to determine the amount of $BAL needed as well and taken from the depositor.
     * NOTE: user has to make sure to give DEX approval to spend their tokens on their behalf by calling approve function prior to this function call.
     * NOTE: Equal parts of both assets will be removed from the user's wallet with respect to the price outlined by the AMM.
     */
    function deposit() public payable returns (uint256 tokensDeposited) {
        require(msg.value > 0, "Must send value when depositing");
        uint256 ethReserve = address(this).balance.sub(msg.value);
        uint256 tokenReserve = token.balanceOf(address(this));
        uint256 tokenDeposit;

        tokenDeposit = (msg.value.mul(tokenReserve) / ethReserve).add(1);
        uint256 liquidityMinted = msg.value.mul(totalLiquidity) / ethReserve;
        liquidity[msg.sender] = liquidity[msg.sender].add(liquidityMinted);
        totalLiquidity = totalLiquidity.add(liquidityMinted);

        require(token.transferFrom(msg.sender, address(this), tokenDeposit));
        emit LiquidityProvided(
            msg.sender,
            liquidityMinted,
            msg.value,
            tokenDeposit
        );
        return tokenDeposit;
    }

    /**
     * @notice allows withdrawal of $BAL and $ETH from liquidity pool
     * NOTE: with this current code, the msg caller could end up getting very little back if the liquidity is super low in the pool. I guess they could see that with the UI.
     */
    function withdraw(
        uint256 amount
    ) public returns (uint256 eth_amount, uint256 token_amount) {
        require(
            liquidity[msg.sender] >= amount,
            "withdraw: sender does not have enough liquidity to withdraw."
        );
        uint256 ethReserve = address(this).balance;
        uint256 tokenReserve = token.balanceOf(address(this));
        uint256 ethWithdrawn;

        ethWithdrawn = amount.mul(ethReserve) / totalLiquidity;

        uint256 tokenAmount = amount.mul(tokenReserve) / totalLiquidity;
        liquidity[msg.sender] = liquidity[msg.sender].sub(amount);
        totalLiquidity = totalLiquidity.sub(amount);
        (bool sent, ) = payable(msg.sender).call{value: ethWithdrawn}("");
        require(sent, "withdraw(): revert in transferring eth to you!");
        require(token.transfer(msg.sender, tokenAmount));
        emit LiquidityRemoved(msg.sender, amount, ethWithdrawn, tokenAmount);
        return (ethWithdrawn, tokenAmount);
    }
}

hey @technophile-04 ! Gj! But I have some questions

why to allow to click "Send" when we already show that input value is wrong (red border)? I think we need to disable buttons.
is "." a wrong value? I think it should be equal to 0, and hence convertible. What do you think?

and it's not a hydration error

Tysm Rinnat for review 🙌

why to allow to click "Send" when we already show that input value is wrong (red border)? I think we need to disable buttons.

Umm not sure about I just wanted to show developers the error message lol so that they can learn, also one thing, I was also wondering weather instead of taking input in ether unit we should take it in wei unit (The input will have * which developer need to hit manualy to multiply by 10^18 ) again for developers learnings
Ideally we should disable "Send" btn and take input in ether unit in production app, but just wanted to keep it raw similar to SE-1 challenge so that people can learn
But I am fine with anything 🙌 cc @ZakGriffith and @Pabl0cks for their thoughts

is "." a wrong value? I think it should be equal to 0, and hence convertible. What do you think?

Umm in SE-2 main branch we are showing error for ".", I think it will be easy to solve just handling only "." case separately but if its get too messier I would probably leave

Good job @technophile-04 and nice suggestions @rin-st !!  🙌

why to allow to click "Send" when we already show that input value is wrong (red border)? I think we need to disable buttons.

Reading you both, after changing my mind a couple of times 🤯, I see this could be an interesting approach, let me know what do you think:

I think is good to let users input ether unit in frontend and wei in Debug contracts. That way we teach them good practices in how to build for the end user, but having in mind the base unit to work with in contracts is wei.
Disabling send button is the best practice to show them in order to build production apps, but I think experiencing the transaction error it's also part of the learning experience.
How about explaining in the README, maybe in a "Side Quest", to ask them what happend when they input a non numeric or negative number, and asking them to prevent it in the frontend by disabling "Send button", or even adding more evaluations in the contract level to give more accurate error messages on each case?

is "." a wrong value? I think it should be equal to 0, and hence convertible. What do you think?

Maybe this is a bit edge case, I think we could assume that if an user writes a "." he is going to input a decimal number, and if he didn't write any number after "." is an error / he forgot to do it? If so I would probably just leave it like that.

Thanks for quick answers!

I was also wondering weather instead of taking input in ether unit we should take it in wei unit (The input will have * which developer need to hit manualy to multiply by 10^18 ) again for developers learnings

If we return it to IntegerInput, we need to add it to EtherInput too, for consistency. Because I think it's strange to input value in Eth to EtherInput, but in wei to IntegerInput below it, especially when values are equivalent at the start and you have a chart comparing that values.

Ideally we should disable "Send" btn and take input in ether unit in production app, but just wanted to keep it raw similar to SE-1 challenge so that people can learn

But it was in Eth in SE-1, or am I wrong?

Umm in SE-2 main branch we are showing error for ".", I think it will be easy to solve just handling only "." case separately but if its get too messier I would probably leave

Yep, let's leave it for now

@Pabl0cks good points, I agree!

I think is good to let users input ether unit in frontend and wei in Debug contracts. That way we teach them good practices in how to build for the end user, but having in mind the base unit to work with in contracts is wei.

I agree with this.  Best to keep the challenge front ends using ether.  Throughout the challenges, we do walk them through using the debug tab while converting to wei, and the front ends using ether.
I also agree with keeping the send buttons available, even when it is an obvious error.  Let them send it to get a feel for how these errors look.

is "." a wrong value? I think it should be equal to 0, and hence convertible. What do you think?

I don't have an opinion on this one, either way is okay.
This looks really good btw!  Very smooth.

I think is good to let users input ether unit in frontend and wei in Debug contracts.

We return it to IntegerInput, we need to add it to EtherInput too, for consistency.

I agree with this. Best to keep the challenge front ends using ether. Throughout the challenges, we do walk them through using the debug tab while converting to wei,

I agree with you all, really appreciate it !! Tysm all 🙌

[X] User will entering the amount in ether uint on dex UI
[X] Keeping the "Send" btn enabled to show the error directly on UI

I think both are handled already so merging this 🙌. Feel free to create more new issues around it, thanks again all !!


Channels support

Updated tools to build images and run tests with the local maven repo
Removed all feature-packs versions from test applications.
Updates test and features applications to rely on org.jboss.eap.channels:eap-8.0-beta (tha tcontains eap-datasources and eap-cloud feature-packs

@jmesnil , here we are ;-). All tests are run with channels.


feat(rate-limiting) set TTL when incrementing counters

Summary
The "local" rate-limiting policy stores its counters in a shared-memory dictionary (shdict).
This PR modifies that behaviour so the counters now expire after one period length (e.g. if the counter is for requests in the current minute, the TTL is set to one minute).
There's a little more detail in my forum post.
Full changelog

update code in the local policy's increment method to pass the init_ttl parameter to shm:incr
add a plugin test for this behaviour
update the dummy shm implementation in globalpatches to support the ttl method

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.


kanata: 1.1.0 -> 1.2.0

Description of changes
https://github.com/jtroo/kanata/releases/tag/v1.2.0

Things done

Built on platform(s)

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

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

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[X] Fits CONTRIBUTING.md.

tests failing
       > failures:
       >     cfg::parse_default
       >
       > test result: FAILED. 15 passed; 1 failed; 0 ignored; 0 measured; 0 filtered out; finished in 0.07s

@NickCao Could you provide some error messages about that failed test?
I cannot reproduce it.  I have built kanata several times and all tests are passed.

See ofborg logs.


Query is reset to "*" after clicking on other controls

Hi guys,
Clicking around Explorer, encountered an issue where query gets reset to * after entering it and then touching some other control afterwards.
To reproduce:

Enter a query, e.g. "racism"
Edit any of the dates in the "For dates"
Query gets reset to "*"

or

Enter a query, e.g. "racism"
Choose collection, e.g. "ABYZ Global English Sources"
Remove the default collection ("US Top News")
Query is reset to "*"

nice find, Linas. I needed to include the updated but unstored edits to the queryForm like this:
const updatedQuery = {
      ...query,
      ..formQuery,
}

in the handling of updates to the various form fields.Due to the complexity of the media handling and dates, we have different methods for each field, but this addresses them all similarly


"Enable PINE, if it is not already (System > Game Settings > Configure PINE)"

Hello I can't find the option specified by the readme.md in my installation

Thanks in advance

Ok, this option was added in pcsx2 1.7.x


Move multi-portal search from code4sa to OpenUp website.

Moves the TRACE page to OpenUp. Next step would be redirecting from code4sa.org/data-desk to this and using this in further comms.
Generally looks fine except for the rounded blue? buttons from the btn class in main.css. Is this intended or should I remove rounding and change the colour to our green?

If it can match the new site's style's, that's ideal.


Lower the gas fee to 1.3 on Tezex

Lower the gas fee to 1.3 on Tezex

This may result in negative profit margins for changemakers.


[Issue #59] Windows 10 Mobile ole32.dll error when playing

Fix errors for native call of PropVariantClear reported in #59 . Also up Windows 10 Demo max tested version to latest Windows 10 SDK (1511).

Note - PropVariantClear for Windows 8 is changed to api-ms-win-core-com-l1-1-0.dll while Windows 8.1 should use api-ms-win-core-com-l1-1-1.dll
Not sure if it's not better to leave the dll import to ole32.dll instead for Windows 8 and implement the change

soory for delay, will hopefully get round to testing this soon. I've got some more UWP fixes of my own I need to merge this with

Thanks :) Shall I return the original "ole32.dll" for Windows 8? And leave the change only for Windows 10?

I have to confess I'm not sure what the best option is. Does [DllImport("api-ms-win-core-com-l1-1-0.dll")] only work on 8 or can it work on 8.1 too? To be honest the UWP is probably the most important thing going forwards, but if it's not too hard to keep the Win8 one working on all platforms then that would be a good idea.

Done. I have reverted Win8 to include the .NET PropVariantNative "ole32.dll" reference. Just to be on the safe side.


ValueError: cannot reshape array of size 0 into shape (0,newaxis,4)

Error faced:
Epoch 00004: val_loss improved from 217.36631 to 104.59965, saving model to ssd300_weights_epoch-04_loss-137.3789_val_loss-104.5996.h5
Epoch 5/700
1/13 [=>............................] - ETA: 17:50 - loss: 114.2628
2/13 [===>..........................] - ETA: 16:58 - loss: 78.6937
3/13 [=====>........................] - ETA: 15:14 - loss: 81.4181Traceback (most recent call last):
File "ssd_training.py", line 243, in
validation_steps=ceil(n_val_samples / batch_size))
File "/usr/local/lib/python3.5/dist-packages/keras/legacy/interfaces.py", line 91, in wrapper
return func(*args, **kwargs)
File "/usr/local/lib/python3.5/dist-packages/keras/engine/training.py", line 2192, in fit_generator
generator_output = next(output_generator)
File "/usr/local/lib/python3.5/dist-packages/keras/utils/data_utils.py", line 793, in get
six.reraise(value.class, value, value.traceback)
File "/usr/local/lib/python3.5/dist-packages/six.py", line 693, in reraise
raise value
File "/usr/local/lib/python3.5/dist-packages/keras/utils/data_utils.py", line 658, in _data_generator_task
generator_output = next(self._generator)
File "/home/ubuntu/SSD/ssd_batch_generator.py", line 909, in generate
batch_y_true = ssd_box_encoder.encode_y(batch_y, diagnostics=False) # Encode the labels into the y_true tensor that the SSD loss function needs.
File "/home/ubuntu/SSD/encode_decode_utils.py", line 858, in encode_y
y_encode_template = self.generate_encode_template(batch_size=len(ground_truth_labels), diagnostics=False)
File "/home/ubuntu/SSD/encode_decode_utils.py", line 803, in generate_encode_template
boxes = np.reshape(boxes, (batch_size, -1, 4))
File "/usr/local/lib/python3.5/dist-packages/numpy/core/fromnumeric.py", line 257, in reshape
return _wrapfunc(a, 'reshape', newshape, order=order)
File "/usr/local/lib/python3.5/dist-packages/numpy/core/fromnumeric.py", line 52, in _wrapfunc
return getattr(obj, method)(*args, **kwds)
ValueError: cannot reshape array of size 0 into shape (0,newaxis,4)`

During the 5th epoch of training, following error has surfaced. Please explain, that in the last line's error 'reshaping of size 0 into shape (0, newaxis, 4)', how does an array of size zero is being passed to be reshaped. Or any other issue..?

My first guess would be that this happens because there is no ground truth box left in the entire batch, which is a situation that the SSDBoxEncoder doesn't account for. What batch size are you using?

Thanks, We have a training set of 380 images, val'n set of 34 images, and batch size of 32 was used.

Is it a possibility that a batch might end up with no ground truth boxes at all?

Each image in my training data has at-least one ground truth box labelled in PASCAL VOC format. That imo should not result in any batch ending up with zero ground truth box.

I'm also getting the same error,
Please reply.

I've updated the data generator, please pull the latest master and try the new version. It's not very likely that this solves your issue, but it might.
In order to narrow down the range of possible causes of the issue, once you pulled the latest master, add print statements to data_generator/object_detection_2d_data_generator.py. Add the line
print("before: ", len(batch_y))
right before this code block and the lines
print("before: ", len(batch_y))
print()

right after it. If the issue keeps occurring, let me know what it prints for the batch where it fails.

Thanks @pierluigiferrari . The updated master is doing perfect training.
Could you explain what was the issue you fixed in that code update?

Great! The fix is a side effect of a quite drastic data generator redesign, so it's hard to say what fixed the issue because we never really figured out what exactly caused the issue. One possible reason is that the newly added SSD data augmentation chain (in case you're using that) makes it impossible for an image to end up with no ground truth boxes at all, so an empty batch becomes impossible. Another possible reason is that I cleaned up a few fragile points in the generator code where some edge cases might have been overlooked previously and now they are handled correctly.
I'll close this issue as it appears to be fixed, let me know if anything else comes up.


Added instructions to start and enable waagent for cent6

Cent7 docs mentioned about starting waagent service using systemctl enable waagent.  Its not mentioned anywhere for cent6. Added the commands to start and enable waagent service in cent6.

@cg-cnu : Thanks for your contribution! The author, @szarkos, has been notified to review your proposed change.

@cg-cnu Sorry for the delay! I'm going to help out by pulling this over into another pull request and fixing the merge conflict. You will still be shown as a contributor for the doc, this just makes it so I can fix the conflict before it gets published.
I'm going to close this pull request because the new one will be the one that is merged with the changes. Thank you for your contribution :)
#please-close  #86425


Update SECURITY.md

Improved the framing of statements
Fixes
Closes #1509
Relates to #1553 & #1554

@abhisar-yadav have you made the changes requested by @CBID2 ?


[SR-16035] 'self' refers to the method 'X.Y.self', which may be unexpected, use 'X.Y.self' to silence this warning

Previous ID
SR-16035

Radar
None

Original Reporter
Gereon (JIRA User)

Type
Bug

Additional Detail from JIRA

Votes
0

Component/s
swift

Labels
Bug

Assignee
None

Priority
Medium

md5: ba77fe5b5033eac72d64559e09eeded9

Issue Description:
I'm in the process of updating some code to use Xcode 13.3 and came across some new warnings that I'm unsure how to remove, and that didn't occur in Xcode 13.2.1.
Here's the gist of the code:
struct Link: Codable { }
let href: URL
{{}}}
struct FooLinks: Codable { }
let `self`: Link // <— this is the culprit
{{}}}
struct Foo: Codable {
let links: FooLinks
{{}}}
The warnings don't show up in the editor, only in the build log, and are as follows:
<unknown>:0: warning: 'self' refers to the method 'Foo.FooLinks.CodingKeys.self', which may be unexpected <unknown>:0: note: use 'Foo.FooLinks.CodingKeys.self' to silence this warning<unknown>:0: warning: 'self' refers to the method 'Foo.FooLinks.self', which may be unexpected <unknown>:0: note: use 'Foo.FooLinks.self' to silence this warning<unknown>:0: warning: 'self' refers to the method 'Foo.FooLinks.self', which may be unexpected <unknown>:0: note: use 'Foo.FooLinks.self' to silence this warning
None of the suggested fixes make any sense to me. Is there a way to get of the warnings without renaming the offending self property?

This has been fixed in Xcode 14.0 beta and can be closed. Thanks!


Added hourly and daily metrics

I've created hourly and daily metrics. This pull request is for 4.0.0 (master) branch.

Support for customizing meters will make it into v4, but I can't force hourly and daily on our users, as I'm already getting lots of complaints about reporting too many metrics.

My client will kill me ;)
When will v4 be available?


Fix tclength parameter name in STC doc

This update docs to match the correct parameter name for "tclength" on STC.
Fixes #634

Fixes #634

@emerzon,
I took care of this on this end.
Thanks for making the contribution.
KJ


Check valid project and stage names

When creating a project, the valid projectname and stage names have to be checked.
Currently, we do not allow any special characters in the names (!, -, ., ...)

Please use a regex to verify the allowed characters and document it.
Please add a test for other characters.
Please document which characters are allowed.


Remove import

https://github.com/mogwai/fastai_audio/blob/e8499b72876292b5013f2382a7eef7317aa674e8/audio/audio.py#L11
I want to discuss removing this line.  I have removed it in my local environment and added the audio folder into the fastai core library and it allows me to do from fastai.audio import *
Just want to discuss what the from audio import * is doing in the current library and see if there is another way to achieve the results.

This is indeed a pain point for using the library currently. You could indeed move the audio folder into the fast ai library but of course we can't do that unless fastai merge our code into there.
We've thought it might be a good idea to make a setup.py file so that you can install the library through pip, then you could just:
from fastai_audio import *

In fact the specific line that your referencing could be removed

I was thinking it could be removed.  I just wanted to make sure there was no unintended consequences to doing that.

Yes it could be removed


Unexpected behavior with create and duplicates enabled

Describe the bug
I've created a tom-select with create: true, duplicates: true. When removing a duplicate value from this tom-select (by pressing backspace), the deleted element isn't always at the caret position and sometimes deletes multiple copies of the duplicated value (while throwing an error, below).
To Reproduce
Steps to reproduce the behavior:

Create a tom-select input with duplicates and create enabled
Enter some values including at least some duplicates
Delete values using backspace. When removing a value that has multiple copies, the bug will occur.

Expected behavior
The element(s) should be deleted.
Additional context
Error message as follows (using latest dist):
Uncaught TypeError: t.options[s] is undefined
    updateOriginalInput tom-select.ts:2077
    removeItem tom-select.ts:1887
    removeOption tom-select.ts:1646
    removeItem tom-select.ts:1880
    deleteSelection tom-select.ts:2266
    onKeyDown tom-select.ts:691
    setup tom-select.ts:324
    C utils.ts:128
    setup tom-select.ts:324
    R tom-select.ts:157
    <anonymous> initialize.js:5
    jQuery 8
    onopen shinyapp.js:94
    createSocket shinyapp.js:91
    connect shinyapp.js:36
    initShiny init_shiny.js:551
    setTimeout handler* init_shiny.js:583
    jQuery 13
tom-select.ts:2077:4

Hmm, I'm not  seeing any errors. Do you get them with this example?

I don't get errors with your example. It seems that it's actually a problem with the autocomplete option. Removing autocomplete="off" from your example causes the error to occur as described.

Hmm, I just tried without autocomplete="off" and I'm still not getting the error.

Would love to get this fixed. If you have any other steps and/or clues on how to reproduce the error, please let me know.

Interestingly I can't get the error log to be thrown again for some reason. However, I can still replicate the behaviour of the issue reliably.
Do the following:

Type test,tes,test (or any string which contains some duplicates and some other values) followed by backspace

I've uploaded an mp4 of this happening to my Dropbox: here.
I'm trying again now and it actually occurs even with autocomplete="off" still present... Example. For some reason I can't quite isolate what's causing this.

Awesome, I see what you're talking about now. Thanks!

Thanks for your quick work! Unfortunately this isn't fully fixed: Ctrl+A, backspace (or any other way of selecting multiple elements like Shift + click or Ctrl + click) still deletes elements in an unexpected manner

Hmm.

still deletes elements in an unexpected manner

Could you describe what you mean by that? I'm not seeing any issues... wondering if your referring to the large gap that can appear between items after an item in the middle was deleted?

Ah that may be my oversight. Yes upon closer inspection, it seems the elements are actually being deleted correctly, but elements right of the cursor are being pushed off to the right side of the input as you describe. Sorry for my over-eager bug reporting! I appreciate your responsiveness.

It's all good. The solution for this is the input_autogrow plugin.


feat(spanner): add samples for Proto columns with null values

This PR adds samples to perform insert and read operations on Proto columns - Proto Messages and Proto Enum that can take null values.

Please review this PR after reviewing PR-2776.


PS2 files tend to crash the tool

Suspecting it's probably something in the art assets, an unexpected compression or something of the sort (but who knows.) The crash is not limited to one single .lvl file, but the most recent one I attempted to open was actually a side .lvl from a PS2 beta build for BF2. Reason why I'm blaming the art assets is because there are half-extracted textures, possibly models too that remain broken and unviewable in the output folder. At the moment I'm not too worried about the art, but I'd like to retrieve and examine the .odf portions of some classes as they were set up during development, before getting settled into the final BF2 setting.
This isn't a terribly important thing and therefore doesn't warrant any hurry if there's no time available to spare on this tool, I simply just like seeing how things were during games' development histories. Still, it would be cool if the PS2 .lvl files (and its beta build's, if they're even any different) could be fully extracted. Or even just the code sections, really. Whichever is easier to make happen.

I do want to eventually polish the v1.2.0 RC properly. Though it definitely isn't a priority. In the meantime does using v1.1.1 release vs the 1.2.0-rc produce anything different? I know other people had reported less crashing from the v1.1.1 release.

Oh, that is good - 1.1.1 handled this one like a champ. I got everything I wanted.


Pushbullet - support multiple api keys and email destinations.

Adds the ability to use multiple API keys in the Pushbullet output plugin, also the addition of email address destinations for messages.
Squashed version of my previous PR with fixes and cleanups.

Thanks! Sorry for the delay, want to update the wiki?

Updated, added an example as well.


editor: added image resize functionality

Added image resize functionality in drafts and setting width of images in publications.

@iskaktoltay looks great! I just pushed some style changes so the image will stay in the center all the time.
The onnly thing I would add is to make the video also resize. do you think this is easy to add?

also we need to respect the width in the publication view

I did add it to publication content

@iskaktoltay looks great! I just pushed some style changes so the image will stay in the center all the time. The only thing I would add is to make the video also resize. do you think this is easy to add?

I'm not sure because video embeds and uploaded videos use different components to display the video. I will take a look

I did add it to publication content

yep I saw it after!
I did change the styles so the image will be centered too!
nice work!

I'm not sure because video embeds and uploaded videos use different components to display the video. I will take a look

I think this PR is ready to go, let's do Videos as a Nice to have if you can!


Installed, but not available/working.

Installed in Fedora 25, atom-1.16.0-0.1.x86_64.
Installed packages language-vhdl 0.7.2, aligner-vhdl 0.1.0, vhdl-entity-converter 0.4.3, Sublime-Style-Column-Selection 1.7.4.
Not showing any VHDL Align option, or operation.  Tried installing package aligner 0.22.3.  I now show Align option, and it's working, but not optimized for VHDL.
Suggestions?
Thanks for your support.

For sure you'll need the Aligner package as this is just an add-on for VHDL language support and I don't enforce that dependency.
Can you be more specific on "not optimized for VHDL".  I currently support aligning on :, :=,  <=, and => operators.  My current workflow is to select the block I want aligned and then trigger Aligner:Align or Ctrl-Alt-/

Thank you for the prompt response!   Here's an example:
---before align---
AF_H     <=     af_int;
FF_H <= ff_int;
---after align---
AF_H <= af_int;
FF_H    <= ff_int;

I could see why that wouldn't work. Normalizing leading whitespace is tricky because folks have very different opinions on what it should be. For leading whitespace for any language I always find it easiest to:

select the block
align it to the left margin by hitting ctrl-{ until everything is aligned
bring it out to the desired indentation with ctrl-}
then use the aligner package to fix anything else

Thank you for your support!


Update and Delete Bot Messages

Docs Build status updates of commit 1413c91:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/bots/how-to/conversations/update-and-delete-bot-messages.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 63af075:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/bots/how-to/conversations/update-and-delete-bot-messages.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 3f8a822:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

msteams-platform/bots/how-to/conversations/update-and-delete-bot-messages.md
:x:Error

Details

msteams-platform/bots/how-to/conversations/update-and-delete-bot-messages.md

[Error] Cannot resolve '~/includes/pre-release-label.md' relative to 'bots/how-to/conversations/update-and-delete-bot-messages.md'.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit eb81f01:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

msteams-platform/bots/how-to/conversations/update-and-delete-bot-messages.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


How would you suggest setting secrets?

Hi, this looks like an interesting project and our team is investigating whether it's a good fit for our workflow. We're wondering how you would suggest giving developers access to change secrets though, since this tool seems to only read from the secret stores. When using it, do you give developers direct access to the secret store to update values? Thanks!

Great question @systemist
Currently we are doing two things for security reasons:

We don't have an opinion on how to authenticate -- meaning, Teller will not ask you for credentials for your secret stores, but it will basically "assume connected" and federate the authentication to how it currently works on your machine. A good example would be anything based on AWS, where almost every AWS SDK "knows" where your authentication details are and knows how to connect to it. When Teller uses this SDK, it will just ask it to connect the best way it knows.
We're restricting to read only, and so -- it would mean lesser privileges, naturally. However we've gotten a few request to "push" values back to the stores -- so we're exploring a push interface.

For the push interface -- I'm happy to get more details especially for your use case so we can build it right 😄

Thanks @jondot. I'll step in here and add my two cents. The basic use case would be the following:

Ability to set a secret on multiple providers at once
Detect "drift" between providers
Ability to sync secrets between providers when drift occurs, sort of like resolving a merge conflict

And for what it’s worth, our team (including @rcII) would be happy to work on an implementation but
wanted to see if you had any existing designs so we didn’t just create a fork that’s only a good fit for us. Our current workflow is to give all devs access to set configs (including secrets) on Heroku but only have DevOps/SecOps have access to AWS secrets and Vault, but want to unify a single developer friendly workflow as Raney mentioned.

Yea I think actually when you draw out the use case this way it's really exciting! since we automate the source, we don't get drifts much and we were looking on rotation as a use case (update in all providers at once because a key has been exposed/stale).
I love that you call it "drift", makes complete sense.
In terms of implementation -- yea happy to accept any suggestion/PR you might have. Basically since we're dealing with what is essentially a K/V store, it's just to map back a value to the key, and push via the relevant provider API (all should support this). Because authority and authentication is federated, we don't care about roles (either user has permissions or they hasn't, but its not for us to decide).
In terms of user experience, that's where it would be more interesting. How would you suggest interfacing this?
For example:
setting secrets
teller put [provider] [KEY] [value]
(remember, path is in the configuration file so there's a small win here to save typing)

or maybe you do want to provide a specific path:
teller put [provider,provider2] [path] [value]
(remember, path is in the configuration file so there's a small win here to save typing)

and then maybe we can map a complete file? (but it's kind of dangerous to encourage people to keep files with secrets on disk)
teller put [provider,provider2] [env_file]

and then, maybe put on all providers this way?
teller put-all [key] value

drift detection
teller drift provider1 provider2 provider3

Or maybe check a key drift?
teller drift K1 K2

sync
Assumption: there's always once source and many sinks and we want to "sync from source to sink":
teller sync [source provider] [dest provider1, dest provider2] --exclude [KEY]

And just for usefulness include an --exclude/--include logic for the sync, optional.
What do you think? can you draw out your perfect interface in your mind?

One more idea after we've discussed it internally, is to encode all this in the configuration itself. So basically we want to model:
[ ] handle/id (a secret has a unique handle, although it may appear in multiple stores), the handle is used to do matching between keys/secrets in multiple stores. so basically this means: add one more property to configuration
[ ] again in the configuration, mark up relevant secrets as origin, meaning this way we can automatically infer where to grab the source data and sync it into any other provider that's has a key/secret that isn't marked as origin

Thanks for the brainstorming and potential implementation criteria @jondot! The one potentially complicated use case that may become more common than not is when organizations separate out secret and non-secret values into Vault (secret) and Consul (non-secret). In this case, a user would need to be able to set a source for two discrete sets of KVs, one secret and one non-secret. However, maybe we don't need to disambiguate between secret and non-secret since the providers handle the encryption portion of the story. I understand that teller was built around secrets and not non-secret configs, so maybe this is a use case that teller shouldn't support by nature. Also, this particular use case is most likely specific to Vault/Consul being used in conjunction. The most useful pieces of your descriptions above would be the following:

Set a source and/or sources in a config file. (i.e. Vault and Consul as sources of truth)
Have a CLI interface that can put a specific secret (i.e.teller put [provider] [KEY] [value]) or KV config to a provider or multiple providers (put-all)
Drift detection and remediation (teller drift provider1 provider2 provider3 and teller sync [source provider] [dest provider1, dest provider2]

Since most app configs contain secret and non-secret values, it would be a nice-to-have to control them all under one tool, but we could easily chunk them into a single source. This may make the user experience a bit rough however, as everything would be redacted in the teller output.

Or maybe one source per path...

No worries, I think what I'll start doing is the skeleton of the 'write' workflow (data model and refactoring) and then update back here

Hi all, happy to introduce the write/put interface, you can jump to the README updates here to see the addition from a user's point of view.
https://github.com/SpectralOps/teller/blob/ee791c2eeea68c9af5bb0c2b5a857023eaac46f5/README.md#bike-multi-write-rotation--sync
In terms of implementation, everything is in place to easily implement a drift fixing feature -- just waiting for some comments about the put interface if there are any

Thanks @jondot, we will do some preliminary testing with Heroku, Vault, and Consul. This looks great!

I was able to set a secret in a single provider (vault)
➜  teller git:(put-interface) teller put TESTING_TELLER=true --path app/data/config --providers hashicorp_vault -c  ~/.teller.yml
Put TESTING_TELLER (app/data/config) in hashicorp_vault: OK.

But unable to put in multiple providers
➜  teller git:(put-interface) teller put TESTING_TELLER_1=true --path app/data/config --providers hashicorp_vault,heroku -c  ~/.teller.yml
Put TESTING_TELLER_1 (app/data/config) in hashicorp_vault: OK.
Error: cannot put TESTING_TELLER_1 in provider heroku: Patch "https://api.heroku.com/apps/app/data/config/config-vars": The requested API endpoint was not found. Are you using the right HTTP verb (i.e. `GET` vs. `POST`), and did you specify your intended version with the `Accept` header?

I suspect this is because the path semantics are different between the two providers? Vault follows a true "path" format, while Heroku's "path" is actually the name of the app that is being configured. I may be doing something completely wrong as well.
.teller.yml :
project: newproject

# Set this if you want to carry over parent process' environment variables
# carry_env: true

#
# Variables
#
# Feel free to add options here to be used as a variable throughout
# paths.
#
opts:
  region: env:AWS_REGION    # you can get env variables with the 'env:' prefix
  stage: development

#
# Providers
#
providers:
  # requires an API key in: HEROKU_API_KEY (you can fetch yours from ~/.netrc)
  heroku:
  # sync a complete environment
    env_sync:
      path: $HEROKU_APP
      source: s1

  # # pick and choose variables
  # env:
  #	  JVM_OPTS:
  #      path: drakula-demo

  # configure only from environment
  # https://github.com/hashicorp/vault/blob/api/v1.0.4/api/client.go#L28
  # this vars should not go through to the executing cmd
  hashicorp_vault:
    env_sync:
      path: app/data/config
      sink: s1

(NOTE: $HEROKU_APP is a placeholder in this snippet for the actual name of the app)

Yes you are correct, there are two ways to put

You "work through" the key-value path configuration in your teller.yml
You supply a path in a "direct mode" where you know there's no configuration defined (kind of like raw setting the value)

And yes in case (2) it makes less sense to set multiple providers, the way I think might be interesting to solve it is perhaps one of two (would be happy to get feedback here):

Restrict the put --path case to just one provider
Accept a full /path/to/FINAL_KEY=value in the case of put --path and then --path just becomes a bool indicating that the mapping includes a full path

What do you think?

If I am understanding correctly, a user can put to multiple providers at once, but their .teller.yml must be configured? Example real-life use case:
A developer needs to set a configuration/secret on Heroku and Vault simultaneously to avoid environment disparity.
Can this be accomplished via CLI flags, or must the teller.yml be configured correctly?

Correct. Think of it like the config file containing the coordinates of every secret or value and how it physically maps into the provider. If this information does not exist there eventually we have to somehow provide it (it may include the path but optionally additional bits of information that is provider specific such as encryption in the case of AWS based stores)

Thanks @jondot. From initial testing, the put functionality seems to be working as expected. Thanks for implementing this. Once the drift-fixing feature is place, we would like to start testing within our org at a wider scale. Would there be a way to run the drift-fix as daemon somewhere? Example use case would be:

Developer makes a change to Heroku config
drift-fix detects the change and syncs the sink provider(s)
We could run this in a container to monitor changes.

Great @rcII, yes I believe that's easily done. We can add a --watch flag that's commonly done in these kind of tools. I would go ahead and add in some kind of standard logging/eventing subsystem so you could get notified of these, I'll look around to see what isn't so heavy to use, and is pluggable (maybe enough to pick a good logging library that can have some plugins to connect to).

Hey @jondot. Just wanted to let you know we believe we now have a path to success with your additions! We're pretty sure we can get around needing the daemon functionality for the time being but would still be a neat addition to the library. Great work and thank you for all your help with this!

I'm closing this issue. You are more than welcome to open new issues with new thoughts for future discussions :).


feat(crypto helper):jwt_implement_ES384_signature_algorithm

@rwwwx can you merge the updated main branch into your branch? now it contains CI configuration for pull requests