Add encryption to SQS queue

NOTE: I haven't tested this and I don't use lifecycled myself! I was browsing the issues for buildkite/elastic-ci-stack-for-aws and saw this issue, and I've recently done the same thing for some SQS queues at $work.
Fixes buildkite/elastic-ci-stack-for-aws#458

Awesome, many thank! I'll test this out.

Also implements https://github.com/buildkite/lifecycled/issues/60.

I suspect this may run into issues with SNS not being able to publish encrypted messages on the queue.


Manual installation on Debian 12

This ticket is to track the issue with building on the current version of Debian as of the time of writing (Debian 12).
Following the manual installation instructions on Debian 12 on a Raspberry Pi 3 B results in a compilation failure when building wheel for numpy.  This occurs when installing pip requirements.txt. The error is simply a return code of 1 from gcc with no accompanying error message.
error: Command "aarch64-linux-gnu-gcc -Wsign-compare -DNDEBUG -g -fwrapv -O2 -Wall -g -fstack-protector-strong -Wformat -Werror=format-security -g -fwrapv -O2 -fPIC -DNPY_INTERNAL_BUILD=1 -DHAVE_NPY_CONFIG_H=1 -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE=1 -D_LARGEFILE64_SOURCE=1 -Ibuild/src.linux-aarch64-3.11/numpy/core/src/common -Ibuild/src.linux-aarch64-3.11/numpy/core/src/umath -Inumpy/core/include -Ibuild/src.linux-aarch64-3.11/numpy/core/include/numpy -Ibuild/src.linux-aarch64-3.11/numpy/distutils/include -Inumpy/core/src/common -Inumpy/core/src -Inumpy/core -Inumpy/core/src/npymath -Inumpy/core/src/multiarray -Inumpy/core/src/umath -Inumpy/core/src/npysort -Inumpy/core/src/_simd -I/home/pi/seedsigner/include -I/usr/include/python3.11 -Ibuild/src.linux-aarch64-3.11/numpy/core/src/common -Ibuild/src.linux-aarch64-3.11/numpy/core/src/npymath -c build/src.linux-aarch64-3.11/numpy/core/src/multiarray/arraytypes.c -o build/temp.linux-aarch64-3.11/build/src.linux-aarch64-3.11/numpy/core/src/multiarray/arraytypes.o -MMD -MF build/temp.linux-aarch64-3.11/build/src.linux-aarch64-3.11/numpy/core/src/multiarray/arraytypes.o.d" failed with exit status 1
It's possible this message from a few pages back in the output is related (although the left hand operand looks fine to me...).
aarch64-linux-gnu-gcc: numpy/core/src/multiarray/common.c
      numpy/core/src/multiarray/arraytypes.c.src: In function ‘get_dummy_stack_array’:
      numpy/core/src/multiarray/arraytypes.c.src:65:26: error: lvalue required as left operand of assignment
         65 |     Py_TYPE(&new_fields) = NULL;
            |                          ^

Full output of the command was too big for Microsoft GitHub to handle (> 64Kb), so I've put it here:
https://controlc.com/56b25e4a
The test system started with a stock Raspberry Pi OS image (Debian 11, aka bullseye) and then upgraded to bookworm (Debian 12) using the standard apt commands.
There were some changes required for doing the manual install on Debian 12. Some examples: Python 3.11 is already installed. Debian wants all python packages to be installed with apt and go as far to only allow pip in a virtual environment, and even then only the pip executable that is in that virtual environment will function.
Below are the commands that should reproduce this issue on Debian 12:
sudo apt install -y libzbar0 libbcm2835-dev python3-virtualenvwrapper git python3-pip python3.11-venv
git clone https://github.com/SeedSigner/seedsigner
cd seedsigner
python3 -m venv `pwd`
./bin/pip install -r requirements.txt

I didn't run through the entire setup process, but I tried to reproduce this issue from last year and was unable to do so. The steps to repro were slightly different because ./bin/pip doesn't seem to be there anymore, but the updated instructions below are what I used to test.
sudo apt install -y libzbar0 libbcm2835-dev python3-virtualenvwrapper git python3-pip python3.11-venv
git clone https://github.com/SeedSigner/seedsigner
cd seedsigner
python3 -m venv `pwd`
. bin/activate
python -m pip install -r requirements.txt

I didn't run through the entire manual instructions again, but if they work now we can close this ticket.


Simplify onboarding

Closes https://github.com/prax-wallet/prax/issues/226
Closes https://github.com/prax-wallet/prax/issues/227
Closes https://github.com/prax-wallet/prax/issues/221

Lots of deletions related to onboarding flow
Removes wallet birthday from being a user-facing concept
Onboarding automatically chooses random rpc + frontend for user
Removes "Forgot password?" flow
Only generates 12 word seedphrases

left some suggestions mainly around rpc selection; great work streamlining the onboarding flow!

Will need final leadership review before deploying, but can follow up with necessary follow up PRs for additional changes


Getting "403 Forbidden" after first setup

The Problem
I ran tvheadend on raspian light with a Pi-TV-Hat without problems. To avoid double power consumption, I now tried to integrate it int HomeAssistant.
I installed the GauthamVarmaK's addon first, but after I found out that the hardware isn't discovered. When searching for a solution, I found out that this addon is outdated. So I uninstalled it, deleted the repository, installed the Daniel Figus repository and then the addon. In the first setup I gave an admin with password and a user with password. After finishing the first setup, I keep on getting "403 Forbidden" and can't log in to TVHeadend anymore.
Environment

Addon version: 5.1.11
HA OS version: Home Assistant OS 12.3  (aarch64 / raspberrypi4-64)
Home Assistant Core: 2024.6.2
Home Assistant Supervisor: 2024.06.0
Installation methode: HA OS
Hardware: RPI 4 4GB RAM

Logs
2024-06-11 14:59:14.174 [  ERROR] http: <IP_ADDRESS>: HTTP/1.1 GET (1) /api/hassio_ingress/eU7H1950nOft3aqcTxO5qP3FPD2C846SoEHIw4HcoGQ/extjs.html -- 403
Steps to reproduce
Click on "Open WEB UI" or sidebar first gives you a login window and then a "403 Forbidden"  page
Additional Information
On a bare metal Raspberry with the latest RaspiOS TVHeadend works fine also with admin and user accounts configured.

Sorry, after looking into the hidden issues, I saw that the was a similar problem in March. I will try to setup the server from scratch and avoid to install the GauthamVarmaK's addon first.

Found out that I needed to put the allowed network to <IP_ADDRESS>/0 instead of my home network.
Now I'm able to log in, but next issue is: It doesn't show my Pi-TV-Hat in DVB-Inputs (on a native Raspbian this is shown)

Hi, is the TV-Hat working on HA OS natively? That is usually the limiting factor, as HA OS does not ship with the required drivers and firmware. If it's not running on HA OS, there is nothing the addon can use.

Found out that I needed to put the allowed network to <IP_ADDRESS>/0 instead of my home network.

If you logon via the HA ingress (web ui button), it will go through HA and nginx reverse proxy and hence mask your real IP. TVH will then only see the HA internal IP as caller. There is a PROXY protocol & X-Forwarded-For config in TVH, which might help. But as HA OS will already do authentication, you might just want to allow <IP_ADDRESS> in addition to your local network and not everything via <IP_ADDRESS>.
TVH is also directly accessible via HA-IP:9981 without the HA ingress and here TVH will see the correct caller IP. This direct access could be used from kodi for example.

Hi, is the TV-Hat working on HA OS natively? That is usually the limiting factor, as HA OS does not ship with the required drivers and firmware. If it's not running on HA OS, there is nothing the addon can use.

Thank you very much for the quick reply! How can I check if it is working on HA OS natively. I installed SSH and did some tests. If I do a ls in /dev, I can't see something like /dev/dvb/adapter0

Found out that I needed to put the allowed network to <IP_ADDRESS>/0 instead of my home network.

If you logon via the HA ingress (web ui button), it will go through HA and nginx reverse proxy and hence mask your real IP. TVH will then only see the HA internal IP as caller. There is a PROXY protocol & X-Forwarded-For config in TVH, which might help. But as HA OS will already do authentication, you might just want to allow <IP_ADDRESS> in addition to your local network and not everything via <IP_ADDRESS>.
TVH is also directly accessible via HA-IP:9981 without the HA ingress and here TVH will see the correct caller IP. This direct access could be used from kodi for example.

Thank you! This is very helpful. I'm totally new to HomeAssistant and didn't know about the <IP_ADDRESS> network address

Thank you very much for the quick reply! How can I check if it is working on HA OS natively. I installed SSH and did some tests. If I do a ls in /dev, I can't see something like /dev/dvb/adapter0

You would need to ssh into the HA OS directly, see documentation here and check there. What is available here, can be used by the addon. Do you see anything in dmesg output regarding dvb?

Closing as no response from reporter


fixed search error in visualizer.

Fixed error where disease search wasn't working. Due to minor changes from Visualizer.tsx.

Forcing into main after checks are passed.


Plot.ly delays action edit  page

Is your feature request related to a problem? Please describe.
When loading action edit page, plot.ly is delaying it.
Describe the solution you'd like
Use one of the bundles provided by Plot.ly

Fixed in release B.4.3


ERR_PACKAGE_PATH_NOT_EXPORTED reported when using as node script

Am I missing something? I have implemented as per the README but am getting this error.
☵ % node ./scripts/todo.js
node:internal/modules/cjs/loader:499
      throw e;
      ^

Error [ERR_PACKAGE_PATH_NOT_EXPORTED]: No "exports" main defined in /Users/shooftie/Code/project/node_modules/leasot/package.json
    at new NodeError (node:internal/errors:387:5)
    at throwExportsNotFound (node:internal/modules/esm/resolve:464:9)
    at packageExportsResolve (node:internal/modules/esm/resolve:688:7)
    at resolveExports (node:internal/modules/cjs/loader:493:36)
    at Function.Module._findPath (node:internal/modules/cjs/loader:533:31)
    at Function.Module._resolveFilename (node:internal/modules/cjs/loader:942:27)
    at Function.Module._load (node:internal/modules/cjs/loader:804:27)
    at Module.require (node:internal/modules/cjs/loader:1028:19)
    at require (node:internal/modules/cjs/helpers:102:18)
    at Object.<anonymous> (/Users/shooftie/Code/independent-web/scripts/todo.js:6:16) {
  code: 'ERR_PACKAGE_PATH_NOT_EXPORTED'
}

It is failing at the first hurdle
const leasot = require('leasot');

What node.js version are you using?

16.18.1, currently.

It works locally for me with the following versions, I'm not sure why it's not working for you honestly

I also setup a build matrix in the past and everything seems to pass (also on Windows, Ubuntu, OSX)
https://github.com/pgilad/leasot/actions/runs/3813158787

Oh you're using it programmatically, you're going to need to use it as an ESM module (and not using commonjs) https://blog.logrocket.com/how-to-use-ecmascript-modules-with-node-js/


4.22 release no longer buildable from source on Debian Stable, but Testing OK

Required information

Distribution: Debian
Distribution version: 11 (bullseye)
The output of "lxc info" or if that fails:

Kernel version: 5.10.0-10-amd64
LXC version: 1:4.0.6-2
LXD version: 4.22
Storage backend in use: ZFS (N/A)

Issue description
The lxd-4.22 tag is no longer buildable from source on Debian Stable following the provided instructions. This functioned as expected with lxd-4.21, and continues to work on Debian Testing/Unstable.
Steps to reproduce

Follow installing.md instructions for source build of lxd-4.22 tag on a Debian Stable system
Upon issuing make, note the following error:

root@server:/usr/local/go/src/github.com/lxc/lxd# make
CC="cc" CGO_LDFLAGS_ALLOW="(-Wl,-wrap,pthread_create)|(-Wl,-z,now)" go install -v -tags "libsqlite3"  ./...
github.com/osrg/gobgp/v3/pkg/server
# github.com/osrg/gobgp/v3/pkg/server
../../../../pkg/mod/github.com/osrg/gobgp/v3@v3.0.0-rc4/pkg/server/server.go:113:24: undefined: net.ErrClosed
make: *** [Makefile:33: build] Error 2

Repeat steps on Debian Testing; make succeeds as expected
Repeat steps on Debian Stable with lxd-4.21; make succeeds as expected

Information to attach

Debian Stable golang version: 2:1.15~1
Debian Testing/Unstable golang version: 2:1.17~1

Oh right I'll fix the announcement but one thing we did in this release is raise the minimal Go, LXC and kernel versions.
You need a minimal of Go 1.16 to build the LXD daemon now. That was forced on us by changes in the upstream net packages. For a few releases now we were pinning a specific older commit for it but that's a very bad idea from a security standpoint so we had to raise the minimal Go version.

https://linuxcontainers.org/lxd/docs/master/requirements/ has the minimal requirements.

Ah, cool, I had suspected it was something like that but I didn't see anything mentioning it in the release announcement. Thanks for the prompt response. 👍

I've updated the announcement now.

bullseye backports seems to have 1.17 https://packages.debian.org/search?keywords=golang&searchon=names&section=all&suite=bullseye-backports FWIW


change tenant connection to system

i need to get data from one tenant  so i set connection using
app(Environment::class)->tenant($website);
now i need to set this data to system database but using
DB::connection('system') not good for me because i need to use eloquent relation to insert this data
i need to set system connection globally again in the same method
anyway?

You can use Model::on('system')

still insert into tenant database  ??
hint : using in model UseTenantConnection

You can use Model::on('system')
still insert into tenant database ??
hint : using in model UseTenantConnection

You can use Model::on('system')

still insert into tenant database
hint : using in model UseTenantConnection

You can use Model::on('system')

still insert into tenant database
because i am using UseTenantConnection in model ??

I don't quite understand why you're doing that @Dev12230.  A model class should use the UsesSystemConnection or UsesTenantConnection trait.  The system connection is ALWAYS available, and the tenant connection is only available if a tenant has been identified.
This means that the two connections exist in parallel and you should have no problem using the system connection at any point assuming your models are configured correctly.

Closing this issue. If you are still having issues with the package, please create a new issue. Thank you


create shader doesnt work

hello, I'm trying to create a sign for fivem but when I try to create a shader I get this error and I don't know how to fix it. I use blender in version 3.4

This is a known bug. To fix this, you have to change your Blender language to English for now, sorry.

This is a known bug. To fix this, you have to change your Blender language to English for now, sorry.

Really? just change the language and it works?

Yes. The error occurs because the code expects there to be a node called "PrincipledBSDF", but if the language isn't English, it's called something else.
If for some reason switching the language doesn't fix your issue let me know and I will reopen the issue.


Add missing tests to build

/azp run

Codecov Report

Merging #1301 (9392c85) into master (98b46c2) will decrease coverage by 82.20%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #1301       +/-   ##
==========================================
- Coverage   83.39%   1.18%   -82.21%
==========================================
  Files         280     252       -28
  Lines       13588   12356     -1232
  Branches      689     689
==========================================
- Hits        11332     147    -11185
- Misses       2256   12209     +9953

Impacted Files
Coverage Δ

...m/src/main/scala/com/microsoft/lightgbm/SWIG.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...ain/scala/org/apache/spark/ml/param/MapParam.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...n/scala/org/apache/spark/ml/param/UDPyFParam.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...cala/org/apache/spark/ml/NamespaceInjections.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...cala/org/apache/spark/ml/param/DataTypeParam.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...ala/org/apache/spark/ml/param/ByteArrayParam.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...ala/org/apache/spark/ml/param/EvaluatorParam.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...la/org/apache/spark/ml/param/ParamSpaceParam.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...la/org/apache/spark/ml/param/TypedArrayParam.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

...a/com/microsoft/azure/synapse/ml/stages/udfs.scala
0.00% <0.00%> (-100.00%)
:arrow_down:

... and 262 more

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 98b46c2...9392c85. Read the comment docs.


Rebase silently failing

Hi,
We are trying to keep our fork of a repo up-to-date, while maintaining a branch with one small commit.
Our Pull config yaml can be found here.
It appears that the first two hardreset rules are being carried out, but the last two rebase rules are not. I can't find any way to introspect what is happening. Could it be to do with unusual characters in the base branch names?
Thanks.

Yes due to url encoding with # in the branch name when checking for changes. I'm currently using probot@v9 which uses<EMAIL_ADDRESS>This issue has been fixed in v17 as seen in https://runkit.com/whe/5f50d3cecb7d24001a3a6be4
I'll url encode for you for now.

Update I fixed it in 0271f31, and it can be seen working shadow-robot/moveit#9 and shadow-robot/moveit#10 🥳

Thanks @wei, incredible turn around time!


Remove view docs dropdown

I'll continue @ericholscher's work here
It looks like this

but we should return to this and potentially remove the link to the build list page when a project doesn't have a build

What would be the solution here? Removing the "No docs" button completely and only show "View docs" when there is docs to show?

but we should return to this and potentially remove the link to the build list page when a project doesn't have a build

I guess we can address that in the site redesign


Typeface app in Teams

Typeface app in Teams doesn't have supported capabilities in non-Teams products but the badge saying that it works across Teams, Outlook and Office. Seems like a bug and the badge should be removed

Thank you for reporting this issue, we will check this and get back to you.

We are checking this internally with engineering team, we will keep you posted on the updates. Thank you.

hello, I still see this issue.

Hello @shahsaachi - We have informed engineering team and let you know the updates if any.

Hello @shahsaachi - Sorry for delay in response.
This issue is fixed and we have checked it in Web browser:

Could you please verify and let us know?


JPEG Transcoding isn't lossless

Upstream for this issue https://github.com/T8RIN/ImageToolbox/issues/1363
JPEG transcoding should be bit-identical, but these additional EXIF tags are being added in ImageToolbox
[IFD0]          ImageWidth                      : 1280
[IFD0]          ImageHeight                     : 960
[IFD0]          Orientation                     : Unknown (0)
[ExifIFD]       LightSource                     : Unknown

Did you tried to do it in sample app without any additional layers? Like this one.
val jxl = JxlCoder.Convenience.construct(buffer5)
val reconstructed = JxlCoder.Convenience.reconstructJPEG(jxl)
Log.d("TAG", "Source size ${buffer5.size}  reconstructed size ${reconstructed.size}")

If it doesn't match you have to ask explanation at libjxl team, this library does nothing on JPEG transcoding only sending to libjxl and passing back, without any changes.
To me it never adds any metadata.
You should check at first if it is matches using library.
Your android OS may patch image on exporting from the app, anyway this is highly likely out of scope of this library.

And at may best understanding JPEG XL promising bit exact transcoding on quality not file size.
However it is highly unlikely that it will write any metadata also

libjxl decodes the file bitexact, the ending file, not just the frame, will have matching hashes if anything went right. I was able to replicate the issue using the toolbox app. I got matching file sizes but the hashes were different.
Using djxl on the "constructed" jxl does produce an image with matching hashes. I do not currently have time to make a demo program to try with but will upload a known bad image when I get a chance to. I verified with sha256sum.

If so then test code
fun sha256(input: ByteArray): String {
      val digest = MessageDigest.getInstance("SHA-256").digest(input)
      return digest.joinToString("") { "%02x".format(it) }
}

val jxl = JxlCoder.Convenience.construct(buffer5)
val reconstructed = JxlCoder.Convenience.reconstructJPEG(jxl)
Log.d("TAG", "Source size ${buffer5.size}  reconstructed size ${reconstructed.size}")
val originalHash = sha256(buffer5)
val reconstructedHash = sha256(reconstructed)
Log.d("TAG", "Original $originalHash")
Log.d("TAG", "Reconstructed $reconstructedHash")
Log.d("TAG", "Matches ${originalHash == reconstructedHash}")

Output
2024-09-24 19:54:26.037  4904-4929  TAG                     com.awxkee.jxlcoder                  D  Source size 395311  reconstructed size 395311
2024-09-24 19:54:26.042  4904-4929  TAG                     com.awxkee.jxlcoder                  D  Original 726b991eeecfb723f1bc0ac5c35b0d1176831586108c9f10aa2ba470b4f27efd
2024-09-24 19:54:26.042  4904-4929  TAG                     com.awxkee.jxlcoder                  D  Reconstructed 726b991eeecfb723f1bc0ac5c35b0d1176831586108c9f10aa2ba470b4f27efd
2024-09-24 19:54:26.043  4904-4929  TAG                     com.awxkee.jxlcoder                  D  Matches true

As I noted, lib itself does nothing, only calling libjxl apis.
Perhaps the problem might be in specific file sizes if stream size do not handled correctly I think this is not really likely however might be, issue in libjxl vesion for specific images, or on exporting layer to your external storage from the app that is preferred option I consider.

Or importing code, to an internal app storage

Or both :)

I think dev of Toolbox suspects something https://github.com/T8RIN/ImageToolbox/commit/d66dadf7473e3b34d0c18fa22488220f2b622c38

yup, just tested their CI and the issue is fixed for me

So, I think the issues looks like have been resolved :)
Closing for now.

Not sure why they didn't link it to my issue, but thanks for figuring it out you two


Using OrderItem.data in price calculation

Hi and thanks for the the work on this project, we're trying to adopt it as our base for doing ecommerce sites with Django.
We have an use case where we want to calculate the price of a product dynamically using properties passed by the user on the order form.
To accomplish this we tried to leverage the "data" parameter passed to the modify_item method of the Order class, but the method sets data on the item after the call to product.get_price
data is set here:
https://github.com/matthiask/plata/blob/next/plata/shop/models.py#L449
call to get_price is done here:
https://github.com/matthiask/plata/blob/next/plata/shop/models.py#L437
So in practice data passed as modify_item parameter is not usable to calculate the price, as it's set after the price calculation.
We resolved our use case by defining our Order model and swapping order of the two calls, but this probably something that should be ported to the base Order implementation.
Do you see any side effect with this?
Best regards
Mauro

Hi,
I probably had a similar issue, and decided to let get_price and get_prices return the base prices, and add upgrade prices only in Product.handle_order_item. In my case:
    def handle_order_item(self, orderitem):
        upgrades = set()
        for upgrade in orderitem.data.get('upgrades', []):
            orderitem._unit_price += upgrade['price_excl_tax']
            orderitem._unit_tax += upgrade['tax']
            upgrades.add(upgrade['id'])

        orderitem.name = unicode(self)
        orderitem.sku = self.generate_sku(upgrades)

(This is from a webshop selling computer hardware; it's possible to select more RAM, a faster SSD etc. when adding a product to the cart.)
Anyway, I don't think assigning item.data earlier would be a problem, but I'm not 100% sure! It's probably even better, because otherwise a get_price implementation looking at item.data would use stale data in the worst case.
Anyway. The general idea was that get_price cannot return the final price anyway, because the price might depend on discounts, might depend on the current user in case of educational discounts, might depend on other items in the cart etc.

Hi @matthiask ,
thanks for your quick response, I understand it better now (I am using the handle_order_item already for adding accessories that are not sellable alone).
In my case, I prefer to create different prices for my products based on their properties.
I am selling prints of different sizes and different materials, and creating a price instance for every combination I am interested in. (as the relation might be non linear or difficult to express by configuration)
My implementation of get_price looks like this:
def get_price(self, currency=None, orderitem=None):

        if currency is None:
            currency = (
                orderitem.currency if orderitem else
                plata.shop_instance().default_currency())

        try:
            return self.prices.get(
                currency=currency,
                size=orderitem.data.get('size', None),
                material=orderitem.data.get('material', None))

        except IndexError:
            raise self.prices.model.DoesNotExist

I swapped calls of price calculation and item data setting as I mentioned before in my Order model.
(And if this is not impacting in other places I'd change it in the default implementation, seems more consistent anyway. If you want i can make a PR for this)
Also, in my order form I only show combinations of size and material for which I have a price.
I think the current API is good enough as it allows multiple implementation, probably we should add some more complex examples like this or similar ones.
Thanks again, regards
Mauro

Mauro, it seems to me that you might want to take a closer look at https://github.com/matthiask/plata-options-product/blob/master/options_product/models.py

Hi Matthias, thanks for the link, It's very useful and probably worth ma.
I'll sure take some ideas from there.
If you agree I'd keep this issue open, as I think my original proposal of swapping the two methods calls is still worth investigating.

That's alright. Do you want to submit a pull request? I'll probably merge this change right away if Travis CI doesn't complain.


VS2019: Extremely high memory usage with v<IP_ADDRESS>

Please let me know if I do anything to provide more information about this bug. As it stands I don't know if I can e.g. force this extension to log anything for me.
Upgrading the GitExt VS2019 extension from v<IP_ADDRESS> (the newest build on GitHub) to v<IP_ADDRESS> (the newst build on VS Marketplace) seems to result in an extremely high memory usage of devenv.exe, circa 2GB for just the IDE loaded with no code! With a bigger solution loaded, devenv.exe easily crashes as it runs out of memory.
v<IP_ADDRESS>, IDE loaded with no code:

v<IP_ADDRESS>, IDE loaded with no code:

For all I know, it wasn't "always" like this as it seems to have worked fine for me until yesterday. To my knowledge I have not changed any settings, the last thing I did in VS before I noticed things started breaking was opening a CMake project via VS. Because I am not sure how to debug this, I do not know if this action had any relevance to the issue or not.
So far I observed this on VS16.11.2 and VS16.11.3.

Hey, I think it does log some unhandled exceptions to the Output window, do you have anything there? Example:

Not when I open VS with no code, sadly:

Hmm I tried opening the main GitExtensions repo on VS 16.11.3 with and without the 3.4.1 extension installed and it seemed roughly about the same memory consumption on my machine and didn't crash 🤷♂
I'm wondering if this is related to my NuGet update changes:

CommandBars update, https://github.com/gitextensions/gitextensions.vsix/pull/25
All other NuGet packages update, https://github.com/gitextensions/gitextensions.vsix/pull/26

I've made 3 test builds by reverting both of these changes and either one. If it wouldn't be too much trouble could you see if any of them fix the memory issue for you?
test-builds.zip

Hmm I tried opening the main GitExtensions repo on VS 16.11.3 with and without the 3.4.1 extension installed and it seemed roughly about the same memory consumption on my machine and didn't crash 🤷♂

That doesn't surprise me, as this version of the extension worked fine for me for a long time, and then I must have did something to trigger the issue. I just can't pinpoint what exactly, the fact I started noticing it after opening a CMake project via VS is just my rough guess.
I tested the builds:

revert-nugetupdates - no change, memory usage is at around 2GB
revert-commandbars-update - seems fixed, memory usage is back to normal!

revert-both - now this is weird - this build also results in high memory usage :/

Those results make no sense, so after revert-both I went back to the revert-commandbars-update build to check it again - 3 out of 3 attempts the commandbars build works fine, whether I load VS with no project or open an existing project using git, memory usage stays at around 300MB.
Is there some sort of plugin settings or cache I could move out/wipe to see if it helps? I tried looking for something like this but didn't find any, and I'd like to avoid resetting all my VS settings (a Repair + restore settings didn't help).

Hmm yes that result doesn't make sense to me, wondering if I got them mixed up somehow... Have done another set:
test-builds-2.zip
I don't think they get cached 😕 I found this article on the /ResetSkipPkgs devenv command line argument, but I think that's for stopping them from being "disabled" like you can see in the GUI.

In the meantime, I compiled the extension myself and attempted to reproduce the issue from an experimental environment - but as expected, it doesn't reproduce there.
Then, I reversed the debugging scheme, and right now I'm using the experimental environment to debug my regular environment - this way the issue reproduces, and given that at the time of writing this post a Memory snapshot collected over 7 million roots, something seems to be leaking references severely.

If this collection ever finishes, I'll hopefully be able to at least pinpoint what is the data that is leaking.

I finally managed to take a memory snapshot, here's the top results. Notice the presence of CommandBars there, which is in line with the revert fixing the issue! Now, if I manage to find what's the oustanding reference...

OK, last post in that tripleposting thread :)
I troubleshooted it further and seems like clearing out the ChangeCommandCaption function (the only one to rely on CommandBars) reliably mitigates this issue. Do you reckon this points at CommandBars having some issue out of gitext's control? If that's the case, I imagine it'd be hard to escalate, given I cannot even figure out the correct repro steps :(


Grammar error when defining arrays

The form: byte[] array = new byte[0]; works but the form:  byte array[] = new byte[0]; throws an exception with message Encountered "[" at line....
Attached is a simple test file.
Reported by: toomasr
Original Ticket: "beanshell/bugs/300":https://sourceforge.net/p/beanshell/bugs/300

https://github.com/beanshell/beanshell/blob/41352867a2698223d933b13a19d76f5d292cf3a6/src/test/java/bsh/SourceForgeIssuesTest.java#L195-L217
Closed: fixed


[Bug] The nvl function reports an error after the vectorization engine is enabled.

Search before asking

[X] I had searched in the issues and found no similar issues.

Version
doris dev1.1.2 branch ee83ab9
What's Wrong?
SQL: select nvl(null,0);
with error
[HY000][1105] errCode = 2, detailMessage = No matching function with signature: nvl(unknown type: NULL_TYPE, tinyint(4)).

What You Expected?
SQL execute succeed.
How to Reproduce?
No response
Anything Else?
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

the nvl is the oracle function name. You can use ifnull function in Doris.
And nvl will be as a alias of ifnull in next release version(1.2)


Zero tokens

Something went wrong during completion. Reason: You exceeded your current quota, please check your plan and billing details.
Got this message after deployment could please help me sir

Hey - I am getting this error as well.  :(

I am dumb.  Had to upgrade to paid and restart docker.  FIXED

How do upgrade to paid? I don't see anything about payment in repo. Btw, I need to integrate my own system for payment and dunno know where its checking

You need to upgrade your OpenAI account to paid one.
@lomeat write me on Telegram (@karfly)


[#354] [Compose] 1/2: Setup UI and Integration tests

https://github.com/nimblehq/android-templates/issues/354
What happened 👀

Add missing HomeViewModelTest in both Template & Sample projects, and clean up testImplementation dependencies.
Add initial HomeScreenTest (instrument test) in both Template & Sample projects, and add necessary androidTestImplementation dependencies & configurations.

Insight 📝

To add HomeViewModelTest, we need to create a CoroutineTestRule with UnconfinedTestDispatcher by default. We will switch to StandardTestDispatcher when needed.
To add HomeScreenTest (instrument test), we need to:

add androidx.compose.ui:ui-test-junit4 dependency 👉  As we switched to use BOM to import Jetpack Compose, we need to define androidTestImplementation(platform("androidx.compose:compose-bom:${Versions.COMPOSE_BOM_VERSION}")) to add androidTestImplementation dependencies https://github.com/gradle/gradle/issues/23347.

In the Sample project:

to grant necessary permissions in testing, we need to add androidx.test:rules and make use of GrantPermissionRule.
to mock UseCase result, we can add io.mockk:mockk-android and io.mockk:mockk-agent-android.

🚧 Known issues 🚧

MockK is having issues with UI instrument test mocking. Version 1.12.3 seems to be a stable version.

java.io.IOException: Unable to dlopen libmockkjvmtiagent.so: dlopen failed: library "libmockkjvmtiagent.so" not found with version 1.12.4 https://github.com/mockk/mockk/issues/297#issuecomment-1134270830.
java.lang.AbstractMethodError: abstract method "boolean kotlin.reflect.KClass.isValue()" with version 1.12.5 and later, even the latest 1.13.4 https://github.com/mockk/mockk/issues/826.

Proof Of Work 📹

Template UI test on the Home screen

https://user-images.githubusercontent.com/16315358/211962183-5edf2a30-e5b3-4c7d-9f39-fcb7d93714e4.mov

Sample UI Test  on the Home screen

https://user-images.githubusercontent.com/16315358/216777392-8ffca290-ab98-4906-89d1-e5c085d26625.mov

@Wadeewee All null checks in CoroutineTestRule has been resolved in https://github.com/nimblehq/android-templates/pull/384/commits/f4284bbf9940c39677e11e23f6252e4532a84f13, please have your final review 👋

@luongvo LGTM aside from @Wadeewee's comments and the failing CI. Not sure how to proceed with this, shall we merge anyway after the import comments or open a new issue to upgrade/downgrade sdkman first to see if it works? 👀 CC: @Tuubz

@ryan-conway I could be too optimistic, but I would wait until they fix it first (should be fast) 🚀

@ryan-conway I think we can skip the issue with verify_newproject_script.yml to speed up PR merging. There are only some small new things introduced since the last success at commit https://github.com/nimblehq/android-templates/pull/384/commits/c80193ac0804bab5a875096becc6b3e5d0adb46d 👌 .

FYI, only verify_newproject_script fails, the rest is passed. @Tuubz We don't know exactly when the sdkman API will be back 💭

@luongvo @ryan-conway SDKMan has apparently been fixed: https://github.com/sdkman/sdkman-cli/issues/1173#issuecomment-1429649116

@Tuubz I resolved your all comments 👨🎤


Update NCIT build

Add new branches from NCIT to cover all annotations found in BioModels
Add ProbOnto

@bgyori I just tested the rebuild and it's looking good. The errors in testing are related to cloudfront and not the improvements for this PR.

Looks good, thanks!


position_the_popover_based_on_a_prop

Bug: T274259
TODO:

[x] left-bottom, left-center, left-top classes
[x] unit tests
[x] a better way for class extensions

@micgro42 whenever you've time, can you look over this PR? I would appreciate some feedbacks on this.

@micgro42 thanks for the feedback.
I updated your change requests. However, I'm having some issues with the story template.
but is not a big deal. we can discuss it tomorrow.

Updated the all template

Not sure why exactly, but that last change broke the off-center positions

I'm noticing the vertical position issues too, plus the pointer looks off when the popover is placed on the left (Chrome/Mac):

Kinda related: Would it be possible to make available the position values in a dropdown in the Controls, instead of multiple radio buttons?

I'm noticing the vertical position issues too, plus the pointer looks off when the popover is placed on the left (Chrome/Mac):

Kinda related: Would it be possible to make available the position values in a dropdown in the Controls, instead of providing multiple radio buttons?

Thanks for the feedback. the dropdown is now added in the controls. we also worked on fixing the position of the pointer

Some places are still not using the correct token, otherwise it looks like things are coming together +1

Thanks for pointing them out.

@micgro42 Thanks for reviewing it.


Error detection when a just created record is being accessed

It'll be easier to show the result:

Compared to current behavior (silent failure): http://play.witheve.com/#gist:ec1ef78e3b2e54d5cdab32604069e236-examples-untitled.eve
Suggestions are appreciated to improve the text of error :)

This change is

Looks like PR is outdated with a new runtime :)


Build failure: Perl

Steps To Reproduce
Steps to reproduce the behavior:

sudo nixos-rebuild switch --upgrade

Build log
❯ sudo nixos-rebuild switch --upgrade
unpacking channels...
building Nix...
building the system configuration...
this derivation will be built:
  /nix/store/38l9y0ihzcykc7y4ahs14gsxm21a4678-nixos-system-wombat11k-23.11pre541036.63678e9f3d3a.drv
building '/nix/store/38l9y0ihzcykc7y4ahs14gsxm21a4678-nixos-system-wombat11k-23.11pre541036.63678e9f3d3a.drv'...
switch-to-configuration syntax is not valid:
Can't locate loadable object for module Fcntl in @INC (@INC contains: /nix/store/jdadgrryz805rf424bs0cdy7daqi33fy-perl-5.38.0-env/lib/perl5/site_perl/5.38.0/x86_64-linux-thread-multi /nix/store/jdadgrryz805rf424bs0cdy7daqi33fy-perl-5.38.0-env/lib/perl5/site_perl/5.38.0 /nix/store/jdadgrryz805rf424bs0cdy7daqi33fy-perl-5.38.0-env/lib/perl5/site_perl /nix/store/3nz3lxfn4yhcmgxrpbx8lbvk5d5bsmyx-perl-5.38.0/lib/perl5/site_perl/5.38.0/x86_64-linux-thread-multi /nix/store/3nz3lxfn4yhcmgxrpbx8lbvk5d5bsmyx-perl-5.38.0/lib/perl5/site_perl/5.38.0 /nix/store/3nz3lxfn4yhcmgxrpbx8lbvk5d5bsmyx-perl-5.38.0/lib/perl5/5.38.0/x86_64-linux-thread-multi /nix/store/3nz3lxfn4yhcmgxrpbx8lbvk5d5bsmyx-perl-5.38.0/lib/perl5/5.38.0) at /nix/store/jdadgrryz805rf424bs0cdy7daqi33fy-perl-5.38.0-env/lib/perl5/site_perl/5.38.0/Config/IniFiles.pm line 10.
Compilation failed in require at /nix/store/jdadgrryz805rf424bs0cdy7daqi33fy-perl-5.38.0-env/lib/perl5/site_perl/5.38.0/Config/IniFiles.pm line 10.
BEGIN failed--compilation aborted at /nix/store/jdadgrryz805rf424bs0cdy7daqi33fy-perl-5.38.0-env/lib/perl5/site_perl/5.38.0/Config/IniFiles.pm line 10.
Compilation failed in require at /nix/store/xfgc46yp5y36nrb4z17rqc0mbbfglvqf-nixos-system-wombat11k-23.11pre541036.63678e9f3d3a/bin/switch-to-configuration line 17.
BEGIN failed--compilation aborted at /nix/store/xfgc46yp5y36nrb4z17rqc0mbbfglvqf-nixos-system-wombat11k-23.11pre541036.63678e9f3d3a/bin/switch-to-configuration line 17.
error: builder for '/nix/store/38l9y0ihzcykc7y4ahs14gsxm21a4678-nixos-system-wombat11k-23.11pre541036.63678e9f3d3a.drv' failed with exit code 1

Additional context
Was working fine a few days ago. Haven't modified my NixOS configuration.
Notify maintainers
@edolstra
Metadata
❯ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 6.1.59, NixOS, 23.11 (Tapir), 23.11pre540331.8efd5d1e2836`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.1`
 - channels(root): `"nixos"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

switch-to-configuration syntax is not valid: is suspiscious. Try checking for corruption: nix-store --verify --check-contents.

Yep, looks like my nix store is corrupted.


fix: code block styling escapes parent

This fixes issue #41418
Add overflow-wrap to code styling
Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] All the files I changed are in the same world language, for example: only English changes, or only Chinese changes, etc.

Closes #41418

Thank you, for getting this PR out.
However, I have added another related issue. Would you mind adding it to this PR?
Link to comment: #41418 (comment)
@ahmadabdolsaheb Are you ok with the proposed change? It means code will wrap:

As opposed to the current:

It should scroll vertically instead of wrapping similar to https://www.freecodecamp.org/learn/scientific-computing-with-python/python-for-everybody/dictionaries-common-applications

It should scroll vertically instead of wrapping similar to https://www.freecodecamp.org/learn/scientific-computing-with-python/python-for-everybody/dictionaries-common-applications

@ahmadabdolsaheb I do not see it scrolling vertically? I see it scrolling horizontally.


Move docker annotation processor to std lib

Purpose

Describe the problems, issues, or needs driving this feature/fix and include links to related issues.

Fixes #
Approach

Describe how you are implementing the solutions along with the design details.

Samples

Provide high-level details about the samples related to this feature.

Remarks

List any other known issues, related PRs, TODO items, or any other notes related to the PR.

Check List

[ ] Read the Contributing Guide
[ ] Required Balo version update
[ ] Updated Change Log
[ ] Checked Tooling Support (#)
[ ] Added necessary tests

[ ] Unit Tests
[ ] Spec Conformance Tests
[ ] Integration Tests
[ ] Ballerina By Example Tests

[ ] Increased Test Coverage
[ ] Added necessary documentation

[ ] API documentation
[ ] Module documentation in Module.md files
[ ] Ballerina By Examples

Codecov Report

Merging #18703 into master will increase coverage by 0.53%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #18703      +/-   ##
==========================================
+ Coverage   15.04%   15.57%   +0.53%
==========================================
  Files          48       47       -1
  Lines        1263     1220      -43
  Branches      199      184      -15
==========================================
  Hits          190      190
+ Misses       1059     1016      -43
  Partials       14       14

Impacted Files
Coverage Δ

...s/vscode/src/project-tree-view/project-overview.ts
0% <0%> (ø)
:arrow_up:

...ool-plugins/vscode/src/debugger/config-provider.ts
0% <0%> (ø)
:arrow_up:

...ugins/vscode/src/project-tree-view/project-tree.ts

tool-plugins/vscode/src/trace-logs/traces.ts
21.05% <0%> (+2%)
:arrow_up:

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 2d69fda...d543cc7. Read the comment docs.


dir option not working for git_status

I know it's mentioned in the docs, but...why not?

It doesn't make sense to me. Why would you only want to see the changed files in a sub-directory?

Seeing changes in a sub directory is not the only use case for the option. I for instance currently use neogit and asked the same question there because sometimes I need to edit my neovim config while I'm working on my main project. So I open the config, edit it and when I open the neogit status, I see the status of my config repo because the current file belongs to that repo, not my project repo.
So what I'm asking is to choose a directory and for neo-tree choose the repo based on the given path: the repo at the directory itself or its closest ancestor repo.

Neo-tree will actually use the current working directory, so if you are using it as a sidebar or float, you can
:tcd the/dir/you/want

or
:cd the/dir/you/want

If you are using it with position=current, then it uses the window's working directory so
:lcd the/dir/you/want

would have the desired effect.
Of course, if you are using a plugin that automatically changes the working directory to match the file you are focused on, that's the entire source of your problems with both neo-tree and neogit. If you get rid of that, this would work the way you want it to without having to fix your working directory.

Fair enough, thank you for the explanation. Closing.


Operators

Allow the use of operators on quantities, for example:
D = Operator('Delta')
p = Quantity('p')
an it is printed accordingly

Issue closed for an indefinite period.


Questions

It seems this requires Keycloak. How does this integrate with existing Openstack users?
Which Openstack release this supports?
In youtube video, I assume this runs on top of k8s, while this repo has docker-compose only. I am wondering if this repo includes everything to test bumblebee? It seems the setup options are all in env-template file, am I missing anything else?

Hi @haiwu Thanks for getting in touch.
They way we run this service is that it is independent of OpenStack. Users have local accounts, which are created by OIDC (in our case, handled by Keycloak).
It does run on K8s, and we use Helm for that. The Helm repo is here https://github.com/NeCTAR-RC/bumblebee-helm
There are a couple of Pull Requests here that might help in evaluating whether this is useful to you or not. They were contributed by others who were also looking to evaluate Bumblebee. I'd have a look at those and I know that some of the code allows for using docker-compose to set it up (see PR#1) that does include Keycloak.
I haven't looked at running this code outside of our environment for a while, so there might be some rough edges. I'll be presenting about this at the Vancouver summit in June, so I'll be making sure it's in an easily testable state by then!
Hope this helps - and please reach out if I can help in any way.
cheers

Thanks @andybotting for fast response! I am thinking of first testing this by creating 2 VMs in the cloud, one with Devstack, and one to host Bumblebee with docker-compose (or maybe this could be a VM with k3d/kind installed, and use bumblebee-helm to install). Would that work to quickly test this?
Or maybe better to wait till June.

I'd really like it to be usable with docker-compose, so I'll try it today and I'll let you know.

@andybotting to integrate keycloak with bumblebee what OIDC_SERVER_URL value I need to set in env file, kyecloak server URL or any other ?

@andybotting could you please help us with the document related to keycloak for reference ?

@andybotting  is it possible to expose windows VM through bumblebee ?

@sanshah1211 We're actually working on that now. No ETA on when that work will be done yet though

@andybotting OK, Thank You


VM Activate media folder rename + lightbox code

#assign: @ManikaDhiman
@alkohli @ManikaDhiman

Can you review this PR?
IMPORTANT: When this content is ready to merge, you must add #sign-off in a comment or the approval may get overlooked.

NOTE @dansisson - the validation flags will need to be addressed before we can merge. thanks
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team

@dansisson @ManikaDhiman - after these changes are live, please confirm that the images are displaying as intended. And that lightbox is working.


Update buildah image version

The previous buildah image is a year old, and runs into SSL errors when the OCP version
was upgraded.

@wcheang @amisstea this new buildah image is not supported for ppc64le architecture.
docker run registry.redhat.io/rhel8/buildah@sha256:144bf3afc949e36a9b25c400c139a2d025d188c3892290c93dc16753dfa516fa
Unable to find image 'registry.redhat.io/rhel8/buildah@sha256:144bf3afc949e36a9b25c400c139a2d025d188c3892290c93dc16753dfa516fa' locally
registry.redhat.io/rhel8/buildah@sha256:144bf3afc949e36a9b25c400c139a2d025d188c3892290c93dc16753dfa516fa: Pulling from rhel8/buildah
Digest: sha256:144bf3afc949e36a9b25c400c139a2d025d188c3892290c93dc16753dfa516fa
Status: Downloaded newer image for registry.redhat.io/rhel8/buildah@sha256:144bf3afc949e36a9b25c400c139a2d025d188c3892290c93dc16753dfa516fa
WARNING: The requested image's platform (linux/amd64) does not match the detected host platform (linux/ppc64le) and no specific platform was requested
exec /bin/bash: exec format error

Do we have a multi-arch image version of buildah?

I have created a new PR to update the current buildah image with its multi-arch version. - https://github.com/redhat-openshift-ecosystem/operator-pipelines/pull/348


Changing Theme Breaks Data Transfer Wizard.

System information:

Operating system (distribution) and version
Windows 10 w/ Latest Patches
DBeaver version
Community Edition Version 7
Additional extensions
Darkest Dark Theme

Connection specification:

Database name and version
N/A
Driver name
N/A
Do you use tunnels or proxies (SSH, SOCKS, etc)?
N/A

Describe the problem you're observing:
Changing the Theme seems to break the Data Transfer Wizard used to export results to another table or file. The wizard still works but will only export to csv and will not let you select another file or format. Neither reseting the UI nor changing the theme back to default resolves the issue.
Steps to reproduce, if exist:
Change Theme, Click Data Export Toolbar button at bottom of results.
Include any warning/errors/backtraces from the logs
N/A

thanks for the bug report

will be resolved in the scope of #9223


JSC_DUPLICATE_OBJECT_KEY exception when running dart2js output through Google Closure Compiler

Hello Dart team :)
This is happening with Dart SDK Version 2.0.0 on MacOSX.
I am experiencing the exception below when attempting to run the compiled output through Closure Compiler at https://closure-compiler.appspot.com/home:
JSC_DUPLICATE_OBJECT_KEY: Object literal contains illegal duplicate key "gam", disallowed in strict mode at line 1014 character 0
gam:function(){return this}},
^

Doing a search in the compiled output shows the duplicate gam keys:
gam:function(){return this},
$isag:1,
gam:function(){return this}},

Any ideas on how to proceed with this?
I'm testing this out as I remember @sethladd stating in a past video suggesting you could improve the output by running through the closure compiler. Is this still the case? Thank you so much
Im blogging on Dart at DEV.to

I think any recommendation to use Closure Compiler with Dart2JS is very outdated. Dart2JS already has implementation details similar to JSC (tree-shaking, optimizations) with specifics to Dart. As such, I can't imagine anyone will work on making JSC compatible with Dart2JS outputs.


no content-type header in response

Describe the bug
Encountering an odd error message.
There is a valid git clone URL  but cargo-generate chokes on it.
Stating

[x] I'm on the latest rust version rustup update
[x] I'm on the latest cargo-generate version cargo install cargo-generate
[ ] .gitconfig  (not applicable)

To Reproduce
Steps to reproduce the behavior:
export GIT_CLONE_URL=http://stappers.it/git/cargo-generate
cargo generate --git ${GIT_CLONE_URL} --name foo --verbose

Verbose version, shows that the URL is correct and shows the odd error:
stappers@juli:/tmp
$ export GIT_CLONE_URL=http://stappers.it/git/cargo-generate
stappers@juli:/tmp
$ echo ${GIT_CLONE_URL}
http://stappers.it/git/cargo-generate
stappers@juli:/tmp
$ git clone ${GIT_CLONE_URL}
Cloning into 'cargo-generate'...
Fetching objects: 4065, done.
stappers@juli:/tmp
$ rm -rf cargo-generate/
stappers@juli:/tmp
$ cargo generate --git ${GIT_CLONE_URL} --name foo --verbose
Error: Please check if the Git user / repository exists.

Caused by:
    no content-type header in response; class=Http (34)
stappers@juli:/tmp
$ cargo generate --version
cargo generate-generate 0.18.3
stappers@juli:/tmp
$

Expected behavior
A succesfull cargo generate action like
stappers@rosa:/tmp
$ cargo generate  --git /srv/git/cargo-generate --name foo --verbose
? 🤷   Which template should be expanded? ›
❯ example-templates/hooks
  example-templates/liquid-rhai-bridge
  example-templates/placeholders

Additional context
Is it a HTTP thingy?

Cargo generate as same as cargo uses libgit2 for cloning.
There I found the following comment

The Accept header field here is application/x-git-upload-pack-result which is what you get when the server sends the packfile. However, glitch's implementation is sending Content-Type: application/x-git-upload-pack-advertisement which is neither what we asked for nor what it's delivering.

I'm not entirely sure if that information is still accurate, but your particular git server does not send the mentioned http header back, only this:
22:55:26.200489 http.c:664              == Info: Connected to stappers.it (2a01:7c8:aaae:103::1) port 80 (#2)
22:55:26.200566 http.c:611              => Send header,<PHONE_NUMBER> bytes (0x000000c2)
22:55:26.200572 http.c:623              => Send header: GET /git/cargo-generate/objects/info/packs HTTP/1.1
22:55:26.200576 http.c:623              => Send header: Host: stappers.it
22:55:26.200578 http.c:623              => Send header: User-Agent: git/2.34.1
22:55:26.200581 http.c:623              => Send header: Accept: */*
22:55:26.200583 http.c:623              => Send header: Accept-Encoding: deflate, gzip
22:55:26.200586 http.c:623              => Send header: Accept-Language: en-GB, *;q=0.9
22:55:26.200588 http.c:623              => Send header: Pragma: no-cache
22:55:26.200590 http.c:623              => Send header:

(see GIT_CURL_VERBOSE=1 git clone http://stappers.it/git/cargo-generate foo)
So I'd say this is an issue of libgit2 and I bet they would appreciate your issue report there, maybe it's a good idea to refer to their closed issue https://github.com/libgit2/libgit2/issues/4844 as well.

Acknowledge

Reported upstream
This issue retitled with text from the underlaying library
Taking label waiting-for-user-input as "report back when you get information from upstream"

Taking label waiting-for-user-input as "report back when you get information from upstream"

Edward Thomson, from  the libgit2 project, asked for elaboration on the issue.
Such information has been provided.

Additional context
Is it a HTTP thingy?

Yes

https://www.git-scm.com/docs/http-protocol

Additional context
Is it a HTTP thingy?

Yes


BUG:Remove Vite logo from the title and add KaamDo logo

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
The title of the website contains VITE logo which should be removed and replaced with the KaamDo logo.
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@PatilHarshh please assign this issue to me .


Add tests for KvBackend trait implements

What type of enhancement is this?
Tech debt reduction
What does the enhancement do?
Add behavior tests for KvBackend implements to ensure all implements have the same behaviors.

@WenyXu Can I try it? It a good chance to learn about KvBackend.


Support restricting access to certain clusters

For the CERN use case of running many different types of workload in a large federation of clusters, restricting access to certain classes of cluster does a few things for us (and lets us use a very simple placement to schedule everywhere else).

Mainly it lets us reserve clusters with particular hardware for only workloads that require that hardware, i.e GPU clusters or for clusters with especially large nodes.
In our own cloud it would also let us restrict access to any clusters we create in our critical power area.

Point number 1 would be possible with separate federations (but would probably lead to too much fragmentation), and for point number 2 deployments should be in both standard and critical power, so separate federations would require managing multiple copies of the deployment.

It would also be one way to designate if a job is important enough to send to an external cloud if our own is at capacity. Some jobs would be fine left in pending until there is room for them.

In the November 21st federation-v2 meeting, taints/tolerations were suggested as the best way to support this. I have started on an implementation for this in #515.
If there is a simpler way to do this then I'm all ears, because the full feature set of taints/tolerations is actually a bit overkill for this. Since these things (hardware/location) are immutable properties of the cluster, it is unlikely that we would need the NoExecute taint, and PreferNoSchedule is possibly covered by cluster weightings in placement. Encoding in the name (i.e starts with "+") that a label is required in the placement to schedule on that cluster would be one possible solution, but I assume that diverging from standard kubernetes behaviour is to be avoided at all costs.
When I say restrict I realise that this is not enforceable (yet) but it is just to mean that the particular clusters are opt-in to schedule on.

I apologize for taking so long to respond to this, I've been away.  I appreciate your taking the time to document your use case, and I think I have a better grasp of your motivation for #515.

I think it unfortunate that kubectl taint is restricted to targeting nodes, since the logic would seem to readily apply to a FederatedCluster that has the same Taints field.

I think you're correct that the semantics of label selection should remain unchanged from Kubernetes.

Assuming a goal of opt-in placement, I think you're on the right path.

#526 seems to be describing a related use case, in which access to member clusters needs to segmented by user.  I'm curious if the namespace placement change described in a comment on that issue might be helpful to your use case.


Mystical Aggriculture essence recepies spam make infusion crystals with 1000 uses, only uses them once before making new ones.

Issue description:
Combining essence creation from mystical aggriculture, which requires a crystal to transform between tiers, means that even though the crystal has 1000 uses, the storage system will create one for each, leaving each on 999 uses, whilst tying to make more
Steps to reproduce:

Using a refined storage system, create a recipie for an infusion crystal
Make recepies for the essences
Try to make 10 essences of tier 2. The storage system will want to create 10 crystals, even though it would only need 1 before for each tier.
...

Version (make sure you are on the latest version before reporting):

Minecraft: 1.12.2
Forge: 14.23.5
Refined Storage: 1.6.9

Does this issue occur on a server? [yes/no]
No

Intended

Why is this intended?


Can't see or run any tests if one of them has an error in the code.

Describe the Bug
I'm newish with VSCode and honestly hate this IDE.
Aside from that I'm new to Angular Testing.
I found your plugin and have spent countless days trying to figure out how to get it to work.
What I had finally discovered was that the projects existing test code that I'm to run has errors in the tests.

A clear and concise description of what the bug is.
see above

Steps to Reproduce the Behavior
Create some tests and then create some tests that do not properly create the services that are needed for the components.
I'd give an example but honestly I don't really know how to do these tests yet. I was to use the project as a learning stage but the project is riddled with issues in the code as well as the tests.

Go to '...'
Click on '....'
Scroll down to '....'
See error

Describe the Expected Behavior

A clear and concise description of what you expected to happen.

Provide Debug Logs

With the karmaTestExplorer.logLevel set to debug, please capture and attach the logs from the Karma Test Explorer Output chanel when the issue occurs.
Please also capture and attach the logs from the Karma Server Output channel when the issue occurs.
I do not know what this setting is or where it is that I would set this value?
If you can provide details on this I may be able to update the ticket with this information.

Screenshots

If applicable, add screenshots to help explain the issue you are having.

Please Provide the Following Information

OS: (e.g. macOS)

Linux

Environment: (e.g. Dev Container with Guest OS and version)

CENTOS8

Node Version: (e.g. 14.x)

14.x

VS Code Version:

1.69.1

Karma Version:

6.3.16

Frameworks: (e.g. Angular, Jasmine, Mocha)

Angular, Jasmine

Additional Context

Add any other context about the problem here.

Only after on a command line running ng test did I see where errors existed in the test files. This extension should identify all *.spec.ts files and also specify if nothing was found in them, tests and pending whereas nothing would be either nothing or errors in those files. Having this extension NOT load at all if some random test buried in the application has an issue is not helpful.

The extension itself is not a test framework, and relies on Angular (ng-test) for discovering tests in the project. In this case, ng test is failing to discover the tests due to the errors in the test files, and so the extension cannot display them either.


feat: allow setting ttl per item in setBatch

Addresses https://github.com/momentohq/client-sdk-javascript/issues/1250
Aimed to make this addition backwards compatible but open to suggestions for a different API design!

i pushed a commit to make the ttl optional on Anita's new type.


read_diann not processing

Hi I installed the package inside my miniconda and I am working in the (ms proteomics) environment within the cd of my report.tsv from DIA-NN.
However
(msproteomics) (msproteomics) PS D:\Data\Dominik\27102024DIANNH1299secondtry\reseponlyfasta> read_diann -o reportmsproteomics.tsv -E Fragment.Quant.Raw reportmsproteomics.tsv -f human.fasta report.tsv
is not starting the processing the desired input table for sitereport. It is just showing me this:
Activate siteloc (use -h for help) ...
and not showing process nor producing the output table
thanks for your input!

Hi, it seems that your command has the same input and output. Can you change the output filename (for example, -o reportmsproteomics_output.tsv)? I assume that your diann report is reportmsproteomics.tsv.
Thang

Hi Thang,
I pasted the command wrongly.
(msproteomics) (msproteomics) PS D:\Data\Dominik\27102024DIANNH1299secondtry\reseponlyfasta> read_diann -o reportmsproteomics.tsv -E Fragment.Quant.Raw -f human.fasta report.tsv
Report.tsv is my diann report.
However the command does not do anything

Update
After installing many additional packages and environments on this windows system it worked
1)Is the code and response listed below correct and fine?
2)Are the maxlfq values in the txt output files for site and phosphopeptide log2 transformed?
3)What do the sites and allsites column in the txt mean? What is M1/M2 etc?
(msproteomics) C:\Users\domin>cd C:\Users\domin\Downloads\msproteomics-1.0.1\msproteomics-1.0.1
(msproteomics) C:\Users\domin\Downloads\msproteomics-1.0.1\msproteomics-1.0.1>read_diann -o reportmsproteomics.tsv -E Fragment.Quant.Raw -f human.fasta report.tsv
Activate siteloc (use -h for help) ...
Loading FASTA files:

Processing human.fasta
Index of J not found in char_tree_getindex.
Failed determining child index in char_tree_enter.
Failed processing a peptide in store_fasta_into_peptide_tree.
Inserted 20467 proteins into peptide tree.
Failed storing FASTA file in peptide tree.

Failed loading FASTA files.
Processing input file report.tsv.
Skipped 10 lines with peptide backbone sequences not found in FASTA files.
Wrote output to file reportmsproteomics.tsv.
(msproteomics) C:\Users\domin\Downloads\msproteomics-1.0.1\msproteomics-1.0.1>sitereport reportmsproteomics.tsv
General setting:
input data        =  reportmsproteomics.tsv
processing tool   =  generic
sample_id_col     =  Run
intensity_col     =  Fragment.Intensity
secondary_id_cols =  ['Precursor.Id', 'Fragment.Rel.Id']
annotation_cols   =  ['Fasta.Files']
normalize         =  none
quant_method      =  maxlfq
Phosphosite setting:
protein_id_col               =  Protein.Ids
site_id_col                  =  Phospho.Site.Specs
site_filter_double_less      =  [['Global.Q.Value', '0.01']]
site_filter_double_greater   =  [['PTM.Site.Confidence', '0.01']]
site_filter_string_equal     =  None
site_filter_string_not_equal =  None
output_site                  =  phosphosite-report.txt
Phosphopeptide setting:
modified_sequence_col           =  Modified.Sequence
regex_str                       =  (UniMod:[0-9]*)
target_modification             =  (UniMod:21)
peptide_filter_double_less      =  [['Global.Q.Value', '0.01']]
peptide_filter_string_not_equal =  None
output_peptide                  =  phosphopeptide-report.txt
Loading data file: reportmsproteomics.tsv
625219 rows x 10 columns read
587458 rows after filtering out <= 0 intensities.
418042 rows after other filtering(s)
Creating site identifiers
Creating site report
Concatenating secondary ids...
574816 rows after peptide filtering
Creating peptide identifiers
Creating peptide report
Concatenating secondary ids...

It looks ok.
The first step, the siteloc program is quite verbal. I normally do not get so many warnings, like 'Index of J not found in char_tree_getindex.'.

why is this happening? I am using a regular human swissprot fasta with contaminants added?
also:
2) How do I adjust the site confidence cutoff? For DIANN I wanna replicate their 90 and 99 site table but with 75 and 51 and 1% values instead?
2)Are the maxlfq values in the txt output files for site and phosphopeptide log2 transformed? I think so based on the supplement of your biorxv publication but I need confirmation.
3)What do the sites and allsites column in the txt mean? What is M1/M2 etc? -> This I have now identified as multiplicity. Is there anyway to collapse this data back to without multiplicity like DIA-NN or PTMprophet does?
4)Why are peptides in the phosphopeptide report included that have a STY_count of 0? Why are some as 0.0 and some as 0.1 labeled?


google_new_transError: 404 (Not Found) from TTS API. Probable cause: Unknown

I'm trying to translate an epub into spanish using the following command:
python epub-translator.py -l es
But the following error is thrown:
google_trans_new.google_trans_new.google_new_transError: 404 (Not Found) from TTS API. Probable cause: Unknown
I'm using python 3.9 on a conda environment. The requirements listed have been installed.

It's a bug from google_trans_new #57. I've fixed it in v1.0.1. Sorry for the late response.


Fix the pull request labeler

It's failing the check for all dependabot PRs

From this article, we see that Dependabot PR's are treated as forks with read-only permissions, so "any workflows that attempt to write to the repository to fail," including the labeler. To fix this, we just added the permissions mentioned in this comment on another issue.


feat: migrate fact_*_engagement reports

Description
This PR migrates the engagement datasets from Aspects to DBT and creates a new model to support the individual learner report dashboard.
The old fact_*_engagement models have been renamed to fact_*_engagement_per_subsection as these names match what their real purpose is. The old name is used to match what Aspects expects from the dataset.

Thanks for the pull request, @Ian2012! Please note that it may take us up to several weeks or months to complete a review and merge your PR.
Feel free to add as much of the following information to the ticket as you can:

supporting documentation
Open edX discussion forum threads
timeline information ("this must be merged by XX date", and why that is)
partner information ("this is a course on edx.org")
any other information that can help Product understand the context for the PR

All technical communication about the code itself will be done via the GitHub pull request interface. As a reminder, our process documentation is here.
This is currently a draft pull request. When it is ready for our review and all tests are green, click "Ready for Review", or remove "WIP" from the title, as appropriate.

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


On-Prem build agent not updating tfx version

I was trying to publish a new task to my local org account and got an error back:
2019-02-11T16:17:37.3840977Z error: Failed Request: Internal Server Error(500) - The stated category 'Build and release' is no longer supported. Refer to the categories listed at https://aka.ms/extensioncategories and republish using one or more amongst them. 0 [ 'error: Failed Request: Internal Server Error(500) - The stated category \'Build and release\' is no longer supported. Refer to the categories listed at https://aka.ms/extensioncategories and republish using one or more amongst them.',
After updating the category to Azure Pipelines, the build failed with:
2019-02-11T16:20:24.7232096Z error: The following categories are not valid: Azure Pipelines. Valid categories are: Build and release, Collaborate, Code, Test, Plan and track, Insights, Integrate, Developer samples.
I checked the tfx version being used and it was 0.5.10. I removed tfx from the local tools folder and ran the build again. This time it installed v 0.6.4 of tfx. It seems the task is not properly updating to the latest version, if a previous version exists.

That's correct it tries to detect a locally installed version and uses that.
You can use npm update<EMAIL_ADDRESS>in a prior task to force it to stay updated.
I've considered updating automatically, but that would be a breaking change to the way the tasks currently works.
I've wanted to create a tool installer instead where you can specify which version strand to use.

Have you considered adding an optional setting to keep the tool up to date?

I did. I prefer the tool installer, as people tend to use more than one task from this suite in their pipeline and that way this can all be handled in one go. Simply using the npm task to install the tool somewhere and then configuring the toolpath also works.


Surprise! Tyler still has issues with order

I know, I know, I've already talked a ton about this in cloudfour/drizzle-builder#31. But it's still not working the way I'd like it to, and I thought I'd document that (even if it has to wait for a future version to be resolved).
The Problem
Here's a hypothetical collection of patterns:
- disabled.hbs
- group.hbs
- overview.hbs
- style-variations.hbs

By default, these will show up in the same order as above... whatever the file system determines "alphabetical" means for these files.
Let's say I want "Overview" to be the first pattern. Currently, that's really easy. I just add the property to my Frontmatter:
---
order: 1
---
{{! etc. }}

And the order changes:
1 overview.hbs
- disabled.hbs
- group.hbs
- style-variations.hbs

Simple.
But what if I want a pattern to sort to the end?
For example, maybe the group pattern should go after all of the patterns pertaining to individual instances.
Currently, that would require that I specify an order for every file:
1 overview.hbs
2 disabled.hbs
3 style-variations.hbs
4 group.hbs

Alternatively you could give everything after overview the same order value. But you'd still have to specify for every single pattern, as any pattern lacking an order property will show up after everything else.
My Recommendation
This use-case is why I recommended (and prototyped) that we mimic the CSS order property in structure. Specifically its initial value of 0.
Having an initial value means negative numbers show up earlier than the default, positive numbers show up later.
This means you'd only need to specify the order property for the patterns you wanted to move, regardless of which direction they moved...
-1 overview.hbs
 - disabled.hbs
 - style-variations.hbs
 1 group.hbs

It also seems intuitive given our target audience to make our order property behave in a manner similar to an order property they're familiar with.

@tylersticka I'm fine with using negative values, but I'd like to consider a few things:
Because we now have helpers that can sort by any property, we might want to focus on creating a common sorting function that possesses the logic to infer default values (e.g. 0) and that isn't limited to just properties named "order".

Also, here's the library used by the helpers: https://github.com/bubkoo/natsort


Question - Implementation of DKIM

Hi
I'm using milter with Postfix, now i'm trying to implement DKIM using this library https://github.com/toorop/go-dkim.
My question is where I should put dkim.Sign method()? Inside Body(), BodyChunk(), Headers ()....?
Seams Dkim needs to get the full message, sign it and replace the message with a signed on.
Best regards

Yes, DKIM signing needs the whole message so it should be done as last step just before the message is send.
If you want to do it in your milter then the Body() is the right place to do it. And make it the last thing you do in your milter.
But it may still fail :(
Im using opendkim for DKIM signing and its the last milter in the filter chain in my postfix.

Body() only includes the body part, not the headers.
Is not correct?

If i have understood correctly Body() doesnt include anything. Its just called when all the body parts are read.
So you need to create the message your self by collecting all headers + body parts and this is why adding
DKIM may fail because you have to get everything just right to add the signature.
Check this for example:
https://github.com/phalaaxx/pf-milters/blob/master/extmilter/main.go


first item open=true

I have a code that generate a new panels.panel inside a #each helper and i want to make only first panel with open=true. There's a way to do this?

{{each}} yield an index as the second param, so you could do this:
{{#each items as |item index|}}
  {{#cp-panel open=(eq (index 0)) as |panel|}}
    ...
  {{/cp-panel}}
{{/each}}

Let me know if you have any follow-up questions!

I already knew the index as second param, but i tested this code and return me an error: "eq is not a helper". I didn't understand

Woops worry - try installing the ember-truth-helpers addon
On Tue, Oct 2, 2018 at 8:49 AM GUILHERME GUIMARAES GALLI <
<EMAIL_ADDRESS>wrote:

I already knew the index as second param, but i tested this code and
return me an error: "eq is not a helper". I didn't understand
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/embermap/ember-collapsible-panel/issues/55#issuecomment-426260800,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ACyXCpkBOmaNvMx-QpMRp_r-v2EgRZoRks5ug2DlgaJpZM4XCuNX
.

Ok, now i'm getting the following error: "index is not a helper". My code is like this:
{{#cp-panels accordion=true as |panels|}}

Serviços

Total

                 {{#each model.Services as |service index|}}                     {{#panels.panel open=(eq (index 0)) as |panel|}}                         {{#panel.toggle}}
❯  {{t                              service.ServiceId}}

{{format-number service.Totals.OK}}
                         {{/panel.toggle}}                         {{#panel.body}}                             {{#each service.Actions as |action|}}

{{t action.ActionId}}

{{format-number action.Totals.OK}}

                             {{/each}}                         {{/panel.body}}                     {{/panels.panel}}                 {{/each}}         {{/cp-panels}}

Ok, now i'm getting the following error: "index is not a helper". My code is like this:
{{#cp-panels accordion=true as |panels|}}

Serviços

Total

                {{#each model.Services as |service index|}}
                    {{#panels.panel open=(eq (index 0)) as |panel|}}
                        {{#panel.toggle}}

❯  {{t service.ServiceId}}

{{format-number service.Totals.OK}}

                        {{/panel.toggle}}
                        {{#panel.body}}
                            {{#each service.Actions as |action|}}

{{t action.ActionId}}

{{format-number action.Totals.OK}}

                            {{/each}}
                        {{/panel.body}}
                    {{/panels.panel}}
                {{/each}}
        {{/cp-panels}}

Try (eq index 0) (no extra parens)

It worked! Thanks for your time Sam.


leight -> light

Same change needs to be applied to the repo description

@beneiltis Make sure to fix it in the repo description as well :)


How are resizing http://techanjs.org ?

Hi,
Quick Question - how are you resizing this page: http://techanjs.org/
Is that code resizing code snippet part of your library?
Thanks,
Vishal

Resizing is not part of the library, as it really depends on how the fundamental d3 chart has been constructed. I've tried to keep techanjs from forcing any particular layout style on a user and there's probably too many user implementation differences to be ever part of the library.
In short the steps are:

Catch a window/element resize
Redraw using standard d3 select & update techniques

  window.onresize = function() {
    d3.select('div#chart').call(resize);
  };

  function resize(selection) {
    // Recalculate dimensions
    // Adjust scales based on new dimensions
    // Perform any other logic, change display tick count, etc
    // Redraw everything
  }

@andredumas , show your resize window code please

Greetings,
I have just gotten your Techan.js installed and am also wondering if I can see you code for resizing the plots as I would like to add it to some of your examples. In particular the examples/plot/plots example.
Can you please share this javascript code and implementation?
Thanks,
Lonnie

+1


fix: Use functional state update

Summary
Update several state updates to use a functional update where the updated value depends on the previous value
Related Issues or PRs
closes #844
How To Test
Component behavior should remain unchanged by this PR.
Jest tests should still pass.
Storybook functionality (e.g. Expanding/Collapsing GovBanner or Password Reset Form, toggle showPassword) should still behave as expected

Warnings

:warning:
This PR does not include changes to tests, even though it affects source code.

  Generated by :no_entry_sign: dangerJS against 614dde528a78fab1b3be3da42abd10ce64d15182


Update WCSS peerDependencies

Alaska Airlines Pull Request
Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes: #86
Summary:
Updating WCSS peerDependencies to the same version used in devDependencies
Type of change:
Please delete options that are not relevant.

[ ] New capability
[x] Revision of an existing capability
[ ] Infrastructure change (automation, etc.)
[ ] Other (please elaborate)

Checklist:

[x] My update follows the CONTRIBUTING guidelines of this project
[x] I have performed a self-review of my own update

By submitting this Pull Request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
Pull Requests will be evaluated by their quality of update and whether it is consistent with the goals and values of this project. Any submission is to be considered a conversation between the submitter and the maintainers of this project and may require changes to your submission.
Thank you for your submission!
-- Auro Design System Team

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Failing to archive via xcodebuild when inside an iOS Framework when building for an XCFramework

Warning
It's possible this issue doesn't describe a valid use case for this package, having looked at this for a wee while I could have tunnel vision on this! Especially as no one else has raised this.

Expected behavior
I have an iOS framework target that depends on swift-nio via SPM. When I try to archive said target with the following incantation
 xcodebuild archive \
-scheme "FooiOSFramework" \
-destination "generic/platform=iOS" \
-archivePath FooiOSFramework \
SKIP_INSTALL=NO \
BUILD_LIBRARY_FOR_DISTRIBUTION=YES

I'd expect it to compile swift-nio and it's pals without compile error. Note that I can archive this same scheme via Xcode directly, and swift-nio builds just fine when I use it as a package dependency in an iOS app - although in that case it's being pulled in via Vapor.
Actual behavior
It starts to compile but alas I hit this (note I've trimmed the front of these paths to not include my machine specific stuff)
The following build commands failed:
	SwiftCompile normal arm64 Compiling\ PriorityQueue.swift checkouts/swift-nio/Sources/_NIODataStructures/PriorityQueue.swift (in target '_NIODataStructures' from project 'swift-nio')
	CompileSwift normal arm64 checkouts/swift-nio/Sources/_NIODataStructures/PriorityQueue.swift (in target '_NIODataStructures' from project 'swift-nio')
(2 failures)

scrolling up the archive log I see:
/checkouts/swift-nio/Sources/_NIODataStructures/PriorityQueue.swift:21:20: error: 'self' used before 'self.init' call or assignment to 'self'
        self._heap = Heap()
                   ^
/checkouts/swift-nio/Sources/_NIODataStructures/PriorityQueue.swift:22:5: error: 'self.init' isn't called on all paths before returning from initializer
    }
    ^
/checkouts/swift-nio/Sources/_NIODataStructures/PriorityQueue.swift:76:25: error: 'self' used before 'self.init' call or assignment to 'self'
            self._queue = queue
                        ^
/checkouts/swift-nio/Sources/_NIODataStructures/PriorityQueue.swift:77:9: error: 'self.init' isn't called on all paths before returning from initializer
        }

As I say, very strange as this archives fine from Xcode directly - however I don't think I can make the archive slices I need to make an XCFramework from Xcode. For context, as I need both the generic iOS device and generic iOS simulator, and I can't figure out how to archive the latter from Xcode.
Steps to reproduce
I've provided my sample framework. It just links swift-nio and imports it into a dummy file for now.

Grab the sample
Inside the sample directory try and archive

xcodebuild archive \
-scheme "ArchiveExample" \
-destination "generic/platform=iOS" \
-archivePath Example \
SKIP_INSTALL=NO \
BUILD_LIBRARY_FOR_DISTRIBUTION=YES

It should fail to compile.

If possible, minimal yet complete reproducer code (or URL to code)
ArchiveExample.zip
SwiftNIO version/commit hash
2.42.0
System & version information
I can't really run nit-diagnose as this pertains to compilation but I'll try and grab this manually

MacOS Version: 12.6
MacOS Arch: m1
Xcode Version: 14.0 (14A309), note I saw the same behaviour in the latest Xcode 13 as well.
Swift Version: 5.7 (although I've tried 5.6 as well with no luck)
swift-nio version: the latest at time of writing 2.42.0. I've tried dialing this back a versions (to 2.40.0 but I'm still seeing issues).
Min Deployment target of the framework that includes swift-nio: iOS 15.0

SwiftNIO doesn't define a stable ABI, so it doesn't promise that it will compile cleanly with BUILD_LIBRARY_FOR_DISTRIBUTION=YES. To use SwiftNIO embedded within an ABI-stable framework, you'd need to hide SwiftNIO from your ABI by importing it as @_implementationOnly. Doing so should allow you to compile NIO without setting BUILD_LIBRARY_FOR_DISTRIBUTION=YES, thereby avoiding the errors.

Ah wonderful thanks :) , closing this.

@empowerMax could you share how you fixed the issue? I am facing the same issue

@Hugo-Persson ah we changed tack in the end. For context I was using Vapor as a mock server for our UI test suite. We changed approach such that we didn't need to depend on Vapor.
As I wasn't pulling SwiftNIO in directly, I couldn't change how it was imported without forking and editing the consuming package that I was importing (Vapor).

Okay @empowerMax I am using BSON library which uses Nio. Which puts me in the same situation :/


Add: auto update to overdue

set invoice status to overdue after credit period

[x] add schedule
[x] update various overdue implementation

Closed via bd23cffe399d52dbdeb207c899625f59225fddb4


[Introspection] dot (and other non-[a-z0-9] characters) in field name

e.g. Vanilla schema

Let's define some of the terms a little bit more:
Invalid character -> everything that does not match the regex _a-zA-Z0-9
Invalid at start of identifier -> everything that does not match the regex a-zA-Z
Replacement strategy:
Everything invalid before the first character valid for start of the the identifier gets dropped.
Everything invalid after the first character valid for the start of the identifier gets replaced with _.

fixed via #1139


feat: umd build.

Adds a UMD build primarily for CDN consumption.
Uses React.createElement instead of react/jsx-runtime for UMD.

Codecov Report
Base: 99.28% // Head: 99.28% // No change to project coverage :thumbsup:

Coverage data is based on head (d54f3bb) compared to base (ac90e29).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #36   +/-   ##
=======================================
  Coverage   99.28%   99.28%
=======================================
  Files           7        7
  Lines         561      561
  Branches      149      149
=======================================
  Hits          557      557
  Misses          4        4

Impacted Files
Coverage Δ

src/component.tsx
100.00% <ø> (ø)

src/control.tsx
100.00% <ø> (ø)

src/highlighter.tsx
100.00% <ø> (ø)

src/hook.tsx
98.52% <ø> (ø)

src/icons.ts
100.00% <ø> (ø)

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

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


Simplified Alternate Image Handling

The image service will (soon) automatically suppot fallback images. We can simplify our web app with standard labels.
New API (in production soon)

GET image.webp?fallback=label1,label2&poster_fallback=1  → returns label1, label2 or standard thumbnail

So that would make the search sequence:

aspect ratio specfic - card_roku_9x13
label  -card
standard poster

API call

GET card_9x13?fallback=card_9x13,card &poster_fallback=1

AC

Specific labels for

Cards
Media background
Channel logo

Support aspect ratio

Format x
Support :  --> automatically transform to x

Webp format is used

Related tickets

https://github.com/jwplayer/ott-web-app/issues/114
https://github.com/jwplayer/ott-web-app/issues/152

Delivered some time ago.


KCL parser should allow if expr after |>

This KCL program parses:
fn f = (i) =>  { return i + 1}

0 |> f(%)

This equivalent program doesn't:
fn f = (i) =>  { return i + 1}

- 0 |> f(%)
+ 0 |> if true {
+   f(%)
+ } else {
+   f(%)
+ }

Because the parser defines a pipeline expression as some expression followed by 1 or more CallExpression, and in the second example, it's followed by an IfExpression. Clearly the pipeline expression parser should allow this too.
Thanks @lf94 for leading me to this.

I think an IfExpression should be a valid expression in a pipeline expression if each clause (not including the conditional) is a CallExpression.   Should this be more permissive such that a valid IfExpression in a pipeline expression is one where each clause (not including the conditional) is an expression that contains substitution operator?

This might not be compatible with the new proposal https://github.com/KittyCAD/kcl-experiments/pull/15.


allow adding plugins and change defaultmode for opensearch dashboards

Signed-off-by: Lu Yu<EMAIL_ADDRESS>Description
Similar to #71 , the change would allow installing external plugins when container starts for opensearch dashboards.
In addition, this change allows to specify defaultMode for the opensearch_dashboards.yml file which is mounted as configmap
Changelog and Readme are updated.
Changes were tested using k8s cluster using the chart
Issues Resolved
#341
Check List

[x] Commits are signed per the DCO using --signoff

For any changes to files within Helm chart directories:

[ ] Helm chart version bumped
[x] Helm chart CHANGELOG.md updated to reflect change

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@BionIT Can you refactor the PR according to contribution guidelines mentioned in  : CONTRIBUTING.md

@BionIT Can you refactor the PR according to contribution guidelines mentioned in  CONTRIBUTING.md

@BionIT Can you refactor the PR according to contribution guidelines mentioned in  CONTRIBUTING.md

@BionIT Can you refactor the PR according to contribution guidelines mentioned in CONTRIBUTING.md

@TheAlgo Sure! Do we need to bump appVersion from 2.3.0 to 2.3.1 as well or just version? The linked PR did both, but OSD latest release is at 2.3.0

@BionIT Can you refactor the PR according to contribution guidelines mentioned in CONTRIBUTING.md

@TheAlgo Sure! Do we need to bump appVersion from 2.3.0 to 2.3.1 as well or just version? The linked PR did both, but OSD latest release is at 2.3.0

appversion keep 2.3.0, bump version.

@BionIT Can you refactor the PR according to contribution guidelines mentioned in CONTRIBUTING.md

@TheAlgo Sure! Do we need to bump appVersion from 2.3.0 to 2.3.1 as well or just version? The linked PR did both, but OSD latest release is at 2.3.0

appversion keep 2.3.0, just bump version.

Thanks @peterzhuamazon and @TheAlgo ! I have bumped version for osd chart, linter was complaining about version bump for os, but I only updated documentation for os chart, and per CONTRIBUTING.md: Chart version should be bumped for every code change except docoumentation change. Please let me know if any advice~

@BionIT Can you refactor the PR according to contribution guidelines mentioned in CONTRIBUTING.md

@TheAlgo Sure! Do we need to bump appVersion from 2.3.0 to 2.3.1 as well or just version? The linked PR did both, but OSD latest release is at 2.3.0

appversion keep 2.3.0, just bump version.

Thanks @peterzhuamazon and @TheAlgo ! I have bumped version for osd chart, linter was complaining about version bump for os, but I only updated documentation for os chart, and per CONTRIBUTING.md: Chart version should be bumped for every code change except docoumentation change. Please let me know if any advice~

Yes this is not ideal but this is a backlog for this repo. Our linter is configured this way. @prudhvigodithi / @peterzhuamazon We need to fix this ASAP. In the meanwhile @BionIT can you bump minor version for OS chart as well so that linter is happy

@BionIT Can you refactor the PR according to contribution guidelines mentioned in CONTRIBUTING.md

@TheAlgo Sure! Do we need to bump appVersion from 2.3.0 to 2.3.1 as well or just version? The linked PR did both, but OSD latest release is at 2.3.0

appversion keep 2.3.0, just bump version.

Thanks @peterzhuamazon and @TheAlgo ! I have bumped version for osd chart, linter was complaining about version bump for os, but I only updated documentation for os chart, and per CONTRIBUTING.md: Chart version should be bumped for every code change except docoumentation change. Please let me know if any advice~

Yes this is not ideal but this is a backlog for this repo. Our linter is configured this way. @prudhvigodithi / @peterzhuamazon We need to fix this ASAP. In the meanwhile @BionIT can you bump minor version for OS chart as well so that linter is happy

Sure, @TheAlgo! Thanks for clarifying! I have bumped version for os chart as well and the linter is happy now :).

Adding @prudhvigodithi to take a look at this.

@prudhvigodithi @DandyDeveloper Please take a look when free. Thanks

Heya, what's the latest on getting this reviewed? I'd like to get this merged.

Hey @BionIT thanks for the PR and contribution, please check the following points.

Adding default mode is not a bad idea but can you share what issues faced for this to get added, if not lets keep what kubernetes sets to the defaultMode to the volume.
Since the change is particular to the OpenSearch Dashboard, we dont have to increment the versions in chart.yaml for OpenSearch, only for the dashboard chart is required.
Can you also please resolve the conflicts?

@TheAlgo @DandyDeveloper @peterzhuamazon @bbarani

Hey @BionIT thanks for the PR and contribution, please check the following points.

Adding default mode is not a bad idea but can you share what issues faced for this to get added, if not lets keep what kubernetes sets to the defaultMode to the volume.
~- Since the change is particular to the OpenSearch Dashboard, we dont have to increment the versions in chart.yaml for OpenSearch, only for the dashboard chart is required.~
Can you also please resolve the conflicts?

@TheAlgo @DandyDeveloper @peterzhuamazon @bbarani @seraphjiang

Per request, resolved conflicts.
Per allowing users to set defaultMode, k8s allow users to specify defaultmode for volumes, why our helm chart want to restrict it to readonly? We have use case to dynamically change the values in the yaml and the current set up prevents that.

Pinging @DandyDeveloper @TheAlgo @prudhvigodithi again for the checks on @BionIT latest changes.
I am ok with this approach, and if any of you give an approval we can follow up with a merge.
Thanks.

Related to kind action need to update to 1.3.0.

Hi @BionIT please have a follow-up backport PR to 1.x.
Thanks.

Backport 1.x: #359