Sentence tokenization with PunktSentenceTokenizer

https://www.nltk.org/api/nltk.tokenize.PunktSentenceTokenizer.html
I am working with llms and store input/outputs in polars (here for adding an embeddings column and using polars for hybrid filters/vector search). I often want as a processing step to tokenize into sentences but nltk is very slow, this seems like an interesting algorithm for a rust implementation that can have a lot of uses from multi sentence string column to list[str] column.

This will be a great addition to the current string stuff and indeed there is a rust crate that does punkt sentence tokenization.
Only 1 problem. One of my goals is to keep the package as slim as possible (while keeping the performance). Punkt will require you to download predefined rule sets and will make the package much bigger. I can go about this like NLTK by asking the user to download these rules. I don't know how to do that yet. Another thing is that the Rust punkt crate is quite old and I am slightly worried about the maintenance.
Is this a high priority issue? Currently I don't think I will work on this in the near future. Hugging face seems to have a Rust-tokenizer crate which has a Python package that does some fancy tokenizations. Maybe that's helpful?

Thanks (again) for the great response, just to clarify I am loving the design package so I am giving suggestions of things that would fit well. Nothing I am asking is a real blocker for me since I am opening issues for algos I already have in my python stack and I think are worth to be "primitives" for dataframe pipelines.
I agree Punkt is low priority as I can always pre-process my strings slowly and keep them in a parquet for lazy processing. Something I observed is that going from utf8 to list[utf8] (after nltk punkt) is crazy expensive in terms of memory and was thinking that it would be better to do it on the spot only when needed to avoid storing the inflated parquet file.
I was literally running punkt  (and waiting 40 mins for 3M rows) while "studying" this package and I felt like it would be a good fit, I did check the Rust Punkt crate but is indeed old :/
I will post more suggestions in the next days and next week I will throughly test and benchmark the knn module on some text embeddings.
Thanks so much for reading and considering my suggestions!

Closing this as not planned


ResNext Code is incorrect.

Why ResNext Code is equal to ResNet?
You need to see files, rmac_resnet.py and rmac_resnext.py in nets.
rmac_resnext.py is just copied from rmac_resnet.py
Please see and modify codes.

closed. because of not answering for a long time.


Create a functionality to borrow the bike once user finds a bike.

Features/Views/Components to be implemented
Input:

Bike ID

Functionality:

Type bike ID
Link to search CallBikes menu
Back to main menu


Enhancing UI

As a GSSOC'22 contributor I would like to work on enhancing the UI of the Hash-Whiteboard.
Kindly assign this issue to me.

Kindly assign this work to me.
I have internship experience and I'll make sure that the work will be done within 2 weeks @IKRA786 @AnuthaDev @varun14gb @rohannair11 @achitJ

@IKRA786 @AnuthaDev @varun14gb @varun14gb @rohannair11


Look into displaying code coverage change over time

This is a suggestion I made for something to add that would be nice. Everytime coverage runs we record the total coverage and update the badge of the repo. It would be helpful to add that percentage and date to a tool so we can view how total coverage change over time, and then make a graph

This is just and idea and not sure if possible


[CodeClean] fix indent

Trivial, fix indent and init null to global val.

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #285. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://ci.nnstreamer.ai/.


Update to react plugin 7.9.1

Enact-DCO-1.0-Signed-off-by: Roy Sutton<EMAIL_ADDRESS>

I'm a little hesitant about the 'react/forbid-foreign-prop-types': 0, rule, but we can see how this plays out over time. We did this at least twice in our code at this point, so we found value in it. I'm not convinced of the value of it if we just turn off the linting rule whenever we want to do it.


[Bug] i64 start option for increment integer provider panics

When using the Increment.integer provider, with the start option, if the value given cannot be converted to i32 (so, higher than i64), the code will panic.
The Random.Number.i32 provider can be used as example as it has an extra check.

It's not happening actually


Bounds not updating when surrounding div resizes.

I have a google-map-react component in a surrounding div. When the surrounding div resizes, the map obviously resizes too. But getBounds doesn't change.
I'm currently fixing this with the following hack:
componentDidUpdate() {
        if (this.map)
            google.maps.event.trigger(this.map, 'resize');
}

Is this necessary or is there a better way?

https://github.com/istarkov/google-map-react/issues/106#issuecomment-187944697
When I will have a time I will play with this https://github.com/sdecima/javascript-detect-element-resize element size detection lib.

Thanks. My method actually has significant problems - although getBounds then gives the correct bounds, it also gives GoogleMap bounds not eq errors in the console, and the markers suddenly jump to the wrong position.

Ok, so I'm now trying to do this with the element-resize-detector library on the DOM node. That part seems to work fine - it correctly detects when the map div changes. Whenever this occurs, I run
google.maps.event.trigger(this.map, 'resize').
But this does not seem sufficient - when the div changes size, the map markers jump to an incorrect place (chrome and IE11). I've even tried adding a debugging button to run google.maps.event.trigger(this.map, 'resize') by force. When I change the div size, the markers move to the wrong place. Clicking my debug button does not do anything. But then if I slightly resize the browser window, the markers jump back to the correct place. So the trigger resize function is not sufficient. Somehow resizing the browser does something else too. Any idea what this is?

No simple ideas now.
As a dirty hack call via component ref  _onWindowResize method before triggering resize event.
If it will be not enough also change center on a small value after trigger.

I think I will add width and height props, and give an example of how to use AutoSizer control from react-virtualized https://github.com/bvaughn/react-virtualized/blob/b12bae6cddb3a604269a42b53a0b970f4a158ef0/source/AutoSizer/AutoSizer.js

Close as in todo list https://github.com/istarkov/google-map-react/issues/147


Revise server side hook configuration to support dynamic hook file extensions.

Feature Description
In the /modules/git/hook.go file each hook is hardcoded without a file extension, causing inconvenience in deploying any script git hook other than bash on Gitea.
Within modules/git/hook.go, the variable "hookNames" is defined as an array without file extensions:
var hookNames = []string{
    "pre-receive",
    "update",
    "post-receive",
}

Screenshots
No response

How does these hooks with extensions work? https://git-scm.com/docs/githooks

Hooks with extensions do work on Gitea, but currently, they do not appear on the frontend. It would be helpful to customize each hook's behavior without having to create a wrapper script that calls the script from bash.
For example, if I wanted to create a pre-receive.ps1 script inside the repository on the server, Gitea would not find it since the hook names are hard-coded. Additionally, calling the script from bash with #!/usr/bin/env pwsh without a file extension won't work either, as since PowerShell 7.2 scripts require a .ps1 file extension on windows.

Hooks with extensions do work on Gitea, but currently, they do not appear on the frontend. It would be helpful to customize each hook's behavior without having to create a wrapper script that calls the script from bash.
For example, if I wanted to create a pre-receive.ps1 script inside the repository on the server, Gitea would not find it since the hook names are hard-coded. Additionally, calling the script from bash with #!/usr/bin/env pwsh without a file extension won't work either, as since PowerShell 7.2 scripts require a .ps1 file extension on windows.

I don't think so. Gitea will create update.d, proc-receive.d, post-receive.d and pre-receive.d. You can put your scripts under these folders.

We close issues that need feedback from the author if there were no new comments for a month. :tea:


Twitter rate limiting problem?

It seems like I am unable to capture any twitter page at all. As soon as I hit "Capture" I get a white screen with the message "Too Many Requests". Is there some sort of global limit on twitter crawls?

Yes, this is a known issue. Unfortunately, not much that can be done on the hosted service.
Please use the Webrecorder Desktop app if you want to archive twitter pages.

Thanks!


New-Win32.ps1  Script failure

Great work on the amazing solution.
Thought I would configure it and have a go. Setup was a breeze.
There seems to be an issue in the very last stage (publish) where it fails to run the New-Win32.ps1.
at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
at System.Console.GetBufferInfo(Boolean throwOnNoConsole, Boolean& succeeded)
at Microsoft.Management.Service.IntuneWinAppUtil.LogUtil.PrintProgress(AppContext context)
at Microsoft.Management.Service.IntuneWinAppUtil.PackageUtil.CreatePackage(String folder, String setupFile, String outputFolder, String catalogFolder)
at Microsoft.Management.Service.IntuneWinAppUtil.Program.Main(String[] args)
WARNING: Unable to detect expected 'Deploy-Application.exe.intunewin' file after IntuneWinAppUtil.exe invocation
Creating default requirement rule
Creating additional custom requirement rules
Creating detection rules
Constructing an icon object
Creating Win32 application
C:\ADOAgent_work\1\s\Scripts\New-Win32App.ps1 : Cannot validate argument on parameter 'FilePath'. Cannot bind
argument to parameter 'Path' because it is an empty string.
At C:\ADOAgent_work_temp\21af28b3-f0b6-4c6d-b7b4-db3447f4c6d3.ps1:4 char:1

. 'C:\ADOAgent_work\1\s\Scripts\New-Win32App.ps1' -TenantID f1xxxx ...

  + CategoryInfo          : InvalidData: (:) [New-Win32App.ps1], ParameterBindingValidationException
  + FullyQualifiedErrorId : ParameterArgumentValidationError,New-Win32App.ps1

##[error]PowerShell exited with code '1'.
Finishing: New-Win32App

Hello i encounter the same issue, anyone found a solution to this?

Found a temporary workaround, will create a pull request now

@ikkhatri my solution works fine, tried it yet? any idea on how to improve it?

I guess this is related to the latest issues with the changes Microsoft made to a specific version of the wrapper-utility that didn't like the output getting redirected (I was trying to hide this in the code of the New-IntuneWin32AppPackage function). I'm about to release a new version of the IntuneWin32App module, but family stuff and moving to a new house has gotten in the way. It should be just around the corner now though, as we're finally settling in :)

I guess this is related to the latest issues with the changes Microsoft made to a specific version of the wrapper-utility that didn't like the output getting redirected (I was trying to hide this in the code of the New-IntuneWin32AppPackage function). I'm about to release a new version of the IntuneWin32App module, but family stuff and moving to a new house has gotten in the way. It should be just around the corner now though, as we're finally settling in :)

Hope everything is going all right with your moving and family stuf.
My workaround "works" but as i am moving closer to releasing a newer version of what we do with pipelines and intune, i would be interested with your solution.
No rush at this point just pure curiousness, but only if you have the time for it of course!

My fix for this issue was to make sure absolute paths are passed to New-IntuneWin32AppPackage
$IntuneAppPackage = New-IntuneWin32AppPackage -SourceFolder (Resolve-Path $SourceFolder).Path -SetupFile $AppData.PackageInformation.SetupFile -OutputFolder (Resolve-Path $OutputFolder).Path

Were you still experiencing this issue with the 1.4.4 version of the IntuneWin32App module?

Yes that's the version I've done my tests with.


[WIP] bazel: Add LLVM toolchain

Ideally we use this toolchain for compiling and can then remove the compiler from our container platform/toolchains
More immediately, adding this means we can use the clang tools (clang-format, clang-tidy, llvm-cov, etc) with bazel - this remove the host requirement, and should prevent issues around running tooling with incorrect versions (in combination with #29397 and some further dev)

Commit Message:
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

@keith i had to set the version to 14.0.0 (our current) as there is not a toolchain for ubuntu < 22 - but that should be ok for now
i also had to shift some of the setup around to get it working, not exactly ideally

there are some llvm threads about improving the release process for these binaries so I'm optimistic that part will get solved in the future. not sure what to say about the binary size problem though. That was always something we were going to have to take the hit on if we moved to this. If this was the default then we could probably stomach doing that download once since you would keep the binaries locally until they were upgraded again anyways

That was always something we were going to have to take the hit on if we moved to this.

one consequence is that we cant use this without removing from the container image otherwise ci downoads 4G x2

If this was the default then we could probably stomach doing that download once since you would keep the binaries locally until they were upgraded again anyways

its more about CI and limiting what an RBE client needs to download on a run (esp where there is no change)
for the compiler i  agree we just need to take the hit
for clang tools im not so sure - for now we can hook up the python way as it ~matches our current version
upgrading tho we will then need an upstream to be available for both the full compiler toolchain and any tooling
as we are in this situation anyway (no current upgrade path) im happy to land this and start using the python distributed tools
its gonna be a bit fragile but not much worse than current
the only other blocker i hit was that i got a compiler error when trying to use for compile toolchain - ill re-enable it so we can see

https://github.com/envoyproxy/envoy/pull/29455/files

this is probably worth another try
the llvm_toolchain has moved to bazel-contrib
there is also a potential gcc toolchain here https://github.com/f0rmiga/gcc-toolchain

I like this - this also will make upgrades of LLVM much easier


dynamic-dynamodb package cannot install with python 2.7 on t1.micro using CloudFormation template

Hello,
I'm using the CloudFormation template to install Dynamic DynamoDB, however I encounter the issue where the stack fails to create. The error points to "Failed in starting dynamicdyanmodb service", upon investigation, it's because the step to start dynamic-dynamodb service failed due to "TypeError: 'module' object is not callable"
More investigation, it turns out the TypeError is due to the need of Python 2.7, which is not updated on the t1.micro instance by default. When logging into the instance, the terminal prompts to do a "sudo yum update", which installs Python 2.7, I then change 2.7 to be the default version, install pip again to point to 2.7, then update dynamic-dynamodb package using pip repo. Finally, this would work, and I was able to successfully use the service.
However, this doesn't solve the CloudFormation stack creation failure. I am unable to insert in the template with "sudo yum -y update" as this would fail, probably because it can't call yum. So what can I that the CloudFormation stack would create smoothly? Am I doing something wrong?
Thanks,
Luke

Sorry for the duplicate, this issue is linked to #270, detailed in #27 and #274. Version 2.1.1 has fixed this issue.


Prepare release v4.4.0

Description
Release v4.4.0

Just like the latest release the e2e test will fail. This is thanks to the latest kustomization folder now points to 4.4.0 but this image don't exist. I need to come up with some good logic to manage this. But for now this will have to work/rebuild the release pipeline.
We have seen that the other PR:s have passed on the master branch and this should be fine. All other pipelines are okay and the image builds without any issues.


[bug]: I can't find a way to drag pictures from the gallery into IP-Adapter.

Is there an existing issue for this problem?

[X] I have searched the existing issues

Operating system
Windows
GPU vendor
Nvidia (CUDA)
GPU model
RTX3060
GPU VRAM
12GB
Version number
5.0.0.a1
Browser
Vivaldi	6.9.3447.41
Python dependencies
No response
What happened
In previous versions, the control layer and gallery would appear on the same page. I could drag and drop pictures into the picture input of IP-Adapter.
In the current version, the gallery and layers cannot be displayed simultaneously. So I can't drag pictures in anymore. I can only re-upload pictures.
What you expected to happen
When dragging an image from the gallery, "Create raster layer" and "Create control layer" will be displayed on the canvas. Perhaps a "Create IP-Adapter" can be added?
How to reproduce the problem
No response
Additional context
No response
Discord username
No response

Will be fixed in next release.


At around 20 (?) seconds, there is a chain menu sound effect

I strongly suspect that this is only occurring because of how we're loading the test map. If this occurs in a real scenario let's revisit.


Remove empty continuation lines.

Empty continuation lines will become errors in a future release.

Thank you for the contribution!


replit starter repo example

Pull Request Description
Added example on how we can use solana agent kit on replit
Related Issue
NA
Changes Made
This PR adds the following changes:

Added replit starter repo in examples

Implementation Details
NA
Transaction executed by agent
NA
Prompt Used
NA
Additional Notes
NA
Checklist

[ x ] I have tested these changes locally and on replit

heyo @The-x-35 instead of including the repo, I'll just include the replit link and attribute you


Brackets window blank with menus missing after "Reopen windows when logging back in"

Launch Brackets
Open the Mac OS menu and choose Shut Down.  Make sure "Reopen windows when logging back in" is checked.
Start back up the computer

Result: Brackets window is blank, and the menu bar is basically empty as well.
This is a feature in Mac OS Lion (10.7) and newer.

I had this same issue with Brackets every time it was reopened on login (Yosemite 10.10). Blank HTML, unresponsive menus. Unresponsive shortcuts. Had to quit from dock icon and then relaunch.
I downloaded the source, and ran the "setup_for_hacking.sh" script, to see if the same issue existed outside the Application folder. Now I can no longer replicate the issue. Even after removing the symlink with "restore_installed_build.sh", I can no longer replicate.
@peterflynn, Are you still having this issue?


Intellij Plugin to add Quill support

Things this plugin could do:

Code highlight with the SQL generated
Query probing
Explain plan

We might use this

any progress on this? Would love to see SQL generated :)

@cambridgemike I don't think anyone has done anything about it. I would love to see it as well.


Nibe F1145 heat pump

Hi
I have a F1145 that I want to control using Home Assistant and esphome (or modbus). Had a look inside and there are 2 controller boards in there (one for home and one for swimming pool I think) but they don't look like any of the other wiring schema's I've seen with the number 11 and 12 being used. Am I not looking in the right place? Anyone here with more Nibe HP experience that can shed a light?

Please look at the official modbus 40 documentation from nibe.

ok thx, I'll have another look. For future reference:

So I have connected terminals 10 and 11 to my esp directly (don't have the modbus 40 device)
Number 10 is connected to A  and 11 to B. These correspond to pin 1 and 3 of the esp according to the specs of the device I am using.

I have this UART setup in my esphome config

id: my_uart
rx_pin: 1
tx_pin: 3
baud_rate: 9600

The esp has ip adres <IP_ADDRESS>, home assistant is <IP_ADDRESS>
I see this in my esphome log :
[14:48:42][C][nibegw:120]: NibeGw
[14:48:42][C][nibegw:122]:  Target: <IP_ADDRESS>:9999
[14:48:42][C][nibegw:125]:  Source: <IP_ADDRESS>
[14:48:42][C][nibegw:127]:  Read Port: 9999
[14:48:42][C][nibegw:128]:  Write Port: 10000

And yet in home assistant I get:

Any tips on what to try to get this working? I have tried swapping RX and TX pins already.
Could it be a signal quality issue? Cable is only 3 meters long.

forgot to mention the heat pump goes into alarm mode if I enable the MODBUS40 accessory in the 5.2 menu so I guess something is wrong with the connection?

my board definition in esphome is :
esp32:
  variant: esp32s3
  board: esp32-s3-devkitc-1

pin definitions:
#define _HOSTNAME             "Ultra-Dongle"
#define _DEFAULT_HOSTNAME _HOSTNAME
#define OTAURL              "http://ota.smart-stuff.nl/ultra/v5/"
#define PROFILE "[ULTRA]"

//poorten
#define IO_BUTTON             0 //multifunctional flash/button

#define RGBLED_PIN           42
#define BRIGHTNESS           50 // Set BRIGHTNESS to about 1/5 (max = 255)
#define DTR_IO               17
#define RXP1                 18
#define TXP1                 -1 //no txp1
#define TXO1                 21
#define O1_DTR_IO            15
#define P1_LED               16
#define SSL_RAND              4
#define LED              	 41 //fake
#define PRT_LED           	 16

#define IO_S0_1             45
#define IO_H2O_2            46
#define IO_WATER     		46 //double
#define GPIO5               5
#define GPIO6               6
#define GPIO7               7
#define GPIO8               8
#define GPIO9               9

//ETH API
#define INT_GPIO            14
#define MISO_GPIO           13
#define MOSI_GPIO           11
#define SCK_GPIO            12
#define CS_GPIO             10

//MODBUS
#define RXPIN       		1
#define TXPIN       		3
#define RTSPIN      		2
#define BAUDRATE  		 9600

What is the tx/rx selection pin supposed to do?
I am correct that on the heat pump I have to enable the MODBUS40 accessory and not the MODBUS setting?

Rs485 chips need to know if they are transmitting or receiving, this is controlled by a pin on the chips. So it depends on what rs485 treanceiver your board is using and how it is connnected to the esp32

Might be the RTSPIN in your case, but without looking at schematics or docs for the board its unknown. Also if its active high or low matters (but that you can test)

This is the dir_pin config option in nibegw

Ok thx, I'll try with
  dir_pin:
    number: 2
    inverted: false

and inverted true and false.

I am correct that on the heat pump I have to enable the MODBUS40 accessory and not the MODBUS setting?

on the same esp hardware I have successfully connected to a Huawei solar modbys RTU using:
uart:
  - id: mod_bus
    baud_rate: 9600
    rx_pin: 1
    tx_pin: 3
    rx_buffer_size: 1024
    data_bits: 8
    stop_bits: 1
    parity: NONE

modbus:
  - uart_id: mod_bus
    id: modbus1
    flow_control_pin: 2
    send_wait_time: 200ms

modbus_controller:
  - id: huawei_inverter
    address: 0x1
    modbus_id: modbus1
    setup_priority: -10
    command_throttle: 750ms
    update_interval: 15s

So the flow control pin is indeed 2 and what NibeGW calls the dir_pin
Is there no need to set data_bits, stop_bits or parity in the UART section for NibeGW?

No usually not. It should defsult to the right thing i think.

I'm still missing something obvious I'm afraid. The heatpump goes into an alarm state the second the modbus option is enabled. I have tried with the dir_pin inverted true and false

Do you get anything logged in the esp device when you turn on modbus in the punp?

I have the F1145 as well and good working for two years.
This is my setup in NibeGW.yaml file
Set pins required for LilyGo T-CAN485 board
output:

platform: gpio
id: ENABLE_PIN # Enable the chip
pin:
number: GPIO19
inverted: true
platform: gpio
id: SE_PIN # Enable autodirection
pin:
number: GPIO17
inverted: true
platform: gpio
id: ENABLE_5V_PIN # Enable 5V pin for RS485 chip
pin:
number: GPIO16
inverted: true

Configure uart that will be used
uart:
rx_pin: GPIO21
tx_pin: GPIO22
baud_rate: 9600
Configure NibeGW
nibegw:
udp:
# The target address(s) to send data to. May be a multicast address.
target:
- ip: <IP_ADDRESS>
port: 10090
# List of source address to accept data from, may be empty for no filter
source:
  - <IP_ADDRESS>

acknowledge:
- MODBUS40
Constant replies to certain requests cabe made
constants:
- address: MODBUS40
token: ACCESSORY
data: [
0x0A, # MODBUS version low
0x00, # MODBUS version high
0x01, # MODBUS address?
]

Can you confirm your Home Assistant add-on settings?
That 10090 should match what is set in home assitant, no? If I use 10090 like in the file you shared I guess I should set it up like this in HA:
choose NibeGW

@bmwmotor can you confirm my wiring (the 10 and 11 port) is ok? I use an external power supply for the esp so I do not use the 5v from the heat pump to power the esp like you seem to do.
I understand it correctly I have to get the pump connected to the esp device before I can connect the home assistant nibe integration to the esp device?

I still worries me that the the home assistant add-on has the text "Verify that The MODBUS40 accessory has been enabled in the heat pump configuration." and my heat pump only allows me to enable a MODBUS setting in it's menu.

@scruysberghs if you get errors toggling that setting. It is the right setting.
Dont bother with home assistant settings until you can turn that setting (modbus) on without errors. It should work standalone. Turn up logging in the esphome device and look if it is receiving data and/or trying to send.
Ps. Pump supplies 12 volt, not 5.

This is how it is connected. The wires are on nr 9, 10,11 en 12.   9 and 12 are power.

You find the correct pinnumbers on the docs with the board

For the F1145 you use the RS485 connection.

In the NIBE F1145 there are 2 menu's to change:

In HA --> ESPHome go to LOGS at your heatpump. It opens a new window : click on wireless.
Then you see connection messages: UDP packet xx bit send.........  Then you know your connection works.
When not look at your router if the IP adress is correct and fixed.
When this is oke then go to the NIBE add-on in HA.  I do not remember how I filled in that ports. Just try some settings
This part in ESPHome config file is sending the messages to your HA - Nibe addon.
Configure NibeGW
nibegw:
udp:
# The target address(s) to send data to. May be a multicast address.
target:
- ip: <IP_ADDRESS>
port: 10090
# List of source address to accept data from, may be empty for no filter
source:
  - <IP_ADDRESS>

Be aware that the board stops communicating when it gets updates in HA-ESPHome. Then the F1145 gives an MODBUS error. Just cancel it on the heatpump and it will work. Good Luck

only thing in my esphome logs  is this
[12:23:10][C][nibegw:120]: NibeGw
[12:23:10][C][nibegw:122]:  Target: <IP_ADDRESS>:10090
[12:23:10][C][nibegw:125]:  Source: <IP_ADDRESS>
[12:23:10][C][nibegw:127]:  Read Port: 9999
[12:23:10][C][nibegw:128]:  Write Port: 10000
when enabling the modbus option nothing is logged and the heat pump goes into alarm.
I do not have the 5.3 menu in @bmwmotor  second picture. Not even after enabling MODBUS.

Do you have Nibe menu 5.2.4?   Scroll all the way down and activate MODBUS there

my esp device ha a SP3485EN chip so I've ordered a LillyGo board to ensure that isn't the problem

Do you have Nibe menu 5.2.4? Scroll all the way down and activate MODBUS there

Yes, that is the option that causes the alarm when activated

and when you leave it on (ignore the alarm), can you then get 5.3 ?
Here is the compleet config file. You have to change IP adress and Wifi etc .
nibegw (1).yaml.txt

@scruysberghs i wonder if you have some old variant of the f1145, since your photos of the boards dont seem to match the modbus40 images.

the pictures in my first post were the wrong boards somewhere in the back of the heat pump. Quite sure the cable now is connected to the right board like in @bmwmotor 's picture.

I found the ports in the ESPHome logfile. You need them in the HA (Nibe) add-on screens
[13:43:36][C][nibegw:122]:  Target: <IP_ADDRESS>:10090
[13:43:36][C][nibegw:125]:  Source: <IP_ADDRESS>
[13:43:36][C][nibegw:127]:  Read Port: 9999
[13:43:36][C][nibegw:128]:  Write Port: 10000

Could your pump have a really old firmware version?

that could well be the case. We are trying to upgrade it now (it never had a firmware update since it was installed)


Bump govspeak version in postgres dockerfile

https://github.com/alphagov/govspeak/tags

Could be done as part of https://github.com/alphagov/govuk-knowledge-graph-gcp/issues/558

Being done as one of the many parts of #552. We ought to configure dependabot to keep reminding us.


Adding hyperlinks from concepts/widgets to examples/widgets

Hello,
I added hyperlinks to the examples for each widget that has an example because I thought it would be useful.
I hope this is the right way to suggest this change.
Good day to you.

Thanks for the PR (and welcome to Ratatui). I changed it a little so that the links were from the root rather than ../../ and merged.


Use size_t * instead of unsigned int * for JsSerializeScript's bufferSize param

See https://github.com/Microsoft/ChakraCore/blob/master/lib/Jsrt/ChakraCommon.h#L1113
Since this is a buffer size, the most portable type for the parameter would be size_t instead of unsigned int *.
@liminzhu: @digitalinfinity mentioned this might not be possible because of compat, but could you confirm that and see if we might want to make this change anyway?

Before we discuss it, @digitalinfinity I remember signature was taking unsigned long * instead of unsigned int * and saw you changed it in the https://github.com/Microsoft/ChakraCore/pull/766. It seems to break compat. Any reason why you did this change?

size_t won't be available in Linux; sizeof(long) is different from sizeof(int) in Linux. changing from unsigned long to unsigned int shouldn't change backwar compat. @liminzhu what is your concern?

size_t won't be available in Linux..

AFAIK We do use many other 'aliased' types. 'size_t' is available in Linux and it's not different than others. In case we have a problem with these aliases are represented with different sub types on varying platforms, that's something else.

Yep, size_t is in the standard. I don't think there's any problem there.

Do we need size_t if we don't support byte code buffer >= 2^32?

Maybe we should define a ChakraByteCodeBufferSize type (or something less verbose) to capture the semantic info about the type (the fact that regardless of platform, we have a max buffer size), so that this question doesn't keep coming up?

The c++ way to do it would be to do a using size_type = std::size_t; (for pre-C++11: typedef std::size_t size_type;) on the type itself, and then use the type alias on all the methods. If you're going to do that however I would consider also considering defining a buffer_traits template that knows how to get all this information or what to fall back to if the aliases aren't defined for that buffer type. I would consider reaching out to the VC++ Libraries team for guidance here as they've had to deal with these issues quite a lot and may be able to suggest coding standards around these sorts of issues.

Closing this. ChakraCore no longer asks for buffer size.


Make pop view animation duration customizable?

Great library - thank you! I'm wondering if the animation duration of the pop view can be made a config. 0.1s seems really short and it'd be great if it's customizable.
https://github.com/tilltue/TLPhotoPicker/blob/bdb091f59aed6ad94b7bd7f64eb9f996f27f2825/TLPhotoPicker/Classes/TLAlbumPopView.swift#L32
Speaking of which, there's a white (light gray?) divider line in black mode. Maybe it can be changed to some darker shade?

Hi, @danqing
I added configure for popup. ( version 1.9.4 )
        var configure = TLPhotosPickerConfigure()
        configure.popup = [PopupConfigure.animation(3)]
        //...

for Example
CustomBlackStylePickerViewController.swift
class CustomBlackStylePickerViewController: TLPhotosPickerViewController {
    override func makeUI() {
        super.makeUI()
        //...
        self.albumPopView.tableView.separatorColor = (your color) // 👈
    }

That's great to know - appreciate the quick turnaround!
For the separator, do you think it makes sense to default the black style color to some darker shade? (great to be able to customize it for sure)


Feature proposal: add a monthly view

Via a monthly view a user could easily see expenses and income.

Reporting has been now added to the app


refactor internal helpers

Great, thank you! I left a last comment.

@allyraza Thank you very much! :smiley:


Use Python 3.6 to generate docs

What was wrong?
Our live doc site is currently broken not showing any of the API docs as seen here.

This PR fixes it as seen on the following screenshot (notice the URL. This is live on a special doctest branch that I pushed into this repo but will delete again now)

How was it fixed?
Apparently readthedocs had issues building our API docs with Python 3.5 (which should work in general, trio uses Python 3.5 for rtd as well and has no issues with the autodoc feature). Instructing readthedocs to build with Python 3.6 instead fixes the issue.
Cute Animal Picture

Maybe python 3.6 required because of trinity/p2p code being python3.6 only due to type annotations?

@pipermerriam nope, the errors where in evm but probably not worth following up.


Some generated code is missing from file

When Plandex runs it generates code in the terminal. I thereafter apply the update and only parts of the generated code is in the file.
This is in a new project with no files to start with.
The file it generated
main.txt
Plandex output:
$gtrends_analysis$ plandex new
✅ Started new plan draft and set it to current plan
plandex (l)oad  👉 load files, dirs, urls, notes or piped data into context
plandex (t)ell  👉 describe a task, ask a question, or chat
plandex (pl)ans  👉 list plans
plandex (cu)rrent  👉 show current plan
$gtrends_analysis$ plandex tell
✅ Context is up to date
💬 User prompt 👇
I want to create a python3 script that does the following.
The inputs should be hardcoded in the beginning of the script
Inputs: data csv file, start_date, end_date defaults  to current day, N_overlap
days that is overlapped, keywords file with a list of keywords to get from
google trends.
outputs: updated data csv file
the csv file should be named data_gtrend.csv
Makes sure a csv file that contains keywords from google trends is up do date.
The data should be fetched in batches that corresponds to calendar quartely
interval i.e. they start at the calendar quarter and ends at the calendar
quarter. The latest quarter will of course end at the current date. If the last
quarter that is in the data file has not ended it should be removed and replaced
with new data. You can add a blank row in the csv file to keep track of where
the last quarter begins.
The data should be normalized using overlaping days. This means the data needs
to be fetched N days before the quarter begins. This will be used to scale the
new data so it has the same amplitude as theold data. All the non empty overlap
days except should be used to calculate correction factor. However they should
be divided into 4 parts and then compared as a sanity check so they are within a
maxumum percentage distance. After that the average correcting factor should be
calculated and used to scale the new data.
print information about the current place in the fetching and eventual errors.
The oldest data should be fetched first.
The columns should be
date, keyword1, keyword2 etc
🤖 Plandex reply 👇
Let's break up this task into smaller subtasks to create a Python script
that fetches Google Trends data for given keywords, normalizes the data
using overlapping days, and updates a CSV file accordingly. The script will
handle inputs such as the data CSV file path, start and end dates, the
number of overlap days, and a file containing keywords. The output will be
an updated CSV file named  data_gtrend.csv .

Set up the Python environment and install necessary libraries: Determine
the libraries needed for CSV file manipulation, fetching Google Trends data,
and handling dates.
Define hardcoded inputs: Code the section where inputs like the data CSV
file path, start and end dates, the number of overlap days, and the keywords
file path are hardcoded.
Read keywords from the keywords file: Implement the functionality to read
keywords from the provided file.
Fetch Google Trends data: Develop the function to fetch Google Trends
data for the given keywords in quarterly batches, including handling the
overlap days for normalization.
Normalize the fetched data: Code the logic to normalize the fetched data
using the overlap days, including the division into four parts for a sanity
check and calculating the average correction factor.
Update the CSV file: Implement the functionality to update the existing
CSV file with the new, normalized data, ensuring the correct format and
handling of quarters.
Logging and error handling: Add print statements throughout the script to
inform the user of the current process stage and handle any potential errors
gracefully.

Given the complexity and the need for thorough implementation, we'll start
with the first subtask: setting up the Python environment and installing
necessary libraries. This involves identifying the libraries required for
CSV manipulation, fetching Google Trends data, and date handling. We'll need
pandas  for CSV file operations,  pytrends  for accessing Google Trends
data, and possibly  datetime  for handling dates, which is part of the
standard library.
Next, we will define hardcoded inputs in the script.
👉 To continue with the plan, we'll start by defining the hardcoded inputs
in our Python script. This includes the path to the data CSV file, the start
and end dates (with defaults to the current day), the number of overlap days
(N_overlap), and the path to the file containing the list of keywords.
• main.py:
import pandas as pd
from datetime import datetime, timedelta
from pytrends.request import TrendReq

# Hardcoded inputs
data_csv_path = 'data_gtrend.csv'
start_date = datetime(2020, 1, 1)  # Example start date, adjust as needed
end_date = datetime.now()  # Defaults to the current day
N_overlap = 5  # Number of overlap days
keywords_file_path = 'keywords.txt'

# Initialize Google Trends API
pytrends = TrendReq(hl='en-US', tz=360)

Next, we'll implement the functionality to read keywords from the provided
file. This step is crucial as these keywords will be used to fetch data from
Google Trends.
• main.py:
def read_keywords(file_path):
    with open(file_path, 'r') as file:
        keywords = [line.strip() for line in file.readlines()]
    return keywords

keywords = read_keywords(keywords_file_path)

After setting up the initial inputs and reading the keywords, the next step
involves fetching Google Trends data. This requires handling the logic for
quarterly data fetching, including the overlap days for normalization. We'll
also need to ensure that we're fetching data in batches that correspond to
calendar quarterly intervals.
• main.py:
def fetch_quarterly_trends(keywords, start_date, end_date, N_overlap):
    # Calculate the start of the current quarter
    current_quarter_start = start_date - timedelta(days=start_date.day - 1)
    current_quarter_start = current_quarter_start -

timedelta(months=(current_quarter_start.month - 1) % 3)
    # Adjust the start date for overlap
    adjusted_start_date = current_quarter_start - timedelta(days=N_overlap)

    # Placeholder for fetched data
    trends_data = []

    # Fetch data in quarterly batches
    while adjusted_start_date < end_date:
        quarter_end = adjusted_start_date + timedelta(days=90)  #

Approximate quarter length
if quarter_end > end_date:
quarter_end = end_date
        # Fetch data for the current batch
        pytrends.build_payload(keywords,

timeframe=f'{adjusted_start_date.strftime("%Y-%m-%d")}
{quarter_end.strftime("%Y-%m-%d")}')
data = pytrends.interest_over_time()
        trends_data.append(data)

        # Move to the next quarter
        adjusted_start_date = quarter_end

    return pd.concat(trends_data)

# Placeholder for normalization and CSV update logic

This code snippet sets up the basic structure for fetching Google Trends
data in quarterly intervals, including the overlap days. The next steps will
involve normalizing the fetched data using the overlap days and updating the
existing CSV file with the new, normalized data. These steps will handle the
normalization logic, including the division into four parts for a sanity
check, calculating the average correction factor, and updating the CSV file
accordingly.
Next, we will implement the normalization of the fetched data and update the
CSV file.
👉 Continuing from the data fetching, the next step involves normalizing the
fetched data using the overlap days. This process includes dividing the
overlap days into four parts for a sanity check, ensuring they are within a
maximum percentage distance, and then calculating the average correction
factor to scale the new data.
• main.py:
def normalize_data(fetched_data, N_overlap):
    # Assuming fetched_data is a DataFrame with the first column as 'date'

and the rest as keywords
# The data is already sorted by date in ascending order
    # Calculate the correction factor for each keyword
    correction_factors = {}
    for keyword in fetched_data.columns[1:]:  # Skip the 'date' column
        # Divide the overlap period into four parts for sanity check
        overlap_data = fetched_data.head(N_overlap)[keyword]
        parts = np.array_split(overlap_data, 4)
        part_means = [part.mean() for part in parts]

        # Sanity check: ensure the means are within a maximum percentage

distance
max_distance = 0.2  # 20% maximum distance
if all(abs(part_means[i] - part_means[i+1])/part_means[i] <
max_distance for i in range(len(part_means)-1)):
# Calculate the average correction factor
correction_factors[keyword] = sum(part_means) /
len(part_means)
else:
print(f"Sanity check failed for {keyword}. Skipping
normalization.")
correction_factors[keyword] = 1  # No scaling
    # Apply the correction factor to scale the new data
    for keyword, factor in correction_factors.items():
        fetched_data[keyword] *= factor

    return fetched_data

After normalizing the data, the final step is to update the existing CSV
file with the new, normalized data. This involves checking for the last
quarter in the existing data, removing it if it has not ended, and appending
the new data.
• main.py:
def update_csv(data_csv_path, new_data):
    # Try to read the existing data
    try:
        existing_data = pd.read_csv(data_csv_path)
    except FileNotFoundError:
        existing_data = pd.DataFrame()

    # If existing data is not empty, remove the last quarter
    if not existing_data.empty:
        last_quarter_start = existing_data['date'].max() -

pd.offsets.QuarterBegin(startingMonth=1)
existing_data = existing_data[existing_data['date'] <
last_quarter_start.strftime('%Y-%m-%d')]
    # Append the new data
    updated_data = pd.concat([existing_data, new_data])

    # Save the updated data to CSV
    updated_data.to_csv(data_csv_path, index=False)
    print(f"Data updated successfully in {data_csv_path}")

# Placeholder for the main logic that orchestrates the fetching,

normalization, and updating process
This completes the implementation of the core functionality for the Python
script. The remaining task is to orchestrate the fetching, normalization,
and updating process within a main function or script block that utilizes
the defined functions to achieve the desired outcome.
All tasks have been completed.
─────────────────────────────────────────────────────────────────────────────────────────────────────
🏗  Built plan
📄 main.py ✅
plandex (ch)anges  👉 review plan changes
plandex (ap)ply  👉 apply plan changes to project files
plandex log  👉 show log of plan updates
plandex rewind (rw)  👉 rewind to a previous state
$gtrends_analysis$ plandex log
📝 Update 99a16a0 | Today | 7:23:05pm CEST
🏗  Build pending changes
✏  Implement data normalization and CSV update logic
• edit → main.py
📝 Update 1ed4daf | Today | 7:22:53pm CEST
Message #4 | 🤖 Plandex reply | 656 🪙
📝 Update 73baf89 | Today | 7:22:30pm CEST
🏗  Build pending changes
✏  Implemented initial setup, keyword reading, and quarterly data fetching
• new file → main.py
• edit → main.py
📝 Update 3ec8f3e | Today | 7:22:25pm CEST
Message #3 | 🤖 Plandex reply | 697 🪙
📝 Update 449ace1 | Today | 7:21:56pm CEST
Message #2 | 🤖 Plandex reply | 409 🪙
📝 Update e8cc20a | Today | 7:21:41pm CEST
Message #1 | 💬 User prompt | 320 🪙
$gtrends_analysis$ plandex ap
Apply changes to 1 file? (y)es | (n)o> y
✅ Applied changes, 1 file updated

Thanks

@JLA4444 thanks for reporting. I'll investigate. Also please read limitations and guideance section in the readme.
I'm working on improving file updates as much as possible, but they still aren't perfect. In cases where changes are applied incorrectly, I'd recommend using plandex changes to reject the change and apply it manually--you can either copy it from the plandex changes TUI or if it's incorrect there as well, use plandex convo to output the full conversation and copy it from there.
And I'll of course continue working to get updates as reliable as possible.

@JLA4444 these kinds of issues should be much rarer in 0.9.0 - https://github.com/plandex-ai/plandex/releases/tag/cli%2Fv0.9.0


Automated cherry pick of #2429: Fix the deadlock between exporter and conntrack polling goroutines

Cherry pick of #2429 on release-1.2.
#2429: Fix the deadlock between exporter and conntrack polling goroutines
For details on the cherry pick process, see the cherry pick requests page.

Codecov Report

Merging #2446 (61f3575) into release-1.2 (b1c7a5b) will decrease coverage by 4.54%.
The diff coverage is 100.00%.

@@               Coverage Diff               @@
##           release-1.2    #2446      +/-   ##
===============================================
- Coverage        59.87%   55.32%   -4.55%
===============================================
  Files              284      284
  Lines            22178    22170       -8
===============================================
- Hits             13278    12266    -1012
- Misses            7474     8585    +1111
+ Partials          1426     1319     -107

Flag
Coverage Δ

kind-e2e-tests
38.79% <100.00%> (-8.30%)
:arrow_down:

unit-tests
42.01% <66.66%> (+0.03%)
:arrow_up:

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

Impacted Files
Coverage Δ

.../flowexporter/connections/conntrack_connections.go
80.00% <100.00%> (+1.31%)
:arrow_up:

pkg/agent/flowexporter/exporter/exporter.go
79.06% <100.00%> (-1.07%)
:arrow_down:

pkg/agent/flowexporter/flowrecords/flow_records.go
83.01% <100.00%> (+4.58%)
:arrow_up:

pkg/apis/controlplane/v1beta2/helper.go
25.00% <0.00%> (-75.00%)
:arrow_down:

pkg/agent/controller/networkpolicy/packetin.go
4.36% <0.00%> (-68.45%)
:arrow_down:

...formers/externalversions/security/v1alpha1/tier.go
0.00% <0.00%> (-64.29%)
:arrow_down:

...ers/externalversions/core/v1alpha2/clustergroup.go
0.00% <0.00%> (-64.29%)
:arrow_down:

...s/externalversions/core/v1alpha2/externalentity.go
0.00% <0.00%> (-64.29%)
:arrow_down:

...xternalversions/security/v1alpha1/networkpolicy.go
0.00% <0.00%> (-64.29%)
:arrow_down:

...versions/security/v1alpha1/clusternetworkpolicy.go
0.00% <0.00%> (-64.29%)
:arrow_down:

... and 64 more

/test-all

/test-networkpolicy


DBZ-6336 MySql in debezium-parser-ddl does not support with keyword p…

After mysql5.7, with cte analysis is supported, but MySqlParser.g4 does not currently support with keyword analysis.
Now improve MySqlParser.g4 to support with cte parsing

@caicancai Applied, thanks a lot! Could you please prepeare a PR for https://github.com/antlr/grammars-v4/tree/master/sql/mysql/Positive-Technologies and add a comment here with the link to it to make sure it is contributed back to upstream?

@jpechane  Ok,thank you for your reminder，i have given prepeare a PR for https://github.com/antlr/grammars-v4/tree/master/sql/mysql/Positive-Technologies


WebGL errors in Chrome 80

The triangle sample looks fine but triggers a cascade of the following two GL errors:
GL_INVALID_ENUM: invalidateFramebuffer: invalid attachment
GL_INVALID_OPERATION : glDrawElements: Source and destination textures of the draw are the same.
These errors seem to be triggered even when post-processing is disabled, which suggests they are unrelated to #2263.

Somewhat unrelated, but helmet.html also triggers a warning:
RENDER WARNING: there is no texture bound to the unit 11
I think this is due to the ubershader mode, which conditionally samples from non-existent textures.  While in ubershader mode, gltfio is meant to provide 1x1 dummy textures to mitigate this warning.

It is surprising to me that I see 4 draw calls on every frame: the first one is the app's, but the subsequent 3 draw calls must be coming from Filament, despite the fact that post-processing is disabled.

Closing this ticket, most of the warnings are now fixed.  Opened #2338 for a related issue.


Not Working Properly.

When I click on extension it shows something like this.

Please look into this matter.
@vigneshpalainsamy @asinghal279 @omkar-pote @raghav0108 @vignespacap

Hi @Preetmungara . Thanks for notifying this issue.
kindly verify once again you followed with bellow steps.
https://github.com/omkar-pote/count-cuckoo#building-locally
- git clone https://github.com/omkar-pote/count-cuckoo.git
- npm i
- npm run build

https://github.com/omkar-pote/count-cuckoo#installing-on-chrome
-Go to Chrome extensions chrome://extensions/
-Enable Developer mode
-Select the "Load Unpacked" option and point to the build directory
Also, Kindly help us with more info.
Your System Os and Browser Details.

Issue was this [on windows].

Adding this "build": "set "INLINE_RUNTIME_CHUNK=false" && react-scripts build",
in place of    "build": "INLINE_RUNTIME_CHUNK=false react-scripts build", in package.json worked for me.
Works like charm 🔥🔥

Thanks @Preetmungara


Crashes with Cerbon's Better Becaons if BeaconsForAll isn't installed.

Mod Name
Cerbon's Better Beacons
Mod Homepage
https://www.curseforge.com/minecraft/mc-mods/cerbons-better-beacons-forge
Describe the bug
The game crashes if you attempt to run connector with CBB unless you also have BFA installed.
Steps to reproduce

Try running Connector with Cerbon's Better Beacons.
Crashes

...
Logs
https://mclo.gs/UpC1Zo9
Additional context
https://github.com/CERBON-MODS/CERBONs-Better-Beacons-FORGE/issues/4
This was reported separately by someone else, but I can also confirm to reproduce the issue independently.

Seems like the Fabric mixin fork shipped by Connector is a bit more strict about resolving indy refrences in mixins

I attempted to run the latest beta (v28) with and without BFA installed, and instead it caused a startup crash for me:
crash-2023-12-02_15.02.20-client.txt
Crash doesn't occur when I switch back with v27. Also doesn't occur if I have IPN removed (that's a false mod suspect)
crash-2023-12-02_14.38.08-client.txt

beta 29 fixed this crash issue!


Add Connection Parameter to Select Meisterplan Region

user has to select region before OAuth sign-in process with Meisterplan starts
use region connection parameter to navigate to the Meisterplan API Gateway
remove script that did the redirect to the API Gateway before
remove all security properties from swagger file. This may fix the issue that the authorization header is not send with the request

When submitting a connector, please make sure that you follow the requirements below, otherwise your PR might be rejected. We want to make you have a well-built connector, a smooth certification experience, and your users are happy :)
If this is your first time submitting to GitHub and you need some help, please sign up for this session.

[ ] I attest that the connector doesn't exist on the Power Platform today. I've verified by checking the pull requests in GitHub and by searching for the connector on the platform or in the documentation.
[ ] I attest that the connector works and I verified by deploying and testing all the operations.
[ ] I attest that I have added detailed descriptions for all operations and parameters in the swagger file.
[ ] I attest that I have added response schemas to my actions, unless the response schema is dynamic.
[ ] I validated the swagger file, apiDefinition.swagger.json, by running paconn validate command.
[ ] If this is a certified connector, I confirm that apiProperties.json has a valid brand color and doesn't use an invalid brand color, #007ee5 or #ffffff. If this is an independent publisher connector, I confirm that I am not submitting a connector icon.

If you are an Independent Publisher, you must also attest to the following to ensure a smooth publishing process:

[ ] I have named this PR after the pattern of "Connector Name (Independent Publisher)" ex: HubSpot Marketing (Independent Publisher)
[ ] Within this PR markdown file, I have pasted screenshots that show: 3 unique operations (actions/triggers) working within a Flow. This can be in one flow or part of multiple flows. For each one of those flows, I have pasted in screenshots of the Flow succeeding.
[ ] Within this PR markdown file, I have pasted in a screenshot from the Test operations section within the Custom Connector UI.
[ ] If the connector uses OAuth, I have provided detailed steps on how to create an app in the readme.md.

Hello @itd-sk,
Congratulations, your Pull Request is approved and merged. Please update your submission in ISV Studio by using the latest commit ID.
Let us know if you have any questions.


Example notebook of OpenAI got ConstraintException

The bug
When using latest version of guidance '0.1.15' and run notebook multiple times ( < 10), it failed with error:
ConstraintException: The model attempted to generate b'THE CAPITAL OF GREENLAND IS NUUK.<|inval...' after the prompt `b'...enland?<|im_end|>\n<|im_start|>assistant\n'`, but that does
not match the given grammar constraints! Since your model is a remote API that does not support full guidance
integration we cannot force the model to follow the grammar, only flag an error when it fails to match.
You can try to address this by improving the prompt, making your grammar more flexible, rerunning with
a non-zero temperature, or using a model that supports full guidance grammar constraints.

This exception also got raised when using select, json etc. gen functionalities with OpenAI models.
To Reproduce
Note: you might need to run multiple times before getting the ConstraintException.

gen call:

from guidance import system, user, assistant

# this relies on the environment variable OPENAI_API_KEY being set
gpt35 = models.OpenAI('gpt-3.5-turbo')

lm = gpt35

with system():
    lm += "You only speak in ALL CAPS."

with user():
    lm += "What is the captial of Greenland?"

with assistant():
    lm += gen('answer', max_tokens=20)

another example:

gpt = models.OpenAI("gpt-4")

with system():
    lm = gpt + "You are a cat expert."

with user():
    lm += "What are the smallest cats?"

with assistant():
    lm += gen("answer", temperature=0.001, max_tokens=15)

select:

with system():
    lm = gpt + "Answer to the questions only with YES or NO."
with user():
    lm += """Life its Hard? """
with assistant():
    lm += "Answer = "

##USE SELECT OR GEN
with assistant():
    lm += select(['YES','NO'], 'answer')

System info (please complete the following information):

OS (e.g. Ubuntu, Windows 11, Mac OS, etc.): Mac OS, jupyter notebook, this doesn't matter as it was also tested in other environment on my server (Linux).
Guidance Version (guidance.__version__): 0.1.15
OpenAI version: 1.30.4

Hmm, we really shouldn't be erroring on unconstrained gen calls like this. It's possible and likely to happen when you use select or json, as we can't meaningfully enforce a grammar on a remote endpoint, so all we can do is check to see if the model followed the grammar. But for an unconstrained gen, we shouldn't have this problem. Thanks so much for pointing this out!

@Harsha-Nori this is something our notebook build fails on quite a bit

Our notebook build appears to be healthier after #879 , but since that just added extra log messages, we've probably got a race condition somewhere :-/


Automated cherry pick of #5854: fix(region): Add CustomizeListFilter for snapshotpolicy

Cherry pick of #5854 on release/3.2.
#5854: fix(region): Modify display field's name for snapshotpolicy

Codecov Report

Merging #5855 into release/3.2 will decrease coverage by 0.00%.
The diff coverage is 0.00%.

@@               Coverage Diff               @@
##           release/3.2    #5855      +/-   ##
===============================================
- Coverage         7.02%    7.02%   -0.01%
===============================================
  Files              641      641
  Lines           108712   108726      +14
===============================================
  Hits              7640     7640
- Misses          100486   100500      +14
  Partials           586      586

Flag
Coverage Δ

#aFlag
7.02% <0.00%> (-0.01%)
:arrow_down:

Impacted Files
Coverage Δ

pkg/compute/models/snapshotpolicy.go
3.34% <0.00%> (-0.10%)
:arrow_down:

Continue to review full report at Codecov.

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


nrf53: fix RTC pretick power usage for events on RTC0

Fixes #63789
For RTC0 events the RTC1 pretick event was not cleared what caused the WDT to be not stopped. This resulted in increased power usage.

Pushed additional commit, marking DNM for a while

@ankuns @anangl there doesn't seem to be any "Fixes #12345" entry. Please add one (required at this stage of the release process if you want this for 3.5)

@jhedberg edited description, added "Fixes #63789"


Widget: order by START date/time if DUE and PRIORITY are indistinguishable

Currently, the widget order is as follows:

by DUE (early due first, no due last)
by PRIORITY (high priority first, no priority last)
by CREATED (new tasks first)

In order to have a better control of tasks with the same (or no) DUE and PRIORITY, I propose to add a new condition START after PRIORITY:

by DUE (early due first, no due last)
by PRIORITY (high priority first, no priority last)
by START (early start first, no start last)
by CREATED (new tasks first)

Then, already started tasks are listed before tasks which haven't been started yet, and tasks which have been started a long time ago come before recently started tasks. This helps the user to keep the number of active tasks low in order to concentrate on finishing already started tasks. Furthermore, it allows to influence the order of tasks without DUE more fine-grained than using priorities.

merged into staging


fix: ensure that per-element investment cost are calculated accurately

Purpose

As originally mentioned in https://github.com/Breakthrough-Energy/PowerSimData/pull/297, fix AC investment cost calculations so that we don't use underground line costs when not appropriate (lines of appropriate rating.voltage that are entirely within NEISO and NYISO J-K regions).
Add tests of individual element cost calculations, which illustrate some bugs (e.g. branches within the SOCO [Southern Company] region were getting NaN cost estimates, and underground line costs were being applied inappropriately to some branches).
Fix the bugs.
Cleanup from https://github.com/Breakthrough-Energy/PowerSimData/pull/391, where we enabled the cost calculation of nuclear generators, but didn't ensure that costs would be calculated for costs cases other than "Moderate".

These code fixes are necessary for https://github.com/Breakthrough-Energy/PowerSimData/issues/439.
What the code is doing

In powersimdata/tests/mock_grid.py: we allow the user to pass data to the MockGrid.storage["StorageData"] field for tests.
In powersimdata/design/investment/tests/test_investment_costs.py: we add tests of the individual element cost calculations (which initially fail)
In powersimdata/design/investment/investment_costs.py:

We add a new function merge_keep_index, which lets us perform a dataframe merge without losing the index of the first dataframe (as demonstrated in the comments on https://github.com/Breakthrough-Energy/PowerSimData/issues/439), and we use this wherever we had been doing a pandas.DataFrame.merge.
We add a new function append_keep_index_name which lets us perform an append without losing the name of the first dataframe index, which we need for merge_keep_index.
In _calculate_ac_inv_costs:

We update the select_mw function to ensure that we only select the cost class for underground lines if the line is entirely within one of the two zones which have significant shares of underground lines (based on the data in the LineRegMult.csv file).
We generalize the get_transformer_mult function to be get_branch_mult, so that it can operate on all types of branches to return the average of the regional multiplier for the to & from ends of the branch. In the process, we fix a bug where if a Line has endpoints with no regional multiplier defined, it would return a NaN multiplier and then this branch would not count towards the total cost (and not be meaningful when results are returned un-summed).
Adding data to the branch data frame is moved after adding data to the bus data frame, so that we can take advantage of knowing the bus regions within the new get_branch_mult function.

In calculate_gen_inv_costs:

We modify the load_cost function so that if we request a cost case that is not "Moderate", then we copy the "Moderate" nuclear costs into what we will return, and warn the user. This allows us to calculate costs for the "Advanced" and "Conservative" cost cases without having NaNs for nuclear.
We ensure that the indices of the storage units that we append to the plant data frame are preserved, so that there is no collision with the plant IDs in the returned non-summed data frame. We generate these non-colliding indices automatically whenever we add storage to the Grid, but they live in the StorageData storage table due to legacy design around the Matpower/MOST schema)

Testing
Unit tests are added which detect the bugs, and the changes fix the bugs so that the tests run successfully. This branch has also been tested (with some additional code in the daniel/congestion_cost_upgrade branch) to successfully analyze per-branch upgrade costs for a real Eastern scenario.
Time estimate
30-60 minutes. The investment_cost module is big, and requires aggregating many different tables to produce results, so it takes a while to trace through all the places where bugs can be introduced. There is definitely room to refactor this module to be structured a bit better, but that is of lower priority than fixing these bugs, which need to be addressed for the trainings that are coming up soon.

Thanks for updating the tests and the Mock object. The logic is clear and all tests passed. I think this is good to go after some minor concerns are addressed.

I found another edge case that also needs fixing: if there are no transformers in the upgraded branches then we get an error caused by trying to apply on an empty data frame. I will add some tests to detect the problem and a fix for it, and let you know when they're ready.

I found another edge case that also needs fixing: if there are no transformers in the upgraded branches then we get an error caused by trying to apply on an empty data frame. I will add some tests to detect the problem and a fix for it, and let you know when they're ready.

New tests and fixes are added, and all comments are addressed (I think).


[Android] icon inline does not work

var React = require('react-native')
var Icon = require('react-native-vector-icons/FontAwesome');
var {
  AppRegistry,
  Image,
  Text,
  View,
  ScrollView,
  StyleSheet,
} = React

export default React.createClass({
  render:function(){
    return (
      <View style={{flex:1,backgroundColor:'white'}}>
        <Text><Icon name="rocket" size={30} color="black"/>{'<='}Icon inlineTest</Text>

        <Icon name="rocket" size={30} color="black"/>
        <Text> {'^^^^^'}only Icon</Text>
      </View>
    )
  }
})

I also use the lastest react-native source code to build  :-）

I'm aware of this bug :disappointed: and unfortunately it will be hard to fix it properly until React Native core supports custom fonts on android. I know what the problem is, I can give it another round and see if I can circumvent it for the time being.

Flexbox layout should work though! :smile:

Yeah,  thanks :)


Can't delete tasks and edit task. Only create

Describe the bug
I can't delete tasks.
When editing, they are not edited, but a new one is created
Language is Russian
To Reproduce
Steps to reproduce the behavior:

Go to 'View Issue/View Issue Details/Things to do'
Write name of task, Click on 'Add'
Click on 'Remove'
Nothing happens
Click 'Edit task'
After edit, the old one remains and a new one is created with the edited name

Expected behavior
I assumed that editing only changes the name
Screenshots
screenshots below
Environment (please complete the following information):

MantisBT version: 2.24.4
Plugin version: 2.4.0
Database: MySQL 5.7.27
PHP 5.7.27
Chrome browser 97.0.4692.71

Additional context
Add any other context about the problem here.

@VadimVadimych thank you very much for the detailed information you have given!
Unfortunately I was not able to reproduce this bug. Would you mind repeat the steps mentioned above with the dev console opened? Maybe we could find a clue within "console" or "network" tab.

Maybe we can find a clue in the "console" or "network" tab. Thanks in advance.
output in the console at the moment of deletion:
new.kof-palitra.ru-1642514711660.log
and from 'network':

This is weird 🤔 I will try to find the cause and fix the problem as soon as possible, but in the meantime could you please show me contents of the "Payload" tab as well?

I don't understand it myself.

Thank you very much. At least now we know that it is related to front-end.

I translated strings into Russian, in case someone might need it
strings_russian.txt

Hey @VadimVadimych 👋🏻 Once again thanks for reporting this issue and all your help.
I've just created a new release v2.5.0. It contains changes which I hope fixes this problem. The Russian language is included as well 😉
Please verify if this fixes the bug because I was not able to reproduce it.

Hello Andrzej!
Thank you so much for your time on this problem!
Everything is working perfectly!
Thank you again.


com.google.android.material.textview.MaterialTextView cannot be cast to com.skydoves.powerspinner.PowerSpinnerView

Please complete the following information:

Library Version [e.g. v1.0.0]
Affected Device(s) [e.g. Samsung Galaxy s10 with Android 9.0]

Describe the Bug:
Add a clear description about the problem.
Expected Behavior:
A clear description of what you expected to happen.

Library Version [e.g. v1.2.4]
while i try to open the layout and get the view by viewbinding i have this problem
java.lang.ClassCastException: com.google.android.material.textview.MaterialTextView cannot be cast to com.skydoves.powerspinner.PowerSpinnerView

Dup #164


PostgreSQL - Role list in the wrong place

Description
Dear DBeaver team,
This is just a graphical issue, but can cause a misunderstanding to the DBeaver's users. In PostgreSQL, the users and roles (which are the same) are created at DBMS level and not within the databases.

Each database shows exactly the same role info, which is correct, but should be displayed just once at the same level that the databases.
As I said, this is just a GUI issue, but my students are getting confused (specially when they see that psql and pgAdmin4 displays the info in the correct way).
Kind regards and thanks in advance,
Fher
DBeaver Version
Community Edition 22.2.5
Operating System
Ubuntu Desktop 22.04.1
Database and driver
PostgreSQL v14.5
PostgreSQL JDBC Driver 42.5.0
Steps to reproduce

Create a PSQL connection.
Connect to the PSQL.
Expand the navigation tree.
Check that "roles" is not at root level (as it should be, sibling to the databases).
Check that "roles" is repeated within every database (where should not be).

Additional context
No response

I concur.  The strange thing is that up until, I don't know the last update, the Roles folder was shown in the correct database cluster level, not in an individual database.
I was forced to use pgAdmin 4 while I was searching around the internet for some reason why dBeaver was suddenly broken.
Annoying as it it, at least I know where the devs hid this vital piece of information and I can stop looking though every one of the probably hundreds of settings in dBeaver.
Thank you @FherStk for pointing it out.
I hope this can be fixed ASAP.  As previously mentioned PostgreSQL, unlike some other RDBMS, roles are global to the cluster.
(running dBeaver CE 22.2.5..202211202223 under Windows 10)
Roles apparently missing:

Role in the wrong location:

Hello.
For many years DBeaver had Role Manager on the database level. But then we tried to switch it to the data source level because, as you said, the role belongs to the data source.
After it, we had some issues: https://github.com/dbeaver/dbeaver/issues/14703.
There is no database choice in the Role manager, and it works poorly for data sources with many databases.
So we returned Role Manager on the database level.
We must do it properly if we return it to the data source level. And this is a complex task.

@LonwoLonwo,
Thanks for jumping into the conversation.  I'm not sure how other RDMS handle it, but there isn't supposed to be a database choice in the role manager in PostgreSQL, at least not insofar as what database the role belongs to.  There is one role per cluster, that role can have access to none of the databases, some of the databases, or all of the databases.  Additionally, the privileges assigned to a particular role can, and often do, differ between databases.  The problem with #1473 appears to be that some folks wanted the ability to change all of the permissions for any role across any database in the cluster.  While that is admittedly a nice to have it breaks the methodology of PostreSQL.
If I have 2 databases; db0 & db1 and one role, bob.  If bob has read-only access to 10 schema in db0 but full access to 10 different schema in db1, how can I see that?
If I delete bob from db1, will I know that I'm also deleting him from db0?  Will I wonder why PostgreSQL will throw up a bunch of errors saying that there are objects owned in db0 by bob?
The problem, as I see it, is that the dBeaver devs, in trying to make a universal database GUI are trying to force a methodology on a RDMS that isn't the same as most of the others that it supports.
There are a number of options that would have been tried, and perhaps they already were, such as:

Keep the Roles at the cluster level and add a db dropdown, perhaps under the description
Add a db column in Roles tab (Owner, Role, Grantor, Admin option, Database)
Add a database top level in the Permissions tab above Schema
Convert the Source tab to a a tree view with the database at the root.

As you can see there are many options to incorporate the fact that PostgreSQL utilizes cluster level roles.  The current solution, that of pretending that roles are database specific is definitely confusing and possibly dangerous.
If you could return them to where they should be, at the cluster level, while you work out how to incorporate this wrinkle into the GUI, that would be appreciated.
In any event, it would have been most helpful, at least for folks like myself who spent an inordinate amount of time trying to fix this new apparent bug, including moving back to pgAdmin4 to access the roles, if changes like this were prominently announced at least in the update notes that  the change was made in.
Thanks again, and I wish you luck.


The size of cuda-relative library files are ZERO byte

Hello
How are you?
I am building an own Jetson docker image from nvcr.io/nvidia/l4t-tensorflow:r32.4.3-tf1.15-py3 or nvcr.io/nvidia/l4t-pytorch:r32.4.3-pth1.6-py3 on x86_64 host machine rather than Jetson.
However, I faced the following issue during building opencv-4.2.0 with cuda support.

So I checked this cublas library file in the base container by using the following command on x86_64 host machine.
$ sudo docker run nvcr.io/nvidia/l4t-tensorflow:r32.4.3-tf1.15-py3  ls -la /usr/lib/aarch64-linux-gnu/
What is strange is that the sizes of most cuda-relative library so files are ZERO byte.

I am confused.
How should I understand this?
Thanks

Hi @rose-jinyang , the l4t-base container that these use is only for Jetson, and the CUDA/GPU libraries are mounted in by the docker runtime- that is why those files are zero bytes.
Also the pytorch/tensorflow wheels that get installed into container are only built for Jetson (aarch64)
I recommend using the containers for x86 on NVIDIA NGC, they have them for pytorch/tensorflow/ect for x86.


Add inline action for start working

And a few linting errors
Fixes #1717

@misolori here are the icons I picked for starting and stopping working. What do you think?

Hmm...the check is often used for "done" or "active" so it feels a bit odd here. You could use the pencil though that could imply editing the issue or title:

You could also try the right arrow:

But i think in general you are going to have a hard time finding a good representation of "Start working".
For stopping, would the square be better for an actual stop icon? We use that in notebooks for stopping an execution:

For "Start working", would the git branch icon make sense?

@RMacfarlane we could use that, though it is the same icon as the SCM in the activity bar:

New icons:

We have the pencil for editing the queries in the same tree, so I think that would be confusing. I also think the scm icon will be confusing since it's already in the activity bar.


Commit Rust 1.60 compatible Cargo.lock

As seen in #3085, not having the Cargo.lock file committed can lead to the inability to compile rav1e if some upstream crates do not respect semver. Many crates, unfortunately, do not consider a change in the minimum Rust version to be a breaking change, so this has been and will continue to be a recurring problem unless we commit Cargo.lock. This also generally is good for making builds reproducible.
The downside is we will have to manually update dependencies or use a tool like dependabot to update them for us (which will run through CI to verify the updates do not break our builds). I consider this preferable to having random build breakages that are outside of our control.

Codecov Report
Base: 86.95% // Head: 86.91% // Decreases project coverage by -0.04% :warning:

Coverage data is based on head (b81e76a) compared to base (7f58549).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3086      +/-   ##
==========================================
- Coverage   86.95%   86.91%   -0.05%
==========================================
  Files          83       83
  Lines       33094    33094
==========================================
- Hits        28778    28763      -15
- Misses       4316     4331      +15

Impacted Files
Coverage Δ

src/asm/x86/lrf.rs
93.48% <0.00%> (-0.64%)
:arrow_down:

src/rdo.rs
85.11% <0.00%> (-0.56%)
:arrow_down:

src/partition.rs
77.58% <0.00%> (-0.16%)
:arrow_down:

src/deblock.rs
96.36% <0.00%> (-0.16%)
:arrow_down:

src/encoder.rs
87.15% <0.00%> (+0.03%)
:arrow_up:

src/me.rs
95.70% <0.00%> (+0.09%)
:arrow_up:

src/context/mod.rs
92.63% <0.00%> (+1.05%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

You are trading hiding bugs for additional discomfort reviewing patches.
Please revert it and if you want to put workarounds add a patch section in Cargo.toml.


Compatibility with Snips after Snips Platform Update 1.1.0 (0.61.1)

Home Assistant release with the issue:

0.87.1
Last working Home Assistant release (if known):
Operating environment (Hass.io/Docker/Windows/etc.):

Hass.io/ RPi 3b
Component/platform:

https://www.home-assistant.io/components/snips/
Description of problem:
Snips Platform Update 1.1.0 (0.61.1) -> hermes API: probability field in hermes/intent/ messages has been renamed confidenceScore (json).
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
snips:

intent_script: !include intent_script.yaml

activateScene:
    action:
      - service: scene.turn_on
        data_template:
          entity_id: scene.{{ (scene) | lower| replace(" ","_") }}

Traceback (if applicable):
Key error: probability

Additional information:
Replacing ["intent"]["probability"] with ["intnet"]["confidenceScore"] in /srv/homeassistant/lib/python3.5/site-packages/homeassistant/components/snips.py fixes the problem

This was already fixed in #21443 which will be released in 0.89 (it's already in the RC: https://github.com/home-assistant/home-assistant/compare/master...rc)


Back-propagating through limited number of Nodes having Target.

❓ Questions & Help

Problem Setting
Say I have a Graph with Nodes: [a,b,c,d,e]. Out of which only [a,c,e] have targets [0,1,0](Node Classification).
Edge_Index goes like [[a,b,c,d,e],[b,d,e,a,e]].
How do I use this and to learn an unified feature extractor in such a setting?

I think the examples/gcn.py example covers this use-case pretty well. You basically just train your model based on the nodes you have labels for (but still use all other nodes to gather information).

Thanks. This would still allow me to use the Feature Extractor for the Nodes with no Label right?

Yes, exactly. This is known as semi-supervised learning. You can have a look at the GCN paper for a more detailed explanation.


Use redis 6.2.4 in CI

What do these changes do?
Use Redis 6.2.4. in CI.
Are there changes in behavior for the user?
No.
Related issue number
Relates to #930.
Checklist

[x] I think the code is well written
[ ] Unit tests for the changes exist
[ ] Documentation reflects the changes
[ ] If you provide code modification, please add yourself to CONTRIBUTORS.txt

The format is <Name> <Surname>.
Please keep alphabetical order, the file is sorted by names.

[ ] Add a new news fragment into the CHANGES/ folder

name it <issue_id>.<type> (e.g. 588.bugfix)
if you don't have an issue_id change it to the pr id after creating the PR
ensure type is one of the following:

.feature: Signifying a new feature.
.bugfix: Signifying a bug fix.
.doc: Signifying a documentation improvement.
.removal: Signifying a deprecation or removal of public API.
.misc: A ticket has been closed, but it is not of interest to users.

Make sure to use full sentences with correct case and punctuation, for example:
Fix issue with non-ascii contents in doctest text files.

Awesome dude! I'll take a look.

awesome thanks @abrookins !


Fix uninstall link in upgrade guide 0.11

NONE

/assign @munnerz

/lgtm

/cherrypick release-0.11

/release-note-none


metaclass conflict

I got TypeError: metaclass conflict importing torchdata
Python Version: Python 3.6.13
PyTorch:
torch==1.7.1+cu110
torchaudio==0.7.2
torchdata==0.2.0
torchvision==0.8.2+cu110

In [1]: import torchdata as td
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-1-d739c1dd990c> in <module>
----> 1 import torchdata as td

~/git/envs/pyenv/lib/python3.6/site-packages/torchdata/__init__.py in <module>
     58 """
     59
---> 60 from . import cachers, datasets, maps, modifiers, samplers
     61 from ._version import __version__
     62 from .datasets import Dataset, Iterable

~/git/envs/pyenv/lib/python3.6/site-packages/torchdata/datasets.py in <module>
    155
    156
--> 157 class Iterable(TorchIterable, _DatasetBase, metaclass=MetaIterable):
    158     r"""`torch.utils.data.IterableDataset` **dataset with extended capabilities**.
    159

TypeError: metaclass conflict: the metaclass of a derived class must be a (non-strict) subclass of the metaclasses of all its bases

type(TorchIterable)
Out[2]: typing.GenericMeta

Dataset inherited torch.Dataset which already inherited GenericMeta metaclass, but it also specified using MetaDataset as metaclass which caused the conflicting
You can reproduce with this
class M_A(type):
    pass

class M_B(type):
    pass

class A(metaclass=M_A):
    pass

class C(A, metaclass=M_B):
    pass

This can be solved with a wrapper:
class MetaDatasetWrapper(MetaDataset, GenericMeta): pass

class Dataset(TorchDataset, _DatasetBase, metaclass=MetaDatasetWrapper):

Hi there, I got the same error. Any updates?

@bearpaw looks like the tests are still failing on your fork/branch with the fix. Any idea what's up?

@bearpaw @ZacCranko Hey, sorry I'm pretty inactive on github lately due to other duties. Merged your fix and should be available in torchdata-nightly shortly.
Some issues have to be resolved about tests (shared memory related), this is fine locally but will fail with gh-actions as it doesn't allow multiple processes (from a quick check).

Also same issue because I upgraded python. DeepMind just recently solved the protein folding problem with AI but we humanity is still at least 20 years away from figuring out how to update python without breaking things.
This change to datasets.py gets it to import. But I need to test more:
diff --git a/torchdata/datasets.py b/torchdata/datasets.py
index 1d32686..b3b2c1e 100644
--- a/torchdata/datasets.py
+++ b/torchdata/datasets.py
@@ -25,9 +25,10 @@ from torch.utils.data import ConcatDataset as TorchConcatDataset
 from torch.utils.data import Dataset as TorchDataset
 from torch.utils.data import IterableDataset as TorchIterable
 from torch.utils.data import TensorDataset as TorchTensorDataset
-
+from torch.utils.data import _typing
 from ._base import Base, MetaDataset, MetaIterable
 from .cachers import Memory
+class GenericMeta(type): pass

 class _DatasetBase(Base):
@@ -154,7 +155,8 @@ class _DatasetBase(Base):
         return self._chain_object((self, other))

-class MetaIterableWrapper(MetaIterable, GenericMeta): pass
+print(type(TorchIterable))
+class MetaIterableWrapper(MetaIterable, GenericMeta, _typing._DataPipeMeta): pass

 class Iterable(TorchIterable, _DatasetBase, metaclass=MetaIterableWrapper):


Add inject option

Supports:

false (as documented)
"body" (append to body of template)
"head" (append to head: default + original)

Thank you for the PR 🥇
A couple of comments:

The inject option you're referring to as documented is actually for html-webpack-plugin.

See this issue https://github.com/jouni-kantola/inline-chunk-manifest-html-webpack-plugin/issues/3
The following section in the readme: https://github.com/jouni-kantola/inline-chunk-manifest-html-webpack-plugin/blob/master/README.md#explicit-inject
By using inject: false with html-webpack-plugin you can already inject the chunk manifest where you need it (i.e. body). With this in mind, do you still think this extra inject is a good configuration option? Let's discuss that for a bit.

After we decide if we should add this option or not, then I'd appreciate tests and docs for it.

Tests fits around here: https://github.com/jouni-kantola/inline-chunk-manifest-html-webpack-plugin/blob/master/test/plugin-inject-manifest-test.js#L4

The last bit that I think is missing is docs for the new configuration property. It probably needs extra clarity in this case, since there already is a similar configuration option for html-webpack-plugin.


Inline breakpoint jumps when saving file with line changes

Testing https://github.com/microsoft/vscode/issues/81175

Start debugging and place an inline breakpoint
Make edits to that line - breakpoint correctly shifts
Save the file - breakpoint jumps to original position

This is actually as designed behavior. Reason being, when you save the file we send the breakpoint back to the extension, and the debug extension tells us where the breakpoint should be positioned.
If you think the new position is not correct you should file this against the node / chrome @roblourens
However I think the behavior here is fine, and there is nothing we can do about this. Since when you stop debugging it should go back to the edited location. Thus closing as designed.
Though I am open for ideas on improvments, in that case I can reopen.
Thanks


Enable Travis Auto Cancellation

This is a new beta feature:  https://blog.travis-ci.com/2017-03-22-introducing-auto-cancellation
When iterating on a branch and force pushing frequently, this will automatically cancel old builds that haven't yet started to avoid running builds which would have stale results.
cc @edunham @larsbergstrom @metajack

That sounds great! Do you think we should enable it on all of our repos? It looks like it's a per-project setting.

I think we should enable the PR auto-cancel but not the branch auto-cancel. That way we'll still know which specific commit introduced a failure. I've enabled PR auto-cancel for most of the repos in the servo org. I wish there was a way to control this more easily than clicking into every repo on Travis CI and then hitting settings and flipping the toggle.

@metajack In their beta issue tracker, the Travis CI folks mention this can be modified via the API. I think it might be nice to make a script for this (similar to the ones in #457).

In the meantime, it might be nice to add this to the checklist for adding new repos on the wiki.

Make the wiki changes.

This is enabled and documented, so I don't see any other action items for this right now. Closing.


Enhancement: Change key layout to optimise insert order

Just a PoC (aka quick hack) at this point, probably some lists:reverse and sext:decode/encodes too many :)
The on-disk layout is incompatible with existing metadata stores, since the key layout is changed.
It should massively speed up inter-node synchronisation.
Should also address issues with increased CPU and/or RAM usage after a new node has joined. (let's see a bit)
I used the following batch and partition sizes for a bit of minimal testing:
vmq_swc.exchange_batch_size = 1000
vmq_swc.swc_groups = 10

Example of sync to an empty joining node: 125k Clients with 100k Subscriptions fully synced in about 3 seconds. (each partition syncs about 12k objects concurrently)
2024-08-29T10:50:00.687532+02:00 [info] <0.581.0> vmq_swc_exchange_fsm:teardown/1:228: Replica meta7: AE exchange with<EMAIL_ADDRESS>synced 12560 objects in 3649 milliseconds
2024-08-29T10:50:01.296803+02:00 [info] <0.615.0> vmq_swc_exchange_fsm:teardown/1:228: Replica meta1: AE exchange with<EMAIL_ADDRESS>synced 12493 objects in 2159 milliseconds
2024-08-29T10:50:01.843329+02:00 [info] <0.651.0> vmq_swc_exchange_fsm:teardown/1:228: Replica meta6: AE exchange with<EMAIL_ADDRESS>synced 12616 objects in 1547 milliseconds
2024-08-29T10:50:02.293390+02:00 [info] <0.710.0> vmq_swc_exchange_fsm:teardown/1:228: Replica meta5: AE exchange with<EMAIL_ADDRESS>synced 12421 objects in 1588 milliseconds
2024-08-29T10:50:02.784669+02:00 [info] <0.775.0> vmq_swc_exchange_fsm:teardown/1:228: Replica meta10: AE exchange with<EMAIL_ADDRESS>synced 12366 objects in 1563 milliseconds
2024-08-29T10:50:02.876436+02:00 [info] <0.788.0> vmq_swc_exchange_fsm:teardown/1:228: Replica meta3: AE exchange with<EMAIL_ADDRESS>synced 12381 objects in 1556 milliseconds
2024-08-29T10:50:02.963893+02:00 [info] <0.795.0> vmq_swc_exchange_fsm:teardown/1:228: Replica meta9: AE exchange with<EMAIL_ADDRESS>synced 12541 objects in 1624 milliseconds
2024-08-29T10:50:03.109823+02:00 [info] <0.822.0> vmq_swc_exchange_fsm:teardown/1:228: Replica meta4: AE exchange with<EMAIL_ADDRESS>synced 12419 objects in 1651 milliseconds
2024-08-29T10:50:03.176218+02:00 [info] <0.810.0> vmq_swc_exchange_fsm:teardown/1:228: Replica meta8: AE exchange with<EMAIL_ADDRESS>synced 12595 objects in 1768 milliseconds
2024-08-29T10:50:07.155449+02:00 [info] <0.1117.0> vmq_swc_exchange_fsm:teardown/1:228: Replica meta2: AE exchange with<EMAIL_ADDRESS>synced 12609 objects in 3748 milliseconds


Color picker

Color picker on toolbar

[ ] Text color
[ ] Highlight color

@zurfyx is this something I could help contribute? I'm new to the project / repo so any additional details would be helpful if so.

Sure, go for it! Don't have a mock but something that looks moderately pretty on the top bar would do the trick.
Heads up, I just realized the Lexical part is not a trivial good-first issue but happy to assist in the process. Basically you want to replicate formatText (https://github.com/facebook/lexical/blob/main/packages/lexical/src/LexicalSelection.js#L931) but instead of format you want to apply a style. You can either abstract it or we can help with this later.
Then, there's a couple important bits here:

Performance - we want to be able to pull the value quickly and put it into the DOM
Size - Reusing the format flag is possible and cheap as we can convert the color to a number but not sure this optimization is worth. Readability wise and performance it's better to pull the color directly from a property. __color
Reusability - I'd argue it's a good fit for TextNode because it's a common feature and we can optimize it on the reconciler (i.e. when to merge adjacent TextNodes)

Does this make any sense?
cc @trueadm @acywatson WDYT of the proposed approach?

We probably want to use __style for this:
https://github.com/facebook/lexical/blob/main/packages/lexical/src/nodes/LexicalTextNode.js#L229
There are also helpers for it:
https://github.com/facebook/lexical/blob/3f36d8d6234731d89928b1e11c780a9debafc87a/packages/lexical-selection/src/index.js#L294

@zurfyx @acywatson appreciate the additional context. that gives me some direction to get started. i'll dig in this week and follow up with any additional questions.


Data viewer is not recognizing objects of pandas.DataFrame subclass

Applies To

[X] Notebooks (.ipynb files)
[ ] Interactive Window and/or Cell Scripts (.py files with #%% markers)

What happened?
A standard pandas data frame, which is an instance of pandas.DataFrame, is able to be expanded in the data viewer. However, an instance of pandas.DataFrame subclass is not.

VS Code Version
1.64.2
Jupyter Extension Version
v2022.1.1301854968
Jupyter logs
No response
Coding Language and Runtime Version
Python 3.9.5 (pandas 1.4.1)
Language Extension Version (if applicable)
Python v2022.2.1924087327
Anaconda Version (if applicable)
No response
Running Jupyter locally or remotely?
Remote

We can fix this, instead of checking whether the type is dataframe we need to check if the type inherits dataframe.
This information needs to come from Python instead of us trying to determine this in node based on the type value.

@DonJayamanne Thanks! Should we submit an issue with the Python extension? I am not sure how it is supposed to work.

This is an issue in this extension, hence the issue belongs here.

The issue still appears in the latest version of VSCode 1.75.1

With the same code example @pw provided, I still don't see MyDF treated the same as a  pandas DataFrame, there is an icon next to the name (highlighted in red) and Size appear to be different when inspecting Variables.
from pandas import DataFrame
class MyDF(DataFrame):
    ...
pd_df = DataFrame(dict(a=[1,2,3]))
my_df = MyDF(dict(a=[1,2,3]))

I can still replicate this issue.


websocket: checks alert app-layer metadata for frames

Ticket
Redmine ticket: https://redmine.openinfosecfoundation.org/issues/6973

Merged in #1831, thanks!


Flakey Test

These 2 drone builds were run on the same code, one failed and one passed:
https://drone.grafana.net/grafana/opensearch-datasource/465
https://drone.grafana.net/grafana/opensearch-datasource/466
Summary of all failing tests
FAIL src/components/QueryEditor/MetricAggregationsEditor/SettingsEditor/index.test.tsx (38.745 s)
  ● Settings Editor › Raw Data › Should correctly render the settings editor and trigger correct state changes

    thrown: "Exceeded timeout of 5000 ms for a test.
    Use jest.setTimeout(newTimeout) to increase the timeout value, if this is a long-running test."

       9 | describe('Settings Editor', () => {
      10 |   describe('Raw Data', () => {
    > 11 |     it('Should correctly render the settings editor and trigger correct state changes', async () => {
         |     ^
      12 |       const metricId = '1';
      13 |       const initialSize = '500';
      14 |       const initialOrder = 'desc';

      at src/components/QueryEditor/MetricAggregationsEditor/SettingsEditor/index.test.tsx:11:5
      at src/components/QueryEditor/MetricAggregationsEditor/SettingsEditor/index.test.tsx:10:3
      at Object.<anonymous> (src/components/QueryEditor/MetricAggregationsEditor/SettingsEditor/index.test.tsx:9:1)

closing as not planned


Unshim ExclusiveModeGpuDiscoveryPlugin

Fixes #3589
As one of the publicly documented classes loaded by Spark outside the
plugin control, ExclusiveModeGpuDiscoveryPlugin may not reside in the
Parallel World locations. Binary dedupe shows
ExclusiveModeGpuDiscoveryPlugin is bitwise-identical across all shims
Signed-off-by: Gera Shegalov<EMAIL_ADDRESS>

 jar tvf dist/target/rapids-4-spark_2.12-21.10.0-SNAPSHOT.jar | grep ExclusiveModeGpu
 10734 Tue Sep 21 16:18:04 PDT 2021 com/nvidia/spark/ExclusiveModeGpuDiscoveryPlugin.class

build

build

we need to finish testing this manually

weird ci hasn't reported, I don't see anything running, kicking again

build

I tried this out manually and its working for me as well:
21/09/22 06:55:01 INFO InternalExclusiveModeGpuDiscoveryPlugin: Running ExclusiveModeGpuDiscoveryPlugin to acquire 1 GPU(s), host has 1 GPU(s)

failed with the scalastyle issue because of import order I mention above

build


Lock down ello-brains version to remove promos

#145167477
#145167489

          1 Message

:book:
Nice, more deletions than insertions :red_circle:

  Generated by :no_entry_sign: danger


[git] Commit messages with 50 characters in glG

truncating at character 60. First 10 chars are used for *, commit hash and spaces.
The 50 char limit is based on http://tbaggery.com/2008/04/19/a-note-about-git-commit-messages.html
Also changed aliases to use the --long-option format when available, so aliases are clearer for those unfamiliar with the options.

Example with the change:
* f443b54 [git] Commit messages with 50 characters in glG    <Eric Nielsen> (89 seconds ago) (HEAD -> git-alias, ericbn/git-alias)
|
* d562acc [history-substring-search] Remove redundant defa.. <Eric Nielsen> (7 days ago) (origin/master, origin/HEAD, master)
|
* ff32968 Warning message for zsh < 5.2                      <Eric Nielsen> (4 weeks ago)
|
...

Yes, with that change, LGTM.


--target is not always a valid command

For example Google's BoringSSL can't be build with this crate as it --target isn't a valid cmake command for it.
I don't know how to disable it but the default of --target "install" causes a build error.

Interesting! Maybe we could add an option like no_build_target which disables passing the --target flag to cmake?

That should work.
I guess the only other approach would be an Option<String>, but that would be an api change.
:+1:

Yeah we can probably do that eventually but for now stick with the backwards compatible option. Would you be willing to send a PR to test this out?

Currently traveling I'll try to submit soon.

Ok, thanks!