Template project: update all npm dependencies

This fixes the one vulnerability reported by GH.
There are still two occurrences of this other vulnerability reported by npm audit.

Guys, I think this change is fairly uncontroversial :)

Thanks @tripu


[fix] #3657: Split view changes from normal messages

Description

Linked issue

Closes #{issue_number}

Benefits

Checklist

[x] I've read CONTRIBUTING.md
[x] I've used the standard signed-off commit format (or will squash just before merging)
[x] All applicable CI checks pass (or I promised to make them pass later)
[ ] (optional) I've written unit tests for the code changes
[x] I replied to all comments after code review, marking all implemented changes with thumbs up

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

5 of 84 (5.95%)  changed or added relevant lines in 7 files are covered.
6982 unchanged lines in 129 files lost coverage.
Overall coverage decreased (-3.1%) to 56.326%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

smart_contract/src/lib.rs
0
1
0.0%

core/src/query/store.rs
3
6
50.0%

core/src/block_sync.rs
0
5
0.0%

core/src/sumeragi/message.rs
0
5
0.0%

core/src/sumeragi/mod.rs
0
20
0.0%

core/src/sumeragi/main_loop.rs
0
45
0.0%

Files with Coverage Reduction
New Missed Lines
%

config/base/derive/src/view.rs
1
99.37%

config/src/block_sync.rs
1
95.0%

config/src/network.rs
1
93.75%

config/src/torii.rs
1
95.45%

config/src/wasm.rs
1
87.5%

core/src/smartcontracts/isi/block.rs
1
87.5%

config/src/kura.rs
2
79.41%

config/src/lib.rs
2
0.0%

ffi/src/option.rs
2
71.43%

config/src/genesis.rs
3
72.92%

Totals

Change from base Build<PHONE_NUMBER>:
-3.1%

Covered Lines:
23025

Relevant Lines:
40878

💛  - Coveralls

please change git commit message to [refactor]


College CGPA Calculator Python

I'd create a CGPA Calculator in python where the user would be able to see their CGPA out of 10 based on the individual pointer grades they obtained for each individual course.

Expected behavior

User can enter their courses and grades
Entry can be done either manually or in CSV format
The script would calculate their CGPA and display it to the users

Do not make more than 5 issues at a time


Teckin SP22 not flashable anymore with update 1.0.7

I'm hereby reporting another device that can not be OTA-flashed anymore after the newest Tuya update.
Device: Teckin SP22
WIFI Firmware v1.0.7
MCU Firmware v1.0.7
Tuya-convert is running on a RPi3 with a fresh install of Raspbian. An Android smartphone is connected for Smartconfig to work properly.
With the newest update applied, the device briefly establishes a connection with the vtrust-flash WiFi network broadcasted by my RPi3 but then immediately disassociates.
I have attached the four log files from tuya-connect as well as a pcap file containing all traffic captured on wlan0 during the process. The pcap file and the log files are from the same flashing attempt. The pcap is zipped since Github does not allow uploading pcaps.
Summary of the information extracted from logs and pcap as far as I can understand it:
SP22 is referenced as Espressi_c2:eb:b5. Its full MAC address is 60:01:94:C2:EB:B5.
My phone's (OnePlus 6) MAC address is 64:A2:F9:82:FE:F3.

the interface wlan0 is created and vtrust-flash is broadcasted
the RPi3 has IP <IP_ADDRESS>
the phone associates, gets IP <IP_ADDRESS>
the Espressi_c2:eb:b5 broadcasts LLC
4-Way-Handshake between RPi3 and Espressi_c2:eb:b5 is completed in frames 1537-1540
Espressi_c2:eb:b5 is assigned IP <IP_ADDRESS>
<IP_ADDRESS> starts DNS request for a3.tuyaus.com in frame 1566 (0xa80b A a3.tuyaus.com)
<IP_ADDRESS> responds
<IP_ADDRESS> tries to send TCP packet from port 56662 to 443 (SSL) in frame 1568
<IP_ADDRESS> starts another DNS request for a3.tuyaus.com in frame 1597 (0x6d1d A a3.tuyaus.com)
<IP_ADDRESS> responds again
<IP_ADDRESS> tries to send another TCP packet from port 29202 to 443 (SSL) in frame 1599
One more ARP reply by Espressi_c2:eb:b5 is recored in frame 1602
Afterwards: silence. No Espressi_c2:eb:b5 is to be seen anymore.

I followed the instructions bit by bit and in my opinion the dnsmasq blocking is working. When I try to visit a3.tuyaus.com from the connected phone, I'm just greeted with dnsmasq's hello world page. This seems to prove the theory that Tuya devices are now trying to communicate via SSL and are disassociating immediately when an invalid packet is received.
If more information is required I'm happy to deliver. Sadly I'm not able to solder to the ESP8266 since the case is glued.
smarthack-mqtt.log
smarthack-smartconfig.log
smarthack-web.log
smarthack-wifi.log
capture.zip

I have just ordered another fresh batch of six SP22s. (For everyone looking to buy some: get the three pack on Amazon DE right now. They're 8€ a pop with a 10€ coupon code that is offered directly on the product page until 21st of May) When they arrive I'll try to flash them with Tuya Convert directly out of the box using only devices that are not connected to the internet in any way to prevent any firmware upgrades or successfull DNS requests.
I'm expecting to get devices with a firmware version that is still flashable. I'll still capture all network traffic generated in the procedure and provide it + logs as above.
I'll also provide the backed up bin files and maybe I'll sacrifice one of the already updated plugs to see if I can extract the newer firmware.

I have received exactly this thee pack from Amazon DE today and have flashed them semi-successfully before they had any chance to connect to the Internet. I have backups of the original firmware in case anyone needs it.

I've foolishly made the mistake of connecting them to the app first to see if they work. All four I had... Obviously they got the update immediately.
We should probably both update this issue as we gather new information since finding someone with the same hardware seems to be quite rare.

I've foolishly made the mistake of connecting them to the app first to see if they work. All four I had... Obviously they got the update immediately.

Hence, I hope https://github.com/ct-Open-Source/tuya-convert/pull/198 will get merged.

I have received the new batch of SP22s this morning and am happy to report that flashing them out of the box with the provided Tasmota 6.5.0 bin works like a charm. Afterwards I connected them to my Wifi and configured them for the Teckin (52) template. Power meter and switching works, only the reported voltage has an offset.
Attached you'll find log files, the network traffic captured on wlan0 during the flash and the original bin files.
I was not able to pry open one of the devices with the new Tuya firmware. They're so well glued together that the ESP8266 would be destroyed trying to open them. If at all, they can be opened from the back. But there's a lip in the injection mold that stops one from inserting any prying tools further than 1mm. So the only option would be sawing them open, which I don't have the appropriate tools for.
device-info-6f77.txt
smarthack-mqtt.log
smarthack-smartconfig.log
smarthack-web.log
smarthack-wifi.log
pcap and bin.zip

Do the red and blue LEDs behave correctly for your SP22s with the Teckin (52) template @sebastianklein96?

@sebastianklein96
Step by Step Instructions for Noobs? 😊

Do the red and blue LEDs behave correctly for your SP22s with the Teckin (52) template @sebastianklein96?

Yes, they do. I didn't change anything except for calibrating the power measurements and it works like a charm. Did you flash the Tasmota bin that was delivered with tuya-convert? Tuya-convert ships with Sonoff Basic. Maybe replace the sonoff-basic.bin in ~/tuya-convert/files/ with the sonoff.bin from the newest Tasmota release.

@sebastianklein96
Step by Step Instructions for Noobs? 😊

Well the instructions are already in the README of this project. I set up a fresh install of Raspbian on my RPi3, cloned the repository, installed the dependencies using the install script.
Afterwards I ran the ./start_flash script, connected a laptop to the vtrust-flash wifi network, plugged in the SP22, waited about five seconds for it to boot and then held the button for about 7 seconds until the led started to flash blue. Then I hit enter in the flash script. Took about 10 seconds until the plug was recognized and the firmware dump began. Afterwards I just did the curl http://<IP_ADDRESS>/flash2 and curl http://<IP_ADDRESS>/flash3 . A new wifi popped up from the plug and I connected to it with the laptop, navigated to <IP_ADDRESS> and put in my wifi information.

@sebastianklein96 it is all working now for me; I had not entered my MQTT server details yet and this is why it was flashing blue. Now the LEDs are working properly.

@sebastianklein96
Whats the different between this way and the normal way of tuya convert flash?

@sebastianklein96 it is all working now for me; I had not entered my MQTT server details yet and this is why it was flashing blue. Now the LEDs are working properly.

@probonopd Glad to hear that!

Whats the different between this way and the normal way of tuya convert flash?
I bought a Digoo Plug . But the WiFi Version is 1.1.1
So the normal way of Flashing with Tuya Convert fails.

@haselchen I literally followed the instructions provided in the Tuya Convert Readme. The description that I gave earlier is just me following those instructions.
In which way does the flash of your Digoo plug fail? Did you update its firmware or connect it to the internet before attempting the flash?

I wanted to flash the plug as always. Step by step following the instructions as always.
The leds are blinking , ready for flashing. So i hit Enter.  After ca. 3seconds. the Plug makes "klick" and the leds are off. Plug is back to normal mode.
I think it is because of the new firmware. (WIFI 1.1.1)
Every owner of a tuya device describes the current failure during flashing.
Why does it work for you with the firmware over 1.0.5?
(Can we also treat the problem in German?)

It only works if you do the flash directly with factory-new devices that have never accessed the Internet or the app, because they update themselves to a new firmware from the factory after which they can no longer be flashed with tuya-convert.
Es funktioniert nur direkt mit fabrikneuen Geräten, die noch nie mit dem Internet oder der App verbunden waren, weil sie sich sonst von selbst auf eine neue Firmware des Herstellers aktualisieren, mit der sie nicht mehr mit tuya-convert geflasht werden können.

As probonopd said, it only works if the plugs didn't have any chance to receive an update. The devices I used for updating all did not have an internet connection and therefor the plugs could be flashed since they had no way to update.
-German-
Wie Probonopd es gesagt hat.  Ich tippe mal, dass der dnsmasq Filter, der mit Tuya Convert ausgeliefert wird, nicht mehr greift, weil die Geräte jetzt über SSL kommunizieren. Auch, wenn der Tipp unethisch ist: falls du den Stecker bei Amazon gekauft hast, melde eine Retoure an und kauf den selben Stecker noch mal. Vielleicht hast du Glück und der neue Stecker kommt mit der alten Firmware. Dann kannst du mit Geräten ohne Internetzugang flashen.

I bought the double socket at Banggood. I think that only devices with new firmware are delivered.
I've connected it to the app now.
In your opinion, is there a possibility that despite connecting to Tuya, a flash (if a solution was found) is still possible?
-German-
Hab die Doppelsteckdose bei Banggood gekauft. Ich denke, dass nur noch Geräte mit neuer Firmware ausgeliefert werden. Weil ich die Dose benutzen will, habe ich sie natürlich jetzt mit der App verbunden.

With the current version of Tuya Convert, I don't think it's possible. The developers are already looking into ways to circumvent the new security measures though. But I don't have any insight on how long that will take. It's probably gonna be a while since they literally have to emulate Tuya's SSL responses as far as I understand. I think your safest bet would be to get rid of the plugs you have now if that is still possible (don't know banggoods return policy).
-German-
Mit der aktuellen Version von Tuya Convert wird das wohl nichts.  Keine Ahnung, ob und wann das was wird. Ich würde aber sagen, dass es nen Moment dauern wird, SSL-Verschlüsselung zu reverse engineeren. Wenn du kannst, sieh zu dass du die Stecker wieder los wirst und dir Ersatz besorgst, von dem du weißt dass er flashbar ist oder den du einfach zurück geben kannst.

I have received 4 x SP111 Go-sund sockets.  The sockets havent connected to the internet yet. Alltough i have succefully flashed 1 device, the other 3 are not working :(
Same as @haselchen  (The leds are blinking , ready for flashing. So i hit Enter. After ca. 3seconds. the Plug makes "klick" and the leds are off. Plug is back to normal mode.)
I don't know how one 1 plug was flashed succes, the other three are not working.

Can't flash Teckin FL41 either. Same issue

Is there a way to check the firmware without using the app?

@eterpstra Asking the same question in three seperate issues isn't exactly good etiquette.
Answer:
https://github.com/ct-Open-Source/tuya-convert/issues/160#issuecomment-494515604

Same issue with brand new Teckin SP27's from Amazon UK, I can only assume they are now coming from the factory patched?

I just bought 4 teckin sp22 and i can't flash them. I didn't connected them to the internet, I just plug them in and when tuya converter is searching (. . . . .) the blinking led of the plug stops blinking. Shoud I return them (Amazon) or do u thing this will be solved soon (less than 1 month)?

@Xan8 just keep an eye on this repository and see if some development is done. If not, just return them before the end of your return period.

@sebastianklein96 But soldering is still possible right?

@eterpstra I don't know anything about that. But if I had to guess, I'd say it should still be possible since I don't know of a way to "write-protect" an esp8266's flash memory. Just guessing though.

Any way to set up an alert so that we would know when a newer version of Tuya convert is uploaded? I have a bricked Teckin SP22 at the moment, but don't want to crack it open due to it being used with mains power etc... (don't feel comfortable trying to seal and glue it all back together afterwards). This one was upgraded accidentally to the latest firmware, and now nothing recognises it.

@baldfox You can click the "Watch" button at the top of the page to get notifications about development activity. You can also try codetheweb/tuyapi if your goal is local control, next best thing to custom firmware.

@baldfox You can click the "Watch" button at the top of the page to get notifications about development activity. You can also try codetheweb/tuyapi if your goal is local control, next best thing to custom firmware.

@kueblc Thanks a lot. I've never really used github before. Appreciate your help!

@baldfox,
Please take look at the solution I am working on, it provides the virtually tasmota features on tuya devices using tuyapi with flashing it. Virtual Tasmota for Tuya Api Devices.

Thanks
Thiru

With the current version of Tuya Convert, I don't think it's possible. The developers are already looking into ways to circumvent the new security measures though. But I don't have any insight on how long that will take. It's probably gonna be a while since they literally have to emulate Tuya's SSL responses as far as I understand. I think your safest bet would be to get rid of the plugs you have now if that is still possible (don't know banggoods return policy).

Would it be possible to implement sslstrip in to the tuya-convert software to strip out the https transactions and downgrade it to http? https://github.com/moxie0/sslstrip

@baldfox,
Please take look at the solution I am working on, it provides the virtually tasmota features on tuya devices using tuyapi with flashing it. Virtual Tasmota for Tuya Api Devices.

Thanks
Thiru

Thanks for this. I looked at it. I really don't want to keep the tuya/smartlife firmware on there though at the moment. I don't want any of my smarthome products phoning home. I guess I'll need to wait for now.

I thought I'd have another attempt at this. If i hold down the teckin sp22 for 5 seconds it blinks rapidly. If I then hold the button again for a further 7 seconds, it flashes blue slowly. It will then throw up an SSID of smartlife-XXXX. I then try and connect to that but I can't get any further. Anyone know the default webpage to try and administer it again? At this junction I think i'd rather just revert the SP22 to default and try and use it with Tuya until the SSL is broken and I can go OTA again.

I don't want any of my smarthome products phoning home

You can try blocking *.tuya(us|eu|cn).com at your router and use codetheweb/tuyapi to control your devices locally.

If i hold down the teckin sp22 for 5 seconds it blinks rapidly. If I then hold the button again for a further 7 seconds, it flashes blue slowly. It will then throw up an SSID of smartlife-XXXX.

tuya-convert only works with the first mode, known as EZ config. It does not work with AP config.

tuya-convert only works with the first mode, known as EZ config. It does not work with AP config.

Thanks for that. It saves me some time. Unfortunately in AP mode the snart-life app doesn't conclude it's registration so it's well and truly fubarred I think.

Hi,
I just bought some TECKIN SP22 from Amazon. Thought it would work out.
Did a fresh Rasperry Pi Setup and got stock here
file:///home/tim/Bilder/Screenshots/Screenshot_20190828_100242.png
It seems that I cant flash them.
Is there a workaround/ a solution already out?
Will there be a solution in the future?
Should I just send them back?
What would you guys do?

I had the same issue. The only thing I can suggest is that we all write to Teckin and ask them for a work around. If there are enough of us it might work. We can highlight the success of Shelly and their approach to open source software as an example of how well they could do.

as of now we have only few alternative options ,

use tuya-api based control without flashing device with tasmota firware (Refer here or here  )

use soldering iron to hard wire flashing with tasmota (You might need to put MCU in reset mode along with putting the esp in flashing mode to avoid the serial data interruptions from MCU mostly for dimmers)

use shelly or sonoff diy devices for easy flashing support

Thanks
Thiru

@timbo-lino,
as of now we have only few alternatives ,
1. use tuya-api based control without flashing device with tasmota firware ([Refer here](https://github.com/thirug010/Virtual-Tasmota-for-Tuya-api-devices/blob/master/README.md) or [here ](https://github.com/codetheweb/tuyapi) )

2. use soldering iron to hard wire flashing with tasmota _(You might need to put MCU in reset mode along with putting the esp in flashing mode to avoid the serial data interruptions from MCU mostly for dimmers)_

3. use shelly or sonoff diy devices for easy flashing support

Thanks
Thiru

Thanks for pointing out all the options.
Tuya-api based control seems to be an overkill for me, because as far as I see, I need to run a service 24/7 to use it.
I think the SP22`s are going back to Amazon and I will replace it with a shelly or sonoff.

TECKIN SP22 Users check: https://github.com/ct-Open-Source/tuya-convert/issues/273
I just sucessfully flashed four SP22 Plugs

Happy to share that support for HTTPS firmware is in the works #279


casper client rs deployhash is conflicting with node's deployhash

running into an issue with the casper-client-rs. During development for checkpoint charlie I ran into type conflicts between the nodes DeployHash and clients DeployHash.

The immediate issue that generated this request was resolved but is this relevant otherwise?  Seems like we "should" be keeping the rust client up to date with the node????

I think this can get closed.  The client is generally kept up to date with the node.  The main branch works with the latest published version of the node, while the dev branch works with the node's dev branch.


Markdown Edge Cases

Sometimes Markdown isn't rendered correctly, for instance:
<Secret/>
Definition:
Function: A function is a self-contained block of code that performs a specific task and can be defined independently of any class or object. It takes input arguments, processes them, and returns a result.
Method: A method is a function that is associated with an object or class. It is called on an object and can access the data within that object.
Usage:
Function: Functions are standalone entities and can be called from anywhere in the code, provided they are in the scope.
Method: Methods are associated with objects or classes and are called on instances of those objects or the class itself.
Syntax:
Function: Functions are defined using the def keyword and can be called using the function name.
Method: Methods also use the def keyword but are defined within a class and are accessed using dot notation (object.method() or class.method()).
Here's a simple example in Python to illustrate the differences:
    # function
    def add(a, b):
        return a + b

    # method
    class Math:
        def add(self, a, b):
            return a + b

    # function call
    add(1, 2)

    # method call
    Math().add(1, 2)

</Secret>

Here's another:


spotify playlist doesnt exist anymore or something?

System OS
MacOS
Python Version
3.9 (CPython)
Install Source
pip / PyPi
Install version / commit hash
4.1.4
Expected Behavior vs Actual Behavior
was supposed to download some songs, but says it cant find my playlist, might have been a new update or something
Steps to reproduce - Ensure to include actual links!
open spotdl
use this link: https://open.spotify.com/playlist/03jfT6tQLwhT29gclp8nX2?uid=a10a8e758e8a835e&uri=spotify%3Atrack%3A2uufSlJp3cSuv6hQXOs05B
watch the errors
Traceback
Last login: Mon Apr 17 11:58:48 on ttys001
bw26168@C02FD2PQQ6L3 ~ % cd desktop/"to upload to psp"/relaxo
bw26168@C02FD2PQQ6L3 relaxo % spotdl https://open.spotify.com/playlist/03jfT6tQLwhT29gclp8nX2?uid=a10a8e758e8a835e&uri=spotify%3Atrack%3A2uufSlJp3cSuv6hQXOs05B
[1] 24672
zsh: no matches found: https://open.spotify.com/playlist/03jfT6tQLwhT29gclp8nX2?uid=a10a8e758e8a835e
[1]  + exit 1     spotdl
bw26168@C02FD2PQQ6L3 relaxo %

Other details
it could be due to miniconda but i did disable it and it still had the same problem. (ps, i'm still pretty new to terminal stuff so i may have just made a dumb mistake) (also the playlist i linked is public so spotdl shouldnt have any problems finding it)

don't know what i did that fixed it but reinstalling it and ffmpeg fixed the problem


ConnectingFailedHandler doesnt follow the same approach?

Describe the bug
No extension methods to use additional client delegates like ConnectingFailedHandler
Which project is your bug related to?

Client
ManagedClient

To Reproduce
Steps to reproduce the behavior:

Create a client or ManagedClient and attempt to use the additional handlers by the apparent new format of UseConnectingFailedHandler, which doesn't exist.
See that other exposed handler methods also dont have accessors like ApplicationMessageProcessed, SKipped, etc.

Expected behavior
In Pre 3.0 you could bind a handler to the ConnectionFailed event to log additional context, but I can't find how to do that now.

In version 4 all these events are now proper .net events supporting async tasks. Please test with latest version 4.


Avoiding PyTorch dependency

The executables (main app, comparator app) that will be distributed to the players will contain MKL and PyTorch dlls (and possibly Anaconda). However, it should be possible to have estimators.py use numpy instead of torch? Or keep the app performance while not relying on NNs.

To not use PyTorch, set use_torch in the LIBRARY section of the configs.ini file(s).
PyTorch is now excluded from the required dependencies when installing the PCGSEPy library if the above flag is set in the configs.ini file at the same level of the setup.py.
For now it is set to False by default as there are no noticeable differences in performances and reduces the executables filesize drastically.

Awesome, that's a huge improvement, Roberto 👍


.eslintrc.json is outdated

When following part 4 of the step-by-step guide, Set Up a Lint Check, I'm getting the following error when running npx eslint webapp:
Oops! Something went wrong! :(

ESLint: 9.8.0

ESLint couldn't find an eslint.config.(js|mjs|cjs) file.

From ESLint v9.0.0, the default configuration file is now eslint.config.js.
If you are using a .eslintrc.* file, please follow the migration guide
to update your configuration file to the new format:

https://eslint.org/docs/latest/use/configure/migration-guide

If you still have problems after following the migration guide, please stop by
https://eslint.org/chat/help to chat with the team.

Running the configuration migrator in the migration guide leads to another error:
  0:0  error  Parsing error: "parserOptions.project" has been provided for @typescript-eslint/parser.
The file was not found in any of the provided project(s): webapp/Component.js

Could you update the documentation and code to take into account these changes to ESLint?
Environment information:
npm --version: 10.8.2
node --version: v22.6.0

Thanks, yes, we should do this. The sample code requires ESLint 8, but the instructions will now result in version 9.

@akudev - we can adopt it to version 9 - I will also adopt the templates by this week.

@ZZzzaaKK - ESLint has been updated to version 9 in the main and testing branch.


How can Services running in a Docker VM and running on localhost register at EUREKA dynamically?

Hello everyone,
I have a problem with my Eureka Configuration. Normally all my services are running as docker containers in a VM. My configuration looks as follows:
eureka:
  instance:
    prefer-ip-address: true
    ip-address: <IP_ADDRESS>

Sometimes I want to debug a service and start it locally as spring-boot application and it still registers with the IP above. As far as I understood, the IP in the config simply overrides the real IP. My idea to solve this problem, is a configuration profile, which overrides the Docker VM IP. However my machine IP is changing regularly. Is there a way to let the configuration of the IP change dynamically according to my machines IP.
eureka:
  instance:
    prefer-ip-address: true
    ip-address: <IP_ADDRESS>
---

spring:
  profiles: local

eureka:
  instance:
    ip-address: TODO change dynamically

If this is not yet implemented, can I implement it myself somehow? Thanks a lot! :)

Closing in favor of stack overflow question http://stackoverflow.com/questions/43044273/how-can-services-running-in-a-docker-vm-and-running-on-localhost-register-at-eur


Helm uninstall does not return real error when it failed

We use helm client but not helm cli to do helm uninstall.
https://github.com/helm/helm/blob/main/pkg/action/uninstall.go#L125
This line will not return concrete error when helm uninstall failed.  Only helm release name is not enough
Could it return the ""errors" which return by "deleteRelease"

I suspect this was written a long time ago, before Helm SDK existed. A PR to adjust this logic, and return an error type which includes those errors would be welcome I think. (note: it isn't possible to make an incompatible change to the API definition and return errs directly from the function.

👋 I'm interested in picking this up, seems like a decent beginner issue


license nit

Hi, obviously it's totally cool to pull node-tap's lib/stack.js into a standalone module, and I appreciate you doing that.
But, since this is based on that code, would you mind updating the license to include the copyright, as required by node-tap's ISC license?  Maybe something like this?
Copyright (c) James Talmage<EMAIL_ADDRESS>(github.com/jamestalmage), Isaac Z. Schlueter<EMAIL_ADDRESS>and Contributors

Extracted from code in node-tap http://www.node-tap.org/

I don't mean to be weird, and I'm definitely not upset and this is not borne out of frustration or conflict.  I'm only mentioning it because being in a new git repo means that the git history doesn't preserve the provenance of the code, and I like to maintain best practices for this sort of thing.
Changing to the MIT license is fine, it's functionally equivalent to ISC anyway.
Thanks :)

Yep - no problem. I meant to ask you how you wanted me to handle attribution.
The license is just the default from the yoeman generator I used, happy to switch to ISC if you prefer.

@isaacs - fixed the attribution. Let me know if you would prefer switching to the ISC license.

I would also happily to move this into https://github.com/tapjs if you would prefer to maintain more control over it.

Awesome, thanks!
I'd really love for you to keep maintaining it, actually, if you don't mind.  But moving it into @tapjs probably makes sense.  I'll add you to the org.

:+1: I don't mind at all!


Red Alert. Show stopping bug.

Summary: Your nginx/1.13.12 isn't sanitizing some user input, and has a command injection vulnerability.  Your bash or gnu readline is intercepting user query strings in transit, and interpreting some of them.   I suggest you take everything offline until you find and eliminate this.
Bash on the server side is interpreting my bash keywords and builtin variables like local, sh, $1, source, man, touch and others.
Reproduce:
Step 1: visit: http://cht.sh/bash/local
Step 2: get blasted in the face with a 500: Internal Server Error The server encountered an internal error and was unable to complete your request. Either the server is overloaded or there is an error in the application.
Links that fail:

http://cht.sh/sh/local  fails as I said above.
http://cht.sh/bash/man fails.
http://cht.sh/bash/sh also fails.
http://cht.sh/bash/$1 fails but 2 works fine, dollarsigns are being interpreted.  So it looks like bash doing the interpreting.

Relevant: https://xkcd.com/327
Little bobby drop tables we call him...

@sentientmachine Thank you very much, for reporting it. Actually, it should not be a bug, not an active bug at least, but that's true, that there were several cache entries that such pages contained. I've refreshed the cache. Can you please check that now everything works fine for you?

Your welcome, all the queries above are working again now.  Glad to hear it's not a bash command injection problem.

@sentientmachine Thank you very much, Eric!


Android Access denied error while using Camera

I'm getting error Access denied finding property "persist.vendor.camera.privapp.list" and access denied finding property "camera.aux.packagelist" for Android api 28 and above ie 8 and 9 android versions.Please help me.

Change
imageReader = ImageReader.newInstance(imageSize.getWidth(), imageSize.getHeight(),ImageFormat.YUV_420_888,2);`
to this
 imageReader = ImageReader.newInstance(imageSize.getWidth(), imageSize.getHeight(),ImageFormat.YUV_420_888,2);


fixunnecessary cast will be added on children in InPredicate

Proposed changes
Issue Number: close #10881
Problem Summary:
We should use equals function rather than Use equals operator to compare two Type in Analyzer#castAllToCompatibleType.
Checklist(Required)

Does it affect the original behavior: No
Has unit tests been added: No
Has document been added or modified: No
Does it need to update dependencies:No
Are there any changes that cannot be rolled back: No

Further comments

@EmmyMiao87 @morningman PTAL


commands as keybinds?

hi,
What about simply add a keybind like this:
': w enter': 'core:save'

While I understand this is not the best, it might "fill a hole" for now.
I have been using and is very effective for me. I even add :n, :p and :A.

Ha, very cool.  Luckily this isn't necessary anymore: https://github.com/lloeki/ex-mode
Well, ex-mode doesn't currently support :n, :p, or :A...  But it's very new.  How hard could it be?  :)

Right, lets close this


Warning has not been reported in some cases.

Plugin name and version
azure-maven-plugin 0.0.1
Plugin configuration in your pom.xml

Expected behavior

Should give a warning like this

If you don't proceed with the login in the browser after 5 minutes, this goal will fail.

Actual behavior

Run mvn azure:select-subscription successfully when user not login, and not give warning

Run mvn azure:login, when the browser pop up, wait 5mins without any operates, no warning tips and the browser still here.

Steps to reproduce the problem

a. Run mvn azure:select-subscription when user not login.
a. Run mvn azure:login
b. The browser pop up, do nothing, wait about 5mins.

Fix in commicrosoftazure-3341.


Update BSK with autofill 16.1.0

Task/Issue URL: https://app.asana.com/0/1209009003685195/1209009003685195
Autofill Release: https://github.com/duckduckgo/duckduckgo-autofill/releases/tag/16.1.0
BSK PR: https://github.com/duckduckgo/BrowserServicesKit/pull/1138
Description
Updates Autofill to version 16.1.0.
Autofill 16.1.0 release notes
What's Changed

[DeviceInterface] Call closeautofillparent only after openManage calls by @dbajpeyi in https://github.com/duckduckgo/duckduckgo-autofill/pull/718
[CredentialsImport] Call credentials flow call first, and then close parent by @dbajpeyi in https://github.com/duckduckgo/duckduckgo-autofill/pull/723
[FeatureToggle] Add new autofill config for partial save trigger by @dbajpeyi in https://github.com/duckduckgo/duckduckgo-autofill/pull/724

Full Changelog: https://github.com/duckduckgo/duckduckgo-autofill/compare/16.0.0...16.1.0
Steps to test
This release has been tested during autofill development. For smoke test steps see this task.

Note to the person reviewing/authoring: If you merge this without passing through partialFormSaves feature toggle, then this feature will stop working.
Please create that mapping and pass it to autofill.js before merging this, JS side expects partial_form_saves as the key.


Auth

Auth is ready for testing 🎉

[x] Sign-in with Google
[x] Sign-in with GitHub
[x] Firestore integration
[x] Account section in "Settings page"
[x] UI
[x] Manual CRUD operation
[x] Permissions
[x] History sync
[x] Testing

Preview deploy: https://deploy-preview-513--postwoman.netlify.com
Flow:

"Login with" button on top-right button
Select any one-time Auth provider (Google, GitHub)
Sign-in (OAuth id)
One time Toast prompt to "Turn on sync" or Enable from "Settings page"
Manual CRUD: Home page > Sidebar > Sync

fixes #514
fixes #507
fixes #26

wow great work liyas!
you did a wonderful job


Xcode 7: Investigate how xcscmblueprint relates to xccheckout

Xcode 7 seems to create another file, .xcscmblueprint in a workspace. Look at how it relates to what we pull out of xccheckout and whether there's a way to force Xcode Server to generate it - might make #81 much easier to implement then.

@czechboy0 would this allow us to force Xcode to generate an xccheckout file? I can't seem to get it to generate one... I have cloned using both SSH and HTTPS, cloned using both git from terminal, source tree and Xcode. (and then opening the project in Xcode). Have you got any tips for forcing one to generate? I am using cocoa pods as well so I have a workspace.

Yeah also try creating a Bot manually from Xcode, that might help. Unfortunately this is something I'm still hunting down, in Xcode 6 the file seemed to have been created right after the project was opened in Xcode. Xcode 7 however seems to be a bit lazier.

@czechboy0 I have tried creating a bot from Xcode by selection Create bot from the menu. Is there a way to get around Xcode not generating it?

Technically, yes, but it's a bit tricky. Basically there is a way to create the checkout file manually, but unfortunately that will make it impossible for you to edit Bots created in such a way.
Let's use this project as an example - in fact, you can see Buildasaur's checkout file here. So what you should do: copy this file, rename it to the name of your project and put it into your project's workspace. Then, you'll need to change the following keys to respect your project: IDESourceControlProjectName, the url in IDESourceControlProjectOriginsDictionary, IDESourceControlProjectPath, IDESourceControlProjectURL, IDESourceControlWCCName.
That should be it. What to change them to should be obvious. If you have any questions, please ask away. I hope this will help you to get Buildasaur working - and I'll keep looking for a reliable way to get this file generated by Xcode :+1:

Okay so say I edit a bot do I just need to give Buildasaur the new xccheckout?
Also it looks like Buildasaur needs to be setup as a seperate github user? Does that mean I need an organisation account?
Okay cool I will give that a try..
Thanks so much for the help @czechboy0!

Buildasaur takes a workspace/project file and it looks inside of it automatically. So now, follow the steps above and once you have that .xccheckout file edited for your project, commit it to your repo so that it stays there. Then, launch Buildasaur and point it at your workspace (which now contains the checkout file) and it should then let you proceed.
Nah, I just use a separate GitHub, but you don't have to. Just generate a personal token in GitHub settings.
No worries, let me know if you got it working! :)

Right I see, I wonder if Xcode isn't generating it because it is in my gitignore... I wasn't planning on committing it to my repo.
Okay cool, is there a way for it to not comment but rather use the github status API so I can use protected branches?

Yeah it uses both the GitHub status and posts a comment. You can disable the comments in the UI at the bottom
Yes, actually have comments tell more about the tests etc is the subject of another ticket, so any contributions are welcome :+1: Such as #49.

@czechboy0 okay cool I will look into it... just trying to get my custom xccheckout file to work at the moment.

@czechboy0 I got it to work. Is there a way to build 2 targets? For example: iOS, OS X, watchOS, tvOS?

Not at the moment, you would need separate Xcode Bots for this - because the Bot needs a scheme, which points to just one target. However, I am working on multi-project support (#12) , which would make it possible - you'd have separate syncers for your iOS target, OS X target etc.

@czechboy0 okay I wonder if I could create two instances of buildasaur. Okay cool.

@czechboy0 in regard to this actual issue after a PR bot has completed my normal bots give me a warning.
Assertion: An error occurred updating existing checkout. Falling back to a clean checkout.

@Brett-Best Could you please move this comment into a new issue? I'll take a look at it and comment there, thanks :+1:

So in #140 I added support for reading the xcscmblueprint file, which actually fixed the issue for some people who started their project on Xcode 7 (never had a xccheckout file).
To me it looks like they are equivalent, just one is xml and was generated by Xcode 6 and the other is JSON and is used by Xcode 7.
The big issue is that we're not yet able to force Xcode to generate them. Closing this issue and moving to a more specific one.

@czechboy0 yep I agree the generation of the blueprint is the issue now.


Refactor Sequence Steps

Also fixes step drag image edge bug.

Coverage increased (+0.2%) to 82.245% when pulling 2275c66bc31aa78f76cd456ae888787892fa6e1a on gabrielburnworth:staging into 6593605d9858a7f13bc265d660398fa1ceffc57f on FarmBot:staging.


Flatcar support

Motivation

Support CoreOS/Flatcar
Feature

As discussed in here https://github.com/falcosecurity/falco/issues/1075 Flatcar requires some modification to the OS to let Falco install the kernel module
Alternatives

Additional context

Discussed this with @leogr that soon driverkit becomes the way to install correct driver. I would like to contribute in this regard for the OS Flatcar. CoreOS is In EOL.
I'm going to partecipate to the community meeting today.

IIRC, as discussed during the community call the best approach should be to add a new target flatcar here.

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle rotten

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

@poiana: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@poiana: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

/reopen

/reopen

@mazzy89: Reopened this issue.

In response to this:

/reopen

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

@mazzy89: Reopened this issue.

In response to this:

/reopen

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

I will work on this in the upcoming days. Sorry it is taking a bit longer.

I will work on this in the upcoming days. Sorry it is taking a bit longer.

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

@poiana: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


set device for small zeros tensor to avoid device misalignment

fixes issue for non CUDA backends which device tensors cannot have CPU data storage.

It seems like this PR has no meaning anymore, as it modified a removed code. closing.


Native methods of UI5 aggregation items cannot be invoked for sap.m.MultiComboBox

Describe the bug
Similar to something what is described in this issue, none of the items methods can be used.
MultiComboBox has an aggregation called items of type sap.ui.core.Item.
Each Items has a property called text.
The below is my code. I'm just invoking the getText method
const multicomboBox =  await browser.asControl({
            selector: {
                id: /.*id$/,
                controlType: "sap.m.MultiComboBox",
                interaction: "root"
            },
            forceSelect: true
        })
await multiComboBox.open();

const items = await multiComboBox.getItems();

const firstItemText = await items[0].getText()

expect(firstItemText).toBeTruthy()

This doesn't work, because the native UI5 methods of sap.ui.core.Item don't seem to be available on each of the items in aggregation. This leads to a type error about missing function
To Reproduce
Invoke getText on items of items aggregation of multiComboBox
Expected behavior
The text of the item
Logs/Console Output
getText is not a function
Screenshots
Runtime Env (please complete the following information):

wdi5/wdio-ui5-service-version: 0.9.13
UI5 version: 1.96.0
wdio-version: 7.25.0
node-version: 14.17.3
OS: macOS Monterey 12.6
Browser + Version: Chrome 106.0.0

Added a test in PR.
Also note the rendered Item is type sap.m.StandardListItem -> getText() needs to be getTitle(). This is a UI5 flaw and will be documented in the wdi5 documentation.

Hi @dominikfeininger, thanks for pointing me to the correct method. (getTitle)
However, the problem persists because no UI5 method is generated for the combobox items. I have attached some screenshots that I took while debugging, please take a look.

Hello colleagues, please reopen this issue because as I mentioned in the previous message the problem still persists.

happy to reopen this, but FTR: we included a test just for this bug and its fix:
https://github.com/ui5-community/wdi5/blob/c3293971d2deff46ef36103e0c72b975dfa1d217/examples/ui5-js-app/webapp/test/e2e/prop-array.test.js#L30
...where we're doing essentially the same thing as you: open a multi combo box and operate on an aggregation item.
maybe the referenced test can help you as a blueprint, specifically the interaction: "root" as part of the combox box selector?

Hello, I tested once more and everything seems to work fine.
Thanks


Add say exercise to the ocaml track

It is on the F# track and looks to be a good intermediate exercise.
http://exercism.io/exercises/fsharp/say/readme
I'm available to do this if it helps.

@stevejb71 it would be wonderful if you could add the say exercise to the OCaml track!
Have you seen the documentation on porting an exercise to an other language?
Please reach out if you need any help.

I've submitted a PR with the exercise in. I would be interested in your feedback.

@stevejb71 Great! I will take look immediately.

Looks good. I have on suggestion that would improve readability, otherwise this is awesome.

@stevejb71 Thanks for porting the say exercise.
And welcome aboard! It is great to see you as an exercism member


chore(deps): update dependency semantic-release to v17.4.0

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

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


grpcio version error

BUG REPORT
High Level Description
pip install -e .[train] throws error due to compatibility issue
SMARTS version
0.4.16
Steps to reproduce the bug
pip install -e .[train]
Resulting and expected behavior
all the dependencies should be installed properly
Error logs and screenshots
ERROR: Could not find a version that satisfies the requirement grpcio==1.37.0 (from smarts[train])
ERROR: No matching distribution found for grpcio==1.37.0

System information

Operating system & Environment: installation on compute canada platform
Python version: 3.7.9
Python packages: grpcio
Date when bug occurred: June 14, 2021

Impact [If known]
unable to complete the library installation process
Way out to remove this error
edit setup.py and change line no. 61 to:
"grpcio==1.38.0"
this worked perfectly for me and I am able to complete the installation process.
please incorporate this change. :)

Just to clarify, I encountered this error while setting up SMARTS on compute canada

The existing commands appear to install SMARTS without any issues:
$ git checkout master
$ python3.7 -m venv .venv
$ source .venv/bin/activate
$ pip install --upgrade pip
$ pip install --no-cache-dir -r requirements.txt
$ pip install --no-cache-dir -e .[train,test,dev]

Yeah, it seems to work perfectly in my local setup, but I was trying to install it on compute Canada, and there I got this error.

Are you using Singularity for deployment in Compute Canada?

earlier I was not but currently, I am trying to use singularity for deployment on CC. There are some version issues when I try to do it by directly cloning the GitHub repo and then doing make sanity-test. I have opened this issue as well: https://github.com/huawei-noah/SMARTS/issues/939 and explained the further complications while setting up directly.

Singularity support is now provided via pull request #1016.
Consider using Singularity on ComputeCanada instead of directly setting up.
For issues on using ComputeCanada, see and dicuss in thread #963.
No updates to grpc since it is not causing any errors currently.


virtualenv support for python actions

Allow a virtualenv directory in a zip file for a python action.
This allows to package python modules installed with pip into the action zip file.
The invoker activates the virtual python environment before invoking the action.
Naming conventions :

The name of the main python module in the zip file must be : main.py
The directory name of the virtual environment must be : virtualenv

How to create a zip file with a virtual environment:

name your python file  main.py
install virtualenv:  (pip install virtualenv)
create the virtual environment named 'virtualenv': virtualenv virtualenv
activate the virtual environment: source virtualenv/bin/activate
install all required modules (e.g. pip install xxx)
deactivate the virtual environment: deactivate
zip the files: zip -r myPython.zip virtualenv main.py
create action: wsk action create actionname --kind python myPython.zip

Add a test?

works nice 👍
(minor: when I played through I saw the action must be named __main__.py instead of main.py)

good to see this.
if the zip includes native (compiled) libraries, did you verify if they will work?
we need tests, of course: please see PythonActionContainerTests and CLIPythonTests.

How do you folks feel about Python 2 vs. Python 3?  In Python 2, virtualenv is the way to go but since Python 3.3 venv has been a module in the standard library and since Python 3.5 "venv is now recommended for creating virtual environments.".
$ python3 -m venv . ; . bin/activate
I know that OpenWhisk is Python 2 currently but I have been submitting PRs to get us to compatibility with both Py2 and Py3.  I provide some of the reasons for this in https://github.com/openwhisk/openwhisk/issues/1899 and it has been suggested in https://github.com/openwhisk/openwhisk/issues/1184#issuecomment-245352118 that we only support one of the two Python versions.

@cclauss good points - we need to accelerate moving to python 3/adding support so all new changes at in the newer domain. on this note, i think once we have python 3, we can mark python 2 deprecated and eventually remove it.

The docs should be updated to show that you can just zip python files without a venv as well (the simpler case).

Simpler but less powerful...  With this technique you can bring along third party modules that are not already included in the Docker container.

Docs should not preclude the simpler use case which is easier for tire kickers.

This will make it super easy to support Python actions in the Serverless Framework plugin. Good work 👏.

I would still be interested in a discussion on the use of the builtin venv vs virtualenv now that Python 3 is the default.

@cclauss

I would still be interested in a discussion on the use of the builtin venv vs virtualenv now that Python 3 is the default.

python 2 is the default

You are correct.  I did missed the changes made in https://github.com/openwhisk/openwhisk/pull/2094  Bummer.  Given the stance of Python.org, we should reopen https://github.com/openwhisk/openwhisk/issues/1899

Playground PG1  #1358

Playground PG1 #1360

PG2/1415 for the latest revisions.

Squashed and merged as https://github.com/openwhisk/openwhisk/commit/d3f4f9591cadda84071db76f8716bb867bd36b83.

允许 zip 文件中的 virtualenv 目录用于 python 操作。
这允许将使用 pip 安装的 python 模块打包到操作 zip 文件中。 调用者在调用操作之前激活虚拟 python 环境。
命名约定：

zip 文件中主要 python 模块的名称必须是：__main__.py
虚拟环境的目录名必须是：virtualenv

如何使用虚拟环境创建 zip 文件：

命名你的python文件  __main__.py
安装 virtualenv: (pip install virtualenv)
创建名为“virtualenv”的虚拟环境：virtualenv virtualenv
激活虚拟环境：source virtualenv/bin/activate
安装所有必需的模块（例如 pip install xxx）
停用虚拟环境：deactivate
压缩文件：zip -r myPython.zip virtualenv__main__.py
创建操作：wsk action create actionname --kind python myPython.zip

well

Allow a virtualenv directory in a zip file for a python action.
This allows to package python modules installed with pip into the action zip file. The invoker activates the virtual python environment before invoking the action.
Naming conventions :

The name of the main python module in the zip file must be : __main__.py
The directory name of the virtual environment must be : virtualenv

How to create a zip file with a virtual environment:

name your python file  __main__.py
install virtualenv:  (pip install virtualenv)
create the virtual environment named 'virtualenv': virtualenv virtualenv
activate the virtual environment: source virtualenv/bin/activate
install all required modules (e.g. pip install xxx)
deactivate the virtual environment: deactivate
zip the files: zip -r myPython.zip virtualenv __main__.py
create action: wsk action create actionname --kind python myPython.zip

well notes, this way is work in now days...


Does not work with base.html file

I tried this project on my latest project and i found out that is actually adds the whole html file from the doctype thereby ignoring all the css i have adjusted in my base.html file. Its a great editor i just wish it worked well with django templates

It looks like you have to use SummernoteWidget which uses iframe to isolate the editor from your pages, not SummernoteInplaceWidget.


Failed to open display, what am I doing wrong?

hey, so I did make install on syspower but without sudo it can't find libsyspower.so and with sudo it can't open a display

what am I doing wrong?
I'm on Arch Linux
with Hyprland 0.43.0-1

Try re-logging in
Or spawn another terminal instance
Or LD_LIBRARY_PATH=/usr/local/lib syspower

hey, the LD_LIBRARY_PATH works and opens the menu but then again, on proceeding with any command, shutdown, reboot or logout, nothing really happens, it just closes the terminal in which I started syspower. that's it.

You have to detach the program so syspower & detach or bind it to a key in hyprland
syspower closes all programs on shutdown/reboot/whatever so when it closes the terminal that it was started from it also closes itself
Not really a bug more of a user error sorta thing, I could probably add a check for it to not kill the process it was started from but ehh...

yes man that works! thanks.


Addition of new GPT3-Turbo and GPT4 models

I added two new models that came out in the last update.
https://openai.com/blog/function-calling-and-other-api-updates

Would be better having the functions array added to ChatCompletionCreateRequest

Would be better having the array added to functions``ChatCompletionCreateRequest

The Model cannot be defined by an object, it must be a string. That's because in case of an update like now, you can manually name the new model and you can have your own models that you have taught.

already changed it. I meant to add a new property in the dto

I have to close this PR. Adding support for new models has been addressed in a different PR. Thank you for contributing.


Enable verify-samples to handle multiple service directories

Similar to the verify-readme.yml changes, update verify-samples to handle multiple service directories.

The following pipelines have been queued for testing:java - templatejava - template - testsjs - templatenet - templatenet - template - testspython - templatepython - template - tests You can sign off on the approval gate to test the release stage of each pipeline. See eng/common workflow


necessary resources?

@hannahbus , indeed a nicely documented package. I have already been experimenting a bit with the example programme from your tutorial, and it works without any problem. However, as soon as we scale up to a problem with more observations and features, sooner or later it crashes. Our IT department indicated to me that this is probably due to the environment I am working in: a docker environment on a workbench. More specifically, ray would not work well in this context.
They advise me to either switch to bare metal or move out to a cloud environment. The latter is far from obvious for privacy reasons.
What I actually wanted to ask, could you give an indication of what kind of environment you guys use to run somewhat larger samples, e.g. how much memory, how many cores etc? Thanks in advance, and definitely continue working on this nice package!

@joostbollens, thanks for the great feedback. Concerning your question, I would like to refer you to the replication repository of our paper, which you find here https://github.com/MCFpy/replication-BBL_2022. In the ReadMe.pdf, there is a section on Computational Requirements, in which you will find the specification of the machine we deployed-- including no. of cores and RAM. I am not sure Ray requires a cloud environment. We run it on our local machines (laptops) and servers. We tested both Unix-based OS and Windows. All worked fine. Please reach out should the issue persist.


Code suggestions

Here's a list of small bugs or suggestions I've found in the codebase. Some of these are just style things that are probably mainly down to personal preference so feel free to ignore. Bear in mind I haven't tested any of this, it's just from looking through the code. Hopefully this isn't out of place.
1) while loop in __parse_cmd
This is the same in iocage and I don't really see why it's there. Every case has an exit statement that ends execution anyway. The while loop would only make sense if you were trying to parse multiple commands passed in the same cmd line, which would be cool, but near impossible to actually implement. As __parse_cmd is pretty much the only thing that actually gets run in the main code, you also don't really need to exit after each case, as when __parse_cmd returns, the code will exit anyway.
You could also put the ;; on the same line, reducing each case to 1 line, and cutting the length of the function by almost 2/3rds. Some people have fairly strict style guidelines that ;; must be on a newline, but I think that's a bit overkill when each case is only 1 line.
  version)  __version ;;
  oldset)	__oldset "$@" ;;

2) wired memory typos
On line 520, $wire_memory-m would probably cause that grub-bhyve command to fail if wired memory was enabled as there's no space between the options. There's also various calls to grub-bhyve that don't have the $wired_memory settings, although that could be intentional?
3) UEFI seems to drop back to normal boot
Could probably do with an exit after the __uefi call, or at the end of the uefi function as it looks to me like it will drop back to the __start function, then probably output Guest is already running? Also the uefi function seem to check firmware but continues anyway.
4) nested ifs
Another style thing so probably down to preference, I just find lots of nesting makes things harder to read and maintain. You could get rid of quite a bit of nesting by switching if statements the opposite way round, then exiting. For example in __install you could do the following:
# check valid guest
if [ ! -d /iohyve/$pool/$name -a ! -d /iohyve/$name ]; then
    echo "Not a valid guest name"
    exit 1
fi

# Check to make sure guest isn't running
local running=$(pgrep -fx "bhyve: ioh-$name")
if [ -n $running ]; then
    echo "Guest is already running"
    exit 1
fi

# normal code

Thank you for your feedback! There's definitely some issues with formatting and a bit of rot in the script. There's been so many hands in iohyve it's hard to keep things standard.
The way things are going it's only going to get worse, so I've been tossing the idea of a feature freeze and audit before we begin work on NETMAP support. (Around v0.8) There's also the idea of splitting into libraries and such as well. It may be prudent to do that at the same time.
I'm closing the issue, but feel free to respond or add any other comments here, they won't fall on deaf ears.


Feature: Dependency Updates

Coverage remained the same at 69.315% when pulling cabd7b3bf12a2cc8ccdfc5902abfc9ba2b761237 on feature/dependency-upgrades into f58aff0aee855f8e5f7b840d76f40355eac502fb on master.


Hamburger menu for right side panel not shown when in Focus mode

Is the bug present when using the default Obsidian theme?
No
Is the bug present when snippets and plugins are disabled?
Yes
Minimal theme version
7.5.4
Describe the bug
Enabling Focus mode on the companion Plugin seems to break styling for the right side panel's hamburger icon. No icon is shown, but the space for it is kept and clicking on it actually works. Issue is not present when disabling Focus mode.
Debug info
SYSTEM INFO:
Obsidian version: v1.5.12
Installer version: v1.5.3
Operating system: Darwin Kernel Version 23.3.0: Wed Dec 20 21:33:31 PST 2023; root:xnu-10002.81.5~7/RELEASE_ARM64_T8112 23.3.0
Login status: logged in
Catalyst license: none
Insider build toggle: off
Live preview: off
Base theme: dark
Community theme: Minimal v7.5.4
Snippets enabled: 3
Restricted mode: off
Plugins installed: 1
Plugins enabled: 1
1: Minimal Theme Settings v7.5.0
RECOMMENDATIONS:
Custom theme and snippets: for cosmetic issues, please first try updating your theme and disabling your snippets. If still not fixed, please try to make the issue happen in the Sandbox Vault or disable community theme and snippets.
Community plugins: for bugs, please first try updating all your plugins to latest. If still not fixed, please try to make the issue happen in the Sandbox Vault or disable community plugins.

This should be fixed in 7.6.9


Adonis Cli fails to install

This started in the last couple of hours, it seemed to work beforehand. Upon installing the adonis cli, i get the following error:
npm ERR! Invalid tag name "dot-prop<EMAIL_ADDRESS>Tags may not have any characters that encodeURIComponent encodes.
adonis cli version: 4.0.12
node: 11.15.0
npm: 6.7.0
Sample Code (to reproduce the issue)
npm i<EMAIL_ADDRESS>

Seems to be fixed now.


Support newer versions?

Tailscale is being removed from official repos.
https://github.com/openwrt/packages/pull/25062

The builds in this repo are built automatically whenever a new tailscale version is released via GitHub actions.
It should continue to work independent of what version of Go is supported in openwrt.

I meant new versions of OpenWRT.

These builds should also work on newer OpenWRT versions as well as there are no external dependencies. Just point the repo to 19.07 and install.

Just tried this on OpenWRT 25.05.5, opkg update fails because Package.sig doesn't exist. You can turn off signature verification, but that's probably not a good idea

I do not have any signing keys that would be trusted by openwrt. So even if I added a signature file, it would appear invalid.


Support for multiple remotes in infrakit CLI

This PR makes the use of -H to connect to remote hosts easier:

A hosts file in $INFRAKIT_HOME/hosts is used to determine the url of the remote infrakit daemons to connect to.  Multiple hosts can be specified in one file:

The format is YAML
It's simply a dictionary of :  <url_list>  where <url_list> is a comma-delimited list of URL remote mux endpoints and  is a logical name.
An environment variable INFRAKIT_HOST is used to determine which hostname to use.  If the environment variable is not set, "default" is used.

This is the order of determining where to connect:

If -H is set, it takes priority ==> continue
If the INFRAKIT_HOST env is set, then use it as a key in the hosts file:

Load the hosts file in INFRAKIT_HOME/hosts
Look up the comma delimited string associated with INFRAKIT_HOST value
Use the host list as connection string for remote client access

If INFRAKIT_HOST env is not set, and hosts file exists, use the "default" host.
If INFRAKIT_HOST env exists and is set to local, then connect locally.
If hosts file does not exist, then assume local mode.

Please sign your commits following these rules:
https://github.com/moby/moby/blob/master/CONTRIBUTING.md#sign-your-work
The easiest way to do this is to amend the last commit:
$ git clone -b "work-0504" git@github.com:chungers/infrakit.git somewhere
$ cd somewhere
$ git rebase -i HEAD~842354613824
editor opens
change each 'pick' to 'edit'
save the file and quit
$ git commit --amend -s --no-edit
$ git rebase --continue # and repeat the amend for each commit
$ git push -f

Amending updates the existing PR. You DO NOT need to open a new one.


Add configmaps GET permission

EES-4318
With the addition of Locust implementation, now Kangal requires an extra permission, a GET for ConfigMaps.

+1

+1


i18n tests

Description
I've modified the test for src/locale/compare.ts to make it test the functionalities instead of running the actual locales test.
I've added the locales test to a pre-push git hook (via Husky) so that the check is run each time we push a branch.
Also added the check to the CircleCI build_and_test job.
On what platform have you tested the change?

[ ] Android - Emulator
[ ] Android - Physical device
[ ] iOS - Emulator
[ ] iOS - Physical device

Screenshots
Checklist

[ ] I have updated mockServer
[x] I have run npm test
[x] I have run npm run test:i18n

Out of scope and potential follow-up

@MikeD-ZOE I've resolved conflicts, are you happy with this?


gltf-loader: support data url for images

This allows the glTF-Embedded variants in the sample models to be used.
The data url format is relatively small, so I didn't include a crate like docs.rs/data-url.
Also fixes the 'Box With Spaces' model as URIs are now percent-decoded.
cc #1802

bors r+


ClusterItem position is centered at the bottom

I am using Clustering and to my suprise, the content of the ClusterItem is not centered in the middle, but at the bottom. This square and marker have the exact same position, but as you can see the square is not in the center of the marker. This looks really weird and wrong when you place ClusterItems with images on the map, they are on top of where they belong:

Reproduction project:
https://github.com/Jasperav/GoogleMapsBug
Relevant code:
GoogleMap(cameraPositionState = CameraPositionState(
    position = CameraPosition.fromLatLngZoom(
        defaultLocation, 18f
    )
)) {
    Marker(state = MarkerState(defaultLocation))
    Clustering(items = listOf(ClusterItemCustom())) {
        Box(
            modifier = Modifier
                .requiredHeight(100.dp)
                .requiredWidth(100.dp)
                .background(
                    Color.Blue.copy(alpha = 0.5f)
                )
        )
    }
}

If you would like to upvote the priority of this issue, please comment below or react with :+1: so we can see what is popular when we triage.
@Jasperav Thank you for opening this issue. 🙏
Please check out these other resources that might help you get to a resolution in the meantime:

Check the issue tracker - bugs and feature requests for Google Maps Platform APIs and SDKs
Open a support case - Get 1:1 support in Cloud Console.
Discord - chat with other developers
StackOverflow - use the google-maps tag

This is an automated message, feel free to ignore.

@kikoso is there any planned work for this issue?

@kikoso You would not expect a marker to be centered around the given position to give more context, this is the exact behavior on iOS (Apple Maps) of my use case. I am converting map related stuff from an iOS app to Android and this is a bit of a problem.

i'd also like to customize the cluster + item location. (i need it to be centered as well)
i'm quite confused why this is not an issue for more developers. and also confused why it's hardcoded that way.


Add Prism GC application

Add a new application Prism GC (garbage-collect command) to delete unused S3 objects on the Prism bucket.

test


fix: URL escaping for asset files

Fixes loading files with ? or other special URL characters in the name.

Tested on a file called twinkle???.wav


Documentation: convert .inc to .rst

It also solves a lot of warning showing up after renaming the files.
The documentation builds without errors and looks fine, but please check.

This should be merged soon to facilitate the ongoing documentation effort.

Can you try to fix that?

I tried but without success. :disappointed:
I don't know what is going wrong, I used exactly the syntax given in the Sphinx documentation.

I tried but without success.
I don't know what is going wrong, I used exactly the syntax given in the Sphinx documentation.

Renaming the file back to .inc solves the problem, so I suspect this is why there were called that in the first place...

Renaming the file back to .inc solves the problem, so I suspect this is why there were called that in the first place...

Given the advantage of using .rst, do we really want to rename back to .inc just to get rid of the warning?

Any opinion on how to proceed? @jvegasbsc @valeriupredoi @ledm
I find .rst useful for the preview function on GitHub.

Given the advantage of using .rst, do we really want to rename back to .inc just to get rid of the warning?

Yes, because I want to add a unit test that the documentation builds without warnings quite soon, because now I have to fix the warnings every few months and ask people to fix their warnings every time I'm reviewing a pull request.

ESMValTool as a whole has a serious problem with it's documentation, which we are slowly addressing. I support any effort that makes it easier to write (and read) documentation, even if it means that we encounter warnings (but not errors!) at compile time.

The problem with allowing warnings is that they are almost always caused by actual errors, so the resulting documentation will not look at all like the author expects.

Ok, let's rename back to .inc. Still this PR contains other minor improvents which are worth merging.

No more warnings! :tada:

I've added the documentation builds without warnings test and the gallery is now automatically generated when you build the documentation.

Very nice, thank you!


resource group big_query_scan_rows_limit didn't worked  when use limit

Steps to reproduce the behavior (Required)
create resource group rg2
to
    (user='root')
with (
    'cpu_core_limit' = '4',
    'mem_limit' = '10%',
    'concurrency_limit' = '2',
    'type' = 'normal',
    'big_query_scan_rows_limit' = '2'
);
select * from lineitem limit 3;

Expected behavior (Required)
ERROR 1064 (HY000): exceed big query scan_rows limit: current is 3 but limit is 2
Real behavior (Required)
query ok
StarRocks version (Required)

You can get the StarRocks version by executing SQL select current_version()

mysql> select current_version();
+----------------------------+
| current_version()          |
+----------------------------+
| BRANCH-2.3-RELEASE 26f8840 |
+----------------------------+

https://github.com/StarRocks/starrocks/issues/8028


Components as Web Components

We started discussion here, I just want to structurize it a bit.

As requested, an example of a (cycle) soon to be web component: http://codepen.io/cem2ran/pen/xGKpBE?editors=101 :tada:

I've created similar example, but a bit earlier, for Cycle 0.20.
https://github.com/erykpiast/autocompleted-select/blob/master/src/js/create-component.js

@cem2ran I like your codepen example, and it shows how it might be important to find a proper way to pass properties to the top-level computer function which is given to applyToDOM. In your example the web component is not using the attributes it was given. The exciting is that I think issue #116 will be able to solve this problem, so we don't need to add another API shortcut for this use case.

This issue was moved to cyclejs/cycle-web#10


PLAT-52544: Clean up samples

Clean up the items specified in DoD of the PLAT-52544.
Enact-DCO-1.0-Signed-off-by: Baekwoo Jung<EMAIL_ADDRESS>

I have applied all the above comments. and I modified appinfo.json and package.json to be identical to other samples.

Too many properties were omitted in samples\package.json. Could you check it?

@ybsung
React-router has been removed because it is an unused module. (using react-router-dom in samples)

LGTM

LGTM

I have applied all the above comments.


return_code accesses non-existent subprocess attribute

I tried to get c.return_code which in turn calls subprocess.returncode. Got:
AttributeError: 'PopenSpawn' object has no attribute 'returncode'
Looking at the call I saw that the pExpect module actually has the following:
subprocess.exitstatus

This is resolved with #19.


Update to Semian 0.21

Semian gem is locked in the semian-postgres.gemspec  to <0.20.0 .
As far as I can see, it doesn't seem to be breaking changes in recent Semian versions, but not sure though.

Great @mschoenlaub, thanks!


Fix pr716 updates

What changes does this PR introduce?
Fix problems with some changes from PR #716 when  generating TAF adjoint with  ALLOW_PSBAR_STERIC defined (in ECCO_OPTIONS.h), see issue #725
What is the current behaviour?
TAF get confused about BEGIN_MASTER/END_MASTER additions (from PR #716) leading to major recomputations and significant slow-down.
What is the new behaviour
fixed by reverting changes when pkg/autodiff is compiled. Also add missing new src code ecco_read_pickup.f to ecco_ad_diff.list.
Does this PR introduce a breaking change?
no
Other information:
None of current AD verification experiments does test ALLOW_PSBAR_STERIC code but this is tested in verification_other 2 AD test-experiments global_oce_cs32 and global_oce_llc90.
Suggested addition to tag-index
Not needed if merged just after PR #716

Would it be a good idea to test this code also in lab_sea? When I define ALLOW_PSBAR_STERIC, however, one out of 3 AD experiments fail (lab_sea.noseaice: Program received signal SIGILL: Illegal instruction.), also there's another recomputation warning.

@mjlosch Thanks for the review and your comments. Below are 3 pieces of answer:

For the sake of quickly fixing this, we can do it this way. I am wondering, why on the first _BEGIN/END_MASTER statement in ecco_init_varia.F needs to be removed.
It's a problem to have these if-statements removed?

I think I did try to only revert changes in ecco_phys.F and keep this _BEGIN/END_MASTER  in  ecco_init_varia.F but was getting I recomp. warning:

TAF WARNING RECOMPUTATION WARNING CALL_STMT ad_input_code.f:XXX in the_main_loop
TAF WARNING  triggered by      : rhosumglob_0,volsumglob_0,xx_gentim2d_dummy[C]
TAF WARNING  still to be recomputed : rhosumglob_0,volsumglob_0
TAF WARNING RECOMPUTATION WARNING CALL_STMT ad_input_code.f:XXX in the_main_loop
TAF WARNING  triggered by      : rhosumglob_0,volsumglob_0,xx_gentim2d_dummy[C]
TAF WARNING  still to be recomputed : rhosumglob_0,volsumglob_0
So I decided to remove it.

Regarding this:

It is possible to use store statements to get rid of the warnings (in 7 files with including new header files like ECCO_OPTIONS.h, ECCO_SIZE.h, ECCO.h), but the proposed solution is much  less intrusive, and if safe, to be preferred in my opinion.

Just to be clear: The alternative solution is to add "store statements" in few files, but this would be not be "less intrusive", right ? So, at the end, you prefer the alternative solution (with storage) or this PR solution (removing BEGIN/END_MASTER) ?
And regarding this:

Would it be a good idea to test this code also in lab_sea? When I define ALLOW_PSBAR_STERIC, > however, one out of 3 AD experiments fail (lab_sea.noseaice: Program received signal SIGILL: Illegal instruction.), also there's another recomputation warning (that could be fixed easily).

I think this could be usefull (at least to compile it), but not sure what the output means in a non-global setup (e.g., the Greatbach correction?). Would need to check few things, so probably better in an other PR. Right now the daily AD tests for verification_other are at the edge of not finishing within the queue time-limit, because of these recomp., so if anything, would prefer to have this fixed soon.

@jm-c your proposed solution (in this PR) is preferred (meaning: I prefer it). My solution with store directive is far more intrusive and we should not go down that path, if your solution is safe.
And yes, let's not waste time on adding this to lab_sea (it didn't change results in my case, and I finally got it to work without issues, but I agree it is meaningless and would serve only to compile and get the extra recomputations). Let's do this later.

Thank you @jm-c for fixing the re-computation issue. Is it still multi-thread safe for adjoint calculations after reverting some changes made in PR#716 regarding multi-threading?

@owang01 Thanks for your comment. After reverting, if pkg/autodiff is compiled,  most of the "multi-threading" changes I made in PR #716, the resulting code is not great once ALLOW_AUTODIFF is defined, I agree.
But there are 2 positive things about this attempt:

There is a clear miss understanding by TAF of what MITgcm multi-threading does, and specially regarding non-tiled global variables (in common block).
There is one little change/improvement that I made in ecco_phys.F that I kept, and this was the addition of local variables VOLsumGlob_1 and RHOsumGlob_1, and this makes the multi-threading safer (before that, the line of code "RHOsumGlob = RHOsumGlob / VOLsumGlob" would have been run for each threads, and the final results in RHOsumGlob would have been the initial RHOsumGlob value divided by VOLsumGlob to the power of the number of threads).


When Are IP Address Changes Effective?

First off thank you very much for providing us with a way to be notified about IP Address changes. This has been helpful. Is there any way you could clarify how quickly after a change has been published we should expect that change to be implemented by the Application Insights service?
For example, if we get a notification about a new IP address being added, should we expect that new IP address to be used by the AI service with in a few minutes, hours, days, or is it already in effect by the time the notification comes out?
Any clarification on this would be helpful so that we can adjust our firewall rules in a timely manner.

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

ID: 92dd03db-51fc-ee03-3244-f0b847303b90
Version Independent ID: 0794a496-ee0c-142d-b37c-a3482d24ffec
Content: IP addresses used by Application Insights and Log Analytics
Content Source: articles/azure-monitor/app/ip-addresses.md
Service: application-insights
GitHub Login: @mrbullwinkle
Microsoft Alias: mbullwin

@Smoles - Thank you for your question, we are looking into this and will get back to you shortly.

@Smoles Thanks for the interesting question!
From what I've gathered from our internal Teams, it turns out that we make sure that new IP addresses are added to this page before the customer traffic is routed to them. There is no formalized wait time beyond that. While it typically took a few days in the past between the docs update and the new IP addresses becoming active, we’re striving to reduce the rollout time to just a few hours for the new regions.
Hope this addresses your query!

Yes this is helpful.
Thank you!


Fix VOC to have task-specific formats

Summary

How to test

Checklist

[ ] I have added unit tests to cover my changes.
[ ] I have added integration tests to cover my changes.
[ ] I have added the description of my changes into CHANGELOG.
[ ] I have updated the documentation accordingly

License

[ ] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.
[ ] I have updated the license header for each file (see an example below).

# Copyright (C) 2023 Intel Corporation
#
# SPDX-License-Identifier: MIT

Codecov Report
Patch coverage: 85.02% and project coverage change: -0.20 :warning:

Comparison is base (8fe4cf0) 78.53% compared to head (adae9da) 78.33%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #997      +/-   ##
===========================================
- Coverage    78.53%   78.33%   -0.20%
===========================================
  Files          233      233
  Lines        26749    26844      +95
  Branches      5320     5350      +30
===========================================
+ Hits         21007    21028      +21
- Misses        4497     4547      +50
- Partials      1245     1269      +24

Flag
Coverage Δ

macos-11_Python-3.8
?

ubuntu-20.04_Python-3.8
78.33% <85.02%> (-0.18%)
:arrow_down:

windows-2019_Python-3.8
?

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

datumaro/plugins/data_formats/voc/base.py
79.88% <77.31%> (-13.11%)
:arrow_down:

datumaro/plugins/data_formats/voc/exporter.py
80.37% <91.48%> (-2.23%)
:arrow_down:

datumaro/plugins/data_formats/voc/format.py
95.52% <100.00%> (+0.23%)
:arrow_up:

datumaro/plugins/data_formats/voc/importer.py
88.88% <100.00%> (-11.12%)
:arrow_down:

... and 5 files with indirect coverage changes

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


Problem with new RabbitMQ 6 client

Is this a bug report?
Yes
Can you also reproduce the problem with the latest version?
Yes, we tried latest 6.2.5 prerelease and we see the same problem.
Environment

Operating system: Windows 10
Visual Studio version: Visual Studio 2019
Dotnet version: .NET 4.7.2 (with netcore 3.1.201 it works properly)

Steps to Reproduce
Create and start a bus using RabbitMQ
var busConfiguration = RabbitMqBusConfiguration.Parse(ConfigurationManager.AppSettings.Get("BusConnectionString"));
return Task.Run(() =>
{
var bus = Bus.Factory.CreateUsingRabbitMq(sbc =>
{
sbc.Host(busConfiguration.ToRabbitMqHostSettings());
});
            bus.StartAsync().Wait();

Expected Behavior
Bus starting properly, like it is using a netcore31 host
Actual Behavior
Bus fails with: RabbitMqConnectionException: ReceiveTransport Faulted, TypeLoadException: Could not load type 'RabbitMQ.Client.NetworkConnection' from assembly 'RabbitMQ.Client, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=89e7d7c5feba84ce
I checked RabbitMQ Client 5.2 and 6.0 and in the netsdk version, RabbitMQ.Client.NetworkConnection has been effectively removed

MassTransit currently is using the 5.1.2 client, it will be updated at some point in the future.


[CGPROD-3016] Disable shop buttons that have scrolled offscreen

Hey- I see what you've done here & it's waaay simpler & cleaner than my original solution, which is awesome. However firing up the shop I can still click on items that are scrolled off the top or bottom (observed on desktop Chrome / macOS.) I think your code might be bumping into the work Nick did to get a11y elems to play nice with RexUI - scrollable-list.js:84 const forwardEventsToButton () => {...} - but not sure without further digging. Do ping if you want to chew it over...

Hey- I see what you've done here & it's waaay simpler & cleaner than my original solution, which is awesome. However firing up the shop I can still click on items that are scrolled off the top or bottom (observed on desktop Chrome / macOS.) I think your code might be bumping into the work Nick did to get a11y elems to play nice with RexUI - scrollable-list.js:84 const forwardEventsToButton () => {...} - but not sure without further digging. Do ping if you want to chew it over...

ye its since I merged in master.... will go through this again

ye its since I merged in master.... will go through this again

oh it does work fine here, I think the thresholds are just a bit off

oh it does work fine here, I think the thresholds are just a bit off

Ok think I've fixed it now

Ok think I've fixed it now

Little bit more investigation on this suggests that my idea isn't so great, since the inner panel is actually larger- i.e. you can be outside the panel, but over a hidden item, and you're still inside the inner panel- it's just masked. So it'd need to check isInTouching() on the outer panel but somehow detect if the click is in the border area. Argh!

Little bit more investigation on this suggests that my idea isn't so great, since the inner panel is actually larger- i.e. you can be outside the panel, but over a hidden item, and you're still inside the inner panel- it's just masked. So it'd need to check isInTouching() on the outer panel but somehow detect if the click is in the border area. Argh!

Little bit more investigation on this suggests that my idea isn't so great, since the inner panel is actually larger- i.e. you can be outside the panel, but over a hidden item, and you're still inside the inner panel- it's just masked. So it'd need to check isInTouching() on the outer panel but somehow detect if the click is in the border area. Argh!

yea I had a mess with this before, I think it's fine like this - especially on touch devices, it wouldn't be noticeable.
if we can find a way to get it a little better without needing to write much code, then we can always come back and improve this further

Little bit more investigation on this suggests that my idea isn't so great, since the inner panel is actually larger- i.e. you can be outside the panel, but over a hidden item, and you're still inside the inner panel- it's just masked. So it'd need to check isInTouching() on the outer panel but somehow detect if the click is in the border area. Argh!

yea I had a mess with this before, I think it's fine like this - especially on touch devices, it wouldn't be noticeable.
if we can find a way to get it a little better without needing to write much code, then we can always come back and improve this further


Change wording for circle enter direction

A more understandable wording might be "enter circle at x o'clock", where x would be the hour position of the first point on the circle with respect to the circle center.

I checked with the pilots, and they prefer directions like north, east etc...
But we still should clarify and ensure, that the direction signifies the position on the circle where we enter the circle, not the direction we fly before entering the circle.


Showing the current Connected Server when hover over tab

Issue Type: Feature Request
At the moment you can't tell which server you are connected to. It would be nice to be able to hover on the tab and see the connected server for the current tab.
SQL Operations Studio version: sqlops 0.30.6 (df7e3ec7d52a82febee5b597a1019cabf021b883, 2018-06-19T21:45:43.589Z)
OS version: Windows_NT x64 10.0.17134

You can show the connection info in the tab name using the  "sql.showConnectionInfoInTitle" setting.

this has been implemented and enabled by default. setting name: queryEditor.showConnectionInfoInTitle


  This commit
does a diff of the inventory on the first message that says
you are loading the cannon.  If the diff and inventory
after that message both have cannon balls in them, then
we know the maximum amount possible were stored so set
the counter to max.
Fixes #13105

Do you know offhand what test cases you were testing ingame to make sure these changes worked? I'd like to add tests to this.

I've added a test for your second commit. I'm happy to write a test for your first commit as well (and maybe the rest of the plugin while I'm at it) if you can share the steps you took to test this ingame

I've added a test for your second commit. I'm happy to write a test for your first commit as well (and maybe the rest of the plugin while I'm at it) if you can share the steps you took to test this ingame

The process to test each case would be:

place the cannon in KQs lair
load with cannonballs and start firing
KQ destroys the cannon
reclaim the cannon from Nulodion
setup the cannon in barb village (dwarfs dont let you place it next to them)

each test case is:

load regular, set back up with excess regular in invent, observe cannon reading 30
1 but with granite cannonballs
load regular, set back up with <= amount of space in cannon of balls in invent, observe only the loaded amount displayed (this is a case i dont believe can be solved since the client can be shut down between destruction and claim, so theres no way to preserve the known number of cannonballs in the cannon)
3 but with granite cannonballs
load regular, set back up with any amount of granite cannonballs in invent, observe the "you cant mix cannonballs message" and the count is set correctly to the message that displays how many balls are loaded
load granite, set back up with only regular in invent, same as 5

Ok, I've added tests which cover the added functionality in your commits. I've also added a commit to remember the previous cannonball count when it is destroyed and later set up. I think this is probably all set now?

Your point on the tests is taken. I've incorporated your recommended changes (along with some further changes on my part to fix up these commits) and I think it's all set now.


Update Carrot.NLog.nuspec

Fix description field

Thank you for your submission and nice catch.
That said, we already managed it with the dotnetcore porting activity.


fix: skip in-use loop devices during vgcreate/vgextend

filter out loop devices based on Major number from lsblk
check whether the back-file is being used by k8s for mounting block
devices and skip using that loop device in volume groups ops

Fixes: #97
Signed-off-by: Leela Venkaiah G<EMAIL_ADDRESS>

well, the fix itself is working, just felt that FilterMap got kinda ugly, so made PR as draft
however PR is re-viewable, below is the log showing how devices are skipped if they are mounted by k8s

{"level":"info","ts":1645098272.500616,"logger":"controller.lvmvolumegroup.vg-manager","msg":"does not match filter","reconciler group":"lvm.topolvm.io","reconciler kind":"LVMVolumeGroup","name":"test","namespace":"lvm-operator-system","Device.Name":"loop1","filter.Name":"usableLoopDevice"}
{"level":"info","ts":1645098272.5372882,"logger":"controller.lvmvolumegroup.vg-manager","msg":"does not match filter","reconciler group":"lvm.topolvm.io","reconciler kind":"LVMVolumeGroup","name":"test","namespace":"lvm-operator-system","Device.Name":"loop2","filter.Name":"usableLoopDevice"}
{"level":"info","ts":1645098272.5835214,"logger":"controller.lvmvolumegroup.vg-manager","msg":"does not match filter","reconciler group":"lvm.topolvm.io","reconciler kind":"LVMVolumeGroup","name":"test","namespace":"lvm-operator-system","Device.Name":"loop3","filter.Name":"usableLoopDevice"}

tested locally

# before restart of vg-manager
sh-4.4# vgs
  VG   #PV #LV #SN Attr   VSize  VFree
  test   3  10   0 wz--n- <4.37t <3.39t
sh-4.4# pvs
  PV           VG   Fmt  Attr PSize  PFree
  /dev/nvme0n1 test lvm2 a--  <1.46t 490.41g
  /dev/nvme1n1 test lvm2 a--  <1.46t  <1.46t
  /dev/nvme2n1 test lvm2 a--  <1.46t  <1.46t

# after restart of vg-manager with 5 in-use loop devices and 1 usable loop devices and so vg is extended correctly
sh-4.4# pvs
  PV           VG   Fmt  Attr PSize  PFree
  /dev/loop5   test lvm2 a--  <5.00g  <5.00g
  /dev/nvme0n1 test lvm2 a--  <1.46t 490.41g
  /dev/nvme1n1 test lvm2 a--  <1.46t  <1.46t
  /dev/nvme2n1 test lvm2 a--  <1.46t  <1.46t
sh-4.4# vgs
  VG   #PV #LV #SN Attr   VSize VFree
  test   4  10   0 wz--n- 4.37t 3.39t

sh-4.4# lsblk
NAME                                            MAJ:MIN RM   SIZE RO TYPE MOUNTPOINT
loop0                                             7:0    0   100G  0 loop
loop1                                             7:1    0   100G  0 loop
loop2                                             7:2    0   100G  0 loop
loop3                                             7:3    0   100G  0 loop
loop4                                             7:4    0   100G  0 loop
loop5                                             7:5    0     5G  0 loop
sdb                                               8:16   0 893.8G  0 disk
|-sdb1                                            8:17   0     1M  0 part
|-sdb2                                            8:18   0   127M  0 part
|-sdb3                                            8:19   0   384M  0 part /boot
`-sdb4                                            8:20   0 893.3G  0 part /sysroot
sr0                                              11:0    1   987M  0 rom
nvme0n1                                         259:0    0   1.5T  0 disk
|-test-6777f3ec--40de--4e8d--ac55--d32b71f7819a 253:0    0   100G  0 lvm
|-test-ca957a21--0ae2--4b05--9e14--2896b38a0fd7 253:1    0   100G  0 lvm
|-test-7901dd21--4942--4f1c--945c--7e5d23e49d01 253:2    0   100G  0 lvm
|-test-32d72da7--580f--4c59--a9ef--5eed39ff139e 253:3    0   100G  0 lvm
|-test-3e617cc8--c26f--40ac--9460--214421330e7a 253:4    0   100G  0 lvm
|-test-6251c743--7326--4c96--9724--3eb34a667087 253:5    0   100G  0 lvm
|-test-0681a68f--94bb--484d--add5--2f72e2859978 253:6    0   100G  0 lvm
|-test-cfd826d7--cc8f--4ea8--9dbe--3b27904671c5 253:7    0   100G  0 lvm
|-test-9d4aa183--72fc--441a--a577--59e4ff4c0f94 253:8    0   100G  0 lvm
`-test-9cd06305--96a3--4b90--8080--01f45b0fa4e0 253:9    0   100G  0 lvm
nvme1n1                                         259:1    0   1.5T  0 disk
nvme2n1                                         259:2    0   1.5T  0 disk

Tested again below scenarios

An usable loop devices exists before vg-manager comes up and reboot vg-manager without creating block PVCs
An usable and unusable loop devices exist after vg-manager reboot
General, no loop devices before vg-manager comes up

btw, losetup already exists in the operator due to installation of util-linux

/cherry-pick release-4.10


feat: add projex app to patch project browsing

What new features does this PR implement?
adds new temporary version of the platform app which just browses projects and displays project details.  This app is called '/app/projex' (a unique name which will be easy to remove) and is deployed to patch.biosimulations.dev and patch.biosimulations.org and linked to temporarily replace legacy biosimulations.org/projects page.
What bugs does this PR fix?
Current production biosimulations.org/projects was not designed for >> 1000 projects, but the current development design is more scalable.  So this fix links out to the dev site for project browsing.  A new app is created just for this purpose so that it can be stable for use in production while not interfering with the development of dev 'platform' app.
Does this PR introduce any additional changes?
Adds a new app, deployed to Netlify as patch.biosimulations.dev and patch.biosimulations.org.
How have you tested this PR?
Tested locally - until all links are live with new domains and API CORS settings, cannot test fully.

:tada: This PR is included in version 9.27.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


rtl version

We won't be supporting rtl for now.

@Dogfalo Why not merge @MetaiR 's awesome work?


Bump user and server api versions

@ayshsandu Can we check why the travis build fails?

@ayshsandu Can we check why the travis build fails?

https://github.com/wso2/identity-api-server, 1.0.6 has not been released by the Travis build time


[FEATURE] Support bucket_sort aggregations

We really want to use newer version because of serializer speed AND really depend on this agg.
When is estimated release?
Elastic.Clients.Elasticsearch version: 8.0.1
Elasticsearch version: 8.5.x
.NET runtime version: .net6
Operating system version: not applicable
Description of the problem including expected versus actual behavior:
A clear and concise description of what the bug is.
Steps to reproduce:
no query language for bucket_sort, found in source code this:
case “avg_bucket”:
throw new Exception(“The aggregate in response is not yet supported.“);
case “bucket_script”:
throw new Exception(“The aggregate in response is not yet supported.“);
case “bucket_count_ks_test”:
throw new Exception(“The aggregate in response is not yet supported.“);
case “bucket_correlation”:
throw new Exception(“The aggregate in response is not yet supported.“);
case “bucket_selector”:
throw new Exception(“The aggregate in response is not yet supported.“);
case “bucket_sort”:
throw new Exception(“The aggregate in response is not yet supported.“);
Expected behavior
support added

Hi, @vladimir-ilnytskyi.
This is not currently available in the 8.0.0 as documented in our supported aggregations in our release notes. We will (re)introduce this (and other) aggregation types in future client releases. Thanks for raising your need for this type, as it helps prioritise that work.

Hello!
Thanks for the reply!
Any rough estimations on when this is planned to be implemented in beta?

@vladimir-ilnytskyi I can't provide any timeframes at the moment.


Restore-db

fixed db_dest_path first creation failing
restore mechanism from polkashots has changed, now working again

Thanks for your contribution :)


[BUG] publish to maven local is not publishing sha1 and md5 checksums

What is the bug?
./gradlew publishToMavenLocal does not generate the checksums
How can one reproduce the bug?
Steps to reproduce the behavior:

/gradlew publishToMavenLocal
check the files

What is the expected behavior?
sha1 and md5 checksums should be generated
Do you have any additional context?
work around has been added here: https://github.com/opensearch-project/opensearch-java/pull/51/files#diff-e21da378399d7f35543533d279a39bf3d1024092580f40330fd8bc56379d6164R87-R109
which can be removed once the checksum generation is handled by gradle

Fixed via #187.