Add mattmahoneyrh to the opendatahub-io members list

I have joined the Operator Scrum Team on the RHODS/RHOAI Project.
@etirelli @VaishnaviHire
New Open Data Hub Member Requirements
New members have reviewed and acknowledged the Open Data Hub:
Code of Conduct
Contribution Guide
Community Membership Guidelines

Enabled 2FA on their GitHub account
Description

New Open Data Hub Member Requirements

[ ] New members have reviewed and acknowledged the Open Data Hub:

Code of Conduct
Contribution Guide
Community Membership Guidelines

[ ] Enabled 2FA on their GitHub account

Pull Request Requirements

[ ] New members are added in alphabetical order
[ ] Only one new member change per commit (if you add two members separate it in two commits
[ ] For individual user changes: Commit message format Add <USERNAME> to <opendatahub-io, mlops-sig, ...>.
[ ] For new team requests: Commit message format Create <TEAMNAME>. If the new team consists solely of existing members, you may
[ ] New GitHub team requests are from an existing opendatahub-io member who will function as the maintainer of the team. Each additional member will follow the same requirements for adding new members

@gmarkley-vi @accorvin @jkoehler-redhat @LaVLaS @anishasthana  Can someone please approve this PR?
Also should I be added to ODH Operator Maintainers.members as mentioned by @etirelli


Dev environment: Windows servers

As a developer, I will need a Windows server (3 instances) in order to begin development work for Sprint 1.

One Windows 2012 R2 server is established in FTL. Delphi XE3 is installed, however, additional components need to be located and installed. Sent email to BC requesting if they know the location of the components and if not can they obtain a copy from the CPRS v31 developement team

This can be closed. The Windows server is up and running. We don't need 3, just the one. (Plus the Linux server Cache, #22).


Search Icon on the wrong position

The search icon seems to be on the wrong postition. I am using latest Google chrome version.

Duplicate of #4690


[feat] virtualcluster: Enable different kubernetes distribution support

The virtualcluster only support vanila Kubernetes now, but the real use case is that many customers are using some different Kubernetes distributions,  it would be great to enable the virtualcluster can support OpenShift and other Kubernetes clusters which has passed the conformance test at https://github.com/cncf/k8s-conformance/tree/master/v1.20

@gyliu513 Thanks for filing the issue. As you may know, we are actively moving VC project to CAPN (https://github.com/kubernetes-sigs/cluster-api-provider-nested) where we defines the interfaces for using customized APIServer/controller manager/etcd. We are going to integrate CAPN model with existing VC framework shortly.
When it is ready, one can implement customized nestedapiserver controller to use other kubernetes distribution like k3s/openshift easily.

Thanks @Fei-Guo that's great! Let me close this one and trace it in https://github.com/kubernetes-sigs/cluster-api-provider-nested


Different options in ApiVersioningOptions.cs between .NET Framework and .NET Core packages

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
We fixed one bug in our Legacy (but still in active development) .NET Framework WebAPI by setting ControllerNameConvention = new OriginalControllerNameConvention();.
Now I want to do the same in .NET Core WebAPI, but IControllerNameConvention is missing in ApiVersioningOptions.cs...
.NET Framework NuGet name and version: Microsoft.AspNet.WebApi.Versioning 5.1.0
.NET Core NuGet name and version: Asp.Versioning.Mvc.ApiExplorer 7.1.0

Expected Behavior
.NET Core should have all options from official documentation.
Controller name should be PrintV2 in Swagger in .NET Core.

Steps To Reproduce
No response
Exceptions (if any)
No response
.NET Version
8.0.100-rc.2.23502.2
Anything else?
.NET Core 7.0.13

This option still exists, but it is now only set through dependency injection (DI). Web API doesn't have out-of-the-box DI support so the option needs to be explicit in the options. In ASP.NET Core, Minimal APIs and MVC Core are separated by the AddApiVersioning() and AddMvc() respectively. I considered moving ControllerNameConvention over to the MVC Core options, but I really wanted to reduce the number of configurable options which are better suited for DI. In fact, the naming convention could always be set via DI. Setting it via the options was a secondary way.
You can find the source at:
https://github.com/dotnet/aspnet-api-versioning/blob/471eb044500495beb8a4f009d9a1c2bf4d5fde63/src/AspNetCore/WebApi/src/Asp.Versioning.Mvc/DependencyInjection/IApiVersioningBuilderExtensions.cs#L66
To achieve your desired configuration, you need only set or replace the service you want to use:
builder.Services.AddSingleton<IControllerNameConvention, OriginalControllerNameConvention>();

In fairness, I missed adding this change to the migration guide. I have updated the section on configuration to note this difference.
I hope that helps.

Thank you. It works :)


Similar function for Movies/Books?

Hi. Is there a chance to "enhance" this application so you work with movies or books? Or maybe create other applications that are appropriate to the purpose?

There are no plans to extend this, mostly because I don't have the time to look into how google stores the files internally, and how to get decrypted copies of those files then.

I understand. However, I want anyway leave open in the case some other developer wanted to try to add this function or simply as "future feature" ..

It is completely out of the scope of this app. Sorry.

OK, thank you the same. But if you had to find the time, please do it

The list of things I want to do is very long, and that is somewhere near the bottom, priority wise, so it is very, very unlikely that this will happen.


chore: updates coraza to latest, fixes nightly build

This PR updates Coraza and coraza-wasilibs dependencies in order to fix the failing nightly build:
github.com/corazawaf/coraza-proxy-wasm/internal/operators imports
	github.com/corazawaf/coraza-wasilibs imports
	github.com/corazawaf/coraza/v3/operators: package github.com/corazawaf/coraza/v3/operators provided by github.com/corazawaf/coraza/v3 at latest version v3.0.0-rc.1 but not at required version v3.0.0-rc.1.0.20230329004849-daf3747c8c45

Should we refresh this repo's .ftw.yaml in this commit?

I made this update a bit more extensive, updating the CRS rule, FTW and the failing tests.
About them:

Several new rules are looking for the Referer header, which is sanitized and removed by Enoy itself
Another iteration will be needed to further dig into some tests that we fixed Coraza side but they are still failing Envoy side.

If it seems right, I would first merge this one to fix the nightly build


Installation Problem

Hi @Proektsoftbg ,
Can you help me? I cannot install the last version of calcpad on my office computer. The problem it's on my side? The IT guy tryed with login privilege but failed too.

Hi, It seems that it is trying to replace a font that is already installed.
What happens if you try to click "Ignore"?

Hi Ned, i don`t know why, but this morning it worked! Mysteries of IT


Remove dlq for outbound kafka action

Insert JIRA Issue number and update link
PR owner checklist
Please make sure that your PR meets the following requirements:

[ ] Your code is properly formatted according to this configuration.
Run mvn clean verify -DskipTests so that imports are correctly set.
[ ] Your commit messages are clear and reference the JIRA issue e.g: "[MGDOBR-1] - $clear_explanation_of_what_you_did"
[ ] The layers in the kustomize folder have been updated accordingly.
[ ] All new functionality is tested
[ ] Pull Request title is properly formatted: MGDOBR-XYZ Subject
[ ] Pull Request contains link to the JIRA issue
[ ] Pull Request contains link to any dependent or related Pull Request
[ ] Pull Request contains description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket

PR reviewer(s) checklist

[ ] Check if code and Github action workflows have been modified and if the modification is safe
[ ] If the modification is safe, add the safe to test label

How to trigger pipelines and use the bots:

Run the end to end pipeline
Annotate the pull request with the label: safe to test. If you want to run the pipeline again, remove and add it again.

Rebase the pull request
Comment with: /rebase.

Deploy BOT
Comment with `/deploy <dev,stable> when the PR has been merged to deploy to a target environment.

Doesn't this remove the entire Error Handler back-channel/DLQ configured on a Bridge?!!?
The DLQ is passed to RHOC for it to write failures to; that RHOSE then responds to etc...
:worried:

Doesn't this remove the entire Error Handler back-channel/DLQ configured on a Bridge?!!?
The DLQ is passed to RHOC for it to write failures to; that RHOSE then responds to etc...
worried

This is removing the DLQ configuration in the SendToKafka action. Up to my knowledge, the DLQ does not apply in this case


Any errors inside the page.open() will cause a silent crash or hang.

Which version of PhantomJS are you using? Tip: run phantomjs --version.
Using 2.1.1

What steps will reproduce the problem?

(function(){
    var page = require('webpage').create();
    page.open('about:blank', function(){
        SYNTAXERROR
        console.log('test');
        phantom.exit();
    });
    page.onLoadFinished = function() {
       phantom.exit();}
}());

This minimal example demonstrates the behavior. It will finish without any console logging or errors. if the onLoadFinished handler is omitted, it will simply hang. Which makes debugging a nightmare.

Which operating system are you using?
Arch Linux

Did you use binary PhantomJS or did you compile it from source?
binary

Please provide any additional information below.

This issue has been fixed in master and released in 2.5 beta version. Closing this issue because we have the tracking issue - 12973

Nice, so i'll just compile the current version? Can't find a binary for 2.5 anywhere.

@maxupp yes or grab the 2.5 beta here https://bitbucket.org/ariya/phantomjs/downloads

Can you update the latest build in https://bitbucket.org/ariya/
phantomjs/downloads. It would be great if you upload the binary which
contains all libraries.
Thanks and Regards
Santosh Hegde
On Fri, Feb 10, 2017 at 7:36 PM, Vitaly Slobodin<EMAIL_ADDRESS>wrote:

@maxupp https://github.com/maxupp yes or grab the 2.5 beta here
https://bitbucket.org/ariya/phantomjs/downloads
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ariya/phantomjs/issues/14812#issuecomment-278949542,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AOXpKZOIxD_Wn7FcAPnNUOPB3lHkXR_iks5rbG7_gaJpZM4L9W8c
.


Align Pipestat output schema with looper output schema

Addresses https://github.com/pepkit/pipestat/issues/20
From previous discussion: if items are present but not defined under samples, they should be assumed to be project-level.
Therefore, this PR takes those extra top-level arguments and adds them to the project level.

Note:
The output schema can be written as so:
pipeline_name: test_pipe
samples:
  number_of_things:
    type: integer
    description: "Number of things"
  percentage_of_things:
    type: number
    description: "Percentage of things"
  name_of_something:
    type: string
    description: "Name of something"
  switch_value:
    type: boolean
    description: "Is the switch on or off"

testing_project_result:
  type: string
  description: "misc project result"
more_number_of_things:
  type: integer
  description: "Number of things, min 20, multiple of 10"
  multipleOf: 10

However, if using:
properties:
  samples:

pipestat will still throw an exception 'properties' is missing required key(s): type, description during schema validation.


Recurrent Attention API for keras

This issue is opened to host a discussion about the recurrent attention API for keras.
Related issues:
#11142.
#8296.
#7633.

From @fchollet in #11142:

I don't think this PR is a good fit to implement attention models at this point.

I believe the Model subclassing API would make this use case far easier.

We should come up with side by side code examples of similar models to figure out whether that is the case (the code doesn't have to work, it would just show the workflow with each API, the API in this PR and the model subclassing API).

Possibly, but first we should make a decision: what's the best way to implement attention models in the Keras API today, and would this PR provide a better API?

To make this decision we need to write quick code examples showing how to build common attention models 1) with the model subclassing API, 2) with the API introduced by #8296, and figure out what looks best.

From @farizrahman4u in #11142.

Shouldn't we first see what an attention model written in Keras (as of now) looks like? Sometimes you can get away with a well documented example (like @fchollet did with seq2seq) instead of making huge changes to the code base to fit niche cases. I would suggest writing an example using a simpler attention model (not MixGaussian.. it has way too much internal knobs) using existing Keras features and see where it goes.

Great to see this moving 👍I'm happy cont. this work as well.
This was the result of my "requirements analysis" (based on common use-case/authoritative papers) as of the API doc:

The attention mechanism should be implemented separately from the core cell so that it can be reused with any standard keras RNN Cells.
Besides the attended, the attention mechanism should have access to the core cell’s states as well as the input at each timestep for computing the attention representation at that timestep.
The attention mechanism should be allowed to be recurrent, i.e. “have state(s) on its own” that is forwarded to the next attention computation.
One should (optionally) be able to get the sequence of attention representations from each timestep as output, for use in later stacked (recurrent) layers.
It should be easily configurable whether the attention is applied “before or after” (in sequence/time dimension) the core recurrent transformation at each timestep.
Masking should be supported and have the same behaviour as for regular keras recurrent layers.
The attended should be allowed to consist of multiple inputs (tensors) of different shape.
It should require minimal efforts for users to write custom attention mechanisms that fulfil requirement 1 - 7.

These should be useful as a reference when discussing tradeoffs and priorities.

Are we talking about Attention only in recurrent models?

Should we consider only the more common dot-product attention or everything else with an overridable method that computes weights?

I think it is important to not be limited to rnn design. I.e. See https://openreview.net/forum?id=HyGBdo0qFm

@bhack

I think it is important to not be limited to rnn design. I.e. See https://openreview.net/forum?id=HyGBdo0qFm

Generally, I agree. The different approach taken in e.g. Attention Is All You Need is of great importance and should be considered for any seq2seq problem. However (!) I'd claim that any feedforward architecture can already be supported by the existing Keras API. You might have to write several custom layers but feedforward attention can be implemented by reusing, and without duplicating, major parts of the existing API (along the lines of @farizrahman4u commet cited above)
Recurrent attention is different; if you want to implement it in a new layer/model you need to reimplement the majority of the RNN logic or wrap the cell as is suggested in the previous PR.
That said, it might still make sense to add layers or models to the API for non-recurrent attention. But I think it is still high prio to support recurrent attention as was initially in the "request for contributions".

In a sense, I think recurrent attention "is supported" since we added support for constants in the RNN - you can quite easily write your own cell wrapper. To me, It's just a question of how standardized and simple you want to make this and if/what ready (cell-wrapper) attention mechanisms should be added to the API.
The main current limitation with this approach is that there is no option to return "state sequences" from the RNN, which is required to feed the attention encoding from on layer to subsequent layers (see point 3) here).

Ok. So usage workflow examples have been requested. Since the heading of this and all preceding issues/PRs has been recurrent attention I'll focus on this and repeat/clarify the workflow of the only concrete suggestion so far. I'll use the architecture in this paper for handwriting synthesis as the use-case (but the workflow would be the same for e.g. this kind image captioning)
xy = Input((None, 2))  # coordinates of handwriting trajectory
text = Input((None, n_characters))  # sequence of characters to be synthesized

cell = MixtureOfGaussian1DAttention(LSTMCell(64), components=3, heads=2)
attention_lstm = RNN(cell, return_sequences=True)
h = attention_lstm(xy, constants=text)
xy_pred = TimeDistributed(Dense(2, activation=None))(h)
# MoG output is used instead of basic regression in the original paper, but this is has nothing to do with the attention mechanism and is left out for brevity.

model = Model(inputs=[xy, text], outputs=xy_pred)
model.compile(optimizer='adam', loss='mse')
model.fit([xy_data[:, :-1], text_data], xy_data[:, 1:])

I think that this workflow is perfectly aligned with Keras guiding principles. Note that no modification of existing classes is required, we've just defined a new RNNCell. For the sake of modularity it reuses/wraps the LSTMCell, which can be replaced by e.g. the GRUCell. This use-case and workflow were one of the main drivers for breaking out the RNNCell and adding support for constants in the RNN.
I honestly can't come up with a reasonable alternative based on the Model subclassing API for this use case. I guess these are other options:
attention_lstm = MixtureOfGaussian1DAttentionRNN(
    cell=LSTMCell(64),
    components=3,
    heads=2,
    return_sequences=True
)
h = attention_lstm(xy, attended=text)

Here, MixtureOfGaussian1DAttentionRNN would be a standalone layer where the attention mechanism has been coupled tightly with the general RNN logic (why I think it is a bad option).
Alternatively something like:
attention_lstm = AttentionRNN(
    cell=LSTMCell(64),
    attention_mechanism=MixtureOfGaussian1DAttention(components=3, heads=2),
    components=3,
    heads=2,
    return_sequences=True  # here return_state_sequences could be supported or other
)
h = attention_lstm(xy, attended=text)

Where both the core cell and and attention mechanism are injected into an new class that connects them. But this requires new interfaces both for the attention_mechanism and for the AttentionRNN which seems completely unnecessary since the existing RNNCell interacface already supports attentive cells (thanks to addition of constants).
Bottom Line

Is this use-case at all of relevans!?
Is there any problem with the suggested approach? If so, please be concrete and/or provide an alternative!
If we can agree that recurrent attention is already supported by using an appropriate (attentive) RNNCell - do you feel that there is a need to add "standard" attentive cells, such as MixtureOfGaussian1DAttention in this example, to the official keras API?

@fchollet @farizrahman4u @gabrieldemarmiesse

Thinking in terms of reducing the number of mental models (cognitive load), I vote for the first one.

Even in that case, I think the "MixtureOfGaussian1DAttention" wrapper is too niche to make its way into the core api (I might be wrong).

We definitely need a standard way to do attention in Keras. I think we should come up with an end to end example, which includes the attention wrapper definition that users can easily extend. For simplicity, use the simplest attention mechanism possible.(MixtureOfGaussian1DAttention has too many moving parts).

I have a feeling that some uncertainty comes from the (motivated!) fuzz about non-recurrent attention mechanisms. If we were to add support for the transformer architecture in Attention is All you Need (or the recent BERT), I definitely think that Model subclassing API is a good place to start - because there are many intricate parts that should be combined the right way. It would look something like:
input_sequence = Input((None, 1))
target_sequence_tm1 = Input((None, 1))
transformer = Tranformer(
    input_tokens=n_input_tokens,  # must be provided if embeddings created internally
    target_tokens=n_target_tokens,  # must be provided if embeddings created internally
    units_model=512,
    units_ff=2048,
    units_keys=64,
    units_values=64,
    layers=6,
    heads=3
)
target_sequence_pred = transformer([input_sequence, target_sequence_tm1])

Where Tranformer would subclass Model. It could also be motivated to expose some of the internals as separate sub-models or layers, which raises the question if the same "Attention layers" can be reused both in the feedforward and recurrent setting. This would be possible (as per my previous suggestion) by defining cell transformations using the functional API, e.g.:
# define complete attentive cell using functional API
units = 32
xy_t = Input((2,))
text = Input((None, n_characters))  # note that this is a sequence
h_tm1 = Input((units,))
c_tm1 = Input((units,))
h_att_t = AttentionMechanism(inputs=concatenate([xy_t, h_tm1]), attended=text)
x_t = concatenate([xy_t, h_att_t])
h_t, c_t = LSTMCell(units)(inputs=x_t, states=[h_tm1, c_tm1])
cell = CellModel(  # creates a valid cell implementation based on functional definition
    inputs=xy_t,
    outputs=h_t,
    input_states=[h_tm1, c_tm1],
    output_states=[h_t, c_t],
    constants=text
)

But we should probably avoid considering this for now and make as few additions as possible. This is why it was decided to not add the RNNAttentionCell base class to the API. ...In the end, the only API addition of the previous PR was one first attentive RNNCell.

Thanks @farizrahman4u, makes sense. So you think something like the RNNAttentionCell base class should be added to the API, to simplify custom implementations? Please let me know if you have suggestions for a simpler, yet relevant, first attention mechanism and corresponding use-case.

Fair enough. I think you can write the whole thing in the example (including the RNNAttentionCell class), submit a PRs, and discuss with @fchollet on what parts can moved into Keras API and what should stay in the example.

As per @farizrahman4u suggestion above, please see https://github.com/keras-team/keras/pull/11421 @fchollet

@andhus thanks for making the PR #11421!
As a Keras fan eagering to implement attention mechanism,
do you think it's okay for me to start using the recurrent_attention_machine_translation.py you provide as a example, or should I wait until the PR is merged?
Thanks in advance.

Is there an update on this? The RFC has been approved for over a year


Don't start interval if disabled #24

@davedoesdev Sorry for not looking this over yet! Been visiting parents over the weekend! 👍

@davedoesdev Thanks again for doing this! 😄

Deployed in 1.2.1 😄

Thanks!


License create

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew tests with your changes locally?

brew create now includes the license DSL inside the template erb
accepts --set-license=<license_id> as a arg for brew create

Pointed to the wrong remote and branch, closing


Feature/MZ-593

Midaz Pull Request Checklist
Pull Request Type

[ ] Auth
[ ] Infra
[ ] Ledger
[x] Mdz
[ ] Transaction
[ ] Pipeline
[ ] Documentation

Checklist
Please check each item after it's completed.

[x] I have tested these changes locally.
[x] I have updated the documentation accordingly.
[x] I have added necessary comments to the code, especially in complex areas.
[x] I have ensured that my changes adhere to the project's coding standards.
[x] I have checked for any potential security issues.
[x] I have ensured that all tests pass.
[ ] I have updated the version appropriately (if applicable).
[x] I have confirmed this code is ready for review.

Additional Notes
Obs: Please, always remember to target your PR to develop branch instead of main.

@maxwelbm I closed this PR because the first commit has not signed. please, reopen with all commits signed.


Algo Binary search in Java

Need to make a readme file which should have the
Definition
Explanation in simple terms
Code related explanation.

Could you assign me the issue?

ok


Make 'supportedSubmitMethods' configurable in springfox-swagger-ui / Enable and disable the "Try it out!" functionality

What's this PR do/fix?
Adds the possibility to UiConfiguration to define the supported submit methods in used swagger-ui ("Try it out!") or to disable the functionality completely (s. Swagger-UI parameter)
Are there unit tests? If not how should this be manually tested?
Changed some of the existing tests to cover new functionality
Any background context you want to provide?
UiConfiguration now holds of list of methods, which will be exposed via configuration REST service to springfox.js. Two constants DEFAULT_SUBMIT_METHODS, NO_SUBMIT_METHODS and a new constructor UiConfiguration(String validatorUrl, String[] supportedSubmitMethods) had been added for convenience.
What are the relevant issues?
#1291 and #1286

Thanks for the PR! :metal:


Unable to setup Slide 'LOCAL IP'

I had to factory reset my slide, and add it as a new entity. But for some reason it doesn't want to integrate now. Even though nothing really changed. Also, it is working fine with the local API through the official app.
Log details (ERROR)
This error originated from a custom integration.

Logger: custom_components.slide.cover
Source: custom_components/slide/cover.py:132
integration: Slide
First occurred: August 27, 2024 at 20:35:09 (1 occurrences)
Last logged: August 27, 2024 at 20:35:09

Unable to setup Slide 'LOCAL IP'

This error originated from a custom integration.

Logger: goslideapi.goslideapi
Source: custom_components/slide/cover.py:111
integration: Slide
First occurred: August 27, 2024 at 20:35:09 (1 occurrences)
Last logged: August 27, 2024 at 20:35:09

Failed request Local API v2. HTTPCode=401

The code I'm running is the following:
#Slide
cover:
  - platform: slide
    host: LOCAL IP
    password: SLIDE PASSWORD
    api_version: 2

Does someone have a pointer for me to solve this issue?

You need to replace LOCAL IP in your configuration code with the actual IP address of your Slide device.
See for more information the README file:

To use this component in your installation With the Local API, add the following to your `configuration.yaml` file for each Slide:
# Example yaml entry

cover:
  - platform: slide
    host: <IP_ADDRESS>
    password: 12345678
    api_version: 2

Configuration variables:

host (Required): The IP address or hostname of your local Slide
password (Required): The device code of your Slide (inside of the Slide or in the box, length is 8 characters). NOTE: With api_version: 2 you can fill in anything here, it is not used by the local API
invert_position (Optional): If the position should be inverted e.g. 0% -> 100% and 100% -> 0% (default = False)
api_version (Optional): The local API version, this is 1 or 2. Firmware version 0.13.8 and 2.0 are using API version 2, all other are using API version 1.  (default = 2)

NOTE: You cannot determine the firmware yourself. If API version 2 does not work, try API version 1. In the following reddit there is a spreadsheet with all known slides and their firmeware version:
https://www.reddit.com/r/slidecurtains/comments/1cwc5u2/which_firmware_version_does_your_slide_have/


Update Elm submission - retry when server is down

Sorry, I forgot to include this in my original submission.
This was a fun project by the way, so thank you.

Thank you back :)


Consider log4j2.formatMsgNoLookups for additionalFields

:green_heart: Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS

Expand to view the summary

Build stats

Start Time: 2021-12-15T07:59:28.456+0000

Duration: 8 min 38 sec

Commit: 612d52a83bcfb50d43efc2c09c40e5c50f878f58

Test stats :test_tube:

Test
Results

Failed
0

Passed
171

Skipped
0

Total
171

:robot: GitHub comments
To re-run your PR in the CI, just comment with:

/test : Re-trigger the build.

run elasticsearch-ci/docs : Re-trigger the docs validation. (use unformatted text in the comment!)


Added method swizzling for default session configuration, is now compatible with AFNetworking

I added method swizzling so we add the SWprotocol to the NSURLProtocols default session every time it gets initialized. Tried to keep same coding style. Also added a isEnabled function to the SWHttpTrafficRecorder class that enables/disables the recorder for a provided session configuration.

@Amindv1 Thanks for the pulling request to enhance the tool and sorry that I was not looking into this earlier due to vacations.
I looked at the pull request and have the following preliminary feedbacks:

If this pull request is merged, after we enable recording for default session config, we do not have a way to remove/stop it; maybe what we can do is to enable recording for default sessions through swizzling introduced in this pull request while users call existing methods like - (BOOL)startRecording(when a session config is not provided, put it around Line 104 of SWHttpTrafficRecorder.m), and then stop recording when - (void)stopRecording is called;

The implementation of  (void)setEnabled:(BOOL)enabled forConfig:(NSURLSessionConfiguration*)sessionConfig can be merged with existing implementation of - (BOOL)startRecordingAtPath:(NSString *)recordingPath forSessionConfiguration:(NSURLSessionConfiguration *)sessionConfig error:(NSError **) error  to avoid duplicate code.

@Amindv1 Thanks for the pulling request to enhance the tool and sorry that I was not looking into this earlier due to vacations.
I looked at the pull request and have the following preliminary feedbacks:

If this pull request is merged, after we enable recording for default session config, we do not have a way to remove/stop it; maybe what we can do is to enable recording for default sessions through swizzling introduced in this pull request while users call existing methods like - (BOOL)startRecording(when a session config is not provided, put it around Line 104 of SWHttpTrafficRecorder.m), and then stop recording when - (void)stopRecording is called;

The implementation of  (void)setEnabled:(BOOL)enabled forConfig:(NSURLSessionConfiguration*)sessionConfig can be merged with existing implementation of - (BOOL)startRecordingAtPath:(NSString *)recordingPath forSessionConfiguration:(NSURLSessionConfiguration *)sessionConfig error:(NSError **) error  to avoid duplicate code.

@JinlianWang I'll have to take a look at it later I've been busy with other projects at the moment.

Hey @Amindv1 curious if you were thinking of revisiting this PR, as I definitely think it's very valuable when working with libraries that don't expose the NSURLSessionConfiguration.

@loudmouth It's not likely anytime soon you can just fork off my repo and use it, everything works the only reason the PR isnt merged is because of the above mentioned things.


[iOS Distribute] The distribute delegate interface uses ! (force unwrap) operator in method selector

Hi there,
when taking a look at the generated swift method selector, the parameter types are annotated using the force unwrap operator. This indicates that the Objective-C method selector and class are lacking the nullability annotation.
optional func distribute(_ distribute: MSDistribute!, releaseAvailableWith details: MSReleaseDetails!) -> Bool

It would be great if the nullability specification could be done on the SDK side.

Hello, this forum is only used for issues with the documentation. For feature request, see the App Center public repo.


Fix link to documentation in README.

Description
One of the links to the documentation in README is broken. I just correct the URL.
Check List (Check all the applicable boxes)

[x] I understand that my contributions needs to pass the checks.
[x] If I created new functions / methods, I documented them and add type hints.
[x] If I modified already existing code, I updated the documentation accordingly.
[x] The title of my pull request is a short description of the requested changes.

Indeed, good catch, thanks!


HTTP Status Codes

Hi all and thanks a lot for the great plugin! 👍
I'm trying to prevent soft 404's when working with vitedge. I already managed to return an HTTP 404 status code using this code in functions/props/NotFound.ts:
import type { EdgeProps } from 'vitedge'
import { NotFoundError } from 'vitedge/errors'

export default <EdgeProps> {
  async handler() {
    throw new NotFoundError('Not Found')
  }
}

I tried with some non-existing pages on my website and they show the correct Vue view. The calls to the vitedge API also return 404. However, the Vue page itself still return a 200 status code. Is it possible to reflect the status code of the API response also on the Vue page?

@clampr Hello! You mean the server-side rendered page for a missing route returns code 200, right?
For now, you can add this to your 404 Vue page component to make it work:
<script>
import { useContext } from 'vitedge'

export default {
  setup() {
    if (import.meta.env.SSR) {
      const { writeResponse } = useContext()
      writeResponse({ status: 404 })
    }
  },
}
</script>

I'll give it a thought about having this automatically applied like we do with 302 code 🤔

Thanks @frandiox 🙂 It does exactly what I was looking for.
I guess it makes sense to apply these status codes automatically as it would be consistent with the redirects. Reading the docs I was expecting it to be applied automatically. But I don’t mind as your snippet does the job, too.

@clampr This should be automatic from v0.16.4 🎉


anybody give me the article about Licode: TURN configuration discussion of coTurn?

I found an article of how to set turn server with Licode on groups.google.com,but I cant open it.
anybody can give me the article?
the URL is
https://groups.google.com/forum/#!topic/lynckia/rT8_N1lcV8M
thanks

What do you mean you cannot see it?
It is public, try a different browser?
Here are the basics:
http://lynckia.com/licode/turn.html


updated installation description

addresses issue #89

Hi @sibirrer maybe you can add the installing requirements line also in the Contributing instructions?
Thank you for addressing the issue !

@alxogm I make you here review my PR that hopefully fixes your issue :)


Update framework for Swift 5.4

Changed @_functionbuilder to @resultbuilder
Housekeeping
Updated version use to 1.6.1

Overall LGTM! Nice to see that the DSL'ish tools like the functionBuilder are finally stable and included in the current Swift version!


new tray window

create new tray window
use this new tray to open a window to set or update wakatime api-key

Before starting reviewing it I run locally and didn't get a tray icon. Does that work in Mac also?

hey @gandarez how did you run it?
i am running it with yarn dev on my mac machine and it works fine
remember to run yarn install first

I run by calling yarn start. Running yarn dev gave the expected result. PR approved.
thanks
I am creating an issue to fix npm start script


Remove Uniqueness Validation type in Mx9

Only Database validation is relevant
Remove migration information from documentation, but expand information about uniqueness validation in validation rules
This is related to JIRA issue [TW-1260]

Thanks for your review Ishan, @ishan1729
I've made one change to clarify rounding numbers, but no other changes.
Is it OK to merge?

Yes, please go ahead.


mgmt/mcumgr: Add Kconfig option for direct image upload

The commit adds CONFIG_IMG_MGMT_DIRECT_IMAGE_UPLOAD that allows
to turn on the direct image upload.
Signed-off-by: Dominik Ermel<EMAIL_ADDRESS>Fix for my blunder, controls: https://github.com/zephyrproject-rtos/mcumgr/commit/31a2aa9cea58d3ceecbf0d5b91361bff7c94aeca

@nvlsianpu @carlescufi Please review.

@nashif Thanks!


Reconvert demo video

I reconverted the webm of the demo video to a gif. It looks much better and is about 1.3 MBs smaller.

@IanMurray Cheers :+


New version: StandardNotes.StandardNotes version 3.152.1

Pull request has been created with Komac v1.5.0 :rocket:
Microsoft Reviewers: Open in CodeFlow

@wingetbot run


Pipenv fails to install psycopg2

Issue description
pipenv install psycopg2 failed, even though the Pipfile and Pipfile.lock files included the entries and hashes for packages.
Also I noticed that pip install psycopg2 also failed: https://gist.github.com/ye/bf1198cc15d9a6a177bae8bc0fc0aff9#file-pip-install-psycopg2-log
Also after the pip install failure, the pip inside pipenv virtual environment is borked.
$ pip -V
dyld: Library not loaded: @executable_path/../.Python
  Referenced from: /Users/ye/.venvs/server-D2-ixepO/bin/python3.7
  Reason: image not found
[1]    5466 abort      pip -V

Expected result
I expect it should install successfully.
Actual result
https://gist.github.com/ye/bf1198cc15d9a6a177bae8bc0fc0aff9
Steps to replicate
pipenv install psycopg2

$ pipenv --support
Pipenv version: '2018.11.26'
Pipenv location: '/usr/local/lib/python3.7/site-packages/pipenv'
Python location: '/usr/local/opt/python/bin/python3.7'
Python installations found:

3.7.4: /Users/ye/.venvs/server-D2-ixepO/bin/python3
3.7.4: /usr/local/bin/python3
3.7.4: /usr/local/bin/python3.7m
2.7.10: /usr/bin/python
2.7.10: /usr/bin/pythonw
2.7.10: /usr/bin/python2.7

PEP 508 Information:
{'implementation_name': 'cpython',
 'implementation_version': '3.7.4',
 'os_name': 'posix',
 'platform_machine': 'x86_64',
 'platform_python_implementation': 'CPython',
 'platform_release': '18.7.0',
 'platform_system': 'Darwin',
 'platform_version': 'Darwin Kernel Version 18.7.0: Tue Aug 20 16:57:14 PDT '
                     '2019; root:xnu-4903.271.2~2/RELEASE_X86_64',
 'python_full_version': '3.7.4',
 'python_version': '3.7',
 'sys_platform': 'darwin'}

System environment variables:

TERM_SESSION_ID
SSH_AUTH_SOCK
LC_TERMINAL_VERSION
Apple_PubSub_Socket_Render
COLORFGBG
ITERM_PROFILE
SQLITE_EXEMPT_PATH_FROM_VNODE_GUARDS
XPC_FLAGS
LANG
PWD
SHELL
SECURITYSESSIONID
TERM_PROGRAM_VERSION
TERM_PROGRAM
PATH
LC_TERMINAL
COLORTERM
COMMAND_MODE
TERM
HOME
TMPDIR
USER
XPC_SERVICE_NAME
LOGNAME
__CF_USER_TEXT_ENCODING
ITERM_SESSION_ID
SHLVL
OLDPWD
ZSH
GOPATH
PAGER
LESS
LSCOLORS
VIRTUAL_ENV_DISABLE_PROMPT
WORKON_HOME
PROJECT_HOME
VIRTUALENVWRAPPER_PYTHON
VIRTUALENVWRAPPER_VIRTUALENV
VIRTUALENVWRAPPER_PROJECT_FILENAME
VIRTUALENVWRAPPER_WORKON_CD
VIRTUALENVWRAPPER_SCRIPT
VIRTUALENVWRAPPER_HOOK_DIR
PIP_DISABLE_PIP_VERSION_CHECK
PYTHONDONTWRITEBYTECODE
PIP_PYTHON_PATH
PIPENV_ACTIVE
VIRTUAL_ENV
_
PIP_SHIMS_BASE_MODULE
PYTHONFINDER_IGNORE_UNSUPPORTED

Pipenv–specific environment variables:

PIPENV_ACTIVE: 1

Debug–specific environment variables:

PATH: /Users/ye/.venvs/server-D2-ixepO/bin:/Users/ye/.cargo/bin:/usr/local/sbin:/Users/ye/Library/Python/3.7/bin:/Users/ye/bin:/Users/ye/.cargo/bin:/usr/local/sbin:/usr/local/opt/gettext/bin:/Users/ye/Library/Python/3.7/bin:/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/usr/local/go/bin:/Users/ye/.cargo/bin:/Users/ye/.poetry/bin
SHELL: /bin/zsh
LANG: en_US.UTF-8
PWD: /Users/ye/website/server
VIRTUAL_ENV: /Users/ye/.venvs/server-D2-ixepO

Contents of Pipfile ('/Users/ye/website/server/Pipfile'):
[[source]]
name = "pypi"
url = "https://pypi.org/simple"
verify_ssl = true

[dev-packages]

[packages]
ipython = "*"
flask = "*"
sqlalchemy = "*"
flask-sqlalchemy = "*"
flask-admin = "*"
flask-login = "*"

[requires]
python_version = "3.7"

Contents of Pipfile.lock ('/Users/ye/website/server/Pipfile.lock'):
{
    "_meta": {
        "hash": {
            "sha256": "39be307e901dd24694fdcd3df287b62eb1265d908ac3d91d854703c3e8acd897"
        },
        "pipfile-spec": 6,
        "requires": {
            "python_version": "3.7"
        },
        "sources": [
            {
                "name": "pypi",
                "url": "https://pypi.org/simple",
                "verify_ssl": true
            }
        ]
    },
    "default": {
        "appnope": {
            "hashes": [
                "sha256:5b26757dc6f79a3b7dc9fab95359328d5747fcb2409d331ea66d0272b90ab2a0",
                "sha256:8b995ffe925347a2138d7ac0fe77155e4311a0ea6d6da4f5128fe4b3cbe5ed71"
            ],
            "markers": "sys_platform == 'darwin'",
            "version": "==0.1.0"
        },
        "backcall": {
            "hashes": [
                "sha256:38ecd85be2c1e78f77fd91700c76e14667dc21e2713b63876c0eb901196e01e4",
                "sha256:bbbf4b1e5cd2bdb08f915895b51081c041bac22394fdfcfdfbe9f14b77c08bf2"
            ],
            "version": "==0.1.0"
        },
        "click": {
            "hashes": [
                "sha256:2335065e6395b9e67ca716de5f7526736bfa6ceead690adf616d925bdc622b13",
                "sha256:5b94b49521f6456670fdb30cd82a4eca9412788a93fa6dd6df72c94d5a8ff2d7"
            ],
            "version": "==7.0"
        },
        "decorator": {
            "hashes": [
                "sha256:86156361c50488b84a3f148056ea716ca587df2f0de1d34750d35c21312725de",
                "sha256:f069f3a01830ca754ba5258fde2278454a0b5b79e0d7f5c13b3b97e57d4acff6"
            ],
            "version": "==4.4.0"
        },
        "flask": {
            "hashes": [
                "sha256:13f9f196f330c7c2c5d7a5cf91af894110ca0215ac051b5844701f2bfd934d52",
                "sha256:45eb5a6fd193d6cf7e0cf5d8a5b31f83d5faae0293695626f539a823e93b13f6"
            ],
            "index": "pypi",
            "version": "==1.1.1"
        },
        "flask-admin": {
            "hashes": [
                "sha256:ca0be6ec11a6913b73f656c65c444ae5be416c57c75638dd3199376ce6bc7422"
            ],
            "index": "pypi",
            "version": "==1.5.3"
        },
        "flask-login": {
            "hashes": [
                "sha256:c815c1ac7b3e35e2081685e389a665f2c74d7e077cb93cecabaea352da4752ec"
            ],
            "index": "pypi",
            "version": "==0.4.1"
        },
        "flask-sqlalchemy": {
            "hashes": [
                "sha256:0078d8663330dc05a74bc72b3b6ddc441b9a744e2f56fe60af1a5bfc81334327",
                "sha256:6974785d913666587949f7c2946f7001e4fa2cb2d19f4e69ead02e4b8f50b33d"
            ],
            "index": "pypi",
            "version": "==2.4.1"
        },
        "ipython": {
            "hashes": [
                "sha256:c4ab005921641e40a68e405e286e7a1fcc464497e14d81b6914b4fd95e5dee9b",
                "sha256:dd76831f065f17bddd7eaa5c781f5ea32de5ef217592cf019e34043b56895aa1"
            ],
            "index": "pypi",
            "version": "==7.8.0"
        },
        "ipython-genutils": {
            "hashes": [
                "sha256:72dd37233799e619666c9f639a9da83c34013a73e8bbc79a7a6348d93c61fab8",
                "sha256:eb2e116e75ecef9d4d228fdc66af54269afa26ab4463042e33785b887c628ba8"
            ],
            "version": "==0.2.0"
        },
        "itsdangerous": {
            "hashes": [
                "sha256:321b033d07f2a4136d3ec762eac9f16a10ccd60f53c0c91af90217ace7ba1f19",
                "sha256:b12271b2047cb23eeb98c8b5622e2e5c5e9abd9784a153e9d8ef9cb4dd09d749"
            ],
            "version": "==1.1.0"
        },
        "jedi": {
            "hashes": [
                "sha256:786b6c3d80e2f06fd77162a07fed81b8baa22dde5d62896a790a331d6ac21a27",
                "sha256:ba859c74fa3c966a22f2aeebe1b74ee27e2a462f56d3f5f7ca4a59af61bfe42e"
            ],
            "version": "==0.15.1"
        },
        "jinja2": {
            "hashes": [
                "sha256:74320bb91f31270f9551d46522e33af46a80c3d619f4a4bf42b3164d30b5911f",
                "sha256:9fe95f19286cfefaa917656583d020be14e7859c6b0252588391e47db34527de"
            ],
            "version": "==2.10.3"
        },
        "markupsafe": {
            "hashes": [
                "sha256:00bc623926325b26bb9605ae9eae8a215691f33cae5df11ca5424f06f2d1f473",
                "sha256:09027a7803a62ca78792ad89403b1b7a73a01c8cb65909cd876f7fcebd79b161",
                "sha256:09c4b7f37d6c648cb13f9230d847adf22f8171b1ccc4d5682398e77f40309235",
                "sha256:1027c282dad077d0bae18be6794e6b6b8c91d58ed8a8d89a89d59693b9131db5",
                "sha256:24982cc2533820871eba85ba648cd53d8623687ff11cbb805be4ff7b4c971aff",
                "sha256:29872e92839765e546828bb7754a68c418d927cd064fd4708fab9fe9c8bb116b",
                "sha256:43a55c2930bbc139570ac2452adf3d70cdbb3cfe5912c71cdce1c2c6bbd9c5d1",
                "sha256:46c99d2de99945ec5cb54f23c8cd5689f6d7177305ebff350a58ce5f8de1669e",
                "sha256:500d4957e52ddc3351cabf489e79c91c17f6e0899158447047588650b5e69183",
                "sha256:535f6fc4d397c1563d08b88e485c3496cf5784e927af890fb3c3aac7f933ec66",
                "sha256:62fe6c95e3ec8a7fad637b7f3d372c15ec1caa01ab47926cfdf7a75b40e0eac1",
                "sha256:6dd73240d2af64df90aa7c4e7481e23825ea70af4b4922f8ede5b9e35f78a3b1",
                "sha256:717ba8fe3ae9cc0006d7c451f0bb265ee07739daf76355d06366154ee68d221e",
                "sha256:79855e1c5b8da654cf486b830bd42c06e8780cea587384cf6545b7d9ac013a0b",
                "sha256:7c1699dfe0cf8ff607dbdcc1e9b9af1755371f92a68f706051cc8c37d447c905",
                "sha256:88e5fcfb52ee7b911e8bb6d6aa2fd21fbecc674eadd44118a9cc3863f938e735",
                "sha256:8defac2f2ccd6805ebf65f5eeb132adcf2ab57aa11fdf4c0dd5169a004710e7d",
                "sha256:98c7086708b163d425c67c7a91bad6e466bb99d797aa64f965e9d25c12111a5e",
                "sha256:9add70b36c5666a2ed02b43b335fe19002ee5235efd4b8a89bfcf9005bebac0d",
                "sha256:9bf40443012702a1d2070043cb6291650a0841ece432556f784f004937f0f32c",
                "sha256:ade5e387d2ad0d7ebf59146cc00c8044acbd863725f887353a10df825fc8ae21",
                "sha256:b00c1de48212e4cc9603895652c5c410df699856a2853135b3967591e4beebc2",
                "sha256:b1282f8c00509d99fef04d8ba936b156d419be841854fe901d8ae224c59f0be5",
                "sha256:b2051432115498d3562c084a49bba65d97cf251f5a331c64a12ee7e04dacc51b",
                "sha256:ba59edeaa2fc6114428f1637ffff42da1e311e29382d81b339c1817d37ec93c6",
                "sha256:c8716a48d94b06bb3b2524c2b77e055fb313aeb4ea620c8dd03a105574ba704f",
                "sha256:cd5df75523866410809ca100dc9681e301e3c27567cf498077e8551b6d20e42f",
                "sha256:e249096428b3ae81b08327a63a485ad0878de3fb939049038579ac0ef61e17e7"
            ],
            "version": "==1.1.1"
        },
        "parso": {
            "hashes": [
                "sha256:63854233e1fadb5da97f2744b6b24346d2750b85965e7e399bec1620232797dc",
                "sha256:666b0ee4a7a1220f65d367617f2cd3ffddff3e205f3f16a0284df30e774c2a9c"
            ],
            "version": "==0.5.1"
        },
        "pexpect": {
            "hashes": [
                "sha256:2094eefdfcf37a1fdbfb9aa090862c1a4878e5c7e0e7e7088bdb511c558e5cd1",
                "sha256:9e2c1fd0e6ee3a49b28f95d4b33bc389c89b20af6a1255906e90ff1262ce62eb"
            ],
            "markers": "sys_platform != 'win32'",
            "version": "==4.7.0"
        },
        "pickleshare": {
            "hashes": [
                "sha256:87683d47965c1da65cdacaf31c8441d12b8044cdec9aca500cd78fc2c683afca",
                "sha256:9649af414d74d4df115d5d718f82acb59c9d418196b7b4290ed47a12ce62df56"
            ],
            "version": "==0.7.5"
        },
        "prompt-toolkit": {
            "hashes": [
                "sha256:46642344ce457641f28fc9d1c9ca939b63dadf8df128b86f1b9860e59c73a5e4",
                "sha256:e7f8af9e3d70f514373bf41aa51bc33af12a6db3f71461ea47fea985defb2c31",
                "sha256:f15af68f66e664eaa559d4ac8a928111eebd5feda0c11738b5998045224829db"
            ],
            "version": "==2.0.10"
        },
        "ptyprocess": {
            "hashes": [
                "sha256:923f299cc5ad920c68f2bc0bc98b75b9f838b93b599941a6b63ddbc2476394c0",
                "sha256:d7cc528d76e76342423ca640335bd3633420dc1366f258cb31d05e865ef5ca1f"
            ],
            "version": "==0.6.0"
        },
        "pygments": {
            "hashes": [
                "sha256:71e430bc85c88a430f000ac1d9b331d2407f681d6f6aec95e8bcfbc3df5b0127",
                "sha256:881c4c157e45f30af185c1ffe8d549d48ac9127433f2c380c24b84572ad66297"
            ],
            "version": "==2.4.2"
        },
        "six": {
            "hashes": [
                "sha256:3350809f0555b11f552448330d0b52d5f24c91a322ea4a15ef22629740f3761c",
                "sha256:d16a0141ec1a18405cd4ce8b4613101da75da0e9a7aec5bdd4fa804d0e0eba73"
            ],
            "version": "==1.12.0"
        },
        "sqlalchemy": {
            "hashes": [
                "sha256:0f0768b5db594517e1f5e1572c73d14cf295140756431270d89496dc13d5e46c"
            ],
            "index": "pypi",
            "version": "==1.3.10"
        },
        "traitlets": {
            "hashes": [
                "sha256:70b4c6a1d9019d7b4f6846832288f86998aa3b9207c6821f3578a6a6a467fe44",
                "sha256:d023ee369ddd2763310e4c3eae1ff649689440d4ae59d7485eb4cfbbe3e359f7"
            ],
            "version": "==4.3.3"
        },
        "wcwidth": {
            "hashes": [
                "sha256:3df37372226d6e63e1b1e1eda15c594bca98a22d33a23832a90998faa96bc65e",
                "sha256:f4ebe71925af7b40a864553f761ed559b43544f8f71746c2d756c7fe788ade7c"
            ],
            "version": "==0.1.7"
        },
        "werkzeug": {
            "hashes": [
                "sha256:7280924747b5733b246fe23972186c6b348f9ae29724135a6dfc1e53cea433e7",
                "sha256:e5f4a1f98b52b18a93da705a7458e55afb26f32bff83ff5d19189f92462d65c4"
            ],
            "version": "==0.16.0"
        },
        "wtforms": {
            "hashes": [
                "sha256:0cdbac3e7f6878086c334aa25dc5a33869a3954e9d1e015130d65a69309b3b61",
                "sha256:e3ee092c827582c50877cdbd49e9ce6d2c5c1f6561f849b3b068c1b8029626f1"
            ],
            "version": "==2.2.1"
        }
    },
    "develop": {}
}

I am having a similar issue on macOS Mojave 10.14.4.
pipenv install psycopg2 fails
However, pipenv run pip install psycopg2 successfully installs psycopg2

^ Expanding on @robertsutherland I had to run pipenv run pip install psycopg2-binary

@HashRocketSyntax
It seems very bad, because when we use the pipenv, we want to record the deps, if pipenv run pip install psycopg2-binary, there is nothing to record in Pipfile/Pipfile.lock.
trying pipenv install psycopg2-binary insead.
if it not works, you(everyone) should consider the bugs of pipenv(and the long-time no-releasing risk) and the benefits of pipenv, then, decide to change to other tool(pip-tools/poetry/or even raw requirements.txt) or keep using this tool

@menghuu that did not work despite checking all homebrew set config and path + adding more libs.

Not an issue related to pipenv
workaround:

Purge the virtualenv by pipenv --rm
Install binary version: pipenv install psycopg2-binary

@frostming can you elaborate why this is not an issue related to pipenv? And why purging virtualenv would make it work?

I heard good things about Pipenv so I thought of giving a go but ran into the issue installing psycopg2 described by others above. I guess I am back to venv and pip but will try again Pipenv in a few months.

@ye Sorry for the late reply
Purging virtualenv is to fix any potential breakage of the venv.

By the way, pipenv install psycopg2-binary didn't work for me.

But your output is about another package google-cloud-python, there is no such package on PyPI, but google-cloud is available.

I've been using pipenv install psycopg2 --sequential (also using --sequential with update and sync). It works for me, but it isn't quite ideal.

Not an issue related to pipenv
workaround:

Purge the virtualenv by pipenv --rm
Install binary version: pipenv install psycopg2-binary

From psycopg installation page:

Note The psycopg2-binary package is meant for beginners to start playing with Python and PostgreSQL without the need to meet the build requirements.
If you are the maintainer of a publish package depending on psycopg2 you shouldn’t use ‘psycopg2-binary’ as a module dependency. For production use you are advised to use the source distribution.

Psycopg devs have already made their point that they won't be fixing this:
https://github.com/psycopg/psycopg2/issues/883
Did anyone find a way to fix this without running additional build steps?

Exporting some flags as per this thread worked for me!
export LDFLAGS="-L/usr/local/opt/openssl/lib" export CPPFLAGS="-I/usr/local/opt/openssl/include"

Exporting some flags as per this thread worked for me!
`export LDFLAGS="-L/usr/local/opt/openssl/lib" export CPPFLAGS="-I/usr/local/opt/openssl/include"

Thanks @vignzpie ! You solved it for me :)

installing dependencies solved it for me
$ sudo apt-get install libpq-dev

installing dependencies solved it for me
$ sudo apt-get install libpq-dev

@pawelrubin my issue was filed for macOS not on linux as you can see in the log: 'platform_system': 'Darwin'.

@ye then you should take either of the following solutions:

Use psycopg2-binary package instead
Install postgres via homebrew and expose the source header files by modifying LDFLAGS and CPPFLAGS env vars.

@frostming yes it was due to a openssl lib linking issue while compiling psycopg2 from source. Thanks.
Both 1 and 2 work now after I upgrade to python 3.8 and installing psychopg2 version 2.8.5 but 1 is not a recommended solution as @shaurmanchic pointed out (https://github.com/pypa/pipenv/issues/3991#issuecomment-564498875)

Exporting some flags as per this thread worked for me!
export LDFLAGS="-L/usr/local/opt/openssl/lib"
export CPPFLAGS="-I/usr/local/opt/openssl/include"

@vignzpie 's solution worked for me on psycopg2 version 2.8.5

If you're here on Linux mint 19 Cinnamon 4.6.7
installing dependencies solved it for me:

sudo apt-get install libpq-dev
pipenv run pip install psycopg2

installing dependencies solved it for me
$ sudo apt-get install libpq-dev

this also help

pipenv install psycopg2-binary is Working on Windows 10 with Python 3.8 as of today.

Install Python 3.8 from the Microsoft Store
Install postgresql-12.5-1-windows-x64 (version 13 is not yet supported!)
Use Application Stack Builder to install PostGIS 3.1
Install osgeo4w-setup-x86_64 (say no to all prompts to set variables)
Modify the Windows Environment (see code block below)
pipenv install psycopg2-binary

set OSGEO4W_ROOT=C:\OSGeo4W64
set GDAL_DATA=%OSGEO4W_ROOT%\share\gdal
set PROJ_LIB=%OSGEO4W_ROOT%\share\proj
set PATH=%PATH%;%OSGEO4W_ROOT%\bin
reg ADD "HKLM\SYSTEM\CurrentControlSet\Control\Session Manager\Environment" /v Path /t REG_EXPAND_SZ /f /d "%PATH%"
reg ADD "HKLM\SYSTEM\CurrentControlSet\Control\Session Manager\Environment" /v GDAL_DATA /t REG_EXPAND_SZ /f /d "%GDAL_DATA%"
reg ADD "HKLM\SYSTEM\CurrentControlSet\Control\Session Manager\Environment" /v PROJ_LIB /t REG_EXPAND_SZ /f /d "%PROJ_LIB%"

These steps were documented in the GeoDjango Windows installation instructions.

pipenv install psycopg2-binary is Working on Windows 10 with Python 3.8 as of today.

Install Python 3.8 from the Microsoft Store
Install postgresql-12.5-1-windows-x64 (version 13 is not yet supported!)
Use Application Stack Builder to install PostGIS 3.1
Install osgeo4w-setup-x86_64 (say no to all prompts to set variables)
Modify the Windows Environment (see code block below)
pipenv install psycopg2-binary

set OSGEO4W_ROOT=C:\OSGeo4W64
set GDAL_DATA=%OSGEO4W_ROOT%\share\gdal
set PROJ_LIB=%OSGEO4W_ROOT%\share\proj
set PATH=%PATH%;%OSGEO4W_ROOT%\bin
reg ADD "HKLM\SYSTEM\CurrentControlSet\Control\Session Manager\Environment" /v Path /t REG_EXPAND_SZ /f /d "%PATH%"
reg ADD "HKLM\SYSTEM\CurrentControlSet\Control\Session Manager\Environment" /v GDAL_DATA /t REG_EXPAND_SZ /f /d "%GDAL_DATA%"
reg ADD "HKLM\SYSTEM\CurrentControlSet\Control\Session Manager\Environment" /v PROJ_LIB /t REG_EXPAND_SZ /f /d "%PROJ_LIB%"

These steps were documented in the GeoDjango Windows installation instructions.

$ sudo apt-get install libpq-dev

You saved me !
Thank you!

pipenv install psycopg2-binary==2.8.5 wasn't working for me. The comments involving LDFLAGS and CPPFLAGS didn't resolve my issue, but this appears to have:

I followed instructions to install PostgreSQL at https://www.postgresql.org/download/macosx/ (I have a mac)
I added this line to my ~/.zshrc: export PATH="/Library/PostgreSQL/13/bin:$PATH" (Replace the 13 with whatever version you download)

$ sudo apt-get install libpq-dev
$ pipenv --rm
$ pipenv install
$ pipenv install psycopg2

Ubuntu 20.04 WSL2 in Windows 10 ..

Using ubuntu 20.04. with multiple python versions 3.8, 3.7..etc and working in a django project using python 3.7 with pipenv, the following worked for me:
Solution:
sudo apt-get update
sudo apt-get install libpq-dev python3.7-dev
pipenv install --python=3.7 psycopg2
pipenv shell

Who so ever is struggling and is unable to install psycopg2 and refer to this comment,
To install psycopg2 in Mac :

run command : export PATH=$PATH:/Applications/Postgres.app/Contents/Versions/14/bin
Make sure you have your version of Postgres.app written in the command in place of 14(which is mine version.)

pipenv install wheel

pipenv install setuptools

pipenv install psycopg2.

and, It worked for me.

I updated python version, and it worked fine for me.

Which version of python are you using ? and did you installed it on mac or windows ?

Which version of python are you using ? and did you installed it on mac or windows ?

I am using Fedora Linux. When I reinstalled the pipenv environment with python 3.10 it worked well.

The problem is there for mac, I tried installing it on mac with the latest version of python, still it was not working.When I followed the above steps only then it worked.

Here in 2024, same error, different solution:
I had the package, psycopg2-binary==* installed in a different repo and ran pipenv graph to identify the version installed there. It happened to be psycopg2-binary==2.9.9. I went to my current repo and specified in the Pipfile, "psycopg2-binary" = "==2.9.9". Then I ran pipenv install and everything worked as intended. I believe specifying the version directly managed to fix something. But I'm not entirely certain why this worked.


I have mixed up colors and sections

tasks with errors are in the priority section, for example

Hi @OlexanderKulyk thank you for reporting that, how can I reproduce this issue?

"@bull-board/nestjs": "^5.9.1",
"@nestjs/bull": "^10.0.1",
"@nestjs/common": "^10.0.0",
"@nestjs/core": "^10.0.0",

I guess this came with the new version of nest

Can you create a minimal repo that reproduces this issue?

This issue probably related to https://github.com/felixmosh/bull-board/issues/650#issuecomment-1827771876
Check that you are using bullAdapter and not the bullMQAdapater


Distance away is displayed even when location is not shared

I noticed that even if I do not allow the website to use my location, the results still show a "# mi away." I expect that if I block my location that it would not have these associated distances. Another thought is that there would be a central part of the map marked and these distances would be relevant to that central map marker.
This relates to issue #137.

Good point. I think this was a legacy of early work where Beck Lake was the default and we didn't rethink this approach--just deciding to move it to a different arbitrary point (in Chicago).
If you didn't share location... that information isn't super useful, could be confusing and can border on creepy.
@shua123 Can we remove these if location is not shared? But add them back after a muni/ZIP search?

Done. If distances are being calculated by using distance from Config.mapCenter, distance won't display in list otherwise it will.
https://github.com/smartchicago/trailsy/commit/c2cf067339c79cdecd45520d78a1cc20b56eed9a

Looks good. Would it be possible to do the list in alpha order? It looks like it's still ordered by distance from Chicago right now.

Yes, now implemented. Sorting by name alpha if there is no search/user location. https://github.com/smartchicago/trailsy/commit/2eb1ef29ba3f8d409ff489f62c1f79765e1f6740

Trails are showing up out of order (following the first POI they are associated with). Not sure if there is a quick fix to this. If not, something we can address later:

Yeah, Trail divs in the list are entirely associated with the POI. Basic process:

Sort Filtered POIs by distance/alphabetical
Iterate through array of Filtered POIs.

Create poi div
Is there an associated Trail_Subsystem? Has a div been created? If no, create a trail subsystem div


WebView showing ERR_CACHE_MISS on app production build

I have same error even thought in androidmanifest.xml the line  uses-permission android:name="android.permission.INTERNET is written.
Originally posted by @anandios in https://github.com/flutter/flutter/issues/55815#issuecomment-818568710

Hi @anandios
Can you please provide your flutter doctor -v and a minimal complete reproducible code sample?
Thank you


Fix non-linear incrementing of session index

This ticket name needs to be improved! And fleshed out with some example data of the problem.

I'll update it once I've narrowed it down.

Still stuck on this one, trying to find a pattern with other users having encountered the same issue.

We're thinking configHeartBeatTimer might somehow be null here which would cause the callback to happen relentlessly but this seems a bit far-fetched.

Hey @BenFradet - configHeartBeatTimer - why don't we create a new ticket that puts a guard on the value of this, rather than just && configHeartBeatTimer? I tried a setInterval with "" as the interval and yes it just triggered relentlessly...

@alexanderdean Do you propose having a default value when it's null?

Hmm, no I'd probably just put guards onto both args in this function:
https://github.com/snowplow/snowplow-javascript-tracker/blob/0ba1bad19e276d381ffe214669042dde17a4d9fd/src/js/tracker.js#L1680-L1683
If either arg is not an int, then don't enable. Less magic that way.

This has altered considerably since this issue was created, including the addition of guarding to prevent relentless callbacks. Likely no longer an issue.


Change the table size

Hi thanks for the lovely tool! I wonder if it is possible to make the table a bit larger to show longer variable name?
|   iter    |  target   | batch_... | schedu... |

https://github.com/fmfn/BayesianOptimization/blob/202dc3cc133d5fc5e787762863490acc101ef8e7/bayes_opt/logger.py#L14
could change here I reckon

Hey @xihajun,
off the top of my head, something like this would work:
from bayes_opt import BayesianOptimization

def black_box_function(a_very_long_variable_name, y):
    """Function with unknown internals we wish to maximize.

    This is just serving as an example, for all intents and
    purposes think of the internals of this function, i.e.: the process
    which generates its output values, as unknown.
    """
    return -a_very_long_variable_name ** 2 - (y - 1) ** 2 + 1

pbounds = {'a_very_long_variable_name': (2, 4), 'y': (-3, 3)}

from bayes_opt.logger import ScreenLogger
from bayes_opt.event import Events

logger = ScreenLogger()
logger._default_cell_size = 20

optimizer = BayesianOptimization(
    f=black_box_function,
    pbounds=pbounds,
    verbose=2, # verbose = 1 prints only when a maximum is observed, verbose = 0 is silent
    random_state=1,
)

for event in [Events.OPTIMIZATION_START, Events.OPTIMIZATION_STEP, Events.OPTIMIZATION_END]:
    optimizer.subscribe(event, logger)

optimizer.maximize(
    init_points=2,
    n_iter=3,
)

Output:
|         iter         |        target        | a_very_long_varia... |          y           |
---------------------------------------------------------------------------------------------
| 1                    | -7.135               | 2.834                | 1.322                |
| 2                    | -7.78                | 2.0                  | -1.186               |
| 3                    | -7.11                | 2.218                | -0.7867              |
| 4                    | -12.4                | 3.66                 | 0.9608               |
| 5                    | -6.999               | 2.23                 | -0.7392              |
=============================================================================================

Of course you can modify your local version of this package to adjust the cell size, as you have discovered.

Thank you so much! @till-m
I will close issue then :)
Alternative way was change the default value here: /lib/python3.8/site-packages/bayes_opt/logger.py at line 14

Hi @till-m, I wonder if there is a way that we can change the table's target to another name result?

maybe I can try to make a pull request if you think might be helpful?

Hey @xihajun,
this name is currently hardcoded: https://github.com/fmfn/BayesianOptimization/blob/202dc3cc133d5fc5e787762863490acc101ef8e7/bayes_opt/logger.py#L96
It would be possible to set it as an (underscore-prepended) property of the model, e.g. ScreenLogger._target_column_title. While I can't imagine that many people need to change the column names, I would not oppose this change since it is very minor. I probably wouldn't opt for adding a setter method though.
@bwheelz36 do you have an opinion on this matter?

I agree, that change would not be a problem but I also wouldn't imagine it being used often.


wp_enqueue_script option

Is it possible to add an option within the theme-options if I want to load via wp_enqueue_script? because the reason why I actually did choose roots was because of it 'hardcoded' stylesheets and scripts.
And if not, can anyone then give some feedback how I can load the scripts manually again then?

what are your reasons for wanting to do that?
also, there's no more theme options (replaced with a more simple roots-config.php)

The reason I want unregistered scripts or 'hardcoded' is because I can then decide exactly how the scripts should be loaded (a custom build of head.js)

roots-scripts.php can really be extended to do lots of things, including taking the enqueued scripts and integrate something like head.js or Modernizr.load
check out http://wordpress.stackexchange.com/a/27783 (roots-scripts.php is based off the examples in the replies)

Ok, those examples are a little overkill, I will deregister the scripts again, it's easier.


Add unit test to swiftTCR

Add some basic unit tests to test if the code is working as expected

Created unit test to the code and can be found under the folder test in swiftTCR


Measurement bottleneck on small benchmarks

If you have a benchmark with test cases that run very quickly, you can't get an idea of the performance difference because they all seem to hit the same "ceiling" in ops/s.
Take these trivial cases: https://jsbm.dev/hEJNNQQm5WzIR
On my machine, in Firefox these all hit ~3.7m ops/s, and on Chrome they hit ~4.8m ops/sec. In reality, these test cases run even faster -- if I run the same cases in https://jsbench.me/ for example (link), these are at least 2 orders of magnitude faster in terms of ops/s. I'm not sure if this is just an inherent trade-off of the worker approach.
I did a bit of profiling and noticed that in the case of these tiny benchmarks that performance.now contributes to a large amount of the runtime:

Whereas for jsbench, it seems to use Date.now for measuring which doesn't even show up on the chart. I have tried a few things but nothing I do is able to close the gap (all measured in Firefox, independent of one another):

Swap performance.now for Date.now is only slightly (3.7m ops/s -> 4.1m ops/s) faster and is still a significant proportion of the measurements.
Use a synchronous function for running the code gave a decent bump (3.7m ops/s -> 6.2m ops/s)
Reduce performance.now() calls by only checking for time limit every 1,000 iterations gave another bump (3.7m ops/s -> 5.2m ops/s)

If I make all the above changes I can hit around 10m ops/s, but it's still nowhere near something like jsbench.
Edit: With some more experimenting I think there is some worker overhead but not that much -- see https://jsbm.dev/WjgHnhqJhhBXv, where I run each benchmark 100 times, on my machine I'm finally able to see some performance difference between the first test case and the rest. If I bump it up to 1000 times, I can see even more differentiation. Doing the math backwards I don't quite get to the numbers of jsbench but it's at least within the same order of magnitude

So I had a bit of a goof and switching from an async function to a sync function actually brings the difference MUCH closer, as it turns out even a useless await on a synchronous function would still slow this down quite a bit. With all of the changes from above applied I get fairly close (only 4-5x worse than jsbench):

So I finally had some time to look into this.
I implemented your suggestions/ideas (with a few modifications) in this PR and the ops/s went from ~5m to ~725m, and much closer to jsbench. You can also now actually tell a difference between the different cases without adding loops to the individual tests.

The changes are not yet live, but you can preview and test them here.
Thanks again for your investigation! 🫶

Awesome, looks great!

Fixed by https://github.com/jsbenchmark/jsbenchmark/pull/8


Routing broken in rc1 w/ swagger

Forking and updating another example for PackageReference Include="SwashbuckleAspNetVersioningShim" Version="1.0.0-beta2"/>, routing of the api endpoints fails with rc1 but works as expected with beta2. The controller endpoints for any version are never hit. Falling back to beta2 resolves the issues.
  <ItemGroup>
    <PackageReference Include="Microsoft.AspNetCore" Version="1.1.1" />
    <PackageReference Include="Microsoft.AspNetCore.Mvc" Version="1.1.2" />
    <!--<PackageReference Include="Microsoft.AspNetCore.Mvc.Versioning" Version="1.1.0-rc1"/>-->
    <PackageReference Include="Microsoft.AspNetCore.Mvc.Versioning" Version="1.1.0-beta2"/>
    <PackageReference Include="Swashbuckle.AspNetCore" Version="1.0.0" />
    <PackageReference Include="SwashbuckleAspNetVersioningShim" Version="1.0.0-beta2"/>
  </ItemGroup>

Complete example at https://github.com/senften/AspNetCoreVersionedWebApi (This is a fork of https://github.com/rh072005/SwashbuckleAspNetVersioningShim)

It appears that your example is no longer correct. In the Startup.cs, it does not contain UseApiVersioning. Unfortunately, this is behavioral breaking change that is required in order to enable route constraints (amongst other things) across API versions. This only affects ASP.NET Core.  The details are called out in the v1.1 RC release notes.
Add the following and you should be back in business:
app.UseMvc().UseApiVersioning();

Thank you very much. That, indeed, works. My apologies. I should have checked the release notes.


Move the functionality to start a proxying session to UserInterface

Move the functionality to start a proxying session to UserInterface
Starting a connection through a given user is an action that is being
taken place on the level of the entire UI, not specific to a single
instance.  This moves the functionality to actually do that out of the
instance code to the top-level.  By doing this, we are able to get
better checking of whether there was a problem connecting or whether a
user aborted the connection.
To keep track of the current status of our connection, we now use a
variable in the UI that is set to whatever instance we are trying to get
access through.
Fixes #1409

:+1:

Comments from the review on Reviewable.io


on dev't system only: exec ./dbupgrade_and_run.sh: no such file or directory

file needs to have end of line as LF, not CRLF. See https://stackoverflow.com/a/58220487/799921


InvalidParameterValueException

ERROR [2017-06-21 21:40:01,884] [io.github.azagniotov.metrics.reporter.cloudwatch.CloudWatchReporter] Error reporting metrics to CloudWatch. The data in this CloudWatch API request may have been discarded, did not make it to CloudWatch.
com.amazonaws.services.cloudwatch.model.InvalidParameterValueException: The value ? for parameter MetricData.member.3.Value is invalid.

According to https://github.com/blacklocus/metrics-cloudwatch/issues/4 this occurs when the sample count is zero. In that implementation those metrics were filtered away. Does this one require a custom MetricFilter to do likewise? Could the same fix be incorporated?

I think this is because the zero filter is only applied in the case of a counter. I think it should be done in stageMetricDatum by checking the metricValue. An example request that fails is,
INFO  [2017-06-21 23:20:48,719] [io.github.azagniotov.metrics.reporter.cloudwatch.CloudWatchReporter] Dry run - constructed PutMetricDataRequest: {Namespace: LoadDocs/Server,MetricData: [{MetricName: HikariPool-1.pool.ActiveConnections,Dimensions: [{Name: Region,Value: us-west-2}, {Name: , Environment,Value: 'dev'}, {Name: Type,Value: gauge}],Timestamp: Wed Jun 21 16:12:56 PDT 2017,Value: 0.0,Unit: None}, {MetricName: HikariPool-1.pool.IdleConnections,Dimensions: [{Name: Region,Value: us-west-2}, {Name: , Environment,Value: 'dev'}, {Name: Type,Value: gauge}],Timestamp: Wed Jun 21 16:13:26 PDT 2017,Value: 30.0,Unit: None}, {MetricName: HikariPool-1.pool.PendingConnections,Dimensions: [{Name: Region,Value: us-west-2}, {Name: , Environment,Value: 'dev'}, {Name: Type,Value: gauge}],Timestamp: Wed Jun 21 16:13:28 PDT 2017,Value: 0.0,Unit: None}, {MetricName: HikariPool-1.pool.TotalConnections,Dimensions: [{Name: Region,Value: us-west-2}, {Name: , Environment,Value: 'dev'}, {Name: Type,Value: gauge}],Timestamp: Wed Jun 21 16:13:33 PDT 2017,Value: 30.0,Unit: None}, {MetricName: PS-MarkSweep.count,Dimensions: [{Name: Region,Value: us-west-2}, {Name: , Environment,Value: 'dev'}, {Name: Type,Value: gauge}],Timestamp: Wed Jun 21 16:13:33 PDT 2017,Value: 3.0,Unit: None}, {MetricName: PS-MarkSweep.time,Dimensions: [{Name: Region,Value: us-west-2}, {Name: , Environment,Value: 'dev'}, {Name: Type,Value: gauge}],Timestamp: Wed Jun 21 16:13:33 PDT 2017,Value: 291.0,Unit: None}, {MetricName: PS-Scavenge.count,Dimensions: [{Name: Region,Value: us-west-2}, {Name: , Environment,Value: 'dev'}, {Name: Type,Value: gauge}],Timestamp: Wed Jun 21 16:13:33 PDT 2017,Value: 11.0,Unit: None}, {MetricName: PS-Scavenge.time,Dimensions: [{Name: Region,Value: us-west-2}, {Name: , Environment,Value: 'dev'}, {Name: Type,Value: gauge}],Timestamp: Wed Jun 21 16:13:33 PDT 2017,Value: 175.0,Unit: None}, {MetricName: blocked.count,Dimensions: [{Name: Region,Value: us-west-2}, {Name: , Environment,Value: 'dev'}, {Name: Type,Value: gauge}],Timestamp: Wed Jun 21 16:13:33 PDT 2017,Value: 0.0,Unit: None}, {MetricName: com.codahale.metrics.jvm.FileDescriptorRatioGauge,Dimensions: [{Name: Region,Value: us-west-2}, {Name: , Environment,Value: 'dev'}, {Name: Type,Value: gauge}],Timestamp: Wed Jun 21 16:13:33 PDT 2017,Value: 0.24365234375,Unit: None}, {MetricName: count,Dimensions: [{Name: Region,Value: us-west-2}, {Name: , Environment,Value: 'dev'}, {Name: Type,Value: gauge}],Timestamp: Wed Jun 21 16:13:33 PDT 2017,Value: 109.0,Unit: None}, {MetricName: daemon.count,Dimensions: [{Name: Region,Value: us-west-2}, {Name: , Environment,Value: 'dev'}, {Name: Type,Value: gauge}],Timestamp: Wed Jun 21 16:13:33 PDT 2017,Value: 53.0,Unit: None}, {MetricName: deadlock.count,Dimensions: [{Name: Region,Value: us-west-2}, {Name: , Environment,Value: 'dev'}, {Name: Type,Value: gauge}],Timestamp: Wed Jun 21 16:13:33 PDT 2017,Value: 0.0,Unit: None}, {MetricName: direct.capacity,Dimensions: [{Name: Region,Value: us-west-2}, {Name: , Environment,Value: 'dev'}, {Name: Type,Value: gauge}],Timestamp: Wed Jun 21 16:13:33 PDT 2017,Value: 2.69517087E8,Unit: None}, {MetricName: direct.count,Dimensions: [{Name: Region,Value: us-west-2}, {Name: , Environment,Value: 'dev'}, {Name: Type,Value: gauge}],Timestamp: Wed Jun 21 16:13:33 PDT 2017,Value: 38.0,Unit: None}, {MetricName: direct.used,Dimensions: [{Name: Region,Value: us-west-2}, {Name: , Environment,Value: 'dev'}, {Name: Type,Value: gauge}],Timestamp: Wed Jun 21 16:13:33 PDT 2017,Value: 2.69517088E8,Unit: None}, {MetricName: heap.committed,Dimensions: [{Name: Region,Value: us-west-2}, {Name: , Environment,Value: 'dev'}, {Name: Type,Value: gauge}],Timestamp: Wed Jun 21 16:13:33 PDT 2017,Value: 7.27711744E8,Unit: None}, {MetricName: heap.init,Dimensions: [{Name: Region,Value: us-west-2}, {Name: , Environment,Value: 'dev'}, {Name: Type,Value: gauge}],Timestamp: Wed Jun 21 16:13:33 PDT 2017,Value: 2.68435456E8,Unit: None}, {MetricName: heap.max,Dimensions: [{Name: Region,Value: us-west-2}, {Name: , Environment,Value: 'dev'}, {Name: Type,Value: gauge}],Timestamp: Wed Jun 21 16:13:33 PDT 2017,Value: 5.726797824E9,Unit: None}, {MetricName: heap.usage,Dimensions: [{Name: Region,Value: us-west-2}, {Name: , Environment,Value: 'dev'}, {Name: Type,Value: gauge}],Timestamp: Wed Jun 21 16:13:33 PDT 2017,Value: 0.09173450157405103,Unit: None}]}

Hi.. Thank you for that. Yeah, I saw that they have a fix for this. Can you issue a PR?

I switched to their library, but I still see error, so it must not have been fully resolved.
Regarding this library, I'm a little concerned that it does the rollups instead of delegating that to the analyzer. Usually one exports monotonically increasing values, report into a time series database, and compute on the dashboard. A quick glance again and not sure if you're providing extra information or not reporting the raw counts.

You are right. I just need to find the time to address this. Thank you for the input!

@ben-manes FYI, @williedoran has addressed an issue where histogram statistic sets were reported by applying conversion by a factor. Now, histogram is reported as is:
https://github.com/azagniotov/codahale-aggregated-metrics-cloudwatch-reporter/pull/6
Thoughts?

Sorry, I'm a bit too swamped to switch contexts and help out. But switching back is pretty quick, so I can give that a shot when you have a release.

Ben, I appreciate your help & time.  Feel free to give it a go when you got time. Cheers

Thank you! :)

Actually, I just released 1.0.5. I think it should be on Maven in a few hours. Thanks!

Sorry, I forgot about this. I reverted back to the original code I had for using this library, with the latest version of course. It sounds like you fixed this so I'll reopen if the issue persists.
Thanks!

@azagniotov Not sure if this is related, but I'm seeing this strange error in the logs.
INFO  [2017-09-05 18:38:14,518] [com.amazonaws.http.DefaultErrorResponseHandler] Unable to parse HTTP response (Invocation Id:c8ef912c-35d9-197e-0788-dc703bbb1b6f) content to XML document ''
org.xml.sax.SAXParseException: Premature end of file.
	at org.apache.xerces.util.ErrorHandlerWrapper.createSAXParseException(Unknown Source)
	at org.apache.xerces.util.ErrorHandlerWrapper.fatalError(Unknown Source)
	at org.apache.xerces.impl.XMLErrorReporter.reportError(Unknown Source)
	at org.apache.xerces.impl.XMLErrorReporter.reportError(Unknown Source)
	at org.apache.xerces.impl.XMLVersionDetector.determineDocVersion(Unknown Source)
	at org.apache.xerces.parsers.XML11Configuration.parse(Unknown Source)
	at org.apache.xerces.parsers.XML11Configuration.parse(Unknown Source)
	at org.apache.xerces.parsers.XMLParser.parse(Unknown Source)
	at org.apache.xerces.parsers.DOMParser.parse(Unknown Source)
	at org.apache.xerces.jaxp.DocumentBuilderImpl.parse(Unknown Source)
	at javax.xml.parsers.DocumentBuilder.parse(DocumentBuilder.java:121)
	at com.amazonaws.util.XpathUtils.documentFrom(XpathUtils.java:162)
	at com.amazonaws.util.XpathUtils.documentFrom(XpathUtils.java:169)
	at com.amazonaws.http.DefaultErrorResponseHandler.parseXml(DefaultErrorResponseHandler.java:124)
	at com.amazonaws.http.DefaultErrorResponseHandler.documentFromContent(DefaultErrorResponseHandler.java:105)
	at com.amazonaws.http.DefaultErrorResponseHandler.createAse(DefaultErrorResponseHandler.java:84)
	at com.amazonaws.http.DefaultErrorResponseHandler.handle(DefaultErrorResponseHandler.java:71)
	at com.amazonaws.http.DefaultErrorResponseHandler.handle(DefaultErrorResponseHandler.java:47)
	at com.amazonaws.http.AwsErrorResponseHandler.handleAse(AwsErrorResponseHandler.java:50)
	at com.amazonaws.http.AwsErrorResponseHandler.handle(AwsErrorResponseHandler.java:38)
	at com.amazonaws.http.AwsErrorResponseHandler.handle(AwsErrorResponseHandler.java:24)
	at com.amazonaws.http.AmazonHttpClient$RequestExecutor.handleErrorResponse(AmazonHttpClient.java:1569)
	at com.amazonaws.http.AmazonHttpClient$RequestExecutor.executeOneRequest(AmazonHttpClient.java:1257)
	at com.amazonaws.http.AmazonHttpClient$RequestExecutor.executeHelper(AmazonHttpClient.java:1029)
	at com.amazonaws.http.AmazonHttpClient$RequestExecutor.doExecute(AmazonHttpClient.java:741)
	at com.amazonaws.http.AmazonHttpClient$RequestExecutor.executeWithTimer(AmazonHttpClient.java:715)
	at com.amazonaws.http.AmazonHttpClient$RequestExecutor.execute(AmazonHttpClient.java:697)
	at com.amazonaws.http.AmazonHttpClient$RequestExecutor.access$500(AmazonHttpClient.java:665)
	at com.amazonaws.http.AmazonHttpClient$RequestExecutionBuilderImpl.execute(AmazonHttpClient.java:647)
	at com.amazonaws.http.AmazonHttpClient.execute(AmazonHttpClient.java:511)
	at com.amazonaws.services.cloudwatch.AmazonCloudWatchClient.doInvoke(AmazonCloudWatchClient.java:1320)
	at com.amazonaws.services.cloudwatch.AmazonCloudWatchClient.invoke(AmazonCloudWatchClient.java:1296)
	at com.amazonaws.services.cloudwatch.AmazonCloudWatchClient.executePutMetricData(AmazonCloudWatchClient.java:1204)
	at com.amazonaws.services.cloudwatch.AmazonCloudWatchAsyncClient$14.call(AmazonCloudWatchAsyncClient.java:774)
	at com.amazonaws.services.cloudwatch.AmazonCloudWatchAsyncClient$14.call(AmazonCloudWatchAsyncClient.java:768)
	at java.util.concurrent.FutureTask.run(FutureTask.java:266)
	at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1142)
	at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:617)
	at java.lang.Thread.run(Thread.java:748)


Error message when try to blur a mask.

When I try to mask a small image and blur his mask that I got this message error on screen shot.

I have the same issue. Did your solved the problem?


Extend langchain.callbacks documentation

Hi to whoever is reading this! 🤗
What's in this PR?
As of the recent addition of ArgillaCallbackHandler in the master branch of LangChain, merged by @dev2049 (but maybe also relevant to @agola11 since it's callback-related), @davidberenstein1957 and @dvsrepo noticed that the Callbacks -> How to guides -> Argilla page was missing, so I've extended the langchain.callbacks documentation a little bit to include the "How to guides" to also allow any other callback integration to live there, and add the missing API Reference for the callbacks.

Hi again @leo-gan, I've fixed the paths to point to the relative path instead of the URL to be generated, but I saw some other references in other pages (not edited in this PR) also point to the URLs to be generated instead of the relative path, is that something we should look into?

Hi @leo-gan and @agola11 I've already addressed the comments, and deployed the docs locally and those now work fine! Feel free to merge whenever you want 😄

Cool, then I'm already done, feel free to merge whenever applicable! 🤗

As of https://github.com/hwchase17/langchain/pull/6300, the docs skeleton has changed and those issues have already been solved, so this PR will be closed! 😄 Thanks for the support in the meantime @leo-gan and @agola11


Reaper reaps specific fields from nested source fields

As a data curator, I would like to ingest records from complex json datasets and apis in a way that I can configure.
AC:

[ ] selector in a schema can be used to fill a schema field from a nested path
[ ] selector should use jpath format.

[
  {
    "data": {
      "usefulField": "field",
      "otherUsefulField": "thing"
    }
  },
  {
    "data": {
      "usefulField": "field2",
      "otherUsefulField": "thing2"
    }
  }
]

could be described as
{
  "name": "usefulField",
  "type": "integer",
  ...
  "selector": "data.usefulField"
},
{
  "name": "otherUsefulField",
  "type": "integer",
  ...
  "selector": "data.otherUsefulField"
}

Tech Notes:

There does not seem to be a jpath implementation for Elixir at the moment.

I really like going in this direction.


Links in field values

Hello. I've been using this hook for an app I'm building that uses a Google Sheet as the data source. It's been great to use and super easy to set up (specially compared to a lot of the other options out there) but I've got an issue where a column I need to view has a link in the data.

This is just returning as a flat string value like "TRK-FORD-13-433" rather than the actual contents of the cell I was expecting.
If I'm just not doing something correctly then please let me know, otherwise is there a way around this?
Thanks

Thanks for the reply. I'll have a look at the Google APIs which I was really trying to avoid.


sum_over_sample does not work with a block of zero samples

With the new AtomicComposition calculator we can produce a descriptor that has gradients where all of them are zero. These zeros are not stored explicitly but only an empty array is saved. This causes problems for the sum_over_samples function as the below example shows
import equistore
from equistore import Labels, TensorBlock, TensorMap

block = TensorBlock(
    values = np.zeros([2,1]),
    properties=Labels(["prop"], np.zeros([1,1], dtype=int)),
    samples=Labels(["structure"], np.array([[0], [1]])),
    components=[]
)

block.add_gradient(
    parameter="positions",
    data=np.zeros((0, 1)),
    samples=Labels(["sample", "structure"], np.empty((0, 2), dtype=np.int32)),
    components=[],
)

tensor = TensorMap(Labels.single(), [block])

equistore.sum_over_samples(tensor, "structure")

The last line will raise an error
---------------------------------------------------------------------------
ValueError                                Traceback (most recent call last)
Cell In[23], line 1
----> 1 equistore.sum_over_samples(tensor, "structure")

File /opt/homebrew/lib/python3.11/site-packages/equistore/operations/reduce_over_samples.py:281, in sum_over_samples(tensor, samples_names)
    219 def sum_over_samples(
    220     tensor: TensorMap, samples_names: Union[List[str], str]
    221 ) -> TensorMap:
    222     """
    223     Create a new :py:class:`TensorMap` with the same keys as as the input
    224     ``tensor``, and each :py:class:`TensorBlock` is obtained summing the
   (...)
    278     :param samples_names: names of samples to sum over
    279     """
--> 281     return _reduce_over_samples(
    282         tensor=tensor, samples_names=samples_names, reduction="sum"
    283     )

File /opt/homebrew/lib/python3.11/site-packages/equistore/operations/reduce_over_samples.py:210, in _reduce_over_samples(tensor, samples_names, reduction)
    207 blocks = []
    208 for _, block in tensor:
    209     blocks.append(
--> 210         _reduce_over_samples_block(
    211             block=block,
...
    104 # change the first columns of the samples array with the mapping
    105 # between samples and gradient.samples
    106 samples[:, 0] = index[samples[:, 0]]

ValueError: cannot reshape array of size 0 into shape (0,newaxis)

This also holds for pure blocks without gradients:
import equistore
from equistore import Labels, TensorBlock, TensorMap

block = TensorBlock(
    values = np.zeros([0, 1]),
    properties=Labels(["prop"], np.zeros([1, 1], dtype=int)),
    samples=Labels(["structure"], np.empty((0, 1), dtype=np.int32)),
    components=[]
)

tensor = TensorMap(Labels.single(), [block])

equistore.sum_over_samples(tensor, "structure")

As already discussed with @DavideTisi we should add a check that if the first dimension of the gradients samples are zero we simply ignore this gradient.

@Luthaf is this behavior of having zero sample supposed to be possible or the TensorBlock should catch that?

That's 100% possible! From a sparsity point of view, it just means that the gradients are 0 everywhere

I understand the meaning just asking why you want to have the object if everything is zeros, also asking for the single block


[Azure.Core] Disable TCP_FAST_OPEN to avoid a WinHttp issue

There is an issue with TCP_FAST_OPEN when the address is invalid or not found.

Fixes https://github.com/Azure/azure-sdk-for-cpp/issues/4490


Adds a configurable timeout for builds

cc @ladyada
This pull request adds an optional configurable timeout parameter, --build_timeout, for building larger Arduino sketches or  "firmware using Arduino" (like WipperSnapper).
Without this pull request, calls to arduino-cli (and UF2 generation) from within this script will time out and fail without any output.
Tested within Adafruit WipperSnapper PR: https://github.com/adafruit/Adafruit_Wippersnapper_Arduino/pull/488
Example failure run, without --build_timeout for ESP32 builds (which take a while to run and fail if they timeout): https://github.com/adafruit/Adafruit_Wippersnapper_Arduino/actions/runs/6174986510/job/16760853258
Example successful run, with --build_timeout enabled for all builds: https://github.com/adafruit/Adafruit_Wippersnapper_Arduino/actions/runs/6174857439

looks good go ahead and merge
this email was probably sent with text to speech, thank you for
understanding any typos or missing punctuation 😊
On Wed, Sep 13, 2023 at 12:10 PM Brent Rubell @.***>
wrote:

cc @ladyada https://github.com/ladyada
This pull request adds an optional configurable timeout parameter,
--build_timeout, for building larger Arduino sketches or "firmware using
Arduino" (like WipperSnapper).
Without this pull request, calls to arduino-cli (and UF2 generation) from
within this script will time out and fail without any output.
Tested within Adafruit WipperSnapper PR:
adafruit/Adafruit_Wippersnapper_Arduino#488
https://github.com/adafruit/Adafruit_Wippersnapper_Arduino/pull/488
Example failure run, without --build_timeout for ESP32 builds (which take
a while to run and fail if they timeout):
https://github.com/adafruit/Adafruit_Wippersnapper_Arduino/actions/runs/6174986510/job/16760853258
Example successful run, with --build_timeout enabled for all builds:
https://github.com/adafruit/Adafruit_Wippersnapper_Arduino/actions/runs/6174857439
You can view, comment on, or merge this pull request online at:
https://github.com/adafruit/ci-arduino/pull/160
Commit Summary

160a7cc
https://github.com/adafruit/ci-arduino/pull/160/commits/160a7cc7b7da3bd9201ffe2d9304cdf8f5b35051
sync brentru timeout with --timeout arg
381d0a9
https://github.com/adafruit/ci-arduino/pull/160/commits/381d0a9e01d87da6c20ae90b759ef83db9b2d09d
fix indentation level
f28f8bc
https://github.com/adafruit/ci-arduino/pull/160/commits/f28f8bc2bf692c9fedaf3e2907b533bc34815ebe
Remove timeout value from sys.argv as well
18ce377
https://github.com/adafruit/ci-arduino/pull/160/commits/18ce377263a7d2e46266c703720cf50d146100f7
Merge pull request #2 from tyeth/tweak-timeout
011ae3e
https://github.com/adafruit/ci-arduino/pull/160/commits/011ae3e22662e23b35e8e3350203e124314ae051
cast
fe15057
https://github.com/adafruit/ci-arduino/pull/160/commits/fe15057af2de27fca63b526afc0ef5108a33a9c9
formatting

File Changes
(1 file https://github.com/adafruit/ci-arduino/pull/160/files)

M build_platform.py
https://github.com/adafruit/ci-arduino/pull/160/files#diff-397680a251a8d6fb57a3be45c67a6fa50d4704abc05b95b9b106cd9901f91124
(19)

Patch Links:

https://github.com/adafruit/ci-arduino/pull/160.patch
https://github.com/adafruit/ci-arduino/pull/160.diff

—
Reply to this email directly, view it on GitHub
https://github.com/adafruit/ci-arduino/pull/160, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJINUIAXXJX43NRJJ4HO73X2HLJBANCNFSM6AAAAAA4WVYHEA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Add css for prefers-color-scheme: dark

to switch to dark mode just change your browser's theme to dark

It needs some adjustments (the logo and the selected elements in the menu), but it's a good start.
Thanks!

It needs some adjustments (the logo and the selected elements in the menu), but it's a good start.
Thanks!


dependencies are not compatible

🐞 Bug report
Command (mark with an x)

[ ] new
[ ] build
[ ] serve
[ ] test
[ ] e2e
[ ] generate
[ ] add
[ ] update
[ ] lint
[ ] extract-i18n
[ ] run
[ ] config
[ ] help
[ ] version
[ ] doc

Is this a regression?
not sure
Description
this is not actually an error but it introduces an issue when installing angular
@angular-devkit/* and @angular/* packages have different versions of dependencies that have breaking changes between them.
for example @angular-devkit/core -><EMAIL_ADDRESS>and @angular/core -><EMAIL_ADDRESS>|| ^7.4.0
in my root package.json , I use<EMAIL_ADDRESS>as a dependency, which is compatible with Angular
but when I run npm install the version 6.6.7 is installed in my root node_module dir because it is required by @angular-devkit/core. (even if my package.json requires v^7.4.0)
this makes the following line typely wrong if used inside an angular project (probably you may need to use a function or extend a class that uses rxjs.Observable from v7.4.0, but your root node_modules has rxjs v6.6.7)
import { Observable} from 'rxjs';

Observable from both versions (i.e 6.6.7 and 7.4.0) are not compatible because there are many breaking changes between those versions, for example, _isScalar is removed from Observable at v7.0.0

the property _isScalar is missing in Observable

this specially happens in monorepos, where the angular project is a workspace inside that monorepo and there is another workspace that extends its functionality.
in this case, the package managers may choose to install<EMAIL_ADDRESS>in the root dir, and<EMAIL_ADDRESS>inside the package's node_module dir.
my suggestion is to update dependencies in @angular-devkit packages to remove that breaking changes between rxjs versions
🔬 Minimal Reproduction
🔥 Exception or Error
🌍 Your Environment

Anything else relevant?

The RxJs version of the application and devkit packages and other node packages have different contexts and are not guaranteed to be always in sync.
The devkit packages also get synced inside of Google which is still dependent on RxJs 6.  While we are looking in updating to RxJs 7, this is not a trivial change due to large set of breaking changes.

in this case, the package managers may choose to install<EMAIL_ADDRESS>in the root dir, and<EMAIL_ADDRESS>inside the package's node_module dir.

This is expected and correct behaviour from package manages. You may need to configure TypeScript path mappings to favour the resolution at package level.
Note: this isn’t a problem if you use the Angular CLI to manage your workspace.


proton client ignore ctrl+c on select from system.numbers

open proton-client
query without limit select number from system.numbers where number % 10 = 0
try pass ctrl+c

In very rare cases ctrl+c work and cause stop but usually client just ignore it.
I think that cause is that select from numbers without limit is very intensive for CPU and client ignore input stream.
So need make to react to ctrl+c as stop output action even on intensive queries

Thanks for report this.

hi @mosinnik thank you for taking the trial.
it can reproduce easily (with commit 0e183fb85b36b739a5c499ea426f94be657aea66)
for the potential race, it may be. but the client display shows only consumption (1.1 CPU).
I will take a look. thanks!

@mosinnik  apologies for the late reply. We have not focused too much on proton-client previously. I spent some time comparing current proton-client vulnerabilities with clickhouse-client, and have tried to address this issue.
However, the official release version will need to be published later. I can trigger a CI action to propose a docker image or my personal local build binary for you if that would help in the meantime?
Just wanted to do a final check - is the pic/video below accurate regarding this issue? Please let me know if any clarification or additional details would be helpful. I'm happy to assist further.

https://harvest-vegetarian-745.notion.site/ctrl-c-cannot-correct-cancel-query-49898eb3bb2c4543a1360574904d7020?pvs=4

@yokofly looks very nice, as for me now it's not an urgent problem so I'll wait release

@yokofly looks very nice, as for me now it's not an urgent problem so I'll wait release

Hi @mosinnik  thanks for the patience :)
wait, umm the above picture or video is my finding problem, is my understanding of the issue accurate?

thank you! I think this new release will solve it :)

Hi, @mosinnik !
we have published the 1.3.22 version, either through docker/ single binary /homebrew.
you can try it in your free time.
docker:
docker run -d --pull always --name proton ghcr.io/timeplus-io/proton:latest
docker exec -it proton proton-client -n

select number from system.numbers where number % 10 = 0

single binary:
curl -sSf https://raw.githubusercontent.com/timeplus-io/proton/develop/install.sh | sh

homebrew:
brew tap timeplus-io/timeplus
brew install proton