scipy imread has been deprecated since v1.0.0.

It might be worth considering to mantain/update the code for removed dependecies
I've got this error trying to use stylize.py on windows.
Restoring vars for relu5_1 from checkpoint models/relu5_1\model.ckpt-15004
W0702 06:19:34.308990  4512 deprecation.py:323] From C:\Program Files\Python37\lib\site-packages\tensorflow\python\training\saver.py:1276: checkpoint_exists (from tensorflow.python.training.checkpoint_management) is deprecated and will be removed in a future version.
Instructions for updating:
Use standard file APIs to check for files with this prefix.
Traceback (most recent call last):
File "stylize.py", line 126, in
main()
File "stylize.py", line 74, in main
content_img = get_img(content_fullpath)
File "C:\Users\JLNR\Documents\WCT-TF-master\utils.py", line 24, in get_img
img = scipy.misc.imread(src, mode='RGB')
AttributeError: module 'scipy.misc' has no attribute 'imread'
I'm going to try using an earlier version of scipy

Thanks for pointing that out. I unfortunately don't have the time to update the repo right now, but at some point in the next few months I'd like to migrate it to TF 2.0 and clean up other bits like this along the way.

The only TF2 implementation I found didn't work so well, so I tried to make this one work, using Anaconda and python v3.8, a custom environment to install all these old package versions. The right combination is tensorflow 1.2.1 with keras 2.0.0... and the changes below.
You can easily replace scipy.misc.imread and scipy.misc.imresize in the code (mainly sytlize.py and utils.py, I didn't try the camera thing).
Here are the BEFORE and AFTER functions
for deprecated scipy.misc.imread:
import imageio
(...)
#BEFORE img = scipy.misc.imread(src, mode='RGB')
img = imageio.imread(src, as_gray=False, pilmode='RGB')

You'll also have problems with the deprecated image resize function from scipy.misc !
from skimage import transform
(...)
#BEFORE style_img_resized = scipy.misc.imresize(style_img, (stylized_rgb.shape[0], stylized_rgb.shape[0]))
style_img_resized = transform.resize(style_img, (stylized_rgb.shape[0], stylized_rgb.shape[0]), anti_aliasing=True)

Made me search a little as I don't program in python, you'll have a lot of deprecation warnings when using the scripts, but it works :)
The only thing that didn't work with the image resize function is when you're trying to use the --ss-stride command-line option. Must be something like a conversion between integers and floats that I don't understand...

The only TF2 implementation I found didn't work so well, so I tried to make this one work, using Anaconda and python v3.8, a custom environment to install all these old package versions. The right combination is tensorflow 1.2.1 with keras 2.0.0... and the changes below.
You can easily replace scipy.misc.imread and scipy.misc.imresize in the code (mainly sytlize.py and utils.py, I didn't try the camera thing).
Here are the BEFORE and AFTER functions
for deprecated scipy.misc.imread:
import imageio
(...)
#BEFORE img = scipy.misc.imread(src, mode='RGB')
img = imageio.imread(src, as_gray=False, pilmode='RGB')

You'll also have problems with the deprecated image resize function from scipy.misc !
from skimage import transform
(...)
#BEFORE style_img_resized = scipy.misc.imresize(style_img, (stylized_rgb.shape[0], stylized_rgb.shape[0]))
style_img_resized = transform.resize(style_img, (stylized_rgb.shape[0], stylized_rgb.shape[0]), anti_aliasing=True)

Made me search a little as I don't program in python, you'll have a lot of deprecation warnings when using the scripts, but it works :)
The only thing that didn't work with the image resize function is when you're trying to use the --ss-stride command-line option. Must be something like a conversion between integers and floats that I don't understand...


If Grafana is unable to connect to the datasource and has public mode enabled, zabbix credentials are viewable in plain text

I'm submitting a ...

[X] Bug report
[ ] Feature requestdd
[ ] Question / Support request

If you don't send bug report, remove sections below
For bug report please include this information:
Software Versions

Grafana
Zabbix
Grafana-Zabbix Plugin

4.2.0
3.2.7
3.3.0

What did you do?
If Grafana is available in public mode (auth.anonymous) and Zabbix is unable to be reached, there will be an error message (red flag in top left corner of panels that use Zabbix. If you click on this panel, and go to the request tab, it is possible to see the output of the request to the zabbix API, which contains cleartext usernames and passwords for the zabbix instance.
What was the expected result?
Cleartext usernames and passwords should not be visible, even during a fault.
What happened instead?
Cleartext usernames and passwords, as well as API URIs are available in cleartext during an error.

That's how Zabbix api works, unfortunately. See #380 for details. Good solution is moving auth-related queries to backend, but I can't do it until we implement backend plugins in Grafana.


Error messages like "Page not found" not documented

We occasionally see Streamlit error messages pop up like this one:

Page not found
The page that you have requested does not seem to exist.
Running the app's main page.

This doesn't make sense to our app's users, and we can't find anything in the documentation about when this error would be reported. That makes it hard to explain to our users what went wrong, and even harder to do anything to avoid it in future.

Thanks for the suggestion! I'm compiling a list of error messages to create a comprehensive troubleshooting section, so I'll be sure to add this.

This error should also be explained in the MPA guide.

Since the user-facing error is currently documented as an image, I've added alt text to make sure it is discoverable by searching. We have new content coming out for multipage apps which will come with a lot of new content, but for this issue in the current state, making sure the current documentation will come up in search should address the issue.


dmenu bind?

Hi! Do you have a way to get the paste in the clipboard like clipd menu?
https://mpov.timmorgan.org/clipboard-history-in-sway-window-manager/

Thanks!


Examples1

begin building template AWS IAM policies to use with control plane pattern

Great work!  Thanks for the contribution.


Update aktualizr example secondary config path

After https://github.com/advancedtelematic/aktualizr/pull/973
tested: built core-image-minimal for qemu

 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.

aktualizr fails for me with Could not read secondary configs from "/usr/lib/sota/secondaries": not a directory. Looks like a bug in aktualizr, sorry.

@patrickvacek thanks! My test was wrong because I kept aktualizr-examples in IMAGES_INSTALL_append. I believe it's only a problem with the recipe, thankfully.

Added a cleanup commit which depends on https://github.com/advancedtelematic/aktualizr/pull/977


Remove Julia 1.3 from tests

Julia 1.3 doesn't need to be in the tests.
It was added when Julia 1.3 was the latest version and I didn't know that the version '1' would always use the latest release.

A more aggressive approach could be to remove the latest release, i.e. '1', from the tests as well. In theory, the builds on 1.0 and nightly should encompass everything in between.

A more aggressive approach could be to remove the latest release, i.e. '1', from the tests as well. In theory, the builds on 1.0 and nightly should encompass everything in between.

In theory.... sometimes nightly does weird stuff though.

One thing we could consider would be switching from Travis to GitHub Actions. Usually GH Actions is much faster.
If you're interested, let me know - I'm happy to make a PR to set up GH Actions for CI testing on this repo.

That sounds great!
I'm curious about GH Actions too, so seeing it applied in a setup I'm previously familiar with would be a useful learning opportunity for me.


mode solver improvements

How can we use different mode solvers to ?

[ ] solve waveguides arbitrary index profiles
[ ] solve multi-layer waveguides with sidewall angles
[ ] compute mode leakage to substrate
[ ] Improve femwell interface from femwell to gdsfactory on docs
[ ] From waveguide cross_section simulate modes

Here is a nice mode solver API
it would be great to port this easy API to femwell and tidy3d mode solvers
@HelgeGehring
@flaport

duplicate of https://github.com/gdsfactory/gplugins/issues/270 ?

yes, i closed the other one


Functions SEARCH and VERIFY

Functions SEARCH and VERIFY like in PL/1.

Functions SEARCH and VERIFY like in PL/1.


Make library a mandatory field for questions

Fixes #136

Pull Request Test Coverage Report for Build 1035

1 of 1 (100.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 99.172%

Totals

Change from base Build 1032:
0.0%

Covered Lines:
479

Relevant Lines:
483

💛  - Coveralls


Theoretical slowdown using batch accumulation

As I understand batch accumulation is just a alias for iter_size parameter in Caffe solver.
https://github.com/NVIDIA/DIGITS/blob/6b8995d89abe300f0a7bbdc11e53ce4aff5671c5/digits/model/tasks/caffe_train.py#L741
As I can see here result of for axample using batch size of 16 and batch size 4 and iter_size 4 should be numerically equivalent? and other settings of solver as learning rate and etc. should not affect result?
What is theoretical slowdown when we use batch accumulation?
How it's work internally? Does it store each batch result of forward pass in GPU memory (iter_size times in total), then average\merge batches to create one batch of batch_size size and then do backward pass and update using one batch of batch_size size ?

I'll refer you to the pull request that added the feature: https://github.com/BVLC/caffe/pull/1977


Fix the assertion within the 'Get Started with Modeling' page.

Description

This change corrects the assertion that Beth does not have the can_share relationship with document:1, even though she is an editor of it.

The relationship definition.

Relationship tuples.

Old assertion.

Fixed assertion.
Review Checklist

[x] I have clicked on "allow edits by maintainers".
[ ] I have added documentation for new/changed functionality in this PR or in a PR to openfga.dev [Provide a link to any relevant PRs in the references section above]
[x] The correct base branch is being used, if not main
[ ] I have added tests to validate that the change in functionality is working as expected

Thank you @brunicius for your contribution. Can you sign the CLA so that we can take the change? Thank you.

You're welcome, I just signed the CLA.


[WFLY-15918] Use java.transaction.xa JPMS module together with java.sql module where necessary

https://issues.redhat.com/browse/WFLY-15918

In upcomming JBoss Modules 2.0.2.Final upgrade all java.transaction.xa exported packages will be removed from java.sql module to be in alignment with Java's JPMS definitions. Thus it is necessary to properly reference java.transaction.xa instead of java.sql.

@fl4via or @bstansberry please review


how to get return value from lua?

See the official lua reference manual, in particular around stack manipulation and lua_call. https://www.lua.org/manual/5.1/manual.html


feat(ts): Add new treesitter standard captures

Description
nvim-treesitter has changed many of its standard captures to better align with the upstream tree-sitter and Helix captures. This adds the new captures whilst leaving the old ones so as not to break back-compatibility.
See nvim-treesitter@1ae9b0e
Screenshots

N/A?

Thanks a lot for this! 🙌
Cross referencing https://github.com/sainnhe/gruvbox-material/issues/194


chore: some housekeeping

Adds some info to client script
Moves renovate to github folder and kills duplicate
Removes multiple scripts in favour of single one

:tada: This PR is included in version 4.1.0-rc.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

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


quickfix to fall back to closest error in the file

[ ] All compiled assets are included (atom packages are git tags and hence the built files need to be a part of the source control)
See https://github.com/TypeStrong/atom-typescript/issues/1191.
This PR makes quickfix to fall back to the closest error in the current file in case no error is on the position of the cursor.

👍


Show overture sequence only once

Currently, the students see the overture sequence after each login, until the first mission is released.
We should change this such that students see the overture sequence only after their first login. Subsequent logins should take them to their room, regardless whether the first mission is released already, or not.

Handled by #667
and beyond SWS3012 by the Engagement team.


Integration into an Angular app

I have zero knowledge of vue.js and I want to use this library into my angular project. Is this doable? if so, any idea how difficult or easy it is just to be able to use the components in this library into a blank angular app?

@LelouchOfTheResurrection technically this is possible but not something we officially support. I'm going to close this issue. I'd suggest stackoverflow as a good place to find some answers.


Publish docker image in GitHub docker registry

It would be nice that docker images would be published on GitHub Packages, so backend developers wouldn't need to install Java, etc to be able to test integration with EFGS on local dev machine.
Currently to build docker images on a Mac you have to:
brew tap AdoptOpenJDK/openjdk
brew cask install adoptopenjdk11
export JAVA_HOME=$(/usr/libexec/java_home -v11)
mvn clean install -P docker
bash -c "cd target/docker && docker-compose up --build"

It would be much easier to just:
# docker pull docker.pkg.github.com/eu-federation-gateway-service/efgs-federation-gateway/backend:latest
docker-compose up --build

I could try to make a PR with GitHub Action workflow changes if this is something you are interested.
Also currently as I understand that without F5 LB there is no way to start EFGS with mTLS locally on developer machine. I have created some k8s configs with envoy proxy (the only one I could find that supports client cert sha256 fingerprints in headers) and it looks like it could be ok for dev/test environment for us - https://gist.github.com/ebeigarts/e129f7bd44ef06fbbb5ce028af8c9cdf

@ebeigarts Thank you for the suggestion and the offer to create a PR. Our devs will review this, please give us some time to get back to you.

@ebeigarts May I ask you to create the PR? Our developers will review your specific suggestions and provide feedback. Thank you!


Async store

Fixes # (issue)
Changes
Please provide a brief description of the changes here.
For significant contributions please make sure you have completed the following items:

[ ] CHANGELOG.md updated for non-trivial changes
[ ] Unit tests have been added
[ ] Changes in public API reviewed

Codecov Report

Merging #85 (6286691) into main (c10bc36) will increase coverage by 0.06%.
The diff coverage is 77.39%.

:exclamation: Current head 6286691 differs from pull request most recent head 40fe87c. Consider uploading reports for the commit 40fe87c to get more accurate results

@@            Coverage Diff             @@
##             main      #85      +/-   ##
==========================================
+ Coverage   92.31%   92.36%   +0.06%
==========================================
  Files         198      202       +4
  Lines        7118     7170      +52
==========================================
+ Hits         6570     6622      +52
  Misses        548      548

Impacted Files
Coverage Δ

.../opentelemetry/sdk/metrics/measurement_processor.h
0.00% <ø> (ø)

...entelemetry/sdk/metrics/state/attributes_hashmap.h
93.94% <ø> (ø)

...etry/sdk/metrics/aggregation/default_aggregation.h
14.71% <27.78%> (+14.71%)
:arrow_up:

...ntelemetry/sdk/metrics/state/sync_metric_storage.h
66.67% <50.00%> (+13.73%)
:arrow_up:

...telemetry/sdk/metrics/state/async_metric_storage.h
93.34% <93.34%> (ø)

...nclude/opentelemetry/sdk/metrics/observer_result.h
100.00% <100.00%> (ø)

sdk/test/metrics/async_metric_storage_test.cc
100.00% <100.00%> (ø)

sdk/test/metrics/observer_result_test.cc
100.00% <100.00%> (ø)

sdk/test/metrics/sync_metric_storage_test.cc
100.00% <100.00%> (ø)


add: added mockups of website

In folder mockup you can find the initial mockup of the website. On the moodle you can find full laboratory work #2.

Solved the issue above


[db specific] Big number aggregations show undefined in Athena

Hi,
While drawing a basic big number chart, I got an undified output although data is a defined number (16 here) . Please see the screenshot below.

I am experiencing this bug with athena datasets, and with all aggregation functions (MAX, SUM, MIN, MAX, AVG, COUNT_DISTINCT). No issues with postgres datasets though.
I am using the current latest image (Digest:sha256:3d9c6413c0d4bf0f11feefbb2636d96cbafd1158dc6fade4a3a0970a69bf1d27), and the version in the front interface is 0.999.0dev.
Thanks

@jafournier thanks for reporting! we currently don't have a good plan to support different drivers in all product aspects. If you have a solution already, we will make sure the PR is reviewed and merged.


http://ladoga.graphics.cs.cmu.edu/xinleic/tf-faster-rcnn/cache.tgz dead

The link is dead. T T

I will have it fixed soon, like in 24 hours. Sorry about that.
Regards,
Xinlei

From: Ruotian(RT) Luo<EMAIL_ADDRESS>Sent: Wednesday, February 28, 2018 1:43:12 PM
To: endernewton/tf-faster-rcnn
Cc: Subscribed
Subject: [endernewton/tf-faster-rcnn] http://ladoga.graphics.cs.cmu.edu/xinleic/tf-faster-rcnn/cache.tgz dead (#302)
The link is dead. T T
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/endernewton/tf-faster-rcnn/issues/302, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ACTwEX_bpa-9-_FzvMNCXGmtnD7XKPodks5tZchvgaJpZM4SXZNM.

It is on Google drive: https://drive.google.com/drive/folders/0B1_fAEgxdnvJSmF3YUlZcHFqWTQ
Check cache.tgz.
Regards,
Xinlei

From: Xinlei Chen<EMAIL_ADDRESS>Sent: Thursday, March 1, 2018 8:01:37 AM
To: endernewton/tf-faster-rcnn
Cc: Xinlei Chen; Your activity
Subject: Re: [endernewton/tf-faster-rcnn] http://ladoga.graphics.cs.cmu.edu/xinleic/tf-faster-rcnn/cache.tgz dead (#302)
I will have it fixed soon, like in 24 hours. Sorry about that.
Regards,
Xinlei

From: Ruotian(RT) Luo<EMAIL_ADDRESS>Sent: Wednesday, February 28, 2018 1:43:12 PM
To: endernewton/tf-faster-rcnn
Cc: Subscribed
Subject: [endernewton/tf-faster-rcnn] http://ladoga.graphics.cs.cmu.edu/xinleic/tf-faster-rcnn/cache.tgz dead (#302)
The link is dead. T T
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/endernewton/tf-faster-rcnn/issues/302, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ACTwEX_bpa-9-_FzvMNCXGmtnD7XKPodks5tZchvgaJpZM4SXZNM.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://github.com/endernewton/tf-faster-rcnn/issues/302#issuecomment-369638971, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ACTwEZFo5rRBca1ezZXdKuAhMGfBNVO_ks5taBtdgaJpZM4SXZNM.


problems when building

Hi, I wanted to give a try to parallel.
I installed rust and cargo on my kubuntu box  with apt-get, cloned the repo and ran:
cargo build
I got the errors below.
Could not compile permutate.
To learn more, run the command again with --verbose.

I have rustc 1.7 installed, I just noticed that the recommended version is 1.12. I will try install it and write the results of that try here.

builds fine with version 1.12


[DISCUSSION] Should picture sharing be added?

Should users be allowed to share pictures?
Pros

It makes the chat more fully features and more pleasant to use.
People expect it.

Cons

Sharing photos is largely against the philosophy of this app.
Other chat apps deal with this well already, and this app shouldn't try to replace them as a chat platform.
 Say, 2 days and 50 messages.
Allows users to enable or disable photo-sharing.

This could be a global toggle,
Or, disabled by default and enable per chat.

Nah

Nah

Middleground is to support maybe Giphy and Tenor embedding into the chat.

Middleground is to support maybe Giphy and Tenor embedding into the chat.
I'd find it frustrating to know that images were supported but were limited in such a way.

Or maybe support link previewing like most messaging systems do now-a-days. But I suppose then you might as well just add both for convenience...


Delete items such as Invoices?

Is there any way to delete items such as invoices with QuickBooksSharp?
Also, any plans to add additional properties to entities such as ProjectRef to Invoice?
(Thank you for your work by the way)

That was missing from 3.0 but please use the latest 3.1 for this.
There is an example here: https://github.com/better-reports/QuickBooksSharp?tab=readme-ov-file#delete-entities


Empty migrations cause errors

I created an empty migration accidentally with migrate dev --create-only. Next time I ran that command, it tried to apply all existing migrations (including the empty one), and that cause this error:
C:\Users\Jan\Documents\throwaway\pscaleMigrationTable>npx prisma migrate dev --create-only
Environment variables loaded from .env
Prisma schema loaded from prisma\schema.prisma
Datasource "db": MySQL database "migration-table" at "<IP_ADDRESS>:50155"

Error: P3006

Migration `20210709211128_foo2` failed to apply cleanly to the shadow database.
Error:
Database error
Error querying the database: Server error: `ERROR HY000 (1105): query was empty'
   0: sql_migration_connector::flavour::mysql::sql_schema_from_migration_history
             at migration-engine\connectors\sql-migration-connector\src\flavour\mysql.rs:337
   1: sql_migration_connector::validate_migrations
             at migration-engine\connectors\sql-migration-connector\src\lib.rs:322
   2: migration_core::api::DevDiagnostic
             at migration-engine\core\src\api.rs:89

(This is against PlanetScale, might be specific to them)

I think I've seen this error elsewhere in tests. If the database doesn't like empty queries/empty migrations, should we do something about it? The error message is already pretty clear, and I don't think we should silently ignore these.

Could be MySQL specific


bug: comment filtering to most recent redirects to home page

Describe the bug
When clicking on 'most recent' on filter comments it redirects to the home page.
To Reproduce
Steps to reproduce the behavior:

Go to any class, change comments filter from 'most upvote' to 'most recent' or to 'most downvote'
It redirects us to the home page

Expected behavior
Should filter out comments based on the seleceted filter.
Screenshots or GIFs
If applicable, add screenshots to help explain your problem.
https://github.com/code100x/cms/assets/72039675/754f852d-791a-4297-992d-c1a000ae3b7c
Info (please complete the following information):

Brave
1.65.x

Additional context
Add any other context about the problem here.

Hi @ASCE-D,
It working as expected.Please check the attached  video.
Comments.webm

In production we can replicate the issue.


scipy.stats.mode does not accept non-numeric values

In Tutorial_Reference_mappying.ipynb, the following lines raise an error
for k in idx_list:
    if faiss_imported:
        idx = labels[k]
    else:
        idx, sim = get_similar_vectors(test_emebd[k][np.newaxis, ...], ref_cell_embeddings, k)
    pred = mode(ref_embed_adata.obs[cell_type_key][idx], axis=0) # I made this change. scipy.stats.mode no longer accepts non-numeric values
    preds.append(pred[0][0])

TypeError                                 Traceback (most recent call last)
/tmp/ipykernel_21913/2423575529.py in ?()
17     if faiss_imported:
18         idx = labels[k]
19     else:
20         idx, sim = get_similar_vectors(test_emebd[k][np.newaxis, ...], ref_cell_embeddings, k)
---> 21     pred = (ref_embed_adata.obs[cell_type_key][idx], axis=0) # I made this change. scipy.stats.mode no longer accepts non-numeric values
22     preds.append(pred[0][0])
23     # preds.append(stat_mode(np.array(ref_embed_adata.obs[cell_type_key][idx])))
24
/usr/local/lib/python3.10/dist-packages/scipy/stats/_axis_nan_policy.py in ?(failed resolving arguments)
519                 # behavior of those would break backward compatibility.
520
521                 if sentinel:
522                     samples = _remove_sentinel(samples, paired, sentinel)
--> 523                 res = hypotest_fun_out(*samples, **kwds)
524                 res = result_to_tuple(res)
525                 res = _add_reduced_axes(res, reduced_axes, keepdims)
526                 return tuple_to_result(*res)
/usr/local/lib/python3.10/dist-packages/scipy/stats/_stats_py.py in ?(a, axis, nan_policy, keepdims)
507         message = ("Argument a is not recognized as numeric. "
508                    "Support for input that cannot be coerced to a numeric "
509                    "array was deprecated in SciPy 1.9.0 and removed in SciPy "
510                    "1.11.0. Please consider np.unique.")
--> 511         raise TypeError(message)
512
513     if a.size == 0:
514         NaN = _get_nan(a)
TypeError: Argument a is not recognized as numeric. Support for input that cannot be coerced to a numeric array was deprecated in SciPy 1.9.0 and removed in SciPy 1.11.0. Please consider np.unique.mode
My current workaround:
from statistics import mode as stat_mode

for k in idx_list:
    if faiss_imported:
        idx = labels[k]
    else:
        idx, sim = get_similar_vectors(test_emebd[k][np.newaxis, ...], ref_cell_embeddings, k)
    # pred = mode(ref_embed_adata.obs[cell_type_key][idx], axis=0) # I made this change. scipy.stats.mode no longer accepts non-numeric values
    # preds.append(pred[0][0])
    preds.append(stat_mode(np.array(ref_embed_adata.obs[cell_type_key][idx])))

Just want to confirm if this is workaround is appropriate. Thanks!

Thank you @xinyu-dev ! and yes, looks like it is related to scipy version changes. The workaround looks good to me. I am also going to update a fix right now


Add version information to non-API ref pages

Currently there is no way for users to tell if a docs page is using the latest version of Qiskit, Qiskit Runtime or not. We should somehow indicate version information (maybe in a notes box in the bottom?) on each page.

Duplicate of https://github.com/Qiskit/documentation/issues/133.


How to centralize the startify theme

How to achieve a similar effect with dashboard theme like centered startify?

changing position = "center" and adding a "width = x" to opts of each element might get you the effect you want without much trouble

One problem I faced is the lengths within startify are not fixed.
So it's not trivial to use the center trick as dashboard.

the newer theta theme is like a centered startify theme

Looks good, thx!


Prevent the presence of multiple root path providers from making the bootstrapper throw

Ignore Nancy.Testing to prevent detecting the FakeRootPathProvider
Remove the use of Single() and decide what should happen is multiple providers are detected

This helped me out in an unexpected way.  I had launched my instance with a new DefaultBootstrapper.  Changed it to new Bootstrapper().  Took me a while to figure out what was going on because the error kept telling me to override the method and I was like "I have."  and it was like "NO YOU HAVEN'T." and in the end I was like "Okay, you're sorta right."
Putting the error here so people can search for this.
MultipleRootPathProvidersLocatedException
More than one IRootPathProvider was found and since we do not know which one you want to use, you need to override the RootPathProvider property on your bootstrapper and specify which one to use. Sorry for the inconvenience.


Support SNIP-9

Closes #1517
Introduced changes
[x] Added outside execution model, defined hashing
[x] Extended base account interface
[x] Added all utilities to deal with SNIP-9 nonce, interface support queries and generation OutsideExecution call
[x] Cleaned up as much as I could to keep changes minimal
[x] Added a bit of docs
[x] Added one test for positive scenario
[x] Pinned hash calculation for OutsideExecution
[x] Added one test for wrong caller scenario
Small things I had to rework on the way
[x] Moved broadcast transaction generation method closer to usage to get rid of Cyclic dependency
[x] Bugfix type data generation
[x] Allowed for incomplete definition of ParameterDict as contains fields is often missing and linter is.
While implementing the feature I realised better what are the problems of current library implementation. I'd love to discuss those and if it proves that my vision is similar to maintainers I'd love to help fixing those.

@franciszekjob Please, take a look.

@baitcode

I fixed all the required steps. With sad heart I got to admit, that some tests in the project are really bad.
Those to are not reproducible, as they depend on the account that is global and is shared among all other tests (and not immutable)

I see, we can think about refactoring these tests in separate PR.

And yes, I need help with test_on_network. I can't reproduce those, neither can I understand what is wrong. You can start the review.

Tests on networks are failing because you haven't added secrets to your forked repository
You need to add:

SEPOLIA_RPC_URL
SEPOLIA_ACCOUNT_ADDRESS
SEPOLIA_ACCOUNT_PRIVATE_KEY

Let me know if it works!

@franciszekjob Added repository secrets on my repo. Still not working.

@baitcode are you sure that the url of node is correct? Have a look at this error
https://github.com/software-mansion/starknet.py/actions/runs/12266627198/job/34225144979?pr=1530#step:10:6834

@franciszekjob
Can you please provide exact instruction how to set up branch, and possibly update contribution documentation. I have everything configured. All values are correct.

I can see this in Job SetUp

Why is that happening?

@franciszekjob
May be this is the problem? How do you think?

@baitcode
Yup, you're right. I think it would be possible to configure repo the way there will be "Allow and run" option but I need to discuss if internally first. Will update you on that.
For the time being, you can either make a PR in your forked repo and see the result of network tests, or run them locally.

@franciszekjob

@franciszekjob Let's finish this!

@franciszekjob please take a look at the last comment.

@baitcode ok, pls request review on github and I will take a loook 👍

@franciszekjob
For some reason this issue appears twice. I replied here

@baitcode just a couple more comments regarding tests, lmk if you need anything 👍

@franciszekjob I need help. For some reason argent account doesn't execute v3 transactions. Reverts with error:
'argent/invalid-tx-version'

For now I've reverted execute_v1 method, as it seems that v3 are actually not supported by argent according to that report.

@baitcode please see my comments

Sure. I'm on it atm.

@franciszekjob Is there anything left?

I squashed and merged so it's fine 🤔

@baitcode you should see the funds on Only Dust 😄


Update README.md

fidel please trigger ok-to-deploy so we can see if the job goes into progress

/ok-to-deploy sha=82fa789

/ok-to-deploy sha=82fa789

/ok-to-test sha=82fa789

/ok-to-test sha=82fa789

/ok-to-test sha=82fa789

/ok-to-test sha=82fa789


[BUG] Custom Driver Stuck on Credential Creation

Rancher Server Setup

Rancher version: 2.7.4
Installation option (Docker install/Helm Chart):

Docker v2.7.4
Helm v2.7.4 on Kubernetes v1.25.10

User Information

What is the role of the user logged in? (Admin/Cluster Owner/Cluster Member/Project Owner/Project Member/Custom)

admin

Describe the bug
I am attempting to setup the Vultr node driver on a fresh install of Rancher(v2.7.4). The node driver can be added to Rancher without issue and it will download as well. Once I attempt to create a cluster using the driver it goes into a loop during the Cloud Credential page. I will add the values, click "create" and then it will bring me right back to creating a credential. I have also attempted to create the Cloud Credential separately, however, the driver does not show up under Cloud Credentials even so the driver is active and downloaded.
This driver worked fine on previous versions of Rancher, however, it is recently not working with newer versions.
To Reproduce
Attempt to use the node driver on Rancher v2.7.4, regardless of whether valid or invalid credentials are used it will loop on that page without any error printed. It is unable to get past the cloud credentials page.
Result
Expected Result
Either print an error on what is wrong, or continue to creating the cluster.

Issue is still present on v2.7.5

@happytreees , thank you for reporting this issue.
Could you please provide some additional information:

You're stating "This driver worked fine on previous versions of Rancher, however, it is recently not working with newer versions." - could you please let us know what is the latest previous version where it worked for you?
Since you've stated "Once I attempt to create a cluster using the driver it goes into a loop during the Cloud Credential page." it looks like you're creating RKE2 cluster as the UI flow for RKE1 clusters is a bit different and requires creating Node Template where Vultr credentials should be provided.
Related to the above, is it possible it was working for RKE1 but maybe never worked for the RKE2 case?
To help reproduce the issue, please provide the exact values used for creating Vultr Node Driver in Rancher.

Hey @snasovich
Thank you for reviewing this issue. I can definitely provide that information for you:

The drive works fine all the way up to v2.6.13 , however, when upgraded to v2.7.0 the issue presents itself immediately and any versions past that. So it seems there was a breaking change or a bug introduced in v2.7.x.

This is for an RKE1 cluster, we do are not using RKE2 at this time.

We first create a Node template under RKE1 Configuration which creates fine. We then go to the "Clusters" tab and attempt to "Create Cluster"
We are then presented with the following page:

It is worth noting that in previous versions this page would just go straight to cluster configuration page, however, in v2.7.x it is no longer doing so as that is the default generated page by rancher. I attempt to put in the values click "continue" and I am brought to this page:

This just becomes a loop and I am unable to move past it as it will have me try to create the credential but because it's not a hard-coded credential it doesn't seem to be able to handle it properly and loops.

We never even attempted to use RKE2 at this point, only RKE1. There does not appear to be an alternate method to create a cluster using RKE1 other than the "Create Cluster" button under the Clusters tab.

Download URL:
https://github.com/happytreees/docker-machine-driver-vultr/releases/download/v0.2.1/docker-machine-driver-vultr
Custom UI URL:
https://sjc1.vultrobjects.com/rancher-ui/component.js
Whitelist Domains:
api.vultr.com
sjc1.vultrobjects.com

@happytreees , thank you for providing this additional information.
I've tried to reproduce it on the latest v2.7-head image and only see a similar issue when "RKE1"/"RKE2/K3s" toggle is set to "RKE2/K3s" (similar because the page for creating Cloud Credentials is slightly different for me).
When RKE1 is selected it goes straight to cluster details page allowing me to use created RKE1 Node Template (please note that I don't have valid API Key but UI allowed me to proceed forward with creating the node template).
Please see attached the video demonstrating the behavior I see.
https://github.com/rancher/rancher/assets/85187633/cb2ed903-b5e0-4c4c-b27c-c7ec9d4bec01
Could you please provide a recording that shows the issue you're seeing?

@snasovich I apologize! I did not notice that toggle as my screen is rather large! That resolved the loop. Thank you so much

@happytreees , no worries - glad it's working as before and nothing got broken recently in regards to how Vultr driver is functioning in Rancher.

Does this not work with RKE2 though?

@FreekingDean not possible due to https://github.com/rancher/ui/blob/master/lib/global-admin/addon/components/cru-cloud-credential/component.js#L234-L253
CC: @snasovich


unsplash link not working

Steps to Reproduce

click the link

Observed Behaviour

the link times out

Image or video please.

Expected Behaviour
https://www.unsplash.com/ should redirect to https://unsplash.com/ seemingly, or in the short term, link directly to https://unsplash.com/
Technical Notes

Thanks for catching this, @brandly! I swear this was working a couple weeks ago.
We're gonna look into why the www hostname doesn't redirect, but I deployed a link update as a band-aid fix.

thanks @joshwcomeau! glad i could help :D


Reading Nested List

I am not sure is it the right place to ask this question, but how would I read nested list from tfrecord?
In particular I am tryring to process frame level data from https://research.google.com/youtube8m/download.html
So "rgb" would have a list of lists, see below.
feature_lists: {
  feature_list: {
    key  : "rgb"
    value: {
      feature: {
        bytes_list: {
          value: [1024 8bit quantized features]
        }
      }
      feature: {
        bytes_list: {
          value: [1024 8bit quantized features]
        }
      }
      ... # Repeated for every second of the video, up to 300
  }
  feature_list: {

I tried reading it as StructField("rgb",  ArrayType(ArrayType(DoubleType, true), true)) but it did not work. Any ideas?

@mpekalski : Please use this repo for any issues and questions. This is our active repo.  https://github.com/tensorflow/ecosystem/tree/master/spark.
Sure will look into it.

@mpekalski We created an issue in https://github.com/tensorflow/ecosystem/issues/36. We are working on the PR.

The fix has been merged to the TensorFlow ecosystem repo


Cannot read properties of undefined (reading 'wireguard')

Description
Whenever I try to open Kubernetes deployment details, it doesn't open & I get this error in the console.

Verified on devnet.
Kubernetes deployment details and cluster management pages are fully functioning without any issues.
Screencast from 13-06-23 12:51:45.webm


[backport 2.6]Add retention lease with followerClusterUUID

Description
Currently, "{followerClusterName}:{followerShardId}" is used as retention lease identifier which results in overlapping usage when two followers are configured for same leader domain with same remote-alias.
The retention lease ID should also include follower cluster specific ID which would provide distinction between retention lease ID’s when 2 follower clusters are connected to a single leader cluster.
New retention lease will be of the format {followerClusterUUID}:{followerClusterName}:{followerShardId}
Issues Resolved
https://github.com/opensearch-project/cross-cluster-replication/issues/554

I just want to confirm that this fix was not backported to 2.6? We just hit this issue in our 2.6 deployment and wanted to confirm this is root cause.

Hi @mrvdcsg , the fix is backported to 2.x
https://github.com/opensearch-project/cross-cluster-replication/pull/851

Hi @mrvdcsg , the fix is backported to 2.x #851

Thank you for the reply. If this happened in May, our build from 2.6 was prior to that backport. Is there a logical way to implement that patch without doing an upgrade? I have a large and active production environment and I have to plan through what a non-disruptive upgrade would look like.


Uncaught TypeError: goog.exportSymbol is not a function

I keep getting this error in the console, stemming from audits.js, even when not using Audits in dev tools.
The "goog" object doesn't contain anything, so thus it is failing when the rest of the code assumes that it has various functions registered as properties.
Seems maybe all of these lines referencing the goog object shouldn't actually be in the extension code?
Using version 2.10.9 in Chrome 48.0.2564.116 m.

+1

+1 So frustrated that it interrupts my workflow. wrapping it with a simple sanity check would preserve my own sanity.

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

+1

+2

+1

+1

+1

I'm afraid this is what I've come to expect from Google. A bug that's been open for over a year, dozens of up-votes, but not even an acknowledgement. ☹

I don't know if it's useful but... +1

+1

+1

+1

+1

Hey all,
This project is in the googlearchive organisation for a reason. It's deprecated & replaced by Lighthouse which catches more than just accessibility issues. Try uninstalling this plugin and give Lighthouse a spin.


⚠ Bofrost.fr has degraded performance

In 2374b20, Bofrost.fr (https://www.bofrost.fr) experienced degraded performance:

HTTP code: 200
Response time: 1420 ms

Resolved: Bofrost.fr performance has improved in 6b98db5 after 6 minutes.


SPOJ User Name not working

The script is unable to fetch the user-name on SPOJ problem pages due to recent update by SPOJ.

Fixed.


Please give us more documentation on how to use this package?

I was try something like
julia> @symmetric f(a::Int, b::String) = 1
ERROR: LoadError: MethodError: no method matching multiset_permutations(::Vector{Any}, ::Vector{Any}, ::Int64)

Closest candidates are:
  multiset_permutations(::Any, ::Vector{<:Integer}, ::Integer)
   @ Combinatorics ~/.julia/packages/Combinatorics/Udg6X/src/permutations.jl:128
  multiset_permutations(::Any, ::Integer)
   @ Combinatorics ~/.julia/packages/Combinatorics/Udg6X/src/permutations.jl:122

Stacktrace:
 [1] multiset_permutations(a::Vector{Any}, t::Int64)
   @ Combinatorics ~/.julia/packages/Combinatorics/Udg6X/src/permutations.jl:125
 [2] var"@symmetric"(__source__::LineNumberNode, __module__::Module, def::Any)
   @ SymmetricDispatch ~/.julia/packages/SymmetricDispatch/5E8Oj/src/symmetric.jl:55
in expression starting at REPL[7]:1

but it fails.

I see some tests in the test dir, I will try to understand how to use it from here.


Fix capitalization of language facets

Language name capitalization differs between languages. This PR adds context for the translation strings so these differences can be properly handled.
For Finnish and Swedish, the language facets (including the "Others" and "All" option) are now switched to lowercase.
Motivation and Context
https://jira.lingsoft.fi/browse/SIMPLYE-321 - Translated items in the Language menu should be capitalized

Honestly in Finnish, this looks a bit wrong. In Finnish, Suomi is the country and suomi is the language (same capitalization in Swedish, but different words for the country and language, so less confusion).

If the following list
suomi
ruotsi
englanti
Muut
Kaikki
feels better than
Suomi
Ruotsi
Englanti
Muut
Kaikki
Then we can revert/cancel this PR. Please confirm.

It does look a bit funny with mixed case for the first letter, but with capitalization it reads like "Finland, Sweden, England, ..." in Finnish. Only after seeing "Other" and "All" also capitalized I might think that they're actually languages.
I think at least the translations themselves shouldn't be capitalized, in case they're (later) used somewhere else. So capitalizing the first letter in the UI (like Sylvain suggested) would be better.
In Finnish, having all of them (including "muut" and "kaikki") in lowercase might be best, but for such a small thing that might be too much work to implement properly.

I agree with @kankaristo, in Finnish having everything lowercase (including "Others" and "All") would be best.
I've now added a context value for the language facet translations and made all of the options lowercase for Finnish and Swedish, while keeping all other filters and English values intact.

In the interface, should we have
Kieli: Kaikki
or
Kieli: kaikki
?
Kieli: suomi
or
Kieli: Suomi
?
In the Ellibs Library app, the language filter is like:
Kielet
[ ] suomi
[ ] ruotsi
[ ] englanti

These look best to me without context:

Kieli: kaikki
Kieli: suomi

But if it has "Kokoelman nimi: Kaikki" next to it, then "Kieli: kaikki" would look weird (although "Kieli: suomi" would look correct at least to me, even if it had "Kokoelman nimi: Kaikki" next to it).
In my opinion, it would be best to lowercase all of those in Finnish ("Kokoelman nimi: kaikki", "Kieli: kaikki", "Kieli: suomi", etc.). That way it would have proper grammar ("suomi" instead of "Suomi", since it's the language, not the country), but it would still have consistent capitalization. The fact that it's not consistent across different app languages won't really be visible to the user.

If "Kaikki" is lowercased for the language filter, it should be as well for other filters.
Will it have an impact on other filters. For example
Saatavuus: Saatavilla nyt
or
Saatavuus: saatavilla nyt
?
What about filter items which are names or set statically via the admin UI?

Realized that the "All" option is still problematic in the current PR version and would require some more refactoring to make it work properly.
I'd suggest we keep it simple and cancel this PR, i.e. keep the options as they are now: "Kaikki", "suomi", "ruotsi", "englanti" and "Muut".

Okay for me at least, that's the simplest and 2nd best option in my opinion. @sylvain-degeilh-lingsoft decides.

Let's put this on hold then. Either delete or put on hold (not sure what are the available options for PRs).


Sitemap and sitemap ping not working

I found that sitemap and sitemap ping of Person Finder are not working.
And it looks like it is failing for long, at least for 7 years.
Maybe we can just remove it because Google probably does a good job finding pages quickly without sitemap, and we actually have seen no issues in these 7 years? WDYT? @nworden
One potential counterargument is that it may still help Google to find a new repository even more quickly.
I noticed that because "sitemap ping" cron job is failing in our production. Also https://google.org/personfinder/japan/sitemap returns an error.
I found multiple problems:

The URL specified in the cron job, /sitemap/ping?search_engine=google, is 404. It misses a repository name.
/sitemap fails due to out of memory.
The datastore contained two entities of StaticSiteMapInfo, both created at 2011, while the code expects only one entity. I have just deleted one of them.

It looks like the sitemap is designed to include links to person pages, which doesn't seem like desirable behavior today (and we're setting noindex on person pages anyway).
I think I could simplify it a lot by just making it a map to repo pages and shared pages (like the terms of service page). Maybe also change it to only ping when a new repo is launched (instead of on a cron every 15 minutes). What do you think? Any reason not to?

SGTM.

Go for it


Release/2.6

The team crossed the great Mojave Desert, collapsed from dehydration, all just to obtain support. In other words we now support macOS Mojave.
[2.6.0] - 2018-10-03
Added

Apple has limited some kickstart command functionality in macOS Mojave, preventing screen
control in some invocations. We verified the ard resource's implementations of kickstart still
function, and added tests to validate the default actions of ard.

Updated Xcode default version to 10.0.

Fixed

Prevented the xcode resource from leaving available Command Line Tools downloads
in Software Updates.

276.1107836 passed.


feat: add context precision for context quality eval algo

Issue #, if available:
Description of changes:
This PR adds the context precision metric under the context quality evaluation algorithm. This is the first of three evaluation metrics under context quality.
Context Precision is a metric that evaluates whether all of the target output relevant items present in the retrieved contexts are ranked higher or not. Ideally all the relevant context chunks must appear at the top ranks. This metric is computed using the model_input, target_output and the retrieved_contexts, with values ranging between 0 and 1, where higher scores indicate better precision.
Notes:

There are no built in datasets for context quality since the dataset context needs to be retrieved by the RAG system.
The default judge model is currently set to a sample Bedrock model because judge model selection is in progress.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Will address default prompt template related changes in a separate PR.


implementing hoa-light object

Name

[ ] Creating the source for a pd object called hoalight at https://github.com/pd-externals/hoalight

Source code

[ ] The source code should be also added to  ZirkoniumSpatializationServer/externals/hoa/ as a submodule

IO

[ ] The pd object should have 4 inlets
[ ] The pd object should have 1 outlet

Type

[ ] The pd object is not a DSP object

Inlet

[ ] The first inlet accepts bang / message
[ ] When the first inlet receives a message, starting with "define_loudspeakers" the object interprets the message as a speaker definition formatted in the style of VBAP object and store the data internally
[ ] The second inlet accepts float and it lets the user control the azimuth of the source (-180 ~ 180)
[ ] The third inlet accepts float and it lets the user control the elevation of the source (0 ~ 90)
[ ] The fourth inlet accepts float and it lets the user control the spread (0 ~ 100)

processing

[ ] The pd object internally encodes/decodes higher-order ambisonics using HOA-light library and output the volumes of each speaker when the first inlet receives a bang

order

[ ] The order of ambisonics is defined by the number of speakers automatically

Looks good.

move this issue to pd-externals/hoalight

Transfer keeps failing, I will have to do it manually.


[Grid] Add auto to TypeScript definitions

Thanks @oliviertassinari for adding auto as a grid size #11804
Here are the updated type definitions

@woobianca Thanks :)


Add support for field value validation before user creation

Requested feature
Add ability to validate some of the received claims or inputted user creation fields before the creation of a User

Closed by #25


[Doc] Dynamically fetch Header height and SideBar width

I am trying to create a loading screen that would only cover the Content area of the layout. In order to style the CSS of my Loader, I need to be able to dynamically fetch the following information from the Layout builder:

Header height
Sidebar width
Sidebar status - is the sidebar open or closed

The following code snippet shows my CSS styling - including the comments  so that you can better understand what I am aiming to achieve:
const useStyles = makeStyles((theme) => ({
  // TODO: Track https://github.com/siriwatknp/mui-treasury/issues/715
  root: {
    alignItems: 'center',
    display: 'flex',
    flexDirection: 'column',
    height: (props) =>
      // 64px is the Header height
      // ideally we would want to fetch this value from the Layout builder
      props.initialLoad ? '100vh' : 'calc(100vh - 64px)',
    justifyContent: 'center',
    width: (props) =>
      // 256px is the Sidebar width
      // ideally we would want to fetch this value from the Layout builder
      // add an if open case to properly handle persistent Sidebar
      props.initialLoad ? '100vw' : 'calc(100vw - 256px)',
    [theme.breakpoints.down('sm')]: {
      width: '100vw',
    },
  },
}));

I am wondering whether, you could provide a working example of how you would approach this problem to achieve the desired responsive behaviour. Also could kindly document that on the website as it could help many other people.

I managed to get somewhere with the following code:
const useStyles = makeStyles((theme) => ({
  root: {
    alignItems: 'center',
    backgroundColor: 'red',
    display: 'flex',
    flexDirection: 'column',
    height: (props) =>
      props.initialLoad
        ? '100vh'
        : `calc(100vh - ${props.headerConfig.md.initialHeight}px)`,
    justifyContent: 'center',
    width: (props) =>
      props.initialLoad
        ? '100vw'
        : `calc(100vw - ${props.edgeSidebarConfig.configMapById.edgeSidebar_id.md.width}px)`,
    [theme.breakpoints.down('sm')]: {
      backgroundColor: 'blue',
      height: (props) =>
        props.initialLoad
          ? '100vh'
          : `calc(100vh - ${props.headerConfig.sm.initialHeight}px)`,
      width: (props) =>
        props.initialLoad || !props.edgeSidebarState
          ? '100vw'
          : `calc(100vw - ${props.edgeSidebarConfig.configMapById.edgeSidebar_id.sm.width}px)`,
    },
    [theme.breakpoints.down('xs')]: {
      backgroundColor: 'green',
      height: (props) =>
        props.initialLoad
          ? '100vh'
          : `calc(100vh - ${props.headerConfig.xs.initialHeight}px)`,
      width: '100vw',
    },
  },
}));

Everything seems to be working fine, with the exception of the temporary Sidebar  - for the xs breakpoint.
My Loader component is instructed to fill the entire viewport - 100vw and 100vh and this instruction is respected as long as the Sidebar is closed. As soon as I open the temporary Sidebar, the Loader component supposed to fill the entire viewport get squashed -  see the following screenshot, the green background is supposed to fill up the entire width.

I suggest another alternative. Can you add position: relative css to Content and then for Loading use position: absolute with top:0, left: 0, bottom: 0, right: 0
<Content> // position: relative
  <Overlay> // position: absolute, top: 0, left: 0, bottom: 0, right: 0
    <Loader />
  </Overlay>
</Content>


Setting vppbranch variable's value to v3.28.0

Setting vppbranch variable's value to v3.28.0
Product Version(s): v3.28.0

Issue:

Link to docs preview:

SME review:

[ ] An SME has approved this change.

DOCS review:

[ ] A member of the docs team has approved this change.

Additional information:

Merge checklist:

[ ] Deploy preview inspected wherever changes were made
[ ] Build completed successfully
[ ] Test have passed

 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.Onong Tayeng seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@ctauchen , @lwr20
Hi Guys, could you please review and merge this PR? We are cutting Calico/VPP v3.28.0 release and this is to make sure the docs point to the right release.

LGTM, pending clean build.

@onong Ready to merge?

@ctauchen yes please :)


Add .csv export capability to Cost Summary in Assessments and Project

In the “Cost Summary” I love how it adds together the totals from the work items…. please also make the materials list exportable to a CSV so we can use it to email to suppliers for quotes, etc.

CSV export button added to cost summary in recent commit.


Is value expected with initial value 0.05?

Continuous plus or minus the time will be wrong, such as an initial value of 0.05, I added twice after 2.05, when I press the time, his value becomes 1.049999999998.

Hi @WangJian0 What is your step and min/max values? What do you mean "press the time"?

Sorry,it's my fault.I mean by min values,but the value changed to 1.049999999998.

So the when it hits the min value you are saying it changes to 1.049999999998?

Can you share the step value and the min/max values, so I may try it out?

initialValue = 0.05
minimumValue={-100}
maximumValue={1000}
and i don't set up stepValue use default value 1;

@WangJian0  Hi again. I have an easy solution to your question. The value you receive in the valueChanged callback is purely a number variable calculated based on value plus or minus stepValue. So the value you got is correct. Now that said if you want to display the number at a certain decimal place you can do so as followed:
// ...
valueChanged(value: number) {
    const displayValue = value.toFixed(2)
    //  ... use in state change or other function
    //  ... or round it
    const roundValue = Math.round(value * 100) / 100
}
// ...


Support lint errors without column

And dont hardcode golangci-lint settings. These should be specified in linter config instead.
Also remove one usage of deprecated util.communicate.
Fixes #2

https://github.com/cixtor/SublimeLinter-golangcilint


Remove runtime type checks for production

Really nice lib!
I thought it might be nice to wrap the validation logic into if(process.env.NODE_ENV!=="production").
This way the lib will be super light in production and additional type checking features could be introduce for development id needed.

This is a great idea. Instead of just doing if(process.env.NODE_ENV!=="production") to avoid the checking it might be nicer to conditionally define the function so that the type checking is completely left out.

Fair enough. The condition will still be env I assume?

@alexeygolev Please take a look at the last commit. If it looks good to you I'll push it to npm.

@paldepind Dead code elimination can't figure it out:) We are still going to have all that validation code in production:) My idea was to eliminate it completely... To do this we need to actually wrap pieces of code into if(process.env.NODE_ENV!=="production"). This way the uglyfier will see if(false) and just remove it from the output on the build stage. This way we reduce the footprint of the library in addition to disabling the validators. Thoughts?

Oh. You mean if one uses something like envify?

I use webpack which does the same kind of think as envify does. React uses the same trick for their warning messages. It lets you be quite verbose for development environments but remove it completely for production. I'm not sure if you find it useful though so the PR is more to illustrate the case)


Try catch finally block is too verbose

   io:ByteChannel destinationChannel = getFileChannel(dstFilePath, io:WRITE);
   try {
       io:println("Start to copy files from " + srcFilePath + " to " +
                   dstFilePath);
       copy(sourceChannel, destinationChannel);
       io:println("File copy completed. The copied file could be located in " +
                   dstFilePath);
   } catch (error err) {
       io:println("error occurred while performing copy " + err.message);
   } finally {
       match sourceChannel.close() {
           error sourceCloseError => {
               io:println("Error occured while closing the channel: " +
                          sourceCloseError.message);
           }
           () => {
               io:println("Source channel closed successfully.");
           }
       }
       match destinationChannel.close() {
           error destinationCloseError => {
               io:println("Error occured while closing the channel: " +
                          destinationCloseError.message);
           }
           () => {
               io:println("Destination channel closed successfully.");
           }
       }
   }```
could be replaced with the following:
   ```try(
       io:ByteChannel sourceChannel = getFileChannel(srcFilePath, io:READ);
       io:ByteChannel destinationChannel = getFileChannel(dstFilePath, io:WRITE);)
   {
       io:println("Start to copy files from " + srcFilePath + " to " +
                   dstFilePath);
       copy(sourceChannel, destinationChannel);
       io:println("File copy completed. The copied file could be located in " +
                   dstFilePath);
   } catch (error err) {
       io:println("error occurred while performing copy " + err.message);
   }```

Currently the suggested approach is not supported. There are some changes coming up related to try-catch block.


Documentation issue

http://docs.anychart.com/7.11.1/Graphics/Paths

Sample parsing error!
{sample}GFX_Paths_1{sample}

Issue with two samples have fixed.
Thank you for your report!


Added flake8 configs in tox.ini

Hey @sourabhtk37 please submit a gerrit review with the same change where I can merge it, you can see more information at https://github.com/redhat-performance/stockpile#contributing. I'm going to close this PR.


Comparison to youtube-dl - table for wiki / README?

I'm curious to hear how annie's goals and usage differ from youtube-dl. As mentioned previously in #57, a short comparison table on the README or wiki would be awesome, especially as both programs have a somewhat long list of options and features. I'd be willing to help tackle this if anyone can point me to any further documentation or roadmap of the project.
Based  on the README and the supported sites list, my impression is that the main difference between the two, besides the implementation language of Go vs Python, is that annie focuses mostly on supporting Chinese sites — is that right?

is that annie focuses mostly on supporting Chinese sites

For now, yes, this may be a big difference between annie and youtube-dl. youtube-dl supports a lot of sites, and annie only supports a few sites (mostly Chinese sites).
The reason why I choose Go as implementation language is that Go is statically-compiled language, it's fast, maintainable, easy to use (compile once, run everywhere, no need to config other environments).
And about the roadmap, I haven’t thought about this yet. Now I'm just constantly improving the stability of download, and other core things (like error handling. etc) and maintaining existing sites.
Maybe I can try to write some further documentation about development and roadmap thing (maybe takes a few days).

Some site fails when using youtube-dl, so I think annie is better performance addressing 'bilibili.com'.
That's annie's goal.


Testing and validating on multi gpu

When I run test on multi gpu I have less samples then exists. In my test set I have 7 samples, also I see this in the logs when start task. But as a result I have only two samples. When I use one gpu everything is okay. The problem is also when validating the model

Do you have similar issue like this: https://github.com/Pointcept/Pointcept/issues/97
Check your evaluation log whether it is completed or not.
In my case, validation and test using 4 GPUs on S3DIS will result only 17 samples out of 68 samples. It was stuck on Syncing ..., the process was not completed. How many GPUs are you using? If it is 4, then most probably you have similar issue during precise evaluation because you get 2 samples out of 7.
Also check if your output is npy or pth.

When I run test on multi gpu I have less samples then exists. In my test set I have 7 samples, also I see this in the logs when start task. But as a result I have only two samples. When I use one gpu everything is okay. The problem is also when validating the model

Hi, the training log only presents the main process (GPU:0) information. So when you enable multi-gpu test, the number of samples is actually the number of samples on the GPU:0 or the number of batches, that is correct.

Do you have similar issue like this: #97
Check your evaluation log whether it is completed or not.
In my case, validation and test using 4 GPUs on S3DIS will result only 17 samples out of 68 samples. It was stuck on Syncing ..., the process was not completed. How many GPUs are you using? If it is 4, then most probably you have similar issue during precise evaluation because you get 2 samples out of 7.
Also check if your output is npy or pth.

Also, for the case of S3DIS, each sample contains a varying number of points, which means the work loads of each GPU are not even. That causes a long waiting time for syncing. Just take it easy and have a break, the testing results will finally be present in your log. Still, we have a new technology for testing, which can significantly reduce the testing time while keeping precise. Some new configs already enable it, you can refer information here. (https://github.com/Pointcept/Pointcept/issues/108#issuecomment-1893709522)

Hi Xiaoyang, thanks! I'm reading that now and I will give a try by waiting the syncing. Anyway, I'm just curious in my case with S3DIS with 4 GPUs, what I see in a result folder is 17 npy files. I believe it comes from 1 GPU (because we have in total 68 samples). After Syncing is completed, I will expect to have 68 npy files. Does that mean that the npy files are the final prediction after precise evaluation?
Sorry for asking you a lot of questions, this is a great codebase and I'd really like to use this for my project.

That case is a bit out of my expectations. As four GPUs are predicting concurrently, it should not merely 17 npy. Yet this case means the other 3 GPUs are real stuck. Give me some time, I will check it on my local machine.
Attach with usual case:

@aldinorizaldy Could you share your script for testing? It was directly started with python or our test script?

Thanks, with your test script: sh scripts/test.sh -g 4 -p python -d s3dis -n semseg-spunet-v1m1-0-base -w model_best

In the meantime, I'll check testing with python, not the script. You're the best Xiaoyang!!

Sadly, In my local machine, the issue seems that did not occur. As shown in the below image, it is still the second iteration of testing, we already have multiple npy saved in the results folder. Maybe you can add some print, so that we can check the running status from other GPUs.

Well, in my case, I only have 2 npy in the second iteration. That's odd, but now it's clear where the problem is. Anyway, thanks for checking it. I'll run some tests to check the GPU's usage.

Well, in my case, I only have 2 npy in the second iteration. That's odd, but now it's clear where the problem is. Anyway, thanks for checking it. I'll run some tests to check the GPU's usage.

If needed, you can also email me your experiment folder, which contains config, cached code, and model weights. I can further check whether it is the code is the reason.

Sure, what is your email address?

Hi @Gofinge I have sent my experiment folder. If you have any chance to look at it, that would be really great!

you have any chance to look at it, that would be really great!

Hi, I received the email. I will check it soon.

Hi @Gofinge I can confirm that the problem is on my machine, not the code. I tested on different GPU cluster on my server, and it turns out now it produces multiple npy files for each iteration. Thanks!!

Hi @aldinorizaldy , I also met this problem, can you tell me how you solved it? I can perform normal testing in debug mode on Vscode, but when I test it in the terminal, the code gets stuck in the Syncing process and the output results are incomplete, only 1/4. I used 4 GPUs for testing.

Hi @aldinorizaldy , I also met this problem, can you tell me how you solved it? I can perform normal testing in debug mode on Vscode, but when I test it in the terminal, the code gets stuck in the Syncing process and the output results are incomplete, only 1/4. I used 4 GPUs for testing.

One temporal solution is to test with a single GPU. Or try to check your local environments.

Hi @aldinorizaldy , I also met this problem, can you tell me how you solved it? I can perform normal testing in debug mode on Vscode, but when I test it in the terminal, the code gets stuck in the Syncing process and the output results are incomplete, only 1/4. I used 4 GPUs for testing.

One temporal solution is to test with a single GPU. Or try to check your local environments.

Thanks for your quick reply. Could you tell me the detailed process of checking? Because when I debug the code, everything seems normal. But it always gets stuck when I run it in the terminal.

Hi @Sylva-Lin I work on a few GPU clusters in a HPC environment. In my case,  the syncing did not work on particular clusters. When I tried different clusters, it worked. For me, it seems caused by the local environment. I'm also curious how to solve this, but I'm sorry I can't help you more 😢

Hi @Sylva-Lin I work on a few GPU clusters in a HPC environment. In my case, the syncing did not work on particular clusters. When I tried different clusters, it worked. For me, it seems caused by the local environment. I'm also curious how to solve this, but I'm sorry I can't help you more 😢

OK, thanks for your reply. I'll try something else.

Good luck! If you solve the problem please update here.


vscode timeout waiting for debuger

Issue Type: Bug
vscode timeout waiting for debuger
i can't debugger my program or run any program anymore
please check and repair this as soon as possible
VS Code version: Code 1.28.0 (431ef9da3cf88a7e164f9d33bf62695e07c6c2a9, 2018-10-05T14:58:53.203Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-8250U CPU @ 1.60GHz (8 x 1800)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledvideo_decode: enabledvideo_encode: enabledwebgl: enabledwebgl2: enabled

Memory (System)
7.90GB (3.57GB free)

Process Argv

Screen Reader
no

VM
40%

Extensions (9)

Extension
Author (truncated)
Version

vscode-sqlite
ale
0.3.1

vscode-markdownlint
Dav
0.20.0

diff-tool
jin
0.0.1

vscode-language-pack-zh-hans
MS-
1.28.2

mssql
ms-
1.4.0

anaconda-extension-pack
ms-
1.0.1

python
ms-
2018.9.0

vscode-yaml
red
0.0.17

codelf
unb
10.7.0

In order to repair, please provide detailed steps so that we can reproduce the problem, e.g., what language are you debugging, when do you get the timout, what is your launch configuration.

Piling on - this just happened to me also.  Running a script yesterday that worked perfectly well.  Updated VS code and did a reboot.  Timeout message - timeout waiting for debugger.    It was closed as the closer claimed the issue must be the python extension.  In the middle of a full fresh clean install of windows on that machine.  I had updated VS code on that one also.
Apparently in my case steps to reproduce are to update to latest vs code - than reboot PC.
Version: 1.28.0 (system setup)
Commit: 431ef9da3cf88a7e164f9d33bf62695e07c6c2a9
Date: 2018-10-05T14:58:53.203Z
Electron: 2.0.9
Chrome: 61.0.3163.100
Node.js: 8.9.3
V8: 6.1.534.41
Architecture: x64
running Python 3.6.5 64-bit - anaconda distribution
using the base Python extension and the anaconda extension on both machines.
I hit F5 to run the script - no color change to the Python status strip - than after a short time the timeout message.

windows 10 pro - version 1809 - fast track have had this build since 9/15/2018.  Assume that was another example of closed issue that came back to bite hard :)

Getting same VSCode timeout message  issue after the October 9, 2018—KB4462919 Windows Update.

OS          Version	10.0.17134 Build 17134
VS Code Version     1.28.1
Temporary fix is to revert your last Windows Update.

KB4462919 - seems an unlikely suspect to me.  I have the issue on a standard Win configuration, likely having this update and on a 2nd PC that's on the fast track, a full generation apart.  This second machine just went bad today after a reboot, but my guess would be that it saw this KB a month or more ago.
Does the revert get you out of version 1.28.1 of VS?  Also think that my first machine failed with this error before the 10/9 date - but since I am just did a full clean windows installation I can't check that anymore.  Seen a few other issues listed for json errors, but I am such a newbie at this that I could not tell you if my json is pretty or pink.  Will update after the clean install done - its takes a month of sundays to get anaconda python back to running condition for tensorflow/keras.

Wau - I was turning chicken as I was installing a fresh version of anaconda and am going with the 1.27.2 VS code install.  KB4469... is now buried in my fresh install of windows and baked in like a burnt cookie.  When I first submitted a bug report the other day on this issue it was quickly closed blaming the extension.  I don't think there was a legit investigation on that closing.
Will try your suggestion later tonight on the fresh install if 1.27.2 also has the issue.  If present in 1.27.2 than a good chance the extension is the issue.
My second machine running a time critical script on Spyder (I do love the VS look and layout in comparison) so will not get a chance to try your fix on it for a day or two.

Add the following setting:
"python.terminal.activateEnvironment": false
worked for me too，Thank you very much！
VS Code version: Code 1.28.1
OS version: Windows_NT x64 10.0.17134

This looks like an issue with the Python debugger, please continue the discussion in the repository of this extension.


Problem compiling with 10.1

With 10.1 I have an error at line 73 in SVGCommon: "Invalid typecast"
Result := WinApi.GDIPAPI.MakeRect(Single(Left), Top, Width, Height);

fixed.


Oliinyk / 2494 Achievement tooltip word break

#2494
Changed style for whole tooltips word break property as overflow-wrap.
Used overflow-wrap: break-word property since word-break: break-word is deprecated one.
Style is global and applied for every tooltip, for example city-filter.

Please, do merge after approval.


Automated cherry pick of #8926: fix(monitor): modify suggestrule process influxdb data

Cherry pick of #8926 on release/3.6.
#8926: fix(monitor): modify suggestrule process influxdb data

/lgtm
/approve


Dashrews/ROX-13082 UUID protos generated PR 4 of 4

Description
This is PR 4 of a 4 PR set.
PR 1:  https://github.com/stackrox/stackrox/pull/3679
PR 2: https://github.com/stackrox/stackrox/pull/3681
PR 3: https://github.com/stackrox/stackrox/pull/3694
The previous PRs dealt with code generation, searcher changes, test constants and fixtures.
This PR deals with the utilization of all that other work.

Protos were updated to use type(uuid) for fields that are UUIDs.
The stores impacted by those protos were re-generated to deal with the UUID type.
Search code was updated and adjusted to handle postgres UUIDs.
Tests were added that use test proto and such within the searcher code to drive the test cases.
Additionally per a comment from @md2119 on PR #3679, I have added the support for regex matching since we can regex match on these columns today and it was essentially as simple as using ::text on the field.

There are a lot of files here, but that vast majority of them are generated.
Checklist

[ ] Investigated and inspected CI test results
[ ] Unit test and regression tests added
~- [ ] Evaluated and added CHANGELOG entry if required~
~- [ ] Determined and documented upgrade steps~
~- [ ] Documented user facing changes (create PR based on [openshift/openshift-docs]~(https://github.com/openshift/openshift-docs) and merge into rhacs-docs)

If any of these don't apply, please comment below.
Testing Performed
Added some UT to cover various UUID paths.

/test

/test go-unit-tests-release
/test go-unit-tests
/test go-postgres-tests

/test go-postgres-tests

/test gke-nongroovy-e2e-tests


[Question] Add column to SummingMergeTree table

Hello.
As I know I can add a column to an existing SummingMergeTree table using Alter table statement.
But is it possible to add this new column to the primary key or to the aggregated columns ?
For example I have a table:
CREATE TABLE Test ( Date Date,  Test1 Int32, Test2 Int32, Test3 Int32) ENGINE = SummingMergeTree(Date, (Date,  Test1), 8192, (Test2, Test3))
I added column:
ALTER TABLE Test  ADD COLUMN Test4 Int32 After Test2
And I got (show create table statement):
CREATE TABLE Test ( Date Date,  Test1 Int32, Test2 Int32, Test4 Int32, Test3 Int32) ENGINE = SummingMergeTree(Date, (Date,  Test1), 8192, (Test2, Test3))
So can i get something like:
CREATE TABLE Test ( Date Date,  Test1 Int32, Test2 Int32, Test4 Int32, Test3 Int32) ENGINE = SummingMergeTree(Date, (Date,  Test1, Test4), 8192, (Test2, Test3))
or
CREATE TABLE Test ( Date Date,  Test1 Int32, Test2 Int32, Test4 Int32, Test3 Int32) ENGINE = SummingMergeTree(Date, (Date,  Test1), 8192, (Test2, Test3,Test4))
?
Thanks in advance

Extending the primary key is not officially supported.
There is an undocumented feature to add a column to primary key of non-replicated table.
https://github.com/yandex/ClickHouse/issues/2210#issuecomment-380482697
This feature exist as a hack.

#147
#2210

Good news are that recently we have added this feature to our plans.

Hi, @alexey-milovidov.
Thanks for the info.
Maybe you could also suggest something about adding a column to the aggregated columns like:
CREATE TABLE Test ( Date Date, Test1 Int32, Test2 Int32, Test4 Int32, Test3 Int32) ENGINE = SummingMergeTree(Date, (Date, Test1), 8192, (Test2, Test3,Test4))
Is it possible ?

You will have to add new aggregated columns by hand:

Execute ALTER ADD COLUMN
Do DETACH TABLE Test
Find the metadata file (usually its path is /var/lib/clickhouse/metadata/<database>/<table>.sql) and add the new column to the list of the aggregated columns.
Do ATTACH TABLE Test

Hi, Alex.
This is great news. Thanks.


Headless renderer still renders

game.renderType HEADLESS becomes CANVAS so it runs through game.render() as usual.
https://github.com/photonstorm/phaser/issues/3001
http://codepen.io/samme/pen/oZdwmJ
This issue is about

A bug in the API (2.6, 2.7)

@samme this issue could then also be closed.


Tilt Signal Strength missing

While the Tilt is in range and sending valid data values, the "Extra Data from Device" Section under "Manage Sensor" constantly lists "Current Signal Strength" as 0

I'd love to see this value.  Even better if it can be reported on the graph page.  Not graphed, but reported.

FWIW, the problem is caused by this: https://github.com/thorrak/fermentrack/blob/dev/gravity/tilt/tilt_monitor_aio.py#L88
Over the past week I've switched gears and am now revisiting all the outstanding Tilt-related items - I'll take a look at getting this resolved.


example-prefix

add example- to all name fields in saturn.json

I like it! We should enforce it in CI.


VISUALS: Reorder (Drag/Drop) Work Items in List

We need to specify the states for drag/drop reordering of work items in the Planner List View
Verification Conditions:

Refer to InVision Wireframes
Read the 3 comments in the InVision Wireframes before starting
Download source file
Create mockups representing the various states for dragging/dropping work items within the planner list view
Post screens to InVision for UX team review

Note the rules of Drag-and-Drop at this point for what we need to visualize.  Users can drop above and below but not on top of other list items.  We'll likely need:

Visual designator for allowed drop location between other work items
Visual designator of not allowed drop location (you won't be able to drop across parent work items, only those items of the same level)
What does the row look like in the "hold" state where users are dragging it around the screen

With the exception of a minor comment, this story is done.


Compute hash of OpenApi document

Fixes #799

I don't think we should be using the GetHashCode override for multiple reasons:

it's not its intended use, especially if caller want to store the hash somewhere for later comparison
it can yield equality between unequal objects
it will vary depending on chip instruction set (x86, 64, arm, arm64)
it requires also overriding the Equals method

I'd instead recommend using a proper hashing algorithm (configurable?) like sha256 or up.
see the documentation of get hash code. https://docs.microsoft.com/en-us/dotnet/api/system.object.gethashcode?view=net-6.0#remarks

I leaned more towards overriding GetHashCode() and tried getting the hash value for each of the property values to avoid collision as I believed the goal was to perform a simple object lookup as opposed to encryption which comes with its own set of complications:

You have to convert the OpenApi document object into a byte array for hash computation
We have to mark all object model types, OpenApiAny and expression types as Serializable for the hashing to take place..

le object lookup as opposed to encryption which comes with its own set of complications:

You have to convert the OpenApi document object into a byte array for hash computation
We have to mark all object model types, OpenApiAny and Expression types as Serializable for the hashing to take place..

Nitpicking: hashing and encryption are two completely different things :) I'm not suggesting to rely on any encryption algorithm, simply on standard hashing.
The nice additional thing with using industry standard hashing is that if we document the process correctly, other libs (from other languages) could interop with it and generate the exact same hash for the same document.
Serialize and convert: well that's the beauty of it, we already have the infrastructure to serialize the document to JSON/YAML, let's use that to produce a terse representation (so white-spacing doesn't impact the result), convert to binary, hash, and voilà! You don't even need to implement changes on the whole object model anymore, just on the document or root object.
What do you think? Also @darrelmiller, any objection with this approach?


Support for PHP8

Using PHP8 results in a 500 server error.  I enabled PHP logging but couldn't get any errors to print out.
Reverting to PHP7 fixes the issue.

Duplicate of #2807.

Duplicate of #2807.

It's OK. Just that you're willing to admit that you missed it is amazing. So is the fact that you even tried to search first! 😸


Updated option set value comparison to allow for decimal numbers

With the PowerFxV1 feature set and numbers defaulting to decimal, an expression such as ErrorKind.Custom = 1000 fails. This should work, and this change fixes it.

✅ No public API change.


Need type safety

We want to use typescript for better type safety.
Currently the project is in javascript just transfer to typescript

to do that we need to change many things that can be damage current version we can plan this idea in version 2.0.0 for now we can close this issue as not planned


Jaeger deployment issues in Kubernetes

Hi,
I'm working to deploy Jaeger in my test env in AWS EKS. The pods to be monitored are in default namespace.
I've created the Jeager Operator (v1.28) in observability namespace, and the jaeger-operator deployment & pod are getting created successfully.
When creating the jaeger instance from the operator of type production, the jaeger is created however neither the jaeger pod, nor the ingress are getting created.
Version:

OS: Amazon Linux
Jaeger version: 1.28
Deployment: Kubernetes

Yaml snippet for tracer deployment with production strategy, storage - cassandra (jtprod.yaml)

apiVersion: jaegertracing.io/v1
kind: Jaeger
metadata:
  name: jaegertracer
  namespace:
spec:
  strategy: production
  sampling:
    options:
      default_strategy:
        type: probabilistic
        param: 0.5
  storage:
    type: cassandra
    options:
      cassandra:
        servers: internal-a1-1179306.ap-southeast-1.elb.amazonaws.com
        keyspace: tccc_jaeger
        username: cassandra
        env:
        - name: CASSANDRA_PASSWORD
          valueFrom:
            secretKeyRef:
              key: password
              name: cassandra-secret
        - name: TRACE_TTL
          value: 2629800
        - name: DEPENDENCIES_TTL
          value: 2629800
      cassandraCreateSchema:
        enabled: false
        datacenter: "dev"
        mode: "test"
  collector:
    autoscale: false
    maxReplicas: 1
    resources:
      limits:
        cpu: 30m
        memory: 128Mi

Results:

$ kubectl apply -f  jtprod.yaml
jaeger.jaegertracing.io/jaegertracer created
$ kubectl get po --all-namespaaces | grep jaeger
NAMESPACE   NAME           STATUS   VERSION   STRATEGY     STORAGE     AGE
observability   jaeger-operator-5786ff487b-xbj92          1/1     Running   0          3h32m
Note: jaegertracer pod is not created
$ kubectl get jaegers
NAMESPACE   NAME           STATUS   VERSION   STRATEGY     STORAGE     AGE
default     jaegertracer                      production   cassandra   24s
$ kubectl get ingress
No resources found
$ kubectl -n observability get deployment jaeger-operator -o jsonpath="
{.spec.template.spec.containers[0].env}"
[{"name":"WATCH_NAMESPACE","valueFrom":{"fieldRef":{"apiVersion":"v1","fieldPath":"metadata.namespace"}}},{"name":"POD_NAME","valueFrom":{"fieldRef":{"apiVersion":"v1","fieldPath":"metadata.name"}}},{"name":"POD_NAMESPACE","valueFrom":{"fieldRef":{"apiVersion":"v1","fieldPath":"metadata.namespace"}}},{"name":"OPERATOR_NAME","value":"jaeger-operator"}]
$ kubectl apply -f  jtprod.yaml
jaeger.jaegertracing.io/jaegertracer created
Expected behavior
Being able to create jaeger pod and connect to agents deployed as sidecars within the target pods

Hi, It would be good to see some operator logs. this will help to diagnostic the issue you are facing.

Hi Ruben,
Sure....  I'm trying to redeploy. I'll share the Operator logs in some time

Hi,
could it be the case that the env section does not work like this? i started version 1.28 and applied jtprod.yaml.
After that the operator has terminated with a panic.
panic: interface conversion: interface {} is map[string]interface {}, not string [recovered]
	panic: interface conversion: interface {} is map[string]interface {}, not string

As far as I can see, it happens because env contains a list and not a string as value. (api)
@rubenvp8510 does it make sense to parse deeper levels at this point or to return an error at Options.UnmarshalJSON?
@venkataphaniv According to the JaegerStorageSpec a secret can be specified in the field secretName.
The values TraceTTL and Timeout seem to be part of JaegerCassandraCreateSchemaSpec.

The pasted CR in the first comment is not valid. The options field is a map of jaeger flags https://www.jaegertracing.io/docs/1.30/cli/


refactor(webpack): move config files out of root

Description:
Move the Webpack config files out of the root directory and into a more suitable location. This avoids unnecessary clutter of the project root.
Motivation and Context:
Root directory is cluttered with things that would be better located elsewhere.
How Has This Been Tested?
Run all webpack related build scripts
Build and manually test the app
Types of changes:
Refactor
Checklist:

[x] My code follows the code style of this project.
[x] I have reviewed and updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[x] I have added tests to cover my changes where needed.
[x] All new and existing tests passed.
[x] My commits have been squashed into a concise set of changes.

Tested ACK bf40af1


[Feature] EventMesh Runtime integrates Workflow capabilities

Search before asking

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

Feature Request
In the process of sending messages by EventMesh Runtime, it is necessary to add and resolve the CloudEvents Extension key, and then decide whether to enable Workflow.
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Hi, I want to participate in your WeOpen-Star activity, and submit a PR.


pagesegmode_test.cc: Disable some code for disable legacy build

Taken from https://github.com/tesseract-ocr/tesseract/pull/3261/files

@stweil, can I merge it?
The second failed test will follow in another PR.


Populate "Approved" drop down option with HRIA Data > CRHP Admin table > "Is accepted by feds" field

There are 29 records with Fed IDs assigned but isacceptedbyfeds is not 'Y'. Should we assign these to be "Accepted"?
There are 3 records where isacceptedbyfeds = 'N'. Should we set those to Ineligibile?

@bferguso

Yes - submitted
I guess we need a state that is "Incomplete" for CRHP, can you please add that? I'll update that other ticket...

@emjohnst , @KatieMcEvoy  This will be affected by any updates that will be made to the BCRHP and CRHP statuses. Perhaps those changes can go in this ticket.

@bferguso updating this ticket in light of the changes:

Do these sites have a submission date? If so, they should be Submitted
If accepted by feds is No - put to "updated required" - that will flag me to take a look at them

Moving to Review/QA - additional changes handled in ticket #575


Cross-compile libwally-core for device

I'm still looking for the correct ./configure incantation to cross compile libwally-core to arm64.

Fixed in f8f68bcf29ada27580110b4ece8deace6ad7ddd5


[Bug]: ModuleNotFoundError: No module named 'llama_index.cli.command_line.rag'; 'llama_index.cli.command_line' is not a package

Bug Description
I am getting this error attempting to upgrade a file using llamaindex-cli upgrade-file <file_path>
(venv) PS C:\Users\aaols\PycharmProjects\experiments> llamaindex-cli upgrade-file indexing.py
Traceback (most recent call last):
  File "C:\Python310\lib\runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "C:\Python310\lib\runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "C:\Users\aaols\PycharmProjects\experiments\venv\Scripts\llamaindex-cli.exe\__main__.py", line 4, in <module>
  File "C:\Users\aaols\PycharmProjects\experiments\venv\lib\site-packages\llama_index\cli\command_line.py", line 4, in <module>
    from llama_index.cli.command_line.rag import RagCLI, default_ragcli_persist_dir
ModuleNotFoundError: No module named 'llama_index.cli.command_line.rag'; 'llama_index.cli.command_line' is not a package

My llama deps
llama-index==0.10.6
llama-index-agent-openai==0.1.1
llama-index-cli==0.1.2.post1
llama-index-core==0.10.10
llama-index-embeddings-openai==0.1.1
llama-index-legacy==0.9.48
llama-index-llms-openai==0.1.2
llama-index-multi-modal-llms-openai==0.1.1
llama-index-program-openai==0.1.2
llama-index-question-gen-openai==0.1.1
llama-index-readers-file==0.1.3
llama-index-readers-github==0.1.5
llama-index-vector-stores-chroma==0.1.2
llama-index-vector-stores-deeplake==0.1.1
llamaindex-py-client==0.1.13

Version
0.10.6
Steps to Reproduce
run the command llamaindex-cli upgrade-file <file_path> on a file that still has some old, pre v0.10.0 imports in it
Relevant Logs/Tracbacks
No response

@aaronjolson can you try pip install -U llama-index-cli ? You must have just installed it -- just caught and fixed that bug

https://github.com/run-llama/llama_index/pull/11068

Here are my upgraded deps, I had blew away my venv.
llama-index==0.10.10
llama-index-agent-openai==0.1.4
llama-index-cli==0.1.3
llama-index-core==0.10.10
llama-index-embeddings-openai==0.1.5
llama-index-indices-managed-llama-cloud==0.1.2
llama-index-legacy==0.9.48
llama-index-llms-openai==0.1.5
llama-index-multi-modal-llms-openai==0.1.3
llama-index-program-openai==0.1.3
llama-index-question-gen-openai==0.1.2
llama-index-readers-file==0.1.4
llama-index-readers-llama-parse==0.1.2
llama-index-vector-stores-chroma==0.1.2
llama-parse==0.3.4
llamaindex-py-client==0.1.13

still getting
(venv) PS C:\Users\aaols\PycharmProjects\ai-experiments> llamaindex-cli upgrade-file indexing.py
Traceback (most recent call last):
  File "C:\Python310\lib\runpy.py", line 196, in _run_module_as_main
    return _run_code(code, main_globals, None,
  File "C:\Python310\lib\runpy.py", line 86, in _run_code
    exec(code, run_globals)
  File "C:\Users\aaols\PycharmProjects\ai-experiments\venv\Scripts\llamaindex-cli.exe\__main__.py", line 4, in <module>
    from llama_index.core.command_line.command_line import main
ModuleNotFoundError: No module named 'llama_index.core.command_line.command_line'

That import no longer exists though 😅🫠
I'll have to take a look in the morning

(That's a slightly different error now though)

@logan-markewich - Yes, a bit different. If you want to close this one I can open a second one tomorrow. Up to you. I appreciate you looking into this!

Yeah I was about to make a pull request for this, ran into the same two issues. For the second issue regarding import main, I manually changed it in the generated script in my bin. For aaron, maybe you can open your .exe file/directory and modify the main.py file in a code editor and do the same.
I opened it in Visual Studio Code and changed the import to from llama_index.cli.command_line import main and it's all working now.
#!/Library/Developer/CommandLineTools/usr/bin/python3
# -*- coding: utf-8 -*-
import re
import sys
from llama_index.cli.command_line import main
if __name__ == '__main__':
    sys.argv[0] = re.sub(r'(-script\.pyw|\.exe)?$', '', sys.argv[0])
    sys.exit(main())

for reference, this is the original code generated:
#!/Library/Developer/CommandLineTools/usr/bin/python3
# -*- coding: utf-8 -*-
import re
import sys
from llama_index.core.command_line.command_line import main
if __name__ == '__main__':
    sys.argv[0] = re.sub(r'(-script\.pyw|\.exe)?$', '', sys.argv[0])
    sys.exit(main())

However I could not figure out how this file got generated from the code though.

(myenv) PS D:\Sudar\Generative AI_Casy> llamaindex-cli upgrade-file chatbot_cli_vers\Chatbot_CLI_4.py
Traceback (most recent call last):
File "", line 198, in _run_module_as_main
File "", line 88, in run_code
File "D:\Sudar\Generative AI_Casy\myenv\Scripts\llamaindex-cli.exe_main.py", line 4, in
ModuleNotFoundError: No module named 'llama_index.core.command_line.command_line'
I'm also facing a error similar to the issue mentioned above
Hope someone here might have a soloution
thanks in advance

(myenv) PS D:\Sudar\Generative AI_Casy> llamaindex-cli upgrade-file chatbot_cli_vers\Chatbot_CLI_4.py
Traceback (most recent call last):
File "", line 198, in _run_module_as_main
File "", line 88, in run_code
File "D:\Sudar\Generative AI_Casy\myenv\Scripts\llamaindex-cli.exe_main.py", line 4, in
ModuleNotFoundError: No module named 'llama_index.core.command_line.command_line'
llama-index==0.10.10
llama-index-agent-openai==0.1.4
llama-index-cli==0.1.3
llama-index-core==0.10.10
llama-index-embeddings-openai==0.1.5
llama-index-indices-managed-llama-cloud==0.1.2
llama-index-legacy==0.9.48
llama-index-llms-openai==0.1.5
llama-index-multi-modal-llms-openai==0.1.3
llama-index-program-openai==0.1.3
llama-index-question-gen-openai==0.1.2
llama-index-readers-file==0.1.4
llama-index-readers-llama-parse==0.1.2
llama-index-vector-stores-chroma==0.1.2
llama-parse==0.3.4
llamaindex-py-client==0.1.13
Nearly facing a kind of above mentioned issue
Hope persons here might have a solution for me
Thanks in advance

@aaronjolson @hkrsmk @Sudar278 found the issue, in the root pyproject.toml. Publishing v0.10.11 shortly

llama-index or llama-index-core v.0.10.11? I've been struggling with the same issue. I blew away the entire venv, did just a pip install llama-index, got 0.10.10 of llama-index and 0.10.11 of core and still get this issue. A simple llamaindex-cli upgrade src gets this error:
/venv/bin/llamaindex-cli", line 5, in
from llama_index.core.command_line.command_line import main
ModuleNotFoundError: No module named 'llama_index.core.command_line.command_line'

@junying1 not core.
Just published, do give it a shot
pip install -U llama-index

ok, NOW its published lol
pip install -U llama-index

aaaaaand it still seems broken.... ugh what did I miss, it worked locally 😢

Proof it works in colab, https://colab.research.google.com/drive/1o9GiupK995a-0UG86716Kli_ZYy3r7wD?usp=sharing
Closing this issue out. For anyone encountering this issue, just run the install above

OK I can confirm it worked within the last 30 seconds LOL. I literally tried a -U llama-index, didn't pick up something. Tried -U llama-index-core, picked up llama_index_core-0.10.11.post1-py3-none-any.whl, but that didn't fix it. Then -U llama-index again and got 0.10.11. That finally ran upgrade. Thanks.

Thanks @logan-markewich @aaronjolson  got the above issue solved but getting a new issue in other side  #11092 your help here is appreciatable


Adjust peer dependencies for vite 5

Are you able to support v5 now it's out? I've tested it, and it appears to work just fine with vite 5, I keep getting conflicts as the peer dependencies are installing v4 whilst other packages are installing v5, would appreciate you updating the peer depdencies to
"vite": "^4.x || ^5.x"

Thankyou!

Sorry, I didn't even realise i was commenting within a namespace.... @liuli-util/vite-plugin-node

Release 0.9.0
https://www.npmjs.com/package/@liuli-util/vite-plugin-node


New unicode character handling using array data structure.

JerryScript-DCO-1.0-Signed-off-by: István Kádár<EMAIL_ADDRESS>

typo in title and commit message: *character

@ktorpi, what are the benefits of this patch? Could you measure it on RPi2?

@LaszloLango, here are the results of the measurements (thanks to @bzsolt):

master: 3543d0c
branch: 9962403

                  Benchmark |                   Perf(+ is better)
                 3d-cube.js |   2.620s ->   2.600s :  +0.763%  (+-0.000%) : [+]
     access-binary-trees.js |   1.400s ->   1.410s :  -0.714%  (+-0.000%) : [-]
         access-fannkuch.js |   7.948s ->   7.934s :  +0.176%  (+-1.307%) : [≈]
            access-nbody.js |   3.082s ->   3.122s :  -1.298%  (+-1.563%) : [≈]
bitops-3bit-bits-in-byte.js |   1.738s ->   1.722s :  +0.921%  (+-0.746%) : [+]
     bitops-bits-in-byte.js |   2.652s ->   2.652s :  +0.000%  (+-1.203%) : [≈]
      bitops-bitwise-and.js |   3.690s ->   3.668s :  +0.596%  (+-1.141%) : [≈]
   controlflow-recursive.js |   0.888s ->   0.898s :  -1.126%  (+-1.475%) : [≈]
              crypto-aes.js |   4.732s ->   4.650s :  +1.733%  (+-1.113%) : [+]
              crypto-md5.js |  21.546s ->  21.928s :  -1.773%  (+-0.874%) : [-]
             crypto-sha1.js |   9.738s ->   9.916s :  -1.828%  (+-0.772%) : [-]
       date-format-tofte.js |   2.222s ->   2.208s :  +0.630%  (+-1.577%) : [≈]
       date-format-xparb.js |   1.086s ->   1.066s :  +1.842%  (+-1.455%) : [+]
             math-cordic.js |   2.982s ->   2.984s :  -0.067%  (+-0.846%) : [≈]
       math-partial-sums.js |   1.656s ->   1.644s :  +0.725%  (+-0.960%) : [≈]
      math-spectral-norm.js |   1.502s ->   1.494s :  +0.533%  (+-0.967%) : [≈]
            string-fasta.js |   4.558s ->   4.428s :  +2.852%  (+-1.500%) : [+]
            Geometric mean: |                 Speed up: 0.241% (+-0.275%) : [≈]

Binary sizes(bytes):
master: 197240
branch: 191980

Nice improvement, LGTM.

After fixing the style issues, LGTM