notary-db.log reports - "ERROR: 1049 Unknown database 'test'". Also /usr/bin/mysqladmin is not found

Reported by @lgayatri under https://github.com/vmware/vic-product/issues/564, which I closed
I verified this on a build from master. Please direct questions for additional details to @lgayatri
Deployed - chin-vic-c2506807-dev-upgrade.OVA
Post power on and registration at 9443 port, checked the logs under harbor and noticed that
notary-db.log says:
Aug 14 05:36:14 <IP_ADDRESS> notary-db[405]: ERROR: 1049 Unknown database 'test'
Aug 14 05:36:14 <IP_ADDRESS> notary-db[405]: 2017-08-14 5:36:14<PHONE_NUMBER>43072 [ERROR] Aborting
Aug 14 05:36:14 <IP_ADDRESS> notary-db[405]:
Aug 14 05:36:16 <IP_ADDRESS> notary-db[405]:
Aug 14 05:36:16 <IP_ADDRESS> notary-db[405]: WARNING: OPENGIS REQUIRED SP-S WERE NOT COMPLETELY INSTALLED!
Aug 14 05:36:16 <IP_ADDRESS> notary-db[405]: GIS extentions might not work properly.
Aug 14 05:36:16 <IP_ADDRESS> notary-db[405]:
Aug 14 05:36:16 <IP_ADDRESS> notary-db[405]: PLEASE REMEMBER TO SET A PASSWORD FOR THE MariaDB root USER !
Aug 14 05:36:16 <IP_ADDRESS> notary-db[405]: To do so, start the server, then issue the following commands:
Aug 14 05:36:16 <IP_ADDRESS> notary-db[405]:
Aug 14 05:36:16 <IP_ADDRESS> notary-db[405]: '/usr/bin/mysqladmin' -u root password 'new-password'
Aug 14 05:36:16 <IP_ADDRESS> notary-db[405]: '/usr/bin/mysqladmin' -u root -h f9736b8317f9 password 'new-password'
Aug 14 05:36:16 <IP_ADDRESS> notary-db[405]:
Aug 14 05:36:16 <IP_ADDRESS> notary-db[405]: Alternatively you can run:
Aug 14 05:36:16 <IP_ADDRESS> notary-db[405]: '/usr/bin/mysql_secure_installation'
Aug 14 05:36:16 <IP_ADDRESS> notary-db[405]:
Aug 14 05:36:16 <IP_ADDRESS> notary-db[405]: which will also give you the option of removing the test
Aug 14 05:36:16 <IP_ADDRESS> notary-db[405]: databases and anonymous user created by default. This is
Aug 14 05:36:16 <IP_ADDRESS> notary-db[405]: strongly recommended for production servers.
Aug 14 05:36:16 <IP_ADDRESS> notary-db[405]:
Aug 14 05:36:16 <IP_ADDRESS> notary-db[405]: See the MariaDB Knowledgebase at http://mariadb.com/kb or the
Aug 14 05:36:16 <IP_ADDRESS> notary-db[405]: MySQL manual for more instructions.
Aug 14 05:36:16 <IP_ADDRESS> notary-db[405]:
Aug 14 05:36:16 <IP_ADDRESS> notary-db[405]: Please report any problems at http://mariadb.org/jira
Aug 14 05:36:16 <IP_ADDRESS> notary-db[405]:
Aug 14 05:36:16 <IP_ADDRESS> notary-db[405]: The latest information about MariaDB is available at http://mariadb.org/.
Aug 14 05:36:16 <IP_ADDRESS> notary-db[405]: You can find additional information about the MySQL part at:
Aug 14 05:36:16 <IP_ADDRESS> notary-db[405]: http://dev.mysql.com
Aug 14 05:36:16 <IP_ADDRESS> notary-db[405]: Support MariaDB development by buying support/new features from MariaDB
Aug 14 05:36:16 <IP_ADDRESS> notary-db[405]: Corporation Ab. You can contact us about this at<EMAIL_ADDRESS>
The installer does not ask to set the password for the DB.
If it is required to set the password, then we should mandate in the OVA deployment UI.
If it is NOT required to set the password, then we can skip showing the above messages
2.There is no file called /usr/bin/mysqladmin
what is 'test' database which we are referring to?

This seems to be an issue of hard code configuration of the mariadb docker image.
https://github.com/dockerfile/mariadb/issues/13
And we don't require the password but we can't skip showing the messages because it's generated by the official docker image of maria db.
We don't plan to make update for this issue in 1.2
However, given we plan to migrate the DB to photon, we may have a chance to fix the misleading message issue in next release.

Cool thanks for looking into it

Verified on VIC 1.3 and this issue has been resolved.
Build on which the scenario was verified - vic-v1.3.0-rc3-2784-7d227d5c.ova


Custom animation

Hi
Help needed. In my project I have rounded slideshow.
When I tap on slideshow it is zoom in, the animation looks good, but when I zoom out the corners of slideshow are staying angled when my slide show is rounded. Looks nasty.
Is it possible to customize the animation?
here link
thanks

Hi, this requires changes in ZoomAnimatedTransitioningDelegate which are bit beyond the scope now. I recommend you to subclass ZoomAnimatedTransitioningDelegate and customize the animation there.


Rename expert_inputs and create a new optional_inputs script

Also be sure to update the references to expert_inputs in other files to reflect the new name

We've done this in the constant_dependency branch, and it should be ready to merge into master


Fix error message in float8 FSDP utils

Fixes #1108
Error message should say "Only supports dynamic scaling".

thanks and welcome Daniel, I added you to our contributor list so you don't need approval to run CI


Fix regressions from 0.0.20180929 release

WIP for closing issue #56

Test this at WireGuard-KT-57-036e797e8328b218371580953ea5d24b70b7d40f.apk.
This is an automatically posted message from Travis-CI

Test this at WireGuard-KT-57-8d6b5e9b532b103e7332d49fc721b339fcefc891.apk.
This is an automatically posted message from Travis-CI

Test this at WireGuard-KT-57-8ed9a40fefe450449532b962cbec5945a9a725b6.apk.
This is an automatically posted message from Travis-CI

Test this at WireGuard-KT-57-314b4826149508db71613ca8a5997111f25ad8c0.apk.
This is an automatically posted message from Travis-CI

Snackbar can wait for now, other fixes are way more mission critical.


tv will not switch inputs

Describe the bug
Tv will power on and off fine but if tv is on say hdmi 2, when powering on it will not switch to the pc's hdmi 4
How to reproduce the bug?
leave tv on hdmi 2, turn tv off, start pc, tv turns on but does not switch to pc input
What was the expected behavior
tv switch to hdmi 4
What is your system?
Win 11, build 22631.3737, LG OLED C1
Log file and screenshots
https://pastebin.com/1WH7wHu4

Hi, can you double check the pastebin link please. It does not seem to work for me. Cheers

seems to have fixed itself, it didnt work after i updated to the latest version, but after switching the settings off and on a couple of times it seems to have fixed it

The only thing that stood out in the log was that the TV was quite slow to respond. Not sure what to make of it but maybe it is related

You got me wondering now if a factory reset of the tv would be a good idea

Yeah it wouldnt be the first time it helped with weird issues. Maybe start with pulling the plug for a few minutes though and see if things run smoothly from there

Hmm, interesting. I have the same or similar issue. Instead of creating a new issue I thought I'd add here.
I have two PC's connected to an LG C2, one on HDMI1 and the other on HDMI4. The one connected to HDMI1 always works correctly and switches the input to HDMI1 when turned on.
The laptop connected to HDMI4 never switches to HDMI4 when turned on.
I pasted a debug log here: https://pastebin.com/m430VVtt

Hi @hapklaar there are no issues with your log, everything looks as expected. Since the TV powers on fine and responds to the request to switch HDMI, but doesn't actually do it, Isuspect that the TV needs slightly more time after being powered on before the HDMI-switch command is sent. Currently on vacation so it'll take a few weeks but I'll have a look

Thanks @JPersson77 for responding while on holiday! No rush at all, I hope you have a good one!
In the mean time, I'll do some additional testing.

I've been testing with alternate inputs, but all behave the same as hdmi4. It works only 1 out of 20something times. Your suspicion about it being a timing issues might be correct, but then I would expect input switching to occur always when the TV is already on but on the wrong input. This doesn't seem to be the case, or maybe I tested it wrong...

I have a similar issue, if a last input was AppleTV then enabling PC turns on TV + AppleTV on its source. I waited quite long time and nothing happened. I have to switch a source manually to PC.

Thanks for reporting! I will have a look as soon as possible. Just for completeness, can you please enable debug level logging and paste a link here wirh the full log

@JPersson77 Here it is.
My steps are:

Turn on PC (hdmi 1), AppleTv is sleeping. TV was turned on by PC
Sleep my PC, TV was turned off by PC
Turn on AppleTV. TV was turned on by AppleTV. Turned off Apple TV, TV was turned off by AppleTV
Turn on PC. TV was turned on by PC. AppleTV was turned on by TV. For 1 sec TV was on hdmi 1 while Apple TV is booting up and then it's taking over the source.

log.txt

But then it sounds like a CEC issue. The app managed to initially and correctly switch to hdmi1 in step 4 if I understand correctly? But then apple tv forced another switch immediately after.

@JPersson77 exactly https://www.reddit.com/r/LGOLED/comments/12j353m/how_to_stop_lg_switching_to_appletv_input_on/
So it's not the issue of your app, thank you anyway. It's small inconvenience for me, the rest functionality is very useful

Yeah that link potentially explains it well.
The app cannot prevent CEC communications between other devices of course. Potentially the hdmi switching command can be delayed or sent multipel times to "override"

@JPersson77 I also thought about the similar fix. Well, will see, might be this issue is affecting someone else and then it would raise the priority

Hi @hapklaar there are no issues with your log, everything looks as expected. Since the TV powers on fine and responds to the request to switch HDMI, but doesn't actually do it, Isuspect that the TV needs slightly more time after being powered on before the HDMI-switch command is sent. Currently on vacation so it'll take a few weeks but I'll have a look

@JPersson77 did you find some time to have a look? Maybe a workaround in the form of a configurable delay could help.

Hi there been a little busy nut this is not forgotten about

Same issue here, LG C3. It used to work normally but recently I took my HDMI cable out and put it back in and my TV switched from my PC being HDMI 3 to HDMI 1 and when I set and applied LGTV Companion to use HDMI 1 instead it still tries to set it to 3 on wakeup.

hi @Sindorius this is a similar but not the same issue.
From your description It sounds like the configuration was not saved properly. Can you confirm you pressed "apply" in the main gui before exiting the GUI? Do you mind enabling debug level logging and upload a log to pastebin and I will have a look

@JPersson77 Yeah I pressed apply. I even tried setting it to another HDMI input #, hitting apply, and setting it back to 1, and hit apply again to see if I could resave settings. But same issue.
How do I enable debug level logging and see the log?

@JPersson77 Yeah I pressed apply. I even tried setting it to another HDMI input #, hitting apply, and setting it back to 1, and hit apply again to see if I could resave settings. But same issue.
Here's the log output: https://pastebin.com/ZdCBAd9h

From the log supplied the app is setting the TV to hdmi input 1 so that, seems correct. Is something else setting it to hdmi input3

Not that I'm aware of. I checked the TV settings and there was something to have it go to 'recent input' when turned back on but that's 1 too.

Hi @hapklaar @vladimir-fsh @Sindorius please see if the latest version v4.1.0 resolves some or all of your issues

Unfortunately my laptop now blocks this executable (Windows Defender ASR, managed from Intune by my employer), so not able to test at the moment unfortunately...

Hi @hapklaar @vladimir-fsh @Sindorius please see if the latest version v4.1.0 resolves some or all of your issues

Thanks for the fix. I can check it after gpu upgrade, in January. Now my pc is without gpu


Bad constant?

After having some build trouble here, I abandoned Ubuntu 16.04's package for xwiimote and built and installed from 'master'. From there, I cleaned, re-configured, and installed xwiimote-bindings and that seemed to go fine.
After running that however, I'm getting this error:
ImportError: /usr/lib/x86_64-linux-gnu/libxwiimote.so.2: version `LIBXWIIMOTE_3' not found (required by /usr/local/lib/python2.7/dist-packages/_xwiimote.so)

I'm assuming this must be a runtime error because of where my libraries live (I didn't configure with any options). I get the same from ldd:
edwin@Rosetta:~/Documents/Code/xwiimote-bindings$ ldd /usr/local/lib/python2.7/dist-packages/_xwiimote.so
/usr/local/lib/python2.7/dist-packages/_xwiimote.so: /usr/lib/x86_64-linux-gnu/libxwiimote.so.2: version `LIBXWIIMOTE_3' not found (required by /usr/local/lib/python2.7/dist-packages/_xwiimote.so)
	linux-vdso.so.1 =>  (0x00007ffc082cc000)
	libxwiimote.so.2 => /usr/lib/x86_64-linux-gnu/libxwiimote.so.2 (0x00007fbd50689000)
	libpthread.so.0 => /lib/x86_64-linux-gnu/libpthread.so.0 (0x00007fbd5046b000)
	libc.so.6 => /lib/x86_64-linux-gnu/libc.so.6 (0x00007fbd500a0000)
	libudev.so.1 => /lib/x86_64-linux-gnu/libudev.so.1 (0x00007fbd50080000)
	/lib64/ld-linux-x86-64.so.2 (0x000056399a5f6000)
	librt.so.1 => /lib/x86_64-linux-gnu/librt.so.1 (0x00007fbd4fe78000)

I'm not knowledgeable enough to know how to work around this however.

After bumping the version in xwiimote to 3 and rebuilding everything (and running ldconfig), it looks like the examples are working as intended. Again, I just threw science at the wall to see what would stick, I don't know the motivations for why the bindings were looking for features in version 3 when that build wasn't enabled yet.

I can confirm. I see the same issue when running the python example, and I used the same fix @RandomInsano offered above.


Async rendering causes too many open files and no caching

Hi, first of all thanks for this great module!
In testing with large numbers of input files (~3500), I consistently get an EMFILE: too many open files error.
It seems that metalsmith-layouts is asking consolidate to render all the input files asynchronously. Each of my input files is using the same template, so what should happen is that consolidate should read the template the first time, then store it in the cache so that it doesn't have to be opened and re-read for subsequent input files. But instead, because all the render jobs are started asynchronously at the same time, consolidate actually begins reading the template file immediately for each input file, before any of its readFile calls actually finishes.
The result is that once the number of input files grows large enough, boom, too many open files.
By changing the each() call to a synchronous forEach() loop I can make the problem disappear, but I'm sure there must be a better and more generalizable solution.
Thoughts?

Ah yeah. We could replace each with eachLimit (https://caolan.github.io/async/docs.html#eachLimit). Set the default to a sensible amount and allow the user to override it. That should take care of it. Feel like submitting a PR?

Yeah, eachLimit would cut the batch size down, but it doesn't really solve the problem. Within a batch of input files, each time we come across a request for a template that hasn't been cached from a previous batch, we'd still open that template file as many times as we have input files in the batch. Right?
I think we would need to first save a list of all the templates that our input files will be using, then ask consolidate to do a render with each of those templates so that it caches the template contents. After all those initial pre-loads are done, then we can do the async each() call to loop through all the input files.

submitted a PR for discussion. Cheers!

Closed from #120


Pangram: Add Pangram exercise

This PR adds exercise for determining if a sentence
is a pangram or not.
Fixes https://github.com/exercism/xgo/issues/395
Signed-off-by: Abhijeet Kasurde<EMAIL_ADDRESS>

Inactivity: Rebased for config.json, and adding commits to address review comments. Will squash + merge shortly.

Thanks @Akasurde !


Debian package for BenchExec

We should find a way to automatically generate Debian packages for BenchExec in our build.

There is a Debian package available starting with version 1.1.


v1.0.0 tagged version is not semantic according to SPM

The released version is tagged v1.0.0 instead of 1.0.0 so it does not work correctly in Swift package manager. We are not able to specify other dependency rules than branch or exact commit.

It looks like the issue here was related to one of the dependencies not being pinned to a stable version. v1.0.1 should fix that with https://github.com/EmergeTools/ETTrace/pull/55 Thanks for reporting!

Works, thank you.


[V3] Package naming

There currently exists com.microsoft.azure.cosmos and com.microsoft.azure.cosmosdb packages. To follow spec, there should just be a com.azure.data.cosmos package.

done


fix(protocol-kit): considering the SafeVersion in the isL1Contract check in SafeBaseContract

What it solves
This PR updates the logic used to determine if a contract is L1 or L2 version. Previously, the check only considered if the chainId was mainnet or if it was manually flagged as an isL1SafeSingleton=true by the user.
Now we also considers whether the Safe version does not have the SAFE_L2_CONTRACTS feature (safeVersion >= 1.3.0)
How this PR fixes it
    const isL1Contract =
      safeDeploymentsL1ChainIds.includes(chainId) ||
      isL1SafeSingleton ||
      !hasSafeFeature(SAFE_FEATURES.SAFE_L2_CONTRACTS, safeVersion)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 77.864%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
783

Relevant Lines:
933

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 77.864%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
783

Relevant Lines:
933

💛  - Coveralls


Clearcoat unit tests

This is a follow-on to #931 that adds unit tests for clearcoat.
Currently the test is failing.  I left a one-line suggestion on #931 that will fix it.  Merge that before merging this.
/cc @donmccurdy

Don submitted a fix to his branch.  So, merging Don's branch in here should change the unit tests from red to green.

And they passed!

@emackey (Sorry, this is just a question, not related to this PR, and I didn't want to make an issue.) What is the meaning of the numbers prefixed to the test files? How did you pick 08_ for this PR for example?

Those numbers have become a little messy I guess.  I picked 08 because I was using that for ORM combinations, and the clearcoat combination is the same sort of thing.  I wanted to reuse the 08_*.png image test assets.  Probably I should put base + alpha combination in there as well, but I haven't got around to it yet.
I'm open to suggestions for re-org, but unfortunately I can't invest a whole lot of time on the test framework.  I just work on it when I get a few spare few moments.

Hmm, well I still don't get it, haha. I didn't want to reorg them. More like, if I wanted to add a test file, how would I know what number I should use?

If it's something new, just make a new number, that should be fine.

A new number as in 01? Or the highest existing number + 1? I was a bit confused doing this for https://github.com/KhronosGroup/glTF-Blender-IO/pull/800 as well. 😅

Highest number plus 1.

Okay. Thanks for answering!


Unable to run Linux

Please paste the output of the following command here: sudo edit-chroot -all

name: xenial
encrypted: no
Entering /mnt/stateful_partition/crouton/chroots/xenial...
crouton: version 1-20171230152552~master:de1361af
release: xenial
architecture: armhf
xmethod: xorg
targets: xfce
host: version 10278.0.0 (Official Build) dev-channel daisy
kernel: Linux localhost 3.8.11 #1 SMP Thu Jan 4 03:26:39 PST 2018 armv7l armv7l armv7l GNU/Linux
freon: yes
Unmounting /mnt/stateful_partition/crouton/chroots/xenial...
Please describe your issue:
Unable to launch Linux
If known, describe the steps to reproduce the issue:
I have tried installing crouton, sucessfully (i think) and am trying to run
bash
sudo startxfce4
i get an error saying:
Entering /mnt/stateful_partition/crouton/chroots/xenial...
/usr/bin/startxfce4: Starting X server
_XSERVTransmkdir: Owner of /tmp/.X11-unix should be set to root
X.Org X Server 1.19.5
Release Date: 2017-10-12
X Protocol Version 11, Revision 0
Build Operating System: Linux 4.4.0-101-generic armv7l Ubuntu
Current Operating System: Linux localhost 3.8.11 #1 SMP Thu Jan 4 03:26:39 PST 2018 armv7l
Kernel command line: cros_secure console= console= loglevel=7 init=/sbin/init cros_secure oops=panic panic=-1 root=/dev/dm-0 rootwait ro dm_verity.error_behavior=3 dm_verity.max_bios=-1 dm_verity.dev_wait=1 dm="1 vroot none ro 1,0 2539520 verity payload=PARTUUID=68f9f34c-d452-5c42-811a-ee3a7080a2a6/PARTNROFF=1 hashtree=PARTUUID=68f9f34c-d452-5c42-811a-ee3a7080a2a6/PARTNROFF=1 hashstart=2539520 alg=sha1 root_hexdigest=b89a76fa5023c7117368c0596185b11b2f07c184 salt=2f712eb6330f916cf53b806ca4c929f03216de9409f3ca5c7f80696ea868d99a" noinitrd vt.global_cursor_default=0 kern_guid=68f9f34c-d452-5c42-811a-ee3a7080a2a6
Build Date: 24 November 2017  09:45:24AM
xorg-server 2:1.19.5-0ubuntu2~16.04.1 (For technical support please see http://www.ubuntu.com/support)
Current version of pixman: 0.33.6
Before reporting problems, check http://wiki.x.org
to make sure that you have the latest version.
Markers: (--) probed, (**) from config file, (==) default setting,
(++) from command line, (!!) notice, (II) informational,
(WW) warning, (EE) error, (NI) not implemented, (??) unknown.
(++) Log file: "/tmp/Xorg.crouton.1.log", Time: Sun Jan  7 15:20:30 2018
(==) Using system config directory "/usr/share/X11/xorg.conf.d"
Error org.freedesktop.DBus.Error.UnknownMethod: Method "ReleaseDisplayOwnership" with signature "" on interface "org.chromium.LibCrosServiceInterface" doesn't exist
method return time=1515356430.236616 sender=:1.9 -> destination=:1.41 serial=617 reply_serial=2
boolean true
MESA-LOADER: failed to retrieve device information
gbm: failed to open any driver (search paths /usr/lib/arm-linux-gnueabihf/dri:${ORIGIN}/dri:/usr/lib/dri)
gbm: Last dlopen error: /usr/lib/dri/exynos_dri.so: cannot open shared object file: No such file or directory
failed to load driver: exynos
EGL_MESA_drm_image required.
(EE)
Fatal server error:
(EE) AddScreen/ScreenInit failed for driver 1
(EE)
(EE)
Please consult the The X.Org Foundation support
at http://wiki.x.org
for help.
(EE) Please also check the log file at "/tmp/Xorg.crouton.1.log" for additional information.
(EE)
Error org.freedesktop.DBus.Error.UnknownMethod: Method "TakeDisplayOwnership" with signature "" on interface "org.chromium.LibCrosServiceInterface" doesn't exist
method return time=1515356430.621424 sender=:1.9 -> destination=:1.43 serial=619 reply_serial=2
boolean true
(EE) Server terminated with error (1). Closing log file.
/usr/bin/xinit: giving up
/usr/bin/xinit: unable to connect to X server: Connection refused
/usr/bin/xinit: server error
Unmounting /mnt/stateful_partition/crouton/chroots/xenial...

Seems this issue has been around for quite a while now and is related to video-armsoc drivers.
Take a look at this comment for a little more info.: https://github.com/dnschneid/crouton/issues/3611
For a couple of workarounds, see this comment: https://github.com/dnschneid/crouton/issues/3291#issuecomment-309564813
Note that in Solution 1 above you may have to substitute 'startxfce4' with your start script.
Hope this helps,
-DennisLfromGA


Cannot "Add Existing Cluster" from duplicate subscription name.

I am trying to add an existing Azure Kubernetes Service cluster.
I have two subscriptions named Visual Studio Enterprise. When I use Kubernetes: Add Existing Cluster the subscriptions I have are correctly listed but when I choose one of the Visual Studio Enterprise subscriptions and hit Next I get the following error reported:
The subscription of 'visual studio enterprise' has more than one match in cloud 'AzureCloud'.

Thanks for flagging this @simonvane.  We can fix this but our future strategy for the "add cluster" UI is via Cloud Explorer.  If you install the AKS extension (https://marketplace.visualstudio.com/items?itemName=ms-kubernetes-tools.vscode-aks-tools) you should be able to see your clusters in the Cloud Explorer (bottom pane of the Kubernetes activity bar) and add kubeconfigs from there.  Let us know if you hit the same problem there - I don't think we've ever tested this scenario!

Thanks for flagging this @simonvane.  We can fix this but our future strategy for the "add cluster" UI is via Cloud Explorer.  If you install the AKS extension (https://marketplace.visualstudio.com/items?itemName=ms-kubernetes-tools.vscode-aks-tools) you should be able to see your clusters in the Cloud Explorer (bottom pane of the Kubernetes activity bar) and add kubeconfigs from there.  Let us know if you hit the same problem there - I don't think we've ever tested this scenario!


Attack - Jailbreak prompt

Include an attack section on the UI, similar to how the defence mechanisms are shown. In this include a button which populates the chat input with a DAN attack. May need to give several options of jailbreak prompts. This requires UI design input as there isn't currently an attack section.

https://learnprompting.org/docs/prompt_hacking/jailbreaking
https://github.com/0xk1h0/ChatGPT_DAN

@PuneetLoona Prototyped this, just another section below the defence mechanisms. Unlike the defences, the attacks are not selectable, so they don't change colour or anything when clicked. There is also not a black border around them when they are hovered over.


Documentation request:  clarify why use this action

I'd like to see something in the README that explains why someone should use this action versus just calling 'docker login' directly.
I'm not trying to be critical or snarky, rather, it's just that when I review the README for this action, it's not apparent to me what functionality or value the action provides versus just running a one liner run: docker login  <loginserver> -u <username> -p <password>.   Would be great if some additional info or context was added.

When you use the above command, the secrets are exposed through the commandline for the process. So if your runner is compromised, your docker credentials are compromised.
The major use of this action is to avoid the above security issue.
We avoid it by creating the file which the same command eventually generates without logging your secrets anywhere; we create that file in a temp directory instead of the home of the runner. This triggers a clean up as soon as the workflow context ends.
The alternative for this action is using the command like this cat ~/my_password.txt | docker login --username foo --password-stdin also additionally cleaning up the my_password.txt file after your workflow is done.


[ru] Localize application-programming-interface.md

Description
This PR adds RU localization of Application Programming Interface
Checklist before opening this PR (put x in the checkboxes)

[x] This PR does not contain plagiarism

don’t copy other people’s work unless you are quoting and contributing it to them.

[x] I have signed off on all commits

signing off (ex: git commit -s) is to affirm that commits comply DCO. If you are working locally, you could add an alias to your gitconfig by running git config --global alias.ci "commit -s".

@tym83 please approve 🥺


Refactor connection & capture logic into components

Ready for Merge 👍  All suggested possible changes have been completed. While there may be minor bugs in this branch, I believe all of them existed before the refactor.

⚠**~NOT READY FOR MERGE~** ⚠
At least the following issues currently exist in this refactor:

[x] streamer: fix viewer count
[x] streamer: fix video fails be received by any receivers (issue on streamer side)
[x] receiver: infobar message not working

Update: With these functional problems fixed, I have suggested possible changes in the comment below.

Ready for Merge 👍  All suggested possible changes have been completed. While there may be minor bugs in this branch, I believe all of them existed before the refactor.


Enable Search for Resource Content File Types

Currently we can only search for various types of files using resource subject keywords, i.e. MATLAB, CSV, Jupyter Notebook. This is insufficient because these labels are usually not descriptive enough to identify resources based on the types of files stored within them. We need a method to explicitly search based on file extension. This would allow advanced searches such as precipitation AND type='.csv, .txt' which would query resources that match both "precipitation" and have either "csv" or "txt" files. This is particularly useful for web apps such as JupyterHub or MATLAB Online.

related to #4072

@danames I am emailing you and Tony about this right now.

This addresses the issue as I see it! Looks great. I think @Castronova  can close this one now.

@danames It's not clear to me that this issue is addressed in #4072 which enables search by aggregation type.

I think there is a subtle point here and Discover allows filtering by Content Type but not by file extension. Alva and I are aware of this and will work this GItHub Issue after several other priority items are handled in Q1/Q2.

Yes I only checked content type, not file type.
On Fri, Feb 12, 2021 at 10:47 AM cuahsimarko<EMAIL_ADDRESS>wrote:

I think there is a subtle point here and Discover allows filtering by
Content Type but not by file extension. Alva and I are aware of this and
will work this GItHub Issue after several other priority items are handled
in Q1/Q2.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/hydroshare/hydroshare/issues/4073#issuecomment-778343188,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA2PZ5AAMOHOWJ2GIIHKHNDS6VSUZANCNFSM4U3BW3OQ
.

--

Daniel P. Ames, Ph.D., P.E.
Professor, Civil & Environmental Engineering,Brigham Young University,
Provo, Utah, USA
Past-President, International Environmental Modelling & Software Society
Editor-in-Chief, Environmental Modelling & Software
http://ceen.et.byu.edu/content/dan-ames
https://ceen.byu.edu/directory/dan-p-ames
http://www.researchgate.net/profile/Daniel_Ames
http://scholar.google.com/citations?user=S0GUCeUAAAAJ

Additional info from deprecated https://github.com/hydroshare/hydroshare/issues/3600#issuecomment-783563230


[Bug]: gpt-4-turbo-2024-04-09 pricing missing

Describe the bug
Pricing is missing again for the newest model.
The current way of handling costs is not optimal. It should be possible to determine that the new model is a gpt-4-turbo model as well, as the prefix is the same.
Steps to reproduce
Use gather_usage_summary([Agent])
from autogen.agentchat.utils import gather_usage_summary
with:
[
{
"model": "gpt-4-turbo",
"api_key": "sk-xxx",
"max_tokens": 4000
}
]
Model Used
gpt-4-turbo-2024-04-09, gpt-4-turbo
Expected Behavior
calculate cost for gpt-4-turbo as per https://openai.com/pricing
Screenshots and logs
No response
Additional Information
No response

can you post some logs on the above issue? Thanks @WebsheetPlugin

v0.2.25 works for me:
'usage_excluding_cached_inference': {'gpt-4-turbo-2024-04-09': {'completion_tokens': 417,
                                                                 'cost': 0.04359,
                                                                 'prompt_tokens': 3108,
                                                                 'total_tokens': 3525},
                                      'total_cost': 0.04359},
 'usage_including_cached_inference': {'gpt-4-turbo-2024-04-09': {'completion_tokens': 417,
                                                                 'cost': 0.04359,
                                                                 'prompt_tokens': 3108,
                                                                 'total_tokens': 3525},
                                      'total_cost': 0.04359}}

Because you have latest version. I would prefer an solution that not requires me to git pull latest version for every model update...

v0.2.25 works for me:
'usage_excluding_cached_inference': {'gpt-4-turbo-2024-04-09': {'completion_tokens': 417,
                                                                 'cost': 0.04359,
                                                                 'prompt_tokens': 3108,
                                                                 'total_tokens': 3525},
                                      'total_cost': 0.04359},
 'usage_including_cached_inference': {'gpt-4-turbo-2024-04-09': {'completion_tokens': 417,
                                                                 'cost': 0.04359,
                                                                 'prompt_tokens': 3108,
                                                                 'total_tokens': 3525},
                                      'total_cost': 0.04359}}

Because you have latest version. I would prefer an solution that not requires me to git pull latest version for every model update...

A) It seems that curently you are hardcoding the price for each model type
seperatly. But if it's not present, then you should simple compare the
prefix. So in this case, I believe the prefix "gpt-4-turbo" is enough to
understand what price range it is :)
B) keep in mind that gpt-4-turbo is now  automaticly pointing to that new
model. Which means that all running aplication with earlier version stop
calculating costs right. So it's an huge problem.
C) As an additonial idea. Which might be good for non openai models, which
pricing were not added. I Propouse to allow setting prompt/completion
prices via llm_settings.
czw., 18 kwi 2024, 13:48 użytkownik Chi Wang @.***>
napisał:

I'd like that too 😄 . How?
—
Reply to this email directly, view it on GitHub
https://github.com/microsoft/autogen/issues/2424#issuecomment-2063675702,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AUEKDGVSNZ2XGCPH3AXL6W3Y56XITAVCNFSM6AAAAABGMG7QLOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANRTGY3TKNZQGI
.
You are receiving this because you were mentioned.Message ID:
@.***>

The issues in A) and B) are true, though I don't have a good solution. The solution in C) makes sense by having the price provided by the user.

Do you think simply checking if "gpt-4-turbo-2024-04-09" starts with "gpt-4-turbo" is not good? I think it's better than calculating the price "0" :)
If you don't like the solution proposed above, then at least let's do C) so I can enter the prices manually and not worry about new model updates.

Do you think simply checking if "gpt-4-turbo-2024-04-09" starts with "gpt-4-turbo" is not good? I think it's better than calculating the price "0" :)
If you don't like the solution proposed above, then at least let's do C) so I can enter the prices manually and not worry about new model updates.

I think it's still a one-time solution and doesn't address the dynamic nature of the problem. C) sounds good to me. Feel free to make a PR.


Optimize contract size

IBC contract (approximately 41k) exceeds Ethereum's hard cap 26k on contract size. Axon can increase this number but not sure how the chain will be impacted. Evaluation and test are needed for this issue.
Refer

https://ethereum.org/en/developers/tutorials/downsizing-contracts-to-fight-the-contract-size-limit

@KaoImin discuss on here

Warning: Contract code size is 42966 bytes and exceeds 24576 bytes (a limit introduced in Spurious Dragon). This contract may not be deployable on mainnet. Consider enabling the optimizer (with a low "runs" value!), turning off revert strings, or using libraries.

ref: https://github.com/synapseweb3/ibc-solidity-contract/actions/runs/5709524579/job/15468457656#step:7:222

We should implement IBCQuerier and CellEmitterHandler similar to other handlers, i.e., delegatecall actual implementation. (Maybe use regular call and don't share storage for CellEmitterHandler.)

We should implement IBCQuerier and CellEmitterHandler similar to other handlers, i.e., delegatecall actual implementation. (Maybe use regular call and don't share storage for CellEmitterHandler.)

maybe it would reduce contract size, but I'm afraid it cannot solve the deployment problem due to the huge gap

the deployment problem due to the huge gap

What do you mean by the deployment problem due to the huge gap?

CellEmitterHandler can be a separate contract or use delegatecall. Queries can be proxies with delegtecall. If having many proxy functions and encoding/decoding params/results turn out to be a problem, we can use a single query proxy entrypoint function:
contract IBCHandler
{
    address queryContract;

    function query(bytes calldata data_) external returns (bool success, bytes memory data) {
        return queryContract.delegatecall(data_);
    }
}

Or we can use some alternative query implementation, e.g. by indexing events.


Change URL to point directly to files on framabag.org

This will prevent the package to "break" as soon as a new version is released (because of the checksum mismatch).
http://wllbg.org/latest-v2-package redirects to the latest release file on framabag.org (currently http://framabag.org/wallabag-release-2.1.4.tar.gz).
I asked about files location and one of the wallabag devs answered on gitter:

They are at http://framabag.org/wallabag-release-2.x.x.tar.gz but it's dirty, they will probably move in the future.

I added a comment with the old URL http://wllbg.org/latest-v2-package in case the direct one breaks.
Also updated the package to 2.1.4.

LGTM. Thanks.


Sentry contrib package.

Submit proxy and context representation to the breadcrumbs the same way we do with py.test plugin.

https://docs.sentry.io/clients/python/advanced/#sanitizing-data

Usage together with Django: https://docs.sentry.io/clients/python/integrations/django/#setup
# project.settings
RAVEN_CONFIG = {
    'processors = ('stories.contrib.sentry.StoryProcessor',)
}

Add stories as breadcrumbs:
>>> from raven import breadcrumbs
>>> def processor(data):
...     data.update({
...         'message': """
... Story.method:
...     pass
...
... Context:
...     a = 1
... """,
...         'category': 'story',
...     })
...
>>> breadcrumbs.record(processor=processor)
>>> from raven.contrib.django.raven_compat.models import client
>>> client.captureException()


fix: xread(_group) timeout should be None by default

xread and xread_group check the timeout parameter to apply blocking operation. If the timeout is None it will be a non-blocking operation. But the default value of timeout parameter was 0.

@gstamac -
Thanks for the submission! Maintenance for this project has resumed.
Just need a couple things from you to get this merged:

[ ] Add an entry to the CHANGES directory.
[ ] Add yourself to CONTRIBUTORS.txt.
[ ] Sign the CLA.

This change has been superseded by #891.


Sign of weight in downvote

This line is same as in upvote: https://github.com/GolosChain/golos-smart/blob/46be1a693a28e7c2215c664391542d858bc902de/golos.publication/golos.publication.cpp#L249 Here must be item.weight = -weight; if action accepts positive weight. Or action must check that weight sign as expected: positive or zero in upvote and negative in downvote

#81

Related to #60


Pretrained subclasses nn.Module now

Addresses issue #987.
I've tried to make some sane choice for the forward() method of each subclass. Some models cannot pass gradients through their decode procedure in any meaningful way - is that unexpected for a forward() method? Should we only pick methods that pass gradients?

Hi @Gastron ,
let me fork this PR and work on an the proposed solution using import lib and external injection of the forward function


Remove Admin API used for Data Migration

What this PR does / why we need it?
This PR removes the Admin API that was previously used for Data Migration. As the Data Migration process has been completed, there is no need for this API anymore.
Any background context you want to provide?
The Admin API was created specifically to handle Data Migration. Now that the migration process has been finished, the API is no longer necessary and can be safely removed.
What are the relevant tickets?

No relevant tickets.
Checklist

[x] Added relevant tests or not required
[x] Didn't break anything

 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.


Botplay L Rank

What is your question?
I tried any song with botplay enabled then finishing a song, appears Results as a L Rank
They not appeared the L Rank while Botplay Enabled nothing wrong Or ELSE?

That is known as a ghost L


Use Normalize.css logo

@DanielPintilei was kind enough to design a truly sleek and memorable logo after I made a request on Twitter. Normalize.css is probably well enough known not to require any branding, but art is another way we can express the commitment and satisfaction we share from building this software together.

I don't think a css file needs a logo :)

logo looks promising & nice..!


Move demos and examples folder into docs

To give all users a good starting point, we should populate the docs.
It would be nice to have some "Getting started" section, listing a few examples:

How to get xrefs
get resources
print all classes using X
find all API calls using a permission
...

As the examples are used in the tests, move them into tests folder

I removed the demos folder in the jadx branch...

the examples folder is basically the folder for the binaries used in tests... more documentation would be nice though...


Error when installing packages to R virtual environment

Issue
I cannot restore project from lockfile. From this post, I possibly need to be using an older version of R? Currently on R v4.3.3 and renv v1.0.7.
Traceback
I created the R environment as below:

renv::init()
This project already has a lockfile. What would you like to do?

1: Restore the project from the lockfile.
2: Discard the lockfile and re-initialize the project.
3: Activate the project without snapshotting or installing any packages.
4: Abort project initialization.

Then when trying to restore project from the lockfile (i.e. selecting option 1) I get the following error:

Installing packages --------------------------------------------------------

Installing MASS ...                           FAILED
Error: Error installing package 'MASS':
================================

installing source package ‘MASS’ ...
** package ‘MASS’ successfully unpacked and MD5 sums checked
** using staged installation
** libs
using C compiler: ‘Apple clang version 15.0.0 (clang-15<IP_ADDRESS>)’
using SDK: ‘MacOSX14.4.sdk’
clang -arch arm64 -I"/Library/Frameworks/R.framework/Resources/include" -DNDEBUG   -I/opt/R/arm64/include    -fPIC  -falign-functions=64 -Wall -g -O2  -c MASS.c -o MASS.o
MASS.c:37:23: error: unknown type name 'Sint'; did you mean 'int'?
VR_sammon(double *dd, Sint *nn, Sint *kd, double *Y, Sint *niter,
^~~~
int
MASS.c:37:33: error: unknown type name 'Sint'; did you mean 'int'?
VR_sammon(double *dd, Sint *nn, Sint *kd, double *Y, Sint *niter,
^~~~
int
MASS.c:37:54: error: unknown type name 'Sint'; did you mean 'int'?
VR_sammon(double *dd, Sint *nn, Sint *kd, double *Y, Sint *niter,
^~~~
int
MASS.c:38:20: error: unknown type name 'Sint'; did you mean 'int'?
double *stress, Sint *trace, double *aa, double *tol)
^~~~
int
MASS.c:168:8: error: unknown type name 'Sint'; did you mean 'int'?
static Sint ord;               / ranks of dissimilarities */
^~~~
int
MASS.c:169:8: error: unknown type name 'Sint'; did you mean 'int'?
static Sint ord2;              / inverse ordering (which one is rank i?) /
^~~~
int
MASS.c:170:8: error: unknown type name 'Sint'; did you mean 'int'?
static Sint n;                  / number of  dissimilarities /
^~~~
int
MASS.c:171:8: error: unknown type name 'Sint'; did you mean 'int'?
static Sint nr;                 / number of data points /
^~~~
int
MASS.c:172:8: error: unknown type name 'Sint'; did you mean 'int'?
static Sint nc;                 / # cols of  fitted configuration */
^~~~
int
MASS.c:181:31: error: unknown type name 'Sint'; did you mean 'int'?
VR_mds_fn(double *, double *, Sint *, double *, Sint *,
^~~~
int
MASS.c:181:49: error: unknown type name 'Sint'; did you mean 'int'?

VR_mds_init_data(Sint *pn, Sint *pc, Sint *pr, Sint *orde,
^~~~
int
MASS.c:188:28: error: unknown type name 'Sint'; did you mean 'int'?
VR_mds_init_data(Sint *pn, Sint *pc, Sint *pr, Sint *orde,
^~~~
int
MASS.c:188:38: error: unknown type name 'Sint'; did you mean 'int'?
VR_mds_init_data(Sint *pn, Sint *pc, Sint *pr, Sint *orde,
^~~~
int
MASS.c:188:48: error: unknown type name 'Sint'; did you mean 'int'?
VR_mds_init_data(Sint *pn, Sint *pc, Sint *pr, Sint *orde,
^~~~
int
MASS.c:189:4: error: unknown type name 'Sint'; did you mean 'int'?
Sint *ordee, double *xx, double *p)
^~~~
int
fatal error: too many errors emitted, stopping now [-ferror-limit=]
20 errors generated.
make: *** [MASS.o] Error 1
ERROR: compilation failed for package ‘MASS’
removing ‘/Users/u2176058/Documents/trachoma-endgame/trachoma-amis-integration/renv/library/R-4.3/aarch64-apple-darwin20/.renv/1/MASS’
install of package 'MASS' failed [error code 1]
Traceback (most recent calls last):
14: renv::init()
13: restore(project = project, library = libpaths, repos = repos,
prompt = FALSE)
12: renv_restore_run_actions(project, diff, current, lockfile, rebuild)
11: renv_install_impl(records)
10: renv_install_staged(records)
9: renv_install_default(records)
8: handler(package, renv_install_package(record))
7: renv_install_package(record)
6: withCallingHandlers(renv_install_package_impl(record), error = function(e) writef("FAILED"))
5: renv_install_package_impl(record)
4: r_cmd_install(package, path)
3: r_exec_error(package, output, "install", status)
2: abort(all)
1: stop(fallback)

Thanks Thibault! Went ahead using R 4.3


Circular references

I don't think this should be supported, because of the extra overhead either in performance or in API complexity, but I think the documentation should state clearly that it's not handled. I mean, reading the code this is obvious, but it would still be a good service :)
P.s. do you align with the handmade manifesto? If you haven't heard of it, do check it out, because it seems you do! 😊🙌
Cheers

Don't be silly, the overhead in both complexity, memory use, and code size is negligible (see for yourself https://github.com/AsyncBanana/microdiff/commit/3b6ab8fc6c12759b9a2910fa08480154ffa4f2f9), and it serves the important purpose of not creating situations where using this library would potentially crash the process or exhaust your server's memory. For such a small addition (less than 2 lines of code) that's a huge win.

That totally depends on the size of the objects. If the user knows that there are no circular references, why have the runtime check it at all?
For big JSON blobs with many objects, this will severely impact performance by having to

reallocate the array
linearly search for references for every addition

Sure, it is assumed that it is small, but performance almost doesn't matter in the small anyway, so why micro-optimize then?

Benchmark it.

I don't have the incentive to put that amount of time in, I don't even use the lib. Just wanted to put this issue out there.

I will try to benchmark this later. If the results show that cyclical reference checking slows nested objects down enough, I will add an option to disable the checking.

For most objects with low depth, as expected, there is little to no difference. However, with highly nested objects, I noticed a difference of around 30%. Should I add an option to disable it? Having the choice would increase code size slightly and reduce performance slightly, but it would overall be faster if you disable checking, and you can still keep checking if you want.

Great work! 🎉🙌
One thing we could try before using a flag is to use a Set instead of an Array. How does that affect performance?
I think a 3% pessimization or so is where I'd feel alright about it personally, because if one only diffs large JSON blobs which just arrived serialized over the network, there is no need to check for circular dependencies, so it would feel silly to miss out on something like every 20th diff (in the case of 5% e.g.)

I decided to add an option in v1.2.0, so by default, cycles are checked, but you can disable it as specified in the README and release notes.

Nicely done!


Upping jtwig-core

Coverage increased (+0.04%) to 97.927% when pulling 5fb4d299508358f6716a2b978d936e2eca25c7a5 on lyncodev:master into bc23f6166188dcfe978e2cfdd8734bdeb4c6e46f on jtwig:master.


kubectl create secret is missing the --save-config flag

What happened:
There is a --save-config flag for the create subcommand:
$ kubectl create --help
[…]
Options:
[…]
    --save-config=false:
        If true, the configuration of current object will be saved in its annotation. Otherwise, the annotation will
        be unchanged. This flag is useful when you want to perform kubectl apply on this object in the future.
[…]

It works for most object types, but not secret:
$ kubectl create --save-config configmap foo --dry-run=client -o yaml
apiVersion: v1
kind: ConfigMap
metadata:
  annotations:
    kubectl.kubernetes.io/last-applied-configuration: |
      {"kind":"ConfigMap","apiVersion":"v1","metadata":{"name":"foo","creationTimestamp":null}}
  creationTimestamp: null
  name: foo

$ kubectl create --save-config secret generic foo --dry-run=client -o yaml
error: unknown flag: --save-config
See 'kubectl create secret --help' for usage.

What you expected to happen:
I expect the flag to work as documented.
How to reproduce it (as minimally and precisely as possible):
kubectl create --save-config secret generic foo --dry-run=client -o yaml

Environment:
$ kubectl version --client --short
Flag --short has been deprecated, and will be removed in the future. The --short output will become the default.
Client Version: v1.26.7
Kustomize Version: v4.5.7

Hi @lindhe I was able to reproduce your issue, but foun that if you switch the order this way [1] it works fine. Let me know your outcome.
[1]:
> kubectl create secret generic foo --save-config --dry-run=client -o yaml apiVersion: v1 kind: Secret metadata:   annotations:     kubectl.kubernetes.io/last-applied-configuration: |       {"kind":"Secret","apiVersion":"v1","metadata":{"name":"foo","creationTimestamp":null}}   creationTimestamp: null   name: foo

Agreed, I get the same results.
For me, that have not looked into the code much, this sounds pretty strange. The --save-config flag is documented as a flag to create, so putting that flag as an argument to create secret is surprising.

The order matters in spf13/cobra. kubectl first needs to identify the command create configmap, create secret , etc.. And after that flags (i.e. --save-config) should be added. If flags are added between create and configmap, that breaks the identification of command. Even though, command is identified successfully, that results in the ignorance of --save-config.
I'd prefer closing this as not a bug.

If that had consistently been the case, I would agree that this is not a bug. But it seems like kubectl create secrets behave differently w.r.t. this compared to other kubectl create x commands. I believe that there is a bug somewhere here, that makes create secret behave differently than (all?) other create subcommands.
And if the flag should indeed go after x in kubectl create x, not before it, I think the documentation (particularly the --help texts) should reflect that. As it stands, kubectl create --help shows --save-config as a flag, implying that kubectl create takes the flag.

I investigated this issue;
First task of cobra is to find the correct command being executed. kubectl create --save-config configmap and kubectl create configmap --save-config are equivalent because there is no other command creating any ambiguity. cobra successfully continues other executions by assuming that the command is create configmap.
On the other hand, kubectl create secret --any-flag generic(or kubectl create --any-flag secret generic) is not equivalent to kubectl create secret generic --any-flag because for the former, cobra detects create secret as command(be aware not create secret generic) which doesn't have any --save-config flag and for the latter, it detects create secret generic which has --save-config command.
I understand the inconvenience in here. But suggested way is to use flags after the commands;
kubectl create secret generic --save-config foo --dry-run=client -oyaml

I'd prefer closing this as it works as intended.
/close

Thank you for the explanation. It makes more sense to me now. Is it possible to remove it from kubectl config --help since it should not be used as a flag for that command?


Repeated fragment in auto-generated query.

I've written a GraphQL request which uses several times the same fragment through different fragments (see the code below and https://github.com/coq/bot/blob/bc65314888df8e8c6c8bad55624f9492a36a1e78/bot-components/GitHub_GraphQL.ml#L30-L47 for the full context).
  fragment Column on ProjectColumn {
    id @ppxCustom(module: "ID")
    databaseId
  }

  fragment Project on Project {
    columns(first:50) {
      nodes { ... Column }
    }
  }

  fragment ProjectCards on ProjectCardConnection {
    nodes {
      id @ppxCustom(module: "ID")
      column { ... Column }
      project { ... Project }
    }
  }

The auto-generated query contains two copies of the Column fragment, this makes the GraphQL query ill-formed.
This is all the more annoying that, in this case, GitHub's GraphQL API doesn't answer with a clear error message but with:
{
  "data": null
}

(separately reported to GitHub)

Ah interesting! How would that work in JS?

I'm an OCaml dev, so I'm not sure I can answer that. But what I did to debug the problem (because I didn't understand why GitHub was answering with an empty data field) was to print the request it was sending, then I confirmed with curl that it was not working:
curl https://api.github.com/graphql -X POST -H "Authorization: bearer $GH_TOKEN" -d '{"query":"query GetMilestoneMergedPullRequests($owner: String!, $repo: String!, $number: Int!)  {\nrepository(owner: $owner, name: $repo)  {\nmilestone(number: $number)  {\npullRequests(first: 100, states: [MERGED])  {\nnodes  {\n...PullRequestWithMilestoneAndCards   \n}\n\n}\n\n}\n\n}\n\n}\nfragment PullRequestWithMilestoneAndCards on PullRequest   {\nid  \ndatabaseId  \nnumber  \nmilestone  {\n...Milestone   \n}\n\nprojectCards(first: 20)  {\n...ProjectCards   \n}\n\n}\nfragment Milestone on Milestone   {\nid  \nnumber  \ntitle  \ndescription  \n}\nfragment ProjectCards on ProjectCardConnection   {\nnodes  {\nid  \ncolumn  {\n...Column   \n}\n\nproject  {\n...Project   \n}\n\n}\n\n}\nfragment Column on ProjectColumn   {\nid  \ndatabaseId  \n}\nfragment Project on Project   {\ncolumns(first: 50)  {\nnodes  {\n...Column   \n}\n\n}\n\n}\nfragment Column on ProjectColumn   {\nid  \ndatabaseId  \n}\n","variables":{"owner":"coq","repo":"coq","number":37}}'

Then I tested the auto-generated query in GitHub's GraphQL explorer:
query GetMilestoneMergedPullRequests($owner: String!, $repo: String!, $number: Int!)  {
  repository(owner: $owner, name: $repo)  {
    milestone(number: $number)  {
      pullRequests(first: 100, states: [MERGED])  {
        nodes  {
          ...PullRequestWithMilestoneAndCards
        }

      }

    }

  }

}
fragment PullRequestWithMilestoneAndCards on PullRequest   {
  id
  databaseId
  number
  milestone  {
    ...Milestone
  }

  projectCards(first: 20)  {
    ...ProjectCards
  }

}
fragment Milestone on Milestone   {
  id
  number
  title
  description
}
fragment ProjectCards on ProjectCardConnection   {
  nodes  {
    id
    column  {
      ...Column
    }

    project  {
      ...Project
    }

  }

}
fragment Column on ProjectColumn   {
  id
  databaseId
}

fragment Project on Project   {
  columns(first: 50)  {
    nodes  {
      ...Column
    }

  }

}
fragment Column on ProjectColumn   {
  id
  databaseId
}

and it highlighted the first Column fragment with the message There can be only one fragment named "Column".

Another simpler (and complete) example where it triggered this issue:
  fragment Milestone on Milestone {
    id
  }

  fragment PullRequest on PullRequest {
    id
    milestone { ... Milestone }
  }

  query issueMilestone($owner: String!, $repo: String!, $number: Int!) {
    repository(owner:$owner, name:$repo) {
      issue(number:$number) {
        id
        milestone { ... Milestone }
        timelineItems(itemTypes:[CLOSED_EVENT],last:1) {
          nodes {
            ... on ClosedEvent {
              closer {
                ... on PullRequest { ... PullRequest }
                ... on Commit {
                  associatedPullRequests(first: 2) {
                    nodes { ... PullRequest }
                  }
                }
              }
            }
          }
        }
      }
    }
  }

I think some production users haven't hit this issue because they use the template tag option (default for rescript-apollo-client). What client are you using? I think this should be fixed regardless btw.

I am calling graphql-ppx directly (I am not aware of any available client for native OCaml).

This issue didn't happen with lower versions? (because handling of this shouldn't have changed too much I think)

Before upgrading to 1.2.0, I didn't use fragments as much because the object encoding didn't make them as essential. So I don't think it's a regression but rather that I had never had an occasion of encountering this issue before.

Hey @jfrolich, do you think this issue is reasonably easy to fix? I tried to look at the code to see if I could help, but I don't understand the code enough to manage to do that.

BTW, you were right that (at least some) JS GraphQL clients do some deduplication: https://github.com/apollographql/graphql-tag/issues/27

I encountered the same issue while trying to implement a data-layer using GraphQL PPX. It generated a duplicate fragment, which yields in server error.


Bot doesn't respond to any commands.

pb !
pb!
!pb
pb help!
/pb
/ pb!
/ !pb
/pb help
!pb help
Nothing works.

The default command prefix is pb! for help is pb!help to change the prefix pb!setprefix <prefix>
If you forked this project, prefix can be found at:
https://github.com/mashwishi/PruneBot/blob/master/config/config.json
https://github.com/mashwishi/PruneBot/blob/master/config/settings.json


cec: support resource name qualification for HTTP HealthCheckFilter

Currently, Envoy resource names within a CiliumEnvoyConfig are qualified by prefixing them with the namespace and name of the CiliumEnvoyConfig. This affects the names itself - but also references from other Envoy resources.
This commit introduces support for qualifying the cluster references of the Envoy HTTP HealthCheckFilter.

/test

/test


Release 2024-12-20

:tada: This issue has been resolved in version 2.6.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Plans on POV demo analyser

Hi, any plans on implementing POV demo analyzer? I've tested valve tool on github and it can read POV demos so I assume that it's not that hard to do if you already implemented GOTV, in terms of code adjustments. Or there are some issues with POVs?

You should be able to parse POV demo if you use the SLOW_PROTOBUF condition when compiling.

@MrMinicop What exactly do you mean? AFAIK both POV and GOTV demos use the same file format which demoinfo is able to parse. Are you getting wrong data from your POV demos? Is the parser crashing?

@main-- I've tried to use demo project to parse POV demos but it generated no data. I'm not pretty sure but as far as I remember there was a note somewhere that POV demo parsing isn't working. Maybe demo project is for GOTV demos only?
@0BuRner I'll try, thanks. Is it documented somewhere?

@MrMinicop What exactly do you mean with that? Are you just not getting any gameevents? Are there never any entities or they don't move? We never consciously limited this project to POV demos and definitely ran some early tests with those, so I'd expect them to work.
Regarding the SLOW_PROTOBUF option mentioned by @0BuRner: All you need to do is clone the git repo, enable the SLOW_PROTOBUF compile option for the demoinfo project and add the protobuf dependency. However, this should not be necessary in practice as SLOW_PROTOBUF is not designed to have any use in production (it's mostly intended for demoinfo debugging). In particular, it doesn't change any major code paths.

@main-- I'll double check POV demos again and get back with information

@main-- Results are ready. I've used this demo project
https://github.com/EHVAG/demostatistics-generator/blob/master/README.md#usage-of-demoinfo-public
When I open GOTV demo with it - it returns xls with stats. When I open POV demo - I get this error:
System.NullReferenceException: Object reference not set to an instance of an object.    at DemoInfo.DP.Handler.PacketEntitesHandler.Apply(PacketEntities packetEntities, IBitStream reader, DemoParser parser)    at DemoInfo.PacketEntities.Parse(IBitStream bitstream, DemoParser parser)    at DemoInfo.DP.DemoPacketParser.ParsePacket(IBitStream bitstream, DemoParser demo)    at DemoInfo.DemoParser.ParseDemoPacket()    at DemoInfo.DemoParser.ParseTick()    at DemoInfo.DemoParser.ParseNextTick()    at DemoInfo.DemoParser.ParseToEnd()    at StatisticsGenerator.MainClass.Main(String[] args) in E:\Downloads\demostatistics-generator-master\StatisticsGenerator\Program.cs:line 209
Line 209 is:
parser.parseToEnd()

That does look like a bug to me - does this happen with all POV demos?
By the way, you can get a better stack trace if you clone the repo and add a project reference instead of using nuget.

@main-- I've tried to record own pov demos and to download them from hltv.org - same result. Interesting fact - after error is thrown, new empty csv file is created with correct map name in the filename.
Also tried to build with project reference, here is the stack:
at DemoInfo.DP.Handler.PacketEntitesHandler.Apply(PacketEntities packetEntities, IBitStream reader, DemoParser parser) in E:\Downloads\demoinfo-master\DemoInfo\DP\Handler\PacketEntitesHandler.cs:line 47
   at DemoInfo.PacketEntities.Parse(IBitStream bitstream, DemoParser parser) in E:\Downloads\demoinfo-master\DemoInfo\DP\FastNetmessages\PacketEntities.cs:line 31
   at DemoInfo.DP.DemoPacketParser.ParsePacket(IBitStream bitstream, DemoParser demo) in E:\Downloads\demoinfo-master\DemoInfo\DP\DemoPacketParser.cs:line 38
   at DemoInfo.DemoParser.ParseDemoPacket() in E:\Downloads\demoinfo-master\DemoInfo\DemoParser.cs:line 653
   at DemoInfo.DemoParser.ParseTick() in E:\Downloads\demoinfo-master\DemoInfo\DemoParser.cs:line 633
   at DemoInfo.DemoParser.ParseNextTick() in E:\Downloads\demoinfo-master\DemoInfo\DemoParser.cs:line 544
   at DemoInfo.DemoParser.ParseToEnd(CancellationToken token) in E:\Downloads\demoinfo-master\DemoInfo\DemoParser.cs:line 529
   at DemoInfo.DemoParser.ParseToEnd() in E:\Downloads\demoinfo-master\DemoInfo\DemoParser.cs:line 520
   at StatisticsGenerator.MainClass.Main(String[] args) in E:\Downloads\demostatistics-generator-master\StatisticsGenerator\Program.cs:line 209

Hey,
there are indeed two different bugs with POV-demos. One is fairly easy to resolve (as @0BuRner said, by enabling SLOW_PROTOBUF) - however the other one will be a bit nasty, and no demoparser implements it yet, and will lead to getting wrong data from demos.
This is definitively open for a pull-requests. Especially to make them at least parse with SLOW_PROTOBUF shouldn't be too hard. Sadly neither me nor @main-- can really help you on the details, for various reasons. However, if you have a pull-request we'll gladly review and merge it.

@moritzuehling I would like to help with coding, but I'm not a C# developer :(
I can help only with testing, so if someone needs to test something - I'm ready.

i ran into the same problem, but i fail to understand on howto enable SLOW_PROTOBUF - is there some documentation somewhere ?

Hello Crimsdings, look at the Microsoft-Documentation on this topic: https://msdn.microsoft.com/en-us/library/4y6tbswk.aspx


Update Fantasy.css

Just some fixed and code sorting.
Ice, just wait before approving this. I'm working on this file and I will let you know when this is good.

There will be changes added to this PR as I'm working on a few things and getting things fixed up and setup so its fits better for this custom #loginContiner and sorting things out. Again this should not be approved till I think its ready. I am working on this on my AMP Dev server so Ice can see what I'm working on if he wants to.
Any and all feed back is welcome if they want to pull this and see what I'm doing.

Please don't open a PR until you're ready for it to be merged. Otherwise, it sends lots of needless notifications.


use JedisCluster to read RedisBloomFIiter

Expected behavior
I can customize jedis to read redis bloomfilter
but I can't use JedisCluster to read redis bloomfliter.
Actual behavior
JedisClusterCommand don't customize.
Jedis version:3.0.0-m1
Java version: jdk 1.8

@joyang1 want to open a PR to add that feature?

ok, I'm implementing this features and then open a PR.

@marcosnils https://github.com/xetorthio/jedis/pull/1905, If use think no problem, please merge this request and release this version. I'm very anxious to use this function.
thanks!!!
😄 😄

moved to https://github.com/RedisLabs/JReBloom/issues/3


Log messages exceeding size limits lost

Describe the bug
When parsing long/multiline log entries (usually due to stack traces and debug information) they often exceed the maximum size resulting in the message being rejected with the following error:

[2022/10/20 00:29:28] [ warn] [output:stackdriver:stackdriver.0] error
{
"error": {
"code": 400,
"message": "Log entry with size 679.5K exceeds maximum size of 256.0K",
"status": "INVALID_ARGUMENT"
}
}

When using a look ahead regex expression with the parse_regex processor there isn't an easy way that I'm aware of to limit the size of the message.
To Reproduce
Steps to reproduce the behaviour:

Create a log receiver with a parse_regex processor with a lookahead expression such as the following:
^(?<time>\d{4}-\d{2}-\d{2}T\d{2}:\d{2}:\d{2},\d{3}) (?<severity>[^ ]*) (?<msg>[\s\S]*?)(?=\g<time>|\Z)
2.Read in a log entry that exceeds the 256KB limit:

2022-10-20T00:22:29,900 INFO [main] Test message1
2022-10-20T00:22:29,941 INFO [main] **** PAD THIS OUT WITH A 256KB MULTILINE PAYLOAD  ***
2022-10-20T00:22:29,969 INFO [main] Test message2

Expected behaviour
A message should be written to stackdriver to indicate that a log had been dropped rather than it just be silently lost and/or a processor should be provided to allow specific fields in the payload to be truncated based on a maximum length so we can at least get a log entry and part of the message.

Hi @jgtr-blis thanks for opening an issue. I am actively working on this. I don't know if we have a public issue tracker for it at the moment, but I will keep this issue updated with progress.

Hi @braydonk, is this issue still being worked on? I've been running into a similar issue for some time now. However we're only using "parse_json" and "modify_fields", not the "parse_regex_ processor OP mentioned.


updateVisionSource replacement?

It seems like token.updateVisionSource() was removed and I can't find a way to force update token vision after adding/removing perceptive's vision channel. It's used in "Toggle VC receiver filter of selected Token" macro which is broken right now.

The bug should now be fixed with v4.2.2.

I mean, it doesn't really fix the issue, since updateVisionSource() does not exist anymore and vision is not updated unless I move the token or reselect it.

Ah, right, sorry, could have sworn that there was reason for why it was no longer necessary in v12. I will look into adding a nother way to update the vision

Should now be fixed with v4.2.3


Rename env variable POSTGRES_PORT -> PGPORT

Postgres requires PGPORT environment variable set if we need to run it on a different port.
This PR

Renames POSTGRES_PORT env variable to PGPORT so that we can run the app with the postgres port we need. Api service fails to connect to postgres without this change
Creates .env.template file for easier developer onboarding, and updates README

@DMalone87 one more PR here


Invite to org

Name:
Why would you like to join the organisation?

Check your Email or org Page Click here

You can also show that you are part of this org, by making yourself public. This will show our profile org on your profile:
Follow This:

After you join the organisation, search your name in the list of people. 🔍
Follow this below: ⬇

haven't been added yet

Let me check

Done invite sent successfully

You can also show that you are part of this org, by making yourself public. This will show our profile org on your profile:
Follow This:

After you join the organisation, search your name in the list of people. 🔍
Follow this below: ⬇

🎉🎊Welcome to Jaidev Studio🎉🎊!!

Why not contribute to...
Unique-Github-Profiles!

haven't been added yet

🎉🎊Welcome to Jaidev Studio🎉🎊!!

Why not contribute to...
Unique-Github-Profiles!

You can also show that you are part of this org, by making yourself public. This will show our profile org on your profile:
Follow This:

After you join the organisation, search your name in the list of people. 🔍
Follow this below: ⬇

do this first


bug: can't exit "filter" view

Steps

trigger <C-f> to open filter view

Expectation
I should able to quit/cancel the filtering by either doing "esc" and/or "q"

Yeah agree on this!
Fixed by #48


Add tddwizard/magento2-fixtures to library list

Codecov Report

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

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


fix : Fix the bug in FAQ section, when I am clicking on the title of a faq

What does this PR do?
FIxes #81
https://github.com/WikiPortal/DoodleCollab/assets/40324640/97292094-3801-4260-a241-3b27b972e73c

Since I've not assigned someone, and someone else has worked on it before you,      thus closing this PR


Error in xcode 7

Hello
while building on xcode 7 an error occurred:
No such module 'KMPlaceholderTextView'
any clue ?

Hi, could you try to open VoiceMemos.xcworkspace?

thnks mozhouqi its working now
btw i tried recoarding in backgraound and it works well but as soon as i play music it stops rec
any clue do u think there is a way to fix it

Yes, you could not record and play at the same time.

is there a work around for this it wont work at all?

You could try to change the recording and playback logic, and set the category of AVAudioSession.sharedInstance() to be AVAudioSessionCategoryPlayAndRecord.


has to be type "string" but received type "object"

After updating js-controller to 3.3.8. the log is full of the following info:
State value to set for "deconz.0.lights.00112345672123.xy" has to be type "string" but received type "object"

Is fixed with version 1.3.15


RSC15e as a part of 0.9

Please review and merge if everything looks good.

Please review and merge if everything looks good.

@mattheworiordan I was sure that test for what you describing already exists, no?
In my PR I only added test for custom rest host, to be sure that request will try it instead of default rest.ably.io.
Could you please take a look again, do I still need any changes here?

In my PR I only added test for custom rest host, to be sure that request will try it instead of default rest.ably.io.

But I am not really clear why this test is needed then given providing a custom restHost is covered in many places, including of course http://docs.ably.io/client-lib-development-guide/features/#TO3k2.  We know that restHost is used by those tests, so I am not clear what we are achieving then if a test for http://docs.ably.io/client-lib-development-guide/features/#TO3k2 already exists?
Can you explain as I feel like we're just duplicating tests unnecessarily now?

You are right, I see that there are couple of places to test overridden restHost.
But please understand me, I just wanted to test it after I read this diff from 0.8:
(unless overriden in ClientOptions#restHost).
So I will close this useless PR, thanks for the feedback @mattheworiordan. :)

Ok, no problem. I am hoping to set up a call with Dmitry and all of you shortly so that we can discuss next steps as perhaps some guidance of expected test coverage would be sensible moving forwards.

Closed as discussed above.


Use v86 for command prompt component

copy/v86#1 (the issue doesn't matter, just a link) could be used as a drop-in solution for Command Prompt.

For https://98.js.org/ I did a similar thing but using JS-DOS. It's very pointless, not being connected to anything! and it doesn't support text selection like the real command prompt.
But you could drop in my solution and change the icon and "MS-DOS Prompt" to "Command Prompt", update the intro text. (To get the intro text right I made a batch file for it to run at start up that clears the screen and prints some text.)
Note the Command Prompt window in Windows XP is normally shown in Windows Classic style.


Fire() deletes variables without asking

Just ran into this when I added
hook.StacktraceConfiguration.Enable = true
to one of my projects and could not understand why it suddenly made my error field (logging with logrus.WithError(...)) disappear from my logfiles..
Turns out it was logrus_sentry "consuming" the error as part of sending it to Sentry.  Looking at the code it does it with several other "special" fields as well, like "http_request".
Not sure what the best solution is here, but this should be configurable by the caller in some way - otherwise it's bound to cause these type of headaches for other people as well.

Why are fields deleted anyhow? I checked some other community developed Hooks, and I didn't see any other hook dropping fields. Could we simply remove this?

@njern /cc @GlenDC
I think some special fields are removed to avoid duplicate data and large payload size when it used.
PR #25 is not to delete keys from logrus.Data and not to affect other hooks.
@njern does this PR solve your problem? if so, I'll merge it.
I'm planning to add an option flag to preserve original data of "special fields" and send duplicated data to Sentry.

Fixed in v0.2.7


npm install and npm usage documentation missing

Alternatively, whatwg-fetch is missing as an import here:
https://github.com/pretenderjs/pretender/blob/6cdf4e5c2383e43de6c9b17d694dd5bee817f6ca/index.d.ts#L1-L2

@jherdman , if you are on browser environment the assumption is that route-recognizer, fake-xml-http-request and whatwg-fetch/dist/fetch.umd.js are installed together with pretender, so they are dependencies. You can take a look at ember-cli-pretender's implementation.
For index.d.ts, can you explain more on what should be imported here?

Thank you, that definitely helps. I think we can use this information to beef up the README with installation instructions — things currently are a bit murky.
I'm a bit of a TypeScript neophyte, so I don't fully understand the significance of the index.d.ts file. But it seems strange to me that
https://github.com/pretenderjs/pretender/blob/6cdf4e5c2383e43de6c9b17d694dd5bee817f6ca/pretender.js#L19
references code that isn't necessarily loaded. And that
https://github.com/pretenderjs/pretender/blob/6cdf4e5c2383e43de6c9b17d694dd5bee817f6ca/pretender.js#L13-L15
references code that looks to be loaded in the index.d.ts file. This line of thinking comes from my above noted FakeFetch error, but I don't see a RouteRecognizer error (maybe it's loaded as a side dependency already???).

@jherdman you don't need to worry about index.d.ts file since mirage is not using typescript. If you're curious about how typescript def loading works, you can checkout their documentation.

Thank you again for the reply, @xg-wang . I really appreciate your time.


MTOUCH build error for iOS on Azure DevOps

The following error occurred when upgrading Xam.Plugin.Media to 5.0.1, I had to rollback to <IP_ADDRESS> for the build to succeed. All other platform builds succeed against 5.0.1.
MTOUCH : error MT2101: Can't resolve the reference 'System.Void CoreImage.CILanczosScaleTransform::set_InputImage(CoreImage.CIImage)', referenced from the method 'UIKit.UIImage Plugin.Media.UIImageExtensions::ResizeImageWithAspectRatio(UIKit.UIImage,System.Single)' in 'Xamarin.iOS, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=84e04ff9cfb79065'.
Azure Devops Agent Specification: macOS-10.14
Xamarin.Forms: <IP_ADDRESS>0310

Seems as though you have an older version of Xamarin build tools on that machine, you will want to make sure the latest versions with latest xcode are installed as this is an iOS 13 api and required 2019 to be installed. Take a look https://devblogs.microsoft.com/xamarin/boots-xamarin-ci/

@jamesmontemagno That seemed to do the trick thanks! The most frustrating thing about Xamarin development for me is managing packages, so many things break when I update dependencies and want to build on DevOps, by the way I love your video content online with tips, keep up the good work!
FYI to get things working I updated MonoVersion from 6.4.0 to 6.8.0 on the hosted build agent, this change also allowed me to move from macOS 6.14 to 6.15 too


Attestations: take snapshot of state at point of attestation

Too big, and not enough people asking for it


Now bot can use berries. Yeah! + Fix #660, #666

If catch rate is low (under 50%). Thrown the berry to its face. Make them know who is the boss!

Fix

https://github.com/PokemonGoF/PokemonGo-Bot/issues/660
https://github.com/PokemonGoF/PokemonGo-Bot/issues/666

don't pull yet. resolve conflicts and fux bugs now.

This should be fine now. please merge it!

@TheSavior updated. review again please.

Would like to have it merged 👍

I like it too!
:+1:

👍

But need solve CI failed and another +1 need.

do a new push and CI check will trigger re-run.

@earthchie Please do a git rebase and then push so the travis CI can run and I will merge in

don't pull yet. Testing in progress.

Alright. tested!
ready to pull

Testing this now...


On rotate the slider is not updated right

I noticed that if you rotate the device with Auto-layout set on the slider, the view is not updated.
This will cause issues like: the rounder edges are not rounder anymore and the slider image is bigger then expected.
Solution:
Add "self.setNeedsDisplay()" at the end of "func update()".
Marco

Thanks.
You can create pull request with your fix and I will approve.


Column-first iteration for Error Diffusion

Transposes the ED filter matrices in the Error Diffusion constructor to iterate over CartesianIndices(img) instead of rows of the image.
I'm temporarily transposing reference tests until they are passed in this PR, then I will update the references.

Not sure why tests pass on binary images but fail on color images.


Expiration time' claim ('exp') is too far in the future

Bug Report
Current Behavior
Following current getting started guide, that is finishing the Configuring a GitHub App, results in an application that throws 401 Unauthorized on a new issuewith the ERROR	 (HttpError): 'Expiration time' claim ('exp') is too far in the future error
I used the npx binary as per the documentation, and chose the basic-ts typescript template.
import { Application } from 'probot' // eslint-disable-line no-unused-vars

export = (app: Application) => {
  app.on('issues.opened', async (context) => {
    const issueComment = context.issue({ body: 'Thanks for opening this issue!' })
    await context.github.issues.createComment(issueComment)
  })

Environment

Probot version(s): 10.9.5
Node/npm version: v14.14.0/6.14.8
OS: Fedora 32

Can you reproduce the problem? It's most likely caused by time difference between your system time and GitHub's system time. I thought we accounted for that case in https://github.com/octokit/auth-app.js/pull/164, but it seems we missed something 🤔

Tried it just now, I see the same issue. Restarted chronyd.service to ensure the best possible time. Is the issue not reproducible on your end?
The stack trace seems to be:
ERROR	 (HttpError): 'Expiration time' claim ('exp') is too far in the future
    HttpError: 'Expiration time' claim ('exp') is too far in the future
        at /home/mczernek/repos/rht/test/node_modules/@octokit/request/dist-node/index.js:66:23
        at processTicksAndRejections (internal/process/task_queues.js:93:5)
        at async getInstallationAuthentication (/home/mczernek/repos/rht/test/node_modules/@octokit/auth-app/dist-node/index.js:258:7)
        at async hook (/home/mczernek/repos/rht/test/node_modules/@octokit/auth-app/dist-node/index.js:424:7)
        at async Job.doExecute (/home/mczernek/repos/rht/test/node_modules/bottleneck/light.js:405:18)
...
// event log
...
INFO	 (http): POST / 401 - 242ms
ERROR	 (probot): Unauthorized
    Error: Unauthorized
        at Request.callback (/home/mczernek/repos/rht/test/node_modules/superagent/lib/node/index.js:883:15)
        at IncomingMessage.<anonymous> (/home/mczernek/repos/rht/test/node_modules/superagent/lib/node/index.js:1126:20)
        at IncomingMessage.emit (events.js:327:22)
        at IncomingMessage.EventEmitter.emit (domain.js:486:12)
        at endReadableNT (_stream_readable.js:1327:12)
        at processTicksAndRejections (internal/process/task_queues.js:80:21)
status: 401

I cannot, but it's great that you can! Let's try to create a minimal test case!
Can you try to run this code on your machine to see if it reproduces the problem and share the output if it does?
https://runkit.com/gr2m/probot-probot-1426

I wrapped the whole repro (excluding the imports) in an async function so that it can be executed. Then, I got the following 401 response:
{
  token: 'public-token-here',
  expiresAt: '2020-12-01T07:57:15.000Z',
  diff: 569.602
}
RequestError [HttpError]: 'Expiration time' claim ('exp') is too far in the future
    at /home/mczernek/repos/rht/test/node_modules/@octokit/request/dist-node/index.js:66:23
    at processTicksAndRejections (internal/process/task_queues.js:93:5)
    at async test (/home/mczernek/repos/rht/test/test.js:41:26) {
  status: 401,
  headers: {
    'access-control-allow-origin': '*',
    'access-control-expose-headers': 'ETag, Link, Location, Retry-After, X-GitHub-OTP, X-RateLimit-Limit, X-RateLimit-Remaining, X-RateLimit-Used, X-RateLimit-Reset, X-OAuth-Scopes, X-Accepted-OAuth-Scopes, X-Poll-Interval, X-GitHub-Media-Type, Deprecation, Sunset',
    connection: 'close',
    'content-length': '121',
    'content-security-policy': "default-src 'none'",
    'content-type': 'application/json; charset=utf-8',
    date: 'Tue, 01 Dec 2020 07:47:03 GMT',
    'referrer-policy': 'origin-when-cross-origin, strict-origin-when-cross-origin',
    server: 'GitHub.com',
    status: '401 Unauthorized',
    'strict-transport-security': 'max-age=31536000; includeSubdomains; preload',
    vary: 'Accept-Encoding, Accept, X-Requested-With',
    'x-content-type-options': 'nosniff',
    'x-frame-options': 'deny',
    'x-github-media-type': 'github.v3; format=json',
    'x-github-request-id': 'A87B:4EDC:DDF582:FBE1E9:5FC5F4F7',
    'x-xss-protection': '1; mode=block'
  },
  request: {
    method: 'GET',
    url: 'https://api.github.com/app',
    headers: {
      accept: 'application/vnd.github.v3+json',
      'user-agent': 'octokit-request.js/5.4.10 Node.js/14.14.0 (linux; x64)',
      authorization: 'Bearer [REDACTED]'
    }
  },
  documentation_url: 'https://docs.github.com/rest'
}

Note that the date in the response is Tue, 01 Dec 2020 07:47:03 GMT. Date on my computer was 08:47 AM. I am in CEST, which is GTM+1. Not sure whether that's relevant.

Thank you for sharing. This is very odd indeed. The 569.602 means that the expiration time is roughly 9minutes in the future, well within the 10min limit.

Note that the date in the response is Tue, 01 Dec 2020 07:47:03 GMT. Date on my computer was 08:47 AM. I am in CEST, which is GTM+1. Not sure whether that's relevant.

The time zone should make no difference.
What I do think is happening is that your machine's time and GitHub's API's server time is out of sync by a bit over 1 minute, which would trigger the error.
The expiration time is 2020-12-01T07:57:15.000Z, it is set 9 minute into the the future, so your local time must have been 2020-12-01T07:48:15.000Z
The server API time was 2020-12-01T07:47:03.000Z at the time the request was received (equals Tue, 01 Dec 2020 07:47:03 GMT)
new Date('2020-12-01T07:48:15.000Z') - new Date('2020-12-01T07:47:03.000Z') is 72000 (72 seconds)
Can run date and compare minutes/seconds with the Date response header of curl -I https://api.github.com | grep "date"? In my case, the two are exactly the same, down to the second
date && curl -I https://api.github.com | grep -Fi "date"
Tue Dec  1 10:04:23 PST 2020
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0  2308    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0
date: Tue, 01 Dec 2020 18:04:23 GMT

Sure:
Tue 01 Dec 2020 07:17:01 PM CET
date: Tue, 01 Dec 2020 18:16:18 GMT

I get RequestError [HttpError]: 'Expiration time' claim ('exp') is too far in the future with the difference above (just verified again to make sure that the error still happens desipte the time difference being less than 1m).
However, it seems that you are correct. When I set the time to that of GitHub (sudo date +%T -s $(curl -sI https://api.github.com | grep -Fi "date" | awk '{ print $6 }') -u), the test case succeets:
$ node test.js
{
  token: 'public-token-here',
  expiresAt: '2020-12-01T18:37:05.000Z',
  diff: 569.146
}

Is it expected that the time window for the authn is so small? After setting the time, I can no longer reproduce the issue even after re-starting chronyd again.
I do worry about the issue cropping up again though; it seems inevitable that the time diff becomes ~40s again since re-synchronizing chronyd did not solve the issue (chronyc -a makestep)...

Don't worry, I'll make sure we have solid solution for this. As I mentioned above, we already account for two related errors caused by time differences between your system and GitHub API's:
https://github.com/octokit/auth-app.js/pull/164/files#diff-3f0d30441d44c825e178251d199e3ad61e2730d7a4d0ccbf882f8197bce87767R16-R26
It's all about reproducing the problem and creating a test in https://github.com/octokit/auth-app.js. If you would like to help, the tests would look like one of these: https://github.com/octokit/auth-app.js/blob/80c9daa5f161fc16b10f003bcb3487637e21340c/test/index.test.ts#L1216-L1481. I'm occupied with other work right now, but I'll get to it eventually

Any news?

I still have this in my backlog

I'm currently experiencing the same issue. The time of the servers runs a few seconds apart, but I get the expiration time error.
date && curl -I https://our.ghes-server.com | grep -Fi "date"
Fri Jan 21 09:19:35 CET 2022
Date: Fri, 21 Jan 2022 08:18:52 GMT

Also having this same issue. Anything being done to resolve it?

I am also having this exact issue

@rlvk-tc
Did you sync the time on your machine?

@rlvk-tc
Did you sync the time on your machine?

I've not done anything to my machine. How would I sync my machine?

It depends on your operating system. Just google it yourself.

It depends on your operating system. Just google it yourself.

That makes sense, I'll try to investigate if this might be my issue and if it helps to sync it. Thanks.

We still need a reproducible test case as we don't seem to cover all possibilities of out-of-sync times between GitHub's API servers and app servers: https://github.com/probot/probot/issues/1426#issuecomment-736744661

We could integrate a ntp time check, and warn if the system clock is more than 10 seconds or so. Does github have a self-owned ntp server?

Does github have a self-owned ntp server?

I have no idea, but it sounds like overkill? @octokit/auth-app is taking care of out-of-sync time already, it just seems that we missed a case.


Switch from chacha20-poly1305-aead to chacha20poly1305 in the default resolver

Hi, chacha20-poly1305-aead hasn't been updated for 4 years, so we should switch to chacha20poly1305 which is actively maintained.

Nice, see also this PR to add XChaCha20Poly1305 using the chacha20poly1305 crate: https://github.com/mcginty/snow/pull/73

Looks so much cleaner than the past implementation we were using, thank you!!

@mcginty done.

Merged and edited your commit message to match the PR. Thanks!


Mounting NFS shared folders fails on MacOSX on vagrant 1.7.4

After going through the setup steps in the README for running locally, my vagrant instance fails to mount the NFS shared folders:
$ vagrant up
Bringing machine 'herokuwp' up with 'virtualbox' provider...
==> herokuwp: Box 'ubuntu/trusty64' could not be found. Attempting to find and install...
    herokuwp: Box Provider: virtualbox
    herokuwp: Box Version: >= 0
==> herokuwp: Loading metadata for box 'ubuntu/trusty64'
    herokuwp: URL: https://atlas.hashicorp.com/ubuntu/trusty64
==> herokuwp: Adding box 'ubuntu/trusty64' (v20150609.0.10) for provider: virtualbox
    herokuwp: Downloading: https://atlas.hashicorp.com/ubuntu/boxes/trusty64/versions/20150609.0.10/providers/virtualbox.box
==> herokuwp: Successfully added box 'ubuntu/trusty64' (v20150609.0.10) for 'virtualbox'!
'...
==> herokuwp: Configuring and enabling network interfaces...
==> herokuwp: Exporting NFS shared folders...
==> herokuwp: Preparing to edit /etc/exports. Administrator privileges will be required...
Password:
The nfsd service does not appear to be running.
Starting the nfsd service
==> herokuwp: Mounting NFS shared folders...

The following SSH command responded with a non-zero exit status.
Vagrant assumes that this means the command failed!

mount -o 'vers=3,udp' <IP_ADDRESS>:'/Users/alexortiz/Development/heroku-wp' /app

Stdout from the command:

Stderr from the command:

stdin: is not a tty
mount.nfs: Connection timed out

I found this stackoverflow answer that seems to suggest the config format for setting up NFS mounts in version 1.6+ is different:
http://stackoverflow.com/questions/26700249/cannot-mount-vagrant-synced-folder-with-nfs

Found the solution for the problem here https://github.com/mitchellh/vagrant/issues/2546
The correct syntax for vagrant version 1.3 to 1.6 is:

config.vm.synced_folder ".", "/vagrant", :nfs => { :mount_options => ["dmode=777","fmode=777"] }

I'm using a brand-new install of vagrant:
```shell
$ vagrant --version
Vagrant 1.7.4

I'm on MacOSX 10.10.3

I did some further digging and it seems that nfsd service is not started, despite what the startup log suggests:
$ vagrant ssh
$ sudo service nfs-kernel-server status
nfsd not running

When I try starting nfsd, it fails:
$ sudo service nfs-kernel-server start
 * Not starting NFS kernel daemon: no exports.

As the service startup message indicates, /etc/exports doesn't have any export NFS configs:
$ cat /etc/exports
# /etc/exports: the access control list for filesystems which may be exported
#		to NFS clients.  See exports(5).
#
# Example for NFSv2 and NFSv3:
# /srv/homes       hostname1(rw,sync,no_subtree_check) hostname2(ro,sync,no_subtree_check)
#
# Example for NFSv4:
# /srv/nfs4        gss/krb5i(rw,sync,fsid=0,crossmnt,no_subtree_check)
# /srv/nfs4/homes  gss/krb5i(rw,sync,no_subtree_check)
#


Contributing guide + improved devex + direct users to the ask forum

Waiting on https://github.com/replit/replit-docs-2/pull/265

I added a contributing.md file and also improved the devex for developing with this on Replit.

btw what's is-prod-frozen doing in this PR?

yeah agree, the links will probably rot

Are we good here?


1.20 Update

Are there any plans on updating this to work with the new followers/necklaces which come with the newest content addition update?

Maybe!


no stable connection to indego 500+

Since yesterday I am not able to get a stabil connection to my indego.
In the logs I found the following lines:
2021-10-01 09:57:21 WARNING (MainThread) [custom_components.indego] Uncaught error: init() got an unexpected keyword argument 'renew_date' on index: 0
2021-10-01 09:57:21 INFO (MainThread) [custom_components.indego] Update state got an exception: init() got an unexpected keyword argument 'enabled'
2021-10-01 09:57:21 DEBUG (MainThread) [custom_components.indego] Refreshing state.
2021-10-01 09:57:21 DEBUG (MainThread) [custom_components.indego] Refreshing initial operating data.
2021-10-01 09:57:21 INFO (MainThread) [custom_components.indego] Update state got an exception: init() got an unexpected keyword argument 'enabled'
2021-10-01 09:57:21 DEBUG (MainThread) [custom_components.indego] Refreshing state.
2021-10-01 09:57:22 INFO (MainThread) [custom_components.indego] Update state got an exception: init() got an unexpected keyword argument 'enabled'
2021-10-01 09:57:22 DEBUG (MainThread) [custom_components.indego] Refreshing state.
2021-10-01 09:57:22 INFO (MainThread) [custom_components.indego] Update state got an exception: init() got an unexpected keyword argument 'enabled'
2021-10-01 09:57:22 DEBUG (MainThread) [custom_components.indego] Refreshing state.
2021-10-01 09:57:22 INFO (MainThread) [custom_components.indego] Update state got an exception: init() got an unexpected keyword argument 'enabled'
2021-10-01 09:57:22 DEBUG (MainThread) [custom_components.indego] Refreshing state.
2021-10-01 09:57:22 INFO (MainThread) [custom_components.indego] Update state got an exception: init() got an unexpected keyword argument 'enabled'
2021-10-01 09:57:22 DEBUG (MainThread) [custom_components.indego] Refreshing state.
2021-10-01 09:57:23 INFO (MainThread) [custom_components.indego] Update state got an exception: init() got an unexpected keyword argument 'enabled'
2021-10-01 09:57:23 DEBUG (MainThread) [custom_components.indego] Refreshing state.
2021-10-01 09:57:23 INFO (MainThread) [custom_components.indego] Update state got an exception: init() got an unexpected keyword argument 'enabled'
2021-10-01 09:57:23 DEBUG (MainThread) [custom_components.indego] Refreshing state.
2021-10-01 09:57:23 INFO (MainThread) [custom_components.indego] Update state got an exception: init() got an unexpected keyword argument 'enabled'
2021-10-01 09:57:23 DEBUG (MainThread) [custom_components.indego] Refreshing state.
2021-10-01 09:57:24 INFO (MainThread) [custom_components.indego] Update state got an exception: init() got an unexpected keyword argument 'enabled'
2021-10-01 09:57:24 DEBUG (MainThread) [custom_components.indego] Refreshing state.
2021-10-01 09:57:24 INFO (MainThread) [custom_components.indego] Updating operating data
2021-10-01 09:57:27 DEBUG (MainThread) [custom_components.indego] Refreshing state.
2021-10-01 09:57:30 DEBUG (MainThread) [custom_components.indego] Refreshing state.
2021-10-01 09:57:33 DEBUG (MainThread) [custom_components.indego] Refreshing state.
2021-10-01 09:57:36 DEBUG (MainThread) [custom_components.indego] Refreshing state.
2021-10-01 09:57:39 DEBUG (MainThread) [custom_components.indego] Refreshing state.
2021-10-01 09:57:42 DEBUG (MainThread) [custom_components.indego] Refreshing state.
2021-10-01 09:57:45 DEBUG (MainThread) [custom_components.indego] Refreshing state.
2021-10-01 09:57:48 DEBUG (MainThread) [custom_components.indego] Refreshing state.
2021-10-01 09:57:51 DEBUG (MainThread) [custom_components.indego] Refreshing state.

Same here since yesterday. Do they have a problem with their api?

Same here since yesterday. Do they have a problem with their api?

the apps for iOS and Android working normal.
Maybe a change in their API?
2021-10-01 09:57:21 WARNING (MainThread) [custom_components.indego] Uncaught error: init() got an unexpected keyword argument 'renew_date' on index: 0
-> this line make me worry about.

Just to say that I have the same problem, too. AFAIK you need to restart HA to get it trying to reconnect. Yesterday it still worked for a while after restart but not anymore. I noticed that the grauonline webinterface has occassional problems too.

I have the same issue too.
The Bosch android app seems to never connect when I am on my home wifi (which has AdGuard protecting it) but is always able to connect when I am on 4G connection.  I cannot get HA to connect at all.
I suspect Bosch have made some changes to the API.  Perhaps ad blockers are a factor for other people in here too?  Perhaps we need to whitelist some new domains.
Are you guys also running ad blockers?

I have no adblocker in place and issues too

OK, then this must be a more global issue.
I just whitelisted "bosch-si.com" in my AdGuard instance and it was still broken thereby confirming it's probably not an ad blocker issue.
I wonder why the app works great on 4G but never on my home wifi then...

I have build a new version of the python library used by the Bosch integration. Please try this change in your config/custom_components/indego/manifest.json:
"requirements": ["pyIndego==2.0.29"],
change to
"requirements": ["pyIndego==2.0.30"],
Please let me know if it works for you!
/J

New version of component released. Should be visible in HACS shortly.