Flag original dates for observed days

When an observed day (i.e. holidays that are moved because they are on a weekend) is returned would it possible to flag what the original date was rather than relying on a string suffix?
Also, in some locations when a holiday falls on a weekend, it is moved to a weekday as an observed day. In some of these locations only the observed day is considered a holiday whereas in others both the observed and original days are considered holidays. This is an important distinction in some jurisdictions.
Is there any chance of supporting an additional flag against observed holidays that indicate the original day is or isn't a holiday?

Your suggestion would change a lot of things in the API of workalendar, if I understand correctly your point.
So far, the holidays() method in workalendar only returns a tuple, composed by a date and a string. I'm really attached to this elementary data structure, because there's no extra data surrounding it. You want a calendar, you have dates and a label. KISS principle.
If we had to return these calendars with an extra flag meaning that it's a "true" holiday or not, this would mean it'd break the API for every workalendar user. Because:

we can't define an extra property to a tuple. Or we would have to define a class that inherits from a tuple, and it's generally a bad idea to try to override a built-in type,
or we may define a core class, Holiday, with core properties such as the date & label, but with extra properties.

But doing either 1. or 2, would mean to change every single calendar and the way they build their holiday list. And also, every 3rd party application using workalendar would have to rewrite the way they're accessing the holidays... That'd represent a lot of workload, caused by a feature you might not be interested in.
Now onto your use-case:
I think that if this kind of distinction between a holiday and a non-holiday should be treated case by case in each calendar that makes this distinction.
If for example May 1st is a non-working day and it's not counted as a holiday if it falls on Sundays, then your ̀holidays() method shouldn't return it in the list. If both the regular day and the observed day are counted as holidays, then make it so by returning both of them in the holiday list. I think it's doable for the precise jurisdictions that demand it ; and I'd be glad to review and integrate these patches to improve the workalendar library accuracy in these calendars.
And if by some ways it appears that your special case can't be processed at the workalendar calendar class, then you'll have to externalise this by building a wrapper around this holiday list. Iterate over the days and remove those who should be removed, because you know how the days must be post-processed, the specifics of it.
What do you think of these alternatives?

@brunobord  Your suggestion of not returning May 1st if it isn't counted as a holiday (as it falls on a Sunday) doesn't address scenarios where users of Calendars need to see the original dates of a holiday once it has moved.
Take for example https://www.industrialrelations.nsw.gov.au/public-holidays/public-holidays-in-nsw/

2 The public holiday standard in the Public Holidays Act 2010 provides that when Australia Day (26 January) falls on a Saturday or Sunday, there will be no public holiday on that day and instead the following Monday is to be the public holiday.

In any case, I understand that given the current simple structure that is returned this would require significant changes that would break projects downstream.


NonText (listchars) invisible unless directly under cursor

When using a dark background, your modifications to peaksea "NonText" colors render them invisible unless they are directly under the cursor. This may or may not be related to issue #224 reported a few years ago.
Here's a direct comparison between your modified version (top) and the original value (bottom). Note how in the top session, only the tab character directly under the cursor is visible, but in the bottom session, all non-text is visible:

Was this change intentional?  I couldn't tell what the motivation was from the commit comment. I personally find it convenient to see all non-text all the time, so I had assumed this was a mistake.  But since that change was made over five years ago, it seems more likely that this is your personal preference (or it would have been changed by now).
If this is your desired behavior, I override it in my_configs.vim to add visibility to the non-text characters.
And in case it's some environmental thing, my setup is as follows:

Ubuntu 18.10
rxvt-unicode 9.22-4
neovim v0.3.1
tmux 2.7.1 with Oh My Tmux! config
set listchars=tab:>-,nbsp:_,eol:$,trail:!,extends:},precedes:{

It was so long time ago this is set, I have reverted this tho' as it does not make sense to do such an overwrite.


Update Set-PnPListItem.md

Fixed typo in Example 6 -UpdateType UpdateOverwriteVersion
In section Parameters;
Not sure if -UpdateOverwriteVersion should be noted as a parameter. It is not available as a parameter. I'd rather include this explanation with parameter UpdateType
Before creating a pull request, make sure that you have read the contribution file located at
https://github.com/pnp/powerShell/blob/dev/CONTRIBUTING.md
Type

[ ] Bug Fix
[ ] New Feature
[x] Sample

Related Issues?
Fixes #X, partially fixes #Y, mentioned in #Z, etc.
What is in this Pull Request ?
Please describe the changes in the PR.
Guidance

You can delete this section when you are submitting the pull request.*
Please update this PR information accordingly. We use this as part of our release notes in monthly communications.
Please target your PR to Dev branch. If you do not target the Dev branch we will not accept this PR.

Thank you!


wording

The C++ STL is one of the most complete and reusable algorithm libraries available.
This single header file brings 80% of that functionality to C99 in a non-intrusive way.
There are no new data structures. Just include the library and call functions on C arrays.

"algorithm libraries" is unspecified and https://en.cppreference.com/w/cpp/algorithm says "The algorithms library defines functions for a variety of purposes (e.g. searching, sorting, counting, manipulating) that operate on ranges of elements. Note that a range is defined as [first, last) where last refers to the element past the last element to inspect or modify."
You might want to clarify this.
"A range of elements" here does not specify if or if not the underlying bits are continuous memory, so you might want to clarify that as well.
If you really want to stick to C++ algorithms, you could mention https://en.cppreference.com/w/c/algorithm and/or constrained algorithms.
I'd rather explain what you code does.

"algorithm libraries" refers to a library of useful computer science algorithms, not any narrow C++ definition.
Ranges are tool to accomplish algorithm implementations, not the definition of what the library does. For example, swap does not operate on ranges.

you could mention https://en.cppreference.com/w/c/algorithm

I'll consider adding a link. It would be helpful as documentation gets filled in.


Support of next_hop_unchanged under address family

Enhancement summary
Looking at the current configuration data model, next_hop_unchanged is only available under router_bgp/peer_groups hierarchy while it is also possible to be configured under router_bgp/address_family_evpn for example (as configured for my customer's IP Fabric spines)
Which component of AVD is impacted
eos_cli_config_gen
Use case example

router bgp xxxx
...
   address-family evpn
      bgp next-hop-unchanged
      neighbor LEAVES-EVPN activate

Describe the solution you would like

Change schema files
Change router-bgp.j2 template

Describe alternatives you have considered
The alternative of changing the customer's configuration to apply the next_hop_unchanged at peer_group level is exposing a limitation which shouldn't be difficult to implement.
Additional context
No response
Contributing Guide

[X] I agree to follow this project's Code of Conduct

According to internal discussions:

The default behaviour for family EVPN is indeed "next-hop-unchanged"
But, looking at the default configuration (show run all), we have "no bgp next-hop-unchanged" (cosmetic bug #295339)
When we explicitly configure "next-hop-unchanged" under address-family evpn, it appears in the running-configuration

So, in IMHO, we need to give the user the ability to explicitly configure "next-hop-unchanged"


docs: fix link to VecDeque

The link to std::collections::VecDeque was invalid because std was not in scope, but VecDeque was in scope via alloc so it didn't need an explicit link. This made rustdoc fail with the following error:
error: unresolved link to `std::collections::VecDeque`
 --> src/with_alloc/vecdeque.rs:8:42
  |
8 | /// Wrapper of the built-in [`VecDeque`](std::collections::VecDeque) struct
  |                                          ^^^^^^^^^^^^^^^^^^^^^^^^^^ no item named `std` in scope
  |
note: the lint level is defined here
 --> src/lib.rs:3:9
  |
3 | #![deny(warnings)]
  |         ^^^^^^^^
  = note: `#[deny(rustdoc::broken_intra_doc_links)]` implied by `#[deny(warnings)]`

error: could not document `ringbuffer`

Thanks!


Seeking Help/Question: Best practice for fine-tuning LLM with Lora but with addtional parameters

Hi PEFT Library, thanks for the cool project!
So I recently run into a use case where I want to fine-tune a LLM with Lora, but also some additional parameters. I'm currently using the Huggingface eco system for training. I wonder what would be the best practice for saving the additional parameters I want to have?
Specifically, I'm training a model with a PLM component. Only the PLM component is lora-adapted, but there is some extra modules that are being fine-tuned along with the lora adapters. The desired behavior is I want to save all parts that require gradients, not just the lora adapter, when calling model.save().
Here is a simplified example of the model I'm working with:
from transformers import PreTrainedModel, LlamaModel
from transformers.modeling_utils import PretrainedConfig
from transformers.modeling_outputs import SequenceClassifierOutput
import torch

class MyModelConfig(PretrainedConfig)
       # .... some useful custom config settings...

class MyABitComplexExtraModel(torch.nn.Module):
       # a somewhat complex pytorch model that takes llama's output hidden state
       # and map it to label

class MyModel(PreTrainedModel):
        def __init__(config:MyModelConfig):
                 # ...... some intializations
                 self.llm = LlamaModel.from_pretrained(config.llama_path) # <-- the llm I want to do LoRa on
                 self.extra_module = MyABitComplexExtraModel()
        def forward(self, llamamodel_inputs, labels):
                 llama_output = self.llm(**llamamodel_inputs)
                 logits = self.extra_module(llama_output)
                 if labels:
                           loss = ....
                           return SequenceClassifierOutput(loss=loss, logits=logits)
                 return logits

# preparing model for training
model = MyModel(config)

from peft import get_peft_model
for param in model.llm.parameters():
     param.requires_grad = False
model.llm = get_peft_model(model.llm, peft_config) #<-- note that the model.extra_module is still being tuned here

Now, when I later train this model using a standard HuggingFace trainer, is there any simply way to make sure every time I call model.save(), not only the adapter but as well as the "extra_module" is saved?
I looked around and currently the only way seems to be somehow inherit the "BaseTunerLayer" in "MyABitComplexExtraModel"? However, this probably require making the whole "MyABitComplexExtraModel" PEFT-compatible, which probably requires implementing its own configs, etc...
I wonder if there'd be easier way to do this, given this is experimental code and will be heavily modified... Thanks!

Now, when I later train this model using a standard HuggingFace trainer, is there any simply way to make sure every time I call model.save(), not only the adapter but as well as the "extra_module" is saved?

I assume you mean save_pretrained? No, unfortunately, I don't think there is an easy way to do this.
You could write your own custom functions to save and load those extra parameters, but probably that's not quite enough for you because you want to use Trainer, which isn't aware of those functions (e.g. when creating checkpoints).
What you could try is to create very light weight wrapper, something along the lines of:
class MyCustomSaveWrapper(nn.Module):
    # wraps a peft model and overrides save_pretrained and from_pretrained,
    # otherwise, it is a pass-through
    def __init__(self, model):
        super().__init__()
        self.model = model

    def __getattr__(self, name):
        return getattr(self.model, name)

    def save_pretrained(self, *args, **kwargs):
        # ensure that extra params are saved
        # take care of removing the extra 'model.' prefix added by this wrapper
        ...

There may be some more steps involved than what I outlined here, but it could be worth a try.

Ok, thanks! Yeah, that do sounds like a great idea... (Though still hope there could be some sort of mixins we can easily inherit...)
Closing and thanks for the quick response!


File "vendor/notty/lwt/notty_lwt.ml", line 68, characters 25-64:
Error: This expression has type (unit -> unit) Lwt.t
       but an expression was expected of type unit Lwt.t
       Hint: Did you forget to provide `()' as argument?
Done: 216/230 (jobs: 1)^J

your code doesnt even compile lol

so many*
On Sun, Sep 27, 2020, 12:42 PM sh0tx420<EMAIL_ADDRESS>wrote:

File "vendor/notty/lwt/notty_lwt.ml", line 68, characters 25-64:
Error: This expression has type (unit -> unit) Lwt.t
but an expression was expected of type unit Lwt.t
Hint: Did you forget to provide `()' as argument?
Done: 216/230 (jobs: 1)^J
your code doesnt even compile lol
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/alexozer/flitter/issues/26#issuecomment-699658293,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAP2NUUIG2KCOS5XQAEAVWLSH5TQTANCNFSM4R3UXUZQ
.

Yeah I have the same problem.Does this have anything to do with "Hint: Did you forget to provide `()' as argument?"
[christophersmith@this-is-pc flitter]$ dune build
File "_none_", line 1:
Warning 58: no cmx file was found in path for module Toploop, and its interface was not compiled with -opaque
File "vendor/notty/lwt/notty_lwt.ml", line 68, characters 25-64:
Error: This expression has type (unit -> unit) Lwt.t
       but an expression was expected of type unit Lwt.t
       Hint: Did you forget to provide `()' as argument?
Done: 212/230 (jobs: 1)[christophersmith@this-is-pc flitter]$ dune build
File "vendor/notty/lwt/notty_lwt.ml", line 68, characters 25-64:
Error: This expression has type (unit -> unit) Lwt.t
       but an expression was expected of type unit Lwt.t
       Hint: Did you forget to provide `()' as argument?
[christophersmith@this-is-pc flitter]$

Fixed in 666c5483bc93efa6d01e0b7a927461269f8e14de. And it's not a bad idea at all to provide a binary release.


Feature: Display "Total Monthly Goals" in Budget Inspector

Explanation of Bugfix/Feature/Modification:
Shows "Total Monthly Goals" in the Budget Inspector.
"Total Monthly Goals" is here defined as the sum of all monthly + target budget by date goal amounts for the month; put another way, it is the number that shows for the "Underfunded" quick budget, if you have not budgeted money towards any goals in that month.
Background: I mostly made this for myself. I use goals heavily ... almost every category has a goal. I also have a spreadsheet where I keep track of the number of months I have covered in my budget. The "Total Monthly Goals" number is really helpful for populating that spreadsheet. It would be nice to see this merged so that I don't have to use a custom built version of the extension 😃

Screenshot of this feature in action:

@jonscheiding just pinging you so we can get the comments addressed and get this thing in!

Thanks @joshmadewell for the follow up and feedback! I should have a chance to get this stuff addressed next Tuesday or Wednesday. Glad you like the feature.

@joshmadewell Responded to / incorporated feedback ... thanks!

Looks good! Thanks!


[fix] Fix data of qk_matmul in prefill stage

In qk_matmul，q belongs to load_act, not load_kv_cache

You are right, I have merged this.


Project properties - last modified date is wrong

The last modified date for all of my projects always just shows the same as the created date.
This is a problem in the project property window & the project details section of the workspace.

Yes, There is even a push requiest to fix this I beleive.

I was just having a look at this but can't find the fix on the forums?
As far as I can tell we have some confusion about what fields should represent what and therefore what fields are set or updated during different actions. e.g.
files that insert or change date_modified:
import.php
gift_this_template.php
duplicate_template.php
new_template.php
files that change or insert date_accessed
edit.php
edithtml.php
gift_this_template.php
So I could be wrong but editing a project only changes date_accessed not date_modified. I'm not sure what date_accessed is meant to represent so if we changed the two edit files to change date_modified instead or as well as date_accessed the displayed date_modified would be correct?

I think this has been fixed https://github.com/thexerteproject/xerteonlinetoolkits/commit/3881c37174b69c553cba27c7badbf9b7863f101f


Update CONTRIBUTORS and README.md

Following https://github.com/milessabin/shapeless/pull/542

Thanks! :-)


lyxal: Watch innblockchain.com

lyxal requests the watch of the watch_keyword innblockchain\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

innblockchain\.com has been seen in 2 true positives, 0 false positives, and 0 NAAs.

Approved by Jeff Schaller in Charcoal HQ


Improve stateful PlayScreen

WHAT
Improve stateful version of PlayScreen.
WHY
In order to provide a version with default implementation of mediaDisplay and controlButtons slots.
HOW

Add default implementation of mediaDisplay slot with TextMediaDisplay;
Add default implementation of controlButtons slot with MediaControlButtons;
Add lifecycle-runtime-ktx dependency to media-ui module;
Add tests;
Add util function rememberStateWithLifecycle;
Add test double FakePlayerRepository - it should be moved in another iteration to a separate test fixtures module;

Checklist :clipboard:

[x] Add explicit visibility modifier and explicit return types for public declarations
[x] Run spotless check
[x] Run tests
[x] Update metalava's signature text files

cc: @dpreussler


Updating mesosphere-shared-reactjs v0.0.20

Only change in this version is this: https://github.com/dcos/mesosphere-shared-reactjs/pull/15, which makes sure that the hanging requests actually works.
This also fixes a problem, where the history server would always be set to offline in the beginning.
Before (with network delay):

Before (without network delay):

After:

mesosphere-ci Retest this please

Tested in different network conditions and it LGTM. I can't reproduce the original problem though.

If you open up ./node_modules/mesosphere-shared-reactjs/utils/RequestUtil.js and replace the createCallbackWrapper with this:
function createCallbackWrapper(callback, requestID) {
  return function () {
    setTimeout(() => {
      setRequestState(requestID, false);

      if (Util.isFunction(callback)) {
        callback.apply(null, arguments);
      }
    }, 2100);
  };
}

That will create a delay for some of the requests to trigger the hanging function for one and not for the other.

Assigning you @rcorral since you have already looked at the code from mesosphere-shared-reactjs.


Space between preview paragraphs and additional line-breaks in editor

Having an issue with react-quill where the editor shows no line-breaks between empty paragraphs while the decoded preview of the editor-text has additional line-breaks.
Here is the way lorem ipsum text appears in my editor

And here is the way that same text appears when I decode it

The text is being decoded with ReactHtmlParser(he.decode(body))
I would like the text to appear as it appears in the editor, or as closely to it as possible. At minimum I would like to get rid of the additional line-breaks.

Hey, I know this is an old thread, but just wanted to give some insight for other user.
This seems to be a CSS issue.
The editor does not appear to have margin for the paragraphs  created
On the decoded page there seems to be a margin that actually separates one  from another.
you can try something like this
.ql-editor p { margin-top: 10px; margin-bottom: 10px; }
and on your decoded body you can do something like this:
.decoded-content p { margin-top: 10px; margin-bottom: 10px; }
this will keep the margins the same and will look very similar.

Hey, I know this is an old thread, but just wanted to give some insight for other user.
This seems to be a CSS issue.
The editor does not appear to have margin for the paragraphs  created
On the decoded page there seems to be a margin that actually separates one  from another.
you can try something like this
.ql-editor p { margin-top: 10px; margin-bottom: 10px; }
and on your decoded body you can do something like this:
.decoded-content p { margin-top: 10px; margin-bottom: 10px; }
this will keep the margins the same and will look very similar.


BatchUploader: introduce limit of records

Originally on 2014-01-20
It would be nice if BatchUploader would have some configurable limits, similar to MultiEdit (see also #77). In this way, any consequent processing jobs (indexing, etc.) would not impact the normal flow of the system.
Example:

1 -> 500 records: add/update on the spot
500 -> above: schedule the job for the evening

Invenio 1/2 is in feature freeze so closing ticket.


Use https as submodule url of google/docsy.git

The https version can be used without registering an SSH key with GitHub.
I stumbled upon this while checking out the django-DefectDojo repository using ansible builtin git. It automatically tries to download the registered submodules. But because of the missing SSH key, the clone fails.
My reasoning for this proposed change:

docsy is a public repo, using https should just work
there's no real reason to use SSH, if you want to contribute to google/docsy.git you can still setup an environment with SSH
if you don't have an SSH key registered with GitHub or you are in an environment where you don't have the key, using the submodule will fail

Good find, thank you @MichaelGissingNC


Disable Rails/LinkToBlank, because it is outdated

This is only a vulnerability in over three years old browsers.
https://github.com/rubocop/rubocop-rails/blob/202d8c1bde34b8a6cf72e8378ef933442c2701a2/config/default.yml#L653
From https://mathiasbynens.github.io/rel-noopener/

2021 update: Browsers now implicitly set rel=noopener for any target=_blank link, following a spec change. If the demo on this page no longer seems scary, congratulations — you’re using a modern browser!

So the cop should be disabled.

I don't think we should be encouraging developers to litter their code with rel=noopener to make it safer for users running three years old browsers. Those users with so old browsers have a ton of remote code execution bugs in their browsers, so putting whether or not rel=noopener is there won't make a big difference to them.

There no doubt that users with old browser have a plethora of issues and really should update. For me there are still a good chunk that don't do that for one reason or another. I can't tell what the consequences would be for those if this is disabled. If it does anything at all I'd prefer to keep it.
That's just my personal opinion, I don't know when a good point would be to disable this.  Maybe now, maybe a year ago already, or just 5 years in the future. :shrug:


Add NN-logo to list of sponsors

NN-logo and link added to README. Thanks to Sigurd@bytesnbrains for coding and thanks to NN for paying.

I am sorry to ask, but what have you contributed to the project?

He created the
https://github.com/bytesandbrains/ProcessOptimizer
fork which has constrains, bokeh_plots and better plots. I'm currently backporting all to scikit-optimize (PRs : #838, #836 and #830).
I'm fine with the change.

I am sorry to ask, but what have you contributed to the project?

Sure, fair question. I mainly drank a lot of coffee and made my boss pay for skilled external coder's work on the bokeh_plot, the categorical plotting, constrained optimization and implementation of latin hyper cube screening.

Thanks, I was not aware of the fork, hence all @sigurdcarlsen contributions have gone unnoticed.
But shouldn't we instead credit Bytes and Brains? how is this company related to Novo Nordisk?
@sigurdcarlsen: what do you think?
(Sorry for my questions, but understand that from the outside, this looks very peculiar. None of the changes made by @sigurdcarlsen have made it into master, yet we should already credit Novo Nordisk, whose connection with @sigurdcarlsen are unknowns.)

The changes from @sigurdcarlsen cannot be easily merged into scikit-optimize. I tried but the repositories are to different and there are to many conflicts.
I decided to copy the important changes and to add them manually into the newest master.

Thanks, I was not aware of the fork, hence all @sigurdcarlsen contributions have gone unnoticed.
But shouldn't we instead credit Bytes and Brains? how is this company related to Novo Nordisk?
@sigurdcarlsen: what do you think?
(Sorry for my questions, but understand that from the outside, this looks very peculiar. None of the changes made by @sigurdcarlsen have made it into master, yet we should already credit Novo Nordisk, whose connection with @sigurdcarlsen are unknowns.)

Sure,
I perfectly well see your point.
My company (Novo Nordisk a/s) paid for the nice work performed by @sigurdcarlsen at his company BytesnBrains. We do not own the code - and it has remained open source at all points of the development. Hence, you are of course free to use the code found at BytesnBrains/ProcessOptimizer under the usual license.
By all means you can wait with the PR until the rest of the other changes have made it to master.
The changes by Sigurd have not gone unnoticed and they are actively being kindly implemented by @holgern atm.

The changes from @sigurdcarlsen cannot be easily merged into scikit-optimize. I tried but the repositories are to different and there are to many conflicts.
I decided to copy the important changes and to add them manually into the newest master.

OK,
Then I understand if our contributions do not qualify for a logo. You decide :-)
/S

As an aside to the discussion about including the logo or not:
Please remember that overly terse, sarcastic, snarky, patronising and so on comments are unproductive towards finding consensus on an issue or pull request. Especially when groups that don't know each other or have a shared history meet for the first time, give people the benefit of the doubt by adding more context and more explanations. Even if it is about things which are obvious to you.
Thanks!

It gets even more tricky when attribution and money are involved. For this it is very helpful for people to use a well established online identity that lets others see who pays you, what work you sponsor (or not), whether you are acting on behalf of someone else, and so on. Basically more transparency helps others understand the context of your work and hence how to evaluate it.


Publish rye to PyPi

I'd like to install rye from PyPi. It simplifies the CI and DevContainer code.

this would also solve #995 since it would allow users to install rye using pyprojectx

Given that uv exists on pypi it might make sense.  That said the desire of Rye is that it's the way you bootstrap Python which implies that pip does not exist.  I do however think that there is potentially a reason to assume that such a world will take longer to materialize ;)

I don't mind adding this, not too hard to do.

One blocker is that rye on pypi is something else entirely. Either the person owning that package is willing to rename or we gotta find some reasonable suffix/prefix.

The name reclamation process is very slow though it may be faster now that they've hired someone to help with support — I haven't heard back since February about my request and there are lots of requests from 2023 without responses.

Maybe we can use astral-rye while rye is transferred.

The process is here - I was lucky that the maintainer of the previous project responded very quickly to my email and was happy to transfer the name over. If that did not happen I would likely not have been able to get the name transferred in any reasonable timeframe, if at all.

I can send the owner an email and see where it goes.


Validate CMSSW/ScramArch against TagCollector

Fixes #4918
I think we were using the default ScramArch value for tests. If it breaks too many things, then we can just drop this PR and fix it in the future :)

All unit tests cleared. I've also created another issue to mock tagCollector data for jenkins.
@ticoann please review and merge


return value of JedisCluster.publish

Expected behavior
as we all known jedis publish return value of clients subscirbe on channel . but  in cluster mode , pub information broadcast by cluster protocol , we run sub/pub on anynode which cause incorrect return value of publish
Actual behavior
mostly return 0
Steps to reproduce:
Redis / Jedis Configuration
Jedis version:
All
Redis version:
3.0+
Java version:
All

@zts1993 thx for reporting. If you run this through the redis CLI in cluster mode, what do you get in return of the publish operation?


feat: do not require top-level schemas to be arrays

Users can now write collections that do not have a "type": "array" as the top-level generator - this will cause an individual object to be generated instead of an array of objects as it was previously. Specifying --size will just cause the single generated value to be overwritten n number of times.

LGTM!


APIConnectionError: Error communicating with OpenAI.

System Info
python 3.11
fastapi==0.95.1
langchain==0.0.180
pydantic==1.10.7
uvicorn==0.21.1
openai==0.27.4

Who can help?
@agola11
@hwchase17
Information

[ ] The official example notebooks/scripts
[X] My own modified scripts

Related Components

[X] LLMs/Chat Models
[ ] Embedding Models
[ ] Prompts / Prompt Templates / Prompt Selectors
[ ] Output Parsers
[ ] Document Loaders
[ ] Vector Stores / Retrievers
[ ] Memory
[ ] Agents / Agent Executors
[ ] Tools / Toolkits
[ ] Chains
[X] Callbacks/Tracing
[X] Async

Reproduction
I am trying to create a streaming endpoint in Fast API, below are the files
main.py
from fastapi import FastAPI
from src.chat_stream import ChatOpenAIStreamingResponse, send_message, StreamRequest

app = FastAPI()

@app.post("/chat_streaming", response_class=StreamingResponse)
async def chat(body: StreamRequest ):
    return ChatOpenAIStreamingResponse(send_message(body.message), media_type="text/event-stream")

src/chat_stream.py
from typing import Awaitable, Callable, Union

Sender = Callable[[Union[str, bytes]], Awaitable[None]]

from starlette.types import Send
from typing import Any, Optional, Awaitable, Callable, Iterator, Union
from langchain.schema import HumanMessage
from pydantic import BaseModel
from fastapi.responses import StreamingResponse
from langchain.chat_models import ChatOpenAI
from langchain.callbacks.base import AsyncCallbackHandler
from langchain.callbacks.manager import AsyncCallbackManager

class EmptyIterator(Iterator[Union[str, bytes]]):
    def __iter__(self):
        return self

    def __next__(self):
        raise StopIteration

class AsyncStreamCallbackHandler(AsyncCallbackHandler):
    """Callback handler for streaming, inheritance from AsyncCallbackHandler."""

    def __init__(self, send: Sender):
        super().__init__()
        self.send = send

    async def on_llm_new_token(self, token: str, **kwargs: Any) -> None:
        """Rewrite on_llm_new_token to send token to client."""
        await self.send(f"data: {token}\n\n")

class ChatOpenAIStreamingResponse(StreamingResponse):
    """Streaming response for openai chat model, inheritance from StreamingResponse."""

    def __init__(
        self,
        generate: Callable[[Sender], Awaitable[None]],
        status_code: int = 200,
        media_type: Optional[str] = None,
    ) -> None:
        super().__init__(
            content=EmptyIterator(), status_code=status_code, media_type=media_type
        )
        self.generate = generate

    async def stream_response(self, send: Send) -> None:
        """Rewrite stream_response to send response to client."""
        await send(
            {
                "type": "http.response.start",
                "status": self.status_code,
                "headers": self.raw_headers,
            }
        )

        async def send_chunk(chunk: Union[str, bytes]):
            if not isinstance(chunk, bytes):
                chunk = chunk.encode(self.charset)
            await send({"type": "http.response.body", "body": chunk, "more_body": True})

        # send body to client
        await self.generate(send_chunk)

        # send empty body to client to close connection
        await send({"type": "http.response.body", "body": b"", "more_body": False})

def send_message(message: str) -> Callable[[Sender], Awaitable[None]]:
    async def generate(send: Sender):
        model = ChatOpenAI(
            streaming=True,
            verbose=True,
            callback_manager=AsyncCallbackManager([AsyncStreamCallbackHandler(send)]),
        )
        await model.agenerate(messages=[[HumanMessage(content=message)]])

    return generate

class StreamRequest(BaseModel):
    """Request body for streaming."""

    message: str

Expected behavior
The Endpoint should stream the response from LLM Chain, instead I am getting this error
Retrying langchain.chat_models.openai.acompletion_with_retry.<locals>._completion_with_retry in 1.0 seconds as it raised APIConnectionError: Error communicating with OpenAI.
Retrying langchain.chat_models.openai.acompletion_with_retry.<locals>._completion_with_retry in 2.0 seconds as it raised APIConnectionError: Error communicating with OpenAI.
Retrying langchain.chat_models.openai.acompletion_with_retry.<locals>._completion_with_retry in 4.0 seconds as it raised APIConnectionError: Error communicating with OpenAI.
Retrying langchain.chat_models.openai.acompletion_with_retry.<locals>._completion_with_retry in 8.0 seconds as it raised APIConnectionError: Error communicating with OpenAI.
Retrying langchain.chat_models.openai.acompletion_with_retry.<locals>._completion_with_retry in 16.0 seconds as it raised APIConnectionError: Error communicating with OpenAI.

ERROR:    Exception in ASGI application
Traceback (most recent call last):
  File "/Project/venv/lib/python3.11/site-packages/aiohttp/connector.py", line 980, in _wrap_create_connection
    return await self._loop.create_connection(*args, **kwargs)  # type: ignore[return-value]  # noqa
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/asyncio/base_events.py", line 1098, in create_connection
    transport, protocol = await self._create_connection_transport(
                          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/asyncio/base_events.py", line 1131, in _create_connection_transport
    await waiter
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/asyncio/sslproto.py", line 577, in _on_handshake_complete
    raise handshake_exc
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/asyncio/sslproto.py", line 559, in _do_handshake
    self._sslobj.do_handshake()
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/ssl.py", line 979, in do_handshake
    self._sslobj.do_handshake()
ssl.SSLCertVerificationError: [SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:992)

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/Project/venv/lib/python3.11/site-packages/openai/api_requestor.py", line 588, in arequest_raw
    result = await session.request(**request_kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Project/venv/lib/python3.11/site-packages/aiohttp/client.py", line 536, in _request
    conn = await self._connector.connect(
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Project/venv/lib/python3.11/site-packages/aiohttp/connector.py", line 540, in connect
    proto = await self._create_connection(req, traces, timeout)
            ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Project/venv/lib/python3.11/site-packages/aiohttp/connector.py", line 901, in _create_connection
    _, proto = await self._create_direct_connection(req, traces, timeout)
               ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Project/venv/lib/python3.11/site-packages/aiohttp/connector.py", line 1206, in _create_direct_connection
    raise last_exc
  File "/Project/venv/lib/python3.11/site-packages/aiohttp/connector.py", line 1175, in _create_direct_connection
    transp, proto = await self._wrap_create_connection(
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Project/venv/lib/python3.11/site-packages/aiohttp/connector.py", line 982, in _wrap_create_connection
    raise ClientConnectorCertificateError(req.connection_key, exc) from exc
aiohttp.client_exceptions.ClientConnectorCertificateError: Cannot connect to host api.openai.com:443 ssl:True [SSLCertVerificationError: (1, '[SSL: CERTIFICATE_VERIFY_FAILED] certificate verify failed: unable to get local issuer certificate (_ssl.c:992)')]

The above exception was the direct cause of the following exception:

Traceback (most recent call last):
  File "/Project/venv/lib/python3.11/site-packages/uvicorn/protocols/http/h11_impl.py", line 429, in run_asgi
    result = await app(  # type: ignore[func-returns-value]
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Project/venv/lib/python3.11/site-packages/uvicorn/middleware/proxy_headers.py", line 78, in __call__
    return await self.app(scope, receive, send)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Project/venv/lib/python3.11/site-packages/fastapi/applications.py", line 276, in __call__
    await super().__call__(scope, receive, send)
  File "/Project/venv/lib/python3.11/site-packages/starlette/applications.py", line 122, in __call__
    await self.middleware_stack(scope, receive, send)
  File "/Project/venv/lib/python3.11/site-packages/starlette/middleware/errors.py", line 184, in __call__
    raise exc
  File "/Project/venv/lib/python3.11/site-packages/starlette/middleware/errors.py", line 162, in __call__
    await self.app(scope, receive, _send)
  File "/Project/venv/lib/python3.11/site-packages/starlette/middleware/exceptions.py", line 79, in __call__
    raise exc
  File "/Project/venv/lib/python3.11/site-packages/starlette/middleware/exceptions.py", line 68, in __call__
    await self.app(scope, receive, sender)
  File "/Project/venv/lib/python3.11/site-packages/fastapi/middleware/asyncexitstack.py", line 21, in __call__
    raise e
  File "/Project/venv/lib/python3.11/site-packages/fastapi/middleware/asyncexitstack.py", line 18, in __call__
    await self.app(scope, receive, send)
  File "/Project/venv/lib/python3.11/site-packages/starlette/routing.py", line 718, in __call__
    await route.handle(scope, receive, send)
  File "/Project/venv/lib/python3.11/site-packages/starlette/routing.py", line 276, in handle
    await self.app(scope, receive, send)
  File "/Project/venv/lib/python3.11/site-packages/starlette/routing.py", line 69, in app
    await response(scope, receive, send)
  File "/Project/venv/lib/python3.11/site-packages/starlette/responses.py", line 270, in __call__
    async with anyio.create_task_group() as task_group:
  File "/Project/venv/lib/python3.11/site-packages/anyio/_backends/_asyncio.py", line 662, in __aexit__
    raise exceptions[0]
  File "/Project/venv/lib/python3.11/site-packages/starlette/responses.py", line 273, in wrap
    await func()
  File "/Project/src/app.py", line 67, in stream_response
    await self.generate(send_chunk)
  File "/Project/src/app.py", line 80, in generate
    await model.agenerate(messages=[[HumanMessage(content=message)]])
  File "/Project/venv/lib/python3.11/site-packages/langchain/chat_models/base.py", line 63, in agenerate
    results = await asyncio.gather(
              ^^^^^^^^^^^^^^^^^^^^^
  File "/Project/venv/lib/python3.11/site-packages/langchain/chat_models/openai.py", line 297, in _agenerate
    async for stream_resp in await acompletion_with_retry(
                             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Project/venv/lib/python3.11/site-packages/langchain/chat_models/openai.py", line 63, in acompletion_with_retry
    return await _completion_with_retry(**kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Project/venv/lib/python3.11/site-packages/tenacity/_asyncio.py", line 88, in async_wrapped
    return await fn(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Project/venv/lib/python3.11/site-packages/tenacity/_asyncio.py", line 47, in __call__
    do = self.iter(retry_state=retry_state)
         ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Project/venv/lib/python3.11/site-packages/tenacity/__init__.py", line 325, in iter
    raise retry_exc.reraise()
          ^^^^^^^^^^^^^^^^^^^
  File "/Project/venv/lib/python3.11/site-packages/tenacity/__init__.py", line 158, in reraise
    raise self.last_attempt.result()
          ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/concurrent/futures/_base.py", line 449, in result
    return self.__get_result()
           ^^^^^^^^^^^^^^^^^^^
  File "/Library/Frameworks/Python.framework/Versions/3.11/lib/python3.11/concurrent/futures/_base.py", line 401, in __get_result
    raise self._exception
  File "/Project/venv/lib/python3.11/site-packages/tenacity/_asyncio.py", line 50, in __call__
    result = await fn(*args, **kwargs)
             ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Project/venv/lib/python3.11/site-packages/langchain/chat_models/openai.py", line 61, in _completion_with_retry
    return await llm.client.acreate(**kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Project/venv/lib/python3.11/site-packages/openai/api_resources/chat_completion.py", line 45, in acreate
    return await super().acreate(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Project/venv/lib/python3.11/site-packages/openai/api_resources/abstract/engine_api_resource.py", line 217, in acreate
    response, _, api_key = await requestor.arequest(
                           ^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Project/venv/lib/python3.11/site-packages/openai/api_requestor.py", line 300, in arequest
    result = await self.arequest_raw(
             ^^^^^^^^^^^^^^^^^^^^^^^^
  File "/Project/venv/lib/python3.11/site-packages/openai/api_requestor.py", line 605, in arequest_raw
    raise error.APIConnectionError("Error communicating with OpenAI") from e
openai.error.APIConnectionError: Error communicating with OpenAI

I got this bug, too. Could you tell me how to fix this bug.

I've got this error only with async functions of langchain, but the error does not say much (on train, I though my connection is just terrible). Later I've set max_retries to lower number:chat = ChatOpenAI(model_name="gpt-3.5-turbo", temperature=0.9, request_timeout=600, max_retries=3) and got errors printed out about wrong SSL certificates. Even minimal request via aiohttp was not successfull.
This helped: https://stackoverflow.com/questions/69605350/aiohttp-raises-an-certificate-error-with-some-sites-that-browser-opens-normally

I get the same problem and find the wrong is from VPN. Sometimes the VPN rules will only support part of OPENAI.
If you can't get an available VPN, try Web IDE like Colab.

This happens mostly on apple silicon machines. Fixed it by,
bash /Applications/Python*/Install\ Certificates.command

https://stackoverflow.com/a/76519628/3297613

thanks, it works!

This happens mostly on apple silicon machines. Fixed it by,
bash /Applications/Python*/Install\ Certificates.command
https://stackoverflow.com/a/76519628/3297613

i got the same question when using agent.arun(), here is my demo
import asyncio
from typing import Any, Dict, List

from langchain.chat_models import ChatOpenAI
from langchain.schema import LLMResult, HumanMessage
from langchain.callbacks.base import AsyncCallbackHandler, BaseCallbackHandler
from langchain.agents import AgentType, initialize_agent, load_tools
from langchain.chains.conversation.memory import ConversationBufferMemory, ConversationStringBufferMemory
from langchain.llms import OpenAI
import os

os.environ["OPENAI_API_KEY"] = 'xxx'

class MyCustomSyncHandler(BaseCallbackHandler):
    def on_llm_new_token(self, token: str, **kwargs) -> None:
        print(f"Sync handler being called in a `thread_pool_executor`: token: {token}")

class MyCustomAsyncHandler(AsyncCallbackHandler):
    """Async callback handler that can be used to handle callbacks from langchain."""

    async def on_llm_start(
        self, serialized: Dict[str, Any], prompts: List[str], **kwargs: Any
    ) -> None:
        """Run when chain starts running."""
        print("zzzz....")
        await asyncio.sleep(0.3)
        class_name = serialized["name"]
        print("Hi! I just woke up. Your llm is starting")

    async def on_llm_end(self, response: LLMResult, **kwargs: Any) -> None:
        """Run when chain ends running."""
        print("zzzz....")
        print("Hi! I just woke up. Your llm is ending")

# llm = OpenAI(temperature=0)
handler1 = MyCustomAsyncHandler()
llm = ChatOpenAI(
        temperature=0,
        verbose=True,
        # callbacks=[handler1],
    )
memory = ConversationStringBufferMemory(memory_key="chat_history", output_key='output')
tools = load_tools(["llm-math"], llm=llm)
agent = initialize_agent(tools, llm, verbose=True, memory=memory, agent=AgentType.CONVERSATIONAL_REACT_DESCRIPTION)

async def run():
    arun = await agent.arun("what is 1+1")
    print(arun)

asyncio.run(run())

same error
Retrying langchain.llms.openai.acompletion_with_retry.<locals>._completion_with_retry in 4.0 seconds as it raised APIConnectionError: Error communicating with OpenAI.

This happens mostly on apple silicon machines. Fixed it by,
bash /Applications/Python*/Install\ Certificates.command

https://stackoverflow.com/a/76519628/3297613

bash /Applications/Python*/Install\ Certificates.command
-- pip install --upgrade certifi
/Library/Frameworks/Python.framework/Versions/3.8/bin/python3.8: No module named pip.main; 'pip' is a package and cannot be directly executed
Traceback (most recent call last):
File "", line 44, in
File "", line 24, in main
File "/Library/Frameworks/Python.framework/Versions/3.8/lib/python3.8/subprocess.py", line 364, in check_call
raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['/Library/Frameworks/Python.framework/Versions/3.8/bin/python3.8', '-E', '-s', '-m', 'pip', 'install', '--upgrade', 'certifi']' returned non-zero exit status 1.

Is this ok?

Was this issue resolved? I'm also using fastapi and I face this issue after sometime. Initially I receive the responses and after some idle time I receive the same error

Was this issue resolved? I'm also using fastapi and I face this issue after sometime. Initially I receive the responses and after some idle time I receive the same error

Hi, did you fix it? I am also encounter the same error on my linux machine.

I am having this issue with Langchain, FastAPI and StreamingResponse in Docker. I am using LCEL, including standard Runnables and custom Runnables.
The issue occurs both when generating via stream() and astream()
The issue does not occur when calling the LCEL chain directly - even on the same machine.
Any insights would be useful.

I am having this issue with Langchain, FastAPI and StreamingResponse in Docker. I am using LCEL, including standard Runnables and custom Runnables. The issue occurs both when generating via stream() and astream()
The issue does not occur when calling the LCEL chain directly - even on the same machine.
Any insights would be useful.

@imarquart In my case this was caused by SSL issue, I’m behind my company’s firewall. Maybe you can have a look.

How to execute
bash /Applications/Python*/Install\ Certificates.command
if I only use python from (Xcode)
/usr/bin/python3
on my Mac?


feat: Add position_intent into order model

Closes https://github.com/alpacahq/alpaca-py/issues/539
Context:

position_intent is missing in order object

Changes:

add position_intent into order object

Thanks for the quick fix


Update link in about section

This link still points to old leetcode extension

taking on the issue

Visit https://github.com/arunbhardwaj/LeetHub-2.0.
See the right end of the repo

Thank you, it's been updated.


Is it possible to list plugins to stdout in plain text?

I'm trying to get a list of plugins from my ansible playbooks.
nvim +PluginList +qall gets a lot of control chars there (as it's probably expected)
Is it possible to get a list of plugins in some machine-readable format?

Is the reason you asked this on the vundle repo? If you use vundle, then you can get a list of installed plugins through :PluginList.

Well, that's gives a lot of unparsable garbage. I want something which can be cleanly processed.

I'm not sure what you mean. It simply opens up a buffer with the list of plugins. Unless you find the prefixes with "Plugin" to be unparsable garbage?
Are you asking how to direct the output of a buffer to a file for postprocessing? You could do something like
vim -c "PluginList" -c "w pluginlist.txt" -c "qa"

if desired. If you are getting different behaviour, you might want to provide an example.

That's better, thanks!
Any way to output that to stdout?

I've seen some people install some other tools for this, but vim isn't particularly good at throwing non-formatted things to stdout. I did a bit of googling to see if this was easy, and if it is then I missed it --- I'm not sure of an easy to way throw this to stdout. If you find something that does this for you, let me know.
Or rather, I'm not sure of an easy way to print a vim buffer to stdout. I think the easiest way for most people to print their plugins would be something like
cat ~/.vimrc | grep "^Plugin"

(adjusted for location and name of one's actual vim config file).


Image caption

Hi,
Is it possible to add image caption? couldn't see any information about it in docs. If not could you point me where +/- it should be added I can try to add pull request.

Hi!
No its not possible at this time i'm afraid. I don't know the docx spec well enough to judge whether this would be difficult to add...
This is the function that is concerned with injecting image-related nodes into the document XML.
https://github.com/guigrpa/docx-templates/blob/f3e33ffdeb657e31f79b9e70f926be53ae59fb88/src/processTemplate.ts#L759-L854
Happy to take a look at a PR.

Should be possible to add this feature in a way similar to how image rotation was added in #221 . See http://officeopenxml.com/drwPic-nvPicPr.php

Hi @jjhbw, have we had this feature in the latest version of docx-template?

@PhuocHoangMinhNguyen i'm afraid not. I didn't get around to it yet. Rotation can be handled from JS though, outside of docx-templates.

@jjhbw It's okay man. Just wondering what do you mean by "didn't get around to it yet"? Is it hard or you haven't had time to look into it? If it is the latter, I will try to look in the code then, we are in dire need of this feature.

Would be great if you could take a look. I'm swamped with other work at the moment, but happy to look at your PR.

I've created a PR for this: #286

Thanks @jwasnoggin ! Looks like a sensible solution to me.

Hi @jjhbw
When are you planing to release this feature ?

The release is lined up, see https://github.com/guigrpa/docx-templates/releases/tag/v4.10.0
The GH Action that we use to publish on NPM broke due to a new 2FA requirement. I'm working with @guigrpa to get that resolved. If you'd rather not wait you can install the commit directly from github in the meantime, see https://stackoverflow.com/a/18049685


Updated validations.py  python script

Updated validations.py python script.
Fixed the behavior of validate_user function in validations.py.
Commented the isnumeric() statements and replaced it with isaplha() instead.
Fixes #<issue_number_goes_here>

It's a good idea to open an issue first for discussion.

[ ] Tests pass
[ ] Appropriate changes to README are included in PR

@googlebot I signed it!


planning & implementing scene loading and data sharing

[x] loading and unloading regions
[x] global state
[x] persist state and scene information over code reload

https://docs.unity3d.com/Manual/PlatformDependentCompilation.html We could automatically remove the dictionary wrapper in the build


Return incorrect information

I get the price of a token via sdk via the example code
For example the price of goETH
But there is a problem, the price received via sdk is very different from the price inside the site (I tested it again today, the price difference was about $ 40!)

I tested it now
The information is as follows!
in site : 1 goETH = 3226 ALGO
with sdk :  1 goETH = 3123 ALGO
What is the problem?
from tinyman.v1.client import TinymanMainnetClient
account = {
    'address': 'ALGORAND_ADDRESS_HERE',
    'private_key': 'base64_private_key_here', # Use algosdk.mnemonic.to_private_key(mnemonic) if necessary
}

client = TinymanMainnetClient(user_address=account['address'])

goETH = client.fetch_asset(386195940)

ALGO = client.fetch_asset(0)

pool = client.fetch_pool(goETH, ALGO)

quote1 = pool.fetch_fixed_input_swap_quote(goETH(1 * 10 ** goETH.decimals), slippage=0.01)

print(quote1)

and returned :
SwapQuote(swap_type='fixed-input', amount_in=goETH('1'), amount_out=ALGO('3123.557726'), swap_fees=goETH('0.003'), slippage=0.01)

Do you still see this issue? Could you post a screenshot if you do? When I check right now I see the exact same values.

Thank you for your contribution.
There is one more similar issue, I am closing this one since it is not active.
Please check https://github.com/tinymanorg/tinyman-py-sdk/issues/36. Feel free to reopen it.


RichTextLabel wrapping is not working if set the Visible Characters property

Godot version:
d5eba83fd57005c110ef594f90c84016b758a0d5
OS/device including version:
macOS 10.13.3
Issue description:
RichTextLabel wrapping is not working if set the Visible Characters property
Steps to reproduce:

Create a RichTextLabel
Set RichTextLabel's Text to "Test, Test, Test, Test, Test, Test, Test, Test, Test"
Wrapping is working now.
Set RichTextLabel's Visible Characters to 40
Wrapping is not working now.

Minimal reproduction project:

Godot version:
612ab4b
OS/device including version:
win 10
I couldn't reproduce it. Everything works as it should. Please check again.

Godot version:
24cd4fe (2 commits after 612ab4b)
OS/device including version:
macOS 10.13.3
Issue description:
stilling not working

Minimal reproduction project:
TestRichText.zip

Can confirm it. Problem appeared exactly in 3.0.1.

This was fixed in 0e1e95c41f02acd5ed5c347ff63d0e38185525ad (will be in 3.0.2)


How to implement global style overriding or globally apply custom styles

What package has an issue
@svelteuidev/core
A clear and concise description of what the bug is
I hope to modify the primary in color through certain configurations, so that all places based on primary can automatically become the color I set. For example, after I change primary to red, buttons everywhere change to red.
I hope to make global modifications, rather than changing in every place where it is used.
I have tried Custom theme but it didn't work.
In which browser(s) did the problem occur?
Chrome, Other - list in description
Steps To Reproduce
+layout.svelte
const newTheme = createTheme("new-theme", {
	colors: {
		primary: "red",
	},
});

<SvelteUIProvider class={newTheme}>
	<NavBar {data} />
	<slot />
	<footer class="bg-white"></footer>
</SvelteUIProvider>

Do you know how to fix the issue
Yes
Are you willing to participate in fixing this issue and create a pull request with the fix
Yes
Relevant Assets
No response

This would be indeed very helpful to be able to override the primary color once and have an easy way to customize the default color palette. But it looks like the blue is hardcoded in a few places (for the buttons at least)

Hey, right now we don't have a workaround besides setting the color prop in each component. We are moving away from our current styling solution, so this will definitely be changed in the future.


ENH: match np.result_type behaviour in some scipy.signal functions

Changes the following to have return type np.float32 when all
arguments have type np.float32, and to have return type np.complex64
when all arguments are have type float32 and complex64.

decimate
lfilter_zi
lfiltic
sos2tf
sosfilt_zi

decimate was also changed to return np.float64 when all arguments are
neither float32, nor float64.  sos2tf and lfiltic already had this
behaviour, which has been maintained.
Adds tests for this behaviour for the above functions and also:

filtfilt (no complex type support with method='gust')
lfilter
savgol_filter (no complex type support)
sosfilt
sosfiltfilt

Fixes #3158.

Reference issue

What does this implement/fix?

Additional information

Remarks:

the test implementation is perhaps over-complicated
since this adds tests for all combinations of (f4, f8, c8, c16) for functions of up to 4 arguments, it adds quite a lot of test cases; on my (admittedly humble) machine the new tests take about 5s, vs. 1min30s for the whole scipy.signal test suite.
I'm not sure how generic the signal functions are supposed to be; I've used

    if result_type.kind in 'bui':
        result_type = np.float64

instead of
    if result_type.kind not in 'cf':
        result_type = np.float64

this is arguably backwardly incompatible, since code relying on the current f4 -> f8 conversion will break
I've tried to maintain existing  (integer-type) -> f8 conversions (sos2tf, lfiltic)

I started a 1.6.0 release notes page on the wiki and added an entry for this, @roryyorke feel free to double-check it looks okay to you:
https://github.com/scipy/scipy/wiki/Release-note-entries-for-SciPy-1.6.0
Thanks for the fix!


Revert [SPARK-19355][SPARK-25352]

What changes were proposed in this pull request?
This goes to revert SPARK-19355 and SPARK-25352.
How was this patch tested?
Existing tests.

cc @cloud-fan @rxin @hvanhovell

Test build #96240 has started for PR 22464 at commit 8ee721c.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/testing-k8s-prb-make-spark-distribution-unified/3232/
Test PASSed.

Merged build finished. Test PASSed.

@viirya, not a big deal but mind leaving few explicit discussion links (in the PRs you pointed out) to the current PR description?

I just put the link here: https://github.com/apache/spark/pull/16677#issuecomment-422650759

Thanks @HyukjinKwon @maropu. I've put it in the PR description.

Test build #96240 has finished for PR 22464 at commit 8ee721c.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
trait BaseLimitExec extends UnaryExecNode with CodegenSupport
case class LocalLimitExec(limit: Int, child: SparkPlan) extends BaseLimitExec
case class GlobalLimitExec(limit: Int, child: SparkPlan) extends BaseLimitExec

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/96240/
Test PASSed.

@viirya Thanks for doing it! To ease the review, can you revert these 4 commits sequentially with git revert commit-hash? Thanks!
https://github.com/apache/spark/pull/22344
https://github.com/apache/spark/pull/22330
https://github.com/apache/spark/pull/22239
https://github.com/apache/spark/pull/16677

@cloud-fan Ok. Let me try it.

@cloud-fan Shall we have 4 PRs to revert these PRs individually?

just one PR including 4 commits


[SPARK-42731][CONNECT][DOCS] Document Spark Connect configurations

What changes were proposed in this pull request?
This PR proposes to document the configuration of Spark Connect defined in https://github.com/apache/spark/blob/master/connector/connect/server/src/main/scala/org/apache/spark/sql/connect/config/Connect.scala
Why are the changes needed?
To let users know which configuration are supported for Spark Connect.
Does this PR introduce any user-facing change?
Yes, it documents the configurations for Spark Connect.
How was this patch tested?
Linters in CI should verify this change.
Also manually built the docs as below:

Merged to master and branch-3.4.


allow overriding of DWN hosts

adds this enhancement: https://github.com/TBD54566975/web5-js/issues/86

Would rather call the property dwnEndpoints, as host makes it seem like an external entity is always involved, when it could simply be a static IP to a box in your own home.

@csuwildcat yeah I agree - was struggling for a name. endpoints is also what DID spec calls them, will change.

@csuwildcat how does this look?

Lgtm

@mistermoe were you going to push up some additions to the PR we discussed yesterday to make overriding other properties possible?  Or should we start with dwnEndpoints and open a separate PR?

i didn't get a chance to add those additional overrides we discussed yesterday. let's start with dwnEndpoints. i'll open a separate PR that introduce the other options

the only thing that gives me pause at the moment is that this dwnEndpoints override option only makes sense in a scenario where an embedded user agent is being used. That said, the only agent that exists today is an embedded agent haha! the long term goal is for desktop & mobile agents to exist and it's in those external agents where you'd be providing your custom dwn endpoints or any other custom did configurations. that wouldn't happen via the web5 api.
Given that we haven't yet finished the first external agent, i think there's value in allowing people to provide custom dwn endpoints today. I think we could do it like so:
const { web5, did } = await Web5.connect({
  techPreview: {
    dwnEndpoints: []
 }
});

namespacing the dwnEndpoints option within a techPreview object hopefully makes it more clear that the option is temporary

Spotted a couple issues with the existing PR:

only 1 of the 2 README files was updated
README example was const { web5, did: myDid } = await connect(... instead of const { web5, did: myDid } = await Web5.connect(....

both resolved with Commit 05e4e34
Still one remaining issue which is if dwnEndpoints is an empty array, the messageAuthorizationKeys and recordEncryptionKeys values aren't populated in the DID document:
with { techPreview: { dwnEndpoints: [] } }:

with { techPreview: { dwnEndpoints: ['https://dwn.your-domain.org'] } }:

Discussing with @mistermoe and @csuwildcat as to how we should handle.

yeah calling connect with dwnEndpoints specified is in some ways assuming that is the special case that a did will be created for you, but wasn't sure where else it belongs in the short term.


Unable to get Pygments to work

On Ubuntu and I've done
sudo apt-get install python-pygments

Here is my bundle
Using posix-spawn (0.3.6)
Using albino (1.3.3)
Using blankslate (<IP_ADDRESS>)
Using diff-lcs (1.1.3)
Using ffi (1.0.11)
Using github-markup (0.7.0)
Using mime-types (1.17.2)
Using grit (2.4.1)
Using mustache (0.99.4)
Using nokogiri (1.5.0)
Using redcarpet (1.17.2)
Using sanitize (2.0.3)
Using rack (1.4.1)
Using rack-protection (1.2.0)
Using tilt (1.3.3)
Using sinatra (1.3.2)
Using gollum (1.3.1)
Using rubypython (0.5.3)
Using pygments.rb (0.2.4)
Using rdiscount (1.6.8)
Using bundler (1.0.21)
Your bundle is complete! Use `bundle show [gemname]` to see where a bundled gem is installed.

But I'm unable to get syntax highlighting to work for any of the lexers.

I didn't have the latest version of this gem (which I got from RubyGems) so it didn't have Pygments support.


Gobrake v5 is not respecting the HTTPClient

This line is not using the HTTPClient provided in the Notifier Options: https://github.com/airbrake/gobrake/blob/08fd4aef072cb2b7f7f1e29dc29be355de64022b/remote_config.go#L238
Which is causing gobrake to be upset in my application.

TLDR: As a result, proxy settings are not honored.

I'm also not sure under what conditions gobrake is attempting to write the config.json to disk, but the filesystem is read-only, so that just causes an extra layer of unhappiness

Thanks!


MultipleObjectsReturned: get() returned more than one Embed

We are getting this quite a lot.
We have  a Sentry report over here: https://beta.getsentry.com/share/group/34313638322e3833333433303131/
MultipleObjectsReturned: get() returned more than one Embed -- it returned 2!
  File "django/core/handlers/base.py", line 137, in get_response
    response = response.render()
  File "django/template/response.py", line 103, in render
    self.content = self.rendered_content
  File "django/template/response.py", line 80, in rendered_content
    content = template.render(context)
  File "django/template/base.py", line 148, in render
    return self._render(context)
  File "django/template/base.py", line 142, in _render
    return self.nodelist.render(context)
  File "django/template/base.py", line 844, in render
    bit = self.render_node(node, context)
  File "django/template/base.py", line 858, in render_node
    return node.render(context)
  File "django/template/loader_tags.py", line 126, in render
    return compiled_parent._render(context)
  File "django/template/base.py", line 142, in _render
    return self.nodelist.render(context)
  File "django/template/base.py", line 844, in render
    bit = self.render_node(node, context)
  File "django/template/base.py", line 858, in render_node
    return node.render(context)
  File "django/template/loader_tags.py", line 65, in render
    result = block.nodelist.render(context)
  File "django/template/base.py", line 844, in render
    bit = self.render_node(node, context)
  File "django/template/base.py", line 858, in render_node
    return node.render(context)
  File "django/template/base.py", line 904, in render
    return render_value_in_context(output, context)
  File "django/template/base.py", line 881, in render_value_in_context
    value = force_text(value)
  File "django/utils/encoding.py", line 83, in force_text
    s = six.text_type(s)
  File "wagtail/wagtailcore/blocks/stream_block.py", line 311, in __str__
    return self.stream_block.render(self)
  File "wagtail/wagtailcore/blocks/base.py", line 216, in render
    return self.render_basic(value)
  File "wagtail/wagtailcore/blocks/stream_block.py", line 201, in render_basic
    [(force_text(child), child.block_type) for child in value]
  File "wagtail/wagtailcore/blocks/stream_block.py", line 201, in <listcomp>
    [(force_text(child), child.block_type) for child in value]
  File "django/utils/encoding.py", line 83, in force_text
    s = six.text_type(s)
  File "wagtail/wagtailcore/blocks/stream_block.py", line 251, in __str__
    return self.block.render(self.value)
  File "wagtail/wagtailcore/blocks/base.py", line 216, in render
    return self.render_basic(value)
  File "wagtail/wagtailcore/blocks/base.py", line 223, in render_basic
    return force_text(value)
  File "django/utils/encoding.py", line 83, in force_text
    s = six.text_type(s)
  File "wagtail/wagtailembeds/blocks.py", line 21, in __str__
    return embed_to_frontend_html(self.url)
  File "wagtail/wagtailembeds/format.py", line 10, in embed_to_frontend_html
    embed = embeds.get_embed(url)
  File "wagtail/wagtailembeds/embeds.py", line 128, in get_embed
    return Embed.objects.get(url=url, max_width=max_width)
  File "django/db/models/manager.py", line 92, in manager_method
    return getattr(self.get_queryset(), name)(*args, **kwargs)
  File "django/db/models/query.py", line 361, in get
    num if num <= MAX_GET_RESULTS else 'more than %s' % MAX_GET_RESULTS

Thanks! It looks like we either need to add a unique constraint, or make the get_embed function handle the error better.
The Embed model is just a cache, so it should be safe to delete the duplicate in the database. Any missing Embed records will be regenerated.

@lee-kagiso I take it you're using SQLite as your database backend (based on the lack of postgresql / mysql client library under Packages in the Sentry report)?
We have a unique_together constraint on the Embed model which should prevent these duplicates from occurring at the database level: https://github.com/torchbox/wagtail/blob/8bf9851e01e8a5128cfc9d235b165d23954fdb41/wagtail/wagtailembeds/models.py#L41. However, according to http://stackoverflow.com/questions/8844283/django-unique-together-not-preventing-duplicates these are not respected by SQLite. If so, I don't think there's much we can do to avoid this on SQLite.

We are using Postgres...
On Fri, Oct 23, 2015 at 2:20 PM, Matt Westcott<EMAIL_ADDRESS>wrote:

@lee-kagiso https://github.com/lee-kagiso I take it you're using SQLite
as your database backend (based on the lack of postgresql / mysql client
library under Packages in the Sentry report)?
We have a unique_together constraint on the Embed model which should
prevent these duplicates from occurring at the database level:
https://github.com/torchbox/wagtail/blob/8bf9851e01e8a5128cfc9d235b165d23954fdb41/wagtail/wagtailembeds/models.py#L41.
However, according to
http://stackoverflow.com/questions/8844283/django-unique-together-not-preventing-duplicates
these are not respected by SQLite. If so, I don't think there's much we can
do to avoid this on SQLite.
—
Reply to this email directly or view it on GitHub
https://github.com/torchbox/wagtail/issues/1745#issuecomment-150557350.

Hmm... what do you get if you enter \d wagtailembeds_embed at the psql command line? (I'm particularly interested to know if there's a UNIQUE CONSTRAINT, btree (url, max_width) line at the bottom.)

Is max_width set toNone in the duplicates? I'm not sure if the unique_together constraint is enforced if one of the fields doesn't have a value.

I get this:
kagisocms=# \d wagtailembeds_embed
                                      Table "public.wagtailembeds_embed"
+---------------+--------------------------+------------------------------------------------------------------+
|    Column     |           Type           |                            Modifiers                             |
+---------------+--------------------------+------------------------------------------------------------------+
| id            | integer                  | not null default nextval('wagtailembeds_embed_id_seq'::regclass) |
| url           | character varying(200)   | not null                                                         |
| max_width     | smallint                 |                                                                  |
| type          | character varying(10)    | not null                                                         |
| html          | text                     | not null                                                         |
| title         | text                     | not null                                                         |
| author_name   | text                     | not null                                                         |
| provider_name | text                     | not null                                                         |
| thumbnail_url | character varying(200)   |                                                                  |
| width         | integer                  |                                                                  |
| height        | integer                  |                                                                  |
| last_updated  | timestamp with time zone | not null                                                         |
+---------------+--------------------------+------------------------------------------------------------------+
Indexes:
    "wagtailembeds_embed_pkey" PRIMARY KEY, btree (id)
    "wagtailembeds_embed_url_12c617386ccf42c1_uniq" UNIQUE CONSTRAINT, btree (url, max_width)

I believe this is now fixed as of #6635 and #6815 - #6635 replaced the old unique_together constraint (with its unexpected treatment of null fields) with a unique constraint on the hash field (where that quirk doesn't arise), and #6815 cleans up the duplicate records that were presumably created as a result of this bug.


Add cheat sheet for git

start and document git commands

Denke das Sheet ist jetzt schon etwas überladen!
Lieber die wichtigsten Befehle mit guter Dokumentation und passendem Workflow als alles.
Eine Sammlung von bestehenden Dokumenten ist ebenfalls hilfreich!

Improve the existing git-cheat-cheat.
Adding a .png to git is not optimal, please document why @TheBlackBlade
Divide into 2 separate (text) files:

[ ] CONTRIBUTE.md (GitHub workflow, see this example
[ ] crib-sheet_git.md (git setup, merging, ssh)

Focus is on necessary commands, collect links to existing sources.
Another thing, (source: Huelk) is wrong in so many ways. Please start an issue to discuss and integrate proper citation!

Adding a .png to git is not optimal, please document why @TheBlackBlade
It's sub-optimal, because a changed pixel in the image file translates to a completely new file for git. I thought his would be neglectable, since the file will not be changed.
What is your professional solution to reference problems, external cloud storage or link libary?

Probably a markdown file with a proper structured collection of links to existing cheat-sheets might be one solution?
e.g.
git
git-cheat-sheet by github
git-cheat-sheet by tower
SQL
PostgreSQL-cheat-sheet
Python/conda
conda-cheat-sheet
pandas
pandas-cheat-sheet
matplotlib
matplotlib-cheat-sheet
seaborn
seaborn-cheat-sheat

@Ludee When I asked the group about this problem, @khelfen mention the "large file storage" extention for git (https://git-lfs.github.com/).
with that, large picture files or videos can be remotely saved and the git only contains a text pointer to the storage location.
Please review and evaluate, if this would be an appropriate method of handling images and videos in the future.

So let's just go on as @nailend suggested? So that we can close all pending issues in this repo

Please review and evaluate, if this would be an appropriate method of handling images and videos in the future.

This looks interesting. I'm not a big fan of too much git extensions. Handling images in Issues an PR is already good. And git isn't made for media and data.


Setup HAOS on Kubevirt

[x] remove github cache for provisioner deploy, can’t cache something in nix build cache (or we should cache that too)
[x] Add back buildbuddy rbe for debugging
[x] Remove the cwd from the deploy step
[x] Remove the looking up 1password files, only env variables
[x] fix HomeKit integrations
[ ] setup teleport ssh server
[ ] setup teleport kubernetes
[ ] setup pulumi
[ ] Cleanup existing multus installation
[ ] Install multus
[ ] Add haos vm to the lan
[ ] Add static ip to haos vm
[ ] Setup haos
[ ] Install esphome
[ ] Install zigbee2mqtt
[ ] Install matter server
[ ] Set root password to random value
[ ] Set Maarten password to random value
[ ] Figure out a way to store haos disk image on zfs

https://www.reddit.com/r/rancher/comments/ilzdp7/howto_set_up_k8s_or_k3s_so_pods_get_ip_from_lan/ Good how to article

https://medium.com/@arbnair97/virtual-machine-orchestration-in-kubernetes-using-kubevirt-91bd0e81a5bd


[GenericTilePolygonEditor] Add support for Pan and Magnify gestures.

Fixes https://github.com/godotengine/godot/issues/94849

Thanks!


_DecoratorContextManager.call() got an unexpected keyword argument 'input_ids'

Traceback (most recent call last):
File "/home/liudi/model/Video-CCAM-main/test.py", line 45, in
response = videoccam.chat(messages, images, tokenizer, image_processor, max_new_tokens=512, do_sample=False)
File "/home/hisense/.cache/huggingface/modules/transformers_modules/modeling_videoccam.py", line 367, in chat
output_ids = self.generate(
TypeError: _DecoratorContextManager.call() got an unexpected keyword argument 'input_ids'

Sorry, but I cannot reproduce this without knowing your settings.


Moderation hold

Allow moderator to approve or reject comments before rendering.

So how do we implement this? Just brainstorming here:

the moderation dashboard must be available only to the domain owner
therefore each domain must be reigstered (reject all requests from unregistered domains)
this means each domain owner must be given a unique access token; this token will be publically known so it can't act as identification for the moderator dashboard

The registration database will store the (domain, owner email, public key, salted password hash) columns. The comment database will now store (domain, url, comment text, name, email*, timestamp, moderator approved boolean).
* email will be a new feature (optional of course)
This design will have zero increase in complexity for the end-user: they just need to register to get the embed code (which will contain the public key as a Commento.init parameter). Then they can login to approve each and every comment.
Feedback on this design?

At this point it should probably be a service haha :D

So, how does that works? I'm just thinking out loud to try to understand the whole flow.
An user wants to plug the commento widget to her website.

She first, must register herself (providing credentials, like: email and password) on some kind of commento web-app?
Then, she creates as many domains as she wants
Then, on her website she will simply plug our javascript/css libs and initialize the widget by calling Commento.init({token: "xxx", domain: "something..."})
Her audience will then submit comments on the domain provided in the initialization configuration.
Finally, She will be able to sign in to her Commento Account, to list all comments on a given domain and then she will be able to moderate the comments

Does that makes sense?

Well it doesn't need to be SaaS for there to be a moderator dashboard. The server just has /admin routes that have a dedicated web interface for those functions. Credentials can either be supplied by the config when you run it, or printed to console on startup or whatever. Just a master password or something is probably sufficient. Multiple moderators and logins is trickier, but could be done down the line.


LIKE fails when new lines are present

LIKE does not search a string when it contains a new line anywhere in the string. It does not error, but does not return the line even though the text is contained within, when the expected behavior is to return all rows that match.
Attached is a patch that may be applied to the latest as of the time of this issue filing. This adds a new SELECT test for the LIKE issue, along with a new memory table, and changes the relevant tests to also look for the new table.
patch.txt

Thank you for reporting this issue, we'll take a look as soon as we can.

Thank you for the catch, @Hydrocharged! Could you open a PR with the tests that you provided? The fix for them is already in https://github.com/src-d/go-mysql-server/pull/817.

The fix is already merged, but the tests no. I'm reopening this so we can track the tests from @Hydrocharged.

Tests merged in #820.


The type 'Bind' does not define the field, constructor or member 'each'.F# Compiler39

The type 'Bind' does not define the field, constructor or member 'each'.F# Compiler39 in SafeClient.fs line 120

Hello, did you try this in the browser or locally?
@davedawkins I think those changes are still unreleased right?

I may be behind on a commit. I had to redo some of the Bind.each naming.
Will sync tomorrow
On Thu, 16 Sep 2021 at 19:16, Angel D. Munoz @.***>
wrote:

Hello, did you try this in the browser or locally?
@davedawkins https://github.com/davedawkins I think those changes are
still unreleased right?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/davedawkins/Sutil/issues/41#issuecomment-921135051,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AACFV3MZYXR63AK4L6EASLTUCIX7LANCNFSM5EFIYHDQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Everything pushed, packaged, released to nuget and the REPL. Thanks for reporting the issue, and thanks Angel for responding.


Question: Http POST very slow, take too much time

Starting a bash on the GRAFANA container and running something like
curl -X POST http://<IP_ADDRESS>:8099/api/store -H 'Content-Type: application/json' -d '{"topic":"ttttest", "value":"test"}'
works as expected.
Let it POST out from Grafana GUI fired by a button, it tooks up to 10s. I do not find the reasons, what might cause that a high delay.
Did someone also faced this issue and how this issue can be improved.
Thanks a lot in advance.

@uid01089 Are you running curl from your laptop/pc, the one you are using to connect to Grafana?
Data Manipulation form execute POST request from the browser, which may cause delay compare to the host you are running the command.


Clarify which values in starter.ini require a docker image rebuild.

Currently it is not clear in the comments of starter.ini and perhaps the README which values require the INVALIDATE_CACHE value to be incremented thus forcing a rebuild of the workspace image.
Make it more clear.

This should be clear enough
####################
# starter.ini
# Configuration for gitpod to build your stack as you like it
# Note: Boolean values should always be set to 1 (true) or 0 (false)
# Note: Version numbers for node packages support semantic versioning or can be left blank
# IMPORTANT NOTE: Unless specified in the comments for a section, configuration values are
# only used first time a workspace is created and not when the workspace is restarted.
# Furthermore, unless stated otherwise in the comments, any changes made to a value before
# a workspace is created for the first time requires a rebuild of the Docker image layers
# To rebuild the Docker image layers, invalidate the docker cache by incrementing the
# value for INVALIDATE_CACHE in .gitpod.Dockerfile
####################


Typo fix

@JoeDavies-MSFT - Can you please assist to check, approve and merge the content of this PR?
Thank you.


阿拉伯数字1..9不能竖向显示

Yeah, of course. The numbers are INTENDED to be displayed "horizontally", but with a clockwise rotation of 90°.


release updates to breadcrumbs.html

Currently deployed version fails to build if meta not set. This is fixed in master. A deploy would be great!
In master [sphinx_rtd_theme/breadcrumbs.html: 9-15]:
{% if meta is defined and meta is not none %}
{% set check_meta = True %}
{% else %}
{% set check_meta = False %}
{% endif %}

{% if check_meta and 'github_url' in meta %}

On 0.1.10-alpha [sphinx_rtd_theme/breadcrumbs.html: 13]:
{% if meta is defined and 'github_url' in meta %}

See also rtfd/readthedocs.org#2590

I've just released 0.2.0


Upgrade to TestNG 6

From<EMAIL_ADDRESS>on March 16, 2011 09:21:31
Upgraded to TestNG 6
Original issue: http://code.google.com/p/powermock/issues/detail?id=316

From<EMAIL_ADDRESS>on March 16, 2011 01:21:42
Labels: Milestone-Release1.4

From<EMAIL_ADDRESS>on March 23, 2011 00:19:13
Summary: Upgrade to TestNG 6

From<EMAIL_ADDRESS>on April 05, 2011 11:53:51
Status: Fixed


Updating Godeps to experimental L3 changes and test fixes

Container godep update to latest ofnet that contains changes to support L3 without bgp for cloud use case and bgp cleanups
L3 test fix for intermittent failures (Need to be revisited after observation).

LGTM


Include AWS CodeStar/CloudFormation support

It would be great if the boilerplate included template.yml and buildspec.yml to integrate with downstream dependencies like CodePipeline/CodeDeploy.

Hi Chad, I think that's a great suggestion! However, I don't have any prior experience working with CloudFormation or CodeStar.
In case you haven't already found out, Travis CI is already set up to deploy directly to AWS Lambda using the JavaScript AWS SDK (see deploy.js).
Nevertheless, I tried out CodePipeline with CloudFormation (following this tutorial) and came up with #17. Since I'm not very familiar with different workflows when using CloudFormation, could I get you to review the PR?
My concern is that this configuration is specific only for CodePipeline - could I kindly get some input regarding how to set this up correctly?


Fix broken checkstyle validations

Fixes https://github.com/stephanenicolas/toothpick/issues/134

Coverage remained the same at 94.712% when pulling 33588312c22b70abe7b8ae8a7358b74369be4cea on eygraber:checkstyle-fixes into 37bcdffb223b6b3f935046e86fc319c12e6a5c11 on stephanenicolas:master.

Great!
Thx @eygraber


Consider publishing crate as arrow-datafusion

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
This is probably a bad idea but we can at least document that we considered this :slightly_smiling_face:
We are already at version 4.0.0 for the datafusion crate, and really that version number does not represent the maturity of the product.
Now that we have moved to a new repo, we could consider publishing a new arrow-datafusion crate and deprecate the existing datafusion crate. This would give us the opportunity to start again with version numbers and reset expectations.
Describe the solution you'd like
See above.
Describe alternatives you've considered
The alternative is to not do this and release 5.0.0 the next time we change an API.
Additional context
None.

@alamb @jorgecarleitao It would be good to get your opinion on this

@andygrove  I personally think we should not confuse things by adding a new crate name. Once we get the arrow-rs versions releasing more regularly then I think we should start releasing datafausion on whatever cadence makes sense for it
I think people can draw their own conclusion of its maturity based on the number / frequency of major releases
Other than marketing / appearance of maturity, I don't think there is any  technical difference between publishing datafusion major versions and pre 1.0 versions 0.1, 0.2, etc.

IMO it is more important to have a version that correctly represents the state of the software than datafusion in cargo.
IMO 0.X is the correct state of the software, both from a security point of view and if we plan to integrate Ballista more closely with DataFusion.
I find it odd that the vast majority of the software in Rust is pre-1, but arrow, parquet, DataFusion, etc, is post-1 (and all of them with an API that is essentially unstable). IMO that is an historical artifact, and we rather correct it sooner than later.
The perception of 3.0 in Rust is that the software had a significant maturation time.
From a users' perspective, the switch is a very low effort and I do think that we have the necessary channels to market any change (mailing, slack, github, cargo.io, etc).

I honestly don't have a strong opinion and can with whatever versioning
scheme. If I am left to my own devices, I would likely keep on the current
versioning scheme
On Sun, May 2, 2021 at 2:13 PM Jorge Leitao @.***>
wrote:

IMO it is more important to have a version that correctly represents the
state of the software than datafusion in cargo.
IMO 0.X is the correct state of the software, both from a security point
of view and if we plan to integrate Ballista more closely with DataFusion.
I find it odd that the vast majority of the software in Rust is pre-1, but
arrow, parquet, DataFusion, etc, is post-1 (and all of them with an API
that is essentially unstable). IMO that is an historical artifact, and we
rather correct it sooner than later.
The perception of 3.0 in Rust is that the software had a significant
maturation time.
From a users' perspective, the switch is a very low effort and I do think
that we have the necessary channels to market any change (mailing, slack,
github, cargo.io, etc).
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/apache/arrow-datafusion/issues/234#issuecomment-830849093,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADXZMLGPGJKCHQXCTOGKUDTLWI23ANCNFSM436N3ARA
.

I personally think it is too late to consider this now. If anyone objects then feel free to re-open or create a new issue.


Hundreds of connections make mongodb query slowly

FerretDB version
Starting FerretDB v1.21.0... {"version": "v1.21.0", "commit": "a3fb274d25bac13aea594ec34d603338bc00bfc3", "branch": "unknown", "dirty": true, "package": "docker", "debugBuild": false, "buildEnvironment": {"-buildmode":"exe","-compiler":"gc","CGO_ENABLED":"0","GOAMD64":"v1","GOARCH":"amd64","GOOS":"linux","go.version":"go1.22.1","vcs":"git","vcs.time":"2024-03-20T06:51:32Z"}, "uuid": "9e87c620-5030-4eae-ae5f-72513027d594"}
Backend
PostgreSQL : 14
Environment
OS: centos 7.9 x86_64
Deployment: Docker using ghcr.io/ferretdb/ferretdb:v1.21.0 image
Deployment details: Docker version 26.0.1,
docker-compse file is :
services:
postgres:
image: postgres
environment:
- POSTGRES_USER=polaruser
- POSTGRES_PASSWORD=123456789
- POSTGRES_DB=ferretdb
volumes:
- ./data:/var/lib/postgresql/data
ferretdb:
image: ghcr.io/ferretdb/ferretdb:1.21.0
restart: on-failure
ports:
- 27017:27017
environment:
- FERRETDB_POSTGRESQL_URL=postgres://@postgres:5432/ferretdb
- log-level=debug
networks:
default:
name: ferretdb
What did you do?
There are serval apps connect to ferretdb, and there are hundreds connectons ,
What did you expect to see?
ferretdb support  hundreds of connectons well.
What did you see instead?
no more connection can be established successfully, meanwhile, the query through mongosh is very slow even the query is "show dbs;",  sometimes, the query spend 15+ seconds.

Did you check the postgres documentation about max_connections?

Did you check the postgres documentation about max_connections?

Hi, thanks for your reply.
Yes ,I have set the postgres max_connections to 10000, and I checked the remain connections in postgres, 9973 connections free.   I login to postgresql and query something , it  executed quickly.  meanwhile, I login use mongsh through ferretdb , query something, it run slowly.  That means, postgres is running well, the ferretdb has performance issue.
ferretdb=#  select max_conn-now_conn as resi_conn from (select setting::int8 as max_conn,(select count(*) from pg_stat_activity) as now_conn from pg_settings where name = 'max_connections') t;
resi_conn
  9973

(1 row)
Is there any  way tuning  the ferrendb parameters to make ferrentdb have a good performance? Or, is there any way to deploy multi-ferretdb architecture ？

Please try running FerretDB with debug logs. There should be messages about connection establishment, etc. It is hard to understand what is going on without logs.

@AlekSi
Ok, here is the debug logs,  and I found the cpu usage is very high of ferretdb, I am running ferretdb by docker, and the docker is running on a 8 core cpus server.

log27017-3.log

Please download http://<IP_ADDRESS>:8088/debug/pprof/profile?seconds=30 (address is configured by --debug-addr / FERRETDB_DEBUG_ADDR) and attach the file


Update title on equality and diversity start page

Context

Changes proposed in this pull request

Guidance to review

Link to Trello card

https://trello.com/c/PhKiDWKY
Things to check

[ ] This code does not rely on migrations in the same Pull Request
[ ] If this code includes a migration adding or changing columns, it also backfills existing records for consistency
[ ] API release notes have been updated if necessary
[ ] New environment variables have been added to the Azure config

thanks @malcolmbaig!


use current cpu arch instead hard-coding

I've already signed a CLA and changed my primary email. Just wondering what should I do next?

/check-cla

I've already signed a CLA and changed my primary email. Just wondering what should I do next?

Did you click on the link signup this project as an employee?

I've already signed a CLA and changed my primary email. Just wondering what should I do next?

Did you click on the link signup this project as an employee?

I signed up as an individual, is that ok?

/check-cla

@wxw-matt
try git reset --soft HEAD ~ 1 and force push
Maybe the comin was the wrong email (CLA email)
Thanks for your tip.
Just pushed, hope it works.

@wxw-matt it appears your still having issues with the CLA. Can you verify the commit email matches the email you used to sign the CLA? (this might help: https://stackoverflow.com/questions/29876342/how-to-get-only-author-name-or-email-in-git-given-sha1)

@wxw-matt it appears your still having issues with the CLA. Can you verify the commit email matches the email you used to sign the CLA? (this might help: https://stackoverflow.com/questions/29876342/how-to-get-only-author-name-or-email-in-git-given-sha1)

Try again to fix this, please have a look again.

/check-cla

/check-cla

/check-cla

Hey there @wxw-matt – sorry to see you're still having issues with your CLA! If you've tried all the above-mentioned suggestions regarding your commit email matching the one you used to sign the CLA, then you may need to log a ticket with the Linux Foundation Helpdesk to get this resolved – see here. Hopefully this will have things fixed swiftly so we can review your contribution.

Testing
/close

/reopen

@wxw-matt , Can you look into the instructions to complete the EasyCLA? Thanks.

Hi Karen,
I have tried following the instructions and in different ways.
However, I never got any luck.
I will do it again today and hopefully it works.
Thanks
Best wishes,
Matt
On Tue, Nov 23, 2021 at 12:20 PM Karen Bradshaw @.***>
wrote:

@wxw-matt https://github.com/wxw-matt , Can you look into the
instructions to complete the EasyCLA? Thanks.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/kubernetes/website/pull/28908#issuecomment-976096328,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANF6Y5TUZNB34MKEZMVNYL3UNLXM7ANCNFSM5AGUQUWA
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Hey there! @wxw-matt, looks like you haven't signed the CLA yet. Could I please have you do that? https://github.com/kubernetes/community/blob/master/CLA.md

I would like to cancel this pull request and resubmit it because I tried different approaches, however, none of them worked.


on sign up after wrong otp person is created

on sing up after wrong otp person is created. So that I can enetr severel persons with the same tax_id, but diffrent birth_date
if otp is incorrect person should not be created

duplicates #2407


Clear User‘s Data While Uninstalling

Issue Type: Feature Request
Add a option for the uninstaller to clear user‘s data and settings.
This is a feature request. I'm not sure if anyone has reported this yet, but I didn't find any.
I think this feature could be considered after this issue https://github.com/Microsoft/vscode/issues/3884 is done, because currently the user's data locations are very complicated.
VS Code version: Code - Exploration 1.32.0-exploration (d0aafbad741c2316c92b369113abe0205f41e84e, 2019-02-13T07:02:19.413Z)
OS version: Windows_NT x64 10.0.18334

#66837

Duplicate. Close now.


np.int error

Dear all,
I am getting the following error:
AttributeError: module 'numpy' has no attribute 'int'. np.intwas a deprecated alias for the builtinint. To avoid this error in existing code, use intby itself. Doing this will not modify any behavior and is safe. When replacingnp.int, you may wish to use e.g. np.int64ornp.int32 to specify the precision. If you wish to review your current use, check the release note link for additional information. The aliases was originally deprecated in NumPy 1.20; for more details and guidance see the original release note at: https://numpy.org/devdocs/release/1.20.0-notes.html#deprecations
I am running it with py39, pytorch 2.3.0 (dgl package  is not compatible with pytorch 2.30) and cu121. Could you help? Thanks
Best

Hi,
I am happy that you're interested in our work!
I don't use "np.int()" in my code, hence I think there is an incompatibility between one of the installed packages and numpy. To resolve this, it would be helpful if you could send the entire error traceback.
Best,
Alex