Enhancement: Improving Accessibility and Performance

Hey! The code had minor issues which are fixed.

@harsxv Please let me know if there is anything!


PHOENIX-6188 Jenkins job history uses too much storage (addendum: fix…

… artifact dir regression)

:confetti_ball: +1 overall

Vote
Subsystem
Runtime
Comment

+0 :ok:
reexec
0m 32s
Docker mode activated.

_ Prechecks _

+1 :green_heart:
dupname
0m  0s
No case conflicting files found.

+1 :green_heart:
@author
0m  0s
The patch does not contain any @author tags.

_ master Compile Tests _

+0 :ok:
mvndep
8m 19s
Maven dependency ordering for branch

_ Patch Compile Tests _

+0 :ok:
mvndep
0m 11s
Maven dependency ordering for patch

+1 :green_heart:
whitespace
0m  0s
The patch has no whitespace issues.

_ Other Tests _

+0 :ok:
asflicense
0m  0s
ASF License check generated no output?

9m 13s

Subsystem
Report/Notes

Docker
ClientAPI=1.40 ServerAPI=1.40 base: https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-38/1/artifact/yetus-general-check/output/Dockerfile

GITHUB PR
https://github.com/apache/phoenix-connectors/pull/38

Optional Tests
dupname asflicense

uname
Linux c33f5cb10546 4.15.0-60-generic #67-Ubuntu SMP Thu Aug 22 16:55:30 UTC 2019 x86_64 x86_64 x86_64 GNU/Linux

Build tool
maven

Personality
dev/phoenix-connectors-personality.sh

git revision
master / b5f62a5

Max. process+thread count
52 (vs. ulimit of 30000)

modules
C:  U:

Console output
https://ci-hadoop.apache.org/job/Phoenix/job/Phoenix-Connectors-PreCommit-GitHub-PR/job/PR-38/1/console

versions
git=2.7.4 maven=3.3.9

Powered by
Apache Yetus 0.12.0 https://yetus.apache.org

This message was automatically generated.


What is it with Rust crates never ever caring to add git tags on the repo?

Hey, it'd be useful for me to package this software for ArchLinux if git tags are provided. If you care about preserving the history of the project, tags are useful too.

Thank your for offering to port to AUR! I think release tags slipped past us in our rush to get things together to announce (we're only 0.3 anyway). I assume most other Rust crates don't have git tags because their authors prefer to stay indefinitely pre-1.0 and just install through cargo.
We also need to figure out a build script before we're for platform distribution. Specifically,

Wayland depdendencies
Installing wlc
Installing other Rust dependencies (they are statically linked in the binary, I assume just running cargo build when we're building from source)
Move our init file to /etc/way-cooler or .config/way-cooler
Later, we'll have some Lua utils to move to /usr/local or something

We also need to update rustwlc to the latest wlc version (been too busy with way-cooler).
@Timidger anything I'm missing?

Yep, looks like all of the dependencies. @vinipsmaker, I went ahead and added tagged the 0.3 release, so you should be all set to make a PKGBULD for Arch.

Thanks guys.
Packages are up:

https://aur.archlinux.org/packages/way-cooler/
https://aur.archlinux.org/packages/way-cooler-git/


chore: bumping min version of shillelagh

SUMMARY

This bumps the minimum version of shillelagh to a higher minimum version to account for releases that have fixed some bugs with creating google sheets.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

did you run pip-compile-multi on this?

@eschutho no, but I don't think that you need to because shillelagh isn't on the list of required installs in the superset's setup.py.

@eschutho ran pip-compile-multi --no-upgrade on this


Remove the native frame around the application

@Crecket this looks awesome +1

@Crecket Looks nice! but now you cant minimize and maximize the screen?


Fix useful spelling

The word usefull doesn't exists, its either use-full or useful and here useful

 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.

Thx !


Repo sync for protected CLA branch

The pull request is created from main638028504873060398sync_temp to main to fix git push error for protected CLA branch

Learn Build status updates of commit 667f97e:
:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the learn.microsoft.com contributor guidesPost your question in the Learn support channel


Edit - Setttings and View Tabs doesn't present any options

First of all, thank you for improving OneNote, I actively use it in Linux, Windows and Mac.
On my Linux, the new version has bug on Edit, View and Settings Tabs. Because all three shows broken content instead of listing options (regardless of opening my notebook) which you can see the images with the following link: OneNote Bug
As a feature request, it would be nice if the OneNote remembers the last opened notebook and shows it directly when it is started. I don't like to go Navigate-> OneNote Home every time when I start the oneNote app. I think this feature was already supported on the old version.

First of all, thank you for improving OneNote, I actively use it in Linux, Windows and Mac.

You're very welcome. The update was long overdue.

On my Linux, the new version has bug on Edit, View and Settings Tabs. Because all three shows broken content instead of listing options (regardless of opening my notebook) which you can see the images with the following link: OneNote Bug

That is curious. Looks like some of those menu options are not being rendered correctly by Electron or your DE. I could use a bit more information to reproduce and fix the problem at my end:

Which version of Linux you are using? Ubuntu/Arch/Fedora and the release.
What's your desktop environment/window manager? Gnome, KDE, XFCE, Openbox etc
Are you using X11 or Wayland?
Which package are you using? The .deb, .AppImage or Snap package?
Also, it looks like Navigate and Help are working fine?
My testing enviroments are Ubuntu 20.04 and 21.10, and Manjaro. but I mostly use KDE and XFCE. From the images it seems like your have Ubuntu with Gnome

As a feature request, it would be nice if the OneNote remembers the last opened notebook and shows it directly when it is started. I don't like to go Navigate-> OneNote Home every time when I start the oneNote app. I think this feature was already supported on the old version.

The app should load the last opened notebook when the app is relaunched. I'll check this behavior. I had changed it during testing. Let me see if I forgot to change it back.

I apologize for the inconvenience. But, while I try to test this, would you mind trying out the following steps?

Method 1:
This will only clear local app settings and reset to default. Your login and app history and everything OneNote related will stay intact. This app does not directly make any changes to OneNote.
A. If you installed the app using the .deb package:

Close the app
Go to /home/<user>/.config/onenote-desktop or /home/<user>/.config/OneNote
Delete the config.json file
Restart the app.
OR
B. If you are using .snap, i.e. you used sudo snap install onenote-desktop or used the Ubuntu App Store to install the app
Close the app
Go to /home/<user>/snap/onenote-desktop/current/.config/OneNote/
Delete the config.json file
Restart the app.

Method 2:
If you don't mind losing you instance login. You will NOT lose any OneNote data. But you will just need to login again, and app settings will be reset to defaults. Not OneNote settings, just the Linux wrapper app.
Instead of deleting the config.json files from method 1, delete the entire config folder. I would recommend this step, but only if you have your login information and can easily login again.

I.e. if .deb installation, delete the folder /home/<user>/.config/onenote-desktop or /home/<user>/.config/OneNote
if Snap installation, delete the folder /home/<user>/snap/onenote-desktop/current/.config/OneNote/

Method 3:
You can achieve either or these results if you use the new Reset App Defaults (soft reset, Method 1) or Reset App - everything (hard reset, Method 2) options. You can find them in the Help section of the app menu.
I would really appreciate your feedback if you try any of these. Thanks

Hi, I just ran a few checks and all the menu items seem to render just fine for me. It could be that the Electron version update is causing some graphics glitches. I recommend manually resetting the app once. Method 2 mentioned above. It could be that the app settings and caches are not migrating correctly after the upgrade.

As a feature request, it would be nice if the OneNote remembers the last opened notebook and shows it directly when it is started. I don't like to go Navigate-> OneNote Home every time when I start the oneNote app. I think this feature was already supported on the old version.

About this, the app still remembers your last the last opened notebook and that should launch when the app is started again.
For more context, I just tested the app on Ubuntu 20.04 and 2.10, with XFCE, LXQT, Gnome and KDE desktop environments. Everything worked as intended. I was unable to reproduce the errors you are experiencing. A bit more information would be helpful.
Thanks, Anuj

Closed because issues already addressed in previous updates/fixes. And didn't hear back from the original author. I was unable to recreate the graphics issue without more information. I am assuming it was solved at their end.


prometheus-bird-exporter: init at 1.3.5-git

Motivation for this change
Add a Prometheus exporter for grabbing some metrics out of the BIRD routing daemon.
Things done

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

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

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

@ofborg test prometheus-exporters.bird

@ofborg test prometheus-exporters.bird

This is a semi-automatic executed nixpkgs-review. If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 108096 run on x86_64-linux 1

1 package built:

prometheus-bird-exporter

This is a semi-automatic executed nixpkgs-review. If you find some bugs or got suggestions for further things to search or run please reach out to SuperSandro2000 on IRC.
Result of nixpkgs-review pr 108096 run on x86_64-linux 1

1 package built:

prometheus-bird-exporter


Add support for standalone Asio

Hi, @laudrup
As we discussed in #79 , the pr is a simple attempt to add support for the non-boost version of wintls. It only contains modifications to the core code, excluding build system and test code. This is still a crude attempt, but should be sufficient for the header only version.
The main tricks are contained in config.hpp. Use C++ using alias to distinguish boost and non-boost versions, and use the WINTLS_USE_STANDALONE_ASIO macro to control whether to use the boost version.  Some boost functions and macros for assertions have been rewritten, and currently we don't depend on any boost components.
As you say, beast currently depends on boost-asio.  So in my opinion, if we use a library like beast, we can just use the boost version of winstl. Well, I'm not sure if there will be a problem mixing boost-asio with standalone-asio in beast.
There's definitely a lot that can be improved here. Looking forward to your review, thanks!

Thank you for your review. It seems that our thinking is feasible, it's so good!  I saw your comment and it was so valuable. But now, let's consider the test code first.
I'm not sure what to do with the test code, so I'll stick with the basic idea of this pr, which is to remove as many relevant boost dependencies as possible. Notice that the main dependency in the test code is beast, and we use stream.hpp from the beast test code. Unfortunately, beast's test code also depends on some of its own basic components.
Fortunately, rewriting them wasn't as hard as I thought it would be, and after rewriting this part of the file, we can get a stream.hpp that is basically free of boost dependencies (but there are still a few dependencies, such as some utility templates, which are features provided in the C++20 STL).
The added code structure is shown below:
└── test
    ├── utils
    │   ├── allocator.hpp
    │   ├── empty_value.hpp
    │   ├── error.hpp
    │   ├── fail_count.hpp
    │   ├── flat_buffer.hpp
    │   ├── impl
    │   │   ├── error.ipp
    │   │   ├── fail_count.ipp
    │   │   ├── flat_buffer.hpp
    │   │   ├── stream.hpp
    │   │   └── stream.ipp
    │   ├── stream.hpp
    │   └── test_config.hpp

As mentioned just now, this part of the code comes from Beast, with some additional modifications. This part of the code totals about 2k lines and can be used to replace the original dependency stream.hpp.  Now we can then use the namespace alias trick again to distinguish whether to use standalone-wintls or boost-wintls for testing.
Now, I've made a first attempt and passed unittest on the boost version. I can push these commit together. Or if you have a better idea, please let me know, thank you!

Yes, in effect what we've done is ported a minimal beast test component to our test code so that we don't depend on beast.  And then we can use this "minimal beast" to build our test code. I think the option "whether to use beast to build test code" should no longer be needed here.
If that's appropriate, I'll try to organize some commits and submit them :-D

@windowsair That sounds great. Looking forward to see the resulting commits.

Thanks a lot for this. Looks really good.
Some general questions/comments:
I'm not really sure I understand which parts of the test code requires either boost or C++17 apart from the Boost.Process part. I would very much like us to not depend on boost at all for this to make sense and as far as I can tell that should be possible.
I initially chose to target C++14 since that is the first C++ standard that Microsofts compiler clams to support. I would rather not change that unless there are some good reasons. It could potentially limit the amount of users able to use this but of course I have no idea how relevant that is. As far as I can tell, the compilers actually being used for testing this code all support C++17 well enough, but if we should target C++17 I would prefer to keep that in a separate discussion.
Regarding Boost.Process. It seems like it is only being used for starting and stopping a process with some command line arguments. Although the WIN32 API pretty terrible it should be relatively simple to write the required code for calling StartProcessEx and other functions to do the same. It doesn't seem like we need to do anything like redirecting streams and we even only have to support one platform.
If you don't feel like it, I might see if I can find the time to implement something to get rid of the Boost.Process dependency.
Thanks once more. This is definitely making good progress. And yes, I can have a look at updating the CI once everything else is in place.

Ah, I actually don't dislike Boost. At first I just wanted to get rid of the Beast dependency, because standalone-wintls couldn't work well with it. Hmmm, since you want the standalone version to not rely on Boost at all, it looks like we have more work to do.
Regarding C++17, the feature used is string_view, you can see the details in stream.hpp:
#if __cplusplus >= 201703L || (defined _MSVC_LANG && _MSVC_LANG >= 201703L)
#include <string_view>
using string_view = std::string_view;
#else
#include <boost/utility/string_view.hpp>
using string_view = boost::string_view;
#endif

Some member functions of the stream class use string_view . I'm not sure it's a good idea to just switch to std::string.
Regarding Boost.Process, I use this in my own projects: https://github.com/sheredom/subprocess.h
Quite simple header-only subprocess library. With it, I can replace Boost.Process in a few minutes. Of course, if you don't like to include external dependencies, I think this is still a good reference for implementing subprocess.

At first I introduced the standalone Beast test just to test standalone-wintls. As you know, Beast uses Boost-asio, while standalone-wintls uses non-Boost-Asio, which may lead to potential problems. Completely removing the Boost part of the test code was not what I originally intended. Of course if it can be done easily, why not do that 😄
I totally agree with you about string_view and Boost.Process. Also, for standalone-Asio, we might need to handle it via CMake FetchContent as well.
What is our next move? Is there any way I can help? Thank You.

If you want to try to make the tests work without depending on boost at all that would be great. Otherwise I can look into that at some point.
As far as I can tell, there are some minor issues that I've commented on that it doesn't look like you've addressed so if that is the indeed the case then please try to fix those.
Otherwise it is fine with me if I hijack this branch and do what is needed to extend the pipeline, update the documentation and probably even changing the name of this project/library.
I'll of course make sure you get the credit you deserve but it's up to you really how much more time you want to spent on this.

Okay, let's see how the code is performing so far.
The first patch is about the main code changes, which were made based on your comments. The other patches are some work to replace Boost, and now we really don't need any Boost code in our test code for standalone version.
The next step might be to add standalone-asio in Cmake, but before that let's see if there are any improvements to be made in these patches :D

Hi @laudrup,
hi @windowsair,
If I may voice an opinion from an interested observer:
I think the original effort from @windowsair is great. Making this library useful for people that do not want to depend on all of boost is probably a good idea.
But the additional efforts of removing all boost dependencies from the test code look way over the top for me.
Effectively forking other (large!) 3rd Party Libs into this Repository does not seem like a good idea to me, when we can just use boost..?

Hi, @jens-diewald ,
Thank you for your attention. As I said, it should work fine even if we don't remove all the Boost parts from the test code.
What I did in the last few patches in fact removed these few Boost dependencies:

Boost.Process: replaced by subprocess.h, about 1k lines code, single file.
string_view: replaced by string_view-lite, about 1700 lines code, single file.
variant: replaced by variant-lite, about 2800 lines code, single file.

If C++17 was available, that would be great, we could just use std::string_view and std::variant, but we chose C++14 as the target, so we can only replace these by bringing in 3rd libraries. It seems to me that it might be of some use to those who don't want to use Boost at all.
I'm not sure if this is too large. Maybe I misunderstood what you meant?

@windowsair First of all, thanks a lot for your great efforts on this PR.
Overall, it looks good now. One major thing though, I'd really prefer that we use CMake FetchContent for handling the external dependencies instead of copying them to this repository directly.
@jens-diewald Of course your input it very much appreciated. I don't have any use for this library myself (at least currently) so I appreciate all the input I can get.
So regarding your comment and some open questions to both of you whom I assume are interested in using this library and most likely have more experience with Windows than I do.
I assumed the only thing we really need from boost for the tests of this library is the test stream in boost.beast. Since that is more or less an internal thing used by the tests of Beast itself I think it can make good sense to copy our own version to this library. Vinnie from the Beast library has more or less encouraged me to do exactly that and it also makes it easier for change or extend the test stream if we would want that. I don't think that is at all unlikely.
The only other thing we actually use from boost is the process library. Since we only use that to start and stop a process, I think it's a bit overkill to pull in an external dependency just for that. It makes sense to use boost.process if we already have boost as a dependency, but writing a simple class that starts a process and stops it on destruction ought to be fairly trivial even if it involves the nastiness of the Win32 API.
@windowsair chose to pull in an external dependency for that which is fine, but I had definitely considered getting rid of that at some point and write the process wrapper code at some later point.
A bit to my surprise it turned out that we actually depend on either C++17 or boost for variant and string_view implementations, which leads me to a related but somewhat different issue.
I cannot remember exactly why I chose to target C++14 for this library initially, but as far as I remember I tried to make it easy for someone already using boost.asio to use this library without any additional requirements on compiler, toolchain etc. Since MSVC only started to be somewhat standards compliant after C++11 and only started supported setting a standards flag after C++14, that's what I aimed for. I'm fairly certain that asio requires at least C++11 so it was an effort not to divert to much from that.
On the other hand, the only compilers/toolchains we actually test with in our CI pipeline all supports C++17 just fine so it might make sense to drop that requirement and instead of supporting a specific C++ standard simply tests against a set of compilers which is more or less what is already being done anyway.
I wouldn't mind dropping the C++14 support and simply document that VS 2017 and VS 2019 is tested and supported while earlier versions might work. According to this official support for VS 2013 stops early next year so the question is really if it is worth the effort to support VS 2015? That version is not available in the Github actions anyway so it's not like it's even been tested.
I would love to hear you guys opinion on that. It would fit me just fine to drop the C++14 support, use std::variant and std::string_view for the tests. If someone really needs support for older versions they are free to open an issue or a pull request.
Please let me know what you think. If you think it sound reasonable I'll drop the C++14 flag and update the master branch so we can rebase this PR and hopefully get this merged soon.
Thanks a lot for your contributions to this project. I very much appreciate it.

Hi, @laudrup
Thank you for your guidance. I think breaking C++14 support might not be a good idea.
Although the CI already supports new enough compiler versions, but we can't be sure what environment the user is using. Perhaps they need the unit test programs we provide to determine if their current compiler and system environment will run the code properly.
Importing external libraries as FetchContent is easy to do, and I want to focus on the subprocess library here:
The functions we currently need are probably to start a process, terminate a process, and determine whether a process is alive or not. I'm not sure we'll need to read stdin/stdout and other functions in the future. The current 3rd libraries basically call some Win32API. Of course there is also some handling of process parameters, posix support and handling of imported functions (no include <windows.h>), and different compiler handling operations.
These parts are not complicated, but they are quite trivial. Third party libraries have taken care of all of this with only about 1000 lines code, which doesn't seem particularly large to me. This is why I tend to use 3rd libraries rather than implementing them myself.
I've prepared the relevant changes regarding the standalone-asio Cmake test. Once we reach a consensus, I will commit it ASAP.

Hi @laudrup and @windowsair,
sorry for the late reply, I did not find any time to respond during the holidays.
Regarding my own requirements for this library: I would be fine with C++20 and all of Boost.
But I am aware that not everyone is in that position. I do suspect, C++14 is still a reasonable requirement today.
@laudrup, what are you trying to get at regarding the end of support for VS 2013? C++14 is still supported (and the default) in VS 2022 and the GitHub pipeline only tests VS 2019 and VS 2022. Or am I missing something?
Also, I think that it is a reasonable assumption, that there are people who would be happy using Standalone WinTLS+Asio and expanding the group of potential users is great!
However, I do not think that it is anywhere near as important, to have the tests not depend on boost.
@windowsair raises the point, that even users may want to run the tests. I did not think of that, and I still do not really think that most (all?) users would want to do that. They should write their own tests for whatever-they-use-wintls-for.
That means, removing the Boost dependency from the tests would only be a simplification for contributors to this library that use the standalone version otherwise. (Also note that the tests depend on OpenSSL as well.)
On the other side - and that may be an opinion you do not share - I fear that introducing more external dependencies will make maintaining this library significantly more work in the long run, while there are only so many man-hours available even now.
So, in my opinion, having our own process wrapper/string_view/variant, be it another non-boost library or self-written, costs time that could better be used for something else.
Regarding the Beast test stream, I do not think that it is possible to reasonably use that with standalone WinTLS+Asio. I do think though, it may be fine to completely skip all tests using that stream for the standalone version. However, depending on the Beast test stream is a little bit awkward by itself. It would be nicer if asio would provide such a stream as an "official" testing facility by itself. So, I am not that opposed to forking the Beast test stream.
So, all in all, I think removing the Boost dependency from the tests costs time better used elsewhere.
If you choose to use the additional libraries, I agree that using FetchContent would be nicer that forking them into the repository.

Hi @windowsair,
Thanks a lot for you work on this and to @jens-diewald for the very valuable input.
I have tried to github actions a bit to test building with different C++ standards. In order to not get an unreasonable amount of test jobs that also means cutting down a bit on which versions of boost to test against. This was meant as a preparation for adding tests for both standalone asio and boost.asio.
I think it's about time I get this merged. While we can spend lots of time continuing discussing the pros and cons of thirdparty dependencies etc. I think it's better to get this merged to the master branch and tagged with a version so we can get this out there and potentially get some feedback.
So, @windowsair if you commit your latest changes I think the best way forward is for me to hijack this branch, remove the boost references from the name(space) and other updates so this will become asio-wintls instead. It might also involve some other minor changes to the changes you already made but I hope you're OK with that.
Then we can finally close this PR. Are you OK with that?

Hi, @laudrup ,
Sure. Let's start merging this!

Hi, @laudrup
The new patches add option ENABLE_STANDALOND_WINTLS to determine whether to enable the standalone version of wintls compilation.
There is also a new change in CI:
Using choco install openssl may fetch different versions of OpenSSL, one being 3.1.1 and the other being 3.2.0, which causes failures in our test cases.
I think this is probably not caused by the changes in this PR. Using the code from the master branch and explicitly specifying OpenSSL version as 3.2.0 will also result in failures for all test cases. See:
https://github.com/windowsair/boost-wintls/tree/openssl-test
https://github.com/windowsair/boost-wintls/actions/runs/7528290666
Openssl version 3.2.0 was recently introduced by choco. It looks like choco in CI randomly installs both version 3.1.1 and 3.2.0, and once it installs version 3.2.0, then our test case fails.  So another patch specified Openssl as version 3.1.1 to bypass this issue. We may need to investigate this issue in the future.
I think there are probably no more commits to push now. Feel free to modify them :)

@windowsair
I've picked up your branch and spent quite some time adding new commits. Some related to handle thirdparty code differently and mainly to rename this library from boost-wintls to simply wintls.
My plan is pretty much to merge this branch as it is and then rename this repository to wintls.
I think you should have a chance to have a look and comment before I do this. Of course your commits will still be in the history mostly unchanged (I've fixed a few conflicts I think). If you want more credit let me know.
@jens-diewald Your input would also be appreciated if you have any.
Thanks!

Hi, @laudrup
Looks really good. Lets see what others say.

I have merged these changes to master so I will close this pull request.
Thanks for you contribution @windowsair


Hiding the posts details table on each post

I am noticing a table categorized by the year of posts on each blog post
For e.g. on https://puneet-gupta.github.io/puneetgupta/2020/06/01/collect-network-trace-via-AutoHealing/

I want to display only the POST content when someone goes to the post and avoid showing this table and list of posts on every post. Is there a way to workaround this?

Get rid of this thing in Front Matter:

Super! That was exactly the problem. Thanks again!


Supporting table cell selection

Provide a general summary of the feature here
I've tried to rebuild Notion's style table with react-aria table to find out that cell selection is not supported. Sure you can hack your way around this with react state but it would be nice if react-aria table selection manages cell selection internally.
🤔 Expected Behavior?
When clicking on a cell you select it. Moving right or left with keyboard or hitting tab will move selection to the next table. If the user wants to select a row it click on the row square icon.
😯 Current Behavior
There is no way to select a cell in the current implementation.
💁 Possible Solution
No response
🔦 Context
I'm trying to build sophisticated tables like those of notions.so or glide https://grid.glideapps.com/
💻 Examples
No response
🧢 Your Company/Team
No response
🕷 Tracking Issue
No response

Thank you for the feature request! At the moment Spectrum Design doesn't have a pattern for this so unfortunately it is unlikely that we can pick this up in the short term. We'd be happy to accept a contribution for this or to prioritize if there is enough community interest for it.


Scrollbar still remains when removing all the items from the array

How to reproduce:

take the demo http://kamilkp.github.io/angular-vs-repeat/
in the form, set 0 as array size
apply

+1


Url.ArticulateCategoryUrl(Model, c) generating whitespace.

Sitemap crawlers are indexing the URL's generated by Url.ArticulateCategoryUrl with white space which is causing google searches to index the wrong pages.
So for example the following:
/blog/categories/Event of the week
is being indexed as:
/blog/categories/event%20of%20the%20week/
Is there anyway around this?

Hi,
What is the Umbraco and Articulate version you are using?

Umbraco version 7.5.4 assembly: 1.0.6136.27241
Articulate 2.0.5

That’s an old version of articulate. Can you try upgrading to v3?
Thx
Sent from my iPhone

On 15 Dec 2017, at 11:34, Ben-TangentDesign<EMAIL_ADDRESS>wrote:
Umbraco version 7.5.4 assembly: 1.0.6136.27241
Articulate 2.0.5
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Hi,
That version isn't compatible with the Umbraco currently installed. We can't upgrade the version due to other package requirements at this time.

Hi, so what is the actual problem with whitespace in the URLs?
The underlying problem with changing the URL for a tag/category is the tag/category exists as-is in the Umbraco database. So it is stored as "Event of the week" which is also how it needs to be looked up.
In order to change a string with spaces or special chars to something nicer for URLs means we also need to fail safe way to convert it back to the actual string value. This can prove quite difficult since there will be all sorts of escaping logic needed. The way tags/categories are stored is done by Umbraco, not Articulate, so Articulate doesn't have some extra storage space to store a tag's URL name to a tag's actual name. To do this would require a custom database table managed by Articulate.

Hi, so what is the actual problem with whitespace in the URLs? Is it just a cosmetic thing?
We can have a config option to turn spaces to hyphens (or something else) but this will cause problems if there is a hyphen in the tag name itself. I could make this an optional configuration thing but know that it would break things if the delimiter is used in any tag name.

It was a purely cosmetic request from the client that they claimed was causing issues with their seo. It was basically causing google to index the pages with '%20%'.
This was apparently effecting the search results but I was yet to see convincing results from this other than a seo company complaining it was knocking the client off top spots in search results.

I am using Articulate 3.0.2 with Umbraco 7.14.0 and I have found that tags and categories do not either replace the spaces with dashes, as is common, or URL encode it.  It seems Articulate should do one or the other.  It would be a nice touch to add a filed in the "root" Blog node, on the Customize tab to configure this.

@lcichanowicz replacing spaces with dashes is problematic because we need to decode that too. I'm sure there's probably a clever way but if your tag also contains dashes, how would you know which dashes to change back to spaces? You'd have to convert the dashes to something as well, but then this problem continues because what if that 'something' is also used in tag.
The URLs mentioned above are URL encoded but let me know if you are seeing other things.


[Backport 2.3] Failed to add Registry Credentials

https://github.com/rancher/rancher/issues/26981

Close in favor of rancher/rancher#27182


etcdserver: rename db file into a formal directory

and rename it to a formal name
follow https://github.com/coreos/etcd/pull/3526#issuecomment-140277647

LGTM


Can't compile library.

Problem
Hey guys,
I have tried compiling library, but got some errors.
The command I used is:
cargo make ci

Errors
On branch md/portable:
error: could not find native static library `dsl`, perhaps an -L flag is missing?

On branch main:
error: could not find native static library `common`, perhaps an -L flag is missing?

On tag v0.1.0:
error: could not find native static library `common`, perhaps an -L flag is missing?

Environment
This behaviour is replicating for both
stable:
stable-aarch64-apple-darwin (default)
rustc 1.81.0 (eeb90cda1 2024-09-04)

and nightly rust compilers:
nightly-aarch64-apple-darwin
rustc 1.83.0-nightly (363ae4188 2024-09-24)

As you can probably see, I am doing this on MacOS.
C++ compiler is:
% gcc --version
Apple clang version 16.0.0 (clang-16<IP_ADDRESS>)
Target: arm64-apple-darwin24.1.0
Thread model: posix
InstalledDir: /Library/Developer/CommandLineTools/usr/bin

cargo-make is:
cargo make --version
cargo-make 0.37.23

Maybe you happened to have such an issue and can point to what I can do to fix this?

Can you try again on md/portable branch after issue cargo clean before or better starts from scratch.

Anyway can you post  here the complete output?

Sure; fetched latest changes from md/portable. Here is complete replication:
git clone https://github.com/HorizenLabs/ultraplonk_verifier
cd ultraplonk_verifier/
git switch md/portable
cargo make ci

Same as I did deiniting submodule and running cargo clean.
Full output

cargo make ci
[cargo-make] INFO - cargo make 0.37.23
[cargo-make] INFO -
[cargo-make] INFO - Project: ultraplonk_verifier
[cargo-make] INFO - Build File: Makefile.toml
[cargo-make] INFO - Task: ci
[cargo-make] INFO - Profile: development
[cargo-make] INFO - Execute Command: "cargo" "clean"
     Removed 0 files
[cargo-make] INFO - Execute Command: "cargo" "build" "--release"
   Compiling proc-macro2 v1.0.89
   Compiling unicode-ident v1.0.13
   Compiling libc v0.2.164
   Compiling memchr v2.7.4
   Compiling glob v0.3.1
   Compiling prettyplease v0.2.25
   Compiling rustix v0.38.40
   Compiling regex-syntax v0.8.5
   Compiling cfg-if v1.0.0
   Compiling either v1.13.0
   Compiling minimal-lexical v0.2.1
   Compiling shlex v1.3.0
   Compiling bitflags v2.6.0
   Compiling libloading v0.8.5
   Compiling bindgen v0.69.5
   Compiling home v0.5.9
   Compiling itertools v0.12.1
   Compiling cc v1.2.1
   Compiling lazy_static v1.5.0
   Compiling lazycell v1.3.0
   Compiling clang-sys v1.8.1
   Compiling log v0.4.22
   Compiling crunchy v0.2.2
   Compiling aho-corasick v1.1.3
   Compiling nom v7.1.3
   Compiling rustc-hash v1.1.0
   Compiling spin v0.9.8
   Compiling thiserror v1.0.69
   Compiling rand_core v0.6.4
   Compiling rand v0.8.5
   Compiling rustc-hex v2.1.0
   Compiling cmake v0.1.51
   Compiling byteorder v1.5.0
   Compiling hex-literal v0.4.1
   Compiling quote v1.0.37
   Compiling substrate-bn v0.6.0
   Compiling syn v2.0.87
   Compiling regex-automata v0.4.9
   Compiling cexpr v0.6.0
   Compiling errno v0.3.9
   Compiling which v4.4.2
   Compiling regex v1.11.1
   Compiling thiserror-impl v1.0.69
   Compiling ultraplonk_verifier v0.2.0 (/Users/my-user/horizen-labs/ultraplonk_verifier)
error: could not find native static library `dsl`, perhaps an -L flag is missing?

error: could not compile `ultraplonk_verifier` (lib) due to 1 previous error
Error while executing command, exit code: 101

Maybe you know the case where successful compilation happens on MacOS darwin arm64? I can try using a different rust/c++ versions than mine.

Maybe you know the case where successful compilation happens on MacOS darwin arm64? I can try using a different rust/c++ versions than mine.

No CI the on Mac works !
https://github.com/HorizenLabs/ultraplonk_verifier/actions/runs/11893139575/job/33137470354

Just a question: did you also compile zkverify before?

Yeah, I tried but stumbled upon the same errors, hence where I tried narrowing down to this repository.

Maybe on your home folder there is a .cargo folder, can you check if you have a a config.toml file in it and, in the case, post it here

Sure:
config.toml
[env]
BARRETENBERG_LIB_DIR_DEBUG = "/Users/my-user/horizen-labs/zkVerify/deps/ultraplonk/debug"
BARRETENBERG_LIB_DIR_RELEASE = "/Users/my-user/horizen-labs/zkVerify/deps/ultraplonk/release"
ROCKSDB_LIB_DIR = "/Users/my-user/horizen-labs/zkVerify/deps/rocksdb/debug"
SNAPPY_LIB_DIR = "/Users/my-user/horizen-labs/zkVerify/deps/snappy/debug"

Ok, try to remove the two BARRETENBERG_LIB_DIR_* lines .... should works

If you have a vscode opened with zkverfy repo close it

magic, it worked 🙂 thank you

magic, it worked 🙂 thank you
(needed to wait a bit for compilation)

I know :laughing:


Fix quoting bug which prevented running quote-requiring commands with sudo.

Produced un-executable (even blocked the test run until ssh timeout) shell commands like this:
sudo /bin/sh -c 'grep -qs -- '^first ' /tmp/testfile'

Hi, thanks for the PR!
Could you add a test in test/integration/test_trusty.py ? I think a simple assert Command.check_output("echo x | grep -qs '^x'") == "x" should be enought (the trusty tests are run with and without sudo).

I added a test case which mimics the real world situation where I first bumped into this problem. The test passes now, but I wasn't able to execute it manually with the "broken" code, so I'm not 100% that it tests the right thing.

Fixup and merged in 25349b2 and v1.1.1 released.
I changed the test, the generated command was a valid shell command but not the good one ;)
Thanks !


sx1276: Initial implementation

This driver was developed using an SX1276 based shield on the Galileo
G2.  It requires 3.3v of operation.  It does not work with Edison, due
to SPI issues.
Signed-off-by: Jon Trulson<EMAIL_ADDRESS>

Thanks, merged with 2a1af1399a09e36f20dfe3f7775a462d897f9d8c!


Shellcheck error on Debian 12/bookworm (testing as of today)

In lib/lib.sh line 402:
        registered=$?
                   ^-- SC2320 (warning): This $? refers to echo/printf, not a previous command. Assign to variable to avoid it being overwritten.

https://github.com/amazonlinux/amazon-ec2-net-utils/blob/dc1f1e9d659d3a4337f9337aa29ba85f49452eae/lib/lib.sh#L402
I'm not a Bash programming expert myself, I don't know if this is a genuine problem or a false alarm, but at least it broke the build

@nodakai Thanks for raising this! Will check it out.


Project Status note bug

when we change status special characters are not saving

https://www.berrycast.com/conversations/31d2327f-1e51-5c2a-9f14-5c64df73cd52
https://www.berrycast.com/conversations/fcda5ca0-e9d3-5ca7-91eb-e5172b4dab16

https://www.berrycast.com/conversations/9b6dc089-deea-5e0c-bb66-24e29726f9ce


Hints not shown in problem statement

Describe the bug
Hints added to a problem statement should be shown in the problem statement to students.
Currently it is only visible in the Code editor but not in the problem statement in the exercise view.
To Reproduce
Follow the steps described in TC345 (TP4, TS53) in TestQuality.
Expected behavior

Here, a question mark should be added like it is in the code editor view:

I guess this feature was not fully developed. Maybe @thilo-behnke can say something about it?

I just tested it and this issue still occurs


Implement a SyncTargetEndpointSlice, to accomodate sharding topology in the Syncer

Implement something like a SyncTargetEndpointSlice, to accomodate sharding topology in the Syncer.
The goal is to be able to run a syncer that would sync resources to a syncTarget, but only for a given number of shards.
This probably also involveschanges to the way the Syncer would be deployed (workload sync command).
@sttts thoughts & design guidelines welcome !

/transfer-issue contrib-tmc


fix: multi deck.gl viewport frozen

add onViewportChange method to handle changes to the viewport
CATEGORY
Choose one

[x] Bug Fix
[ ] Enhancement (new features, refinement)
[ ] Refactor
[ ] Add tests
[ ] Build / Development Environment
[ ] Documentation

SUMMARY
Multi deck.gl chart's viewport was frozen. This adds a proper onViewPortChange method to handle for pan/zoom interactions.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

TEST PLAN
visual inspection
ADDITIONAL INFORMATION

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

fixes: https://github.com/apache/incubator-superset/issues/7841
REVIEWERS

Codecov Report

Merging #7854 into master will decrease coverage by 0.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #7854      +/-   ##
==========================================
- Coverage    65.8%   65.79%   -0.02%
==========================================
  Files         461      461
  Lines       22090    22094       +4
  Branches     2422     2423       +1
==========================================
  Hits        14536    14536
- Misses       7433     7437       +4
  Partials      121      121

Impacted Files
Coverage Δ

...t/assets/src/visualizations/deckgl/Multi/Multi.jsx
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


I have a question about the function of the agtype json operator in version 1.5.0

i have a question about the phenomenon found in the version of age 1.5.0
The json operator function was fine in 1.4.0 as below query
select properties->'ott' from ott_service_1.ott_company;

However, starting with 1.5.0 and later versions
When you create a query, such as version 1.4.0
ERROR: invalid input syntax for type agtype error occurs, however
select properties-> 'ott'::text as ott_company_name from netflix_2.ott;
I did type casting like this, and the problem was solved.

Is this the phenomenon intended in 1.5.0? and Will this continue in future versions?

@shinhanbyeol What version of PG is this on, what version of PG did it previously work on?

@jrgemignani  1.5 ,1.4 Both were done in pg 14 version

@shinhanbyeol There appears to be something weird with the definition of ->
@Zainab-Saad Could you take a look?

psql-16.1-5432-pgsql=# select agtype_object_field(properties, 'name') from xyz._ag_label_vertex;
 agtype_object_field
---------------------
 "John"
(1 row)

psql-16.1-5432-pgsql=# select agtype_object_field_agtype(properties, 'name') from xyz._ag_label_vertex;
ERROR:  invalid input syntax for type agtype
LINE 1: select agtype_object_field_agtype(properties, 'name') from x...
                                                      ^
DETAIL:  Expected agtype value, but found "name".
CONTEXT:  agtype data, line 1: name
psql-16.1-5432-pgsql=#
psql-16.1-5432-pgsql=# select properties->'name' from xyz._ag_label_vertex;
ERROR:  invalid input syntax for type agtype
LINE 1: select properties->'name' from xyz._ag_label_vertex;
                           ^
DETAIL:  Expected agtype value, but found "name".
CONTEXT:  agtype data, line 1: name
psql-16.1-5432-pgsql=#

@Zainab-Saad Thank you!

The error you are getting is because the operator -> is overloaded (in version 1.5.0) with the following possible operands
agtype -> agtype
agtype -> text
agtype -> int
In the query select properties->'ott' from ott_service_1.ott_company;, 'ott' is of the type unknown and when such a type is provided as second argument to the operator ->, postgres chooses the operator agtype -> agtype instead of agtype -> text. And 'ott' is not in valid agtype format (should be '"ott"'), so you get an error.
You can read why specific operator is choosen when unknown type is provided in the operator type resolution section here


List has no rows for assignment to SObject

Describe the bug
When I try to deploy a child trigger, I get this error:
List has no rows for assignment to SObject (workstudy): [] (dlrs) Class.dlrs_workstudy_EmploymentTest.testTrigger: line 11, column 1
I also get this warning:
dlrs_workstudy_EmploymentTrigger:Test coverage of selected Apex Trigger is 0%, at least 75% test coverage is required
To Reproduce
Steps to reproduce the behavior:

Create new DLRS using new wizard or creating DLRS manually
click on "Manage Child Trigger"
Click on "Deploy"
See error

Expected behavior
Expect child trigger to deploy successfully so I can proceed with deploying the DLRS
Screenshots
If applicable, add screenshots to help explain your problem.

Additional context
Add any other context about the problem here.

@ktallman this error is usually caused by the child record needing more detailed setup for the Apex Test that would be used to cause the trigger to fire.
DLRS provides a method to customize the test method body to suite your data better. If you're not familiar with writing Apex test code for you system you may need to engage a developer who understands your systems requirements and can do the customizations for you.


Optional Characteristic in air quality sensor do not appear

I want add the optional Characteristics like PM2.5 Density and PM10 Density to the air quality sensor. I have already set the required characteristics and add the optional characteristics to sensor, but in homekit app it only shows the air quality, I can't find PM2.5 Density and PM10 Density.
This is my core code
sensor
  .addService(Service.AirQualitySensor, "Air quality Sensor")
  .getCharacteristic(Characteristic.AirQuality)
  .on('get', function(callback) {
    callback(null, AIRQUALITY_SENSOR.aqi);
  });

sensor
  .getService(Service.AirQualitySensor)
  .addCharacteristic(Characteristic.PM2_5Density)
  .on('get', function(callback) {
    // AIRQUALITY_SENSOR.getStatus()
    callback(null, AIRQUALITY_SENSOR.pm2_5Density);
  });

sensor
  .getService(Service.AirQualitySensor)
  .addCharacteristic(Characteristic.PM10Density)
  .on('get', function(callback) {
    callback(null, AIRQUALITY_SENSOR.pm10Density);
  });

setInterval(function() {
  AIRQUALITY_SENSOR.ramdomizeQuality();
  // update the characteristic value so interested iOS devices can get notified
  sensor
    .getService(Service.AirQualitySensor)
    .setCharacteristic(Characteristic.AirQuality, AIRQUALITY_SENSOR.aqi);
  sensor
    .getService(Service.AirQualitySensor)
    .setCharacteristic(Characteristic.PM2_5Density, AIRQUALITY_SENSOR.pm2_5Density);
  console.log("Set");
  sensor
    .getService(Service.AirQualitySensor)
    .setCharacteristic(Characteristic.PM10Density, AIRQUALITY_SENSOR.pm10Density);
}, 3000);

This is the screen shoot

deleted accessory program add the newly
then it will be shown

AirQualitySensor
.addService(Service.AirQualitySensor, ismi2)
.getCharacteristic(Characteristic.AirQuality)
.on('get', function(callback) {
callback(null, SENSOR.getQuality());
});
AirQualitySensor
.getService(Service.AirQualitySensor)
.getCharacteristic(Characteristic.AirParticulateDensity)
.on('get', function(callback) {
callback(null, SENSOR.Density());
});
AirQualitySensor
.getService(Service.AirQualitySensor)
.getCharacteristic(Characteristic.AirParticulateSize)
.on('get', function(callback) {
callback(null, SENSOR.Size());
});
AirQualitySensor
.getService(Service.AirQualitySensor)
.getCharacteristic(Characteristic.StatusLowBattery)
.on('get', function(callback) {
callback(null, SENSOR.Battery());
});

It works but your code should be like this for Characteristic.StatusLowBattery, Characteristic. AirParticulateSize ,  Characteristic.AirParticulateDensity
AirQualitySensor
  .getService(Service.AirQualitySensor)
  .addCharacteristic(Characteristic.AirParticulateDensity)
  .on('get', function(callback) {
    callback(null, SENSOR.Density());
  });

It should be addCharacteristic
Also, what if I want to add two or more AirParticulateDensity, what should I do.
Right now I change the AirParticulateDensity and size every 3 seconds, is any other good ways?

@Nero5023 The Home app can't show some type of Characteristic value (at the moment, maybe later it will show), but the Elgato Eve app display the values!


can't install on vs2013 premium

The latest version I could install was 0.30.4.
All later versions failed telling me no compatible products have been found.
Looking at the manifest I saw that the version range is 12-16, but only community edition.

/cc @hmemcpy

I was literally typing a comment, but forgot to press the button!!!
Hmm, Community is the "minimum" version, but I don't remember if they added this logic in 2013. Is your Visual Studio updated with the latest update?

Yes VS is updated.
in the vsixmanifest there's this:
  <Prerequisites>
    <Prerequisite Id="Microsoft.VisualStudio.Component.CoreEditor" Version="[15.0.26004.1,16.0)" DisplayName="Visual Studio core editor" />
  </Prerequisites>`````

The last version not containing this is installable on vs2013

Using VS2013 Ultimate Update 5.
I got round this by

(optional) Changing the .vsix to .zip
unzipping the vsix to a folder
editing the .vsixmanifest file, in my case I changed the InstallationTarget element to <InstallationTarget Version="[12.0,16.0)" Id="Microsoft.VisualStudio.Ultimate" /> (ref. https://msdn.microsoft.com/en-us/library/hh696828.aspx )
select all the files
right click, send to "Compressed folder"
change the extension back to .vsix
Double click to install

It then installs with no issue.
However I did not have nice icons on paket.dependencies or paket.references in my c# .net 4.6.2 dll projects, or in my solution folders. The tree view of packages does however show under the files, as do the icons on each depenency, and I have commands in the context menus as expected.


Custom websocket error codes?

Currently partykit can throw errors, but does not allow for the error code to be displayed. It would be great to be able to control this error code returned back for more flexible error handling (similar to http)
Previous API:
throw new Error('Room does not exist...');
New API:
throw new PartyKitError({code: number, reason: string});
https://discord.com/channels/1051830863576453180/1104146566303977553/1142883806861332571

Can it be you are confusing close events with errors?
Per WHATWG, WebSocket errors do not have any identifiers that would act as additional error information (that's not ideal, I know). What you've showcased looks a lot like a CloseEvent (code and reason hint at that) but that doesn't necessarily indicate an error.
Can you tell me more about how you'd use a newly proposed PartyKitError? How and where would you throw it? How would the client get it?


How to destroy form from outside of it?

Is this currently possible? It would be nice to destroy the form from a saga or something similar, after an async submit has occurred that is outside the scope of the form.

You can use the destroy action creator for this


Ability to add custom svg icons

Thank you for this stunning font-service.
It would be very convenient and as well bringing endless flexibility, when it would be able to add custom svg codes in a custom user related collection, which is always be generated along the other icon selections.
If anyone want to add a custom icon, which is not available for selection, it is really hard work to add this into the fontello package manually for all the generated file types.

SVG import is supported, with some prepare.
https://github.com/fontello/fontello/wiki/How-to-use-custom-images


Overriding PublicOrigin with the empty string results in unlogged relative urls

Issue / Steps to reproduce the problem

Set IdentityServerOptions.PublicOrigin to string.Empty.
IdentityServerApplicationBuilderExtensions.ValidateOptions(...) does not log the fact that the PublicOrigin has been overwritten, because the IsPresent() extension method checks for !string.IsNullOrEmpty

if (options.PublicOrigin.IsPresent()) logger.LogDebug("PublicOrigin explicitly set to {0}", options.PublicOrigin);

The BaseUrlMiddleware uses the null-coalescing operator to set the IdentityServerOrigin value, so it does get set to the empty string:

var origin = _options.PublicOrigin ?? request.Scheme + "://" + request.Host.Value;
context.SetIdentityServerOrigin(origin);

Impact:
You end up with relative urls in your discovery document:
"jwks_uri":"/.well-known/openid-configuration/jwks","authorization_endpoint":"/connect/authorize",...
I'm not sure if more is impacted by this - I haven't taken a close look.
I would open a PR changing the BaseUrlMiddleware to also check PublicOrigin.IsPresent(), but that's potentially a breaking change if anyone is depending on relative urls in their discovery document. If that is valid, then I believe the logging should be changed to log the fact that the PublicOrigin is explicitly being set to the empty string.

I pushed a fix to prevent empty config values for PublicOrigin. Have a look, please.


How to track work on Serverless Framework compatability

Austen Collins mentioned that you are working on integration with the Serverless Framework v1.0. \0/
What is the best way to track this effort? In particular know when it is worth trying it out?

Integration is complete


[firestore-bigquery-export] Better error logging on bigquery extension when partition is set to none

Describe the problem
We have a firebase customer trying to set up the bigquery extension and is confused because they have partition option set to none, but still getting warnings: Cannot partition an existing table ${table.dataset.id}_${table.id}.
Looking at the code it might be because this line logs a warning regardless of the partitioning settings.
https://github.com/firebase/extensions/blob/fdbf5004c8d5e7b9749bbab3b2e16950ad812007/firestore-bigquery-export/firestore-bigquery-change-tracker/src/bigquery/partitioning.ts#L131
The customer is asking if they can avoid these warning logs, so I wonder if we should have better logging/messaging here?

This is definitely something we should look into, this can be done through:

Improved messages
Verbose messaging

Or a combination of both, maybe.

It would seem that there is a deeper issue here. Firstly, per the definition of bigquery.Table, exists()returns aPromise`, thus will always evaluate as being truthy here:
https://github.com/firebase/extensions/blob/41d9a3b5e46487d725c6bdd8c2f6ad1f0d438d7a/firestore-bigquery-export/firestore-bigquery-change-tracker/src/bigquery/partitioning.ts#L132-L137
In fact, to emphasize, the same method is called again in the following statement with await:
https://github.com/firebase/extensions/blob/41d9a3b5e46487d725c6bdd8c2f6ad1f0d438d7a/firestore-bigquery-export/firestore-bigquery-change-tracker/src/bigquery/partitioning.ts#L139
Additionally, the comment "No table provided, cannot evaluate" does not seem to correspond with the logic that follows. It appears to actually be in reference to the previous line.
Lastly, the content of the log message indicates that the table cannot be partitioned, but the scope of the method is only to check if the table has already been partitioned. These are conflicting concerns.
I suspect that said block of code was mistakenly included here and that the errant logging brought forward in this issue is a result of that, not due to the logging of it in general.

We're also experiencing a ton of logs related to partitioning even though we do not have partitioning configured.


Working with jOOQ Flyway using Testcontainers

Port of Spring Boot Guide https://testcontainers.com/guides/working-with-jooq-flyway-using-testcontainers
I added a Test Resources section to show how you can simplify testing.
To generate guide locally:
./gradlew workingWithJooqFlywayUsingTestcontainersBuild
Open HTML locally:
open build/dist/index.html
Code available:
build/code/working-with-jooq-flyway-using-testcontainers

@sivaprasadreddy  can you review this?

We could get Gradle working, but it would need a change in starter
If we generate a build with this plugin
    id("dev.monosoul.jooq-docker") version "5.0.7"

Some config:
configurations {
    jooqCodegen {
        extendsFrom(implementation)
    }
}

tasks {
    generateJooqClasses {
        basePackageName.set("example.micronaut.jooq")
    }
}

And an additional dependency:
    jooqCodegen("org.postgresql:postgresql")

Then the Gradle build works...  However I can't find a way to add the config or dependency without changing starter 🤔

Something is unclear to me. What do we recommend? Using test containers directly or test resources? If we want to use test resources, why explain how to do it with test containers before?

We recommend test resources. Explaining how to do it with test containers first shows what Test Resources gives you out-of-the-box (no testcontainers dependencies, more straightforward tests to write, etc.).


BUG:Text not visible in Darkmode

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
In the Literary Realms section ,while we are switching dark mode on,the text is not visible
We can change the color to a darker one so that visibility is fine
From this:

To this:

Plz @anuragverma108 Assign this issue to me
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

Assigned!
Make sure to attach before & after screenshots in your PR. Also, star the repo before working on it. While creating your PR, ensure you pull all existing changes beforehand to avoid conflicts.
Happy open-source contributing!

I'd like to contribute to this issue too, can you assign this to me as well sir?


V5 develop crashes on code that works on v4...

I just spent 3 days fixing up code to compile with V5- develop. (mainly AKManager and Double->Float changes all over the place). What was working on v4.9.4 is now crashing in startup. with required condition is false: [AVAudioEngineGraph.mm:3893:MakeVirtualConnection: (IsNodeInGraph(conn.srcNode) && IsNodeInGraph(conn.destNode))]
I'm I doing something wrong, is v5 not ready to test, or is it not compatible with code written for v4? I couldn't find a roadmap or a migration guide :-)
Any guidance on what the recommended changes are coding for v4 vs v5 will help me test the develop branch better!

V5 is not even in beta yet. It will not be backwards compatible but the changes are pretty easy.
There will be a migration document. v5 is probably at least a few months away.
I'm happy to review your project if you really want to be ready for v5.

Ah ok! Got it :-)
I'll wait -- once it's ready to be tested, I'll be happy to. Looking forward.

could you put your crash report here?

It's this assert:
required condition is false: [AVAudioEngineGraph.mm:3893:MakeVirtualConnection: (IsNodeInGraph(conn.srcNode) && IsNodeInGraph(conn.destNode))]
I'm trying to narrow it down for you so there's a small amount of code that reproduces the issue.
I'll update this once I have it.


RFD - Bitnami retention policy considerations

Status
Draft 🚧

Author(s)
@viniciusdc

Date Created
08-12-2022

Date Last updated
08-12-2022

Decision deadline
NA

Title
Considerations around Bitnami retention policies
Summary
Just a note regarding using Bitnami as the repo source for helm charts, as happened in the past with Minio, they have a 6m retention policy for their repo index, which means that old versions will be dropped from the main index after that period. This is, in the future, our deployments are bound to break if the version is not found by Helm.
User benefit

Right now, we are bound to have broken deployments of the old Nebari versions in the future; as an example, v0.4.0 and v0.4.1 are still (at this date) broken due to the fact these versions have in their source code a pointer to a Minio version that does not exist anymore in the main index.yaml (fixed on v0.4.2).

Design Proposal

Alternatives or approaches considered (if any)
There is two ways to suppress this problem:

Every six months, we update our chart versions or validate these somehow, originally proposed here
We pin the repository source on each service with the last available hash for the index, the same as we did for Minio

Best practices

User impact

Unresolved questions

This may be left as it is if we increase our release schedule, which will give the user freedom to use aways an "up-to-date" version of Nebari.


Installation on Windows without Node.js

I'm unable to install the package.
This message is being displayed:
Installing<EMAIL_ADDRESS>failed.Hide output…

<EMAIL_ADDRESS>postinstall C:\Users\AKASH_~1\AppData\Local\Temp\apm-install-dir-115922-2412-1wx0rj0\node_modules\atom-beautify\node_modules\cross-spawn\node_modules\spawn-sync
node postinstall

'node' is not recognized as an internal or external command,
operable program or batch file.
npm ERR! Windows_NT 6.2.9200
npm ERR! argv "C:\Users\akash_000\AppData\Local\atom\app-1.0.19\resources\app\apm\bin\node.exe" "C:\Users\akash_000\AppData\Local\atom\app-1.0.19\resources\app\apm\node_modules\npm\bin\npm-cli.js" "--globalconfig" "C:\Users\akash_000.atom.apm.apmrc" "--userconfig" "C:\Users\akash_000.atom.apmrc" "install" "C:\Users\AKASH_~1\AppData\Local\Temp\d-115922-2412-1uns2x5\package.tgz" "--target=0.30.7" "--arch=ia32"
npm ERR! node v0.10.40
npm ERR! npm  v2.13.3
npm ERR! code ELIFECYCLE
npm ERR<EMAIL_ADDRESS>postinstall: node postinstall
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>postinstall script 'node postinstall'.
npm ERR! This is most likely a problem with the spawn-sync package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node postinstall
npm ERR! You can get their info via:
npm ERR!     npm owner ls spawn-sync
npm ERR! There is likely additional logging output above.

Do you know if node is installed on your computer?

I'm not sure. How can I install it?
Sent from my OnePlus One.
On Oct 22, 2015 8:08 PM, Austin Cheney<EMAIL_ADDRESS>wrote:
Do you know if node is installed on your computer?
—
Reply to this email directly or view it on GitHub.

I had node installed before I installed Atom in all my environments so I cannot say if its included or a prerequisite.  @Glavin001 would be better able to answer that.
Try to install Node and let me know if you are still having problems: https://nodejs.org/en/

Looks like Node.js is not installed.
spawn-sync will be using your global node.js installation instead of Atom's. That may be room for improvement actually...

How do I install it?
Sent from my OnePlus One.
On Oct 24, 2015 3:13 AM, Glavin Wiechert<EMAIL_ADDRESS>wrote:
Looks like Node.js is not installed.
spawn-sync will be using your global node.js installation instead of Atom's. That may be room for improvement actually...
—
Reply to this email directly or view it on GitHub.

As @prettydiff indicated:

Try to install Node and let me know if you are still having problems: https://nodejs.org/en/

Specifically, you can go to https://nodejs.org/en/download/ and get the appropriate Windows installer, download, run it, profit.

I am going to close this issue. @akashpb let us know if that does not work out for you.

cross-spawn has since been removed so this will no longer happen See https://github.com/Glavin001/atom-beautify/pull/955


Slovak language

@getneil any problem?

@getneil conflicts solved check it right now

@dany4everr can you fix the lint issue and potential conflicts. thanks

fixed

conflicts solved @getneil


Remove states from RPC logs, include only rule unique IDs

This PR removes the states from Booster's rewrite and equation traces. Only rule unique IDs are included in the RPC logs now.
Here's a quick performance comparison on the execute request test-fountry-bug-report/state-007.send with log-successful-rewrites and log-successful-simplifications set to true:

Git revision
Time
Comment

georgy/rpc-logs-only-labels
3.286555 s
this PR

main (0ef6ecd)
3.259145294
Georgy's log fix reverted

0096f93
35s
logs with states, -N still there

The performance with unique IDs only seems to be on par with not collecting traces at all.

KEVM performance
No difference with main where the logs are not emitted at all.

Test
georgy-rpc-logs-only-labels time
master-0ef6ecd time
(georgy-rpc-logs-only-labels/master-0ef6ecd) time

erc20/hkg/transferFrom-failure-1-spec.k
34.15
39.95
0.8548185231539424

erc20/ds/approve-failure-spec.k
27.05
28.16
0.9605823863636364

erc20/ds/approve-success-spec.k
27.97
29.08
0.9618294360385145

erc20/hkg/transfer-success-2-spec.k
41.22
35.71
1.1542985158218986

TOTAL
130.39
132.9
0.9811136192626033


pyLODE fail, GitHub action.  New version rdfLib at fault?

I have a github action set up to run pyLODE on the ttl in my github repo. Last week it worked; when I did a push today I got an error on the action. looking at the logs, the difference seems to be this:
works (last week):
...

Collecting rdflib>=5.0.0
Downloading rdflib-5.0.0-py3-none-any.whl (231 kB)

...

Successfully installed isodate-0.6.0 markdown-3.3.4 pylode-2.10.3 rdflib-5.0.0 rdflib-jsonld-0.5.0

... starts generating html files
Doesn't work (today)
...

Collecting rdflib>=5.0.0
Downloading rdflib-6.0.0-py3-none-any.whl (376 kB)

...

Successfully installed isodate-0.6.0 markdown-3.3.4 pylode-2.10.3 rdflib-6.0.0 rdflib-jsonld-0.5.0

then:

/usr/local/lib/python3.8/dist-packages/pylode/profiles/ontdoc.py:763: UserWarning: Code: domainIncludes is not defined in  namespace SDO
for o in self.G.objects(subject=s, predicate=SDO.domainIncludes):
/usr/local/lib/python3.8/dist-packages/pylode/profiles/ontdoc.py:819: UserWarning: Code: rangeIncludes is not defined in namespace SDO
for o in self.G.objects(subject=s, predicate=SDO.rangeIncludes):

.... stack trace omitted...

File "/usr/local/lib/python3.8/dist-packages/pylode/profiles/base.py", line 467, in _make_schemaorg_metadata
return g.serialize(format="json-ld").decode("utf-8")
AttributeError: 'str' object has no attribute 'decode'
Error: Process completed with exit code 1.

any ideas?

You are right, problem is that rdflib is not pinned to 5.0.0.
Workaround:
pip uninstall rdflib
pip install rdflib==5.0.0


Running with Administrator Permissions documentation is missing Mouse without Borders

Provide a description of requested docs changes
The PowerToys Running with Administrator Permissions page has a list of Affected PowerToys utilities. Missing from that list is Mouse Without Borders.

Mouse Without Borders

Use Service

Hello and thanks for your message. You're welcome to send in a Pull Request to update this info. Go to https://github.com/MicrosoftDocs/windows-dev-docs/blob/docs/hub/powertoys/administrator.md and click the "edit" pencil. You will edit the file in a copy repository and then submit a Pull Request to  be merged.

Resolved with pr #4966.


Add Konveyor 1.2.2 to 1.2.5

Thanks submitting your Operator. Please check below list before you create your Pull Request.
Updates to existing Operators

[x] Is your new CSV pointing to the previous version with the replaces property?
[x] Is your new CSV referenced in the appropriate channel defined in the package.yaml ?
[x] Have you tested an update to your Operator when deployed via OLM?
[x] Is your submission signed?

Your submission should not

[X] Modify more than one operator
[X] Modify an Operator you don't own
[X] Rename an operator - please remove and add with a different name instead
[X] Submit operators to both upstream-community-operators and community-operators at once
[X] Modify any files outside the above mentioned folders
[X] Contain more than one commit. Please squash your commits.

Operator Description must contain (in order)

[X] Description about the managed Application and where to find more information
[X] Features and capabilities of your Operator and how to use it
[X] Any manual steps about potential pre-requisites for using your Operator

Operator Metadata should contain

[X] Human readable name and 1-liner description about your Operator
[X] Valid category name1
[X] One of the pre-defined capability levels2
[X] Links to the maintainer, source code and documentation
[X] Example templates for all Custom Resource Definitions intended to be used
[X] A quadratic logo

Remember that you can preview your CSV here.

Are the bundle errors in the above tests something to worry about currently?

@jmontleon, I will look deeper, bear with us.

Thank you


2D stretch mode doesn't scale properly

Godot version:

4.0, b55d859c5f092d145cd0d2c8df9ac5d42c500626
OS/device including version:

Windows, Vulkan
Issue description:

I set my resolution to 320x180. I set the test resolution to scale up to 1280x720. I set the stretch mode to 2D. When I start the game the camera is really far zoomed out, and doesn't show the camera region that is represented in the editor. When I resize the window, it doesn't stretch the content, it just stays the same size and shows more of the screen.

Steps to reproduce:
Open repro project, run game.
Minimal reproduction project:

repro.zip

Related to #37941.

I have this problem, too

Duplicate of #37941.


Issue with Limiting Input Length in Input Dialog

[ ] I have verified there are no duplicate active or recent bugs, questions, or requests
[x] I have verified that I am using the latest version of Material Dialogs.
[x] I have given my issue a non-generic title.
[x] I have read over the documentation (before asking questions on how to do something).

Include the following:

Material Dialogs version: <IP_ADDRESS>
Device OS version: Android 5.1
Device Manufacturer: Motorola
Device Name: Moto E- XT1044

Also, please wrap Java with correct syntax highlighting.
inputRangeRes(2, 20, R.color.material_red_500)

Reproduction Steps

Enter some text in input dialog with inputRangeRes defined.
Erase the text in input dialog
Now the positive text button is enabled even if we have defined input range.

Expected Result
After setting the input range,if I erase the text inside the input dialog , it should follow the input range defined.
Actual Result
After setting the input range,it works well but if I erase the input totally so positive text button  becomes enabled even if I have defined the input range.

inputRangeRes(2, 20, R.color.material_red_500)  works well initially but once I have given some input & then erasing the input causes the input dialog to its default behaviour.

I'm unable to reproduce any problems with this.
There's actually usage of inputRange in the sample project, it works fine there too.

Link: https://github.com/abhinituk/StockHawk/blob/Detail_Activity_Implementation/app/src/main/java/com/sam_chordas/android/stockhawk/ui/MyDialogFragment.java
You can have a look at it. Material dialog has default behaviour if we erase the input.
If you wish, you can clone repo & run the project to reproduce the error.
https://github.com/abhinituk/StockHawk/tree/Detail_Activity_Implementation

It looks like that may be a private repo, it returns a 404

It's a private repo which I have forked from Udacity GitHub.If you wish I can mail you the complete project zip file with detailed information to reproduce the error.

Well, I probably won't be able to see much more there.
As long as you're using a newer version of the library, it should work. It's worked fine with in all of the apps I use it in.

I am using the latest version of the library but still the problem exists. I have updated the comment which shows that positive text button is disabled earlier but when I write some input & erase the input, positive text button is enabled.
For more information please see the project:
StockHawk.zip

I'm still unable to reproduce any issues with this in the sample project, or any of my applications.
When the input becomes empty, the button disables correctly for me.

I've made a small change which may help, though.

https://github.com/afollestad/material-dialogs/releases/tag/<IP_ADDRESS>

Still the issue exist after updating the library version.

Well, I'm still unable to reproduce any issues so it must be on your end.

`public class MyDialogFragment extends DialogFragment {
private static final String TAG = "tag";
private static final String ADD = "add";
private static final String SYMBOL = "symbol";
private final String LOG_TAG = getClass().getSimpleName();
private String mInputSymbol;
private Intent mServiceIntent;
@Override
public Dialog onCreateDialog(Bundle savedInstanceState) {

    MaterialDialog dialog = new MaterialDialog.Builder(getActivity())
            .title(R.string.symbol_search)
            .content(R.string.content_test)
            .inputRange(1, 20)
            .inputType(InputType.TYPE_TEXT_FLAG_CAP_CHARACTERS)
            .positiveText(R.string.positive_text)
            .negativeText(R.string.negative_text)
            .alwaysCallInputCallback()
            .input(R.string.input_hint, R.string.input_prefill, new MaterialDialog.InputCallback() {
                @Override
                public void onInput(@NonNull MaterialDialog dialog, CharSequence input) {

                    if (input.toString().contains(" ")) {
                        dialog.setContent(R.string.whitespace);
                        dialog.getActionButton(DialogAction.POSITIVE).setEnabled(false);
                    } else {
                        dialog.setContent(R.string.content_test);
                        dialog.getActionButton(DialogAction.POSITIVE).setEnabled(true);
                        mInputSymbol = input.toString();

                    }
                }
            })

            .onPositive(new MaterialDialog.SingleButtonCallback() {
                @Override
                public void onClick(@NonNull MaterialDialog dialog, @NonNull DialogAction which) {
                    Cursor c = getActivity().getContentResolver().query(QuoteProvider.Quotes.CONTENT_URI,
                            new String[]{QuoteColumns.SYMBOL}, QuoteColumns.SYMBOL + "= ?",
                            new String[]{mInputSymbol}, null);
                    if ((c != null ? c.getCount() : 0) != 0) {
                        Toast toast =
                                Toast.makeText(getActivity(), R.string.stock_already_saved,
                                        Toast.LENGTH_LONG);
                        toast.setGravity(Gravity.CENTER, Gravity.CENTER, 0);
                        toast.show();
                        return;
                    } else {
                        // Add the stock to DB
                        mServiceIntent = new Intent(getActivity(), StockIntentService.class);
                        mServiceIntent.putExtra(TAG, ADD);
                        mServiceIntent.putExtra(SYMBOL, mInputSymbol);
                        getActivity().startService(mServiceIntent);
                    }
                    if (c != null) {
                        c.close();
                    }
                }
            })
            .build();

    return dialog;
}

}
`

Please have a look at DialogFragment class which I am using.

It's not an issue with this library.
You're manually enabling the submit button inside of onInput.
@abhinituk

.alwaysCallInputCallback() makes onInput get invoked everytime you modify the input.

Alright @afollestad , Thanks for helping me out.


Running of buy and hold example strategy generates NotImplementedError: Could not find signature for display:

After setting up QSTrader using the instructions in the readme file, I attempted to run the example buy_and_hold_backtest.py. This generated the error:

Traceback (most recent call last):
  File "C:\Users\Andre\Anaconda3\envs\python27_env_tradeEngine\Lib\site-packages\qstrader\examples\buy_and_hold_backtest.py", line 85, in <module>
    main()
  File "C:\Users\Andre\Anaconda3\envs\python27_env_tradeengine\lib\site-packages\click\core.py", line 716, in __call__
    return self.main(*args, **kwargs)
  File "C:\Users\Andre\Anaconda3\envs\python27_env_tradeengine\lib\site-packages\click\core.py", line 696, in main
    rv = self.invoke(ctx)
  File "C:\Users\Andre\Anaconda3\envs\python27_env_tradeengine\lib\site-packages\click\core.py", line 889, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "C:\Users\Andre\Anaconda3\envs\python27_env_tradeengine\lib\site-packages\click\core.py", line 534, in invoke
    return callback(*args, **kwargs)
  File "C:\Users\Andre\Anaconda3\envs\python27_env_tradeEngine\Lib\site-packages\qstrader\examples\buy_and_hold_backtest.py", line 81, in main
    run(config, testing, tickers, filename)
  File "C:\Users\Andre\Anaconda3\envs\python27_env_tradeEngine\Lib\site-packages\qstrader\examples\buy_and_hold_backtest.py", line 65, in run
    results = backtest.simulate_trading(testing=testing)
  File "C:\Users\Andre\Anaconda3\envs\python27_env_tradeengine\lib\site-packages\qstrader\trading_session\backtest.py", line 68, in simulate_trading
    self._run_backtest()
  File "C:\Users\Andre\Anaconda3\envs\python27_env_tradeengine\lib\site-packages\qstrader\trading_session\backtest.py", line 54, in _run_backtest
    self.statistics.update(event.time, self.portfolio_handler)
  File "C:\Users\Andre\Anaconda3\envs\python27_env_tradeengine\lib\site-packages\qstrader\statistics\simple.py", line 50, in update
    current_equity = PriceParser.display(portfolio_handler.portfolio.equity)
  File "C:\Users\Andre\Anaconda3\envs\python27_env_tradeengine\lib\site-packages\multipledispatch\dispatcher.py", line 161, in __call__
    (self.name, str_signature(types)))
NotImplementedError: Could not find signature for display: <long>

I added the below code to the price_parser.py file and the backtest worked. However, the results were not correct; I suspect it is due to my introduction of the below code.
    @staticmethod
    @dispatch(long)
    def display(x):
        return round(x / PriceParser.PRICE_MULTIPLIER, 2)

What would the correct solution to the original problem be? The platform is Windows+Anaconda (Python 2.7)
Thanks.

Hi Drex91,
I'm going to have access to my Windows machine briefly next week (currently only have access to my Mac laptop!), so I'll try and replicate the error and then fix.
Although, @femtotrader and @ryankennedyio are the experts with regards multiple dispatch, as they jointly built up a lot of code for this aspect, so they might have more insight before then!
Cheers,
Mike.

My version of Python is
$ python --version
Python 3.5.2 :: Anaconda 4.1.1 (x86_64)

under Mac OS 10.10.5 (64 bits)
Are you using 32 bits or 64 bits OS ?
This may help
http://stackoverflow.com/questions/2104884/how-does-python-manage-int-and-long

union-types are supported by multiple-dispatch
http://multiple-dispatch.readthedocs.io/en/latest/resolution.html#union-types
maybe we should define in price_parser.py
int_t = (int, long)

and replace (nearly) every int by int_t
so code could be:
from __future__ import division
from multipledispatch import dispatch

int_t = (int, long)

class PriceParser(object):
    """
    PriceParser is designed to abstract away the underlying number used as a price
    within qstrader. Due to efficiency and floating point precision limitations,
    QSTrader uses an integer to represent all prices. This means that $0.10 is,
    internally, 10,000,000. Because such large numbers are rather unwieldy
    for humans, the PriceParser will take "normal" 2dp numbers as input, and show
    "normal" 2dp numbers as output when requested to `display()`

    For consistency's sake, PriceParser should be used for ALL prices that enter
    the qstrader system. Numbers should also always be parsed correctly to view.

    """

    # 10,000,000
    PRICE_MULTIPLIER = 10000000

    @staticmethod
    @dispatch(int_t)
    def parse(x):  # flake8: noqa
        return x

    @staticmethod
    @dispatch(float)
    def parse(x):  # flake8: noqa
        return int(x * PriceParser.PRICE_MULTIPLIER)

    @staticmethod
    @dispatch(int_t)
    def display(x):  # flake8: noqa
        return round(x / PriceParser.PRICE_MULTIPLIER, 2)

    @staticmethod
    @dispatch(float)
    def display(x):  # flake8: noqa
        return round(x, 2)

    @staticmethod
    @dispatch(int_t, int_t)
    def display(x, dp):  # flake8: noqa
        return round(x / PriceParser.PRICE_MULTIPLIER, dp)

    @staticmethod
    @dispatch(float, int_t)
    def display(x, dp):  # flake8: noqa
        return round(x, dp)

but I'm not sure of results as I haven't been able to reproduce this issue.

in fact long doesn't exist with Python 3 but exist with Python 2
Maybe we might use six here https://pythonhosted.org/six/
six.integer_types
Possible integer types. In Python 2, this is long and int, and in Python 3, just int.


dubbo 与Springboot整合

dubbo 与Springboot整合时@Reference注解无法注入服务

you can use dubbo-starter maintained by @mercyblitz

@qq282485068 let me know if you are using https://github.com/dubbo/dubbo-spring-boot-project, and if you confirm it is the bug from the project I refer here, pls. report the issue on that project, instead of here.

OK. Thanks, Let' me try!


Interaction cannot continue because the desired element was not found

Hello,
I am running into a strange issue with below implementation, exception provided below.
grey_accessibilityLabel("LoginID")).assert(grey_sufficientlyVisible())
The Label indicated above is based on accessibility inspector and I am confused why the desired element is not found on the screen, there was another element on the same screen I was  easily able to tap with the script  based on accessibility label.any thoughts?is there any way I can capture and log all the tappable or editable elements on a screen ?
EarlGrey.selectElement(with:
grey_accessibilityLabel("Contact Us")).atIndex(UInt(0))
.perform(grey_tap()) //This implementation was sucessful
Exception: com.google.earlgrey.ElementInteractionErrorDomain
Exception Name: com.google.earlgrey.ElementInteractionErrorDomain
Exception Reason: {
"User Info" :   {
"NSLocalizedDescription" : "Interaction cannot continue because the desired element was not found."
},
"Error Info" :   {
"Search API Info" : "",
"Recovery Suggestion" : "Check if the element exists in the UI hierarchy printed below. If it exists, adjust the matcher so that it accurately matches element.",
"Assertion Criteria" : "assertWithMatcher:sufficientlyVisible(Expected: 0.750000, Actual: 0.000000)",
"Element Matcher" : "((respondsToSelector(isAccessibilityElement) && isAccessibilityElement) && accessibilityValue('Enter Online ID'))"
},

When you tap on an element, EarlGrey only checks if any part of the element is available for tapping on. The visibility check ascertains that over 75% of the element is visible on the screen.

how about replaceText and typeText?all of the implementations are failing for this element LoginID are failing which was identified using accessibility inspector.I would like to be able submit a userid and password and was not expecting it to fail when another element on the same screen was easily identifiable, please advice how to trroubleshoot this.
Thanks
EarlGrey.selectElement(with:
grey_accessibilityLabel("LoginID"))
.perform(grey_replaceText("Testuser1")
EarlGrey.selectElement(with:
grey_accessibilityLabel("LoginID"))
.perform(grey_typeText("Testuser1")

I attempted multiple approaches  (I included few below)but haven't had any luck able to enter userid and password in the textbook.EarlGrey seems to be a perfect fit for my requirement, any pointers to overcome this issue are much appreciated
EarlGrey.selectElement(with:grey_anyOf([grey_accessibilityLabel("Login ID"),grey_kindOfClassName("ABLineTextField"),grey_interactable()])).atIndex(0).perform(grey_replaceText("testuser1"))
EarlGrey.selectElement(with:grey_allOf([grey_accessibilityLabel("Login ID"),grey_kindOfClassName("ABLineTextField"),grey_interactable()])).atIndex(0).perform(grey_replaceText("testuser1"))


You should be using Alarm Manager to keep BLE running

Describe the bug
So according to the epidemiologists that we have been working with on OpenTrace you want to scan every 5 minutes.  The only way to do that on Android is to use AlarmManager and cheat by having your onRecieve method schedule another alarm.  You aren't doing that, and thus your app is subject to getting dozed by the operating system.
To Reproduce
Run this app long enough and it will stop scanning and broadcasting on some handsets under some manufacturer specific battery optimizations, etc.
Expected behaviour
There's really no reason an Android app should be limited.  Our app scans every 10 seconds, runs for days, doesn't drain the battery.
Add any other context about the problem here.
You can see a proper implementation here in our project:
https://github.com/kunai-consulting/OpenTrace/blob/develop/android/app/src/main/java/ai/kun/socialdistancealarm/alarm/BLETrace.kt
The TCN project works the same way as our does.

Thanks Aaron. Do you have any details of handsets which exhibit this behaviour? Part of our beta trial is looking at things like battery optimisation - and it's really helpful to know which specific devices we should be testing.
I'll pass on the point about using Alarm Manager.

@edent Huawei/Honor devices very aggressively kill apps in the background (my wife has one and despite having added exceptions for many apps in the power settings, it still kills things that have been in the background for a while). Samsung is another manufacturer that has a reputation for aggressive power management, but I don't have many issues with my own Samsung device. Hope this is useful.

Here is a website that details how to prevent specific devices from specific vendors from killing your app: https://dontkillmyapp.com/

I'm pasting this message in every active GitHub issue, so you may receive duplicate notifications.
Today, I'm happy to announce that NHSX has released the full git commit history for the Isle of Wight Beta apps.

Android - https://github.com/nhsx/COVID-19-app-Android-BETA
iOS - https://github.com/nhsx/COVID-19-app-iOS-BETA

As discussed, we have redacted API keys, sensitive domain names, and some of the developers' personal details. I am still waiting on final approval to publish the server-side code.
I would like to personally thank the community for your comments, bug reports, and vulnerability disclosures. They all went into helping the development process.
The beta trial of this app has now ended and we've moved to the next phase of app development. It is our intention to publish the source code of future apps as the binaries are released to the public.
Once again, thank you for being part of this.
Terence Eden
Head of Open Technology - NHSX


[DOCS] Fix typo in docs

What changes were proposed in this pull request?
Fix typo in docs
How was this patch tested?

Test build #71715 has started for PR 16658 at commit 2667f05.

Test build #71715 has finished for PR 16658 at commit 2667f05.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71715/
Test PASSed.

cc @srowen

To cut down on the number of these PRs, please spend some time to spell check the other docs too.

@srowen I have checked on one sweep of all the docs.

Test build #71915 has started for PR 16658 at commit 9e1e32a.

Test build #71915 has finished for PR 16658 at commit 9e1e32a.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/71915/
Test PASSed.

Merged to master


Need to Add Entitlement for Delivery Service for Integration Tester

Currently, the integration tester is not granted any entitlements for the delivery service (see https://github.com/Azure/osdu-infrastructure/blob/master/docs/osdu/integration-test-data/user_info_1.json). This causes delivery service integration tests to fail until those entitlements are granted. The entitlement that needs to be added is service.delivery.viewer (see https://community.opengroup.org/osdu/platform/system/delivery/-/blob/master/delivery-core/src/main/java/org/opengroup/osdu/delivery/api/DeliveryApi.java#L53).
Outstanding questions

Does this need to be added for both tenants?
Are there any other permissions that need to be granted to run the delivery integration tests?

This feature/code has fully migrated to GitLab.


Remove expand param from quote endpoints

This removes the unnecessary logic behind the expand param to get the content of a quote.
The frontend does not use this feature for now and it's good to remove complexity especially as we need to add public facing API as well.
The quote endpoints will always return its content for now.

Codecov Report

Merging #485 into develop will decrease coverage by 0.01%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop     #485      +/-   ##
===========================================
- Coverage    96.47%   96.45%   -0.02%
===========================================
  Files          114      114
  Lines         3627     3608      -19
  Branches       336      334       -2
===========================================
- Hits          3499     3480      -19
  Misses          87       87
  Partials        41       41

Impacted Files
Coverage Δ

datahub/omis/quote/views.py
100% <100%> (ø)
:arrow_up:

datahub/omis/quote/serializers.py
100% <100%> (ø)
:arrow_up:

datahub/company/models/adviser.py
84.21% <0%> (-0.28%)
:arrow_down:

datahub/search/company/serializers.py
98.94% <0%> (ø)
:arrow_up:

datahub/company/admin.py
89.47% <0%> (+0.58%)
:arrow_up:

Continue to review full report at Codecov.

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


Decomposing Uncertainty with GPflow

discussion
Hi,
is there any way to get aleatoric and epistemic uncertainty in GPflow (i.e. seperate stds)?
Best,
Tim

Hi @TimHahn1981, thanks for your interest in GPflow! I am not quite sure what kind of answer you are expecting - you can certainly build GP models with a heteroskedastic likelihood, e.g. see the varying noise notebook in the GPflow documentation. Are you looking for a model in which you model the variance with another GP, e.g. a chained GP (Saul et al, http://proceedings.mlr.press/v51/saul16.html)? The GPflow multioutput framework (tech report, example notebook) makes this easy - though you'll still have to implement a custom likelihood. If that is what you want, it may be more helpful if you ask a question on stackoverflow.com (tagged gpflow), so that other users can also benefit from the answer.

Hi @TimHahn1981, without a response from you there isn't anything we can do for you, so I'm closing this issue for now. You are very welcome to re-open it with your response!

(Note- we're in the process of adding heteroskedastic likelihoods to GPflow, see #1455)

And we're discussing the design in the #likelihood-design channel on the gpflow.slack.com chat (invite link in GPflow's README.md)


In mxnet or offical mxnet imagenet pretrained model, When processing images,the data layout is batchSize x channel x width x height or batchSize x channel x height xwidth ？

#cv2.imread returns [height, width, channel] in BGR
cv2img = cv2.cvtColor(cv2.imread(finename), cv2.COLOR_BGR2RGB)
#http://mxnet.io/tutorials/python/predict_imagenet.html
cv2img.shape
cv2img = np.swapaxes(cv2img, 0, 2)
cv2img = np.swapaxes(cv2img, 1, 2)#this return (channel,height,width)

in tutorials-predict_imagenet I see the predict imagenet the pre-trained model forward image_shap is as above(channel,height,width).
but I know, in the mxnet pre-trained model for example vgg-imagenet-pretrained-model and , the image-shape is (batch_size,channel,width,height). but this is
and, in the offical mxnet readme, When processing images, MXNet assumes the data layout is
batchSize x channel x width x height.

This issue is closed due to lack of activity in the last 90 days. Feel free to ping me to reopen if this is still an active issue. Thanks!


Improve TemplateBuilder & remove Java MessageFormat dependency

Purpose

Fix for https://github.com/siddhi-io/siddhi/issues/858 ...

Codecov Report

Merging #1431 into master will decrease coverage by 0.02%.
The diff coverage is 80.95%.

@@             Coverage Diff              @@
##             master    #1431      +/-   ##
============================================
- Coverage     67.57%   67.55%   -0.03%
  Complexity       41       41
============================================
  Files           634      634
  Lines         26718    26753      +35
  Branches       4219     4225       +6
============================================
+ Hits          18055    18073      +18
- Misses         6892     6909      +17
  Partials       1771     1771

Impacted Files
Coverage Δ
Complexity Δ

...io/siddhi/core/util/transport/TemplateBuilder.java
52.45% <80.95%> (+12.45%)
0 <0> (ø)
:arrow_down:

...e/util/statistics/memory/ObjectSizeCalculator.java
73.58% <0%> (-2.52%)
0% <0%> (ø)

...ut/stream/state/AbsentStreamPreStateProcessor.java
85.21% <0%> (-1.41%)
0% <0%> (ø)

...y/api/execution/query/output/stream/UpdateSet.java
24.44% <0%> (-1.14%)
0% <0%> (ø)

...e/src/main/java/io/siddhi/core/util/Scheduler.java
81.64% <0%> (-0.53%)
0% <0%> (ø)

...core/src/main/java/io/siddhi/core/table/Table.java
27.54% <0%> (-0.48%)
0% <0%> (ø)

.../io/siddhi/core/util/snapshot/SnapshotService.java
71.63% <0%> (-0.42%)
0% <0%> (ø)

...nt/stream/holder/SnapshotableStreamEventQueue.java
75.91% <0%> (-0.4%)
0% <0%> (ø)

...ore/util/parser/helper/DefinitionParserHelper.java
74.48% <0%> (+0.05%)
0% <0%> (ø)
:arrow_down:

...ore/table/record/AbstractQueryableRecordTable.java
85.93% <0%> (+0.05%)
0% <0%> (ø)
:arrow_down:

... and 8 more

Continue to review full report at Codecov.

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


add multiview helper to make the sentry multiview aware.

:scroll: Description
Currently, the Sentry Dart Plugin does not completely support the new multi-view feature for the web. Especially the SentryScreenshotWidget,SentryUserInteractionWidget and the WidgetsBindingIntegrationare incompatible.
Therefore we deactivate these features if we detect a multi-view application.
To find out if multiViewEnabled is set in the flutter_bootstrap.js I check the PlatformDispatcher.instance.implicitView return value.
In a regular non multiview app PlatformDispatcher.instance.implicitView returns a FlutterView object. If you try to call PlatformDispatcher.instance.implicitView you receive null. This is also explained here in the Flutter Docs
An alternative approach I found out, could be accessing the __flutterState object and look at the elements. In a regular non multiview application the first element is always a meta event.

For a MultiView App the __flutterState only contains flutter-view elements for the number of active views.
In this example, there were two active views:

:bulb: Motivation and Context
Based on the discussion here I added a MultiViewHelper, which is only active for the Web Platform.
:green_heart: How did you test it?
local
:pencil: Checklist

[x] I reviewed submitted code
[ ] I added tests to verify changes
[x] No new PII added or SDK only sends newly added PII if sendDefaultPii is enabled
[x] I updated the docs if needed
[x] All tests passing
[x] No breaking changes

:crystal_ball: Next steps

I'll add an integration test for this