Contribute to the language status

Hi  👋
Currently the Python extension contributes individual items to the status bar. On my macOS I see the Python version being used on the left side (picture). There might be other items?
I suggest to use the language status bar, Typescript is already using this (picture attached).
This way all the Python status bar items will come under a common item making the whole status bar experience cleaner for our users.
The status bar items which are actually progress our current advise is to use the withProgress-API. With that you announce a long running thing to VS Code. Our current thinking is to add a new progress location which is the extension’s language status. With that we can render something on the composite item as well as on the item itself. This is being discussed here, please chime in.
fyi @brettcannon @jrieken

I contributed to the discussion at https://github.com/microsoft/vscode/issues/129037#issuecomment-930471834 where I mentioned what we need in order to adopt the API wholesale (we already use it on web and plan on using it for virtual and untrusted workspaces as-is). @karthiknadig plans to bring up our use-case at the next API sync.

@brettcannon thanks. I missed your comment in that issue, sorry about that.
I agree that it might be a shock to the user that in one milestone we move to the right and move it inside the language status. Due to that I propose the following:

This release simply move your status bar contribution to the right. I see you have explained the reasoning why it is left here but I think we should just change this
Listen to user feedback
After we hopefully see that the users did not really get alarmed by this change move the item inside the language status

Typescript only got the first complaint about this change 2 months after the release. So it was smooth sailing, let's try to do the same for Python :)

I'm afraid the level of detail that users need to have about their environment for Python is bit more nuanced and extensive than TypeScript, so I don't think we can equate their experience to ours unfortunately.
We can talk about moving to the right sooner rather than later, but it will require updating a ton of screenshots so it isn't a quick fix. I would rather not try to squeeze it into this iteration unless open capacity shows up unexpectedly.

Moving to this is also already in the deferred items list in our iteration plan, so we do hope to tackle this in the upcoming iteration.

[ ] Move status bar item to the right
[ ] Look into all the screenshots we need to upgrade
[ ] Discuss with VSCode about the capabilities we may need, like pinning.

I attempted to do a basic move

@isidorn Right now it appears weird as it "Python" appears twice. Each status bar has a priority number attached to it, can I set such a number which ensures it always appears next to the language status item? In that case we can simply remove "Python".

@karrtikr thanks for giving it a try!
Yeah, I think you can and that is exactly what Typescript is doing (once the user pins the typescript version there is no duplication).
As for what is the exact right priority number for this @jrieken might help.

There is no exact priority number because it's an open system for all extensions. Tho, we want to ensure that pinned language status items are always directly after the language name. So, how about using the new language status API and then we discuss if pinned by default should be a thing or not?

Sure. We have decided to follow the approach described by @isidorn in https://github.com/microsoft/vscode-python/issues/18040#issuecomment-972950995, but have the 3rd step behind an experiment, as we suspect Users would likely want pinning by default. We can discuss after that.

I see. For the priority you can try something like 100.09999 but pinned language status items are meant to win. So, that might lead to <Python> <Some_Python_Language_Status> <Your Status Bar Item>

Nice, thanks! I'll try it out.

Works great:

We plan to remove bitness "64 bit" to make this even shorter. We discussed as a team again and right now the plan is to A/B experiment with this hack, when using language status API, and the control group (where this is on the left as it is). We can decide based on that whether pinning is needed.

@isidorn I am working on adopting the language status API and here's how it looks:

@isidorn I am working on adopting the language status API and here's how it looks:

After pinning and hovering:

When interpreter selected is invalid:

After pinned:

It looks good. However I discovered a bug where the language status item does not appear when a language mode is changed: https://github.com/microsoft/vscode/issues/140589. We have a getting started experience where we ask the user to create an untitled file and then select an interpreter using the status bar (now language item), so we would need that bug to be addressed before we can adopt the language status API.

[x] Add experiments in experiment tower
[ ] Add pythonInterpreterInfoPinned experiment to where the pinned interpreter status bar item is moved to the right https://github.com/microsoft/vscode-python/issues/18282
[ ] Add pythonInterpreterInfoUnpinned experiment to change interpreter status bar item into a language status item https://github.com/microsoft/vscode-python/issues/18283
[ ] Look into any screenshots we need to update with the getting started experience regarding each experiment
[ ] Rollout the pythonInterpreterInfoPinned experiment
[ ] Wait for https://github.com/microsoft/vscode/issues/140589 to get fixed after which rollout the language status API experiment

@karrtikr thanks for looking into this and thanks for filling that issue with nice steps. We plan to look into it this milestone.
I do agree that it looks good!

Joh just pushed a fix for the issue you filled. It should be in Insiders on Monday so try it out and let us know if there is anything else we can do to help you adopt the language status. Thanks

Sure, thanks a lot for the quick fix!


Add Internvl2TemplateWithAngles template

PR type

[ ] More Models or Datasets Support

PR information
Template A is added to support entering image angles，and the dataset format is as follows:
{"query": "please describe the image.", "response": "An old man combing the hair of a young boy.", "images": ["/mnt/g/dongyongfei786/custom-swift/examples/data_processing/output/train/235812.jpg"], "angles": [0]}
add custom templates：
--template internvl2-angle

Thank you very much for your PR. Could you please describe the reason for submitting it?😊

Thank you very much for your PR. Could you please describe the reason for submitting it?😊

Hello, I'm glad to receive your reply. The reason for this PR is that when extracting multimodal image information, the usual practice is to classify the images into angles (such as 0, 90, 180, 270, etc.), and then input the corrected angle images into the model. Based on our experience, this approach works better with a small number of samples for fine-tuning training. The original data format did not support the input of image angles, so we are submitting this PR.


rclone is available with restic-exporter?

Hello,
I'm trying to implement rclone with your restic-exporter, but I get an error:
restic-exporter-3  | 2023-06-14 15:27:00 ERROR    Unable to collect metrics from Restic. Exception: Error executing restic snapshot command: Fatal: unable to open repository at rclone:pcloud:OVH: exec: "rclone": executable file not found in $PATH  Exit code: 1

Here my docker-compose.yaml :

@ngosang Could you confirm rclone is working with your restic-exporter?
Thanks

I fixed this by mounting the rclone executable from my host inside the docker container.
While I'm not sure if this is best practice. It does allow the rclone version to be in sync with the version I'm running on my host machine.
restic_exporter:
    image: ngosang/restic-exporter
    container_name: restic-exporter
    environment:
      TZ: Europe/Amsterdam
      RESTIC_REPO_URL: rclone:gd-backup:
      RESTIC_REPO_PASSWORD:
      REFRESH_INTERVAL: 1800 # 30 min
    volumes:
      - /docker:/data:ro
      - /usr/bin/rclone:/usr/bin/rclone:ro
      - /mnt/config/rclone.conf:/root/.config/rclone/rclone.conf
    ports:
      - "8001:8001"
    restart: unless-stopped

Hello @koenvdheuvel,
Thanks for your example. It's working well.

Thank you both. I'm not going to include rclone in the Docker image because it weights 50+ MB but I will document your solution.


⚠ gorzdrav.org has degraded performance

In 7b10b94, gorzdrav.org (https://gorzdrav.org) experienced degraded performance:

HTTP code: 200
Response time: 6102 ms

Resolved: gorzdrav.org performance has improved in d0e3826.


Avoid using OpenStruct where Struct is enough

On PR #1036, I saw a CI failure that is unrelated to its changeset.
https://norsk.cf-app.com/builds/446820782
The error NameError: uninitialized constant LicenseFinder::OpenStruct implies that we should explicitly bundle "ostruct" gem, but in this case I'd suggest to simply use Struct instead of OpenStruct.

@amatsuda Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.

@pivotal-cla This is an Obvious Fix

@amatsuda This Pull Request contains an obvious fix. Signing the Contributor License Agreement is not necessary.

Ruby 2.6 CI fails https://norsk.cf-app.com/builds/446825268 due to ba4910e02d77ed7047a4d6dfaa8b690bfbc3846b.
Perhaps you'd better either revert this or drop Ruby < 2.7 support.

@amatsuda Thanks for the PR. I saw this after but added the fix by explicitly requiring openstruct

@xtreme-shane-lattanzio No problem, I'm totally fine with that. But maybe I should have wrote in the PR description that use of ostruct is now mildly warned due to performance reason https://github.com/ruby/ostruct/pull/56
and that's why I suggested to switch to Struct rather than simply adding a require statement.


Table not opening within 5 seconds after clicking on availability button.

As per the docs https://github.com/yashwanthm/cowin-vaccine-booking/wiki/Usage-Guide
Point 6
"If you are not able to see the table within 5 seconds, then the tracking is not happening. Please refresh and start over. This can also happen when there are no vaccination centers in your pincode. Chaniging to district based tracking will help"
I do not see a pop-up as mentioned.
Moreover, I was using the bot to look for slots.
The slots opened ( as per notification from some other app : telegram group ) and the bot could not book it.
Please guide for the same.

I think some people are mistaking the placeholder text with the actual value on the input.
I've made a fix for it. Try it now.
I'm not sure of what you mean by pop-up. If the app detects that the slots have opened up, it will speak up and show you the captcha.


Fix onchain logic

This is a direct PR from Brians fork to our own, I have yet to read through this completely and I will be leaving my own comments below. I have requested review from Brian, Siegfried, and Dz.
Thanks all.

Looking through the code, I'll make one last fix of a function name. Give me a min.


fix typo in problem-details-service

While going through the samples in the docs I stumbled across this typo in the example.
Fix the Arguement typo to Argument.

@timdeschryver  thanks! I don't check this repo often, feel free to ping me for quicker reviews.


HI, I have questions about the training data.

Thanks for your contribution, can you give more details about how to get the training data? I have tried to looking for it but failed, please give me a hand! Many Thanks！

Hi there,
please refer to the gitter chat (https://gitter.im/DLTK/DLTK) if you have any questions or email the respective authors of the project (email address should be in the respective README.md file). Since this is neither a bug or issue, I am closing this.


Add: HSE24

Channel Name
HSE24
Alternative Names (optional)
No response
Network (optional)
No response
Owners (optional)
No response
Country
IT
Subdivision (optional)
No response
City (optional)
Rome
Broadcast Area
c/IT
Languages
ita
Categories (optional)
shop
NSFW
FALSE
Launched (optional)
No response
Closed (optional)
No response
Replaced By (optional)
No response
Website (optional)
https://www.facebook.com/lundhalseyconsolesystems/videos/hse-24/306298704420152/
Logo
https://s3.amazonaws.com/playstore/images/53386f676c88d4fbc232ab58128aa9ee
Notes
No response

Already in the database: https://iptv-org.github.io/channels/it/GM24

From 5 July 2021 HSE24 changes its name to GM24

https://logos.fandom.com/wiki/GM24


Use ESLint to lint JavaScript code blocks inside .cshtml

Linting external JavaScript files is great, but let's also cover inline script as it's still widely used.
The html plugin already supports inline <script> block in html files - maybe it's enough to add .cshtml to the list of html extensions? Maybe not.

I think you will get syntax errors from the Razor. Copy a script block into a JS file first to check what can you expect before investing time setting up the plugin,

What do you mean by this:

Copy a script block into a JS file first to check what can you expect

?

What I wrote in the previous sentence. :) I expect that you will get syntax errors in the script block if you treat cshtml as html. You can verify it by just taking the content of the script block and putting it into a JS file and running ESLint on it.
Also see razor-vue-lint. It doesn't seem to be maintained, but the readme shows an attempt at wrapping Razor syntax for ESLint.

Oh, I totally forgot about code inside of script blocks, too - that'd be tricky to work around.

Due to C# and JS being mixed, this would indeed be tricky. I think we can't realistically tackle this, rather, we should focus on having as little inline JS as possible. Serious issues also impacting runtime will then be surfaced during UI tests.

Perhaps the html eslint plugin could be invoked by the UI test like we do with html-validate?

Hmm, interesting, yeah, we could do that, I guess.


Issue/7356 remove deprecated std iso6

Description
drop all usages of deprecated std resources from core test
(https://github.com/suite/inmanta-core/issues/7356)
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[x] Attached issue to pull request
[x] Changelog entry
[x] Type annotations are present
[x] Code is clear and sufficiently documented
[x] No (preventable) type errors (check using make mypy or make mypy-diff)
[x] Sufficient test cases (reproduces the bug/tests the requested feature)
[x] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )
[ ] If this PR fixes a race condition in the test suite, also push the fix to the relevant stable branche(s) (see test-fixes for more info)

Processing this pull request

Merged into branches iso6 in 33bc2cb1242f410c4f1b6a5c3e424be3a7ebaac7


Unit conversion of ut1_to_tt in cirs_to_itrs(), itrs_to_cirs(), tod_to_itrs(), and itrs_to_tod()

I think there is another error related to the time calculation, specifically on this line in cirs_to_itrs. ut1_to_tt should not be subtracted from jd_tt_low.

Hi @hannorein,
I think it was actually correct, since cel2ter() needs UT1-based JD, not TT-based JD as its inputm, whereas cirs_to_itrs() takes a gumentTT-based JD as its input. So, when making the call, the JD argument needs to be converted from TT-based to UT1-based.

The issue affects itrs_to_cirs(), tod_to_itrs(), and itrs_to_tod() also. The fix has been committed and pushed to the v1.0.1 branch.

I noticed the units don’t work out. But your are right about the UT vs TT

Given the severity of this bug, I also added the fix directly to the main branch...


Feature Request

Please add this features;
1. Bold text
2. Italic text
3. Underlined text
4. Limit text lenght : 1024 (Teamspeak not allowed higher value than 1024)

I haven't looked into this project for quite a while, and looking back on it the code seems like a huge mess.
Either way, going to see what I can do to reactor it or start anew.

Well the new version is there to use, however I was unable to compile v8 on windows so far therefore no release yet.


Links opening in Firefox instead of Tor Browser

Describe the bug
Since the latest Tor Browser update, when the open links in torbrowser option is enabled and default Firefox is running, links will be opened in Firefox instead of Tor Browser. This is not a bug with Tweelectron, but a long time issue with Tor Browser which was "fixed" until now.
Still opening this issue to warn people.
Might also be dependent on operation system, so if it works for you, please tell me.
To Reproduce
Steps to reproduce the behavior:

open Firefox
open Tweelectron
enable open links in torbrowser option
click on a link

Expected behavior
Link should open in Tor Browser
Desktop (please complete the following information):

OS/Distro: Solus
App Version: 1.1.0 (probably 1.0.10 too)

Ah, I see why I didn't notice this sooner.
I used Firefox dev before which doesn't have this problem.
So this problem was always there with default Firefox.
Solution:
Use Firefox Developer Edition or any other browser as default browser when using the open links in tor option.
Keeping this issue open, until I find a way to avoid this problem.

Closing since there is no solution


Error with certificate module

Receiving the following error ;
Parse error: syntax error, unexpected '[' in /usr/share/nginx/html/report/app/modules/certificate/certificate_model.php on line 77
Initially, I thought it was something to do with me migrating away from sqlite to mysql but I have now been able to replicate the same error with MR creating a fresh sqlite db.
For my setup, I'm running php-fpm 5.3.3 on Centos 6.

Its working fine for me now. Thanks!


Handle Move in TreeSelectionMode.

Previously, a NotifyCollectionChangedAction.Move would cause TreeSelectionNode to throw a NotSupportedException. This PR allows TreeSelectionNode to handle moves.
Note that if the moved item is currently selected, it will be unselected: it does not try to preserve a moved selection as that will require an API change in TreeSelectionModelIndexesChangedEventArgs to notify of multiple index ranges being changed.

Customer has tested and fixes their issue 🎈


Possibly a nice typo in the init

https://github.com/PFython/log2d/blob/de42d65877646a85fe3bef77e0fd86de2ba3fef7/log2d/__init__.py#L39-L41
isn’t “keyword” meant to be “key”?

@MatusGasparik the version of the code on GitHub isn't even the same as the actual version from PyPI 😬...
from inspect import getsource
from log2d import Log

print(next(filter(lambda l: "setattr" in l, getsource(Log).splitlines()), None))

'            setattr(self, key, value)'

Thanks - synced now.

Oops. Did I title this a „nice typo“??? Definitely a result of doing this on iPhone mini with a help from an intelligent text assistant…


Add Support for Wallaby

Wallaby support is only half-implemented. Right now, the tests run and pass, however source maps are not working, so updating the code does not run the tests. I'm not sure if Babel needs to be configured via a Webpack loader, Wallaby compiler or Wallaby preprocessor.

Fixed with c8e539e265167ee33542f6710ae2072228795731


Expand platforms we test in travis / cleanup cruft

Remove the old awesome customers stuff that isn't coming back and isn't used in learn chef anymore anyways. Nuke some extra fluff in the kitchen configs and avoid inspec deprecation warnings. Remove Debian 7 testing and add Amazon2 / Ubuntu 18.04

i'm very confused as to why the ruby-2.5.0 failures are back in this branch...

Yep same here. I wonder if we're landing on different boxes again


[Maven-project] Incorporate maven-proto-plugin into build of Maven projects

Reason for this PR

See #522
What changes are included in this PR?

Add maven-proto-plugin into pom.xml of java-info.
When the maven project is built, this plugin will compile protobuf into java source code into java-info/src/main/java/org/apache/graphar/proto.
Potential problem:

Are generated-code located in  java-info/src/main/java/org/apache/graphar/proto suitable?
Once the code was generated, developer should remove the code manually to avoid compile error.

Are these changes tested?

No
Are there any user-facing changes?

If protobuf is in maven build already let's remove it from buf.gen.yaml: https://github.com/apache/incubator-graphar/blob/main/buf.gen.yaml#L33

Maybe we can put the location of generated code to maven-projects/src/main/java/org/apache/graphar/proto/? since we want the generated code can be a independent package like arrow java's flatbuf

Why it would raise compile error if not delete the generated code?

maven-projects/src/main/java/org/apache/graphar/proto/

With modifications, the code well generated at maven-projects/proto/src/main/java/org/apache/graphar/proto, and will be clean automatically when regenerating.

Guys, why do we need both, buf-java plugin and maven-protobuf plugin? For me, if we do not want to store the code in the repository, let's remove java-generation from buf and leave only the maven plugin. Or did I miss something?

Guys, why do we need both, buf-java plugin and maven-protobuf plugin? For me, if we do not want to store the code in the repository, let's remove java-generation from buf and leave only the maven plugin. Or did I miss something?
hi, Sem, you are not miss anything. We do need leave only the maven plugin. @Thespica, could you delete the java-generation from buf if we use maven plugin.

maven-projects/src/main/java/org/apache/graphar/proto/

With modifications, the code well generated at maven-projects/proto/src/main/java/org/apache/graphar/proto, and will be clean automatically when regenerating.

Sorry for spell the path wrong, I suggest we can put the generated code to maven-projects/format/src/main/java/org/apache/graphar/proto, since it's the generated code for format.

maven-projects/src/main/java/org/apache/graphar/proto/

With modifications, the code well generated at maven-projects/proto/src/main/java/org/apache/graphar/proto, and will be clean automatically when regenerating.

Sorry for spell the path wrong, I suggest we can put the generated code to maven-projects/format/src/main/java/org/apache/graphar/proto, since it's the generated code for format.

I think it's okay to naming module as proto, since the package name is proto.

Hi @SemyonSinchenko
I have transfered to the newer plugin, could you review again?

It looks cool. Thank you!
I still have some concerns. It seems to me that we should use a maven-shade plugin to shade protobuf and rename generated classes... Otherwise, users may face dependency hell. But it is out of scoop for the current PR. Let's go step by step.


Hotfixes that don't have an issue break our convention of putting the issue in the merge message

We should discuss if we loosen our convention or make it part of a hotfix to always create an issue for it

I am in favor of loosening the convention.
Hotfixes can vary widely in complexity (flawed code to just typos) so not all of them need to be documented in an issue. I would argue that if there is an issue to link, we should do so and if not we can add a separate convention for the commit message. Something like [hotfix] oh no semicolons DO have a meaning in JS

Maybe the root of the problem is what we define as a hotfix.
I propose the following definition: It's the action needed to fix a functionality that does not work in production
Consequences of this definition are

There must always be an issue relating to it
We always branch from main (because production is part of the definition)
There must be a release attached to it

Things that are not production breaking, should not demand a hotfix, therefore can be attached to a release. This definition encapsulates the "this is urgent" concept, so anything related to style/convention/would-be-nice-to-have fall outside of this category.
What do you think?

I agree with shouldn't loosen the convention, once we do we'll open the door to commits that work their way into a release and we can't easily track them down.
What I suggest is to open an issue right before each release. It would serve different purposes:

any commit in the release branch can refer to the release issue
We can have a PR pointing to the release (with the usual Fixes #issue-number we also close it automatically)
the VERSION bump (and the changelog updates) would neatly refer to the issue and be part of the above-mentioned PR


Docker resources added

hey @yashdev9274 thanks for merging and making me one of the first contributors. and an apologize for making 3 commit as I was testing something.

great work, and thanks for adding resources.
Do check out other sections also and If your interested in WEB3 then check out my this repo


Improve pivot chart behavior

Before
When there is no keys in pivot setting, chart showed only one group even when there is more than one group.

After
When there is no keys but groups and values, show the multi-bar chart as one group named 'values'.

Before

After
Use origin value name as legend instead of 'values'.

Cool fixes!

Tested. Nice improvements!
I'm merging it


feat: modify divide plugin to help p2c/shortResponse plugin support all proxy plugins.

modify divide plugin to help p2c/shortResponse plugin support all proxy plugins.

Make sure that:

[x] You have read the contribution guidelines.
[x] You submit test cases (unit or integration tests) that back your changes.
[x] Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

Codecov Report

Merging #5187 (94eccf1) into master (fe5077c) will increase coverage by 0.02%.
Report is 1 commits behind head on master.
The diff coverage is 75.55%.

@@             Coverage Diff              @@
##             master    #5187      +/-   ##
============================================
+ Coverage     61.96%   61.98%   +0.02%
- Complexity     8575     8577       +2
============================================
  Files          1243     1244       +1
  Lines         37582    37587       +5
  Branches       3556     3553       -3
============================================
+ Hits          23287    23298      +11
+ Misses        12263    12261       -2
+ Partials       2032     2028       -4

Files
Coverage Δ

.../org/apache/shenyu/plugin/divide/DividePlugin.java
87.23% <100.00%> (+10.52%)
:arrow_up:

...java/org/apache/shenyu/plugin/brpc/BrpcPlugin.java
41.37% <100.00%> (ø)

...henyu/plugin/alibaba/dubbo/AlibabaDubboPlugin.java
65.38% <100.00%> (ø)

...va/org/apache/shenyu/plugin/motan/MotanPlugin.java
44.44% <100.00%> (ø)

...java/org/apache/shenyu/plugin/sofa/SofaPlugin.java
82.75% <100.00%> (ø)

...e/shenyu/plugin/springcloud/SpringCloudPlugin.java
93.93% <100.00%> (ø)

...nyu/plugin/response/strategy/RPCMessageWriter.java
84.61% <100.00%> (ø)

.../apache/shenyu/plugin/sentinel/SentinelPlugin.java
60.00% <50.00%> (-3.16%)
:arrow_down:

.../shenyu/plugin/apache/dubbo/ApacheDubboPlugin.java
70.00% <0.00%> (ø)

...java/org/apache/shenyu/plugin/grpc/GrpcPlugin.java
38.88% <0.00%> (ø)

... and 2 more

... and 21 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


⚠ Bofrost.lu has degraded performance

In aba4f78, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 2370 ms

Resolved: Bofrost.lu performance has improved in 3a8d732 after 31 minutes.


Adds an example of selenium service

This is a small example on how selenium can be added to ddev to e.g. run Codeception tests locally.

Yay!

Yeah.. Will have a look.

Looks like the "same", but the difference is that the example you're linking to is using the Chrome stand-alone only, and not the Selenium as well.
I would need to check up on that if I did "to much".
I'll add some information on how this container itself can be tested.
Thanks for your feedback.

Ping @tomasnorre , looking forward to getting this in.

Pong ;) Yeah, me too. Got away from it as I was focusing on the release of the TYPO3 Crawler.
Will see when I'll get to this, I honestly forgot about it because of my focus somewhere else.

As I'm not really used to use selenium, I'm not sure what else to add, if you have any ideas, please let me know.

BTW. Do I need to sign CLA? and if so how do I do that?

You have already signed the CLA, thanks! It doesn't have to be done for every PR.

Ok. I wasn't aware of that. But good to know :D
I knew i didn't have to every time, but didn't know I already did :)

@tomasnorre I've used the files from this PR to run Laravel Dusk tests and it works really well, thank you. However, I did get an error regarding volumes. I changed line 13
- /dev/shm:/dev/shm
to
- /ddev/shm:/ddev/shm
Which fixed my volume error. Not sure if this is an issue other people may come across.

Happy that I was useful for you too.
The problem you are referring to is unknown to me. I will test if your suggestion works in my setup too if so I have no problem using your suggestion.

@Michael-Brooks /dev/shm is a special linux driver for shared memory. You don't mention what kind of error you got, but it wouldn't make any sense to change "dev" to "ddev" there.
I don't understand why one would make /dev/shm a volume, so that line doesn't make any sense to begin with (to me).
(Looking at @tomasnorre 's source, https://github.com/SeleniumHQ/docker-selenium#version-3, it looks like this volume was possibly put there to allow two containers to communicate with each other)

@tomasnorre I apologize, but I haven't gotten you to add anything here that would explain how to manually test and validate this, despite several requests starting March 30. It seems like a really valuable PR, and it will be excellent, but it needs much more detail, especially manual testing detail, and explaining that it's not just selenium but chromedriver.
I'm going to close this for now, but let's get it reopened and get it merged!

It's perfectly fine. I have so many open things. I'll keep it on my todo list and when I get to it, I'll reopen.


I should be able to generate a "Paper Key" with just my PGP key

I would love to be able to generate a paper key from just my PGP key with Keybase. I would then use this single paper key to "on board" all my devices.
This could generate a graph of:
+--------+                                                   +-----------------+
| Device | +---+                                       +---+ | Social Identity |
+--------+     |                                       |     +-----------------+
               |                                       |
+--------+     |   +-----------+         +---------+   |     +-----------------+
| Device | +-----> | Paper Key | <-----> | PGP Key | <-----+ | Social Identity |
+--------+     |   +-----------+         +---------+   |     +-----------------+
               |                                       |
+--------+     |                                       |     +-----------------+
| Device | +---+                                       +---+ | Social Identity |
+--------+                                                   +-----------------+

Why?
For users like myself who do manage their own PGP key very well, but want to extend into Keybase and allow Keybase to create their device keys, with a single point of uplink contact to my true identity, my PGP key.

I agree, I really wish I could start with a Paper Key and then follow with my devices later. I'd really like to reuse my existing paperkey, much like I can reuse my PGP key as wel...

with a single point of uplink contact to my true identity, my PGP key.

Also one thing to consider. Adding this feature will NOT make the paper key your "single point of uplink" as you are probably imagining it.
In keybase, let's say you were able to make the graph you describe; revoking the paper key will not revoke the devices...
It seems like you are assuming that all the devices are connected by the paper key and revoking the paper key will be a kill switch for you. That's not true. You will need to revoke each device one by one.


Documentation for DelChr function is not complete (and very unclear)

The DelChr function has three parameters. The following case is missing from the documentation:
[the Github interface does not do code formatting of whitespace right]
If you omit both Where and Which parameters, then all spaces are removed from String .
Example:
var
    String: Text;
    NewString: Text;
    Text000: Label '   Windy West Solutions    ';   // note: can be multiple leading and trailing spaces
    Text001: Label '>%1< is transformed to >%2<';
begin
    String := Text000;
    NewString := DELCHR(String);
    MESSAGE(Text001, String, NewString);
end;
The message window displays the following:
> Windy West Solutions < is transformed to >WindyWestSolutions<

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 815ff32e-5231-8233-e84b-7e08c2bb1d71
Version Independent ID: 0ac36fe2-c3fb-a160-923b-509986272830
Content: DelChr Method - Business Central
Content Source: dev-itpro/developer/methods-auto/text/text-delchr-method.md
Service: dynamics365-business-central
GitHub Login: @SusanneWindfeldPedersen
Microsoft Alias: solsen

Thanks for your contribution - the topic will be updated with the next refresh.


** BUILD FAILED **

-scheme "SQLite iOS" -configuration Release -sdk iphoneos ONLY_ACTIVE_ARCH=NO BITCODE_GENERATION_MODE=bitcode CODE_SIGNING_REQUIRED=NO CODE_SIGN_IDENTITY= CARTHAGE=YES clean build) failed with exit code 65:
** CLEAN FAILED **
The following build commands failed:
Check dependencies
(1 failure)
The following build commands failed:
Check dependencies
(1 failure)

I saw this error yesterday too when I tried to use Carthage to install SQLite.swift as a dependency of my project.
I was able to get SQLite.swift to build if I installed manually and used Command-B to initiate the build.

did it's work, can you tell me you step ?

Can you retry with the latest instructions?

I am using Xcode 8.0.
I changed my Cartfile line from what was in the old SQLite.swift readme
github "stephencelis/SQLite.swift" ~> 0.10.1

to what is now in the new SQLite.swift readme
github "stephencelis/SQLite.swift" ~> 0.11.0

and then the build initiated by carthage update did succeed.
I did see a screenful of warnings at the end of the build that were like
<module-includes>:1:1: warning: umbrella header for module 'SQLite' does not include header 'sqlite3.h'

After following the Carthage instructions for Adding frameworks to an application,  I can now import SQLite in my AppDelegate file and building my project succeeds.

Closing

I have my project written in Swift 2.3 and I cannot use
github "stephencelis/SQLite.swift" ~> 0.11.0
When I use
github "stephencelis/SQLite.swift" ~> 0.10.1
I get the same above error.


feat: Add Aws support

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

GitHub release
v1.1.0

Your semantic-release bot :package::rocket:


Add tagging to this repository

While we currently track upstream OPA stable releases, we don't have tags set up on this repo. This makes it hard for users that want to build from source and track stable versions. We should update the vendoring update automation to tag the repository.
Also, I noticed that the automation is leaving branches around. Is there anyway we can avoid this?

We tag the repo manually while merging the automated pull requests. Also branches created by the automation are deleted during merge.


scanIterator on Cluster

Connection failed. TypeError: cluster.scanIterator(...) is not a function or its return value is not async iterable
Environment:

Node.js Version: 14.4.0
Redis Server Version: 6.0.5
Node Redis Version<EMAIL_ADDRESS>

This happens for me too


Still active development?

Hi.
I just came across your project on discourse and love the idea!
The last commit is over half a year ago. Do you still actively develop the project?

Active, no.
Bought a house, and have been busy with real life activities. But it's still on my list of things get around to.


fix indentation on validation emails

As the title says, fix the indentation on the verification email's that get sent out

Fixed in 612a863020a1df5d5fb6209d0f066fc32c77a376


org.jscep.transport.TransportException: 501 Not Implemented

Just upgraded against EJBCA Community <IP_ADDRESS> version and getting the above error.
Our acceptance tests pass within a docker container but when ran from VM and enrolling to localhost we get his failure.
I noticed strange chars from the stdout which are just due to copy/paste:
2015-07-01 13:36:03 [âœ—] Certificate Rollover
2015-07-01 13:36:03 Using [dn=C=UK, O=Moneysupermarket.com Group PLC, CN=root.tsm.internal; serial=723928201668310617] for issuing entity
2015-07-01 13:36:03 Requesting certificate verification.
2015-07-01 13:36:03 Certificate verification passed.
2015-07-01 13:36:03 Verifying signature of RA certificate
2015-07-01 13:36:03 RA and CA are identical
2015-07-01 13:36:03 Verifying signature of RA certificate
2015-07-01 13:36:03 RA and CA are identical
2015-07-01 13:36:03 Determining capabilities of SCEP server
2015-07-01 13:36:03 Sending GetCACaps(MSMCA) to http://localhost:8080/ejbca/publicweb/apply/scep/pkiclient.exe?operation=GetCACaps&message=MSMCA
2015-07-01 13:36:03 Received '200 OK' when sending GetCACaps(MSMCA) to http://localhost:8080/ejbca/publicweb/apply/scep/pkiclient.exe?operation=GetCACaps&message=MSMCA
2015-07-01 13:36:03 CA capabilities:
2015-07-01 13:36:03 [âœ—] Certificate Rollover
2015-07-01 13:36:03 [âœ“] HTTP POST
2015-07-01 13:36:03 [âœ“] Certificate Renewal
2015-07-01 13:36:03 [âœ—] SHA-512 Message Digest
2015-07-01 13:36:03 [âœ—] SHA-256 Message Digest
2015-07-01 13:36:03 [âœ“] SHA-1 Message Digest
2015-07-01 13:36:03 [âœ—] Triple DES Encryption
2015-07-01 13:36:03 SHA-1 PKCS#10 Fingerprint: [79765e06961fc5f83899071265c8dceaf3298263]
2015-07-01 13:36:03 Encoding pkiMessage
2015-07-01 13:36:03 Encoding message: org.jscep.message.PkcsReq@7008b996[transId=21b7616c184657b02de1e333bf05128293491177,messageType=PKCS_REQ,senderNonce=Nonce [32ed469ca95eb3bab3399a517dcd949b],messageData=org.bouncycastle.pkcs.PKCS10CertificationRequest@39282562]
2015-07-01 13:36:03 Encoding pkcsPkiEnvelope
2015-07-01 13:36:03 Encrypting pkcsPkiEnvelope using key belonging to [dn=C=UK, O=Moneysupermarket.com Group PLC, CN=root.tsm.internal; serial=723928201668310617]
2015-07-01 13:36:03 Finished encoding pkcsPkiEnvelope
2015-07-01 13:36:03 Signing pkiMessage using key belonging to [dn=CN=smoketest.inf.internal,OU=OPS,O=Moneysupermarket.com Group PLC,L=Chester,ST=Cheshire,C=UK; serial=9847771967412809691]
2015-07-01 13:36:03 Signing org.bouncycastle.cms.CMSProcessableByteArray@136c2147 content
2015-07-01 13:36:03 Finished encoding pkiMessage
2015-07-01 13:36:03 Sending org.bouncycastle.cms.CMSSignedData@26973f1b
2015-07-01 13:36:03 Received '501 Not Implemented' when sending org.bouncycastle.cms.CMSSignedData@26973f1b to http://localhost:8080/ejbca/publicweb/apply/scep/pkiclient.exe?operation=PKIOperation
org.jscep.transport.TransportException: 501 Not Implemented
2015-07-01 13:36:03 org.jscep.transport.TransportException: 501 Not Implemented

any ideas ?

The strange characters are ticks and crosses, used to indicate support for certain capabilities.  I suspect something in your execution environment isn't using unicode.
I have no idea about the EJBCA 501, I'm afraid.

Interesting might be due to upgrade then, I will check the details later in the week let you know thanks

:thumbsup:

Hey found the issue the CA was built without export Java policies by accident so not an issue with your super code.. closing

@pauldavidgilligan-msm could you please explain a bit more about the issue and how to fix it? I do not understand what you mean with "CA was built without export Java policies". Am having exactly the same error and am not getting any closer to a solution. You're my first and best lead at this point.

@lweijl I suspect @pauldavidgilligan-msm means that the server running EJBCA wasn't using the unlimited strength JCE policy files.  See here:
http://www.ejbca.org/docs/installation.html#Installation guide

correct yes see above


Other libs

I found this repository, which have a really good toolbar, but try to do the same thing as you.
As well this one, which have the drawer style, and also doing the same as you.
Can this lib ask to merge with them about those feature and might others?
Thanks.

See #320 for more.
I should note that react-native-material-ui doesn't actually polyfill Ripple and implementing a standalone toolbar is a waste of time as the app bar is usually the responsibility of whatever navigator library you use.


Fix exports entry in package.json

Closed in favor of https://github.com/devrnt/react-use-wizard/pull/167


Charge moves are not properly stripped

As pointed out by Foobs, Iguana does not currently properly strip charge commands, specifically the kind that specify a time (such as ~5a; ~a is fine).
To quote Elecbyte's documentation of KFM:
;   tilde (~) - to detect key releases
;          egs. command = ~a       ;release the a button
;               command = ~D, F, a ;release down, press fwd, then a
;          If you want to detect "charge moves", you can specify
;          the time the key must be held down for (in game-ticks)
;          egs. command = ~30a     ;hold a for at least 30 ticks, then release

We could probably just detect these and replace them with more 'normal' inputs during tokenization, shouldn't be too hard (probably).

This has now been fixed by d798ba1 using a simple regex to strip this data.
Maybe in the future we could make them distinguished in the movelist somehow, but this'll do for now.

Thinking about this again now that I understand charge characters a bit better, yeah this could definitely be refined to wrap the charge input in [] when outputting them. There's also a proposal for a dedicated "CHARGE" glyph in Ikemen GO by default, so I might just wait and see if that gets merged.


Add ability to specify output directory

Currently here - https://github.com/aurelia/bundler/blob/master/lib/bundler.js
I think we assume src and dist are in the same directory which is not always the case, such as in the new skeletons for asp.net 5 - https://github.com/aurelia/skeleton-navigation/tree/master/skeleton-typescript-asp.net5/src/skeleton-navigation-typescript-vs
Is there a configuration option (or can we add one) to fix this?

We only assume dist is within baseURL. They may be in different folder. This is how we specify the output directory:
  "dist/app-build": {            // Bundle filename and output path.
    "includes": [
      "[*]",                            // Module names to be included in the bundle. May be a pattern too. eg. `*`, `**/**/*`, `[*]`
      "*.html!text",
      "*.css!text"
    ],
    "excludes": [
      "dist/donot-bundle"               // Module names to be excluded
    ],
    "options": {
      "inject": true,                   // Default is true
      "minify": true,                   // Default is false
      "rev": true                       // Set it to true for revison suport. Default is false
    }
  },

We can specify another folder such as another/app-build as long as another folder lives within baseURL.

Ah good point thanks sorry forgot about >.<


[Provider] xAI

Title:

Added xAI to Provider

Description:

Done with chat completion part in xai
Integrate xAI provider configuration

Related Issues:

#693

Hi @Akshay-66 ! Thanks for the PR. We will review this today.

@Akshay-66 Please resolve conflicts.

Thank you @narengogi and @vrushankportkey for guiding me in every step possible and thanks to @VisargD and @b4s36t4 for reviews and allowing me to do everything 😄

Thank you @Akshay-66 for the contribution!


[BUG] Unable to create Alert

Request Type
Bug
Work Environment

Question
Answer

OS version (server)
Ubuntu 20.04.1 LTS

OS version (client)
Ubuntu 20.04.1 LTS

TheHive4py version / git hash
1.8.0

Problem Description
Unable to create Alert from Samples/test-alert-create.py. Running fail with python error in models.py
Steps to Reproduce

Add APIKey to test-alert-create.py
Run  python3 test-alert-create.py -> Got error
Remove 1.8.0 vesrion: pip3 uninstall thehive4py
Install 1.7.1 version: pip3 install thehive4py==1.7.1
Run  python3 test-alert-create.py -> Alert created successfully

Possible Solutions
Temporary solution: downgrade thehive4py
Complementary information
Error:
Traceback (most recent call last):
File "test-alert-create.py", line 20, in
AlertArtifact(dataType='file', data='pic.png'),
File "/home/user/.local/lib/python3.8/site-packages/thehive4py/models.py", line 697, in init
file_object = open(data[0], 'rb')
FileNotFoundError: [Errno 2] No such file or directory: 'p'

Hello @Rodionc thanks for this issue. In fact it looks like a regression related to alerts with file artifacts.
I'll fix it in a 1.8.1 patch release

Hello @Rodionc thanks for this issue. In fact it looks like a regression related to alerts with file artifacts.
I'll fix it in a 1.8.1 patch release

Fixed this issue to make sure the code below is correct:
For Alert Artifacts
inmemory_file = open("hello.txt", "rb")
artifacts = [
    AlertArtifact(dataType='file', data='sample.txt'),
    AlertArtifact(dataType='file', data=(inmemory_file, 'hello.txt'))
]

For Case Observables
CaseObservable(dataType='file', data='sample.txt', tags=['thehive4py'])

inmemory_file = open("hello.txt", "rb")
CaseObservable(dataType='file', data=(inmemory_file, 'hello.txt'), tags=['thehive4py'])

Fixed this issue to make sure the code below is correct:
For Alert Artifacts
inmemory_file = open("hello.txt", "rb")
artifacts = [
    AlertArtifact(dataType='file', data='sample.txt'),
    AlertArtifact(dataType='file', data=(inmemory_file, 'hello.txt'))
]

For Case Observables
CaseObservable(dataType='file', data='sample.txt', tags=['thehive4py'])

inmemory_file = open("hello.txt", "rb")
CaseObservable(dataType='file', data=(inmemory_file, 'hello.txt'), tags=['thehive4py'])


🐛 [cloud_firestore] Error while mixing FieldPath.documentId and field name in where conditions

Bug report
Describe the bug
Firestore is not returning data when in "where" conditions are mixed string (as a field) and FieldPath.documentId . In this case with 2 where conditions I'm getting an error:

Client specified an invalid argument. Note that this differs from failed-precondition. invalid-argument indicates arguments that are problematic regardless of the state of the system (e.g., an invalid field name).
invalid-argument
W/Firestore(11362): (22.1.2) [WatchStream]: (fcb8fd6) Stream closed with status: Status{code=NOT_FOUND, description=Target id not found: 74, cause=null}.

With only one "where" condition I have no trouble to get expected document, no matter which one I persist (conditions are not excluding themselves).
Steps to reproduce
Steps to reproduce the behavior:

Try to retrive data with 2 "where" conditions, where one is referring to field, and one to FieldPath.documentId
Get an error instead of data

Expected behavior
It should return docs that fulfill both conditions.
Sample project
_firestore
                .collection(Event.COLLECTION_NAME)
                .where(Event.FIELD_STATUS, isNotEqualTo: EventStatus.STATUS_DELETED)
                .where(FieldPath.documentId, whereIn: eventIds)
                .snapshots()

Additional context
eventIds is a valid list of document ids - this condition is working as aspected when first "where" condition is not present.

Flutter doctor
Run flutter doctor and paste the output below:
Click To Expand
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 2.0.6, on Mac OS X 10.15.6 19G2021 darwin-x64, locale pl-PL)
[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.3)
[✓] Xcode - develop for iOS and macOS
[✓] Chrome - develop for the web
[✓] Android Studio
[✓] Android Studio (version 4.2)
[✓] IntelliJ IDEA Ultimate Edition (version 2019.2.2)
[✓] Proxy Configuration
[✓] Connected device (2 available)

• No issues found!

Flutter dependencies
Run flutter pub deps -- --style=compact and paste the output below:
Click To Expand
Dart SDK 2.12.3
Flutter SDK 2.0.6
event_app 0.0.35+35

dependencies:
- cached_network_image 3.0.0 [flutter flutter_cache_manager octo_image]
- cloud_firestore 2.1.0 [cloud_firestore_platform_interface cloud_firestore_web firebase_core firebase_core_platform_interface flutter meta]
- firebase_auth 1.1.4 [firebase_auth_platform_interface firebase_auth_web firebase_core firebase_core_platform_interface flutter meta]
- firebase_core 1.1.1 [firebase_core_platform_interface firebase_core_web flutter meta]
- firebase_crashlytics 2.0.3 [firebase_core firebase_core_platform_interface firebase_crashlytics_platform_interface flutter stack_trace]
- firebase_storage 8.0.6 [firebase_core firebase_core_platform_interface firebase_storage_platform_interface firebase_storage_web flutter]
- flutter 0.0.0 [characters collection meta typed_data vector_math sky_engine]
- flutter_spinkit 5.0.0 [flutter]
- fluttertoast 8.0.6 [flutter flutter_web_plugins]
- provider 5.0.0 [collection flutter nested]
- qr_code_scanner 0.4.0 [flutter]
- smooth_page_indicator 0.3.0-nullsafety.0 [flutter]
- url_launcher 6.0.3 [flutter url_launcher_platform_interface url_launcher_linux url_launcher_macos url_launcher_windows url_launcher_web]

dev dependencies:
- flutter_test 0.0.0 [flutter test_api path fake_async clock stack_trace vector_math async boolean_selector characters charcode collection matcher meta source_span stream_channel string_scanner term_glyph typed_data]

transitive dependencies:
- archive 3.1.2 [crypto path]
- async 2.5.0 [collection]
- boolean_selector 2.1.0 [source_span string_scanner]
- characters 1.1.0
- charcode 1.2.0
- clock 1.1.0
- cloud_firestore_platform_interface 5.0.1 [collection firebase_core flutter meta plugin_platform_interface]
- cloud_firestore_web 2.0.1 [cloud_firestore_platform_interface firebase_core firebase_core_web flutter flutter_web_plugins js]
- collection 1.15.0
- crypto 3.0.1 [collection typed_data]
- fake_async 1.2.0 [clock collection]
- ffi 1.0.0
- file 6.1.1 [meta path]
- firebase_auth_platform_interface 4.2.2 [firebase_core flutter meta plugin_platform_interface]
- firebase_auth_web 1.1.2 [firebase_auth_platform_interface firebase_core firebase_core_web flutter flutter_web_plugins http_parser intl js meta]
- firebase_core_platform_interface 4.0.1 [collection flutter meta plugin_platform_interface]
- firebase_core_web 1.0.3 [firebase_core_platform_interface flutter flutter_web_plugins js meta]
- firebase_crashlytics_platform_interface 3.0.3 [collection firebase_core flutter meta plugin_platform_interface]
- firebase_storage_platform_interface 2.0.4 [collection firebase_core flutter meta plugin_platform_interface]
- firebase_storage_web 1.0.6 [async firebase_core firebase_core_web firebase_storage_platform_interface flutter flutter_web_plugins http js meta]
- flutter_blurhash 0.6.0 [flutter meta pedantic]
- flutter_cache_manager 3.0.2 [clock collection file flutter http image path path_provider pedantic rxdart sqflite uuid]
- flutter_web_plugins 0.0.0 [flutter js characters collection meta typed_data vector_math]
- http 0.13.3 [async http_parser meta path pedantic]
- http_parser 4.0.0 [charcode collection source_span string_scanner typed_data]
- image 3.0.2 [archive meta xml]
- intl 0.17.0 [clock path]
- js 0.6.3
- matcher 0.12.10 [stack_trace]
- meta 1.3.0
- nested 1.0.0 [flutter]
- octo_image 1.0.0+1 [flutter flutter_blurhash]
- path 1.8.0
- path_provider 2.0.1 [flutter path_provider_platform_interface path_provider_macos path_provider_linux path_provider_windows]
- path_provider_linux 2.0.0 [path xdg_directories path_provider_platform_interface flutter]
- path_provider_macos 2.0.0 [flutter]
- path_provider_platform_interface 2.0.1 [flutter meta platform plugin_platform_interface]
- path_provider_windows 2.0.1 [path_provider_platform_interface meta path flutter ffi win32]
- pedantic 1.11.0
- petitparser 4.1.0 [meta]
- platform 3.0.0
- plugin_platform_interface 2.0.0 [meta]
- process 4.2.1 [file path platform]
- rxdart 0.27.0
- sky_engine 0.0.99
- source_span 1.8.0 [charcode collection path term_glyph]
- sqflite 2.0.0+3 [flutter sqflite_common path]
- sqflite_common 2.0.0+2 [synchronized path meta]
- stack_trace 1.10.0 [path]
- stream_channel 2.1.0 [async]
- string_scanner 1.1.0 [charcode source_span]
- synchronized 3.0.0
- term_glyph 1.2.0
- test_api 0.2.19 [async boolean_selector collection meta path source_span stack_trace stream_channel string_scanner term_glyph matcher]
- typed_data 1.3.0 [collection]
- url_launcher_linux 2.0.0 [flutter]
- url_launcher_macos 2.0.0 [flutter]
- url_launcher_platform_interface 2.0.2 [flutter plugin_platform_interface]
- url_launcher_web 2.0.0 [url_launcher_platform_interface meta flutter flutter_web_plugins]
- url_launcher_windows 2.0.0 [flutter]
- uuid 3.0.4 [crypto]
- vector_math 2.1.0
- win32 2.0.5 [ffi]
- xdg_directories 0.2.0 [meta path process]
- xml 5.1.0 [collection meta petitparser]

@msmietankopzu Could you provide a example (dummy data) of one of the records in your collection? Should help making debugging the exact issue easier.

agenda and buttons are collections
agenda

1620873000000 "sdas"
buttons
details "EVENT_DETAILS"
code "123123123"
description "DESCRIPTION"
image "LINK_TO_IMAGE"
status "DELETED"
title "DUMMY TITLE"

I'm afraid that query isn't possible because it ultimately results in two inequality checks due to using the documentId field as well as isNotEqualTo. See this SO answer for further clarification. Although, we should probably throw an error before this query gets to native platforms and confirm with a test. Thanks for raising this issue, @msmietankopzu.


16506 Draft (Slider Widgets)

Draft files for 16506. Includes:

Update for main Widgets landing page, added descriptions of properties which many widgets have in common.

Also added explanations + video for some properties which didn't have them

Slider widget landing page, with use-cases, links to subpages, events, styles
Individual sub-pages included for:

Category Slider
Number Slider
Range Slider

Updates complete and ported to GitBook, see here for active CR

Can you please close this @spark-c ?


[BUG][WEBJS] - Unable to Send Poll

I am Using this :
curl -X 'POST'
'http://<IP_ADDRESS>:3001/api/sendPoll'
-H 'accept: /'
-H 'X-Api-Key: 933933'
-H 'Content-Type: application/json'
-d '{
"chatId"<EMAIL_ADDRESS>"poll": {
"name": "How are you?",
"options": [
"Awesome!",
"Good!",
"Not bad!"
],
"multipleAnswers": false
},
"session": "7891XXXX33_J1V_WHA"
}'
Getting Error :
{
"statusCode": 422,
"message": "The method is not implemented by the engine.",
"error": "Unprocessable Entity"
}

Are you using WEBJS engine? Because polls are available only in NOWEB right now
https://waha.devlike.pro/docs/engines/noweb/

Comment +1 in https://github.com/devlikeapro/waha/issues/189 to get a notification when it's ready in WEBJS!


Intermittent Build Failure due to CommonMark

Details

Project URL: https://readthedocs.org/projects/fyrd/
Build URL (if applicable): https://readthedocs.org/projects/fyrd/builds/5785772/
Read the Docs username (if applicable): mike.dacre

Expected Result
Build to work
Actual Result
About 80% of the time, I get the following error:
Running Sphinx v1.5.3
making output directory...

Traceback (most recent call last):
  File "/home/docs/checkouts/readthedocs.org/user_builds/fyrd/envs/latest/lib/python3.5/site-packages/sphinx/application.py", line 512, in setup_extension
    mod = __import__(extension, None, None, ['setup'])
  File "/home/docs/checkouts/readthedocs.org/user_builds/fyrd/envs/latest/lib/python3.5/site-packages/sphinxarg/ext.py", line 13, in <module>
    from sphinxarg.markdown import parseMarkDownBlock
  File "/home/docs/checkouts/readthedocs.org/user_builds/fyrd/envs/latest/lib/python3.5/site-packages/sphinxarg/markdown.py", line 1, in <module>
    from CommonMark import Parser  # >= 0.5.6
ImportError: cannot import name 'Parser'

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "/home/docs/checkouts/readthedocs.org/user_builds/fyrd/envs/latest/lib/python3.5/site-packages/sphinx/cmdline.py", line 295, in main
    opts.warningiserror, opts.tags, opts.verbosity, opts.jobs)
  File "/home/docs/checkouts/readthedocs.org/user_builds/fyrd/envs/latest/lib/python3.5/site-packages/sphinx/application.py", line 199, in __init__
    self.setup_extension(extension)
  File "/home/docs/checkouts/readthedocs.org/user_builds/fyrd/envs/latest/lib/python3.5/site-packages/sphinx/application.py", line 516, in setup_extension
    err)
sphinx.errors.ExtensionError: Could not import extension sphinxarg.ext (exception: cannot import name 'Parser')

Extension error:
Could not import extension sphinxarg.ext (exception: cannot import name 'Parser')

The other 20% of the time, it works fine, with no changes to the code. I just have to fire off builds over and over again until one randomly works. It also works fine on my own computers.

Now this happens every time, it is clearly caused by sphinx-argparse trying to use CommonMark and for some reason rtfd is installing CommonMark version 0.5.4, which is not working.
I have made this work for me by just removing sphinx-argparse and doing manual documentation, but that is hardly a solution.

I need this to be fixed also please.

same problem here:
https://readthedocs.org/projects/mpy-dev-tools/builds/5988423/

I'm hitting the same problem: https://readthedocs.org/projects/rucio/builds/6710110/
I tried to enforce CommonMark==0.7.4 in the project requirements and configured in rtds but it didn't help.
Would you know any work around  ?

This issue now has a workaround implemented in sphinx-argparse: https://github.com/ribozz/sphinx-argparse/pull/84.
The issue still isn't fixed though, there still should be a way to use CommonMark greater than 0.5.4 in rtfd builds.

I'm not sure why is this happening, you should be able to download a new version of CommonMark by adding it to your requirements.txt file. Have anyone tried to wipe the environment before testing with a requirements file? (versions -> edit -> wipe)

I managed to get this working for some projects by editing requirements.txt:
sphinx-argparse==0.2.1
CommonMark==0.7.5
(see https://github.com/ZELLMECHANIK-DRESDEN/dclab/blob/master/docs/requirements.txt)
For other projects, this did not work (see #4855).

I read the whole thread and it's not clear to me what's the problem on Read the Docs. I mean, there are projects that are failing on their builds but it seems it's because of a problem in the external dependency sphinx-argparse.
What would be the action that we can take from Read the Docs, if any?
I was able to build properly in my dev instance without modifycation:

mpy-dev-tools
fyrd

limmbo failed because a conda package resolution problem:
ResolvePackageNotFound:
  - libiconv==1.15=0
  - libxml2==2.9.8=0

and rucio failed with an issue that doesn't seems to be related to RTD:
Could not import extension rucio.common.doc.argparse.ext (exception: No module named rucio.common.doc.argparse.ext)

Please, let me know if there is any action that we can take from our side. Otherwise, I will close this issue.

I have not tested this on Python 2.7, but for Python 3 I would consider this problem resolved.
In general, the problem with sphinx-argparse in combination with RTD seems to be that detailed error messages are missing (see e.g. https://github.com/rtfd/readthedocs.org/issues/4855#issuecomment-438197912). Not sure who would be responsible for that.

the problem with sphinx-argparse in combination with RTD seems to be that detailed error messages are missing

Not sure to follow here. When you build locally in your computer do you have a better detailed message that explain the reasons (*)?
(*) those that you finally found with your custom extension, I think.

Yes, but I currently don't have a minimal example to support my claims. Give me some time and I will open a new issue.

I can reproduce the missing error messages locally, so this is not related to RTD.
https://github.com/ribozz/sphinx-argparse/issues/102

@paulmueller thanks for sharing your research here!
Now, I think that we can close this issue on RTD and people interested in this problem can subscribe to the issue that you opened in the sphinx extension.
Thank you all for debugging and helping us to make Read the Docs better ;)


How to set Accounts.ui.config ?

Hi,
I use your package for several days, it is excellent!
However I have a problem with the configuration.
When I set requested the permissions of the connection, I realize that they are not considered.
Do I have misconfigured service?
Accounts.ui.config({
	requestPermissions : {
		facebook : ['user_friends', 'user_likes', 'user_posts'],
		google : ['https://www.googleapis.com/auth/userinfo.email', 'https://www.googleapis.com/auth/userinfo.profile', 'https://www.googleapis.com/auth/youtube','https://www.googleapis.com/auth/youtube.readonly']
	},

	"requestOfflineToken" : {"google": true},
	"forceApprovalPrompt" : {"google": true}
});

I found it. I should use the first arg of signInWith
// ON THE CLIENT:
Meteor.signInWithGoogle ( {
   // Add persmissions here
}, function (error, mergedUserId) {

  // mergedUsers is set if a merge occured
  if (mergedUserId) {
    console.log(mergedUserId, 'merged with', Meteor.userId());
  }
});

:+1:


Inaccurate information with recent changes

Inaccurate information with recent changes

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 8751595d-4839-d2ee-1e2c-16d07ef84a2d
Version Independent ID: e66f96b3-5b88-9304-1ec0-9e3bb8a2cfbe
Content: How to manage app passwords - Azure Active Directory
Content Source: articles/active-directory/user-help/multi-factor-authentication-end-user-app-passwords.md
Service: active-directory
Sub-service: user-help
GitHub Login: @curtand
Microsoft Alias: curtand

@djlause which information is inaccurate?

@curtand could this be updated with the latest screenshots?

Hey @djlause
Thanks for providing feedback that helps improve our documentation. Unfortunately, at this time we have been unable to review your issue in a timely manner and we sincerely apologize for the delayed response.  We are currently updating all the end-user content and will be migrating to the Microsoft Support site. We are closing the issue as we are migrating the content to a new site. #please-close


Explore separating IO into its own crate

The core crate currently contains data structures and IO, which are only loosely related. I've had a lot of churn in how I think about our IO after v0.9.0, especially with the advent of stac-geoparquet, and so we may want to split the two APIs to give us the ability to flex the IO API more while keeping the core stable.

I'm going with "no" for two reasons:

stac_io::read feels awkward to me — just doesn't vibe with a new crate
The Href trait is pretty tightly tied to IO and it needs to live in stac b/c it's an attribute on the structs


Question regarding to production mode optimisation.

I have some questions regarding to have babel-plugin-typecheck running in production mode. Is there any way that I can cut off the overheads of type checking in production mode, which means the type check will be enable in development only. My current config for .babelrc is:
{
  "stage": 0,
  "plugins": ["typecheck"],
  "optional": ["runtime"]
}

And my production config I include  new webpack.optimize.UglifyJsPlugin(). However, I got the production code compiled to the following:
      if ("number" != typeof t) throw new TypeError("Value of argument 'a' violates contract, expected number got " + (null === t ? "null" : t instanceof Object && t.constructor ? t.constructor.name : typeof t));
      if ("number" != typeof e) throw new TypeError("Value of argument 'b' violates contract, expected number got " + (null === e ? "null" : e instanceof Object && e.constructor ? e.constructor.name : typeof e));

Any idea on this? Also thanks for this wonderful plugin.

If you remove typecheck from your .babelrc and compile your code, everything will work as normal without the typechecks, but I agree this isn't very convenient. It comes down to the inability to pass options to babel plugins which should be changing soon

Another point though is that is the overhead real or imagined? The vast, overwhelming majority or apps can get away with leaving this on and it will make no discernible difference to performance at all.

Thanks @phpnode, maybe I overthink about it.

If you remove typecheck from your .babelrc and compile your code, everything will work as normal without the typechecks

What is removing the type annotations in this case?

Another point though is that is the overhead real or imagined? The vast, overwhelming majority or apps can get away with leaving this on and it will make no discernible difference to performance at all.

When you have several 1000 modules with type annotations, it will make a difference to bundle size. And it's questionable to be throwing those errors in production.

Oh, TIL Babel strips Flow types by default :)

it's questionable to be throwing those errors in production.

Well, at least with this enabled then you'll gen an early error, so typically you can find the source of the bug faster. With typechecks disabled you'll either get an error further down the stack  or unwanted behaviour and it'll be much harder to track down.


pre-commit: PR109472

Link: https://github.com/llvm/llvm-project/pull/109472
Requested by: @goldsteinn

/reopen

/reopen

/close

/remove-label reviewed

/add-label reviewed,regression

/remove-label regression


Create a UI Page to Display the Point System

We need a new page on the site that displays the point system for Hacktoberfest contributions. The point system is used to allocate points to different types of contributions (e.g., documentation, bug fixes, etc.). The page should be built using React, and it will display the following point system:

Documentation: 1 point
Bug fix: 2 points
Level 1 contribution: 3 points
Level 2 contribution: 5 points
Level 3 contribution: 8 points

Requirements:

Create a new page that shows the point system using a table or other user-friendly layout.
The page should be styled with tailwindcss, following the existing site's theme.
Ensure the page is responsive and accessible.

Acceptance Criteria:

The page successfully displays the point system in a readable format.
The page should be integrated into the existing site's navigation.

Additional Notes:

Discuss any design improvements if needed, but keep the UI clean and easy to understand.
Ensure that the component or page can be reused or updated easily if the point system changes.

can you assign me this task @jalaym825

can you assign me this task @jalaym825

It's already closed and live on website. You can work on points page.


cli: execute: auth: rcon: dial tcp <IP_ADDRESS>:25575: connect: connection refused

Describe the bug
After starting the server, it showed the below message:
cli: execute: auth: rcon: dial tcp <IP_ADDRESS>:25575: connect: connection refused
To Reproduce
Using docker composer to deploy
Expected behavior
rcon connect successfully
Actual behavior
Error message showed
OS information

OS:  Synology DSM
Palworld Version:  Latest
Docker Image Version:  2.6.1

Hardware information

Memory:  20g

docker-compose.yml contents
services:
   palworld:
      image: thijsvanloef/palworld-server-docker:latest
      restart: always
      container_name: Palworld-Server
      stop_grace_period: 30s
      ports:
        - 8211:8211/udp
        - 27015:27015/udp
        - 25575:25575/tcp
      environment:
        - PUID=xxx
        - PGID=xxx
        - PORT=8211
        - PLAYERS=2
        - SERVER_PASSWORD=xxx
        - MULTITHREADING=true
        - RCON_ENABLED=true
        - RCON_PORT=25575
        - TZ=xxxx
        - ADMIN_PASSWORD=xxxx
        - COMMUNITY=false
        - SERVER_NAME=xxx
        - SERVER_DESCRIPTION=xxx
        - AUTO_REBOOT_ENABLED= true
        - AUTO_REBOOT_CRON_EXPRESSION= 55 11 * * *
        - AUTO_REBOOT_WARN_MINUTES= 5
      volumes:
         - ../palworld:/palworld:rw
      network_mode: bridge

Container/Host Logs
[S_API FAIL] Tried to access Steam interface SteamUser021 before SteamAPI_Init succeeded.
[S_API FAIL] Tried to access Steam interface SteamFriends017 before SteamAPI_Init succeeded.
[S_API FAIL] Tried to access Steam interface STEAMAPPS_INTERFACE_VERSION008 before SteamAPI_Init succeeded.
[S_API FAIL] Tried to access Steam interface SteamNetworkingUtils004 before SteamAPI_Init succeeded.
cli: execute: auth: rcon: dial tcp <IP_ADDRESS>:25575: connect: connection refused

Player logging is attempting running rcon before the server finishes booting and rcon is started yet.
Not a big deal but should still be fixed.

Player logging is attempting running rcon before the server finishes booting and rcon is started yet.
Not a big deal but should still be fixed.

There is no user currently trying to log in. It showed up just after starting the instance.
Is there anything can do to confirm the rcon is working properly?

You can run rcon to verify it's running correctly docker exec palworld-server rcon-cli info

To clarify player logging runs on a timer. While it would be great to trigger based on a new player joining/leaving I don't know of a way for us to implement that.

Thanks @Dashboy1998, just wondering the benefit of running rcon? Is it related to server management? Considering only two local users probably not worth to activate the rcon?

Rcon is the only way to manage the server unless you do only in game commands.
Rcon is the only way for the container to trigger a save or properly shutdown.
Without rcon there's no way for the container to save so your backups might not actually be useful as player data and world data save out of sync.

Thanks for the quick fixed :)

This issue is still happening in my instance. I can't even connect to rcon via docker exec palworld-server rcon-cli info and Waiting for RCON port to open to show player logging... gets spammed repeatedly in the logs.
Here's my docker-compose:
services:
  palworld:
    image: thijsvanloef/palworld-server-docker:latest
    restart: always
    ports:
      - 8211:8211/udp
      - 27015:27015/udp
    environment:
      PUID: 1000
      PGID: 1000
      PORT: 8211
      SERVER_PASSWORD: "<redacted>"
      MULTITHREADING: true
      RCON_ENABLED: true
      RCON_PORT: 25575
      ADMIN_PASSWORD: "<redacted>"
      COMMUNITY: false # Enable this if you want your server to show up in the community servers tab, USE WITH SERVER_PASSWORD!
      SERVER_NAME: "<redacted>"
      SERVER_DESCRIPTION: "<redacted>"
      # Custom game settings
      PLAYERS: 32
      GUILD_PLAYER_MAX_NUM: 32
      COOP_PLAYER_MAX_NUM: 32
      PAL_EGG_DEFAULT_HATCHING_TIME: 24
    volumes:
      - ./palworld:/palworld/


[4.8] fix freeze, add SkipFreezeOnSet

This is a cherry-pick of the following upstream PRs to openshift-4.8 branch:

https://github.com/opencontainers/runc/pull/2946
https://github.com/opencontainers/runc/pull/2958
https://github.com/opencontainers/runc/pull/3019
https://github.com/opencontainers/runc/pull/3085
https://github.com/opencontainers/runc/pull/3093 (modulo some tests)
https://github.com/opencontainers/runc/pull/3092
https://github.com/opencontainers/runc/pull/2994
https://github.com/opencontainers/runc/pull/3167

(as requested by @ingvagabund)

Looks good based on https://github.com/openshift/origin/pull/26696. The success rate of failing sig-storage tests went down based on the latest CI results.


allowTextSelectionOnPress isn't working for usePress

Provide a general summary of the issue here
When I use allowTextSelectionOnPress inside onPress, it does not allow selection.
🤔 Expected Behavior?
Text selection should work.
😯 Current Behavior
Text selection does not work
💁 Possible Solution
No response
🔦 Context
No response
🖥 Steps to Reproduce
https://codesandbox.io/p/sandbox/busy-ioana-c3rf9t?file=%2Fsrc%2FApp.js%3A2%2C15
Version
3.34.1
What browsers are you seeing the problem on?
Chrome
If other, please specify.
No response
What operating system are you using?
MacOS
🧢 Your Company/Team
No response
🕷 Tracking Issue
No response

@devongovett do you recall what this prop was intended for more specifically? it looks like it was only used in date segments, which makes me think the use case was possibly more specific than the docs would lead me to believe.


Update logo

@kuzminadya Thanks for the awesome logo 💃

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@223ac03). Click here to learn what that means.

@@           Coverage Diff           @@
##             master   #120   +/-   ##
=======================================
  Coverage          ?    92%
=======================================
  Files             ?     13
  Lines             ?    488
  Branches          ?     84
=======================================
  Hits              ?    449
  Misses            ?     39
  Partials          ?      0

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 223ac03...1d568cc. Read the comment docs.

@vigneshshanmugam FYI https://github.com/reactjs/redux/tree/master/logo was used as a reference

@w0rm 👍


Function based views

Hello, is there a way to use this lib with function based views?

Hi @Leelith. Have you been able to check README out about this lib?
https://github.com/marcelo225/django-keycloak-auth#how-to-use

Hi @Leelith. Have you been able to check README out about this lib? https://github.com/marcelo225/django-keycloak-auth#how-to-use

Hello, yes! As far as I understand, this describes the use with class-based views. I want to apply this to a function with @api_view decorator for example. I already wrote additional code for this, which adds @keycloak_roles decorator, but maybe I just didn’t understand how to work with the existing functionality.
My addition work like this
@api_view
@keycloak_roles({'GET': ['some_role',...]})
def some_api_function(request):
    ...

@Leelith did you find any solution to use this lib with function based views ?

@Leelith did you find any solution to use this lib with function based views? The docs doesn't have any solution for this.

Hello @SamuelBen-22, for now I'm using the decorator I wrote
https://github.com/Leelith/django-keycloak-auth/tree/decorator
Usage:
from django_keycloak_auth import keycloak_roles
...
@keycloak_roles({'GET': ['judge'], ...})
@api_view(('GET', ...))
def my_api_function(request):
    ...

@Leelith , could you create a PR about this? I'll merge it if you propose this PR, right?
Thanks!

@Leelith PR was merged into master.
The new version is 0.97 now
Thanks for helping all of us!

@Leelith PR was merged into master.
The new version is 0.97 now
Thanks for helping all of us!


Add missing recommended key lengths/digest to Cert system

Add missing key lengths to the certificate management pages, for keys of size 3072 (for current use), 7680, 15360 (for long term resistance), 8192 and 16384 (common binary exponents), and the missing digest "whirlpool".
The key lengths are supported by OpenSSL and for certain uses are currently recommended (eg long term resistance to replay/decryption). See keylength.com for citations.
The "whirlpool" digest is the only non-SHA1/2/3 family digest to be recommended and is especially valuable for users wishing to avoid the SHA families (for whatever reason).
Files changed - user manager, cert manager, ca manager, certs.inc

Merged to master only, for now at least. If it checks out in more widespread usage, might cherry-pick back for 2.3.2.


GUA-231: Wire up Personal Tax Account

Wires up the personal tax account journey.
Not entirely clear on the order from the Figma .
But have assumed it's
Start -> Choose -> Home?

Can you add a button that takes you to the start of the journey to the home page like we have for DBS? See https://github.com/alphagov/di-solid-prototype/blob/main/src/views/index.njk#L11-L14
Also, which of these pages takes a user to login/ prove their identity? Before we do that we'll need to set some stuff in their session so the IPV journey can send them back to the right place - https://github.com/alphagov/di-solid-prototype/commit/8602fa796b48a0df906b291576f996c1c7ad5119#diff-01a773a52b374d5a72322dd915e3c93f3af30a99ed38011c54642287877d0cb5

Cool have added the button,
It's unclear to me if this PTA journey just needs to be there for demonstration purposes, or if it also needs to repeat the DBS read journey 🤔
I'll start a slack thread about that and do a separate PR for that work.


Bring back collision check somewhere

Node collision has been removed for now, but I'm not sure it is necessary to being it back.
Maybe it's good to have a check just once somewhere in the GA, but not during mutation because that's probably quite heavy - O(n^2).

Benchmarks are passing so this doesn't seem necessary.


azure-eventhub python library bundle installation

Customer is trying to install azure-eventhub library in their onprem VM to test eventhub. But their network setup does not allow downloading libraries from internet. They can however download the whl file in their personal machine and FTP to their work machine. But even that does not work, because azure-eventhub has dependency on azure-core that requires download from internet.
Is it possible to get a whl file to install azure-eventhub python library with all its dependencies bundled in one file?

Hi @arunansu , Ill ask a few people within my team and see what options are available to help you out and get back to you asap.

@arunansu the best way to create an installer would be to use the pip download option. You would have to make sure that the machine downloading the dependencies has the same OS and python version as the VM where it will be installed (  if not, then at least know the OS and Python version)
Here are the steps to go about it:-

Create a folder for example eventhub-deps
Then run the following command to download the dependencies in the folder created above: pip download azure-eventhub -d "eventhub-deps" --python-version "3.7" --platform "Windows". In this step python version and platform should be the same as the VM where it will be installed.
Copy the whl and the dependencies folder on to the VM and then run pip install azure-eventhub.whl -f ./ --no-index --no-deps inside the eventhub-deps folder

This should now install eventhub sdk on to the VM and resolve all dependencies locally :)
Please let me know if you have any further questions.


SystemCommand: Annotate system_command and its derivatives.

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

References https://github.com/Homebrew/brew/pull/14629

Thanks @Kentzo! Could you explain a little bit about how you ran into this issue?

The lack of annotations for system_command caused one of the tests to fail in my sudo PR because sorbet saw a type cast from Array to untyped.Adding an annotation snowballed quickly into a larger diff.Best RegardsIlya KulakovOn Apr 29, 2023, at 3:16 AM, Mike McQuaid @.***> wrote:
Thanks @Kentzo! Could you explain a little bit about how you ran into this issue?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

I have opened https://github.com/sorbet/sorbet/issues/6984, since having the shorthand
out, err, status = system_command! "true"

is much more convenient to use and it mirrors Open3.capture3.


Remove wrong debug print

Remove error message from debug print for "updatable objects "

updatable_objects_repository_reply doesn't have the 'error_message' property, and the script throws an exception during the import.


Problem: The parser must be rebuilt in nodeJS each time it is changed, causing developer slowdown

Solution: Automate rebuilding the parser from the parser schema file
Testcase: Isolate rebuilder in test, make sure that it outputs is a valid parser

Issue looks like it's good to go. Going to close it.


bokeh_patch returning hover tool

I saw that the ShaftElement.bokeh_patch() method returns a bokeh hover.
Is there a reason for us to return the hover instead of returning bk_ax?
@rodrigomoliveira1

Actually, yes.
It was necessary to prevent the code from creating a "hover" icon for each shaft element in the model.
Returning a bokeh hover allows me to collect all the information in a single "hover" and then add it to the layout. Otherwise, the bokeh.figure would display lots of hover icons on the toolbar.
You'll notice it in DiskElement.bokeh_patch() too.

Hi there!
I have marked this issue as stale because it has not had activity for 45 days.
Consider the following options:

If the issue refers to a large task, break it in smaller issues that can be solved in
less than 45 days;
Label the issue as wontfix or wontfix for now and close it.


Remove version: P-StSoftware.WinSSHTerm version 2.28.0

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://dl.winsshterm.net/WinSSHTerm-2.28.0-x64.msi
Response Error: The remote server returned an error: (404) Not Found.
(Automated message - build 911)

https://github.com/microsoft/winget-pkgs/issues/142854


Assisted Wallet UI and Flow

Implement Assisted Wallet UI and Flow.
https://www.figma.com/design/m5eMFTKD391AiJAXaEzgxB/Bitcoin-Keeper?node-id=711-8439&t=fFmMYfh6IEVsJGl7-0

Dev task


Changes to README.md; removed top level test directory (which I think is obsolete?)

I'd also like to include a contributor thank-you at the bottom of the readme. Let me know if you have a preference of URL for me to link to :)

Sorry, I'll get onto this today.
On Fri, 16 Dec 2022 at 02:09, Jahfer Husain @.***>
wrote:

If you don't have time to update this branch, you can provide me with
commit access by following these instructions:
https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/working-with-forks/allowing-changes-to-a-pull-request-branch-created-from-a-fork
—
Reply to this email directly, view it on GitHub
https://github.com/jahfer/clj-activitypub/pull/2#issuecomment-1354079396,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABJ66W2HYWTLQXAKPA7CSWLWNPFMPANCNFSM6AAAAAAS5TYITI
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Simon Brooke :: http://www.journeyman.cc/~simon/
    ;; Stultus in monte


fix(exporter): ignore duplicate LVs when collecting metrics

Pull Request template
Why is this PR required? What issue does it fix?:
When LVs have more than a single segment(i.e. spread over several), collecting the metrics does no longer work, as lvs returns several entries for the same LV
What this PR does?:
As I did not want to change the behaviour of ListLVMLogicalVolume (even though its currently only used in metrics collection. I opted for skipping any further occurrences of the LV (per UUID) during metric collection.
Does this PR require any upgrade changes?: NO
If the changes in this PR are manually verified, list down the scenarios covered::
Assuming a VG with 100GB

Create 2 PVCs with 10GB size
Delete the PVC that sits before the second 10GB on the VG
Create a PVC with 85GB in size, so the LV will end up having two segments
Collect metrics and see no scrape errors

Checklist:

[x] Fixes #211
[x] PR Title follows the convention of  <type>(<scope>): <subject>
[ ] Has the change log section been updated?
[ ] Commit has unit tests
[ ] Commit has integration tests
[ ] (Optional) Are upgrade changes included in this PR? If not, mention the issue/PR to track:
[ ] (Optional) If documentation changes are required, which issue on https://github.com/openebs/openebs-docs is used to track them:

@kmova thanks for running the ci tests. I am a bit confused by the failures - it seems to me that some test framework dependencies need to run on a newer golang version ( if I read this correctly https://github.com/openebs/lvm-localpv/actions/runs/3488761623/jobs/5842170432#step:6:74 )
Maybe you have any other hints on where to look

Hi @patrickjahns What do you mean by LV will end up having two segments?

@Ab-hishek
Please see the outputs of lvdisplay and lbs in the issue https://github.com/openebs/lvm-localpv/issues/211
LVs can have several segments, when the size does not fit into a consecutive "block" of a volgroup. lvdisplay just shows that there the volume is segmented, the output of lvs shows a entry for each segment ( the start extent/block )

@patrickjahns - probably a good idea to bump up the golang version in a different PR and then merge this. Something similar to this: https://github.com/openebs/jiva-operator/pull/201/files
Would be able to take up the go bump PR?

@kmova
I'll take a look on the weekend

@kmova would you be able to take a look at this and the corresponding go version bump in https://github.com/openebs/lvm-localpv/pull/214

Can somebody please take a look at this PR? I verified that this patch indeed fixes the problem on our cluster(s) and would like to have it in the next release.
The PR code is a bit older, it needed a very small tweak, but otherwise still applies clean.

I've rebased the changes to the latest develop - it would be really nice if any maintainer takes a look now. But my hopes are little after that long period of time and not receiving feedback in slack either 🤷

I've rebased the changes to the latest develop - it would be really nice if any maintainer takes a look now. But my hopes are little after that long period of time and not receiving feedback in slack either 🤷

Thank you for your quick response, let's hope the OpenEBS maintainers can make some time free to look at this.

@niladrih @hrudaya21 @dsharma-dc PTAL


PHP Inbuilt Server doesn't hit breakpoints

When using the inbuilt server option from PHP Storm, I can't hit breakpoints in my code.

This works if I switch PHPStorm to use the native Windows PHP version, but then I have all the weirdness of running PHP in Windows, which is why I found your great project.

A lot has changed since opening this issue - closing now to tidy up.


feat(CategoryTheory): allow diagram chases by arguing up to refinements

This PR introduces the most basic results which enable doing diagram chases in general abelian categories by arguing "up to refinements".

See also https://leanprover.zulipchat.com/#narrow/stream/335062-homology/topic/Diagram.20chases.20in.20general.20abelian.20categories

Note: I have pushed an update to the lean toolchain because this PR was on a buggy version of the toolchain. WARNING: checking out old commits of this PR using v4.2.0-rc2 or v4.2.0-rc3 can cause lake clean to delete your mathlib folder! If you need to do so, make sure to delete lakefile.olean manually before running any lake commands.


ng-mock version improvement, it is better to follow angular versions

Describe the feature or problem you'd like to solve
ng-mock versioning is confusing, it is better to follow angular versions.
So that we can choose which version to install based on the ng-mock version

Proposed solution
It is better to follow angular versions

Additional context

Hi, you should use the latest one for all angular version.


feat: allow load on adapter factories to return a Promise

I think we should allow load to return a Promise instead of just data.
The use case is doing asynchronous tasks before instantiating my adapter ie. reading creds from a file, fetching creds from a server, creating tmp files the adapter may use. So load would look something like:
export default () => ({
  load: async () => {
    const creds = await (await fetch('https://foo.bar')).json()
    return creds
  },
  link: creds => () => () => {
    return createAdapter(creds)
  }
})

I could see this, or allowing link to return a Promise of an adapter impl. I think either would work.

Sounds good, maybe let's get hyper as is running on Deno first and deployed to a dev env, then make adjustments? Otherwise, we could end up with a mess.


Add a statistics tab to the dictionary

The dictionary should contain a tab at the very left (not selected by default; the noun tab should remain the default tab) whose label is a glyphicon instead of a word (to keep its width small). The tab should display statistics about the dictionary, such as entry counts by part of speech, and a bar showing a graphical representation of what proportion of the dictionary storage each part of speech takes up.

Implemented in 415aab9. 😜


LF-12310: Exponential stream backoff on errors and cycle detection.

@ross-pfahler  should check for sanity...


feat(lib): methods for measuring time and duration

Committed without pre-commit hook because tests are still missing.
Types of changes

[ ] Enhancement (project structure, spelling, grammar, formatting)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

A description of the changes proposed in the Pull Request
I am opening this PR because I want to figure out if I am heading in the right direction.
So far, the plugins I have seen in if are basically functions or operators that map parameters to other parameters. They are performing a calculation. But they are not measuring anything.
But if you actually want to measure, you are also interested in the timestamp and the duration of the measurement. And you don't want to write that by hand into your manifest files. Thus, I wrote this timer methods that can manipulate timestamp and duration.
Does that conform with how you imagine if should be used? If so, do you think this plugin should find its way into the unofficial plugins? I would have to add tests then.

I addressed your comments, but still need to add some tests.

Hi @manushak, a short update on the changes I pushed:
a) I added tests.
b) I changed the approach: TimerStart is now just writting a timer/start value to each input but does not change timestamp or duration. This logic moved to TimerStop. That has the advantage that TimerStop can check if TimerStart was run before by looking for the timer/start field. Of course, that also changes the input and return values of both methods. TimerStop consumes the resets now.
c) The example manifest from the README now uses a simple plugin that can execute shell commands (instead of the measurement plugin). I opened a PR for that plugin here #59.

Hi @alexzurbonsen i'm struggling to understand the intended use case for this - if you want to measure the time it takes to execute a pipeline, why not do this externally (i.e. time IF execution)? If you have a need for this, then you are of course encouraged to maintain the plugins in your own repository, but I'm not sure it's something we want to maintain in the IF repo.

Hi @jmcook1186,
The idea is, that once you actually measure something with a plugin, and not just put measured values as inputs into the pipeline, then at least I would want to automatically write timestamp and measurement duration in the inputs. In this [draft PR](the need for this can be seen in this draft PR: https://github.com/Green-Software-Foundation/if-unofficial-plugins/pull/63) the measure-webpage plugin is loading a webpage and measuring its page weight for example. I would want to know at which time that measurement was done.
But as  I also wrote in the summary of the PR, I am not sure you had imagined IF to be used in this way, because plugins I have seen are pretty static so far and not measuring things.

Similar to the other PR - this seems like a potentially useful idea, but it's not really something we need in our standard library - recommend maintaining this in an external repo. We'll be creating a public registry for external plugins soon.


[SPARK-31385][SQL] Restrict micros rebasing via switch arrays up to 2037 year

What changes were proposed in this pull request?

Generate rebasing arrays for micros up to 2037 in RebaseDateTimeSuite.generateRebaseJson().
Exclude 4 time zones from the black list in generateRebaseJson().
Re-generate JSON files with rebasing info - gregorian-julian-rebase-micros.json and julian-gregorian-rebase-micros.json.

Why are the changes needed?

sun.util.calendar.ZoneInfo resolves DST after 2037 year incorrectly. See https://github.com/AdoptOpenJDK/openjdk-jdk8u/blob/aa318070b27849f1fe00d14684b2a40f7b29bf79/jdk/src/share/classes/sun/util/calendar/ZoneInfo.java#L55-L62 . By restricting the rebase arrays to 2037 year, we follow the behaviour of ZoneInfo which uses DST of 2037 for all years beyond 2037.
To enable optimization of micros rebasing via switch arrays for the time zones:

Asia/Tehran
Iran
Africa/Casablanca
Africa/El_Aaiun

Does this PR introduce any user-facing change?
No
How was this patch tested?
By existing test suites RebaseDateTimeUtils, DateTimeUtilsSuite and DateFunctionsSuite.

@cloud-fan @HyukjinKwon Please, review the PR.

Test build #121451 has started for PR 28253 at commit 052a70a.

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-K8s/26135/
Test PASSed.

Test build #121451 has finished for PR 28253 at commit 052a70a.

This patch fails PySpark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

jenkins, retest this, please

Test build #121454 has started for PR 28253 at commit 052a70a.

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-K8s/26138/
Test PASSed.

Test build #121454 has finished for PR 28253 at commit 052a70a.

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/121454/
Test PASSed.

LGTM, merging to master/3.0!


Client refactor with new api names for JSON format

Pull Request
What does this PR do?
This PR partially fixes #182 for JSON format
[BREAKING CHANGES]
AddDocumentsAsync is now AddDocumentsJsonAsync
AddDocumentsInBatchesAsync is now AddDocumentsJsonInBatchesAsync
UpdateDocumentsAsync is now UpdateDocumentsJsonAsync
UpdateDocumentsInBatchesAsync is now UpdateDocumentsJsonInBatchesAsync
The README and code samples have been updated to reflect the changes.
The test infrastructure has been updated to easily leverage it when we will add NDJSON and CSV.
I introduce a JsonHttpContent, this allow to remove some custom code from HttpExtensions (more will be removed later) and easily follow what Adding and Updating functions are doing.
Let me know if things are not clear, this PR is a bit big.

Hi @alallema,
No problem for the delay, just take your time and do not hesitate to ask me questions if something is not clear.
I thought that AddDocuments method would disappear in favor of Json, Ndjson and Csv ones.
What AddDocuments is supposed to do if we have support for the three formats ?

Ok, I went a bit fast, I've just checked the signature of the methods.
This is PR is not in line with the issue :)
I will make new ones.

What AddDocuments is supposed to do if we have support for the three formats?

The AddDocuments method is supposed to send an Object, 'Classor anArrayaccording to the SDK, while theAddDocumentsJson AddDocumentsCsvandAddDocumentsNdJsonmethods are supposed to retrieve this document in string format and send it as is. To note: Before v0.23, all SDKs sentContent-Type: application/jsonon every request. After I added the possibility to sendNdJsonorCsvwe left the sending ofContent-Yypeonly forPOSTandPUTrequests with the appropriateContent-Type. The AddDocumentsmethod keep Content-Type: application/json` header.
Hope it's more clear, sorry maybe the issue isn't not clear enough.

This is fine, I didn't check the methods signatures :)


Don't encode image resource URI twice

After  #84667 and 8caf39ef37932f5b9492a88a7f921625ef120080, issue #84197 and #83165 are happening again.
This fixes both issues by encoding the resource URI just once so that it gets parsed correctly here
This PR fixes #84197 and #83165
cc @mjbvz

Thanks!


Can't find variable: expect

Added this to Gemfile:
group :test, :development do
  gem 'teaspoon-mocha'
end

Then ran bundle followed by the installation generator:
✗ rails generate teaspoon:install --coffee
      create  spec/teaspoon_env.rb
      create  spec/javascripts/support
      create  spec/javascripts/fixtures
      create  spec/javascripts/spec_helper.coffee
+============================================================================+
Congratulations!  Teaspoon was successfully installed.  Documentation and more
can be found at: https://github.com/modeset/teaspoon

I then created spec/javascripts/test_spec.js.coffee with the following content:
describe "My great feature", ->

  it "will change the world", ->
    expect(true).to.be(true)

  it "will destroy the world", ->
    expect(true).to.be(false)

Then tried to run the tests:
✗ rake teaspoon
Starting the Teaspoon server...
Teaspoon running default suite at http://<IP_ADDRESS>:65447/teaspoon/default
FF

Failures:

  1) My great feature will change the world
     Failure/Error: Can't find variable: expect

  2) My great feature will destroy the world
     Failure/Error: Can't find variable: expect

Finished in 0.01000 seconds
2 examples, 2 failures

Failed examples:

teaspoon -s default --filter="My great feature will change the world"
teaspoon -s default --filter="My great feature will destroy the world"
rake teaspoon failed

Nothing else has changed. Running OS X 10.11.1.

Mocha doesn't ship with an assertion syntax. You'll need to add Chai and enable expect.
In your spec_helper.coffee:
#= require support/chai
window.expect = chai.expect