Add stub of str.format()

When I tried to implement str.format() got an issue with position arguments. Could you please give advice how to work with position arguments in particular in my case?

Could you please mark the PR as WIP and question?

I think this may be hitting a bug in the Numba internals. Irrespective of this, I'm unsure how good an idea it is to try and implement the format spec directly in Numba, it's hugely involved https://github.com/python/cpython/blob/04c79d6088a22d467f04dbe438050c26de22fa85/Objects/unicodeobject.c#L2689.

I think this may be hitting a bug in the Numba internals. Irrespective of this, I'm unsure how good an idea it is to try and implement the format spec directly in Numba, it's hugely involved https://github.com/python/cpython/blob/04c79d6088a22d467f04dbe438050c26de22fa85/Objects/unicodeobject.c#L2689.

Maybe I could try to implement simple version of the method if you gave me advice how to work with position arguments.

I think this may be hitting a bug in the Numba internals. Irrespective of this, I'm unsure how good an idea it is to try and implement the format spec directly in Numba, it's hugely involved https://github.com/python/cpython/blob/04c79d6088a22d467f04dbe438050c26de22fa85/Objects/unicodeobject.c#L2689.

Maybe I could try to implement simple version of the method if you gave me advice how to work with position arguments.

I think this is hitting a bug, I don't think you can work with what you have written because of this bug. The bug will need fixing to make this possible, perhaps fix that first?

@densmirn were you planning on continuing with this PR? It's just a stub at present. If not, I propose we close it? Thanks.

@densmirn were you planning on continuing with this PR? It's just a stub at present. If not, I propose we close it? Thanks.

Let's close the PR.

@densmirn were you planning on continuing with this PR? It's just a stub at present. If not, I propose we close it? Thanks.

Let's close the PR.

ok, thanks for confirming.


 I'm going to use somethig else.

Don't give up on us yet, we're working on it!
Dupe of https://github.com/Microsoft/vscode/issues/35783

@gonoiu hahahahahahaha


Fix embeded enums

We were missing a trailing } both in de and ser when parsing (non-newtype / embedded) enum structs. It was symetrical, but not valid json.
Now this is fixed, and compatible with typical parsers.

Once again... sorry


benchmarking with autosklearn (zeroconf)

Great initiative, thanks for making this public!
You might be interested in extending your benchmarking to the auto-sklearn. https://github.com/automl/auto-sklearn
I have created a script that can take in a sparse dataset in the pandas HDFS dataframe .h5 format and run a binary classification on it on multiprocessing cluster with auto-sklearn. https://github.com/Motorrat/autosklearn-zeroconf Myself I will try to duplicate your benchmark, but just in case you are on it you might want to try out yourself.

Thanks @Motorrat for feedback/comments/info. I probably won't have time to extend the benchmark, but contributions are welcome :)


Object being "left behind" even though animation is supposed to move it away

Describe the bug

When I apply MoveToTarget to a mobject, the mobject does move, but a copy of it stays in the old position.
Code:

the full construct function:
        tensordot = TexText("$ \otimes $")
        vector = TexText(r"$v^{i}$").next_to(tensordot, LEFT).set_color_by_gradient(GREEN)
        covector = TexText(r"$w_{j}$").next_to(tensordot, RIGHT).set_color_by_gradient(YELLOW)
        self.play(Write(tensordot), Write(vector), Write(covector))
        text1 = TexText(r"$ \leftarrow $ values $ \rightarrow $").next_to(tensordot, DOWN)
        dvector = TexText(r"$v = \begin{bmatrix}1\\2\end{bmatrix}$").next_to(text1, LEFT)
        dcovector = TexText(r"$w = \begin{bmatrix} 3, 4 \end{bmatrix}$").next_to(text1, RIGHT)
        self.wait()
        self.play(Write(text1), Write(dvector), Write(dcovector))
        tgroup = VGroup()
        tgroup.add(tensordot, vector, covector)
        self.wait()
        self.play(FadeOut(text1), FadeOut(dvector), FadeOut(dcovector))
        matrix = TexText(r"$ W^{i}_{j} = v^{i}w_{j} $").set_color_by_gradient(BLUE)
        self.play(Transform(tgroup, matrix).set_run_time(2))
        self.wait()
        m12 = TexText(r"$M^{1}_{2} = v^{1}w_{2} = 1 \cdot 4 = 4$").set_color_by_gradient(RED)
        m11 = TexText(r"$M^{1}_{1} = v^{1}w_{1} = 1 \cdot 3 = 3$").next_to(m12, UP).set_color_by_gradient(RED)
        m21 = TexText(r"$M^{2}_{1} = v^{2}w_{1} = 2 \cdot 3 = 6$").next_to(m12, DOWN).set_color_by_gradient(RED)
        m22 = TexText(r"$M^{2}_{2} = v^{2}w_{2} = 2 \cdot 4 = 8$").next_to(m21, DOWN).set_color_by_gradient(RED)
        matrix.generate_target()
        matrix.target.shift(2*UP)
        self.play(MoveToTarget(matrix)) # supposed to move but was stuck
        self.play(Write(m11))
        self.play(Write(m12))
        self.play(Write(m21))
        self.play(Write(m22))
        self.wait()

Wrong display or Error traceback:

the object "matrix" gets stuck in the old position and a copy of it moves to the target.
Additional context

Use ReplacementTransform instead of Transform:
self.play(ReplacementTransform(tgroup, matrix).set_run_time(2))

@TonyCrane  thats exactly it thanks


Support servlet API

Motivation:
#2672
#253
Armeria supports servlet interface because it is familiar to java web developers.
Modifications:
Result:
Close #2672
Close #253

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Do Minh Hien seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

Close #2672
Close #253

If you want to solve two issues, how about solving them in separate PRs?

Close #2672
Close #253

If you want to solve two issues, how about solving them in separate PRs?

Servlet support multipart, I think this PR resolved #253. However, if we need separate PR to resolve each issue, I will do that.

Servlet support multipart, I think this PR resolved #253. However, if we need separate PR to resolve each issue, I will do that.

I didn't take a look at closely at the multipart logic, but if it's solved only when using Servlet, then this PR actually does not close #253.
So forget about making separate PRs. 😄

Servlet support multipart, I think this PR resolved #253. However, if we need separate PR to resolve each issue, I will do that.

I didn't take a look at closely at the multipart logic, but if it's solved only when using Servlet, then this PR actually does not close #253.
So forget about making separate PRs. 😄

Agree, I will remove close #253 :)

@dominhhien This PR is really huge so it's very hard to review. :thinking:
Do you mind splitting this PR into multiple PRs?
For example, I think we can get rid of multipart, filter, etc. and focus on implementing ServletService (which implements HttpService).
If we complete the first step, then we can go on to the next step.
If we don't publish this servlet until we have all we need, it shouldn't be a problem.
What do you think?

@dominhhien This PR is really huge so it's very hard to review. 🤔
Do you mind splitting this PR into multiple PRs?
For example, I think we can get rid of multipart, filter, etc. and focus on implementing ServletService (which implements HttpService).
If we complete the first step, then we can go on to the next step.
If we don't publish this servlet until we have all we need, it shouldn't be a problem.
What do you think?
@minwoox Agree, I will split this PR. First, we focus on ServletService. Please, wait a moment, I change pull request content now.

@dominhhien This PR is really huge so it's very hard to review. 🤔
Do you mind splitting this PR into multiple PRs?
For example, I think we can get rid of multipart, filter, etc. and focus on implementing ServletService (which implements HttpService).
If we complete the first step, then we can go on to the next step.
If we don't publish this servlet until we have all we need, it shouldn't be a problem.
What do you think?

@minwoox  I changed pull request content to focus on ServletService implementation.

Could you sign the ICLA first?

Could you sign the ICLA first?

I signed ICLA. I don't know why it is not complete. Please help me check this.

@minwoox I resolved your reviews.

@trustin @minwoox @ikhoon Could you help me check why AppVeyor build failed? I read tracing log, but I can't figure out the reason.

Also there's no way to set servletName.
Servlet could be mapped using serveltName or path(urlPattern).

Also there's no way to set servletName.
Servlet could be mapped using serveltName or path(urlPattern).

Url maps to servletPath, servletPath maps to servletName, servletName maps to servlet class. I think both servlet path and servlet name can have the same value. Therfore our API will be simpler.

@minwoox I resoved your reviews. Please, help me review further.

@minwoox I added context path tests and throw exception after server is initialized. Please, take a look. Thank you so much.

Yes, I understood. I want to confirm do you mean this comment #2686 (comment) exclude setSessionTrackingModes()?

Yes, we need to add ensureUninitialized().

Could you address this issue #2686 (comment)
please?

I tried
httpRequest.headers().contentType().charset()

but httprequest stuck and timeout. Therefore I used:
ServletUtil.decodeCharacterEncoding(getContentType())

httprequest is working. I don't know the reason httpRequest.headers().contentType().charset() is not working.

httpRequest.headers().contentType()

Oh we should check whether the contentType exists or not, first. 😉

I'm still fixing, please wait a bit more. 😉

I realized that we cannot use servletName as the URL mapping.
A user has to use ServletRegistration to map a URL to the servlet:
final DefaultServletRegistration servletRegistration =
    new DefaultServletRegistration(servletName, servlet, ...);
servletRegistration.addMapping(urlMappings);

Because we use the fluent builder pattern, we cannot just return ServletRegistration from ServletBuilder:
public ServletBuilder servlet(String path, String servletClass) {
    ...
}

So let me just add String... urlMappings as a parameter for now:
public ServletBuilder servlet(String path, String servletClass, String... urlMappings) {
    ...
}

We might need to an API that exposes ServletRegistration later.

I realized that we cannot use servletName as the URL mapping.
A user has to use ServletRegistration to map a URL to the servlet:

Yes, I lacked the case of mapping multiple urls to one servlet name.

I added more commit PTAL @dominhhien.

LGTM, thank @minwoox !

examples/servlet-container/

We are not a container, so perhaps better renaming to examples/servlet-adapter ?

You might also want to run a subset of JakartaEE Servlet TCK:

https://cwiki.apache.org/confluence/display/TOMCAT/Servlet+TCK+5.0
https://github.com/eclipse-ee4j/jakartaee-tck
We could skip some tests for container-side, but others may still be useful. Not sure how much effort will be required to run a TCK, though...

Please ignore my comments about missing features. I had a chat with @minwoox that your plan is to start with a small implementation and add more on top of it, because it's hard to implement everything at once.

Please add an integration test that tests a few well known Servlet-based applications, so we are sure this is useful to a user who's looking for a lightweight alternative to TomcatService and JettyService. Otherwise, there would be no point of implementing Servlet API by ourselves.

@trustin Do you mean the integration test is like this:
REQUEST(ajax/submit (Frontend) => servlet request => grpc/thrift (Backend))
RESPONSE (grpc/thrift response (Backend) => servlet response => ajax/submit response (Frontend))

Do you mean the integration test is like this

No. Pick a few well known open source web applications and run it. I guess we can do this later in a follow-up PR though, given that this PR will not implement some of the essential functionalities such as sessions and filters.

I changed this PR to merge servlet branch instead of master because we got lots more work to do. 😄
Also, could you rename the module from servlet to servlet4? Then, I think we can merge this and go on.

Thanks a lot for your effort and patience, @dominhhien!
Let's move on. 😄

Yes, I will create new pull request to implement next servlet features.

🎉🎉🎉🎉🎉 Congratulations on your first merge!
Let's keep going. 😉


Pilot roster page styling updates

https://github.com/lonestarvirtual/skyOS/blob/bd3a66329d6df6e29e2f164a35a31cf81b2c891a/app/views/pilots/index.html.haml#L3
Change this class to just .container instead of .container-fluid to match styling of other pages.
Consider adding a "View" button on a far right column to highlight the ability to view more details about a pilot.

Would you want the button coded in without the functionality? Is there a view that it should reference when it is clicked?

I was thinking similar to the Pilot logbook view page:

Instead of the link being on the Pilot ID (which is somewhat hidden), make another column on the far right similar to above that has a 'View' button with maybe the magnifying glass icon?
Open to discussion.

Ok, I can get the button programmed in. Ignore the last pull request for this.


Colum sizes don't save position when re entering the folder

Description
I can change column sizes in a folder but if I go back then to it again or close the program it resets to the wide as possible sizes.
Steps To Reproduce
No response
Expected behavior
It should save the custom sizes and not reset
Files Version
<IP_ADDRESS>
Windows Version
10.019043.1237
Relevant Assets
debug.log
debug_fulltrust.log

Some additional informational on <IP_ADDRESS>:
Columns sizes are saved after closing if firstly open any folder with unchanged column size instead of a changed folder. If open folder with changed columns sizes after opening another folder columns sizes restore to changed values even if firstly sizes were reset to default.
Columns sizes are saved if navigate through different folders without app closing.
P.S. Don't we need to change columns sizes for all folders not for only one? Maybe we can implement it with an appropriate option in the preferences?

Merging with #6448


Plans to upgrade buster base images to bullseye?

Hello,
Debian recently announced a new stable build under the name of bullseye. This is the new stable build that is replacing buster as stable.
In the cypress base images, buster is currently being used.
This is currently causing issues for our CI/CD system when trying to build cypress
E: Repository 'http://security.debian.org/debian-security buster/updates InRelease' changed its 'Suite' value from 'stable' to 'oldstable' E: Repository 'http://deb.debian.org/debian buster InRelease' changed its 'Suite' value from 'stable' to 'oldstable' E: Repository 'http://deb.debian.org/debian buster-updates InRelease' changed its 'Suite' value from 'stable-updates' to 'oldstable-updates'
This can be fixed with the inclusion of --allow-releaseinfo-change when running apt-get update for the image that we build off of a cypress browser image. This command can be run once to fix a runner at source. We are using on-demand runners instead of a dedicated runners which is causing us to have to run this command each time for this build. This is not ideal as it may incur unintended consequences for any other packages updating unknown to us.
The bullseye images would be available in dockerhub now from the node core team. Understand that it may be a big undertaking but would there be any plans to upgrade the cypress base images to bullseye on your roadmap currently?

As there is node:16-bullseye already out there, what is holding this to be upgraded from buster to bullseye? Anything that I am missing here?

This will be addressed in #597.


Order of loading property is different in apk & companion

With the same project & same blocks, the property value from another component when initializing a component's property is different.
Full extension codes

It seems like the companion (ReplForm) would create the components first and then set up properties. But in compiled applications (Form) would initialize the components with the property

@ColinTree Can you send me the AIA for this project?

@ewpatton please try
CanvasAddonsTest_checkpoint1.zip

@ewpatton Getting any progress on this?


converts build-common-utils pipeline.yml to 1ES template

Description
This PR converts build-common-utils pipeline.ymll to the new 1ES template required by Microsoft with the minimal required set of template changes required. This new template enables security and auditing checks by Microsoft. Read more on how the logic for converting to a 1ES pipeline works on the official 1ES wiki
A temporary 1ES folder was created for converted pipeline templates. This enables other pipelines to remain unchanged and use the original templates while new PR's are opened for individually converted other pipelines to 1ES as required by Microsoft.

Once all pipelines are moved over, we can start moving the files out of the 1ES folder and eventually remove the folder since it will no longer have any files (This is after we convert the rest of the required pipelines).

Breaking Changes
Reviewer Guidance

Confirm the newly converted pipelines run successfully. Look at the completed steps and compare with a recent successful run on main to ensure the expected steps run (plus the new SDL and 1ES security checks)
Make sure all the relevant templates and pipelines for build-common-utils pipeline.yml have been converted
Make sure there are no unnecessary changes or 1ES template and/or pipeline conversions
Make sure the yml is valid -- a successful pipeline run should suffice for confirming this

Those paths are not included in the original file so I don't think there's anything to update unless I'm missing something. If my assumption is correct, do you want me to add those file paths in?

They are, lines 46-48. Maybe you were looking at the triggers for prs, where they indeed don't show up (correctly, because PR builds don't run the bits of the pipeline that actually publish stuff).

Actually, I think we're missing updates on a few trigger.path.include entries. Besides build-npm-package, I think two of the include-publish-npm-package* files also had changes and thus have 1ES versions, right? So we need to update the paths for those.

Those paths are not included in the original file so I don't think there's anything to update unless I'm missing something. If my assumption is correct, do you want me to add those file paths in?

They are, lines 46-48. Maybe you were looking at the triggers for prs, where they indeed don't show up (correctly, because PR builds don't run the bits of the pipeline that actually publish stuff).

Updated those lines to the 1ES folder.


[Feature Request] PROPOSAL: when el-table selection changed, update the summary cell

Existing Component
Yes
Component Name
el-table
Description
When show-summary and type="selection" is enabled, the summary should update if the selection-change method is triggered.

In this case, you can write your own summary-method.

@Leopoldthecoder I have no idea how to do it,  is there any method to know the selected row status, and rerender the table.
Give me a hint

https://jsfiddle.net/gdmzumwq/


Error kmeans example

Hi,
When I run the k-means example I got the following error
Py4JJavaError: An error occurred while calling o109.buildDF.
: java.lang.AssertionError: assertion failed: Op type not registered 'StridedSlice'

Does anyone help me?
Thanks

It looks like it is an issue due to using different versions of tensorflow. I am going to make a new release, which should help.

This example is blocked by #78

This is fixed in master, I am going to publish a new release.


Cache misses when a build request for a cross-targeting project without preceding per-TF requests

A non-graph-based build scheduler may ask for projects to build in topographical order without awareness of TargetFramework global properties that may apply. When a cross-targeting project's build is requested, its "dependencies" include the per-TargetFramework projects, and if they were not built previously, the plugin reports a cache miss.
I expect/hope that instead, the plugin can replay the cache from each TargetFramework in the project if necessary so that the whole project's output can be reconstituted as would happen without the cache.

This was fixed by #93. However, note that the $(MSBuildCacheGetResultsForUnqueriedDependencies) property (or GetResultsForUnqueriedDependencies setting on the plugin) must be set to opt-into this behavior.


[WIP] Bump up cluster-api version to v1.7.1 and go version to 1.21.9

I have bumped up the cluster-api version to v1.7.1 and use the version same as cluster-api v1.7.1 for all other packages.
Also, I have bumped the go version to 1.21.9 (same as cluster-api version)
After the version bump,  controlPlane.Logger() is completely deprecated, so the code is modified to always use ctrl.LoggerFrom(ctx) for logging.
Also, controlPlane.Logger is used a lot in failure_domain.go, so I removed failure_domain.go file and machine_collection.go. Some duplicated code in machine_filters.go is also removed. The code has not modify any logic, except that logic for MachineWithDeleteAnnotation is fixed.

Any reason why FilterableMachineCollection is introduced previously?

Any reason why FilterableMachineCollection is introduced previously?

This is copied from older version cluster api code, and FilterableMachineCollection is removed after refactoring to machie_collections.go. It is removed as we could simply reuse machine_collections.go to avoid code duplication.
If we need to customized machine_collections with different filtering logic, it will be put inside machine_filters.go.


Module Inclusion

Hello,
I tried to use your package in an es module context only to realize it didn't work because your main index.js file possesses commonjs syntax.
I decided to fork and modify the code to include module and umd support using microbundle (more info: https://github.com/developit/microbundle).
I didn't add or remove procedures, I just transformed the index.js into ES6 Module syntax which is now supported by modern browsers. There's a build step which converts this file into different targets (into dist) :

CommonJs (fallback) for people using plain JS in Node.JS scripts.
ESM (along with the module property in package.json) for people using ES6 Modules for modern browsers, experimental Node.JS module support (future), and TypeScript module resolution.
UMD for Node & CDN's

The test passes and the module.exports is unchanged which means existing projects won't be affected. I also tested the compilation result in my initial project and the node resolver uses the mjs as intended (i.e. @vdegenne/clipboard-copy for live testing).
I think this is one step into JavaScript evolution towards trying to make modular programming easier. And this PR should be considered.
If you merge this PR, the next step for you is to build the distribs npm run build, change the versioning, and push the update to the npm repository for everyone's sake. Also modifying the readme according to the new terms.
Have a nice day.

Thank you for this PR. I would like to support ESM in this package, but I don't like how microbundle seems to do a lot more than just this. It's doing some transpiling of async-await to Promise, etc.
Is there are an tool that just takes ESM and outputs CJS and updates package.json? cc @mikeal

Rollup would work fine in this case. Microbundle is based on it.

Btw would you like to add node es modules support as well?

Hmm, since there is no need for this package in Node.js (right?), couldn't we just switch to just doing ES modules and skip the build step altogether? and release that as a new major version...
It seems that that is much easier for us to maintain, and would still be supported by all bundlers, and modern web browsers without a bundler.
e.g. after this change, when working on the package, I know have to remember to run npm run watch or otherwise I will test old code. And I also run the chance of reloading to quickly in the browser after making a change, and then testing the previous version without my latest changes...

and would still be supported by all bundlers

Are ESM-only packages supported by browserify?

Are ESM-only packages supported by browserify?

Would you even need to use Browserify at that point? ESM packages are natively supported in the browser. I have seen some posts suggesting you might be able to Babelify with Browserify to use ESM-only packages, but have not tried it.

You could also support ESM by adding a wrapper which wouldn't require using Rollup or Microbundle: https://nodejs.org/api/packages.html#packages_approach_1_use_an_es_module_wrapper
If we can go ESM-only that'd still be preferable for reduced complexity, but if we need to support both it wouldn't be too hard. Let me know what path forward you'd prefer and I can help


datetime and timedelta constructor validation

datetime and timedelta contructor function currently do not implement and validation of input arguments to the contructor. For example, the datetime contructor needs to validate the following:

Is year > 0?
Is 0 < month < 13?
Is 0 < day < 32?
etc.

and raise an exception if any of the above tests fail. Similarly for timedelta constructor.
Currently, these constructors are implemented as pure elemental functions which allows the user to instantiate an array of datetimes or timedeltas in a familiar Fortran array syntax. Once the above validations and raising exceptions are implemented, these functions cannot be pure (nor elemental) anymore, because only an impure procedure can write to standard output and stop the program. Consequently, this will require implementing a separate version of constructors that will accept arrays as actual arguments, and which will be used for overloading the contructor.

Or use the new impure elemental... If compiler support exists. Alternatively, do something creative with pointers to allow a state full error stack to be passed around and the check it when the function returns.

@zbeekman Did not know about this feature. Is it F2008?

@milancurcic
Alternatively, provide a basic class encapsulating errors handling that will be the parent ancestor of all classes: all pure procedures will remain pure and exception handling vould be done outside the pure procedure. I have not tried this approach, but I found suggestions on that on clf-google-group.
@zbeekman can you elaborate more on both elemental impure procedures and pointers aproach? :pray:
Datetime rocks! My toy project that exploits it is not abbandoned...

impure elemental
From MCR:

Elemental procedures as defined in Fortran 95 and 2003 are required to be pure, a condition
which aids parallel evaluation. While this is advantageous for performance, it does prevent
other possibilities where one wishes to perform impure processing elementally on arrays
of arbitrary rank. In such cases, one was forced to provide a separate function for each
permutation of conformant ranks; for a procedure with two arguments, that was 22 separate
procedures (8 cases where both arguments had the same rank, 7 where the first was scalar andthe second was an array, and 7 where the first was an an array and the second was scalar). With
the increase of maximum rank to 15, this increases to 16+15+15 = 46 separate procedures.
The impure prefix on the procedure heading allows one to define an impure elemental
procedure, which processes array argument elements one by one in array element order. An
example is shown in Figure 20.9. This example is impure in three ways: it counts the number
of overflows in the global variable overflow_count, it logs each overflow on the external
unit error_unit, and it terminates the program with stop when too many errors have been
encountered.

Code example not included due to licensing concerns....

Only the requirements relating to ‘purity’ (lack of side-effects) are lifted: the elemental
requirements remain, that is:
• all dummy arguments of an elemental procedure must be scalar non-coarray dummy
data objects and must not have the pointer or allocatable attribute;
• all dummy arguments of an elemental procedure must have specified intent;
• the result variable of an elemental function must be scalar, must not have the pointer
or allocatable attribute, and must not have a type parameter that is defined by an
expression that is not a constant expression;
• in a reference to an elemental procedure, all actual arguments must be conformable;
and
• in a reference to an elemental procedure, actual arguments corresponding to intent out
and inout dummy arguments must either all be arrays or all be scalar.

comment about pointer
Pointer arguments with intent(in) can be used to allow pure functions to have arguments that are effectively intent(inout) because intent attached to a pointer only refers to the pointer association, the value of the data it is pointing to can change.
In the end, though, using an error handling class with an error stack is the way to go, IMO.

@zbeekman Thank you for the summary. At first I felt embarrassed that I missed this in my MRC, but then I realized its described in the green (new) book which I don't own yet. I still own the blue book (95/2003). :)
Anyhow, these are great recommendations. I like the idea of an extensible parent class with error handling methods as a most general and re-usable approach. I will take some time to digest this information.

Get the green one! (Or see when they're going to publish their next book... maybe Fortran 2015 will be covered)

@zbeekman I skipped the green book and got the red book last year. :)


ScanRetriesExhaustedException when reading from bigtable in a managed VM setup

I am not able to do any bigtable operations from the managed VM setup. The bigtable operations are working fine from my local machine, but timeout on the cloud(Managed VM). The setup was working fine on managed VM as well till yesterday.
I followed the steps as given here, and the setup was working fine for a couple of months (till yesterday, when I started getting timeouts consistently).
I am using version 0.2.2 of cloud-bigtable-client with hbase1.1 (also version 0.2.2).
The error I get in the log is:
Caused by: com.google.cloud.bigtable.grpc.scanner.ScanRetriesExhaustedException: Exhausted streaming retries.
at com.google.cloud.bigtable.grpc.scanner.ResumingStreamingResultScanner.backOffAndRetry(ResumingStreamingResultScanner.java:142)
at com.google.cloud.bigtable.grpc.scanner.ResumingStreamingResultScanner.next(ResumingStreamingResultScanner.java:112)
at com.google.cloud.bigtable.grpc.scanner.ResumingStreamingResultScanner.next(ResumingStreamingResultScanner.java:42)
at com.google.cloud.bigtable.hbase.adapters.BigtableResultScannerAdapter$1.next(BigtableResultScannerAdapter.java:43)
at org.apache.hadoop.hbase.client.AbstractClientScanner$1.hasNext(AbstractClientScanner.java:94)
... 11 more
Caused by: com.google.cloud.bigtable.grpc.scanner.ScanTimeoutException: Timeout while merging responses.
at com.google.cloud.bigtable.grpc.scanner.ResponseQueueReader.getNext(ResponseQueueReader.java:113)
at com.google.cloud.bigtable.grpc.scanner.ResponseQueueReader.getNextMergedRow(ResponseQueueReader.java:64)
at com.google.cloud.bigtable.grpc.scanner.StreamingBigtableResultScanner.next(StreamingBigtableResultScanner.java:64)
at com.google.cloud.bigtable.grpc.scanner.StreamingBigtableResultScanner.next(StreamingBigtableResultScanner.java:28)
at com.google.cloud.bigtable.grpc.scanner.ResumingStreamingResultScanner.next(ResumingStreamingResultScanner.java:101)
... 14 more

Exception in thread "pool-2-thread-2" java.lang.IllegalAccessError: tried to access field sun.security.ssl.Handshak
er.localSupportedSignAlgs from class sun.security.ssl.ClientHandshaker
at sun.security.ssl.ClientHandshaker.processMessage(ClientHandshaker.java:278)
at sun.security.ssl.Handshaker.processLoop(Handshaker.java:913)
at sun.security.ssl.Handshaker.process_record(Handshaker.java:849)
at sun.security.ssl.SSLSocketImpl.readRecord(SSLSocketImpl.java:1035)
at sun.security.ssl.SSLSocketImpl.performInitialHandshake(SSLSocketImpl.java:1344)
at sun.security.ssl.SSLSocketImpl.startHandshake(SSLSocketImpl.java:1371)
at sun.security.ssl.SSLSocketImpl.startHandshake(SSLSocketImpl.java:1355)
at sun.net.www.protocol.https.HttpsClient.afterConnect(HttpsClient.java:559)
at sun.net.www.protocol.https.AbstractDelegateHttpsURLConnection.connect(AbstractDelegateHttpsURLConnection
.java:185)
at sun.net.www.protocol.http.HttpURLConnection.getOutputStream(HttpURLConnection.java:1093)
at sun.net.www.protocol.https.HttpsURLConnectionImpl.getOutputStream(HttpsURLConnectionImpl.java:250)
at com.google.bigtable.repackaged.com.google.api.client.http.javanet.NetHttpRequest.execute(NetHttpRequest.
java:77)
at com.google.bigtable.repackaged.com.google.api.client.http.HttpRequest.execute(HttpRequest.java:965)
at com.google.auth.oauth2.ServiceAccountCredentials.refreshAccessToken(ServiceAccountCredentials.java:222)
at com.google.cloud.bigtable.grpc.io.RefreshingOAuth2CredentialsInterceptor.refreshCredentialsWithRetry(Ref
reshingOAuth2CredentialsInterceptor.java:268)
at com.google.cloud.bigtable.grpc.io.RefreshingOAuth2CredentialsInterceptor.doRefresh(RefreshingOAuth2Crede
ntialsInterceptor.java:238)
at com.google.cloud.bigtable.grpc.io.RefreshingOAuth2CredentialsInterceptor$1.run(RefreshingOAuth2Credentia
lsInterceptor.java:166)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
at java.lang.Thread.run(Thread.java:745)

Tried upgrading to bigtable client 0.2.3. This is the error I am getting now. (I have 'com.google.auth:google-auth-library-appengine:0.3.1' in the classpath)
Caused by: java.io.IOException: Application Default Credentials failed to create the Google App Engine service account credentials class com.google.bigtable.repackaged.com.google.auth.appengine.AppEngineCredentials. Check that the component 'google-auth-library-appengine' is deployed.
at com.google.bigtable.repackaged.com.google.auth.oauth2.DefaultCredentialsProvider.tryGetAppEngineCredential(DefaultCredentialsProvider.java:241)
at com.google.bigtable.repackaged.com.google.auth.oauth2.DefaultCredentialsProvider.getDefaultCredentialsUnsynchronized(DefaultCredentialsProvider.java:138)
at com.google.bigtable.repackaged.com.google.auth.oauth2.DefaultCredentialsProvider.getDefaultCredentials(DefaultCredentialsProvider.java:67)
at com.google.bigtable.repackaged.com.google.auth.oauth2.GoogleCredentials.getApplicationDefault(GoogleCredentials.java:54)
at com.google.cloud.bigtable.config.CredentialFactory.getApplicationDefaultCredential(CredentialFactory.java:181)
at com.google.cloud.bigtable.config.CredentialFactory.getCredentials(CredentialFactory.java:100)
at com.google.cloud.bigtable.grpc.io.CredentialInterceptorCache.getCredentialsInterceptor(CredentialInterceptorCache.java:85)
at com.google.cloud.bigtable.grpc.BigtableSession$6.call(BigtableSession.java:317)
at com.google.cloud.bigtable.grpc.BigtableSession$6.call(BigtableSession.java:309)
at java.util.concurrent.FutureTask.run(FutureTask.java:262)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
... 1 more
Caused by: java.lang.ClassNotFoundException: com.google.bigtable.repackaged.com.google.auth.appengine.AppEngineCredentials
at java.net.URLClassLoader$1.run(URLClassLoader.java:366)
at java.net.URLClassLoader$1.run(URLClassLoader.java:355)
at java.security.AccessController.doPrivileged(Native Method)
at java.net.URLClassLoader.findClass(URLClassLoader.java:354)
at java.lang.ClassLoader.loadClass(ClassLoader.java:425)
at java.lang.ClassLoader.loadClass(ClassLoader.java:358)
at org.eclipse.jetty.webapp.WebAppClassLoader.loadClass(WebAppClassLoader.java:450)
at org.eclipse.jetty.webapp.WebAppClassLoader.loadClass(WebAppClassLoader.java:403)
at java.lang.Class.forName0(Native Method)
at java.lang.Class.forName(Class.java:195)
at com.google.bigtable.repackaged.com.google.auth.oauth2.DefaultCredentialsProvider.forName(DefaultCredentialsProvider.java:266)
at com.google.bigtable.repackaged.com.google.auth.oauth2.DefaultCredentialsProvider.tryGetAppEngineCredential(DefaultCredentialsProvider.java:225)
... 12 more

The latest version of managed-vm-gae example uses 0.2.3 w/ netty boring SSL static.

I do have netty boring SSL in the classpath.  I was able to get the setup to work, however yesterday the servers again stopped working (Without any update to the servers). This is the gradle file
I am using. (I was using version 0.2.3. I have now retried with 0.2.3 and 0.2.4, couldnt get either to work).
appengineSdk 'com.google.appengine:appengine-java-sdk:1.9.34'
compile 'com.google.appengine:appengine-endpoints:1.9.34'
compile 'com.google.appengine:appengine-endpoints-deps:1.9.34'
compile 'io.netty:netty-tcnative-boringssl-static:1.1.33.Fork13:linux-x86_64'
compile 'com.google.auth:google-auth-library-appengine:0.3.1'
compile 'com.google.auth:google-auth-library-credentials:0.3.1'
compile 'com.google.auth:google-auth-library-oauth2-http:0.3.1'
compile 'com.google.cloud.bigtable:bigtable-client:0.2.4'
compile 'com.google.cloud.bigtable:bigtable-hbase-1.1:0.2.4'

Finally got it to work. Had to put in GOOGLE_APPLICATION_CREDENTIALS in the appengine-web.xml.

Glad it worked for you, but that shouldn't be necessary.  Is your Cloud Bigtable Cluster in a different project?  If so, you can add your project as an editor of that and get access.

Our cloud bigtable cluster is in the same project. The setup is rather
vanilla, very similar (AFAICT to the one in managed-vm-gae example. I have
tried to mimic that in our setup. I can get the example to work, but not
our setup w/o having to specify the GOOGLE_APPLICATION_CREDENTIALS var,
which does look like a hack).
The classnotfoundexception is telling though, because the package name has
bigtable.repackaged, it appears that there are some classes that should be
shaded and part of the cloud-bigtable-jars that have gotten missed.
Thanks for your responses
Vinay
On Tue, Mar 29, 2016 at 11:57 PM, Les Vogel<EMAIL_ADDRESS>wrote:

Glad it worked for you, but that shouldn't be necessary. Is your Cloud
Bigtable Cluster in a different project? If so, you can add your project as
an editor of that and get access.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/cloud-bigtable-examples/issues/145#issuecomment-203038990

@sduskis Can you suggest anything?
Do you have all the Bigtable API's enabled?  Are you using IAM?

I'll google-auth-library-appengine to our pom.xml as a shaded and included jar.  That may help.

I deployed a new version of cloud-bigtable 0.2.5-SNAPSHOT that has the google-auth-library-appengine dependency.

@sduskis Many thanks. This fixes the issue. I was able to get our setup working w/o the need for GOOGLE_APPLICATION_CREDENTIALS.

Spoke too soon...approximately 10% of the queries are giving this error
java.util.concurrent.ExecutionException: java.lang.RuntimeException: com.google.cloud.bigtable.grpc.io.IOExceptionWithStatus: Error in response stream
at java.util.concurrent.FutureTask.report(FutureTask.java:122)
at java.util.concurrent.FutureTask.get(FutureTask.java:188)
at com.yawasa.server.storage.TopicTable.freeFutures(TopicTable.java:109)
at com.yawasa.server.storage.TopicTable.get(TopicTable.java:96)
at com.yawasa.server.ReadHandler.doPost(ReadHandler.java:60)
at javax.servlet.http.HttpServlet.service(HttpServlet.java:644)
at javax.servlet.http.HttpServlet.service(HttpServlet.java:725)
at org.eclipse.jetty.servlet.ServletHolder.handle(ServletHolder.java:808)
at org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1669)
at com.google.apphosting.utils.servlet.ParseBlobUploadFilter.doFilter(ParseBlobUploadFilter.java:125)
at org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1652)
at com.google.apphosting.runtime.jetty9.SaveSessionFilter.doFilter(SaveSessionFilter.java:37)
at org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1652)
at com.google.apphosting.utils.servlet.TransactionCleanupFilter.doFilter(TransactionCleanupFilter.java:50)
at org.eclipse.jetty.servlet.ServletHandler$CachedChain.doFilter(ServletHandler.java:1652)
at org.eclipse.jetty.servlet.ServletHandler.doHandle(ServletHandler.java:585)
at org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:143)
at org.eclipse.jetty.security.SecurityHandler.handle(SecurityHandler.java:553)
at org.eclipse.jetty.server.session.SessionHandler.doHandle(SessionHandler.java:223)
at org.eclipse.jetty.server.handler.ContextHandler.doHandle(ContextHandler.java:1127)
at org.eclipse.jetty.servlet.ServletHandler.doScope(ServletHandler.java:515)
at org.eclipse.jetty.server.session.SessionHandler.doScope(SessionHandler.java:185)
at org.eclipse.jetty.server.handler.ContextHandler.doScope(ContextHandler.java:1061)
at com.google.apphosting.vmruntime.jetty9.VmRuntimeWebAppContext.doScope(VmRuntimeWebAppContext.java:299)
at org.eclipse.jetty.server.handler.ScopedHandler.handle(ScopedHandler.java:141)
at org.eclipse.jetty.server.handler.ContextHandlerCollection.handle(ContextHandlerCollection.java:215)
at org.eclipse.jetty.server.handler.HandlerCollection.handle(HandlerCollection.java:110)
at org.eclipse.jetty.server.handler.HandlerWrapper.handle(HandlerWrapper.java:97)
at org.eclipse.jetty.server.Server.handle(Server.java:497)
at org.eclipse.jetty.server.HttpChannel.handle(HttpChannel.java:310)
at org.eclipse.jetty.server.HttpConnection.onFillable(HttpConnection.java:257)
at org.eclipse.jetty.io.AbstractConnection$2.run(AbstractConnection.java:540)
at org.eclipse.jetty.util.thread.QueuedThreadPool.runJob(QueuedThreadPool.java:635)
at org.eclipse.jetty.util.thread.QueuedThreadPool$3.run(QueuedThreadPool.java:555)
at java.lang.Thread.run(Thread.java:745)
Caused by: java.lang.RuntimeException: com.google.cloud.bigtable.grpc.io.IOExceptionWithStatus: Error in response stream
at org.apache.hadoop.hbase.client.AbstractClientScanner$1.hasNext(AbstractClientScanner.java:97)
at com.yawasa.server.storage.TopicPopulator.get(TopicPopulator.java:50)
at com.yawasa.server.storage.TopicTable.get(TopicTable.java:118)
at com.yawasa.server.storage.TopicTable.access$000(TopicTable.java:41)
at com.yawasa.server.storage.TopicTable$1.call(TopicTable.java:88)
at com.yawasa.server.storage.TopicTable$1.call(TopicTable.java:85)
at java.util.concurrent.FutureTask.run(FutureTask.java:262)
at java.util.concurrent.Executors$RunnableAdapter.call(Executors.java:471)
at java.util.concurrent.FutureTask.run(FutureTask.java:262)
at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1145)
at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:615)
... 1 more
Caused by: com.google.cloud.bigtable.grpc.io.IOExceptionWithStatus: Error in response stream
at com.google.cloud.bigtable.grpc.scanner.ResultQueueEntry$ExceptionResultQueueEntry.getResponseOrThrow(ResultQueueEntry.java:66)
at com.google.cloud.bigtable.grpc.scanner.ResponseQueueReader.getNextMergedRow(ResponseQueueReader.java:69)
at com.google.cloud.bigtable.grpc.scanner.StreamingBigtableResultScanner.next(StreamingBigtableResultScanner.java:42)
at com.google.cloud.bigtable.grpc.scanner.StreamingBigtableResultScanner.next(StreamingBigtableResultScanner.java:27)
at com.google.cloud.bigtable.grpc.scanner.ResumingStreamingResultScanner.next(ResumingStreamingResultScanner.java:158)
at com.google.cloud.bigtable.grpc.scanner.ResumingStreamingResultScanner.next(ResumingStreamingResultScanner.java:44)
at com.google.cloud.bigtable.hbase.adapters.BigtableResultScannerAdapter$1.next(BigtableResultScannerAdapter.java:43)
at org.apache.hadoop.hbase.client.AbstractClientScanner$1.hasNext(AbstractClientScanner.java:94)
... 11 more
Caused by: com.google.bigtable.repackaged.io.grpc.StatusRuntimeException: UNAUTHENTICATED: Request had invalid authentication credentials.
at com.google.bigtable.repackaged.io.grpc.Status.asRuntimeException(Status.java:431)
at com.google.cloud.bigtable.grpc.scanner.ReadRowsResponseListener.onClose(ReadRowsResponseListener.java:72)
at com.google.bigtable.repackaged.io.grpc.internal.ClientCallImpl$ClientStreamListenerImpl$3.runInContext(ClientCallImpl.java:462)
at com.google.bigtable.repackaged.io.grpc.internal.ContextRunnable.run(ContextRunnable.java:54)
at com.google.bigtable.repackaged.io.grpc.internal.SerializingExecutor$TaskRunner.run(SerializingExecutor.java:154)

FWIW your managed-vm-gae example requires a google sign in...a test case which does not require a user sign in..and just rely of service account would nail the problem....
I tried it with a test case..and had production reports about failures an hour after...:(

@sduskis - I suspect it's unrelated, but we just ran across this.

@lesv: Is there anything I need to do for this bug?  I'm not sure about how managed vms work.

No - this sounds like it's something for me - at least initially - not sure what I was thinking when I asked you to look at it.
@vc13819 - you encountered issues after an hour - was it a single long running session (ie. no chance for an explicit credential refresh)?  Or were you doing multiple connections.

On Wed, Apr 6, 2016 at 12:36 AM, Les Vogel<EMAIL_ADDRESS>wrote:

No - this sounds like it's something for me - at least initially - not
sure what I was thinking when I asked you to look at it.
@vc13819 https://github.com/vc13819 - you encountered issues after an
hour - was it a single long running session (ie. no chance for an explicit
credential refresh)? Or were you doing multiple connections.

It was a single session. We are running 2 instances of the server, and they
both started failing after about an hour.
We did not do any explicit credentials refresh.
With the GOOGLE_APPLICATION_CREDENTIALS things have continued to work
without hiccups (touchwood!!)

—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/cloud-bigtable-examples/issues/145#issuecomment-205950813

There's a bug in the AppEngineCredentials handling of expiration.  There's a default expiration of about 1 hour.  That expiration information is not being passed around correctly.  I submitted a fix for that problem, so that the refreshing logic can kick in and fix the problem.

This requires the updated client to be fixed it should be in 0.2.5-SNAPSHOT now if you wish to try it.

Fixed


Update SAW TLS handshake proofs

Description of changes:
Update SAW TLS handshake proofs for recent no_client_cert change
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Cleared Cache and restarted build


Icons demo page

Fixes: #110

@skvale thanks very much for the PR! Just checking, have you run yarn flow against this as I think I see some props on Icon that shouldn't be allowed?

I'm curious why payments is a special case prefix.
I added
+prefix?: "fa" | "fe" | "flag" | "payment",

but, yes, flow did not approve of those props.

OK, I'm done making churn on this PR now 👍


feat: Implement generic traits

Description
Problem*
Resolves #3471
Summary*
Implements support for generics on the trait directly. E.g. trait Into<T> { ... }
Additional Context
The old trait_generics test has been renamed to trait_impl_generics - I think this is more accurate. There is a new test in trait_generics now which tests the generic traits added by this PR.
I've discovered two new bugs in writing this PR, which are commented in the trait_generics test. I'll make issues for them now.
Documentation*
Check one:

[ ] No documentation needed.
[x] Documentation included in this PR.
[ ] [Exceptional Case] Documentation to be submitted in a separate PR.

PR Checklist*

[ ] I have tested the changes locally.
[ ] I have formatted the changes with Prettier and/or cargo fmt on default settings.

Issue/PR #4000 🎉

Ah, forgot to update some code I commented out. PR is not done yet, apologies.

Just checked to avoid duplicating work. This fixes https://github.com/noir-lang/noir/issues/3474 !

@Thunkar I didn't finish rewriting the check for the parameters / return type of a trait method yet at the time. I tested again after fixing the trait_impl_generics test and #3474 is fixed now in this PR

@kevaundray @TomAFrench @vezenovm, this PR is ready to review again

One question, this fails on me with the code in this PR:
trait Serializable<N> {
    fn serialize(self) -> [Field; N];
}

global DATA_LEN = 2;

struct Data {
    a: Field,
    b: Field,
}

impl Serializable<DATA_LEN> for Data {
    fn serialize(self) -> [Field; DATA_LEN] {
        [self.a, self.b]
    }
}

fn sum<T, N>(data: T) -> Field where T: Serializable<N> {
    let serialized = data.serialize();
    serialized.fold(0, |acc, elem| acc + elem)
}

fn main(data: Data) -> pub Field {
    sum(data)
}

It generates invalid SSA:
Initial SSA:
acir fn main f0 {
  b0(v0: Field, v1: Field):
    v3 = call f1(v0, v1)
    return v3
}
acir fn sum f1 {
  b0(v0: Field, v1: Field):
    v3, v4 = call f2(v0, v1)
    inc_rc v4
    inc_rc v4
    v8 = call f3(v3, v4, Field 0, f4)
    return v8
}
acir fn serialize f2 {
  b0(v0: Field, v1: Field):
    return [v0, v1]
}
acir fn fold f3 {
  b0(v0: Field, v1: [Field], v2: Field, v4: function):
    v3 = allocate
    store v2 at v3
    inc_rc v1
    inc_rc v1
    jmp b1(Field 0)
  b1(v5: Field):
    v8 = lt v5, v0
    jmpif v8 then: b2, else: b3
  b2():
    v10 = cast v0 as u64
    v11 = lt v5, v10
    constrain v11 == u1 1 'Index out of bounds'
    v13 = array_get v1, index v5
    v14 = load v3
    v15 = call v4(v14, v13)
    store v15 at v3
    v16 = add v5, Field 1
    jmp b1(v16)
  b3():
    v17 = load v3
    return v17
}
acir fn lambda f4 {
  b0(v0: Field, v1: Field):
    v2 = add v0, v1
    return v2
}

It thinks that serialize returns a slice instead of a fixed length array

Thanks for the test @sirasistant, it turned out to be a very good one since it highlighted a lot of requirements I hadn't yet done. I had thought I was missing something since the original PR was quite small, and indeed I was missing a lot. I also thought the Into<T> test was sufficient, but your example highlights the need to test more. I added your example as a test, and added a variant of it where the trait method is called with a static method which gave me some additional issues while testing that are resolved now.
This PR should be ready for review now.


SW-2600 Handle deleted species in list withdrawals wrapper

show  just like we do in the inventory table

LGTM! Thanks

Thanks, Charlie!


Firestore uses or overrides a depreacated API /unchecked or unsafe operations. PathProvider Plugin uses or override a deprecated API , SharedPreferences Plugin has been deprecated

I ma developing an app with firestore, pathprovider , sqflite db, and sharedpreferences ...  and I am using minSDKVersion 23, I don't know why it shows up after i do flutter clean ..
Note: F:\Flutter\flutter\.pub-cache\hosted\pub.dartlang.org\cloud_firestore-0.12.11\android\src\main\java\io\flutter\plugins\firebase\cloudfirestore\CloudFirestorePlugin.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: F:\Flutter\flutter\.pub-cache\hosted\pub.dartlang.org\cloud_firestore-0.12.11\android\src\main\java\io\flutter\plugins\firebase\cloudfirestore\CloudFirestorePlugin.java uses unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.
Note: F:\Flutter\flutter\.pub-cache\hosted\pub.dartlang.org\path_provider-1.6.0\android\src\main\java\io\flutter\plugins\pathprovider\PathProviderPlugin.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
F:\Flutter\flutter\.pub-cache\hosted\pub.dartlang.org\shared_preferences-0.5.6+1\android\src\main\java\io\flutter\plugins\sharedpreferences\SharedPreferencesPlugin.java:25: warning: [deprecation] getFlutterEngine() in FlutterPluginBinding has been deprecated
    setupChannel(binding.getFlutterEngine().getDartExecutor(), binding.getApplicationContext());
                        ^
1 warning

and here is my output for  flutter doctor
[ظê?] Flutter (Channel stable, v1.12.13+hotfix.7, on Microsoft Windows [Version 6.3.9600], locale en-US)
[ظê?] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
[ظê?] Android Studio (version 3.5)
[ظê?] Connected device (1 available)

 No issues found!

same issue

Hi @sunaarun
Please look at this answer
Closing, as this isn't an issue with Flutter itself,
if you disagree please write in the comments and I will reopen it
Thank you

Hi @sunaarun
Please look at this answer
Closing, as this isn't an issue with Flutter itself,
if you disagree please write in the comments and I will reopen it
Thank you

Hey @iapicca  , I see this warning only after do flutter clean , I don't know why this appear to me ..
and I tried to search for a solution to remove this warning , all i did doesn't remove the warning ..
and I saw the answer , what i understand is to update the plugins i used in app ?! ..
please can you help me and explain to me

I have this too.  I tried enabling Lint to get more details, but that didn't work.  A flutter clean is required to see this.

I think the deprecation warning is related to getFlutterEngine being deprecated.


Possible mistake in _triage_segments function

Is your feature request related to a problem? Please describe.
In the line 1965 (in the function "_triage_segments") in in scipy/signal/_spectral_py.py, win = get_window(window, nperseg) should be win = get_window(window, nperseg, False). Without the "False", the output comes a bit weird as the window is periodic. The window should be symmetric instead.
Describe the solution you'd like.
No response
Describe alternatives you've considered.
No response
Additional context (e.g. screenshots, GIFs)
No response

Hi @Rakib1503052 thanks for reporting. Can you post some details with code execution maybe? Would you like to make a PR to fix it as well?

In current version of scipy, this code
x = np.arange(90) _,y = welch(x,fs=1.0,window='hamm',nperseg=20,noverlap=10,nfft=20,detrend=False,average='mean')
Produces,
[ 37324.4892, 13844.9124, 1.63003019, 0.192535571, 0.257295057, 0.240885088, 0.212325371,    0.188962883, 0.173046561, 0.163979745, 0.0805234021 ]
The above call is equivalent to this call in MATLAB, [y,f] = pwelch(x,20,10,20,1.0) which produces
35443.9998544396 14803.1319632977 23.1801433919644 2.59088336922622 0.863454188318468 0.452279330900834 0.298664399597121 0.226918416387484 0.190156005988099 0.172135278038071 0.0833391264471670
With the fix in PR #15593, the same call gives,
[ 35443.9999, 14803.1320, 23.1801434, 2.59088337, 0.863454188, 0.452279331,    0.298664400, 0.226918416, 0.190156006, 0.172135278, 0.0833391264 ]


subdag content is not executed by trigger_dag

HI, I think I got a problem with triggered dag containing a subdag
I am using airflow 1.6.1
python3 / celery / mysql
when I trigger my dag like this
airflow trigger_dag all_reload
the subdag within 'all_reload' does not get scheduled.
the sceduler give me some
[celery] queuing ('all_reload', 'test_reload_data', datetime.datetime(2015, 12, 7, 18, 24, 20)) through celery, queue=default
[...]
2015-12-07 18:45:31,052 - root - INFO - Loaded DAG <DAG: all_reload.test_reload_data>
2015-12-07 18:45:31,052 - root - INFO - Loaded DAG <DAG: all_reload>
2015-12-07 18:45:31,069 - root - INFO - Checking state for <DagRun all_reload @ 2015-12-07 18:24:20: None, externally triggered: True>
2015-12-07 18:45:31,074 - root - INFO - Marking run <DagRun all_reload @ 2015-12-07 18:24:20: None, externally triggered: True> successful

The subdag is marked has scheduled but nothing happens and the scheduler mark the job as successfull.
I have other subdags on the same config (daily scheduled) and everything is running fine.
Here is an example dag definition to reproduce the problem:
from datetime import datetime, timedelta
import time

from airflow import DAG
from airflow.operators import DummyOperator, SubDagOperator, PythonOperator

def sleep_2():
    print("sleep 2")
    time.sleep(2)

default_args = {
    'owner': 'airflow',
    'depends_on_past': False,
    'start_date': datetime.now() - timedelta(days=1),
    'email'<EMAIL_ADDRESS>    'email_on_failure': False,
    'email_on_retry': False,
    'retries': 0,
    'retry_interval': timedelta(minutes=1),
}

def build_reload():

    main_dag = DAG('all_reload', default_args=default_args, schedule_interval=None)

    global_reload = DummyOperator(
        task_id='global_reload',
        dag=main_dag
        )

    subname = 'test_reload_data'

    task = PythonOperator(
        task_id='sleep_for_2',
        python_callable=sleep_2,
        dag=main_dag)
    task.set_upstream(global_reload)

    dag2 = DAG('all_reload.' + subname, default_args=default_args, schedule_interval=None)

    PythonOperator(
        task_id='sleep_for_2_sub',
        python_callable=sleep_2,
        dag=dag2)

    update_ui = SubDagOperator(
        task_id=subname,
        dag=main_dag,
        subdag=dag2,
        )
    update_ui.set_upstream(task)

    return main_dag

main_dag = build_reload()

Well I just saw here
https://medium.com/handy-tech/airflow-tips-tricks-and-pitfalls-9ba53fba14eb#.x3wfdr7ei

Even though subdags are triggered as part of a larger dag, if their schedule is set to None or ‘@once’, the subdag operator will succeed without doing anything.

maybe this is my issue, I'll check this again, anyway It's a strange behavior.

@jlowin @thibault-ketterer can you guys please create a Jira for this if it is still an open issue?


fix(#72): correct openapi spec response types

closes #72

This makes the spec match the implementation. If we had noticed this earlier, I might have opted for the other direction, but frontends are already built.


chore: update permissions

What this PR does / why we need it:
Introduces top level read-only permissions in the CI workflow
Moves write permissions to within the single job (kind of a vanity change but stops certain security tooling from complainting)
Which issue(s) this PR fixes:

part of https://github.com/grafana/plugin-tools/issues/742
Special notes for your reviewer:
Tested with a custom plugin and CI/Release still appeared to execute as expected.
CI workflow may require more changes as per the issue

(Maybe it would be useful to automatically open a PR in the https://github.com/grafana/plugin-workflow-tester/ repo if any of the workflow template files change to see if they pass. However as we don't edit these too often maybe it doesn't worth the effort.)

(Maybe it would be useful to automatically open a PR in the https://github.com/grafana/plugin-workflow-tester/ repo if any of the workflow template files change to see if they pass. However as we don't edit these too often maybe it doesn't worth the effort.)

Nice, did not know we had this repo.

Hmm, maybe I am doing something wrong, but this didn't work for me (failing workflow). Are you sure read-only is a valid permission value? As far as I can see here to define access for all the available scopes we can either use read-all or write-all.

🤦


update font family for serial icon to brand-icons

fixes https://github.com/microsoft/pxt-microbit/issues/4499
This icon is now under a new font family, 'brand-icons' instead of 'Icons' so we override the font family for this element.

Thanks for all your help, @riknoll!


How to star the application?

I want to know the architecture of the project,and how to star?

star？not start？


Files pyclariuscast.so and libcast.so Cast Python Exmples

We just started using the Clarius US probes and wanted to read out frames in Python on a Windows machine.
In the Cast Python example the following step is given:

Copy the python programs to the extracted libs folder (where pyclariuscast.so and libcast.so are placed).

Where could we find the pyclariuscast.so and libcast.so files?

Hi, download the binaries from the release section https://github.com/clariusdev/cast/releases. Thanks!

I've downloaded the cast-11.0.0-windows.x64.zip binary and the source code zip and still can't find pyclariuscast.so and libcast.so

Shares object or .so files are for Unix like operating systems, you'll find dll and lib files only in Windows operating system packages

It appears that pyclariuscast.so is missing in the ubuntu 22 released binaries for version 11.1.

The Linux packages were updated in the release section to include the missing pyclariuscast.so files--thank you for finding the issue. @AriBunnell you can re-download the Linux packages from the release section and try again.


OnAppeared

[Enter feedback here]

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

ID: e2969e5f-e192-94e3-223a-e9f354503104
Version Independent ID: 190a6979-1884-ec70-a9b4-a8e41b5a4423
Content: MasterDetailPage.OnAppearing Method (Xamarin.Forms)
Content Source: docs/Xamarin.Forms/MasterDetailPage.xml
Product: xamarin
Technology: xamarin-forms
GitHub Login: @dotnet-bot
Microsoft Alias: dotnetcontent

@zdananic23 Can you explain more about the issue with the docs you are seeing?
If this is an issue with Xamarin.Forms or an enhancement request please log your feedback here
https://github.com/xamarin/Xamarin.Forms

Microsoft support for Xamarin will end on May 1, 2024 for all Xamarin SDKs including Xamarin.Forms. In preparation for this, all issues and PRs in this repository are being closed.
.NET MAUI is the replacement for Xamarin.Forms, and its API docs are built from /// in the dotnet/maui repository. Therefore, any issues with the .NET MAUI API docs should be opened in that repository.


Fix marshalling complete secrets

How to categorize this PR?

/area control-plane
/kind bug
/priority 3
What this PR does / why we need it:
This PR fixes issue #398 (Problem with marshalling complete secrets).
Which issue(s) this PR fixes:
Fixes #398
Special notes for your reviewer:
Release note:

- bugfix concerning the import of a complete secret or config map

Example resources (installations ,secrets, config maps) can be found here for the following scenarios:
an installation imports values

from a secret,
from the data of a config map,
from the binary data of a config map;

and for each of these cases,

the complete data are bound to a single import parameter,
or each item is bound to a separate import parameter.


[SPARK-46894][PYTHON][FOLLOW-UP] Includes error-conditions.json into PyPI package

What changes were proposed in this pull request?
This PR is a followup of https://github.com/apache/spark/pull/44920 that includes error-conditions.json into PyPI package.
Why are the changes needed?
So PyPI packages work with pip install. https://github.com/apache/spark/actions/runs/8913716987/job/24479781481 is broken for now.
Does this PR introduce any user-facing change?
No, the main change has not been released out yet.
How was this patch tested?
Manually tested the packaging, and check the file exists in the package.
Was this patch authored or co-authored using generative AI tooling?
No.

cc @nchammas FYI

and @itholic too

There is a test at https://github.com/apache/spark/blob/master/dev/pip-sanity-check.py but that only runs a basic test that does not import pyspark.errors whereas https://github.com/apache/spark/actions/workflows/build_python_connect.yml tests its full compatibility (only for pyspark-connect package though.

Oh, much simpler.

Merged to master.
It will be tested in the scheduled job.


cups-browsed.c: Introduce NotifLeaseDuration directive

The PR implements two things:

NotifLeaseDuration directive for cups-browsed.conf - it will make
lease duration for notifications configurable by users. IMO it is not
useful for regular users, but it is helpful during sanity testing
(for verifying that we actually renew the subscription when time
comes). The current hardcoded 1 day is unusuable for that :( .
I implemented the lowest threshold to 300s to prevent a possible DoS.
Subscription renewal is set to happen in the middle of NotifLeaseDuration,
not one minute before lease expiration. This was a problem on busy servers,
where cups-browsed was busy and wasn't able to renew the subscription
before cupsd removed it. Then if some jobs had come before the subscription
was created again, the queue got disabled. The proposed approach is based
on behavior of DHCP.

Sorry for the noise, I'll return with the PR tomorrow.


Google Sign In / Android - missing Guava Proguard rules

When trying to build release, I am getting proguard errors:
Note: the configuration refers to the unknown class 'com.google.android.gms.common.api.internal.BasePendingResult$ReleasableResultGuardian'
Warning: com.google.common.util.concurrent.FuturesGetChecked$GetCheckedTypeValidatorHolder$ClassValueValidator$1: can't find superclass or interface java.lang.ClassValue
Warning: com.google.common.base.AbstractIterator: can't find referenced class com.google.errorprone.annotations.CanIgnoreReturnValue
Warning: com.google.common.base.Converter: can't find referenced class com.google.errorprone.annotations.concurrent.LazyInit
Warning: com.google.common.base.Converter: can't find referenced class com.google.errorprone.annotations.CanIgnoreReturnValue
Warning: com.google.common.base.Converter: can't find referenced class com.google.errorprone.annotations.CanIgnoreReturnValue
Warning: com.google.common.base.Converter: can't find referenced class com.google.errorprone.annotations.CanIgnoreReturnValue
Warning: com.google.common.base.Converter: can't find referenced class com.google.errorprone.annotations.CanIgnoreReturnValue
Warning: com.google.common.base.Function: can't find referenced class com.google.errorprone.annotations.CanIgnoreReturnValue
Warning: com.google.common.base.Joiner: can't find referenced class com.google.errorprone.annotations.CanIgnoreReturnValue
Warning: com.google.common.base.Joiner: can't find referenced class com.google.errorprone.annotations.CanIgnoreReturnValue
Warning: com.google.common.base.Joiner: can't find referenced class com.google.errorprone.annotations.CanIgnoreReturnValue
Warning: com.google.common.base.Joiner: can't find referenced class com.google.errorprone.annotations.CanIgnoreReturnValue
Warning: com.google.common.base.Joiner: can't find referenced class com.google.errorprone.annotations.CanIgnoreReturnValue
Warning: com.google.common.base.Joiner: can't find referenced class com.google.errorprone.annotations.CanIgnoreReturnValue
Warning: com.google.common.base.Joiner: can't find referenced class com.google.errorprone.annotations.CanIgnoreReturnValue

Warning: there were 696 unresolved references to classes or interfaces.
         You may need to add missing library jars or update their versions.
         If your code works fine without the missing classes, you can suppress
         the warnings with '-dontwarn' options.
         (http://proguard.sourceforge.net/manual/troubleshooting.html#unresolvedclass)

FAILURE: Build failed with an exception.

I need to create proguard file and add Guava 20 rules:
-dontwarn com.google.common.base.**
-keep class com.google.common.base.** {*;}
-dontwarn com.google.errorprone.annotations.**
-keep class com.google.errorprone.annotations.** {*;}
-dontwarn   com.google.j2objc.annotations.**
-keep class com.google.j2objc.annotations.** { *; }
-dontwarn   java.lang.ClassValue
-keep class java.lang.ClassValue { *; }
-dontwarn   org.codehaus.mojo.animal_sniffer.IgnoreJRERequirement
-keep class org.codehaus.mojo.animal_sniffer.IgnoreJRERequirement { *; }

to build release with proguard enabled (which is default currently). You need to fix plugin sources and add proguard rules there.
BTW. You actually should not use Guava, as most Java 8 features is available now by desugaring: https://developer.android.com/studio/write/java8-support or simply use Kotiln.

Hi @giaur500
Can you please provide your flutter doctor -v, your pubspec.yaml and the steps you did to build your app that shows the logs?
Thanks

flutter doctor -v
[✓] Flutter (Channel stable, 1.22.1, on Linux, locale pl_PL.UTF-8)
    • Flutter version 1.22.1 at /home/michal/android/flutter
    • Framework revision f30b7f4db9 (7 days ago), 2020-10-08 10:06:30 -0700
    • Engine revision 75bef9f6c8
    • Dart version 2.10.1

[✓] Android toolchain - develop for Android devices (Android SDK version 29.0.2)
    • Android SDK at /home/michal/Android/Sdk
    • Platform android-29, build-tools 29.0.2
    • ANDROID_HOME = /home/michal/Android/Sdk
    • Java binary at: /home/michal/android/studio/jre/bin/java
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)
    • All Android licenses accepted.

[✓] Android Studio (version 4.0)
    • Android Studio at /home/michal/android/studio
    • Flutter plugin version 50.0.1
    • Dart plugin version 193.7547
    • Java version OpenJDK Runtime Environment (build
      1.8.0_242-release-1644-b3-6222593)

[✓] VS Code (version 1.50.1)
    • VS Code at /usr/share/code
    • Flutter extension version 3.12.2

[✓] Connected device (1 available)
    • Android SDK built for x86 64 (mobile) • emulator-5554 • android-x64 •
      Android 10 (API 29) (emulator)

• No issues found!

publspec.yaml - default as app created, only added google sign in dependency:
google_sign_in: ^4.5.5
To build, use:
flutter build apk --split-per-abi

Hi @giaur500
Thanks for providing that, I'm still not able to reproduce the issue with the same configuration.
Can you please provide your /android/app/build.gradle and the complete output of your flutter build apk --split-per-abi?
If the logs are too long you can provide it in a .txt file.

Complete build output:
https://pastebin.pl/view/351569a2
build.gradle:
def localProperties = new Properties()
def localPropertiesFile = rootProject.file('local.properties')
if (localPropertiesFile.exists()) {
    localPropertiesFile.withReader('UTF-8') { reader ->
        localProperties.load(reader)
    }
}

def flutterRoot = localProperties.getProperty('flutter.sdk')
if (flutterRoot == null) {
    throw new GradleException("Flutter SDK not found. Define location with flutter.sdk in the local.properties file.")
}

def flutterVersionCode = localProperties.getProperty('flutter.versionCode')
if (flutterVersionCode == null) {
    flutterVersionCode = '1'
}

def flutterVersionName = localProperties.getProperty('flutter.versionName')
if (flutterVersionName == null) {
    flutterVersionName = '1.0'
}

apply plugin: 'com.android.application'
apply from: "$flutterRoot/packages/flutter_tools/gradle/flutter.gradle"

android {

    signingConfigs {
        release {
            keyAlias 'key'
            keyPassword '123abc'
            storeFile file('../store.jks')
            storePassword '123abc'
        }
    }

    compileSdkVersion 29

    lintOptions {
        disable 'InvalidPackage'
    }

    defaultConfig {
        // TODO: Specify your own unique Application ID (https://developer.android.com/studio/build/application-id.html).
        applicationId "my.app"
        minSdkVersion 16
        targetSdkVersion 29
        versionCode flutterVersionCode.toInteger()
        versionName flutterVersionName
        testInstrumentationRunner "androidx.test.runner.AndroidJUnitRunner"
    }

    buildTypes {
        release {
            //minifyEnabled=false
            //shrinkResources=false
            signingConfig signingConfigs.release
            proguardFiles getDefaultProguardFile('proguard-android.txt'), 'proguard-rules.pro'
        }
    }
}

flutter {
    source '../..'
}

dependencies {
    testImplementation 'junit:junit:4.12'
    androidTestImplementation 'androidx.test:runner:1.1.0-alpha4'
    androidTestImplementation 'androidx.test.espresso:espresso-core:3.1.0-alpha4'
}

I still cannot reproduce this issue, this will need further investigation.

I don't know how can I help. I am always experiencing that with Guava + Proguard. Now as I can see, your plugin uses Guava without any Proguard configuration, so I think that may be the same problem.


[spark] Fix Comaptibility in getting NamedReference from spark32 to spark35

Purpose
Fix Comaptibility in getting NamedReference from spark32 to spark35 with a common api to get NamedReference
spark3.3+ ：

spark3.2：

then sql would error out as ：

Linked issue: close #xxx

Tests

API and Format

Documentation

Try to PaimonPushDownTest => PaimonPushDownTestBase, add this to 3.2

Paimon 3.2 version is from 3.2.4 and 3.2.4 is no problem about it. this is 3.2.4 and 3.2.0 version code


Development Environment missing commands

Dear Zipline Maintainers,
Before I tell you about my issue, let me describe my environment:
Environment

Operating System: Linux bones-alphai 4.8.0-39-generic #42~16.04.1-Ubuntu SMP Mon Feb 20 15:06:07 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux
Python Version: 2.7 (in a virtualenv)
Python Bitness: 64
How did you install Zipline:  from the source code

Now that you know a little about me, let me tell you about the issue I am
having:
Description of Issue
I was setting up Zipline on dev environment (based on release 1.0.2 working dir) by following the instruction on the wiki.
After installing everything I ran thenosetest command
nosetests --with-timer --with-coverage --cover-package=zipline --timer-top-n=1

What did you expect to happen?
Test passing
What happened instead?
I got two failures

ImportError: No module named talib
ImportError: No module named blaze

What steps have you taken to resolve this already?
To fix them, I had to execute the following commands:
pip install -r ./etc/requirements_blaze.txt
pip install -r ./etc/requirements_talib.txt

It would be nice to have them in the documentation
Regards
Daniele

Hey @bonesmccoy thanks for opening up this issue. I was considering moving the dev environment info from the wiki to zipline.io; it'll take some time to do this on my end as I'm looking at some other stuff right now though feel free to submit a PR that fixes this

Sorry for the silly question: how do I submit a PR for updating the wiki ? thanks

Oh, sorry if my response was vague. I meant that if you want to move the dev environment info to the zipline docs (on http://zipline.io), you could submit a PR for that (I'd like to move the dev environment docs myself to zipline.io but I'm currently looking at some other issues)

Thanks for the explanation. I'll work on it!


Add v2.7.1 image of aws load balancer controller

Please select the kind of this pull request, e.g.:
/kind enhancement
What this PR does / why we need it:
Which issue(s) this PR fixes:
Special notes for your reviewer:
Related to https://github.com/gardener/gardener-extension-provider-aws/pull/886

/ok-to-test
/approve


HIVE-26957: Add convertCharset(s, from, to) function

What changes were proposed in this pull request?

The function converts the string s from the from charset to the to charset.It is already implemented in clickhouse.
Why are the changes needed?

Does this PR introduce any user-facing change?

How was this patch tested?

There are also some unused imports which should be removed. It's also a good idea to check the code smells and resolve them whenever possible.

@SourabhBadhya Hello, I am a contributor to hive for the first time. Could you please help me review the code? Thank you.

Please consider adding the following test cases -

Null and empty string input
Null, empty and Invalid "from" character set input
Null, empty and Invalid "to" character set input

For the cases which throw an exception - Consider writing negative test cases using TestNegativeLlapLocalCliDriver or by writing Junit tests.

Please consider adding the following test cases -

Null and empty string input
Null, empty and Invalid "from" character set input
Null, empty and Invalid "to" character set input
1/2 arguments given for convertCharset() UDF.

For the cases which throw an exception - Consider writing negative test cases using TestNegativeLlapLocalCliDriver or by writing Junit tests.

This is a good idea, I will modify the code according to your suggestion.


refactor: Improve tedge_api structure

Proposed changes

Clarify tedge_api with sub-modules for measurements, commands and the store.
Reduce the visibility of private structs

Types of changes

[ ] Bugfix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Improvement (general improvements like code refactoring that doesn't explicitly fix a bug or add any new functionality)
[ ] Documentation Update (if none of the other choices apply)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Paste Link to the issue

Checklist

[x] I have read the CONTRIBUTING doc
[x] I have signed the CLA (in all commits with git commit -s)
[x] I ran cargo fmt as mentioned in CODING_GUIDELINES
[x] I used cargo clippy as mentioned in CODING_GUIDELINES
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Further comments

Some tests are flaky.

In this run , "Migrate Legacy Configuration Files" is falling.
In that one, the whole test suite on "Configuration Lock File" is failing.


Required MIT License.

add MIT license to the project.

year: 2021
name: harshal kaigaonkar
is that correct ?

thanks, man.

@RohanTheProgrammer your welcome sir
That was a pretty fast response BTW


Allow filters to be predefined, or have all set by default

With the changes in #14 , the filter now defaults to America only. So when we need our end users to test, we have to get them to manually check the filters, increasing the number of steps... They're also not sure what they're ticking -- they're end users.
Can the filters be set using a GET or Hash identifier? This will more allow us to provide pre-configured links to our end users and get results more easily.

Thanks for the feedback, will add it next week.


Added function to improve platform compatibility

Platform
Operating System: Windows 11
Python Version: 3.12.7
Selenium Version: 4.25.0
Summary
I added a function _get_platform_dependent_params to make the package compatible with other platforms. When I first tried this package, I encountered errors coming from get_firefox_installation_path, get_undetected_firefox_path, patch_libxul_file, and the Firefox class constructor. The errors stem from hard-coded strings referring to Firefox in the Linux OS. The function that I added will help to at least put all the hard-coded paths in one place and adapt values based on the user platform.
I think this also fixes #1 although I wasn't able to test it out in MacOS. The changes should be minimal if it still doesn't work in it.
Aside from this, I also made a minor change in the constants in which the Firefox path key in "windows" is changed to "win32".

The changes look pretty good to me!
I will test this on a Windows machine, but I am unable to test on macOS as I do not have access to a macOS device, and emulating it is quite ... problematic.
Thanks for contributing to this project, I will merge it as soon as I am able to test it on Windows

I have just re-run on my Mac, it logged "FileNotFoundError: Could not find libxul.dylib (What the hell?!)"

If possible, could you specify the location of this file within your Firefox installation? A quick search tells me that the file is in "/Applications/Firefox.app/Contents/MacOS". However, I'm not familiar with Mac OS, hence I'm not sure if the path is correct or not

There's no file named "libxul.dylib" in "/Applications/Firefox.app/Contents/MacOS"

Okay, got it! I'll try to research more about this. It may take a while though since I don't have direct access to a Mac.


Demo using binder

Create a binder config to run on mybinder, allowing people to try it out easily.

I guess we can set up a JupyterLite demo directly, instead of Binder?
We can follow the same approach as https://github.com/jupyterlite/demo and deploy to GitHub Pages via GitHub Actions.

OK, I will take a look.


Tool suggestion: identify elements adjacent to those matching a predicate

I just came up with another function that I don't think already exists and might make a useful addition to more-itertools: identifying elements in an iterable that match a predicate or are adjacent to those matching a predicate.  The design I have in mind returns a tuple for each element in the iterable with a boolean indicating whether it is or is adjacent to a "selected" element, as well as the element itself.
>>> list(adjacent(lambda c: c in 'aeiou', 'thursday'))
[(False, 't'), (True, 'h'), (True, 'u'), (True, 'r'), (False, 's'), (True, 'd'), (True, 'a'), (True, 'y')]

In my application it's important to know where the (equivalents of the) vowel-centered groups end and begin, in addition to knowing which elements are in those groups, so I pass the result of adjacent() through groupby(). (This is akin to choosing context lines in a differ.) If I wanted an iterable of just ['h', 'u', 'r', 'd', 'a', 'y'] in that example, I could use instead use filter() and map() or a generator, i.e. (e[1] for e in adjacent(...) if e[0]). I think this flexibility is important.
The question I want to bring up before sending in a pull request is how to generalize this. My base implementation is the following:
def adjacent(predicate, iterable):
    i1, i2 = tee(iterable)
    selected = chain([False], map(predicate, i1), [False])
    adjacent = map(any, windowed(selected, 3))
    return zip(adjacent, i2)

(designed to avoid calling predicate() more than once per item) It's easy enough to change the number of elements of "context" by increasing the second argument to windowed(), and that would be a straightforward generalization. Is it also worthwhile to support arbitrary "masks" by using stagger() instead of windowed()? E.g. passing offsets=(-1, 1) to "mark" only elements which are before or after those which satisfy the predicate, leaving out the ones which do satisfy the predicate themselves? Or offsets=(0,1,2) to "mark" items which satisfy the predicate and the two that follow them?

My first impression was that this was a straightforward application of windowed. However, after playing with it for a bit in the interpreter, I couldn't come up with a more elegant method than yours; this is tricky to get right without calling predicate more frequently.
So, I'm open to a PR for this. For generalizations, I'm good with an optional n argument (I guess verify that it's at least 3?).

Yeah, sounds like you went through the same thought process I did. It seemed easy at first but was not so straightforward to get right. I'll get that PR started shortly.
Incidentally, I wonder if there's a use for a "companion" itertool which combines the (bool, item) pairs into groups. I expect a common, if not the most common, use case for adjacent() is to iterate over groups of items which are within a given distance of one that satisfies the predicate; this would be used in generating a contextual diff, for example, and it's also how I use it in the application that inspired me to come up with this algorithm. But the logic to convert the (bool, item) pairs into groups is itself a little messy, because we don't want to bool to appear in the group. Here's what I've come up with:
def adjacent_groups(predicate, iterable):
    for key, group in groupby(adjacent(predicate, iterable), lambda t: t[0]):
        yield (key, (t[1] for t in group))

Come to think of it, this could be generalized into another itertool, basically a groupby() wrapper that allows the user to customize what appears in the group as well as what is used for the key.
def better_groupby(iterable, keyfunc=None, valuefunc=None):
    for key, group in groupby(iterable, keyfunc):
        yield key, map(valuefunc, group)

Then adjacent_groups() could just be implemented as
better_groupby(adjacent(predicate, iterable), lambda t: t[0], lambda t: t[1])

(or with operator.itemgetter()). I'm not sure if this warrants inclusion in more-itertools, since it's not hard to do on a case-by-case basis, but personally I would find it really handy to have this function available for code readability, if nothing else. And even if you don't want the generic better_groupby() as an itertool (under a different name of course), I think there is a good case for something like adjacent_groups() simply because of how common I expect that usage to be.

I'm positive on better_groupby (maybe groupby_transform?), but I'd say leave adjacent_groups as an exercise for the reader?


Objectives

Hi all and welcome to the new better more organised GitHub repo!
I created a list of the things I would like to get done in this summer work but please feel free to voice opinions and add to it!
I will expand on each point as we go but this is the general “Objectives” issue of the work!

[ ] New git repo with improved scripts
[ ] Improved write up of the project work
[ ] Our own line model (inspo relxill)
[ ] Look more into degeneracies
[ ] Write a degeneracies—finder tool

It would be fantastic if "Improved write up of the project work" could translate into a journal article, e.g., in MNRAS. I will create a template and upload to the repository soon.

It would be fantastic if "Improved write up of the project work" could translate into a journal article, e.g., in MNRAS.

That is the optimal outcome yes! There’s a few things I would really like to expand on — like degeneracies presented self-consistently and more real-life analysis of the high spin high inclination cases. Do you think we could compare with some actual data sets? I also want to do more Overlaying to actually compare our outcomes against literature I think that would be super fun and interesting!


[cxx-interop][Runtime] Initialize metadata of a Swift array of C++ references correctly

This fixes a crash at runtime when destroying a Swift array of values of a C++ foreign reference type.
Swift optimizes the amount of metadata emitted for _ContiguousArrayStorage<Element> by reusing _ContiguousArrayStorage<AnyObject> whenever possible (see getContiguousArrayStorageType). However, C++ foreign reference types are not AnyObjects, since they have custom retain/release operations.
This change disables the _ContiguousArrayStorage metadata optimization for C++ reference types, which makes sure that swift_arrayDestroy will call the correct release operation for elements of [MyCxxRefType].
rdar://127154770

@swift-ci please test

@swift-ci please test

Assuming we don't care about ABI, LGTM.
Wondering what kind of ABI change do we expect for foreign reference types from this PR?

Assuming we don't care about ABI, LGTM.

Wondering what kind of ABI change do we expect for foreign reference types from this PR?

Someone who knows more about the Swift runtime and ABI should comment, but I think this change might cause FRTs to lose some protocol witnesses. There might be some other edge cases too beyond the runtime implications.

As far as I can tell, this function is only used for runtime casting decisions. This shouldn't be an ABI break, exactly, but some casts that previously succeeded will now fail, and casts to AnyObject will presumably produce a boxed value now instead of just producing the value verbatim. This should be OK, because almost any real-world use of such a cast would have crashed when it tried to retain/release the C++ reference.

@swift-ci please test

@aschwaighofer I had to tweak the getContiguousArrayStorageType<AnyObject> logic in SILOptimizer to make this work for optimized builds, could you please look over to check if I did it right?


Enable GPU test runs

We'd like to be able to run test on GPU (even if that is only manual for now).

https://github.com/maciejkula/spotlight/pull/35


[SYMBOL] Enable Get Index from Symbol

it does not make too much sense to get index from a grouped symbol, an error should be raised, @ZihengJiang can you followup a pr to fix this?


error in 1.8

Version of Boardwalk:1.1
Version of Android:5.0.2
Device model:Sony xperia Z1 C6903
Description of the problem:when I hit any mobs the world be comes black

Can you help me

@zhuowei

Help


Fix races in unit tests

What this PR does / why we need it:
Fix some races in the unit tests to finally allow enabling the code coverage checks again, and to detect race issues in the code.
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

NONE

/test

/test

/test pull-kubevirt-goveralls

/test pull-kubevirt-goveralls

/cc @dhiller

/cc @dhiller

/test pull-kubevirt-goveralls

/test pull-kubevirt-goveralls

/close
@dhiller is taking over.


Change the ID of InputGetVerifiedSenders from int to int64

To ensure that IDs can be handled uniformly in int64

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

Comparison is base (4b55044) 67.39% compared to head (49a7b6e) 67.39%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #21   +/-   ##
=======================================
  Coverage   67.39%   67.39%
=======================================
  Files           9        9
  Lines         782      782
=======================================
  Hits          527      527
  Misses        180      180
  Partials       75       75

Flag
Coverage Δ

unittests
67.39% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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


Avoid unnecessary Vec allocation when executing GraphQLBatchRequest

This is a minor improvement which eliminates unnecessary Vec collection of GraphQLRequests before feeding them into future::join_all(), which is OK with accepting Iterators anyway.
Additionally:

Makes GraphQLBatchRequest/GraphQLBatchResponse code a bit more laconic.

Thank you!


Only boot backup server for backups

Modify pre- and post-backup scripts in restic_local so the pre-backup script wakes the backup server and the post-backup one shuts it off.

Backup server will also serve as a monitoring server in the future, so it should not be taken offline.


Cleaning up old modules

I've made a number of updates to older modules so that they're consistent with how the newer ones act/feel:

I deleted file name patterns that aren't being used
Deleted vt_climatology.py because everything it does can be done by vt_time_agg.py
Deleted vt_seas_vars.py because the script underlying it is crazy (if I get time later in the week I'll add seas_vars.py from the old cct repo)

@captainceramic Are you happy with these edits?

Also, I double checked and after these changes are made the Nino3.4 workflow works (with Crop instead of Timeslice)

I'm very happy with those - thanks!


Minor improvements to the render tree dump.

Also fixes https://github.com/flutter/flutter/issues/6510


ES6 Template Literals And "require-spaces-inside-object-brackets"

I ran into an interesting issue with the "require-spaces-inside-object-brackets" rule when using ES6 template literals.
For example this will cause a JSCS error:
var id = 1;
var url = `www.example.com/${id}/profile`

It seems like this rule should not be applied within strings.
For more detail about the specific error see https://github.com/emberjs/ember.js/pull/10637

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/9626767-es6-template-literals-and-require-spaces-inside-object-brackets?utm_campaign=plugin&utm_content=tracker%2F281640&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F281640&utm_medium=issues&utm_source=github).

Hmm this shouldn't be an issue since the rule is checking for an ObjectExpression. If esnext is enabled esprima should parse it as a TemplateLiteral (and I see it is enabled in the ember .jscsrc. Can you try testing on master or a later release of jscs?
Going to look into it more.

@chadhietala what version of JSCS are you using? 1.11 correctly only does Objects, where as 1.10.* did all curlies.

Also, this is dependent on a bug in esprima which will be fixed in the next release.

@mikesherov Yea Ember is using an older version. Will bump and report back.

@mikesherov Yes this appears to be effected by the Esprima bug. I bumped everything up to the latest and I still get the error.

You can work around it by depending on his pull request branch on esprima,
and setting "esprima": "node_modules/esprima" in your jscsrc
On Tue, Mar 17, 2015, 07:36 Chad Hietala<EMAIL_ADDRESS>wrote:

@mikesherov https://github.com/mikesherov Yes this appears to be
effected by the Esprima bug. I bumped everything up to the latest and I
still get the error.
—
Reply to this email directly or view it on GitHub
https://github.com/jscs-dev/node-jscs/issues/1171#issuecomment-82387250.

yeah as per @nevir
package.json
"esprima": "mikesherov/esprima#templateStrings"

.jscsrc
"esprima": "node_modules/esprima"

but I dono if that works that easily since its broccoli-jscs?

Can close this now?

Yes @hzoo thanks!


Search - Repeated Searches

Repeating a search after going to a different page does not work.

Go main page type "test" into search box.
From the hamburger menu choose "Listing Management"
Return to the main page by clicking the shopping bag icon in the top menu bar.
Type "test" into the search bar.

Expected Result:  Search results are displayed.
Actual Result:  The word "Searching..." is displayed on your screen forever (This is an estimate, I waited 30 seconds).

Can this be moved to resolved? @clarkjefcoat

Passed on Latest.  IE, FF and Chrome

Passed on new staging. IE, Chrome and FF


Console errors when go to checkout page

I see these errors when go to checkout page:

Error concerns layout service, there was an unexpected missed data or something. Now it is not reproducible

Wei told issue was fixed,


JUnit: No annotation with failure details

Each failing test will produce an annotation with failure details:

No annotation/comment is being created with more information about the failed test.
The summary comment is created with no issue but without information about what test failed.
That's the only comment I get:

My setup is PHPUnit generating a junit.xml file.
Is this behaviour expected or is it a bug? Thanks!

Have you clicked on the see this check link? That is the place where GitHub displays annotations.

@EnricoMi hey, I did click but I was in the Github iOS app which throws a “Something went wrong” - I just tested using the website and it shows the annotation. I was expecting to be a comment in the PR though, that’s why I didn’t bother to try elsewhere. Thanks!


Fix of Footer's component code example

What does this PR do?
Fix the documentation of Footer component.
Where should the reviewer start?
What testing has been done on this PR?
How should this be manually tested?
Any background context you want to provide?
What are the relevant issues?
Screenshots (if appropriate)

Thanks for your contribution! Welcome to the team 🎉