Access to current state from within transition?

Hi Daniel,
I'm having some trouble with keeping state between many calls to process. e.g: I have a menu that prompts for an eta, then confirms that eta, and I would like to then keep that eta in state:
  state :prompt, _twilio, _job do
    gather method: "GET" do
      say dgettext("call_flows.job.update_eta", "Enter the ETA time in minutes and then press #, or press # to go back"), voice: "alice", language: Gettext.get_locale(TelephoneProvisioner.Gettext)
    end
    redirect method: "GET"
  end

  state :confirmation, _twilio, %{eta: %{unconfirmed: minutes}} do
    gather numDigits: 1, method: "GET" do
      say dgettext("call_flows.job.update_eta", "The time you entered is %{eta} minutes", %{eta: minutes}), voice: "alice", language: Gettext.get_locale(TelephoneProvisioner.Gettext)
      say menu_option("1", dgettext("call_flows.job.update_eta", "if this is correct")), voice: "alice", language: Gettext.get_locale(TelephoneProvisioner.Gettext)
      say menu_option("2", dgettext("call_flows.job.update_eta", "to update the ETA")), voice: "alice", language: Gettext.get_locale(TelephoneProvisioner.Gettext)
    end
  end

  state :confirmed, _twilio, job do
    say dgettext("call_flows.job.update_eta", "ETA confirmed at %{eta} minutes", %{eta: minutes}), voice: "alice", language: Gettext.get_locale(TelephoneProvisioner.Gettext)
    redirect method: "GET"
  end

  def transition(:prompt, %{Digits: minutes} = twilio, job) do
    job = Map.put(job, :eta, %{unconfirmed: minutes})
    state :confirmation, twilio, job
  end

  def transition(:confirmation, %{Digits: "1"} = twilio, job) do
    state :confirmed, twilio, job
  end

  def transition(:confirmation, %{Digits: "2"} = twilio, job) do
    state :prompt, twilio, job
  end

The problem is that I don't have access to the job.eta.unconfirmed from the :confirmation state when I get to the def transition(:confirmation, %{Digits: "1"} = twilio, job) - job is newly injected from the process call on my HTTP endpoint.
Is there a way to use the existing state in transitions?
As a side note, you can see that I'm having to repeat voice: "alice", language: Gettext.get_locale(...) for all of my say calls - is there a way around this?
Cheers,
Brenton.

This sounds like it could be a bug. I'll see if I can reproduce it in a test case and then fix it. The existing state should be passed through for the whole duration of a call.

state.name is passed to transition here for pattern matching against, but the state.options isn't.
It'd be a breaking change, but you could pass the whole state, and patten match from within that instead, that way a user could combine current state data, twilio, and options on to create the data for the next state.

@brentonannan I wrote this test case to try to reproduce, and couldn't: https://github.com/danielberkompas/telephonist/commit/f26bbd3fa554114a5d3356160159d626c608b8ff
Are you sure you are on the latest version of Telephonist?
Regarding the say problem, it looks like you're stuck with that for now. ExTwiml needs to be updated to support passing a variable for verb options, like so:
options = [voice: "alice"]
say "Hello", options
say "World", options

It's actually in the second transition - from :confirmation to :confirmed that the data is lost - because it's available as part of the :confirmation state, but that's not available in the transition.

Also, what would be nice for ExTwiml is if known tags could have known attributes set in config (although that wouldn't help with dynamic ones like my language: Gettext.getlocale(...). I've been trying, unsuccessfully so far, to write macros to wrap say, not sure if I'm barking up the wrong tree there.

@brentonannan I just made a new release of ExTwiml which allows you to set default attributes for verbs: https://github.com/danielberkompas/ex_twiml/releases/tag/v2.1.0


Sidebar icon disappears

Sidebar icon disappears, I can’t find installed extension icon on my code-server, is it a known issue? (However it’s clickable)
I installed GitLen

Web Browser: Safari & vsapp
Local OS: iPadOS 13.4.1
Remote OS: Raspbian (Raspberry Pi4)
code-server --version: 3.3.0

@michael34435 Can you try upgrading to v3.3.1?

upgraded, not works

Is it possible to get any logs? 🤔

Managed to reproduce this.
Not sure if it's a bug with code-server or we built the extension wrong.
cc @cmoog

Not a build issue given that we're pulling the release directly from Github here.

Looks like it's a Safari issue, icon shows up in Chrome.

yup, i had tried chrome(in macOS) this morning. it works

But Safari in macOS ... no icons like iPad too


Highlight issue with Array

VSCode Version: 1.8.1

let x: Array<{ name: string, fn(): any, field: string }>

notice after fn() the highlight does not work.

This issue was moved to Microsoft/TypeScript-TmLanguage#398

Thanks. btw, "This issue IS moved to ...". Please delete this comment after you see it. 🌷


Verify integrity from .torrent?

Hello!
If I have the files from a torrent. And I have the .torrent file. With RHash, is it able to read the .torrent file to verify all the files are good?
Thanks!

RHash can create a torrent file (with --torrent option), but not verify it.
The program can verify only Bittorent InfoHash (BTIH) and only uTorrent version of it. You can calculate it using --btih option.
Torrent support will not be developed in future, because InfoHash value is not uniquely defined - various torrent clients calculate different InfoHash for the same file.


Added Romanian language

Coverage remained the same at 94.629% when pulling 67fe802daa92f158d409197463bfcef79b36ef74 on laurentiuonac:master into 44f48893802ef8cdd0b003a778e3ed2248dcdf9b on MarshallOfSound:master.

Changes Unknown when pulling 1505b23b2469291473de931978fd5dbea690fce5 on laurentiuonac:master into ** on MarshallOfSound:master**.


get-methods for direct permissions

AccessControlContext doesn't have any getters for direct permissions, only for effective permissions. To keep symmetry with the setters, which only affect direct permissions, we will add the corresponding get-methods.

commit 4ec81c2 completes the related changes and closes this issue


Restarting i3-gaps breaks desktop updating.

If you restart i3-gaps using $mod+shift+r the desktop display will stop updating.

Also experiencing this issue.

For now I'd say restart the bar when restarting the WM.

so for the lazy: i3 that works for me
exec_always
exec_always --no-startup-id killall polybar 2>/dev/null | true && polybar --reload pascalwhoop

the 2> hides the error message if no polybar exists (kind of redundant but clean), the |true avoids errors if no polybar started yet (at boot), the rest is explained in the link


Wither ore cannot be found

The wither ore does not spawn. I have searched several places but I can't find anything.

Hey! MisterGamerDeveloper i have send this report to my developer team. you will get a message back soon as possible.

Hey MisterGamerDeveloper  This is indeed a mistake on our part. we will fix this as soon as possible in the next update! We are sorry.

ok thank you! when will the update come ?

The update come in: April ( Curseforge & Modrinth )

alright thanks! i will wait for the update

Your welcome! and thank you for the bug report.

no problem


User Access Administrator role details

Hi, it'll be great to have additional details of the permissions granted to "User Access Administrator".
The documentation on this and "Access control in Data Lake Store" discuss how ACLs are applied but not as much on how they are administered. It discusses super-users and owning users / groups but not the user access administrator.
This page states "Add and remove roles" but this suggests it's for the resource level not the data access level. And the "Data access rights" section suggests it's regarding reading data not applying data acls.
Upon experimenting (and the name gives it away) the user access administrator can alter data ACLs but it would be helpful to have this explicitly documented.
thanks
Alex.

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

ID: b9c63a1e-74b7-2d5f-3d9f-20ee3621c01c
Version Independent ID: 8719864f-eac5-e495-8ae1-785ad6c8cf01
Content: Overview of security in Data Lake Store
Content Source: articles/data-lake-store/data-lake-store-security-overview.md
Service: data-lake-store
Product: unspecified
GitHub Login: @nitinme
Microsoft Alias: nitinme

@AlexKeySmith Thanks for the feedback! I have assigned the issue to the content author to evaluate and update as appropriate.

@twooley
Hi Tessa:
I've been tasked with looking at issues older than 90 days. Is this one ready to be closed now?
Thanks,
Paul Goldman
Sr Technical Writer


App/Nav: Desktop - help menu

Figma:
https://www.figma.com/file/RMXc7jUWiC6amxz9Q2JK5L/Onboarding-and-Wallet-UX?node-id=718%3A30
The links in the help menu have changed for some reason.

[ ] Change the links in the help menu back to the text only links (see screenshot) HIGH
[ ] Copy update "Make a test trade" -> "Trading tutorial" + move this to the top of the list (this link does not have an icon) HIGH

Comparison:

looks done. closing


Add game import for custom schedule leagues for whole region

as of now custom schedule leagues games can only be imported per league.
support an option fo import a list for multiple leagues on region basis

covered by #577


[Bug] - contributors page - link as text is wrong

What happened?
When you go to the contributors page, the link to the page is wrong. It should be "contributor" instead of "contributer"

Also the text "React Us" should be replaced by "React out to Us", this looks more friendly and professional.

How can we reproduce this bug?
Just go to the Contributors page
Desktop Information (Optional)
None
Urgency (Optional)
High priority
Record

[X] I agree to follow this project's Code of Conduct
[X] I have checked the existing issues
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

Please assign me this issue under GSSoC'23

Too easy issue


Add dependency for wicd-gtk to use the systray

wicd-gtk supports systray notifications as long as the python notification library is installed.  This change adds that library as a dependency.

Built locally, thanks.


Add support for laptops that run Debian + KDE

Debian will always be running KDE

Not longer relevant, Debian did not work out because of an extremely outdated version of KDE


fix: use dedicated EL context when creating connector

Issue
https://gravitee.atlassian.net/browse/APIM-1155
Description
Fixes the issue related to the endpoint target based on an EL expression that uses API properties. Api properties can, under certain circumstances be mixed and leads to errors during connector instantiation. This ends with 503 for the final user.

:tada: This PR is included in version 2.0.11 :tada:
The release is available on:

2.0.11
GitHub release

Your semantic-release bot :package::rocket:


Ebook version 0.1, closes #449

[X] Have you followed the contributing guidelines?
[X] Have you explained what your changes do, and why they add value to the Guides?

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.
This is a first attempt, the script could be improved.

sofitchafodze  instagram code

Hi @MikeMcQuaid, I don't know Ruby, I could try in python, maybe.

@elbaulp Leaving it in Bash should be fine but I'd rather something like a Rubygem was installed or used in that case rather than manual perl substitutions. Thanks!

Passing on this as there's been no movement in almost 2 months, sorry.


Allow denotation char inside mentions

Fixes #177

I think this should be fixed by PR #252, I also ran into this issue a while back. Would love for you to verify whether this actually covers your issue too, in which case the PR can be closed

@TimDirks Yes, I believe your PR has fixed this issue. Maybe we should also mention that allowedChars needs to be updated to allow the @ symbol as well? Default allowedChars don't allow @ today.
Anyway, to illustrate that your code works:
https://github.com/quill-mention/quill-mention/assets/3159565/f727ba4e-0754-4201-a696-08f838a46411


Deploy Checklist: New Expensify 2023-03-31

Release Version: 1.2.93-4
Compare Changes: https://github.com/Expensify/App/compare/production...staging
This release contains changes from the following pull requests:

[ ] https://github.com/Expensify/App/pull/16465
[ ] https://github.com/Expensify/App/pull/16627
[x] https://github.com/Expensify/App/pull/16650
[ ] https://github.com/Expensify/App/pull/16657
[ ] https://github.com/Expensify/App/pull/16670
[ ] https://github.com/Expensify/App/pull/16673
[ ] https://github.com/Expensify/App/pull/16676
[ ] https://github.com/Expensify/App/pull/16704
[ ] https://github.com/Expensify/App/pull/16714
[ ] https://github.com/Expensify/App/pull/16724
[x] https://github.com/Expensify/App/pull/16726
[x] https://github.com/Expensify/App/pull/16781

Internal QA:

[ ] https://github.com/Expensify/App/pull/16360 - @arosiclair

Deploy Blockers:

[ ] https://github.com/Expensify/App/issues/16761

Deployer verifications:

[ ] I checked the App Timing Dashboard and verified this release does not cause a noticeable performance regression.
[ ] I checked Firebase Crashlytics and verified that this release does not introduce any new crashes. More detailed instructions on this verification can be found here.
[ ] I checked GitHub Status and verified there is no reported incident with Actions.

cc @Expensify/applauseleads

:rocket: All staging deploys are complete, @Expensify/applauseleads please begin QA on version https://github.com/Expensify/App/releases/tag/1.2.93-4 :rocket:

Checking off https://github.com/Expensify/App/issues/16761 since we fixed this blocker in the previous deploy but forgot to remove the label.

Regression is completed

Test passed for #16360!


Implement auto redirect

TODO

[x] When writing to file, always use forward slashes
[x] When writing to file, always surround in quotes
[x] Write tests

My PR to this branch addresses the comments to rediraffe.py.
A few upkeep things:

Could you add a few more tests, covering more complex setups & rediraffe_auto_redirect_perc? The one test case was passing but was relying on an ambiguous variable.
Can you further separate the checkdiff and writediff tests? (change test names or wrap in separate classes / files). I had created the tests when there was only 1 builder.

Depends on #9. This PR wraps filenames in quotes.

Closes #4.


Delete user and logout

Before submitting a PR to the dev branch:

[X] Tests passing
[X] Black formatting
[X] Rebase/merge the dev branch
[ ] Note in the CHANGELOG

Additional checks before submitting a PR to the master branch:

Change version in setup.py (?)

Fixes #292.

https://github.com/ScilifelabDataCentre/dds_cli/pull/303


Is it possible to suppress INFO messages using an app-specific environment variable?

Hi there,
Is there an environment variable we can use to set the logger level to error or something else? I'm trying to suppress these messages:
 ...[INFO] Language server with id 2 sent unhandled notification telemetry/event

and any other INFO messages, if possible.
Thank you.

This log comes from your editor, not from ElixirLS

Not quite, I believe. In this instance, Zed does honor the RUST_LOG environment variable, and I can observe its influence in the logs. The errors mentioned above begin to occur when I use the elixir-ls server, so if I'm not mistaken I believe these logs belonged to this elixir-ls.

You are mistaken. The log is originating in the editor. It's complaining that it does not support this LSP notification

You're correct, and I apologize for the oversight. Additionally, I've discovered that the bootstrapping of the server extensions is not respecting the RUST_LOG environment variable. Sorry for any confusion caused.


cups-2.2.2: cupsd won't start when gssapi has been disabled during compilation

In Gentoo bug #606472 we got a report that cupsd from cups-2.2.2 fails to start with its default cupsd.conf file if cups has been compiled without gssapi support.
We've fixed this by conditionally removing the kerberos policy part from cupsd.conf (see gentoo/gentoo@7ee57377c41da4814963ba9a162d03bc466448d8) but perhaps this can be fixed in a more convenient way.

Hmm, OK, we'll look into a better long-term solution.

Fixed in TOT.


odrin7

Expected behavior
Actual behavior
Information about the Issue
Steps to reproduce the behavior

...
...

Looks like this was opened accidentally


Please add tvOS support

Please add in the pod file support for tvOS.

Unfortunately, that is not in our roadmap. You are more than welcome to implement by forking from this repository. Thank you!


adds email_field to FormBuilder

Fixes https://github.com/joecorcoran/judge/issues/31
Pretty straight forward, but thought I'd get another collaborators eyes on it.

+1 @jamesmk


Option to return logits instead of log_probs in forward method

Hi,
With some colleagues, we developed a QA system that uses a whole QA pipeline (Retriever, Reader, Ranker). We use your older version of BertForQuestionAnswering as Reader and now we wish to update it to be compatible with your new release and to add others models as well (XLNet, XLM).
Our system uses the logits outputted by the model in order to rank the answers between different paragraphs (using probabilities outputted by the softmax layer is an incorrect approach for such systems).
However, we understand that in your new API, the forward method of QA models now can only return probabilities. We suggest you add the option to output the raw logits as well.
We could of course overcome this by using the functions self.start_logits and self.end_logits, but we think that this feature can be useful for other users as well

Well, the output of the BertForQuestionAnswering model haven't changed, the returned start_score and end_score are still scores before the softmax.
Can you point more specifically to the changes you are referring to?

I'm sorry. Indeed the BertForQuestionAnswering remains the same.
Actually, I was referring to the implementation of XLNetForQuestionAnswering, which is pretty different from BertForQuestionAnswering (I thought you had standardised the implementation for all QA models and did not check the BERT implementation before posting the issue here)
Please correct me if I am wrong, but I do not see the forward() method outputting the Start and End's logits here (only the softmax probas - start_log_probsand end_log_probs:
https://github.com/huggingface/pytorch-transformers/blob/268c6cc160ba046d6a91747c5f281f82bd88a4d8/pytorch_transformers/modeling_xlnet.py#L1226-L1290

Oh yes the official XLNet implementation uses a beam search for Question Answering so the output is more complex.
I'll see if I can come up with a standardized way to use both.

Thanks, I get it. Maybe caching the start_logits during the search and returning the logits of chosen End and Start positions would be a solution.
By the way, FYI the examples in the documentation for XLNetForQuestionAnswering and XLMForQuestionAnswering are incorrect, both show start_scores and end_scores in the outputs and the example in XLNetForQuestionAnswering uses XLM instead of XLNet
XLNetForQuestionAnswering:

[XLMForQuestionAnswering](https://huggingface.co/pytorch-transformers/model_doc/xlm.html#xlmforquestionanswering:

This is silly/nitpicky - but can we change the title of this issue? Got very worried I had been using the BERT heads incorrectly until I read into the weeds of the comments..

Hello, any news on this standardized head? @thomwolf?
What do you think about my proposition about caching the start_logits during the Beam search and outputting both start_logits and end_logits as it is done with BertForQuestionAnswering?
Or do you have any other ideas?
I can try to work on that and do a PR if you need


Cleanup statement has unrecognized argument

az group delete --name myResourceGroupAG --location eastus
tosses:
az: error: unrecognized arguments: --location eastus
Removing ' --location eastus ' works

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

ID: b66ca071-31b4-d637-725e-f9a3c6f27fe4
Version Independent ID: 6f748c5d-e2bc-8783-5eec-5a0c1b11b34d
Content: Enable web application firewall - Azure CLI
Content Source: articles/application-gateway/tutorial-restrict-web-traffic-cli.md
Service: application-gateway
GitHub Login: @vhorne
Microsoft Alias: victorh

Thanks for the question! We are investigating and will update you shortly.

@justSteve Thanks for the feedback! I have made the change, and it should reflect on the doc within 24 hours.
We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


Add documentation for config file

Let's consider it as a first draft

Should be good enough by now.


Feature/refactor3 iol

Thanks, looks good from my perspective too. You can go ahead and merge it after checking the line note I left above if you want (or i'll do it in the morning)


Debugging of UEI gateway issue

Description
The UEI gateway was not getting deployed on UEI infra.
Goals

[ ] Identify the root cause of the connectivity issues.
[ ] Implement a fix to resolve the connectivity issues.
[ ] Ensure the gateway operates without interruptions.
[ ] Document the debugging and resolution process.

Expected Outcome
The UEI gateway should function without any connectivity issues, ensuring reliable data transmission across the Beckn platform.
Acceptance Criteria

[ ] The root cause of the issue is identified and documented.
[ ] A fix is implemented and tested.
[ ] Detailed documentation of the debugging process and resolution is provided.

Mockups / Wireframes
N/A
Product Name
Beckn
Domain
UEI
Tech Skills Needed

Network debugging
Gateway configuration
Data transmission protocols
System monitoring and logging

Complexity
High
Category
Bug
Sub Category
Connectivity Issues

UEI team was deploying the Gateway on there infra. But they were getting below error.
java.lang.NullPointerException: Cannot invoke "in.succinct.onet.core.adaptor.NetworkAdaptor$Domain.getExtensionPackage()" because the return value of "in.succinct.onet.core.adaptor.NetworkAdaptor$Domains.get(String)" is null
    at in.succinct.onet.core.adaptor.NetworkAdaptor.create(NetworkAdaptor.java:518)
    at in.succinct.onet.core.adaptor.NetworkAdaptor$5$1.create(NetworkAdaptor.java:558)
    at in.succinct.json.JSONAwareWrapper.get(JSONAwareWrapper.java:143)
    at in.succinct.json.JSONAwareWrapper.get(JSONAwareWrapper.java:124)
    at in.succinct.beckn.Request.getContext(Request.java:42)
    at in.succinct.beckn.gateway.controller.BgController.act(BgController.java:244)
    at in.succinct.beckn.gateway.controller.BgController.search(BgController.java:263)

We have check the same issue on our infra and we were also getting the same issue.
After debugging further we got to know that automic registry was down due to which gateway was not getting deployed.
To fix this issue Venky suggested to remove networks folder from the code and rebuild the docker image.
We did the required change on gateway build machine and rebuild the docker image the push it to the docker hub.
With the latest docker image we successfully deployed gateway  on UEI infra.


Implement Linode detail resize tab

This has been implemented! https://github.com/linode/manager/pull/1107


Thank you!

Just wanted to drop in and congratulate you on the next version. We are using your RSDI in our production applications. We use Vue and Clean Architecture and your solution has helped us keep the DI implementation separate from the infrastructure layer. I was searching for such a solution and by accident stumbled on yours. Great job.
If you need any feedback or help my email is dima[at]dreamvention.com

🙌  thank you for the feedback.

I need some help with this:
[ERROR] 01:43:45 Error: require() of ES Module \node_modules\rsdi\dist\index.js from \src\infrastructure\di.ts not supported.
Instead change the require of index.js in \src\infrastructure\di.ts to a dynamic import() which is available in all CommonJS modules.

I updated the version and make all adjustments needed


Feature Request: Allow sync (or bumps) for specific dependencies

Context
vendir currently supports the --directory flag:
$ vendir sync -d my_dependency

But it doesn't save the lock file. I would like vendir to be able to isolate sync/bumps to a specific dependency.

Hey @ciriarte!
Let us know if this captures your intent.
Currently vendir sync only syncs an entire vendir.yml file.
You can use vendir sync --directory to override contents of particular directories by pointing them to local directories. However, this does not update the lock file.
To support syncing only a subset of the dependencies in the vendir.yml, vendir sync would provide the ability to target the given dependencies, and update the vendir.lock.yml with the new metadata.

@ciriarte we wouldnt expect to modify anything other dependencies -- even if they were modified on disk, intentionally or unintentionally -- right?

@ciriarte we wouldnt expect to modify anything other dependencies -- even if they were modified on disk, intentionally or unintentionally -- right?

My expectation is that vendir will only update the directory it is managing and I've instructed in the vendir sync -d some_dir call.

Currently vendir sync only syncs an entire vendir.yml file.
You can use vendir sync --directory to override contents of particular directories by pointing them to local directories. However, this does not update the lock file.

@pivotal-dean
My understanding is that vendir sync -d has a couple of options per the help command:

It does sync local dirs if used with dir/sub-dir[=local-dir] I'm not really interested in this workflow at the moment 🚫
It synchronizes with a remote if the optional [=local-dir] is omitted. This does something close to my desired outcome ✅

If my understanding of 2 is correct, then you captured my wishes perfectly in the following sentence:

To support syncing only a subset of the dependencies in the vendir.yml, vendir sync would provide the ability to target the given dependencies, and update the vendir.lock.yml with the new metadata.

included in https://github.com/k14s/vendir/releases/tag/v0.10.0


What will happen if steam broken accidently?

I am new comer to c++&grpc, after read your example code: async_streaming_server_alarm.cc, i don't understand what will happen if stream rpc call was broken accidently.
In such cases, CallData seems have no chance to delete itself.
Maybe i can use AsyncNotifyWhenDone() to release memory of CallData ?

The status of CallData is modified by the server, in this example it will be set to FINISH after processing 6 times regardless of the state of the rpc.
This is an async server, so be design it's not waiting for any notifications from the client.

Thank you for quick reply.
So for this example, i suppose, there is not task generated by server in such cases:

client close connection
client interrupt rpc call
connection broken

And I notice the example use this line to control the process:
void PutTaskBackToQueue()
{
           alarm_.Set(cq_, gpr_now(gpr_clock_type::GPR_CLOCK_REALTIME), this);
}

Is it too heavy? (too much resource consumption, because it runs again and again for nothing to do)
Why not just idle the task, and push task to queue if there are some data to write?

I think what you want is a synchronous client-server connection, which is not covered by these examples. You can go to gRPC's official repo and have a look at the synchronous examples there.
The short answer is there's no other simple way to put a task back to the queue (at least at the time writing). You can read more about it in our blog post: https://www.gresearch.co.uk/article/lessons-learnt-from-writing-asynchronous-streaming-grpc-services-in-c/


NPM?

Awesome package. But how would I pull this in through npm?

I have not figure out how to make the npm version. I am working on it. If you have any suggestion, let me know

Now, you have the option to do. It was implemented in the commit #8
NPM installation command > npm i google-autocomplete-vue


True zero on axis

Hi,
How can I get a true zero on Y axis? Even if put I set -1000 as lower, no negative ticks are labelled.
Best.
library(ggcyto)
#> Loading required package: ggplot2
#> Loading required package: flowCore
#> Loading required package: ncdfFlow
#> Loading required package: RcppArmadillo
#> Loading required package: BH
#> Loading required package: flowWorkspace
data("GvHD")
fs2 <- GvHD[subset(pData(GvHD), Patient %in% 5:7 & Visit %in% c(5:6))[["name"]]]
autoplot(fs2, "Time", "FL3-H", bins = 64)  + scale_y_flowCore_fasinh(b = 1/5) + ggcyto_par_set(limits = list(x = c(0,1000), y = c(-10, 2e3)), facet = facet_wrap("name"), lab = labs_cyto("marker")) # + theme(axis.text.y=element_blank())
#> Warning: Removed 10 rows containing non-finite values (stat_binhex).
#> Warning: Removed 85 rows containing missing values (geom_hex).

sessionInfo()
#> R version 3.5.3 (2019-03-11)
#> Platform: x86_64-w64-mingw32/x64 (64-bit)
#> Running under: Windows 7 x64 (build 7601) Service Pack 1
#>
#> Matrix products: default
#>
#> locale:
#> [1] LC_COLLATE=English_United Kingdom.1252
#> [2] LC_CTYPE=English_United Kingdom.1252
#> [3] LC_MONETARY=English_United Kingdom.1252
#> [4] LC_NUMERIC=C
#> [5] LC_TIME=English_United Kingdom.1252
#>
#> attached base packages:
#> [1] stats     graphics  grDevices utils     datasets  methods   base
#>
#> other attached packages:
#> [1] hexbin_1.27.2             ggcyto_1.10.0
#> [3] flowWorkspace_3.30.0      ncdfFlow_2.28.1
#> [5] BH_1.69.0-1               RcppArmadillo_<IP_ADDRESS>.0
#> [7] flowCore_1.48.1           ggplot2_3.1.0
#>
#> loaded via a namespace (and not attached):
#>  [1] tidyselect_0.2.5    purrr_0.2.5         lattice_0.20-38
#>  [4] pcaPP_1.9-73        colorspace_1.3-2    htmltools_0.3.6
#>  [7] stats4_3.5.3        yaml_2.2.0          XML_3.98-1.16
#> [10] rlang_0.3.4         pillar_1.4.0        glue_1.3.1
#> [13] withr_2.1.2         Rgraphviz_2.26.0    BiocGenerics_0.28.0
#> [16] RColorBrewer_1.1-2  matrixStats_0.54.0  plyr_1.8.4
#> [19] robustbase_0.93-3   stringr_1.3.1       zlibbioc_1.28.0
#> [22] munsell_0.5.0       gtable_0.2.0        mvtnorm_1.0-8
#> [25] evaluate_0.12       labeling_0.3        latticeExtra_0.6-28
#> [28] Biobase_2.42.0      knitr_1.20          parallel_3.5.3
#> [31] DEoptimR_1.0-8      Rcpp_1.0.1          KernSmooth_2.23-15
#> [34] corpcor_1.6.9       scales_1.0.0        backports_1.1.2
#> [37] graph_1.60.0        IDPmisc_1.1.18      gridExtra_2.3
#> [40] digest_0.6.18       stringi_1.2.4       dplyr_0.8.1
#> [43] grid_3.5.3          rprojroot_1.3-2     tools_3.5.3
#> [46] magrittr_1.5        lazyeval_0.2.1      tibble_2.1.1
#> [49] cluster_2.0.7-1     crayon_1.3.4        rrcov_1.4-7
#> [52] pkgconfig_2.0.2     MASS_7.3-51.1       data.table_1.11.8
#> [55] flowViz_1.46.0      assertthat_0.2.1    rmarkdown_1.10
#> [58] R6_2.4.0            compiler_3.5.3

Created on 2019-09-06 by the reprex package (v0.2.1)

Hey @SamGG , could you please give me your sessionInfo() output? It will help me figure this out.

Hi Jacob. Request updated. Best

Hi @SamGG , I'm having trouble reproducing the plot you're seeing. It seems to be giving me a true zero on the y axis. That is, this:
autoplot(fs2, "Time", "FL3-H", bins = 64) + scale_y_flowCore_fasinh(b=1/5) + ggcyto_par_set(limits=list(x=c(0,1000), y=c(-10, 2e3)), facet=facet_wrap("name"), lab=labs_cyto("marker"))

gives me this:

If I just set the y axis to a minimum of 0, it seems to do that appropriately:
autoplot(fs2, "Time", "FL3-H", bins = 64) + scale_y_flowCore_fasinh(b=1/5) + ggcyto_par_set(limits=list(x=c(0,1000), y=c(0, 2e3)), facet=facet_wrap("name"), lab=labs_cyto("marker"))

I should mention that I am testing using Bioconductor's 3.8 Docker image to test this, which is at R 3.5.2 and ggplot2 3.2.1, so it's possible there is a difference there that is responsible for the misrepresentation of zero. There are minor version bumps on ggcyto and flowWorkspace over what you have but I don't think they are relevant to this. My full sessionInfo() for reference:
R version 3.5.2 (2018-12-20)
Platform: x86_64-pc-linux-gnu (64-bit)
Running under: Debian GNU/Linux buster/sid

Matrix products: default
BLAS: /usr/lib/x86_64-linux-gnu/blas/libblas.so.3.8.0
LAPACK: /usr/lib/x86_64-linux-gnu/lapack/liblapack.so.3.8.0

locale:
 [1] LC_CTYPE=en_US.UTF-8       LC_NUMERIC=C
 [3] LC_TIME=en_US.UTF-8        LC_COLLATE=en_US.UTF-8
 [5] LC_MONETARY=en_US.UTF-8    LC_MESSAGES=en_US.UTF-8
 [7] LC_PAPER=en_US.UTF-8       LC_NAME=C
 [9] LC_ADDRESS=C               LC_TELEPHONE=C
[11] LC_MEASUREMENT=en_US.UTF-8 LC_IDENTIFICATION=C

attached base packages:
[1] stats     graphics  grDevices utils     datasets  methods   base

other attached packages:
[1] ggcyto_1.10.2             flowWorkspace_3.30.2
[3] ncdfFlow_2.28.1           BH_1.69.0-1
[5] RcppArmadillo_0.9.700.2.0 flowCore_1.48.1
[7] ggplot2_3.2.1

loaded via a namespace (and not attached):
 [1] pcaPP_1.9-73        Rcpp_1.0.2          plyr_1.8.4
 [4] RColorBrewer_1.1-2  pillar_1.4.2        compiler_3.5.2
 [7] DEoptimR_1.0-8      BiocManager_1.30.4  zlibbioc_1.28.0
[10] tools_3.5.2         tibble_2.1.3        gtable_0.3.0
[13] lattice_0.20-38     pkgconfig_2.0.2     rlang_0.4.0
[16] graph_1.60.0        Rgraphviz_2.26.0    parallel_3.5.2
[19] mvtnorm_1.0-11      hexbin_1.27.3       gridExtra_2.3
[22] stringr_1.4.0       withr_2.1.2         dplyr_0.8.3
[25] cluster_2.1.0       IDPmisc_1.1.19      stats4_3.5.2
[28] grid_3.5.2          tidyselect_0.2.5    data.table_1.12.2
[31] glue_1.3.1          robustbase_0.93-5   Biobase_2.42.0
[34] R6_2.4.0            rrcov_1.4-7         XML_3.98-1.20
[37] latticeExtra_0.6-28 purrr_0.3.2         corpcor_1.6.9
[40] magrittr_1.5        scales_1.0.0        matrixStats_0.54.0
[43] BiocGenerics_0.28.0 MASS_7.3-51.4       assertthat_0.2.1
[46] colorspace_1.4-1    labeling_0.3        KernSmooth_2.23-15
[49] stringi_1.4.3       flowViz_1.46.1      lazyeval_0.2.2
[52] munsell_0.5.0       crayon_1.3.4

As for the negative ticks, it's just not adding them because there is no data in that range. But you could always add explicit ticks if that were desired. As a side note, it looks like I am really having trouble reproducing your plots in the recent versions of the packages because the bins are being re-evaluated appropriately during the re-scaling of the axis. The same plotting code above in the current trunk branch gives me this:

which is clearly problematic and I'll be opening an issue for that soon. I'll still loop back to figure out the discrepancy over the zero on the axis as time allows.

Hi. Thanks for looking at my question, but don't waste your time on that. I am currently updating all packages on order to get an up-to-date setup with R3.5.3. I will let you about the result. I guess I will have to switch to R3.6 and to Bioc 3.9. Best

Sounds good, and I appreciate it. I'm actually trying to run down this scaling issue that persists in Bioc 3.9, so your post has actually been really helpful in that regard. But I think I've found the problem and I'll have it fixed soon.


Zero lmkey test

Test around ZERO LM Keys
Resolves #13

Wow 3 consecutive failures in github actions ...

Broken ppa/archive index, the package references is older thanthe one actually available in the repo ...

And rerunnign jobs is broken in the first step, wow, congrats @github


Prepare for Erlang/OTP 22

Addresses: #1196
Comes with https://github.com/vernemq/plumtree/pull/24

Thanks for the PR much appreciated :) I merged vernemq/plumtree#24, can you fixup the reference there, and squash the commits for each of the three fixes (one commit per changelog entry)?

I just merged #1211, so I guess this should just be rebased and squashed.

Sure, I will rebase an squash when I reach my computer.
On Tue, Jun 11, 2019, 00:30 Lars Hesel Christensen<EMAIL_ADDRESS>wrote:

I just merged #1211 https://github.com/vernemq/vernemq/pull/1211, so I
guess this should just be rebased and squashed.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/vernemq/vernemq/pull/1213?email_source=notifications&email_token=AABJD6AXCTKGRORRPP3O7VLPZ4S5LA5CNFSM4HWPTOJKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXL447A#issuecomment-500682364,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AABJD6GGIF5NNKODR4BGGSTPZ4S5LANCNFSM4HWPTOJA
.

Squashed and re-based, ready to merge :crossed_fingers:


Is there an easy way to determine if a list if tight or loose

I'm working on building an AST from the token stream returned by the markdown-it parser. Everything has been very smooth, but I couldn't find an easy way to determine from the tokens whether a list is tight or loose.
(For comparison, the commonmark.js AST provides that information on the list nodes).
Did I miss it or is it not available? Is there anyway to identify that information from the result of parsing at all?
Thanks!

See tokens https://markdown-it.github.io/#md3={"source"%3A"- tight\n\nloose\n\n- foo\n\n- bar"%2C"defaults"%3A{"html"%3Afalse%2C"xhtmlOut"%3Afalse%2C"breaks"%3Afalse%2C"langPrefix"%3A"language-"%2C"linkify"%3Atrue%2C"typographer"%3Atrue%2C"_highlight"%3Atrue%2C"_strict"%3Afalse%2C"_view"%3A"debug"}}.
You can check in li if p has property hidden true or false.

See tokens https://markdown-it.github.io/#md3={"source"%3A"- tight\n\nloose\n\n- foo\n\n- bar"%2C"defaults"%3A{"html"%3Afalse%2C"xhtmlOut"%3Afalse%2C"breaks"%3Afalse%2C"langPrefix"%3A"language-"%2C"linkify"%3Atrue%2C"typographer"%3Atrue%2C"_highlight"%3Atrue%2C"_strict"%3Afalse%2C"_view"%3A"debug"}}.
You can check in li if p has property hidden true or false.

Thanks @puzrin.
I'm unclear how that relates to the question (although possibly the rendering in HTML is identical?).
For instance:
- > quote1
- > quote2

is a tight list according to the commonmark spec (and commonmark.js AST):
https://spec.commonmark.org/dingus/?text=- > quote1
- > quote2
but the p property is false in all markdown-it tokens:
https://markdown-it.github.io/#md3={"source"%3A"- > quote1\n- > quote2\n"%2C"defaults"%3A{"html"%3Afalse%2C"xhtmlOut"%3Afalse%2C"breaks"%3Afalse%2C"langPrefix"%3A"language-"%2C"linkify"%3Atrue%2C"typographer"%3Atrue%2C"_highlight"%3Atrue%2C"_strict"%3Afalse%2C"_view"%3A"debug"}}

Probably i misunderstand your question. AFAIK, the only difference between tight/loose lists is dropping <p> tag for one-liner content. I told you how to check with tokens if <p> should be dropped or not - via "hidden" property. May be now ideal, but there are no more differences in data. Direct attr on list parent is not available.
If you need something different - could you reword?

Probably i misunderstand your question. AFAIK, the only difference between tight/loose lists is dropping <p> tag for one-liner content. I told you how to check with tokens if <p> should be dropped or not - via "hidden" property. May be now ideal, but there are no more differences in data. Direct attr on list parent is not available.
If you need something different - could you reword?

Thanks for the follow up. Sorry if my question wasn't so well formulated. I'm still wrapping my head around this distinction between tight and loose lists in commonmark. But reading the spec:
https://spec.commonmark.org/0.29/#lists

A list is loose if any of its constituent list items are separated by blank lines, or if any of its constituent list items directly contain two block-level elements with a blank line between them. Otherwise a list is tight. (The difference in HTML output is that paragraphs in a loose list are wrapped in  tags, while paragraphs in a tight list are not.)

It seems to me that the distinction is not simply about one-liner content.
For the example I gave above:
- > quote1
- > quote2

The commonmark AST gives you a tight="true" attribute
<document xmlns="http://commonmark.org/xml/1.0">
  <list type="bullet" tight="true">
    <item>
      <block_quote>
        <paragraph>
          <text>quote1</text>
        </paragraph>
      </block_quote>
    </item>
    <item>
      <block_quote>
        <paragraph>
          <text>quote2</text>
        </paragraph>
      </block_quote>
    </item>
  </list>
</document>

While the following example:
- > quote1

- > quote2

Result in an AST with tight="false"
<document xmlns="http://commonmark.org/xml/1.0">
  <list type="bullet" tight="false">
    <item>
      <block_quote>
        <paragraph>
          <text>quote1</text>
        </paragraph>
      </block_quote>
    </item>
    <item>
      <block_quote>
        <paragraph>
          <text>quote2</text>
        </paragraph>
      </block_quote>
    </item>
  </list>
</document>

The application could then decide to render those differently (or distinguish when transforming to another format).
As far as I can tell, the token stream is identical in those two examples, so I was looking for a way to recover that tight/loose distinction in some way.

You can do more deep abalyzis, if you wish, but that's a bit out of this project scope. Difference in data was described in first answer - fake paragraph tokens.

You can do more deep abalyzis, if you wish, but that's a bit out of this project scope. Difference in data was described in first answer - fake paragraph tokens.

That won't work if the token stream is identical in both cases, but understanding if you consider this out of scope.
Summary for anyone else bumping into this:

markdown-it does not expose information about tight v loose lists in parse tokens
this distinction can sometimes be inferred from the hidden attribute in tokens within the list, but not always (see above examples)
this is out of scope for the project

Note, reference parser does NOT provides AST. Because columns info missed in tokens. It provides tree-like data. That's not enough for exact reverse transform.


Screenshot permissions set to 644.

Hello,
I'm using phantomjs 2.0.0 development to screenshot websites, but my concern is that the files have permissions set to 644 .I am running phantomjs from a limited user.My current work scenario requires that i have 664 permissions, so that i can delete the screenshots from another user. Is there a possibility that  phantomjs can save files with different permissions? Also, can this be implemented in future versions?
Thank you very much for your time in advance.

Thank you for submitting this ticket!
This issue tracker primarily serves as the tool to report and track bugs, defects, and development tasks.
For general question, code review request, or other related discussion, please post it to either:

the official PhantomJS users mailing-list
Stack Overflow (with phantomjs tag)

If you have not read it yet, please check also a few tips on Effective Q&A.


[Feature Request]: Connect to the HuggingFace Hub to achieve a multimodal capability

Is your feature request related to a problem? Please describe.
The HuggingFace Hub provides an elegant python client to allow users to control over 100,000+ huggingface models and run inference on these models to achieve a variety of multimodal tasks, like image-to-text, text-to-speech, etc. By connecting to this hub, a text-based LLM like gpt-3.5-tubor could also have the multimodal capability to handle images, video, audio, and documents, in a cost efficient way.
However, it still needs some additional coding work to allow an autogen agent to interact with a huggingface-hub client, such as wrapping the client method into a function, parsing different input/output types, and model deployment management. That's why I'm seeking if autogen could have an out-of-box solution for the connecting.
Other similar works: JARVIS, Transformers Agent
Describe the solution you'd like

The simplest and most straightforward way is to provide a huggingface-hub toolkit with inference functions. Users can then easily register this toolkit to an autogen agent according to their requirements. However, it's not clear to me where is the best place to place this toolkit.
The second approach is to provide a huggingface_agent, like Transformers Agent. This agent would essentially consist of a pairing between an assistant and a user-proxy agent, both are registered with the huggingface-hub toolkit. Users could seamlessly access this agent to leverage its multimodal capabilities, without the need for manual registration of toolkits for execution.
The third approach is to create a multimodal capability and register this capability to any given agent by hooking the process_last_received_message method. However, it may not be straightforward for some tasks such as text-to-image.

Additional context
I'd like to hear your suggestions and make contributions in different ways.

Re 1. We have some on going work in https://github.com/microsoft/autogen/pull/1929 and https://github.com/microsoft/autogen/pull/2414 for adding functions as a function store. This is similar to your idea of adding built in hugging face tools. It's best to discuss with @gagb @afourney and @LeoLjl about this direction and see if you can combine effort.
Re 2. Sounds interesting! I think we can start with a notebook example to show how this work. And then we can decide on whether to just do a notebook PR or a contrib agent.
Re 3. cc @WaelKarkoub @BeibinLi we do have text to image capability already.

@whiskyboy we implemented VisionCapability that adds the vision modality to any LLM: https://microsoft.github.io/autogen/docs/notebooks/agentchat_lmm_gpt-4v/#behavior-with-and-without-visioncapability-for-agents.
We also implemented an ImageGeneration capability that allows for any LLM to generate images: https://microsoft.github.io/autogen/docs/notebooks/agentchat_image_generation_capability
Other multimodality features are currently being worked on that you can track the progress of in this roadmap https://github.com/microsoft/autogen/issues/1975. Let me know if you have other ideas that we could add to the roadmap

@ekzhu
It's good to know there will be a function store in AutoGen soon! I will also try to provide a PoC of the #2 approach in these two days.
@WaelKarkoub
Thank you for sharing this awesome roadmap! I'm also thinking of adding some similar multimodal capabilities like TTS or document QA but with non-openai models (more specifically, with open-source models in huggingface hub). Although the current implementation of some capabilities accepts a customer process function, a built-in support of huggingface models is also attractive (to me at least). Additionaly, we can achieve more capabilities like image-to-image and audio separition by leveraging hf-hub.

@whiskyboy just for awareness, I have a PR that handles text-to-speech and speech-to-text https://github.com/microsoft/autogen/pull/2098. I'm experimenting with architecture but it mainly works.

@WaelKarkoub @ekzhu
Drafted a PR here: #2599


fixes #297

use color-all mixin to set the fill property in addition to the color property in inverted context

The thought was that if you are in a <div class="stripe--inverted" > it should fill the svg icon as well with the needed inverted color vs adding new css.


Lost pairing ?

Hello,
Today I disconnect the power from the ESP32 and I used the remote controls to control the shades. But now the ESP32 is not working. If I press the button from the shade nothing will move. I need to pair again or how I can synchronize the remote and ESPSomfy again ?

Did you change the address of the shade to the same as the address of any other remote?  You must not do that although there are several solutions online that do this.  Eventually one or the other will stop working and the shade will need to be reset and re-paired.

No I have this remote with 5 channel
I set the shade into the prog mode via remote and then I add shade give a name and press there the prog button
after this I press on link remote and press one button and it work Wirrheit problems few days
Then I remove the power frei esp half day and I used the remote to control the shades in this time. Then I plug the power back and nothing work. All three shades are not working but on the Transmitter setting I see the dBm change when I press button. I try also to link remote again but not help

Yeah it sound like you aren't trying to emulate an existing remote.  You did the correct thing.  The Pair Shade function is what links ESPSomfy RTS with the motor.  And it should not be removed from the motor unless the motor is factory reset or it is unpaired.

Ok I test today and now I have the same problem,
but I make a reboot of the ESP and then it work, maybe its some problem with the Transmitter module, I will check next time if I press the button on a SDR it will sent some signals or not.  The shades show the correct position on the webinterface but I dont check if I press open on the remote that ESPSomfy detect it when I have this problem

I have never seen an instance where the radio just stops responding.  When you press the Save button ESPSomfy RTS reinitializes the radio registers.  This leads me to think that the transceiver at some point lost power from the ESP32.
This is likely either the power supply, wiring, or the transceiver.  It sounds like the ESP never skipped a beat and as soon as the radio was re-initialized it came back on line.

You are right, I change the TRX Module and now I work, i think the old modul has some problem with the power

Great glad you got it working.  I am closing this for now open another issue if you have any more problems.


Danger local not using DANGER_GITHUB_HOST to actually make requests

The DANGER_GITHUB_HOST environment variable is not actually being used to make requests with local. As you can see below, the correct pull request URL is found, but when the request starts it actually uses api.github.com instead. Obviously that's not going to work.
$ export DANGER_GITHUB_HOST="myinternalgithub.com"
$ export DANGER_GITHUB_API_TOKEN=XXXXXXXX
$ bundle exec danger local --use-merged-pr=11
Running your Dangerfile against this PR - https://myinternalgithub.com/someorg/somerepo/pull/11
Turning on --verbose

bundler: failed to load command: danger (/Users/michele/.rvm/gems/ruby-2.3.0/bin/danger)
Octokit::Unauthorized: GET https://api.github.com/repos/someorg/somerepo/pulls/11: 401 - Bad credentials // See: https://developer.github.com/v3

This is a documentation issue. There are 2 env variables and they are not referenced on the Troubleshooting page that talks about local, just the FAQ. Made a PR: https://github.com/danger/danger.systems/pull/85


Migration to Manifest V3

https://developer.chrome.com/docs/extensions/migrating/checklist/
https://developer.chrome.com/docs/extensions/migrating/

Manifest V3 is supported generally in Chrome 88 or later. For extension features added in later Chrome versions, see the API reference documentation for support information. If your extension requires a specific API, you can specify a minimum chrome version in the manifest file.

Now arxiv2notion works on the latest chrome  114.0.5735.134（Official Build)


Use hash_to_curve to derive global parameters

Motivation
Use hash_to_curve to derive global parameters.
Pedersen CRH (& compressed variant) updates:

Removes usage of an RNG from the setup method in favor of hash_to_curve
Updates group trait to ProjectiveCurve

Bowe Hopwood Pedersen CRH updates:

Removes usage of an RNG from the setup method in favor of hash_to_curve
Reduces in-memory parameter size by 50%

Pedersen commitment (& compressed variant) updates:

Removes usage of an RNG from the setup method in favor of hash_to_curve
Updates group trait to ProjectiveCurve
Reduces serialized commitment parameter size by 50%

Schnorr signature updates:

Removes usage of an RNG from the setup method in favor of hash_to_curve
#258
Switches group trait from AffineCurve to ProjectiveCurve
Adds CryptoRng trait to methods where Rng is used

Group encryption updates:

Removes usage of an RNG from the setup method in favor of hash_to_curve
Removes SG signature group and SignatureScheme implementation/variant
Adds CryptoRng trait to methods where Rng is used

DPC changes:

Extracts account commitment, encryption, and signature parameters out of DPC logic internals
Accounts now utilize FromBytes(ToBytes()) to convert between encryption key and signature key

Parameter changes:

Removes account commitment, account encryption, account signature, record commitment, inner circuit ID CRH, local data commitment, local data CRH, and serial number nonce CRH parameters files and RNG-based generation scripts

Related PRs
Builds on #241

Still working through some final changes, but the primary changes regarding using hash_to_curve for CRH, Commitment, Encryption, and Signature are implemented with tests passing and ready for review.

I will start looking at the code. Note that I ran the tests locally, and the same as the CI, gadgets/integration-testnet12 are not passing.

Correct, the snarkvm-gadgets module has 1 Merkle tree test failing, and the .integration tests are failing due to loading SNARK proving and verifying key files that have not be regenerated on the new fields/groups.

I have ignored those single integration tests for testnet1 and 2 given the other integration tests will synthesize the circuits from scratch and verify transaction generation works. We can resample these parameters in another PR.

Got it. I am going through the files (40% done). Will finish soon.

Great, I'm debugging the Merkle tree gadget at the moment, will keep file changes minimal!

Same, can't seem to isolate the issue. The fact that the Pedersen compressed CRH Merkle Tree variant passes, yet the Pedersen CRH MT variant fails seems odd.
The compressed CRH calls the standard CRH as a subroutine and invokes to_affine() after. Given the failure is that the X Coordinate Conditional Equality/conditionally_equals fails, could it be that the comparison is comparing two different representations that technically are equivalent (e.g. unreduced form)?

I think I know why. Two ProjectiveCurve elements, even if they are the same, may not be "Eq".

Unconfirmed. But see this code:
            let previous_is_left = AllocatedBit::alloc(&mut cs.ns(|| format!("previous_is_left_{}", i)), || {
                Ok(&previous_hash == left_hash)
            })?
            .into();

If two ProjectiveCurve "gadgets" are not Eq even if they are the same, then it would also target the wrong root.

Wait nope, that is not right. They should be affine gadgets...let me think about it.

I just checked it does seem to be AffineGadgets.
Related, but unrelated, shouldn't this be a ProjectiveGadget for performance?

Nope. Inside curves, Affine is better.

But I would say previous_hash == left_hash does seem to be dangerous.

I definitely want to get to the bottom of this issue, however want to propose one way to remedy this issue.
Ray has PR #249 which we could rebase to with a more generalized and refreshed implementation of Merkle tree.

I agree. So merge this to testnet2 first and then see if #249 can fix it?

I think that works.
By the way, your suspicion is correct. If I add the following println!:
            println!("{}: {} {}", i, previous_hash == *left_hash, previous_hash == *right_hash);

The output looks as follows:
0: true false
1: false false
2: false false
3: false false

Actually let me have a try on this:
impl<P: ShortWeierstrassParameters, F: PrimeField, FG: FieldGadget<P::BaseField, F>> AllocGadget<SWProjective<P>, F>
    for AffineGadget<P, F, FG>

I have concerns that the implementation is not correct.

Ah, nope, in our cases I would be Edwards curve. Let me check a little bit.

I think we can narrow it down: either the parameters are not actually the same or there is an issue in allocating the tree path. The second is very likely because it explains why the compressed version passes.

Agreed, it should be from a lower level of abstraction than Merkle Tree gadget given other variants of this test are passing.
In the meantime, I'll follow the proposed course of merging this into testnet2 and migrating to #249 and testing the version there.

Ok. I think this might be the explanation:
In the native version, when computing MIDDLE nodes in the Merkle tree, we to_bytes projective curve elements. In twisted Edwards, this means that we to_bytes [x, y, t, z].
In the constraint version, it to_bytes affine curve elements.
We need to change the native version.

That sounds like the culprit! It would also explain why the compressed gadget variant passes. Given we reduce to affine and use the x-coordinate on it's own, this issue wouldn't arise then.

The fix would be easy: just change the output type if Bowe-Hopwood (uncompressed) CRH and Pedersen (uncompressed) CRH from G to G::Affine.

Should I make and push a fix to testnet2, or you will fix it?

That sounds like a good idea, feel free to make the fix and push to testnet2!


Fixes typos and phrasing in README.md

Hey @slmingol - Thanks a lot for this PR. I believe that your fix on "Kubernetes" may need to be changed though because it's really "Kubernetes" [1] and not "Kubernete's"
If you can fix that I will merge it.
Thanks again!
[1] https://kubernetes.io/


new feathure Add GZIP support for large message

[x] I have searched the issues of this repository and believe that this is not a duplicate.
[x] I have checked the FAQ of this repository and believe that this is not a duplicate.

Environment

Dubbo version: 2.7.7
Operating System version: windows10
Java version: 14

We sometimes has very large data to retrieve from dubbo service, in order to reduce the transportation cost, please add a feathure to compress the message.
suggested:

dubbo.protocol.gzip.enabled=true, to enable the compress function.
dubbo.protocol.gzip.payload=1M, messages larger than 1M bytes will be compressed.

Reference:
https://github.com/lshm/dubbo-gzip/blob/master/dubbo-remoting/dubbo-remoting-api/src/main/java/com/alibaba/dubbo/remoting/exchange/codec/ExchangeCodec.java

It would be great if someone can help to add this support.
One thing to keep in mind, make sure it's optional and extensible.

FYI, it's not recommended to transport large data in one TCP connection.

gRPC has compress support and there is a example for their API: https://github.com/AmiDavidW/grpc-java/blob/a111ab75dd246548b9ea6fef58466a05d1aba5b0/examples/src/main/java/io/grpc/examples/experimental/CompressingHelloWorldServerAllMethods.java#L52

It would be great if someone can help to add this support.
One thing to keep in mind, make sure it's optional and extensible.

I'd like to help, but, how can I help?

I have tested merge the code from those sample to 2.7.7, and it works.  BTW, the original code has some bug, need to fix.


Documentation dispute: C420 really same as C420mpeg2?

The description for Colorspace::C420mpeg2 now reads:
https://github.com/image-rs/y4m/blob/58375d69120a33e2a21320e17449e84e4de9949d/src/lib.rs#L248
However, sources that I can find disagree with this conclusion:

https://chromium.googlesource.com/webm/libvpx/+/refs/heads/main/y4minput.c indicates that there is a half-pixel (relative to Y size) or quarter-pixel (relative to C size) displacement
https://github.com/xiph/rav1e/blob/bc516fcd18bcc2de7f009c2837469c901f7c605f/src/bin/decoder/y4m.rs#L104 indicates that it's, well, different

I recommend changing the text to:
    /// 4:2:0 with alternated vertically-displaced chroma planes, 8-bit.
    C420paldv,
    /// 4:2:0 with consistently vertically-displaced chroma planes, 8-bit.
    C420mpeg2,

or something more specific than that, if people want to ask about "hey how alternated is that".

I concur. Can you submit a pull request with these changes? Thanks!


Carthage error: Module compiled with Swift 3.0 cannot be imported in Swift 3.0.1

The binary should be built with the newest version of Swift: 3.0.1

@Anders123fr
You can use with carthage update --no-use-binaries
See @rad3ks's comment: https://github.com/kishikawakatsumi/KeychainAccess/pull/293#issuecomment-260373336

Yes I know, but I still think that it should comply with the latest Swift version, 3.0.1 :)

Any news on this one?

I guess someone will do the release of newest version compiled with Swift 3.0.1, for now use --no-use-binaries

As others I am looking for version compiled with Swift 3.0.1. Thanks

I add myself to the list, and will even mention Swift 3.0.2 👍

It would be great to get this resolved, this is the only Carthage package I'm using that needs --no-use-binaries but using that flag means my updates take a long time as now all packages have to be built instead of downloaded.

I was just recently advised to use my own fork and upload my own binary for the release version I'm interested in. This way I'm able to point to KeychainAccess fork in Cartfile and not use --no-use-binaries. It's a great strategy to boost the carthage bootstrap speed.
To achieve that you should:

Fork this repo
Follow the instructions
Go to releases in your fork
Edit & upload the KeychainAccess.framework.zip
Use your fork in Cartfile

Unfortunately it's required to repeat those steps for each new release.

+1

Please update Carthage to 0.20. Carthage uses pre-built binary only when they were built with the same version of the Swift compiler from 0.20. No longer need to --no-use-binaries :)
https://github.com/Carthage/Carthage/releases/tag/0.20.0

Cool 👍


[AMQ-9595] Fix recoverNextMessages when there are messages consumed farther than maxBatchSize.

What problems does this PR solve?
Fixes a bug when a durable consumer can received messages which it's already acknowledged
Why is it beneficial to merge into ActiveMQ?
It fixes a bug
How do you make sure this PR is well tested?
I couldn't find any tests that test that part of code directly. I wrote a higher level test that covers that edge case.

Do you have a unit test the reproduces the issue?

Do you have a unit test the reproduces the issue?

the test I added reproduces the issue. without the fix the test will fail


Deleting saved rolls

We can save dice rolls with /r $<nickname> = <dice roll>. I don't see a way to delete them and free up saved roll space.
I tried issuing "/r $<nickname> =" in hopes that it would "unset" the roll using that, to no luck.
Can you please either document the method in the readme, or add one (and document it)?
Thanks!

So this feature was only added last week, roll removal isn't implemented yet, I will fix it soon. Thanks for the heads up!

You can now list the memorized rolls with /r $ and remove them with /r $name =.

The listing stops after the first 25 rolls.
On 07/09/2017 01:02 PM, Artem wrote:

You can now list the memorized rolls with |/r $| and remove them with
|/r $name =|.

--
Andrew W. Ragland
<EMAIL_ADDRESS>

It must be a Discord limit.
Please try again, I have updated the bot to send multiple embeds.

Fixed, thanks.
On 07/09/2017 01:21 PM, Artem wrote:

It must be a Discord limit.
Please try again, I have updated the bot to send multiple embeds.

--
Andrew W. Ragland
<EMAIL_ADDRESS>

You're welcome. Thanks for testing it on a short notice.


Installation issue

Hello,
I'm not sure if I have installed correctly node-bluetooth-hci-socket as some warnings pop up. I've visualized the following when I installed it:

Could anyone help me out?
Thanks

Hi @mz92341,
Looks ok to me. xpc-connection is an optional dependency and needed on OS X only. The package.json error looks like it's because you don't have a package.json in your home folder.


use veryShortWeekdaySymbols

use weekdaySymbols.map { String($0.first!) } will return a incorrect Symbol in other language, e.g. zh

Wow. Did not know that such an instance method existed. Thanks for the contribution!


format error

After deleting this template, please provide the following information:

Forge Version: 31.2.31; 1.15.2
JEI Enchantment Info version: 1.1.0

only protection disappears this "format error", that is so weird.

This was the result of percent symbols not being properly escaped.


Mario Jump

Make a mario character jump/run/ move on key press and overcome obstacles

Can you please assign it to me?

@Shreyaar12 I want to work on this issue under GSSoC'23

Can you please assign it to me?

@Shreyaar12


Fix vertical alignment of upload icon in the FilePicker (on mobile)

@freenow/wave version: 1.29.0

Relevant code
Inside FilePicker.tsx
<Outliner
                minHeight="5.125rem"
                alignItems="stretch"
                data-testid="filepicker-outliner"
                display="flex"
                disabled={disabled}
                hasValidFile={validFileSelected}
                error={error}
                justifyContent="space-between"
                onClick={onClickHandler}
                px={2}
                py={validFileSelected ? '1.25rem' : 3}
            >

What was expected to happen?
The current setting of alignItems="stretch" in the <Outliner> is causing the upload icon on mobile screens to not center vertically correctly.

Solution
Change the value for alignItems to center

The issue is with icon positioning in flexbox on mobile only


Add possibility to disable time axis ticks

Why there is no drawTicks option in TimeScaleOptions?
I think there should be possibility to disable time aixs ticks as it's possible right now on a price axis.

OK, give me a couple of hours


⚠ Bofrost.be has degraded performance

In 31489a8, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 2065 ms

Resolved: Bofrost.be performance has improved in 9575f88 after 3 hours, 44 minutes.


Enable WriteCore feature for alertsmanagement

We need to enable WriteCore feature to advisor. Therefore add use-write-core: true and do a regen.

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.ResourceManager.AlertsManagement


Unable to display two ipyleaflet maps in separate interactive windows

Environment data

VS Code version: 1.51.0
Extension version : v2020.10.332292344
OS and version: Windows 10 Pro 2004
Python version : 3.8.6
Type of virtual environment used : N/A
Relevant/affected Python packages and their versions: ipyleaflet==0.13.3
Value of the python.languageServer setting: Pylance

Expected behaviour
Show two ipyleaflet maps in separate interactive windows like in jupyter notebooks.

Actual behaviour
The first interactive window can show a map correctly, while the second interactive window failed.

Steps to reproduce:
Open an interactive window and run the below code, a map will show correctly:
from ipyleaflet import Map, basemaps, basemap_to_tiles

m = Map(
    basemap=basemap_to_tiles(basemaps.NASAGIBS.ModisTerraTrueColorCR, "2017-04-08"),
    center=(52.204793, 360.121558),
    zoom=4
)
m

Open another interactive window and run the code again, the map would not show.

@xiejx5 Thanks for the detailed report and sorry about the issue here. I tried this out locally and I confirm that I see the same issue:

We'll discuss this at our triage meeting, but no specific ETA yet.

@IanMatthewHuff  did you get any errors in the webview console? That's where I'd suspect we'd show any problems.


add conan and cmake ways to build libconfuse

I suggest to add new modern way to build libconfuse. It was tested on Ubuntu 22 with conan 2.0, works ok. Thanks for help.
Build & Install
using conan (better)
mkdir my_genimage && cd my_genimage
//to uninstall conan use: pip uninstall conan
//install conan
pip install conan --pre --user
pip install conan --user --upgrade
//download test example
git clone https://github.com/libconfuse/libconfuse
//put to original libconfuse/ folder files CMakeLists.txt and config.h.in (replace) and conanfile.txt
cd libconfuse/
rm -Rf build/
conan profile detect --force
conan install . --output-folder=build --build=missing
cd build
cmake .. -DCMAKE_TOOLCHAIN_FILE=conan_toolchain.cmake -DCMAKE_BUILD_TYPE=Release
cmake --build .
sudo checkinstall --pkgname="libconfuse" --nodoc --pkgversion="1" --default
//to remove use: dpkg -r build-libconfuse
using cmake only
mkdir my_genimage && cd my_genimage
mkdir build-libconfuse
git clone https://github.com/libconfuse/libconfuse
//put to original libconfuse/ folder files CMakeLists.txt and config.h.in (replace)
cd libconfuse/
cmake -S libconfuse/ -B build_libconfuse/ -DCMAKE_BUILD_TYPE=Release
cmake --build build_libconfuse/ --parallel
cd build_libconfuse/
sudo checkinstall
//to remove use: dpkg -r build-libconfuse
P.S. If you still haven't conan install and update it:
pip install conan --pre --user
pip install conan --user --upgrade

Co-Authored-By: myd7349<EMAIL_ADDRESS>

As one of the maintainers, who also do not like or want to maintain cmake (or conan for that matter), I do not approve this PR.
Other maintainers may feel differently, and may be willing to take care of it.  If so, please speak up.

Diar @troglobit @cgzones I still think libconfuse needs CMake... Do you change your opinion?

Diar @troglobit @cgzones I still think libconfuse needs CMake... Do you change your opinion?

No, I have not.

I would like this to be merged too :(

No response from other maintainers in over a year. So I'm pulling rank and closing this PR.


Doesn't work in Idea-2017 EAP (idea-IU-171.2014.21)

The sbt console does not work for me on the latest 2017 Idea EAP version (under fedora 25).

Problem seems to be solved with more recent updates.


How to get product description and link to detail page

Hi there!
How do I get the description of a product and the link to the its detail page?
Best,
Carl

Hi, i search too the description of the product, can't find anything. Any help ?
Thanks !

Amazon doesn't provide the description in their APIs - the reasoning that I read up on is Amazon doesn't want duplicate content to affect their SEO efforts.


Run Workflow System Task not working for Users

Describe the bug
When creating a user workflow and using the Run Workflow system task, no workflows show up in the drop down of workflows to choose from.
This may be a UI bug as it could be building the drop up based on data already held of the workflows loaded.
What project does this issue relate to?
Boomerang Flow
To Reproduce

Create two User workflow: one for running a sleep or simple function. The second workflow to use the system task for Run Workflow.
On editing the Run Workflow system task, the workflows drop down will never get filled in.

Environment
Flow 3.5.0
Additional Contenxt
No response

For team workflows, the dropdown is just the workflows associated with particular team returned from GET workflow/teams. However, for user workflows, the workflows are not associated with a particular team but with a user. Therefore, the UI will need to populate the dropdown from GET workflow/workflows/user and exclude the workflow that is currently being edited.

Thanks for the analysis.

Do we need to also check system and template workflows?
Do we need to exclude the current workflow?

I've already checked the system and template workflows. They are already calling GET /workflow/workflows/system and GET /workflow/workflows/template, respectively.
From what I have observed from the UI behavior, the UI is excluding the workflow that is currently being edited from the Run Workflow dropdown, perhaps to keep the workflow from executing itself.

Thanks @amhudson. In discussing (2), i think we remove the limitation for excluding the current workflow. The user can decide, and I am not even sure what benefit excluding it would provide, in that a user could always use two workflows to go in a loop back and forth, so there are ways to do it anyway.

Opened a PR for this and pushed tag 3.5.8 to dev and qa to test things
https://github.com/boomerang-io/flow.client.web/pull/111


Unbind event handlers after not needed anymore.

Unbind event handlers once there's no more elements left. This purges obsolete event handlers from piling up on page where lazyload() is called repeatedly.

this seems quite useful :+1:


Depreciation in Ember 2.1

In 2.1 a deprecation was added:

DEPRECATION: The initialize method for Application initializer 'metrics' should take only one argument - App, an instance of an Application. [deprecation id: ember-application.app-initializer-initialize-arguments] See http://emberjs.com/deprecations/v2.x/#toc_initializer-arity for more details.

If I get a chance later, I'll try to create a PR to fix this. But if someone else has time to do it sooner, that'd be great.

Thanks for reporting! It's a simple fix, I have done it for my other addons and overlooked this one.


Mapping/Model + Row converters for DplaPlace.exactMatch and DcmiCollection.isShownAt

Add DcmiCollection.isShownAt to DplaMapModel, row and model converter
Add DplaPlace.exactMatch to model and row converters
Add tests for Model and Row converters for DplaPlace and DcmiCollection
Add mapping to NARA for DcmiCollection.isShownAt
Fix up base URI for NARA URIs (catalog.archives.org -> catalog.archives.gov)

Left a couple of comments that aren't that important!


Update plug-in

Updated some dependent plug-ins

@microsoft-github-policy-service agree [company="{your company}"]

@microsoft-github-policy-service agree


Describe better limitations and difference with other solutions

Done: https://github.com/artalar/stylerun/commit/082400bace4355b66031af158601b94cb181c1b7#diff-b335630551682c19a781afebcf4d07bf978fb1f8ac04c6bf87428ed5106870f5R47


Add UML diagram about AWS Getting started using Elastic Beanstalk

Relates to #12

https://docs.aws.amazon.com/elasticbeanstalk/latest/dg/GettingStarted.html

[ ] Step 1: Create an example application
[ ] Step 2: Explore your environment
[ ] Step 3: Deploy a new version of your application
[ ] Step 4: Configure your environment
[ ] Step 5: Clean up


ETCD-612 Added callback to provide preconditions for revision controller

/lgtm

/test unit

@p0lyn0mial I have made all the changes as suggested.
Verified that the revision rollout is blocked against by removing etcd static pod manifest, which brings the cluster to quorum-unsafe state. Given the current way of evaluating the revision precondition, the RevisionController doesn't enter a degraded when the precondition check returns error(the case mentioned in the first sentence). I wanted to confirm if the RevisionController not degrading in this case is okay.

I wanted to confirm if the RevisionController not degrading in this case is okay.

It’s fine for me, but we can change it if you think it would be valuable. This can be done in a new PR. I believe the etcd operator will degrade anyway when the quorum cannot be restored. Additionally, any errors returned by the precondition function for the revision controller will be logged.
/lgtm
holding off in case you want to add the change that will make the controller to go degraded.
/hold

With the etcd operator degrading, it is acceptable for the revision controller to not degrade.
Thank you @p0lyn0mial for the review and feedbacks.
/hold cancel


chore: add dockerimage-replication module to integ check workflow exempt list

Issue

Closes #222

Changes

add dockerimage-replication module to integ check workflow exempt list

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

AWS CodeBuild CI Report

CodeBuild project: IDFModuleTestsInfrastructur-59JoFoVH64CI
Commit ID: 6dc1b995d7c693942c152415c4f415609fa22350
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


Automobilista crash to desktop with D9VK

With D9VK, The game goes to desktop after a some time with a lot of Stutter. You can see a video here:
https://youtu.be/k_122qOl2UY
Without D9VK the game don't crash to desktop .
Please describe your issue as accurately as possible. If you run into a problem with a binary release, make sure to test with latest master as well.
Software information
Autombilista (Steam ID 431600)
System information

GPU: Nvidia GeForce GTX-1060-6GB
Driver: 430
Wine version: Proton 4.11-7
D9VK version: 0.22

Please test with master builds are here: https://git.froggi.es/joshua/d9vk/-/jobs
You also ignored and deleted the most important part of the issue template. If I didn't want that information I wouldn't put it on there.

Seems that with the last master (4ad9c589), the game works great and don't crash to desktop. I only could test about 2 minutes now, but this night I will play more time to confirm it.

After playing more time I can confirm that with the version 4ad9c58, the game don't crash. I tried the new 0.30 "official" version and with it the game continues crashing.

Well nothing after that can cause a crash ever.

Unless it was the rebase, what happens with the build for the ZERO fix on git.froggi.es jobs

Could this be another case of the game running out of virtual address space, in which case the first thing I suggest is to patch the game to be Large Address Aware.

Seems that now it don't crash with Proton 4.11-9. This happens now that I've managed to make a trace.... I upload it to Mega:
https://mega.nz/#!ABwQjA4K!2osCv_WhXaALoTJvDGdx-qzClx9e783RCtH0C73DGKY
I see that there are stuttering and FPS are low (without D9VK I think is more higher)

It crash to desktop again. I found this problem when you crash your car in the pits. I found another problem. The colors, especially in menus, is very contrasted and strong.

Hey, I can't repro this on the trace:
https://store.steampowered.com/app/431600/Automobilista/ is the game right?

Yes, is this, but some days ago I was playing a lot of time with the last version without problems. Take a look to the video, it works great:
https://youtu.be/Xs3Pzn5ee5k


Strip down talon and make it Python 3 compatible

@timabbott, I have stripped down talon and added it to server.lib.

Automated message from Dropbox CLA bot
@umairwaheed, it looks like you've already signed the Dropbox CLA. Thanks!

Also, it is not Python3 compatible.

Cool, glad this works!  I'd rather not end up maintaining our own special fork of talon in-project though; how much did you have to modify talon to split this out?  https://github.com/mailgun/talon/issues/96 suggests that there's interest in talon upstream in it being possible to install talon without the ML stuff; do you think it'd be reasonably easy to add either a --no-ml installation option that we could specify in requirements.txt to talon that installed approximately the files you have here without needing the extra dependencies?
If not, I would still probably prefer to be maintaining a fork of the talon repo with a few commits making these modifications and including that in requirements.txt, since then we have a way to rebase our changes to take upstream updates.

(FYI I merged reverting the original talon commit as f1ba3ded42726fc5007546508149c8834dd77465, since the install performance issue was pretty bad and I think we're not going to keep that precise solution).

There was not much I did, first, I ran 2to3 script to make the code Python 3 compatible; second, I made some more modifications on top of 2to3 script for Python 3 compatibility; third, I added mypy annotations.

do you think it'd be reasonably easy to add either a --no-ml installation option that we could specify in requirements.txt to talon that installed approximately the files you have here without needing the extra dependencies?

Sure.

@timabbott, I have created the initial PR, https://github.com/mailgun/talon/pull/97. I still need to figure out how to exclude packages using setuptools.

@timabbott, my fork now installs stripped down version of Talon but it still is not Python 3 compatible. Mostly, the problem is unicode.

Cool.  How hard do you think fixing the Python 3 unicode compatibility issues will be?  Since @sharmaeklavya2 is very close to having Zulip fully support Python 3, we're going to need to get it working with Python 3 soon regardless...

@timabbott, well, they have good test coverage so I am hoping that it won't be a huge problem.

OK.  I think we should probably just do that work as part of merging this, then, since I'm not sure how to easily merge this without breaking the python 3 tests otherwise....

Yep, I am working on it. Hopefully, it would be done today.

OK, the Talon part is done. I am just facing a weird problem while installing the package via requirements file. It seems that the install-option given to Talon pollutes the global environment of pip and this argument also given to another package listed down the order.

@timabbott, done. Please review.

Merged, thanks @umairwaheed!  Can you report a pip bug for the install-option issue?  Seems like that's something that could get annoying for us in the future, and thus is definitely worth making sure the pip developers are aware of it.

Thanks.

Can you report a pip bug for the install-option issue?

Reported @ https://github.com/pypa/pip/issues/3845.

@timabbott, can I close this PR?

Yep!
-Tim Abbott (mobile)

Merged already.


CORS policy on custom domains wasn't set

Fixed on the server, make sure it's working now.

Fixed


Add phpdbg package

The phpdbg package should be added for debugging and code coverage reporting.

phpdbg should be available with the php 7.0 and 7.1 packages, and now the php 5.6

Thanks, what is the package name? I cannot see anything in the pkgsrc. I tried phpdbg but only got an error.

It looks like the phpdbg interactive debugger is distributed with php 5.6 and newer. It should be part of the base php package. Yesterday I recompiled the php 5.6 package and added the phpdbg in the build.

Indeed, it's there, thanks!


Add override keyword to MockAsyncResponseReader::Destroy when available

Future versions of gRPC will include Destroy() in grpc::ClientAsyncResponseReaderInterface<Response>, when that happens we need to add the override keyword to any Destroy() member functions in our mocks.  We may need to do this with some #ifdef based on gRPC version.

For context, this is waiting on the approval and implementation of https://github.com/grpc/proposal/pull/238.

@coryan When is it safe to do this? Do you use gRPC from HEAD in the open source release?

@coryan When is it safe to do this?

gRPC releases every 6 weeks or so, the robots send us PRs to update our builds the first Monday after each gRPC release.  So after your changes make it to gRPC, wait anywhere form 0 to 6 weeks, then the next Monday. However ...

Do you use gRPC from HEAD in the open source release?

No, we do not hate ourselves that much. Our builds are (mostly) pinned to the latest gRPC release.  We have builds that are pinned to the oldest release we claim to support (1.33.1).  We have some builds that are pinned to whatever release is included with some package managers.
I am not sure if that answers your question about "when is it safe"...

You mentioned off-thread the possibility of conditionally adding the override keyword based on version; is there some variable that can be ifdef'd? (I'm not sure it's worth it, but if you think it's important.)

Well, this makes me sad: I expected gRPC to follow, oh, I don't know, 30 years of software development conventions and include a macro declaring its version... I cannot find any such macro.

(I'm not sure it's worth it, but if you think it's important.)

I am 99.4% certain clang-tidy will generate a warning if you redefine a function as virtual and already is in the base class:
https://clang.llvm.org/extra/clang-tidy/checks/modernize-use-override.html
When the time comes, we can probably silence clang-tidy with // NOLINT(modernize-use-override) (or // NOLINTNEXTLINE(modernize-use-override) if it does not fit.

Oh, and I forgot to mention, at least one of our builds compiles with warnings-as-errors and with clang-tidy turned on.

Wow, that's extremely tidy. Sounds good.

Discussed this. Still want it, but we're waiting until our min gRPC version is >= ???

Strike 2.  Let's see if we have increased the minimum gRPC version to > 1.38 the next time (we are currently at 1.35).

We no longer use grpc::ClientAsyncResponseReaderInterface<> in our mocks, this is now obsolete.


[Bug]: Streaming from edge function works, but chunks arrive together at once at the end

Summary

Edge function in pages/api/*.ts works technically, but is not really streaming
Next.js application without any Netlify-specific configuration
On Vercel, things work as expected, e.g. headers are set properly, etc.
On Netlify, it works, but there is a bug or a missing piece to make streaming work end-to-end

This issue template is killing me. Just wanted to ask whether this is a known limitation/issue? Trying to fill out the fields as requested. Happy to provide more details!
I'm pointing to the reproduction repository I'm working with, but it's not really a repro since you would need API keys etc. so it's just to get an idea of what I'm working on.
(By the way, can't debug this locally properly, since netlify dev eventually results in self is not defined coming from the streaming edge function (this does not happen using next dev; it also happens using netlify dev --live); not sure how to go about this. I expected Next.js didn't use Webpack anymore, but it does and somehow netlify is different from next here.)
A link to a reproduction repository
https://github.com/kadena-community/7-docs-demo/blob/main/src/pages/api/completion.ts
Expected Result
See a streaming response with the first chunk coming in early on and then chunks arriving over time subsequently
Actual Result
Technically a streaming response, but the first and last chunk are one millisecond apart
Steps to reproduce
To see the issue right away:

Visit https://thunderous-medovik-731432.netlify.app
Select "Need suggestions?" and pick one
See that the response takes some time and is printed at once
See in the network tab that the response was indeed streaming, but all chunks arrive at the same time (see screenshots)

To get something going locally:

Create a Next.js application
Add a API function like pages/api/my-function.js
Call OpenAI's /chat/completions function and set stream: true
See that the response takes some time and is printed at once
See in the network tab that the response was indeed streaming, but all chunks arrive at the same time (see screenshots)

Next Runtime version
4.36.1
Is your issue related to the app directory?

[ ] Yes, I am using the app directory

More information about your build

[ ] I am building using the CLI
[ ] I am building using file-based configuration (netlify.toml)

What OS are you using?
None
Your netlify.toml file

`netlify.toml`
# Paste content of your `netlify.toml` file here

Your public/_redirects file

`_redirects`
# Paste content of your `_redirects` file here

Your next.config.js file

`next.config.js`
# Paste content of your `next.config.js` file here. Check there is no private info in there.

Builds logs (or link to your logs)

Build logs
# Paste logs here

Function logs

Function logs
# Paste logs here

.next JSON files

generated .next JSON files
# Paste file contents here. Please check there isn't any private info in them
# You can either build locally, or download the deploy from Netlify by clicking the arrow next to the deploy time.

Hi @webpro
The team at Netlify have now released the new v5 Next.js runtime which has support for both pages and app router, on-demand and time-based revalidation, automatic fine-grained cache control, and automatic image optimization using Netlify's image CDN.
You can find the documentation and additional information on the new runtime here: Next.js on Netlify
The v4 runtime is now in the maintenance support phase with no new features being added. Occasional bug fixes and security patches will be applied when needed.
Thanks!


[Question]: Why does crate.io have newer versions than in this repository?

Sorry that I'm very new to Rust. Are newer versions on crate.io just new builds based on the same source code of lazy-static?

It doesn't: https://github.com/rust-lang-nursery/lazy-static.rs/releases
1.1.1 is maintenance/bugfix release for an older version. See https://github.com/rust-lang-nursery/lazy-static.rs/pull/162

Thank you for the clarification!


Use super() for HttpSession init call

This seems to be the only place in the code where you're not calling super() to init the superclass. With this change, HttpSession can participate in diamond class patterns which I have a specific use case for (integrating with requests_oauthlib).

Thanks!


expose environment variables to ssh session

as one start a ssh container, passing --env for various stuff, assumes the ssh connection will also see those variables.
But sshd sessions do intentionally hide the parent environment, so we end with minimal set of variables from /etc/profile.
This workaround is inspired by comment on https://github.com/docker/docker/issues/2569#issuecomment-27973910

@reviewbybees @ydubreuil @carlossg

This pull request originates from a CloudBees employee. At CloudBees, we require that all pull requests be reviewed by other CloudBees employees before we seek to have the change accepted. If you want to learn more about our process please see this explanation.

It is a 🐝 for me !

:bee: too!

@reviewbybees done


APPDUX-167: First phase of configuration management

Motivation
JIRA:
https://issues.redhat.com/browse/APPDUX-167
Design spec:
https://marvelapp.com/11id0249/screen/69959965
What
Add multi-tab interface for Solution Explorer settings page. Revise existing solution patterns tab and create new Managed Integration schedule-related tab as described by JIRA story and design spec. The new tab:

Fetches the daily backup and maintenance window data from the rhmi config, and formats as to spec in local time and UTC time
Allows you to write a new start time for daily backups
Prevents daily backups from being scheduled during first hour of next maintenance window (by disabling it in the dropdown)

Why
Allow users to make backend OpenShift changes directly via Solution Explorer vs. editing the config YAML in OpenShift.
Verification Steps

Log in with admin account.
Go to the Settings page.
Click the Managed Integration schedule tab.
Select a start time for backups and click Save.
Verify that if you go to settings again, the Next daily backup date/time has changed. Bonus points, go to the RHMIConfigs YAML on OpenShift and verify that the values have changed accordingly.
Verify that the Start time dropdown displays the current Next daily backup.
Verify that the weekly maintenance window date/time reflects what is in the YAML file.
Verify that the backup dropdown has disabled the time in which the weekly maintenance window occurs so that it is not selectable.
Click the Solution Pattern content tab.
Verify that the syntax checking and ability to add new github repos still works.
Verify that the new 'Learn more...' link at the bottom navigates to the expected URL. Note that currently it loads the correct page but it looks like Doc has not created the expected bookmark. Not a bug with this code.

Checklist:

[x] Code has been tested locally by PR requester
[ ] Changes have been successfully verified by another team member

Progress

[ ] Finished task

Additional Notes
Functionality is complete, and most CSS and other related changes are complete. Sending this as a draft pull request because of known CSS issues that are being fixed... wanted to get the core functionality/code into review.
Known CSS issues:

Selectable area on tab card should not be selectable
Daily backups dropdown is too wide
Need to remove extra gray space at top where 'last saved' info will eventually be
'Permissions needed' page should expand to new tab size

Testing:
For now, you can test on my OpenShift cluster (available until 6/27):
https://solution-explorer.apps.cluster-uxddev-730d.uxddev-730d.example.opentlc.com/
This build disables security so you can log in as a non-admin, for ease of testing. Can also load a version with security if desired.
Also, you can load either of the following docker images onto your own OpenShift cluster, the first disables security so you don't need to be logged in as Admin, the second has security enabled and is how it will eventually be released:
docker.io/mfrances17/dev-tutorial-web-app:config
docker.io/mfrances17/dev-tutorial-web-app:configWithSecurity
Screen caps
Backup tab:

Solution Patterns tab:

Conflicting maintenance window time disabled:

RHMIConfig YAML write succeded:

Looking good in Chrome -- Firefox seems to be having some trouble with the backups dropdown:

I had the same experience as Jenny with Firefox.
Found two issues:

The behavior of the Save button on Solution Pattern content tab. To recreate:

Type a syntax-valid repository URL in the list (I had actually typed two but this may be recreated with just one.) - Save is enabled.
Click the Home link to exit to Solution Explorer
Navigate back to Solution Pattern content tab
The repository link(s) are removed but the Save button remains enabled - it should be disabled
I did this initially in Firefox and then attempted to test in Chrome but could not because the same state existed - the Save button was enabled with no URLs listed in the field.
In both browsers, I tried a scenario in which I then entered a syntax-valid URL and then clicked Save. When I returned to the Solution Pattern content tab, the URL was saved as expected, but Save button was still enabled.

On Firefox and Chrome, a tooltip labelled "Solution Pattern content" appears when the cursor rests on any location in the white space on the tab.

@phcox The Save button on the solution patterns tab has always been enabled by default. This was by design so you can have an empty field to clear existing added repos. I'll look into the other issues.

@jenny-s51 thanks, i'll investigate the firefox dropdown issue

@phcox @jenny-s51 i've fixed the tab title display on hover issue and the dropdown date issue that was occurring in firefox. i've updated my server with the latest code.

@mfrances17 Working as expected now in Firefox -- this is looking good!
This change you made:
 .pf-c-dropdown__menu {     min-width: var(--pf-c-dropdown__toggle--MinWidth); }
for some reason doesn't completely align the width of the dropdown items with the dropdown toggle width, but maybe that's just a minor nit so we can merge this. Also, it would definitely be nice to make the dropdown items list scrollable at some point.

Great work!


Skeleton Wand Feathers One Sided - Invisible from the back

When viewing the skeleton wand from the back (changing your view moving your mouse) the feathers on the wand disappear when viewed from the back.
To Reproduce (if applicable)
Steps to reproduce the bug. For example:

Wield Skeleton Wand.
Spin view with mouse.
See disappearing act!

Expected behavior (if applicable)
Expect all elements from the wand to be visible from all angles.

Thanks for the report. As explained in discord (for the benefit of anyone curious or reading this just now): This is because the wand uses the vanilla valheim material GoblinStuff_mat, which uses the Custom/Creature shader which is not double sided. This causes the feathers to not render when not viewed head on.
To fix, I need to change it to use a different shader. Possibly the Standard Two Sided shader which will render both sides, but this might affect the look of the weapon.

Fixed in next release (whenever that comes)


Don't push/import a commit to a repo if no changes occurred

srpmproc will push new commits, even if empty, to a repository. this is sometimes annoying, because it doesn't make it clear to the user if it was already imported at some point in the past. There should be a way for it to see that the repo it is importing to (assuming it is exists) has the same content with no changes occurring with its import (post-patch, if applicable), notify the user, and exit cleanly.

This is my take at it https://github.com/rocky-linux/srpmproc/compare/main...lecler-i:srpmproc:tweak/13-no-change-no-publish?expand=1
It just works, I didn't try any edge-cases tho, just to launch an repo sync multiple time in a row with no changes to my upstream


⚠ Mobitopup has degraded performance

In 822b88a, Mobitopup (https://napcard.voz.tech) experienced degraded performance:

HTTP code: 200
Response time: 6120 ms

Resolved: Mobitopup performance has improved in f3b1725.


Massive flickering in every single keystroke

Issue Type: Bug
Testing #43968.

Start creating a new folder in the explorer
Type 'folder1/'
-> once the '/' is in the input box every keystroke afterwards causes a full flicker of the information widget shown below the input box.

This also happens on other platforms.
VS Code version: Code - Insiders 1.21.0-insider (1a84a882737184488d965426747a05474f54a759, 2018-02-27T06:17:53.785Z)
OS version: Linux x64 4.4.0-59-generic
Reproduces without extensions

We need to debounce


[orga] pluralize lists on team page

No more "1 Pending Invitations" or "1 Team Members" on the team page! 🎉

Is it a bird? :bird: Is it a plane? 🛩  No, it's Captain Pluralizer! 👨🚀
Thanks!

Oh no, I've got a nickname :D


ci: Switch from actions/setup-elixir to erlef/setup-elixir

From here:

Please note: This repository is currently unmaintained by a team of developers at GitHub. It is now maintained by the Erlang Ecosystem Foundation at erlef/setup-elixir.

👍


changing the hashtag of the banner to "#GazaGenocide"

Update Banner.jsx by changing the hashtag from the not very used "#PalestineLivesMatter" to the hashtag that is continuesly updated and used, "#GazaGenocide"

Thank you


Allow admin to add new environments in the UI

The administration section of the app has an environments tab. However it is not clear how an administrator can add a new environment. When I set up the application I do not see any environments and I need to be able to easily add new environments for new studies.
offline use case
For offline environments we need to provide docker images or tar.gz environments as files that will be transferred to the offline machine via USB drive. The admin would then need to upload a file (docker image or tar.gz) to Arachne datanode to add the new environment.
online use case
For online machines the admin should be able to provide the url for the docker image or tar.gz file through the datanode UI which will then be downloaded from the internet and added as a new runtime environment for Arachne.
Thanks!

To review after v2.0.3 and to specify

I think we can skip this request. Let's focus on deployment with the current features.