vagrant: 2.0.0 -> 2.0.1 (fixes vagrant with virtualbox 5.2)

Motivation for this change
With the recent virtualbox bump to 5.2 vagrant is no longer able to interface with virtualbox. Version 2.01 fixes that.
While doing that I figured that the custom rake gem doesn't seem to have any purpose anymore. Our standard pkgs.rake works just as well so that is the 2nd commit in this PR.
Things done

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

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Fits CONTRIBUTING.md.

@grahamc is there a way to get the output of @GrahamcOfBorg? The command that I can see in the tooltip of "grahamcofborg-eval-package-list" works on my machines (TM).

Hey @andir, a couple things:

if you rebase this branch to master, it'll definitely fail. the piping of stderr to /dev/null isn't going to help people figure this out, so I'll remove that.
The failure is because master fails to evaluate packages: https://github.com/NixOS/nixpkgs/pull/31626 and yours is based on master. Orivej is sending a patch for this right now. When that happens, we'll rerun the checks for this branch.

Thanks for the input. I used the latest unstable channel as base for my PR - as I do always.
So the checks succeeded now. Without anyone touching master and this PR.. So your theory sounds a bit strange to me. Maybe I am missing something.

@GrahamcOfBorg vagrant

LGTM should we backport this?

It probably doesn't hurt since it is basically just bug fixing and adding support for ruby 2.4 (but previous versions are still support (we've 2.3 as "stable" currently)).
The relevant changelog is
## 2.0.1 (November 2, 2017)

FEATURES:

  - core: Introduce Ruby 2.4 to Vagrant [GH-9102]
  - providers/virtualbox: Virtualbox 5.2 support [GH-8955]

IMPROVEMENTS:

  - command/destroy: Introduce parallel destroy for certain providers [GH-9127]
  - communicators/winrm: Include APIPA check within ready check [GH-8997]
  - core: Clear POSIXLY_CORRECT when using optparse [GH-8685]
  - docs: Add auto_start_action and auto_stop_action to docs. [GH-9029]
  - docs: Fix typo in box format doc [GH-9100]
  - provisioners/chef: Handle chef provisioner reboot request [GH-8874]
  - providers/salt: Support Windows Salt Minions greater than 2016.x.x [GH-8926]
  - provisioners/salt: Add wget to bootstrap_salt options when fetching installer file [GH-9112]
  - provisioners/shell: Use ui.detail for displaying output [GH-8983]
  - util/downloader: Use CURL_CA_BUNDLE environment variable [GH-9135]

BUG FIXES:

  - communicators/ssh: Retry on Errno::EPIPE exceptions [GH-9065]
  - core: Rescue more exceptions when checking if port is open [GH-8517]
  - guests/solaris11: Inherit from Solaris guest and keep solaris11 specific methods [GH-9034]
  - guests/windows: Split out cygwin path helper for msys2/cygwin paths and ensure cygpath exists [GH-8972]
  - guests/windows: Specify expected shell when executing on guest (fixes winssh communicator usage) [GH-9012]
  - guests/windows: Include WinSSH Communicator when using insert_public_key [GH-9105]
  - hosts/windows: Check for vagrant.exe when validating versions within WSL [GH-9107, GH-8962]
  - providers/docker: Isolate windows check within executor to handle running through VM [GH-8921]
  - providers/hyper-v: Properly invoke Auto stop action [GH-9000]
  - provisioners/puppet: Fix winssh communicator support in puppet provisioner [GH-9014]
  - virtualbox/synced_folders: Allow synced folders to contain spaces in the guest path [GH-8995]

(https://github.com/hashicorp/vagrant/blob/v2.0.1/CHANGELOG.md#201-november-2-2017)

This patch was applied to master: https://github.com/NixOS/nixpkgs/commit/d7aefea2ef4248a8138de15baad228ebce21e737 and I triggered the evaluation checks  to run again. That is why it started passing again :)

Ahhh, okay.. Since I didn't know the exact error I didn't think of that fixing it... Totally makes sense now :-)


Cursor in last line is cut off

Expected behavior
if command line have two or more full lines the cursor should be first in next line
Actual behavior
the cursor look like bar cursor but it's base cursor and if you type one character it jump to 2dn position in next line.
Steps to reproduce
Open
https://codepen.io/jcubic/pen/MVbJmE
the cursor need to be at the end

in version 1.14.0


In the statsd metrics, blank spaces must be sanitized (according to etsy statsd format)

Environment

Telemetry Streaming Version:
BIG-IP Version:

Summary
A clear and concise description of what the bug is.
Please also include information about the reproducibility and the severity/impact of the issue.
Steps To Reproduce
Expected Behavior
In our lab, we have some metrics that are coming in with spaces in the metric names and this is not correct. (ex: ARP flood stats)
I expect that there is some sanitization that is performed.
The F5 telemetry plugin must sanitize the metric names and remove spaces if any, like its mentioned here:
https://github.com/statsd/statsd/blob/db4ec7b16b5d98b5b797cdb5892d8e2d25cbbc74/stats.js#L168-L176
function sanitizeKeyName(key) {
  if (keyNameSanitize) {
    return key.replace(/\s+/g, '_')
              .replace(/\//g, '-')
              .replace(/[^a-zA-Z_\-0-9\.]/g, '');
  } else {
    return key;
  }
}

Actual Behavior
A clear and concise description of what actually happens.
Please include any applicable error output.
f5telemetry.server-CVv15d01-ve-service-63kgfh-ad-testbed-dcn.dos-DeviceStats.ARP flood stats.common-aggDetected:0|g|#common.status:Ready,deviceType:Static Vector - Inline

Thank you for your feedback. I have added this issue to our internal product backlog as AUTOTOOL-2938.

This has been resolved in the 1.27.0 release.


Arduino compatibility

Can Arduino uno be replaced with teensy board? is the code compatible?
Thanks

Hi, it runs on Teensy by default. Unfortunately you need at least an Arduino Mega to run this. Please refer to the google groups if you have non issue/bug questions. :) Thanks!

Thanks a lot for the quick response, im really impressed with this project and going to continue building one :)


Doesn't compile (qoi.h no such file or directory)

Describe the bug
Doesn't compile
To Reproduce
Steps to reproduce the bug:

Environment: Fedora 38 KDE
Install dependencies in README.md
Run make -j 8 in terminal

src/parsers/QoiParser.cpp:5:10: fatal error: qoi/qoi.h: No such file or directory
    5 | #include <qoi/qoi.h>
      |          ^~~~~~~~~~~
compilation terminated.
make: *** [Makefile:175: build/src/parsers/QoiParser.cpp.o] Error 1
make: *** Waiting for unfinished jobs....

Expected behavior
Should compile

check if your Dotto's source directory has the following files:

dependencies/qoi/qoi.cpp
dependencies/qoi/qoi.h

if not, this means you didn't pull the submodules, basically Dotto has few third party libraries that are kept at LibreSprite/DottoDependencies.
but there is a shortcut to this repository in the main dotto repository which is called a "submodule".
while cloning the repository you need to pass a flag --recursive which clones all the submodules too but you probably forgot that.
so if you don't want to re-clone everything you can run this command in your existing cloned repo:
git submodule update --init --recursive

Oh ok  sorry I didn't follow the instructions properly


APNG support

Hi,
Do you think you can easily add APNG support ?
https://wiki.mozilla.org/APNG_Specification

Hi there,
Theoretically yes, but it is not on my list of priorities for now. I'm concentrating on common web formats.
Maybe after V1 is released.
Cheers
James


Create Lord Hacker Ultimate

https://github.com/wix-incubator/corvid-stackdriver-telemetry-adapter/assets/147946049/796c46e0-40e1-4fdd-8411-542315bcdf24
We are LORD HACKER ULTIMATE  Digital Assets Recovery Agency
Are you a victim of BITCOINS SCAM?
Are you having a case that needs an Urgent legit Hacker Response, Do you want to get your lost funds/Crypto back from the SCAMMERS Do you want to catch your cheating Spouse or partner, Do you want to upgrade your credit score and clean your criminal records
Hurry up right now and contact us immediately at Lord Hacker Ultimate Digital Assets recovery Agency through Our hotline<EMAIL_ADDRESS>and you can also visit us on our YouTube page: @lordhackerultimate to subscribe, or our website: https://lordhackerultimatee.wixsite.com/hacker,

Beware of Hackers while trading on Bitcoin, I learned this the hard way. As I was trading on Bitcoin on the 28th of August 2023, my account was hacked, and the $718,000 that I was trading with disappeared in the air, I was devastated and in distress, and I immediately started looking for a way to recover my lost Crypto! I went through several ads and I came across an Ad on Lord Hacker Ultimate Digital Assets Recovery, after doing thorough research I noticed several people had gotten help from him and were actually happy after getting back all their lost funds, I personally inbox several of them and they confirmed of the excellent services they received from Lord Hacker Ultimate Digital Assets Recovery, that's when I made contact too and as advertised I am here to testify that he helped me recover all my lost $718,000. He also gave me tips on how to keep my crypto account safe from hackers. If in any case, you have lost funds while trading on crypto, quickly contact them now at: L.H.ULTIMATE@ FASTSERVICE.COM Or website: lordhackerultimatee.wixsite.com/hacker their services are 100% guaranteed and he doesn't waste time recovering what you lost.

Beware of Hackers while trading on Bitcoin, I learned this the hard way. As I was trading on Bitcoin on the 28th of August 2023, my account was hacked, and the $718,000 that I was trading with disappeared in the air, I was devastated and in distress, and I immediately started looking for a way to recover my lost Crypto! I went through several ads and I came across an Ad on Lord Hacker Ultimate Digital Assets Recovery, after doing thorough research I noticed several people had gotten help from him and were actually happy after getting back all their lost funds, I personally inbox several of them and they confirmed of the excellent services they received from Lord Hacker Ultimate Digital Assets Recovery, that's when I made contact too and as advertised I am here to testify that he helped me recover all my lost $718,000. He also gave me tips on how to keep my crypto account safe from hackers. If in any case, you have lost funds while trading on crypto, quickly contact them now at: L.H.ULTIMATE@ FASTSERVICE.COM Or website: lordhackerultimatee.wixsite.com/hacker their services are 100% guaranteed and he doesn't waste time recovering what you lost.

Hello, my name is Savannah....I’m going to share with you how I got my stolen crypto back from the scammers, after falling victim to a cryptocurrency investment scam, I was introduced to this cryptocurrency scam by a guy I met on Instagram, and after investing all my life savings, they swindled me off $4,600,000. It was the saddest moment of my life because I couldn’t believe what had happened, I was bankrupt and my credit was damaged. I reported to the authorities and there was nothing they could do to get these scammers or return my money. I thought all hope was lost and was going into depression until I came across an article online about a hacker who could help me recover my stolen funds, at first I thought it was impossible to do that but I realized I had no other choice but to give it a try, to my surprise Lord Hacker Ultimate Digital Assets Recovery Agency was able to recover all my stolen funds and upgrade my credit score, I wanted to put this out there for anyone who might be going through the same situation and has been swindled by this scammers, can also contact Lord Hacker Ultimate Digital Assets Recovery Agency to help you recover your stolen funds/Crypto. Contact them at: L.H.ULTIMATE@ FASTSERVICE.COM Or website: lordhackerultimatee.wixsite.com/hacker
Uploading Lord Hacker Ultimate Digital Assets recovery.mp4…

https://github.com/wix-incubator/corvid-stackdriver-telemetry-adapter/assets/148729263/68c36667-b64d-40fa-8419-9c4140c027a2

Hello, my name is Savannah....I’m going to share with you how I got my stolen crypto back from the scammers, after falling victim to a cryptocurrency investment scam, I was introduced to this cryptocurrency scam by a guy I met on Instagram, and after investing all my life savings, they swindled me off $4,600,000. It was the saddest moment of my life because I couldn’t believe what had happened, I was bankrupt and my credit was damaged. I reported to the authorities and there was nothing they could do to get these scammers or return my money. I thought all hope was lost and was going into depression until I came across an article online about a hacker who could help me recover my stolen funds, at first I thought it was impossible to do that but I realized I had no other choice but to give it a try, to my surprise Lord Hacker Ultimate Digital Assets Recovery Agency was able to recover all my stolen funds and upgrade my credit score, I wanted to put this out there for anyone who might be going through the same situation and has been swindled by this scammers, can also contact Lord Hacker Ultimate Digital Assets Recovery Agency to help you recover your stolen funds/Crypto. Contact them at: L.H.ULTIMATE@ FASTSERVICE.COM Or website: lordhackerultimatee.wixsite.com/hacker
Uploading Lord Hacker Ultimate Digital Assets recovery.mp4…

Cryptocurrency has become a popular form of digital currency, with many investors putting their trust and money into it. However, this also makes it a target for broke hackers and scammers looking to make a quick buck. I learned this the hard way when my cryptocurrency was stolen by a group of skilled scammers. I lost 820, 649 USDT to those heartless scammers through Crypto investment, Feeling helpless and devastated, I didn't know where to turn until I came across an article online about Lord Hacker Ultimate Digital Assets Recovery Agency and their good work and recommendation on recovering stolen Funds/Crypto from scammers, I did some research about them on my own, which I was satisfied with, so I contacted them via their hotline and report my case, they gave me their procedure to follow which I did, in my greatest surprise. With their expertise in hacking and stolen BTC recovery, they were able to help me recover my stolen crypto back to my wallet in just 4-5 hours, they also helped me increase my credit scores and gave me a sense of relief. quickly contact them if you are a scam victim through their hotline: L.H.ULTIMATE@ FASTSERVICE.COM, website: lordhackerultimatee.wixsite.com/hacker, or Telegram: @lordhackerultimate

Finally, after an unregulated broker hoarded my trading money for more than three months while denying me access to my trading account, I was able to recover every penny I lost—roughly $350,000—that I had lost. I had never thought I would ever see my lost funds again. I'm overjoyed to have received all of my money returned without any hassle. My counterpart, who introduced me to Lord Hacker Ultimate Digital Assets Recovery, a licensed binary options recovery specialist, deserves all the credit. Do you want to withdraw money from your wallet? Does your broker want you to make further deposits before you can withdraw money? Have you ever lost money on a Bitcoin investment made online? Are you unsure of what to do? Use one of the approved methods listed below to get in touch with Lord Hacker Ultimate, Digital Assets Recovery right now for assistance. Information: Email<EMAIL_ADDRESS>WhatsApp No: +19095063423, Website: lordhackerultimatee.wixsite.com/hacker, YouTube page: @lordhackerultimate, Telegram: @lordhackerultimate**

As a Retired combat Marine, I never believed I could easily fall prey to internet fraudsters after many years serving in the National Guard. I want to tell the world the wonders of what the [LORD-HACKER ULTIMATE]Hacking/Recovery agency is doing, it's as important as what US troops are doing on the battlefield. You are waging warfare against these criminals and terrorists on the internet. I thought after investing with a fake Bitcoin investment website, I would lose all my money, but you saved the day by ensuring my funds were recouped from that fake crypto investment platform. Thank you for defending and protecting the most vulnerable of our society against these online predators. Keep up the good work, His contact is Below.
Mail: L.H.ULTIMATE@FASTSERVICE.COM,, Telegram: @lordhackerultimate, YouTube Channel: https://youtube.lordhackerultimate.com & Website: lordhackerultimate.wixsite.com/lordhackerultimate.

I am aware firsthand of the devastation that losing a Bitcoin can cause, particularly if its value has increased. I recently lost my Bitcoin due to a phishing assault. I was inconsolable and believed I would never see my money again. However, I then learned about Bitcoin recovery services. I was initially dubious, but I made the decision to give it a shot. I'm so happy I did. Within a few weeks, the Bitcoin recovery staff at Lord Hacker Ultimate Digital Assets Recovery was able to retrieve my Bitcoin. They were competent, skilled, and professional. I'm extremely appreciative of their support. To speak to a rep of Lord Hacker Ultimate Digital Recovery Assets, Email<EMAIL_ADDRESS> WhatsApp No: +19095063423, website: lordhackerultimatee.wixsite.com/hacker, YouTube page: @lordhackerultimate

✅ Verified in all 🇱🇷 services, Recover Bitcoin, iCloud bypass, Bank loader, game hacks ,Wi-Fi recovery ,Instagram , Facebook ,Snapchat , Twitter ,Gmail, cashapp, PayPal, and other hacking job👌Contact Him 22 on Instagram for quick recovery now https://instagram.com/22cracker?igshid=YWYwM2I1ZDdmOQ==

I want to testify about Lord Hacker Ultimate blank atm cards which can withdraw money from any ATMs around the world. I was very poor before and had no job. I saw so many testimonies about how Lord Hacker Ultimate sent them the atm blank card and used it to collect money in any ATM and become rich. I emailed them also and they sent me the blank atm card at an interesting Price which I paid to them before sending it to them. I have used it to get 900,000 dollars. withdraw a maximum of 5,000 USD daily. Lord Hacker Ultimate is giving out the card just to help the poor at an amazing price. To hack and take money directly from any ATM vault with the use of an ATM-programmed card that runs in automatic mode.
You can also contact them for the service below
•	Western Union/MoneyGram Transfer
•	Bank Transfer
•	PayPal / Skrill Transfer
•	Crypto Mining
•	CashApp Transfer
•	Bitcoin Loans
•	Recover Stolen/Missing Crypto/Funds/Assets
you can contact them for yours at their hotline:
Email: L.H.ULTIMATE@ FASTSERVICE.COM,
Website: lordhackerultimatee.wixsite.com/hacker,
Telegram: @lordhackerultimate,
Signal No: +16266210821,
WhatsApp No: +16266210821

I had already given up on trusting any online cryptocurrency or Bitcoin investment platform after losing $784k to an online BTC investment scammer in March this year. Fortunately, I was able to find Lord Hacker Ultimate Digital Assets Recovery Agency online, a company that is an expert in both cryptocurrency and blockchain networks. It was a horrible time for me, but Lord Hacker Ultimate essentially saved my life by assisting me in Recovering and pulling back all of my transactions to the scammers in a matter of hours. I also received the same amount that I had lost to those scammers. I would recommend anyone in a similar circumstance not to worry; Lord Hacker Ultimate got you covered. You can reach them at Email<EMAIL_ADDRESS>Email: L.H.ULTIMATE@ FASTSERVICE.COM, WhatsApp: +16266210821, Telegram: @Lordhackerultimate, or Signal: +16266210821.


AAD Security Groups

Does a similar process exist for preventing regular users from creating Azure AD security groups?

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

ID: fb2953d4-8abc-5e9c-e30d-43b059b57b9a
Version Independent ID: 467981d4-20b5-8b70-2d42-d3e38b918281
Content: Manage who can create Office 365 Groups
Content Source: Office365-Admin/create-groups/manage-creation-of-groups.md
Service: o365-administration
GitHub Login: @MikePlumleyMSFT
Microsoft Alias: mikeplum

@SteveColegrove  Thank you for your feedback. Only global and user management administrators have permissions to create, edit, or delete Azure AD security groups; for more information about administrator roles, see Assigning Admin roles here(https://docs.microsoft.com/en-us/office365/admin/add-users/assign-admin-roles?view=o365-worldwide)
Hope this helps!
Thanks
Sri


optimize skiplist by reduce 1 malloc

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

313 of 313 (100.0%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 99.715%

Totals

Change from base Build<PHONE_NUMBER>:
0.1%

Covered Lines:
1051

Relevant Lines:
1054

💛  - Coveralls


A question about testing

Hi! The best model I got after training is 73.13/91.16, but the test result is always close to 0. I don’t know why and try many ways, can you point out the reason for this? Thanks very much!
I use this instruction “CUDA_VISIBLE_DEVICES=0 python3 test_models_three_crops.py  kinetics --archs='resnet50' --weights /data1/Masters/TDN/best.pth.tar --test_segments=8 --test_crops=3 --batch_size=32 --full_res --gpus 0 --output_dir ./result  -j 4 --clip_index 0”

Due to my negligence, I have found the problem, thank you for your help！


code hint requires a script setup block: even if an emply block will work, but the block cannot be omitted

Fixed in 0.22.13, thanks for the report!


Overlapping between popup

Hi, I'm trying to test lspsaga with nvim. When 2 function is activated at the same time, popups are overlapped with each other. Can we have a workaround or able to set the priority of each popup?
Config:
nnoremap <silent> K         :Lspsaga hover_doc<CR>
autocmd CursorHold  * :Lspsaga show_line_diagnostics

Demo

fix it later.

Currently pressing K doesnt close the diagnosis


PUBHandler prepends loglevel, module and line to log message

I have successfully implemented a PUBHandler, however, for all loglevels except info the subscriber seems to receive a message bytestring which has prepended the loglevel as a string, the originating module name and the line in the module.
Examples:
INFO - seems fine
2019-01-14 23:38:49,764 INFO     TS_T1: starting cycle...
DEBUG - loglevel, module name and line prepended to the log message
2019-01-14 23:38:49,765 DEBUG    **DEBUG sensor.py:191** - TS_T1: take_reading...
ERROR- same as debug plus appends "- None"
2019-01-14 23:42:48,936 ERROR    **ERROR foundation.py:489** - TS_T1: starting cycle...done. **- None**
CRITICAL - same as debug
2019-01-14 23:38:49,770 CRITICAL **CRITICAL foundation.py:489** - TS_T1: starting cycle...done.
It can be filtered with message.split(b" - ")[-1:][0].decode('utf-8') but that seems awkward. Is there a way to suppress this at the source, or am I perhaps not understanding the way PUBHandler emits logrecords fully ?
Many thanks.

Further to my above post, I either don't understand how to use PUBHandler, or it is not really useful in its current form. I'd suggest to change it so that it receives a logrecord object via recv_pyobj()  which can then be handled by passing it to the existing logger(s):
record = recv_pyobj()
logger.handle(record)

Wrong project?

Issue can be removed, I think.

PUBHandler was written a very long time ago, and should probably be updated quite a bit. I don't think I properly understood what this should do. If you'd like to take a stab at updating it, that would be cool. I wouldn't use pyobj methods (those should generally not be used in production), but I would structure the messages differently to preserve the logrecord format.

Ok, that explains why it doesn't really work nowadays. Just to be clear, all I wanted to use for was to handle logging to a single stream in a multiprocess python program. Since I already use zmq for inter-process communication, it wasn't a big leap to add PUBHandler.  In the end I managed to achieve the same thing by way of a Queue object as described in the python mp documentation.
Apart from all the surrounding 'stuff', there are really only two lines of code in the mp logger class:
                    record = logger_q.get()
                    self.logger.handle(record)  # No level or filter logic applied - just do it!

In my opinion, the PUBHandler should do exactly the same - listen for a logrecord to arrive and then pass it to the local logger(s).
Assuming that's what is desired (?) then I could have a stab at updating it accordingly.

Yeah, that sounds great. The one thing I would do is make sure to serialize the fields as e.g. JSON not the record object as pickle, since pickle raises unnecessary security issues on the receiving end.

Hi, I recently needed similar functionality (passing logrecords over zmq and serializing/deserializing) and ended up writing my own version for a project that I am working on.
It is here if it is of any use to anyone:
https://github.com/klover1/pyzmq/tree/logrecordpubhandler
I have been using it for quite a while so let me know if there are any issues with it.
Thanks!


Productize the Chat widget

Chat Widget Enhancements
a. ability to style the widget
b. allow multi-user chat where each user's chat style can be configured
c. chat text could come from server with embedded EDL. Client should process the EDL and display widgets inline
d. Provide the ability to specify System Message and Prompt right on the client and the ability to change it. These would be in EDL as configurations for the chat widget with js methods to change them on the fly
e. provide onMessageReceived and beforeMessageSent methods that allow users to call any action.
f. Developer should be able to process the message from the server, insert EDL into it and call a method to display it. Client should convert the EDL into inline widgets
g. saveChat method - that saves the chat in client's storage. Can take optional id as an argument

Ability to securely save and use the API keys (in case you are hitting the LLM directly)

Provide Kitchen Sink

Document it

Update Schema

cc @amin-nas @vusters

TODOs:

Schema
Docs


Can I use BPMN with piper?

No. BPMN isn't supported.
On Mon, Apr 1, 2019, 4:41 AM Jim Liu<EMAIL_ADDRESS>wrote:

—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/creactiviti/piper/issues/33, or mute the thread
https://github.com/notifications/unsubscribe-auth/AC6VBnGr8ynObCr5SqatSSAerYGxb3s5ks5vccZOgaJpZM4cVKP6
.

If I want to design a pipeline, I must write yaml as convention of piper,Right?
but if I want to design the pipeline with a visual tool or draw a model and convert it to yaml, how can i do? Because they are not developer and do not family with yaml.

@liu1084 There's currently no graphical workflow editor compliant with the yaml format of piper, so you have to write the workflow by hand for now.

closing due to inactivity.


Adding pause&resume capability to Refresher, improving tests along the way

A review would be appreciated =) Fixes #56.

Merging into devel, only adds new features and shouldn't break anything that already works


HSEARCH-3626 Fix order requirements in various backend integration tests

https://hibernate.atlassian.net//browse/HSEARCH-3626

Pull Request Test Coverage Report for Build 1

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

Totals

Change from base Build 308:
0.0%

Covered Lines:
18721

Relevant Lines:
20965

💛  - Coveralls


fix(core): do not collect component twice, closes #177

What kind of change does this PR introduce?

[x] Bugfix
[ ] Feature
[ ] Docs
[ ] New Binding issue #___
[ ] Code style update
[ ] Refactor
[ ] Build-related changes
[ ] Other, please describe:

Does this PR introduce a breaking change?

[ ] Yes, and the changes were approved in issue #___
[x] No

Checklist

[x] When resolving issues, they are referenced in the PR's title (e.g fix: remove a typo, closes #___, #___)
[x] I have added a convincing reason for adding this feature, if necessary

Other information

Could you first report a bug and provide a minimal reproduction?

If PR is a bugfix, it's better to associate it with an issue, or create it if there is no existing issue, thanks a lot!

This solution is not good enough, for this case, a tuple can be used as a stack item to avoid the need to find each time.

I've opened a draft PR #178 and you can either update the changes to this PR to that, or just let me merge that in.

Close this, you can just merge #178


OutOfMemory with Dynamic Dataset LuaJit, preprocess_nthreads>1

see http://forum.opennmt.net/t/out-of-memory-with-dynamic-dataset/827

Fixed by a1b1501.


Unable to use With..

This is my class, and when I do  var modelSearch = new PostSearchRq();
there is no method With
Maybe I am doing smth wrong,
Thank you
using Lombok.NET;
using Newtonsoft.Json;

[AllArgsConstructor]
// [NoArgsConstructor]
[With]
public class PostSearchRq
{

    [JsonProperty("currency", Required = Required.Always)]
    public string Currency ;

    [JsonProperty("departure", Required = Required.Always)]
    public Departure Departure;

    [JsonProperty("arrival", Required = Required.Always)]
    public Arrival Arrival;

    [JsonProperty("occupancy", Required = Required.Always)]
    public Occupancy Occupancy ;

}

[AllArgsConstructor]
[NoArgsConstructor]
[With]
public class Arrival
{
    [JsonProperty("locationId", Required = Required.Always)]
    public string LocationId;

    [JsonProperty("returnDateTime", Required = Required.Always)]
    public DateTimeOffset ReturnDateTime;
}

[AllArgsConstructor]
[NoArgsConstructor]
[With]
public class Departure
{
    [JsonProperty("locationId", Required = Required.Always)]
    public string LocationId;

    [JsonProperty("dateTime", Required = Required.Always)]
    public DateTimeOffset DateTime ;
}

[AllArgsConstructor]
[NoArgsConstructor]
[With]
public class Occupancy
{
    [JsonProperty("adultsNo", Required = Required.Always)]
    public long AdultsNo ;

    [JsonProperty("childrenNo", Required = Required.Always)]
    public long ChildrenNo;

    [JsonProperty("childrenAges", Required = Required.Always)]
    public long[] ChildrenAges;
}

Classes annotated with [AllArgsConstructor], [NoArgsConstructor], or [With] must be partial in order for the code generation to work.


ceph-nfs: add ganesha_t type to selinux

Since RHEL 8.1 we need to add the ganesha_t type to the permissive
SELinux list.
Otherwise the nfs-ganesha service won't start.
This was done on RHEL 7 previously and part of the nfs-ganesha-selinux
package on RHEL 8.
Closes: https://bugzilla.redhat.com/show_bug.cgi?id=1786110
Signed-off-by: Dimitri Savineau<EMAIL_ADDRESS>(cherry picked from commit d75812529069244734732d05cc5aa3ddbc99b7c5)

jenkins test centos-container-all_daemons

jenkins test centos-container-all_daemons

unrelated CI failure


seedling Uplift CAPI to v1.2.6

What this PR does / why we need it:
Uplifts CAPI to the latest patch release, which includes security fixes.

/test-ubuntu-integration-release-1-2

/test unit
/test golangci-lint

Thanks for the uplift.
/lgtm


AUTO_INCREMENT

Resolve #159
Attempt #164
Attempt #165

Note: Built on #167; will rebase when merged.
@panarch not ready for merge. Any ideas on how to clean things up?

Optional feature auto-increment sounds good :)
I saw that you added optional methods in Store and StoreMut, get_generater and set_generater.
And.. it looks like you are considering SledStorage implementations to work both with auto-increment and without auto-increment.
Let's make it simple, AlterTable was simple to support as optional, but AutoIncrement might be different because it may require storage implementation code changes.
You can simply consider to run test based on..

features = [sled-storage]
features = [sled-storage, auto-increment]
features = [sled-storage, alter-table, auto-increment]

Let's exclude features = [sled-storage, alter-table] requirement.
Then, you don't need to provide specific implementation patterns to users.
Some storage makers may want to use a different strategy on implementing auto increment.
Current methods, get_generater and set_generater look like providing specific pattern.
How about making this simple as possible?
something like.. only having a single trait method, you can simply make a new store trait rather than putting methods into Store and StoreMut.
pub trait AutoIncrement<T: Debug>  {
    async fn generate_id(table_name, column_name) -> Result<T>;
}

Then using this, we can only need to call generate_id on auto increment constraint column, and put that id into Row. That's all we need to do.
Preparing space for auto increment id, it is all up to storage implementation.
I like current choice to prepare new slots for auto increment, but I don't think it is good to be exposed to store interface.
(btw, little bit off the topic, you might worry about this may make INSERT query execution not to be atomic. Let's don't care about it for now, we can also restore atomic in INSERT using different pr)

Oh I just found a mistake in my prev opinion. AutoIncrement trait may not need to use T because it returns numeric id rather than key T.

This makes sense.
Will give it a go.

Hello,
Could there be an issue when loading/restoring old dumps into tables having AUTO_INC columns.
The value should be taken from the dump dataset. Other dumped tables might refer as FOREIGN KEY.
There should be some ANSI like
DO not autoinc on table(s)
Restor table(s)
DO autoinc on tables(s)

There are a set of solutions depending on supplier
How to do in gluesql

@willy610
The idea is to only auto increment when there is no value
(aside: or, still a bit of a mixed idea on whether to do this and defaults for NULLs)
Thus if you are INSERTing data, as long as the AUTO_INCREMENT column has values, it should be fine.
Is that your concern?

@KyGost
Well that sounds good.

Always dump the value
and accept new value on insert
but no change on update!?

I did had some concerns some years ago on a complex system.
(Perhaps I was confused by the Export/Import options in the tool phpmyadmin.php.
Your comment on aside: is still actual )

@willy610 yup, it only ever bothers generating a value if it doesn't have one.
As for UPDATE depends on whether we do defaults for NULL or not. If we do, using NULL would generate a new value.
In all other UPDATE circumstances however, we just leave the value unchanged :-)

I've spent a little too long from this issue and don't remember what was in my way.
I've rebased it to main and it seems to be working. Moving out of draft.

This PR has become a bit of a mess. Will make a new one.


Default to lazy setup (and an additional check).

In relation to https://github.com/trbs/pid/issues/4.
The following PR is a culmination of fixing the following issue:
with pid.PidFile(lazy=True):
    # (1) pid is locked at this point
    pidfile2 = pid.PidFile(lazy=True)
    with raising(pid.PidFileAlreadyRunningError):
        # (2) the pidfile2._setup() isn't called when performing check()
        pidfile2.check()

I made a test and the appropriate fix to support double lazy checks. I then tried the testsuite defaulting lazy=True and that also worked. Furthermore, I moved the logging to the _setup() phase, because we cannot make assumptions whether the internal logging stream handle would be available after daemonization. It isn't elegant but I beleive it is the correct approach when combining pid with a daemon manager.

Coverage decreased (-0.37%) to 88.89% when pulling 59a61f8dd39fa6c7418a5775f028f9f2a796a14f on nnathan:lazy into b233bfebdce258a835e9654de77bb78015006761 on trbs:lazy.

You inferred the correct case, I like the third approach best. Let me fix up the code (after testing) and let's reevaluate the PR.

trbs, thanks for the feedback. I've implemented the suggested changes:
(1) use the logger property to automatically setup a logging instance (and clean up close()),
(2) remove the term_signal_handler you introduced, I didn't really see the obvious solution was to simply set the register_term_signal_handler=False. I was very confused by the TERM signal handler because I'm not quite familiar with atexit(), but now I understand the intentions when implementing this option.

Coverage decreased (-0.37%) to 88.89% when pulling 17692d1b05c91915ba633c11b8b471a6af3c5e5a on nnathan:lazy into b233bfebdce258a835e9654de77bb78015006761 on trbs:lazy.

Coverage decreased (-0.37%) to 88.89% when pulling 17692d1b05c91915ba633c11b8b471a6af3c5e5a on nnathan:lazy into b233bfebdce258a835e9654de77bb78015006761 on trbs:lazy.

Coverage decreased (-0.45%) to 88.82% when pulling 1cb6565c8e1ed681d26164462141ff2ee48d179b on nnathan:lazy into b233bfebdce258a835e9654de77bb78015006761 on trbs:lazy.

Nice !


⚠ Bofrost.de has degraded performance

In 266da2b, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1342 ms

Resolved: Bofrost.de performance has improved in 6a00cf5 after 309 days, 5 hours, 11 minutes.


test: Reduce login jail time in tests

I think this would be better if we could configure the login time out with a gluon server config option.

Agreed, done


exclude-subviews - not working on 5.4.0 (and :head)

I have an UITextField with a clear-button. The problem is that the border specified in Button in style.nss is also applied to the clear-button inside the text field even though I have set

Button {
    ...
    corner-radius: 0;
    exclude-views: UIAlertButton;
    exclude-subviews: UITableViewCell,UITextField;
}

Here's how it looks

Any advice on this?

Problem is that exclusion class names can have garbage spaces before and after which causes a mismatch. Best to fix it in the parser, but I couldn't track it down. Not sure when this may be applied, but this can also be fixed in UIView+NUI.m.

(void)setNuiClass:(NSString*)value
{
if (![value isEqualToString:kNUIClassNone]) {
// Set class to none if view is in the exclude
NSMutableArray *excludeViews = [NSMutableArray arrayWithArray:[[NUISettings get:@"exclude-views" withClass:value] componentsSeparatedByString:@","]];
// Add global exclusions to the list
[excludeViews addObjectsFromArray:[NUISettings getGlobalExclusions]];
if (excludeViews.count) {
NSString *currentClass = NSStringFromClass([self class]);
for (NSString *item in excludeViews) {
if ([item containsString:currentClass]) {
value = kNUIClassNone;
break;
}
}
}
}
if (![value isEqualToString:kNUIClassNone]) {
// Set class to none if any view superviews is in the exclude
NSMutableArray *excludeSubviews = [NSMutableArray arrayWithArray:[[NUISettings get:@"exclude-subviews" withClass:value] componentsSeparatedByString:@","]];
// Add global exclusions to the list
[excludeSubviews addObjectsFromArray:[NUISettings getGlobalExclusions]];
if (excludeSubviews.count) {
UIView *superView = self;
while (superView != nil) {
NSString *currentClass = NSStringFromClass([superView class]);
for (NSString *item in excludeSubviews) {
if ([item containsString:currentClass]) {
value = kNUIClassNone;
break;
}
}
superView = superView.superview;
}
}
}
objc_setAssociatedObject(self, kNUIAssociatedClassKey, value, OBJC_ASSOCIATION_RETAIN_NONATOMIC);
}

So I took a shot at it, and I think I have located the spot where to fix it, but I'm not sure if this will break anything else:
In NUIParserDelegate.m line 48
change from
[contents appendFormat:@" %@", tokenString];  (notice the space before %@)
to
[contents appendFormat:@"%@", [tokenString stringByTrimmingCharactersInSet:[NSCharacterSet whitespaceAndNewlineCharacterSet]]]; (without the space before %@ and extra measures for other misplaced characters)

Appreciate it. Looks like you're fixing the root cause of the problem whereas I was only patching the place where the problem manifested itself.
Unfortunately, there are a bunch of pull requests in the system but nothing's been approved in months. I sent an email to Tom asking if he was still working on NUI, but got crickets.
As such, we decided to go with Pixate instead. Thanks again.

+1!!!!

Another way around this bug:
    UIButton *clearButton = [theTextField valueForKey:@"_clearButton"];
    clearButton.nuiClass = @"none";

Yet another way which seems to work better in iOS 9:
UIButton *defaultClearButton = [UIButton appearanceWhenContainedIn:[UITextField class], nil];
defaultClearButton.nuiClass = @"none";

Fixed by #312.


Did you make those data preparation code for the tutorial on your own? What's your issue? 🙌 🙌 🙌

Well I can simply say Thanks to Siraj for introducing to such an enormous topics in such a simple way


sqlsrv_driver > Change @@IDENTITY with SCOPE_IDENTITY()

In the sqlsrv_driver.php the function insert_id() doesn't work well in some cases, returning an unexpected ID. It can be solved changing @@IDENTITY with SCOPE_IDENTITY().
More info: http://www.databasejournal.com/features/mssql/article.php/10894_3307541_2/Getting-the-Wrong-Identity-in-Microsoft-SQL-Server-identity-Columns.htm

Please explain what these "some cases" are.
And it's identity, no f in there - you made that typo 4 times... That's probably a bad thing to judge by, but it does give the impression that you're not quite familiar with what you're talking about, and in turn making me question whether I should take your suggestion seriously. Am I right to be worried about this?

Yes, I made a typo, so that's a bad thing to judge.
If you are interested in the topic you can refer to the link where you can read:
"@@IDENTITY is usually the first one suggested in books and news groups for discovering new identities. However, its real purpose is not at first evident and may produce the wrong identity. @@ IDENTITY will return the last identity used in your session, but across all scopes. Meaning if your statement causes a trigger to fire, and that trigger insert into a table with an identity field, the triggers new identity will be returned, not your statements new identity. For example, create a test table with an identity, and a trigger on the products table that will insert on the test table.
(...) The function SCOPE_IDENTITY returns the new identity created on any table in this session in this scope, giving us the new value we would expect."
If you use SQL Server Audit to audit a table named 'books' for example, when you insert a row CI won't return the new book ID but the Audit table new row ID.
PD> Sorry, I'm not fluent in English


Migrate server module tests to JUnit 5, part 2

This PR continues migration of the server module to the new JUnit 5 test structure.
The following packages were migrated:

bc
bus
command
commandbus
commandstore

@armiol PTAL.


build issues on Android X

Hi,
Thanks for providing the library.
I tested it out on my project which was recently migrated to android X. But, it's causing build errors in the project.
I need to know, if this library does have support for android X projects or not ?

Any solution for this issues?

@tbarthel-fr ??

@vbv1 @rejsiperpalaj in case you are still interested, just created a fork and migrated to AndroidX
https://github.com/amanzan/BlurDialogFragment
You can get it from jitpack:
    implementation 'com.github.amanzan:BlurDialogFragment:3.2.1'

@amanzan your fork fix this issue! It's work for me.


BUG REPORT: Error Messages Terminal freezing.

When user tries to interact with the terminal the program completely pauses. Connections still work but drop out every 30 seconds or so. .
the fix is to scroll all the way down in the terminal and backspace anything that was being typed into the terminal.
Repeating error until I manually fixed the terminal.
Suggestions is not to mess with terminal in any way if operating normally.
OpentakRouter v1.0.9 windows.
Example of error messages after the terminal caught up:
at dpp.opentakrouter.TakTcpPeer.OnReceived(Byte[] buffer, Int64 offset, Int64 size)
[17:47:19 INF] peer=dpp-public-tcp event=cot type=a-f-G-E-V-C
[17:47:19 ERR] peer=dpp-public-tcp type=unknown error=true forwarded=false
System.InvalidOperationException: There is an error in XML document (0, 0).
---> System.Xml.XmlException: Root element is missing.
at System.Xml.XmlTextReaderImpl.Throw(Exception e)
at System.Xml.XmlTextReaderImpl.ParseDocumentContent()
at System.Xml.XmlReader.MoveToContent()
at Microsoft.Xml.Serialization.GeneratedAssembly.XmlSerializationReaderEvent.Read11_event()
--- End of inner exception stack trace ---
at System.Xml.Serialization.XmlSerializer.Deserialize(XmlReader xmlReader, String encodingStyle, XmlDeserializationEvents events)
at System.Xml.Serialization.XmlSerializer.Deserialize(TextReader textReader)
at dpp.cot.Event.Parse(String payload)
at dpp.cot.Message.Parse(Byte[] data, Int32 offset, Int32 length)
at dpp.opentakrouter.TakTcpPeer.OnReceived(Byte[] buffer, Int64 offset, Int64 size)
[

The hanging terminal thing is the result of something called "Quick Edit Mode" in Windows. When you select any text in the terminal it pauses the output to make it easier to select text. This has the unfortunate side effect of also pausing the running program. More discussion on the topic at https://stackoverflow.com/questions/33883530/why-is-my-command-prompt-freezing-on-windows-10.
For the error - I actually removed the default peering with the public instance. It was meant as a demonstrative configuration option only. This is resolved in https://github.com/darkplusplus/opentakrouter/pull/25 and contained in the default configuration for v1.0.10.


DMA2D artifacts

After long time found out that I D caches on the Cortex M7 affect the graphics.
In case a user needs to enable them, they should properly configure the MPU.

Great! Thanks for sharing this!
Won't disabling the cache cause huge performance loss?

Indeed, it does affect the performance a lot, which is something I didnt notice using the theme.
Running the benchmark gave me those results bellow.
Left side CPU results, middle DMA2D, right DMA2D with I&D cache off
Plain benchmark 73ms   40ms    56ms
Wallpaper ON     75ms    43ms   103ms

Wp recolor     104ms    80ms  360ms
Shadow          111ms    87ms  387ms
Opacity           113ms    89ms  389ms
Will dig into it further!

@kisvegabor  Please check this one.
Now the performance is much better.
Plain benchmark 39ms
Wallpaper ON 41ms
Wallpaper ON + Wp recolor 80ms
Wallpaper ON + Wp recolor + Shadow 88ms
Wallpaper ON + Wp recolor + Shadow + Opacity 89ms

In another, unrelated project with STM32F7 I found that having write-through D-cache enabled with a framebuffer still led to graphical artifacts.
Can we just turn off caching altogether for that MPU region? It shouldn't affect performance as we almost never read from that section of memory.
Also, if we are going to be updating this project to STM32CubeIDE, there is no real need for the separate Atollic project to exist anymore, correct?

In another, unrelated project with STM32F7 I found that having write-through D-cache enabled with a framebuffer still led to graphical artifacts.

It's very strange that there were artifacts even in write-through mode.

Can we just turn off caching altogether for that MPU region? It shouldn't affect performance as we almost never read from that section of memory.

I don't know what caching option there are in F7 MPUs but if it's possible then it sounds promising.

Also, if we are going to be updating this project to STM32CubeIDE, there is no real need for the separate Atollic project to exist anymore, correct?

I've already heard good things about CubeIDE (I still use Ac6) and probably it'd be worth to update to it as it's the current recommended ST IDE. In this case Atollic project can be archived.

It's very strange that there were artifacts even in write-through mode.

I agree. I didn't have a chance to investigate further, but I'm suspicious that it may have been related to errata 1259864.

I don't know what caching option there are in F7 MPUs but if it's possible then it sounds promising.

As far as I know there are options for write-back, write-through, and no caching. I will double check the Cortex-M7 reference manual but it should mainly involve changing the IsCacheable property for the MPU region @papadkostas added to MPU_ACCESS_NOT_CACHEABLE.

As far as I know there are options for write-back, write-through, and no caching. I will double check the Cortex-M7 reference manual but it should mainly involve changing the IsCacheable property for the MPU region @papadkostas added to MPU_ACCESS_NOT_CACHEABLE.

Maybe easier to try it :slightly_smiling_face:
I'd try it (at least I'd try the CubeIDE too) I don't have that board with moe now.

Maybe easier to try it :slightly_smiling_face:

@papadkostas will have to try it, because I don't have the F769 (only the F746).

@embeddedt thanks! just tried it and pushed the changes.
The performance haven't affected at all.

Thank you for the tests and fixes. It can be merged from my side.
@embeddedt ? (Note the update to CubeIDE)

Merged. It appears that we actually only have one Atollic project (for STM32F746). I will archive that now and add a note that STM32CubeIDE is the way forward.

I will archive that now and add a note that STM32CubeIDE is the way forward.

Great! Thanks!

should I implement the functions gpu_fill_cb fill and gpu_blend_cb as well?

should I implement the functions gpu_fill_cb fill and gpu_blend_cb as well?

It'd be great!

Not working as I expected,should i open a PR for discussion?
Here's how performing and you can see the issues as well.
Also I've noticed something when benchmarking on #2.
When pressing opacity, the opacity is not updated completly until you hit any other buton than opac.
While this is visible in the benchmark, on the demo is working fine. Only appears with DMA2D!

@papadkostas
What if you set disp_drv.gpu_blend_cb = NULL. Is it working with gpu_fill_cb only?

If i disable the blend function, the redering gets back to normal times but still when I press the shadow button it crashes the application and the opacity button creates those weird lines and snow on the buttons. Video here

It's strange because gpu_fill_cb should be used only for fully covering (opacity=100%) drawings.
Can you give a link the current version of the driver?

It's strange because gpu_fill_cb should be used only for fully covering (opacity=100%) drawings.
Can you give a link the current version of the driver?

What do you mean by "version of the driver"?
The filling function is here

What do you mean by "version of the driver"?

Yes, I meant the latest version of the driver (as you linked it).
Are you sure this line is correct? I think instead of TFT_HOR_RES you should use dest_width because when lvgl redraws only a button the width of dest is only the width of the button.
Besides, does it work if use line-by-line fill instead of rectangle fill?
Similarly to this.

@kisvegabor But dest_width - width always equals 0, right?

But dest_width - width always equals 0, right?

No, they can be definitely different.
For example, if you press a  button dest_width will be the width of the button because only it is refreshed. However, when e.g. its left border is drawn width will equal to border_width.
The same thing happens if you refresh the entire screen. dest_width will be hor_res but width will be the size of the children being drawn.


selected python version is not preserved on uv sync

I just started to check for uv as an alternative.
I face the problem, that, even if python 3.12. is stored in toml and lock file uv sync generates an environment with the current active python, e.g. 3.11.
I tried to predefine a fresh venv with python 3.12 but this is deleted on calling uv sync.
Since I'm new to uv it might be my fault. I'ts all very basic and I'm sure, there are solutions.
Maybe related, maybe a different problem: If i generate named venvs e.g. venv311 and venv312 uv sync also fails because there is no .venv directory.

Ok. The hint for  uv python pin 3.12 did it. Since I found a related problem here I did not dig deeper in the web to find this. Sorry and thank you.

Thanks for following up!


Curry undocumented

In API section on github

Closing (https://gcanti.github.io/fp-ts/modules/function.ts.html#curry-function)

Actually, it no longer exists. Or am I just dumb?

@antonkatz Auto currying is not type-safe with TypeScript. The recommended solution is manual currying.

Couldn't this help to bring this one back: https://v10i.dev/p/auto-currying-in-typescript ?

Is there an explanation why it is not safe, or some documentation around that fact?


Unable to deposit into BCH saver vault

Overview
When trying to deposit into BCH saver vault, getting the error" Transaction Failed"
References and additional details

Log into app.shapeshift and connect to wallet of choice
Navigate to Earn page and select BCH saver vault
Enter flow to deposit and broadcast the transaction
Note that the error pops up "Transaction Failed"

Acceptance Criteria
Should be able to deposit into BCH saver vault
Need By Date
No response
Screenshots/Mockups
No response
Ownership

[X] If my bounty needs engineering or needs product I have added the respective labels on the right
[X] As the sponsor of this bounty I will review the changes in a preview environment (ops/product) or review the PR (engineering)

Estimated effort
No response
Sponsor / Stakeholder
No response
Bounty Hunters

Join our discord
Include an expected timeline for you to complete work in the work plan when you apply for this bounty!
Please refer to this link for some basic info
Please do not start work on this issue until you are approved in Gitcoin.

Current debugging:
gomes — Yesterday at 11:20 PM
Prod also failed on me for the same reason - regular sends are happy and so is reconciliation though so the node is somehow not happy with this precise send (the final one for savers deposits)

/api/v1/send is failing on us with a 520. Reconciliation Tx that's broadcasted before that is somehow happy. Not sure if there's something in the deposit Tx which is built which is currently wrong because of upstream shenanigans, as we haven't changed anything ther recently, or if our send endpoint is rugging us. Will continue investigating - if it's the latter, no web change is going to be able to fix this, though that would be weird as the previous reconciliation Tx is working.

Confirmed our built Tx is looking sane, compared against a THORSwap Tx which was broadcasted fine.
Moving this to blocked, as this looks to be caused by our node currently struggling with Txs with OP_RETURN_DATA Txs - regular sends (i.e without OP_RETURN_DATA) are unaffected.
Confirmed with a monkey patch without OP_RETURN_DATA (effectively burning money since the Tx cannot be identified by THOR) and was able to proceed with the send


Some instructions for developers

Readme is now done, but we could still use:

comments in source code
some documentation on architecture (it's not too complicated so this should be short)

I would like to try


Add support for Subscription timestamps and automatic cleanup

Design Issue: https://github.com/realm/engineering/issues/12
This PR adds new capabilities to subscriptions:

3 new timestamps to Subscriptions: createdAt, updatedAt and expiresAt
Makes it possible to update the underlying query for a named subscription
Subscriptions which have parsed their expiresAt date, will now be removed automatically when the app is started or whenever a subscription is created/modified.

Design thoughts:
By hooking the cleanup up to creating/modifying existing subscriptions we do not have truly empheral subscriptions. For that, we would need to tie it in with the destruction of the notification token. However, it isn't clear to me how to accomplish that relatively easy. Also, I'm not sure it matters.
The reasoning being:

If we tie the cleanup to the Notification token, it will have to be asynchronous. Meaning an app crash can disrupt the cleanup. Cleaning up on startup catches these cases. Worst case you have subscription living longer on the server if the app isn't re-opened, but this is true either way.

In all other cases, the user will navigate to a screen in the app that also contains a subscription (presumably). Which will trigger cleanup?

The only concern I have is that running cleanup on all subscription modifications might slow down creating new subscriptions. It is unclear at which point we have so many subscriptions that running the cleanup query actually impacts creating the subscription to a noticeable degree.

TODO:

[x] Add automatic cleanup when writing subscriptions
[ ] Add tests for timestamps
[ ] Add tests for automatic cleanup
[ ] Determine the name of the underlying fields. E.g. camelCase vs. under_score
[x] Set expiresAt to far in the future when upgrading old Realms to mimic old behaviour.
[ ] Create issues in all repos for exposing this.
[ ] Verify the usefulness of this implementation by implementing at least on binding API before merging.

Wouldn't it make sense to make the expiresAt value optional? I realize there's no practical difference between a huge value and no value, but still, would be nice to semantically indicate when you expect a subscription to last forever.

Do we have any logic for updating the expiresAt? Sometimes (often?) it would be nice to have it relative to when the data was last used in the app (so you for example could have a subscription stay around for 2 days after last use, just as a cache in case someone did the same query again).

@astigsen Right now the idea was to set expiresAt either when creating or updating the subscription (e.g. replacing the query). Making it relative to expiresAt sounds useful, probably even more useful than a fixed expiresAt. I see you also made a similar comment in https://github.com/realm/engineering/issues/12, sorry for having missed that.
I agree that this use case is probably more natural (and safer). I'll look into changing expiresAt to be an offset from last used.

Making expiresAt optional would also be good because it makes the migration a no-op. It probably wouldn't ever be an issue, but setting it to max_timestamp after creating the column opens up the possibility of weirdness if the file is then opened with an older version of Realm which creates more subscriptions without setting the timestamp.
Performing cleanup when adding a new subscription is very unlikely to ever be an issue performance-wise. The write transaction overhead is going to dominate everything else, so as long as you can do the cleanup in the same transaction...

Ready for review


Add contrib opencv packages to the opencv check.

This commit adds contrib versions of opencv to the list of candidate opencv packages, in addition to opencv-python and opencv-python-headless.

Now I see that this PR tries to solve the same thing as https://github.com/WASasquatch/was-node-suite-comfyui/pull/461

Couldn't hurt. So many builds.


Fix a number of typos

This PR fixes a number of typos in the comments, the attribute strings, along with method and variable naming.

Very nice! :satisfied:

@sean-gilliam Thanks!

Almost sad to see words like ingore and recusion go :cry: :smile:
Still: thank you :+1:


Add support for embedded-hal-async I2C traits.

This uses maybe-async-cfg to automatically remove the async and .await keywords from the blocking version.
It also exposes the performance mode settings, since those were previously not exposed; let me know if you want me to split that into its own PR.

This is great, thanks!
Learning more about async at the moment and also looked into the maybe_async crate as well.


Evaluating against ground truth files gives imperfect success score (OTB100 metric)

As a sanity check, using ground truth files as tracking results and evaluating against them should supposedly yield perfect scores. However, the code actually gives a success score of 0.952 (=20/21) instead of 1.000. Is this expected behavior?

@bkkm78 hi ,are u solved this question? I met the same question.I evaluated the siamrpn++ results .but the success score is 0.665.hope u answer .thanks

@bkkm78 hi ,are u solved this question? I met the same question.I evaluated the siamrpn++ results .but the success score is 0.665.hope u answer .thanks


Header.cap component does not include package name

When building a cap everything works fine with JCDK 3.0.3 that I'm testing but the generated Header.cap component does not include a name (The package_name_info from the CAP file format specification has length 0). I see the same behavior if I use the regular ant tools so probably it is related to the internal converter but I wonder if is worth trying to fix that in this project. If so, I can have a look if someone can provide some guidance where to look to fix that.

Where and how does the issue mandate itself? It is a project without a package (default package). I understand that building works, but afterwards installation or using the cap with some other tool fails?

Is purely a cosmetic issue, the resulting cap installs with no issues and if anything, is only that a manual parser would not find a name to display. I would like to fix it just for completeness sake, but I have not seen any impact other than cosmetic.

Okay. As ant-javacard is a simple wrapper for the original Oracle code, I don't believe there is a "properly fit" place for such a hack (or it would be justified to add it).
Otherwise two possible locations:

Modification to the cap file: https://github.com/martinpaljak/ant-javacard/blob/master/src/pro/javacard/ant/JavaCard.java#L507
Adding some parameter to the process: https://github.com/martinpaljak/ant-javacard/blob/master/src/pro/javacard/ant/JavaCard.java#L473

As ant-javacard is a usability wrapper, I don't intend to fix non-blocking issues with the Oracle developer kits. Meaningful minimalistic patches that fix annoying issues are still welcome, but they must be patches, not bug reports. Thus closing this for now.


Issue 313

Goal: Add code output section with a button run it and displaying the changed output state
Problem: Correct output state is not displaying

Made some fixes based on feedback, switching to a fxnal component with jsx the fix. Closing this PR to start another one.


fix(git): properly log errors from git

Log the command and the Stderr that results in an git error:
$ deis git:remote -a test
Error: Error when running 'git remote add deis ssh://git@deis-builder.<IP_ADDRESS>.nip.io:2222/test.git'
fatal: Not a git repository (or any of the parent directories): .git

Current coverage is 8.83% (diff: 0.00%)

Merging #199 into master will decrease coverage by <.01%

@@            master      #199   diff @@
========================================
  Files           49        49
  Lines         3460      3462     +2
  Methods          0         0
  Messages         0         0
  Branches         0         0
========================================
  Hits           306       306
- Misses        3115      3117     +2
  Partials        39        39

Powered by Codecov. Last update 272b1cd...07637bd

I unset all the LGTMs because I greatly changed the PR. Same idea, but I'm not adding the error checking to every function and added more error checks in the cmdr.GitRemote method as well.

For some reason this is pointing to CI build 3, but build 4 passed https://ci.deis.io/job/Deis/job/workflow-cli/job/PR-199/4/.


python310Packages.cftime: 1.6.1 -> 1.6.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python310Packages.cftime is: "Time-handling functionality from netcdf4-python"
meta.homepage for python310Packages.cftime is: "https://github.com/Unidata/cftime"
meta.changelog for python310Packages.cftime is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS
Warning: a test defined in passthru.tests did not pass
found 1.6.2 with grep in /nix/store/992a2k61cpc08d1a4lnb85x2qd5gfx3w-python3.10-cftime-1.6.2
found 1.6.2 in filename of file in /nix/store/992a2k61cpc08d1a4lnb85x2qd5gfx3w-python3.10-cftime-1.6.2

Rebuild report (if merged into master) (click to expand)

24 total rebuild path(s)

24 package rebuild(s)

First fifty rebuilds by attrpath
python310Packages.arviz
python310Packages.boltztrap2
python310Packages.cftime
python310Packages.datashader
python310Packages.h5netcdf
python310Packages.meshio
python310Packages.netcdf4
python310Packages.pygmt
python310Packages.pymc
python310Packages.sfepy
python310Packages.sunpy
python310Packages.wrf-python
python39Packages.arviz
python39Packages.boltztrap2
python39Packages.cftime
python39Packages.datashader
python39Packages.h5netcdf
python39Packages.meshio
python39Packages.netcdf4
python39Packages.pygmt
python39Packages.pymc
python39Packages.sfepy
python39Packages.sunpy
python39Packages.wrf-python

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/992a2k61cpc08d1a4lnb85x2qd5gfx3w-python3.10-cftime-1.6.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python310Packages.cftime https://github.com/r-ryantm/nixpkgs/archive/00f0293f1789f3c491ca8cc27206e00f4a05e77d.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/992a2k61cpc08d1a4lnb85x2qd5gfx3w-python3.10-cftime-1.6.2
ls -la /nix/store/992a2k61cpc08d1a4lnb85x2qd5gfx3w-python3.10-cftime-1.6.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

2 packages failed to build:

python310Packages.sunpy
python39Packages.sunpy

22 packages built:

python310Packages.arviz
python310Packages.boltztrap2
python310Packages.cftime
python310Packages.datashader
python310Packages.h5netcdf
python310Packages.meshio
python310Packages.netcdf4
python310Packages.pygmt
python310Packages.pymc (python310Packages.pymc3)
python310Packages.sfepy
python310Packages.wrf-python
python39Packages.arviz
python39Packages.boltztrap2
python39Packages.cftime
python39Packages.datashader
python39Packages.h5netcdf
python39Packages.meshio
python39Packages.netcdf4
python39Packages.pygmt
python39Packages.pymc (python39Packages.pymc3)
python39Packages.sfepy
python39Packages.wrf-python

Maintainer pings

@ofborg eval


A cleaner right side of the navbar with a Github link

Coverage decreased (-0.07%) to 81.34% when pulling 4052cd1dc73a5c68f8a7fd3b5accd8542329aa9a on mistercrunch:nav_right into ab71ee4f93abe6a2f83f367258fcc6f812640008 on airbnb:master.

Coverage decreased (-0.03%) to 81.373% when pulling 4052cd1dc73a5c68f8a7fd3b5accd8542329aa9a on mistercrunch:nav_right into ab71ee4f93abe6a2f83f367258fcc6f812640008 on airbnb:master.


Hitting enter does not indent line properly

VSCode Version: 1.19.2
OS Version: W10LTSB

Steps to Reproduce:

Type a line of code that opens a block, like an HTML div tag or a PHP function or a Javascript function, then hit Enter.
Notice your cursor is plopped at the at the same horizontal point as the beginning of the above opening block line, instead of one indent inward.

I read this may have something to do with TypeScript, but implied it deletes an empty line, but that's not what's happening here as it's keeping the other indents. Also I'm not using TypeScript, and as mentioned it happens when coding PHP and HTML too, and I'm not seeing that I'm using any code formatting TypeScript anything.
I thought this previous bug was the problem - https://github.com/Microsoft/vscode/issues/41702 - but apparently that's part of this bigger issue.

Does this issue occur when all extensions are disabled?: Yes

If you change the language to HTML and then back to PHP the problem is solved until you close the editor.

I need a code sample to reproduce...


tf.strings.substr crashes(aborts)

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04): Linux Ubuntu 18.04
Mobile device (e.g. iPhone 8, Pixel 2, Samsung Galaxy) if the issue happens on mobile device: N/A
TensorFlow installed from (source or binary): binary
TensorFlow version (use command below):2.1.0
Python version:3.7.6
Bazel version (if compiling from source):N/A
GCC/Compiler version (if compiling from source):N/A
CUDA/cuDNN version:N/A
GPU model and memory:N/A

Describe the current behavior
tf.strings.substr crashes(aborts)  when len(pos) > len(input)
Describe the expected behavior
expect an exception message if the input unexpected instead of crash.
Standalone code to reproduce the issue
import tensorflow as tf
tf.strings.substr(input='abc', len=1, pos=[1,-1])

import tensorflow as tf
tf.strings.substr(input='abc', len=1, pos=[1,2])

Output:
2021-02-03 22:46:41.234297: F ./tensorflow/core/framework/tensor.h:806] Check failed: new_num_elements == NumElements() (2 vs. 1)
Aborted (core dumped)

@rmothukuru
I ran the code shared on tf 2.4 and nightly, colab crashes, please find the gist here.

According to documentation https://www.tensorflow.org/api_docs/python/tf/strings/substr an error should be thrown out gracefully (instead of a crash). Added a PR #46974 for the fix.


Handle HTTP Basic Auth for client's access to token endpoint

This PR closes #132 and #199
Section 4.1.3 says "If the client type is confidential or the client was
issued client credentials (or assigned other authentication
requirements), the client MUST authenticate with the authorization
server as described in Section 3.2.1.", and that is HTTP Basic
Authentication.
This commit is based on #292 PR.
@night I have refactored your PR #292 (request.cliend_id are not patched in _get_client_creds_from_reponse anymore) and fixed Python 3.x compatibility issues.

@lepture The tests are all green and I think it is ready to be merged.

lgtm 👌

@night Thank you for your work! Your PR was a great starting point!

@frol Thanks for your work


UX improvement on discussion forum buttons:  "report", "vote" and "like"

Before this commit the buttons (report, follow and vote) in discussion adapter waited for api to complete before appearing selected, this creates a bad user experience as users expect immediate responses and has the potential to create unexpected results if users click lots of times very quickly.  Since most of the time api calls will succeed it is better to have the buttons respond immediately assuming they will work.  Then if the api call fails turn them back.
Description
LEARNER-XXXX
There was a learner that someone brought up about these buttons when a user clicks lots of times it creates unexpected results, this learner seems to have been removed (or I just can't find it).

@adamsythe Following are some of my observations after I manually tested the PR and did some code review:

Since we are only updating the view itself and not the data behind it, if the server call takes some time to succeed/fail and the user scrolls down and up again so that the adapter recreates the updated row, the change would be reverted. What are your thoughts on this case? (Link to testing video: https://openedx.atlassian.net/secure/attachment/65579/state_reverting.mp4)
The indentations you've added don't go well with the overall project, can you revert them?

For example:

@miankhalid  thanks so much for the feedback and sorry it has taken so long for me to get back to it.  Ill try my best to push something that fixes the issues you raised as soon as possible

@miankhalid
Thanks I changed the branch to fix the spacing and to update with the current master of edx.
As for your UX question.
The far majority of cases of people using the discussion will be with internet, so i think it is more confusing for a user to click a button while they have slow internet and wait a couple seconds for the click to be registered than the unlikly case that a user does not have internet and it takes a user a couple of seconds to wait for it to have incorrectly changed (or in your case scrolled away and came back).  If you look major apps they all use assume success for that reason and I believe app users instinctively understand this.  Here is an article on mistakes android developers make and number 7 refers to this point . https://www.toptal.com/android/top-10-most-common-android-development-mistakes
Of course the absolute winner would be to have local storage and if the click fails to store it until the user gets internet and then try the api call again - but edx is still a bit away from that sort of tech.

Thanks @miankhalid  for the really great feedback, will update asap

@miankhalid  updates done

@adamsythe any ETA on the fix?

@miankhalid
Fantastic!  Done: let me know what you think

Sorry for the nits again and again.
But the squashed commit message and the title still can be minified somehow. Plz have a look at this great article:
http://chris.beams.io/posts/git-commit/

Thanks @miankhalid  that was a great article to follow.  Always keen to learn more about best practices, please let me know if it looks better now.

No problem, glad I was able to help. Keep up the good work 👍

@adamsythe 🎉 Your pull request was merged!
Please take a moment to answer a two question survey so we can improve your experience in the future.


chore(localization): update translations

This PR was open while I was editing the CrowdIn configuration and is not valid.


Replace Function Improvements

-Replace function improvements.

Release updates.
test action update.

Codecov Report
Patch coverage: 100.00% and project coverage change: +0.44 :tada:

Comparison is base (31db7dd) 96.42% compared to head (1bea606) 96.87%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #13      +/-   ##
==========================================
+ Coverage   96.42%   96.87%   +0.44%
==========================================
  Files           1        1
  Lines          28       32       +4
==========================================
+ Hits           27       31       +4
  Misses          1        1

Impacted Files
Coverage Δ

src/replace.js
96.87% <100.00%> (+0.44%)
:arrow_up:

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Any particular reason Accordion is misspelled as accordeon?

In documentation and in the library itself, accordion is misspelled as accordeon in class names accordeon-trigger js-accordeon-trigger Is there an intentional reason for this?

Hi! Thanks a lot for reporting! This isn’t intentional and an error I obviously made based on the German spelling: "Akkordeon". 😬 Will fix this ASAP.


Add network filter to top collection

I wish you all the best for the new year
lets start with one addition I believe will make user less confused about the networks

lets do something like that, but without the "network:" please

changing based on the network switch in the navbar

https://www.figma.com/file/JXNtryaNYhY0XJNArzvPQx/Handoff?type=design&node-id=1287-18959&mode=design&t=ICUuvAohcPrUiimF-11

👋

👋


fix(leaving-ibm): token updates

Related Ticket(s)
https://github.com/carbon-design-system/carbon-for-ibm-dotcom/issues/6205
Description

[X] Update label to use $label-02 token
[x] Update link to use Link large variant from Carbon core

@photodow checking what happened to the react wrapper

@photodow so the duplicated style block was for the react wrapper. I restructured it so we don't have to use it anymore.


[SPARK-21893][SPARK-22142][TESTS][FOLLOWUP] Enables PySpark tests for Flume and Kafka in Jenkins

What changes were proposed in this pull request?
This PR proposes to enable PySpark tests for Flume and Kafka in Jenkins by explicitly setting the environment variables in modules.py.
Seems we are not taking the dependencies into account when calculating environment variables:
https://github.com/apache/spark/blob/3a07eff5af601511e97a05e6fea0e3d48f74c4f0/dev/run-tests.py#L554-L561
How was this patch tested?
Manual tests with Jenkins in https://github.com/apache/spark/pull/20126.
Before - https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85559/consoleFull
[info] Setup the following environment variables for tests:
...

After - https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/85560/consoleFull
[info] Setup the following environment variables for tests:
ENABLE_KAFKA_0_8_TESTS=1
ENABLE_FLUME_TESTS=1
...

cc @srowen, @vanzin, @yhuai and @zsxwing who I believe are primarily related with this change.

Test build #85561 has started for PR 20128 at commit 3b1533a.

Ah I probably missed that when moving these behind a profile.  If it resolves the problem, yes go for it.

Test build #85561 has finished for PR 20128 at commit 3b1533a.

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

Merged to master.
Thank you @srowen, @felixcheung and @ueshin for reviewing this.


Update Introduction.md

Fix for Issue 40:
https://github.com/AMWA-TV/nmos-dns-sd-implementation-guide/issues/40

@rmsporter any comments?

Ah, it's good that you changed that wording but it's not the bit I was referring to which is right at the end of Introduction.md in the "Conclusions" part.


Add ruamel.yaml as requirement

Without ruamel.yaml installed, MkDoxy doesn't work.
File ".../MkDoxy/mkdoxy/utils.py", line 5, in
from ruamel.yaml import YAML
ModuleNotFoundError: No module named 'ruamel'

I'm not sure what's best, should this be versioned or not? In any case, nothing works for me without the yaml package.

This is a duplicate of #71
The consensus was to use PyYAML instead and drop ruamel.yaml dependency.

I have merged this PR with ruamel.yaml for successful work of this plugin.
If anyone adds PR with better PyYAML I will merge it to the master.


Test and fix user-facing issues

addresses #124 (#73, #107, #113, #116)

latest commit resolves #105

412802b closes #116


fix: fixes infinite loop when running into an error

Description

fixes infinite loop when running into an error

Testing required outside of automated testing?

[x] Not Applicable

Screenshots (if appropriate):

[x] Not Applicable

Rollback / Rollforward Procedure

[x] Roll Forward
[ ] Roll Back

Reviewer Checklist

[ ] Description of Change
[ ] Description of outside testing if applicable.
[ ] Description of Roll Forward / Backward Procedure
[ ] Documentation updated for Change

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Removed Azure CNI

According to LINK
Kubernetes docs allow content for third-party projects only when:

Content documents software in the Kubernetes project
Content documents software that's out of project but necessary for Kubernetes to function
Content is canonical on kubernetes.io, or links to canonical content elsewhere

This entry does not meet any of these requirements for inclusion and should therefore be removed.

This CNI plugin is useful if you have a Kubernetes cluster on Azure and want cluster networking.
We should not remove it. Networking is prerequisite for a working Kubernetes cluster.
/hold

This is inconsistent.  Azure CNI is NOT required for Kubernetes.
Perhaps it is useful.  Physical networking on-prem is also useful (and dare I say required for cluster functionality).  In fact you are not going to get very far with the Azure CNI without a physical network to interconnect the nodes at L2 or L3, load balance across ingress', firewall service ports on the physical network, for the purposes of using the CNI in conjunction with Azure.
Just because Azure is a big service provider, doesn't mean they should get preferential treatment in how the rules are applied.
The rule in question is:
"Content documents software that's out of project but necessary for Kubernetes to function"
Azure CNI is NOT NECESSARY for Kubernetes to function.  Therefore this content should be removed from this page.

SIG Docs tech lead here.
Networking is required for Kubernetes. Any CNI plugin that provides Kubernetes-compatible networking does count as providing functionality necessary for Kubernetes to function. We should not remove this section.

If "networking" is required for Kubernetes, what is in scope?
You are saying CNI is in scope (even though you do not NEED them to function in-project).  K8s will function just fine without a CNI.
What about physical networking?  Will your k8s cluster work without a network to connect the nodes?  No, therefore physical networking is necessary to function in-project.
How about a vswitch or OVS?  Is this necessary for k8s to function?  No.
The only rule that is clear is:
Content documents software that's out of project but necessary for Kubernetes to function
An Azure CNI is entirely unnecessary for a kubernetes cluster to function properly in-project.

You are saying CNI is in scope (even though you do not NEED them to function in-project). K8s will function just fine without a CNI.

@ixodie , I don't agree with this statement.
I agree with @sftim's comment that Azure CNI should not be removed

Can you build a cluster without using one of these CNIs?  Sure, it will use kubenet right?  Do you need the Azure CNI in order to run k8s?  Of course not.
The goal of 20232:
Our goal is to minimize and eliminate third-party content except when necessary for Kubernetes
to function in-project.
"In project" means software in the Kubernetes org, which includes the kubernetes and kubernetes-sigs repositories.
K8s functions perfectly well without this CNI.  This is a third party CNI specifically for use in a big cloud (Azure).  Why does Azure get preferential treatment?  A non-Azure k8s admin will never need this CNI, it is NOT NECESSARY.
Please reread the below very slowly:
**Our goal is to minimize and eliminate third-party content except when necessary for Kubernetes
to function in-project.
"In project" means software in the Kubernetes org, which includes the kubernetes and kubernetes-sigs repositories.**

Here's the KEP that the content policy is based on: https://github.com/kubernetes/enhancements/tree/master/keps/sig-docs/1326-third-party-content-in-docs

It says:

Clearly define what documentation is required so that readers understand how to deploy, operate and consume Kubernetes clusters using features from in-project code and its mandatory dependencies.

Azure CNI does not match this goal in any way.

@kubernetes/sig-docs-en-reviews for additional reviews

Closing this PR as the reviewers are in agreement that Azure CNI should not be removed
/close


how to display BoundingBox3DArray in noetic rviz?

Please see https://github.com/Kukanani/vision_msgs_visualization


Add name parameter to GardenerConfigInput

Description
Changes proposed in this pull request:

ShootCR name is no longer generated in Provisioner, it is now moved to parameter in GardenerConfigInput

Related issue(s)

#309

/retest

/test pre-master-control-plane-gke-provisioner-integration

/test pre-master-control-plane-gke-provisioner-integration


Change app_domain for aws staging to staging.govuk.digital

config in hieradata_aws/staging.yaml still pointed to staging.publishing.service.gov.uk and is causing the
govuk_crawler_worker to get confused.

Furthermore, the targets of crawler as they are defined in hieradata_aws/common.yaml.
Changed this now to prevent the crawler from timing out trying to access Carrenza.

Solo: @schmie

I don't think it should time out trying to access Carrenza, they're all public-over-the-Internet URLs?

Hi @issyl0, both assets urls (see below) do result in a 404 from my workstation. I would assume that is the same from awd->carrenza internal?!
https://assets.digital.cabinet-office.gov.uk/
https://assets.publishing.service.gov.uk/


Add versioning to ETH exporter

Add a version to the output data of the ETH exporter.

We have other ideas on how to recompute metrics around S3 data lake


[Autocomplete] Fix option height border-box

fixes #18978

[x] I have followed (at least) the PR section of the contributing guide.

@MariyaVdovenko Thank you
@gespispace If you can provide a reproduction once we release the change, it would be perfect.


Update Releasenote

Checklist

[ ] At least one test case is included for this feature or bug fix.
[ ] ReleaseNotes has added or is not needed.
[ ] This PR has passed all of the test cases on Nodejs and Chrome Browser.
[ ] This PR has passed all of the test cases on QT/QML.
[ ] This is an API breaking change.
[ ] Requires a major version change.

Issue Resolved / Feature Added
Update release note for PR https://github.com/iLib-js/iLib/pull/220
Resolution
Links

oh. , you are right. I also forgot to add them. I updated it too. Please review this again.


Aggregate by family

Currently the web page reports raw per file data, it might be better to aggregates averages per family and then offer drill down for per file info

Will this be in scope for current contract?

Yes, I am working on this right now. This update will be part of the current contract.

Feature added with 1fac030


Tool to set environment variables when ssh'ing.

Should be configurable per host. E.g. might want to set TERM=xterm for some (old) hosts.

This is an over-engineered idea... closing.


AUDIO_SESSION_MICROPHONE not work

audio_session: ^0.1.8
Here is what I added：
target 'Runner' do
  use_frameworks!
  use_modular_headers!

  flutter_install_all_ios_pods __dir__
end

post_install do |installer|
  installer.pods_project.targets.each do |target|
    flutter_additional_ios_build_settings(target)

    # ADD THE NEXT SECTION
    target.build_configurations.each do |config|
      config.build_settings['GCC_PREPROCESSOR_DEFINITIONS'] ||= [
        '$(inherited)',
        'AUDIO_SESSION_MICROPHONE=0',
        'DISABLE_PUSH_NOTIFICATIONS=1'
      ]
    end
  end
end

but  appconnect reply：
ITMS-90683: Missing Purpose String in Info.plist - Your app‘s code references one or more APIs that access sensitive user data. The app‘s Info.plist file should contain a NSMicrophoneUsageDescription key with a user-facing purpose string explaining clearly and completely why your app needs the data. If you're using external libraries or SDKs, they may reference APIs that require a purpose string. While your app might not use these APIs, a purpose string is still required. For details, visit: https://developer.apple.com/documentation/uikit/protecting_the_user_s_privacy/requesting_access_to_protected_resources

The added section looks correct to me. I'm not sure exactly what may be causing this for you, although I do notice that
  flutter_install_all_ios_pods __dir__

is different from my example:
  flutter_install_all_ios_pods File.dirname(File.realpath(__FILE__))

Not sure if that makes any difference.
One other thing that may be useful is others have reported that even without adding the above code, but instead adding the missing key to Info.plist to describe your microphone usage (even though you don't use the microphone you can add a description like "This app does not use the microphone"), they have had their accepts accepted and the user will never see that description because your app will never actually use any of the microphone APIs.

Uncomment this line to define a global platform for your project
platform :ios, '9.0'
source 'https://github.com/CocoaPods/Specs.git'
CocoaPods analytics sends network stats synchronously affecting flutter build latency.
ENV['COCOAPODS_DISABLE_STATS'] = 'true'
project 'Runner', {
'Debug' => :debug,
'Profile' => :release,
'Release' => :release
}
def flutter_root
generated_xcode_build_settings_path = File.expand_path(File.join('..', 'Flutter', 'Generated.xcconfig'), FILE)
unless File.exist?(generated_xcode_build_settings_path)
raise "#{generated_xcode_build_settings_path} must exist. If you're running pod install manually, make sure flutter pub get is executed first"
end
File.foreach(generated_xcode_build_settings_path) do |line|
matches = line.match(/FLUTTER_ROOT=(.*)/)
return matches[1].strip if matches
end
raise "FLUTTER_ROOT not found in #{generated_xcode_build_settings_path}. Try deleting Generated.xcconfig, then run flutter pub get"
end
require File.expand_path(File.join('packages', 'flutter_tools', 'bin', 'podhelper'), flutter_root)
flutter_ios_podfile_setup
target 'Runner' do
use_frameworks!
use_modular_headers!
flutter_install_all_ios_pods dir
end
post_install do |installer|
installer.pods_project.targets.each do |target|
flutter_additional_ios_build_settings(target)
target.build_configurations.each do |config|
  config.build_settings.delete 'IPHONEOS_DEPLOYMENT_TARGET'
  config.build_settings['GCC_PREPROCESSOR_DEFINITIONS'] ||= [
    '$(inherited)',
    'AUDIO_SESSION_MICROPHONE=0',
    'DISABLE_PUSH_NOTIFICATIONS=1'
  ]
end

end
end
The above content is automatically generated by the flutter tools
After many attempts, apple still replied to the same content
Using the vlidate app provided with Xcode, no problems were reported：
App "xxx" successfully validated.
Your apo successfullv passed all validation checks.

Version 13.3 (13E113)
pod --version
1.11.2

We may need to wait until there is more than one report of this to know whether it is an audio_session bug. Until then, my previous advice should apply.

grep -r recordPermission
Binary file ./Payload/xxx.app/Frameworks/audio_session.framework/audio_session matches
建议：
 else if ([@"getRecordPermission" isEqualToString:call.method]) {
        [self getRecordPermission:args result:result];

把这些也全都加上条件编译。
苹果的扫描可能是直接字符串匹配，即使我们用的是苹果系统的API方法已经加了条件编译，但是可能我们自定义方法，但是苹果可能不那么认为。
- (void)getRecordPermission:(NSArray *)args result:(FlutterResult)result {
#if AUDIO_SESSION_MICROPHONE
    result([self recordPermissionToFlutter:[[AVAudioSession sharedInstance] recordPermission]]);
#else
    result(FlutterMethodNotImplemented);
#endif
}

I had to google translate that but are you saying that maybe Apple is doing simple string matching and incorrectly picking up the name of one of the plugin's own methods? Feel free to experiment by renaming the method if that's the case, I would be interested to hear if that works for you.

I had to google translate that but are you saying that maybe Apple is doing simple string matching and incorrectly picking up the name of one of the plugin's own methods? Feel free to experiment by renaming the method if that's the case, I would be interested to hear if that works for you.

It's just a guess, and I'm not sure. I'll try it next time.
It has been proposed today, using the method of adding NSMicrophoneUsageDescription to Ingo.plist.
Thank you anyway. Your plug-in works well and improves our work efficiency.

The fix was introduced in v0.1.3 and verified as working. Can you report whether the bug is present if you downgrade to that version?

It started working after I added
        'PERMISSION_MICROPHONE=0'

to Podfile also. Didn't check v0.1.3, but I think I'll go with this Podfile now as it's working w/o adding key to plist:
post_install do |installer|
  installer.pods_project.targets.each do |target|
    flutter_additional_ios_build_settings(target)

    target.build_configurations.each do |config|
      config.build_settings['GCC_PREPROCESSOR_DEFINITIONS'] ||= [
        '$(inherited)',
        'AUDIO_SESSION_MICROPHONE=0',
        'PERMISSION_MICROPHONE=0'
      ]
    end
  end
end

Maybe you are using some other plugin that gave you separate instructions to add PERMISSION_MICROPHONE?
@hc2088 is that also the case for you? Is this issue because you're also using another plugin that requires you to disable its microphone code?

PERMISSION_MICROPHONE

Unlike my case, There is no error of cocoapods. I didn't use PERMISSION_MICROPHONE .  What I encountered was that after uploading apple, the IPA package failed to pass the verification. My later solution was in info Plist add "Privacy - Microphone Usage Description"

As far as I can see, it could certainly be the same in your case.

There is no error of cocoapods.

That is the same in both cases.

I didn't use PERMISSION_MICROPHONE

That is also consistent in both cases.  In other words, maybe your app is using another plugin that has its own requirements for what you must add to the Podfile, and so you can't get it to work by just following the requirements of one plugin, you must follow the requirements of all plugins that you use. That is why in @AlekseiAfanasev 's case both AUDIO_SESSION_MICROPHONE was needed (a requirement for audio_session), and PERMISSION_MICROPHONE was also needed (a requirement for some other plugin that @AlekseiAfanasev was using in the same app). If you look at your dependencies and are unsure which one it might be that is causing this, you could share your dependencies below and see if any suspects stand out to me.


No controller input

My controller is detected by windows, passes input to other apps (like Cemu), yet even when java edition was in focus, the controller did not work.
Legacy4J version: 1.4.3b
Controller: DualShock 4 (connected via bluetooth)
here is a video showcasing that issue:
https://www.youtube.com/watch?v=XBQE_t-wLZY

When you connect it after opening the game, does the controller detected toast appear?

When you connect it after opening the game, does the controller detected toast appear?

yes it does appear

Have you tried connecting with a cable? It seems to be working with him, at least

Have you tried connecting with a cable? It seems to be working with him, at least

st the moment i dont have a microusb cable that actually works (it can charge my controller if plugged into the wall, but when connecting to my laptop it does nothing at all), so i cant really test that right now. if that is sufficient to close the issue, then im fine with that

I've another DS4 controller and it keeps happening too


Login controller

Create login endpoint

Hello, I'd like to work on this.

Okah

Do I also need to create a skeleton backend myself? I saw #1 but it seems there's no progress yet on main branch.
And if so, what technologies should I use?

@kofta999 can you work on the project setup before this issue #1

Sure!

Comment in that issue, let me assign it to you


Replace Travis with Github actions

@bors r+

:pushpin: Commit 04b562765d15ef51756853afe2aa5f05bdebed52 has been approved by Amanieu

:hourglass: Testing commit 04b562765d15ef51756853afe2aa5f05bdebed52 with merge a66b4c31e94301ac97bb9025d3ebb38b1cc8adfa...

:boom: Test timed out

@bors r+

:pushpin: Commit e796274587b38bff232dedbe2ea707e1d9146b74 has been approved by Amanieu

:hourglass: Testing commit e796274587b38bff232dedbe2ea707e1d9146b74 with merge a513f60399005556d2e253df46e7013397821598...

:pushpin: Commit bb898b9c501f4dd180a3b6b1502c29534b0c1959 has been approved by Amanieu

:hourglass: Testing commit bb898b9c501f4dd180a3b6b1502c29534b0c1959 with merge 12da8a06ef99426537c8c8b8a6cd71428a6a8aad...

:sunny: Test successful - checks-actions
Approved by: Amanieu
Pushing 12da8a06ef99426537c8c8b8a6cd71428a6a8aad to master...


sozu: 0.15.3 -> 0.15.4

Description of changes
This PR updates Sozu to its latest version: https://github.com/sozu-proxy/sozu/releases/tag/0.15.4
Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

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

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

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

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 255284 run on x86_64-linux 1

1 package built:

sozu


Will you merge "sign in with apple"?

Based on https://github.com/ananay/passport-apple

Sure. Maybe you can make a PR?

https://github.com/lipp/login-with/pull/77


features extraction

How does the service extract features to rank the articles? This documentation says title may be used to generate features. The API says other cognitive services such as entity linking, video analysis are used to extract features automatically but the question is from what source? Is it from the reading the content at the given URL?

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

ID: 8639b1b2-a841-de45-1e43-f12881932a6e
Version Independent ID: 54e50295-cff1-881d-44fd-2efdef770dd0
Content: API - Azure Cognitive Services
Content Source: articles/cognitive-services/custom-decision-service/custom-decision-service-api-reference.md
Service: cognitive-services
GitHub Login: @slivkins
Microsoft Alias: slivkins

@trungngv Thanks for the feedback! I have assigned the issue to the content author to investigate further and update the document as appropriate.
@slivkins Hi, this is a document question, the customer need more accurate details, can you please check it and update the document as necessary? Thanks a lot!

Custom Decision Service is a lab. It is not a supported preview or GA service.
#please-close