feature request: Support map attribution printing

Currently, there seems to be no map attribution printing functionality as configuration, but it would be nice to support it from a point of view of license.
The following is OpenStreetMap attribution guidelines for printed maps.
https://osmfoundation.org/wiki/Licence/Attribution_Guidelines#Books,_magazines,_and_printed_maps

Books, magazines, and printed maps
For a printed map and similar media (that is ebooks, PDFs and so on), the credit must appear beside the map if that is where other such credits appear, or in a footnote/endnote if that is where other credits appear, or in the “acknowledgements” section of the publication (often at the start of a book or magazine) if that is where other credits appear. The URL to openstreetmap.org/copyright must be printed out.

You can put an attribution text in the JasperReport template, do you need something other?
What do you imagine?
Having an attribution sting on the layers, and the print will create a string array with all the attributions, and put it in the context?

@sbrunner Thanks for your reply!

You can put an attribution text in the JasperReport template, do you need something other?

Yes, it's possible to add attribution text directly (or via parameter) in the JasperReport template, but I thought this is a common use case, so supporting it in this library side is nice.

What do you imagine?
Having an attribution sting on the layers, and the print will create a string array with all the attributions, and put it in the context?

Yes, my image is similar with other frontend map libraries (OpenLayers, Leaflet, etc.),

OpenLayers example: https://openlayers.org/en/latest/examples/attributions.html
Leaflet example: https://leafletjs.com/examples/quick-start/

but more simple map level configuration (like attributes/map/attributions as text string) may be enough.
And control configuration image is similar with current scalebar and northArrow.


update docs on composeExchanges

The code and code-docs says its right to left (and tested it locally, it is right left) but the official docs says its left to right
Helpful links:

https://github.com/urql-graphql/urql/blob/88bed21c015dc673d900bf1a806e43ccb2900879/packages/core/src/exchanges/compose.ts#L12
https://github.com/urql-graphql/urql/commit/e6710d29ed426df9d21d9d9daa18ca182647db53
https://commerce.nearform.com/open-source/urql/docs/api/core/#composeexchanges

Summary
I was using composeExchanges. Read official docs but it works in the opposite direction.
Set of changes
Update docs

Both are correct, actually. Sorry about the confusion!
Let me explain.
The TSDocs state:

[...] calling them from
right to left, with the prior Exchange’s ExchangeIO function
as the ExchangeInput.forward input.

The docs site states:

It chains them in the order that they're given, left to right.

Both are correct.
The exchanges themselves are chained from left to right, meaning, the operations run through from left to right, just like the exchanges array on the Client are given (i.e. natural order, basically)
However, the TSDocs explain the code and what's going on there. What's going on in the code is that the exchanges are called from right to left, which composes them to be chained in the natural order, as given.
This is further complicated by the operation results running in reverse order basically, so I know it's confusing, but the docs must state left-to-right to clarify that that's the expected order

@kitten thanks for the quick reply. I got your point. I will close this PR but I think the docs not be a bit more clear as you said :)
thanks

@kitten Actually, I'm still confused. The docs say, 'It chains them in the order that they're given, left to right.' What does that mean? One would assume that it executes in the order they are provided, i.e., the first item will execute first and so on. By reading this, one can never assume that they will execute from right to left, i.e., the last will execute first and then it travels towards the left

It means that the requests that come in will be executed in the order of the exchanges left-to-right, and then return as a result from right-to-left

If I understand this correctly, is it something similar to this diagram?

If this is the case then maybe the docs could be updated to make it a bit more obvious?

That's a great suggestion :+1:
Originally, this diagram served to explain this, but it isn't quite up-to-date and good enough anymore: https://urql.dev/goto/docs/architecture/#the-client-and-exchanges
Generally, we'll need to work on some documentation changes to get things back into shape


console: missing kernel.h include in header

console.h requires k_fifo but kernel.h is not included.
CC @pfalcon.

@lemrey : How did you hit it? (Generally, we should back such things with regression tests. Maybe we'll be too lazy to write one, but we should not be lazy to discuss how to write one ;-).

I hit a compiler warning; perhaps it is possible to reproduce by including console.h before kernel.h:
warning: ‘struct k_fifo’ declared inside parameter list will not be visible outside of this definition or declaration

Thanks. Fairly speaking, I had an idea that all Zephyr apps should start with #include <zephyr.h>. Dunno where I got the from though, probably because all in-tree samples do that. Anyway, it's an easy fix, posted as https://github.com/zephyrproject-rtos/zephyr/pull/9541


Add recursive loading of child directories for custom checks

Description
Custom checks directory flag is good but currently its only loads all the custom checks from a given directory. This becomes less manageable  and less maintainable when there are alot of custom check files present and when custom checks files are unrelated to one another for the same provider as shown below
Current Behaviour
Current functionality only provides support for all the tfchecks files located in same directory when custom checks directory flag is passed.
Example: tfsec -custom-check-dir custom_checks/aws only loads all the tfchecks files from custom_checks/aws as shown in below directory structure
custom_checks
└──  aws
    ├── README.md
    ├── lambda_tags_tfchecks.yml
    ├── ec2_iam_profile_tfchecks.yml
    ├── elb_tags_tfchecks.yml
    ├── k8s_custom_001_tfchecks.yml
    └── ec2_tags_tfchecks.yml

Proposed Solution
Proposed solution is to load _tfchecks.json or _tfchecks.yaml or _tfchecks.yml  from all the child directories (as shown in below directory structure) within the given custom_checks/aws directory when the command tfsec -custom-check-dir custom_checks/aws is used.
This will give proper directory isolation for all the custom checks related to given resources and a will be better maintainable and easy to manage custom check files.
Note The current functionality of loading only *_tfchecks.* is still maintained so that only tfchecks files are loaded
custom_checks
├── aws
│   ├── README.md
│   ├── aws_ec2
│   │   ├── ec2_iam_profile_tfchecks.yml
│   │   ├── ec2_tags_tfchecks.yml
│   │   └── staging
│   │       ├── custom_001_tfchecks.yml
│   │       └── custom_002_tfchecks.yml
│   ├── aws_elb
│   │   └── elb_tags_tfchecks.yml
│   ├── aws_lambda
│   │   └── lambda_tag_tfchecks.yml
│   └── global_tfchecks.yml
└── k8s
    └── k8s_custom_1_tfchecks.yml

Solution
This PR adds recursive file loading in internal/app/tfsec/custom/loader.go when loading custom checks files from given directory. The logic of identifying tfchecks in a given directory is unchanged. But now when the loader identifies a child directory, it loads any tfchecks files present within the child directory.

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

Hi @owenrumney,
Wondering if you have any updates on this PR to allow tfsec to support this functionality? Let me know if you need any updates from my end.

Hi @japan-p - this is a nice idea but it possibly needs either a little more refinement or a little more understanding at my end :)
Are you thinking that all of the custom checks across the entire directory structure will all be applied to all templates in the scanned directories?
It seems it'd be more intuitive/practical if custom checks at each directory level were only applied to that level. Does that make sense? Or am I missing a use case here?


win: add cast to ensure that printf precision value has 32bits during…

… 64bit build

I haven't seen the message in #955, but CI seems fine with this change: https://ci.nodejs.org/view/libuv/job/libuv-test-commit/79/

I'm building libuv as part of our project, which means the warning flags are probably more strict. In any case, the modifier is expected to be 32bit so the cast is necessary.

Landed in 7b07d18ad91ed2db58644185f1eaa60fdc7303af. Thanks!


[BUG] Noisy point clouds near the bottom-right corner of OAK-D PRO camera

Hello everyone, in our lab we are currently testing our OAK-D PRO camera to detect point clouds of cylindrical PVC pipes. However, we have noticed that, keeping a fixed distance from the camera (z-axis of the camera) of about 35 cm, the point clouds become increasingly noisy as we move closer to the bottom-right corner of the camera's field of view. Generally, we could say that the affected area is the triangle with vertices at the bottom-left corner, bottom-right corner, and top-right corner. I am attaching the three point clouds separately, as well as a point cloud with the three overlaid, and two photos of the current setup. The room is moderately lit without any direct light sources pointed at the camera. Any ideas on why this noise is occurring?
Luxonis material.zip

Hi @Luca0101 ,
First, let me share docs on improving pointcloud noise:
https://docs.luxonis.com/hardware/platform/depth/configuring-stereo-depth/#Configuring Stereo Depth-6. Fixing noisy pointcloud
I'd say better lighting could help improve this, along with:

Lower confidence threshold. this will result in less points, but also less noise
Use decimation filter /w median mode: https://docs.luxonis.com/hardware/platform/depth/configuring-stereo-depth/#Configuring Stereo Depth-2. Fixing noisy depth-Stereo postprocessing filters-Decimation filter

Feel free to share pointclouds of the pipes after trying out these suggestions.
Thanks, Erik

Hi @Erol444, thank you for these precious suggestions. We'll try them in the future as soon as we have some time and post the results here.
Gianluca


Thirdweb react fails to load listings after wagmi and rainbowkit update

We use rainbowkit and wagmi as our wallet provider. We have used wagmi as the query client for some time. Recently we updated those packages and I believe a function was deprecated (we needed to fix breaking changes in our code)
The error we are getting is contract?.getAddress is not a function
We instantiate the contract like so
    const { contract: marketplace } = useContract('0xe304ff5363427FD35B366c9f651bD2F95B98F3D2', 'marketplace');
    const { data: listings, isLoading: loadingListings } = useActiveListings(marketplace);

Our provider configs are (one for wallet connected one for now)
                <ThirdwebProvider
                    activeChain="goerli"
                    queryClient={wagmiClient?.queryClient}
                >
                    <Component {...properties} />
                </ThirdwebProvider>

Or
                <ThirdwebSDKProvider
                    activeChain="goerli"
                    queryClient={wagmiClient?.queryClient}
                >
                    <Component {...properties} />
                </ThirdwebSDKProvider>

Does the react sdk need an update or is this perhaps another issue? It only appears now after merging changed with updated wagmi and rainbowkit packages
As soon as I remove wagmi as query client the listings load. But then I can't get the users wallet to interact with the listings without wagmi as the query client (as far as I can tell)

I am also getting the same error with the marketplace V3, with this to instantiate it
    const { contract: marketplace } = useContract('0x2eC2ceb7E69c91d8bD3f8cb9e29102154e125630', 'marketplace-v3');
    const { data: listings, isLoading: loadingListings } = useValidDirectListings(marketplace);

I have resolved this for now by providing the signer prop instead with the signer from Wagmi


Send test results to GA server

https://github.com/nprapps/dailygraphics/blob/master/graphic_templates/_base/js/analytics.js

Example of tracking something:
https://github.com/nprapps/elections16/blob/master/www/js/app.js#L251
GA track takes obj with, in hierarchical-ish order: slug, action, label (eg which slide), value (eg time)

Sample tracking code in #33
Will check back in later to see how's its looking on Analytics:


Comment on photos

This would be a dag structure like the photo directories, something like:

a file called parent that contains the root cid of the target photo
a file called comment that contains the actual comment text (much like caption file in the photo dag)
a file called meta which is much like the photo meta file, containing, created, username, and peer_id fields.
a file called last which contains the cid of the last comment OR photo update this peer made in that room, this way we still only need to republish HEAD update for each room

That covers the storage mechanism. We can do an indexing setup just like photos. So, a table in the SQLite db that has columns for the fields we want to index: parent, created, username, peerId.

V2 architecture has a "block type" for a comment stubbed out: https://github.com/textileio/textile-go/blob/master/repo/model.go#L27


Linked Lists in C

I want to add C program for Circular And Doubly Circular Linked List.

@sjatin-cse Doubly linked list already implemented.


Integration Tests via Travis

[x] Remove code style checks from Travis (in favor of Scrutinizer)
[x] Configure Integration Tests (create user/etc)
[x] Fix test_encrypt_archive(), encrypt_archive() with gpg integration test issue

@manasdk @lakshmi-kannan, this is where I'm in stuck and need your help/ideas:

log: https://travis-ci.org/StackStorm/st2/jobs/71434949#L1473
errored test: st2debug test_encrypt_archive()
function: st2debug encrypt_archive()

I've included this PR in shared st2:travis-integration-tests branch, so you could commit to it/debug.

Except of this error, Integration tests in Travis should work OK (verified before in my forked branch).

Hm, looking at the changes, we don't run "lint" task anymore? Or am I missing something and we run it as part of a separate job now?

Ignore my last comment. I just now saw the "Remove code style checks from Travis (in favor of Scrutinizer)" line in the PR description.
I'm personally not too big of a fan of that. I'd prefer to run everything on Travis CI. This way you only need to go to a single page to see the results.

There are also some advantages behind that, see: https://github.com/StackStorm/st2/pull/1724
And errored build looks pretty nice:
https://scrutinizer-ci.com/g/armab/st2/inspections/708d945a-2906-4fe0-9f44-7ef410714e13
Also in usual Travis build if something exited with error - further execution is stopped. See Scrutinizer ^^ log. All tasks are processed, which is especially good for code style checks.

When code in module A is touched during tests from module B, they are both included in reports now.
  (+4%)
@Kami @lakshmi-kannan @manasdk this PR is ready for review now :eyes:

When code in module A is touched during tests from module B, they are both included in reports now.
That's awesome.

LGTM.

LGTM. I'll merge.


proposal: syscall: add Syscall21 and Syscall24

Currently Go supports calling up to 18 arguments when calling function from a Windows DLL. But there are some SDKs that have functions with 22 and more parameters. It seems there is no workaround for this issue.

Any idea where we should stop?

If you have a syscall with 21 parameters, you probably missed some. (With apologies to Alan Perlis)

I think that MSVC allows at most 256 parameters in a function definition, so we probably shouldn't need more than that.

But there are some SDKs that have functions with 22 and more parameters.

Can you provide some reference, please.
Thank you.
Alex

For example a SDK for LED matrix display controller. One of the functions is:
int HDAPI Hd_Rt_SendRealTimeText(int nSendType, void *pStrParams, int nRealTimeAreaIndex, int nMaxPageCount, int nColor, int nGray, int nX, int nY, int nWidth, int nHeight, void *pText, int nTextColor, int nBackGroupColor, int nStyle, void *pFontName, int nFontHeight, int nShowEffect, int nShowSpeed,int nStayTime, int nLiveTime, int bSaveToFlash, void *pDeviceGUID);

This is going to be sufficient, since 24 is the highest number. https://www.youtube.com/watch?v=RkP_OGDCLY0

@xc218m is this a theoretical issue? an internet search doesnt even show that this is a valid function [1].
and I dont see even a single use at GitHub, under any language [2].
I am all in favor of Go being robust, but wasting time is a different matter.

https://google.com/search?q=Hd_Rt_SendRealTimeText
https://github.com/search?q=Hd_Rt_SendRealTimeText

It's not theoretical. It's from SDK for these controllers:
https://www.huidu.cn/en/monochromatic-and-dual-color-network-port-control-card.html
You can't find it, because it is not publicly available.
A already used another language for the SDK, so it's not a big problem for me.
I tried to add Syscall24 myself but I think that has to be changes in asmcgocall function too.

cc @zx2c4 @mattn

It's not theoretical. It's from SDK for these controllers:
https://www.huidu.cn/en/monochromatic-and-dual-color-network-port-control-card.html

Just so we're clear: this isn't actually to call any Windows system API or something, but just to call some arbitrary C functions with tons of arguments from that SDK, correct? (Not that that's necessarily a bad thing.)

Any idea where we should stop?

We could add a variadic function and then we'd be done, even when NTv6 extends the max syscall list to 640KB. We already support variadic syscalls for Proc.Call and LazyProc.Call.

Just so we're clear: this isn't actually to call any Windows system API or something, but just to call some arbitrary C functions with tons of arguments from that SDK, correct? (Not that that's necessarily a bad thing.)

I agree with Jason sentiment, that this is pretty rare.
And since 24 is the Highest Number, Syscall24 should be the last API we will ever need. It shouldn't be difficult to add these APIs. We should also adjust runtime.TestSyscall18 to verify the change.

We could add a variadic function ...

Adding Syscall21 and Syscall24 is easy comparing to adding and implementing new API. And there is only 1 Syscall21 user so far. So we should not invest too much effort here.
Alternatively we can say no to Syscall21 and Syscall24. We would loose users of those controllers.
Alex

However, it also looks like the SyscallNN functions then eventually trampoline back into a common, variadic cgo stub anyway.

If we're already trampolining back to a varidac sub, we should just make the primary API into this varadic, and cut out all the middle pieces. We can then rework mksyscall and that codegen around the more direct varadic route. The SyscallXX helpers will then be left around for backwards compat. That would address the concerns of this proposal, and also clean up the API considerably.

Good point, writing a varargs version seems clearly preferable.  The signature would presumably be something like
SyscallN(trap uintptr, args ...uintptr) (r1, r2 uintptr, err Errno)

I think we can do that safely using the //go:uintptrescapes annotation.

OK, so it sounds like we should do SyscallN.

We could limit this to Windows, but it seems like we should just implement it for all systems.
Non-Windows systems have a clearer limit, and we can handle N up to that limit and then return an error otherwise.

Based on the discussion above, this proposal seems like a likely accept.
— rsc for the proposal review group

No change in consensus, so accepted. 🎉
This issue now tracks the work of implementing the proposal.
— rsc for the proposal review group

Any info about performance changes? 👀

I wouldn't expect any performance changes from this, but I think it would be great if someone wanted to measure and report back.


Training error

Hello!
I get this error at the beginning of the first epoch:
Traceback (most recent call last):
File "/home/diogene/Documents/Alexandra/SGDepth-master/train.py", line 228, in _run_epoch
for batch_idx, batch in enumerate(self.train_loaders):
File "/home/diogene/Documents/Alexandra/SGDepth-master/loaders/init.py", line 25, in iter
for batch_idx, *group in zip(length_iter, *infinite_iters):
File "/home/diogene/Documents/Alexandra/SGDepth-master/loaders/init.py", line 30, in _make_infinite
for batch in loader:
File "/home/diogene/anaconda3/envs/torch_110/lib/python3.7/site-packages/torch/utils/data/dataloader.py", line 582, in next
return self._process_next_batch(batch)
File "/home/diogene/anaconda3/envs/torch_110/lib/python3.7/site-packages/torch/utils/data/dataloader.py", line 608, in _process_next_batch
raise batch.exc_type(batch.exc_msg)
AttributeError: Traceback (most recent call last):
File "/home/diogene/anaconda3/envs/torch_110/lib/python3.7/site-packages/torch/utils/data/_utils/worker.py", line 99, in _worker_loop
samples = collate_fn([dataset[i] for i in batch_indices])
File "/home/diogene/anaconda3/envs/torch_110/lib/python3.7/site-packages/torch/utils/data/_utils/worker.py", line 99, in
samples = collate_fn([dataset[i] for i in batch_indices])
File "/home/diogene/Documents/Alexandra/SGDepth-master/dataloader/pt_data_loader/basedataset.py", line 182, in getitem
sample = self.load_transforms(sample)
File "/home/diogene/anaconda3/envs/torch_110/lib/python3.7/site-packages/torchvision/transforms/transforms.py", line 61, in call
img = t(img)
File "/home/diogene/Documents/Alexandra/SGDepth-master/dataloader/pt_data_loader/mytransforms.py", line 61, in call
sample[key] = pil.fromarray(sample[key])
File "/home/diogene/anaconda3/envs/torch_110/lib/python3.7/site-packages/PIL/Image.py", line 2642, in fromarray
arr = obj.array_interface
AttributeError: 'NoneType' object has no attribute 'array_interface'
This was the logged summary
_Starting initialization
Loading training dataset metadata:
/home/diogene/Documents/Alexandra/SGDepth-master/Dataset/kitti_zhou_split/train.json
/home/diogene/Documents/Alexandra/SGDepth-master/Dataset/kitti_zhou_split/train.json

Can use 43760 images from the kitti (zhou_split) train split for depth training
/home/diogene/Documents/Alexandra/SGDepth-master/Dataset/cityscapes/train.json
Can use 2975 images from the cityscapes train set for segmentation training
Loading validation dataset metadata:
/home/diogene/Documents/Alexandra/SGDepth-master/Dataset/kitti_kitti_split/validation.json
Can use 1159 images from the kitti (kitti_split) validation set for depth validation
/home/diogene/Documents/Alexandra/SGDepth-master/Dataset/cityscapes/validation.json
Can use 500 images from the cityscapes validation set for segmentation validation
Summary:
Model name: zhou_full
Logging directory: /home/diogene/Documents/Alexandra/SGDepth-master/ckpt/sgdepth_eccv_test/zhou_full
Using device: cuda (GeForce RTX 2080 Ti)_

Do you have any idea why I get this error? Thanks in advance!

Hi @Ale0311
I think the important lines are sample[key] = pil.fromarray(sample[key]) and AttributeError: 'NoneType' object has no attribute 'array_interface'. This suggests that the sample does not contain an image but a 'NoneType'-Object. I would suspect that the dataset structure is either not as described or that parts of the dataset are missing. To trace back, which image is causing this error, you could try to debug the __get_item__() function in the BaseDataset class in dataloader/pt_data_loader/basedataset.py. Here I would check, if the paths that the images are loaded from actually exists on your system.
Alternatively, you could try to train with a reduced number of loaders for depth and semantic segmentation by setting --depth-validation-loaders "", --segmentation-validation-loaders "", --depth-training-loaders "", and/or --segmentation-training-loaders "" in the argument parser. If the error dissapears, then you know that this loader/dataset caused the error.

Thank you for your response.
Meanwhile I managed to fix the error. The problem was that I hadn't downloaded the gtFine folder from cityscapes dataset.
It is working now.
But just to be safe, are there any other folders I need to download for cityscapes? In the cityscapes folder I have the .json files you provided and the downloaded gtFine and leftImg8Bit folders,both with train, val and test folders inside. Do I also need the trainextra archive?
Thank you again for your help and response! 😊

That sounds good!
For Cityscapes just the 2975 images + segmentation labels are used, so you do not have to download the trainextra archive. Just make sure that if you want to run the evaluation on the KITTI 2015 dataset, that you have downloaded it and preprocessed the disparity maps to depth maps.
Otherwise, I think everything should be working without further dataset downloads.

Thanks again! Yesterday I started the training script and I have just checked the results:

I think everything is ok, because the results are really close to those that you presented.


NamespaceLifecycle admission control plugin

If a Namespace is undergoing graceful termination, prohibit any CREATE actions in that namespace.  This admission control plug-in will be enabled once we have a NamespaceController that is actually doing the graceful termination.
See #5195 for design details.

Assigning to @erictune, who I believe is more familiar with admission control plugins.

LGTM

@bgrant0607 - made the update to move to fields.Everything() so this should be good if we get green travis


LoRA-trainer-XL not working

With free colab it does not work. Do I have to wait until they fix it?　I am very sad.

Does not work too

i change xformers to !pip install xformers==0.0.24 --no-deps and it work now

i change xformers to !pip install xformers==0.0.24 --no-deps and it work now

It worked!　Thanks. I am very happy.


Unable to Refresh, Takeover Dataset

Hi Jayendran,
Thanks for the wonderful post.
I have tried your steps.
When I'm trying to takeover dataset,refresh the dataset and apply schdeule refresh, its throwing errors as SPN is not having access.
DO we need to add the SPN to Gateway similar to User PrincipalName.
And Does the cloud sources also needs to be added with SPN to Gateway.
Thanks in advance.
Regards,
Srinivas K

Hi @jyothsnakundan , If you are talking about dataset that using cloud source then there is no gateway is needed - such a scenario if you add the spn as workspace admin all dataset operations will work
If you have a dataset that is using on-prem or need a gateway (to refresh) then your SPN should have access to the gateway. There is a way to add the SPN as admin to the gateway https://learn.microsoft.com/en-us/powershell/module/datagateway/add-datagatewayclusteruser?view=datagateway-ps#example-4
But I remember from my earlier days when they released these features of adding SPN to gateway there are some limitation which I captured here https://github.com/jayendranarumugam/GatewayAutomation/blob/master/README.md#limitation-as-of-may-2020 I personally never tried this so not sure how it will work for you now.

@jyothsnakundan I'm closing this issue for now. If you need any further help let me know


add missing dependency

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

AWS CodeBuild CI Report

CodeBuild project: GitHubPipelineAutoBuildProj-eNGjrh4O6eRH
Commit ID: 3046676da3b70e7f711ef8441175e85aa358cbeb
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

Failed, although npm run test passed.

AWS CodeBuild CI Report

CodeBuild project: GitHubPipelineAutoBuildProj-eNGjrh4O6eRH
Commit ID: 7ee965cee9afce7e9c725b1e702656e7c8e69560
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubPipelineAutoBuildProj-eNGjrh4O6eRH
Commit ID: c4eb0217fe5970a7ec833d2c6bb21feb5e89e771
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: GitHubPipelineAutoBuildProj-eNGjrh4O6eRH
Commit ID: 5c829ee65dad161029c66f2e89896be64ae226e8
Result: FAILED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

I'm going o close this PR without merging as I realize I don't quite understand why/where this dependency is missing. Don't hesitate to re-open after clarifying this...

AWS CodeBuild CI Report

CodeBuild project: GitHubPipelineAutoBuildProj-eNGjrh4O6eRH
Commit ID: c9dde4c6bcf90a97b77790359d815bd7ff314b79
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Use correct attribution in the footer of documentation pages

To be inline with ASF policy, we should update the text in the footer used in the docs (e.g. https://arrow.apache.org/adbc/current/index.html) to match the footer used in the main website (https://arrow.apache.org/). i.e. add the trademark attribution test.

Already done. https://github.com/apache/arrow-adbc/commit/650994d59e2c44971ff292d7bc880ea8aabd1125


When passing data from the route to app with hydrate-client option all markdown data gets added

I tried to make a webpage with a hydrated app having hash-based routing. But as soon I set the hydrate-client={{data}} option, not only the data which I append gets added, but also all the markdown data from the template. So I have to create a new object on the static page and pass this object to the routes object, otherwise, all the markdown data gets added to the generated static HTML file.
I created a repository to highlight the problem (the first branch shows how to circumvent the problem):
https://github.com/floratmin/template/tree/routes
https://github.com/floratmin/template/tree/routes-markdown

I just had a quick look into your code and it seems you used the @elderjs/plugin-markdown set to the routes blog, which will add your pages markdown to data on these routes. This is the expected behavior in order to generate your blog pages.
What I propose you to do is to add an object into your data (in route.js), something like hydratate and put all the data you need for your hydrate component inside it, then pass this object. I guess is always better to separate what you need for SSG and hydrate component.

Sorry if I was not totally clear. The route of concern is the app route (e.g. /en/app). The rest of the repo is the standard elder.js template. I am not intending to use the @elderjs/plugin-markdown in my project at all. I stumbled over this behaviour and this could add bloat to parts of the webpage where route data is passed to hydrated clients.

@floratmin that is the designed behavior. You should only hydrate the absolute minimum so you’ll want to restructure the data object if the whole page needs to be hydrated. It is a different setup than a SPA.

Thank you very much.


Cant run apf on n Apple Silicon M1

Hello
I have mac on Apple Silicon M1. When I try to run apf I get en error
apf -r 9003 -k namespace/pod

==> : Forwarding local listening ports to (==>) remote ports
<== : Forwarding to local ports from (<==) remote listening ports (use -r option)

panic: Failed to establish manager stream: EOF
goroutine 1 [running]:
main.main()
/home/runner/work/autoportforward/autoportforward/cmd/apf/apf.go:123 +0xe8c
Any ideas on how to fix this?

Could you check the following things for me to debug?

the container's kernel arch (x64 or aarch64)
The m1 version I built can only run on the aarch64 kernel. You can tweak the build.sh, or maybe I can update the script later.

if the pod is runing using read-only root fs
The apf needs the root to be writable, as it need to copy a small binary into the container.


Adding attachment >3 MB not possible if no client secret is used

Hi,
maybe it's a bug, but I'm unable to send attachments which are bigger than 3 MB via a managed identity or a normal app registration with certificate based authentication.
After creating a client secret for the same app registration it worked. (But I don't want to use a client secret.)
Is it possible, that the function which handles the upload somehow doesn't use the already existing session?
Because the upload doesn't work, but the email is send without problems (and without an attachment of course).

I used the following commands:
Connect-MgGraph -Identity
or
Connect-MgGraph -ClientId $ClientId -TenantId $TenantId -CertificateThumbprint $Thumbprint
Send-EmailMessage -From 'XXX' -To 'XXX' -HTML "XXX" -Subject 'XXX' -Graph -ReplyTo "XXX" -MgGraphRequest -Attachment "C:\temp\XXX.pdf" -Verbose
(By using the parameter -Credential and the function ConvertTo-GraphCredential for generating the credential object and removing the parameter -MgGraphRequest it worked.)
Thanks,
Markus

Try using PS 7, it should provide better error reporting. Also do you have proper rights to upload files? As ReadWrite Emails? Send is not enough.

Same with PS7 (do I have to "reinstall" the module for PS7?):

Yes, both, my managed identity and the app registration have Mail.Send and Mail.ReadWrite. (That should be fine, because it worked with the client secret for the same app registration.)

You don't have to reinstall module.
Just to clarify:

Does it work for 4MB plus with Connect-MgGraph and MgGraphRequest with ClientSecret?
It doesn't work with Connect-MgGraph and MgGraphRequest with Certificate?

And I assume in all cases the mailbox does exists? The one you're sending it with?

I'm now using the following workaround without the need for a client secret:
$null = Connect-AzAccount -Identity
[System.Object]$AccessToken = (Get-AzAccessToken -ResourceTypeName 'MSGraph' -AsSecureString -WarningAction 'SilentlyContinue').Token
[System.Object]$Credentials = ConvertTo-GraphCredential -MsalToken ([System.Net.NetworkCredential]::new('', $AccessToken).Password)
[System.String[]]$Attachments = @(XXX)

Send-EmailMessage -From 'XXX' -To 'XXX' -HTML "XXX" -Subject 'XXX' -Graph -ReplyTo "XXX" -Credential $Credentials -Attachment $Attachments -DoNotSaveToSentItems


Fix symbol types where an incomplete type check occurs

The PR fixed the fragmented types that are currently presented on an
incomplete type check i.e. if a referenced nuget package is not
restored etc.
e.g. types may be represented by ?25354-> ?25345  rather than ’a -> ‘b
Ive fixed all the occurrence that I can see happen although there could
be further issues in FSharpMemberOrFunctionOrValue.FullType as I only
touched ValRef’s

Any comments?

It looks good.  Unit tests?

Unit test will require a discrete fragmented type failure case, the only one I know of is when a nuget package is not restored, Ill try and get a small unit test to cover this.

@7sharp9 This kind of problem should occur whenever necessary referenced DLLs are not available and you have code that uses constructs in those DLLs.

Closed in favour of #442


Floats are completely lost after Json.LoadReport<>()

"paeaefae:aeae": {
    "type": "float",
    "default": 3.2,
    "range": [ 0.0, 2.0 ]
}

When read by the diagnoser, the default value for this actor property is 3. Just logging jsonc.prase(diagnoser.document) contains the 3.2 value.
Most likely happens in other places too, but this is where I noticed it

@DaanV2 Thoughts?

I mean if its wrong, it should be updated. Im unsure what you want to gauge 🤔

I was hoping you'd have an idea on how it can be fixed. Since it seems like the type cast is where the data is lost

Looked into it, but nothing is turning it to 3, where are you finding this?

I ran it with the latest extension, and the only error I'm getting is:
Default value is not within the range: 3.2

That's odd, it must be something with my system's environment then. Either way this isn't an issue then and the pr can be merged


Credo report occurs on v0.6.0_rc

https://github.com/rclex/rclex/runs/4899585539?check_suite_focus=true
Checking 17 source files ...

  Software Design
┃
┃ [D] → Found a TODO tag in a comment: # TODO nodeに紐付いているpub,subをきちんと終了させる
┃       lib/rclex/node.ex:238 #(Rclex.Node.handle_call)
┃ [D] → Found a TODO tag in a comment: # TODO: has_keyで見る
┃       lib/rclex/node.ex:173 #(Rclex.Node.handle_call)

  Code Readability
┃
┃ [R] ↗ Function/macro/guard names should be written in snake_case.
┃       lib/rclex/nifs.ex:309:7 #(Rclex.Nifs.readdata_std_msgs__msg__String)
┃ [R] ↗ Function/macro/guard names should be written in snake_case.
┃       lib/rclex/nifs.ex:306:7 #(Rclex.Nifs.setdata_std_msgs__msg__String)
┃ [R] ↗ Function/macro/guard names should be written in snake_case.
┃       lib/rclex/nifs.ex:303:7 #(Rclex.Nifs.init_msg_std_msgs__msg__String)
┃ [R] ↗ Function/macro/guard names should be written in snake_case.
┃       lib/rclex/nifs.ex:294:7 #(Rclex.Nifs.readdata_geometry_msgs__msg__Twist)
┃ [R] ↗ Function/macro/guard names should be written in snake_case.
┃       lib/rclex/nifs.ex:291:7 #(Rclex.Nifs.setdata_geometry_msgs__msg__Twist)
┃ [R] ↗ Function/macro/guard names should be written in snake_case.
┃       lib/rclex/nifs.ex:288:7 #(Rclex.Nifs.init_msg_geometry_msgs__msg__Twist)
┃ [R] ↗ Function/macro/guard names should be written in snake_case.
┃       lib/rclex/nifs.ex:300:7 #(Rclex.Nifs.create_empty_msg_std_msgs__msg__String)
┃ [R] ↗ Function/macro/guard names should be written in snake_case.
┃       lib/rclex/nifs.ex:297:7 #(Rclex.Nifs.get_typesupport_std_msgs__msg__String)
┃ [R] ↗ Function/macro/guard names should be written in snake_case.
┃       lib/rclex/nifs.ex:285:7 #(Rclex.Nifs.create_empty_msg_geometry_msgs__msg__Twist)
┃ [R] ↗ Function/macro/guard names should be written in snake_case.
┃       lib/rclex/nifs.ex:282:7 #(Rclex.Nifs.get_typesupport_geometry_msgs__msg__Twist)

  Refactoring opportunities
┃
┃ [F] → Function takes too many parameters (arity is 9, max is 8).
┃       lib/rclex/nifs.ex:244:7 #(Rclex.Nifs.rcl_wait_set_init)

  Warnings - please take a look
┃
┃ [W] ↗ There should be no unused return values for Enum functions.
┃       lib/rclex/publisher.ex:47:5 #(Rclex.Publisher.publish)
┃ [W] ↗ There should be no unused return values for Enum functions.
┃       lib/rclex/job_executor.ex:42:8 #(Rclex.JobExecutor.handle_cast)
┃ [W] ↗ There should be no unused return values for Enum functions.
┃       lib/rclex/job_executor.ex:34:8 #(Rclex.JobExecutor.handle_cast)

Please report incorrect results: https://github.com/rrrene/credo/issues

Analysis took 0.3 seconds (0.08s to load, 0.2s running 52 checks on 17 files)
188 mods/funs, found 3 warnings, 1 refactoring opportunity, 10 code readability issues, 2 software design suggestions.

Showing priority issues: ↑ ↗ →  (use `mix credo explain` to explain issues, `mix credo --help` for options).
Error: Process completed with exit code 4.

This report about "Code Readability" could be fixed (disappeared) from #98


installing text classifier or medicall image ....

Hi
I have the same problem also
am using ubuntu 14.04.5
when I install the pip install -e $DIGITS_ROOT
Usage:
pip install [options] ...
pip install [options] -r ...
pip install [options] [-e] ...
pip install [options] [-e] ...
pip install [options] <archive url/path> ...
have above mentioned lines
and then this is going to happen
pip install $DIGITS_ROOT/plugins/data/sunnybrook/
Exception:
Traceback (most recent call last):
File "/usr/lib/python2.7/dist-packages/pip/basecommand.py", line 122, in main
status = self.run(options, args)
File "/usr/lib/python2.7/dist-packages/pip/commands/install.py", line 257, in run
InstallRequirement.from_line(name, None))
File "/usr/lib/python2.7/dist-packages/pip/req.py", line 173, in from_line
return cls(req, comes_from, url=url, prereleases=prereleases)
File "/usr/lib/python2.7/dist-packages/pip/req.py", line 71, in init
req = pkg_resources.Requirement.parse(req)
File "/usr/local/lib/python2.7/dist-packages/pkg_resources/init.py", line 2833, in parse
req, = parse_requirements(s)
File "/usr/local/lib/python2.7/dist-packages/pkg_resources/init.py", line 2781, in parse_requirements
yield Requirement(line)
File "/usr/local/lib/python2.7/dist-packages/pkg_resources/init.py", line 2790, in init
raise RequirementParseError(str(e))
RequirementParseError: Invalid requirement, parse error at "'/plugins'"
Storing debug log for failure in /tmp/tmpm_CQsC
it would be wonderful if you could help me out

Hello, you need to set DIGITS_ROOT to something that makes sense, i.e. the actual root of our DIGITS repository - see this doc.

I already saw that and installed my digits based on that
now I wanted to add the text classifier and sunnybrook

Closing due to inactivity.


Clean up CFL condition implementation

We currently have hard-coded magic numbers in the CFL condition that only work for certain discretizations and might lead to problems when we start playing with different discretizations.
I think this could be calculated automatically since we have the FD coefficients at the symbolic level.

Can we close this?

is this still an issue?


Issues with CP-SAT python type annotations

python, version 9.9.3963
this release includes some py.typed files.  I like to add annotations to my python code and check them strictly, so I am totally in favour of the idea here.
however, the experience of the annotations as they are today is awkward - they generate lots of false positives on simple code.
Here are the cases that I have found myself having to # type: ignore:
from ortools.sat.python import cp_model
model = cp_model.CpModel()

result: error: Call to untyped function "CpModel" in typed context  [no-untyped-call]
from ortools.sat.python import cp_model
solver = cp_model.CpSolver()

result: error: Call to untyped function "CpSolver" in typed context  [no-untyped-call]
      status = solver.Solve(model)
      if status in (cp_model.FEASIBLE, cp_model.OPTIMAL):
          pass

result: error: Non-overlapping container check (element type: "CpSolverStatus", container item type: "ValueType")  [comparison-overlap]
  class SolutionPrinter(cp_model.CpSolverSolutionCallback):
      def __init__(self):
         super().__init__()

result: Call to untyped function "__init__" in typed context  [no-untyped-call]
please consider either improving the annotations to make them more useful, else not publishing py.typed until they are more ready for consumption.

I think three out of those four can be resolved by adding  -> None to __init__() methods

Another one:
      model = cp_model.CpModel()  # type: ignore[no-untyped-call]
      foo = model.NewIntVar(1, 26, "foo")
      bar = model.NewIntVar(1, 26, "bar")
      baz = model.NewIntVar(1, 26, "baz")
      quux = model.NewIntVar(1, 26, "quux")
      thud = model.NewIntVar(1, 26, "thud")
     model.Add(quux == foo + bar + baz + thud)

result:
foo.py:14: error: No overload variant of "__radd__" of "LinearExpr" matches argument type "SupportsComplex"  [operator]
foo.py:14: note: Possible overload variants:
foo.py:14: note:     def __radd__(self, LinearExpr | IntVar | Integral | int, /) -> LinearExpr | IntVar | Integral | int
foo.py:14: note:     def __radd__(self, LinearExpr | IntVar | Integral | int | Number | float, /) -> LinearExpr | IntVar | Integral | int | Number | float
foo.py:14: note: Left operand is of type "LinearExpr | int | SupportsComplex"

interesting. Which type checker are you using ? pytype ? pylance ? mypy ?
Pytype does not complain on the cp-sat code.

mypy

https://github.com/google/or-tools/issues/4120#issuecomment-1986861312 was not fixed by my pull request: I don't know whether you want to reopen this, or handle it separately, or neither...

Sent me the patch
Le sam. 9 mars 2024, 20:10, David Hotham @.***> a
écrit :

#4120 (comment)
https://github.com/google/or-tools/issues/4120#issuecomment-1986861312
was not fixed by my pull request: I don't know whether you want to reopen
this, or handle it separately, or neither...
—
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/4120#issuecomment-1986951410,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUPL3IUVLVX76LZLD5YKHTYXNNCJAVCNFSM6AAAAABEOB37TCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBWHE2TCNBRGA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

I have no patch for that one, it looks harder than the others!

possibly __add__ and __radd__  (and __sub__ and so on) on a LinearExpr should always just return another LinearExpr?

ah, because there's a special case at https://github.com/google/or-tools/blob/2319f87902cd269092f06809c2b91ba4f65b56db/ortools/sat/python/cp_model.py#L468-L469 (and also in __rmul__)
most pedantically that could return LinearExpr | Literal[0], less precisely perhaps LinearExpr | int or similar variation

Not sure the Literal[0] is accepted with older python version ( i believe we support 3.8+).
I pushed a fix for mul and rmul, as well as missing ValueType.

I reverted the ValueType hack. It breaks our code.
Note that while mypy complains heavily, especially with arithmetic and SupportComplex, pytype validates everything I tried flawlessly.

pybind11::class_(m, "Domain", DOC(operations_research, Domain))
.def_static("all_values", &Domain::AllValues,
DOC(operations_research, Domain, AllValues))
.def_static("from_values", &Domain::FromValues,
DOC(operations_research, Domain, FromValues), arg("values"))
.def_static("from_intervals", &Domain::FromVectorIntervals,
DOC(operations_research, Domain, FromVectorIntervals),
arg("intervals"))
.def_static("from_flat_intervals", &Domain::FromFlatIntervals,
DOC(operations_research, Domain, FromFlatIntervals),
Laurent Perron | Operations Research | @.*** | (33) 1 42 68 53
00
Le lun. 11 mars 2024 à 16:57, proshkin @.***> a écrit :

There is also an issue with Domain class signature:
It looks like methods like FromFlatInterval are classmethods, but they
defined as regular methods in sorted_interval_list.py
If I do Domain.FromFlatIntervals([0, 2, 5, 5, 8, 10]), I got "Parameter
'flat_intervals' is unfilled' warning, but it works fine.
I can do Domain(0,0).FromFlatIntervals([0, 2, 5, 5, 8, 10]) to get rid of
the warning, but it is suboptimal.
On Mon, Mar 11, 2024 at 5:04 AM Laurent Perron @.***>
wrote:

Closed #4120 https://github.com/google/or-tools/issues/4120 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/4120#event-12072584547, or
unsubscribe
<
https://github.com/notifications/unsubscribe-auth/ABXB25XPTGMSEA6DK5W55MDYXWMWRAVCNFSM6AAAAABEOB37TCVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJSGA3TENJYGQ2TINY>

.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

—
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/4120#issuecomment-1988783628,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUPL3MMR64D5NGW6PUMJLDYXXH75AVCNFSM6AAAAABEOB37TCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBYG44DGNRSHA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

Why are they not static in sorted_interval_list.py?
On Mon, Mar 11, 2024 at 9:00 AM Laurent Perron @.***>
wrote:

pybind11::class_(m, "Domain", DOC(operations_research, Domain))
.def_static("all_values", &Domain::AllValues,
DOC(operations_research, Domain, AllValues))
.def_static("from_values", &Domain::FromValues,
DOC(operations_research, Domain, FromValues), arg("values"))
.def_static("from_intervals", &Domain::FromVectorIntervals,
DOC(operations_research, Domain, FromVectorIntervals),
arg("intervals"))
.def_static("from_flat_intervals", &Domain::FromFlatIntervals,
DOC(operations_research, Domain, FromFlatIntervals),
Laurent Perron | Operations Research | @.*** | (33) 1 42 68 53
00
Le lun. 11 mars 2024 à 16:57, proshkin @.***> a écrit :

There is also an issue with Domain class signature:
It looks like methods like FromFlatInterval are classmethods, but they
defined as regular methods in sorted_interval_list.py
If I do Domain.FromFlatIntervals([0, 2, 5, 5, 8, 10]), I got "Parameter
'flat_intervals' is unfilled' warning, but it works fine.
I can do Domain(0,0).FromFlatIntervals([0, 2, 5, 5, 8, 10]) to get rid
of
the warning, but it is suboptimal.
On Mon, Mar 11, 2024 at 5:04 AM Laurent Perron @.***>
wrote:

Closed #4120 https://github.com/google/or-tools/issues/4120 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/4120#event-12072584547,
or
unsubscribe
<

https://github.com/notifications/unsubscribe-auth/ABXB25XPTGMSEA6DK5W55MDYXWMWRAVCNFSM6AAAAABEOB37TCVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJSGA3TENJYGQ2TINY>

.
You are receiving this because you are subscribed to this
thread.Message
ID: @.***>

—
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/4120#issuecomment-1988783628,

or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/ACUPL3MMR64D5NGW6PUMJLDYXXH75AVCNFSM6AAAAABEOB37TCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBYG44DGNRSHA>

.
You are receiving this because you modified the open/close state.Message
ID: @.***>

—
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/4120#issuecomment-1988790001,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABXB25WPXPKCAH2T4GSVTT3YXXIK3AVCNFSM6AAAAABEOB37TCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBYG44TAMBQGE
.
You are receiving this because you commented.Message ID:
@.***>

There are no sorted_interval_list.py
Laurent Perron | Operations Research | @.*** | (33) 1 42 68 53
00
Le lun. 11 mars 2024 à 17:07, proshkin @.***> a écrit :

Why are they not static in sorted_interval_list.py?
On Mon, Mar 11, 2024 at 9:00 AM Laurent Perron @.***>
wrote:

pybind11::class_(m, "Domain", DOC(operations_research, Domain))
.def_static("all_values", &Domain::AllValues,
DOC(operations_research, Domain, AllValues))
.def_static("from_values", &Domain::FromValues,
DOC(operations_research, Domain, FromValues), arg("values"))
.def_static("from_intervals", &Domain::FromVectorIntervals,
DOC(operations_research, Domain, FromVectorIntervals),
arg("intervals"))
.def_static("from_flat_intervals", &Domain::FromFlatIntervals,
DOC(operations_research, Domain, FromFlatIntervals),
Laurent Perron | Operations Research | @.*** | (33) 1 42 68 53
00
Le lun. 11 mars 2024 à 16:57, proshkin @.***> a écrit :

There is also an issue with Domain class signature:
It looks like methods like FromFlatInterval are classmethods, but they
defined as regular methods in sorted_interval_list.py
If I do Domain.FromFlatIntervals([0, 2, 5, 5, 8, 10]), I got
"Parameter
'flat_intervals' is unfilled' warning, but it works fine.
I can do Domain(0,0).FromFlatIntervals([0, 2, 5, 5, 8, 10]) to get rid
of
the warning, but it is suboptimal.
On Mon, Mar 11, 2024 at 5:04 AM Laurent Perron @.***>
wrote:

Closed #4120 https://github.com/google/or-tools/issues/4120 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/4120#event-12072584547,
or
unsubscribe
<

https://github.com/notifications/unsubscribe-auth/ABXB25XPTGMSEA6DK5W55MDYXWMWRAVCNFSM6AAAAABEOB37TCVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJSGA3TENJYGQ2TINY>

.
You are receiving this because you are subscribed to this
thread.Message
ID: @.***>

—
Reply to this email directly, view it on GitHub
<
https://github.com/google/or-tools/issues/4120#issuecomment-1988783628>,

or unsubscribe
<

https://github.com/notifications/unsubscribe-auth/ACUPL3MMR64D5NGW6PUMJLDYXXH75AVCNFSM6AAAAABEOB37TCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBYG44DGNRSHA>

.
You are receiving this because you modified the open/close
state.Message
ID: @.***>

—
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/4120#issuecomment-1988790001,

or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/ABXB25WPXPKCAH2T4GSVTT3YXXIK3AVCNFSM6AAAAABEOB37TCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBYG44TAMBQGE>

.
You are receiving this because you commented.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/4120#issuecomment-1988805154,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUPL3P6MKARNOPY5AVTNTTYXXJEHAVCNFSM6AAAAABEOB37TCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBYHAYDKMJVGQ
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

encoding: utf-8
module ortools.util.python.sorted_interval_list
from C:\Users...\ortools\util\python\sorted_interval_list.cp312-win_amd64.pyd
by generator 1.147
no doc
imports
import pybind11_builtins as __pybind11_builtins
class Domain(__pybind11_builtins.pybind11_object):
....
def FromFlatIntervals(self, flat_intervals, p_int=None): # real
signature unknown; restored from doc
"""
FromFlatIntervals(flat_intervals: List[int]) ->
ortools.util.python.sorted_interval_list.Domain
This method is available in Python, Java and .NET. It allows building
a Domain object from a flattened list of intervals (long[] in Java and
.NET, [0, 2, 5, 5, 8, 10] in python).
"""
pass

On Mon, Mar 11, 2024 at 9:21 AM Laurent Perron @.***>
wrote:

There are no sorted_interval_list.py
Laurent Perron | Operations Research | @.*** | (33) 1 42 68 53
00
Le lun. 11 mars 2024 à 17:07, proshkin @.***> a écrit :

Why are they not static in sorted_interval_list.py?
On Mon, Mar 11, 2024 at 9:00 AM Laurent Perron @.***>
wrote:

pybind11::class_(m, "Domain", DOC(operations_research,
Domain))
.def_static("all_values", &Domain::AllValues,
DOC(operations_research, Domain, AllValues))
.def_static("from_values", &Domain::FromValues,
DOC(operations_research, Domain, FromValues), arg("values"))
.def_static("from_intervals", &Domain::FromVectorIntervals,
DOC(operations_research, Domain, FromVectorIntervals),
arg("intervals"))
.def_static("from_flat_intervals", &Domain::FromFlatIntervals,
DOC(operations_research, Domain, FromFlatIntervals),
Laurent Perron | Operations Research | @.*** | (33) 1 42 68 53
00
Le lun. 11 mars 2024 à 16:57, proshkin @.***> a écrit :

There is also an issue with Domain class signature:
It looks like methods like FromFlatInterval are classmethods, but
they
defined as regular methods in sorted_interval_list.py
If I do Domain.FromFlatIntervals([0, 2, 5, 5, 8, 10]), I got
"Parameter
'flat_intervals' is unfilled' warning, but it works fine.
I can do Domain(0,0).FromFlatIntervals([0, 2, 5, 5, 8, 10]) to get
rid
of
the warning, but it is suboptimal.
On Mon, Mar 11, 2024 at 5:04 AM Laurent Perron @.***>
wrote:

Closed #4120 https://github.com/google/or-tools/issues/4120 as
completed.
—
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/4120#event-12072584547,

or

unsubscribe
<

https://github.com/notifications/unsubscribe-auth/ABXB25XPTGMSEA6DK5W55MDYXWMWRAVCNFSM6AAAAABEOB37TCVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJSGA3TENJYGQ2TINY>

.
You are receiving this because you are subscribed to this
thread.Message
ID: @.***>

—
Reply to this email directly, view it on GitHub
<
https://github.com/google/or-tools/issues/4120#issuecomment-1988783628>,

or unsubscribe
<

https://github.com/notifications/unsubscribe-auth/ACUPL3MMR64D5NGW6PUMJLDYXXH75AVCNFSM6AAAAABEOB37TCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBYG44DGNRSHA>

.
You are receiving this because you modified the open/close
state.Message
ID: @.***>

—
Reply to this email directly, view it on GitHub
<
https://github.com/google/or-tools/issues/4120#issuecomment-1988790001>,

or unsubscribe
<

https://github.com/notifications/unsubscribe-auth/ABXB25WPXPKCAH2T4GSVTT3YXXIK3AVCNFSM6AAAAABEOB37TCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBYG44TAMBQGE>

.
You are receiving this because you commented.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/4120#issuecomment-1988805154,

or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/ACUPL3P6MKARNOPY5AVTNTTYXXJEHAVCNFSM6AAAAABEOB37TCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBYHAYDKMJVGQ>

.
You are receiving this because you modified the open/close state.Message
ID: @.***>

—
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/4120#issuecomment-1988842865,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABXB25XHZ2S75OEEDTCOGTLYXXKW3AVCNFSM6AAAAABEOB37TCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBYHA2DEOBWGU
.
You are receiving this because you commented.Message ID:
@.***>

This is a pyi file.
Le lun. 11 mars 2024, 17:58, proshkin @.***> a écrit :

encoding: utf-8
module ortools.util.python.sorted_interval_list
from
C:\Users...\ortools\util\python\sorted_interval_list.cp312-win_amd64.pyd
by generator 1.147
no doc
imports
import pybind11_builtins as __pybind11_builtins
class Domain(__pybind11_builtins.pybind11_object):
....
def FromFlatIntervals(self, flat_intervals, p_int=None): # real
signature unknown; restored from doc
"""
FromFlatIntervals(flat_intervals: List[int]) ->
ortools.util.python.sorted_interval_list.Domain
This method is available in Python, Java and .NET. It allows building
a Domain object from a flattened list of intervals (long[] in Java and
.NET, [0, 2, 5, 5, 8, 10] in python).
"""
pass
On Mon, Mar 11, 2024 at 9:21 AM Laurent Perron @.***>
wrote:

There are no sorted_interval_list.py
Laurent Perron | Operations Research | @.*** | (33) 1 42 68 53
00
Le lun. 11 mars 2024 à 17:07, proshkin @.***> a écrit :

Why are they not static in sorted_interval_list.py?
On Mon, Mar 11, 2024 at 9:00 AM Laurent Perron @.***>
wrote:

pybind11::class_(m, "Domain", DOC(operations_research,
Domain))
.def_static("all_values", &Domain::AllValues,
DOC(operations_research, Domain, AllValues))
.def_static("from_values", &Domain::FromValues,
DOC(operations_research, Domain, FromValues), arg("values"))
.def_static("from_intervals", &Domain::FromVectorIntervals,
DOC(operations_research, Domain, FromVectorIntervals),
arg("intervals"))
.def_static("from_flat_intervals", &Domain::FromFlatIntervals,
DOC(operations_research, Domain, FromFlatIntervals),
Laurent Perron | Operations Research | @.*** | (33) 1 42 68 53
00
Le lun. 11 mars 2024 à 16:57, proshkin @.***> a écrit :

There is also an issue with Domain class signature:
It looks like methods like FromFlatInterval are classmethods, but
they
defined as regular methods in sorted_interval_list.py
If I do Domain.FromFlatIntervals([0, 2, 5, 5, 8, 10]), I got
"Parameter
'flat_intervals' is unfilled' warning, but it works fine.
I can do Domain(0,0).FromFlatIntervals([0, 2, 5, 5, 8, 10]) to get
rid
of
the warning, but it is suboptimal.
On Mon, Mar 11, 2024 at 5:04 AM Laurent Perron @.***>
wrote:

Closed #4120 https://github.com/google/or-tools/issues/4120
as
completed.
—
Reply to this email directly, view it on GitHub
<
https://github.com/google/or-tools/issues/4120#event-12072584547>,

or

unsubscribe
<

https://github.com/notifications/unsubscribe-auth/ABXB25XPTGMSEA6DK5W55MDYXWMWRAVCNFSM6AAAAABEOB37TCVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJSGA3TENJYGQ2TINY>

.
You are receiving this because you are subscribed to this
thread.Message
ID: @.***>

—
Reply to this email directly, view it on GitHub
<
https://github.com/google/or-tools/issues/4120#issuecomment-1988783628>,

or unsubscribe
<

https://github.com/notifications/unsubscribe-auth/ACUPL3MMR64D5NGW6PUMJLDYXXH75AVCNFSM6AAAAABEOB37TCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBYG44DGNRSHA>

.
You are receiving this because you modified the open/close
state.Message
ID: @.***>

—
Reply to this email directly, view it on GitHub
<
https://github.com/google/or-tools/issues/4120#issuecomment-1988790001>,

or unsubscribe
<

https://github.com/notifications/unsubscribe-auth/ABXB25WPXPKCAH2T4GSVTT3YXXIK3AVCNFSM6AAAAABEOB37TCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBYG44TAMBQGE>

.
You are receiving this because you commented.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
<
https://github.com/google/or-tools/issues/4120#issuecomment-1988805154>,

or unsubscribe
<

https://github.com/notifications/unsubscribe-auth/ACUPL3P6MKARNOPY5AVTNTTYXXJEHAVCNFSM6AAAAABEOB37TCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBYHAYDKMJVGQ>

.
You are receiving this because you modified the open/close
state.Message
ID: @.***>

—
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/4120#issuecomment-1988842865,

or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/ABXB25XHZ2S75OEEDTCOGTLYXXKW3AVCNFSM6AAAAABEOB37TCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBYHA2DEOBWGU>

.
You are receiving this because you commented.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/google/or-tools/issues/4120#issuecomment-1988956050,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACUPL3JISXXMDB42IDVPRMLYXXPFDAVCNFSM6AAAAABEOB37TCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSOBYHE2TMMBVGA
.
You are receiving this because you modified the open/close state.Message
ID: @.***>

This is a bit g of mypy. It was fixed in 1.9. I need to check it is really fixed.

On 9.10, mypy 1.10.0, code such as:
from ortools.sat.python import cp_model
...
status = solver.Solve(self.planner_model.model)
if status is cp_model.INFEASIBLE:
    raise RuntimeError("Infeasible")

is flagged with this typing error:
error: Non-overlapping identity check (left operand type: "CpSolverStatus", right operand type: "ValueType")  [comparison-overlap]


aiextract and parametric types

Not quite sure what's up with this example, but aiextract doesn't seem to know what to do here:
using PromptingTools

# Return types
abstract type AbstractThing end
struct Dolphin <: AbstractThing end
struct NotDolphin <: AbstractThing end

# Wrapper type
struct ThingHolder{C<:AbstractThing}
    dolphin_or_not::C
end

# Wrapper type that works
struct ThingHolder2
    dolphin_or_not::Union{Dolphin,NotDolphin}
end

# Call aiextract
tester = "Look it's probably a dolphin."
a = aiextract(tester; return_type=ThingHolder)
b = aiextract(tester; return_type=ThingHolder2)

In both cases, I get a Dict:
julia> a.content
Dict{Symbol, Any} with 1 entry:
  :dolphin_or_not => "dolphin"

julia> b.content
Dict{Symbol, Any} with 1 entry:
  :dolphin_or_not => "dolphin"

I think my expectation here would be that I'd get a ThingHolder, but the fallback seems to be to a Dict. Anyway thought I'd type this one up in case it's unexpected behavior.
Here's the warnings indicating that the JSON could not be read:
┌ Warning: There was an error parsing the response: ArgumentError("invalid JSON at byte position 19 while parsing type AbstractThing: ExpectedOpeningObjectChar\n{\"dolphin_or_not\":\"dolphin\"}\n"). Using the raw response instead.
└ @ PromptingTools ~/.julia/packages/PromptingTools/nuNH1/src/llm_openai.jl:649
[ Info: Tokens: 76 @ Cost: $0.0 in 0.8 seconds
PromptingTools.DataMessage(Dict with keys: dolphin_or_not)

┌ Warning: There was an error parsing the response: MethodError(JSON3.read, (StructTypes.SingletonType(), UInt8[0x7b, 0x22, 0x64, 0x6f, 0x6c, 0x70, 0x68, 0x69, 0x6e, 0x5f, 0x6f, 0x72, 0x5f, 0x6e, 0x6f, 0x74, 0x22, 0x3a, 0x22, 0x64, 0x6f, 0x6c, 0x70, 0x68, 0x69, 0x6e, 0x22, 0x7d], 19, 28, 0x22, NotDolphin), 0x0000000000007b0a). Using the raw response instead.
└ @ PromptingTools ~/.julia/packages/PromptingTools/nuNH1/src/llm_openai.jl:649
[ Info: Tokens: 78 @ Cost: $0.0 in 0.7 seconds
PromptingTools.DataMessage(Dict with keys: dolphin_or_not)

I’ll have a look later but there are two challenges:

not sure my simple JSON schema generator inspects parametric types
even if it did, I leverage JSON3 reading into types via StructTypes and I think you would need to define the rule how pick the subtypes of the abstract thing (eg, a subfield that tells you which one to use). See: https://juliadata.org/StructTypes.jl/stable/#StructTypes.AbstractType

It might be easier to enable passing a vector of types and pass both to the “tools”. At the moment, I fix it to one only.


Fix stale github action

change to run at 7AM US west time, 11PM China time, 3PM UTC
run more operations per run (from 30 to 100)
From old issues/PRs to new ones

alluxio-bot, merge this please


how to use ground_truth?

Hi, It is a very nice job !
here I have some confusion  about the  usage of  ground_truth.
I want to  augment the  image and it's masks(here every image has two different mask ) simultaneously,
How can I realize the idea with ground_truth?
show  me some example will be excellent
thank  you

Hi @zhaoxin111, yes this is something I am working on - it's been requested a few times so it seems to be functionality that many people would find useful. So I'll try to make it a priority and get it added soon.

Hello @mdbloice ,
I am working on a image segmentation task where each image has only one mask. I succeed augmenting both data and masks by adding p.ground_truth("ground_truth_path") and then p.sample(N), which will automatically produce the augmented images and masks and store them in "output" folder.
However, when I try "Augmentor.Pipeline.keras_generator()". It seems that this function can only return a generator with a one-hot encoded label as image labels instead of a whole mask.
It would be very kind of you to improve this feature in "keras_generator()" also. Thank you!

Is there any way to transform a pair of images (image + ground truth) on the fly without saving them on a disk?

@tamkaho
Hello tamkaho,  augmentation applied both on images and ground truth(without saving them to disk) can be achieved by keras.preprocessing.image.ImageDataGenerator(), and then use flow() or flow_from_directory() method. A detail example can be found in keras documents, which shows how to do the augmentation: https://keras.io/preprocessing/image/
I have already implemented this in my own project. But I would like to use Augmentor library to do it , ,because for my own dataset, this augmentation effect is better than the keras built-in library I mentioned above.

Hello,  @ChenWentai,
I am also want to augment the image and its ground truth annotation simultaneously, but the ground truth is bounding box for me .  If so, can you share me your processing procedure about augmenting both the image and its ground truth annotation as refer? Thanks in advance!

@ChenWentai Hi there, In your ground_truth directory, does it contain images and bounding box coordinate (mask) text files? It sounds like you applied transformation to image and also to the location of bounding box (mask) in augmentation. Can you elaborate?

@blateyang @jaelim
Hello, I was using Warwick-QU dataset and my ground truth is the labeling of objects: 0 for background and 1 for objects. So it does not contain any bounding box. Sorry I cannot help in this case!

It doesn't matter. I'll try to make bbox ground_truth by myself.
At 2018-09-09 05:09:24, "CHEN Wentai"<EMAIL_ADDRESS>wrote:
@blateyang@jaelim
Hello, I was using Warwick-QU dataset and my ground truth is the labeling of objects: 0 for background and 1 for objects. So it does not contain any bounding box. Sorry I cannot help in this case!
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or mute the thread.

@ChenWentai @blateyang I found out what mask (an example of this is shown in the repo) is; it is not a bounding-box (as in object detection model) as I incorrectly assumed.

@ChenWentai
my annotation was generated using makesense.ai which is formatted as coco or vgg json format, is that possible to user ground_truth to augment my own dataset?


consider presenting Age column vs Created At column

fixed by https://github.com/cppforlife/knctl/commit/e64c7db1761a60aa89623262466f5cb757655474


Remove ManagedZone API

What
Originally ManagedZone was thought of in the context of multiple tenants where we might control the credentals and the zones. This is no longer the case. ManagedZone as remained in place largely as a reference to a credential to the underlying DNS provider. We do not need this indirection. Additionally the ManagedZone API adds some confusion around what should happen when you delete a managed zone resource.
How
Remove the ManagedZone resource and update DNSPolicy to use a label selector for secrets in that namespace.  Each secret found will reference a particular zone and particular provider and the dns record controller will propigate these records into each valid provider

[ ] Add ProviderRef label selector to API
[ ] Remove all references to ManagedZone
[ ] Update all existing documentation
[ ] Update existing tests
[ ] Bump API version to v1alpha2

@mikenairn would appreciate any addiitional thoughts you have on this one.

https://github.com/Kuadrant/kuadrant-operator/pull/793


Play with vulns V2

⛔ Snyk has found 1 issues.

Icon
Severity
Issues

Critical
0

High
0

Medium
1

Low
0

✅ security/snyk check is completed. No issues were found. (View Details)
⛔ code/snyk check is completed. Snyk found 1 issues. (View Details)


Add pc.AnimationComponent#getAnimations

Currently, there is no way to get a list of animations made available in pc.AnimationComponent. This PR fixes that.
I confirm I have signed the Contributor License Agreement.

@slimbuck This is something you should greenlight since you're in the process of rewriting the animation system. Over to you. 😄

This PR is no longer necessary due to https://github.com/playcanvas/engine/pull/1967#issuecomment-609062878


Add more decent Documentation

Is your feature request related to a problem? Please describe.
Although having a auto-generated API documentation available it's always better to have a decent documentation, which makes onboarding (devx) much more comfortable. The better the documentation, the better the acceptance/adoption
Describe the solution you'd like
Add an examples folder with structured examples for most common use cases. BurstJS Examples can be a starting point, showing off how to use in vanilla js web sites, or modern apps like React, Svelte, and show how to use with NodeJS
[UPDATE]
Most probably, we will go with gitbook and establish a much more dev-friendly documentation

This will be done with gitbook.... changing title

wokring on docs now: https://signum-network.gitbook.io/signumjs/

Documentation has been migrated to the following platform:
https://docs.signum.network/signum/signumjs


Code for sum of two integers

Sum of integers.

Hi Manvi, you have been invited to our org as member, review code and help out !


Support other Testing Library packages

Depends on #1
Tests are currently only based on DOM Testing Library, but we may want to use libraries that support other environments, such as Puppeteer Testing Library

On second thought, now that Chrome supports step export, it's easier to use this extension with framework specific packages like React Testing Library. We can investigate those independently and packages that require different environments as part of #1.


Replace

Maybe you can replace $this->getOwningPlugin(); in line 82 to $this->owningPlugin->
On Commands.php

I think that can be changed with the next update

Bad idea, Bruh


oh-my-posh: 19.8.3 -> 19.9.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/jandedobbeleer/oh-my-posh/releases.
meta.description for oh-my-posh is: A prompt theme engine for any shell
meta.homepage for oh-my-posh is: https://ohmyposh.dev
meta.changelog for oh-my-posh is: https://github.com/JanDeDobbeleer/oh-my-posh/releases/tag/v19.9.0
Updates performed

Golang update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 19.9.0 with grep in /nix/store/78ddf6zsad6xdjxblisliab6dbfpysq5-oh-my-posh-19.9.0
found 19.9.0 in filename of file in /nix/store/78ddf6zsad6xdjxblisliab6dbfpysq5-oh-my-posh-19.9.0

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
oh-my-posh

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/78ddf6zsad6xdjxblisliab6dbfpysq5-oh-my-posh-19.9.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/189843931643ae4294116a04003651d5bcfa4383#oh-my-posh

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/78ddf6zsad6xdjxblisliab6dbfpysq5-oh-my-posh-19.9.0
ls -la /nix/store/78ddf6zsad6xdjxblisliab6dbfpysq5-oh-my-posh-19.9.0/bin

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

1 package built:

oh-my-posh

Maintainer pings
cc @lucperkins @urandom2 for testing.

@urandom2 looks inactive

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/jandedobbeleer/oh-my-posh/releases.
meta.description for oh-my-posh is: A prompt theme engine for any shell
meta.homepage for oh-my-posh is: https://ohmyposh.dev
meta.changelog for oh-my-posh is: https://github.com/JanDeDobbeleer/oh-my-posh/releases/tag/v19.11.2
Updates performed

Golang update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 19.11.2 with grep in /nix/store/bdrvgdvznr3lp22l583qcw0j6qhvcl4f-oh-my-posh-19.11.2
found 19.11.2 in filename of file in /nix/store/bdrvgdvznr3lp22l583qcw0j6qhvcl4f-oh-my-posh-19.11.2

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
oh-my-posh

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/bdrvgdvznr3lp22l583qcw0j6qhvcl4f-oh-my-posh-19.11.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/1f12e5982434f7e1fc139ba226589227d036427e#oh-my-posh

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/bdrvgdvznr3lp22l583qcw0j6qhvcl4f-oh-my-posh-19.11.2
ls -la /nix/store/bdrvgdvznr3lp22l583qcw0j6qhvcl4f-oh-my-posh-19.11.2/bin

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

1 package built:

oh-my-posh

Maintainer pings
cc @lucperkins @urandom2 for testing.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/jandedobbeleer/oh-my-posh/releases.
meta.description for oh-my-posh is: A prompt theme engine for any shell
meta.homepage for oh-my-posh is: https://ohmyposh.dev
meta.changelog for oh-my-posh is: https://github.com/JanDeDobbeleer/oh-my-posh/releases/tag/v19.11.4
Updates performed

Golang update

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 19.11.4 with grep in /nix/store/l9bdss783czl5djw8s12y8dng269wf59-oh-my-posh-19.11.4
found 19.11.4 in filename of file in /nix/store/l9bdss783czl5djw8s12y8dng269wf59-oh-my-posh-19.11.4

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

1 total rebuild path(s)

1 package rebuild(s)

First fifty rebuilds by attrpath
oh-my-posh

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/l9bdss783czl5djw8s12y8dng269wf59-oh-my-posh-19.11.4 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

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

Or:
nix build github:r-ryantm/nixpkgs/e53dda131a6ecd9dbf69b995c0d69005f6f16463#oh-my-posh

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/l9bdss783czl5djw8s12y8dng269wf59-oh-my-posh-19.11.4
ls -la /nix/store/l9bdss783czl5djw8s12y8dng269wf59-oh-my-posh-19.11.4/bin

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

1 package built:

oh-my-posh

Maintainer pings
cc @lucperkins @urandom2 for testing.

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

1 package built:

oh-my-posh


Breaks Nuxt-WindiCSS with Nuxt 3

Encountering a similar issue when trying to #178  combine nuxt-windicss and daisyui - this issue only crops up when transforming daisyui (as below). The difference here is that I am using Nuxt v3.
const { transform } = require("windicss/helpers");

module.exports = {
  plugins: [transform("kutty")],
};

The error message that crops up:
 ERROR  Cannot start nuxt:  Maximum call stack size exceeded                                                                                13:11:14

  at guessClassName (node_modules/windicss/lib/index.mjs:364:24)
  at node_modules/windicss/lib/index.mjs:367:67
  at Array.map (<anonymous>)
  at guessClassName (node_modules/windicss/lib/index.mjs:367:41)
  at node_modules/windicss/lib/index.mjs:367:67
  at Array.map (<anonymous>)
  at guessClassName (node_modules/windicss/lib/index.mjs:367:41)
  at node_modules/windicss/lib/index.mjs:367:67
  at Array.map (<anonymous>)
  at guessClassName (node_modules/windicss/lib/index.mjs:367:41)

Sample repo is here. Any way to overcome this issue?

@Destaq
I'm not sure if it's from Nuxt or Windi, there are issues about this (#1, #2) on Nuxt repo but they don't seem helpful for this case. You might want to open an issue on Windi repo or Nuxt repo.
I tested your project with Tailwind (instead of Windi) and everything works just fine.
Here's a sample project config with Nuxt3 and Tailwind:
https://github.com/saadeghi/daisyui/tree/master/examples/nuxtjs-v3

this issue is still present(without nuxt)
import { transform } from 'windicss/helpers'
import { defineConfig } from 'vite-plugin-windicss'

export default defineConfig({
    darkMode: false,
    theme: {
        extend: {
            colors: require('daisyui/colors/windi'),
        },
    },
    extract: {
        include: ['index.html', 'src/**/*.{vue,html,jsx,tsx}'],
        exclude: ['node_modules/**/*', '.git/**/*'],
    },
    plugins: [transform('daisyui')],
})

There are similar reports from Vite+Windi so I'm pretty sure the problem is from Windi itself.
I would suggest using Tailwind CSS 3.0 instead of Windi. Now with JIT mode, I'm not sure if there's an advantage of using Windi instead of Tailwind CSS 🤷♂

If anyone still experiencing this error try updating windicss to latest version, 3.5.1 fixed it for me

That's great! Thanks @old-zoomer