Setting concurrency = 1 breaks helmfile template with --set

Using a concurrency of 1 breaks helmfile template because the arguments are set on helm on the first thread, the second thread then does a helm fetch --set, which results in --set is an invalid argument.
PR to fix this incoming...

@mumoshu would be great to get this merged and released ASAP as i've changed the interface for helm template slightly (no --concurrency), so minimising the impact of this breaking change.

Makes sense. In the future, it would be nice to have concurrency option for the helm-fetch part of lint and template commands though.
Thanks a lot for the fix anyway!

Yup I agree and I'm working on that, just wanted to prevent this issue in
the interim.
Do you think it should be configurable or we just do them all concurrently
regardless?  I can't really see anyone wanting to control it
On Tue, 4 Sep 2018, 12:56 pm KUOKA Yusuke<EMAIL_ADDRESS>wrote:

Makes sense. In the future, it would be nice to have concurrency option
for the helm-fetch part of lint and template commands though.
Thanks a lot for the fix anyway!
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/roboll/helmfile/issues/288#issuecomment-418339878,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABavie-LU42vB6FpzQLdxyCcPDHw5ONKks5uXmp9gaJpZM4WYwaX
.

I'd hope that it is configurable. Sequential download is always slow, and may harm u/x for serious users who is managing several dozens of releases with helmfile 😄Plus concurrently downloading dozens of charts doesn't seem to make sense either. Perhaps @sstarcher and @osterman would have more opinions, because they seem to manage many releases with helmfile?
But I completely agree with your YAGNI point. So, my stance is welcome any contribution about making  its concurrency configurable!

I have never used the lint or template commands of helmfile since they have been added.
On 26 releases
helmfile lint  17.25s user 6.87s system 198% cpu 12.165 total```

helmfile lint --concurrency=1  16.73s user 6.64s system 78% cpu 29.764 total

In my one off example looks like the command takes a little over 2x as long.

@sstarcher - I've just raised https://github.com/roboll/helmfile/pull/292 which parallelises those releases.


Inline: Split out InlineExhaustivePass from InlinePass

InlinePass is the generic inliner that other inliners will derive from. Ultimately should move the main inlining routine into InlinePass, parameterized by a function for inline policy and a function for what do if the function is not inlinable.

Rebased and pushed into master as e28bd39997801


Why OPTIONS method instead of GET for /accounts ? And so Authorization:Bearer ... is'nt send.

Hello, i try to connect my api with ember-simple-auth jj-abrams branch  (latest commit ...254d38d) and try to load user data after session authentification succeed, this is done by a OPTIONS method (I work with password grant).
The dummy exemple try it with a GET method ... so where come from that http verb ?
I had read that OPTIONS request can't have custom headers.
The hash.beforeSend seems to have correct argument "Bearer {$my_token}.
Could you explain or search with me what's wrong ?

It sounds like you might be running into a CORS problem, which is controlled by the browser and server.
The OPTIONS method is a "preflight" request the browser makes automatically when a cross domain request is initiated. The browser considers a request as cross domain when either the protocol (e.g http), domain (e.g localhost or example.com) or port (e.g. 80, 8080, etc) are different.
The browser is expecting to receive back from the server some Access-Control-* HTTP headers that tell it whether or not the initiating domain has the capabilities to execute the desired request.
If the server response allows the browser to make the request, then the browser will then make the request (GET, POST, PUT, etc.)
For example, this basic response example from the server would allow any domain to access the server.
    HTTP/1.1 200 OK
    Access-Control-Allow-Origin: *

Here's some more details on the subject:
https://developer.mozilla.org/en-US/docs/Web/HTTP/Access_control_CORS

@joelkallman great explanation 👍

Thanks you very much, with that, that's work now for me !


Specify all modules at main of the library

I think the library needs to support multiple versions of UUID at one time without requiring more than 2 times. I am using multiple versions at one time, and now there are more lines in my code. It is not a big deal for me. However, I thought if I can do like below(code), it is really helpful.
const uuid = require('uuid')

const v1Result = uuid.v1(); // to use v1
uuid.v5('main', v1Result); // use v5 right after v1

According to #154, the versions were separated into submodules and the structure of the library was changed. The only change I want is specifying all submodules at the main(index.js). From structure, I think there are no changes to use this library.

80+% of [opensource] projects using this library use v4 and nothing else.  Adding dependencies to the top-level module increases the footprint of this module for those users by 6-8X.  Unless/until packagers are able to effectively "tree shake" the unused code out of bundles (which, to my knowledge, they do not currently do), I would suggest we not make this change.
@ctavan may have an opinion on this, based on his recent ESM work.

@broofa I believe once we land an ESM variant of this library it will be safe to export all versions from the top-level, at least my experiments with webpack indicated that: https://github.com/kelektiv/node-uuid/tree/a66f7935eb060eb963b6dc21ab90f015cbb078dc/examples/browser-webpack/dist
We could then also include all versions in the CommonJS-build assuming this build will mainly target Node.js users but I think that is a separate discussion.
Until we have a viable alternative for the Web platform I agree with you that we should not further in crease the footprint of the CommonJS-module variant of this module.

Yeah, that's pretty much my understanding - that tree-shaking is feasible with ESM modules, but isn't practical for CommonJS codebases.

I see why there aren't all versions at the main entry. I am sorry to do this work without considering other uses because I am not good at front-end technologies such as webpack. I can't talk pretty enough about this subject but now understood why. Thanks!


Fix: Improve Sentry Integration tests

:scroll: Description
This PR improves the tests which make sure that every platform uses the correct integrations.
#skip-changelog
:bulb: Motivation and Context
https://github.com/getsentry/sentry-dart/issues/390
:green_heart: How did you test it?
I've written new tests.
:pencil: Checklist

[ ] I reviewed submitted code
[ ] I added tests to verify changes
[ ] I updated the docs if needed
[ ] All tests passing
[ ] No breaking changes

:crystal_ball: Next steps

Codecov Report

Merging #421 (e0d995a) into feature/support-desktop (a874f04) will not change coverage.
The diff coverage is n/a.

@@                   Coverage Diff                    @@
##           feature/support-desktop     #421   +/-   ##
========================================================
  Coverage                    90.02%   90.02%
========================================================
  Files                           54       54
  Lines                         1684     1684
========================================================
  Hits                          1516     1516
  Misses                         168      168

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 a874f04...e0d995a. Read the comment docs.

I would like to land #413 before this, so I can implement https://github.com/getsentry/sentry-dart/issues/390#issuecomment-810164155 in this PR.


Add support to Napalm vars for env var retrieval

Proposed Changes
I would see if we can add to the Napalm variables in settings.py to allow them to be retrieved as environment variables.

Justification
This would be useful for Golden Config/nautobot-nornir which offer environment variables as an option for Napalm secrets to retrieve config.

@susanhooks Are you thinking that a setting set in settings.py is exposed to plugins/the app as an environment variable?

@bryanculver I got some further clarification in slack, I misunderstood how it was working. Closing the issue.


Support zoom

Hi, love this app, but I am using it on a  planet computers gemini which has a portrait screen, and it could really use some kind of UI scaling as the contact list takes up nearly half the screen. Is this possible? Thanks

Done with 1.5.0

Awesome, thank you!
On Sat, 29 Feb 2020 at 19:50, Leonhard Künzler<EMAIL_ADDRESS>wrote:

Closed #13 https://github.com/92lleo/WhatsappWebToGo/issues/13.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/92lleo/WhatsappWebToGo/issues/13?email_source=notifications&email_token=AAHGCJTH3IVAZT5NIQ4T273RFFTJFA5CNFSM4JAB3W62YY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOW7QYY6A#event-3085012088,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHGCJWXPJXMFC3A4676ZOLRFFTJFANCNFSM4JAB3W6Q
.

@static-void did it work in your usecase?

It actually has not helped, I can now zoom in but I want to zoom out :S
On Mon, 2 Mar 2020 at 07:12, Leonhard Künzler<EMAIL_ADDRESS>wrote:

@static-void https://github.com/static-void did it work in your usecase?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/92lleo/WhatsappWebToGo/issues/13?email_source=notifications&email_token=AAHGCJXTKLMPUJFEFTUZOXDRFNL6PA5CNFSM4JAB3W62YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENOFQTQ#issuecomment-593254478,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHGCJUSET7XBVWHLMWJ3KDRFNL6PANCNFSM4JAB3W6Q
.

Thanks for the feedback. I'll look into this again and see what i can do

Could you please try the attachted apk? I changed the initial zoom setting to a hardcoded value. If this works for you, I need to build some kind of settings page as this does not really work on phone screens.
wawtg_29_test.zip

Will try it later today, thanks a lot for the support on this issue!
On Mon, 23 Mar 2020 at 18:15, Leonhard Künzler<EMAIL_ADDRESS>wrote:

Reopened #13 https://github.com/92lleo/WhatsappWebToGo/issues/13.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/92lleo/WhatsappWebToGo/issues/13#event-3156473526,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHGCJRRNEKEBWWSSN777QLRI6RNFANCNFSM4JAB3W6Q
.

The zoom is perfect now, but it always zooms in on the chat when you click
to write text, is it possible to optionally disable that? But if not it is
anyway a great improvement! Definitely the best of this the whatsapp web
apps!
On Wed, 25 Mar 2020, 05:49 Ed Robbins<EMAIL_ADDRESS>wrote:

Will try it later today, thanks a lot for the support on this issue!
On Mon, 23 Mar 2020 at 18:15, Leonhard Künzler<EMAIL_ADDRESS>wrote:

Reopened #13 https://github.com/92lleo/WhatsappWebToGo/issues/13.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/92lleo/WhatsappWebToGo/issues/13#event-3156473526,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAHGCJRRNEKEBWWSSN777QLRI6RNFANCNFSM4JAB3W6Q
.

Hi, I love this app.
I've noticed, on the login page you are constantly zoomed in and it's not possible to zoom out. Most browser have an option in case websites block zooming which allows you to zoom anyways. I don't know but maybe an adaption this force zoom option would work.

@theFireProjet I actually enabled all that settings. The only workaround is to set the initial zoom lower like detailed above. Those zoomsettings need some finetuning before releasing it, unfortunately I can't give an eta.
@static-void I'll look into it. Thanks for the feedback!


Use adapter Relation class instead of BaseRelation

Not sure if this is necessary for Athena adapter for now.


Add integration test field on Feature Review page

This pull request adds a field to the Feature Review page, which displays the status of the integration test build.

@FundingCircle/prod-ops @bliof @0mega  👀

@sgerrand there might be some confusion with what the name represents. My understanding was that the IntegrationTest will represent the tests that are run on Jenkins for our apps. So it's part of the core model of shipment tracker to represent other application's result of running the integration tests, as opposed to tests that we run against shipment tracker code.

Whoops, apologies. 😊

We discussed with @bliof and we agreed that actually there is no reason of a new event type.
We can re-use the BuildEvent, as this is actually a build and just have a new type of source, which states that this is a integration tests build.

@bliof @0mega That's ready for review again 👀

@mbdimitrova @bliof Could you take a final look at this pull request 👀

@bliof I added a BuildHelper for constructing builds.

@sgerrand I added factories for builds and removed the previous helper. 🙂


Remove branches through options object

Yeah, people will want to remove branches, specially the 'behavior' one. Let's add an options object to component's instantiation.

@germanattanasio I've come across this requirement as well and wouldn't mind implementing it.  My proposed solution would add a Options.exclude property which is an array of ids to exclude from rendering.  The valid ids would be the same ones the library is already checking for, specifically:

personality
needs
values
sr
sbh
blank

Using this solution, I could exclude the social behavior from rendering in the sunburst chart if I passed the value Options.exclude = ['sbh'].

I like it and it think it's not a breaking change if you just add exclude. I'm happy with a PR @soumak77.


Multiple Exercise custom report

Background
Creates efficiencies by supporting the creation of custom reports that span multiple exercises.
User Story
As user, I would like functionality so that benefit.
Users are: Candidate, Admin, Assessor, QT participant, Panellist, Digital team.
Benefit(s) to user (if not already clear from User story)
Feature(s) Description
Clearly describe the new functionality/change(s) to existing functionality.
Include a link to a Miro board/image if appropriate.

[ ] Be able to choose both live and archived exercises

[ ] Create a new multiple exercise custom report on its own under the exercises drop down menu

[ ] Create two separate dropdown boxes: one for exercises and one for fields.

[ ] Add new text at the top of the page: 'Select the exercises you would like to include from the drop down menu.
Choose the fields you wish to include in your custom report from the dropdown below and click generate to produce your report on screen. You can save your fields for future use. Download the report when you are satisfied with the content.'

Acceptance Criteria
Specify the requirements of the primary user(s) for the change, update, or new feature - use the format:
It's done when...
Definition of Done

[ ] User stories/acceptance criteria met
[ ] Internal reviews passed (feedback actioned)
[ ] User testing passed
[ ] Relevant technical documentation updated
[ ] User guidance updated
[ ] Deployed and merged without errors

User Testing Steps
Specify for users what is being tested (but not how to test it.)
Feature Champion

Q Is there a filter based on the exercise type as above i.e. legal etc?
A The custom report currently only works per exercise - not across multiple exercises - therefore if I understand your question then the answer is no. The fields that can be included in the report are from the applications, not from the exercise
Q Is the custom report exercise specific i.e. if it's a legal exercise, the fields will only relate to legal questions. If it's non-legal, the fields will relate to non-legal exercise questions?
A It should be exercise-specific, that would make sense, but I’m not sure if that’s actually the case. We’re in the same boat on that one - i.e. would need to compare two exercises (legal vs non-legal) and spot the difference

Hey team! Please add your planning poker estimate with Zenhub @drieJAC @HalcyonJAC @tomlovesgithub @warrensearle


Self Hosted Git Repo Support

Hey guys,
thank you for this great project!
It would be awesome to have support for self hosted git repositories. Then drone would be the perfect solution for a enterprise project I'm working on and I could get rid of Jenkins :)
Cheers
Sebastian

The latest version of Drone completely relies on 3rd party version control systems (ie GitHub, Bitbucket, etc) for user accounts, user login and repository permissions. This means that Drone will not support generic Git repositories.

I will add that all is not lost. We are planning on adding dynamic plugins (using JavaScript) that you may be able to use to hack something together. It will just take a bit of creativity to fully implement the required methods:
type Remote interface {
	// Authorize handles authentication with thrid party remote systems,
	// such as github or bitbucket, and returns user data.
	Authorize(w http.ResponseWriter, r *http.Request) (*model.Login, error)

	// GetRepos fetches all repositories that the specified
	// user has access to in the remote system.
	GetRepos(user *model.User) ([]*model.Repo, error)

	// GetScript fetches the build script (.drone.yml) from the remote
	// repository and returns in string format.
	GetScript(user *model.User, repo *model.Repo, hook *model.Hook) ([]byte, error)

	// Activate activates a repository by creating the post-commit hook and
	// adding the SSH deploy key, if applicable.
	Activate(user *model.User, repo *model.Repo, link string) error

	// ParseHook parses the post-commit hook from the Request body
	// and returns the required data in a standard format.
	ParseHook(r *http.Request) (*model.Hook, error)

Sad, sad day indeed.... :cry: :sob:

well there go my dream of moving away from bamboo

@ramonski FYI, I have written a specific extension to use drone with our self-hosted mercurial repositories:
https://github.com/cedk/drone/tree/trypod
http://hg.tryton.org/trypod
It should not be too difficult to adapt it for other custom usage.

well there go my dream of moving away from bamboo

I think someone could easily create a deamon that implements the minimum required REST endpoints (login, get repository, upload ssh key) to hook into Drone. This daemon could sit on your Git server, or it could probably even sit on your Drone server and use ssh to communicate with your Git server.
So your dream isn't dead :)
It will just require an extra module, which won't be included in the core Drone codebase.
If nobody wants to create this daemon, I will do it. I just have a pretty long backlog at the moment so it would take a few months. If someone wants to tackle this now I'm happy to offer my assistance.

@bradrydzewski @cedk - I've made a fork of the Drone project in which I'm starting a plugin for Stash. I'll definitely be taking up your offers for help. I'll see how far I get in the next few days before I call for help though. I'm using the BitBucket plugin and the Trypod plugins as a base reference for now.

Really cool that you are trying it! Thanks and good luck! :)

Have there been any developments since this issue was closed? I'm interested in using drone for our deployments where I work, but all our repos are self-hosted.

@ChiperSoft the upcoming 0.4 release of Drone will feature a plugin model for hooking up custom remote systems. One just needs to satisfy the following interface:
https://github.com/drone/drone/blob/0.4.0/pkg/remote/remote.go#L9
Self-hosted raw git repositories won't be officially supported, however, I'm confident they can be added through an external plugin once 0.4 is ready.

https://github.com/drone/drone/issues/10#issuecomment-108590120
Now it's 0.8 and I can't find how to make a external plugin to support my custom remote.

@spacemeowx2 ypu should implement the Remote interface described in https://github.com/drone/drone/blob/master/remote/remote.go (see other implementation in the same folder).

@mavimo So after I writing it how can I plugin it into drone without compile again?

Currently it's not possible without recompilation, but maybe there will follow some API comparable to secrets and registry.

Got it, thank you.


Question about reproducing UCF101 result

Hi, thanks for your awesome work! I recently run the public naive SFT and CIL 512 checkpoint following the B.3 Evaluation setting and get the following result, which may be different from paper reported result.

Method
FVD (2048-16f)
IS (10k)

pre fs=3
434.60
13.16

pre fs=8
288.01
13.31

cli fs=8
409.69
13.68

sft fs=8
373.77
13.19

I notice the paper said ``we sample 16 frames at 3 fps'',  so I change the inference shell script to FS=3 (NOTE that the FS is actually fps in dynamicrafter) but find the pretrained checkpoint have higher fvd than the paper said. So I also add the FS=8 (fps=24/3=8), but I found CIL at the same fps rate has larger fvd than pre.
So I would like to kindly request more detail about the evaluation detail in the paper, does you evaluate fvd on exact 2048 generated videos, or generate more video but only sample 2048 files randomly each time?
Second, I use RAFT to calculate motion score on the generated 2048 videos following the paper description in page 17, I get the following results:

Method
Motion Score

pre fs=3
91.5

pre fs=8
78.17

cli fs=8
73.78

sft fs=8
85.89

It seems that sft has higher motion score than the pretrained model, which is contrary to the Table1. Could you help me to  find problems ?

Hi @LZY-the-boys , thank you for your attention to our work and sorry for replying late.  The evaluation settings are clarified as follows:

For UCF101, we sample 16 frames at 3 fps.  In other words, 3 frames per second in the UCF101 video are sampled, resulting in a total of 16 frames serving as ground-truth for further evaluation. Here, the fps does not refer to the FPS in the DynamiCrafter setting, which remains at its default value in its official repository.
We generate 10K videos to evaluate the IS metric, and randomly sample 2048 of them for FVD evaluation.
Notably, the bash scripts in our repository applies the inference strategy, namely Analytic-Init, which makes them unsuitable for direct use in the ablation of training strategies. For example, in inference_512.sh the initial timestep M is set 940 (0.94T) while in inference_CIL_512.sh it's 1000 (T). Also, in both scripts, whether_analytic_init is set true. For fair validation on the effectiveness of TimeNoise, there's a need to set M to 1000 and whether_analytic_init=0. I geuss this is probably also the cause of the inconsistency between your results and that in the paper, especially the motion scores.
To validate the improvement on the motion score,  it's also recommended to run our scripts to generate some demos following readme.md. I believe the visual comparison results can demonstrate the elative degree of motion between SFT and TimeNoise.


Get rid of features/warnings.

Use libc from crates.io
Don't use as_slice.

Cool! Thanks!

Just released 0.2.5 with these changes.


merge remainder of 1.2.1 (ipy3 compatibility) to master

also fixes up the moment situation https://github.com/twosigma/beaker-notebook/issues/1388
and closes https://github.com/twosigma/beaker-notebook/issues/1328
davide please double check the finalStuff/doFinish stuff , i am not sure how to repro that.

It does not work at all:
Failed to load resource: the server responded with a status of 404 (Not Found)
beaker.js:409 ReferenceError: requirejs is not defined

It looks like this was the first merge to master that started a trend of failing specs
https://travis-ci.org/twosigma/beaker-notebook/builds/57523288
Before this point master was green.
It might be worth investigating why the specs went red. any thoughts @buob @scottdraves @ildipo ?

Thanks for looking into this, Sam.

Looking at their contents surely it is this one #1393, and in particular
the change in location of require.js that's causing the problem?
On Tue, Apr 28, 2015 at 7:13 PM, Sam Saccone<EMAIL_ADDRESS>wrote:

It looks like this was the first merge to master that started a trend of
failing specs
https://travis-ci.org/twosigma/beaker-notebook/builds/57523288
Before this point master was green.
It might be worth investigating why the specs went red. any thoughts @buob
https://github.com/buob @scottdraves https://github.com/scottdraves
@ildipo https://github.com/ildipo ?
—
Reply to this email directly or view it on GitHub
https://github.com/twosigma/beaker-notebook/pull/1393#issuecomment-97258699
.

--
BeakerNotebook.com
ScottDraves.com http://www.ScottDraves.com
@Scott_Draves http://twitter.com/scott_draves

this PR contains the commit that caused master to start being red
build:
https://travis-ci.org/twosigma/beaker-notebook/builds/57358180
commit sha:
https://github.com/twosigma/beaker-notebook/commit/3da2e14d6e9634a1d34f5e20f8d06c1f7f862254
sorry I copied the incorrect build number from travis
it very well might be related to requirejs it looks as if nothing is loading. I have not dug in to the cause of the error, just trying to give us a decent starting place


Creating a new org for upstream contributions

creating a new GH org Instead of using a working directory for upstream contribution

[ ] Propose 3 alternatives for an contrib / upstream org and have maintainers vote

We could move there the following:

[ ] https://github.com/flatcar/ignition
[ ] https://github.com/flatcar/gentoo

created - https://github.com/flatcar-hub


Competition Submission Issue

Hi,
The result submission of the competition website seems out of service recently. Is there an alternative way to submit my new result and get updates of the leaderboard?
Thanks.
Haonan Wang

We have installed codalab service on our own server and will release the service soon (by next week). We encourage contestants to submit the prediction results to our own server now.
Thanks very much for your patient！We will email all contestants with detailed instructions once we are done!

Thx!


ARROW-15200: [C++][Gandiva] Enable RTTI when building LLVM dependency using vcpkg [WIP]

When generating Gandiva's jar file it is possible that an error of UnsatisfiedLinkError may occur based on an LLVM's symbol being used, more specific the llvm::ObjectCache.
To fix that problem, it is necessary to compile the LLVM dependency with RTTI enabled when generating the jars in the nightly build.

@github-actions crossbow submit java-jars

@github-actions crossbow submit java-jars

The nightly build is being broken when it is executing the set up of the last job, I need to check if it is related to the changes I made.

@kou @kszucs the java-jars nightly build is broken when is executing the checkout in the last step I will check and fix it.

@kou @kszucs the java-jars nightly build is broken when is executing the checkout in the last step I will check and fix it.

Apparently the problem is in the checkout actions https://github.com/actions/checkout/issues/672#issuecomment-1010606946

@github-actions crossbow submit java-jars


NVOffset Validator Check

Before the latest Studio Patch 3.10.1 the validator was picking up any table with a missing offset once the first one is set.
When updating tables that have an offset originally it will flag there is one missing.

Is a follow up of https://github.com/syd711/vpin-studio/issues/599

gorgatron — Today at 4:03 PM
FWIW I think the rules around NVOffset go like this:

an NVOffset of 0 (or not set) when other tables for the same rom have something set is wrong -- it won't switch the nv files
two vpx files that have the same NVOffset are going to share the same high scores etc.
almost certainly you don't want two tables with different gameId (VPS) to share the same high scores (e.g. Dr. Who and Dr. Strange)
-but you might have different authors of the same table (e.g. VPW and Ext2k) that you do want to have the same NVOffset (this is the specific issue I filed -- these files are both trying to simulate the same table per VPS)

The first rule we definitely want to be warned about.  The second is just how it works and really the last two are what policy you want -- those two are my opinion based on how I have NVOffset working (now that I understand it)


New release with updated Jackson

To use Spring 5 in our project, we need jackson to be at 2.9+. The last release of Fahrschein still has a lower jackson version. Master is currently at 0.15.0 as a snapshot version and jackson has been upgraded. Is there anything that prevents creating a new release with the upgraded version? If so, what things need to be done?

Do you notice any problems running fahrschein with jackson 2.9.x? It should be compatible with all versions starting from 2.6.

Sorry, I was not clear. The problem is that there is no fixed version with a jackson version of 2.9.x yet. It has been upgraded on master, but this is a snapshot version, so we cannot just use it for production. It would be nice to have a fix version 0.15.0. So I was wondering what's missing for this.


use same build targets as OpenTelemetry .NET SDK

Expand the build targets to include all those also supported by the OpenTelemetry .NET SDK.
Fix #23.

@nblumhardt Any comments on the supported targets? If not, I'll merge the change.


Add tile loading events to image tile sources

This adds tileloadstart, tileloadend, and tileloaderror type events to tile sources.

Here's the example demonstrating a simple progress bar monitoring tile loading for a single source:
http://tschaub.net/ol-tile-load-events/examples/tile-load-events.html

This looks like a solid piece of much welcome work @tschaub
LGTM

Thanks for the review @bartvde.  I reworded that commit message.  Will merge when Travis goes green.

Now that we register an additional listener for tile events, it will be nice to avoid creating tiles that are not needed.  These listeners aren't unregistered until the tile cache for a source is full.  See #3246 for a change that avoids unnecessary tile creation.

hey @tschaub forgot to ask, but do you have plans to add events for image sources as well or not? If not, I can probably take a crack at it.

@bartvde yes - I'd like to do the same.  But I'm not working on it currently, so it would be great if you wanted to take it on.  If I do find some time for it, I'll open a ticket first to indicate that I'm working on it.

Nice work @tschaub.


0.17.0 Kafka Supervisor becomes "unhealthy" due to missing inputFormat when Overlord is restarted

Affected Version
0.17.0
The Druid version where the problem was encountered.
0.17.0
Description
When provisioning a Kafka Supervisor in druid 0.17.0, a payload using the inputFormat property in the ioConfig will be accepted and the supervisor will work properly until you restart the Overlord service. When the Overlord service is restarted, it will fail to create tasks due to there no longer being any "inputFormat" data in the ioConfig.
This is due to the way the json marshalling is handled in the configuration persistence. When you issue a command to create a supervisor with an ioConfig like this:
"ioConfig": {
"topic": "some-cool-topic",
"inputFormat": {
"type": "json"
},
"replicas": 1,
"taskCount": 2,
"taskDuration": "PT1800S",
"consumerProperties": {
"bootstrap.servers": "kafka:9092"
},
"pollTimeout": 100,
"startDelay": "PT5S",
"period": "PT30S",
"useEarliestOffset": true,
"completionTimeout": "PT3600S",
"lateMessageRejectionPeriod": null,
"earlyMessageRejectionPeriod": "PT3600S",
"stream": "npav-ts-metrics",
"useEarliestSequenceNumber": true
},
It will be accepted, but anytime you try to retrieve this supervisor, you will get back an ioConfig that looks like this:
"ioConfig": {
"topic": "some-cool-topic",
"replicas": 1,
"taskCount": 2,
"taskDuration": "PT1800S",
"consumerProperties": {
"bootstrap.servers": "kafka:9092"
},
"pollTimeout": 100,
"startDelay": "PT5S",
"period": "PT30S",
"useEarliestOffset": true,
"completionTimeout": "PT3600S",
"lateMessageRejectionPeriod": null,
"earlyMessageRejectionPeriod": "PT3600S",
"lateMessageRejectionStartDateTime": null,
"stream": "npav-ts-metrics",
"useEarliestSequenceNumber": true,
"givenInputFormat": {
"type": "json",
"flattenSpec": {
"useFieldDiscovery": true,
"fields": []
},
"featureSpec": {}
}
},
Notice that the inputFormat is gone and givenInputFormat is there instead. This supervisor will work properly until you restart the overlord, at which point the supervisor ioConfig will contain the following:
"ioConfig": {
"topic": "some-cool-topic",
"replicas": 1,
"taskCount": 2,
"taskDuration": "PT1800S",
"consumerProperties": {
"bootstrap.servers": "kafka:9092"
},
"pollTimeout": 100,
"startDelay": "PT5S",
"period": "PT30S",
"useEarliestOffset": true,
"completionTimeout": "PT3600S",
"lateMessageRejectionPeriod": null,
"earlyMessageRejectionPeriod": "PT3600S",
"lateMessageRejectionStartDateTime": null,
"stream": "npav-ts-metrics",
"useEarliestSequenceNumber": true,
"givenInputFormat": null
},
Notice that there is no longer a valid inputFormat, nor is there a valid givenInputFormat. When in this state, the supervisor is unable to create new ingestion tasks.

I will be fixing this

I cannot reproduce this issue. I believe it was fixed in this PR: https://github.com/apache/druid/pull/9136
by @clintropolis
I think we can close this issue :)

I will try and reproduce, but the version of the code I generated this bug from includes the changeset mentioned in  https://github.com/apache/druid/pull/9136 so I think it may still be an issue

@acdn-mpreston
Hmm.. can you try to reproduce this with 0.17.0 and 0.18.0? I cannot reproduce this bug in either version. Also. the changeset mentioned in #9136 rename the serialize/deserialize name of givenInputFormat to inputFormat, so you should not see "givenInputFormat" in the spec when you try to retrieve this supervisor.

I can confirm that this is resolved in the 0.17.1 branch. Closing this bug as resolved. Thank you for looking into it!


do you try to use Resnet in wgan-gp?

Have you compared the difference between the network structure of DCGAN and the structure of Resnet in WGAN-GP?Is the effect of Resnet will be better than the structure of DCGAN.

@tuoniaoren I have tried Resnet but it is too slow, so I didn't finish that experiment.

ok,i add inception v3 in the net of D,the result  is better.i will try add more this block


Group instance ID support (group.instance.id) (static membership)

Hello!
It is possible to add group instance ID support for static membership. This allows in some cases reduce consumer rebalances.
Details: KIP-345

The underlying protocol library already supports it,
however we do not have the plan to support it in brod yet, not at the moment.
ref: https://github.com/kafka4beam/kafka_protocol/blob/7147d6e16652b9b1c39758250bdf93ee934ec706/src/kpro_schema.erl#L499-L517

@zmstone thanks for reply!
Correctly I understand that we should pass this to coordinators state and after to join_group and sync_group requests body?
https://github.com/kafka4beam/brod/blob/master/src/brod_group_coordinator.erl#L616
https://github.com/kafka4beam/brod/blob/master/src/brod_group_coordinator.erl#L652

Hi @anoskov
I'm afraid it's more than just adding the fields.
I had a quick look at KIP before, and just quickly revisited it,
It seems that "Client Behavior Changes" implies brod_group_coordinator should change in its rebalance behaviours.


Uglify seems not to do anything

Hi there.
First, thanks for this starter kit, it was a great help.
From a fresh newly created project using your cli command, I updated the index.ts to add just a comment :
export default class Main {
  constructor() {
    // Hello from the comments
    console.log("Typescript Webpack starter launched");

    if (/test/.test("test")) {
      console.log("hello there");
    }
  }
}

let start = new Main();

After a npm run build, If you inspect the app.bundle.js generated, you'll see that :

comments are not stripped
code is not at all uglified
there is a huge amount of source mappings

A tiny part of the generated file :

sourcesContent:['export default class Main {\n  constructor() {\n    // Hello from the comments\n    console.log("Typescript Webpack starter launched");\n\n    if (/test/.test("test")) {\n      console.log("hello there");\n    }\n  }\n}\n\nlet start = new Main();\n']}

Hello @jscti ! Thank you for reporting this issue and sorry for this delayed response. Indeed there was an issue in the way uglifyJs and istanbul was configured for the coverage.
Those issues should have been fixed with the latest version of the CLI => v0.3.0 https://www.npmjs.com/package/create-ts-lib
I made a couple of tests in order to ensure most of the comments are stripped as well as the code to be well uglified.
Here are some bundle size comparisons of the project sample coming along with the CLI:
npm run build:dev # Output a developer bundle

npm run build # Output a production bundle, minified with tree shaken

Also, I personnaly use this CLI as a base scaffolder for this library https://bundlephobia.com/result?p=morphism. The improvements in the bundle size from the last 3 versions come from the tweak of this CLI. This is something I will keep on enhancing.
Thanks and let me know if you find other improvements! 🙏🏽

Closing this issue with #22 merged

Thanks for that ;)


[jax_export] Add the next part of multi-platform lowering support.

We change the lowering rule selection code to work when MultiContext.platform is a tuple of strings, and emit conditional code to select the lowering based on the platform index argument.
These changes will not affect the normal JAX lowering paths (when MultiContext.platform is a string).
These changes should work for most primitives, with the exception of the few ones that actually access ModuleContext.platform inside the lowering rules (most primitives just register different rules for different platforms, which is taken into account by these changes).
Previous PR in this series: #17316.

@hawkinsp any thoughts here?

I have no objection, provided you also do the cleanup to prevent rules from referring to .platform in their lowering contexts.

That is the plan, in subsequent PRs


Update npm package to include latest bin versions in its optional dependencies

Which version of the AzCopy was used?
Note: The version is visible when running AzCopy without any argument
Latest npm package version 3.0.0
Which platform are you using? (ex: Windows, Mac, Linux)
Linux
What command did you run?
Note: Please remove the SAS to avoid exposing your credentials. If you cannot remember the exact command, please retrieve it from the beginning of the log file.
Install the npm package
What problem was encountered?
The binary included in the npm package's optional dependencies is using an outdated version 10.19.0 which has several security vulnerabilities.
How can we reproduce the problem in the simplest way?
Install the package in any OS, they all use the same version.
Have you found a mitigation/solution?
Manually remove from node_modules the outdated version, download the latest version for my OS (https://aka.ms/downloadazcopy-v10-linux) and manually place it in the same location after package is installed.
Doing this manually is not a big issue but it has to be done constantly for a service that installs npm dependencies.
It would be great if this was updated to use the latest, more secure versions of the binaries.

@MRayermannMSFT @JasonYeMSFT could you please take a look at this request

Hi @JorgeLievanoLeanIX , thanks for opening an issue!
Yes, we've definitely done a poor job of keeping the npm distributions of the executables up to date. We had everything ready to go for publishing 10.22.0 to npm, but unfortunately during testing with azcopy-node, a regression was discovered that makes 10.22.0 incompatible with azcopy-node.
And it is exactly for that reason why we don't have azcopy-node simply download the latest AzCopy. The way AzCopy is used via azcopy-node is a wee bit different than how you normally use AzCopy via a CLI, so there's extra testing and verification we do to make sure the latest version is compatible.
However, again, you are correct that it is important that the executables on npm are generall kept up to date due to security fixes that new releases contain. We're already in the process of re-evaluating how we prioritize publishing new versions to npm, and hopefully you can see a change in cadence going forward.
For today though, if you are concerned about deploying the older versions of AzCopy, please feel free to take advantage of the exe option on AzCopyClient:

This option allows you to specify a different AzCopy executable to be used, instead of the ones in the executable npm packages. This should allow you to download and manually make use of 10.22.0 yourself. Though, as I said earlier, there is an issue with 10.22.0 and AzCopy node. Specifically, if you make use of OAuth authentication, 10.22.0 has a regression in how it receives tokes shared in the way that azcopy-node shares them. But if you don't make use of that, then you should be fine to use 10.22.0.
Let us know if you have any further questions.

Thanks for your answer @MRayermannMSFT!
We do not use OAuth with azcopy so using version 10.22.0 should work just fine for us.
Hopefully it's not too long before the npm package starts including these updated binaries as well 🙏.

Closing this as there is no action item open for AzCopy here. Feel free to post your comments in this regard here.


aria-labelledby is not inserted for <title> elements in icon files.

In docs/meta-data.md it states that:

... even without the meta file being specified, svg-sprite will try to find these two elements [<title> and <description>] in your files and set the aria-labelledby attribute accordingly.

If my icon file has:
<svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 18 28">
<title>testicon</title>
<path d="M16.797 11.5c0 0.125-0.063 0.266-0.156 0.359z"></path>
</svg>

and I have this config:
{
    "mode": {
        "symbol": {
            "dest": ""
        }
    }
}

Then my sprite comes out as:
...
<symbol viewBox="0 0 18 28" id="testicon"><title>testicon</title><path d="M16.797 11.5a.54.54 0 0 1-.156.359z"/></symbol>
...

From the docs above, I would expect the sprite to be:
...
<symbol viewBox="0 0 18 28" id="testicon" aria-labelledby="atesticon-title"><title id="atesticon-title">testicon</title><path d="M16.797 11.5a.54.54 0 0 1-.156.359z"/></symbol>
...

Is there any other way to get aria-labelledby and aria-hidden added than using a meta file.

@tech4him1 First of all sorry for the late reply!
As shown in the example and specified by WAI-ARIA, aria-labelledby is referencing the ID(s) of an element (or multiple elements). You're example doesn't have an id attribute on the <title> element, so referencing it cannot work.
Besides: No, svg-sprite doesn't support another way of adding ARIA labels, I'm sorry.


[kwok] Respect HostNetwork of Pod

What would you like to be added?
If spec.hostNetwork is true then status.podIP is set to status.hostIP.
Why is this needed?
NONE

/good-first-issue

@wzshiming can you please tell me about this issue more

@swastik959 Update details of the issue

@wzshiming do i need to change https://github.com/kubernetes-sigs/kwok/blob/9ae9f320fdbf247ae1a1ec8fd7e9579cb68310af/pkg/kwok/controllers/pod_controller.go#L422

I expect the following changes
https://github.com/kubernetes-sigs/kwok/blob/8e2871ed268e8efbd3934eeb5432a9afe6844ac2/pkg/kwok/controllers/pod_controller.go#L466-L477
https://github.com/kubernetes-sigs/kwok/blob/8e2871ed268e8efbd3934eeb5432a9afe6844ac2/pkg/kwok/controllers/pod_controller.go#L420-L430
https://github.com/kubernetes-sigs/kwok/blob/5491b502b4ae22a4b2bd51d7b4b3a2ee8f0c864d/stages/pod-fast.yaml#L60
https://github.com/kubernetes-sigs/kwok/blob/5491b502b4ae22a4b2bd51d7b4b3a2ee8f0c864d/stages/pod-general.yaml#L97

Very interesting. As I posted in the slack channel (#sig-scheduling), I made some modification to previous release v0.0.1 of kwok for testing the CNI operator that I'm working on.
<EMAIL_ADDRESS>There, I did modify the pod_controller configurePod function.
And, in addition to that, I also made a mapping between hostName and hostIP and configure that.
The reason is that Pods are scheduled by (DaemonSet) controller and hostIP is set later (not manually set).

https://github.com/kubernetes-sigs/kwok/blob/122f36ce87128371d2b9bbcff43837d764b35e37/pkg/kwok/controllers/pod_controller.go#L126
It looks like we just need to change it to func(nodeName string) string and get the IP corresponding to the nodeName from the node_controller.

/assign

Do we need to just change this

https://github.com/kubernetes-sigs/kwok/blob/122f36ce87128371d2b9bbcff43837d764b35e37/pkg/kwok/controllers/pod_controller.go#L126
It looks like we just need to change it to func(nodeName string) string and get the IP corresponding to the nodeName from the node_controller.

or any further changes required.

I think the point is the current node controller does not keep information of IP corresponding to the nodeName.
In my understanding of the code, we need to add that mapping in the node controller watch and pass that mapping to the pod controller. We also need to be careful of map read/write concurrency because it might be multiple threads access that map (from node controller and pod controller).
( Another way is connecting to API server to get node information when configuring pod but I may not recommend this way because it will cause one extra communication to API server which does not exist when there is a kubelet process. )
I have already implemented some changes to solve this issue modified from v0.0.1. Although there are multiple changes in latest version but the significant points to solve this issue are the same as much as I see. I can work on this with some additional effort on rebasing.

@sunya-ch
Looks good, with the addition that we should use a structure and perhaps support CIDR #78 or Dual Stack in the future.


Problem with promise needle version when export it to other source file

Hello!
I have Node 8.9 and needle version 2.0.1 :
"dependencies": { ...  "needle": "^2.0.1" ... }
I need to work with Promise version of needle API, but seems it doesn't work.
I add needle as the module and trying to launch as function (I saw the docs), but have troubles :
const needle = require('needle');

// ...
module.exports = {
  getStats(url) {
      needle('get', url).then(res) => {
        console.log(res.body);
      }
  },
//...
}

and I got this when I call getStats to other source file (which has also imported needle) :
needle('get', url).then(res) => {
      ^
SyntaxError: Unexpected token (

Maybe I did something wrong, I'm pretty newbie in ES6 standart and Node as well.  Maybe I failed when make import or something like that.
Thanks for attention!
Cheers, Alexander.

Are you sure you have native support for ES6 syntax? Try with the regular syntax and see if it helps:
module.exports = {
  getStats: function(url) {
      needle('get', url).then(function(res) {
        console.log(res.body);
      })
  }
}

Your code works fine, thank you!
Maybe it was my mistake or some Node`s fail.


[MAINTAINERS] New maintainer candidate: Stefan Pingel

I propose to add Stefan as a Besu project maintainer.
Over the past few months, his contributions to the Private Transactions feature have added a lot of value to the project. You can see his past contributions cliking here. I have worked close to him and can attest that he is committed to the project and will help to keep this project in a high-quality standard.
Voting ends two weeks from today, midnight UTC on 2020-03-02 countdown
For more information on this process see the Becoming a Maintainer section in the MAINTAINERS.md file.

@pinges Please let us know if you accept this nomination!


Load and manipulate a script

The proposed solution on issue #63 was not exactly what I was looking for. I wish I could load a script, insert and modify the JavaScript values from Java, also, call functions in a simple way. I really liked the JavaScriptCore API for iOS, so I tried to do something like trying not to break the API that you already have.

Thanks for the submission, but I don’t think this is a good fit for our API. I believe you might be able to accomplish something quite similar on top of this API.


StaticLLMPipeline: Decide when to enable NPUW_DQ_FULL property

Mirror of https://github.com/openvinotoolkit/openvino.genai/pull/1258

This one can be merged, I believe - functionally it depends on https://github.com/openvinotoolkit/openvino.genai/pull/1272 to be in the release branch, but as the config is text-based there's no ABI-level dependency on that PR - it is generally a look-ahead feature.


feat: impl some web3 interface

Which docs this PR relation:

mining
get_block_transaction_count_by_hash
get_transaction_by_block_hash_and_index
get_transaction_by_block_number_and_index

Which toolchain this PR adaption:
No Breaking Change
Special notes for your reviewer:

Codecov Report

Merging #103 (86082e6) into main (d353891) will decrease coverage by 0.32%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #103      +/-   ##
==========================================
- Coverage   39.40%   39.07%   -0.33%
==========================================
  Files          75       75
  Lines        5020     5062      +42
==========================================
  Hits         1978     1978
- Misses       3042     3084      +42

Impacted Files
Coverage Δ

core/api/src/jsonrpc/impl.rs
0.00% <0.00%> (ø)

core/api/src/jsonrpc/mod.rs
0.00% <ø> (ø)


feat(NumberTheory/Fermat): fermat_eq_fermat_sub_one_pow_two_add_one

could also be called fermat_add_one I suppose

IMHO fermat_succ is better

In general I agree, but there are other formulas describing fermat n + 1 in terms of fermat n

In general I agree, but there are other formulas describing fermat n + 1 in terms of fermat n

Then you have to fermat_succ_eq_blah_blah, or fermat_succ_eq' (primed naming scheme is strongly discouraged if you have several formulas for fermat (n + 1) I think).

Please hold for #17618

Please hold for #17618

Rebased, should be good to go now.

Thanks!
bors merge


JAVA使用socket连接一个无效IP时返回success

(NOTE: This is used to report product bugs:
To report a security vulnerability, please visit https://istio.io/about/security-vulnerabilities
To ask questions about how to use Istio, please visit https://discuss.istio.io)
Bug description
在K8S1.21版本中安装了istio1.8.2
启动一个pod运行以下代码时，总是显示成功。
经过测试，在未安装istio的K8S集群中会显示连接失败，我在想是istio接管了这个连接返回了成功吗？
以下示例代码中使用的IP是一个任意无效IP测试
import java.io.IOException;
import java.net.InetSocketAddress;
import java.net.Socket;

public class HelloJava{
 public static void main(String[] args){
  System.out.println("Hello Java");
                String host = "<IP_ADDRESS>";
                int port = 6789;
                System.out.println("check.connectable.host:"+ host +", port:" + port);
                Socket socket = new Socket();
                try {
                   socket.connect(new InetSocketAddress(host, port));
                } catch (Exception e) {
                   System.out.println("check.connectable.error:" + e);
                   return;
              } finally {
                   try {
                socket.close();
            } catch (IOException e) {
                e.printStackTrace();
            }
        }
        System.out.println("check.connectable.host: "+host+", port: "+port+", success");
 }
}

运行结果：
check.connectable.host: <IP_ADDRESS>, port: 6789, success
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
[ ] Upgrade
Expected behavior
Steps to reproduce the bug
Version (include the output of istioctl version --remote and kubectl version --short and helm version --short if you used Helm)
How was Istio installed?
Environment where the bug was observed (cloud vendor, OS, etc)
Additionally, please consider running istioctl bug-report and attach the generated cluster-state tarball to this issue.
Refer cluster state archive for more details.

Hi @guilinonline . Sorry you are having issues. Unfortunately, I am not sure we can help as is, as 1.8 is EOL (https://istio.io/latest/news/support/announcing-1.8-eol-final). For future issues it would be ideal if the report was in English to allow the most number of contributors to understand it. Thanks!

Drive by: the tcp connection will be accepted by istio-proxy.
Don't rely on the socket connect() behavior.

When socket connects to an invalid IP through connect, it always returns success
Software version
kubernetes: 1.21.4
istio: 1.11.0
docker: 19.3.9
linux: CentOS Linux 7 (Core)   3.10.0-1160.11.1.el7.x86_64
create project
kubectl create ns test
kubectl label namespace test istio-injection=enabled
kubectl create deployment web --image=gozap/oraclejdk8
kubectl exec -it web-xxxxxxxx -n test -- sh

After entering the pod, create a java file test
vim HelloJava.java

import java.io.IOException;
import java.net.InetSocketAddress;
import java.net.Socket;

public class HelloJava{
	public static void main(String[] args){
		System.out.println("Hello Java");
                String host = "<IP_ADDRESS>";  //This is an invalid IP for testing purposes only
                int port = 6789;
                System.out.println("check.connectable.host:"+ host +", port:" + port);
                Socket socket = new Socket();
                try {
                   socket.connect(new InetSocketAddress(host, port));
                } catch (Exception e) {
                   System.out.println("check.connectable.error:" + e);
                   return;
              } finally {
                   try {
                socket.close();
            } catch (IOException e) {
                e.printStackTrace();
            }
        }
        System.out.println("check.connectable.host: "+host+", port: "+port+", success");
	}
}

javac HelloJava.java
java HelloJava

Execution result returned successfully
However, it should have returned failure. Why did it return success? Because I used an invalid IP to test.

@lambdai
Drive by: the tcp connection will be accepted by istio-proxy.
Don't rely on the socket connect() behavior.

If this is a normal situation, how can I judge whether a socket can connect to a destination IP
Thank you very much for answering this question

Drive by: the tcp connection will be accepted by istio-proxy.
Don't rely on the socket connect() behavior.

If this is a normal situation, how can I judge whether a socket can connect to a destination IP
Thank you very much for answering this question

This is not possible, Envoy will always accept the connection. If you need
to you would need to disable iptables capture for that ip
On Tue, Aug 17, 2021, 7:50 PM guilinonline @.***> wrote:

When socket connects to an invalid IP through connect, it always returns
success Software version
kubernetes: 1.21.4
istio: 1.11.0
docker: 19.3.9
linux: CentOS Linux 7 (Core) 3.10.0-1160.11.1.el7.x86_64
create project
kubectl create ns test
kubectl label namespace test istio-injection=enabled
kubectl create deployment web --image=gozap/oraclejdk8
kubectl exec -it web-xxxxxxxx -n test -- sh
After entering the pod, create a java file test
vim HelloJava.java
import java.io.IOException;
import java.net.InetSocketAddress;
import java.net.Socket;
public class HelloJava{
public static void main(String[] args){
System.out.println("Hello Java");
String host = "<IP_ADDRESS>";  //This is an invalid IP for testing purposes only
int port = 6789;
System.out.println("check.connectable.host:"+ host +", port:" + port);
Socket socket = new Socket();
try {
socket.connect(new InetSocketAddress(host, port));
} catch (Exception e) {
System.out.println("check.connectable.error:" + e);
return;
} finally {
try {
socket.close();
} catch (IOException e) {
e.printStackTrace();
}
}
System.out.println("check.connectable.host: "+host+", port: "+port+", success");
}
}
javac HelloJava.java
java HelloJava
Execution result returned successfully
However, it should have returned failure. Why did it return success?
Because I used an invalid IP to test.
@lambdai https://github.com/lambdai
Drive by: the tcp connection will be accepted by istio-proxy.
Don't rely on the socket connect() behavior.
If this is a normal situation, how can I judge whether a socket can
connect to a destination IP
Thank you very much for answering this question
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/istio/istio/issues/34739#issuecomment-900771775, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAEYGXJUDG2MNYKV23KIALDT5MNYBANCNFSM5CJUVGEQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email
.

This is not possible, Envoy will always accept the connection. If you need to you would need to disable iptables capture for that ip
…
On Tue, Aug 17, 2021, 7:50 PM guilinonline @.***> wrote: When socket connects to an invalid IP through connect, it always returns success Software version kubernetes: 1.21.4 istio: 1.11.0 docker: 19.3.9 linux: CentOS Linux 7 (Core) 3.10.0-1160.11.1.el7.x86_64 create project kubectl create ns test kubectl label namespace test istio-injection=enabled kubectl create deployment web --image=gozap/oraclejdk8 kubectl exec -it web-xxxxxxxx -n test -- sh After entering the pod, create a java file test vim HelloJava.java import java.io.IOException; import java.net.InetSocketAddress; import java.net.Socket; public class HelloJava{ public static void main(String[] args){ System.out.println("Hello Java"); String host = "<IP_ADDRESS>"; //This is an invalid IP for testing purposes only int port = 6789; System.out.println("check.connectable.host:"+ host +", port:" + port); Socket socket = new Socket(); try { socket.connect(new InetSocketAddress(host, port)); } catch (Exception e) { System.out.println("check.connectable.error:" + e); return; } finally { try { socket.close(); } catch (IOException e) { e.printStackTrace(); } } System.out.println("check.connectable.host: "+host+", port: "+port+", success"); } } javac HelloJava.java java HelloJava Execution result returned successfully However, it should have returned failure. Why did it return success? Because I used an invalid IP to test. @lambdai https://github.com/lambdai Drive by: the tcp connection will be accepted by istio-proxy. Don't rely on the socket connect() behavior. If this is a normal situation, how can I judge whether a socket can connect to a destination IP Thank you very much for answering this question — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub <#34739 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAEYGXJUDG2MNYKV23KIALDT5MNYBANCNFSM5CJUVGEQ . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&utm_campaign=notification-email .

thank you
This is not a best practice. If I manually modify the IPtable rule, istio will modify it again. Can I configure istio?

istio iptables has some per pod parameters traffic.sidecar.istio.io/excludeOutboundIPRanges
see full list https://istio.io/latest/docs/reference/config/annotations/

Do not change police reports have been filed

istio iptables has some per pod parameters traffic.sidecar.istio.io/excludeOutboundIPRanges
see full list https://istio.io/latest/docs/reference/config/annotations/

Thank you very much. I have solved my problem by using the method you provided
apiVersion: apps/v1
kind: Deployment
metadata:
  name: web3
spec:
  selector:
    matchLabels:
      app: web3
  template:
    metadata:
      annotations:
        traffic.sidecar.istio.io/excludeOutboundIPRanges: <IP_ADDRESS>/32


Unexpected behavior on 'Align' rule with statements

Bug Report

TSLint version: 5.7.0
TypeScript version: Version 2.5.2
Running TSLint via:  VSCode

TypeScript code being linted
let foo = 2
let bar = 3
; [foo, bar] = [bar, foo]

with tslint.json configuration:
"align": [true, "parameters", "statements" ],

Actual behavior
tip: [tslint] statements are not aligned (align)
Expected behavior
no warning

seems caused by leading semicolon, it requires a valid form like
let foo = 2
let bar = 3;
[foo, bar] = [bar, foo]

I would expect to get this error in this case.
If you prefix the third line with "; " then the statement in the third line indeed does not align with the statement on the line before, as the third line statement starts two characters more indented.

Since ;s are sometimes inserted by ASI compensation techniques (sigh, ASI), it seems reasonable that the align rule could take in an option to allow this.

💀 It's time! 💀
TSLint is being deprecated and no longer accepting pull requests for major new changes or features. See #4534. 😱
If you'd like to see this change implemented, you have two choices:

Recommended: Check if this is available in ESLint + typescript-eslint ✅
Not Recommended: Fork TSLint locally 🤷♂

👋 It was a pleasure open sourcing with you!
If you believe this message was posted here in error, please comment so we can re-open the issue!

🤖 Beep boop! 👉  TSLint is deprecated 👈 (#4534)  and you should switch to typescript-eslint! 🤖
🔒 This issue is being locked to prevent further unnecessary discussions. Thank you! 👋


Getting error when we are adding reposestry

Could not find a file named "jitsi_meet_plugin/jitsi_meet_web/pubspec.yaml" in git://github.com/lennhv/jitsi_meet_plugin.git 1f2b36bac16ec4db37fb213d894c4a8771f82249.

This way a i am adding in pubspace.yml
jitsi_meet_web:
git:
url: git://github.com/lennhv/jitsi_meet_plugin.git
path: jitsi_meet_plugin/jitsi_meet_web

Hi @chilakam-dinesh-reddy  i added the instructions for use the code in a project. Please review it and let me know if work for you.

yes @lennhv  it totally working fine   tq u so much


Publish cmd_vel only if a gamepad is connected

Create a timeout or some other mechanism that would detect is a gamepad is connected or not. If not, then turn off publishing the zero cmd_vel messages as it conflicts with any other node e.g. move_base, teleop_keyboard to publish control the robot.

solved in https://github.com/robotont-demos/demo_teleop


Exported chart is shrunk in 0.4.5

🐛 Bug Report
Exported chart looks different between 0.4.4 and 0.4.5. It looks shrunk in 0.4.5.
🔬 How To Reproduce
Steps to reproduce the behavior:

Install datawrapper v0.4.4.
Execute code sample and see exported chart.
Install update to v0.4.5 (it only updates the datawrapper package).
Execute code sample again and see exported chart.

Code sample
dw = Datawrapper(access_token=token)
df = pandas.DataFrame(
    [("01/01/2020", 2), ("01/02/2020", 4)], columns=["date", "messages"]
)
chart = dw.create_chart(title="Shrunk", chart_type="column-chart", data=df)
dw.export_chart(chart_id, filepath=f"{chart["id"]}.png")

Environment

OS: Windows 10
Python 3.8.3

Screenshots

📈 Expected behavior
Exported chart should not look shrunk.
📎 Additional context

Hi @actionbrk , would you mind sharing both versions of the chart? Shrunk and non-shrunk?
I just tried recreating it and I get the same chart for both 0.4.4 and 0.4.5

Hi @chekos,
I recreated a new venv (still in Python 3.8.3) and pip installed datawrapper (v0.4.4), pandas and python-dotenv. I ran the same code. As with my old venv, the chart looks nice in 0.4.4 :

Then I updated datawrapper with pip install -U datawrapper. Not much new packages were installed :
Successfully installed commonmark-0.9.1 datawrapper-0.4.5 rich-9.13.0 shellingham-1.4.0 typer-0.3.2 typing-extensions-<IP_ADDRESS>

I ran the code again and got a shrunk chart :

I first thought I had a problem with dependencies. I pip uninstall datawrapper and pip install datawrapper==0.4.4 and ran the code again to make sure the new dependencies coming along with 0.4.5 are not the problem ; but going back from 0.4.5 to 0.4.4 fixes it.
Here are the installed packages I have (pip freeze) :
backcall==0.2.0
certifi==2020.12.5
chardet==4.0.0
click==7.1.2
colorama==0.4.4
commonmark==0.9.1
datawrapper==0.4.5
decorator==5.0.7
idna==2.10
ipython==7.22.0
ipython-genutils==0.2.0
jedi==0.18.0
numpy==1.20.2
pandas==1.2.4
parso==0.8.2
pickleshare==0.7.5
prompt-toolkit==3.0.18
Pygments==2.8.1
python-dateutil==2.8.1
python-dotenv==0.17.0
pytz==2021.1
requests==2.25.1
rich==9.13.0
shellingham==1.4.0
six==1.15.0
traitlets==5.0.5
typer==0.3.2
typing-extensions==<IP_ADDRESS>
urllib3==1.26.4
wcwidth==0.2.5

Maybe the Python version has something to do with it? I think I will try to do the same with Python 3.9.

I created a new venv with Python 3.9.4. I went through the same steps and I ended up with the same problem (and same dependency list).

Sorry for the delay in response! i'm taking a look at this today. In datawrapper 0.4.5 we changed the default width to 100 (and the unit is still defaulted to "px") whereas before it was set to 600. That was a mistake on our part! Thanks for catching it @actionbrk
This will be fixed in 0.4.6 but in the mean time you can just dw.export_chart(chart_info['id'], title = "cool chart", width = 600) to restore previous behavior (or change it to whatever works for you!)
Again, thanks for catching this AND letting me know, I hadn't heard from other users but this is not the behavior I want by default 🚀


opentofu: fix mkProvider

The change in #358522 worked for the actual providers that live under plugins, but didn't account for actualProviders and mkProvider, which aren't packages and don't have an override function. So if you try to use mkProvider with opentofu, for instance, it will throw an error.
This change does the simple thing and only tries to override entries in plugins that actually have an override attribute, and passes through other values unchanged.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

looks like it works - thanks for the fix!

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 360647

x86_64-linux


Core: add deviation to requestPerSec assertion

I mean something like this:
.assertions(
    global.requestPerSec(target=10, deviation=.05, inclusive=true),

)

where "target" is target intensity and "deviation" is deviation in percent.

The syntax wouldn't work with Gatling DSL. target and deviation should be defined on the target, not on the metric.
With #3845, we have:
global.requestsPerSec.around(mean = 10, plusOrMinus = 5, inclusive = true)

where plusOrMinus is an absolute value.
How should we have a way to define plusOrMinus as a percentage of the mean? Additional boolean?
Contributions welcome.
Please note that such contribution have to land before we release 3.4.0 or it will be too late.

May be add another target then? I can try to implement it if you approve this way.

No, not a new target, this is not a new metric. This would be merely a new shortcut built on top of between, just like around, see https://github.com/gatling/gatling/blob/master/gatling-core/src/main/scala/io/gatling/core/assertion/AssertionBuilders.scala#L67-L68.
The hard thing is to come up with proper names for around based on absolute deviation values and this new version based on relative ones.

I've created changes but Im not familiar with PRs at all. Should I create PR into master or another branch?  You have to fork it, push there and then create a PR.


not showing the correct pokemon data

need to fix the readline, the unique pokemon showed is charmander

add the line

int choose = 0;
and set'd a try with choose of a pokemon

        try

        {
            Console.Write("Choose a number:  (0 to 10) ");
            choose = Convert.ToInt32(Console.ReadLine());
            if (choose > 10) choose = 10;
        }
        catch {
            choose = 1;
        }


Implement a process to ensure logic contracts can be initialized

Currently, the logic contract of a proxied contract could be "hijacked" by anyone. In most cases this isn't dangerous, but there are specific situations (e.g. when a selfdestruct is reachable) where this could pose problems. To avoid any headaches around this, the Celo smart contracts should have a mechanism for automatically locking the logic contract on deployment.
One proposal for achieving this:

To the Initializable contract, add a constructor that sets initialized to 1, thus blocking proper initialization.
Since unit tests use contracts unproxied, this constructor could take an additional boolean argument testingDeployment that, when set to true, would skip the above step.

@yorhodes @aslawson Moved this one up as well. None of our CoreContracts should be vulnerable, but it should be good practice?

@yorhodes @aslawson Moved this one up as well. None of our CoreContracts should be vulnerable, but it should be good practice?

IIUC - this feels like an generalized approach for #6544 . Can this solution be applied for ReleaseGold contract deployments as well?

IIUC - this feels like an generalized approach for #6544 . Can this solution be applied for ReleaseGold contract deployments as well?

I'd interpret this ticket as solving it for the Celo Core Contracts, and #6544 to solve it for releaseGold contracts. I think the solution is going to be similar, but still different enough to track differently IMO

I'd interpret this ticket as solving it for the Celo Core Contracts, and #6544 to solve it for releaseGold contracts. I think the solution is going to be similar, but still different enough to track differently IMO


enums are always lowercased in queries despire enum_name leading to type does not exist

Description
After running the CLI to generate the types from my existing db, I got
#[derive(Debug, Clone, PartialEq, Eq, EnumIter, DeriveActiveEnum)]
#[sea_orm(rs_type = "String", db_type = "Enum", enum_name = "ModuleType")]
pub enum ModuleType {
    #[sea_orm(string_value = "Sink")]
    Sink,
    #[sea_orm(string_value = "Source")]
    Source,
    #[sea_orm(string_value = "Transformation")]
    Transformation,
}

A simple query with it always results in a lowercase moduletype not existing - which is correct. The generator correctly extracted the type name with capitals.
Query(SqlxError(Database(PgDatabaseError { severity: Error, code: "42704", message: "type \"moduletype\" does not exist", detail: None, hint: None, position: Some(Original(65)), where: None, schema: None, table: None, column: None, data_type: None, constraint: None, file: Some("parse_type.c"), line: Some(270), routine: Some("typenameType") })))

Steps to Reproduce

Have a enum with capitals in it
sea-orm-cli generate entity
run any insert

Expected Behavior
inserts work
Actual Behavior
Query(SqlxError(Database(PgDatabaseError { severity: Error, code: "42704", message: "type \"moduletype\" does not exist", detail: None, hint: None, position: Some(Original(65)), where: None, schema: None, table: None, column: None, data_type: None, constraint: None, file: Some("parse_type.c"), line: Some(270), routine: Some("typenameType") })))

Reproduces How Often
always
Workarounds
Reproducible Example
Versions
sea-orm v0.12.12

Hey! I'm facing the same issue, have you found a workaround?

I am also unsure where the lowercase is being applied. If anyone have found it, a PR would be appreciated!

Has the same issue in postgres, my enum name is like ModuleType in pg and enum_name is set, but orm is lowercasing in on insert and so on to  moduletype


Configuration and build:  Fix solaris tags

The shared_target attrribute for Solaris built with gcc wasn't right
and shared libraries couldn't be properly built.
Fixes #12356

@rainerjung Can you confirm that this addresses your iusse please.

Confirmed, looks good to me!

This pull request is ready to merge

Merged
3a19f1a9dd16f5855a7e2a42cd3e7f136f849bd3 Configuration and build:  Fix solaris tags


Logos updated to svg version

Add space for ...
Etherscan: https://viewblock.io/zilliqa/address/zil1pdxm02u4v6wmjtwswqm305uyth2lkwurawkvre
CoinGecko: N/A
Website: https://tokentalks.carrd.co/
Twitter: https://twitter.com/TokenTalksTT

Is there anything else that needs to be done? I have noticed the website does not show new projects after they were merged into master.
Sent with ProtonMail Secure Email.
‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
On Wednesday, April 7, 2021 12:57 PM, Joel Lim @.***> wrote:

Merged #17 into master.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.

It does show up, we've just haven't pushed them to production yet. I'll get the greenlight to push to production and you should see the new projects @evesnow91 let me know when greenlit, thanks.

And it's up on production! @JoWProDev

Yep, I see it there now! Great. Thank You!
Sent with ProtonMail Secure Email.
‐‐‐‐‐‐‐ Original Message ‐‐‐‐‐‐‐
On Thursday, April 8, 2021 6:18 PM, Joel Lim @.***> wrote:

And it's up on production! @JoWProDev
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.


Cold wallet: Cost of transaction exceeds limit

A user got the error message "Cost of transaction exceeds limit" when trying to sign a transaction containing many inboxes.
The situation should be improved so that either signing completes successfully, or a hint that this will probably happen is shown before the transaction is transferred to the cold device.

A limit restrricting transactions to 100 input boxes will be implemented.

Done


Make sure whe have the right name for the Express 4 JS template.

The Id is pulled from the vspackage.resx file (and the matching localized files)

Replaced with #1519


Add multiple images preview

Description
Added a feature to preview multiple images at once before uploading them.
Fixes #326
Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes
[ ] Any dependent changes have been merged and published in downstream modules

@Tushar19varshney in the subsequent PRs i will implement a image cropper. can you please give your inputs on it?

@carpecodeum Yes, react image cropper is a good idea. You can also implement cropper when uploading profile picture. Go ahead!


cannot execute: required file not found

Every time I try to launch the provided binary, I get the error bash: ./steam-tui: cannot execute: required file not found. I am on Ubuntu 23.04.

Can you do a ldd steam-tui? Might be bad libraries. Where did you get the binary?

Might just be dupe of #54

Woops. just meant to comment,

I'm getting this same error, here is the output of ldd steam-tui
linux-vdso.so.1 (0x00007fff434a9000)
libssl.so.1.1 => /usr/lib/libssl.so.1.1 (0x00007fe53c915000)
libcrypto.so.1.1 => /usr/lib/libcrypto.so.1.1 (0x00007fe53be00000)
libgcc_s.so.1 => /usr/lib/libgcc_s.so.1 (0x00007fe53c8f0000)
libm.so.6 => /usr/lib/libm.so.6 (0x00007fe53c113000)
libc.so.6 => /usr/lib/libc.so.6 (0x00007fe53ba00000)
/nix/store/mhhlymrg2m70r8h94cwhv2d7a0c8l7g6-glibc-2.34-210/lib/ld-linux-x86-64.so.2 => /usr/lib64/ld-linux-x86-64.so.2 (0x00007fe53c9dd000)

I fixed this by just copying the ld-linux-x84-64.so.2 into the "expected" nix directory


chore(docker): undo signing for now

😢

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 28.372%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3712

Relevant Lines:
13769

💛  - Coveralls


Update babel in environment: "parcel"

Bug report
Packages affected

[ ] sandpack-client
[x] sandpack-react

Description of the problem
When I use Private Fields in environment: "parcel",  preview throw an error.
/src/index.js: Unexpected character '#' (2:2)

  1 | class App {
> 2 |   #privateField = 1111;
    |  ^
  3 | }
  4 | console.log(new App());

Probably, babel-env is outdated?
babel-env support private fields in 7.10.0.

7.10.0 Released: Class Fields in preset-env, '#private in' checks and better React tree-shaking · Babel

https://1-0-4-sandpack.codesandbox.io/static/js/babel.6.26.min.js is used.
What were you doing when the problem occurred?
I write private fields synbol # and get an error.
What steps can we take to reproduce the problem?
import { Sandpack } from "@codesandbox/sandpack-react";

const APP_CODE = `
class App {
  #privateField = 1111;
}
console.log(new App());
`.trim();

export default function App() {
  return (
    <Sandpack
      customSetup={{
        environment: "parcel",
        files: {
          "/src/index.js": {
            code: APP_CODE
          }
        }
      }}
      template="vanilla"
    />
  );
}

Link to sandbox: link (optional)
https://codesandbox.io/s/sandpack-parcel-es2022-forked-tgk89o?file=/src/index.js
Your Environment

Software
Name/Version

Sandpack-client version

Sandpack-react version
1.0.4

Browser
Firefox/Chrome

Operating System

Maybe Related

https://github.com/codesandbox/sandpack/issues/137

As a context, I want to use native ES2022 syntax in sandpack.
But, sandpack transpile the code by babel.

@DeMoorJasper do you think can we support this on bundler v1? or maybe on v2?

@danilowoz Skipping babel transpilation no, but updating babel probably. Already works with v2

@azu would you mind using the new bundler? It should work fine
<Sandpack options={{ bundlerURL: "https://83c59996.sandpack-bundler.pages.dev/" }} />

Thanks!
I've confirmed that private class fields works with new bundler
https://codesandbox.io/s/sandpack-parcel-es2022-with-new-bundler-m8vvv1?file=/src/App.js


Cannot Set the history_length to 0 Despite Being Provided

I had a look at the troubleshooting section

[x] Yes
[ ] No

Describe the bug
Cannot set the history_length to 0 despite provided.
To Reproduce
Steps to reproduce the behavior:

Configure Workflow…
Set history_length to 0 and save
Talk to ChatGPT
See error and found error_message: This model's maximum context length is 4097 tokens. However, your messages resulted in 26794 tokens. Please reduce the length of the messages. in log.

Expected behavior
Talk to ChatGPT with history_length set to 0.
The problem seems to be caused by this logic in workflow/src/text_chat.py
    return history[-__history_length:]

Cuz it will just return the full history if __history_length is 0.
Suggested and tested fix:
    return history[len(history) - __history_length:]

Relevant information from the ChatFred_Error.log file
Date/Time: 2023-07-31 05:53:06.224087
model: gpt-3.5-turbo
workflow_version: 1.5.1
error_message: This model's maximum context length is 4097 tokens. However, your messages resulted in 26794 tokens. Please reduce the length of the messages.
user_prompt: ?
temperature: 0.0
max_tokens: None
top_p: 1
frequency_penalty: 0.0
presence_penalty: 0.0

Additional context
Add any other context about the problem here.

Yes, please fix this... additionally, if I set it to a low number such as 1 or 2, it doesn't mind the setting. It just shows everything. IOW, it doesn't work :(

Thanks for reporting this bug. Unfortunately, I am quite busy recently. Maybe someone wants to create a PR?

I created a PR #80 ;)

Hey @Sponge-bink!
Thanks for your PR. I already created a new release: https://github.com/chrislemke/ChatFred/releases/tag/v1.5.2
@toolman11 Could you check if this fix is also solving your issue? Thanks!

No change for me. I deleted the workflow, downloaded ChatFred-1.5.3.alfredworkflow and installed it, added API key, set the "ChatGPT history length" to 1, then 3, then 0, and the complete history is still there (no changes to the history list I see when I invoke Alfred and type "cf ". I'm using the latest (August 2nd, I believe) version of Alfred 5.

That's not what that option means, it just control how many recent messages got to actually send to ChatGPT (to provide better context / cut down token usage). You will see many history messages from Alfred before you send the query, it's there so that you can reuse a prompt (hit Tab key) or just view the result again (hit Return).
What you are experiencing is an expected behavior.

Understood... especially now that I read the sub-text :) Sorry for the mis-understanding.


Running aur-update -s without root adds aur-pkglist to ~

Running aur-update -s without root creates aur-pkglist file in home directory ~. This file contains a list of all directories in ~.

This has been fixed in another PR (fix-update-all branch).


Broken links (404)

The links for windows installers for node.js 10.19.0 are broken.
(https://nodejs.org/dist/latest-v10.x/node-v10.19.0-x64.msi and https://nodejs.org/dist/latest-v10.x/node-v10.19.0-x86.msi)
Working links are https://nodejs.org/dist/v10.19.0/node-v10.19.0-x64.msi and https://nodejs.org/dist/v10.19.0/node-v10.19.0-x86.msi

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

ID: 5bee09a8-022e-1b5a-4aab-d6e7c6216808
Version Independent ID: 49edce1e-f2be-e989-917f-c6b389363e2f
Content: Set up your SharePoint Framework development environment
Content Source: docs/spfx/set-up-your-development-environment.md
Product: sharepoint
Technology: sharepoint-framework
GitHub Login: @spdevdocs
Microsoft Alias: spdevdocs

Vesa has updated the Node.JS version 10.20.1 - 117e2d3,
Please close the incident

@PrasadKasireddy in the future, if you reference issues correctly in the PR submission, it will auto close them.
FWIW, we've moving away from direct links to downloads so we aren't constantly chasing the latest patch version released.


Typo fix

I had to look up devdicate, just in case! Great work with carbonvote so far guys.
jd

Thanks! 👍


Names of country flags barely useful

Example data for a country flag emoji:
{
"name": "REGIONAL INDICATOR SYMBOL LETTERS UY",
"unified": "1F1FA-1F1FE",
"variations": [],
"docomo": null,
"au": null,
"softbank": null,
"google": null,
"image": "1f1fa-1f1fe.png",
"sheet_x": 36,
"sheet_y": 7,
"short_name": "flag-uy",
"short_names": [
"flag-uy"
],
"text": null,
"texts": null,
"category": "Flags",
"sort_order": 243,
"added_in": "6.0",
"has_img_apple": true,
"has_img_google": true,
"has_img_twitter": true,
"has_img_emojione": true,
"has_img_facebook": true,
"has_img_messenger": true
}

That "name": "REGIONAL INDICATOR SYMBOL LETTERS UY" is causing usability issues with emoji-mart (see missive/emoji-mart#96).
I gather from #20 that you are generating these names automatically. Is there some way I can help you generate more useful names for these flags? A name like "[country name] flag" would be great, though I'd also find just "[country name]" pretty ok!

This is fixed in PR #100 which will be release 4.0.0
The NAME property now contains e.g. Israel Flag instead of REGIONAL INDICATOR SYMBOL LETTERS IL


field_at should ignore EcsIterCppEach

In Flex-Rust, field_at() is implemented in terms of field(), which does the check for EcsIterCppEach. This is not done upstream. The only thing that seems to work without setting the flag at the moment is field_at_untyped()

fixed, thanks! I will introduce this in the next minor version later today


Upload Python Project

I would like to add a rock-paper-scissor project program. please assign this to me

I want to contribute to this project. I love making games. Please assign me this project @saraswati258

I also want to contribute to the project plz assign me

I would like to contribute to this issue would you please assign it to me


Improved scikit-learn compatibility

When using a KerasRegressor / KerasClassifier inside a scikit-learn ensemble model (Voting/Stacking/Bagging), an error is raised because such model requires its base estimators to have the attribute _estimator_type, which can be either "regressor" or "classifier". The addition fixes this issue.

@EdoardoBotta Thank you for the PR! would you please provide a colab notebook that reproduces the error?

voting_issue.zip

https://colab.research.google.com/drive/1mzbRXyWmTD6rqchfRPnf1uSGp0v5Zkij?usp=sharing

We are planning to remove the sklearn API support in Keras.
Here is an open-source replacement for this feature.
https://github.com/adriangb/scikeras


Update gitignore

update gitignore
remove files from .idea/ on the remote repository
with issue #3

After merge #6 , there was conflicts with this PR, so I've just resolved it.
Please review it again ;)


fix set_timeout backwards compatibility

When given None as a timeout value, the set_timeout method would not actually set the timeouts to None.  This would leave the values at 3 s if the TSocket object was instantiated with default args.  This change ensures that calling set_timeout(None) really sets the timeouts to None.

This issue is causing our impyla builds to fail.  If this change is ok, could you merge and release?  Sorry for the trouble @lxyu.


Enable hermetic build

Description
Enable hermetic build
Type of change

[ ] Refactor
[ ] New feature
[ ] Bug fix
[ ] CVE fix
[ ] Optimization
[ ] Documentation Update
[x] Configuration Update
[ ] Bump-up dependent library
[ ] Bump-up library or tool used for development (does not change the final image)

Related Tickets & Documents

Related Issue #
Closes #

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 94.83%. Comparing base (d3076b3) to head (de5d901).
Report is 74 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1553      +/-   ##
==========================================
+ Coverage   94.75%   94.83%   +0.07%
==========================================
  Files          59       60       +1
  Lines        2631     2671      +40
==========================================
+ Hits         2493     2533      +40
  Misses        138      138

/retest

/retest

/lgtm

/approve

/override "Red Hat Konflux / ols-enterprise-contract / lightspeed-service"


Start scan each start docker minidlna

Start scan each time start docker minidlna

is not perfect .
Help for issue : https://github.com/geekduck/docker-minidlna/issues/7


Redirect to resource not working when using direct: true

When i use direct: true, an xhr request as expected is sent in the background but when i submit the form after that, i get an error when i try to redirect to the resource.
  def create
    @upload = Upload.new(upload_params)
    if @upload.save
      redirect_to @upload
    else
      render :index
    end
  end

  def upload_params
    params.require(:upload).permit(:file)
  end

Error
No route matches {:action=>"show", :controller=>"uploads", :id=>nil} missing required keys: [:id]
Problem is that with direct: true, no ID is passed in the params anymore so it does not know where to redirect to.
Am i using direct uploads incorrectly? It works fine without direct uploads though.

Bump. Anyone an idea on this one?

Tested with version 6.2 and could not recreate.

This is probably not related to Refile, it seems unlikely that it would affect the create action in such a way.


What is Application object mean?

Existing documentation URL(s)

https://developers.cloudflare.com/fundamentals/setup/account/account-security/scim-setup/

What changes are you suggesting?
The step 2 on Configure user permissions on Okta section. It's mention about application object, which doesn't have screenshot which application object does it mean. This cause a lot of confusion.
Additional information
No response

Hi @kamontat! The application object in this case is not be a tab or link to navigate to.
It is the specific SCIM application that you have created in Okta, where you want to add the user permissions. You can find this in your Okta App Integration Catalog. Clarifying this step in #17779.


Impossible to install new version

Hello, thank you so much for this awesome downloader ! 💯 🥇
I followed the recommended procedure to install the new version :
Download the zip file from the latest release (https://github.com/puemos/hls-downloader/releases)
Open chrome://extensions/
Enable Developer mode
Drop the zip file into the page
But  i can't install the new version :(
we hope that there will be an xpi and crx version (because mercury browser based on firefox allows us to install extensions and addons outside the firefox store) and we hope the versions on firefox store and edge store and opera store will be updated soon, thank you :)

The zip is broken (empty) in the latest version (v4.0.0). Use this version instead: https://github.com/puemos/hls-downloader/releases/tag/v3.3.0

Did you try to download https://github.com/puemos/hls-downloader/releases/download/v4.0.0/extension-archive.zip?

Did you try to download https://github.com/puemos/hls-downloader/releases/download/v4.0.0/extension-archive.zip?

The downloaded zip is just empty...

Can you try this one? https://github.com/puemos/hls-downloader/releases/download/v4.0.0/extension.zip

This one does have a dist folder with content inside.

Great, so try dropping this into the chrome://extensions/.

This does not work because the contents need to be directly in the root of the zip not inside a subfolder dist.
It would be possible to extract the dist folder from the zip and load this instead...

Yes, can can also drop the dist folder directly.

Nope this does not work because the dist folder is not a zip. You can either load the folder via the button load unpacked extension as a folder or drag the zip where the content is located in the root direktly into the window

Else it does not work...

Got it, can you try to load it with "load unpacked extension" and select the dist folder?

Version https://github.com/puemos/hls-downloader/releases/tag/v3.3.0 works without any problems...

I think this has to do with the new policy that google introduced...

https://developer.chrome.com/docs/extensions/reference/manifest/content-security-policy

If you bump up the manifest version to version 3 this error message pops up:
Invalid value for 'content_security_policy'.

Ohh thank you @puemos and @Mqxx  for your valuable explanations :)
It seems a little too complicated to install new version in chromium browsers :(....maybe on firefox and its forks like mercury browser, the installation will be simpler, or on the forks of chromium like thorium browser and uazo browser and unggogled chromium which can make possible to bypass the strict rules of chrome to install the extensions and we hope for extensions well compatible with these web browsers if chrome web store and mozilla firefox store and opera browser and microsoft edge browser does not allow installation of the new version, thank you :)
https://librewolf.net/
https://github.com/pulse-browser/browser
https://github.com/dothq/browser
https://floorp.app/en/
https://www.waterfox.net/
https://mullvad.net/en/browser
https://duckduckgo.com/app
https://github.com/uazo/cromite
https://github.com/Alex313031/Thorium-Win
https://chromium.woolyss.com/
https://github.com/ungoogled-software/ungoogled-chromium
https://github.com/textbrowser/dooble
https://whale.naver.com/en/
https://www.fenrir-inc.com/jp/sleipnir/

Yeah, the best solution is to figure out the migration to mv3.

Great Mr @puemos , thank you so much :) 🥇
It say firefox and brave browser will keep manifest v2, so it seems we can still use manifest v2 extension with frefox and brave....
To work around problems with manifest v3 and escape censorship,  perhaps it will be good idea to replace extensions with scripts:
https://openuserjs.org/
https://greasyfork.org/en
https://www.userscript.zone/
or use these extensions to integrate extensions with various web browsers !
extension to integration with  web browsers :
https://foxified.org/
https://crosspilot.io/
Chameleon
https://chrome.google.com/webstore/detail/chameleon/dmpojjilddefgnhiicjcmhbkjgbbclob
https://sereneblue.github.io/chameleon/
https://en.savefrom.net/18/user.php?vid=387
https://github.com/anime-vsub/extension-animevsub-helper/blob/main/install-on-chrome.md
https://addoncrop.com/v27/
Orange Monkey script manager
https://greasyfork.org/fr
https://www.tampermonkey.net/scripts.php
https://webextension.org/listing/download-with.html
https://github.com/belaviyo/download-with/
https://github.com/FilipePS/Traduzir-paginas-web
intall outside store
https://github.com/tsaost/autoload-temporary-addon
https://tsaost.github.io/autoload-temporary-addon/
"The way around this is to download the Chrome extension files into a folder
and then load that folder (Load unpacked) in Developer mode.
we can solve this issue very easily. Download the .crx, right click
and extract by 7-zip (or rename .crx to .zip)
and then load the extension as unpacked in chrome after
enabling dev mode in the extensions page.
(Bonus point, loading it unpacked is more ease of mind, since the js code
is not minified, you can read it in 5 minutes and know exactly what it's doing!)
Install
Firefox
Desktop users, download from Mozilla Addons.
Mobile users
You can install TWP using this alternative browser to Firefox Mobile Iceraven.
Or follow this tutorial and install TWP on Firefox Nightly.
Vivaldi, Opera, Maxthon, Chromium and Yandex
Download this CRX file TWP_Chromium.crx
Open your browser's extension manager, you can find it at this link: chrome://extensions
Activate developer mode
Reload the extension manager page to avoid errors
Drag and drop the TWP_Chromium.crx file into the extension manager
Note 1: In Opera, Maxthon and Yandex you don't need to enable developer mode
Note 2: In Yandex you need to reactivate the extension every time you open the browser
Chrome, Edge and Brave (With folder without auto update)
Download and extract this ZIP file TWP_Chromium.zip
Open your browser's extension manager, you can find it at this link: chrome://extensions
Activate developer mode
Reload the extension manager page to avoid errors
Drag and drop the TWP_Chromium folder into the extension manager
Chrome, Edge and Brave (With CRX and auto update)
By default, these browsers block the installation of extensions outside the official extension store, however, changing a windows registry it is possible to reverse this, allowing the installation of certain extensions. If you want to do this, follow the tutorial below:
Download this and run this file twp-registry-install.reg. It edit the necessary windows registries
Close your browser and reopen it
Download this CRX file TWP_Chromium.crx
Open your browser's extension manager, you can find it at this link: chrome://extensions
Activate developer mode
Reload the extension manager page to avoid errors
Drag and drop the TWP_Chromium.crx file into the extension manager
Note: If you want to undo registry changes, download and run this twp-registry-uninstall-self.reg. If you want a deeper removal download and run this other file twp-registry-uninstall-all.reg "


[Resteasy-2246] Dynamic features should be resolved once for each resource method

Hi @asoldano and @tterem
This PR is a fix proposal for the DynamicFeature multiple invocations issue.
Its is based on @tterem  PR https://github.com/resteasy/Resteasy/pull/2037 where I just removed the @Ignore("RESTEASY-2246") on test testDynamicFeatureProcessing.
As you will maybe notice, I try in my first commit to no change the public interface JaxrsInterceptorRegistryListener to preserve backward compatibility.
So I was doing a kind of merge between old and new providers. Unfortunately many others tests was failing because, doing things this way I was loosing the sorting provided by JaxrsInterceptorRegistryImpl.createArray() invoked in the postMatch() method.
So in my second commit I rollback everything and modify interface JaxrsInterceptorRegistryListener to get the new provider registered on the parentProviderFactory that trigger the update.
This way no need to rebuild no new resourceMethodProviderFactory and reconfigure everything, we just have to update filter and interceptor registry based on parentProviderFactory  updates.
Let me know what you think please.

@NicoNes thanks, we'll review soon!