Failed to decode repository data: json: cannot unmarshal number into Go value of type main.PluginPackages

Description of the problem or steps to reproduce
Press Ctrl+e, run plugin related commands such as plugin search whatever or plugin install vcs, hit return.
Micro becomes "minimized" (not visible) and shows this error:
Failed to decode repository data:
 json: cannot unmarshal number into Go value of type main.PluginPackages

Press enter to continue

Specifications
$ micro -version
Version: 1.4.1
Commit hash: 1856891
Compiled on August 10, 2018

Running the linux64 binary from the 1.4.1 github release. Running in urxvt v9.22 on Debian Unstable with no custom config at.

I've same problem.

I’m seeing a similar issue. Here’s my screen fetch:

Go and micro versions:
➜  omnisync git:(master) ✗ go version
Ago version go1.11 linux/amd64
➜  omnisync git:(master) ✗ micro --version
Version: 1.4.1
Commit hash: 1856891
Compiled on August 10, 2018

If you need anything else let me know! Loving the editor so far.

I have looked in to this and in the plugin json data which is pulled from the micro plugin github repo has a missing plugin
// Rubocop plugin
"https://raw.githubusercontent.com/computerers/micro-rubocop/master/repo.json"
is no longer on github.
Because the plugin is missing it is causing the error.
Needs removing from the plugin repo.


WIP: Creating Homepage

For now it looks like this.

@alaxalves It's great! Fix the rubocop issues! :smiley:


Reviewfeedback, Python design document

It gets complicated with ARM wheels. There is a separate project that currently is providing wheels for raspberry pi. If we want to mandate ARM compatibility for binary extensions, then we (I) need to provide much more guidance as to which versions of ARM (beyond bitness - i.e. ARM v6, v7, v8 etc.) as well as which OS's we'd be testing on. Which I quite honestly don't know yet.

I'll mention requirements for platform support in the next revision.


Integrate notifications flyout.

Integrate basic notifications fly-out. More changes will follow once we confirm its deployment on the Dynamo side.
@QilongTang @filipeotero

@reddyashish It does not build with Node.js v16? what is your local version of it?

@reddyashish It does not build with Node.js v16? what is your local version of it?

It does, v16.13.2

Would you check out the Github Action build error?

When you have this cleaned up, would you also create a gif for the latest state?

Created a clean PR to avoid the dependency conflicts. Closing this and will be using https://github.com/DynamoDS/NotificationCenter/pull/6


Log collection setting stored in comment

What this PR does:
Log collection setting stored in comment
Checklist

[x] Changes manually tested
[ ] Automated Tests added/updated
[ ] Documentation added/updated
[x] CLA Signed: DataStax CLA

Sample log from local: INFO  [vert.x-eventloop-thread-2] 2024-06-12 12:45:48,709 CreateCollectionOperation.java:108 - Executing CreateCollectionOperation for movie with property {"collection":{"name":"movie","schema_version":1,"options":{"vector":{"dimension":1536,"metric":"cosine","service":{"provider":"openai","modelName":"text-embedding-ada-002"}},"defaultId":{"type":""}}}}


[NET-999][system-probe] Add runtime compilation telemetry

What does this PR do?
Adds telemetry related to runtime compilation to the system probe expvar endpoint
Motivation
NET-999
Additional Notes
Describe your test plan

Add the following to your system_probe.yaml file:

system_probe_config:
  debug_port: 6064
  enable_runtime_compiler: true

Run the system probe
Check that curl localhost:6064/debug/vars | jq .compiler returns the following stats:

{
  "CompilationDuration": [n > 0],
  "CompilationSuccess": [0 if runtime compilation failed, 1 if it succeeded],
  "CompilerEnabled": 1
}

This sets up the expvar data, but I think we also want it reported to the backend as actual telemetry we can report on. @leeavital is that correct?


[FEATURE] Add login button to website

Add login/sign in button to the website in the right hand corner.

I want to work it on

pls assign me this issue

I haven't started yet so please assign it to me

@Vignesh2131 go ahead

for the confirmation, the issue is adding only the login and sign-in button in the navbar. Is that it?


Language name

A (the?) key language name quality is, unique searchability.
Languages named helpfully for language learners and users (hello Stack Overflow):

Clojure, ClojureScript
JavaScript
Kotlin
SQL
GraphQL
Datomic
LINQ

Ambiguous names with previous and ongoing confusion, or too short to be unique:

Go
Rust
C
Python

Maybe not too late to consider a name more unique than Morel. It'd add possibilities like the use of Google Trends, and search of future references in academic papers

I agree. Naming is important. It's also hard.
Fun fact. About 20 years ago I was working on Mondrian, an implementation of the MDX (multi-dimensional expressions) language. It was easy to find examples until Acura launched an SUV of the same name.
Also in my younger years, I played guitar in a number of bands. Choosing a name was always the hardest part. Some of the bands broke up before we managed to agree on a name.
Morel has already been renamed once. It used to be smlj, which was very searchable but not very pronounceable, and didn't convey the emphasis on relational algebra.
Maybe it will be renamed again. Or better still, someone will fork it and it will evolve into something better. (BCPL yielded B yielded C; Oak was evolved into Java.)

If people get into the habit of using the tag "MorelLang", that wouldn't be a bad thing. GoLang has worked pretty well for Go.

I'm going to close this issue because there's nothing concrete we can do. Feel free to re-open if you have suggestions.


[Bug Report] ttnn.max wrong results

Describe the bug
ttnn.max returns unexpected results
To Reproduce
with ttnn.manage_device(device_id=0) as device:
    x = -torch.ones((1, 2, 3, 4), dtype=torch.float32)
    x_tt = ttnn.from_torch(x, dtype=ttnn.bfloat16, layout=ttnn.TILE_LAYOUT, device=device)
    y_tt = ttnn.max(x_tt, dim=-1)
    print(ttnn.to_torch(x_tt))
    print(ttnn.to_torch(y_tt))

Output:
TorchTensor([[[[-1., -1., -1., -1.],
               [-1., -1., -1., -1.],
               [-1., -1., -1., -1.]],

              [[-1., -1., -1., -1.],
               [-1., -1., -1., -1.],
               [-1., -1., -1., -1.]]]], dtype=torch.bfloat16)
TorchTensor([[[[0.],
               [0.],
               [0.]],

              [[0.],
               [0.],
               [0.]]]], dtype=torch.bfloat16)

Expected behavior
Resulting value should be -1.
Please complete the following environment information:

OS: Ubuntu 20.04
Version of software: a7d9204fc9cdf579e0a512e3c26fb291b398c75b

Hi @mywoodstock it blocks us from solving other issues for example unstable softmax: https://github.com/tenstorrent/tt-metal/issues/12661
It should be fixed ASAP.
cc @jvasilje

@dmakoviichuk-tt Don't think I'm the right person for op --max would be under elt-wise?

Hey @caixunshiren,
is this bug the same one you mentioned earlier (the reduce issue), which requires LLK changes?

Hey @caixunshiren,
is this bug the same one you mentioned earlier (the reduce issue), which requires LLK changes?

I'm not sure if they are the exact same issue, but it is related to reduce_revert_delta function
here is the issue: https://github.com/tenstorrent/tt-metal/issues/9370

@caixunshiren Does it fail on the main branch too now ? We put a workaround for the issue in #9370 which fixes it, but we have not closed the issue since it is a temporary workaround.

@caixunshiren Does it fail on the main branch too now ? We put a workaround for the issue in #9370 which fixes it, but we have not closed the issue since it is a temporary workaround.

No it doesn't! We haven't observed ndpcc so far

I think the problem is the same as in softmax, the padded values are all 0s, and when use max/reduce it will pick those values, the Softmax fix is here: https://github.com/tenstorrent/tt-metal/pull/13068
@dmakoviichuk-tt please assign this ticket to the reduce op owner as I'm unfamiliar with it, I can walk through the details of the fix.

So I had a look at ttnn/cpp/ttnn/operations/reduction/generic/device/kernels/compute/reduce_w.cpp
The function does not call reduce_revert_delta(), so is the ND PCC you are facing may be related to the reduce_revert_delta when you are running a big program, but as for the above test case it is not related to #9370

Could you point me to the issue/code where you are seeing the ND issue ? @caixunshiren

Could you point me to the issue/code where you are seeing the ND issue ? @caixunshiren

Yes it is this issue: https://github.com/tenstorrent/tt-metal/issues/9370

@ttmtrajkovic @dmakoviichuk-tt @yugaoTT  - who is working on a fix for this issue? there are 3 of you assigned, and it's a P0 bug/=.
A P0 bug has 1 assignee, and should receive daily updates on progress - otherwise it's not a P0.

@ttmtrajkovic @dmakoviichuk-tt @yugaoTT - who is working on a fix for this issue? there are 3 of you assigned, and it's a P0 bug/=.
A P0 bug has 1 assignee, and should receive daily updates on progress - otherwise it's not a P0.

Initially there was some idea that this issue was related to an issue we had been looking at in llk (ND issue) so Milos was assigned. I am removing Milos from the assigned list.

@jvasilje  @amahmudTT  do you have any updates on this issues?

@amahmudTT  please don't answer my questions in my comments :)

@amahmudTT please don't answer my questions in my comments :) I started thinking I have a split consciousness.

Damn, I never knew this was even possible lol

I changed the shape to (1, 2, 32, 32) and it looks like -1 values are returned. Next steps: I will need to look at how the padding is set up.

@bbradelTT  does autoformat cause issue here too?

In a way. autoformat uses pad, which ignores the pad value and either pads with 0s or fails with an assert (depending on the dimensions passed in). I opened up a P0 since that's blocking my other P0 as well: https://github.com/tenstorrent/tt-metal/issues/15511


iPad issues

I am glad to find UT Notes, which has a lot of potential in improving markdown math input on iOS. Thanks for sharing this app!
I noticed a couple of issues on iPad:
For the multi-window view, it will be nice to have one window in markdown and another window in "live" preview. Can you offer a mechanism to save in the middle of editing, instead of pressing "Done" in the end, so that the preview can catch up with the edits? This will also help prevent accidental loss of editing work.
If there are multiple display-math formula:

$$ ... $$
and
$$ ... $$,

the preview of the entire document renders correctly. However, the "math preview" box can get confused, sometimes mistaking "$$ and $$" as a math formula, as evident in the light-blue color coding. Can the algorithm be improved to fix this glitch? In fact, UT Notes can recognize the following format without any problem:

$$
...
$$
and
$$
...
$$.

Thank you for the feedback!
The synchronized "live" preview is a great idea. I'll add it to the feature list of v1.2.0.
And for the parsing issue of display mode formula, it's a known issue and I am working on solving it. This bug will be fixed on the next version of UT Notes, which hopefully can be released in next week.
Thanks again for your feedback. UT Notes is a small app which has just started, and I will continue to improve it. Please let me know if you have any more advice for the app.

Thanks for the quick response. I am glad that these improvements are in the works.
UT Notes has a lot of potential. I still remember the moment when I found Typora, which immediately became the default markdown editor on my mac. I encourage you to look into blending the markdown source code and preview further, in the spirit of Typora.


Update to JDK 11

Updating plugin to jdk 11, which will unblock fixes for several issues.

Just use 2.361.x baseline.

#403


Does not work with AAI Programmable Vehicles

Need to 1) detect and delete the AAI dummy character, and 2) figure out the AAI remote interface for when vehicles are destroyed and later recreated.

The main problem right now is actually that Earendel implicitly refuses to support script_raised_built for the creation of autonomous vehicles.  This means the only possible means of compatibility are to ban them completely, or to turn off the AI and teleport them to a shadow realm.

As of December 2022, turning off the AI before loading fixes the problem of them being completely ignored by AAI after unloading.  The problem occurs when the AI is still in "On" or "Auto" modes when you load them to a wagon.  The specific AI-version of the car entity created in that case does not trigger the AAI PV script to detect it when it is unloaded.


20 connect general gui with webscrape

Implemented a refresh button, which is able to update the current village's fields and infra
Implemented a faster login, where the program gets the login credentials from file if it exists
Implemented the save email button, if it's clicked then it'll create a file and write the email in it, then it'll use it in the next login.

Closes #20


iOS: Search box loses focus

See https://forum.newsblur.com/t/ios-search-box-is-broken/7664/3

So the user, rightfully, mentioned that this should have gone out. But the problem is that it was on top of other UI changes and so I'm wondering when you think we'll be ready for a release. I've noticed that we're building a lot but there's never a clear moment for a release. Would you be alright with noting that you feel a build is ready for release so I can give it a full test and then we'll fix any remaining issues.
Right now we're waiting on a rather substantial UI refresh but what is your estimate on a release? Can we cherry pick?

The changes in the Dejal branch should always be in a releasable state, at least at the end of each month. You are welcome to merge them into master and release at your convenience. If I'm working on a bigger project that might not be finished by the end of the month, I do so in a different branch (e.g. the Catalyst one, or a local one).
Since I tend to do NewsBlur work in the third or fourth week of the month, you could do a release at the beginning of each month if you wish, if there are worthwhile changes. There will be a couple week period when I'm not changing anything, so is safe to release. (A guaranteed indicator I'm done for the month: when I send the invoice.)


⚠ Bofrost.de has degraded performance

In f3b93f2, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1412 ms

Resolved: Bofrost.de performance has improved in 92dda32 after 6 minutes.


feat(init): initial creation of extracted react-console

[ ] Removal PR to be merged after this one https://github.com/patternfly/patternfly-react/pull/8546
[ ] Move console issues from the the main repo here

I suggest let's merge it to test the github workflows (which can only be tested in a merged state) and send commits on top then.


Remove Mutation body from FailedMutation

Fixes #2093

This change is

@coryan
In  /bigtable/internal/table.cc
BulkApply()
line 112
 auto failures = mutator.ExtractFinalFailures();
  if (!status.ok()) {
    return failures;
  }

If I add something like:
  //if (!status.ok()) {
   // return failures;
 // }
  google::cloud::Status gc_status = bigtable::internal::MakeStatusFromRpcError(status);
  if(!gc_status.ok()){
	  std::vector<FailedMutation> failures_new;
	  for(auto f: failures){
		  failures_new.emplace_back(gc_status, f.original_index());
	  }
	  return failures_new;
  }

then in the last function in   /table_bulk_apply_test.cc
I get
 std::cout << failures.front().status().code() << '\n';  // FAILED_PRECONDITION

which is consistent with what it is expected instead of UNKNOWN
I have not made this change yet. I know that you want to remove Noex::Table so it can wait.

Codecov Report

Merging #2169 into master will decrease coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #2169      +/-   ##
==========================================
- Coverage   92.74%   92.74%   -0.01%
==========================================
  Files         306      306
  Lines       17861    17853       -8
==========================================
- Hits        16565    16557       -8
  Misses       1296     1296

Impacted Files
Coverage Δ

google/cloud/bigtable/internal/bulk_mutator.cc
98.73% <100%> (-0.05%)
:arrow_down:

google/cloud/bigtable/internal/table.cc
98.42% <100%> (ø)
:arrow_up:

google/cloud/bigtable/mutations.h
98.96% <100%> (-0.06%)
:arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 58ddfc9...c602ba9. Read the comment docs.


⚠ deces.matchid.io search API has degraded performance

In e5173fd, deces.matchid.io search API (https://deces.matchid.io/deces/api/v1/search?q=pompidou) experienced degraded performance:

HTTP code: 200
Response time: 4198 ms

Resolved: deces.matchid.io search API performance has improved in 00ebac7 after 37 minutes.


Unable to update from Asset Store in 32bit edtior

Hi,
I realize that it might not be the best place to report that specific issue but it seems that more fitted Unit forum thread for Zenject does not exist. Problem: under Unity 5.2.4 Zenject does not update above version 2.8. Weirdest thing is that there is [Update] button enabled (so Unity sees that there is an update on the server) but after downloading it says "Zenject 2.8 finished downloading". No such problems with standard (64 bit) Unity build

I can't seem to reproduce this issue.  Maybe you could try deleting the old Zenject packages in your asset store cache?
On windows they are here:
C:\Users\YOUR USER NAME\AppData\Roaming\Unity\Asset Store
C:\Users\YOUR USER NAME\AppData\Roaming\Unity\Asset Store-5.x

Going to assume this is fixed


Configuring (e.g., Table of Contents)

How do I disable the ToC suggestion? Also, is there a way to configure what warnings show in the linting?

Also, is there a way to configure what warnings show in the linting?

I don't see any and I think there won't be any.
https://github.com/artempyanykh/marksman/issues/84

For the "Create Table of Contents" code action, you could comment in https://github.com/sublimelsp/LSP/issues/1846 on how you would expect the different kinds of code actions to be presented in the editor. There are three different code action kinds "quickfix", "refactor" and "source" currently defined in the LSP specs, and this ToC action has the "source" kind.
Having more opinions would increase the chance of moving forward with improvements.
From the side of the LSP package, there is currently no way to disable only a particular code action - the only thing possible would be "show_code_actions": "" to disable all code actions for all servers.

I don't know much about LSP (or code actions), I just know that if I need a ToC, I'll have pandoc generate it so it was annoying to have the suggestion always show on the right side of the line I was editing. Again, I don't know anything about LSP or the underlying architecture, only that it'd be nice to turn an action off somehow.

See this marksman issue I've just created: https://github.com/artempyanykh/marksman/issues/90
We'll see what comes out of it.

My 2¢: the easiest would likely be adding a Marksman config to disable the TOC code action (can be a part of https://github.com/artempyanykh/marksman/issues/91).

I'm thinking that on our (ST LSP) side we will start filtering out source actions so this problem will be fixed anyway. It's not an issue that the code action exists, it's only an issue that it appears in the view for every line in the file.
Given how we will behave and how VSCode behaves, I don't think it will really be necessary to add configuration for it.

Sublime LSP from 1.20.0 is filtering out source code action for implicit code action requests.


gui: refactor with new function drawTextInBBox()

Additional refactoring after #3528
I added a function that removes most code duplication from drawInstanceNames() and drawITermLabels(). It takes parameters for a string, color, font, rect, but also a painter pointer. Perhaps this is passing around the QPainter pointer too many times?

Pointers are cheap to pass so no worries about that.  Thanks for the improvement.

FYI - it would be good to update your master branch in your fork.  The message id error has been fixed there.


docs: reference in helix-query.yaml is outdated

the reference in here...
https://github.com/adobe/helix-project-boilerplate/blob/main/helix-query.yaml#L2
...seems very outdated and possibly targeting a different audience.

Fixed


Error when starting bot.

Encountered an error in node main. Error: write EPROTO 7012:error:1408F10B:SSL routines:ssl3_get_record:wrong version number:c:\ws\deps\openssl\openssl\ssl\record\ssl3_record.c:332:
at WriteWrap.onWriteComplete [as oncomplete] (node:internal/stream_base_commons:98:16) {

errno: -4046,
code: 'EPROTO',
syscall: 'write'
}
Node main closed. Code: 1006. Reason: .
Node main disconnected.
I've configured everything correctly, lavalink, etc. Any idea?

which branch are you running?

i think i see your problem, try to set LAVA_SECURE to false

i think i see your problem, try to set LAVA_SECURE to false

I'm running djs13, when setting LAVA_SECURE to false, it connects then show's -
Node main is now ready.
Node main closed. Code: 1011. Reason: .

use this latest dev build https://ci.fredboat.com/repository/download/Lavalink_Build/8871:id/Lavalink.jar since the lavalink client used by the bot use doesn't support master build anymore

use this latest dev build https://ci.fredboat.com/repository/download/Lavalink_Build/8871:id/Lavalink.jar since the lavalink client used by the bot doesn't support master build anymore

Works now! Thanks :D


Added support for Windows

Small fix to support Windows

I checked older windows PR and there are conflicts so I reopened this one.

Tried it and works great for me! Cheers! Could this get merged in?

Thanks @pevecyan @dougd ✌

Magic thanks! 👍


Add instance for RemoteHost

Yes, it doesn't fail on this missing instance (fails on something else, but that's a different issue).


decouple authentication information from HOMEBREW_ARTIFACT_DOMAIN

add support for credentials usage in combination with other mirroring
features. previously HOMEBREW_DOCKER_REGISTRY_TOKEN and
HOMEBREW_DOCKER_REGISTRY_BASIC_AUTH_TOKEN where only used in combination
with HOMEBREW_ARTIFACT_DOMAIN which affects all curl download strategies.
HOMEBREW_BOTTLE_DOMAIN is only used for bottle artifacts, whose mirror might
also require credentials.
this change removes the requirement of using HOMEBREW_ARTIFACT_DOMAIN.
related to #13226

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

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

Thanks so much for your first contribution!  You rock, @UiP9AV6Y!


would like to be able to append array to a DataFrame

It would be useful to be able to append an array of values of the right types in the right order to a DataFrame.

Resolved by implementation of push! in #621.


Incompatible with guzzlehttp/promises ^2.0

https://github.com/8ctopus/php-ga-measurement-protocol/blob/master/src/Network/HttpClient.php#L51
Line 51 in src/Network/HttpClient.php calls unwrap(). This method is removed in the PromiseInterface of guzzlehttp/promises v2.
There are two ways out:

Requireguzzlehttp/promises ^1.5

Apply this fix.
https://github.com/theiconic/php-ga-measurement-protocol/issues/109#issuecomment-1590153733

I'm terribly sorry, I somehow missed the notification that you opened an issue.
The library has become useless as universal analytics data collection has been stopped by google.
Let me know if you still need the fix applied.


DASK slow down the IDE

What happened:
Hi there, I have installed a new Anaconda 2021.11 with a new Spyder 5.1.5 on my Windows 10 with a New DASK
When I open the new file and import packages, everything goes well, but if I read parquet, it kills my machine.
A simple +,-,/  or another task like asking timedelta(hours=200) takes 10 or more seconds.
Do you have an idea to solve that problem?
 It seems this is more of a usage question and we encourage people to ask such questions on the Dask Discourse. Would you mind opening a discussion topic about this over there? If you do so, feel free to close this issue and we will continue the conversation on discourse.
To get the conversation started though, I think you're looking for the %timeit magic command, rather than timedelta. You can use this is an IPython session (info on using IPython in Spyder here), e.g.:
import dask.dataframe as dd
%timeit dd.read_parquet("~/tmp/x.parq")

Second, since you're lazily reading in the parquet file, I wouldn't expect the jump in memory you described since you haven't actually executed the function yet (more on that here). It's a bit hard to test, though, without a minimal reproducer.
Looking forward to continuing the conversation on Discourse!

Hi,
I have created almost the same ISSUE on SPYDER GIT ISSUE.... and they figured out, that the problem is not about the DASK.
It is about the Spyder Variable Explorer. When we run a part of the python script the Spyder Variable Explorer refreshes every variable size ....  While checking the DASK data frame size is not a fast action, it takes time.  The (next) Spyder 5.2.2 will solve this problem.
https://github.com/spyder-ide/spyder/issues/16937
Spyder Support answer:
"...... I think to fix this we should give up computing the representation I mentioned after a timeout, and only show those results we could compute before it..."

Thanks for the update @korosig! We'll go ahead and close this issue, then.


I get an error when I try to run the UI

Environment Information (please complete the following information):

OS: [Windows]
Python Version [3.11]
LLM that you're using: [glm-4]
Other Configurations except the LLM api/key related: [e.g., code_verification: true]

When I try to run the UI I get this error:
(taskweaver) D:\TaskWeaver\playground\UI>chainlit run app.py
Traceback (most recent call last):
File "", line 198, in run_module_as_main
File "", line 88, in run_code
File "D:\software\anaconda\envs\taskweaver\Scripts\chainlit.exe_main.py", line 4, in
File "D:\software\anaconda\envs\taskweaver\Lib\site-packages\chainlit_init.py", line 24, in
from chainlit.action import Action
File "D:\software\anaconda\envs\taskweaver\Lib\site-packages\chainlit\action.py", line 5, in
from chainlit.telemetry import trace_event
File "D:\software\anaconda\envs\taskweaver\Lib\site-packages\chainlit\telemetry.py", line 12, in
from chainlit.config import config
File "D:\software\anaconda\envs\taskweaver\Lib\site-packages\chainlit\config.py", line 469, in
config = load_config()
^^^^^^^^^^^^^
File "D:\software\anaconda\envs\taskweaver\Lib\site-packages\chainlit\config.py", line 438, in load_config
settings = load_settings()
^^^^^^^^^^^^^^^
File "D:\software\anaconda\envs\taskweaver\Lib\site-packages\chainlit\config.py", line 408, in load_settings
features_settings = FeaturesSettings(**features_settings)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "D:\software\anaconda\envs\taskweaver\Lib\site-packages\pydantic_internal_dataclasses.py", line 135, in init
s.pydantic_validator.validate_python(ArgsKwargs(args, kwargs), self_instance=s)
pydantic_core._pydantic_core.ValidationError: 1 validation error for FeaturesSettings
multi_modal
Input should be a dictionary or an instance of MultiModalFeature [type=dataclass_type, input_value=True, input_type=bool]
For further information visit https://errors.pydantic.dev/2.6/v/dataclass_type

same error here

@uebhh Looks like the guys merged a bad PR request at Chainlit. In your .chainlit/config.toml comment out the following lines in [features] and you are good to go for now. This is me assuming your app does not require multi modal or speech to text capabilities in the Chainlit UI.
multi_modal = false
[features.speech_to_text]
    enabled = false

I have mine like
# Authorize users to upload files with messages
# multi_modal = false

# Allows user to use speech to text
#[features.speech_to_text]
#    enabled = false
    # See all languages here https://github.com/JamesBrill/react-speech-recognition/blob/HEAD/docs/API.md#language-string
    # language = "en-US"

Hope it helps for now.

It seems chainlit has a version upgrade and the configurations have changed and it is incompatible with the previous.
The fix would be change the line
multi_modal = true

to
multi_model.enabled = true

Refer to https://docs.chainlit.io/backend/config/features

@uebhh Looks like the guys merged a bad PR request at Chainlit. In your .chainlit/config.toml comment out the following lines in [features] and you are good to go for now. This is me assuming your app does not require multi modal or speech to text capabilities in the Chainlit UI.
multi_modal = false
[features.speech_to_text]
    enabled = false

I have mine like
# Authorize users to upload files with messages
# multi_modal = false

# Allows user to use speech to text
#[features.speech_to_text]
#    enabled = false
    # See all languages here https://github.com/JamesBrill/react-speech-recognition/blob/HEAD/docs/API.md#language-string
    # language = "en-US"

Hope it helps for now.

Thank you, after the modification, it can be started.

It seems chainlit has a version upgrade and the configurations have changed and it is incompatible with the previous. The fix would be change the line
multi_modal = true

to
multi_model.enabled = true

Refer to https://docs.chainlit.io/backend/config/features

Thank you, it worked.


Impact on running VMs with Av1 after retirement of size.

Hello team,
For https://learn.microsoft.com/en-us/azure/virtual-machines/av1-series-retirement we do not share what will happen to the virtual machines in case the due date is passed.
On https://learn.microsoft.com/en-us/azure/virtual-machines/nv-series-retirement#how-does-the-nv-series-migration-affect-me we share it.
Wondering if we can update this doc with this information.

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

ID: e4c428ba-3330-a43a-5ef4-258769cbdba2
Version Independent ID: f1f1bc9f-d2af-b820-0871-87b1e7f0e6d4
Content: Av1-series retirement - Azure Virtual Machines
Content Source: articles/virtual-machines/av1-series-retirement.md
Service: virtual-machines
Sub-service: sizes
GitHub Login: @rishabv90
Microsoft Alias: risverma

Note: Confirmed with A series PM that the deallocation of VM if no action is done will occur.
Suggesting wording:
"On August 31, 2024, we retire Basic and Standard A-series virtual machines (VMs). Before that date, migrate your workloads to Av2-series VMs, which provide more memory per vCPU and faster storage on solid-state drives (SSDs).
The remaining VMs with these specific sizes on your subscription will be set to a deallocated state. These VMs will be stopped and removed from the host. These VMs will no longer be billed in the deallocated state."

@clmendes-MSFT
Thanks for your feedback! We will investigate and update as appropriate.

@clmendes-MSFT
Thank you for bringing this to our attention.
I've delegated this to content author @@rishabv90, who will review it and offer their insightful opinions.

@rishabv90
Could you please review add comments on this, update as appropriate.

@Naveenommi-MSFT , i spoke with @rishabv90  to confirm the above information.

@Naveenommi-MSFT any updates on this item?
We need this improved asap.

@rishabv90  can we have this doc updated asap?

This issue has already been addressed vide PR: https://github.com/MicrosoftDocs/azure-docs/pull/116469. Doc has been updated and published. Closing this issue.
#please-close


Intl gives an error if you try to initialize e.g. en_US and haven't provided a locale for it.

It would be nice if you could indicate what locale the default message texts were in and have an initialize not give you an error in that case. Otherwise you need to special-case it in your initialization routine.

This issue has been moved to dart-lang/intl#28.


Deploy the Pretrained Model on Jetson Nano

The fork branch corresponding to PR 8878 was accidentally deleted and it can't be merged. This PR will solve the problem, please approve it again, thanks.
@Hzfengsy

It would be great if you can make the CI green

@BBuf Could you please update this PR? Then we can merge it quickly

@BBuf Could you please update this PR? Then we can merge it quickly

Ok, I will deal with it quickly.

this pr has passed ci, please merge it. @Hzfengsy

@BBuf
Have you ever tried to use auto scheduler on jetson nano or xavier nx?
https://discuss.tvm.apache.org/t/compiled-model-from-auto-scheduler-on-jetson-is-pretty-slower-in-jetson-nx/15393
Do you have any idea about the above problem?


Remove fancy_time display

Displays timestamps as dd Mon yyyy hh:mm:ss instead of as fancy_time
The fancy_time display broke tests whenever we switched from daylight to standard or vice versa.
Normal display is more precise and accurate than fancy time.
See Slack conversation starting 2023-03-1-2 0619 PDT
Delivers Pivotal 184679272

Manual Tests

Go to Activity Feed
Timestamps should look like 15 Feb 2023 22:10:30

Coverage: 93.503% (-0.08%) from 93.58% when pulling 627e30cfe9a8a540bedecf4b1017a29e6fc0a6fb on jdc-fancy_time-184679272 into 064787ba9fd155d48ce8ad3fe0be443b2d734aff on main.

The marginal coverage decrease is due to the removal of previously tested lines.
There is no newly-uncovered line.
Coveralls hard-to-interpret statistics say that coverage of app/extensions fell,
But every extension has coverage that is unchanged, or -- for TimeExtensions -- increased!


Update title when dynamic part is changed

Once dynamic path resolved it doesn't respond on real data changes. Say we have a page corresponding some project, and project.title property is a part of page title. When user updates the project the page title will not adjust itself for a new name until reload/history manipulation. Some registering/detecting mechanics would be great to implement, for example as in use-react-router-breadcrumbs.
Btw, thank you for such a simple yet useful project! Wonder why it has so little stars, it really saves a lot of time and brain cells :)

Thank you for your feedback. Your feature request sound good but we currently don't have time to implement it. I would appreciate a PR from anyone who wants to do it.


Won't build for Linux platform

Causes "Cannot build player while editor importing or compiling scripts" on pressing the build button for Linux. (works for Win and Mac)
Same happens when trying to build the example project.

Thank you for reporting that I will check this

diff --git a/Assets/unity.webp/Runtime/NativeWrapper/Dec/Decode.cs b/Assets/unity.webp/Runtime/NativeWrapper/Dec/Decode.cs
index cb463333..07b13bf3 100755
--- a/Assets/unity.webp/Runtime/NativeWrapper/Dec/Decode.cs
+++ b/Assets/unity.webp/Runtime/NativeWrapper/Dec/Decode.cs
@@ -36,7 +36,7 @@ namespace WebP.NativeWrapper.Dec

 #if UNITY_EDITOR_WIN || UNITY_STANDALONE_WIN
         const string DLL_NAME = "libwebp";
-#elif UNITY_EDITOR || UNITY_STANDALONE_OSX
+#elif UNITY_EDITOR || UNITY_STANDALONE_OSX || UNITY_STANDALONE_LINUX^M
         const string DLL_NAME = "webp";
 #elif UNITY_ANDROID
                const string DLL_NAME = "webp";
diff --git a/Assets/unity.webp/Runtime/NativeWrapper/Demux/Demux.cs b/Assets/unity.webp/Runtime/NativeWrapper/Demux/Demux.cs
index 0e8d3c8f..46fdeda0 100755
--- a/Assets/unity.webp/Runtime/NativeWrapper/Demux/Demux.cs
+++ b/Assets/unity.webp/Runtime/NativeWrapper/Demux/Demux.cs
@@ -11,7 +11,7 @@ namespace WebP.NativeWrapper.Demux
         public const int WEBP_DEMUX_ABI_VERSION = 0x0107;
 #if UNITY_EDITOR_WIN || UNITY_STANDALONE_WIN
         const string DLL_NAME = "libwebpdemux";
-#elif UNITY_EDITOR || UNITY_STANDALONE_OSX
+#elif UNITY_EDITOR || UNITY_STANDALONE_OSX || UNITY_STANDALONE_LINUX^M
         const string DLL_NAME = "webpdemux";
 #elif UNITY_ANDROID
                const string DLL_NAME = "webpdemux";

this fixed it for me

I installed from the .unitypackage and the files seem to be organized differently in the git repo

Yes, this issue's problem caused by missing define flag what @heeen said.
I commited 218c6d2f3865992c73574dff0f815294d2852a48 then update release to 0.3.2


Changed ss.frame to ss.get_frame in Documenation

Fixes #849

Use ss.get_frame instead of `ss.frame

Updated some of the outputs in the documentation

Thanks @iamabhishek0! Merging when the tests pass :)

Unrelated AppVeyor failure, restarting.


CSS

CSS
User Story
As a boot camp student
I want the prework notes to be structured on a webpage
So that I can easily find and read the information
Acceptance Criteria
GIVEN a Prework Study Guide website
WHEN I view the study guide
THEN I see a dark blue header and footer, and four boxes with a shadow

CSS added to or HTML for the prework study guide


Inconsistency between list_models and list_spaces

If I do
repos = api.list_spaces(author="hackathon-somos-nlp-2023")

The SpaceInfo objects I get contain the likes attribute. If I do
api.list_models(author="hackathon-somos-nlp-2023")

The ModelInfo objects don't contain this attribute. Setting full is required to get likes. I would suggest to be consistent and have likes in both.

cc @coyotte508 ;)

@Wauplin you can use the expand query param to select fields returned

@coyotte508 @osanseviero I've looked into this issue a bit. I made a script to check all inconsistencies by listing the fields returned by default and those that are optional (ie. returned when full=True). Here is the output:
✅ Default in model is consistent with optional in dataset.
❌ Default in model but optional in space: {'tags'}
❌ Default in dataset but optional in model: {'downloads'}
❌ Default in dataset but optional in space: {'tags', 'cardData'}
❌ Default in space but optional in model: {'likes'}
❌ Default in space but optional in dataset: {'likes'}

To be consistent across all 3 methods, we should:

return downloads and likes by default on the /api/models endpoint
return likes by default on the /api/datasets endpoint
return tags and cardData by default on the /api/spaces endpoint

That being said, I've tried the suggested expand parameter. My problem with it is that if I do https://huggingface.co/api/models?expand[]=likes&expand[]=downloads&limit=1 I will only get the likes and downloads fields instead of having the default ones plus those two.
What I could do is to use expand with all the current default + likes + downloads but that would make huggingface_hub very constrained. If a new default field is added server-side, I would need to update hfh to add the field to the expand list. This is against the current philosophy which is "if a field is added server-side", older versions of hfh should support it out of the box.

To be consistent across all 3 methods, we should:
* return **downloads** and **likes** by default on the `/api/models` endpoint
* return **likes** by default on the `/api/datasets` endpoint
* return **tags** and **cardData** by default on the `/api/spaces` endpoint

WDYT @Pierrci @julien-c ?

So my suggestions for the inconsistency issue are either:
1. we update the "expand" behavior server-side to _append_ fields instead of replacing them + we update `huggingface_hub` accordingly

2. we fix it server-side by adding the missing default fields (downloads/likes for models, likes for datasets and tags/cardData for spaces)

3. we fix the requested fields in `huggingface_hub` to be consistent. Drawback: the fields are hardcoded => less flexible for retro-compatibility when new fields are added

4. we don't care about it? confused

In any case, I think I'll push the consistency script so that we can rerun it regularly (not in every CI though).
What's your opinion @julien-c @coyotte508 @osanseviero. I'd rather prefer 1. so that consistency doesn't have to be necessarily handled server-side in the future. Otherwise I'd prefer 2. rather than 3. or 4.

@Wauplin I'm not sure we're going to add new fields in the list endpoints. But we could do a variation on 1: add a "defaultFields" option to expand: &expand[]=defaultFields, so you could do &expand[]=defaultFields&expand[]=likes&expand[]=downloads for the models. Not that we'll still not return the deprecated modelId even if expand is set :sweat_smile:
WDYT?

I'm not sure we're going to add new default fields in the listing endpoints.

If that's the case then maybe we should hardcode them in huggingface_hub. The advantage I see in that is to be explicit when anyone reading the code (+we would have explicit attributes for ModelInfo DatasetInfo SpaceInfo instances.
And keep the "check inconsistencies script" just in case to check if the list of default fields evolves and we forgot to update hfh accordingly.

But we could do a variation on 1: add a "defaultFields" option to expand:

Would work as well but maybe let's not complexify the API and hardcode things client-side instead.

I would go with option 2 from @Wauplin, it seems reasonable to me

Fixed by a server-side update (private PR: https://github.com/huggingface/moon-landing/pull/6154)


[dy] Refactor workspace

Description

This PR refactors workspace code into a Workspace class to perform workspace related operations.
There are also some bug fixes in this PR that were found during testing.
How Has This Been Tested?

[x] Tested with workspaces in local kubernetes
[x] Tested with multi dev environments in AWS ecs

Checklist

[x] The PR is tagged with proper labels (bug, enhancement, feature, documentation)
[x] I have performed a self-review of my own code
[ ] I have added unit tests that prove my fix is effective or that my feature works
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation

cc: @wangxiaoyou1993

yeah i'll add more comments

@wangxiaoyou1993 added some docstring comments and also refactored the workspace classes to use config classes


Vue 2 - input value is not updated in some cases

Vue.js version
2.0.0.rc.7 and older
Reproduction Link
https://jsfiddle.net/Lonzpg7h/
Steps to reproduce

Type one letter into input field - value won't show up but console logs that event was fired
Type more text - you will see now that value is updating correctly
Now repeat step 1 but with removed this.isDirty = value !== '' or 'is-dirty': this.isDirty declaration - value shows up immediately

What is Expected?
I think input value should be visible on the first keystroke as well
What is actually happening?
In some case input value doesn't show up until you enter text for the second time.
Any idea what's going on here?

Also noticed that when I remove v-model from my component, it's working correctly

Hi, thanks for filling this issue.
For v-model to work on custom components, you need to manually $emit an 'input' event: https://jsfiddle.net/gypskp9h/
Checkout the docs here: http://rc.vuejs.org/guide/components.html#Form-Input-Components-using-Custom-Events

@fnlctrl  so I always need to emit this input event?
Maybe i will try to explain what I am trying to do here. Basically i am trying to create custom textfield component but i need to recreate lazy modifier. I thought I'd handle it this way:
onInput(e) {
  const value = e.target.value;

  this.isDirty = value !== ''; // btw. this need to be updated immediately, so I can't use onChange listener

  if (!this.lazy) {
    this.$emit('input', value);
  }
}

but if lazy prop is enabled i would emit input event on blur
if (this.lazy) {
  this.$watch('focused', (focused) => {
    if (!focused) {
      this.$emit('input', this.$refs.input.value)
    }
  })
}

but as you see there's this issue when i don't emit input event inside onInput function and I am not sure how to handle this.

@fnlctrl also you didn't explain part when removing this.isDirty = value !== '' fixes the issue

I always need to emit this input event
Yes.. this is how using v-model on components work.

you didn't explain the part when removing this.isDirty = value !== '' fixes the issue
Sorry I didn't notice it.. I only noticed the $emit being missing. I don't quite have the time to dig into it now, so I'll leave this open.. Maybe @posva @LinusBorg would kindly explain what's going on here : )

@sqal
In vue 2.0,
<textfield v-model="text"></textfield>

means if you don't fire input event from textfield, the value prop of textfield will always remain the initial value of text which is ''.
Since you never fire any input event, so value prop never changes, so the only thing that can trigger the re-render of textfield is the isDirty state.
When you input a char for the first time, the char appears first, this is the native behavior of input tag. Then this input cause the change of isDirty from false to true, which cause the re-render of textfield, notice that it's value is still empty, so the content of the input gets cleared.
For the subsequent char inputs, the native behavior of input still takes effect so you can see them as well. But this time, isDirty remains true, so no re-render occurred on textfield so the content is not cleared.
If this.isDirty = value !== '' is removed, even the first re-render does not occur because isDirty has not changed, so you can see what you have input immediately, that is just the consequence of textfield not being re-rendered.
I modified your code into a template-based one to make things clearer.

So what happens here is this:

onInput is triggered for the first time
this changes isDirty from true to false
this leads to a re-render of the component because isDirty is used in the render function
since the prop this.value is still an empty string '', the content of the input is reset.
subsequent calls to oInput don't lead to a re-render because this.isDirty is already true

This only happens because you don't $emit the change event. If you do,

the parent gets the value
that triggers an updtae of the value prop in the child
that would lead to the new value being inserted in the input upon that first re-render.

...and everything would be fine. Your example broke the cycle, so to speak.

Thank you @Centaur and @LinusBorg for detailed explanation. Your answers helped me to understand Vue a little bit better :)

Have review your code made changes.
you just have to emit the event while changes happens in child
https://jsfiddle.net/f5k0kLob/


Cubic spline interpolation

cubicInterpolationMode has only two modes for interpolations:

default (custom weighted cubic interpolation)
monotone (monotone cubic interpolation)

We need cubic spline interpolation and as I understand from source code its not the same, isn't? If so is there any approach how to provide custom interpolation mode or some workaround?
Thanks!

@mrandreev unfortunately the only way to add this would be to modify the code. I'm not exactly sure what the math for cubic spline interpolation is otherwise I could try an implementation. Do you have any documentation on how this might work in a chart?

hey @etimberg, thanks it will be really helpful!
I had some research on 'cubic spline interpolation' method so if shortly:

math definition
CSPL.js lib
cubic-spline lib

Actually its popular method for smooth interpolation and already implemented by community (I hope it will be easy to integrate).
Also could you please clarify about the "default" mode. As I see from source code the default method is similar to Bezier curve (but not sure). Just want to make sure its not the same one (to cubic spline) but with diff approach.
Thanks!

Thanks @mrandreev. The default is a Bezier curve with a tension of 0.1.

@etimberg hey there, I would like to clarify is there any possibility to support 'cubic spline interpolation' mode in near future? It will be really useful mode. Thanks!

I think, that Bézier is a cubic spline curve. Another cubic spline curve is b-spline. b-spline can be drawn using Bézier, so that would be possible to implement (as the canvas only supports Bézier).
See: https://www.maissan.net/articles/simulating-vines/2
I'm not sure if b-spline is something that is requested here though.


Device NOT become unavailable when actually offline

Device disconnected from network - offline in cloud and locally.
When sending command - integration log "!! Local4 | Device offline".
But device and entities still "available".
Integration version: master + #987.
2022-11-23 18:36:16 [D] 1000f3e195 => Local4 | {'switch': 'on'} !! Timeout 1
2022-11-23 18:36:16 [D] 1000f3e195 => Cloud4 | {'switch': 'on'} |<PHONE_NUMBER>000
2022-11-23 18:36:22 [W] Cloud ERROR: {'error': 504, 'reason': 'Request Timeout', 'deviceid': '1000f3e195', 'apikey': 'xxx', 'sequence': '1669221375000'}
2022-11-23 18:36:28 [D] 1000f3e195 => Local4 | {'cmd': 'info'} !! Can't connect: Cannot connect to host ewelink_1000f3e195.local.:8081 ssl:default [Connect call failed ('<IP_ADDRESS>', 8081)]
2022-11-23 18:36:28 [D] 1000f3e195 !! Local4 | Device offline

Looks like devices/entities become unavailable only on Integration startup.

After some tests: looks #987 have some logical issues.
Pure master version behaves as expected - after failed command device become unavailable.
Will check what happens w/out sending command to disconnected device and report later.
2022-11-23 19:03:54 [D] 1000f3e195 => Local4 | {'switch': 'on'} !! Timeout 1
2022-11-23 19:03:54 [D] 1000f3e195 => Cloud4 | {'switch': 'on'} |<PHONE_NUMBER>000
2022-11-23 19:03:59 [W] Cloud ERROR: {'error': 504, 'reason': 'Request Timeout', 'deviceid': '1000f3e195', 'apikey': '831b1f86-1d78-4abf-a5b4-dae5c21e48e5', 'sequence': '1669223033000'}
2022-11-23 19:04:03 [D] 100171fd9d <= Local0 | {'switch': 'off', 'startup': 'off', 'pulseConfig': {'pulse': 'off', 'switch': 'off', 'pulseWidth': 500}, 'sledOnline': 'on', 'tempUnit': 0, 'timeZone': 3, 'sensorType': 'AM2301', 'currentTemperature': '-1.3', 'currentHumidity': '86.6', 'autoControlEnabled': 0, 'fwVersion': '1.0.2', 'rssi': -60} | 658891
2022-11-23 19:04:04 [D] 1000f3e195 <= Cloud3 | {'online': False} | None
2022-11-23 19:04:05 [D] 1000f3e195 => Local4 | {'cmd': 'info'} !! Can't connect: Cannot connect to host <IP_ADDRESS>:8081 ssl:default [Connect call failed ('<IP_ADDRESS>', 8081)]
2022-11-23 19:04:05 [D] 1000f3e195 !! Local4 | Device offline
2022-11-23 19:04:05 [D] 1000f3e195 => Local4 | {'cmd': 'info'} !! Can't connect: Cannot connect to host <IP_ADDRESS>:8081 ssl:default [Connect call failed ('<IP_ADDRESS>', 8081)]
2022-11-23 19:04:05 [D] 1000f3e195 !! Local4 | Device offline


cancel wms queries when the results are no longer needed

Peek 2022-10-28 16-56.webm
I've used NDVI from EPSG:3857. Then I created an expression that uses this 2x, adds both and divides it by two. Then… use it like in the video. Moreover, I don't see cancel logs anymore.

bors merge


What properties can appear more than once?

File: engine/reference/commandline/build.md, CC @mstanleyjones
The classic example is tag argument:

docker build -t one -t two

The recent docs don't mention what properties can appear more than once.
Example:
before: -t, --tag=[]
now: --tag, -t
This is not good.

This is stopping code being generated to wrap execution of Docker commands.

@tiborvass @FrenchBen can we modify our YAML generator to somehow capture that a value can be listed more than once?
@RehanSaeed I don't understand your second comment.

@mstanleyjones If there was a column like int or int[], bool, bool[], string or string[] it'd be perfect.
Also, if you have the docs in another format (i.e. yaml) I'd be more than happy to use it (rather than generated human readable docs).
I'm using it for wrapping docker commands as a Cake plugin.

We generate it directly from Docker code itself and some docs in the docker/cli repository. What you see here should not be edited directly.

Guys, any news on this?

@tiborvass do you have any thoughts on this?

👍

Guys!

I'm sorry, but this is not an issue that can be addressed in this repository.  Currently, I do not think there is a programmatic way to even figure out this info. If it does exist or needs to be created, that will be in some combination of moby/moby or docker/cli. We are still waiting on technical review to find out whether it's feasible and where the work should be done. @thaJeztah @tiborvass @FrenchBen if you would like to weigh in, that would be great.

No, there's currently no easy way to add this information to the generated YAML, other than manually adding an annotation to each flag to indicate if the flag can be used multiple times.
The packages used to create the command-line (https://github.com/spf13/cobra, and the https://github.com/spf13/pflag) will always accept a flag to be specified multiple times, but if only a single value is expected, later flags override earlier flags (which complicates things a bit).
To automate docker, instead of scraping the docs, I would really recommend using the remote API instead, which is designed for this, and is documented here: https://docs.docker.com/engine/api/v1.30/
While looking at this, I noticed some information was not included in the YAML files used to generate the documentation, and opened a pull request https://github.com/docker/cli/pull/523. That pull request may be useful as well as it shows how the YAML is generated, and perhaps could be used as a starting point if you don't want to use the API and automate generating code to use the CLI.

Closing this for now, since we can't actually do what you asked for, and you can use the remote API directly instead. I'm looking forward to the improvements in https://github.com/docker/cli/pull/523!

But even as a CLI user, how can I know?

Never mind, I'll parse sources myself. After an effort of something like six hours, I'm able to produce more metadata than your tools.

Contributions are welcome to that script :innocent:

@thaJeztah The thing is, that I hacked a C# console app that parses go code. Do you think you could benefit from it? Besides, CLI is really not very friendly written when it comes to documentation. Perhaps a limit of Go, I don't know.


Insert a block should be highlighted directly in the list

The location of the button to add a markdown in a digest is not obvious :

It should be great if we can add a little circle button with "+" label between two bookmark cards to add this block :


[MRG] channel names include channel type again in mne 0.24.1

Seems https://github.com/braindecode/braindecode/pull/357/files was reverted again in MNE... @agramfort @hubertjb @cedricrommel

ok anyways thanks for feedback :)


feat: linked a Leptos wrapper I built

What is in this PR?
I documented a wrapper around RemixIcon built to support RemixIcon in Rust's Leptos framework
The package is available on crates.io
https://crates.io/crates/leptos-remix-icon

do better!

do better!

What would that mean?


change: add branches and pullrequests during org project creation

will close #267

@shreddedbacon if branches and pullrequests aren't provided they're set to true, but the api doesn't allow non-string values, so currently this PR sends values as stringified values. seems wrong

branches and pullrequests are string values, so they should be free text boxes that are optionally filled in by the user
the API will default these to a string representation of "true" if they are not provided.

Can we put a tooltip (?) next to branches and pullrequests that expands what the API has in the descriptions for those fields? It might be a bit helpful
    """
    Which branches should be deployed, can be one of:
    - \`true\` - all branches are deployed
    - \`false\` - no branches are deployed
    - REGEX - regex of all branches that should be deployed, example: \`^(main|staging)$\`
    """
    branches: String
    """
    Which Pull Requests should be deployed, can be one of:
    - \`true\` - all pull requests are deployed
    - \`false\` - no pull requests are deployed
    - REGEX - regex of all Pull Request titles that should be deployed, example: \`[BUILD]\`
    """
    pullrequests: String

and maybe even one next to production environment
    """
    Which environment(the name) should be marked as the production environment.
    *Important:* If you change this, you need to deploy both environments (the current and previous one) that are affected in order for the change to propagate correctly
    """
    productionEnvironment: String

Hmm, bit too spacey?

This looks fine. I wonder if the first - of each line could be a dot though maybe?

Like this

true - all pull requests are deployed
false - no pull requests are deployed
regex of all Pull Request titles that can be deployed, example: '[BUILD]'


Popup window from another popup window with forms

Hello, great team!
Tell me please, how to open a new form in a popup window from the current one popup correctly ?
If I open a new popup window, my window is empty without fields and buttons...
https://jsfiddle.net/7L86ng2b/

You can just use current popup and output content inside of it. If you must open another popup, then close previous. Here is the code that works https://jsfiddle.net/fnbwuLrt/2/
By the way, you cannot open more then one popup at a time. If you need more then one, then you can use messages as here: https://w2ui.com/web/demos/#/popup/5


CachingAssetBundle._loadString does not handle load() returning null

 Future<String> _fetchString(String key) async {
    final ByteData data = await load(key);
    return UTF8.decode(data.buffer.asUint8List());
  }

Yet clearly some implementations of load() do return null.

Yeah, if you attempt to loadString with an invalid key you hit this.
[ERROR:../../lib/tonic/logging/dart_error.cc(16)] Unhandled exception:
NoSuchMethodError: The getter 'buffer' was called on null.
Receiver: null
Tried calling: buffer
#0      Object._noSuchMethod (dart:core-patch/object_patch.dart:44)
#1      Object.noSuchMethod (dart:core-patch/object_patch.dart:47)
#2      CachingAssetBundle._fetchString.<_fetchString_async_body> (package:flutter/src/services/asset_bundle.dart:138:29)
#3      _RootZone.runUnary (dart:async/zone.dart:1404)
#4      _FutureListener.handleValue (dart:async/future_impl.dart:131)
#5      _Future._propagateToListeners.handleValueCallback (dart:async/future_impl.dart:637)
#6      _Future._propagateToListeners (dart:async/future_impl.dart:667)
#7      _Future._completeWithValue (dart:async/future_impl.dart:477)
#8      _Future._asyncComplete. (dart:async/future_impl.dart:528)
#9      _microtaskLoop (dart:async/schedule_microtask.dart:41)
#10     _startMicrotaskLoop (dart:async/schedule_microtask.dart:50)

The theory behind asset bundles is that you know what's in them, so you'd never request something that wasn't there...
...but yeah, we should at a minimum display a pretty error in checked mode.

Fixed by https://github.com/flutter/flutter/pull/9325


NoMethodError (undefined method `init_journal' for nil:NilClass):

Hello,
I migrated our Redmine to version 3.2, and we now get error 500 with these errors :

When validating a time log :
NoMethodError (undefined method init_journal' for nil:NilClass): app/controllers/timelog_controller.rb:112:in create'
lib/redmine/sudo_mode.rb:63:in `sudo_mode'

When deleting a project :
NoMethodError (undefined method init_journal' for nil:NilClass): app/controllers/projects_controller.rb:225:in destroy'
lib/redmine/sudo_mode.rb:63:in `sudo_mode'

If I remove the plugin, all is fine.
I successfully reproduced the issue on a vanilia installation of Redmine 3.2 (just to be sure), but for the time logging only, no problem at deleting projects. As the errors are the same, I guess this is related.
Thanks for your help and your great plugin :+1:
Pierre

Hi.
I made a bugfix, please try.

Hi annikoff,
Sorry for my late feedback, but everything looks good now !
Thanks a lot :)
Pierre

Hi. You are welcome :)
Thank you for the issue.


Option to remove blank lines

Just what it says on the tin…

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/2684163-option-to-remove-blank-lines?utm_campaign=plugin&utm_content=tracker%2F214563&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F214563&utm_medium=issues&utm_source=github).

Suggest splitting into two separate options:

Remove empty lines inside blocks
Number of newlines after closing brace

For example, I always add two newlines after a closing brace, and remove empties inside the braces.

a {
	background: #fff

	color: #04c
}
b {
	font-weight: bold;
}

would become:

a {
	background: #fff;
	color: #04c;
}

b {
	font-weight: bold;
}

Regardless, this is an awesome tool to have when inheriting a poorly written stylesheet. Thanks!

+1

+1

+1

Hi All!
I'm closing this because unless a PR is created it won't likely be addressed.
If you feel strongly about this request I encourage you to dig in and submit a PR 😸


Tests depend on 'liquid' binary

When you run make test at the root of the project BUT you don't have liquid binary on your PATH, the tests will fail with pattern match failure here. But there are no instructions on how to set up liquid anywhere in the repo.
I think the tests could point user to what's wrong (e.g. pattern match Nothing -> error "these tests require liquid binary to run") and the repo should provide the instructions about how to set that up.
I can work on this, but could use some help (see https://github.com/haskell/haskell-ide-engine/pull/898)

Resolved by adding comment to doc/Hacking.md in https://github.com/haskell/haskell-ide-engine/pull/898 and also there's been test added which checks that liquid binary is present in https://github.com/haskell/haskell-ide-engine/pull/900 which makes the reason for failures without liquid clearer.


error TS2694: Namespace 'React' has no exported member

Thank you for reporting an issue or suggesting an enhancement. We appreciate your feedback - to help the team to understand your needs, please complete the below template to ensure we have the necessary details to assist you.
Category

[ ] Question
[ ] Typo
[ ] Bug
[ ] Additional article idea

If you are planning to share a new feature request (enhancement / suggestion), please use SP Dev UserVoice at http://aka.ms/sp-dev-uservoice.

Expected or Desired Behavior
If you are reporting a bug, please describe the expected behavior. If you are suggesting an enhancement please describe thoroughly the enhancement, how it can be achieved, and expected benefit.
Observed Behavior
If you are reporting a bug, please describe the behavior you expected to occur when performing the action. If you are making a suggestion, you can delete this section.
Steps to Reproduce
If you are reporting a bug please describe the steps to reproduce the bug in sufficient detail to allow testing. Only way to fix things properly, is to have sufficient details to reproduce it. If you are making a suggestion, you can delete this section.
Submission Guidelines
Delete this section after reading

All suggestions or bugs are welcome, please let us know what's on your mind.
If you are reporting an issue around any of the documents or articles, please ensure that you have clear reference on the specific file or URL, which should be fixed.
If you have technical questions about the framework, we’ll be monitoring #spfx, #spfx-webparts, and #spfx-tooling on (SharePoint StackExchange)[http://sharepoint.stackexchange.com/]. You can also alternatively submit your question to (SharePoint Developer group)[https://network.office.com/t5/SharePoint-Developer/bd-p/SharePointDev] at Office Network.
Remember to include sufficient details and context.
If you have multiple suggestions or bugs please submit them in separate bugs so we can track resolution.

Thanks for your contribution! Sharing is caring.

@bharav - this isn't really actionable... you only have information in the title and haven't actually given any other information (or I missed it).  Can you give more context?

We closed this issue as it had not activity within last 180 days. This is a generic process we have decided to perform for issues, which have not been explicitly marked still to be "work in progress" based on tags. We are performing this cleaning to make sure that old issues that have already been solved (but not closed) or are no longer relevant are cleaned out and make the issues more manageable. If this issue still valid, we would ask you to open a new issue and follow the guidance in the issue template related on the recommended location. We do apologize any inconveniences this might cause. Please do remember that issues in the issue lists are also messages for others in the community, so you can also check if you can assist on any of them. “Sharing is caring!”


Strong consistency support for ets

"Strong consistency only support the leveldb backend currently."
Do you have a plan to support ets backend in some future?

Hi,
Right now it does not make any sense, since the strong consistency backend (riak_ensemble) goes to disk for the tree.


Setting Proxy-Authorization Header Manually

I know you guys have addressed a variety of concerns relating to configuring proxies and that proxying is functionally complex, but I'd like to raise one issue I was having today. We have a new proxy in our environment that requires auth with a yubikey rather than a password. We can simply set an environment variable to a token returned from a yubikey auth endpoint that expires every 12 hours and pass it along with the Proxy-Authorization header in tools like curl, wget, etc... However, it gets a little trickier with requests, as the default api seems to only support including the proxy-user:proxy-pass in the proxy-url.
In our case, we can't do this, as a yubikey token is only good once. As a temporary workaround, I've subclassed HTTPAdapter and modified the proxy_headers method:
class CustomHTTPAdapter(HTTPAdapter):
    def proxy_headers(self, proxy):
        return {
            'Proxy-Authorization': 'Basic ' + environ.get('proxy_token', str())
        }

and then mount it on a Session:
import requests

from mypkgs.adapters import CustomHTTPAdapter

s = requests.Session()
s.mount('https://', CustomHTTPAdapter())
s.get('https://someurl')

It would be incredibly useful to simply be able to do something like requests.get(uri, proxy_headers=proxy_headers). Am I barking up the wrong tree here or do you have any ideas or feedback?

Generally speaking we're disinclined to add a separate proxy_headers kwarg. The vast majority of users don't need it, and those that do are relatively well-served by the proxy_headers override of the HTTPAdapter.
Requests is extremely reluctant to add new kwargs to the top-level API if it can possibly be avoided, and in this case I think it can.
Sorry! =(

Thanks for the feedback! In that case, is there an easy way to mount a custom HTTPAdapter on the main requests api rather than on a session? I didn't note any in the documentation, but maybe I missed it.

Nope: the requests API manufactures a new Session on each request to avoid having any implicit global state that can lead to all kinds of weird bugs.


Events directly after each other not rendering properly

I am using v31.0-cf5c9 and this issues exists on canary v31.0-14a08 as well.
trace.pb.zip
In the attached protobuf file (sorry its big), I have the event Reduce and DRelu that, based on their start time and duration, should be right beside each other.
However, there is a gap between them:
Reduce starts at 25us 840ns with a duration of 155ns
DRelu (and TxAct) start at 25us 995ns.

Duplicate of https://github.com/google/perfetto/issues/282


fix: Handle comments at the beginning of YAML files

If the first line of the YAML file contains a comment, the
multiple-document detector incorrectly classifies the file as containing
more than one YAML document and fails. This patch fixes that issue.
Signed-off-by: Charith Ellawala<EMAIL_ADDRESS>

Codecov Report

Merging #803 (48ad4f1) into main (b1a874c) will increase coverage by 0.19%.
The diff coverage is 33.33%.

:exclamation: Current head 48ad4f1 differs from pull request most recent head 5aa3c07. Consider uploading reports for the commit 5aa3c07 to get more accurate results

@@            Coverage Diff             @@
##             main     #803      +/-   ##
==========================================
+ Coverage   54.09%   54.28%   +0.19%
==========================================
  Files         100      100
  Lines       10593    10595       +2
==========================================
+ Hits         5730     5752      +22
+ Misses       4363     4344      -19
+ Partials      500      499       -1

Impacted Files
Coverage Δ

internal/util/serdeio.go
63.51% <33.33%> (-1.77%)
:arrow_down:

internal/engine/engine.go
69.97% <0.00%> (+0.80%)
:arrow_up:

internal/storage/index/builder.go
84.35% <0.00%> (+5.02%)
:arrow_up:

internal/compile/errors.go
55.55% <0.00%> (+15.87%)
:arrow_up:


Cleanups while going through some of the library

Opinion on those refactors loosely held, happy to accomodate. But some of the same was recently merged in #1810, so I thought this rounds things up a bit.

Codecov Report

Merging #1833 (3111d9d) into main (661a242) will decrease coverage by 53.80%.
The diff coverage is 37.25%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@             Coverage Diff             @@
##             main    #1833       +/-   ##
===========================================
- Coverage   73.50%   19.71%   -53.80%
===========================================
  Files         410      410
  Lines       50510    50486       -24
===========================================
- Hits        37129     9953    -27176
- Misses      13381    40533    +27152

Flag
Coverage Δ

integration-tests
19.71% <37.25%> (+<0.01%)
:arrow_up:

unit-tests
?

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

Impacted Files
Coverage Δ

crates/executor/src/blockchain_tree/chain.rs
0.00% <0.00%> (-85.19%)
:arrow_down:

crates/executor/src/blockchain_tree/mod.rs
0.00% <0.00%> (-90.76%)
:arrow_down:

crates/net/discv4/src/lib.rs
15.69% <0.00%> (-50.27%)
:arrow_down:

crates/net/discv4/src/test_utils.rs
0.00% <0.00%> (-88.69%)
:arrow_down:

crates/net/downloaders/src/bodies/request.rs
0.00% <0.00%> (-85.46%)
:arrow_down:

crates/net/nat/src/lib.rs
27.33% <0.00%> (-17.67%)
:arrow_down:

crates/net/network/src/eth_requests.rs
76.69% <0.00%> (+0.73%)
:arrow_up:

crates/primitives/src/block.rs
28.40% <0.00%> (-48.80%)
:arrow_down:

crates/revm/revm-inspectors/src/tracing/types.rs
0.00% <0.00%> (ø)

crates/rpc/rpc-engine-api/src/engine_api.rs
0.00% <0.00%> (-92.21%)
:arrow_down:

... and 8 more

... and 262 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

Oops, sorry for merging this prematurely.
looks like ruftmt has problems formatting let else?

Looks like it. A bit annoying


Uninitialized memory reads and out-of-bound in 1.0.12

Hello.
I've done some testing using american-fuzzy-loop. This tool found few serious bugs.
valgrind output:
==85864== Invalid read of size 8
==85864==    at 0x4E3D172: yajl_buf_append (in /lib/libyajl.so.1)
==85864==    by 0x4E3E0F0: yajl_gen_number (in /lib/libyajl.so.1)
==85864==    by 0x401A40: reformat_number (input.c:25)
==85864==    by 0x4E3CD93: yajl_do_parse (in /lib/libyajl.so.1)
==85864==    by 0x4012C0: main (input.c:149)
==85864==  Address 0x60540c2c8 is not stack'd, malloc'd or (recently) free'd
==85864==
==85864==
==85864== Process terminating with default action of signal 11 (SIGSEGV)
==85864==  Access not within mapped region at address 0x60540C2C8
==85864==    at 0x4E3D172: yajl_buf_append (in /lib/libyajl.so.1)
==85864==    by 0x4E3E0F0: yajl_gen_number (in /lib/libyajl.so.1)
==85864==    by 0x401A40: reformat_number (input.c:25)
==85864==    by 0x4E3CD93: yajl_do_parse (in /lib/libyajl.so.1)
==85864==    by 0x4012C0: main (input.c:149)
==85864==  If you believe this happened as a result of a stack
==85864==  overflow in your program's main thread (unlikely but
==85864==  possible), you can try to increase the size of the
==85864==  main thread stack using the --main-stacksize= flag.
==85864==  The main thread stack size used in this run was 8388608.
--85864-- REDIR: 0x50c64f0 (libc.so.6:free) redirected to 0x4c2ed80 (free)
Example PoC:
[[{} ,[[[{} ,[{} ,[[],[[[9],[9],[],[[[9],[[[[],[[9],[9],[[9],[[[[],[[9],[[[9],[[],[[[[],[[9 ,[{} ,[[],[[[9],[9],[],[[[9],[[[[[9],[[[[],[[9],[9],[7],[],[[[9],[[[[[9],[[[[],[[9],[[[9],[[],[[[[],[[9],[9],[],[[[9],[[[[[9],[[[[],[[],[[9],[[[[],[[9],[[],[],[]],[[]],[],[],[[9],[[],[],[[],[[],9],[],[[[9],[[[[[9],[],[9],[],[[[9],[[[[[9],[[[[],[[9],[[[[],[[9],[[],[],[]],[[],[[9],[[9],[9],[],[[[9],[[[[[9],[[[[],[[9],[[[[],[[9],[[],[],[]],[[]],[],[],[[9],[[],[],[[],[[],[[[9],[9],[],[[[9],[9],[[[[],[[9],[[[[],[[9],[[],[],[]],[[[9],[9],[[[[],[[9],[[[[],[[9],[[],[],[]],[[]]C[],[],[[9],[[],[],[[],[[],[[[9],[9],[],[[[9],[[],[[[[],[[9],[[[[[
PoC has been tested on code from: http://lloyd.github.io/yajl/yajl-1.0.12/index.html

I can reproduce this on the 1.x branch, but I can't reproduce in master. A git bisect suggests it was fixed by #25, so anyone on any 2.x.y release and/or master should be ok.

A long time ago, I have done a ticket:

https://github.com/lloyd/yajl/issues/219


feat: Support app deletion properly

Codecov Report
Attention: Patch coverage is 36.36364% with 21 lines in your changes missing coverage. Please review.

Project coverage is 46.88%. Comparing base (b943d7e) to head (8691ced).

Files
Patch %
Lines

principal/event.go
0.00%
10 Missing :warning:

internal/manager/application/application.go
75.00%
4 Missing :warning:

agent/outbound.go
0.00%
3 Missing :warning:

principal/callbacks.go
0.00%
3 Missing :warning:

internal/event/event.go
0.00%
1 Missing :warning:

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #89      +/-   ##
==========================================
- Coverage   46.92%   46.88%   -0.05%
==========================================
  Files          51       51
  Lines        3207     3227      +20
==========================================
+ Hits         1505     1513       +8
- Misses       1568     1580      +12
  Partials      134      134

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


x/net/http2: misbehaved streams can cause connections to exhaust flow control

Please answer these questions before submitting your issue. Thanks!
What version of Go are you using (go version)?
go1.11 darwin/amd64
Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
GOARCH="amd64"
GOBIN=""
GOCACHE="/Users/jnewman/Library/Caches/go-build"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/jnewman/go"
GOPROXY=""
GORACE=""
GOROOT="/usr/local/go"
GOTMPDIR=""
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
GCCGO="gccgo"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/_t/hg9f_j4x1q743pqh00kdv6m00000gn/T/go-build803039253=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do? / What did you expect to see? / What did you see instead?
Use httputil.ReverseProxy with an http2.Transport. I'm still trying to find a minimal reproducible example.
Basically, the http2.Transport is closing an HTTP/2 stream and then sending a number of DATA frames on the connection. The http2.Server is detecting that the stream has been half-closed by the remote and is following RFC7540 and responding with an ErrCodeStreamClosed (see http2/server.go).
However, since the http2.Server does not send a window update the remote http2.Transport does not add back the sent DATA frames to its flow control. This causes a re-used http2.Transport to eventually expend all connection flow control and halt sending.
A proposed fix that I've applied to my fork is to make the following patch
diff --git a/http2/server.go b/http2/server.go
index 56859d1..7a39456 100644
--- a/http2/server.go
+++ b/http2/server.go
@@ -1586,6 +1586,8 @@ func (sc *serverConn) processData(f *DataFrame) error {
        // "open" or "half-closed (local)" state, the recipient MUST respond with a
        // stream error (Section 5.4.2) of type STREAM_CLOSED.
        if state == stateClosed {
+               sc.inflow.take(int32(f.Length))
+               sc.sendWindowUpdate(nil, int(f.Length))
                return streamError(id, ErrCodeStreamClosed)
        }
        if st == nil || state != stateOpen || st.gotTrailerHeader || st.resetQueued {

Although, one could argue that http2.Transport/httputil.Reverse proxy should be fixed to make sure that data frames aren't sent after a close.

I think this is happening since i'm cancelling the upstream request which is causing the http.Request.ctx to be cancelled, which causes the http2.Transport to send a RESET_STREAM frame (see http2.Transport.awaitRequestCancel), but the writer is still writing DATA frames (see http2.Transport.writeRequestBody).

I think this might be related to https://github.com/golang/go/commit/8f38f28222abccc505b9a1992deecfe3e2cb85de#diff-d863507a61be206d112f6e00e6d812a2 since it appears that the http.Server marks the stream as stateClosed if the handler panics, even though the remote side hasn't sent a RESET_STREAM frame. Therefore, the stream should be in "half-closed (local)" and so should send back WINDOW_UPDATE frames but it's not.

/cc @bradfitz

Can you capture the logs (from setting GODEBUG=http2debug=2) while this happens?

I can capture some debug logs, I was using then to debug this and had to
add my own logging to capture the flow control values. Would you like me to
keep my log statements in there too?
On Tue, Oct 16, 2018, 7:58 PM Brad Fitzpatrick<EMAIL_ADDRESS>wrote:

Can you capture the logs (from setting GODEBUG=http2debug=2) while this
happens?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/golang/go/issues/28204#issuecomment-430472009, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ACu-MlsmdhrhqhK-x99LEqbcpPVLjVLHks5ulpzhgaJpZM4XbrWd
.

Hey @prashantv - glad to see you can repro my issue! I looked over your example and it's possible you're also being hit by https://github.com/golang/go/issues/28634. Although, even with patches for this issue and #28634, I was still seeing some flow control issues when using grpc with a context with a timeout...

@prashantv You are correct. I did introduce a bug, but there is another one there too. Once we receive a data frame that does not cause a connection error, we must account for the data. The take() logic should be before all the additional state checks regardless of the path taken.
I believe there may also be a similar bug on the client side.

You don't need a misbehaved stream to trigger this. If the server doesn't read the request body, flow control gets exhausted.
Here's a client/server where the server ignores the body on the first few POST (it just WriteHeader). The client blocks because it's available() flow is 0.
It's broken (server-side) from 1.11.0 through 1.12beta2, and fixed in 1.12rc1 by https://github.com/golang/net/commit/ed066c81e75eba56dd9bd2139ade88125b855585 . We see this in production when the client is Chrome, as well as Go, so I don't think it's a misbehaving client.
Client
package main

import (
	"bytes"
	"log"
	"net/http"
)

func main() {
	for {
		body := bytes.Repeat([]byte{0}, 128*1024*1024)
		_, err := http.Post("https://myhost:8443", "", bytes.NewReader(body))
		if err != nil {
			log.Fatal(err)
		}
	}
}

Server
package main

import (
	"fmt"
	"io/ioutil"
	"log"
	"net/http"
)

func main() {
	var i int
	http.HandleFunc("/", func(w http.ResponseWriter, r *http.Request) {
		fmt.Println(i)
		i++
		if i <= 5 {
			w.WriteHeader(http.StatusAccepted)
			return
		}
		if _, err := ioutil.ReadAll(r.Body); err != nil {
			log.Fatal(err)
		}
		w.Write([]byte("OK"))
	})
	log.Fatal(http.ListenAndServeTLS(":8443","cert.pem","cert.key",nil))
}

To make this easier for others to find, could the issue's title be updated to something like "Not reading request body can cause connections to exhaust flow control"?

To make this easier for others to find, could the issue's title be updated to something like "Not reading request body can cause connections to exhaust flow control"?

The title should make it clear whether this is a Server or Transport issue. Which is it? I see discussion of both in here. (I haven't looked into this yet.)

I don't know enough to say if it's Server or Transport, but I guess Server. It's an accounting error in http2serverConn.flow, in the connection-level flow control.
What I think is happening is that when the HTTP handler returns without reading the body (say it returns a 404), any DATA frames in flight are counted towards the connection-level flow control by the client, but not by the server, so they get out of sync.
Once the HTTP handler returns the server closes the stream like this:

http2: Framer 0xc00013c1c0: wrote HEADERS flags=END_STREAM|END_HEADERS stream=5 len=26
http2: Framer 0xc00013c1c0: wrote RST_STREAM stream=5 len=4 ErrCode=NO_ERROR

The in-flight DATA frames are considered invalid:

http2: Framer 0xc00013c1c0: wrote RST_STREAM stream=5 len=4 ErrCode=STREAM_CLOSED

If I print out the flow control values (http2flow.n) at the connection level, and use the client/server from my previous comment, when it blocks the server has 1,073,807,359 bytes but the client has 0.
This is the same as @prashantv described above.
They added a clarification to the spec for this:

A receiver that receives a flow-controlled frame MUST always account
for its contribution against the connection flow-control window,
unless the receiver treats this as a connection error
(Section 5.4.1).  This is necessary even if the frame is in error.
The sender counts the frame toward the flow-control window, but if
the receiver does not, the flow-control window at the sender and
receiver can become different.

In http2 the flow control value are always relative, which makes problems like this very difficult to catch (it survived all of Go 1.11.X). Personally I think the inability to re-sync flow control values makes the protocol brittle.
I started a discussion about that on the IETF HTTP Working Group's mailing list, here https://lists.w3.org/Archives/Public/ietf-http-wg/2019JanMar/0155.html .
The emerging consensus seems to be that the protocol is fine but better debugging support would be welcome.

Heya, I was looking at the latest http2 server code and this looks to be fixed here. Is this issue worth closing?

The issue reported was fixed.


RigidDynamicBody3d > RigidBody3D

RigidDynamicBody3d was renamed RigidBody3D recently, this is a small change to update that in tbloader

Thanks! Was this done for Beta 10?

I think it was it Alpha 15 at the end of August? Relevant MR for Godot is here.

That was quite a while ago! Thank you 🎉


To add support for websocket

Karate supports HTTP protocol, it would be great if it can support Websocket.

closing due to inactivity

@4M01 and others - reopening this, and we've made some progress adding websocket helpers to the code. can you provide some examples of test scenarios / requirements

@ptrthomas I will try to provide some example for this by Sunday. Hope that is fine

commit message was not descriptive enough, so a few notes:

doc pending
not sure if websocket should be first-class keyword and if clients should be auto-closed at end of scenario
two new methods on karate JS API, listen() and signal()
the combination of JS and karate works very nicely for callbacks !

I think the new "native" support for an "await" has turned out really well.
the CDC example which includes a JMS message listener has been revamped in prev commit.
Feature: payment service

Background:
* def QueueConsumer = Java.type('mock.contract.QueueConsumer')
* def queue = new QueueConsumer(queueName)
* def handler = function(msg){ karate.signal(msg) }
* eval queue.listen(handler)
* url paymentServiceUrl + '/payments'

Scenario: create, get, update, list and delete payments
    Given request { amount: 5.67, description: 'test one' }
    When method post
    Then status 200
    And match response == { id: '#number', amount: 5.67, description: 'test one' }
    And def id = response.id
    * json shipment = karate.listen(5000)
    * print '### received:', shipment
    * match shipment == { paymentId: '#(id)', status: 'shipped' }

and something I just found out is that function(o){ } gets converted to java.util.function.Consumer<Object> via the magic of Nashorn / JVM JS interop !
which leads us to the code behind the queue.listen(handler) that you see in the Background above:
    public void listen(java.util.function.Consumer<String> handler) {
        setMessageListener((message) -> {
            TextMessage tm = (TextMessage) message;
            try {
                handler.accept(tm.getText());
            } catch (Exception e) {
                throw new RuntimeException(e);
            }
        });
    }

and yes, instead of String, the usual rules of JS conversion (e.g. Map --> JSON) apply
the improved API is designed so the karate.listen(timeout) call will return the value passed to karate.signal(value) which makes things pretty elegant if I may say so
which gives us this beauty that can send as well as receive from a websocket connection while ignoring messages we are not interested in:
    * def handler = function(msg){ if (msg.startsWith('hello')) karate.signal(msg) }
    * def socket = karate.websocket(demoBaseUrl + '/websocket', handler)
    * eval socket.send('Billie')
    * def result = karate.listen(5000)
    * match result == 'hello Billie !'


failed to decode envelope: unexpected end of JSON input

Errors like these may occur in the standard output:
go-flutter: failed to pop route after escape key press: failed to decode envelope: unexpected end of JSON input
This is the case because InvokeMethod always expects a response which it wants to decode, but we don't always get a response. Currently responses are not implemented (https://github.com/flutter/flutter/issues/18852), and even when they are, some methods don't return a response.
Java has InvokeMethod vs InvokeMethod with callback. Perhaps we should also provide two methods, one which doesn't expect a response and therefore doesn't try to decode it (resulting in decode error)

fix taged in tag: v0.29.0


jakogut/makefile fixes

Looks good to me, I have another cleanup of this file in progress over here but I will rebase with your changes.
https://github.com/balena-os/leviathan/pull/547 (5th commit)


ibc: move IBC GRPC queries into grpc service

Is your feature request related to a problem? Please describe.
Previously, due to implementation details of the Go cosmos relayer, we implemented a bunch of the required IBC GRPC queries in the pd tendermint ABCI query interface, in info.rs. This is suboptimal and incompatible with Hermes.
Describe the solution you'd like
Create a tonic GRPC service that wraps these IBC query RPCs, and expose it in the pd GRPC api.

I think we will want to maintain the existing stub translation code in the ABCI query interface for now, at least while we're still using the Go relayer, rather than removing it entirely.  But once we have a real GRPC service implementation, we can have the translation code call it, rather than process the requests directly.

WIP https://github.com/penumbra-zone/penumbra/pull/2954


Using the unmanaged DNS provider does not work

How to categorize this issue?

/area usability
/kind bug
What happened:
When creating a shoot with the unmanaged DNS provider (setting .spec.dns.providers[0].type=unmanaged) it fails with:
Error from server (Forbidden): error when creating "dev/local/shoot.yaml": shoots.core.gardener.cloud "my-shoot" is forbidden: 2 errors occurred:
	* given Shoot uses non-registered dns type: spec.dns.providers[0].type ("unmanaged")
	* given Shoot uses non-registered extension type: spec.dns.providers[0].type ("unmanaged")

What you expected to happen:
Shoot creation should succeed.
How to reproduce it (as minimally and precisely as possible):
See above
Anything else we need to know?:
Do we still need this functionality? It obviously seems to be unused. Should we consider dropping it?
cc @timuthy
Environment:

Gardener version: master

/assign

Do we still need this functionality? It obviously seems to be unused. Should we consider dropping it?

I was about to remove the functionality entirely from the code since this particular use case hasn't worked for ~9 months (see https://github.com/gardener/gardener/commit/322ba4a7e31783036566c8b409f97fc5a92bd744). However, unmanaged providers can also be used for default and internal domains and we simply don't know if this feature is used or not. At least I'm reluctant to remove this feature w/o any deprecation period and migration path. #5241 fixes the problem for now.

OK, thanks @timuthy. Shall we then close this ticket for now? It was created under the assumption that everything related to unmanaged providers does not work. I assume that we don't want to spend the effort of deprecating and migrating existing systems with unmanaged providers?

I would also suggest to close this issue. We can still tackle the removal in the future if there are strong arguments to do so.

OK, then let's merge your PR and close this issue with it. Thanks!


Setting up client timeout

How can a client control the timeout for executing a command with Redis? To prevent back pressure issues, it'll be nice to setup a timeout on the client side to abandon operations running longer than X ms.

The high water mark is documented from now on and can be used to circumvent this issue.


No warning or error for volumeMounts without matching volume

Is your feature request related to a problem? Please describe.
Extract from example Deployment/Pod with a typo on the volume name. kubevious guard gives the all OK ✅ .
              volumeMounts:
                - mountPath: /data
                  name: storage-volutypo
          volumes:
            - configMap:
                name: prometheus-server
              name: config-volume
            - name: storage-volume
              persistentVolumeClaim:
                claimName: prometheus-server-efs

@frimik, thanks for reporting it.
I've added a rule to validate volume mounts. It is also aware of volumeClaimTemplates and will work in StatefulSets.
This capability requires version 1.0.55, so to try it out, install it using npm. It will take some time until it is available on homebrew.
$ npm install -g<EMAIL_ADDRESS>

The rule catches it now, perfect!


Solve scrollbar changes pr373

Fixes #(issue number)
Describe the changes you've made
A clear and concise description of what you have done to successfully close your assigned issue. Any new files? or anything you feel to let us know!
Deploy your changes to github page or netlify and mention deployment link:
Add the deployment link here.
Describe if there is any unusual behaviour of your code(Write NA if there isn't)
A clear and concise description of it.
Additional context (OPTIONAL)
Add any other context or screenshots about the feature request here.
Test plan (OPTIONAL)
A good test plan should give instructions that someone else can easily follow.
How someone can test your code?
Checklist

[x] I have mentioned related issue number in PR description.
[ ] I have added screenshots and related demo links of my changes..
[x] The title of my pull request is a short description of the requested changes.
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] Open Source Program names(OPTIONAL: If you participated in any open-source program then please mention the program name here)

Testing link: https://cobaltmon.github.io/Movie-Streaming-Website/home.html

No, not that.
The effect is fine on every page.
Make it like before.
Change the effect only on movies, web series, and kids pages and make them like other pages.

Oh that. Understood

Hi. Can you check if its been corrected?

Nope.
Check this is current effect https://qazimaazarshad.github.io/Movie-Streaming-Website/crime.html.
All scroll bars should be like this. Only movies, web series, and kids pages are different make them similar.

I suggest create a new PR.


About experiments

Thank you very much for your excellent study. I have some questions about the experiment:
Are there any additional experiments on the performance of FOMO in the OWD benchmark results?
How is the inference speed (FPS) evaluated?

Hi @xxyzll,
The supplementary has all the typical results on the OWD benchmarks (M/S-OWODB in Table 8, and the WI/A-OSE in Table 9).
I don't believe I ever evaluated FPS -- I generally don't do that because it is very machine-daepenant (e.g., if you run inference on an A100, you would have a higher FPS then on a titanRTX). If you wanted to evaluate FPS on your system, I would just run inference on some 1000 images and see how long that takes (eval and train of course have additional steps, which make them slower).
Best,
Orr