Help for java

anyone have compiled version and example for java?
i'm new in java :(

What have you tried and what problems do you have?

compile error
c++: internal compiler error: Killed (program cc1plus)
Please submit a full bug report,
with preprocessed source if appropriate.
See <file:///usr/share/doc/gcc-7/README.Bugs> for instructions.
CMakeFiles/tdcore.dir/build.make:326: recipe for target 'CMakeFiles/tdcore.dir/td/telegram/AnimationsManager.cpp.o' failed
make[2]: *** [CMakeFiles/tdcore.dir/td/telegram/AnimationsManager.cpp.o] Error 4
CMakeFiles/Makefile2:274: recipe for target 'CMakeFiles/tdcore.dir/all' failed
make[1]: *** [CMakeFiles/tdcore.dir/all] Error 2
Makefile:140: recipe for target 'all' failed
make: *** [all] Error 2

You have not enough RAM to compile using GCC. You can use clang instead, which need less than 2 GB.

Guys, can u compile java tdlib version? I have problems too... Two days of googling is killing me

@rickpickles What problems do you have?

@levlam im complete new in cmake, and after two days i give up, Why library for java need cmake or something for use?

@rickpickles You are not supposed to know how to use cmake, all required commands are written in README. For which OS do you try to compile TDLib? Do you have some problems not covered in README?

@levlam Я никогда раньше не сталкивался с подобным способом сборки, на windows based машинах если следую readme получаю:
E:\sources\td>cd jnibuild
E:\sources\td\jnibuild>cmake -DCMAKE_BUILD_TYPE=Release -DTD_ENABLE_JNI=ON
CMake Error: The source directory "E:/sources/td/jnibuild" does not appear to contain CMakeLists.txt.
Specify --help for usage, or press the help button on the CMake GUI.
E:\sources\td\jnibuild>cmake -DCMAKE_BUILD_TYPE=Release -DTD_ENABLE_JNI=ON -DCMAKE_INSTALL_PREFIX:PATH=E:\sources\td
CMake Error: The source directory "E:/sources/td/jnibuild" does not appear to contain CMakeLists.txt.
Specify --help for usage, or press the help button on the CMake GUI.
E:\sources\td\jnibuild>cmake -DCMAKE_BUILD_TYPE=Release -DTD_ENABLE_JNI=ON -DCMAKE_INSTALL_PREFIX:PATH=E:\sources\td\CMakeLists.txt
CMake Error: The source directory "E:/sources/td/jnibuild" does not appear to contain CMakeLists.txt.
Specify --help for usage, or press the help button on the CMake GUI.
E:\sources\td\jnibuild>

You should use exactly the line from README.md:
cmake -DCMAKE_BUILD_TYPE=Release -DTD_ENABLE_JNI=ON -DCMAKE_INSTALL_PREFIX:PATH=../example/java/td ..

 My steps for build

i get sources by downloading zip
i have path for sources E:\sources\td:
im in this path in "jnibuild" folder
im trying to use cmd
E:\sources\td>cmake -DCMAKE_BUILD_TYPE=Release -DTD_ENABLE_JNI=ON -DCMAKE_INSTALL_PREFIX:PATH=../example/java/td ..
CMake Error: The current CMakeCache.txt directory E:/sources/td/CMakeCache.txt is different than the directory e:/telega/td where CMakeCache.txt was created. This may result in binaries being created in the wrong place. If you are not sure, reedit the CMakeCache.txt
CMake Error: The source directory "E:/sources" does not appear to contain CMakeLists.txt.
Specify --help for usage, or press the help button on the CMake GUI.

@levlam
im trying again. Can u make a complete lib for code monkeys like me? XD
E:\sources\td>cmake -DCMAKE_BUILD_TYPE=Release -DTD_ENABLE_JNI=ON -DCMAKE_INSTALL_PREFIX:PATH=../example/java/td ..
-- Building for: NMake Makefiles
-- The CXX compiler identification is unknown
-- The C compiler identification is unknown
CMake Error at CMakeLists.txt:3 (project):
The CMAKE_CXX_COMPILER:
cl

is not a full path and was not found in the PATH.
To use the NMake generator with Visual C++, cmake must be run from a shell
that can use the compiler cl from the command line.  This environment is
unable to invoke the cl compiler.  To fix this problem, run cmake from the
Visual Studio Command Prompt (vcvarsall.bat).
Tell CMake where to find the compiler by setting either the environment
variable "CXX" or the CMake cache entry CMAKE_CXX_COMPILER to the full path
to the compiler, or to the compiler name if it is in the PATH.
CMake Error at CMakeLists.txt:3 (project):
The CMAKE_C_COMPILER:
cl

is not a full path and was not found in the PATH.
To use the NMake generator with Visual C++, cmake must be run from a shell
that can use the compiler cl from the command line.  This environment is
unable to invoke the cl compiler.  To fix this problem, run cmake from the
Visual Studio Command Prompt (vcvarsall.bat).
Tell CMake where to find the compiler by setting either the environment
variable "CC" or the CMake cache entry CMAKE_C_COMPILER to the full path to
the compiler, or to the compiler name if it is in the PATH.
-- Configuring incomplete, errors occurred!
See also "E:/sources/td/CMakeFiles/CMakeOutput.log".
See also "E:/sources/td/CMakeFiles/CMakeError.log".

You need to run C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\vcvarsall.bat before invoking cmake as it suggests.


Update for 6.3

Updates DalamudPackager and upgrade project to .net7.

@Infiziert90 also submitted a PR with some added features that I merged. Thanks for doing this, but that other PR also updates the plugin to .net7, so I'm going to close this PR.


TypeError: Cannot read properties of undefined (reading 'startLineNumber')

run code-oss
open light bulb menu
error below is printed

  ERR Cannot read properties of undefined (reading 'startLineNumber'): TypeError: Cannot read properties of undefined (reading 'startLineNumber')
    at Object.onFocus (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/editor/contrib/codeAction/browser/codeActionController.js:209:115)
    at ActionList.onFocus (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/platform/actionWidget/browser/actionList.js:235:37)
    at UniqueContainer.value (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/platform/actionWidget/browser/actionList.js:149:67)
    at Emitter._deliver (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/base/common/event.js:870:26)
    at Emitter._deliverQueue (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/base/common/event.js:881:22)
    at Emitter.fire (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/base/common/event.js:904:22)
    at vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/base/common/event.js:98:93
    at UniqueContainer.value (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/base/common/event.js:1206:34)
    at Emitter._deliver (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/base/common/event.js:870:26)
    at Emitter.fire (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/base/common/event.js:899:22)
    at Trait._set (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/base/browser/ui/list/listWidget.js:122:28)
    at Trait.set (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/base/browser/ui/list/listWidget.js:113:25)
    at List.setFocus (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/base/browser/ui/list/listWidget.js:1302:24)
    at ActionList.onListHover (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/platform/actionWidget/browser/actionList.js:248:24)

@aiday-mar do you know if this is related to https://github.com/microsoft/vscode/issues/201812?

I looked at the file codeActionController and the stack trace refers to the method onFocus and that the diagnostic inside of diagnostic.startLineNumber appears to be undefined. I think this is probably not linked to https://github.com/microsoft/vscode/issues/201812. It looks however that the diagnostics array is empty, and the code is trying to access the first non-existent diagnostic.

@aiday-mar yep I see that now! looks like some flow from the light bulb widget exposed a bug with onFocus which i wrote a few months ago 👍

I have encountered this today when running code-oss. Find the unminified trace below.
I believe the flaw is accessing the array of diagnostics because there is no check that it has any elements. Hence it continues with undefined
https://github.com/microsoft/vscode/blob/7c471f6415d8cb9b5e1b77bc1663e187fe8ca5cb/src/vs/editor/contrib/codeAction/browser/codeActionController.ts#L291
2024-01-24 10:23:53.268 [error] Cannot read properties of undefined (reading 'startLineNumber'): TypeError: Cannot read properties of undefined (reading 'startLineNumber')
    at Object.onFocus (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/editor/contrib/codeAction/browser/codeActionController.js:209:40)
    at ActionList.onFocus (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/platform/actionWidget/browser/actionList.js:235:37)
    at UniqueContainer.value (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/platform/actionWidget/browser/actionList.js:149:67)
    at Emitter._deliver (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/base/common/event.js:870:26)
    at Emitter._deliverQueue (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/base/common/event.js:881:22)
    at Emitter.fire (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/base/common/event.js:904:22)
    at vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/base/common/event.js:98:93
    at UniqueContainer.value (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/base/common/event.js:1206:34)
    at Emitter._deliver (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/base/common/event.js:870:26)
    at Emitter.fire (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/base/common/event.js:899:22)
    at Trait._set (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/base/browser/ui/list/listWidget.js:122:28)
    at Trait.set (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/base/browser/ui/list/listWidget.js:113:25)
    at List.setFocus (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/base/browser/ui/list/listWidget.js:1302:24)
    at List.focusNext (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/base/browser/ui/list/listWidget.js:1311:22)
    at ActionList.focusNext (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/platform/actionWidget/browser/actionList.js:201:24)
    at ActionWidgetService.focusNext (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/platform/actionWidget/browser/actionWidget.js:52:32)
    at Action2.run (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/platform/actionWidget/browser/actionWidget.js:185:31)
    at handler (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/platform/actions/common/actions.js:389:52)
    at InstantiationService.invokeFunction (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/platform/instantiation/common/instantiationService.js:46:24)
    at CommandService._tryExecuteCommand (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/workbench/services/commands/common/commandService.js:81:59)
    at CommandService.executeCommand (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/workbench/services/commands/common/commandService.js:50:33)
    at WorkbenchKeybindingService._doDispatch (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/platform/keybinding/common/abstractKeybindingService.js:259:54)
    at WorkbenchKeybindingService._dispatch (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/platform/keybinding/common/abstractKeybindingService.js:143:25)
    at vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/workbench/services/keybinding/browser/keybindingService.js:192:51
2024-01-24 10:23:56.221 [error] Cannot read properties of undefined (reading 'startLineNumber'): TypeError: Cannot read properties of undefined (reading 'startLineNumber')
    at Object.onFocus (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/editor/contrib/codeAction/browser/codeActionController.js:209:40)
    at ActionList.onFocus (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/platform/actionWidget/browser/actionList.js:235:37)
    at UniqueContainer.value (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/platform/actionWidget/browser/actionList.js:149:67)
    at Emitter._deliver (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/base/common/event.js:870:26)
    at Emitter._deliverQueue (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/base/common/event.js:881:22)
    at Emitter.fire (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/base/common/event.js:904:22)
    at vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/base/common/event.js:98:93
    at UniqueContainer.value (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/base/common/event.js:1206:34)
    at Emitter._deliver (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/base/common/event.js:870:26)
    at Emitter.fire (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/base/common/event.js:899:22)
    at Trait._set (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/base/browser/ui/list/listWidget.js:122:28)
    at Trait.set (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/base/browser/ui/list/listWidget.js:113:25)
    at List.setFocus (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/base/browser/ui/list/listWidget.js:1302:24)
    at ActionList.onListHover (vscode-file://vscode-app/Users/jrieken/Code/vscode/out/vs/platform/actionWidget/browser/actionList.js:248:24)

yep, realized that the fix I pushed last week was not an actual fix 🔢
this PR should do it, checking that the array is not empty.  I am curious though, which code action was giving this error? I wasn't able to see the error on any of the move to actions or copilot actions

just for personal reference, I am curious which code action was giving this error? I wasn't able to see the error on any of the move to actions or copilot actions

Hard to say but since I am seeing this with insiders and code-oss means it must be an extension that is install in both and that's either TS or copilot.

@jrieken gotcha, that makes sense. iirc, it could be both, since code actions from the lightbulb, when there are no errors in the line, shouldn't have diagnostics attached to them.
either way, if you could verify the fix that would be great 🥂

Change looks good, added verified

if (diagnostic.startLineNumber && diagnostic.startColumn) {

I think in some parts you are a bit over-cautious. Like above you can assume that startLineNumber and startColumn are always defined. Otherwise it would be a bug with somewhere upstream that you shouldn't swallow. The fine line of failing fast and continuing gracefully ;-)

makes sense! that was something I added in the first PR quickly since I thought that would fix the initial issue, removing now for debt 👍


Part 1 : OpenSMTPD

Let's start to configure OpenSMTPD!

Reading the OpenSMTPD Faq, we need to create a system user named vmail to handle multiple domains.

Should we create some code?
Here on github? Fork and create pull request?

We can discuss together and share here the following:

requirements
all the steps to configure OpenSMTPD
make some tests
write a script

And then suggest a pull request.

Because we configure the virtual users in /etc/mail/passwd we need to create only one single system user: vmail.
All virtual deliveries occur under the same user: vmail
This user don´t login, so we give him /sbin/nologin vmail as login shell
The home directory  for this user is /var/vmail. Because this exists on every openbsd system, we don´t need to check (...well - I think!? Or maybe a mkdir -p /var/vmail first?)
So we have to create it like in the FAQ:
# useradd -m -g =uid -c "Virtual Mail" -d /var/vmail -s /sbin/nologin vmail
I would check for existance first and only run useradd if it not exists:
# getent passwd vmail || useradd -m -g =uid -c "Virtual Mail" -d /var/vmail -s /sbin/nologin vmail

I think that new created files should not accessible by others, so maybe we need to give the vmail`` user a umask 007. So it makes sense to create a loginclass for the vmailuser in/etc/login.conf`:
# loginclass for vmail
#
vmail:\
        :umask=007:\
        ...

and create the user with useradd's L flag:
# getent passwd vmail || useradd -L vmail -m -g =uid -c "Virtual Mail" -d /var/vmail -s /sbin/nologin vmail

But before create a loginclass it´s better to check if it already exists:

But wait... maybe someone uses a hashed database and then we need to rehash /etc/login.conf with  # cap_mkdb /etc/login.conf after creation of the new loginclass.
So we need to check if there exists a hashed database first: if [ -r  /etc/login.conf.db ]; then ....

Regarding the prerequisites, you ll need:

at least a registered domain name with these DNS records set: MX, SPF, DMARC, CAA and DKIM

The requirements :

A vmail user to handle the virtual domains and users
install the package pkg_add opensmtpd-extras to get
Configuration file for OpenSMTPD: /etc/mail/smtpd.conf
Manage the aliases table using the file: /etc/mail/aliases
Manage the domains table using the file: /etc/mail/domains
Manage the passwd table using the file: /etc/mail/passwd
Manage the virtuals table using the file: /etc/mail/virtuals
Manage the blacklist table using the file: /etc/mail/blacklist

at least a registered domain name with these DNS records set: MX, SPF, DMARC, CAA and DKIM

PTR

While MX and A records are the bare minimum for a mailserver, I'd say PTR, SPF, and DKIM are very much essential these days if you want to have reliable delivery to insert big corp name mail users.

Clearly you need all of these dns records.

I wouldn´t argue against setting al of them for a real world mailserver.
I wanted to document that SPF, DMARC, DKIM and PTR are not required to act as a mailserver.
But definitely we need them :-/

You need to be familiar with these terms :

Name
Description

SMTP
Simple Mail Transfert Protocol

MTA
Mail Transfert Agent (OpenSMTPD)

MUA
Mail User Agent (Outlook, Thunderbird ... )

MDA
Mail Delivery Agent (Dovecot)

Mbox
mail storage format

Maildir
mail storage format

POP3
mail access protocol (Post Office Protocol)

IMAP
mail access protocol (Internet Mail Access Protocol)

The main script will be named deploy.
sh deploy without parameters will install all the box.
In case of maintenance, It will be possible reinstall each module like:
./deploy opensmtpd
./deploy letsencrypt
./deploy dovecot
...

I just pushed bin/deploy and lib/opensmtpd ; work in progress.
Just run ./bin/deploy or ./bin/deploy -d

Installer is now ready.
You can use it like this : ./bin/deploy [-d] [library]

I added the linking just for best practice.

If it's not used it might be better to leave it out. You don't know if someone's going to run python 2 or python 3 only scripts and get compatibility issues.
If a particular python is used we could specify python 3 in the shebang line.

We are going to start at the beginning.
A first simple configuration of smtpd.conf to handle local mail :
# Tables
table aliases file:/etc/mail/aliases

# Accept only internal mail
listen on lo0 hostname localhost

# Deliver to local mbox
accept for local alias <aliases> deliver to mbox

# Accept to send anywhere
accept from local for any relay

To test :
# Send to local and check you get it
mail -s test01 root
hello
.
mail

# Send to external address
mail -s test02<EMAIL_ADDRESS>hello
.

Do you think we need that ?

queue compression
         Enable transparent compression of envelopes and messages.  The
         only supported algorithm at the moment is gzip.  Envelopes and
         messages may be inspected using the smtpctl(8) or gzcat(1)
         utilities.

 queue encryption [key key]
         Enable transparent encryption of envelopes and messages.  key
         must be a 16-byte random key in hexadecimal representation.  It
         can be obtained using the openssl(1) utility as follow:

$ openssl rand -hex 16

Compression&encryption always good to use, imho. Safety (y)

Although I basically agree with @deksar I would say that in this sutuation we don´t need to encrypt the queue.
Encrypting (and also compression) will add an extra layer of complexity (but of course also an extra layer of security).
I won´t argue against encryption here but I am not sure if encrypting the queue is a benefit ?!?

If it works, lets use queue compression and encryption, but if it interferes with debugging problems, perhaps look to add that when we know the complete setup is working?

I think final user will be able to add this (queue compression & encryption) manually.

Some useful commands :
# Check your smtpd.conf
smtpd -n

# Get the stats
doas smtpctl show stats

# View the queue
doas smtpctl show queue # or mailq

# Get details on a specific message in the queue
doas smtpctl show message 025a5c7b6cc771cc

# Force the queue distribution
doas smtpctl schedule all

# Remove all messages in the queue
doas smtpctl remove all


Proxy Issues?

Latest testcafe, latest Chrome, Node v6.12.3
I think I have run into proxy problems, as my working fails with this
http://<IP_ADDRESS>:57400/HJXflD_DG/http://localhost:8080/onewssi/user/v1/?filter=businessUnit(XXXX%20XXXXXXX)
The URL http://localhost:8080/onewssi/user/v1/?filter=businessUnit(XXXX%20XXXXXXX) is up and running.
Do I need to configure the Testcafe proxy?

Hi @leegee,
How does it look? The test fails with some error? Or it can't just open a page? Do you use a system proxy on your machine?

In this particular setup, several proxies — it's horrible and not my doing — but the mistake here was mine, with a misconfigured mock. Thanks for your time — loving the TestCafe experience!

Glad to hear this! =) I close this since the issue is resolved


Feature/chap 947 tabbar

@nkiefera9s Wanted to do the CR - but still all the checks are failing :)

@nkiefera9s Wanted to do the CR - but still all the checks are failing :)

Haven't had time to fix the linter issues yet :/ will write you a message when i'll fix them


.load() -> .on('load') for jQuery3

These are the ones I could still find. See e.g., #8975

Oh, oops, wrong upstream.

And never mind, dupe of 10729


[Feature] Display the full JSON-RPC call in the logs that failed

Currently if something goes wrong with a JSON RPC call (eth-calls, decoding, etc) there is just a message like this displayed:
failed to process trigger: block #21041413 (0x0000…b378), transaction a44fa1857fa5448419ff94682de1ffeccaa74eb0d6dc9d0de8fe37c6c2a90746: Decoding function outputs for the call Function { name: "withdraw", inputs: [], outputs: [Param { name: "", kind: Uint(256), internal_type: None }], constant: None, state_mutability: NonPayable } failed, raw output: : Invalid name: please ensure the contract and method you're calling exist! failed to decode empty bytes. if you're using jsonrpc this is likely due to jsonrpc returning 0x in case contract or method don't exist

(From this example subgraph: https://thegraph.com/hosted-service/subgraph/bsamuels453/yearn-fantom-validation?selected=logs, deployment id: QmNhTkng5LLvdCXbK6JSoeU8tSW18wo3tPpm5mMM3Sd4FY)
This alone does not provide enough information to reproduce.
What is the expected behavior?
It would be great to have the exact JSON RPC call written in the logs so we can reproduce and troubleshoot the underlying issue.

For context, this is the transaction that failed with the above error: https://ftmscan.com/tx/0xa44fa1857fa5448419ff94682de1ffeccaa74eb0d6dc9d0de8fe37c6c2a90746

logging RPC call details is very helpful since currently there is nothing subgraph devs can do when a smart contract call fails :)

Hi @leoyvens, I know some devs who'd be interested in working on this. Is there a budget for fixing this issue, and if so, how much?
I'd put this issue up on OpenQ.
Thanks!


Move to markdown-it-py markdown parser implementation

Using an alpha release of ExecutableBookProject/MyST-Parser#123
All the current tests pass, with no changes to the expected outputs. I also took the opportunity to split up the code in NotebookParser.parse in to separate functions.

From a quick scan of the documentation, I don't see any issues.
However, I do note that the plotly and ipywidgets no longer show, moving from:

https://myst-nb.readthedocs.io/en/stable/use/interactive.html
https://myst-nb.readthedocs.io/en/latest/use/interactive.html

I assume this is because this notebook is now being executed during the build, and is thus encountering  jupyter/nbclient#39

In 74cf6a3 I have introduced some nice, self-documenting render tests, see tests/nb_fixtures/basic.txt

In 536a1dd I have added a simple fix for cell index / line number reporting:
Line number are now set as <cell index>*10000 + <line number> (I'm guessing a cell won't have more than 10000 lines!). This is a simple solution to addresses #71, that doesn't require any complex overrides of the sphinx reporting machinery.  We should also document this.

I think this looks great, and agreed about the widgets not showing up. I don't think that should block us here


CMS Planning

[ ] evaluate CMS options
[ ] determine architecture and source-of-truth

Photos

[ ] Social cards out of sanity - correct dimensions

#275 - add additional fields here

https://github.com/opennem/cms

@nc9 my google login is<EMAIL_ADDRESS>

additional schemas to create/import:

https://github.com/opennem/opennem/blob/master/opennem/db/fixtures/facility_status.json
https://github.com/opennem/opennem/blob/master/opennem/db/fixtures/fueltechs.json
https://github.com/opennem/opennem/blob/master/opennem/db/fixtures/networks.json
https://github.com/opennem/opennem/blob/master/opennem/db/fixtures/network_regions.json


[README] Include badges and link to documentation

Adds Carthage and CocoaPods badge along with link to rendered documentation: http://cocoadocs.org/docsets/Result/0.3.2/Structs/Result.html

Nice! Thank you, Kyle!


Feat/delay simulation

Description
Adds a filter to simulate delay.

draft because it includes a commit from #154 which should be merged first.


Cannot set cookie in IE

I'm getting the following error in IE:
LOG: boomerang.rt: [error] cannot set start cookie
I found that commenting out these lines resolves the issue:
			((typeof domain !== "undefined") ? "; domain="
					+ (domain !== null ? domain : impl.site_domain ) : "") +

in the debugger: impl.site_domain = "localhost"
Any suggestions?

boomerang isn't designed to work against localhost.  You need a fully qualified domain name.  Create an alias for your host in /etc/hosts and use that instead.  Perhaps something like localhost.localdomain would work.


Config file name doesn't match mod name.

I see that this is a continuation of DropOff, and I assume that's why the config file is still named as such. I almost posted an issue that I had no config file before I saw the DropOff name up top of the GitHub page.


TypeScript noEmitHelpers flag

Suggestion
✅ Viability Checklist
My suggestion meets these guidelines:
✅ This would change the runtime behavior of existing JavaScript code
⭐ Suggestion
I was writing some sort of code, noticed that whenever I compile my typescript code, it generates some helper functions even though JavaScript has the ability to process the code without the use of helper functions.
Even I tried to compile with '--noEmitHelpers' flags, but it only removes the helper functions not the usage.

You forgot to fill out most of the issue template.
This is working as intended. --noEmitHelpers is meant to avoid emitting duplicate helpers when you're including the tslib NPM package, which brings these helpers. The helpers are still needed if you target a JavaScript version that does not provide the features you're trying to use.
Most likely you just need to adjust the target version.

OK. What should be the target version? I am using node.js v18.0.0.

You can find a list of supported features for various runtimes here: https://kangax.github.io/compat-table/es2016plus/
Node 18  supports most of ES2022.

At my current node.js version, I should use ES2023?

Thanks a lot. It worked with ES2022.


Combine adjacent image and text links for the same resource

Describe the bug
Adjacent text and images which navigate to the same destination should be presented and announced as one link. A description of this WCAG standard can be found here: https://www.w3.org/WAI/WCAG20/Techniques/html/H2
To Reproduce
Steps to reproduce the behavior:

On the sandbox or demo site, go to any entry that displays a thumbnail next to the textual description.
If the thumbnail has a link that takes the user to the same place as the text link, these two links should be combined semantically in the html code. E.g.:

        Products page
Expected behavior
Expect that screen readers and other assistive technology does not encounter the same links a number of times, but rather just once (see code above)

@hostle83
I can't figure out where this issue is detected. I thought in the search page where we have thumbnails and item title with the same url, but using the axe devtools  the problem is not reported

Can i ask where did you find the problem? thanks

@atarix83 : I'm able to detect this issue using WAVE, but not using Axe DevTools.  I think it's worth noting that different scanners will find different issues sometimes.
If I run WAVE scanner on that Search Results page (where thumbnails are shown), I see a large number of "Alerts". In WAVE, an "Alert" is an accessibility warning, but not an accessibility error.
Some of these alerts appear next to the Title link of every Item in the search results and say: Redundant Link: Adjacent links go to the same URL.
The full alert details from WAVE say:

What It Means
Adjacent links go to the same URL.
Why It Matters
When adjacent links go to the same location (such as a linked product image and an adjacent linked product name that go to the same product page) this results in additional navigation and repetition for keyboard and screen reader users.
How to Fix It
If possible, combine the redundant links into one link and remove any redundant text or alternative text (for example, if a product image and product name are in the same link, the image can usually be given alt="").
Standards and Guidelines
2.4.4 Link Purpose (In Context) (Level A)

So, I think this ticket is describing an accessibility issue, but it's more of a low to medium priority issue.  It's not an accessibility error, but instead is describing a change that would improve accessibility of these pages.

@tdonohue
for what i can see. considering how the components are structured, if we really want to resolve this issue the easiest solution is to remove the link from the thumbnail and leave only the title as a link.

@atarix83 : Good point.  I think the question then is which behavior we feel is better.  As I noted, WAVE reports this issue as a warning and not an error.  So, it's not deemed high priority by WAVE and it undetected by Axe DevTools.
At this point, I think this needs more discussion & might be flagged as lower priority.  There are two clear options:

Keep the link on the thumbnail (which may provide better usability but results in an accessibility warning)
Remove the link on the thumbnail (which fixes the accessibility warning, but may be seen as detrimental to usability for users who want to be able to click on that image to go to the Item page)

I don't have a strong opinion at this time, but would be interested in what others think.

In my opinion if it's only reported as a warning i prefer the option 1 where usability is kept as is


RequestHeaderModifier doesn't supports "Host" header

apiVersion: gateway.networking.k8s.io/v1
kind: HTTPRoute
metadata:
name: example
namespace: vaibhav
spec:
hostnames:

example.test.com
parentRefs:
kind: Gateway
namespace: envoy-gateway-system
name: web-gateway
rules:
backendRefs:

kind: Service
name: example-svc
namespace: vaibhav
port: 80
weight: 1
filters:

type: RequestHeaderModifier
requestHeaderModifier:
set:

name: Host
value: test.vaibhav.example.com
matches:

path:
type: PathPrefix
value: /
timeouts:
request: 600s
backendRequest: 600s

This is the HTTProute i am using and the service which i am referring needs this Header Host " test.vaibhav.example.com" in order to communicate the deployed app. But its not able to add this header and my svc sends 404, as the host didn't match up with any deployed app. I need to apply the HTTProute like this only.
Note: I tried directly by creating a ClusterIP svc and used it a reference the httproute, it works fine.

@vverma01232 the Host header can be modified using https://gateway-api.sigs.k8s.io/reference/spec/#gateway.networking.k8s.io%2Fv1.HTTPURLRewriteFilter
closing this issue for now, please let us know if you hit any issues

Thanks its working!

Also i wanted to know i am using the External Auth with Security Policy, so is there any way to pass the custom header in the Security policy backend red service, because right now its just forwarding the Authorization header, and also how can i send the body to the Security Policy backend ref service?

you can use headersToExtAuth to add headers that are part of the request
https://gateway.envoyproxy.io/docs/api/extension_types/#extauth
https://gateway.envoyproxy.io/docs/tasks/security/ext-auth/
to add headers that are not, you can use the earlyRequestHeaders feature
https://gateway.envoyproxy.io/latest/tasks/traffic/http-request-headers/#early-header-modification
https://github.com/envoyproxy/gateway/issues/4267#issuecomment-2373467387
this is only available in v0.0.0-latest

What about the payload or my reqbldy is there a way to that as well?

you can using ext proc https://gateway.envoyproxy.io/docs/tasks/extensibility/ext-proc/

So there is not a way to send the req payload to the External Auth Security Policy right?
Note : I only have to use the External Auth

there isnt, the API doesnt allow for it


Pipelines

Add pipelines to demo collection

Closes #72

Updated the readme as requested.  Requesting help from @m3ller for assistance on the unit test.

@randomir: you had pointed out some issues in other demo unit tests. I made some changes to Victoria's unit test, could you just make sure that I didn't suffer any of the pitfalls that you noticed in the other tests?


fix(card-storybook): replace require with static image reference

📝 Description
This PR is related w:

remove require on card storybook
add card assets to static dir on storybook config

⛳ Current behavior (updates)

🚀 New behavior

💣 Is this a breaking change (Yes/No):
No

Why we should move the images?

Other storybooks and all refer to the public directory, and we need to stay consistent.

Fixed


iocage failing to set properties - it's failing on a release lookup. :/

We're trying to set a property on an iocage:
- name: Set iocage's hostname
  iocage:
    state: set
    tag: "{{ iocage_name }}"
    properties: { hostname: "{{ iocage_name }}" }

This used to work with older versions (pre the python changes).
However, we are now getting a failure relating to a release version:
TASK [iocage-instance : Configure the hostname]     *******************************************************************
fatal: [influxdb-test-upgrade]: FAILED! => {"changed": false, "failed": true
 "module_stderr": "Shared connection to s1.truespeed.com closed.\r\n",
"module_stdout": "Traceback (most recent call last):\r\n  File
\"/tmp/ansible_RMpgHs/ansible_module_iocage.py\", line 681, in <module>\r\n
main()\r\n  File \"/tmp/ansible_RMpgHs/ansible_module_iocage.py\", line 497, in main\r\n
raise Exception(\"Not running a release: {}\".format(out))\r\n
Exception: Not running a release: 10.3-RC3\r\n\r\n", "msg": "MODULE FAILURE", "rc": 0}

This is documented in issue #7
However, not withstanding that, there is no need to look up versions for this action, and so it should succeed without failure. Can the iocage module be taught to only do release version lookups for the actions that need them (create, fetch, etc)?

isn't this fixed by your PR as well ?

No, alas. Now I just get an exception. :)

This looks like it does it #10

Thank you !
Beware, there is a big change in iocage (no more tag but name, replacing also uuid's). Actual code has been updated.

Eek - thanks for the heads up. I'm pretty worried about the amount of flux in iocage now. It's been pretty stable for a long time, and now lots of things are changing; I'm not sure whether the changes are good or no yet!


MudDataGrid SingleSelection

Description

DataGrid SelectColumn Checkbox dont working correctly when MultiSelection=false and SelectedItems has delegate

fixes #7013
How Has This Been Tested?

A lot of debug inspection, visual tests.
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Check bad behavior https://try.mudblazor.com/snippet/waQRkqFSItViYYbP
Checklist:

[X] The PR is submitted to the correct branch (dev).
[X] My code follows the code style of this project.
[ ] I've added relevant tests.

Replaces PR #7014

@ScarletKuro PR is correct now?
I created a new branch from upstream/dev

@tjscience 😃

@ScarletKuro can we fix this in next release?

@ScarletKuro can we fix this in next release?

This has to be reviewed by @tjscience first, he is busy right now.

@wesProg23 I wonder if you can make a bUnit test based on the reproduction code you have provided in the issue. The test will ensure that the code doesn't break in the future, as it will serve as a safeguard for detecting any regressions.

I'm sorry @ScarletKuro .
I'm stucked in tests. I'm not familiar with it. But i really need this implementation because the component is really bugged.
Can we Close this PR, and a team member do a new PR with properly tests?
I'm really sorry.

Leave the PR as is. I will find time and push to this PR a test when I will have time.

@wesProg23
Hi. I tried to write a test for your changes, but these changes do not work.
I tried to look visually and I recorded a gif

As you can see on the left, I have your changes and your reproduction code: https://try.mudblazor.com/snippet/waQRkqFSItViYYbP
As you can see on the recording when I try to click on any of the checkbox, nothing happens at all. Only when I click a row and then click on checkbox it will select.

@wesProg23 Hi. I tried to write a test for your changes, but these changes do not work. I tried to look visually and I recorded a gif
As you can see on the left, I have your changes (I pulled the completely from your PR via github desktop) and your reproduction code: https://try.mudblazor.com/snippet/waQRkqFSItViYYbP And when I try to click on any of the checkboxes, nothing happens at all. Only when I click a row and then click on checkbox it will select. Are you sure you committed the correct changes https://github.com/MudBlazor/MudBlazor/pull/7021/files? From code I just see that when you deselect an item, it just gets the last selected item and put's it to the Selection.

I'm sorry. I guess your branch isnt with my changes.

Thats correct behavior in my branch

Fixed the commit history, but I will check the new changes later.

Thats correct behavior in my branch

Line SelectedItem = item; was not present in your PR.

Any new?

Hello,

Any new?

Unfortunately, there is no news yet. I noticed that you have once again pushed changes to the PR branch, which has caused the fix I applied for your PR to be overwritten. This further delays the review of your changes and now irrelevant commits have made their way into the pull request:

I kindly request that you refrain from making any further modifications to the "datagrid-singleselection" branch unless specifically instructed to do so.

Added test.


APK size increasing drastically

I am working on a Fintech app for which I needed to generate some PDFs for the user.
I used this library for that purpose, and while it is an excellent port for Apache PDFBox, the APK size of my app increased drastically by ~7MB.
On analyzing the APK, I found that a folder named "fontbox" in tom_roush had a raw size of 7.5 MB.
Is there a way to exclude particular fonts/other unnecessary files so I can reduce the size?

Yes, I had the same issue. The port is great, but APK size increases by too much.

+1

FontBox is part of Apache PDFBox and, as far as I know, not optional. That is, all three PDFbox components have to be present at runtime. But this is not what is taking up that space. It is the font resources. This is probably to satisfy the PDF standard related to default fonts and international character support.
Others have taken a stab at lightening up the library: https://github.com/tripplemac/pdfbox-android-lite
It may be that these resources are not strictly necessary, depending on what you are doing. But someone who have to dig into that.

As jverne has said, fontbox is required for most of PdfBox's functionality. The reason that is is seperate is because it works as a standalone font processing library.
If for your app doesn't allow the user to input their own PDFs, you can determine which encoding fontbox will load for your PDFs. In Acrobat, Document Properties -> Fonts -> Encoding should tell you which resources will be loaded. The rest of the resources should be safe to remove.

In Acrobat, Document Properties -> Fonts -> Encoding, the type of encoding mentioned as 'Ansi'. There was no such file in Fontbox folder which I could remove. I could remove the fonts basis on the ones being used, but they do not really take up much space. I'm attaching the screenshot. Can you please help?

@TomRoush You mentioned that we can remove the unwanted resources. But how it is possible? And when will the new 2.0 version will be coming

Any update with this size issue?

@ameerparappurath @norbdev The latest release of this library has these unwanted resources removed. You don't need to do anything.

If I analyze the APK with the latest release (<IP_ADDRESS> Feb 16 2019)
/assets/com: 1.7MB (fontbox 1.3MB)
/com/tom_roush: 1.5MB (fontbox: 1.1MB)
total: 3,2MB


Recurring journal

[x] Crud
[x] Add column for profile name in creation screen.
[x] Default repeat every (like week,month etc....)
[x] never expires tick button and functions
[x] Date column not required
[x] Exchange rate column if they choose currency other than base currency.(set base currency in default)
[x] All additional information in column headings and in every line of journal to access reporting tag.(same as manual journal)
[x] No need entity type column in creation screen.
[x] Need transaction type column and default value set as 'not applicable'.
IN DETAILED WINDOW
[x] Work in overview.(Use actual data instead of dummy data)
[x] Remove status like draft,approved(status like active,stopped,expired).
[x] Options in active stage : 1.stop 2.delete
[x] Options in stopped: 1.resume 2.delete
[x] Options in expired: 1. delete
[x] History in detailed view.
IN LISTING
[x] Remove difference column in listing
[x] Show last and next journal date.

Completed


Fixed macros for out of crate uses without use luisa_compute::*;

Currently, the if_!, etc; macros use local paths to the if_then_else functions. This pull request fixes this by replacing them with absolute paths using $crate::lang::.

Whoops, I added another commit to main for clearing up the warnings. Should be fixed.

Thanks for the fixes!

Whoops, I added another commit to main for clearing up the warnings. Should be fixed.

No worries. Looking forward to another PR to clean up the warnings :-)


Question: using 2.0a XSD files instead of the supplied 2.0b schema

Hey Stan,
Would it be possible for you to support both 2.0a and 2.0b schema with openleadr? The client we're trying to integrate with is sending us requests on a 2.0a namespace such as the following
<?xml version="1.0" encoding="UTF-8"?>
<ns1:oadrRequestEvent xmlns="http://docs.oasis-open.org/ns/energyinterop/201110" xmlns:ns1="http://openadr.org/oadr-2.0a/2012/07" xmlns:ns2="http://docs.oasis-open.org/ns/energyinterop/201110/payloads">
    <ns2:eiRequestEvent>
        <ns2:requestID>FCFB4D0617944DE3B8DACC477435D07D</ns2:requestID>
        <venID>XXX</venID>
    </ns2:eiRequestEvent>
</ns1:oadrRequestEvent>

And serving this request is resulting in the following error
XML schema validation of incoming message failed: Element '{http://openadr.org/oadr-2.0a/2012/07}oadrRequestEvent': No matching global declaration available for the validation root. (<string>, line 0).

At the moment that is not possible; OpenLEADR is 2.0b only. If you find a neat way to support both, I'd be open to any pull requests to get started.


Dataset.reduce pass through non-numeric scalars

[ ] Closes #xxxx
[ ] Tests added
[ ] User visible changes (including notable bug fixes) are documented in whats-new.rst
[ ] New functions/methods are listed in api.rst

Trying to reduce a non-numeric scalar results in an error. This PR passes them through unchanged.
import xarray as xr
ds = xr.Dataset(data_vars={"y": ((), "string")})
ds.mean()

TypeError: the resolved dtypes are not compatible with add.reduce. Resolved (dtype('<U6'), dtype('<U6'), dtype('<U12'))

I did not find an issue for that
Alternatively we could skip them (this is done with non-numeric arrays)
Numeric scalars are still reduced

I'm pretty sure this was intentional -- mean of a string is not well defined.
Could we make this opt-in, by adding a new keyword argument -- perhaps something like ds.mean(skip_nonnumeric=True)?
As you note, there's also the question of whether to drop or pass through non-numeric values. I'm not sure which is more intuitive.


requirements: Remove Django upper bound constraint

Remove Django upper bound constraint, we have no specific reason or inconvenience to preserve this restriction
Resolves #225

Codecov Report

Merging #226 (5c9da22) into develop (195235b) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##           develop     #226   +/-   ##
========================================
  Coverage    81.02%   81.02%
========================================
  Files           32       32
  Lines         2525     2525
  Branches       375      375
========================================
  Hits          2046     2046
  Misses         306      306
  Partials       173      173

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 195235b...5c9da22. Read the comment docs.


fixed #210, add local backup feature

Need to file a follow-up bug to refactor this part. I write this patch quickly and some of codes are redundant and can be written cleaner. Remember to move some codes out of container.


Update workflow file to fix respawn error

Note:  No code has been changed, only compiled with updated libraries.

@ShogunPanda could you merge this?

Sure. Let me do that.
I won't release a new version tho, as I plan to rewrite the helper in Rust.


[BUG]

Hi,
Just i've just started setting up LME and instantly run into an issue I can find an answer to.  When trying to import the config on our WEC server I get the following error.
c:>wecutil cs lme_wec_config.xml
Invalid config file. Error = 0x80070057.
The parameter is incorrect.
Server 2019, 1809, OS build 17763.1999

Hello,
if you open the lme_wec_config.xml file in notepad (or similar) does it look the same as the one in the link below?
lme_wec_config.xml
Regards,
Duncan

Hi Duncan,
I should have checked that, no it doesn't.  I manually updated the file and all is working now, thanks for the quick reply.
Regards,
Paul.


Fix swapped reason/message when pod is initializing

Fixes #474
Proposed Changes

Fixed swapped reason/message when pod is initializing.

Release Note

None

I signed it!

/ok-to-test

/lgtm
/approve
I'll make sure to fix this in #464 too.


Date formats incorrect for LY and CZ

The output of epidemiology.csv has invalid date formats at the start of the file (500+ lines).
e.g.
01.04.2020,CZ_10,,3,,,,3,,
07/21/2020,LY_BA,,,,,46,3,37,

I created #26 to deal with this centrally, PTAL

Looks like this is fixed, closing.


Installs unlicensed binary-only agent extension for secure boot

Describe the bug
If I create a secure boot VM, azure-cli instructs the agent to install the unlicensed and binary-only agent extension Microsoft.Azure.Security.LinuxAttestation.GuestAttestation without asking the user.
I intend to fix that once and for all in the whole Debian distribution by disabling all extensions in the agent if not disabled by default.
Related command
az vm create --security-type TrustedLaunch --enable-secure-boot
Errors
Installs Microsoft.Azure.Security.LinuxAttestation.GuestAttestation.
2023-07-03T07:37:38.695057Z INFO ExtHandler [Microsoft.Azure.Security.LinuxAttestation.GuestAttestation-<IP_ADDRESS>] Target handler state: enabled [incarnation 2]
2023-07-03T07:37:38.695276Z INFO ExtHandler [Microsoft.Azure.Security.LinuxAttestation.GuestAttestation-<IP_ADDRESS>] [Enable] current handler state is: notinstalled
2023-07-03T07:37:38.695509Z INFO ExtHandler [Microsoft.Azure.Security.LinuxAttestation.GuestAttestation-<IP_ADDRESS>] Downloading extension package: https://umsavq3r0szsqptncs4x.blob.core.windows.net/4251b645-6705-d410-2ae4-d2b2db7d31f3/4251b645-6705-d410-2ae4-d2b2db7d31f3_<IP_ADDRESS>.zip

Issue script & Debug output
Unrelated
Expected behavior
No extension is installed without the users consent.
Environment Summary
azure-cli latest
Additional context
No response

Thank you for opening this issue, we will look into it.

@sirfamelin @Bab95 Could you please take a look at this issue?

Guest Attestation Extension installation is enabled by default as Trusted Launch as default initiative. In case user wishes to opt out there's a flag available for it.
--disable-integrity-monitoring
https://learn.microsoft.com/en-us/cli/azure/vm?view=azure-cli-latest#az-vm-create

I know that it can be disabled. But as one of the responsible for the Debian distribution I care more about the default.

why it's a concern for a single distro?

Why not? The software as it is now is neither distributable, security supported, nor governed by any of the checks that we as a distribution do.

Hi @GabstaMSFT, do you mind taking a look at this? Thanks!

@Howie425

@credbbl we are having internal discussion on this.

Hi Bastian as mentioned in our chat, our engineering teams are looking into this and discussing options.

Hello @credbbl, we will be shifting the agent in question (GA) to opt-in by default. The new behavior will require the user to specify “—enable-integrity-monitoring” if they wish GA to be installed. Auto-upgrades will be disabled by default as well and will require users to opt-in via “—enable-integrity-monitoring-autoupgrade.”
It is a generally accepted consensus by us moving forward that all agents/extensions should be only opt-in by default.


kak-bundle.kak: keep track of loaded sources

Hah. This worked out nicely. It's even a bit cleaner, as %sh doesn't have to emit try/catch-es.
It works in my setup, it seems.

I nice benefit is that this makes bundle immune to configuration errors (stray symlinks etc)

Looks good to me.


Tailwind Template not running

Seems the TailWind template in the builder website is missing a '}', so the template won't build and run.
This issue is related to the https://github.com/BuilderIO/jsx-lite/commit/de2c766eef82fb89196620087cfa1b54511c3ff2 commit.
import { useState } from "react";

export default function MyComponent(props) {
  const [name, setName] = useState("Steve");

  return (
    <div
      css={{
        padding: "10px",
      }}
    >
      <link
<EMAIL_ADDRESS>        rel="stylesheet"
      />

      <input
        className="shadow-md rounded w-full px-4 py-2"
        placeholder="What is your name?"
        value={name}
        onChange={(event) => setName(event.target.value)}
      />

      <h1
        className="text-lg"
        css={{
          marginTop: "10px",
        }}
      >
        Hello,
        {name}!
      </h1>
    </div>
  );

Thanks @phiberber! fixed over here #110


Scheduled pipelines are not displayed

I would like to monitor nightly builds.
However, scheduled pipelines are not displayed on the dashboard.
Did I miss something in the configuration or is this expected behavior?

Probably a bug... Someone needs to verify though.

I have the same issue, did you ever find a solution to this @LennardBoediger or @timoschwarzer ?


Debugger doesn't stop at breakpoint with remote debug

Environment data

VS Code version: 1.31.1
Extension version (available under the Extensions sidebar): 2019.1.0
OS and version: Windows_NT x64 6.1.7601
Python version (& distribution if applicable, e.g. Anaconda): 3.6.2 64-bit
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): N/A
Relevant/affected Python packages and their versions: None

Expected behaviour
VSCode debugger stops at the breakpoint with access to the code
Actual behaviour
VSCode pauses, but nothing happens, no visual "stopping", not even tries to create cmd-tab with connection cmd-line
Steps to reproduce:

trigger some method on a remote server (actually it is the same PC with linked py files from my sources)
hit "Start debugging" button in the editor with prepared attach config
"Remote" server starts waiting, but nothing happens in the editor

Logs
debug18628.log

Thanks for the report @ruselcmplx.
I think I will need a bit more information to reproduce your issue. Here's what I've done so far, maybe you can use this to reproduce the issue on your machine to see what we might be able to sort out.
Reproduction, Setup

Download this workspace, and unzip it someplace (for example, C:\dev\4456_test).

4456_test.zip

Open the workspace in VS Code, and open the terminal (I'm using the Powershell terminal window, adjust for your local setup).
Select an appropriate Python interpreter for your workspace (the system Python 3.6 should work fine).
Create another directory outside of the workspace. From the terminal (for example): mkdir C:\dev\4456_remote
Copy the file list.py from the workspace to this new folder. From the terminal (for example): cp C:\dev\4456_test\sources\modules\list.py C:\dev\4456_remote\
In the terminal, CD to the new directory: cd C:\dev\4456_remote
Add a virtual environment by issuing this command: py -3.6 -m venv .venv
Activate the virtual environment by issuing this command: .venv\Scripts\Activate.ps1
Install necessary dependencies for the workspace. Issue this command: python -m pip install -U ptvsd
Start the script (which will make it begin to wait for connections): python list.py
Add a breakpoint at line 5 of line.py inside of VS Code.
Start the debugger in the VS Code editor.

Expected results
The VS Code editor will break at line 5 of list.py.
Observed Results
The VS Code editor opens a different read-only window with the content of list.py in it, and breaks on line 5. (This is due to a related bug reported here #4166).

Please use this reproduction as an example that we can use to figure out what your specific problem is. Alter my repro and see if you can supply us with a minimum reproduction case. Thanks!

Everything is done well with this case
debug10196.log
But still doesn't work with the symlinked src files to server

Comparing logs shows that in my case process stops at "setBreakpoints" command From Client not followed by anything (To Client commands, setExceptionBreakpoints, process/attach etc.). Code.exe loads some CPU for a second and nothing happens

I cannot reproduce your issue, as I cannot get the remote/local debugging to even start when using soft linked files on Windows, which I believe is what you are describing with this setup.
Question:
If you do not use a linked file, but instead copy the file (or deploy it) to the server folder, does it correct the problem?

@ruselcmplx, please reply with an answer to @d3r3kk's question.  Thanks!

Question:
If you do not use a linked file, but instead copy the file (or deploy it) to the server folder, does it correct the problem?

Nope, nothing changed with copy. Perhaps the problem in Cyrillic symbols in remote path? But it was working just in January.

@ruselcmplx, thanks for giving that a try.  So thus far possible causes have been symlinks (on Windows) and Cyrillic symbols, right?
Let's simplify things and try to avoid those two issues at the same time.  Do you see the problem with a minimal project, not using symlinks, but using the same launch.json (with "localRoot" adjusted appropriately)?  Here's an example:
# local
<workspace root>/
    spam.py

# remote
# Assuming there is no __init__.py in the "modules" folder...
C:/server/modules/
    spam.py

# spam.py
print('before')
print('during')
print('after')

I tried to reproduce the scenario with HTTPServer and everything went OK:
#local
<workspace root>/
    spam.py

#remote
C:/servers/modules/cgi-bin/
    spam.py

#spam.py
#!/usr/bin/env python3

def test():
    import ptvsd
    ptvsd.enable_attach(address=('<IP_ADDRESS>', 1010), redirect_output=True)
    ptvsd.wait_for_attach()
    print('before')
    print('during')
    print('after')

print("Content-type: text/html")
print()
test()
print('Ready')

debug19428.log
So I suppose the problem is in the complexity of my working server and perhaps some implicit consequences.

@ruselcmplx, we'd still like to find the cause of the problem. :)  Would you mind trying that simple project but add a Cyrillic character to the module's filename?

Its been a month since the information was requested. Closing due to inactivity.
We'll be happy to reopen the issue once the requested information has been provided.


cnf-network:MetalLB: speaker pods replaced by frr-8ks pods

Hi Evgeny, you test the code? If so this looks like an easy fix to migrate the cnf gotests to 4.17.

Hi Evgeny, you test the code? If so this looks like an easy fix to migrate the cnf gotests to 4.17.

Yes, it was tested and looks good. I'm working on cnf gotests PR as well, it's should be ready soon

lgtm

lgtm


Recorded future playbook updates

Required items, please complete
Change(s):

Added incident creation to RecordedFuture-Alert-Importer Playbook.
Update concurrency in RecordedFuture-IOC_Enrichment playbook
Updated readme

Reason for Change(s):

Fixes

Version Updated:
-No Detections/Analytic
Testing Completed:

Yes

Hello @RecordedFutureOskbo, Thanks for raising this PR. This PR will be investigated and we will update you about the same before 16 July, 2024

Hello @RecordedFutureOskbo, Sorry for the delay in investigation. Can you try to resolve the branch conflicts.

Merged from Ms

Hello @RecordedFutureOskbo, Can you update the branch from master once.

Merged from master and repacked the solution to include changes done by Microsoft in the solution.


Words are sometimes concatenated

Describe the bug
I've seen at least one occurrence of two words being incorrectly concatenated.
To Reproduce
I was not able to understand why they were concatenated. They were separated by something like closing tags and labels.
It was the titles of the first and second tabs, from the tabbed markdown extension.
Expected behavior
No incorrect word concatenation.
Screenshots
/
System (please complete the following information):

MkDocs SpellCheck version: 0.2.0
Python version: 3.8
OS: Linux

Additional context
/

Since tags are stripped, text before and after is concatenated. A space should be inserted each time a tag is stripped.


new release?

you made 27 commits but no release :c

WIP releases have been getting posted on my discord. Release is very very soon, just need to test everything is in order.


Update Add-RabbitMQQueue.ps1

Fix small syntax error

Yikes! Thanks for catching that

you're welcome :)


Update dependencies (Re-Resizable and others)

Reviewer - this PR has made changes to one or more package.json files.

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Reviewer - this PR has made changes to one or more package.json files.

Snapshot stored with reference name:
refs/pull/15301/merge
Test environment:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15301/merge/index.html
To test a playground add it to the URL, for example:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15301/merge/index.html#WGZLGJ#4600
Links to test babylon tools with this snapshot:
https://playground.babylonjs.com/?snapshot=refs/pull/15301/merge
https://sandbox.babylonjs.com/?snapshot=refs/pull/15301/merge
https://gui.babylonjs.com/?snapshot=refs/pull/15301/merge
https://nme.babylonjs.com/?snapshot=refs/pull/15301/merge
To test the snapshot in the playground with a playground ID add it after the snapshot query string:
https://playground.babylonjs.com/?snapshot=refs/pull/15301/merge#BCU1XR#0

WebGL2 visualization test reporter:
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15301/merge/testResults/webgl2playwright/index.html

Visualization tests for WebGPU (Experimental)
https://snapshots-cvgtc2eugrd3cgfd.z01.azurefd.net/refs/pull/15301/merge/testResults/webgpuplaywright/index.html


Invalid Message-Authenticator in packet 224 when using radtest

When using radtest, which is a test tool that comes with FreeRadius, I get an exception  System.InvalidOperationException: Invalid Message-Authenticator in packet 224.
$ radtest user password localhost 1812 0 12345

Received packet from <IP_ADDRESS>:56039, Concurrent handlers count: 1
Failed to receive packet from <IP_ADDRESS>:56039
System.InvalidOperationException: Invalid Message-Authenticator in packet 224
   at Flexinets.Radius.Core.RadiusPacket.Parse(Byte[] packetBytes, IRadiusDictionary dictionary, Byte[] sharedSecret)
   at Flexinets.Radius.RadiusServer.GetResponsePacket(IPacketHandler packetHandler, String sharedSecret, Byte[] packetBytes, IPEndPoint remoteEndpoint)
   at Flexinets.Radius.RadiusServer.HandlePacket(IPEndPoint remoteEndpoint, Byte[] packetBytes)
01e00050b12b2bc26bd8156b7b962b27e0fd924a0106757365720212503043021b6fdb74b3b81454c790a64004067f000101050600000714501200000000000000000000000000000000070600000001

Checked the secret? That is what you would get with an incorrect secret if a message-authenticator attribute is included

Thanks. It wasn't the secret but something else in the params I was passing that I didn't realize until I tool a closer look. The server and port needed to be connected with a :. Sorry about that.

well it did result in a small bug fix so, yay anyway https://github.com/vforteli/Flexinets.Radius.Core/commit/aa8fc93daa46ceca3e8d1fe89f8ef109211c3391


New logger: generate padding fields

I propose an alternative to the serialization: add auto-generated padding fields to the struct where necessary (ie. for nested types).
Advantages:

only one memcpy necessary when logging a message
no code bloat with many auto-generated functions
automatic handling of (newly added) nested types in the message template

Padding is added at the end of each struct as well, which is needed for nested types. ATM this is logged as well, so that the readout still works. If we use this, we can cut these, so that there is only a small overhead in size for nested structs. This then requires the log reader to ignore padding bytes at the end of a struct format definition (if it is not used in a nested struct).
@dagar @kd0aij what do you think?
Tested on SITL & Pixracer.

This sounds good to me as long as we're confident no additional padding is possible on any platform/compiler.

Generated log looks OK except that parameter changes aren't recorded

@dagar yes that's the case. structs are always aligned & padded according to the largest member, which is the 64bit timestamp in our case.


update to use php-http/httplug:^2.0

when will there be a release to require:
"php-http/httplug": "^2.0",

??
currently the composer.json requires
"php-http/httplug": "^1.0",

This is all that is needed. The composer tests pass as well
$ git diff origin/master
diff --git a/composer.json b/composer.json
index 2dadf57..b74ec1a 100644
--- a/composer.json
+++ b/composer.json
@@ -12,7 +12,7 @@
     ],
     "require": {
         "php": "^7.1",
-        "php-http/httplug": "^1.0",
+        "php-http/httplug": "^2.0",
         "react/http-client": "~0.5.9",
         "react/dns": "^1.0|^0.4.15",
         "react/socket": "^1.0",
@@ -21,7 +21,7 @@
         "php-http/discovery": "^1.0"
     },
     "require-dev": {
-        "php-http/client-integration-tests": "^0.6",
+        "php-http/client-integration-tests": "^2.0",
         "php-http/message": "^1.0",
         "phpunit/phpunit": "^4.8.36 || ^5.4"
     },
diff --git a/src/Client.php b/src/Client.php
index 21d3389..946807d 100644
--- a/src/Client.php
+++ b/src/Client.php
@@ -79,7 +79,7 @@ class Client implements HttpClient, HttpAsyncClient
     /**
      * {@inheritdoc}
      */
-    public function sendRequest(RequestInterface $request)
+    public function sendRequest(RequestInterface $request): ResponseInterface
     {
         $promise = $this->sendAsyncRequest($request);

Hello @jraubswitch !
I've just created the PR based on your suggested changes.


[Linux] Certificates stop working after 1 month

Hi,

I probably opened the issue in the wrong repo but I don't really know if it's related with the client or the server. I think it's related with the server configuration, that's why I opened it here.

I have a Openbazaar-go running in a Debian9 machine and the client running on an Ubuntu machine.
I generated the certs like this:
openssl genrsa -out rootCA.key 4096
openssl req -x509 -new -nodes -key rootCA.key -days 1024 -out rootCA.crt -subj "/C=DE/ST=Germany/L=Walldorf/O=SAP SE/OU=Tools/CN=rootCA"
openssl genrsa -out server.key 4096
openssl req -new -key server.key -out server.csr -subj "/C=DE/ST=Germany/L=Walldorf/O=SAP SE/OU=Tools/CN=${MI_IP}"
openssl x509 -req -in server.csr -CA rootCA.crt -CAkey rootCA.key -CAcreateserial -out server.crt

It happened twice that the client started to show "insecure-response" error and it stopped working even if the server was running.
I realized it happened exactly one month after I created the certs.
I have no idea about certificates and stuff so it will probably be that I misconfigured something but I see -days 1024 there.
Is there any configuration error or is it related with Openbazaar itself?
I followed this guide:
https://github.com/OpenBazaar/openbazaar-desktop/blob/master/docs/connectionIssues.md
(the link of the trusted certs how-to is broken, btw)
Thank you!

You got it. I think you can either omit that field or set it very high if you want it to last forever.

Done!
Thank you very much!
The certificate creation and stuff is very messy.

Well waddaya know, here it is exactly 30 days later and again my cert expired...

also @hoffmabc no idea why but the /util/Socket.js file in my application does not match the file on github:
https://pastebin.com/XATHUW7R (/Applications/OpenBazaar2.app/Contents/Resources/app/prod/utils/Socket.js)
vs:
https://github.com/OpenBazaar/openbazaar-desktop/blob/master/js/utils/Socket.js

Ok maybe it's the instructions at https://github.com/OpenBazaar/openbazaar-go/blob/master/docs/ssl.md
should the last line have a days 1024 command?

@ekaitz-zarraga  I have solved this problem I think:
the instructions here:
https://github.com/OpenBazaar/openbazaar-go/blob/master/docs/ssl.md
are missing one more -days 1024 at the very end of the last line, to make the server.crt certificate 1024 days until expiry. Otherwise, openssl gives it a default of 30. I'll make a PR.


Request for sharing models

Dear Authors,
Thank you for sharing your work and code :)
Would it be possible for you to also upload the models (i.e. influence models & model checkpoints at each 10k timesteps) to huggingface? It would be really helpful for gaining more insights from your work..!
Thank you in advance :)

@hbin0701 Hi, thanks for your interest and reasonable request.
I have updated README to add model checkpoint links.
I shared those with my own huggingface account. We will migrate those (and larger files) to our group huggingface organization later.

Thank you! Sharing them have been of a great help! :)


Apple Computer(MAC) browsing table slide problem

Hi,Thanks for serenity .
Here's a serious problem, I don't know how to solve it, and the official Demo has the same problem.
It's not smooth to slide up and down on the MAC computer when you browse the table
I hope the official gives the solution, thank you

What is not "smooth" exactly? Any screenshots?

Closing due to inactivity.

I can't upload the video. Please use MAC to browse the Demo table and roll with a roller. There is a problem with the interface.

@volkanceylan I can't upload the video. Please use MAC to browse the Demo table and roll with a roller. There is a problem with the interface.


[Feature] Map signature overview mode/list mode

So, the ability to fairly quickly add and list the signatures of a system is very helpful both for W-space and K-space.
However i have found that when my primary goal is to use the mapper as an exploration aid, having to mouse over each system one by one to see when it was last scanned becomes a bit tedious.
Additionally, when used to share sig results among corp mates, in order to see what signatures a system has, it is required to pan to the system to select it (can be a decent amount of panning if it is a large chain/map), then pan down to the info pane, then select signatures. Doing this for each system to  find if any interesting sites have been recently scanned is a bit difficult.
So, the idea i propose is a mode that could be toggled, that would instead of displaying the map, would display a table or list of systems.

a column for the system name, mouse over and selecting still doing exactly the same as it does on the map now.
a column telling how long since it was last scanned
a column for flags/notes (npc kills, you are here, PvP activity and such)
a column listing the signatures of the most recent scan.

The signatures could easily be listed in a short hand, for example:
ABC-123	Cosmic Signature	Combat Site	Mul-Zatah Monastery	100%	1.22 AU
GBY-392	Cosmic Signature	Wormhole	Unstable Wormhole	100%	51.23 AU
DNT-631	Cosmic Signature	Data Site	        Local Serpentis Production Installation	0%	52.28 AU
GDW-000	Cosmic Signature	Relic Site		0%	8.14 AU
RFT-749  Cosmic Signature    Gas Site          Calabash Nebula  100%  5.47 AU
could be represented as:
123-CMB - MZM ▌392-WH ▌631-DATA - LSPI ▌000-RELIC ▌749-GAS - CN
A working example of such a list in excel* can be found here:
https://docs.google.com/spreadsheets/d/1maRQ90fbFigouBpbz8rrSGAFdQHxDKnP6LE-IK_o6a4/edit#gid=575127803

The example given is a slightly modified version of the latest version of the E-UNI Project Solitude Coordinated Scanning Endeavor.

If you are in any way interested in this idea but want/need any more information please let me know.
I really like what your tool can already do, and would love to see it grow to be the best for both WH mapping, and K-space exploration signature logging.

Interesting idea! Is there more demand for it?

Well, I am an E-Uni member, and there are a good number of use that use a spreadsheet like the one I linked you, over this mapper tool simply because of the quick ability to see all the systems all at once, and all of their sigs all at once.
So, i can 100% say if such a feature were to be implemented we would more than likely swap over to your tool for its other great benefits. So, that's at LEAST roughly 50 E-Uni members that would love it to be added.
Also, i am sure that there are many others who would love the quick overview mode for both k-space and w-space, for many uses that i cant think of. :)

I'd like to second the motion for a list/table view of all signatures for a selected campus' map.

Can none of the EUNI guys maybe code that themselves? Make them aware that this is an open source project if they don't know yet.
I like the idea though and will probably have a shot at it when I have some time.

My group is k-space based and this type of feature would be a great boon.  I'm thinking of having system group lists that you can save or maybe attach to a map.  So you can add which systems you want to show in the sig overview list.
This feature is second on my list of things I want to work on in wspace, but I am merely a learning programmer so I'm not sure when I'll get to the level where I can effectively tackle something like this.

columns could include :

how long since last visited by your pals (i.e. where should you go looking for pewpew)
activity indicator (based on wh kills from known inhabitants last 7 days or something like this)
pals in system count
and with some form of support for auto-deletion or expiration date on sigs and WHs :
WH count
WH relics & data count
NS relics count
unscanned sigs count
unprobed sigs count


[benchmark] Fix swiftpm based benchmark build on Linux.

Please test with following pull request:
apple/swift-integration-tests#36
@swift-ci please test linux platform

Build failed
Swift Test Linux Platform
Git Sha - 50b4e0eb99e515918946b739722cef6af6f0ef4a

Please test with following pull request:
apple/swift-integration-tests#36
@swift-ci please test linux platform

Please test with following pull request:
apple/swift-integration-tests#36
@swift-ci please test linux platform

Please test with following pull request:
apple/swift-integration-tests#36
@swift-ci please test linux platform

Please test with following pull request:
apple/swift-integration-tests#36
@swift-ci please test linux platform

Build failed
Swift Test Linux Platform
Git Sha - 50b4e0eb99e515918946b739722cef6af6f0ef4a

Please test with following pull request:
apple/swift-integration-tests#36
@swift-ci please test linux platform

Please test with following pull request:
apple/swift-integration-tests#36
@swift-ci please test linux platform

Please test with following pull request:
apple/swift-integration-tests#36
@swift-ci please test linux platform

Build failed
Swift Test Linux Platform
Git Sha - a2727a242e3daddbb1f5367702d66528a6b2a39c

@swift-ci build-toolchain

@swift-ci build toolchain

Please test with following pull request:
apple/swift-integration-tests#36
@swift-ci please test linux platform

@swift-ci smoke benchmark

Please test with following pull request:
apple/swift-integration-tests#36
@swift-ci please test linux platform

@swift-ci smoke benchmark

Please test with following pull request:
apple/swift-integration-tests#36
@swift-ci please test linux platform

Build comment file:
No performance and code size changes

How to read the data
The tables contain differences in performance which are larger than 8% and differences in code size which are larger than 1%.
If you see any unexpected regressions, you should consider fixing the regressions before you merge the PR.
Noise: Sometimes the performance results (not code size!) contain false alarms. Unexpected regressions which are marked with '(?)' are probably noise. If you see regressions which you cannot explain you can try to run the benchmarks again. If regressions still show up, please consult with the performance team (@eeckstein).

Hardware Overview
  Model Name: Mac Pro
  Model Identifier: MacPro6,1
  Processor Name: 12-Core Intel Xeon E5
  Processor Speed: 2.7 GHz
  Number of Processors: 1
  Total Number of Cores: 12
  L2 Cache (per Core): 256 KB
  L3 Cache: 30 MB
  Memory: 16 GB

@swift-ci smoke test os x platform

Looking at those two compiler errors @aciidb0mb3r points out above… Aren't these bugs in how Glibc imports the RUSAGE_SELF and _SC_PAGESIZE  constants?

Feel free to file on swift.org.

Filed SR-9152.


[EDA] : Add county fields to address

Note: FOR INTERNAL USE ONLY. For support questions, please use the EDA Chatter group in The Power of Us Hub.
For feature requests, please use the Ideas tab in The Power of Us Hub.

Do you want to request a feature or report a bug?
feature

What is the current behavior?
No field to store County data

What is the expected behavior?
There should be fields in EDA to store County to addresses

What is the motivation / use case for changing the behavior?

Acceptance Criteria

Given: I am entering address
When: I try to enter the county information
Then: I should be able to add the data to the fields

**lurch: attach W-041323

Tracking W-041323


2.0 migration from Forest back to SQlite

Hi, I'm trying to upgrade to 2.0 and testing.
ForestDB is removed, but is somewhere some tutorial solution how to migrate current users back to SQLite?
Because we have about few thousands users still on ForestDB. We disable forest for new users when deprecation was announced, but still we have too much old users on forest and need to migrate them with 2.0 back to SQLite.
Thanks
We can do it before migrate to 2.0 (until we have time now), but I have no idea how, because I know, that somewhere was written that migration is possible only from SQLite to ForestDB :'(

Hi @mtrakal,
Currently, CBL does not provide migration from ForestDB to SQLite.
FYI: I believe your product uses CouchDB as a backend. Honestly speaking, CBL 2.x supports only new replication protocol. New replication protocol is not compatible with CouchDB.
Thanks,
Hideki

Oh, thanks for info.
Do you plan to support older 1.4.x for maintainance for some time when will be found some major bugs?
Thanks again for info.

Hi @mtrakal,
We will keep maintaining CBL v1.x.


Additional API support for labels.

Implements the ability to manage repository labels and an issue's labels via the API.  See corresponding API documentation in comments below.
As a part of implementation, an ID field has been added to the client-side Label struct, therefore issue API responses will also include this field in the label information of the issue.
This PR depends on gogits/go-gogs-client#35

Labels
List labels for a repository
GET /repos/:owner/:repo/issues

Response
Status: 200 OK
Content-Type: application/json

[
  {
    "id": 5,
    "name": "Feature Request",
    "color": "#0052cc"
  },
  {
    "id": 6,
    "name": "Help Wanted",
    "color": "#006b75"
  }
]

Get a single label
GET /repos/:owner/:repo/issues/:index

Response
Status: 200 OK
Content-Type: application/json

{
  "id": 6,
  "name": "Help Wanted",
  "color": "#006b75"
}

Create a label
Only users with write access to a repository can create an issue.
POST /repos/:owner/:repo/labels

Parameters

Name
Type
Description

name
string
Required The name of the label

color
string
Required The label color as a hex value e.g, #006b75

Example
{
  "name": "Help Wanted",
  "color": "#006b75"
}

Response
Status: 201 Created
Content-Type: application/json

{
  "id": 6,
  "name": "Help Wanted",
  "color": "#006b75"
}

Edit a label
Issue owners and users with write access can edit an issue.
PATCH /repos/:owner/:repo/labels/:index

Parameters

Name
Type
Description

name
string
The name of the label

color
string
The label color as a hex value e.g, #006b75

If a parameter is empty, that part of the label will remain unchanged
Example
{
  "name": "Help Wanted",
  "color": "#006b75"
}

Response
Status: 200 OK
Content-Type: application/json

{
  "id": 6,
  "name": "Help Wanted",
  "color": "#006b75"
}

Delete a label
Only users with write access to a repository can delete labels.
DELETE /repos/:owner/:repo/labels/:id

Response
Status: 204 No Content

Issue Labels
List labels for an issue
GET /repos/:owner/:repo/issues/:index/labels

Response
Status: 200 OK
Content-Type: application/json

[
  {
    "id": 5,
    "name": "Feature Request",
    "color": "#0052cc"
  },
  {
    "id": 6,
    "name": "Help Wanted",
    "color": "#006b75"
  }
]

Add one or more labels to an issue
Only users with write access to a repository can add labels to issues.
POST /repos/:owner/:repo/issues/:index/labels

Parameters

Name
Type
Description

labels
int array
Required Array of label IDs

Example
{
  "labels": [1,6]
}

Response
Status: 200 OK
Content-Type: application/json

[
  {
    "id": 1,
    "name": "Bug",
    "color": "#e11d21"
  },
  {
    "id": 5,
    "name": "Feature Request",
    "color": "#0052cc"
  },
  {
    "id": 6,
    "name": "Help Wanted",
    "color": "#006b75"
  }
]

Replace all labels on an issue with one or more labels
Only users with write access to a repository can replace labels on an issue.
PATCH /repos/:owner/:repo/issues/:index/labels

Parameters
Same as adding labels to an issue
Response
Status: 200 OK
Content-Type: application/json

[
  {
    "id": 1,
    "name": "Bug",
    "color": "#e11d21"
  },
  {
    "id": 6,
    "name": "Help Wanted",
    "color": "#006b75"
  }
]

Remove labels on an issue
Only users with write access to a repository can remove labels on an issue.
DELETE /repos/:owner/:repo/issues/:index/labels/:id

Parameters
Same as adding labels to an issue.
Response
Status: 204 No Content

Remove all labels on an issue
Only users with write access to a repository can remove labels on an issue.
DELETE /repos/:owner/:repo/issues/:index/labels/

Response
Status: 204 No Content

Thanks for the feedback.  I updated the code accordingly.
I noticed some extra logic is needed on adding/replacing label IDs to account for bad input (e.g., a set of label IDs contains one or more invalid IDs, where they either do not exist in the DB or are not valid for the repository).

The second commit adds the following:
-On add/replace issue labels, look up the label IDs first and make sure they are associated with the same repository as the issue. Do not modify the issue's labels unless all are valid.
-Extra error types to handle add/replace issue labels.
One of the extra error types is used when more than one label passed to the server is not valid e.g., a client attempts to add three labels to an issue but one does not exist and the other is not associated with the issue's repository. With this extra type, both errors can be passed back to the client so they have a complete picture of what went wrong. I gave this a generic name as it might be useful in other areas.

The latest commit should address all comments except those related to error handling and new error types.

Thanks!


[Google Cloud Platform Search] ... error

Extension
https://www.raycast.com/klaa97/gcp-search
Raycast Version
1.82.1
macOS Version
14.2.1
Description
Error:
RangeError: "length" is outside of buffer bounds

Buffer.utf8Write:buffer:1066:13
Sl.SD:searchProjects.js:5:28309
we.finish:searchProjects.js:5:27525
searchProjects.js:15:42847

Steps To Reproduce

open extension gcp from raycast to seach
can not seach

Current Behaviour
can not search for gcp project
Expected Behaviour
search for gcp project

@pernielsentikaer Which version of Node is Raycast using? It appears that version 22.7.0 suffers from a critical error: https://github.com/nodejs/node/issues/54518 tied to protobufs, and the problem seems similar here.

Indeed the exception happens when initializing the google-cloud ProjectsClient here:
https://github.com/raycast/extensions/blob/3536d843f032492e720f0f618cd7ce44a9ce44ad/extensions/google-cloud-platform-search/src/projects.tsx#L5
I tried console.log(process.versions.node) and it is using 22.7.0, so that's the issue.

@pernielsentikaer Which version of Node is Raycast using? It appears that version 22.7.0 suffers from a critical error: nodejs/node#54518 tied to protobufs, and the problem seems similar here.

how to check node version raycast using

fork and change extension code like @m-gentil does work

Unfortunately, this was done accidentally. We are going to publish a hotfix (1.82.3) that changes it back to 20.15.1.


now deployment failing (sh: nuxt: not found)

package.json
{
  "name": "www",
  "version": "1.0.0",
  "description": "personal website",
  "author": "tchret",
  "private": true,
  "scripts": {
    "dev": "nuxt",
    "build": "nuxt build",
    "start": "nuxt start",
    "generate": "nuxt generate",
    "lint": "eslint --ext .js,.vue --ignore-path .gitignore .",
    "precommit": "npm run lint"
  },
  "dependencies": {
    "@nuxtjs/axios": "^5.0.0",
    "@nuxtjs/markdownit": "^1.2.0",
    "emoji-mart-vue": "^2.4.5",
    "lodash": "^4.17.5",
    "nuxt": "^1.0.0",
    "vue-lodash": "^1.0.4",
    "vue-markdown": "^2.2.4",
    "vue-markdown-loader": "^2.4.0"
  },
  "devDependencies": {
    "babel-eslint": "^8.2.1",
    "cross-env": "^5.0.1",
    "eslint": "^4.15.0",
    "eslint-loader": "^1.7.1",
    "eslint-plugin-vue": "^4.0.0",
    "node-sass": "^4.7.2",
    "nuxt-sass-resources-loader": "^1.1.1",
    "sass-loader": "^6.0.6",
    "style-loader": "^0.19.1"
  }
}

This question is available on Nuxt.js community (#c2634)

How did you solve it?

same issue here

It happened to me also. I first installed using yarn.
Then installed a dependency using NPM and the error started happening.
Calling yarn fixed this issue for me.


Initial support for Unicode emojis

This is not meant to be a complete fix for #131 and #400, I didn’t want this PR to get too complex. This is only a first step in the right direction.
This makes sure that data from the emojibase-data package is preprocessed and placed into the assets/emojis directory. There is one file for each locale, with some files being identical to English because emojibase-data  doesn’t have translations for Czech for example.
The getCustomEmojis function has been modified to load both custom emojis and the locale-specific Unicode ones. The callers can then handle Unicode emojis with minimal changes.
Issues explicitly not addressed here:

Both code and UI presentation still speak about “custom” emojis. This can be addressed in a subsequent cleanup.
Quality of emojibase-data translation varies. For example, the gender neutral “scientist” is translated as “Wissenschaftler(in)” to German, the female “woman scientist” as “Wissenschaftlerin”. Both get the shortcode “wissenschaftlerin” – and Phanpy is using that shortcode as key, so React will complain about duplicate keys. Probably best to address this upstream, though some solution for duplicate shortcodes might also be in order.
Also given the varying quality of translations, searching both translated and English emoji descriptions/shortcodes might be a good idea.
I see tracking of recent emojis in the code but I don’t see it working in the UI. This code probably needs adapting as well.
Emoji variants with different skin tones are present in the data but cannot currently be selected in the UI.
Emoji subgroups are present in the data, not sure whether these need to be supported.
Depending on OS support, some emojis currently won’t display. It’s possible to detect unsupported emojis, a better idea might be using a custom emoji font however.

I tried running Prettier and got tons of changes, none of which are related to the code I touched. I guess I ignore this test failure then?

@palant your prettier is probably running without following the repo's config? You can run it on specific files too.
Few notes:

Are there screenshots or some deployed preview? Would be interesting to see how this works. (I've tried to enable automatic PR/branch deployments from Cloudflare Pages on my side, but doesn't seem to work on PRs opened by others)
I prefer this to be separate from Custom Emojis. It's confusing enough to combine both of them, and their short codes could clash.
Recent emojis only work for clicked/selected ones.

To be honest, I'm not a fan of adding an emoji picker — I personally use the OS's native emoji picker (and 3rd-party native emoji picker apps) that works across all web sites and apps. Also, if I wanted to add one, probably could use this https://github.com/nolanlawson/emoji-picker-element but the concerns are the huge file sizes (images, shortcode mappings, locale files) and maintenance work required to update it every year as the folks from Unicode Consortium keep adding more.
But I'm interested to see how this works. Preprocessed data is nicely done. If you try npm run build, could check out the bundle sizes via Sonda's report (generated sonda-report.html).

@palant your prettier is probably running without following the repo's config?

It clearly does respect the repo’s config, it corrects double quotes into single quotes for example. I’m running the same command as that test, yet it finds way more issues for some reason.

* Are there screenshots or some deployed preview? Would be interesting to see how this works. (I've tried to enable automatic PR/branch deployments from Cloudflare Pages on my side, but doesn't seem to work on PRs opened by others)

These are screenshots made on Linux (KDE):

Note that I explicitly put the categories of the server’s custom emoji’s on top (it’s also what Mastodon does). This doesn’t quite work for the “other” category which is always placed at the bottom, but this could be addressed.

* I prefer this to be separate from Custom Emojis. It's confusing enough to combine both of them, and their short codes could clash.

Quite frankly, I’d consider it more confusing if they were separate. As far as users are concerned, there is no difference between custom emojis and Unicode ones.
Short code clashes don’t really seem to be an issue (other than for React which can be addressed). It’s a bit confusing if multiple icons show up with the same shortcut in the search, but the users can just select the icon they like. And Unicode emojis aren’t inserted as their shortcuts but rather as the actual Unicode symbol.

* Recent emojis only work for clicked/selected ones.

I am aware of that. Still, they don’t show up for me despite clicking. But that’s a side issue, I can look into it if the general direction is approved.

To be honest, I'm not a fan of adding an emoji picker — I personally use the OS's native emoji picker (and 3rd-party native emoji picker apps) that works across all web sites and apps.

Well, I didn’t add one – there is one already, and it doesn’t do too badly. It could do with a category selector at the top, otherwise I don’t see big issues however.
Linux doesn’t have a native emoji picker, and native emoji pickers aren’t always too convenient. Besides, they won’t let you pick the server’s custom emojis, and providing a unified experience for all emojis no matter the source would be beneficial.

Also, if I wanted to add one, probably could use this https://github.com/nolanlawson/emoji-picker-element but the concerns are the huge file sizes (images, shortcode mappings, locale files) and maintenance work required to update it every year as the folks from Unicode Consortium keep adding more.

I did look into this one first. I then decided that adding the necessary functionality to the existing emoji picker would be simpler. It works with the same data, but its preprocessing won’t reduce file size quite as much. The only benefit I see would be detection of broken emojis – something that would be fairly easy to address.
An emoji font would be quite heavy. The decision whether to use one is independent of an emoji picker however. It would be a way to provide a unified experience across all platforms, including the ones with less than optimal emoji support. There is already #476 on that at the very least.

If you try npm run build, could check out the bundle sizes via Sonda's report (generated sonda-report.html).

The bundle sizes aren’t affected, these JSON files aren’t compiled into the bundles. I considered compiling them into the bundles and using dynamic loading to make sure only the required locales are being loaded, but this turned out unnecessarily complex. So it’s static files and fetch now which is much simpler.
The file sizes vary between 260 KiB (Chinese) and 375 KiB (Ukrainian). Altogether it’s 6.5 MiB which could be reduced to 4.6 MiB by removing data duplicating the en locale.

I’m running the same command as that test, yet it finds way more issues for some reason.

Ok, it’s definitely a versions mismatch. I’ve been running prettier 3.4.1, the test is running some version that is considerably older. It isn’t version 2.8.0 which is listed in package-lock.json but that version is closer. I guess the test somehow cached a version from when it was run originally?
If I may suggest, it would be a good idea to add prettier as an explicit devDependency, probably with a script to run it. This will make certain that everyone is on the same version.


Added threads dependency to connection_pool_test target (fix for Fedora 22)

also made Doxygen dependency optional to suppress warnings of cmake.
Without threads dependency make fails on connection_pool_test target.

Merged, thank you!


Update all RTK APIs to have the same base url

Description
Some of the RTK APIs are using different base urls. I think this is a remnant of the unified fides merge. Some are using the base url that is based off of process.env.NEXT_PUBLIC_FIDESOPS_API
https://github.com/ethyca/fides/blob/6f886898e518e3ed61d12e30b88426b8737f98b8/clients/admin-ui/src/constants.ts#L3-L7
while others are based off of process.env.NEXT_PUBLIC_FIDESCTL_API
https://github.com/ethyca/fides/blob/6f886898e518e3ed61d12e30b88426b8737f98b8/clients/admin-ui/src/features/taxonomy/taxonomy.slice.ts#L10
Desired Changes

Consolidate all of the .env files to have one API value for the backend. Use NEXT_PUBLIC_FIDES_API instead of the two pre unified fides variables

https://github.com/ethyca/fides/blob/6f886898e518e3ed61d12e30b88426b8737f98b8/clients/admin-ui/.env.development#L1-L3

Update all RTK APIs to use the BASE_URL variable in constant.ts
Update any other areas of the codebase that are directly referencing process.env and either update them to use BASE_URL where appropriate or create new constants for the process.env variables. This will allow us to easily keep track of what code is using any specific process.env variable.

@TheAndrewJackson some background on how the next rewrites work https://github.com/ethyca/fides/pull/686#discussion_r880827842


Added functionality for Incident

Two pieces of valuable information are not included with an incdient:

Whether the summary_details includes a subject or a description
The extra data contained in the details log entry

With this small code update, an incident is now aware of both of these items.  There are two new methods for incidents:

incident.get_detail_log_entry(requester_id)  #Returns json of log entry
incident.has_subject()  # Boolean return, if no subject, than it has details (incident.trigger_summary_data.subject/details)

I haven't taken a look at this too much yet but before I do can you do the following:

Fix the flake errors that are causing the build to fail
Remove debug printing
Remove the .idea/dictionaries/dguimaraes.xml file from the commit and squash all commits into a single commit. (Possibly add .idea to the .gitignore to prevent those making it in in the future)

Thanks!

I made a couple of the changes you requested.  Still trying to figure out how to squash the commits.  Also, Trying to understand the error that is occuring so I can fix it.

I can handle the squash when I merge so don't worry about it.

Yes!!  That is much simpler.  I write a lot of ruby and maintain a lot of perl.  I was looking for a better way to do this.  Thank you!  I'll commit and submit the change.

This suggestion helped me to understand your code a lot better.  I see how that works and I implemented the changes.  I wanted your input on the name of the method before I make another commit.  I dropped the word get from the method name so it is just called details_log_entry
i.e. incident.details_log_entry()
Your solution was more elegant.  The requester_id had no business being there nor did making the extra call to the PD API.  Thank you for taking the time to step though this with me!

Yeah, when I initially wrote this library it was made, possibly too, generic to adapt to new options being added to the API without needing code changes. The result being the code is a bit complex to wrap your head around at first and the lack of docs doesn't help.
No problem!

I really like generic code and I'm always striving to generalize mine.  I'm not in a development role so I don't have a lot of support in that area (even though I spend most of my time developing).  I really appreciate this opportunity to walk though this with you.

Don't know if you missed this comment because I double commented on the same line but reiterating it here:

Yeah, thinking about it more I think it would make the most sense to have something like

def get_trigger_log_entry(self, **kwargs):
    match = TRIGGER_DETAILS_ID_RE.search(self.trigger_details_html_url)
    return self.log_entries.show(match.group("trigger_id"), **kwargs)

This way you get easy access to the triggering log event and still have control over the specific options you pass through.
Would that meet your needs?

Specifically an incident has many log entries. This method is asking about a specific log entry, the one that triggered the incident. I do see the value is having easy access to the log entry that triggered the incident but I'm not sure it's worth having a method that always returns the detailed output opposed to just allowing that to be passed through. For example with the method above you could just call incident.get_trigger_log_entry(include=["channel"])

Sorry, I did miss that.  I fixed it.

I manually squashed the commits when merging so this won't close cleanly. You'll want to fix up your local fork or refork before making future contributions.
I'll close this manually. Thanks for the contribution! This has been pushed to PyPI as pygerduty-0.32.0


Apple 2FA codes received by phone call don't work

Build/Submit details page URL
No response
Summary
Your Apple account should be giving you 2FA codes by phone call

Try to start a build that requires your Apple credentials (App Store build in my case)
Log in with your password (type it in or get it from your keychain)
You'll be asked to enter a 2FA. Apple will call and give it to you. Enter it
eas-cli will say Invalid Code

Tried this more than a few times and finally just tried to get Apple to give me codes via SMS. Once I switched to SMS, everything worked as expected.
Managed or bare?
Bare
Environment
System:
OS: macOS 13.1
Shell: 5.8.1 - /bin/zsh
Binaries:
Node: 16.18.1 - ~/.nvm/versions/node/v16.18.1/bin/node
npm: 8.19.2 - ~/.nvm/versions/node/v16.18.1/bin/npm
Managers:
CocoaPods: 1.11.3 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: DriverKit 22.2, iOS 16.2, macOS 13.1, tvOS 16.1, watchOS 9.1
IDEs:
Android Studio: 2021.3 AI-213.7172.25.2113.9123335
Xcode: 14.2/14C18 - /usr/bin/xcodebuild
npmPackages:
expo: ~46.0.19 => 46.0.19
react: 18.0.0 => 18.0.0
react-dom: 18.0.0 => 18.0.0
react-native: 0.69.6 => 0.69.6
react-native-web: ~0.18.7 => 0.18.10
npmGlobalPackages:
eas-cli: 3.7.2
expo-cli: 6.0.8
Expo Workflow: bare
Error output
No response
Reproducible demo or steps to reproduce from a blank project
No demo needed

How did you manage to change your 2FA method from phone call to SMS?

@andresespinosapc sorry just saw the comment now.
It was tedious. There's no explicit way to do it but it was something along the lines of logging into my Apple account and asking them to send my 2FA code another way. The last method you used is the one they'll continue using afairc

Just an FYI for anyone with this issue, Apple defaults to SMS if you are already on a phone call.
All you really have to do it log in through the CLI while you are on a phone call and it should work.