AttributeError: module 'tensorflow' has no attribute 'contrib'

Click to expand!
Issue Type
Bug
Have you reproduced the bug with TF nightly?
Yes
Source
source
Tensorflow Version
2.11.0
Custom Code
Yes
OS Platform and Distribution
No response
Mobile device
No response
Python version
3.8
Bazel version
No response
GCC/Compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current Behaviour?
When I run the first set of commands, this is the results I get from the code so you can get a idea of the problem I'm facing.

Looking in indexes: https://pypi.org/simple, https://us-python.pkg.dev/colab-wheels/public/simple/
Requirement already satisfied: tensorflow==2.11.0 in /usr/local/lib/python3.8/dist-packages (2.11.0)
Requirement already satisfied: tensorflow-estimator<2.12,>=2.11.0 in /usr/local/lib/python3.8/dist-packages (from tensorflow==2.11.0) (2.11.0)
Requirement already satisfied: numpy>=1.20 in /usr/local/lib/python3.8/dist-packages (from tensorflow==2.11.0) (1.21.6)
Requirement already satisfied: absl-py>=1.0.0 in /usr/local/lib/python3.8/dist-packages (from tensorflow==2.11.0) (1.3.0)
Requirement already satisfied: wrapt>=1.11.0 in /usr/local/lib/python3.8/dist-packages (from tensorflow==2.11.0) (1.14.1)
Requirement already satisfied: google-pasta>=0.1.1 in /usr/local/lib/python3.8/dist-packages (from tensorflow==2.11.0) (0.2.0)
Requirement already satisfied: six>=1.12.0 in /usr/local/lib/python3.8/dist-packages (from tensorflow==2.11.0) (1.15.0)
Requirement already satisfied: gast<=0.4.0,>=0.2.1 in /usr/local/lib/python3.8/dist-packages (from tensorflow==2.11.0) (0.4.0)
Requirement already satisfied: libclang>=13.0.0 in /usr/local/lib/python3.8/dist-packages (from tensorflow==2.11.0) (<IP_ADDRESS>)
Requirement already satisfied: typing-extensions>=3.6.6 in /usr/local/lib/python3.8/dist-packages (from tensorflow==2.11.0) (4.4.0)
Requirement already satisfied: tensorflow-io-gcs-filesystem>=0.23.1 in /usr/local/lib/python3.8/dist-packages (from tensorflow==2.11.0) (0.29.0)
Requirement already satisfied: flatbuffers>=2.0 in /usr/local/lib/python3.8/dist-packages (from tensorflow==2.11.0) (23.1.21)
Requirement already satisfied: termcolor>=1.1.0 in /usr/local/lib/python3.8/dist-packages (from tensorflow==2.11.0) (2.2.0)
Requirement already satisfied: tensorboard<2.12,>=2.11 in /usr/local/lib/python3.8/dist-packages (from tensorflow==2.11.0) (2.11.2)
Requirement already satisfied: keras<2.12,>=2.11.0 in /usr/local/lib/python3.8/dist-packages (from tensorflow==2.11.0) (2.11.0)
Requirement already satisfied: protobuf<3.20,>=3.9.2 in /usr/local/lib/python3.8/dist-packages (from tensorflow==2.11.0) (3.19.6)
Requirement already satisfied: packaging in /usr/local/lib/python3.8/dist-packages (from tensorflow==2.11.0) (21.3)
Requirement already satisfied: astunparse>=1.6.0 in /usr/local/lib/python3.8/dist-packages (from tensorflow==2.11.0) (1.6.3)
Requirement already satisfied: opt-einsum>=2.3.2 in /usr/local/lib/python3.8/dist-packages (from tensorflow==2.11.0) (3.3.0)
Requirement already satisfied: h5py>=2.9.0 in /usr/local/lib/python3.8/dist-packages (from tensorflow==2.11.0) (3.1.0)
Requirement already satisfied: setuptools in /usr/local/lib/python3.8/dist-packages (from tensorflow==2.11.0) (57.4.0)
Requirement already satisfied: grpcio<2.0,>=1.24.3 in /usr/local/lib/python3.8/dist-packages (from tensorflow==2.11.0) (1.51.1)
Requirement already satisfied: wheel<1.0,>=0.23.0 in /usr/local/lib/python3.8/dist-packages (from astunparse>=1.6.0->tensorflow==2.11.0) (0.38.4)
Requirement already satisfied: requests<3,>=2.21.0 in /usr/local/lib/python3.8/dist-packages (from tensorboard<2.12,>=2.11->tensorflow==2.11.0) (2.25.1)
Requirement already satisfied: werkzeug>=1.0.1 in /usr/local/lib/python3.8/dist-packages (from tensorboard<2.12,>=2.11->tensorflow==2.11.0) (1.0.1)
Requirement already satisfied: tensorboard-data-server<0.7.0,>=0.6.0 in /usr/local/lib/python3.8/dist-packages (from tensorboard<2.12,>=2.11->tensorflow==2.11.0) (0.6.1)
Requirement already satisfied: tensorboard-plugin-wit>=1.6.0 in /usr/local/lib/python3.8/dist-packages (from tensorboard<2.12,>=2.11->tensorflow==2.11.0) (1.8.1)
Requirement already satisfied: google-auth-oauthlib<0.5,>=0.4.1 in /usr/local/lib/python3.8/dist-packages (from tensorboard<2.12,>=2.11->tensorflow==2.11.0) (0.4.6)
Requirement already satisfied: google-auth<3,>=1.6.3 in /usr/local/lib/python3.8/dist-packages (from tensorboard<2.12,>=2.11->tensorflow==2.11.0) (2.16.0)
Requirement already satisfied: markdown>=2.6.8 in /usr/local/lib/python3.8/dist-packages (from tensorboard<2.12,>=2.11->tensorflow==2.11.0) (3.4.1)
Requirement already satisfied: pyparsing!=3.0.5,>=2.0.2 in /usr/local/lib/python3.8/dist-packages (from packaging->tensorflow==2.11.0) (3.0.9)
Requirement already satisfied: rsa<5,>=3.1.4 in /usr/local/lib/python3.8/dist-packages (from google-auth<3,>=1.6.3->tensorboard<2.12,>=2.11->tensorflow==2.11.0) (4.9)
Requirement already satisfied: cachetools<6.0,>=2.0.0 in /usr/local/lib/python3.8/dist-packages (from google-auth<3,>=1.6.3->tensorboard<2.12,>=2.11->tensorflow==2.11.0) (5.2.1)
Requirement already satisfied: pyasn1-modules>=0.2.1 in /usr/local/lib/python3.8/dist-packages (from google-auth<3,>=1.6.3->tensorboard<2.12,>=2.11->tensorflow==2.11.0) (0.2.8)
Requirement already satisfied: requests-oauthlib>=0.7.0 in /usr/local/lib/python3.8/dist-packages (from google-auth-oauthlib<0.5,>=0.4.1->tensorboard<2.12,>=2.11->tensorflow==2.11.0) (1.3.1)
Requirement already satisfied: importlib-metadata>=4.4 in /usr/local/lib/python3.8/dist-packages (from markdown>=2.6.8->tensorboard<2.12,>=2.11->tensorflow==2.11.0) (6.0.0)
Requirement already satisfied: idna<3,>=2.5 in /usr/local/lib/python3.8/dist-packages (from requests<3,>=2.21.0->tensorboard<2.12,>=2.11->tensorflow==2.11.0) (2.10)
Requirement already satisfied: chardet<5,>=3.0.2 in /usr/local/lib/python3.8/dist-packages (from requests<3,>=2.21.0->tensorboard<2.12,>=2.11->tensorflow==2.11.0) (4.0.0)
Requirement already satisfied: urllib3<1.27,>=1.21.1 in /usr/local/lib/python3.8/dist-packages (from requests<3,>=2.21.0->tensorboard<2.12,>=2.11->tensorflow==2.11.0) (1.24.3)
Requirement already satisfied: certifi>=2017.4.17 in /usr/local/lib/python3.8/dist-packages (from requests<3,>=2.21.0->tensorboard<2.12,>=2.11->tensorflow==2.11.0) (2022.12.7)
Requirement already satisfied: zipp>=0.5 in /usr/local/lib/python3.8/dist-packages (from importlib-metadata>=4.4->markdown>=2.6.8->tensorboard<2.12,>=2.11->tensorflow==2.11.0) (3.11.0)
Requirement already satisfied: pyasn1<0.5.0,>=0.4.6 in /usr/local/lib/python3.8/dist-packages (from pyasn1-modules>=0.2.1->google-auth<3,>=1.6.3->tensorboard<2.12,>=2.11->tensorflow==2.11.0) (0.4.8)
Requirement already satisfied: oauthlib>=3.0.0 in /usr/local/lib/python3.8/dist-packages (from requests-oauthlib>=0.7.0->google-auth-oauthlib<0.5,>=0.4.1->tensorboard<2.12,>=2.11->tensorflow==2.11.0) (3.2.2)
Looking in indexes: https://pypi.org/simple, https://us-python.pkg.dev/colab-wheels/public/simple/
Requirement already satisfied: gdown in /usr/local/lib/python3.8/dist-packages (4.6.0)
Requirement already satisfied: requests[socks] in /usr/local/lib/python3.8/dist-packages (from gdown) (2.25.1)
Requirement already satisfied: tqdm in /usr/local/lib/python3.8/dist-packages (from gdown) (4.64.1)
Requirement already satisfied: filelock in /usr/local/lib/python3.8/dist-packages (from gdown) (3.9.0)
Requirement already satisfied: beautifulsoup4 in /usr/local/lib/python3.8/dist-packages (from gdown) (4.6.3)
Requirement already satisfied: six in /usr/local/lib/python3.8/dist-packages (from gdown) (1.15.0)
Requirement already satisfied: certifi>=2017.4.17 in /usr/local/lib/python3.8/dist-packages (from requests[socks]->gdown) (2022.12.7)
Requirement already satisfied: chardet<5,>=3.0.2 in /usr/local/lib/python3.8/dist-packages (from requests[socks]->gdown) (4.0.0)
Requirement already satisfied: urllib3<1.27,>=1.21.1 in /usr/local/lib/python3.8/dist-packages (from requests[socks]->gdown) (1.24.3)
Requirement already satisfied: idna<3,>=2.5 in /usr/local/lib/python3.8/dist-packages (from requests[socks]->gdown) (2.10)
Requirement already satisfied: PySocks!=1.5.7,>=1.5.6 in /usr/local/lib/python3.8/dist-packages (from requests[socks]->gdown) (1.7.1)
Looking in indexes: https://pypi.org/simple, https://us-python.pkg.dev/colab-wheels/public/simple/
Requirement already satisfied: gdown in /usr/local/lib/python3.8/dist-packages (4.6.0)
Requirement already satisfied: tqdm in /usr/local/lib/python3.8/dist-packages (from gdown) (4.64.1)
Requirement already satisfied: filelock in /usr/local/lib/python3.8/dist-packages (from gdown) (3.9.0)
Requirement already satisfied: six in /usr/local/lib/python3.8/dist-packages (from gdown) (1.15.0)
Requirement already satisfied: beautifulsoup4 in /usr/local/lib/python3.8/dist-packages (from gdown) (4.6.3)
Requirement already satisfied: requests[socks] in /usr/local/lib/python3.8/dist-packages (from gdown) (2.25.1)
Requirement already satisfied: chardet<5,>=3.0.2 in /usr/local/lib/python3.8/dist-packages (from requests[socks]->gdown) (4.0.0)
Requirement already satisfied: certifi>=2017.4.17 in /usr/local/lib/python3.8/dist-packages (from requests[socks]->gdown) (2022.12.7)
Requirement already satisfied: idna<3,>=2.5 in /usr/local/lib/python3.8/dist-packages (from requests[socks]->gdown) (2.10)
Requirement already satisfied: urllib3<1.27,>=1.21.1 in /usr/local/lib/python3.8/dist-packages (from requests[socks]->gdown) (1.24.3)
Requirement already satisfied: PySocks!=1.5.7,>=1.5.6 in /usr/local/lib/python3.8/dist-packages (from requests[socks]->gdown) (1.7.1)

When I Initialize Tacotron and Waveglow this is the error i'm receiving.

/usr/local/lib/python3.8/dist-packages/gdown/cli.py:121: FutureWarning: Option `--id` was deprecated in version 4.3.1 and will be removed in 5.0. You don't need to pass it anymore to use a file ID.
  warnings.warn(
Downloading...
From: https://drive.google.com/uc?id=1qTdOAdKmMKe-HZA-1SXCAARBAdZXeSAN
To: /content/merged.dict.txt
100% 7.94M/7.94M [00:00<00:00, 50.5MB/s]
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-4-3dc3859c8d60> in <module>
     48
     49 # initialize Tacotron2 with the pretrained model
---> 50 hparams = create_hparams()

/content/tacotron2/hparams.py in create_hparams(hparams_string, verbose)
      6     """Create model hyperparameters. Parse nondefault from given string."""
      7
----> 8     hparams = tf.contrib.training.HParams(
      9         ################################
     10         # Experiment Parameters        #

AttributeError: module 'tensorflow' has no attribute 'contrib'

Tensorflow 1 is incompatible with google colab, the code was created to run using tensorflow 1 so this is why the code is not working. I have been facing this error for nearly two months i really would appreciate if you can solve this. I have been told to code this myself but I have no knowledge on coding.

Standalone code to reproduce the issue
https://colab.research.google.com/drive/1ecuUD2sxfuM6IOhjZEW3ls-jvbpJEei4?usp=share_link#scrollTo=Q0BWBVdCS9ty

Relevant log output
No response

You are using a TF 1.15 or older colab while installing TF 2.x. TF 1.15 is no longer supported.

Hello @Colabnoob, Adding to the @mihaimaruseac points,
The error message you are encountering suggests that the code you are trying to run uses the tf.contrib module, which has been deprecated and removed in TensorFlow 2.x.
To resolve this issue, you need to update the code to use a non-deprecated API. In this case, it seems like the code is using the tf.contrib.training.HParams class, which can be replaced by the tf.config.HParams class.

This should resolve the error and allow you to use the new tf.config.HParams class in your code:
# import the new API
import tensorflow as tf

# replace the old API with the new one
hparams = tf.config.HParams()

# set hyperparameters as desired
hparams.add_hparam('param_name_1', param_value_1)
hparams.add_hparam('param_name_2', param_value_2)
...

The above is an example to do this

Let me know if it helps ?

Also if #59492 is the same issue, please close one

Hi thanks for the help I really appreciate this, I am trying to fix this now so I will let you know when I do this

Sure !!!

So I have successfully updated the Hparams.py file and unfortunately it's still saying the same error. "AttributeError  What hyperparameters did you want me to change in the hparams.py file? AttributeError: module.   'tensorflow._api.v2.config' has no attribute 'HParams' is there any information I can provide to help you get a better understanding? Thanks

@mihaimaruseac adding to your point
To resolve this issue, you could try updating TensorFlow to the latest version (2.x) or if you need to use a specific older version, you can use the appropriate API for that version. If you could share the exact version of TensorFlow you are using, I can provide more specific guidance.

Also you have provided that it's 2.11 version of TensorFlow but it's not, so please correct it in your issue description

I do prefer to use waveglow for synthesis so i would appreciate if you can solve this problem, it's very frustrating.

@Colabnoob
As per the above comment, TF 1.15 is no longer supported. could you refer to Hyperparameter Tuning with the HParams Dashboard, As per documentation try with tensorboard.plugins.hparams and let us know if it helps.
Thank you !

Thanks for the reply, I got a name error with "tensorboard.plugins.hparams" I visited the website about hyperparameter tuning and I had no clue. Im a beginner when it comes to coding and I was hoping someone could solve my problem but I think it cannot be solved unless I somehow build the hyper parameters myself, which would take a far while and was hoping I could solve this finally, I'm not sure if to abandon Google colab and use Juypter notebook if that will make any difference. In Google colab I'm not using a GPU, could that be the problem?

The issue is that you are mixing different versions of TF between TF and other packages installed in your ecosystem.
This is not really a TF issue, you need to make sure your Python environment (local, in colab, in Jupyter, does not really matter) contains a consistent set of dependencies. Look at the output of pip list for example.
In any case, since this is not a TF issue, this should be closed

I just discovered I trained my model using tensorflow 1 so I think the model is incompatible in the latest tensorflow, if so I apologise for the inconvenience I caused. I can understand it’s hard to diagnose the error. But the model works in HIFI-GAN so I’m really not sure what is the issue here.


collapsible items in groups DataTree

Having collapsible items (like xarray.Datasets) in groups repr_html_ would help to have user friendly overview of python objects.
I am wondering whether this feature is already available or not.

Hi @agrouaze, the HTML repr (which you screenshotted) already has collapsible sections for each group in the tree. (And each group corresponds to a single Dataset object.) Is this what you were asking for?

It'd be nice to make each Node under "groups" collapsible (like next to TAO).

Also why is it called "Groups" instead of "Nodes"?

Well @dcherian summed it up perfectly, it would be nice to have each single dataset collapsible.

Closed in favour of: https://github.com/pydata/xarray/issues/9343


Consider alternate FP / SA rendering

The code i used to obtain this screenshot
HTML
ffg-forcepowers.html.txt
ffg-signatureability.html.txt
SCSS
ffg-star-wars-alternate-ui-actor.scss.txt

Upon further review, this is too similar to the existing layout and I don't think it's worth the effort for the minor refactor.
If others feel strongly about it, I can consider it for a future release (or you can file a PR and I can gate it behind a setting.)
Pulling from the milestone.


Allow scheduling of section pages (_index.md)

I use sections to categorise content. Compared with Hugo's taxonomies, sections have several benefits. They can define custom titles, params, and contain content. Since recently they can also define settings for all posts in a particular section.
But I also schedule all my posts in advance. And that's a problem since with Hugo 0.64, a date or publishDate front matter variable on a section page has no effect. Those pages are simply rendered, even with future dates. As a result I get empty categories, since the posts haven't published yet.
A good solution for my use case is when the actual section page (_index.md) can be scheduled like regular content pages.
(Related: issue 6312, which is about section pages respecting the draft front matter variable.)

This will be fixed in next Hugo.

That is great. Thank you. :bowing_man: :slightly_smiling_face:


Shared Notes content is lost once disconnected

When the user launch SharedNotes activities, the content stay in the Journal but is lost once disconnected/reconnected.

Fixed in https://github.com/llaske/sugarizer/commit/3c4396d4e30e34430356565c632de23772423d7f


Futhark loses path to headers when importing

Hi, I have a simple binding of a custom cimgui build:
import futhark

 importc:
    path "."
    path "./cimgui"
    define CIMGUI_DEFINE_ENUMS_AND_STRUCTS
    "cimgui.h"

{.passL:"-Icimgui.lib".}

It appears to work fine when compiling just this file,
but if I import this in a parent file, I get:
Fatal: 'cimgui.h' file not found. I'm sure I can edit the path but then if I import this in another place I imagine that would break too?
I'm also difficulty with getting outputPath to act reliably but perhaps that's another ticket :)

That is indeed the correct solution. I use that style in the docs in some places, but it's not very consistent. I guess it should be possible to change the working directory of Clang to where the Futhark macro was called from, that would make paths behave a bit better.

I'm also getting undeclared identifier when trying to use any of the header functions/types in this case, like imvec2() or echoing an enum value.
Do I need to include the generated nim file before use?

and thanks!

I can pad out the docs in the future with a pull request if you're interested


[SPARK-19228][SQL] Migrate on Java 8 time from FastDateFormat for meet the ISO8601

What changes were proposed in this pull request?
Add support for inferring DateType and custom "dateFormat" option.
Fix an old bug with parse string to SQL's timestamp value in microseconds accuracy.
Add a type-widening rule in findTightestCommonType between DateType and TimestampType.
How was this patch tested?
Fix some tests to accord with an internationally agreed way to represent dates.
Add an end-to-end test case and unit tests.

cc @viirya @mgaido91 @dongjoon-hyun

ok to test

Test build #90870 has started for PR 21363 at commit 65179a2.

Test build #90870 has finished for PR 21363 at commit 65179a2.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/90870/
Test FAILed.

@sergey-rubtsov, would we be able to add a configuration to control this behaviour? Sounds we should better have a configuration to control this behaviour for now since the date / timestamp parsing logic is affected.

@HyukjinKwon please, clarify, how to add a configuration to control this behaviour? Do you mean to keep backward compatibility?
But it seems, now, we don't handle a Spark "DataType" at all.
I can change the behaviour in next way:
Hanlde "DataType" if option "dateFormat" was set by customer, if it is not set, ignore it at all, is it okay?

@sergey-rubtsov we have to keep backward compatibility. If a user upgrades, with your change a running application may break because of data not being anymore timestamp but date. We can add a new entry in SQLConf for that.
Moreover, we have to mention in the migration guide all the behavioral changes we introduce.

Can one of the admins verify this patch?

ok to test

Test build #91599 has started for PR 21363 at commit 65179a2.

Test build #91599 has finished for PR 21363 at commit 65179a2.

This patch fails Scala style tests.
This patch does not merge cleanly.
This patch adds no public classes.

Build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/91599/
Test FAILed.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Looks difficult because the behaviours themselves are different. One possibility is a fallback and the other possibility is configuration.

I am going to suggest to close this since it's being active more then few weeks. It should be good to fix. Let me leave some cc's who might be interested in this just FYI. Feel free to take over this when you guys find some time or are interested in this.
Adding @xuanyuanking, @mgaido91, @viirya, @MaxGekk, @softmanu who I could think of for now. Feel free to ignore my cc if you guys are busy or having more important fixes you guys are working on.

@xuanyuanking Are you still working on this? If not, I could continue.

I'm going to close this since @sergey-rubtsov is inactive.
Most of feasible changes extracted from here are merged. One last part is date type inference which's going to need some more discussions (IMHO).


Investigate support for OpenAPI 3.1.0

https://github.com/OAI/OpenAPI-Specification/releases/tag/3.1.0-rc0
https://github.com/OAI/OpenAPI-Specification/blob/master/versions/3.1.0.md

https://www.openapis.org/blog/2020/10/23/implementers-draft-oas3.1-rc1-available-for-feedback-please-respond-by-nov-8

https://github.com/OAI/OpenAPI-Specification/releases/tag/3.1.0-rc1

Looks like this is almost a year old. I'd like to encourage teams at my company to adopt OAS 3.1 as they start publishing their documents into Apicurio Registry. Seems like this issue needs to be resolved before registry supports OAS 3.1, right?
Is it possible to get a timeline estimate for OAS 3.1 support?
Thanks for all your hard work on Apicurio!

There is progress on this, but not enough where I can give a timeline estimate.  You are right that this needs to be accomplished before OAS 3.1 support is available in Registry.
Ping to @Apicurio/developers on this, fwiw.

So I know it's been a long road, but Apicurio Data Models version 2.0 (release candidate 1) was just released and it features support for all released versions of OpenAPI and AsyncAPI:
https://github.com/Apicurio/apicurio-data-models#project-status
The new version takes a code generation approach to creating the core models for all the versions of the specifications we support.  It's a breaking change, as many APIs and also the surface area of the models has changed.  The core ideas are largely preserved.
There's still work to be done, in particular in the validation and operational transformation areas.  But with the release of 2.0.0.RC1 I think we can close this issue and start to open more granular issues for missing or broken functionality.
Note that I have a PR open to upgrade Registry to data models 2.0 here:
https://github.com/Apicurio/apicurio-registry/pull/3100


certificates printing

I could not find the printing of certificates.
This could be added to WCA Live or it would be good to export so that we can copy/paste.

Please, see #13.

Sorry, I did not know there were closed issues as well. :-)
And exporting the data?

WCA exposes so called WCIF with relevant information and WCA Live synchronizes results there, so they are all available. At this very moment getting WCIF via API requires authentication (only organizers/delegates can do that), but we plan to make it available publicly (omitting the secret data like emails and birthdates). Once this is done it should be even simpler to write integrated software.


pdf file with font source_han_sans_normal(思源黑体)can not be rendered correctly

pdf file with font source_han_sans_normal(思源黑提体)can not be rendered correctly. Some charaters have lost, can not be seen. How can I resolve it

Which Software you Used to convert your worked file (like indd, docx, ppt etc.) into Pdf
Sometimes many Software will doesn't convert pdf's Properly Will Result like this!
So, Use DoPdf which is a open-source Pdf Converter which Tutorials are available on YouTube!
DoPdf will change Your Document Text into a Simple Process
Then import it into the Bartekc Android Viewer!
Hope, This will solve your Problem!


Implemented TODO on function remove_section_from_cartesian_grid in cartesian grids

You can also add a test on this operation

I will put a test on this with my next PR, where I will start implementing functions to use the AttributeStore more easillly


[Bug] hubspot_engagement_contact_enabled needs to be taken into consideration within hubspot__contacts model

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the issue
Currently if you are using the hubspot__contacts end model, it offers the ability to turn on/off the engagements portions using this config.
When inspecting the code block that leverages this config you can see it is pulling from the int_hubspot__engagement_metrics__by_contact model. However, upon inspecting that intermediate model you can see the model is dependent on the following variables.
https://github.com/fivetran/dbt_hubspot/blob/6b1900d539e71b531e65b1ae889e6b2758c29e0f/models/marketing/intermediate/int_hubspot__engagement_metrics__by_contact.sql#L1
As you can see there is an inconsistency where the intermediate model is able to be disabled by the hubspot_engagement_contact_enabled variable. Whereas the end model does not take this additional variable into consideration.
We will need to update the hubspot__contacts end model to take the hubspot_engagement_contact_enabled variable into consideration. If we do not, a customer may have the intermediate model disabled, but not the end model. Which will cause a runtime failure.
Relevant error log or model output
13:52:29  Running with dbt=1.5.1
13:52:29  Unable to do partial parsing because saved manifest not found. Starting full parse.
13:52:34  Encountered an error:
Compilation Error
  Model 'model.hubspot.hubspot__contacts' (models/marketing/hubspot__contacts.sql) depends on a node named 'int_hubspot__engagement_metrics__by_contact' which is disabled

Expected behavior
The package is able to run without any errors regardless of the variable configurations.
dbt Project configurations
dbt=1.5.1
Package versions
v0.14.0
What database are you using dbt with?
snowflake
dbt Version
1.5.1
Additional Context
No response
Are you willing to open a PR to help address this issue?

[ ] Yes.
[ ] Yes, but I will need assistance and will schedule time during our office hours for guidance
[ ] No.

This update has been released in the latest version, so closing this issue out!


[BUG] algolia search icon not visible on website navbar

Describe the bug.
There used to be an Algolia Search Icon besides the Language Picker element on Navbar. It is no longer visible, though we can click it as it is present over there but is very small in size.
Expected behavior

In wireframe:

Screenshots

In wireframe:

(That circle besides the Language Picker is where the search icon used to be)
How to Reproduce

Visit AsyncAPI.com

🥦 Browser
Google Chrome
👀 Have you checked for similar open issues?

[X] I checked and didn't find similar issue

🏢 Have you read the Contributing Guidelines?

[X] I have read the Contributing Guidelines

Are you willing to work on this issue ?
No, someone else can work on it

Hi @anshgoyalevil , I've looked into this issue and seems like IconLoupe component i.e Search icon is not rendered correctly as a children prop in AlgoliaSearch file.

initially children i.e. IconLoupe is not treated as a function but a React Node and that's obvious. But later it's treated as a function. I'm attaching a screenshot for the reference.

if (children) {     if (typeof children === 'function') {       console.log("children is a function");       const result = children({ actionKey });       console.log("Result of children function:", result); // Log the result of the function       setChildren(result);  // Set the resulting JSX component     } else {       console.log("children is not a function, setting directly");       setChildren(children); // Directly set children if it's a React node     }   }

As soon as i do setChildren(children) the search bar appears on the homepage. If we've to fix this issue, we can do it that way.
But yes, we can't do this as we can't use the actionKey then.
These are my findings, thank you!

@amanbhoria can't we just do this instead

it solves the issue

@maverickf0 this seems to be working fine but I'm wondering how this issue occurred on the first place. I mean how this got merged in the master branch.

PRs for this issue
https://github.com/asyncapi/website/pull/3400
https://github.com/asyncapi/website/pull/3447


Rancher agent refuses to work with rancher server behind proxy doing SSL termination

Using HAProxy as load balancer and SSL termination to Rancher Servers.
Have HAProxy node setup to route requests to 3 rancher/server nodes. The proxy also does SSL termination.
The UI for rancher/server worked without issue. rancher/server was also started with the custom CA certificate via the -v <some-dir>/certfile.crt:/var/lib/rancher/etc/ssl/ca.crt
When trying to add a host the initial container that pulls configs from rancher/server always errored out stating it wasn't able to connect to https URL.
The only output from the rancher-agent container is:
Updating certificates in /etc/ssl/certs...
WARNING: rancherAddedCA.pem does not contain a certificate or CRL: skipping
1 added, 0 removed; done.
Running hooks in /etc/ca-certificates/update.d...
done.
Found container ID: 47a9edf85039306a5f293e0d4c3a1f5c0c43cf7a22a65f15e24caad4530d2970
Checking root: /host/run/runc
Checking file: 47a9edf85039306a5f293e0d4c3a1f5c0c43cf7a22a65f15e24caad4530d2970
Found state.json: 47a9edf85039306a5f293e0d4c3a1f5c0c43cf7a22a65f15e24caad4530d2970
time="2017-04-12T18:46:30Z" level=info msg="Execing [/usr/bin/nsenter --mount=/proc/17434/ns/mnt -F -- /var/lib/docker/overlay/8ccb117f367d70b3cb7759029ef0254dfa31bf49e9946752b17146661acaf451/merged/usr/bin/share-mnt --stage2 /var/lib/rancher/volumes /var/lib/kubelet -- norun]"
Found container ID: 47a9edf85039306a5f293e0d4c3a1f5c0c43cf7a22a65f15e24caad4530d2970
Checking root: /run/runc
Checking file: 47a9edf85039306a5f293e0d4c3a1f5c0c43cf7a22a65f15e24caad4530d2970
Found state.json: 47a9edf85039306a5f293e0d4c3a1f5c0c43cf7a22a65f15e24caad4530d2970
INFO: Starting agent for 2AFA257F05E22220C007
INFO: Starting agent for 2AFA257F05E22220C007
INFO: Starting agent for 2AFA257F05E22220C007
INFO: Starting agent for 2AFA257F05E22220C007
INFO: Starting agent for 2AFA257F05E22220C007
INFO: Starting agent for 2AFA257F05E22220C007
INFO: Starting agent for 2AFA257F05E22220C007
INFO: Starting agent for 2AFA257F05E22220C007
INFO: Starting agent for 2AFA257F05E22220C007
INFO: Starting agent for 2AFA257F05E22220C007
INFO: Starting agent for 2AFA257F05E22220C007
INFO: Starting agent for 2AFA257F05E22220C007

It continue like that for some time and then exits.
No relevant logs from any rancher/server that I can see

Useful
Info

Versions
Rancher v1.5.3 Cattle: v0.177.10 UI: v1.5.8

Access
ldap admin

Orchestration
Kubernetes

Route
admin-tab.ha

Did you add the certificate to the agent host trying to connect to the rancher server?

I added to the initial script line that the rancher server prodived when
you do the "Add Host" that script ran without issue and starts the
rancher-agent automatically.
How do I add the CA file to the agent after it has been started already?
On Wed, Apr 12, 2017, 11:05 PM Ahmad E<EMAIL_ADDRESS>wrote:

Did you add the certificate to the agent host
https://docs.rancher.com/rancher/v1.5/en/installing-rancher/installing-server/basic-ssl-config/#adding-hosts
trying to connect to the rancher server?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/rancher/rancher/issues/8490#issuecomment-293765779,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEeMqnvKCgz4PN_j-1HQu1tH7WukYAdIks5rvZECgaJpZM4M74js
.

Nvm misread the last few lines of that document and I will try that in the morning and report if it was successful

Better logging message for the agent when it cant connect to the rancher server. Covers the case where self signed certs are in play.

I have found often that you have to clear down /var/lib/rancher on the agent docker-engine when you are making changes like this.

@aemneina agent log issue is tracked here: https://github.com/rancher/rancher/pull/8452

This has been resolved.


_checkForConnectedDevices throws in Chrome (sometimes)

Sorry i could not reproduce the error but we see it in our Sentry logs often.
TypeError: Illegal invocation
  at _checkForConnectedDevices(webpack://egr-onlineplanner/./node_modules/@babylonjs/core/DeviceInput/InputDevices/webDeviceInputSystem.js:177:38)
  at _enableEvents(webpack://egr-onlineplanner/./node_modules/@babylonjs/core/DeviceInput/InputDevices/webDeviceInputSystem.js:143:18)
  at new e(webpack://egr-onlineplanner/./node_modules/@babylonjs/core/DeviceInput/InputDevices/webDeviceInputSystem.js:36:14)
  at new e(webpack://egr-onlineplanner/./node_modules/@babylonjs/core/DeviceInput/InputDevices/internalDeviceSourceManager.js:50:39)
  at new e(webpack://egr-onlineplanner/./node_modules/@babylonjs/core/DeviceInput/InputDevices/deviceSourceManager.js:15:49)
  at attachControl(webpack://egr-onlineplanner/./node_modules/@babylonjs/core/Inputs/scene.inputManager.js:435:37)
  at attachControl(webpack://egr-onlineplanner/./node_modules/@babylonjs/core/scene.js:1522:28)
  at new t(webpack://egr-onlineplanner/./node_modules/@babylonjs/core/scene.js:665:19)

Browser: only in chrome (different versions 79, 80, 96, 97, 98)
OS: Windows 10
Maybe someone can give me a hint would could cause it.

We will be happy to help in our forum, https://forum.babylonjs.com (just as the issue template has suggested). Closing and looking forward to talking to you there.

ok thanks https://forum.babylonjs.com/t/checkforconnecteddevices-throw-typeerror-illegal-invocation/32722


Increase daemon-containerd timeout and order daemon-kubelite after it

Summary

Increase the start timeout of daemon-containerd.
Order the start of daemon-kubelite after it.
Closes #3684
Changes
Please see above; no user facing changes.
Testing
Please see #3684 for reproduction steps.
Possible Regressions

Apparently no regressions from increasing the daemon-containerd timeout to 5 minutes.
There might delays in the setup/commands performed by daemon-kubelite until
up to waiting for containerd socket in run-kubelite-with-args, as it now starts
after daemon-containerd, and not concurrently with it anymore.
Checklist

[x] Read the contributions page.
[ ] Submitted the CLA form, if you are a first time contributor.
(I think this ^ might not be needed per association with Canonical already?)
[ ] The introduced changes are covered by unit and/or integration tests.

Notes

For documentation purposes, reproduction/test steps with original & modified snaps:
$ multipass launch -c 2 -d 5g -m 2g -n test-mk8s jammy
Launched: test-mk8s
$ multipass shell test-mk8s

Original:
$ sudo snap install microk8s --classic --channel=latest/edge
microk8s (edge) v1.26.1 from Canonical✓ installed

$ snap info microk8s | grep latest/edge:
  latest/edge:           v1.26.1  2023-01-20 (4541) 180MB classic

Setup reproducer:
$ sudo cp /snap/microk8s/current/bin/containerd /usr/local/bin/containerd.orig

$ cat <<EOF | sudo tee /usr/local/bin/containerd
#!/bin/sh
/usr/bin/sleep 95
exec /usr/local/bin/containerd.orig "\$@"
EOF

$ sudo chmod +x /usr/local/bin/containerd

$ DIR=/etc/systemd/system/snap.microk8s.daemon-containerd.service.d
$ sudo mkdir -p $DIR
$ cat <<EOF | sudo tee $DIR/hack.conf
[Service]
ExecStartPre=/usr/bin/mount --bind /usr/local/bin/containerd /snap/microk8s/current/bin/containerd
EOF

$ sudo reboot

...
Systemd service definitions:
$ systemctl show snap.microk8s.daemon-containerd.service | grep TimeoutStartUSec=
TimeoutStartUSec=1min 30s

$ systemctl show snap.microk8s.daemon-kubelite.service | grep After= | grep -ow 'snap[^ ]\+service'
snapd.apparmor.service

...
Number of restarts:
$ sleep 300

$ systemctl show snap.microk8s.daemon-containerd.service | grep NRestarts
NRestarts=4

$ systemctl show snap.microk8s.daemon-kubelite.service | grep NRestarts
NRestarts=15

Modified:
$ git log --oneline -2
3129d213e116 (gh-mfo/daemon-containerd-timeout) Increase daemon-containerd timeout and order daemon-kubelite after it
a560168de7bb (origin/master) bump default install version to 1.26 (#3683)

$ snapcraft
...
Snapping |
Snapped microk8s_v1.26.1_amd64.snap

$ multipass transfer microk8s_v1.26.1_amd64.snap test-mk8s:

...
$ multipass shell test-mk8s

$ sudo killall sleep
$ sudo systemctl stop snap.microk8s.daemon-{containerd,kubelite}.service

...
$ sudo snap install ./microk8s_v1.26.1_amd64.snap --classic --dangerous
microk8s v1.26.1 installed

$ sudo reboot
$ multipass shell test-mk8s

...
Systemd service definitions:
$ systemctl show snap.microk8s.daemon-containerd.service | grep TimeoutStartUSec=
TimeoutStartUSec=5min

$ systemctl show snap.microk8s.daemon-kubelite.service | grep After= | grep -ow 'snap[^ ]\+service'
snapd.apparmor.service
snap.microk8s.daemon-containerd.service

...
Number of restarts:
$ systemctl show snap.microk8s.daemon-containerd.service | grep NRestarts
NRestarts=0
$ systemctl show snap.microk8s.daemon-kubelite.service | grep NRestarts
NRestarts=0

Thanks for your work on this @mfoliveira and all the effort you put into testing. We'll make sure to review and move forward with this unless there are any potentially unwanted side-effects by these changes.

Thanks, @neoaggelos!
The target for this fix are backports down to 1.24 -- is there a stable backports process or timeline I could/should follow?
e.g., is it fine to just send backport PRs now, or wait for 1.27 to be released and then send the backports? (or something different)
cheers,
Mauricio

@mfoliveira Feel free to create PRs that cherry-pick the commit and target the release branches 1.26, 1.25 and 1.24.

@mfoliveira Feel free to create PRs that cherry-pick the commit and target the release branches 1.26, 1.25 and 1.24.

Thanks, @neoaggelos. I created PRs #3761,  #3762, and  #3763.


Elixir is revised

Hi,
Elixir patterns & filters are revised and tested.
@martinezmatias @monperrus

Hello @martinezmatias ,
Travis says:
The job exceeded the maximum log length, and has been terminated.
What can we do to fix it? Could you increase the limit? Should we turn off the logging?

Hi @khaes-kth
Thanks for this new feature!

Should we turn off the logging?

It seems that the default log level is log4j.rootLogger=INFO.
The problem can be that there is a logging call in level info that is called several times, or even there is a sysout that would be removed or replaced by a log.

Could you increase the limit?
I am not aware, but I say we should fix the problem in our code (e.g., changing min log level, removing unnecessary logs, etc)

Regards
Matias

The problem was that the final-result was printed by System.out and due to the large test suite it produced a huge log.
Now I am logging the result at the debug level and it is not printed by default because the default log level is INFO. We can also log it at INFO level and change the log-threshold if you think that would be better.

Hi @khaes-kth
I am not sure if replace the sysout from the  StdOutput by the log.debug, because the idea of that output is to print something at the end.
I would say that I would be necessary to change the toString of the sub-class of FinalResult that cause the large output. WDYT?

Hi @martinezmatias ,
Yes, that's an other option. The problem is that I do not see anything that can be removed from the output without losing information. For example, what follows is the output for one diff:
FileDiff [diffFolder=C:\other\daneshgah\phd-kth\TB\coming\target\classes\repairability_test_files\Elixir\elixir1]
FineGrainDifftAnalyzer

-one
Update TypeReference at elixir:6
	float to double

PatternInstanceAnalyzer

----For Diff:Update TypeReference at elixir:6
	float to double

: number instances found: 1
ChangePatternInstance [actions=[Update TypeReference at elixir:6
	float to double
]]
RepairabilityAnalyzer

----For Diff:Update TypeReference at elixir:6
	float to double

: number instances found: 1
ChangePatternInstance [actions=[Update TypeReference at elixir:6
	float to double
]]

What do you think?

Hi @khaes-kth
The sysout is called by the StdOutput processor, which is used by Coming by default (i.e., if no fillter is set).
One strategy that we could do is to create an EmptyOutput which basically does nothing.
I would say that:

When a user  uses Coming via command line, Coming uses, by default, the StdOutput. (So we should change the code to set the output according to)
Otherwise, it uses the emptyoutput processor. For example, the Travis Ci, which does not run the tool from a Jar, will print nothing.
(A correct way would be to configure travis to indicate Coming to use the emptyOuput, but, as it runs mvn test I am not sure how to pass as parameter that info)

(Note that, even the solution of passing to log.debug that single syso, it could be problematic for somebody that checkouts the project, run in and does not see nothing on the screen due to the default log level is INFO)
WDYT?

Thank you @martinezmatias for your comments. I agree with all of them.
Do you have any idea how we can implement the second feature? (how to make Travis execution to use EmptyOutput). We might even be able to configure Travis to ignore logs (I am not a professional Travis user!)
Another (not very good) option would be to increase the log limit. Isn't it better to log more rather than log nothing in Travis?

Hi @khaes-kth

Another (not very good) option would be to increase the log limit. Isn't it better to log more rather than log nothing in Travis?

That could be a nice solution, maybe in this post we have the solution: https://stackoverflow.com/questions/4782089/how-to-change-maven-logging-level-to-display-only-warning-and-errors (remember that travis calls mvn test)
A maybe dirty but easy to implement solution (related to point 1) could be:

Inside the main https://github.com/SpoonLabs/coming/blob/HEAD/src/main/java/fr/inria/coming/main/ComingMain.java#L128 we mark somewhere e.g. (ComingProperties) that the program was launched from the main.
Then we load the output processor, at https://github.com/SpoonLabs/coming/blob/HEAD/src/main/java/fr/inria/coming/main/ComingMain.java#L246 we check the mark: if it's `on' we add the StdOut (as done here), otherwise no.

Hi @martinezmatias
I used sure-fire maven plugin here to pass the required argument while executing mvn test. When "mvn test" is executed by travis, "executed_by_travis" is set to true and an empty output is generated here.

Hi @khaes-kth
Great solution.
Thanks!
Matias

Hi @martinezmatias
Thanks!


opt: reduce row usage of push opcode

 PUSH, step ratio 0.2449364759864078, row ratio 0.5537811087018458, height 9
 MEMORY, step ratio 0.035950046533774864, row ratio 0.07224896095902775, height 8
 ADD_SUB, step ratio 0.055528861762223235, row ratio 0.05579829447214979, height 4
 SHL_SHR, step ratio 0.02115447048892929, row ratio 0.04782850767400538, height 9
 DUP, step ratio 0.16642642252667575, row ratio 0.04180848588187447, height 1
 CMP, step ratio 0.032582299850659045, row ratio 0.03274039308480445, height 4
 SWAP, step ratio 0.10668138432569314, row ratio 0.02679975380546717, height 1
 BITWISE, step ratio 0.025669328831461162, row ratio 0.02579387949952044, height 4
 JUMPI, step ratio 0.04881934073545008, row ratio 0.024528109021550722, height 2
 JUMPDEST, step ratio 0.07058848992489665, row ratio 0.01773274844007925, height 1
 MUL_DIV_MOD, step ratio 0.008272190117524836, row ratio 0.016624655556014695, height 8

pub fn get_num_rows_required_no_padding(block: &Block<F>) -> usize {
        let mut num_rows = 0;
        let mut counter = HashMap::<ExecutionState, usize>::new();
        let mut step_num = 0;
        for transaction in &block.txs {
            step_num += transaction.steps.len();
            for step in &transaction.steps {
                let height = step.execution_state.get_step_height();
                num_rows += height;
                *counter.entry(step.execution_state).or_insert(0) += height;
            }
        }
        // TODO: change to log::trace?
        let mut counter_vec: Vec<_> = counter.into_iter().collect();
        // DESC
        counter_vec.sort_by_key(|(e, c)| -(*c as i64));
        let print_top_k = 10;
        for (idx, (e, usage) )in counter_vec.iter().enumerate() {
            if idx > print_top_k {
                break;
            }
            let height = e.get_step_height();
            log::debug!("evm circuit row usage: {:?}, step ratio {}, row ratio {}, height {}",
            e, (usage / height) as f64 / step_num as f64,   *usage as f64 / num_rows as f64, height);
        }

        // EndBlock
        num_rows += 1;
        // Plus 1 so we can be sure there is an unused `next` row available
        num_rows += 1;

        num_rows
    }

at least we can change like this now
pub const BYTECODE_TABLE_LOOKUPS: usize = 8;

another solution i am considering is to move some ‘PUSH out of boundary then pad 0’ logic into bytecode circuit

done


BUG: reproducible builds problem in _stats_pythran.cpython-311-x86_64-linux-gnu.so

Describe your issue.
While working on reproducible builds for openSUSE, I found that our
python-scipy 1.10.1 package varies between builds for multiple reasons. This issue is about one reason:
There is a timestamp embedded in _stats_pythran.cpython-311-x86_64-linux-gnu.so
Reproducing Code Example
build it twice - we use scratch VMs, but it might be possible to reproduce by removing a few relevant files

https://code.opensuse.org/package/python-scipy/blob/master/f/python-scipy.spec#_182 has our build description.

Error message
--- old /usr/lib/hpc/gnu13/python-scipy/1.10.1/lib64/python3.11/site-packages/scipy/stats/_stats_pythran.cpython-311-x86_64-linux-gnu.so (objdump)
+++ new /usr/lib/hpc/gnu13/python-scipy/1.10.1/lib64/python3.11/site-packages/scipy/stats/_stats_pythran.cpython-311-x86_64-linux-gnu.so (objdump)
@@ -6289,8 +6289,8 @@
  198c0 5f415252 41595f41 5049206e 6f742066  _ARRAY_API not f
  198d0 6f756e64 005f4152 5241595f 41504920  ound._ARRAY_API
  198e0 6973204e 554c4c20 706f696e 74657200  is NULL pointer.
- 198f0 32303233 2d30372d 32352031 373a3537  2023-07-25 17:57
- 19900 3a31392e 35343936 39330030 2e31332e  :19.549693.0.13.
+ 198f0 32303339 2d30382d 32372030 373a3536  2039-08-27 07:56
+ 19900 3a33382e 39373931 30360030 2e31332e  :38.979106.0.13.
  19910 31002873 73732900 5f5f7079 74687261  1.(sss).__pythra
  19920 6e5f5f00 756e6162 6c652074 6f20616c  n__.unable to al
  19930 6c6f6361 74652000 20627974 65730050  locate . bytes.P

SciPy/NumPy/Python version and system information
scipy-1.10.1
numpy-1.24.2
openSUSE-Tumbleweed-20230728

There's nothing in the _stats_pythran file that would generate a timestamp. I reckon it's something introduced by pythran in its build process.

@bmwiedemann can you share what tools/process you use to figure out if a particular binary has changed?

@bmwiedemann, can you check that none of the other .so (not made with Pythran) have this issue?

For diffing, you can use https://github.com/bmwiedemann/reproducibleopensuse/blob/master/filterdiff with a first param of "hexdump -C" and then the two .so files.
There are many differing .so files, and they all have a __pythran__ string right after the timestamps.

It probably comes from https://github.com/serge-sans-paille/pythran/blob/e3babfd430/pythran/cxxgen.py#L683 via pythran/toolchain.py:187

Can you confirm that Pythran HEAD solves this issue? This should be fixed by https://github.com/serge-sans-paille/pythran/pull/2131

I think this issue can be closed now as pythran addressed this issue.


VMtFileExample

This the base example for the VMT converter.
Oak.txt

https://mega.nz/#!J05AmBjJ

Thank you very much
I'll try to convert that as fast as I can...

Closing issue as it's done...


[Feat] Styleless Build

Hey! How are you?
First of all, I want to say thanks for the code: it's a amazing chatbot solution (and the docs is superb).
Is your feature request related to a problem? Please describe.
Due to some frameworks being incompatible with importing CSS on external modules (#79 / #80) would be nice to have a "styleless" version of library. I know it's not a trivial case to not use the default CSS but developers would be allowed to import as needed too.
Describe the solution you'd like
Just a "index.nocss.js" version in dist folder.
Describe alternatives you've considered
The issues have workarounds but this solution is not too cost too.
Additional context
I would be happy to make a PR.

Hey @mariovalney! Thank you for the kind words 😊
With regard to the CSS import issue, the library actually started out requiring all developers to import the css file manually. From v1.5.0 onwards, manual importing was no longer necessary as the library handled it. This update was made because:

Less stuffs for users to do, less mistakes
The library needed the css file to work correctly, might as well the library import it
Makes a simple thing even simpler 😊

That said, I'm aware it caused issues for NextJS users who are using pages instead of the app directory. From what I've gathered, these are the only users affected because of how pages worked. This caused issues with quite a number of libraries and the app directory is suppose to overcome this limitation (and is also the encouraged way forward).
With that in mind, it felt a little unnecessary to solution for an approach that will eventually be "outdated". Adding an extra index.nocss.js and exporting it is a simple and viable solution, but some concerns come to mind:

Potential for confusion (unknowing users may make use of the wrong imports)
Significant increase in package size (even if unused parts are tree-shaken, developers may perceive a larger library size negatively)

Taking into account that there are existing workarounds to get things to work, there wasn't really a strong reason to take on the cons of adding an extra solution. After all, the recommended approach forward for NextJS users is the app directory as well and we adopt the same recommendation for users facing import issues. This is probably not a satisfactory answer but I hope it sheds some light on the situation 🥹

Thank you for the detailed explanation and context!
I understood that app directory being the encouraged approach, but it’s worth noting that NextJS itself allowed a gradual transition from pages to app precisely because it represents a significant paradigm shift.
Given this, I thought it might be interesting to explore a solution, and since I’ve already created an example, you’re welcome to check out the link if you change your mind: https://github.com/tjtanjin/react-chatbotify/compare/main...mariovalney:react-chatbotify:feat/276-nocss-build
Anyway, thanks for your time!


feat(core): add support for eip-4844 blob transactions

Description
Implements eip-4844 blob transactions as TxBlob type. KZG blob commitments/proofs cannot be computed (yet?) by this library.
https://eips.ethereum.org/EIPS/eip-4844

Solution

Current dependencies on/for this PR:

master

PR #3   👈

This stack of pull requests is managed by Graphite.


print layout names

Add a subcommand to print out all of the available layout names

Add the is_layout field to the config
Add eager callback flag for listing available layouts
Add error codes to mypy
Rewrite commands as click groups instead of eager options
Add mock filestore
Remove dummy code and write layout names to stdout!

good job Zach, but I'd prefer if you kept refactoring separate from feature work in the future. I'll let it pass since it's pre-1.0 :ship:


Example Unclear

Whilst Example 1 above will execute it expects a value to be entered for TargetedGroups as that is a REQUIRED value.  This should be made more clear.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 693f12cd-c3c6-c797-0fe0-88f0bbfa276c
Version Independent ID: 1a1e7369-e6da-0d71-5340-ac83678e433f
Content: Set-DeviceConditionalAccessRule (exchange)
Content Source: exchange/exchange-ps/exchange/devices/Set-DeviceConditionalAccessRule.md
Service: exchange-powershell
GitHub Login: @chrisda
Microsoft Alias: chrisda

@officedocsbot assign @yogkumgit

Hello @dusted1969 thanks for your feedback.
I created a pull request to correct the example, as I could verify, it should be merged and live in a couple of days.
Thanks again

@chrisda Please merge the PR #3583

@get-itips  Thank you very much for the contribution and updating the documentation with PR. @Dusted1969 Hope this update is helpful for you. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.

@officedocsbot close


execute ./kylin.sh start   error on ubuntu linux 0.14

when i run ./kylin.sh start on ubuntu linux 0.14 for kylin-0.7.1-incubating version  ,but i got an error ./kylin.sh: 23: ./find-hive-dependency.sh: Syntax error: "(" unexpected .
is anything uncompatible ?

This Github Issues is closed already.
Please open JIRA for any Kylin relative issue: http://issues.apache.org/jira/browse/KYLIN
or send email to our mail list<EMAIL_ADDRESS>and remember to subscribe our mail list via<EMAIL_ADDRESS>


Using the tutorial example to analyze the nicer data, an error occurs,

Hello, I am trying to use the tutorial example and pyxspec, heasoftpy to analyze nicer data, and an error occurs,
detailed code：
# get the on-axis rmf
res = hsp.quzcif(mission='nicer', instrument='xti',detector='-',
             filter='-', date='-', time='-',expr='-',codename='MATRIX')
rmf = [x.split()[0] for x in res.output if 'nixtiref'  in x][0]

# get the on-axis arf
res = hsp.quzcif(mission='nicer', instrument='xti',detector='-',
             filter='-', date='-', time='-',expr='-',codename='SPECRESP')
arf = [x.split()[0] for x in res.output if 'nixtiaveonaxis'  in x][0]

IndexError                                Traceback (most recent call last)
Cell In[7], line 4
1 # get the on-axis rmf
2 res = hsp.quzcif(mission='nicer', instrument='xti',detector='-',
3              filter='-', date='-', time='-',expr='-',codename='MATRIX')
----> 4 rmf = [x.split()[0] for x in res.output if 'nixtiref'  in x][0]
6 # get the on-axis arf
7 res = hsp.quzcif(mission='nicer', instrument='xti',detector='-',
8              filter='-', date='-', time='-',expr='-',codename='SPECRESP')
IndexError: list index out of range
Note:
heasoftpy                 1.2.1                    pypi_0    pypi

@mfacorcoran?

i'll check it out

I set up caldb locally, for example:
 caldbinfo INST NICER XTI
...  using gpcaldbinfo 1.0.0
** caldbinfo 1.0.2
...  using infocaldb   1.0.1
...... environ-var/logical CALDB defined
.........   CALDB path = /home/ethan/workdir/caldb
... environ-var/logical CALDBCONFIG defined
...   CALDBCONFIG file = /home/ethan/workdir/caldb/software/tools/caldb.config
...... environ-var/logical CALDBALIAS defined
.........   CALDBALIAS file = /home/ethan/workdir/caldb/software/tools/alias_con
fig.fits
...... CALDB is configured for the XTI instrument onboard NICER
......... Cal Index File: /home/ethan/workdir/caldb/data/nicer/xti/caldb.indx
......... Instrument directory: /home/ethan/workdir/caldb/data/nicer/xti
... CALDB appears to be set-up & accessible
** caldbinfo 1.0.2 completed successfully
At the same time, I also called the python built-in library os, and edited the following code in the jupyter cell:
import os

os.environ['CALDB'] = '/home/jian/workdir/caldb'

Still have the same error，
# get the on-axis rmf
res = hsp.quzcif(mission='nicer', instrument='xti',detector='-',
             filter='-', date='-', time='-',expr='-',codename='MATRIX')
rmf = [x.split()[0] for x in res.output if 'nixtiref'  in x][0]

# get the on-axis arf
res = hsp.quzcif(mission='nicer', instrument='xti',detector='-',
             filter='-', date='-', time='-',expr='-',codename='SPECRESP')
arf = [x.split()[0] for x in res.output if 'nixtiaveonaxis'  in x][0]

IndexError                                Traceback (most recent call last)
Cell In[7], line 4
1 # get the on-axis rmf
2 res = hsp.quzcif(mission='nicer', instrument='xti',detector='-',
3              filter='-', date='-', time='-',expr='-',codename='MATRIX')
----> 4 rmf = [x.split()[0] for x in res.output if 'nixtiref'  in x][0]
6 # get the on-axis arf
7 res = hsp.quzcif(mission='nicer', instrument='xti',detector='-',
8              filter='-', date='-', time='-',expr='-',codename='SPECRESP')
IndexError: list index out of range

If you do the following in your notebook:
import heasoftpy as hsp
import os
and set chatter=5 when running hsp.quzcif:
res = hsp.quzcif(mission='nicer', instrument='xti',detector='-',
...:              filter='-', date='-', time='-',expr='-',codename='MATRIX', chatter=5)
print (res.output)
for o in res.stdout:
print(o)
what do you get?
res.output should be:
['https://heasarc.gsfc.nasa.gov/FTP/caldb/data/nicer/xti/bcf/rmfbase/nixtirmfbase20170601v001.fits 2',
'https://heasarc.gsfc.nasa.gov/FTP/caldb/data/nicer/xti/cpf/rmf/nixtiref20170601v003.rmf 2',
'https://heasarc.gsfc.nasa.gov/FTP/caldb/data/nicer/xti/cpf/rmf/nixtidiagresp20170601v001.rmf 2', '']
this is what I get when I run that notebook cell

I can't run it, it seems that there is an error here,
Create the nicerl2 task
tstart = Time.now()
print(f'Start at: {tstart.iso[:19]}')
nicerl2 = hsp.HSPTask('nicerl2')

nicerl2.clobber="yes"
#nicerl2.cldir=outdir
#nicerl2.mkffile=mkf
# add the KP values to the mkf file during nicerl2 processing
nicerl2.geomag_path="https://heasarc.gsfc.nasa.gov/FTP/caldb/data/gen/pcf/geomag/"
nicerl2.geomag_columns="kp_noaa.fits(KP)"

resl2 = nicerl2(indir=nicerobsID, noprompt=True, cldir=outdir, mkfile=mkf)

tend = Time.now()
print(f'End at: {tend.iso[:19]}')
print(f'nicerl2 took: {(tend.mjd-tstart.mjd)*86400} seconds')

if resl2.returncode != 0:
    print('\n')
    for o in resl2.output[:]:
        print(o)

Start at: 2023-05-05 12:58:38
End at: 2023-05-05 12:58:38
nicerl2 took: 0.16225771978497505 seconds
ERROR: input directory<PHONE_NUMBER> does not exist at /home/ethan/workdir/heasoft-6.31.1/x86_64-pc-linux-gnu-libc2.31/bin/nicerl2 line 253.
Task nicerl2 1.27 terminating with status -1
nicerl2 1.27
  ang_dist = 0.015
  attfile = $INDIR/auxil/ni$OBSID.att
  autoscreen = yes
  br_earth = 30
  calstation = FLIGHT
  cldir = /home/ethan/tmp/nicer/nicerl2_output/4142010107_out
  cleanup = 1
  cleanup_ufa_files = 0
  clfile = $CLDIR/ni$OBSID_0mpu7_cl.evt
  clobber = 1
  cor_range = *-*
  dec = OBJ
  detlist = launch
  elv = 15
  filtcolumns = NICERV4
  geomag_columns = kp_noaa.fits(KP)
  geomag_path = https://heasarc.gsfc.nasa.gov/FTP/caldb/data/gen/pcf/geomag/
  gtifiles = NONE
  gzip_thresh = 50
  history = 1
  hkdir = $INDIR/xti/hk
  hkpat = $HKDIR/ni??????????_?mpu?.hk{,.gz}
  incremental = 0
  indir =<PHONE_NUMBER>
  issmanfile = CALDB
  leapinit = AUTO
  min_fpm = 7
  mkfile = /home/ethan/tmp/nicer/nicerl2_output/4142010107_out/ni4142010107.mkf
  mpugtimerge = OR
  mpugtiscr = DEFAULT
  mpulist = 0-6
  nicercal_filtexpr = EVENT_FLAGS=bxxxx00
  nicerclean_args = NONE
  nicersaafilt = yes
  nimaketime_gtiexpr = NONE
  niprefilter = 1
  niprefilter2 = 1
  niprefilter2_coltypes = FILTCOLUMNS
  noise25scr = DEFAULT
  noiseextscr = DEFAULT
  orbfile = $INDIR/auxil/ni$OBSID.orb
  overonly_expr = NONE
  overonly_range = *-30
  overonlyscr = DEFAULT
  picalfile = CALDB
  pifastcalfile = CALDB
  pirange = 20:1500
  prefilter_columns = FILTCOLUMNS
  ra = OBJ
  robofile = CALDB
  roundrobbinscr = DEFAULT
  saafilt = no
  saaregfile = NONE
  st_valid = yes
  tasks = ALL
  timebiascalfile = CALDB
  trackfilt = yes
  trumpetfilt = yes
  trumpetkeep = GOOD
  ufafile = $CLDIR/ni$OBSID_0mpu7_ufa.evt
  ufdir = $INDIR/xti/event_uf
  ufpat = $UFDIR/ni??????????_?mpu?_uf.evt{,.gz}
  underonly_range = 0-500
  underonlyscr = DEFAULT
  vehiclefile = CALDB

MPU List: 0-6
Requested task operations: SCREEN,CALMERGE,MKF
Processed prefilter_columns: NICERV4,-MPU_OVER_COUNT,-MPU_UNDER_COUNT,-MPU_XRAY_COUNT,-MPU_ALL_COUNT
Processed niprefilter2_coltypes: BASE,NICERV4
Processed geomag_columns: kp_noaa.fits(KP)

the problem above seems to be that the input obsid directory,<PHONE_NUMBER> does not exist.  But that seems to be separate from the hsp.quzcif error.
  Please see https://heasarc.gsfc.nasa.gov/docs/heasarc/caldb/caldb_remote_access.html for instructions on how to download the caldbconfig and caldbalias files and how to define the environment variables.  Hopefully that solves your problem

Very strange, I can run it in sciserver_cookbooks, but an error will appear on my own computer.

Hi Jian,
you should add these statements:
os.environ['CALDB'] = '/FTP/caldb'
os.environ['CALDBCONFIG'] = '/FTP/caldb/software/tools/caldb.config'
os.environ['CALDBALIAS']= '/FTP/caldb/software/tools/alias_config.fits'

to your copy of the notebooks, then caldb access should work properly.

When I run this code,
import os

os.environ['CALDB'] = 'https://heasarc.gsfc.nasa.gov/FTP/caldb'
os.environ['CALDBCONFIG'] = '/home/jian/workdir/caldb/software/tools/caldb.config'
os.environ['CALDBALIAS']= '/home/jian/workdir/caldb/software/tools/alias_config.fits'

[' ...... environ-var/logical CALDB defined', ' .........   CALDB path = https://heasarc.gsfc.nasa.gov/FTP/caldb', 'CALDB_PATH=(https://heasarc.gsfc.nasa.gov/FTP/caldb)', ' ... environ-var/logical CALDBCONFIG defined', ' ...   CALDBCONFIG file = /home/jian/workdir/caldb/software/tools/caldb.config', 'CALDBCONFIG=(/home/jian/workdir/caldb/software/tools/caldb.config)', ' ...... environ-var/logical CALDBALIAS defined', ' .........   CALDBALIAS file = /home/jian/workdir/caldb/software/tools/alias_con', '           fig.fits', 'CALDBALIAS=(/home/jian/workdir/caldb/software/tools/alias_config.fits)', ' ...... CALDB is configured for the XTI instrument onboard NICER', ' ......... Cal Index File: https://heasarc.gsfc.nasa.gov/FTP/caldb/data/nicer/xt', '           i/caldb.indx', 'CALDB_INDEX=(https://heasarc.gsfc.nasa.gov/FTP/caldb/data/nicer/xti/caldb.indx)', ' ......... Instrument directory: https://heasarc.gsfc.nasa.gov/FTP/caldb/data/ni', '           cer/xti', 'CALDB_INST_DIR=(https://heasarc.gsfc.nasa.gov/FTP/caldb/data/nicer/xti)', 'gtcalf 1.2.3  : Searching CALDB for quality values =  0', ' ...  using cifsl3 1.0.1', ' ...... Searching for CAL_QUAL =          0', ' WARNING - cifsl3 1.0.1: Row    5 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row    8 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row    9 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   11 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   12 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   16 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   17 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   23 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   27 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   32 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   33 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   45 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   47 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   48 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   49 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   50 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   51 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   52 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   55 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   56 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   58 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   59 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   60 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   61 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   62 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   63 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   64 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   65 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   66 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   67 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   68 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   69 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   70 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   71 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   72 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   73 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   74 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   75 quality    0 matches quality =    0', ' WARNING - cifsl3 1.0.1: Row   76 quality    0 matches quality =    0', ' ...... Quality: Found         39 matches out of         76 rows', ' ...... Telescope: Found         39 matches out of         39 rows', ' ...... Instrument XTI matches instrument= XTI Row number          5', ' ...... Instrument XTI matches instrument= XTI Row number          8', ' ...... Instrument XTI matches instrument= XTI Row number          9', ' ...... Instrument XTI matches instrument= XTI Row number         11', ' ...... Instrument XTI matches instrument= XTI Row number         12', ' ...... Instrument XTI matches instrument= XTI Row number         16', ' ...... Instrument XTI matches instrument= XTI Row number         17', ' ...... Instrument XTI matches instrument= XTI Row number         23', ' ...... Instrument XTI matches instrument= XTI Row number         27', ' ...... Instrument XTI matches instrument= XTI Row number         32', ' ...... Instrument XTI matches instrument= XTI Row number         33', ' ...... Instrument XTI matches instrument= XTI Row number         45', ' ...... Instrument XTI matches instrument= XTI Row number         47', ' ...... Instrument XTI matches instrument= XTI Row number         48', ' ...... Instrument XTI matches instrument= XTI Row number         49', ' ...... Instrument XTI matches instrument= XTI Row number         50', ' ...... Instrument XTI matches instrument= XTI Row number         51', ' ...... Instrument XTI matches instrument= XTI Row number         52', ' ...... Instrument XTI matches instrument= XTI Row number         55', ' ...... Instrument XTI matches instrument= XTI Row number         56', ' ...... Instrument XTI matches instrument= XTI Row number         58', ' ...... Instrument XTI matches instrument= XTI Row number         59', ' ...... Instrument XTI matches instrument= XTI Row number         60', ' ...... Instrument XTI matches instrument= XTI Row number         61', ' ...... Instrument XTI matches instrument= XTI Row number         62', ' ...... Instrument XTI matches instrument= XTI Row number         63', ' ...... Instrument XTI matches instrument= XTI Row number         64', ' ...... Instrument XTI matches instrument= XTI Row number         65', ' ...... Instrument XTI matches instrument= XTI Row number         66', ' ...... Instrument XTI matches instrument= XTI Row number         67', ' ...... Instrument XTI matches instrument= XTI Row number         68', ' ...... Instrument XTI matches instrument= XTI Row number         69', ' ...... Instrument XTI matches instrument= XTI Row number         70', ' ...... Instrument XTI matches instrument= XTI Row number         71', ' ...... Instrument XTI matches instrument= XTI Row number         72', ' ...... Instrument XTI matches instrument= XTI Row number         73', ' ...... Instrument XTI matches instrument= XTI Row number         74', ' ...... Instrument XTI matches instrument= XTI Row number         75', ' ...... Instrument XTI matches instrument= XTI Row number         76', ' ...... Instrument: Found         39 matches out of         39 rows', ' ...... Detector: Found         39 matches out of         39 rows', ' ...... Filter: Found         39 matches out of         39 rows', ' ... CNAME MATRIX matches CNAME = MATRIX from CIF row number         59', ' ... CNAME MATRIX matches CNAME = MATRIX from CIF row number         62', ' ... CNAME MATRIX matches CNAME = MATRIX from CIF row number         75', ' ...... CAL_CNAME: Found          3 matches out of         39 rows', ' ...... Boundary: Found          3 matches out of          3 rows', ' ... Found          3 rows after final time selections', 'https://heasarc.gsfc.nasa.gov/FTP/caldb/data/nicer/xti/bcf/rmfbase/nixtirmfbase20170601v001.fits 2', 'https://heasarc.gsfc.nasa.gov/FTP/caldb/data/nicer/xti/cpf/rmf/nixtiref20170601v003.rmf 2', 'https://heasarc.gsfc.nasa.gov/FTP/caldb/data/nicer/xti/cpf/rmf/nixtidiagresp20170601v001.rmf 2', '']

I think you left out the call to hsp.quzcif in the above, but this looks like the output you'd received if you have chatter = 5, so I think this is what you expect.

Thank you, I would like to ask a question! I use Conda to automatically generate a base environment when creating an environment on my local computer. I want to change the default base environment to HEASoft's Python environment (similar to the settings on SciServer). What should I do? thank you!


Upgrade github actions to v3

Questions
Answers

Description?
Please check the fixed ticket.

Type?
improvement

BC breaks?
no

Deprecations?
no

Fixed ticket?
PrestaShop/PrestaShop/issues/34538

Sponsor company
Your company or customer's name goes here (if applicable).

How to test?
PHP tests pass without warning of deprecated node12

The impacted tests are green, that's good for me


KAFKA-9498; Topic validation during the topic creation triggers unnecessary TopicChange events

This PR avoids generating unnecessary TopicChange events during the topic validation. It does so by adding a registerWatch field in the GetChildrenRequest request. This allows to not register the watch when topics are queried from the topic validation logic.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

ok to test

Are there places where we call this method where we only care about one topic?

No.

@ijuma or @hachikuji Do you know why we verify that the topic does not collide (Topic.hasCollision) with others during the topic validation? This check is done for every topic when a CreateTopicPolicy is used and it sounds quite expensive when the cluster has thousands of topics. It this is really necessary, what about using the metadata cache? The guarantee is not strong anyway because a colliding topic could be created between the verification and the insertion.
    if (zkClient.topicExists(topic))
      throw new TopicExistsException(s"Topic '$topic' already exists.")
    else if (Topic.hasCollisionChars(topic)) {
      val allTopics = zkClient.getAllTopicsInCluster
      // check again in case the topic was created in the meantime, otherwise the
      // topic could potentially collide with itself
      if (allTopics.contains(topic))
        throw new TopicExistsException(s"Topic '$topic' already exists.")
      val collidingTopics = allTopics.filter(Topic.hasCollision(topic, _))
      if (collidingTopics.nonEmpty) {
        throw new InvalidTopicException(s"Topic '$topic' collides with existing topics: ${collidingTopics.mkString(", ")}")
      }
    }

@dajac we check for collisions because Kafka converts metric tags (topic names) dots to underscores - https://github.com/apache/kafka/blob/trunk/core/src/main/scala/kafka/metrics/KafkaMetricsGroup.scala#L95
Originally introduced by this commit and asked here

@dajac We call that method in every case, right? Not just in case the policy is defined (createTopicWithAssignment calls it as well). I agree that the cost is too high for what it's doing. Relying on the metadata cache for that use case seems reasonable.
In addition, have we verified that it's safe to call getAllTopicsInCluster without registering watches? Since watches are one shot, this could break things. cc @junrao

@ijuma Yes, that method is called in all cases. My mistake. OK, I will try to address this in the another PR if you don't mind.
Regarding calling getAllTopicsInCluster without registering watches, I think that it is safe. There are two important places where the watch must be registered in the controller: 1) in the initializeControllerContext method; and 2) in the processTopicChange method. The former does the initial registering and the latter re-register every time the event is triggered (and processed). To my knowledge, this is sufficient. I have added the testGetAllTopicsInClusterTriggersWatch which mimics this and also verifies that an extra call without registering a watch does not break it.
That being said, having a second confirmation would be appreciated for sure.

@junrao Thank you for the review. I have addressed your comments.
Regarding the performance improvement, I have run a load test which created 5000 topics (10 requests/sec, 50 topics/request) with 4 partitions and a replication factor of 3.
Prior to this change, 5000 ChangeTopic events were placed in the queue. As a consequence, the request batching was degraded substantially leading to significant amount of UpdateMetadataResponseReceived and LeaderAndIsrResponseReceived. During this test, the controller was busy during 20 minutes to process the backlog of events.
With this change, 300 ChangeTopic events were placed in the queue. The controller took 3 minutes to process all the events and had room to process other events.

@junrao I have updated the PR based on your comments. Thanks!

@junrao Could you start the tests please?

ok to test

retest this please

https://builds.apache.org/job/kafka-pr-jdk11-scala2.13/4861/

@hachikuji or @junrao I think this one is in good shape now. What about merging it?
One test failed:

kafka.api.ConsumerBounceTest.testRollingBrokerRestartsWithSmallerMaxGroupSizeConfigDisruptsBigGroup

I think that it is a flaky one.

retest this please

retest this please


Use preload instead of prefetch for these resources.

See Chrome warnings

And Addy's comment about using preload and prefetch.
https://github.com/GoogleChromeLabs/preload-webpack-plugin/issues/8#issuecomment-277105884

Thanks for submitting this issue @pxwee5. @Hovspian confirmed that this is fixed now (D29769, for internal tracking) so I'm closing this.


WMS ID#11441

Add here your ID#11441
General requirements

Information in the workshop is adequate and updated
Code is correct and working
Links are correct
Make sure you enter the help email link in your manifest.json
Please make sure WMS URLs are updated as needed after your PR is approved

Checklist - Refer to the QA document for the complete list
Please confirm that the following is completed before submitting your PR

[ ] All filenames are lower case (including folders, images, files, etc.)
[ ] Filenames are descriptive
[ ] Your workshop folder structure should be similar to the one used in the sample workshop (https://github.com/oracle-livelabs/common/tree/main/sample-livelabs-templates/sample-workshop)
[ ] Are you using multiple versions (desktop/, sandbox/, tenancy/)? Make sure that each of them contains a manifest.json and an index.html
[ ] Image references in markdown contain an alternate text

Hi,
Please make sure the filenames (the images) are descriptive
Thank you

The rest lgtm!
I noticed that the workshop outline on WMS are not updated, please update it and I'll merge


Keycloak benchmark documentation should mention the kvm2 setup

Description
When running the rebuild.sh script as described in step 3 of the documentation, I've run into a pop-up window requesting authentication and saying that System policy prevents management of local virtualized systems. After entering the password, the command was left blinking indefinitely, and no progress was ever made.
It turns out that in my local machine I had never set up kvm2 properly (https://minikube.sigs.k8s.io/docs/drivers/kvm2/), and my user had to be added to the libvirt group. A simple command sudo usermod -aG libvirt $USER was enough to fix the group membership and allow the rebuild.sh script to finish.
Thus I'm requesting that the documentation is updated with a small section on to verify the kvm2 setup before running the script.
Also, when running the task command in step 4, it first failed with task: Failed to run task "tlsdisableagent": exit status 1, which turns out to happen because of a missing JAVA_HOME env variable. It is probably a good idea to mention the JAVA_HOME requirement in the docs as well.
Discussion
No response
Motivation
No response
Details
No response

@sguilhen thanks for raising this, both valid comments. @ahus1 do you think adding the details around these in the https://www.keycloak.org/keycloak-benchmark/kubernetes-guide/latest/error-messages docs would be a good place or should I add it directly under the relevant installation docs for both

kvm privilege issue
missing java_home requirement

@kami619 - IMHO we should put users on the right path before they get the error message. Therefore I'd recommend to update the docs:

https://www.keycloak.org/keycloak-benchmark/kubernetes-guide/latest/prerequisite/prerequisite-minikube could be enhanced with a Linux section explaining that KVM2 is necessary, how to set it up in a nutshell, link to the minikube page above, and a one-line shell command to verify that is has been setup correctly.
Java is not yet listed here https://www.keycloak.org/keycloak-benchmark/kubernetes-guide/latest/installation ... so it should be added, with a page of the usual structure, plus a one-liner to test that it is set up.

Once that is complete, and we think people will still face this situation, consider the following:

Add a check rebuild.sh that shows a good error message is the prerequisites on Linux about kvm2 are not met
Add a task to the taskfile before running Maven to see if Java is set up correctly and show a descriptive error message.

Let me know if you have the time for this, and if you want to split this task. Whatever you choose, I'd be happy to review things.

@ahus1 ok, I will take this up, thank you for the suggestion and I think it is better to make the user to not go through the user in general, I shall look into creating a PR for this.


RootModel.getattr redirect

Initial Checks

[X] I have searched Google & GitHub for similar requests and couldn't find anything
[X] I have read and followed the docs and still think this feature is missing

Description
I propose a __getattr__ for RootModel to redirect property access to root.
It could even test for the property in .root.
Things would just work, without having to add .root
from typing import List, Union

from pydantic import BaseModel, RootModel

class C(BaseModel):
   items: List[int]

class D(BaseModel):
    data: str

class A(RootModel[Union[C,D]]):
   def __getattr__(self, name):
       if not name.startswith("model_") and not name.startswith("__") and hasattr(self.root, name):
           return getattr(self.root, name)

data = {"items":[1,2,3]}
a = A.model_validate(data)
assert a.root.items == a.items

Link to existing problems with basic inheritance wrt. to __getattr__ in BaseModel.
Affected Components

[ ] Compatibility between releases
[ ] Data validation/parsing
[ ] Data serialization - .model_dump() and .model_dump_json()
[ ] JSON Schema
[ ] Dataclasses
[ ] Model Config
[ ] Field Types - adding or changing a particular data type
[ ] Function validation decorator
[ ] Generic Models
[X] Other Model behaviour - model_construct(), pickling, private attributes, ORM mode
[ ] Plugins and integration with other tools - mypy, FastAPI, python-devtools, Hypothesis, VS Code, PyCharm, etc.

We need to look into this. In my understanding this should just work already.
For instance, we have the following in tests and it works.
    class Nested(RootModel):
        root: int = 42

    Model = RootModel[Nested]

    m = Model()
    assert m.root.root == 42

I don't think we should add this by default. The point is that when you create a RootModel, it's still a normal model with a field called root, it just happens to serialize and deserialize without an extra "container" layer. This __getattr__ would change that behavior and makes it less consistent with how other models work.
That said, @commonism I wouldn't be opposed to adding some tests that if you do define this getattr, it works properly.
Also, @commonism, note that __getattr__ is only called if you don't find the attribute the "normal" way and would otherwise get an AttributeError. So I don't think you actually need to check for the "model_" prefix, and depending on what you are doing you might be able to drop the "__" prefix as well:
class A(RootModel[Union[C,D]]):
    def __getattr__(self, name):
        if hasattr(self.root, name):  # maybe: and not name.startswith("__")
            return getattr(self.root, name)
        raise AttributeError(name)

Yeah, I've misunderstood the proposal. I agree with @dmontagu that RootModel is just a specially precooked BaseModel subclass that has the root field and just serializes slightly differently. I think that in Python in should behave like a Model with a field. There is no functional benefit in it mimicking the root field type except a syntax sugar. I see how this could complicate the debugging of the code.
Also, a potential implementation raises some questions:

Should we prioritize root attributes (using __getattribute__) over RootModel own attributes?
Is it an error to have the same attributes on root and RootModel?
What should happen for RootModel that is an attribute of another RootModel?
Should we pass comparison operations to the underlying type as well?
Should we pass __repr__ and/or __str__ to the underlying type?

IMO, RootModel isn't the same as the root attribute type and shouldn't act like it.

For now I'll avoid entering this rabbit hole.
It starts with getattr, then there is length, getitem, set{attr,item} and then there is too many questions to answer consistently.
For my own use the savings do not relate to the costs currently.


proj: support cross compile

Description of changes
Add a cmakeFlag to point to a sqlite3 binary that can be run at build time

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[x] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Doesn't adding sqlite to nativeBuildInputs help?

Doesn't adding sqlite to nativeBuildInputs help?

I tried that out, and it didn't work. It still tries to execute the binary from the hostPlatform during build.

Builds for me, didn't testing further otherwise.


Error when gulp.src does not find any files

gulp.task('taskname', function() {
  return gulp.src('*.html')
    .pipe(header('Hello'));
});

When there are no html files in '*.html', it throws the following error:
node_modules/gulp-header/index.js:58
    var file = _file.clone({ contents: false });
                    ^
TypeError: Cannot read property 'clone' of undefined

Apparently, TransformStream never gets called, but EndStream does. _file is undefined and, therefore, does not have a clone property.
Proposed solution is to add the following code to the beginning of EndStream function:
if(!_file) {
  return cb();
}

+1 we get the same error.

+1 same error in our build scripts

The same error. Returned to 1.2.2 version.

Looks like the changes for 1.5.1 should resolve this, can you confirm?

It looks good. Thanks.


Add example and documentation for custom token usage

Fixes #78

This looks good to me - thanks!
@tj - you may want to add @jrouleau as a collab on this repo. 2 PRs today and a few issue comments :)


Query :: null compensation logic is not working for some cases where non-nullable type is expected

E.g.:
context.Tags.Where(t => t.Gear.HasSoulPatch)

or
context.Tags.Where(t => t.Gear.HasSoulPatch ? true : false)

Navigation expansion converts t.Gear.HasSoulPatch expression to bool? and we fail to construct Expression, because bool is expected

Fixed in 3f532a773a453b8f77e06e169ff9e5061618662b


Skip bogus pretty print tests only in old node versions

This should keep our coverage to 100% in newer node versions at least

Coverage increased (+14.8%) to 100.0% when pulling 67e8d2985cef7f37bccd0ede9d2db6f0ce117551 on juanjoDiaz:fix_pretty_print_tests into<PHONE_NUMBER>b65984a67c2921751c9972f049d546 on zemirco:master.


Update URL param when audit tab is toggled

[x] change URL to /protocols/audit when "audit" is enabled
[x] add URL param for ?state=pending (if filtering is enabled on backend, if not can filter on UI)

Implemented


Symfony4 support and fix issues

Symfony 4 has been released and the WebSocket package has many obsolete calls and dependency injection has changed. When are you going to do something about it?

Hey,
Symfony has been released yesterday night, master should work on. If any issue comes up, send a pull request to fix. As soon as the bundle work good on sf4, I'll tag it

When i run composer require "gos/pubsub-router-bundle":"master@dev"
I get:

./composer.json has been updated
Loading composer repositories with package information
Updating dependencies (including require-dev)
Your requirements could not be resolved to an installable set of packages.

  Problem 1
    - Installation request for gos/pubsub-router-bundle master@dev -> satisfiable by gos/pubsub-router-bundle[dev-master].
    - gos/pubsub-router-bundle dev-master requires symfony/framework-bundle ~2.3|~3.0 -> satisfiable by symfony/framework-bundle[2.3.x-dev........
etc,

Does this mean the pubsub-router-bundle needs to be updates as well?

Great. I just installed the dev-master version and got the following notice on composer update:
Script cache:clear returned with error code 1
!!
!!  In GosWebSocketExtension.php line 210:
!!
!!    Notice: Undefined index: server
!!
!!
!!

new rc tagged ! Anyone have the same issue as @neandher ?

Tried it yesterday with latest rc and also had the error neandher received

got the same.
^1.8.12 works fine with symfony 3.4 but crashes with 4.0

We have to figured out where come from the leak, it's annoying, and I dont have time those days ...

It looks like the reason for this is circural dependencies mentioned in #292.
I will handle this tomorrow.

added quick fix in #296

thanks very much @tkuska i have merged and released it as new RC

@neandher @rogierkn do you still have memory leak issue with latest rc ?

any news with about memory leak and last rc ? If it's fix, we are ready to tag a new version and be compatible with symfony 4 ! :pray:

@ProPheT777 I will check it in a few hours and let you know

@ProPheT777 Created a new S4 project. Works like a charm!

nice to hear

@rogierkn @ProPheT777
S$ ERROR in session.subscribe after connect to ws server
console log
Disconnected for Connection was closed properly [WS-1007: ] with code 0

Please support yaml extension as well
https://github.com/GeniusesOfSymfony/PubSubRouterBundle/pull/18
Thank you

¿¿can close this issue??


Allow non-interactive install of fetch apply

The currently recommended way to install fetch apply is to download https://raw.githubusercontent.com/P5vc/FetchApply/master/install and run it interactively, answering questions as they come up. It's not possible at the moment to run it non-interactively.
A possible solution would be to allow passing all options as arguments on the command line, for example:
bash /tmp/install --install-path=/var/lib --log-file=/var/log/fetchapply.log --git-url=https://github.com/P5vc/ServerConfigurations.git --cron-frequency="10 1 * * *"

A --non-interactive flag could also be used to tell the installer to take default values when they are not specified, instead of interactively asking the user if it's not specified. This would mean that all values would be taken as defaults unless overridden using the command line argument as described above.

Thank you for the suggestion! This is a great idea! We'll be sure to implement this change within a few days. :+1:

@emmetog Once again, thank you for the suggestion! The change has now been implemented! Feel free to test it out and let us know what you think.
As a thank you for contributing (https://github.com/P5vc/Documentation/blob/master/Contributing/contributionGuide.md#perks-of-contributing), you're eligible for 30 days of a free, @P5.vc email and 30 days with a free, premium Priveasy Docs account. If you're interested, just respond on this thread and we'll hook you up. :+1:

@P5vc thanks! I'm using the new install script and it works a treat, thanks!


Condition DDPM

It seems that condition was not used in DDPM, in models.temporal.py. Function "apply_conditioning" only replaces the initial state.

Yes, currently state conditioning in the model corresponds to setting a state in the trajectory to a particular value (there is no conditioning).

Yes, in guided sampling for return maximization, apply_conditioning only serves to replace the current state, with the rest of the conditioning coming from the value function.
In the Maze2D goal-conditioned RL experiments, the conditions contain both a current and a goal state.


can't recognize "|" character correctly in “az webapp create” command in powershell

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Describe the bug

following command works well in cmd window but fails in powershell
az login
az configure --defaults group="" location=""
az webapp create --name "" --plan "" -r "DOTNETCORE|3.1"
it reports following error
'3.1' is not recognized as an internal or external command,
operable program or batch file.
To Reproduce

run following command in powershell:
az login
az configure --defaults group="" location=""
az webapp create --name "" --plan "" -r "DOTNETCORE|3.1"
Expected behavior

a webapp hosted .net core app should be created successfully.
Environment summary

az --version
azure-cli                         2.14.1
core                              2.14.1
telemetry                          1.0.6
Extensions:
application-insights              0.1.11
azure-iot                         0.10.3
Additional context

it looks like a powershell issue but my suggestion is not using "|" character in the string.

webapp

hi @hanhsia could you pls does this https://github.com/Azure/azure-cli/blob/dev/doc/quoting-issues-with-powershell.md#workaround-the-stop-parsing-symbol work for you?

You can solve this by "double escaping" in powershell.
--runtime '"node|12-lts"'
Working example:
az webapp create  --name "$FrontendAppName" --resource-group "$ResourceGroup" --plan $Plan --runtime '"node|12-lts"'

yes, i use this way to workaround it. The doc is great.

happy to see if works for you. :) is it ok to close the issue?

sure, please close it.


Ad player controls overlapping

Hi @Kiro705
I have tried the plugin and it works fine but the UI of ad player is distorted as can be seen in below image

Player controls like volume etc. are overlapping on playbar and resulting in bad experience.  Below is the link to the video, any help is appreciated
https://api.vadoo.tv/landing_page?vid=WK1aotg5SFG0qJRUCwysEa01jdFOh0SI

@Kiro705 Any help is appreciated as this is causing a poor user experience

Hello @Anil-matcha ,
Sorry for the delay in my response.
I was able to see the mis-styled ad controls on your test page. However, the styling looks correct in the plugin's sample page. The sample uses this stylesheet. I would recommend comparing your player with this example to see what style might be missing or being overwritten.
Thank you,
Jackson
IMA SDK team

Hi @Kiro705, I have imported the same scripts from the sample page but the styling is broken. The videojs skin I am using for the player is a custom made skin but it shouldn't impact the ad player as ad player is overlay on videojs player if I am not wrong.
Is there any custom css for ad player which can be controlled to prevent this issue ?