Environment variables missing - project won't start.

Nice boilerplate. Thanks for this.
However it does not work as provided. Environment variables missing - project won't start.
I tried setting it up with some sane defaults, but still then the project wont load.

export HOST_HTTP="http://localhost"
export HOST_URL="/graph"
export HOST_PORT=8000
export POSTGRES_USER="postgres"
export POSTGRES_PASSWORD=""
export POSTGRES_SERVER="<IP_ADDRESS>"
export POSTGRES_PORT=5432
export POSTGRES_DB="fastapi_strawberry"

You get the error ERROR:    [Errno 8] nodename nor servname provided, or not known

I'm having a similar problem. I am running the app from the readme instructions, but I can't visit the /graphql endpoint due to missing auth. I know there must be some unset key or env vars, but I'm not sure what they are. Please advise.


fixed ubrs typo; added new matches for dung/raids

I do not play wow any more, if you wish to continue this project (not that I got very far) please feel free to continue it in your fork and I'll happily leave a note sending people there.


Add shell command to ping API with existing XML file.

Make it possible to just hit the API with a file which already exists

This would be a nice feature for the adminhtml interface, too.


Token-swap: Add "production mode" hard-coded fee constraints

In the dex situation, the token swap program may have multiple frontends interacting with it, but the program creator would like to be the default destination for trading fees, along with enforcing certain fees on its exchange.  This provides a little framework for setting and enforcing these constants, enabled using the "production" feature.
Currently, only the owner key is configured through an environment variable.  I wanted to statically create a Pubkey based on &'static str using a const function, but the only const function for creating is from an array, so I'm not sure if we can go the full way to creating a Pubkey at this point.  We could eventually also include the fee parameters as env variables, but the conversion &'static str -> u64 wasn't clear either.
Additionally, there's an optional "host account" parameter added to the swap instruction.  If provided, the program creator shares some of the trading fee with this host account.
I had some trouble working with and without the constraints in testing, but I was pretty happy with the idea of just factoring out the constraints from the normal process function in the end.  Any better ideas are absolutely appreciated!
Note that a JS end-to-end test for production mode still needs to be added for CI.

Great thanks!  Prettier and eslint clashed on how a line should look for a long ternary operator, so they would keep clobbering each other's changes and then reporting an error on check.  I updated the problematic line to avoid a ternary operator.


How to get tooltip placement (which is exposed as x-placement attribute on tooltip) and set class based on this

Using v-tooltip directive. When tooltip is opened popper.js tries to find suitable placement based on surrounding space (at least that is how I understand it) and has this information exposed on x-placement attribute. How to get the placement information and use this in Vue - for example to expose it as css class (one reason is that some browsers (IE for example) have performance problem with css attribute selectors.
I managed to get my case working using popper.js custom modifiers but I wonder if there is a more Vue way
https://jsfiddle.net/xhwL2sme/2/

Moved this to stackoverflow as it seems this is more of an usage question:
https://stackoverflow.com/questions/58861160/using-vue-v-tooltip-how-to-get-tooltip-placement-which-is-exposed-as-x-placemen


Zube automation keeps reopening issues

Description
In some cases (e.g. https://github.com/rancher/dashboard/issues/9683) Zube keeps reopening issues after PRs get merged.

The github bot re-opened 9683 correctly after the merge of the PR. things were a bit different then, but i think from now...

issues labelled as QA/None are not re-opened (to check)
issues labelled as QA/Dev Automation are re-opened and moved in to the QA Review zube column
all other issues are re-opened and moved to the To Test zube column

Can we close this and re-open when we see it again - the GH Bot should move issues that are dev-automation to the new 'QA Review' pipeline in Zube.


Check NugetRestore flag before writing warning

If nuget.exe isn't in the Tools folder on the agent, we always emit a
warning that we can't find it and that package restore won't be
performed.  We emit this warning even if the MSBuild/VSBuild task has
the nuget restore option unchecked.  So now we'll only emit the warning
if we can't find nuget.exe and the option is checked.

Approved!


communities.json: spellfix for Wiesbaden

Freifunk in Mainz, Wiesbaden und Umgebung

as found on https://freifunk-mwu.de

Meh, this will generate a new timeseries of course. Sad.

Not sure if you can manually rename the old.


[Feature]: Create a better leaderboard for OpenHands

What problem or use case are you trying to solve?
Currently, there is no comprehensive leaderboard that effectively tracks the performance of open-sourced models within OpenHands. A clear leaderboard would facilitate comparison and improvement.
Describe the UX of the solution you'd like
An online sheet that continuously updates and tracks performance metrics for various models, possibly with the ability to click on links for deeper insights specific to each model.
Do you have thoughts on the technical implementation?
The leaderboard could utilize existing performance data stored in a central database, automatically populated and refreshed at intervals, ensuring real-time updates.
Describe alternatives you've considered
Using a static document or manual tracking methods, but these lack the dynamic features and timeliness that a real leaderboard would provide.
Additional context
This initiative was discussed in a recent thread on Slack with users expressing interest in better tracking mechanisms for performance.
Issue Created By: Graham Neubig on Slack

Currently this doc is our best leaderboard: https://docs.google.com/spreadsheets/d/1wOUdFCMyY6Nt0AIqF705KN4JKOWgeI4wUGUP60krXXs/edit?gid=0#gid=0

Probably separate feature request, linking in with the feedback method, could users run and submit a benchmark from their setup from the UI (not swe, maybe something simpler or lite)?


Added .github workflow files

I realized that when I created the directory structure using cookiecutter that I selected the wrong option at the time and did not include the github action/workflow files - these have now been added and pushed to the main directory!

Thanks Mona!


Expose FieldType Data via Celestial API

Part of Issue #101 includes making Field data mutable. As part of that, the type of field should be able to be set by the user.
With the current design, we should load that FieldType data as content set data via the Celestial API. This is also useful for use with the Field API call too when using the API externally.

Completed and verified:
[josh@jimmy1 ~]$ curl -v 'https://inquisition.carlso.net/api/v1/tuning/?t=field_type' | python -m json.tool
  % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
                                 Dload  Upload   Total   Spent    Left  Speed
  0     0    0     0    0     0      0      0 --:--:-- --:--:-- --:--:--     0*   Trying <IP_ADDRESS>...
* TCP_NODELAY set
* Connected to inquisition.carlso.net (<IP_ADDRESS>) port 443 (#0)
* ALPN, offering h2
* ALPN, offering http/1.1
* successfully set certificate verify locations:
  CAfile: /etc/pki/tls/certs/ca-bundle.crt
  CApath: none
} [5 bytes data]
* TLSv1.2 (OUT), TLS handshake, Client hello (1):
} [512 bytes data]
* TLSv1.2 (IN), TLS handshake, Server hello (2):
{ [108 bytes data]
* TLSv1.2 (IN), TLS handshake, Certificate (11):
{ [2750 bytes data]
* TLSv1.2 (IN), TLS handshake, Server key exchange (12):
{ [333 bytes data]
* TLSv1.2 (IN), TLS handshake, Server finished (14):
{ [4 bytes data]
* TLSv1.2 (OUT), TLS handshake, Client key exchange (16):
} [70 bytes data]
* TLSv1.2 (OUT), TLS change cipher, Client hello (1):
} [1 bytes data]
* TLSv1.2 (OUT), TLS handshake, Finished (20):
} [16 bytes data]
* TLSv1.2 (IN), TLS handshake, Finished (20):
{ [16 bytes data]
* SSL connection using TLSv1.2 / ECDHE-RSA-AES256-GCM-SHA384
* ALPN, server accepted to use http/1.1
* Server certificate:
*  subject: CN=inquisition.carlso.net
*  start date: Aug  5 17:30:51 2018 GMT
*  expire date: Nov  3 17:30:51 2018 GMT
*  subjectAltName: host "inquisition.carlso.net" matched cert's "inquisition.carlso.net"
*  issuer: C=US; O=Let's Encrypt; CN=Let's Encrypt Authority X3
*  SSL certificate verify ok.
} [5 bytes data]
> GET /api/v1/tuning/?t=field_type HTTP/1.1
> Host: inquisition.carlso.net
> User-Agent: curl/7.59.0
> Accept: */*
>
{ [5 bytes data]
< HTTP/1.1 200 OK
< Date: Tue, 14 Aug 2018 23:53:19 GMT
< Server: Apache/2.4.25 (Debian)
< strict-transport-security: max-age=2592000; includeSubDomains
< X-Frame-Options: sameorigin
< X-XSS-Protection: 1
< X-Content-Type-Options: nosniff
< Content-Security-Policy: default-src 'self'; connect-src https:;
< X-Permitted-Cross-Domain-Policies: none
< Referrer-Policy: no-referrer
< Cache-Control: max-age=0
< Vary: Accept-Encoding
< Content-Length: 234
< Content-Type: application/json
<
{ [234 bytes data]
100   234  100   234    0     0    720      0 --:--:-- --:--:-- --:--:--   720
* Connection #0 to host inquisition.carlso.net left intact
{
    "data": [
        {
            "type_id": 1,
            "type_key": "hst",
            "type_name": "log_source"
        },
        {
            "type_id": 2,
            "type_key": "src",
            "type_name": "traffic_source"
        },
        {
            "type_id": 3,
            "type_key": "dst",
            "type_name": "traffic_destination"
        }
    ],
    "data_source": "default",
    "status": "success"
}
[josh@jimmy1 ~]$


more of a question than an issue

does this search for contents on word document and pdf?

Not out of the box no. This is just meant for simple site content search and a starting point for more advanced searching like PDF searching.


Updated WPF Sample

The WPF sample app now works and updates a file called CurrentVersion.txt, which the app uses to display the version number.

Thanks
A few comments:

How is the text file going to be updated back to an older version after a successful update? or it shouldn't?

I'm not sure I like the change to MemorySource. Originally it was meant to be used for in memory feeds only - I don't see how we can use it for updating actual files.

It shouldn't. The update mechanism in the demo app is provided purely for demo purposes.

What is an in-memory feed for? Is it only for demo apps?

Ok, so want to put that in a readme. Or a website. Damn we need a website :)

Not necessarily, otherwise it wouldn't have been in the core lib. Think for example of situations when you want the app to amend the feed before proceeding, or to initialize it's own update, or to read it from a source it is going to be easier to just use MemorySource to pass it to NAU

Do you know about github pages? http://pages.github.com/
I'm still no sure I'm clear on what the memory source would be used for. If its supposed to wrap an existing source shouldn't it take that original source as input?
So how would you update a file using the MemorySource?


Example app is down

http://deadbolt-2-scala.herokuapp.com/ is not working right now

Thanks for the report - this is now fixed.


replace bbmap with something slim

bbmap is using a lot of space but only does one tiny step in the pipeline (split fasta file). Replace!

fixed


Simplify by defining a common usage pattern: TTA.define_and_execute()

'VirtualAssistant.command' is too complex (72).  72 is the measurement of the method's McCabe complexity.  I have rarely seen numbers in 20's but I have never a number so high.  This PR's changes do not lower that number but some work should go into refactoring this method to make it more understandable for the first time reader.  ;-)
Also:

Sort imports with isort (optional but does make imports easier to comprehend)
Use isinstance() when comparing types
Use random.choice() on a tuple instead of random.randint() on a dict
Remove repetition from time_of_day logic.

@cclauss firstly, conversion from TTA.define() and TTA.execute() to TTA.define_and_execute() is incomplete, such as this line.
Combining TTA.define() and TTA.execute() into a single function will reduce the flexibility for developers who want to write more complex blocks. (i.e. execute the same command over and over again under a loop with unknown iteration count) So these kind of functions need to be as atomic as possible.
Replacing print("_______________________________________________________________\n") with print("_" * 63 + "\n") will make it hard to find with a text search in the repository. When I write a comment lines like #print "TTS process started." (without spacing after #), it indicates a commented code line, not an actual comment. My way contradicts with Flake8's default though.
I saw 'VirtualAssistant.command' is too complex (72) warning too. But VirtualAssistant.command() is an unusual function. Do you have a better idea than using a long chain of if-elif-else? We can choose to encapsulate the each block under an if/elif statement with defining a function but why would we do that? What is the benefit of doing that? Performance?

I do not advocate doing away with TTA.define() and TTA.execute().  This PR says that the combination is a common pattern, not a universal pattern.  The two functions have their place as separate functions and there were several instances where I did not attempt to merge them because I understood that they were not directly combined.  I also did not try to find all the places in all files where they could be combined but was focused on shortening the special VirtualAssistant.command() method.
I think we can use dict lookup to simplify VirtualAssistant.command() and also process ("SEARCH" in com or "FIND" in com) differently but (given outside pressures) it will take me time to find the right formulation.  I have no trouble removing the ('_? * 63) parlor trick.

I think that splitting VirtualAssistant.command() is the good idea. It would be much more readeble if to split it for handlers with for example such base class:
class BaseHandler:
	def can_handle(self, command):
		return False

	def handle(self, command):
		return None

@cclauss I suggest you to replace all “BLABLA” in com pattern with a regex ilke this line. I think a simple dictionary lookup is not suitable for this task. We should use massive amount of regex or even NLP methods in a single function. If there is another alternative then please let me know. I will be waiting for your right formulation. :blush:

Fixed with #55

@alexpantyukhin I like your idea in https://github.com/DragonComputer/Dragonfire/pull/50#issuecomment-341686540 more and more.  It would be possible to create plugin vocabulary where just putting a handler in the right directory would add logic to Dragonfire.  I will explore that.

@cclauss yeah, It would be interesting to implement. I think I can implement this if @mertyildiran agree with that.

I started on a transatlantic flight last night...  The issues we face are:

Sharing state down into so many files
Different handler types:

exact match
in match
search and find
regex match
keyboard commands

Priority: which handlers to run first and which last.

The exact match handles should run earlier and use a dict for speed
I am not a fan of debugging regex so I would keep these to a minimum

I will submit a DO NOT MERGE pull request please feel free to use as inspiration or ignore and close.


Hotfixes: Timers on Projects page and in Invoice Form

Coverage increased (+0.03%) to 47.774% when pulling 2c2a64eab0edb2339f8fa8677f84c6ea37ee03a8 on develop into cc486d90f2dbb861faa99d5a092b91478976a0e2 on master.

Coverage increased (+0.03%) to 47.774% when pulling 2c2a64eab0edb2339f8fa8677f84c6ea37ee03a8 on develop into cc486d90f2dbb861faa99d5a092b91478976a0e2 on master.


⚠ Jenkins Plugins Artifactory has degraded performance

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

HTTP code: 200
Response time: 1241 ms

Resolved: Jenkins Plugins Artifactory performance has improved in 3f5301a after 23 minutes.


Mount Suggestion Ticket

Bringing this ticket back from Stratos, we should be able to add mounts to the server easily, such as the recent Galarian Bird mounts.
If you have a suggestion for a mount, use the following format:

Pokemon Name:
Land, Water, or Flying Mount:
Why this should be added:

Please no troll Pokemon, I'm not making Magikarp rideable, but Pixelmon has a lot of mounts missing/unimplemented and Oblique can add them manually.
If I approve of your idea, I will use a thumbs up mark emoji, rejections will have an thumbs down emoji, and implemented ones will have a rocket.

Pokémon Name: Zamazenta/Zacian
Land, Water, or Flying Mount: Land
Why this should be added: They're large doggos. They're also pretty new, and a lot of people were kind of sad, myself included, that they weren't mountable from the release. We can ride other similar four-legged dog-type Pokémon; Such as Arcanine and Ninetales. Even some that are legendary, such as Raikou/Entei/Suicune.

Pokemon Name: Genesect
Land, Water, or Flying Mount: Flying
Why this should be added: It was a flying mount on Stratos following my request to Faty at the time. The reasoning being that during its idle animation, Genesect folds itself up into a “hoverboard” shape and briefly floats above the ground

Name: Dragapult
Mount Type: Flying
Reason: Before the Pixelmon 8.2.0, Dragapult was a pretty well-known flying mount who was a faster flyer than most other mounts. Whether or not you add it's alternate form (when it moves over the ground), Dragapult is no longer rideable, in turn it formed various turn-offs for various people.

Name: Eternatus
Land, Water, or Flying Mount: Land/Flying
Why this should be added: Eternatus is a giant extraterrestrial space dragon, It would seem fitting that it would be able to be used as a flying mount, even at it's smallest size it's still quite large compared to the player so it wouldn't look odd. It's even seen in it's appearance on Pokemon Journeys that it has the ability to fly! And it can learn fly :)


feat(cloudtrail): bump cloudtrail plugin version to 0.11.0

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area plugins

/area registry

/area build

/area documentation

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

Thanks for your pull request. Before we can look at it, you'll need to add a 'DCO signoff' to your commits.
:memo: Please follow instructions in the contributing guide to update your commits with the DCO
Full details of the Developer Certificate of Origin can be found at developercertificate.org.
The list of commits missing DCO signoff:

ffcde78 bump cloudtrail plugin version to 0.11.0

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: matteopasa
Once this PR has been reviewed and has the lgtm label, please assign leogr for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jasondellaluce, matteopasa
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

LGTM label has been added.  Git tree hash: 4301e2993a5d2f681fd9737f6706e67a368e7bd6


Void with ResponseType attribute

Take a look at the following snippet:
public class ValuesController : ApiController
{
    public void Put()
    {
    }

    [ResponseType(typeof(void))]
    public IHttpActionResult Post()
    {
        return StatusCode(HttpStatusCode.NoContent);
    }
}

I expect the two operation to have the same result in the responses field.
Both should have:
"responses":{"204":{"description":"No Content"}}

But instead the Post operation gets the following:
"responses":{"200":{"description":"OK","schema":{"$ref":"#/definitions/Void"}}}

This generates the following on swagger-ui:

Which is clearly wrong. I need the Post operation to have IHttpActionResult, so I can return with BadRequest() or Unauthorized() etc. That is why I used the ResponseType attribute to specify when it succeeds it should return 204 No content, without a schema reference.
How can I remove these strange "schema":{"$ref":"#/definitions/Void"} references from my operations?

This is a SB bug - just isnt handled because I never thought of the case where you would use ResponseType instead of void on the action signature. Simple fix ... will go out with next release

In the meantime, you could do the following as a workaround ...
[SwaggerResponseRemoveDefaults]
[SwaggerResponse(204, "No Content")]
public IHttpActionResult Post()
{
    ...
}

Should be resolved in latest release - 5.2.0

Thanks I needed this too! Great release! :+1:

What about .NET Core version? It has no "SwaggerResponseRemoveDefaults" attribute, and "[ProducesResponseType(StatusCodes.Status204NoContent, Type = typeof(void))]" does not seem to work.


Adding journaltypes for easydao

This closes: https://github.com/foam-framework/foam2/pull/1915
This is needed for the RoutingJournals redesigning.

The actual functional changes look fine but I would really prefer if you didn't run all the code through a formatter.


Questions about the inptus of borzoi prediction

Hi, thanks for your great work. I wonder if ther input seq lengh of borzoi (524288) is a very strict requirement, as I found that many genes do not have such window size in their chrom (like 200 out of 18000 candidates). Is it possible to adjust its window size for fine-tuning model? Thanks a lot.

Hi, this isn't strict, but the self-attention relative position encodings are set based on the sequence length. It would be safer to just pad with N's on each side. We have a transfer learning pipeline on the way that will make it easier to adjust the sequence length, and maintain the original self-attention sequence length. https://github.com/calico/baskerville/blob/transfer/docs/transfer/transfer.md


Shrink acceptance test data - PR9

A component of https://github.com/metoppv/improver/issues/1804
To reduce the data volume of the acceptance test data repository, work has been done to coarsen the resolution of test data relating to the following CLIs:

spot-extract
neighbour-finding

This PR updates the checksums and removes a redundant test for spot-extract. It also corrects a doc-string.
The changed acceptance test data is fundamentally changed due to the nature of these tests and needs careful review.
The relevant acceptance test data can be found in branch shrink_atd_9
Plotting of the original and modified data can be seen here.

@brhooper you should now find that the data are different in all of the KGO. I have significantly increased the search radius for the nearest land or similar altitude neighbour to achieve this for the global data on the coarse grid. The figure below, covering some of South America, shows sites (circles) and the chosen neighbouring grid points (triangles) that correspond to different options (e.g. nearest, nearest-land, etc); sites and associated neighbours share the same colours.

@bayliffe: Please can you resolve the conflicts on this PR?


preserve context when adding a new context provider

This is common need when creating a domain specific logger: we may want to provide domain specific ids (like user/session ids) inline with some generic ones (like Otel ids). Currently, we have to create a single lambda with all of the behaviour in it, which leads to lots of duplicated across apps.

/dobby version minor


Reference to prior work

Hey Everybody,
first of all congrats to the paper, it's really interesting and am looking forward to seeing its impact on the space!
I wanted to point out that a few colleagues and me followed a similar approach to introduce translational equivariance in kernelizable attention (as is implemented in the Performer) for image classification tasks and posted it on ArXiv in the beginning of February https://arxiv.org/abs/2102.07680.
While the approach proposed in your work is more generic, we would highly appreciate if you could also refer to our prior work in the publication.
Best,
Max

Thank you for informing us of the related work. We will refer to your paper in the final version.


Flush option for ListGroup

Issue description
Feature request: Flush mode for List group

components: List Group
reactstrap version #5.0.0-beta2
import method es (if umd, are you using the "full" version (only for v5+)?)
react version #16.3.0
bootstrap version #4.0.0

What is happening?
Using list group inside card. Have to manually put list-group-flush inside className
What should be happening?
flush option on ListGroup component so don't have to use className
Steps to reproduce issue
<Card>
  <ListGroup flush>
    <ListGroupItem>...</ListGroupItem>
  </ListGroup>
</Card>

Error message in console
None
paste error message with stacktrack here
Code

<Card>
  <ListGroup flush>
    <ListGroupItem>...</ListGroupItem>
  </ListGroup>
</Card>

the following should work right?
<Card>
  <ListGroup
     flush={true}>
    <ListGroupItem>...</ListGroupItem>
  </ListGroup>
</Card>

https://github.com/reactstrap/reactstrap/blob/master/src/ListGroup.js#L28

thanks. it actually works. Just didn't found such options in the docs.


[3.x] Take into account the clear icon's size even when not visible in LineEdits

Backport of #61726.

Thanks!


feat: zstd support

Closes #10
Code complete but needs testing

It might be better to use zstd-rs which is in pure Rust, just like the other compression libs?

There are two implementations in this PR, the pure Rust default that uses ruzstd, and zstd if you enable fast_zstd or max_perf.
This is the same as the zlib backend which uses pure Rust miniz_oxide by default, but if you enable fast_zlib it'll switch to the zlib-ng backend which needs a C compiler.


[KYUUBI #699][DOCS] Add document for kyuubi-extension-spark_3.1 module

Why are the changes needed?

Make Kyuubi SQL extension readable.
How was this patch tested?
The screen snapshot is:

nit: use extensions or extension

Codecov Report

Merging #702 (21d9cf7) into master (cd32e4a) will increase coverage by 0.04%.
The diff coverage is n/a.

:exclamation: Current head 21d9cf7 differs from pull request most recent head d9d6360. Consider uploading reports for the commit d9d6360 to get more accurate results

@@            Coverage Diff             @@
##           master     #702      +/-   ##
==========================================
+ Coverage   80.52%   80.57%   +0.04%
==========================================
  Files         122      122
  Lines        4730     4730
  Branches      572      572
==========================================
+ Hits         3809     3811       +2
+ Misses        600      599       -1
+ Partials      321      320       -1

Impacted Files
Coverage Δ

...org/apache/spark/kyuubi/SQLOperationListener.scala
97.43% <0.00%> (+5.12%)
:arrow_up:

Continue to review full report at Codecov.

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

cc @yangrong688 this is the late docs.

cc @yangrong688 this is the late docs.

I got it, thanks.

thanks, merging to master for v1.3.0.


Upgrade @octokit/rest to v17

Why?
Upgrades @octokit/rest to v17. This let's us remove the custom graphql property, since that's now included on the Octokit class.
Closes #108
How?
I've removed src/github in favor of just creating the Octokit instance as Toolkit.github directly in the constructor.
Note: This is currently failing to build because of type issues. I'm not sure how to resolve them - it looks like @octokit/rest isn't exporting a class, and I can't figure out how to grab the type of the export:

cc @gr2m

[x] Tests have been added/updated (if necessary)
[x] Documentation has been updated (if necessary)

Might be the same problem as reported in https://github.com/octokit/rest.js/issues/1624


IntegralDomain is misnamed

IntegralDomain is the wrong name for that concept. In abstract algebra, an IntegralDomain is not enough to provide quot and rem. Technically, I think we need an Euclidean Domain.

My suggestion would be an Euclidean Ring which would imply a quotient, a remainder, and an absolute value according to Euclidean Ring on Mathworld. The latter is not yet available as a method or member of a concept.
That name also gives rise to the association of the Ring concept, indicating the similarity of multiplication, division with quotient and remainder.

Also, and in relation to the recent discussion about unique, shouldn't quot and rem be divides and modulus, respectively div and mod for consistency ? There's a std::divides and std::modulus, as well as a mpl::divides and mpl::modulus. Similarly, operator/ and operator% are usually referred to as division and modulus. Sorry for being pedantic about that...

Yes, I think we're looking at an Euclidean Ring (which is an Euclidean Domain). I would need to add the norm method, and the euclidean algorithm would be defined by the two functions quot and rem.
Regarding the name of quot and rem: There's a subtle difference between div/mod and quot/rem in Haskell. quot truncates towards zero, whereas div rounds towards minus infinity IIRC. Since what we're doing in C++ is truncate towards zero, I went with that name. This is the historical reason, but you are right that the correct name we should be using is div/mod for consistency with the standard library. I'll create an issue.


Gamepad issue

Bevy version
bevy = { version = "0.13", features = ["dynamic_linking"] }
The release number or commit hash of the version you're using.
The last : https://github.com/bevyengine/bevy/releases/tag/v0.13.2

the Rust version

"latest stable release" of Rust

the operating system

Ubuntu 22.04.4 LTS

I used the test from the "gilrs" repo : https://gitlab.com/gilrs-project/gilrs
it's giving me this result
DragonRise Inc.   Generic   USB  Joystick   is Wired
SystemTime { tv_sec:<PHONE_NUMBER>, tv_nsec: 686915000 } New event from 0: ButtonPressed(Unknown, Code(EvCode(EvCode { kind: 1, code: 289 })))
SystemTime { tv_sec:<PHONE_NUMBER>, tv_nsec: 686915000 } New event from 0: ButtonChanged(Unknown, 1.0, Code(EvCode(EvCode { kind: 1, code: 289 })))
SystemTime { tv_sec:<PHONE_NUMBER>, tv_nsec: 822909000 } New event from 0: ButtonReleased(Unknown, Code(EvCode(EvCode { kind: 1, code: 289 })))
SystemTime { tv_sec:<PHONE_NUMBER>, tv_nsec: 822909000 } New event from 0: ButtonChanged(Unknown, 0.0, Code(EvCode(EvCode { kind: 1, code: 289 })))
I tested this example and it works for
https://bevyengine.org/examples/Tools/gamepad-viewer/
but when I try to receive the "GamepadButtonType::West" event in my code, I got nothing
if button_inputs.just_pressed(GamepadButton::new(pad, GamepadButtonType::West)) { println!("west"); }
Not just the "GamepadButtonType::West" but those also "South,East,North,West".

Check if your gamepad UUID is configured for your OS in here: https://github.com/mdqinc/SDL_GameControllerDB. This is the gamepad database that Bevy ends up depending on in the end, via gilrs.


Restore host verification

Using the definition of a valid homeserver name from here.

Done in e8f7bd9, cb857db.


Course selection for sync

As an user, I want to choose the courses to be included in the sync to download only the contents I am interested in.

[ ] show available courses
[ ] edit selection
[ ] show my selection
[ ] only selected courses are synced
[ ] selection is persisted

Does the persistence part work yet?
For me, version 1.0 stores an empty array in the .ilias-downloader.json file and thus no courses are synced unless I run course selection again.


Inconsistency in Language Display for Delivery and Pickup Time Tags in Arabic

Overview:
When changing the language to Arabic on the Enatega Food Delivery website and subsequently modifying the delivery or pickup time, an inconsistency is observed. While the delivery time tag remains in Arabic, the pickup time tag switches back to English, leading to a discrepancy in language presentation.
Steps to Reproduce:
Access the Enatega Food Delivery website.
Change the language to Arabic.
Navigate to the order placement or modification section.
Adjust the delivery and pickup times.
Observe that the delivery time tag remains in Arabic, but the pickup time tag switches to English.
Expected Behavior:
Both the delivery and pickup time tags should consistently appear in Arabic when the language is set to Arabic, ensuring a cohesive and localized user experience.
Actual Behavior:
The pickup time tag switches to English after changing the language to Arabic, causing a language inconsistency.
https://github.com/Ninjas-Code-official/Enatega-Multi-Vendor-Food-Delivery-System/assets/152266796/88d6386b-46ab-45bb-a8a2-fb387827b8a5

Resolved!

waiting for publish


fix(SeedReviewScreen): allow overflow for long passphrases [#263]

LGTM, tested and does correct issue


how to choose PRODUCT_REPO and OCP_RELEASE

export OCP_RELEASE=<release_version>
export LOCAL_REGISTRY='<local_registry_host_name>:<local_registry_host_port>'
export LOCAL_REPOSITORY='<repository_name>'
export PRODUCT_REPO='openshift-release-dev'
export LOCAL_SECRET_JSON='<path_to_pull_secret>'
export RELEASE_NAME="ocp-release"
it seems okd images located here: https://quay.io/repository/openshift/okd-content
but how to set the PRODUCT_REPO and OCP_RELEASE?
the tag here is like: 4.4.0-0.okd-2020-05-23-055148-beta5-multus-whereabouts-ipam-cni
not like: 4.4.0-x86_64

export PRODUCT_REPO='okd'
export OCP_RELEASE="<name of the image you're deploying>"


index: use Node's CommonJS JSON loader

Node natively supports loading JSON via require() since 2011, which is a cleaner and easier way to load JSON:

https://nodejs.org/en/blog/release/v0.5.2

Node's require() safely performs regular JSON decode whenever it detects a ".json" file extension, which is what our "detectable.json" filename is using.

The createRequire() API is only necessary because arRPC is not a CommonJS project. However, if the package.json was changed to "type": "commonjs" we would be able to remove those two lines to make the code even cleaner, since require() would then be available automatically.

Using createRequire() is a completely normal, supported interoperability feature of Node, and is documented here:

https://nodejs.org/api/module.html#modulecreaterequirefilename

This is ready to merge. Everything works exactly as before.

This is just a change to change? I don't see any benefit since the project will never move to commonjs.

This is just a change to change? I don't see any benefit since the project will never move to commonjs.

It's a change to help downstream apps that integrates arRPC, so that they don't need to patch that command when they include arRPC in ASAR bundles or something like that.
With the change to use require() the code works for downstream bundles.

@CanadaHonk Hi Oliver, I noticed that you're very busy with other projects. However, since this is such a small fix, I'm wondering if we can move ahead with this so that I can then alert the downstream project that they don't need to patch arRPC any longer. :)

as i pretty much not like node to run in background, i have an idea to run it with bun. ive tried it just work normally.
i have an idea where we can bundle into a single bun executable. though, it wasnt good at dealing with files when it wrapped with __dirname. so i think the optimal way is to just put like this and casually update the process.

@hUwUtao @CanadaHonk Yes, exactly. We need to merge this otherwise arRPC doesn't work when bundled (such as inside ASAR packages).


Cisco xrd ssh error

I tried to do a lab of srl Linux and Cisco xrd as provided by the documentation. I am able to login the Nokia srl using ssh but with the Cisco xrd, I am not able to ssh. I get no route to host ssh error when I tried to ssh into the node. All other nodes works except the xrd. I have not change anything with the startup config shared in the lab yml file.

Check boot log
docker logs

below are the logs i get when i run the docker logs --details
ctrace_enable_configuration(): inotify_init() failed. Too many open files (24)
ctrace_enable_configuration2 failed with error 0x2
ctrace_enable_configuration(): inotify_init() failed. Too many open files (24)
ctrace_enable_configuration2 failed with error 0x2
ctrace_enable_configuration(): inotify_init() failed. Too many open files (24)
ctrace_enable_configuration2 failed with error 0x2
ctrace_enable_configuration(): inotify_init() failed. Too many open files (24)
ctrace_enable_configuration2 failed with error 0x2

I cannot even ping the node (<IP_ADDRESS>)

Try increasing /proc/sys/fs/inotify/max_user_instances
see e.g. https://stackoverflow.com/questions/32281277/too-many-open-files-failed-to-initialize-inotify-the-user-limit-on-the-total

@steiler and @hellt Thanks a lot. after increasing /proc/sys/fs/inotify/max_user_instances it is working now. I really appreciate the help.

Thanks


make $compile accept DDO without a factory

could we add the possibility to define a directive without a factory. this would only be syntactic sugar and I could make a PR for this. This would make the creation of directives easier for beginners.
angular.module('sample', []).directive('test', {
    template: 'Hello World!'
});

Thanks for your suggestion. In my opinion, this is not something really needed. Just another way to do something. Also, it's a golden opportunity for beginners to learn what a factory is. On a side note, we have the angular.component helper lined up for 1.5, which will help create "idiomatic components" for angular 1 (also good for migration to ng2): #12166

We now have the component helper in master for these simple cases.


Hi,nice to see this lib

Hi,
I hava updated this lib and some issues have been fixed:

export blob as image
code refactoring

https://github.com/li-yu/SQLiteToExcel
Ps. I have not uploaded to the jCenter.

Thanks Bro, I will add this feature to my library.


Kafka Consumer - handler concurrency limit?

I did not see anything in the documentation related to this. Basically, if I have a consumer for a topic with 100 partitions, and it is the leader for 50 of those partitions, I don't want it to process all 50 partitions in parallel. I might want to limit it to only processing 5 or 10 partitions concurrently, to limit resource usage, etc.
If there isn't such a limit defined, consider this a feature request  :)

Is this something you've experienced as a problem, or are you considering that it could become a problem?
The overhead of one consumer consuming from many partitions should generally be fairly minimal. Each fetch would involve more requests depending on the number of brokers, but all in all I wouldn't consider that a very large cost.
If the problem is that you're processing a whole batch of messages in parallel, so that you're constrained on resources because you're processing 1000 messages at the same time, you can control the number of messages that you get in each batch with maxBytesPerPartition and/or maxBytes so that each fetch contains fewer messages. You could also in your application code simply process the messages in "mini batches". For example, if you get 1000 messages in the batch, maybe process 100 in parallel at a time.
We are experimenting a bit with creating utilities for simplifying higher level patterns outside of kafkajs itself, so maybe this is a candidate, but my gut feeling is that if you just control the batch size, you already have a good solution. Let me know if I'm missing something.

Exactly what @Nevon said, the concurrency will be limited by the size of your cluster. KafkaJS only makes requests to the leaders, so it should be a fraction of your cluster. However, I don't disagree with the feature request, I think it's a nice feature to have. @scmorse, are you having issues with resource usage?

So, the full story is that we used to use the library no-kafka, which had a consumer concurrency limit config value. We set it to 10. Now we're upgrading to use kafkajs because this lib has support for creating topics, but it is missing a few things that no-kafka had, such as the concurrency control (and some group admin functions like listGroups / fetchConsumerLag). We may be okay even without this, but it makes it a riskier upgrade, because it greatly increases the number of concurrent messages being processed.
We're not processing batches of messages from one partition in parallel, the concern was more about processing messages from many different partitions in parallel. If I have a topic with 100 partitions for example, then between all of my consumers I'd have 100 messages being processed concurrently, which could put significant strain on a database or other resources. Whereas with a concurrency limit of 10 on each of my 2 consumers, I'd only have at most 20 messages being processed concurrently.

I'm working on this now. You can follow on this branch: https://github.com/tulios/kafkajs/tree/accept-max-in-flight-requests-on-the-connection

Implemented on PR #216


Add libhdf5-dev installation

Motivation and context

How has this been tested?

Checklist

[x] I submit my changes into the develop branch
[ ] I have created a changelog fragment
[ ] I have updated the documentation accordingly
[ ] I have added tests to cover my changes
[ ] I have linked related issues (see GitHub docs)
[ ] I have increased versions of npm packages if it is necessary
(cvat-canvas,
cvat-core,
cvat-data and
cvat-ui)

License

[x] I submit my code changes under the same MIT License that covers the project.
Feel free to contact the maintainers if that's a concern.

Changes summary
Running command from https://docs.cvat.ai/docs/contributing/development-environment/
docker compose -f docker-compose.yml -f docker-compose.dev.yml up -d --build \
  cvat_opa cvat_db cvat_redis_inmem cvat_redis_ondisk cvat_server

Resulted in build error:
90.77       running build_ext
90.77       Loading library to get build settings and version: libhdf5.so
90.77       error: Unable to load dependency HDF5, make sure HDF5 is installed properly
90.77       Library dirs checked: []
90.77       error: libhdf5.so: cannot open shared object file: No such file or directory
90.77       [end of output]
...
------
failed to solve: process "/bin/sh -c DATUMARO_HEADLESS=1 python3 -m pip wheel --no-deps     -r /tmp/cvat/requirements/${CVAT_CONFIGURATION}.txt     -w /tmp/wheelhouse" did not complete successfully: exit code: 1

Installing libhdf5-dev solved the error and allowed image to be built.

Summary by CodeRabbit

New Features

Enhanced Docker image functionality with the addition of HDF5 support for improved data handling and processing capabilities.

Closing in favour of #8093


Update PetClinicApplication.java

@Seshathri-saravanan Please sign the Contributor License Agreement!
Click here to manually synchronize the status of this Pull Request.
See the FAQ for frequently asked questions.


Move basic gulp tasks to core

Work starts from simplifying es5 template.
Move reading configuration / services etc. from projects gulpfile to core...

done in 0.0.4


Help with SkinChangers Image files

People, I'm building a function to display the skins images on the skinchanger.
ex: you selected the Dragon Lore, it will be displayed in the preview.
A problem I found and having images of skins, it seems to me that the valve stores a link in the csgo folder (\ csgo \ scripts \ items)
but I have no idea of converting the values inside those files into a variable, if someone can help me with that I will be happy, I intend to make everything available through a pull request.

https://github.com/ocornut/imgui/wiki/Image-Loading-and-Displaying-Examples

Its pretty useless i think also accept my friend request in discord

https://github.com/ocornut/imgui/wiki/Image-Loading-and-Displaying-Examples

so, i already understood how the image system works, i already managed to use images in the menu.

Its pretty useless i think also accept my friend request in discord

already accepted.

https://yougame.biz/threads/42556/

Use ItemDefinitionIndex() functions to find the current weaponID, and beforehand, use AddFontFromMemoryTTF() (ImGUI) to load an icon base Font; which is a true font file with all of the weapon icons saved inside.
You can find one on UC I think, take a look.

Use ItemDefinitionIndex() functions to find the current weaponID, and beforehand, use AddFontFromMemoryTTF() (ImGUI) to load an 'icon base' font, that's what I call them at least; which is a true font file with all of the weapon icons saved inside.
You can find one on UC I think, take a look.

Thanks


Add Information about User Management (Fallback Domain)

Make your new onmicrosoft.com domain your fallback domain
Where can I find a guide, to change all User to this new Fallback Domain?
Would it be possible to Link or Add that here?
(User with *.onmicrosoft .comDomain as Primary E-Mail)

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

ID: 0a66bd91-6de1-5128-0f77-df1573b7f800
Version Independent ID: 0a66bd91-6de1-5128-0f77-df1573b7f800
Content: Add and replace your onmicrosoft.com fallback domain in Microsoft 365 - Microsoft 365 admin
Content Source: microsoft-365/admin/setup/add-or-replace-your-onmicrosoftcom-domain.md
Service: o365-administration
GitHub Login: @efrene
Microsoft Alias: efrene

@weyCC81 Thank you for your feedback. Fallback domains are for the entire organization. So the document talks about changing it to a new onmicrosoft.com domain if you create one (it has steps to do that ).
Let's say you have contoso.onmicrosoft.com as your current fallback domain, you can change it to the new one contoso1.onmicrosoft.com so in case your custom domain is no longer active or has been removed all users in the organization will move to this fallback domain. No need to change it for individual users.
Hope this helps!
Thanks
Sri

@msbemba Thank you very much for the contribution and sharing this explanation. @weyCC81 Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.

Hi @msbemba
I have added one as we have a spelling error in the initial one and would like to switch sharepoint in the future.
It is still not clear to me what action could start such a fallback, and/or what do you mean with "no longer active".
I thought you can not remove them if you have added them once (up to 5 max)?

@weyCC81 That is correct. You cannot remove onmicrosoft.com domain, however the custom domains like contoso.com can be removed leaving just the onmicrosoft.com.
Once you add another onmicrosoft.com domain to the tenant using https://docs.microsoft.com/en-us/microsoft-365/admin/setup/add-or-replace-your-onmicrosoftcom-domain?view=o365-worldwide#add-a-new-onmicrosoftcom-domain
You can make that one as fallback domain using the steps mentioned here - https://docs.microsoft.com/en-us/microsoft-365/admin/setup/add-or-replace-your-onmicrosoftcom-domain?view=o365-worldwide#make-your-new-onmicrosoftcom-domain-your-fallback-domain
Once the fallback domain is changed, Sharepoint sites will start reflecting correctly now.
Hope this helps!
Thanks
Sri


BBCode [img] shows up immediately

Describe the bug
When using an image tag in a dialogue, it shows up as soon as the dialogue starts typing out.
Affected version

Dialogue Manager version: 2.31.0
Godot version: 4.2.1.stable

To Reproduce
Steps to reproduce the behavior:

Make a dialogue containing an [img][/img] tag
Run the dialogue
See the image show up immediately

Expected behavior
The image should only show up when the dialogue has reached that point.
Screenshots
See short video showing the issue in action.
https://github.com/nathanhoad/godot_dialogue_manager/assets/2348534/0aaa2c8f-1988-4fe0-9834-06a1e256c7d0

This is a property on RichTextLabels. I've reverted it back to the default value (which shows images as they are typed out) in the DialogueLabel.

Downloaded the dialogue manager recently for godot and had this issue, for some reason my version didn't have that revert on it, but the fix worked for me!


[sentences/a 4-words sentence/a 4-words from 16] Wrong stress in a word "чертит" in the exercises 45, 46

Preconditions:
1 Log in as an existing user. Open: groups/speech exercises/sentences/a 4-words sentence/a 4-words from 16/45 (46) exercise
https://brainup.site/groups/2/series/4/subgroup/31/exercise/1353/task/1345
https://brainup.site/groups/2/series/4/subgroup/31/exercise/1354/task/1346
Step:
Press the red button
Actuai result: черти́т
Expected result: че́ртит

Corrected


Increase the verbosity of failing audits

Debug through graphql-http instead of the auditing server. A great step forwards would already be to attach the full response as a part of the audit result.

:tada: This issue has been resolved in version 1.12.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


H3Core.nonZeroLongArrayToList takes too much memory

Trying to polyfill big area with resolution 11. I can see that internally algorithm prepared result https://github.com/uber/h3-java/blob/3a1e9bc9bcd1d62e9e8d4361230fe47bb5395242/src/main/java/com/uber/h3core/H3Core.java#L691 but when it tries to copy it to List it gets out of memory on this line:
https://github.com/uber/h3-java/blob/a500880eef493dca44903bf7ad0e249aed5b85b1/src/main/java/com/uber/h3core/H3Core.java#L1242-L1254
As you can see in the stacktrace, result is ready, but it just tries to copy it to ArrayList:

Memory, the arrow shows to the place when nonZeroLongArrayToList started :

Scala code:
import scala.collection.JavaConverters._
import java.util.{Collections => JCollections}

import beam.utils.ProfilingUtils
import com.uber.h3core.AreaUnit
import com.uber.h3core.util.GeoCoord

def main(args: Array[String]): Unit = {

  val h3Core = com.uber.h3core.H3Core.newInstance

  val xMin = -106.645646
  val xMax = -93.508292
  val yMin = 25.837377
  val yMax = 36.500704

  val rectangle = java.util.Arrays.asList(
    new GeoCoord(yMin, xMin),
    new GeoCoord(yMax, xMin),
    new GeoCoord(yMax, xMax),
    new GeoCoord(yMin, xMax),
  )

  val holes = java.util.Collections.emptyList[java.util.List[GeoCoord]]()
  val resolution = 11
  val hexes = h3Core.polyfillAddress(rectangle, holes, resolution)
  println(s"Generated ${hexes.size()}")
}

The easiest fix is just to get the total number of non-zero elements and allocate an array with that size and copy elements over. Other solution can be moving all zero elements in the original array to the end of the array and wrap it by ArrayList via Arrays.asList(array).subList(index, IDX_OF_FIRST_ZERO);. I can create PR if this sounds good.
Thanks.

Hi, sorry for the delayed response! Yes, I think counting the number of non-zero indexes is a reasonable change. Off hand I don't know the performance of the other option, but I assume you want to make sure it does not keep the entire result array alive. Alternately we could do something like allowing the ArrayList to dynamically resize, but I suspect that would always be slower.
I'm curious if the result array itself already takes up roughly 8GB memory - is that being shown in your memory profiler with the 8GB of memory already being used?

In my case the size of the ArrayList is only one small part of the problem. The actual problem is, that the number returned by maxPolyfillSize is way too big. I opened another issue here: https://github.com/uber/h3/issues/708


Create Alarm-Clock.py

Created a new Python Project which will help many beginners to take there first step

Only web dev projects is acceptable


com.google.api.core.ApiFuture should also return timestamp and not just message id

I would like to know the timestamp of the message which was generated by the server when the message was published for one of my audit use cases as well as logging. Currently we only receive the message id, but would like to extend this to  object which can contain metadata of the message such as timestamp , message id (in future we can add any more information) rather than just returning string.
public com.google.api.core.ApiFuture publish(PubsubMessage message) should be
public com.google.api.core.ApiFuture publish(PubsubMessage message) , whereas PubsubMessage should contain the server timestamp populated.

Since this request is more of a service-wide request and not something limited to the Java client library, we will track it via the issue tracker.


container throttling widgets

Utilization is not enough. We need a way of visualizing if a container resource is being throttled, either by a hardware limitation or configured software limit.

[ ] throttled cpu
[ ] throttled network io

@brendangregg can you list the metrics that would be necessary to implement this?

For CPU, according to the Kernel docs, perhaps
cpu.stat:
- nr_periods: Number of enforcement intervals that have elapsed.
- nr_throttled: Number of times the group has been throttled/limited.
- throttled_time: The total time duration (in nanoseconds) for which entities

@spiermar good timing - I've been working on a console-based tool for reporting on containers.  From a PCP POV, the metric hierarchies that are in some way related to containers are:

cgroups.*
proc.*
containers.*
network.interface.*
pmcd.hostname
filesys.*

(those are all influenced by the --container option too - see http://pcp.io/docs/lab.containers.html for more details)
@meteorfox see src/pmdas/linux_proc/cgroups.c line 584 - those CPU metrics you mentioned are currently missing from PCP, but easily added.

| [...] those CPU metrics you mentioned are currently missing from PCP [...]
pcp master branch now has:
$ pminfo -t cgroup.cpusched
cgroup.cpusched.shares [Processor scheduler cgroup shares]
cgroup.cpusched.periods [Number of CFS elapsed enforcement intervals]
cgroup.cpusched.throttled [Number of times CFS group was throttled/limited]
cgroup.cpusched.throttled_time [Total time group was throttled/limited]
(will be in pcp-3.10.7)


PydanticUserError when instantiating SyntheticDataGenerator

I am attempting to use this library to generate tabular synthetic data. Any time I try to instantiate the SyntheticDataGenerator class from tabular_synthetic_data (either through the create_openai_data_generator function or directly), I get an error like the below:

PydanticUserError: SyntheticDataGenerator is not fully defined; you should define BaseCache, then call SyntheticDataGenerator.model_rebuild().
For further information visit https://errors.pydantic.dev/2.10/u/class-not-fully-defined

versions (all installed via pip)
langchain_experimental: 0.3.3
langchain: 0.3.11
langchain_openai 0.2.12
langchain_core: 0.3.24
Here is simplified example code that causes the error:
from langchain_openai import ChatOpenAI
from langchain_core.prompts import FewShotPromptTemplate
from langchain_experimental.tabular_synthetic_data.base import SyntheticDataGenerator
from pydantic import BaseModel

OPENAI_TEMPLATE = PromptTemplate(input_variables=["example"], template="{example}")

examples = [
    {"example": "toothbrush_name: Crest Brush, price: 10"},
    {"example": "toothbrush_name: Generic toothbrush, price: 5"}
]

class ToothbrushSchema(BaseModel):
    toothbrush_name: str
    price: int

output_schema=ToothbrushSchema

llm=ChatOpenAI(temperature=1)

prompt_template = FewShotPromptTemplate(
    examples=examples,
    suffix=("Generate synthetic data about {subject}."),
    input_variables=["subject"],
    example_prompt=OPENAI_TEMPLATE,
)

prompt=prompt_template

chain = llm.with_structured_output(ToothbrushSchema)

sdg = SyntheticDataGenerator(template=prompt_template,llm_chain=chain)

I got it to work by creating a dev install of langchain_experimental and editing line 27 tabular_synthetic_data/base.py
BEFORE:
llm: Optional[BaseLanguageModel] = None
AFTER:
llm: Optional[BaseChatModel] = None
I am not sure why this worked. BaseLanguageModel is the parent class of BaseChatModel.


enhancing README and utilizing venvs

Hi, Manning asked me to review your book and I accepted. Only at Ch1-2 but pretty good thus far.
I've started getting into the code aspects of your book.
This pull request contains the following:

Updated README with steps on how to use your jupyter notebooks using virtual environments.
Added an requirements.txt with some initial libraries (I will add more as I step deeper through your notebooks)

Thanks,
Omer

improvement suggestions to the first notebook.
updated to the gpt4o-mini model, updated code to pull API from a local file and added the file to .gitignore so it doesn't get uploaded to github


try_cast documentation first draft

updated documentation to include try_cast
es upstream fix

The case when TRY_CAST fails with an error if the the cast is explicitly not permitted is not documented

othwerwise ack


Messages spammed players

We've been using Harbor fine for several days but it suddenly spammed message. Not sure I can provide any more info. Latest version for paper b273 for MC 1.15.2.
https://puu.sh/FJDjH/65333426a5.png

We've been using Harbor fine for several days but it suddenly spammed message. Not sure I can provide any more info. Latest version for paper b273 for MC 1.15.2.
https://puu.sh/FJDjH/65333426a5.png

What Harbor version?

1.6.1


Keda operator gives error "Webhook not found. Unable to update certificate." after disabling admission webhook

Report
We deployed keda 2.13.0 after disabling admission webhook but we are constantly getting below error of unable to update certificate. Is disabling admission webhook disbales cert updation / rotation for metric server and keda-operator as well??
Expected Behavior
Keda operator should able to rotate / update certificate for keda-operator, keda-metricserver and ignore webhook service.
Actual Behavior
keda-operator throwing error "Webhook not found. Unable to update certificate."
Steps to Reproduce the Problem

Disable admission webhook
Deploy keda operator and keda-metric service with keda managed cert

Logs from KEDA operator
{"level":"error","ts":"2024-07-15T08:03:38Z","logger":"cert-rotation","msg":"Webhook not found. Unable to update certificate.","name":"keda-admission","gvk":"admissionregistration.k8s.io/v1, Kind=ValidatingWebhookConfiguration","error":"ValidatingWebhookConfiguration.admissionregistration.k8s.io \"keda-admission\" not found","stacktrace":"github.com/open-policy-agent/cert-controller/pkg/rotator.(*ReconcileWH).ensureCerts\n\t/workspace/vendor/github.com/open-policy-agent/cert-controller/pkg/rotator/rotator.go:816\ngithub.com/open-policy-agent/cert-controller/pkg/rotator.(*ReconcileWH).Reconcile\n\t/workspace/vendor/github.com/open-policy-agent/cert-controller/pkg/rotator/rotator.go:785\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Reconcile\n\t/workspace/vendor/sigs.k8s.io/controller-runtime/pkg/internal/controller/controller.go:119\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler\n\t/workspace/vendor/sigs.k8s.io/controller-runtime/pkg/internal/controller/controller.go:316\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem\n\t/workspace/vendor/sigs.k8s.io/controller-runtime/pkg/internal/controller/controller.go:266\nsigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2\n\t/workspace/vendor/sigs.k8s.io/controller-runtime/pkg/internal/controller/controller.go:227"}

KEDA Version
2.13.0
Kubernetes Version
None
Platform
None
Scaler Details
No response
Anything else?
No response

We have exactly the same configurations and error with Keda version 2.15.2.


Support for 0.10?

Do you plan on supporting 0.10? Is this module still alive?

I haven't worked on it or used it in a while. It could be alive if you want to bring it back to life and add 0.10 support :) I personally am not using it right now cause I am on a different project so won't have the time to do the 0.10 support myself.

Thanks for the reply. I'm checking out hot-shots too but if that doesn't work out I'll take a crack at this.


Extract created table/view table name without building table catalog

Hi.
I am looking for a way to extract the table/view name from table/view create statements without building a table catalog and analyzing the statement.
For example, consider the following:
String sql = "create or replace table project2.dataset2.table2 as SELECT column1 from project1.dataset1.table1";
List<List<String>> tables = Analyzer.extractTableNamesFromStatement(sql, ZetaSqlAnalyzerFactory.getAnalyzerOptions());

With the above, I am able to retrieve the names of the tables involved in the query, in this case project1.dataset1.table1. What I am looking for is how to extract the created table project2.dataset2.table2 without building the table catalog and analyzing the statement. I can retrieve them building a table catalog and analyzing the statement, my issue is that I am not always able to build the table catalog, so I am looking for a way to bypass this step.
Best regards

You've got the right function call, but you need to explicitly enable create and replace statements in LanguageOptions (and whatever else you want to support).
You can see an example of this in AnalyzerTest.testExtractTableNamesFromStatementWithAnalyzerOptions:
`public void testExtractTableNamesFromStatementWithAnalyzerOptions() {
AnalyzerOptions analyzerOptions = new AnalyzerOptions();
analyzerOptions
.getLanguageOptions()
.setSupportedStatementKinds(
ImmutableSet.of(
ResolvedNodeKind.RESOLVED_INSERT_STMT,
ResolvedNodeKind.RESOLVED_QUERY_STMT));
List<List<String>> tableNames =
    Analyzer.extractTableNamesFromStatement(
        "insert into baz select count(1) from foo.bar", analyzerOptions);

assertThat(tableNames).containsExactly(Arrays.asList("baz"), Arrays.asList("foo", "bar"));

}`

Hi @matthewcbrown  thanks for the reply.
I have now added the RESOLVED_CREATE_TABLE_AS_SELECT_STMT to the list of supported statements.
AnalyzerOptions analyzerOptions = new AnalyzerOptions();
        analyzerOptions
                .getLanguageOptions()
                .setSupportedStatementKinds(
                        ImmutableSet.of(
                                ZetaSQLResolvedNodeKind.ResolvedNodeKind.RESOLVED_CREATE_TABLE_AS_SELECT_STMT));
        String sql = "create table project2.dataset2.table2 as SELECT column1 from project1.dataset1.table1";
        List<List<String>> tables = Analyzer.extractTableNamesFromStatement(sql, analyzerOptions);

When running the above, I still only get the output [[project1, dataset1, table1]] and not project2.dataset2.table2, am I missing something? Even if this would work, I am trying to extract the created table name only, so I am trying to retrieve project2.dataset2.table2, would this be possible somehow?
Thanks

ahh, you are right.  Looking at the code, this doesn't currently seem possible.  I'll ask around to see if anyone can think of a workaround.

@cthrysoe
Can you provide the steps to extract tables from statements? I am facing issues while setting up and running that file


Encrypt the password

encrypt shenyu-admin login password #5255

do you test this pr?

yes in h2 database

add my wechat: yijia1949

I have added

This topic also involves client meta-information registration login.

本主题还涉及客户端元信息注册登录。
Can you be more specific?

本主题还涉及客户端元信息注册登录。
Can you be more specific?

see:
https://github.com/apache/shenyu/blob/master/shenyu-register-center/shenyu-register-client/shenyu-register-client-http/src/main/java/org/apache/shenyu/register/client/http/HttpClientRegisterRepository.java

Codecov Report
Attention: 3 lines in your changes are missing coverage. Please review.

Comparison is base (85c2509) 61.14% compared to head (99250ee) 61.07%.

Files
Patch %
Lines

...u/admin/service/impl/DashboardUserServiceImpl.java
85.00%
2 Missing and 1 partial :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #5436      +/-   ##
============================================
- Coverage     61.14%   61.07%   -0.08%
+ Complexity     8652     8645       -7
============================================
  Files          1258     1258
  Lines         38375    38392      +17
  Branches       3670     3671       +1
============================================
- Hits          23466    23447      -19
- Misses        12799    12827      +28
- Partials       2110     2118       +8

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

the password of the database should be encrypted

the password of the database should be encrypted

The database password change needs to be compatible with upgrade and downgrade scenarios. I think another topic can be used to change the database password.

need to change


713 deleting clinvar export submission sets does not work backport

Codecov Report

:exclamation: No coverage uploaded for pull request base (anthenea@f105b2d). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@             Coverage Diff             @@
##             anthenea     #715   +/-   ##
===========================================
  Coverage            ?   83.06%
===========================================
  Files               ?      129
  Lines               ?    12722
  Branches            ?        0
===========================================
  Hits                ?    10567
  Misses              ?     2155
  Partials            ?        0

Flag
Coverage Δ

javascript
83.06% <0.00%> (?)

python
83.06% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


Can I set the uniqueness based on job class only?

Hi, I'm interested to only have one job running at any given time based on job's class name and not the arguments.
Any idea how to do so?
or alternatively, can we filter by queue?
Thanks.

Hey @olimart
You can define a custom lock_key_arguments to return [] which results a lock key like this one.
Alternatively, you can define a lock_key to set it to any value you want.

Ah I see, instead of defining criteria, you let them blank 👍
Thanks a lot.


container stats are not working for  centos 7 hosts with docker 1.11.2.

Rancher Version: v1.1.0-rc1
Docker Version:  docker 1.11.2
Host os version - CentOS Linux 7 (3.10.0-327.22.2.el7.x86_64)
Container stats are not showing up and is stuck in "Connecting" state.

 cattlevalidationtest.core.test_host_api.test_host_api_containerstats
 Error Details
assert <built-in method startswith of str object at 0x7f45557d8508>('[')  +  where <built-in method startswith of str object at 0x7f45557d8508> = ''.startswith
 Stack Trace
client = <cattle.Client object at 0x7f4547f2a050>

    def test_host_api_containerstats(client):
        container = client.create_container(name=random_str(),
                                            imageUuid=TEST_IMAGE_UUID)
        container = client.wait_success(container, timeout=600)

        assert len(container.hosts()) == 1

        stats = container.containerStats()
        conn = ws.create_connection(stats.url + '?token=' + stats.token)
        result = conn.recv()
        conn.close()
        assert result is not None
>       assert result.startswith('[')
E       assert <built-in method startswith of str object at 0x7f45557d8508>('[')
E        +  where <built-in method startswith of str object at 0x7f45557d8508> = ''.startswith

tests/validation/cattlevalidationtest/core/test_host_api.py:72: AssertionError
2.7 sec	1
 cattlevalidationtest.core.test_host_api.test_host_api_service_containerstats
 Error Details
assert <built-in method startswith of str object at 0x7f45557d8508>('[')  +  where <built-in method startswith of str object at 0x7f45557d8508> = ''.startswith
 Stack Trace
client = <cattle.Client object at 0x7f4547f2a050>

    def test_host_api_service_containerstats(client):
        env = client.create_environment(name=random_str())
        env = client.wait_success(env)
        assert env.state == "active"

        image_uuid = TEST_IMAGE_UUID
        launch_config = {"imageUuid": image_uuid}

        service = client.create_service(name=random_str(),
                                        environmentId=env.id,
                                        launchConfig=launch_config)
        service = client.wait_success(service)
        assert service.state == "inactive"
        service.activate()
        service = client.wait_success(service, timeout=600)
        assert service.state == "active"
        stats = service.containerStats()

        conn = ws.create_connection(stats.url + '?token=' + stats.token)
        result = conn.recv()
        conn.close()
        assert result is not None
>       assert result.startswith('[')
E       assert <built-in method startswith of str object at 0x7f45557d8508>('[')
E        +  where <built-in method startswith of str object at 0x7f45557d8508> = ''.startswith

tests/validation/cattlevalidationtest/core/test_host_api.py:98: AssertionError

cattlevalidationtest.core.test_native_docker.test_native_container_stats
 Error Details
assert 'per_cpu_usage' in ''
 Stack Trace
client = <cattle.Client object at 0x7f454e379ad0>, socat_containers = None
native_name = 'native-test-219800', pull_images = None

    def test_native_container_stats(client, socat_containers, native_name,
                                    pull_images):
        docker_client = get_docker_client(host(client))
        docker_container = docker_client. \
            create_container(NATIVE_TEST_IMAGE,
                             name=native_name,
                             tty=True,
                             stdin_open=True,
                             detach=True,
                             command=['/bin/bash'])
        rancher_container, _ = start_and_wait(client, docker_container,
                                              docker_client,
                                              native_name)

>       assert_stats(rancher_container)

tests/validation/cattlevalidationtest/core/test_native_docker.py:277:
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _

container = {'tty': True, 'hostname': u'c73f0d1c4ce3', 'dataVolumes': [], 'registryCredentialId': None, 'createdTS':<PHONE_NUMBER>000...ommand': [u'/bin/bash'], 'entryPoint': [u'/entry.sh'], 'createIndex': None, 'dnsSearch': [], 'ports': [], 'user': None}

    def assert_stats(container):
        stats = container.stats()
        conn = ws.create_connection(stats.url + '?token=' + stats.token,
                                    timeout=10)
        result = conn.recv()
>       assert 'per_cpu_usage' in result
E       assert 'per_cpu_usage' in ''

tests/validation/cattlevalidationtest/core/test_container.py:224: AssertionError

 cattlevalidationtest.core.test_container.test_container_stats
 Error Details
assert 'per_cpu_usage' in ''
 Stack Trace
client = <cattle.Client object at 0x7f454fc2a6d0>, test_name = 'test-920524'

    def test_container_stats(client, test_name):
        cleanup_items = []
        try:
            container = client.create_container(name=test_name,
                                                imageUuid=TEST_IMAGE_UUID,
                                                networkMode=MANAGED_NETWORK,
                                                attachStdin=True,
                                                attachStdout=True,
                                                tty=True,
                                                command='/bin/bash')
            cleanup_items.append(container)
            container = client.wait_success(container)

>           assert_stats(container)

tests/validation/cattlevalidationtest/core/test_container.py:210:
_ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _ _

container = {'tty': True, 'hostname': None, 'dataVolumes': [], 'registryCredentialId': None, 'createdTS':<PHONE_NUMBER>000, 'labels':...'/bin/bash'], 'entryPoint': None, 'createIndex': None, 'dnsSearch': [u'rancher.internal'], 'ports': None, 'user': None}

    def assert_stats(container):
        stats = container.stats()
        conn = ws.create_connection(stats.url + '?token=' + stats.token,
                                    timeout=10)
        result = conn.recv()
>       assert 'per_cpu_usage' in result
E       assert 'per_cpu_usage' in ''

tests/validation/cattlevalidationtest/core/test_container.py:224: AssertionError

The same issue is also seen with RHEL 7.2 (3.10.0-327.22.2.el7.x86_64) and docker 1.11.2

I can confirm this as well for RHEL 7.2. Once fixed we'll upgrade to 1.11. Happy to test a dev version with fix once available.

+1
Facing same issue on Centos 7 and Docker 1.11.1
The stats works on the VM/host but not for containers (baseimage: centos) deployed on the VM. Strange.

+1 same issue on Ubuntu 16.04 LTS.

I have this as error on host agent:
https://github.com/rancher/rancher/issues/3384
And looks like same ws:// problem just close connection
level="error" msg="Couldn't find container for id." error="token is expired" id=""
Even with 1.10.3:
Containers: 7
Running: 5
Paused: 0
Stopped: 2
Images: 3
Server Version: 1.10.3
Storage Driver: aufs
Root Dir: /var/lib/docker/aufs
Backing Filesystem: xfs
Dirs: 32
Dirperm1 Supported: true
Execution Driver: native-0.2
Logging Driver: json-file
Plugins:
Volume: local
Network: null host bridge
Kernel Version: 4.6.4-1.el7.centos.x86_64
Operating System: CentOS Linux 7 (Core)
OSType: linux
Architecture: x86_64
CPUs: 2
Total Memory: 11.73 GiB

@djcenox token expired issues are often due to the date/time of the compute nodes not being in sync with the rancher server host. Make sure those are in sync.

Omg... done. Everything is working fine even with 1.11.2.
Thanks.

@djcenox great!

The time zone on all nodes must be sync with rancher-server time zone. Syncing the time zone, resolved the issue for me.

Due to the move from using cadvisor to docker stats, please re-test.

Tested with Version 1.2.0-pre4-rc4. The container stats are working with Centos7 and RHEL7 with Docker 1.11.2.


Fix phpstan warning

Coverage remained the same at 100.0% when pulling 8f985e173d21512ec18fe02150491a91a8e9ff4a on GrahamCampbell:patch-1 into 58fbc5921dfff252daf6de4cff8132c6be8cd9e9 on phpDocumentor:master.


some code fix

@commandcobra7 It's good that you're contributing to the yapb project, but please make your commit messages more informative, and not something like "code fix".
This will make it easier to understand what issue you have solved.

@commandcobra7 It's good that you're contributing to the yapb project, but please make your commit messages more informative, and not something like "code fix".
This will make it easier to understand what issue you have solved.
And there is no need to close this pull request, we can edit the commit message ourselves.

@Vladislav4KZ I wrote below about the change, if you don't approve, I will remove it.
1-Collision case the bot can jump if it's not on the ladder, so it won't press the jump button when going up the ladder.
2-Removed unnecessary codes in ladder handling.


CI/CD pipeline - flutter  app

Hi , I am currently working on setting up a CI/CD pipeline using Jenkins and have encountered an issue during the release of the APK. Specifically, I am receiving the following error:

What went wrong:
Execution failed for task ':flutter_logs:generateReleaseRFile'.

Could not resolve all files for configuration ':flutter_logs:releaseCompileClasspath'.
Failed to transform RxLogs-1.0.20.aar (com.github.umair13adil:RxLogs:1.0.20) to match attributes {artifactType=android-symbol-with-package-name, org.gradle.category=library, org.gradle.dependency.bundling=external, org.gradle.libraryelements=aar, org.gradle.status=release, org.gradle.usage=java-api}.
Could not find RxLogs-1.0.20.aar (com.github.umair13adil:RxLogs:1.0.20).
Searched in the following locations:
https://jitpack.io/com/github/umair13adil/RxLogs/1.0.20/RxLogs-1.0.20.aar

Also facing the same issue with codemagic

Also facing this issue. In the past this PR(#57 ) solved it

any idea how I can use this PR(#57) in my code? and what version of flutter_logs should I downgrade to?

any idea how I can use this PR(#57) in my code? and what version of flutter_logs should I downgrade to?

@kwaku-pistis well, if you need to solve this asap you should fork this repo and change the dependency "com.github.umair13adil:RxLogs:1.0.20" for "com.github.umair13adil:RxLogs:1.0.23" in the build gradle

any idea how I can use this PR(#57) in my code? and what version of flutter_logs should I downgrade to?

@kwaku-pistis well, if you need to solve this asap you should fork this repo and change the dependency "com.github.umair13adil:RxLogs:1.0.20" for "com.github.umair13adil:RxLogs:1.0.23" in the build gradle

@aberutti-convorelay  thank you

the url is wrong should be
https://jitpack.io/com/github/umair13adil/RxLogs/v1.0.20/RxLogs-1.0.20.aar

Hi  @kwaku-pistis and @aberutti-convorelay
I have resolved the issue. It was related to the packages you installed in the project

@hamzaoubouzid Hi. Can you elaborate please? How do you resolve it?

Hi, I have the same issue here ! Can you elaborate please? How do you resolve it?
I'm using flutter_logs: 2.1.12
when I try to build shows this:

I'm using this flutter version:

Tks!

@hamzaoubouzid Hi. Can you elaborate please? How do you resolve it?
can you send me your pubspec.yaml

hi  @aberutti-convorelay  can you send me your pubspec.yaml

Please use version 1.0.22. There is an issue at jCenter that's why the version had to be updated.


Support the "back" button

Currently, back button quits the app instead of going back to previos page if there is one.

Indeed, that's a bug. I did not plan to make the app as a dedicated browser, but I guess not everybody wants to install Tinfoil or FaceSlim so separate Facebook from the main browser.
I'm also thinking about using https://touch.facebook.com instead of https://m.facebook.com so it's nicer for the user-experience.

Re: dedicated browser
What do you think about an option to open the links with other browsers?
Or tap-and-hold to copy/share the link?
Then no navigation from the notifications page would be necessary/possible.
Re: touch interface
Yes, that should look better. Did you consider directly opening the notifications page?
https://touch.facebook.com/?soft=notifications
Thanks!

As clicking from the notification just asks Android to browse to facebook, it should show the dialog "Choose the application to continue the action", so a standard browser can be chosen (I've chosen Tinfoil for Facebook).
Taping the notification opens the most specific link:

If you have only friends requests, it opens https://m.facebook.com/friends/center/requests/
If you have only messages, it opens https://m.facebook.com/messages/
If you have only notifications, it opens https://m.facebook.com/notifications.php
If you have a combination of the above, it opens https://m.facebook.com/ with additional buttons to open more specific pages

If you have another behavior, could you please tell me, I've not noticed any malfunctions in this process.

You're right. First I've chosen WebApps, but if you've disabled third-party requests (except for white-listed ones, saved per web app) it doesn't work smoothly when opened externally (from a link) vs. opening a web app directly. So, I switched back to FacebookNotifications as the default app.
The scenario I proposed would be:

tapping the notification
FacebookNotifications opens
any subsequent link in the app opens the "Choose the application to continue the action" dialog
But I'm not sure if it's feasible or possible at all...

I don't see the benefit of the workflow you suggest compared to starting directly with "3".
Also, I have not heard of WebApps so I'm not sure to understand exactly what is the problem: when you open a Facebook link, you are always redirected to the main page of the website and not to the more specific link? (I mean, loading test.com/deep/link.html loads simply test.com)?

Sorry for this digression. I figured that the workflow I suggested is actually a workaround for WebApps' misbehaviour. Please disregard this.
The problem is that when the page is opened "externally" in WebApps, 3rd party domains are disabled, so Facebook is opened without resources from fbcdn.net and akamaihd.net which makes it unusable until the domains are manually unblocked.

The "back" button handling has been added.


Mel normalization

Hi @Rayhane-mamah regarding the mel-features normalization in the [-4, 4] range, I was wondering why this range? and why not the usual zero-mean unit-variance range of [-1 1]? Any insight would be helpful.

"I mainly scale to [-4, 4] to enable for more detail in the output mel spectrogram (in the model's vision). Purely based on experience, outputs scaled to large domains tend to make T2 converge faster and better.."
https://github.com/Rayhane-mamah/Tacotron-2/issues/29

Thanks for the quick answer


"The Art of Darts": Block "get random value from [list]" in tutorial can't be clickable

Describe the bug
"The Art of Darts": Block "get random value from [list]" in tutorial can't be clickable
Steps to reproduce the behavior

Navigate to https://arcade.makecode.com/--skillmap#space
The Art of Darts---->Start
Click step7
Click block "get random value from [list]"

Actual behavior
Block "get random value from [list]"  in tutorial can't be clickable
Expect behavior
Block "get random value from [list]" in tutorial can be clickable. And open it in toolbox if you click in tutorial.
Screenshot

Additional context
1.OS: Windows11
2.arcade version: 1.13.5
3.Microsoft MakeCode version: 8.6.4

I'm not actually sure what's going on here. It doesn't appear to be the markdown (unless "arrays" is no longer the proper way to address the Arrays category)...but it appears that nothing tied to the Arrays category is opening it.  @jwunderl  Can you take a look?
FYI, the Images category seems to be opening just fine, so it's not the fact that it's hidden under Advanced.

Huh, did this every work? Looks like it needs another special case as the 'namespace' is Array but our category is Arrays

will put up a fix

Thanks, @jwunderl !


Include JvmTestEnvironment in Build Server Capabilities

It was requested by @jastice here https://github.com/JetBrains/intellij-scala/pull/500#pullrequestreview-360335173

Looks good, but a thing to consider is if language/env extensions shoudl declare their capabilities deparately from BuildServerCapabilities. Any opinion @olafurpg @jvican ?

I don’t think that’s necessary. In Metals we declare lsp extensions in the initialize handshake.

Of course I can move it to another Capabilities structure, please just let me know :)


Tree chart type: 'broken' modified in latest version

Version
4.2.1-rc1
Steps to reproduce
Creating a tree diagram (using echarts version 3 or 4) with straight lines and square corners like in echarts version 2 using type:'broken'
What is expected?
Tree chart with straight lines and square corners
What is actually happening?
On the new version of echarts tree chart, type:’broken’ modifies the texture of the line, not the shape of the line as in older version.

The link below shows the type of  broken tree chart in echarts version 2.
https://echarts.baidu.com/echarts2/doc/example/tree1.html#-en

@BrianWahinya Can you provide the code in ECharts v4?

// executed once html doc is 100% loaded
$(document).ready(onReady);
// main function
function onReady(){
var data = {
name: "Total (15 a 24 años)",
children: [ {
name: '20 a 24 años',
children: [ {
name: "Urbana",
value: 20.9
},
{
name: "Rural",
value: 27.5
} ]
},
{
name: '15 a 19 años',
children: [ {
name: "Urbana",
value: 13.3
},
{
name: "Rural",
value: 16.9
} ]
} ]
};
//Generate Tree series
var serie = {
type: 'tree',
data: [data],
//lineStyle: { color:'#C41188',       width:1.5, type:'solid', curveness:0.5 },
//lineStyle: { color:'#C41188',       width:1.5, type:'dotted', curveness:0.5    },
//lineStyle: { color:'#C41188',       width:1.5, type:'dashed', curveness:0.5    },
lineStyle: { color:'#C41188',          width:1.5, type:'broken', curveness:0.5    },
label: { normal: {  position: 'bottom', align: 'center' } },
leaves: { label: { normal: { position: 'right', verticalAlign: 'middle', align: 'left' }  }  }
};
var option = { series: serie };
var myChart = echarts.init(document.getElementById('1.3.5'));
if (option && typeof option === "object") {
myChart.setOption(option, true);
};
$(window).on('resize', function(){
if(myChart!= null && myChart != undefined){
myChart.resize();
}
});
};

I think the API has changed since then, I tried this myself and tree charts seem to work fine, but I think the option you're looking for is the curveness inside the lineStyle option: https://ecomfe.github.io/echarts-doc/public/en/option.html#series-tree.lineStyle.curveness
Here's a working codepen for version 4.1.0:
https://codepen.io/jivanvl/pen/ZPpypZ

@Jivanvl your option is valid and i had prior knowledge of it. But i dont want that design of tree chart. Check the link below for the design that I would like to get help in. I'm not sure if they removed that 'broken' functionality and thats why I am enquiring. Its present when you use version 2 but its different when you try to use version 4.
https://echarts.baidu.com/echarts2/doc/example/tree1.html#-en

Hi @BrianWahinya, I know what you want, the straight-line layout of tree in version4 will be released in 4.7.0 with this PR.


Would it be possible to add a "/timeout" endpoint to httpbin?

My use case is building a client that understands the unreliability of distributed systems, and attempts backoffs, retries and other stuff. Having an endpoint which intentionally never returns a response or similarly represents the unreliability here would be really useful. Let me know if I can provide more detail.

So /delay/:n can do that, but I think it only let's you delay for 10 seconds at most. It absolutely returns a response, but I'm not sure we want an endpoint that doesn't ever return a response.

This could potentially tie up too many resources and effect the performance of the site.
On Thu, Jun 16, 2016 at 6:23 AM -0500, "Ian Cordasco"<EMAIL_ADDRESS>wrote:
So /delay/:n can do that, but I think it only let's you delay for 10 seconds at most. It absolutely returns a response, but I'm not sure we want an endpoint that doesn't ever return a response.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

This could potentially tie up too many resources and effect the performance of the site.

Right. I suppose we could attempt to add an endpoint that just randomly closes the connection, but that's hard to do in plain flask. :/ Sorry @samphippen


[SPARK-9130][SQL] throw exception when check equality between external and internal row

instead of return false, throw exception when check equality between external and internal row is better.

cc @rxin

Merged build triggered.

Merged build started.

LGTM

Test build #37599 has started for   PR 7460 at commit 402daa8.

let's see if we have any places that we mistakenly compare

Test build #37599 timed out     for PR 7460 at commit 402daa8     after a configured wait of 175m.

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #37632 has started for   PR 7460 at commit 8a20911.

Test build #37632 has finished for   PR 7460 at commit 8a20911.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Thanks - merging this in.