KAFKA-2825: Extended existing ducktape replication tests to include controller failover

I can't really comment on the Python but the logic looks good to me.

@apovzner: A few minor comments, plus a possible approach to clean up minikdc.py, otherwise looks good.

Closed this one and opened new PR https://github.com/apache/kafka/pull/570


Add CI provider

We need a CI provider that:

can be used in the STX repository -- Is Github actions is an option?  Perhaps yes, if this is going to be a public repo

CI provider added so we can close the issue


Create charguia.txt

@ReserveHotelOnline Please provide us with an URL of a page or another proof (.pdf or a screenshot are OK) showing that the school recognizes the domain which you are submitting as an official email domain for the students.

@ReserveHotelOnline Thank you for the clarification. Pull request merged.


restructure docs at RTD

At last DRWG meeting, it was mentioned that the docs should be restructured to integrate better and make it easier for users to navigate and find what they need. My initial crack at this for the table of contents.
This simplifies the top level entries.
Introduction to AIRR Standards
    AIRR Data Model
    AIRR Data Commons
    AIRR Curated Germline Sets
Getting Started
    Examples & Tutorials
    Templates
AIRR Standards
    Study Reporting (MiAIRR)
    Software guidelines
    AIRR Data Model
        Repertoire schema
        Rearrangement schema
        GermlineSet schema
    AIRR Data Commons
        Query Interface
        AIRR-compliant repositories
    AIRR Curated Germline Sets
        Homo sapiens
        Mus musculus
        Macaca mulatta
Software
    Python API
    R API
    CAIRR
    OGRE
    ADC API
Community
    Supporting Applications
    Resources
Glossary

I guess Introduction and Getting Started would be user-focused while AIRR Data Model and Software would be more reference info.

Overall, I like it!  A few suggestions:

Introduction
Getting Started

Examples & Tutorials
Templates

Standards

Data Reporting (MiAIRR)
Software guidelines
Data Formats

Annotation and Analysis (Rearrangement schema)
Metadata (Repertoire schema)
Germline Databases (GermlineSet schema)

Software & Databases

Python API
R API
CAIRR
OGRE
ADC API

Community

Supporting Applications
Resources

Glossary

On this topic... sort of...
I tried this a while ago, and there was some difficulty I don't recall, but I think we should revisit switching the theme to bootstrap.  We now have a lot of pages that are very long, and you very easily lose the navigation sidebar. We also have some tables, which could use the extra width.

That's good, I like the top-level standards. I modified my TOC in-place.
I want to make sure that the high-level concepts, like how all the schema are related to each other in the AIRR Data Model, or the architecture of the AIRR Data Commons, get described properly up-front instead of being hidden in reference doc, so I guess the introduction and getting started sections is the best for that.
OGRE and inferred allele submission seems kinda hidden, but not sure how to call it out more.

OGRE and inferred allele submission seems kinda hidden, but not sure how to call it out more.

Is there a way to show the top two levels instead of just one? Or to make the second level visible on mouse-over?

Is there a way to show the top two levels instead of just one? Or to make the second level visible on mouse-over?

When you click on a TOC link to go to that page, the TOC shows the next level down, so kinda... The front page currently shows the fully expanded TOC. I think if we add as a top-level topic in the Introduction then that might be good enough. I'm not sure about mouse-over, possible if we can grab some code from somewhere.

I like it! I would only add some short text to the Software & Databases items, similar to section iii Data formats. I think Python API and R API are self explanatory, but the other items aren't. Maybe CAIRR data submission, `OGRE inferred allele submission'. I would make it easier for new users to identify what can be done with the resources available.

Make sure to look at docs from the "metadata" branch

I've committed a rough restructuring of the top level, take a look.
http://docs.airr-community.org/en/metadata-docs/

metadata-docs branch is merged


UsbFile::listFiles not returning contents of folder

Problem
When calling UsbFile::listFiles on a particular folder on an SD card I get an empty list returned, even though the folder contains a number of subfolders.
Expected behavior
All files and folders in the folder should be returned
Actual behavior
No files or folders are returned
Code where problem occurs
I have tracked the problem to FatDirectory::listFiles() -> FatDirectory::init() -> FatDirectory::readEntries()
In readEntries() the expected subfolders just aren't found (some entries are found, but are removed again as the isDeleted flag is set on them). I don't know what is happening, could it be some FAT corruption issue? The SD card and its folders are read correctly in Windows and in the normal Android file manager.
Edit: If needed I can provide a 32 GB image of a card where the issue occurs for reproduction.
The more I look at this the more it seems like a FAT issue. For example, moving or renaming the folder does not fix the problem, but copying the subfolders back into the same folder (in Windows) and removing the original ones does work, and calling listFiles on the source folder will then work again. This is probably related to the device that writes the card which is based on some custom version of Android I think. There haven't been any issues reading SD cards from the device using an earlier version of the library, however.

Fixed with v0.9.4


feat(clojure_lsp) add support for shadow-cljs in clojure_lsp

Shadow-cljs is a build tool for cljs with many features such as npm integration and live reloading.
Link: https://github.com/thheller/shadow-cljs

Do you actually need this as a root marker? Is there a situation where you would have this root marker and not the others?
Yes, there are situations where you are using shadow-cljs standalone without boot, deps.edn/clojure cli, and leiningen.


Trouble building locally

I'm trying to modify the Dockerfile so that I can build a local copy of silverbullet without having to install deno on my host; so far the change is pretty simple:
diff --git a/Dockerfile b/Dockerfile
index f9d6c28..57f6f48 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -7,8 +7,8 @@ FROM denoland/deno:alpine-1.28.3
 #   docker run -v myspace:/space -it zefhemel/silverbullet
 VOLUME /space

-# Copy the bundled version of silverbullet into the container
-ADD ./dist/silverbullet.js /silverbullet.js
+ADD . ./
+RUN deno task build bundle && cp ./dist/silverbullet.js /silverbullet.js

Oddly, when I run deno task bundle in the container as above, the dist/ folder doesn't contain silverbullet.js. This is odd because the ./script/build_docker.sh script seems to assert that this would be the case.

Right. deno task doesn't allow you to specify multiple tasks in one go, you have to run them separately. This should work:
RUN deno task build && deno task bundle && cp ./dist/silverbullet.js /silverbullet.js

Is installing Deno on your host system really problematic enough to jump through all these hoops and accept the significant waiting for every docker build cycle?

Is installing Deno on your host system really problematic enough to jump through all these hoops and accept the significant waiting for every docker build cycle?

I'm a little paranoid and prefer to keep my host as free of dependencies as possible, especially when it comes to the javascript ecosystem. :)

Your suggestion worked, thanks!


Feature/applics 811 aria translations

Describe your changes
A number of terms that were missing translations have now been translated and the required code implemented.
See ticket APPLICS-811-aria-translations for a full list.
Useful information to review or test
Full test instructions have been added to the ticket.
Type of change 🧩

[X] New feature (non-breaking change which adds functionality)

Checklist before requesting a review

[X] I have performed a self-review of my own code
[X] I have double checked this work does not include any hardcoded secrets or passwords
[X] I have provided details on how I have tested my code
[X] I have referenced the ticket number above
[X] I have provided a description of the ticket
[X] I have included unit tests to cover any testable code changes

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

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

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


SuperBuilder have attibute to specify access modifier for builderImpl class

As per discussions here https://stackoverflow.com/questions/52734182/lombok-superbuilder-example-with-json-annotations involving Jan Rieke, it is acknowledged that there is a compatibility issue with Jackson, which by inference means requiring the explicit declaration of the builderImpl class that would otherwise be generated so that the access modifier can be made package private allowing Jackson to create a builderImpl instance
It would be nice to have the ability to specify the access modifier in the SuperBuilder annotation using the AccessLevel class.
I understand it could be argued that Jackson should support the ability to specify a method in its JsonDeserialize annotation to create a builder instance and I will likely raise a feature request to that effect also.  I certainly understand if that is also your preferred option.

Just a few further thoughts:
A parameter builderMethod on @JsonDeserialize would be helpful to avoid the unnecessary broadening of visibility. However, we still need @JsonPOJOBuilder on the builder implementation class because of withPrefix.
We could also add an onBuilderImpl annotation parameter for @SuperBuilder so that users don't have to copy the rather complex builder class definition from delombok.

I agree with your first comment, that would be the preferred method and I will raise an enhancement request to that effect today.
Could you please elaborate on what the onBuilderImpl annotation parameter would do?
With regards to the @JsonPOJOBuilder issue, on that page I referenced Andreas gives a solution which I've been using successfully in conjunction with both @Builder and @SuperBuilder annotations (relevant snipped below).
public static ObjectMapper createObjectMapper() {
final ObjectMapper mapper = new ObjectMapper();
mapper.setAnnotationIntrospector(new JacksonAnnotationIntrospector() {

    @Override
    public JsonPOJOBuilder.Value findPOJOBuilderConfig(final AnnotatedClass ac) {
        if (ac.hasAnnotation(JsonPOJOBuilder.class)) {
            return super.findPOJOBuilderConfig(ac);
        }
        return new JsonPOJOBuilder.Value("build", "");
    }
});

return mapper;

}

FYI: "Set" methods for @SuperBuilder can be prefixed with the next version (PR #2357), making @JsonPOJOBuilder unnecessary if you specify the prefix in @SuperBuilder.
However, you still need to broaden the visibility. Making the builder implementation class package-private by default simply feels wrong, and another annotation parameter also. So I bumped the issue FasterXML/jackson-databind#2354 again, asking if they'd accept a PR.

Hmm, now I'm a bit confused. I just checked it again and it also works with the builder implementation class private. I simply cannot remember why a thought a few month ago that making it package-private is necessary. I simply may have made a mistake back then.
So from my POV, this issue can be closed.

I found the problem: Eclipse's compiler does not complain about private access, but javac does. javac gives MyDto.MyDtoBuilderImpl has private access in MyDto on @JsonDeserialize. Looks like a bug in ecj to me.
Although the Jackson people seem quite positive about adding support for specifying a static factory method, lets keep this issue open until it is really implemented.

@janrieke Thanks for the updates.  With regards to your comment about PR #2357 its really nice to see such flexibility implemented for those who need it.  For me though I'm more in agreement with @rzwitserloot and his comments at https://github.com/rzwitserloot/lombok/pull/2174#issuecomment-524099192 so I'll likely stick with using the JacksonAnnotationIntrospector.
I've read the comments on the Jackson issue I raised and I'm hoping they will allow the specifying of a static builder method.  I'm not a fan of subverting the protection system without compelling reasons, and I don't see a valid reason when a builder method is available for use.  If they do come up with some fix so you can instantiate the private impl builder with java then you will be in a situation where you are specifying a class in the JsonDeserialize annotation that you can't see in the source code.

@janrieke 's the effective maintainer of the SuperBuilder feature at this point. Not because we don't care, on the contrary - because he's sending the pull requests before we even realize there's a problem :) – so I'll leave this in your capable hands and if you need anything from us, let us know.
One thought does come up: If there's this whole song and dance routine you have to go through to make (super)builder act precisely in the right way so that it works best with Jackson, maybe we should just introduce a single annotation that means: "Just make it all so that jackson is as happy as can be". I prefer to wait for the jackson feature request/PR IF that would mean you can just put @(Super)Builder on a class and jackson will just work, no further configuration needed, but if it's a mix of telling (S)B to make the class package private as well as introduce a 'with' prefix as well introduce some annotations on various generated nodes, I'm in favour of @Jacksonify @SuperBuilder or @SuperBuilder(forJackson = true) or something in that vein.

A dedicated Jackson switch could be a solution. However, what I do not want is to use "with" as setterPrefix in this case just to make Jackson happy. But that would mean generating Jackson annotations in Lombok.
That seems to be dangerous terrain to me. I'll think about it.

I opened a thread on Google Groups to continue the discussion: https://groups.google.com/forum/#!topic/project-lombok/L7OVH_noUAY

For reference, I have opened a feature request with Jackson at https://github.com/FasterXML/jackson-future-ideas/issues/47.


feat(FN-970): fix month indexing in session

Introduction
Report month is stored in the session, it was indexed at 0 but we want to use index 1 to make it easier to read
Resolution
When we set the month increase the value by 1

Codecov Report

:exclamation: No coverage uploaded for pull request base (main-stbr@851e9f8). Click here to learn what that means.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             main-stbr    #2299   +/-   ##
============================================
  Coverage             ?   15.98%
============================================
  Files                ?     1256
  Lines                ?    29493
  Branches             ?     4516
============================================
  Hits                 ?     4714
  Misses               ?    21803
  Partials             ?     2976

Flag
Coverage Δ

api-test
0.48% <0.00%> (?)

ui-test
39.30% <0.00%> (?)

unit-test
3.48% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
:mega: Codecov offers a browser extension for seamless coverage viewing on GitHub. Try it in Chrome or Firefox today!


Use pulumi/schema-tools

Fixes #404
https://github.com/pulumi/pulumi-spotinst/pull/311 servers as an example that the flags I'm passing work.

One tweak that may warrant another example PR just to make sure the github action context I suggested is in fact the correct one.
It looks like Mikhail's version automagically works on main, but I don't know why, and I'm not sure this is true for pulumi/schema-tools.

Second example PR is https://github.com/pulumi/pulumi-spotinst/pull/312. Your suggestion worked perfectly. 🙇

LGTM! Thank you for including the extra confidence check.
@thomas11 - you probably want to use github.ref or github.base_ref` in azure-native actions, since schema-tools takes any branch name. Contexts are vrey confusing, but this is an okay starting point.
Full disclosure: I have definitely dumped the full github context so I could see what was available to me in a given action, since the documentation is so confusing (different information is available based on the context).


Allow mixed case partitions for Hive sync_partition_metadata

The Hive system.sync_partition_metadata procedure allows discovering partitions on the file system that match the Hive naming convention. For compatibility with Hive's MSCK REPAIR TABLE, partition names must be in lowercase by default. However, for users that wish to use it with mixed case names, we can add a new allow_mixed_case argument to allow this behavior.

Hello, I would like to work on this. Thanks!


Cannot login

Cannot login, updated app, accepted new terms & conditions, hard coded version to 1.19.0 per apple App Store.
Anyone else?

Same for me. From today.

Same here: Tried reverted version 1.7.3 with the same result...
[Homebridge Panasonic AC Platform] Login failed. Skipping device discovery.
[12/11/2023, 10:55:40 AM] [Homebridge Panasonic AC Platform] The Comfort Cloud server might be experiencing issues at the moment.The plugin will try to log in again in 12 minutes.
[12/11/2023, 10:55:40 AM] [Homebridge Panasonic AC Platform] If the issue persists, make sure: configured is the correct email and password in plugin settings,field "Emulated Comfort Cloud app version (override)" in settings is empty or have the latest version of Panasonic Comfort Cloud from the App Store (like 1.19.0), the latest version of this plugin is installed, all terms and conditions after logging into the Panasonic Comfort Cloud app are accepted. Restart Homebridge if you change plugin settings.

Other plugins (also for comfort cloud) do not work either!
Panasonic had a technical break tonight - they probably changed something in the API (AGAIN!)

Try override version in settings to 1.19.1

This new error msg really concerns me

This new error msg really concerns me

Yes. Me too. I have already written to Panasonic to turn it off or give you a choice. But I didn't get an answer.

Almost for sure the error is located on Panasonic's end, so we have to wait until they resolve this issue.
In the meantime, we have corrected the plugin code to better indicate errors and manage login process.

It looks like they have started to crack down on unofficial applications. As such, I don't think there will be a "resolution" to this issue from the manufacturer. They have every right to prevent unauthorised apps from talking to their services, as unfortunate that may be for community projects like this one.

This is a problem with this and any other plugin that use similar APIs. We can hardly blame the manufacturer for making sure the T&Cs are followed.

I have already written to Panasonic to turn it off or give you a choice. But I didn't get an answer.

What do you expect? I assume the purpose of this move is to make people use the official apps. Best case would be if they are working on official HomeKit support, which would save us a lot of time and trouble.

In another house I have Gree and there plugin has not been updated for 2 years and everything works flawlessly. Ehh.. Panasonic still has to change something...

I'm concerned Panasonic might be clamping down on useful home integration tools such as this (well, at least until they support HomeBridge!).
That said, @mkz212 has pointed out that, if you log out of and back into the Comfort Cloud app, it says 'Login ID or password is incorrect', which suggests there maybe an issue at their end.

Now, when Panasonic unlock my account, again I can login in Panasonic Comfort Cloud app but in Homebridge I have error 500 😞

I've just tested and am getting the same here. Do you think we could use WireShark to look at what's going on?

@jandersonhill I still hope that it will be good because browsing the Internet I encountered many such threads from different periods of time and those problems were somehow solved, most often Panasonic changed something in API and that's it. You can try.

Noticed the same issues regarding this at the github repo for the HACS plugin (home assistant).
For example: https://github.com/sockless-coding/panasonic_cc/issues/147#issuecomment-1851310578
They claim, that after accepting 2 to 3 new EULA stuff it started working again. So I figured I logout off the comfort cloud app, and log back in. But I already accepted the new EULA stuff and unfortunately this didn't fix the error 500 for this homebridge plugin.

Noticed the same issues regarding this at the github repo for the HACS plugin (home assistant). For example: sockless-coding/panasonic_cc#147 (comment)
They claim, that after accepting 2 to 3 new EULA stuff it started working again. So I figured I logout off the comfort cloud app, and log back in. But I already accepted the new EULA stuff and unfortunately this didn't fix the error 500 for this homebridge plugin.

Oke I did some more investigastion, and it seems highly unlikely that anything is working at the moment. When I manually push the same auth request to the the auth URL that is used for this plugin as-well as the Home Assistant version, I get the following error:
<TITLE>ERROR: The request could not be satisfied</TITLE>

403 ERROR

The request could not be satisfied.

Request blocked.
We can't connect to the server for this app or website at this time. There might be too much traffic or a configuration error. Try again later, or contact the app or website owner.

If you provide content to customers through CloudFront, you can find steps to troubleshoot and help prevent this error by reviewing the CloudFront documentation.

Generated by cloudfront (CloudFront)

So maybe, panasonic did change the auth / API location. Or they still have issues.
What is kinda strange, because I was able to re-login using the Comfort Cloud APP. 🤔 ....

I don't see any EULA to accept. Still have error 500.

@mitchelp
If they changed something or have a problem, at the beginning they all had an error 500 in HB and in the Panasonic application they could connect.
But due to too many error logging attempts from HB, these accounts were blocked and then in HB error 429 and in the application information about blocking for 24h.
Now there is no 24 h ban and we return to error 500 again.

@mkz212 there must be a change somewhere at Panasonic. Because why is it possible to (re) sign in to the comfort cloud app, and impossible for the homebridge plugin to sign in.  So this needs some research.

@mitchelp That's what I wrote - either they changed something or something doesn't quite work. A few months ago they also changed something and it didn't work - then a new header had to be added. Do you have the option to check what commands the working application sends?

@mkz212  I tried, but i'm not that well experienced with intercepting traffic. It also uses SSL what doesn't make it easier. 😄 Maybe someone else is more experienced with this?

I don't get it @mkz212 how can it work for people here:
https://github.com/sockless-coding/panasonic_cc/issues/147
I looked at the code, and this plugin uses the following payload towards auth/login:
        payload = {
            "clientId": self._settings.clientId,
            "language": "0",
            "loginId": self._username,
            "password": self._password
        }

And this plugin does the following:
    return axios.request<ComfortCloudAuthResponse>({
      method: 'post',
      url: 'https://accsmart.panasonic.com/auth/login',
      headers: this.getBaseRequestHeaders(),
      data: {
        'loginId': this.config.email,
        'language': '0',
        'password': this.config.password,
      },
    })

Maybe the clientId is missing? Or did I mis it in de code somwhere? Or it is added here?:
https://github.com/homebridge-panasonic-ac-platform/homebridge-panasonic-ac-platform/blob/6af81143455360c064bf1d58b7bf249841a26776/src/types.ts#L28

@mitchelp
I also watched this thread and looked through this code. I also tried to combine with adding clientId and changing headers - but I failed to achieve anything. I also do not understand why one works there and another does not.

I noticed your replies in the thread @mkz212, the only noticeable difference was the clientId. Really strange indeed that one is working and another isn't. Seems to me impossible.
I also don't think this is related to a blockage for my IP, because the official app is working just fine.

I noticed your replies in the thread @mkz212, the only noticeable difference was the clientId. Really strange indeed that one is working and another isn't. Seems to me impossible.
I also don't think this is related to a blockage for my IP/User, because the official app is working just fine.

If you had a ban, the error code in HB would be 429, not 500.

They blocked my account again for 24 hours due to incorrect login attempts. 😓

I've added 2 accounts to my AC to prevent this ^

I've added 2 accounts to my AC to prevent this ^

It doesn't make a difference to me because I don't use a bad application from Panasonic anyway 😅

I had a look at the Cloudflare side of things and found the following:
‘Error 403 from Cloudflare is a forbidden status code. You get this response when trying to scrape a website as a bot. That can happen when using tools like Python, NodeJS, cURL, etc.

Looking briefly at the app that appears to be working, they are using the python library: pypi.org/project/urllib3/ as the HTTP client, which may not be blocked.
It's possible that changing the HTTP library we are using to something else might work, but it's only a matter of time before that also fails. There is an alternative mentioned here that appears to impersonate a specific JA3 fingerprint, but I haven't had the time to read through it yet:
https://medium.com/cu-cyber/impersonating-ja3-fingerprints-b9f555880e42

https://github.com/sockless-coding/panasonic_cc
In this plugin, it is said that some users work. Here I see that there is some SSL certificate.

Looking briefly at the app that appears to be working, they are using the python library: pypi.org/project/urllib3/ as the HTTP client, the fingerprint of which may not be blocked by Cloudflare.

No luck:
import json
import urllib3
url = "https://accsmart.panasonic.com/auth/login"
headers = {
"Accept": "application/json; charset=UTF-8",
"Content-Type": "application/json",
"User-Agent": "G-RAC",
"X-APP-TYPE": "0",
"X-APP-TIMESTAMP": "1",
"X-APP-NAME": "Comfort Cloud",
"X-CFC-API-KEY": "Comfort Cloud",
"X-APP-VERSION": "1.19.0",
}
with open("pana.json", "r") as file:
data = file.read()
http = urllib3.PoolManager()
response = http.request(
"POST",
url,
headers=headers,
body=json.dumps(json.loads(data)).encode("utf-8"),
)
print(response.status)
print(response.data.decode("utf-8"))
pana.json:
{
<EMAIL_ADDRESS>"language":"0",
"password":"XXXXX"
}
->500
{"code":5001,"message":"DB system error due to db system"}

No luck:

You're getting the original error code, as opposed to the Cloudflare rejection error code - it's possible they are related, but I'd like to be sure. If I have a moment this evening, I'll try to run your code in my environment.
It's also possible we're hitting two layers of protection...

https://github.com/sockless-coding/panasonic_cc/issues/147#issuecomment-1852777101

Ok, looking at the work done in the link above, it does seem they're using certificates plus other measures to prevent third-party use of the API. What I find silly is that Panasonic's AC division are relegating themselves from a brand that could be recommended for use in smart homes to one that cannot.

Well it is going to take some time, but someone will reverse engineer this, and than we can use it again.

Agreed

I added PR that solves the problem!

A quick release would be nice ;-)

Same problem here too. Though new version 1.8.4 is released but it isn't showing in homebridge.

It should be visible soon.

Trying to release, but the workflow failed. Am looking into this now...

Version 1.8.4 released! If this solves the problem, please close the Issue.

Version 1.8.4 released! If this solves the problem, please close the Issue.

Did not solve it for me. I still get this when trying to reinstall the integration

This error originated from a custom integration.
Logger: custom_components.panasonic_cc.config_flow
Source: custom_components/panasonic_cc/config_flow.py:60
Integration: Panasonic Comfort Cloud (documentation, issues)
First occurred: 11:26:19 (1 occurrences)
Last logged: 11:26:19
Unexpected error creating device
Traceback (most recent call last):
File "/config/custom_components/panasonic_cc/config_flow.py", line 60, in _create_device
devices = await self.hass.async_add_executor_job(api.get_devices)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/panasonic_cc/pcomfortcloud/session.py", line 207, in get_devices
self.login()
File "/config/custom_components/panasonic_cc/pcomfortcloud/session.py", line 106, in login
self._create_token()
File "/config/custom_components/panasonic_cc/pcomfortcloud/session.py", line 164, in _create_token
raise ResponseError(response.status_code, response.text)
custom_components.panasonic_cc.pcomfortcloud.session.ResponseError: Invalid response, status code: 500 - Data: {"code":5001,"message":"DB system error due to db system"}

I thought it was a Homebridge plug? And you will show me Home Assistant.

Version 1.8.4 released! If this solves the problem, please close the Issue.

Did not solve it for me. I still get this when trying to reinstall the integration

This error originated from a custom integration.
Logger: custom_components.panasonic_cc.config_flow
Source: custom_components/panasonic_cc/config_flow.py:60
Integration: Panasonic Comfort Cloud (documentation, issues)
First occurred: 11:26:19 (1 occurrences)
Last logged: 11:26:19
Unexpected error creating device
Traceback (most recent call last):
File "/config/custom_components/panasonic_cc/config_flow.py", line 60, in _create_device
devices = await self.hass.async_add_executor_job(api.get_devices)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/local/lib/python3.11/concurrent/futures/thread.py", line 58, in run
result = self.fn(*self.args, **self.kwargs)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/config/custom_components/panasonic_cc/pcomfortcloud/session.py", line 207, in get_devices
self.login()
File "/config/custom_components/panasonic_cc/pcomfortcloud/session.py", line 106, in login
self._create_token()
File "/config/custom_components/panasonic_cc/pcomfortcloud/session.py", line 164, in _create_token
raise ResponseError(response.status_code, response.text)
custom_components.panasonic_cc.pcomfortcloud.session.ResponseError: Invalid response, status code: 500 - Data: {"code":5001,"message":"DB system error due to db system"}

My bad. I just did not get the update at first (looked for it in the wrong place first, found it then in HACS). Now up an running again.
Many many thanx!

Hello
Can confirm the update has fixed the issue.
Really really appreciate your efforts to keep this working.
Thanks, Miguel

Closing

Oops. Closing.


Strict mode error when using on-load

Hi folks, I know this issue has been discussed before but I'm still getting issues when attempting to use the on-load library with create-choo-app that's using the latest bankai build.
onload(div, (el) => {     console.log('Loaded')   }, (el) => {     console.log('Unloaded')
results in:
Uncaught TypeError: 'caller', 'callee', and 'arguments' properties may not be accessed on strict mode functions or the arguments objects for calls to them
I can solve this by switching to budo or (according to another test someone did) by disabling babelify. However, it seems like something that should work out of the box.
full gist here: https://gist.github.com/callil/52db52b6786d053a84ad8190a46058d8

To avoid this error in strict mode, you should apply the yo-yoify transform

ok I will give that a shot - also a solution here I missed: https://github.com/shama/on-load/issues/15
  const uniqueIdentifier = 1   onload(div, (el) => {     console.log('Loaded')   }, (el) => {     console.log('Unloaded')   }, uniqueIdentifier)

I think using yoyoify no longer helps because choo doesn't support onload attributes in template tags anymore. The identifier solution should probably be documented in the onload readme


[MRG] Change minimum numpy ver to 1.11.0  for NiLearn compatibility

LGTM then.

Are you still getting that error?

I can only check that  at NSP :-(

If nobody protests I'll merge within 1hour.


Re-implement count() as an OpenRosa extension

Will this break backwards compatibility? Or was it introduced in the orxe2 branch?

I don't expect so, but here's some background:
count() is an XPath 1.0 function, which should be implemented by the browser's evaluator, but appears to be missing from Firefox.  Chrome will pass the tests on this branch with or without the function definition in openrosa-extensions.js.
In master, there is no special handling for count(), so backwards compatibility is unlikely to be an issue.
To add confidence, in orxe2 there are already tests for count() at https://github.com/medic/openrosa-xpath-evaluator/compare/orxe2#diff-163f6516542d896c89e3aecbeecae9f8R79-R106.
Removing this code improve support for count() - note the call to isNativeFunction() at https://github.com/medic/openrosa-xpath-evaluator/compare/orxe2#diff-a4a35cb2fe6ada1511345deee2fe1cc9R162: this is code introduced in orxe2 which checks if the first item in an expression is a call to a specific list of native functions.  This should not be necessary if the XPath Expr is properly parsed.  Also, it can be seen from the NATIVE_FUNS regex that this only happens when an Expr starts with the native function: https://github.com/medic/openrosa-xpath-evaluator/pull/54 / https://travis-ci.org/github/medic/openrosa-xpath-evaluator/builds/699959051.
I've added these tests to this PR as well, and they now pass.
Ultimately a similar change will be needed for each of the functions with special handling in extended-xpath - this code should not need any knowledge of specific functions (neither native, nor OpenRosa extensions), and their presence indicates bugs elsewhere.


Cloudflare validation with DDNS CNAME?

Hi,
I have an issue with DDNS and CNAME entries, my setup:

I have a Synology NAS, that syncs my real IP to dns1.myds.me
I have dns2.eu as my main domain handled by cloudflare, with a CNAME * -> dns1.myds.me

And the problem:
When I try to create a cert for sub.dns2.eu, the tool tries to create the challenge record in dns1.myds.me, which is not available in Cloudflare, nor have I any possibility to change the dns for dns1.myds.me
what I checked:

https://api.cloudflare.com/client/v4/zones/?per_page=50&page=1 returns the zone with name dns2.eu
tried to set up _acme-challenge.dns2.eu and _acme-challenge.sub.dns2.eu CNAME pointing to dns2.eu
tried "AllowDnsSubstition": true/false

Any idea how to fix it?

OMG, there is a typo in the documentation: AllowDnsSubstitution is spelled AllowDnsSubstition

Nice catch, I'll be sure to update the docs!

Fix released in <IP_ADDRESS>


Update pricing

Post updated pricing docs on July 31, Friday, before release on Aug 1.

Users will no longer be charged for query duration.
Users will be charged for the following:

Total number of query counts (includes tasks)
9 cents per GB for data transfer

Request from Tim:

Contact Balaji for pricing deck walkthrough
Work w marketing to ensure our language and theirs aligns.

Notes:

CSV Download does not affect charges. (Initial query brings the data to the browser. The CSV Download button simply packages up the data resident in the browser with no additional Flux query invocation.) Only charge for the initial query (API call - 1 count) and the data transfer cost.
Writing Data Back into Influx does not affect charges. Only charge for initial query (API call - 1 count). No data transfer cost bc data is not transferred (moved through an internal gateway call).

Relevant URLs
We could add details on this page (we currently don't tell them how they're charged).
https://v2.docs.influxdata.com/v2.0/account-management/pricing-plans/

@gunnaraasen can you review the timeline and content with @noramullen1 ?


vending machine - moore machine

solves #4

!bounty 625

Points for TestBench and FSM. Done efficiently.

!bounty 500


Roll Packages from fde908d13896 to 8e24a73d1eea (4 revisions)

https://github.com/flutter/packages/compare/fde908d13896...8e24a73d1eea
2024-04-26<EMAIL_ADDRESS>Bump github/codeql-action from 3.25.2 to 3.25.3 (flutter/packages#6615)
2024-04-25<EMAIL_ADDRESS>[go_router] Update redirect documentation example code for clarity (flutter/packages#6213)
2024-04-25<EMAIL_ADDRESS>[go_router_builder] add support to go_router_builder for initializing a StatefulShellBranch with observers (flutter/packages#5865)
2024-04-25<EMAIL_ADDRESS>Roll Flutter from dba4f7747441 to 5d3bca4500b1 (17 revisions) (flutter/packages#6612)
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-packages-flutter-autoroll
Please CC flutter-ecosystem<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To file a bug in Flutter: https://github.com/flutter/flutter/issues/new/choose
To report a problem with the AutoRoller itself, please file a bug:
https://issues.skia.org/issues/new?component=1389291&template=1850622
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/main/autoroll/README.md

Commit queue failed; closing this roll.


Can't bind to 'options' since it isn't a known property of 'input'

I cannot use the [options] property of an  control to configure the ng2-currency-mask component.
The error "Can't bind to 'options' since it isn't a known property of 'input'" is thrown.
The ng2-currency-mask (v4.4.1) package installed correctly and the CurrencyMaskModule has been imported into my module.
Can you suggest any potential fix for this, please?
Thanks

I had to explicitly import 'CurrencyMaskDirective' into my component, and move my module import down a level (we have a multi-level module hierarchy).

Also is there a way to not default the input box to '£0' for example? This is wiping out my input box placeholder text.

Hi. Are you having this problem yet? I recreated the project with angular-cli and it's working with no errors.

CurrencyMaskDirective

Its is falling for me alse


[Dubbo-4575] Upgrade to use fastjson 1.2.58

What is the purpose of the change
fix #4575
Brief changelog
Upgrade to use fastjson 1.2.58
Verifying this change
UT pass
Follow this checklist to help us incorporate your contribution quickly and easily:

[x] Make sure there is a GITHUB_issue field for the change (usually before you start working on it). Trivial changes like typos do not require a GITHUB issue. Your pull request should address just this issue, without pulling in other changes - one PR resolves one issue.
[x] Format the pull request title like [Dubbo-XXX] Fix UnknownException when host config not exist #XXX. Each commit in the pull request should have a meaningful subject line and body.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Write necessary unit-test to verify your logic correction, more mock a little better when cross module dependency exist. If the new feature or significant change is committed, please remember to add sample in dubbo samples project.
[x] Run mvn clean install -DskipTests=false & mvn clean test-compile failsafe:integration-test to make sure unit-test and integration-test pass.
[x] If this contribution is large, please follow the Software Donation Guide.

Codecov Report

Merging #4579 into master will decrease coverage by 0.09%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             master    #4579     +/-   ##
===========================================
- Coverage     63.29%   63.19%   -0.1%
- Complexity      448      450      +2
===========================================
  Files           770      770
  Lines         33008    33008
  Branches       5216     5202     -14
===========================================
- Hits          20891    20861     -30
- Misses         9714     9737     +23
- Partials       2403     2410      +7

Impacted Files
Coverage Δ
Complexity Δ

...ache/dubbo/remoting/transport/AbstractChannel.java
75% <0%> (-12.5%)
0% <0%> (ø)

.../apache/dubbo/remoting/transport/AbstractPeer.java
58.69% <0%> (-8.7%)
0% <0%> (ø)

...onfig/spring/extension/SpringExtensionFactory.java
80.48% <0%> (-4.88%)
0% <0%> (ø)

.../dubbo/remoting/transport/netty4/NettyChannel.java
61.17% <0%> (-4.71%)
0% <0%> (ø)

...pache/dubbo/remoting/transport/AbstractServer.java
44.79% <0%> (-3.13%)
0% <0%> (ø)

...bo/rpc/protocol/dubbo/telnet/LogTelnetHandler.java
68.57% <0%> (-2.86%)
0% <0%> (ø)

...pache/dubbo/registry/support/AbstractRegistry.java
78.54% <0%> (-2.69%)
0% <0%> (ø)

.../dubbo/metadata/identifier/MetadataIdentifier.java
58.13% <0%> (-2.33%)
0% <0%> (ø)

...va/org/apache/dubbo/remoting/exchange/Request.java
84.44% <0%> (-2.23%)
0% <0%> (ø)

...in/java/org/apache/dubbo/common/utils/IOUtils.java
86.95% <0%> (-2.18%)
0% <0%> (ø)

... and 11 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 05e44db...3c093fc. Read the comment docs.


[bug]: no python3.9 or above I have 3.11.3

Is there an existing issue for this?

[X] I have searched the existing issues

OS
Linux
GPU
amd
VRAM
No response
What version did you experience this issue on?
2.3.5
What happened?
[thedarrens@TheGrimGrin InvokeAI-Installer]$ ./install.sh
which: no python3.11.3 in (/home/thedarrens/.bin:/usr/local/sbin:/usr/local/bin:/usr/bin:/usr/lib/jvm/default/bin:/usr/bin/site_perl:/usr/bin/vendor_perl:/usr/bin/core_perl:/var/lib/snapd/snap/bin)
which: no python3.9 in (/home/thedarrens/.bin:/usr/local/sbin:/usr/local/bin:/usr/bin:/usr/lib/jvm/default/bin:/usr/bin/site_perl:/usr/bin/vendor_perl:/usr/bin/core_perl:/var/lib/snapd/snap/bin)
A suitable Python interpreter could not be found
Please install Python 3.9 or higher before running this script. See instructions at  for help.
Press any key to exi
Screenshots
No response
Additional context
Fresh install of Arch Linux.
Contact Details
<EMAIL_ADDRESS>

duh does not work with 3.11.3 my appologies. missed that bit


Also send sqmid to extension telemetry events

This PR makes sure that sqmid is also sent to extension telemetry.
Continuation of https://github.com/microsoft/vscode/pull/195377
I tried VS Code out of source and it seems to be ok. I have not verified that the telemetry properly gets sent. We can do that in Insiders, and if something is off we can iterate fast.
Open questions:

Should sqmid be all lowercase like all the other extension telemetry events?
Is it ok to add it to vscode.proposed.telemetry.ts? Otherwise I could not get it compiling. If this looks good we can finalize it long term.

fyi @devinvalenciano

I just verified and in latest VS Code Insiders the sqmid is properly sent as part of extension telemetry:

I just verified and in latest VS Code Insiders the sqmid is properly sent as part of extension telemetry:

Are those curly braces part of the ID?


Support for Parcel v2

New Feature Proposal
For more information, see the CONTRIBUTING guide.
Description
Since Parcel v2 is out since a while (and since we already have Parcel v2 working up to some degree) it makes sense to also publish a piral-cli-parcel2 bundler plugin.
Background
From our first tries with Parcel v2 there are quite some challenges and problems ahead with this one. Right now its not clear if we can achieve an app shell and a pilet build just without any config changes. This would be a bummer as Parcel (v1) was always config-less, which made it so appealing (and even in Webpack you may not require a config to get started with).
Discussion
Anything in particular that we need to respect?

This is done - dedicated repo: https://github.com/smapiot/piral-cli-parcel2


Add npm-run-all to devDependencies

The npm-run-all (run-p) package used for npm run watch was not part of the devDependencies. It now is.

Good catch. Thank you :)


Fails to extract .msi Files on Win7 x64

Console
λ scoop install 7zip
Installing '7zip' (16.04).
Loading 7z1604-x64.msi from cache
Checking hash of 7z1604-x64.msi... ok.
Extracting... Exit code was .
Failed to extract files from C:\Users\Jokler\scoop\apps\7zip\16.04\7z1604-x64.msi.
Log file:
  ~\scoop\apps\7zip\16.04\msi.log

msi.log
=== Logging started: 07.08.2017  19:45:19 ===
=== Logging stopped: 07.08.2017  19:45:19 ===

The _tmp folder already contains the extracted files so I have no idea what the problem is.

This issue will appear when you execute scoop install 7zip or scoop install yarn (any .msi app) inside cmder, try to startup a cmd to run the command, and problem solved.
Don't know why meet this issue inside cmder.
duplicate #1107 #1445

Well that's annoying because I usually have cmd completely replaced by cmder.
Since the other issue is closed I'd assume there are no plans to fix it?

I have no idea about it. I use cmder, too.
pin: @lukesampson @r15ch13

I always use cmder and never had this problem. 😕
So I can't investigate it 😞

Which ConEmu version is shown in your settings titlebar?
I use the preview version 170305

I'm on 161206
I use, these settings in case that makes a difference.

I reverted back to 161206 and tried it using the now hooked cmd/powershell and normal cmder instances. No problem installing 7zip. 😕

I tried installing 7zip again and it actually works for me too now.
Are cmder updates silent? Because then it might have been a problem on cmders side.

I am assuming this is fixed by updating cmder so I will close this issue for now.


🐛 [firebase_messaging] Quick reply in notification using fcm for IOS

Is it possible to implement a quick reply notification to ios using fcm?
sample

Hi @bhanuka96
The package does not provide an API to do this. Please take a look at the native options
https://developer.apple.com/documentation/usernotifications/setting_up_a_remote_notification_server/generating_a_remote_notification
As far as I can tell this requires some native implementation and you would need to find a specific guide for this.
Thank you


[bn] Localize content/bn/docs/tasks/tools/_index.md

Signed-off-by: Saber Imtiaz<EMAIL_ADDRESS>Localizes content/bn/docs/tasks/tools/_index.md to Bengali
This is an intial PR as may need some improvements!
Tracking issue: #31677

/label tide/merge-method-squash

/lgtm

Signed-off-by: Noor E Jannat Nafia<EMAIL_ADDRESS>Localizes content/bn/docs/tasks/tools/included/verify-kubectl.md to Bengali
This is an initial PR as may need some improvements!
Tracking isse: 31677


Error: error sending request for url (https://ssc-dao.genesysgo.net/): operation timed out

When doing command metaboss -r https://ssc-dao.genesysgo.net snapshot mints --candy-machine-id 9agQaYR2Hpmfw5dsamUv33UmP9dWuvmjxTwEL7GkrbYr --v2
error sending request for url (https://ssc-dao.genesysgo.net/): operation timed out
Caused by:
0: error sending request for url (https://ssc-dao.genesysgo.net/): operation timed out
1: operation timed out
Same thing with multiple RPC, including my own quicknode rpc and project serum.

I have the latest version of metaboss as well, please advise

ty ser, -t worked


Add company name to create warehouse form submission

I want to merge this change because it adds company name to create warehouse form submission
SALEOR-1507

PR intended to be tested with API branch:
Screenshots

Pull Request Checklist

[ ] All visible strings are translated with proper context.
[ ] All data-formatting is locale-aware (dates, numbers, and so on).
[ ] Translated strings are extracted.
[ ] Number of API calls is optimized.
[ ] The changes are tested.
[ ] Data-test are added for new elements.
[ ] Type definitions are up to date.
[ ] Changes are mentioned in the changelog.

Test environment config

API_URI=https://master.staging.saleor.rocks/graphql/

Task linked: SALEOR-1507 Add company name to create warehouse form submission


Feat:crab-going-live-preparations

Added crab network

@1022ranji  and @wuminzhe account migration on Crab is ready for testing, since it's using the dev net, don't forget to ALLOW INSECURE CONTENT on your chrome

@1022ranji the last build has fixed the menu options order

not switch to Crab Dev after connect to my metamask

Account migration has nothing to do with Metamask, it is connected to Polkadot

my mistake

No worries mate.....

@1022ranji  the last code build contains the multi wallet support and also has fixed the darwinia staking link issue

@1022ranji  and @wuminzhe  check the last PR, I changed the Crab environmment to livenet. If it's all good then we should merge this to the main branch.


Component intro transitions regression on Edge

Intro transitions are correctly animated on initial load.  However, if a component is rendered later, its intro transitions are not animated.
0.7.3: http://jsfiddle.net/29fLzj6g/1/
Edge: http://jsfiddle.net/29fLzj6g/

Click "Tab One" and observe the differences between the two versions

Just wanted to note that I'm looking into it and, potentially, a way to resolve the issues with transitions holding up non-transitioning elements.

Fixed :smile:


Split NonbondedForce in to_openmm

Description
This takes https://github.com/openforcefield/openff-toolkit/pull/920 and tries to implement it here.
This is the script I'm currently using to set a trace:
from openff.toolkit.topology import Molecule

from openff.system.stubs import ForceField

mol = Molecule.from_smiles("ClC#CCl")
mol.name = "HPER"
mol.generate_conformers(n_conformers=1)

parsley = ForceField("openff_unconstrained-1.0.0.offxml")

out = parsley.create_openff_system(topology=mol.to_topology())
out.box = [4, 4, 4]
out.positions = mol.conformers[0]

out['Electrostatics'].method='pme'
out.to_gro("out.gro", writer="internal")
out.to_top("out.top", writer="internal")

from openff.system.drivers.gromacs import _run_gmx_energy, _write_mdp_file
from openff.system.drivers.openmm import get_openmm_energies
_write_mdp_file(openff_sys=out)
print(_run_gmx_energy(top_file='out.top', gro_file='out.gro', mdp_file='auto_generated.mdp'))
print(get_openmm_energies(out))

Pretty close to working now, but not quite there (at least while using GROMACS as a reference):
# GROMACS
Energies:

Bond:          		1.0871152877807617 kJ / mol
Angle:         		0.17977555096149445 kJ / mol
Torsion:       		0.0 kJ / mol
Nonbonded:     		None
vdW:           		-0.30658775568008423 kJ / mol
Electrostatics:		2.093082904815674 kJ / mol

# OpenMM
Energies:

Bond:          		1.0871620178222656 kJ / mol
Angle:         		0.17984433472156525 kJ / mol
Torsion:       		0.0 kJ / mol
Nonbonded:     		0.00011830426331371768 kJ / mol
vdW:           		None
Electrostatics:		None

and this report is missing -2.4035933017730713 kJ/mol in 1-4 interactions (including both vdW and electrostatics).
Checklist

[ ] Add tests
[ ] Lint
[ ] Update docstrings

Okay, adding the electrostatic contribution instead of subtract it (what was I doing?) gets pretty close:
Energies:

Bond:          		1.0871152877807617 kJ / mol
Angle:         		0.17977555096149445 kJ / mol
Torsion:       		0.0 kJ / mol
Nonbonded:     		None
vdW:           		-0.30658775568008423 kJ / mol
Electrostatics:		2.093082904815674 kJ / mol

> /Users/mwt/software/openff-system/openff/system/drivers/openmm.py(164)_get_openmm_energies()
    163     ipdb.set_trace()
--> 164     report = EnergyReport()
    165

ipdb> omm_energies['CustomBondForce'] + omm_energies['CustomNonbondedForce'] + omm_energies['Nonbonde
dForce']
Quantity(value=1.790536213885506, unit=kilojoule/mole)
ipdb> 2.093082904815674 + -0.30658775568008423
1.7864951491355896

Will wrap this up into an actual test next.

This deserves some more rigorous testing, but I will leave that for future work.


Is it possible to toggle between HTML output and Markdown output?

I have read through the CKEditor document and look like we can only define one output for the editor from the start. I think this feature is super nice to have

@hieuhlc Hi, you can toggle it via configuration in plugins.js, but since strapi v4.4.0 there are custom fields and the official ckeditor plugin for strapi is currently developing may be you should write it to them.

YEA! just all the following LoC in your Editor config of CKEditor 5 Plugin plugins.js or plugins.ts :
    ckeditor: {     enabled: true, config: {       editor: {         removePlugins: [""],         }     }  }

I had to define a custom field for markdown content and used a specific markdown editor for that field


when browsing db, report critical exception

Describe the bug
build release with .net9, using redis insight browsing keys(about 400k, data size 9G), server output exception:
crit: Session[0] [<IP_ADDRESS>:50364] [01FA0ABC] ProcessMessages threw a GarnetException: Garnet.common.GarnetException: Failed to write to response buffer    at Garnet.common.GarnetException.Throw(String message, LogLevel logLevel) in C:\garnet\libs\common\GarnetException.cs:line 57    at Garnet.server.RespServerSession.WriteOutputForScan(Int64 cursorValue, List`1 keys, Byte*& curr, Byte* end) in C:\garnet\libs\server\Resp\ArrayCommands.cs:line 435    at Garnet.server.RespServerSession.NetworkSCAN[TGarnetApi](TGarnetApi& storageApi) in C:\garnet\libs\server\Resp\ArrayCommands.cs:line 382    at Garnet.server.RespServerSession.ProcessOtherCommands[TGarnetApi](RespCommand command, TGarnetApi& storageApi) in C:\garnet\libs\server\Resp\RespServerSession.cs:line 700    at Garnet.server.RespServerSession.ProcessArrayCommands[TGarnetApi](RespCommand cmd, TGarnetApi& storageApi) in C:\garnet\libs\server\Resp\RespServerSession.cs:line 570    at Garnet.server.RespServerSession.ProcessMessages() in C:\garnet\libs\server\Resp\RespServerSession.cs:line 437    at Garnet.server.RespServerSession.TryConsumeMessages(Byte* reqBuffer, Int32 bytesReceived) in C:\garnet\libs\server\Resp\RespServerSession.cs:line 312
Steps to reproduce the bug
build release with .net9, using redis insight browsing keys(about 400k, data size 9G), server output exception:
Expected behavior
No response
Screenshots
No response
Release version
No response
IDE
Visual Studio 2022
OS version
Windows 11 26100
Additional context
No response

Is it only happening with that specific data set? Is it possible for you to find the exact key that causes this issue?

Was the specific key or value extremely large, what is the max size of keys and values.


Develope a complex example / blog post / tutorial

I think I will wait for some popularity before diving into this. So far the micro-lib works for me. Closing this for now

I think I will wait for some popularity before diving into this. So far the micro-lib works for me. Closing this for now


IOS Login failed

When I try to login via Facebook I get an error,
from the console browser the error is:
Error: LoginManager.logIn failed.
and in xcode the error is:
FBSDKLog: starting with Graph API v2.4, GET requests for /.../aem_conversion_configs should contain an explicit "fields" parameter.
Help please?

Anyone fixed it?


No way to see databases/apps past first page of pagination

@rwjblue -- Want to take a look at this?  We already paginate app.operations so it shouldn't be much to port this over to apps/dbs.

We might also consider adjusting the size of the database divs. (That was a suggestion from several customers.)

@chasballew: Adjusting them to be larger or smaller?

Smaller.


Windows 10 LTSB or LTSC.

Microsoft Windows [Version 10.0.10240]
When is the Linux Subsystem going to support my build. And how do I install it.

I suggest installing the most recent version of Windows 10:  https://support.microsoft.com/en-us/help/4028685/windows-get-the-windows-10-creators-update
Fall Creator's Update is due very soon.

Duplicate of #1281.


⚠ Jenkins Plugins Artifactory has degraded performance

In 345f2fe, Jenkins Plugins Artifactory (https://get.jenkins.io/plugins/artifactory/) experienced degraded performance:

HTTP code: 200
Response time: 1255 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 513b9ef after 12 minutes.


When three maps are open, stack horizontally rather than vertically

From the default view, when the user clicks "Add Map" a third map is added and the maps are stacked vertically:

This isn't the most effective use of the space, and the top map in particular is occluded by the control panel. Since most monitors are wider than they are high, it makes more sense to line up three maps horizontally instead, which would make each map closer to a square shape than a narrow rectangle.

Agree, and the more portrait-style orientation is closer to mobile device proportions... but then I guess doing the three-panel "phone" view is also an option for that.

Seems fair to me, we can take a look at doing this.

How can I see this change? When I try adding a map in Maperture, I still see the vertical stacking.

This was just deployed to GitHub Pages so you should now see the desired behavior at the link you referenced.

Oh wow, we did this at the exact same time @ebrelsford haha

This is so much better, thanks! Is there an automatic procedure to update client-specific forks of Maperture? Or should I file an issue in the relevant repo?

@kateler, @ebrelsford and I are planning to start doing a weekly port of recent changes over to client-specific forks to keep things up to date. Although if something is a blocking bug, we will bring it in asap.
@ebrelsford I don't think we've decided on a specific day yet, but my inclination would be maybe Wed or Thurs to allow time for any back and forth for ports that aren't straightforward. What do you think?

@aparlato Let's do Thurs and start today--and let's move further conversation around this to the relevant channels/repos.


Typos and Errors in examples folder

Within the examples folder, long_url_example.py file retrieves a shortened URL instead with qc.get_short_url(). Furthermore, the commented-out short URL returns an error page which states Chart not found.

Hi @akelshareif, the long_url_example is named as such because the URL shortener takes a long URL and turns it into a short one.  I agree it would be more aptly named short_url_example.
The commented-out short URL is an example output but is not meant to persist forever.  QuickChart short URLs expire after 3-6 months, which is why it returns Chart not found.

Thank you for the clarification regarding the short URL expiration @typpo. As a suggestion, maybe adding a comment within the long_url_example file indicating that the short URL displayed is an example and has expired could clarify this for future readers of the file. I also agree that the long_url_example would be more aptly named short_url_example. Thank you for your time and speedy response. The issue will now be closed.

Thanks! I've made both suggested changes.


fix(matchers): a more robust isDomNode

Summary
This PR fixes #6204
Replace the naive implementation of isDomNode just like jasmine following this fix - https://github.com/jasmine/jasmine/commit/ced2b114e47d3e0e108781dab3627aabc8515813
Test plan
Added a test for a fake dom Node that throws the correct error only with the code modifications in this PR.

Codecov Report

Merging #6246 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #6246   +/-   ##
=======================================
  Coverage   64.22%   64.22%
=======================================
  Files         220      220
  Lines        8481     8481
  Branches        3        4    +1
=======================================
  Hits         5447     5447
  Misses       3033     3033
  Partials        1        1

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 77d2e8e...8d3d92d. Read the comment docs.


Update Ecker2015 solid-phase diffusivity functions to more closely match the paper

The solid-phase diffusivity functions in Ecker2015 and Ecker2015_graphite_halfcell are updated to better match the functions in the original paper. A new citation Yuan2023, for which these parameters were used, is also added.
Advantages:

The new functions reproduce phase separation behaviour in a similar way to the Cahn-Hilliard equations
With them, the results in Yuan2023 can be reproduced.

Disadvantages:

This level of physical accuracy comes at a noticeable computational cost, comparable with that of ORegan2022
The results in Richardson2019 are no longer reproduced. However, that paper still made a major contribution towards our understanding of the Ecker2015 parameters and is therefore still cited.

My preference would be to add a new Yuan2023 parameter set so that the Ecker results don't change from version to version.

@DrSOKane Is this one fixed by this change as well?
#2152

What's the formula for D_s_n and D_s_p actually used by Ecker? I can't find it in Part I or Part II.

 It's possible that they used an interpolation between the data points. LiionDB has this data (I put it there!) so an alternative approach would be to use that.

@DrSOKane This PR was marked as merged because the commits were also present in #4168


Can Instance variables from generic classes be inherited?

Hello,
I found that automatic initialization of instance variables on classes and inherited classes works as expected:
class Result
  @errors = [] of String
end

a = Result.new
pp a.@errors # => []

class OtherResult < Result
end

b = OtherResult.new
pp b.@errors # => []

This also works when dealing with a generic class, as expected:
class GenericResult(T)
  @errors = [] of String
end

c = GenericResult(Int32).new
pp c.@errors # => []

class Specific
end

d = GenericResult(Specific).new
pp d.@errors # => []

However seems to fail the moment the generic class is used as parent of a new specialized class:
class SpecificResult < GenericResult(Specific)
end

e = SpecificResult.new
pp e.@errors # ERROR: doesn't have an instance var named '@errors'

I was hoping this scenario be consistent with the general inheritance, but perhaps is a limitation?
Thank you in advance for your time :heart: :heart: :heart:

This should work. We'll need to redesign how types are represented because right now generic inheritance is kind of a mess internally.
As a workaround you can set @errors = [] of String inside an initialize.

Thank you @asterite, that is precisely the approach I'm taking to workaround the issue.
Saw #846 might be similar, but wasn't 100% sure.
Cheers.

Thank you @asterite!!!  😘

:+1: @asterite codes fast as an asteroid! :-)


Big molecules: GPU usage and getting first x results

Dear all,
you have done great work on this package as I'm working on larger molecules (>20 carbon atoms) I would like to know if it is possible to:

take advantage of GPU in the calculations
receive the first 100 results, as an example of the structure generated. This would be particularly helpful when dealing with task to take to complete.
Take care and continue!

Dear @dphilip
Thanks for your feedback. I will address two questions together. surge has the output option called "-m" that helps users to split the generation process into independent sets. By that, user can generate small subset of the chemical space instead of generating the whole. This feature is useful for handling large molecules. Additionally, splitting the process using the -m option facilitates parallelisation, allowing users to leverage GPUs effectively.  Below is the screenshot for the -m option. For more details, please refer to the user manual.

Thanks Mehmet, I am already trying it out! Is the package configured to take advantage of a GPU?

Thanks Mehmet, I am already trying it out! Is the package configured to take advantage of a GPU?

Besides that -m option's usage, the current version of surge does not have any specific configuration for GPU.

Great, thanks for making this clear! Right now I am using a bold -m1/100, to get a response from an otherwise immense task.
Thanks again!


Add step for generate AI embeddings

Added new "compute-ai-embeddings" step
It relies on the openai config implemented in https://github.com/datastax/pulsar-transformations/pull/71
It adds/fill a new field embedding-field into the record value with the embeddings of the specified fields. The model must specifed.

Config example:
--user-config '{"openai": {"url": "https://datastax-openai-dev.openai.azure.com", "access-key": "KEY", "provider": "azure"}, "steps": [{"type": "compute-ai-embeddings", "model": "text-embedding-ada-002", "fields": ["field1"], "embeddings-field": "embeddings"}]}'

Supported providers:

OpenAI
Azure

Record schema supported:

[x] AVRO
[ ] JSON
[x] Key Value (With structured value)

Notes:

Each model has a max input size (calculated after the tokenization). If the input exceeds the limit, the function will fail. We could support in the future the chunking of the inputs.
In case of multiple input fields specified, the field values are concatenated and sent as single token. Dot product is not supported.

It adds/fill a new field embedding-field into the record value with the embeddings of the specified fields. The model must specifed.

I think it would be nice to be able to apply a template to the record. Then the user can have more complex embedding inputs.
That's what I'm doing for the chat completions.
Something like
"embeddings": [
    "The name of the user is {{ value.user }}",
    "The destination topic is {{ destinationTopic }}"
]

Actually, we only compute 1 embedding so that would be
"text": "The name of the user is {{ value.user }}, the destination topic is {{ destinationTopic }} and the message key is {{ key }}"

It adds/fill a new field embedding-field into the record value with the embeddings of the specified fields. The model must specifed.

I think it would be nice to be able to apply a template to the record. Then the user can have more complex embedding inputs. That's what I'm doing for the chat completions. Something like
"texts": [
    "The name of the user is {{ value.user }}",
    "The destination topic is {{ destinationTopic }} and the message key is {{ key }}"
]

It makes sense. We could add it later

It makes sense. We could add it later

I think it should be done from the start. Otherwise there will be backward compatibility issues.

We are in a prototype mode
it's fine to have it this way


feat(taints): to add taint feature to aks module

Fixes #152
Changes proposed in the pull request:
A feature to add taints to the default nodepool has been added. This will help to follow Microsoft best practices for system nodepools as we will be allowed to add the CriticalAddonsOnly=true:NoSchedule taint to the system nodepool.

Hi @carvido1 , thanks for opening this pr, but unfortunately, according to terraform-azurerm-provider pr #8982, since the v2.35.0, the node_taints argument is no longer be configured.

Hello @lonegunmanb .
According to the official azurerm documentation for the resource azurerm_kubernetes_cluster_node_pool the node_tains attribute is available.
The node_taints was removed from the resource azurerm_kubernetes_cluster because it was implemented at node_pool level. This happened when AKS started using VMSS instead of the old nodepools.
Please, reconsider the PR as it can be used to follow some best practices that Microsoft recommends.
Thanks in advance.

Hi @carvido1 , thanks for clarifying. According to the provider's code:

The AKS API has removed support for tainting all nodes in the default node pool and it is no longer possible to configure this. To taint a node pool, create a separate one.

So I think we cannot add taints to the default node pool anymore, sorry.

Hello.
I've reviewed the code @lonegunmanb. With a test I could have arrived to the return null ...
Thanks for your review. We can close the PR ;)


There is an error when select table cell

There is an error when select table cell
In my nextjs project.
I want to select mul cell in table,then an error:
TypeError: Class constructor Selection cannot be invoked without 'new'.
This has nothing to do with this plugin---@toast-ui/editor-plugin-table-merged-cell.
I had this problem using basic features.
Reproduce / Screenshots

My project info:
 "dependencies": {
    "@toast-ui/editor-plugin-chart": "^3.0.1",
    "@toast-ui/editor-plugin-code-syntax-highlight": "^3.0.0",
    "@toast-ui/editor-plugin-color-syntax": "^3.0.3",
    "@toast-ui/editor-plugin-table-merged-cell": "^3.0.2",
    "@toast-ui/editor-plugin-uml": "^3.0.1",
    "@toast-ui/react-editor": "^3.1.7",
    "axios": "^0.27.2",
    "html-react-parser": "^1.4.14",
    "next": "^12.1.7-canary.29",
    "prismjs": "^1.28.0",
    "quill-better-table": "^1.2.10",
    "react": "18.1.0",
    "react-dom": "18.1.0",
    "react-quill": "^2.0.0-beta.4",
    "sharp": "^0.28.3",
    "styled-components": "^5.3.5"
  },
  "devDependencies": {
    "@types/node": "17.0.35",
    "@types/react": "18.0.9",
    "@types/react-dom": "18.0.5",
    "eslint": "8.16.0",
    "eslint-config-next": "12.1.6",
    "typescript": "4.7.2"
  },
  "resolutions": {
    "prosemirror-model": "1.17.0",
    "prosemirror-view": "1.18.1"
  },

Device info:

Device: [PC]
OS: [win 11]
Browser [google chrome]

error screenshot

@jiayoujun Is it on SSR env?  We don't support SSR yet.

try use prosemirror-state with version 1.3.4 and prosemirror-transform version 1.4.2
prosemirror-state after1.3.4 ported to typescript and lead to "Class constructor Selection cannot be invoked without 'new'"

There is an error when select table cell
In my nextjs project. I want to select mul cell in table,then an error: TypeError: Class constructor Selection cannot be invoked without 'new'. This has nothing to do with this plugin---@toast-ui/editor-plugin-table-merged-cell. I had this problem using basic features.
Reproduce / Screenshots

My project info:
 "dependencies": {
    "@toast-ui/editor-plugin-chart": "^3.0.1",
    "@toast-ui/editor-plugin-code-syntax-highlight": "^3.0.0",
    "@toast-ui/editor-plugin-color-syntax": "^3.0.3",
    "@toast-ui/editor-plugin-table-merged-cell": "^3.0.2",
    "@toast-ui/editor-plugin-uml": "^3.0.1",
    "@toast-ui/react-editor": "^3.1.7",
    "axios": "^0.27.2",
    "html-react-parser": "^1.4.14",
    "next": "^12.1.7-canary.29",
    "prismjs": "^1.28.0",
    "quill-better-table": "^1.2.10",
    "react": "18.1.0",
    "react-dom": "18.1.0",
    "react-quill": "^2.0.0-beta.4",
    "sharp": "^0.28.3",
    "styled-components": "^5.3.5"
  },
  "devDependencies": {
    "@types/node": "17.0.35",
    "@types/react": "18.0.9",
    "@types/react-dom": "18.0.5",
    "eslint": "8.16.0",
    "eslint-config-next": "12.1.6",
    "typescript": "4.7.2"
  },
  "resolutions": {
    "prosemirror-model": "1.17.0",
    "prosemirror-view": "1.18.1"
  },

Device info:

Device: [PC]
OS: [win 11]
Browser [google chrome]

error screenshot

123123123

try use prosemirror-state with version 1.3.4 and prosemirror-transform version 1.4.2 prosemirror-state after1.3.4 ported to typescript and lead to "Class constructor Selection cannot be invoked without 'new'"

works for me ! thx

Hi everyone! After adding prosemirror-state and prosemirror-transform problem with runtime error was resolved, but I have one more problem - wrong paste table. Does anybody have this problem? Or maybe anybody knows how it fix?
https://github.com/nhn/tui.editor/assets/133666945/7c538b63-eb1c-4e6e-9e37-1d1e2345acb0

Hello ~ I also have same problem like the obove.
I am using originally node v20.9.0.
So, after installing NVM and downgrade the node version as v16.19.0.
and then, it works.
hope that it will be helpful to another one having the same issues~

try use prosemirror-state with version 1.3.4 and prosemirror-transform version 1.4.2 prosemirror-state after1.3.4 ported to typescript and lead to "Class constructor Selection cannot be invoked without 'new'"

How does this work?
Should I add them to package.json dependencies, or resolutions?
I'm currently using yarn package manager.


Reconcile clustersetip in broker

Reconcile clustersetip in BrokerReconciler

@tpantelis We discussed this earlier and not 100% sure we will need this. Pushing this PR for now in case we do need it for addon reconcilation.


SSR: Add support for world/unsigned normals

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

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

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


Not able to change listen from <IP_ADDRESS>:9000 to unix:/var/run/php5-fpm.sock

Not able to change listen from <IP_ADDRESS>:9000 to unix:/var/run/php5-fpm.sock
I have tried
I tried
php::fpm::pool::listen: 'unix:/var/run/php5-fpm.sock'
In the hiera yaml file but makes no difference.

Any feedback please guys?

Short answer, your direct use case:
php::fpm::pools:
    www:
        listen: /var/run/php5-fpm.sock

I'm on the road atm, going to push some more docs on configuring pools soon.
(php::fpm::pool is a resource no class => no direct data binding)

Thank you so much. Thats now worked. Yup definitely some more verbose documentation would be a great help. Thanks again.

Hi can you show me how to install phing using your module please. Looks like it only enables pear but doesnt let you install phing through it? Is that correct?


Attempting to code highlight a Nagios plugin causes a delay

From downstream: designcreateplay/NodeBB#477
A user is attempting to utilise the marked plugin to render a piece of code. They have the "highlight" property set to the following:
marked.setOptions({
    highlight: function (code, lang) {
        return hljs.highlightAuto(code).value;
    }
});

Other options hidden as they are not relevant.
When marked tries to marked() the following code, it takes a very long time (up to 30s):
#!/bin/bash
# ========================================================================================
# CPU Utilization Statistics plugin for Nagios
#
# Written by    : Andreas Baess based on a script by Steve Bosek
# Release    : 1.1
# Creation date : 3 May 2008
# Package       : DTB Nagios Plugin
# Description   : Nagios plugin (script) to check cpu utilization statistics.
#       This script has been designed and written on Unix plateform (Linux, Aix, Solaris),
#       requiring iostat as external program. The locations of these can easily
#       be changed by editing the variables $IOSTAT at the top of the script.
#       The script is used to query 4 of the key cpu statistics (user,system,iowait,idle)
#       at the same time.
#
# Usage         : ./check_cpu.sh [-w <warn>] [-c <crit]
#                                [-uw <user_cpu warn>] [-uc <user_cpu crit>]
#                                [-sw <sys_cpu warn>] [-sc <sys_cpu crit>]
#                                [-iw <io_wait_cpu warn>] [-ic <io_wait_cpu crit>]
#                                [-i <intervals in second>] [-n <report number>]
# ----------------------------------------------------------------------------------------
# ========================================================================================
#
# HISTORY :
#     Release   |     Date  |    Authors    |   Description
# --------------+---------------+---------------+------------------------------------------
#   1.1 |    03.05.08   | Andreas Baess | Changed script to use vmstat on Linux because
#               |               |               | iostat does not use integers
#               |               |               | Fixed output to display the IO-wait warning threshhold
# --------------+---------------+---------------+------------------------------------------
#   1.0 |    03.05.08   | Andreas Baess | Changed script so that thresholds are global
#               |               |               | and output can be parsed by perfprocessing
#               |               |               | changed default warning to 70 critical to 90
# =========================================================================================

# Paths to commands used in this script.  These may have to be modified to match your system setup.

Let me know if I am using marked incorrectly, or if the problem lies upstream (with hljs). Namely, I don't know how to pass a language from my markdown markup to marked, so I left used highlightAuto.

This does not appear to be valid markdown. Each of the # would render as a header.


Services API Endpoint - POST - 500 Status Code

Environment

Python version: 3.7
Nautobot version: 1.0.0b3

Steps to Reproduce

Attempt to POST to api/ipam/services

{
    "name": "http",
    "protocol": "tcp",
    "device": "UUID",
    "ports": [
        "80",
     ]
}

500 Error pointing to TypeError: '<' not supported between instances of 'str' and 'int'

This is also reproducible via pynautobot on 1.0.0b3.
# Working on 1.0.0b2
>>> nb.ipam.services.create({"name": "http", "protocol": "tcp", "ports": ["80"], "device": dev.id})
http
# not working on 1.0.0b3
>>> nb_b3.ipam.services.create({"name": "http", "protocol": "tcp", "ports": ["80"], "device": dev.id})
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/Users/myohman/Library/Caches/pypoetry/virtualenvs/nautobot-ansible-modules-DR2vR1FY-py3.8/lib/python3.8/site-packages/pynetbox/core/endpoint.py", line 271, in create
    req = Request(
  File "/Users/myohman/Library/Caches/pypoetry/virtualenvs/nautobot-ansible-modules-DR2vR1FY-py3.8/lib/python3.8/site-packages/pynetbox/core/query.py", line 386, in post
    return self._make_call(verb="post", data=data)
  File "/Users/myohman/Library/Caches/pypoetry/virtualenvs/nautobot-ansible-modules-DR2vR1FY-py3.8/lib/python3.8/site-packages/pynetbox/core/query.py", line 282, in _make_call
    raise RequestError(req)
pynetbox.core.query.RequestError: The request failed with code 500 Internal Server Error but more specific details were not returned in json. Check the NetBox Logs or investigate this exception's error attribute.

Logs from 1.0.0b3 instance.
19:59:43.164 DEBUG   nautobot.core.api.views.ModelViewSet views.py                  get_serializer_class() :
  Using serializer <class 'nautobot.ipam.api.serializers.ServiceSerializer'>
19:59:43.466 ERROR   django.request :
  Internal Server Error: /api/ipam/services/
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/django/core/handlers/exception.py", line 47, in inner
    response = get_response(request)
  File "/usr/local/lib/python3.7/site-packages/django/core/handlers/base.py", line 181, in _get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/usr/local/lib/python3.7/site-packages/django/views/decorators/csrf.py", line 54, in wrapped_view
    return view_func(*args, **kwargs)
  File "/usr/local/lib/python3.7/site-packages/rest_framework/viewsets.py", line 125, in view
    return self.dispatch(request, *args, **kwargs)
  File "/opt/nautobot/nautobot/core/api/views.py", line 195, in dispatch
    return super().dispatch(request, *args, **kwargs)
  File "/usr/local/lib/python3.7/site-packages/rest_framework/views.py", line 509, in dispatch
    response = self.handle_exception(exc)
  File "/usr/local/lib/python3.7/site-packages/rest_framework/views.py", line 469, in handle_exception
    self.raise_uncaught_exception(exc)
  File "/usr/local/lib/python3.7/site-packages/rest_framework/views.py", line 480, in raise_uncaught_exception
    raise exc
  File "/usr/local/lib/python3.7/site-packages/rest_framework/views.py", line 506, in dispatch
    response = handler(request, *args, **kwargs)
  File "/usr/local/lib/python3.7/site-packages/rest_framework/mixins.py", line 18, in create
    serializer.is_valid(raise_exception=True)
  File "/usr/local/lib/python3.7/site-packages/rest_framework/serializers.py", line 220, in is_valid
    self._validated_data = self.run_validation(self.initial_data)
  File "/usr/local/lib/python3.7/site-packages/rest_framework/serializers.py", line 422, in run_validation
    value = self.validate(value)
  File "/opt/nautobot/nautobot/core/api/serializers.py", line 40, in validate
    instance.full_clean()
  File "/usr/local/lib/python3.7/site-packages/django/db/models/base.py", line 1209, in full_clean
    self.clean_fields(exclude=exclude)
  File "/usr/local/lib/python3.7/site-packages/django/db/models/base.py", line 1251, in clean_fields
    setattr(self, f.attname, f.clean(raw_value, self))
  File "/usr/local/lib/python3.7/site-packages/django/db/models/fields/__init__.py", line 652, in clean
    self.run_validators(value)
  File "/opt/nautobot/nautobot/utilities/fields.py", line 181, in run_validators
    self.base_field.run_validators(part)
  File "/usr/local/lib/python3.7/site-packages/django/db/models/fields/__init__.py", line 604, in run_validators
    v(value)
  File "/usr/local/lib/python3.7/site-packages/django/core/validators.py", line 313, in __call__
    if self.compare(cleaned, limit_value):
  File "/usr/local/lib/python3.7/site-packages/django/core/validators.py", line 347, in compare
    return a < b
TypeError: '<' not supported between instances of 'str' and 'int'
19:59:43.871 ERROR   django.server :
  "POST /api/ipam/services/ HTTP/1.1" 500 174953

Expected Behavior
Successful POST (201 response code)

Observed Behavior
500 Error

I was able to reproduce this and it looks like the quick fix is to make sure your list of ports are integers, for example:
{
    "name": "http",
    "protocol": "tcp",
    "device": "f864f4e2-a5fc-4c47-be44-d5475eebb706",
    "ports": [
        80
    ]
}

Even still, this is still a bug that should at least return a 400, not result in a crash.

It's a side effect from having switched from  ArrayField to JSONArrayField for Service.ports. The API serializer needs to be wiser about this, because it thinks it's just a JSONField and isn't doing any "smart" type detection.


Mandatory inputs issue solved

I have add mandatory function, please marge my PR

Kindly mention the issue it solves, so that it will be easier to award points.

@Keshab1113
Mention the issue this PR solves, so that it will be easier to award points.

issue 52

#52

Thanks for your contribution!


Relationship between Queryable and ToConnection?

Thank you for the excellent library.
I was working with some of my older code when I realized that there are basically two ways to execute a query: With the Query as the receiver (e.g., "SELECT * FROM foo;".fetch(conn)), or with a Queryable as the receiver (e.g., `conn.query("SELECT * FROM foo;")').
But, there are different types involved. The query-receiver API needs ToConnection instances, and the queryable-receiver API needs Queryable instances.
In practice, we know that Pool, Conn, and Transaction are ToConnection; and that Conn, and Transaction are Queryable.
Since ToConnection is only there to get a Conn, and Conn is a Queryable, it may make sense to have ToConnection also be Queryable.
That would also provide a partial "bridge" between the different API styles.
Previously, I had written some functions to accept a Queryable so that it could be called with a Conn or a Transaction, but that limits me to one style of query syntax. I realize now that I could just always accept a Conn, and just have any transaction context pass in the Transaction's inner Conn. But that makes the call site more noisy.
It would be really cool if I could write a function that would accept any of Conn, Transaction, or Pool as input, and be able to use whichever query syntax I felt like using that day.
So, maybe the answer to that would be for ToConnection to "inherit" Queryable.
Or is this all way off base?

Hi.

Since ToConnection is only there to get a Conn, and Conn is a Queryable, it may make sense to have ToConnection also be Queryable.

ToConnection is one step behind Queryable. More precisely, it is only partially a Queryable (it is, when ToConnection is Immediate and its not, when ToConnection is Mediate. See ToConnectionResult). It won't be possible to define Queryable for the Mediate part because you'll need to give away self that is behind a mutable reference.
Regarding the Query trait, it was created to allow a caller to conveniently emit a 'static futures. That's why ToConnection is consuming (IntoConnection would be more conventional name). It might be the case that Queryable is enough for you, but the Query is quite convenient for some cases:
impl Workstation {
    /// Returns a 'static stream of workstations matching the given filter.
    pub fn stream<T: ToConnection<'static, 'static> + 'static>(
        conn: T,
        filter: Option<SqlFilter<WsField>>,
        order: Option<Order<WsField>>,
    ) -> BoxStream<'static, crate::Result<Self>> {
        // ...
    }
}

Thanks for the reply!
I did see that ToConnection has two states, but I assumed that since all of the Queryable stuff is Futures anyway, you could somehow implement it to return a Future that resolves whenever the connect is eventually available. But, now that you mention needing to consume self, I can understand why that wouldn't work.

Regarding the Query trait, it was created to allow a caller to conveniently emit a 'static futures.

Ah! Fair enough. I hadn't thought that there might actually be different capabilities between using Query vs. Queryable APIs, other than how you get one.
Thanks again. I've been happily using your mysql crates for several years. I just never really thought much about the traits/types involved until recently.


require() in generated JS code for a type only import holding a const enum

TypeScript Version:  2.5.2 / nightly (2.6.0-dev.20170921)
Code
File foo.ts exports a const enum + a function:
export const enum ConstFooEnum {
  Some,
  Values,
  Here
};
export function fooFunc(): void { /* removed */ }

File index.ts which imports Foo.ts but uses only the const enum values:
import * as Foo from "./foo";

function check(x: Foo.ConstFooEnum): void {
  switch (x) {
    case Foo.ConstFooEnum.Some:
      break;
  }
}

Expected behavior:
Generated Javascript code should not require("./foo"). Docu (http://www.typescriptlang.org/docs/handbook/modules.html) indicates that type only imports should not result in a require().
Actual behavior:
Generated JS code contains line var Foo = require("./foo"); even Foo is not used anywhere.
"use strict";
Object.defineProperty(exports, "__esModule", { value: true });
var Foo = require("./foo");
function check(x) {
    switch (x) {
        case 0 /* Some */:
            break;
    }
}

In above simple code sample I can workaround the issue by doing importing only the ConstFooEnum via import { ConstFooEnum } from "./foo"; but if the enum is inside some namespace (e.a. enum gets re-exported from an inner module) it's not that easy.

@weswigham can you take a look.
I do remember we did this for a reason.. something to do with namespace merging scenarios.. but do not remember what exactelly thought...
maybe @vladima remembers :)

IIRC references to const enums are transpiled as normal enum references under isolatedModules (since there's no type info to work with), so to account for this they're still marked as value-space reads of the reference in order to have more consistent behavior between isolatedModules compilations and normal ones.

it should be an error to use const enums under --isolatedModules. i think it would make sense to not mark as referenced if we are compiling with --isolatedModules false.

Looking at our code, I feel like there's intent that the import should be elided here, as the identifier is checked. I'm pretty sure if you wrote it as:
import {ConstFooEnum} from "./foo";

function check(x: ConstFooEnum): void {
  switch (x) {
    case Foo.ConstFooEnum.Some:
      break;
  }
}

it'd get elided.

Yep. Using a destructuring import pattern causes ellision to occur as expected!
We do check that references to const enums and const enum only modules aren't considered value references; but in
import * as Foo from "./foo";

function check(x: Foo.ConstFooEnum): void {
  switch (x) {
    case Foo.ConstFooEnum.Some:
      break;
  }
}

we also check Foo in case Foo.ConstFooEnum.Some:. Foo is not a conest enum only module or a const enum (it's a module alias), and so gets marked as a reference.

@mhegazy I think it's fine to use const enums in --isolatedModules; what is bad is that you can export them.

That is true. not sure if there is much need for using const enums within a single file though. we can allow it, but so far i have not seen any requests for that.


Debug feature: List c-source files

Consider this:
MainControl.py
def makeSourceDirectory(main_module):
    ...

    module_filenames = pickSourceFilenames(
        source_dir = source_dir,
        modules    = ModuleRegistry.getDoneModules()
    )

    if Options.options.show_source_fies:
        from distutils.util import strtobool

        def userYesNoQuery(question):
            print(question, end="")
            while True:
                try:
                    return strtobool(input().lower().strip())
                except ValueError:
                    print("Please respond with 'y' or 'n'.\n")

        print("Compiling:")
        for module in ModuleRegistry.getDoneModules():
            if module.isCompiledPythonModule():
                print("  {}".format(module_filenames[module]))

        if not userYesNoQuery("Proceed (y/n)? "):
            sys.exit(0)

OptionsParsing.py
debug_group.add_option(
    "--show-source-files",
    action  = "store_true",
    dest    = "show_source_fies",
    default = False,
    help    = """\
Provide a summary of source files before compilation happens
"""
)

I've made a fast test how long would take to that code display the numpy c source files and the execution was painfully slow, few minutes.
Can you think of a much faster way (the order of ms/few_seconds, ie: [0s-10s]) that warn the user about which c source files will be generated/compiled?

Why should the user care which C source files are generated?
The final scope is only known at the end of optimization, because an import statement may or may not be present, so I fail to see how this is better than --generate-c-only option and an ls command.
Anyway, I fail to see the motivation here. I discussed on the mailing list that a report of the compilation will be added, and it probably will have to be created incrementally, but it definitely is not my priority right now.

Why should the user care which C source files are generated?

I totally agree with you here... remember I'm the guy who already insisted zillion times about UIs providing the minimal feature set that cover important users use-cases, no more, no less. In fact, end-users will only care about 2 things, speed (runtime & build) & correctness. They just want stuff working out of the box and many of them won't try twice if they don't see any benefit on the first attempts, sad but true :)
Anyway, let me clarify a little bit here, this issue is some sort of addon on top of #136 , reason why I opened this new one was mainly to self-answer one of the few questions I've written there... I just didn't want to make longer #136.
That said, you can observe I've written "Debug feature" as a title and then you can also see in one of those throw-away example snippets I've used debug_group... so this was intended to be a feature for "advanced/expert" nuitka users :)
Rationale behind this is related - as explained in the other thread - to the fact that building meaty packages is a quite expensive operation in many cases, so if we advanced users knew very soon (a range of [0s-10s] would be ideal) which type of content will be generated/compiled/added into the final extension module  that'd be awesome and it would help us to know whether the options we passed to nuitka are the correct ones or not.
Unfortunately, I don't see how an incremental logging would help here... You mean adding some logging to the writeSource function itself, right? Tbh, to me is the same whether I see printed c files along 400s multiple times rather than all of them after 400s. Probably I'll have to wait 400s mostly of the times :/
Anyway, please consider the whole explained context from the other issue #136 so you have a wider perspective why I'm asking/interested on this one.
Right now I've got some scripts that compiles ~200 packages living in a requirements.txt... And many packages will be built ok out of the box decently fast, which it's quite awesome... but in many other cases, huge packages like numpy, pandas, matplotlib, pyopengl, sympy, custom frameworks... well, I'd like to have a finer grain control so I'll know exactly which content to ship or not before deploying anything. When I say finer control I mean, if my project will just use a little % of a huge meaty package I just want to include that little % of the package. As it's not the same spending 900s/1000s rather than <100s :)
Hope it makes more sense now and you can understand a little bit better the "motivation" behind this. Feel free to close this issue though so we can continue the discussion on the other thread or leave this one open, whatever works better for you, your choice.
And just for the record, it's of course low-prio stuff... I was just interested to know from you if this was achievable easily or not.
But the fact you've already stated this won't be easy to achieve (till optimization stage has ended) makes me think that having a report in the range of 0s-10s will be quite challenging/difficult/impossible to achieve :(
In any case, let me ask you, don't you think end-users & advanced should understand perfectly which type of content will be included on their final extension modules? I mean, it's like if I was going to build projects like {chromium, qt, blender, cmake, ...} from scratch and I didn't know exactly how to configure them nor knowing which stuff to discard on the compilation, that's just crazy :/

As suggested I am closing this, largely due to my failure to understand. I need to catch up with your comments, but the time to generated the C code should actually be fairly minimal in comparison to reading and optimizing them, so I don't see the point yet.
What I also do not like is to talk about file names at all. In my mind, the split per module is arbitrary and subject to change, and has nothing to do with modules. I hope to inline modules to their users eventually, would that mean they are not included?
Moving --show-inclusion or what it is named up might be an option, but like I said, need to catch up if I can ever make it.


Next.js 13 app directory support

I've tried implementing this with the experimental app directory in Next.js 13 using a layout.tsx and page.tsx - would be really useful to see how to implement next-hubspot with this situation

app/providers.tsx
'use client';

import { HubspotProvider } from 'next-hubspot';

const Providers = ({ children }) => (
  <HubspotProvider>
    {children}
  </HubspotProvider>
)

export default Providers.

app/layout.tsx
import Providers from "./providers";

const RootLayout = ({ children }) => (
  <html>
    <body>
      <Providers>
        {children}
      </Providers>
    </body>
  </html>
);

export default RootLayout;

Add 'use client'; to any component in which you use useHubspotForm hook.
Everything else stays the same.

Thanks @snelsi 👌🏼

Hello. Will it not destroy all SSR by adding 'use client'? 🤔

Hello. Will it not destroy all SSR by adding 'use client'?

Hubspot form can't be server-side rendered. It requires scripts to be executed on the client side. So you should mark any component that imports and uses useHubspot hook with use client.
But adding Providers component from the example above will not affect other components on your page. The children is still rendered on the server and just passed down as a prop

Doesn't all children automatically become client-sided if the parent has 'use client'? My point here was that maybe it's not best to wrap it with the Provider at the very top level. 😉

@vygandas 👇
https://nextjs.org/docs/getting-started/react-essentials#rendering-third-party-context-providers-in-server-components

@vygandas
https://nextjs.org/docs/getting-started/react-essentials#recommended-pattern-passing-server-components-to-client-components-as-props


socket.io instead of simple-websocket?

Hello,
WebSockets are very slow to establish and this can hinder overall performance of tracker, isn't it?
So is it possible to use socket.io instead of simple-websocket for better performance?

WebSockets are indeed unfortunately slow to establish, but we can't switch to something like socket.io because WebTorrent is a standard that is implemented by other clients, lots of them written in C++, Java, Python, etc. We don't want to make all of these implementors reverse-engineer what socket.io is doing and support that. Then the spec would be tied to whatever implementation changes socket.io happens to make in the future.
There might be other ways to make this faster, like pre-connecting to the websocket server even before client.add or client.seed is called.

Let's move discussion to this issue: https://github.com/feross/webtorrent/issues/574