'Select all' checkbox callback

This function solves the problem (这个功能解决的问题)
Hi!
I cant find callback that represent click on checkbox "Select all" in header of table?
How can I change behavior of that event, I want to involve 'reverse selecting'.
Expected API (期望的 API)
Prorpose to involve 'on-select-all' callback to change behavior of checkbox.

https://www.naiveui.com/zh-CN/os-theme/components/data-table#select
You can look at the on-update:checked-row-keys event of this demo, you can get the selected keys.

Closed first, if you still have questions, I will reopen.

Sorry for so long pause.
But the problem is  that mentioned callback run always:

when you press 'Select all'
when you press any checkbox.
But i need to catch only 'Select all' - cause I need to reverse currently selected checkboxes

You want to reverse the current page selection? Why it's important?
I think it should be done in a dropdown menu like this. We can add more selection type.

Thanks you guys, I solve this by setting event on "thead .cp-checkbox-box__border"
But now one more question: do we have option to select how "Select All" box should behave with pagination:

select all on current page;
select all on all pages.

You want to reverse the current page selection?
I think it should be done in a dropdown menu like this not in header checkbox. We can add more selection type.

Thank you.
In Selection callback we have Pagedata, which represent data on current page.
How can I get this out of that callback?

ow can I get this out of that ca

Currently click on checkbox won't emit corresponding row data.

We may consider add row data for onUpdateCheckKeys callback. However the behavior of header checkbox won't be changed.

Thank you, I accessed pageData via ref="table" on data-table component.
In this case if you need access pagedata anywhere:
table.value.paginatedData
So I created reverse selecting like this:
 let checkedArray: never[] = []   if (table) {     table.value.paginatedData.map((item: { key: never }) => !currentCheckedRowKeys.value.includes(item.key) && checkedArray.push(item.key))   }   currentCheckedRowKeys.value = checkedArray

Closed as implemented in 0a7181a87f401b2b57f579bc98bc3aaeab1f9e49.


Empty context

Hi,
I have noticed that there are some blank lines in the extracted context files, as if there were no context for certain sentences.
For instance, line n. 7697 in file OpenSubs.c1.context.src, right after sentence "One mile from the exit, it starts shaking, violently shaking like it's having a nervous breakdown."
Is this normal?
Thanks

I checked the original data (in the separate documents from OpenSubtitles) and lines 7697 and 7698 in OpenSubs.current.src ("At the health club, people are working out and they're training...") correspond to the first lines of a document (1992/1992_697738_4618033.en). So that's why these two lines are empty in OpenSubs.c1.context.src - there is no previous context to give and the choice was to leave them blank to indicate the beginning of documents. You might want to replace them with a symbol indicating that these are the start of a document, explicitly saying that there is no context.

OK, thank you!


Update PR template: rework tests related action item wording

Proposed change

Update tests checkbox wording.
Type of change

[ ] New country/market holidays support (thank you!)
[ ] Supported country/market holidays update (calendar discrepancy fix, localization)
[x] Existing code/test/process quality improvement (best practice, cleanup, refactoring, optimization)
[ ] Dependency upgrade (version update)
[ ] Bugfix (non-breaking change which fixes an issue)
[ ] Breaking change (a code change causing existing functionality to break)
[ ] New feature (new python-holidays functionality in general)

Checklist

[x] I've followed the contributing guidelines
[x] This PR is filed against beta branch of the repository
[x] This PR doesn't contain any merge conflicts and has clean commit history
[x] The code style looks good (make pre-commit)
[x] I've added tests to verify that the new code works and all tests pass locally (make test, make tox)
[x] The related documentation has been added/updated (check off the box for free if no updates is required)

Coverage: 99.885%. Remained the same when pulling 4a797ceac9f3a6f279bde7ef207ec25637afa71d on arkid15r:update-pr-template into 4a72729acf0df368f798857a350592ea25565a3a on dr-prodigy:beta.


User self registration is visible though it was disabled from super tenant

Describe the issue:
In 5.11.0 pack user self registration feature is still available in myaccount login page though it was disabled from super tenant. when a user tries to register it throws ‘self registration is disabled for tenant - carbon.super’ error.
https://user-images.githubusercontent.com/28682701/170188287-09e56f6d-4aa8-4206-be1c-beffb6f9faa2.mp4
How to reproduce:

Login to the admin console.
Navigate to the Manage->User onboarding.
Disable user self registration under self registration.
Navigate to the my account https://localhost:9443/myaccount

Expected behavior:
Since the user self registration feature was disabled in super tenant, it shouldn’t be available in myaccount login page.
Environment information (Please complete the following information; remove  any unnecessary fields) :

Product Version: IS 5.11.0
OS:  Linux

Optional Fields
Related issues:

Suggested labels:

This issue is being closed due to extended inactivity. Please feel free to reopen it if further attention is needed. Thank you for helping us keep the issue list relevant and focused!


Update RFC:7451 description with extension introduced in XSD definition 2.0

The epp_nameserver_admin_v1 branch of the specification and the sister branch for the XSD definition holds changes to the extension portfolio, even though this is a small thing the RFC:7451 contribution should be updated accordingly.

Changed titled, since the official revision of the XSD is 2.0, it includes changes proposed in 1.5 and 1.6

Minor rephrasing in the description in should cover the change, since we now also have extensions to host related commands.
Ref: https://github.com/DK-Hostmaster/epp-service-specification/commit/c24f35e183cdf27cd2d2574377616611091396bf

As described in RFC:7451, the modifications have been sent to IANA (ref: https://tools.ietf.org/html/rfc7451). This will result in an update to the list available at: http://www.iana.org/assignments/epp-extensions/epp-extensions.xhtml


Issue 5403 - foreach requires front to be a function in a range

Allow using any reasonable symbol as front/back/empty.
Remove references to the old range interface (head/toe/next/retreat)
http://d.puremagic.com/issues/show_bug.cgi?id=5403

Please add test cases!

@WalterBright
Rebased, added test cases (and made it actually work)


dubbo-test 这个下面的测试用例哇,都报一样的错误

cannot reproduce. it works on my laptop. there's one thing you may need to pay attention to, which is, you need to specify -Djava.net.preferIPv4Stack=true before you launch the application.


Wind Speed Max

I am using pytrafikverket via Home Assistant, but I am missing a measurement that's most relevant to me.
Is it possible to add Measurement.Wind.ForceMax to the mix? I played around with Trafikverket's API console and found that it existed there, with that name. I also tried to patch this library and the Home Assistant module, but without success. Not sure where I failed in the flow. I am not too experienced with Python.

Hey there, if it is in the api we could certainly add it.
It has to be added to both pytrafikverket library and home assistant in that order.

That's what I tried, but failed with.
I made these two patches:
https://pastebin.com/LP5Nd8Br
https://pastebin.com/UtWGa5Pw
It's likely that I am missing something. My understanding for Python isn't too great, and even less how Home Assistant's modules works. It seemed simple though, besides the fact that it didn't work. :)

https://github.com/home-assistant/home-assistant.io/pull/14256

pytrafikverket is updated and pull requests created for home assistant docs and home assistant sensor code for trafikverket weatherstation

It's now available in the 115 release of Home Assistant


Linting for no-use-before-define

What changes did you make?

Enables no-use-before-define rule

Is there a ticket that you are fixing?
Please link it here if so
Changelog

[x] I have added a Changie entry or given a reason why this does not need an entry in this section.

Not needed for linting

Updated this to point to the feature branch


Get-DbaAgentOperator Repair

Type of Change

[X ] Bug fix (non-breaking change, fixes #)
[ ] New feature (non-breaking change, adds functionality)
[ ] Breaking change (effects multiple commands or functionality)
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Purpose
Fix the misbehaviour: this function did return only the operators of the first sql instance.
Approach
A parameter "Operator" was added, but this was already used in a foreach loop. After the first iteration through the sql instances, this parameter would contain the result of the last iteration through the operators of the first instance. For all consecutive sqlinstances, the check for operators being equal to the $operator would return nothing.
Renaming the variable in the 'foreach operator' loop, solved this.

Merci, my friend 🌈


"autocomplete" property

The form and input elements make mention of the "autocomplete" option but there is no corresponding property in purescript-halogen/src/Halogen/HTML/Properties/Indexed.purs.

Yes... I wrote the refinements "exhaustively", even though we have not yet implemented every HTML attribute. So there are likely other examples of this.
If I don't get around to adding this soon enough, PRs are always welcome!

On Oct 14, 2015, at 7:35 AM, Michael Swan<EMAIL_ADDRESS>wrote:
The form and input elements make mention of the "autocomplete" option but there is no corresponding property in purescript-halogen/src/Halogen/HTML/Properties/Indexed.purs.
—
Reply to this email directly or view it on GitHub.


Operation serialization

will be implemented in https://github.com/ScorexFoundation/sigmastate-interpreter/pull/253


Bugfix for Issue #850.

This fixes bug #850. Instead of committing the transaction on the RESOURCE_METHOD_FINISHED event, which is emitted from Jersey before the ON_EXCEPTION event, the transaction is commited on RESP_FILTERS_START event, which is emitted after the ON_EXCEPTION event.

Thanks for the fix.  I'm not familiar enough with hibernate to know if this is a correct fix or not.  Is there someone else more familiar who can comment?

I will take a look today on this.

Looks great and works correctly. I've squashed the commits and pushed to master.
@phxql, thanks for contribution!


share/usbid: Don't print error when missing

It's always missing during lxd init when run from the snap, even if
the daemon itself has access to it.
Closes #7256
Signed-off-by: Stéphane Graber<EMAIL_ADDRESS>

@stgraber why is the database always missing on startup btw?

lxd init is run prior to the daemon starting and it's the daemon startup script which populates the snap mount namespace.

@stgraber I see, thanks.


fix ignore stacktrace for error code INVALID_ARGUMENT INVERTED_INDEX_NOT_IMPLEMENTED

Proposed changes
Issue Number: close #xxx

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall

run buildall


Website: update bug bounty information on developers page

Once I have the up to date info I can update the design and assign to a dev
Also need to update licenses in footer:

@matt-bullock Do you have the updated info/design?

@tomkysar Could you get us the updated info for this page


Print geojson lines according to WGS84

According to the RFC

The coordinate reference system for all GeoJSON coordinates is a  geographic coordinate reference system, using the World Geodetic System 1984 (WGS 84)

Thus, lines represented in geojson are not necessarily straight as they are currently printed in UI.

A line between two positions is a straight Cartesian line, the  shortest line between those two points in the coordinate reference  system

This can make huge differences especially for large lines/polygons

This GeoJSON polygon
{
  "type": "FeatureCollection",
  "features": [
    {
      "type": "Feature",
      "properties": {},
      "geometry": {
        "coordinates": [[[-55,  35 ],[58,35],[58,55], [-55,55],[-55, 35]]],
        "type": "Polygon"
      }
    }
  ]
}

is printed in geojson.io shuch as:

And should be printed such as:

Thanks for your comments.  The rendering you see here is done by Mapbox GL JS, which is projecting the WGS84 coordinates into webmercator (EPSG 3857).  The globe view further transforms the webmercator tiles to create the globe effect, but the line drawn between two points is the shortest distance between them in the webmercator coordinate plane, not the shortest distance on the surface of the earth.
Closing as this isn't actionable in geojson.io


Sanitizing JSON in release_notes parameter

Fixes https://github.com/fileformat/bitrise-step-appcenter-app-release/issues/15
The trimming of release notes could cause side effects and might be better off behind a flag (sanitize_release_notes maybe?), but this should solve the very common case where a GitHub commit message is the release note and it contains line breaks.

I believe https://github.com/fileformat/bitrise-step-appcenter-app-release/commit/53de7d586a5118ba8931c7d72f87fdfd059ec627 encodes the user input now, so this PR is no longer necessary.


The scale should filter out any ticks that fall within discontinuities

Looking at this example, the scale has a gap between 5-20, however, the axis has a tick at '6'. This should be filtered out.
https://bl.ocks.org/ColinEberhardt/b60919a17c0b14d745c881f48effe681
The current tick filter implementation simply removes overlapping ticks.

I'm having a similar problem with using d3.scaleTime() with discontinuitySkipWeekends(). The timeline shows Sun and Mon removed and still shows Sat. Do you know if this is likely the same issue?

Yes, that's the same issue. The current tick logic 'clamps down' each tick value, then removes duplicates. As a result a tick that is within a discontinuity may still be rendered. In some senses this is correct, but confusing for the user.
We should make fixing this a priority!


use port 4445 for selenium server; closes #229

I want to merge this but we need to run tests on it before we can do that. Any ideas why tests didn't run? @sedubois

@asood123 don't know top of my head either. Related to being a fork? @boneskull can you also rebase on master just to be sure?

as per @asood123 i'm going to re-open this from the main repo

@asood123 I'll need to have write access here too

(was going to make a new branch here and re-create this PR from it)

(unless of course you'd like me to figure out why the PR's aren't running)


intent recognition does not work

Rasa NLU version: 0.13.3
Operating system (windows, osx, ...): ubuntu 16.04
Content of model configuration file:
language: "en"

pipeline:
  - name: "tokenizer_whitespace"
    #  - name: "intent_entity_featurizer_regex"
  - name: "ner_crf"
  - name: "ner_synonyms"
  - name: "intent_featurizer_count_vectors"
  - name: "intent_classifier_tensorflow_embedding"

path: "../model/current/mondelez_nlu_model"
data: "../data/"

    #pipeline:
    #  - name: "nlp_spacy"
    #    model: "en_core_web_lg"
    #    case_sensitive: false
    #  - name: "tokenizer_spacy"
    #  - name: "intent_featurizer_spacy"
    #  - name: "ner_crf"
    #  - name: "ner_synonyms"
    #  - name: "intent_featurizer_ngrams"
    #    max_number_of_ngrams: 10
    #  - name: "intent_classifier_sklearn"

Issue:
intents never get recognized when online training. recognized intent name is always user's text speech:
Chat history:
    bot did:        action_listen

    user said:      yo

             whose intent is:       {'confidence': 1.0, 'name': 'yo'}

    bot did:        action_listen

    user said:      hello

             whose intent is:       {'confidence': 1.0, 'name': 'hello'}

    bot did:        action_listen

    user said:      hey

             whose intent is:       {'confidence': 1.0, 'name': 'hey'}

the 'greet' json data am using is from github examples, and it looks like this:
{
"rasa_nlu_data": {
"common_examples": [
{
"text": "yo",
"intent": "greet",
"entities": []
},
{
"text": "hey",
"intent": "greet",
"entities": []
},
{
"text": "howdy",
"intent": "greet",
"entities": []
},
{
"text": "hey there",
"intent": "greet",
"entities": []
},
{
"text": "hello",
"intent": "greet",
"entities": []
},
{
"text": "hi",
"intent": "greet",
"entities": []
},
{
"text": "good morning",
"intent": "greet",
"entities": []
}
]
}
}

Can you share the command you're using to do online training?

looks like you're not passing a trained NLU model, please try adding this to your command -u models/nlu

Hi. Yes, that was it. Thanks.


Multiple Select Date.

Can this bootstrap-datetimepicker can have multitple select on the dates?

+1

+1

+1

I suspect it's not easy to extend the tool to include the selection of multiple date/times, but it would be great to have. Yours is the best DateTime picker I found, but I need the multiple selection option (though I need it only for the dates and not the times). I would use it to allow the user to schedule items, with exceptions on certain days, or alternatively to select the dates on which action need to take place. In either case I'm going to need multiple selections.
But even if you don;t build it in: big thumbs up for a datetime picker well built !!

+1

+1
Here is an example of old bootstrap datepicker http://bootstrap-datepicker.readthedocs.io/en/latest/options.html#multidate which is no longer maintained it seems. This is the playground: http://eternicode.github.io/bootstrap-datepicker/?markup=input&format=&weekStart=&startDate=&endDate=&startView=0&minViewMode=0&maxViewMode=2&todayBtn=false&clearBtn=false&language=en&orientation=auto&multidate=&multidateSeparator=&keyboardNavigation=on&forceParse=on#sandbox

+1

+1

or multiple select Hours

This has been implemented on V5 https://github.com/tempusdominus/bootstrap-3


Show indicators of state on session sidebar

I have applied the changes to the sidebar, but this can probably be improved. #610
such as:
session_status is similar to cell_status from cell_component.ex.
maybe we can unify (:
The result is this:

Thanks @jeantux! I've just pushed added support for scrolling to relevant cell on click:
https://user-images.githubusercontent.com/17034772/139667938-25ca1d76-4666-4281-922c-85f4a85715d3.mp4
@josevalim anything else we need?

Ship it! Thank you @jeantux!


Documentation for jax.pure_callback does not mention ShapeDtypeStruct

Description
The documentation page for jax.pure_callback states that result_shape_dtypes needs to be a PyTree whose leaves have shape and dtype members, but does not mention jax.ShapeDtypeStruct that already has those fields. An example in that documentation page would also be helpful, as would better errors for users who pass a raw shape instead of a structure as that argument.
What jax/jaxlib version are you using?
No response
Which accelerator(s) are you using?
No response
Additional system info
No response
NVIDIA GPU info
No response

Thanks for the report! I added this in #18075


feat: option to disallow public org registration

The PR adds the Admin APIs for limiting an instance and resetting an instances limits.
On the Admin API, the property disallow_public_org_registration is available and resettable, wheras on the System API, the property audit_log_retention is still available and resettable. Calling the reset endpoints from the Admin API and from the System API only reset their managed properties and don't interfere which each other.
Under the hood, it's the same limits aggregrate with the instance ID as resource owner, but that's not transparent to the API.
If a public org registrations are disallowed, the endpoint /ui/login/org/register returns the status 404 on GET and 409 on POST without rendering a form.
Besides that, the PR also:

fixes the permission requirements for the limiting methods on the system API (from authenticated to system.limits.write and system.limits.delete)
silences the integration tests
Adds the option to avoid updating certain columns on upserts. The option is used to only set the creation date when inserting a restriction. For rolling out to other projections, there is a follow-up PR #6930.

TODO

[x] DefaultInstance property
[x] Integration Test
[x] Docs

Definition of Ready

[ ] I am happy with the code
[ ] Short description of the feature/issue is added in the pr description
[ ] PR is linked to the corresponding user story
[ ] Acceptance criteria are met
[ ] All open todos and follow ups are defined in a new ticket and justified
[ ] Deviations from the acceptance criteria and design are agreed with the PO and documented.
[ ] No debug or dead code
[ ] My code has no repetitions
[ ] Critical parts are tested automatically
[ ] Where possible E2E tests are implemented
[ ] Documentation/examples are up-to-date
[ ] All non-functional requirements are met
[ ] Functionality of the acceptance criteria is checked manually on the dev system.

@stebenz can you approve again, plz?
Thanks @muhlemmer for fixing the linter.


Fix restriction of renegade refit in german.

Currently you are not able to equip renegade refit on U-Wings while the
language is set to german. Removing this line should fix this.

Ah, well. This is obviously the same as #589.
I did check if there is an issue on this bug (there is none), and decided it would be less work to write a pull request than to write an issue... did not check if there is already another pull request.


Add additional example to documentation

I have added another example that I find is missing from the documentation in order to implement MSAL for Angular in any project.

Can anyone review this small documentation change (@jo-arroyo, @tnorling or @peterzenz)?

@bvwidt Thank you for your interest in contributing to our library. Unfortunately, the html tags you are looking to add aren't strictly necessary for every application using MSAL Angular. While it is needed for applications using the MsalRedirectComponent, applications using popup or standalone Angular applications won't need them, so we will not be adding it to our initialization doc. Please see our doc on redirects for more details, and we welcome contributions from you in the future.


View variable in data viewer does not have a test

Test is a little hard to write.

Have to run debugger with a breakpoint from python (command should work)
Have to wait for breakpoint to hit
Have to figure out way to view variable in data viewer (command might work, not sure what the context is)
After that would be verification of the dataviewer opening, which we should already have.

@rchiodo @sadasant I actually wrote this as part of my data viewer fixing. Didn't realize there was a bug already. Closing.

@IanMatthewHuff thank you!


Update Packages WIP

I'm upgrading some vulcan packages to monorepos and cleaning up some dependencies.
Mostly this is centered around getting storybook, eslint, and babel packages correct.
I noticed that when I added the babel rules that Kevin suggested, the meteor build timed out.
Any suggestions welcome!

For this latest eslint stuff I keep getting this error:
  1:1   error    Definition for rule 'babel/new-cap' was not found                                                          babel/new-cap
  1:1   error    Definition for rule 'babel/array-bracket-spacing' was not found                                            babel/array-bracket-spacing

There is few chance that I can investigate this but ping me when you think it's mergeable. Ping @EloyID @juliensl  this might be useful to the platform


Version Bump needed

First off, thank you so much for this project!
The latest merged fix (expiresToSeconds) needs a version bump. The error still persists in the NPM package. Could you please update the version?

We're running into the same issue. Could we get a version update on the NPM package? Thanks!

Created a PR for this bump. Just need someone to merge it in: https://github.com/DAlperin/remix-redis-session/pull/8


NPM package doesn't provide the built versions

Hello, it seems the npm package doesn't provide either the browser minified package or the ES module built versions.
Is that a choice you made?

Yes, use the CDN npm package if you really want build assets.
https://www.npmjs.com/package/@highlightjs/cdn-assets

Yes, use the CDN npm package if you really want build assets.
https://www.npmjs.com/package/@highlightjs/cdn-assets


Repo keeps being 'bare' even if it is not

Gitea version (or commit ref):

$ ./gitea --version
Gitea version 6aacf4d

Git version:

$ git --version
git version 2.7.4

Operating system:

$ cat /etc/os-release
NAME="Ubuntu"
VERSION="16.04.1 LTS (Xenial Xerus)"
ID=ubuntu
ID_LIKE=debian
PRETTY_NAME="Ubuntu 16.04.1 LTS"
VERSION_ID="16.04"
HOME_URL="http://www.ubuntu.com/"
SUPPORT_URL="http://help.ubuntu.com/"
BUG_REPORT_URL="http://bugs.launchpad.net/ubuntu/"

Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[x] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[x] No
[ ] Not relevant

Log gist:

$ sudo journalctl -u gitea

Dec 28 15:24:05 DIR-101 gitea[26904]: [Macaron] 2016-12-28 15:24:05: Started GET /mura/repo-manifests for <IP_ADDRESS>
Dec 28 15:24:05 DIR-101 gitea[26904]: 2016/12/28 15:24:05 [D] Session ID: 18e0d643614cbd89
Dec 28 15:24:05 DIR-101 gitea[26904]: 2016/12/28 15:24:05 [D] CSRF Token: vbaOsqCBITYP1Sg1-JR3E0CfNCOOMTQ4MjkwMTIyMjQ1Mjk2MjkwNg==
Dec 28 15:24:05 DIR-101 gitea[26904]: 2016/12/28 15:24:05 [D] Bare repository: /mura/repo-manifests
Dec 28 15:24:05 DIR-101 gitea[26904]: 2016/12/28 15:24:05 [D] Template: repo/bare
Dec 28 15:24:05 DIR-101 gitea[26904]: [Macaron] 2016-12-28 15:24:05: Completed /mura/repo-manifests 200 OK in 25.286817ms

Description
It shows the 'Quick Guide' page like I had never pushed any file.
I looked into the code by finding the string Bare repo and the symbol IsBare and found there is not many way to set it to false (The expected correct value). Not sure what's going on but literally I can not add any new repo from now on.
P.S.

This can be reproduced with HEAD (f686a32e),
Using SQLite from shell to read data/gogs.db, it shows the column is_bare of the repo is actually 1.

$ sqlite3 data/gogs.db

In SQLite:
sqilte> select is_bare from repository where name == "repo-manifests";
1

It appears that this happens only when git push with SSH and HTTP does not exhibit the problem.

Hi, I cannot reproduce this. Could you give any further information?

@lunny I am trying to diagnose the problem further. Can you enlighten me about where the entry point is when users doing git push?
I suppose it is gitea serv (cmd/serve.go) but I am not sure since I didn't see the logging message I added there. 😕

Yes, you are right. When push via SSH, gitea serv will be invoked. Config file is ~/.ssh/authorized_keys file. You can find command="/Users/lunny/gopath/src/code.gitea.io/gitea/gitea serv key-1 ..., the default log will be write to log/serv.log. And notice don't print anything to console.

Push via SSH does through cmd/update.go (called from .git/hooks/update :)

It turns out this is caused by ~/.ssh/authorized_keys which has to be regenerated. (Thanks Denis Denisov @denji).


Working on v2

I have made some modifications since I became one of the maintainers, but I have not been able to actively participate in the development over the past few years.
I had discovered some problems but had put them off due to the complexity of the current code base.
Several major events have occurred during this time.

latex-utensils releases additional 3 major releases.

textlint-plugin-latex2e depends on v3.x. However, the latest is v6.1.
There are known to be some difficulties in updating.

Starting with textlint v13.0.0, all AST NodeTypes are now defined!

https://textlint.github.io/blog/2023/01/27/textlint-13.html#-breaking-changes

LaTeX Workshop, the VSCode extension that led me to introduce latex-utensils, unified-latex started being used

https://github.com/James-Yu/LaTeX-Workshop/issues/3877

This project is a viable solution for using textlint in LaTeX, even in the year 2024, with no alternative.
I began to think that it might be appropriate to rebuild the entire project once in order to maintain it on an ongoing basis.
This means building v2 from scratch.
Which parser library is better for us?
For v2 I re-evaluated several LaTeX parsers written in TypeScript or JavaScript.
Sample TeX source is as follows:
% This is the beginning of the document
\documentclass{article}  % This is a comment

\begin{document}

This is a paragraph that contains inline math: $E=mc^2$.
This is a second line.

% This is the beginning of the nested document
\begin{itemize}
    \item First item
    \item Second item
    \item Third item
\end{itemize}

\begin{tabular}{|c|c|c|}
    \hline
    Column 1 & Column 2 & Column 3 \\
    \hline
    Row 1 & Row 1 & Row 1 \\
    \hline
    Row 2 & Row 2 & Row 2 \\
    \hline
\end{tabular}

\end{document}

latex-utensils v6.2.0
Generated AST of sample tex source is here.

Pros

We already know this parser
Fast
Less dependencies

Cons

space, linebreak, softbreak do not have location information.
Comments can be parsed, but it is not in AST. We should calculate the position of them in AST.

completeComment func is implemented for this.

unified-latex v1.6.0
Generated AST of sample tex source is here.

Pros

Most whitespace has location information.
Comments are included in AST by default.

Cons

whitespace and parbreak around \item has no location information

https://github.com/siefkenj/unified-latex/issues/73
Maybe, we should implement the feature to calculate position of them.

whitespace may be a softbreak

latex-utensils parses it as softbreak: https://github.com/pddg/eval-js-latex-parsers/blob/88f3ed88f8da01540da3e73d64b4d076ad6c94fb/latex-utensils/output.json#L310-L312
unified-latex parses it as whitespace (softbreak is not defined in unified-latex): https://github.com/pddg/eval-js-latex-parsers/blob/88f3ed88f8da01540da3e73d64b4d076ad6c94fb/unified-latex/output.json#L514-L528

Large dependencies
Compatibility with ES module

https://github.com/siefkenj/unified-latex/issues/72
We can use it via "module": "ESNext" and "moduleResolution": "Bundler", now.

Slower than latex-utensils

https://github.com/James-Yu/LaTeX-Workshop/issues/3877#issuecomment-1544277944

completeBlank and completeComment is hard to maintain. I think it might be worth the transition.
LaTeX.js v0.12.6
(As far as I could tell) this could not be used as a mere parser.
It may be possible to define a Generator that generates textlint ASTs from LaTeX ASTs.
However, I do not know how to create a Generator. Document says TODO, now.
https://latex.js.org/api.html#class-generator
Create original one
Another option would be to make your own, but the reason other parsers produce such output in the first place is because that is the way TeX syntax is written.
Therefore, it is difficult to imagine that a home-built parser would be significantly easier for us to use.
Goal of v2

Correct AST of textlint is constructed from LaTeX source.

We should fix them:

#84
#69

Add snapshot test.

By verifying that the snapshot of expected output matches the actual output, we should be able to implement more tests more easily.

Add integration test if possible.

Ensure that violation of rule can be detected while using this plugin.
Ensure that fixable problem provided by some rules can be fixed while using this plugin.

Non Goal

Keep AST compatible with v1
Enhance unit tests currently implemented

How do we do
As maintainers, we should work on one of the following

Work on v2
Continue to improve on v1
Declare Deprecated and archive repository

Which one do you think is better? @tani @kn1cht

Thank you for revisiting our project.
I would like to express the gratitude for your deep survey and valuable comments.
I agree with your comment that our project depends on the old library and it should be updated to the latest version. Further, I am happy to hear that I could collaborate with you again. Let's reboot our project and make it better.
In my opinion, I would like to use unified-latex libarary. Even if we use the latex-utensils, we need to momdify the upstream code to make it work with our project. I think that it is time to change the library to unified-latex because it is more feature-rich. (For my personal preference, I tend to use the new library for better experience :wink:.)

Hello, I am glad to hear that you are doing well!
I also agree with working on v2.

whitespace and parbreak around \item has no location information

This may be resolved by parseMinimal method of unified-latex.
https://github.com/pddg/eval-js-latex-parsers/blob/e31b34348473fdecbc1f628c07d49fb9373c3f8f/unified-latex-minimal/output.json#L726-L740

Looking at the reference, markdown-to-ast, it appears that whitespace, etc. caused by List nesting is simply ignored.
Each ListItem has no whitespace before or after it, and whitespace or line breaks exist only in the raw value of the List.
We might simply remove all elements without position from the AST and then convert them.

I have written a sample that works to some extent, but have found additional problems and reported them to unified-latex.
https://github.com/siefkenj/unified-latex/issues/77#issue-2140725952

Because latex combines elements of formatting and text, it is often difficult to express in ASTs prepared for mere text.
I am currently struggling with the description environment. What kind of AST should this be expressed as in textlint?
\begin{description}
  \item[item1] desc
  \item[item2] desc
\end{description}

I have two ideas.
1. Consider headings and their descriptions to be lists.
- item1
  - desc
- item2
  - desc

pros

Easy to understand

cons

2. Consider the heading as a Header and the description as a Paragraph.
3. Consider headings and their descriptions as an independent Paragraph.

Because latex combines elements of formatting and text, it is often difficult to express in ASTs prepared for mere text.
I am currently struggling with the description environment. What kind of AST should this be expressed as in textlint?
\begin{description}
  \item[item1] desc
  \item[item2] desc
\end{description}

I have two ideas.
1. Consider headings and their descriptions to be lists.
- item1
  - desc
- item2
  - desc

textlint AST will be as follows:
List
  ListItem
    Paragraph
    List
      ListItem
        Paragraph
  ListItem
    Paragraph
    List
      ListItem
        Paragraph

pros

Nothing (i think)

cons

Need to generate elements of the list that don't actually exist.

2. Consider the heading as a Header and the description as a Paragraph.
- ## item1
  desc
- ## item2
  desc

textlint AST will be as follows:
List
  ListItem
    Header
    Paragraph
  ListItem
    Header
    Paragraph

pros

Simple

cons

TxtHeaderNode requires depth parameter that can be 1|2|3|4|5|6. However, we don't know what the depth to be used here.

https://github.com/textlint/textlint/blob/6cef9504c25a7ab771dd45986e4710463c094369/packages/%40textlint/ast-node-types/src/NodeType.ts#L142

3. Consider headings and their descriptions as an independent Paragraph.
- item1
  desc
- item2
  desc

textlint AST will be as follows:
List
  ListItem
    Paragraph
    Paragraph
  ListItem
    Paragraph
    Paragraph

pros

Implementation will be simple

cons

It is not actually a paragraph and can be false positive for some rules.

Since Re:VIEW also has definition list, I examined the behavior of textlint-plugin-review. As a result, it output AST similar to the idea 1.
input (test/chunker-test.js)
 : Alpha
    DEC の作っていた RISC CPU。
    浮動小数点数演算が速い。
 : POWER
    IBM とモトローラが共同製作した RISC CPU。
    派生として POWER PC がある。
 : SPARC
    Sun が作っている RISC CPU。
    CPU 数を増やすのが得意。

output:
It seems that the Re:VIEW plugin generates ListItem element for each header and list item.
This is close to idea 1, but the difference is whether the ListItem hierarchy is flat or nested. Personally, I feel that the flat hierarchy is simpler.


Drop empty tags

What does this PR do?
Drop empty tags when parsing dogstatsd package.
Motivation
We had a bug in our application metrics code that caused empty tags to be inserted into the tag list.  When this happened, the metric would not be tagged with any of the tags after that empty value in the dogstatsd packet.
Additional Notes
This break existing behavior, but I'm not sure  existing behavior on the datadog side is intended.

Sorry this went so long without getting attention 😞 I don't see any reason not to merge it, so feel free to reopen the PR if you want us to give it a proper review this time 😅@mention me if you do.


NoSuchException when displaying a poll that contains too old issue(s)

Why not appear https://github.com/aosn/aosn.github.io/issues/4 in https://api.github.com/repos/aosn/aosn.github.io/issues?state=all ?

We must retrieve next page.
See response header:

< Link: https://api.github.com/repositories/42379719/issues?state=all&page=2; rel="next", https://api.github.com/repositories/42379719/issues?state=all&page=2; rel="last"


might not be able to determine fileid in lib/file/delete

https://github.com/zenodraft/zenodraft/blob/53cc30c616b085f467ee557ec1b8d8913be6a19f/src/lib/file/delete.ts#L20
fails if files was empty.
Also, replace == with ===

This has been fixed since, closing.


Immediately exits when run from tmux

When I run it from the Terminal, it works fine, but when I run it like this:
tmux new-session -s way-displays way-displays

It exits immediately.

Removed ~/.tmux ~/.zsh*:
way-displays starts normally with:
tmux new-session -s way-displays way-displays

A second attempt:
tmux new-session -s way-displays1 way-displays
[exited]

It seems that the error is not displayed:
E [11:48:11]
E [11:48:11] ERROR: another instance 5881 is running, exiting

tmux new-session "/bin/sh -c way-displays > /tmp/wd.log 2>&1"
[exited]
cat /tmp/wd.log
E [11:58:50]
E [11:58:50] ERROR: another instance 9467 is running, exiting

Gentle nudge @xeruf


[BUG] stuck update message

Please confirm these before moving forward

[X] I have searched for my issue and not found a work-in-progress/duplicate/resolved issue.
[X] I have tested that this issue has not been fixed in the latest (beta or stable) release.
[X] I have checked the FAQ section for solutions
[X] This issue is about a bug (if it is not, please use the correct template)

Describe your issue
I had updates on queue for Windows Terminal and Visual Studio. Both were not recognized by WingetUI as sucessfully installed, despite they actually being so
Steps to reproduce the issue
try updating winget pkg suck as visual studio and windows terminal
WingetUI Log
log1.txt
Package Managers Logs
log2.txt
Relevant information

Screenshots and videos

related to https://github.com/marticliment/WingetUI/issues/1218

I had a similar problem with Chocolatey updating playnite from 10.19 to 10.20, unfortunaly I didn't save the logs.  First I updated normally, when it failed I used admin mode. It worked but I had the same situation as this issue

Please test latest beta: https://github.com/marticliment/WingetUI/releases/download/2.1.2-beta/WingetUI.Installer.exe

It is somewhat hard to test this. How about closing it for now and I reopen it if it happens again?


What's new in 3.1

Fixes #15955
Internal review URL

@jkotalik please review Support for shared queues in HTTP.sys

@guardrex  can you complete the 3 stubbed out Blazor sections?

@Tratcher

@Tratcher  while you're at it I need content for
<!-- TODO
## Breaking changes for SameSite cookies
-->

Yes ... and the bit you have there should use the new Component Tag Helper. I'll update that.
I'll send a PR to your branch with the bits.


Use character instead of one-character string where available

There're several places where a "one-character" string is created to be passed to a method that accepts a character directly. This last version is normally faster (and has not the overhead of creating the String object).
In this commit, I replaced most usages (ignoring files signalled as generated) for String.indexOf, String.lastIndexOf and StringBuilder.append .

#21 is 21 days old :-)

No one looked into this pull request in the last three months and now it has conflicts, so I'll close it.
This wouldn't make a big difference in performance, but I believe that any little thing helps (and this is an almost "free lunch" optimisation).
:-(

Hello, @smmribeiro!
I am Oscar, the Product Marketing Manager of MPS.
First of all, sorry for didn't check the pull request on time. We have been very busy with the version 2017.2, but this is not an excuse.
I invite you to solve the conflicts, and I will make sure that someone checks it out.
We appreciate your contribution in MPS!
Best,

Hello @oscarmarcelo88 ,
Thank you for the feedback (sometimes it's just that that is needed). I'll try to redo the pull request until the end of this week.
Regards,
Sérgio Ribeiro
Porto - Portugal

Perfect!
Please ping me when you have it ready.
Best,

@oscarmarcelo88, I've fixed the conflicts and opened a new PR: PR#23.
Regards,
Sérgio.

Got it!
Let me process it with someone from the team.
Thanks!
Oscar

Thank you @oscarmarcelo88 .
I'm now on my vacations, but I have one or more thoughts that may improve MPS... I'll try to work on them and create new PR's for you guys :-)

That will be awesome @smmribeiro! If you need any kind of support for the PR let us know.
Enjoy your vacations,
Oscar Rodriguez


Android aggresive cache clearing leads to crash

I have the cache put inside a subdirectory in the application cache directory. On devices with low internal storage (Nexus One for example) this will eventually lead to low storage situations where Android will just clear the whole cache directory, including subdirectories.
This gives this:
java.io.FileNotFoundException: /data/data/<package>/cache/images/-1944396650.0.tmp (No such file or directory)
at org.apache.harmony.luni.platform.OSFileSystem.open(Native Method)
at dalvik.system.BlockGuard$WrappedFileSystem.open(BlockGuard.java:232)
at java.io.FileOutputStream.<init>(FileOutputStream.java:94)
at java.io.FileOutputStream.<init>(FileOutputStream.java:66)
at <package>.DiskLruCache$Editor.newOutputStream(DiskLruCache.java:686)

It's thrown because the images directory is now gone. I though listening for ACTION_DEVICE_STORAGE_LOW could solve this but that is broadcasted after Android has cleared the directory.
If would be nice if it was possible to just reset the cache when the whole directory has gone missing or something.

Hi Lexs,
Is it possible to write cache into a cache directory which is located in SD Card?(/mnt/sdcard/data/somepackagename/cache/images/)
In my opinion, Android would never clear this, and this could be a workaround.

Give the dev branch a try. Threw a quick sanity check and attempted directory recreation.

Gerrit tracking: https://android-review.googlesource.com/31430


[Provider's observer / Provider family] provider argument property is not set with the argument of the family

Describe the bug
Provider's observer, provider argument property is not set with the argument of the family when I consume it.
To Reproduce
class ProviderEventBusObserver extends ProviderObserver {
  @override
  void didUpdateProvider(
    ProviderBase<dynamic> provider,
    Object? previousValue,
    Object? newValue,
    ProviderContainer container,
  ) {
    if (provider.from == null) {
// something
    } else {
      print(provider.name);
      print(provider.argument); // will be null
    }
  }
}

When checking the generated code
// GENERATED CODE - DO NOT MODIFY BY HAND

part of 'retrieve_work_detail_use_case.dart';

// **************************************************************************
// RiverpodGenerator
// **************************************************************************

String _$retrieveWorkDetailUseCaseHash() =>
    r'472450562511bef31c31c25b65f4887b8ba05d3a';

/// Copied from Dart SDK
class _SystemHash {
  _SystemHash._();

  static int combine(int hash, int value) {
    // ignore: parameter_assignments
    hash = 0x1fffffff & (hash + value);
    // ignore: parameter_assignments
    hash = 0x1fffffff & (hash + ((0x0007ffff & hash) << 10));
    return hash ^ (hash >> 6);
  }

  static int finish(int hash) {
    // ignore: parameter_assignments
    hash = 0x1fffffff & (hash + ((0x03ffffff & hash) << 3));
    // ignore: parameter_assignments
    hash = hash ^ (hash >> 11);
    return 0x1fffffff & (hash + ((0x00003fff & hash) << 15));
  }
}

typedef RetrieveWorkDetailUseCaseRef = AutoDisposeFutureProviderRef<Work>;

/// See also [retrieveWorkDetailUseCase].
@ProviderFor(retrieveWorkDetailUseCase)
const retrieveWorkDetailUseCaseProvider = RetrieveWorkDetailUseCaseFamily();

/// See also [retrieveWorkDetailUseCase].
class RetrieveWorkDetailUseCaseFamily extends Family<AsyncValue<Work>> {
  /// See also [retrieveWorkDetailUseCase].
  const RetrieveWorkDetailUseCaseFamily();

  /// See also [retrieveWorkDetailUseCase].
  RetrieveWorkDetailUseCaseProvider call(
    String workId,
  ) {
    return RetrieveWorkDetailUseCaseProvider(
      workId,
    );
  }

  @override
  RetrieveWorkDetailUseCaseProvider getProviderOverride(
    covariant RetrieveWorkDetailUseCaseProvider provider,
  ) {
    return call(
      provider.workId,
    );
  }

  static const Iterable<ProviderOrFamily>? _dependencies = null;

  @override
  Iterable<ProviderOrFamily>? get dependencies => _dependencies;

  static const Iterable<ProviderOrFamily>? _allTransitiveDependencies = null;

  @override
  Iterable<ProviderOrFamily>? get allTransitiveDependencies =>
      _allTransitiveDependencies;

  @override
  String? get name => r'retrieveWorkDetailUseCaseProvider';
}

/// See also [retrieveWorkDetailUseCase].
class RetrieveWorkDetailUseCaseProvider
    extends AutoDisposeFutureProvider<Work> {
  /// See also [retrieveWorkDetailUseCase].
  RetrieveWorkDetailUseCaseProvider(
    this.workId,
  ) : super.internal(
          (ref) => retrieveWorkDetailUseCase(
            ref,
            workId,
          ),
          from: retrieveWorkDetailUseCaseProvider,
          name: r'retrieveWorkDetailUseCaseProvider',
          debugGetCreateSourceHash:
              const bool.fromEnvironment('dart.vm.product')
                  ? null
                  : _$retrieveWorkDetailUseCaseHash,
          dependencies: RetrieveWorkDetailUseCaseFamily._dependencies,
          allTransitiveDependencies:
              RetrieveWorkDetailUseCaseFamily._allTransitiveDependencies,
        );

  final String workId;

  @override
  bool operator ==(Object other) {
    return other is RetrieveWorkDetailUseCaseProvider && other.workId == workId;
  }

  @override
  int get hashCode {
    var hash = _SystemHash.combine(0, runtimeType.hashCode);
    hash = _SystemHash.combine(hash, workId.hashCode);

    return _SystemHash.finish(hash);
  }
}
// ignore_for_file: unnecessary_raw_strings, subtype_of_sealed_class, invalid_use_of_internal_member, do_not_use_environment, prefer_const_constructors, public_member_api_docs, avoid_private_typedef_functions

In the constructor part :
  RetrieveWorkDetailUseCaseProvider(
    this.workId,
  ) : super.internal(
          (ref) => retrieveWorkDetailUseCase(
            ref,
            workId,
          ),
          from: retrieveWorkDetailUseCaseProvider,
          name: r'retrieveWorkDetailUseCaseProvider',
          debugGetCreateSourceHash:
              const bool.fromEnvironment('dart.vm.product')
                  ? null
                  : _$retrieveWorkDetailUseCaseHash,
          dependencies: RetrieveWorkDetailUseCaseFamily._dependencies,
          allTransitiveDependencies:
              RetrieveWorkDetailUseCaseFamily._allTransitiveDependencies,
        );

I expect there a call of the argument parameter?
In the comment it said:
  /// If this provider was created with the `.family` modifier, [argument] is
  /// the variable that was used.
  final Object? argument;

Thank you in advance for your answer if I missing something or not here.
Best regards

argument is not set on generated providers, as they can have more than one argument.
argument works only for the .family modifier
Although now that we have records, we could have it record a record of all arguments for logging purpose without needing to cast the provider.

It think it can be handled even if there is multiple argument as argument is type of Object. As you said record can be great.
For my specific usecase I find a different way by using the value of the family provider instance to do my things so it's okay but I wanted to open "issue" about this. So it's not a really needed feature at the time.
Because from the comment I really expected something filled here.
The name is maybe arguments and not argument somehow? (like the ...args in JS)
I use riverpod with generators so I didn't know it's working with the family modifier. I supposed that I was using a family "modifier" as Im using the generated version of the same behavior, sorry.

You closed it too fast!
I just implemented it :P

I agree on argument vs arguments. But I don't think that's worth a breaking change.

You closed it too fast!
I just implemented it :P

You didn't know yet but you already implemented it! ^_^


FF 25 gadget is causing a crash when used for hooking

Hi there, I'm getting a crash when I place the FF 25 (indirect jump) gadget for EPT hooking and I double checked that I'm doing it right. While mov (register), (address); jmp (register) gadget works fine, I tried to change PTE read permission to 1 when vm-exit was triggered by instruction fetch
if (qualification.execute_access) {
  pte->read_access = 1;
  pte->write_access = 1;
  pte->execute_access = 1;
  pte->page_frame_number = hook->exec_pfn;
}

and the FF 25 gadget worked fine, but as expected, my executable page was exposed, and so was the hook. I think the crash is happening because the instruction can't read the address where we want to jmp? I'm not really sure about this and I think I'm missing something here. I place the hook on first bytes of the function. Here is the analysis of the crash dump, although there is little information.
ERROR_CODE: (NTSTATUS) 0xC0000005 (Access violation)
READ_ADDRESS:  ffffffffffffffff
EXCEPTION_PARAMETER1:<PHONE_NUMBER>000000
EXCEPTION_PARAMETER2:  ffffffffffffffff
Attempt to read from address ffffffffffffffff

Yeah this isn't supported at the moment. It is a little confusing, but if you think about it, this shouldn't be possible anyways:
FF 25 00 00 00 00             jmp [rip+0]
11 22 33 44 55 66 77 88       address

The memory read [rip+0] will return the contents of the original page (which is just garbage) and it'll try to jump to that. A potential fix is to enable MTF-exiting, allow read/write/execute, then remove those permissions after the instruction is done executing. Or just use something like:
push 0x11223344
mov dword ptr [rsp+4], 0x55667788
ret

(The gadget might not be exactly right, but the idea is the same)


Simpify queries code with object property shorthand

@webmaster128  Hi Simon, could you please review this when you have time? thanks.

Please avoid opening issues that add no value to the project. If we decide the object property shorthand is better (which I don't agree with), such things should only be done through auto-fixing lint tooling.


Have muting use Topic model

@timabbott This is an initial look at how muting topics would work with a Topic table, to help address #1019.  So far I only touch the back end code.  The current series may actually be a minor performance regression, because the front end still pushes O(N) muted topics to us, and now we have to update O(N) rows on the back end, which is presumably more expensive than writing an O(N) json field in Python.  I doubt that performance is terrible in either case.
Additional commits would make the front end just communicate O(1) changes to the server.
There is also no data backfill yet in this series.  I'm thinking of just doing a lazy backfill--if you try to examine which topics are muted, we check for UserProfile.muted_topics having "migrated" as its value.  If muted_topics is not "migrated" but is instead still valid legacy json, we do the migration to UserTopic and write "migrated" to UserProfile.muted_topics.
This series also does not address approximate_unread_count, which seems to have totally non-sensical handling of muted topics, and it has no test coverage to boot.  I'll open a separate ticket for that.

Ugh, this pull request is kind of screwed up, but basically the last five commits are a preview of the ongoing effort to make muting work with the new Topic table.  (The new Topic table is not on master, and all the early commits on this pull request are on their own development branch that is being monitored elsewhere.)


Server Validators

Hi guys,
There seems to be a bug in the example for server validation (https://surveyjs.io/Examples/Library/?id=validators-server&platform=Vue&theme=default).
The error in the console is: Uncaught TypeError: Cannot read property 'add' of undefined.
I can't get the server validation to work.

browser: chrome
surveyjs platform (angular or react or jquery or knockout or vue): vue
surveyjs version:  1.0.61

you can test it on the example you posted with this  :
survey.onIsValidatingOnServerChanged = surveyValidateQuestion;
It was working not long ago... it seems there is a bug...

Thank you for the report. We'll work on the fix.

We have to fix our example.
Please use the following example for now.
Thank you,
Andrew

We've fixed the issue. Fix will be available in the next minor update.

Thanks for the quick fix!!


Cannot pass parameters to underlying test runner

Description
Summarized in https://github.com/dotnet/cli/issues/3114. Basically runners like Xunit have the ability to change behavior based on command line parameters, this is a must have and worked before the move to VSTest.
This used to be accomplished by passing in the args after a double hypen, eg:
dotnet test myproj.csproj -- -teamcity -xml myfile.xml
Steps to reproduce

Try to pass parameters to any VSTest enabled XUnit project, (eg: -teamcity, or -xml output.xml).

Expected behavior
XUnit to receive parameters passed after the double hyphen.
Actual behavior
Everything after the double hyphen is ignored.
Environment
Happens on all environments.

@StrangeWill with the move to vstest, the adapters do get these parameters as run settings. I am not sure if xunit adapter understands these parameters and provides the functionality.
From your example, it looks like the scenario is to produce xunit xml results and enable teamcity integration. It is possible to achieve this using following logger extensions:
https://github.com/Faizan2304/LoggerExtensions
https://github.com/JetBrains/TeamCity.VSTest.TestAdapter

I guess due to the backwards-compatible breaking changes made they opted to use their own build command as of 2.3 to get off the dependency of VSTest and any further disruption to the platform:
https://github.com/xunit/xunit/issues/1154#issuecomment-295184454
https://xunit.github.io/docs/getting-started-dotnet-core
Is there a reason that VSTest broke this established syntax?

One of the guiding principles for vstest is to provide an uniform user experience (across CLI, IDE/Editors, CI) to run tests irrespective of the test frameworks used, and provide great extensibility for developers (e.g. adapters, loggers etc.). We're evolving the platform towards that. This comment also summarizes the thought process: https://github.com/dotnet/cli/issues/4921#issuecomment-265051207.

We (@xunit) have always viewed VSTest as a way to get integration with Visual Studio. The availability of vstest.console.exe in desktop CLR and dotnet test (or dotnet vstest) in .NET Core is a down-level experience, a common denominator runner platform that does not offer xUnit.net-specific features. For that, we have xunit.console.exe for desktop CLR and dotnet xunit in .NET Core. These things aren't mutually exclusive; they simply serve different purposes.
FYI: Since test adapters that write to VSTest are easier to automate, they're also easier to consume by other third party runners which are also looking to provide a common denominator runner platform (for example, Resharper or VS Code).

@bradwilson yeah just noticed that the new beta has dotnet xunit functionality, which I know has been talked about since .NET Core dropped the ability to pass parameters onto xunit.console.exe prior to the VSTest changes.
This comment: https://github.com/dotnet/cli/issues/4921#issuecomment-281677977 suggests that's probably have 3rd party testing frameworks will work entirely moving forward anyway. So I'll just pull all the bleeding-edge stuff for xunit and rework my CI around the new functionality.

FYI: Since test adapters that write to VSTest are easier to automate

Funny enough due to this change the only thing that has broken is automation, specifically with static code analysis platforms being as they're no longer receiving the expected Xunit output from the CLI (and trx reports are no good).

I meant automation from a code perspective (i.e., using our VSTest adapter as an API that they can program against, by simulating the Visual Studio APIs themselves). 😉

Guess I'm going to close it with these notes:
VSTest is primarily to create a unified API for communicating with test runners, this relies on runsettings for any specific configuration for the test runners (and support for such). This exposes 3rd party testing frameworks to tools like Resharper and whatnot easily.
Libraries like xUnit are putting out their own test runners so that they can expose their own specific tooling behaviors. This will probably be best for CI setups. While this issue has made upgrading our projects to 1.1 frustrating it looks like xUnit has been hard at work at resolving it.
I guess the most aggravating part for most of us is breaking the CLI out of the gate from commands that worked fine in the 1.0 release and us being a little high and dry for a few months but I guess the issue is coming to a close anyway.

Unfortunately, it took me some time to get dotnet xunit working the way I wanted it to, which left us in this in-between state for a while. The beta releases of xUnit.net are generally speaking very stable, and you should feel comfortable using the beta runners in CI scenarios (I have personally used almost exclusively beta builds for the last 10 years, on primarily large codebases (including CodePlex and ASP.NET MVC/Web API). On the rare occasions that issues are discovered in beta builds, we almost always immediately fix them and offer new builds right away.

Yeah I had been on beta xunit for awhile before 2.2 hit stable, it was always good to me, I just didn't check for a beta again because I didn't even know that 2.3 was in the works or what was coming with it.
Thanks for the work on Xunit though. 👍

I faced with this issue today. Any news about it ? Is there any version that fix this ?
Thanks !

@ChrisProlls there's a xunit logger for dotnet test available here: https://github.com/Faizan2304/LoggerExtensions. More information on loggers (teamcity etc.) are documented here: https://github.com/Microsoft/vstest-docs/blob/master/docs/report.md.
Pre 1.0 dotnet-cli versions used to support passing args to the underlying test runners directly. This is no longer the case with 1.0 and later versions. Current architecture is a single runner with adapters for variety of test frameworks. It supports configuring the test framework specific configuration with a run settings (which can be configured from command line as well). The reporting aspects are configured directly with the runner (independent of the test framework adapter).


Format of Json data

Hi,
Can you provide a sample of Json which is acceptable? Or  Can you  add a feature where we can assign a javascript object / array as rows?

You can see a lot of examples to load a CSV or array:
https://bossanova.uk/jexcel/examples/creating-a-table-from-an-external-csv-file


Adding Netlify badge

Signed-off-by: Tihomir Surdilovic<EMAIL_ADDRESS>Many thanks for submitting your Pull Request :heart:!
What this PR does / why we need it:
Special notes for reviewers:
Additional information (if needed):

The greeting action is failing across the board. Will remove it like we did in the spec repo


createUploadSession fails when fileSystemInfo supplied

The documented properties for createUploadSession  include: fileSystemInfo
Per: https://learn.microsoft.com/en-us/onedrive/developer/rest-api/api/driveitem_createuploadsession?view=odsp-graph-online#properties
I would like to supply a lastModifiedDateTime.
If the fileSystemInfo object is supplied at all, with any properties, the request fails.
These are the available properties of fileSystemInfo:
{
  "createdDateTime" : "datetime",
  "lastAccessedDateTime": "datetime",
  "lastModifiedDateTime" : "datetime"
}

Example successful request:
{
    "item": {
<EMAIL_ADDRESS>"replace"
    }
}

Example failing request:
{
    "item": {
        "fileSystemInfo": {
            "lastModifiedDateTime": "2023-11-16T01:06:28.7971778+00:00"
        },
<EMAIL_ADDRESS>"replace"
    }
}

ID's from Graph Explorer:
"client-request-id": "2b69e814-af57-232a-8793-32234145768a",
"request-id": "9b3f0a6d-0970-4fa1-9991-e184d3724785"
Please could you fix graphAPI so that it matches the documentation (not the other way around)?

It works for me,
example (idea http):
### create session with meta
POST https://graph.microsoft.com/v1.0/me/drive/root:/test-modtime/test-file-session:/createUploadSession
Authorization: {{token}}
Content-Type: application/json

{
  "item": {
<EMAIL_ADDRESS>"replace",
    "fileSystemInfo": {
      "createdDateTime": "2024-04-24T02:28:27.433Z",
      "lastModifiedDateTime": "2024-04-25T02:28:27.433Z"
    }
  }
}

<> 2024-04-26T140903.200.json

### upload
PUT https://api.onedrive.com/rup/6349fe61df03d513/eyJSZX
Content-Length: 96
Content-Range: bytes 0-95/96

< main.go

<> 2024-04-26T141705.201.json
#{
# ...
#  "createdDateTime": "2024-04-26T06:17:04.35Z",
#  "lastModifiedDateTime": "2024-04-26T06:17:04.35Z",
#  "name": "test-file-session",
#  "size": 96,
#  "fileSystemInfo": {
#    "createdDateTime": "2024-04-24T02:28:27.433Z",
#    "lastModifiedDateTime": "2024-04-25T02:28:27.433Z"
#  },
#  "lenses": []
#}


Balanced_consumer keeps timing out and exiting

I am using a pykafka 2.5.0 (and 2.8.0-dev.2 just to make sure this is not version-related) to consume messages from a kafka cluster.
On a development machine, it all works fine, but on the production environment, where i am trying to migrate, my consumer doesn't seem to connect to the zookeeper. I tried to start a simple consumer, and it works fine. The zookeeper servers are pingable and reachable.
This is my log output (starting from the balanced consumer instanciation) :
2018-07-05 13:03:50,891 - pykafka.handlers - INFO - RequestHandler worker: exiting cleanly INFO:pykafka.handlers:RequestHandler worker: exiting cleanly 2018-07-05 13:03:51,212 - pykafka.handlers - INFO - RequestHandler worker: exiting cleanly INFO:pykafka.handlers:RequestHandler worker: exiting cleanly WARNING:kazoo.client:Connection dropped: socket connection error: None 2018-07-05 13:04:50,295 - pykafka.balancedconsumer - ERROR - Stopping consumer in response to error Traceback (most recent call last):   File "/opt/consumer/lib/python2.7/site-packages/pykafka/balancedconsumer.py", line 332, in start     self._zookeeper_connection_timeout_ms)   File "/opt/consumer/lib/python2.7/site-packages/pykafka/balancedconsumer.py", line 382, in _setup_zookeeper     self._zookeeper.start()   File "/opt/consumer/lib/python2.7/site-packages/kazoo/client.py", line 560, in start     raise self.handler.timeout_exception("Connection time-out") KazooTimeoutError: Connection time-out ERROR:pykafka.balancedconsumer:Stopping consumer in response to error Traceback (most recent call last):   File "/opt/consumer/lib/python2.7/site-packages/pykafka/balancedconsumer.py", line 332, in start     self._zookeeper_connection_timeout_ms)   File "/opt/consumer/lib/python2.7/site-packages/pykafka/balancedconsumer.py", line 382, in _setup_zookeeper     self._zookeeper.start()   File "/opt/consumer/lib/python2.7/site-packages/kazoo/client.py", line 560, in start     raise self.handler.timeout_exception("Connection time-out") KazooTimeoutError: Connection time-out 2018-07-05 13:04:50,296 - pykafka.balancedconsumer - DEBUG - Stopping <pykafka.balancedconsumer.BalancedConsumer at 0x7f14c6b8bd90 (consumer_group=consumer_radio_data)> DEBUG:pykafka.balancedconsumer:Stopping <pykafka.balancedconsumer.BalancedConsumer at 0x7f14c6b8bd90 (consumer_group=consumer_radio_data)> Traceback (most recent call last):   File "scripts/pykafka_consumer.py", line 78, in <module>     # Rework following if, pre-prepare it   File "/opt/consumer/lib/python2.7/site-packages/pykafka/balancedconsumer.py", line 766, in __iter__     message = self.consume(block=True)   File "/opt/consumer/lib/python2.7/site-packages/pykafka/balancedconsumer.py", line 756, in consume     raise ConsumerStoppedException pykafka.exceptions.ConsumerStoppedException 2018-07-05 13:04:50,324 - pykafka.balancedconsumer - DEBUG - Finalising <pykafka.balancedconsumer.BalancedConsumer at 0x7f14c6b8bd90 (consumer_group=consumer_radio_data)> DEBUG:pykafka.balancedconsumer:Finalising <pykafka.balancedconsumer.BalancedConsumer at 0x7f14c6b8bd90 (consumer_group=consumer_radio_data)> 2018-07-05 13:04:50,325 - pykafka.handlers - INFO - RequestHandler.stop: about to flush requests queue INFO:pykafka.handlers:RequestHandler.stop: about to flush requests queue 2018-07-05 13:04:50,325 - pykafka.handlers - INFO - RequestHandler.stop: about to flush requests queue INFO:pykafka.handlers:RequestHandler.stop: about to flush requests queue 2018-07-05 13:04:50,325 - pykafka.handlers - INFO - RequestHandler.stop: about to flush requests queue INFO:pykafka.handlers:RequestHandler.stop: about to flush requests queue 2018-07-05 13:04:50,325 - pykafka.handlers - INFO - RequestHandler.stop: about to flush requests queue INFO:pykafka.handlers:RequestHandler.stop: about to flush requests queue

Thanks for the issue report. How are you providing the ZooKeeper host string to pykafka? The standard way to do this is with zookeeper_hosts here.

I am calling it like this :
consumer = topic.get_balanced_consumer(     consumer_group='consumer_radio_data',     auto_commit_enable=False,     zookeeper_connect='zookeeper01.address:2181,zookeeper02.address:2181,zookeeper03.address:2181',     auto_offset_reset=OffsetType.LATEST,     reset_offset_on_start=True, )
If needed, i could post more code chunks, or the whole thing :)

What happens when you run this from the same server you're trying to make a BalancedConsumer on?
from kazoo.client import KazooClient
zk = KazooClient('zookeeper01.address:2181,zookeeper02.address:2181,zookeeper03.address:2181')
zk.start()

Outputs the following :
No handlers could be found for logger "kazoo.client" Traceback (most recent call last):   File "<stdin>", line 1, in <module>   File "/opt/openei/lib/python2.7/site-packages/kazoo/client.py", line 560, in start     raise self.handler.timeout_exception("Connection time-out") kazoo.handlers.threading.KazooTimeoutError: Connection time-out
And on my dev machine, it works like a charm, outputs nothing and runs properly.

That error indicates that the machine you're attempting to run on is unable to connect to the ZooKeeper cluster you're pointing pykafka to. Once that's resolved, you should find that pykafka can run just fine.

How is it unreachable if it is pingable and traceroute'able ?

It's possible that the particular port 2181 on which you're trying to connect to the zookeeper nodes is not accessible.

Might be, but why is it accessible from another machine  and not this one, particularly ?

The answer to that depends on the particular configuration of the two machines in question as well as any firewalls or other network configurations you might have set up that could be limiting traffic between the two machines.


drag-n-drop doesn't seem to work

https://vueflow.dev/examples/drag-n-drop

drag-n-drop doesn't seem to work

Thanks, I will check it out

@poerlang
Fixed in Versions ^0.2.x - You can check it out on the example page


Sometimes microphone not working.

In karaoke ruleset, microphone cannot working after restart the game, but can work if quit to song selection then entry into game play.

Maybe fixed in #25 ?
Need to make a check.

Fixed in #29


Update Kptfiles in Sub Packages

@mortent
This PR makes sure that the nested Kptfiles are updated during kpt pkg sync/update. This PR also makes sure that the upstream subpackages(locally pulled for comparison and update) are set with local setters so that the resources are correctly identified for merge.
This PR will fix the issue: https://github.com/GoogleContainerTools/kpt/issues/794

/lgtm


Use prop-types package instead React.PropTypes

React 15.5 brings a new package called prop-types: https://facebook.github.io/react/blog/#migrating-from-react.proptypes.
Please update definitions.

Any updates?

#19 merged, waiting for @CezaryDanielNowak to update npm registry.

Published.


Misleading "not found in this scope" error

I'm loving the new serde!
This is a bit of an unfortunate error message, and it's probably rustc's fault, but I thought I would report it:
/// A parsed version of our configuration file.
#[derive(Clone, Debug, Deserialize, Serialize)]
pub struct Config {
    /// Configuration for each notifier.
    notifiers: HashMap<String, NotifierConfig>,
}

# No definition of NotifierConfig.

Here, the error[E0412]: is correct, but the brightly colored "not found in this scope" below it points to the wrong thing.
error[E0412]: cannot find type `NotifierConfig` in this scope
 --> src/config.rs:6:35
  |
6 | #[derive(Clone, Debug, Serialize, Deserialize)]
  |                                   ^^^^^^^^^^^ not found in this scope

The fix, of course, is to implement NotifierConfig. I know you can't do much about this error, but I thought you might like to know that there was a minor ergonomic issue.

Thanks! This is a limitation of Macros 1.1 and will be fixed by https://github.com/rust-lang/rust/pull/40939.

This can also happen, when one forgets to add the following in the main module of the application:
// In order to use the Serialize and Deserialize macros in the model,
// we need to declare in the main module, that we are using them.
#[macro_use]
extern crate serde_derive;


removing the Firefox quirk - Fixes #969

Fixes #969

@darthtrevino - can we merge this if it looks good ?

Looks great, let me test your branch and I'll merge it in


bpf: Implement Segment Routing Header (SRH) support

Until now, we have only supported the reduced encapsulation format for SRv6. That was possible because we also only support a single segment for now so we can always use the reduced encapsulation format.
However, some routers (e.g., FRR) don't support reduced encapsulation yet. Thus, we should also have a fallback to include a full Segment Routing header when encapsulating and decapsulate accordingly.
See the commits for details.

/test

Looks good to me. Have you ever tested this with FRR (Linux implementation)?

Not yet, but I have compared the packets' layout with that of packets emitted by FRR and they're the same, so hopefully we're good :-)

/test-1.16-4.9

/test-1.24-net-next

/test-1.16-4.9

/test-1.16-4.9

/test-vagrant

/k8s-1.16-kernel-4.9

/test-1.16-4.9

4.9 has new test failures, which confirms the previous and current failures are flakes. They correspond to known flake https://github.com/cilium/cilium/issues/16159 and are clearly not complexity issues. Marking ready to merge.

It looks like this PR picked up a merge conflict and needs a rebase.

/test-vagrant

/test-jenkins

/test-1.16-4.9

/test-1.16-4.9


chore: fix Github action setup gardener test cluster

Description
Fix Github action setup gardener test cluster:

Fail the job if the deploy scripts step fails.
Add proper exec permissions to script files.

Example run(s)

https://github.com/kyma-project/kyma-companion/actions/runs/10734812742

/hold

/hold cancel


Removed redefined types to support GCC

The current build fails with GCC 4.4.7 due to redefined types.
gcc      -I../lib  -O3 -std=c99 -Wall -Wextra -Wundef -Wshadow -Wcast-align -Wstrict-prototypes     -DZSTD_VERSION=\"r0.0.1\"  ../lib/zstd.c xxhash.c bench.c fileio.c zstdcli.c -o zstd
../lib/zstd.c:108: error: redefinition of typedef âBYTEâ
../lib/fse.c:95: note: previous declaration of âBYTEâ was here

..and so on
I've suggested a header for these type definitions. I've tested both GCC and Clang with this configuration.
This is the latest version of GCC available for RedHat/CentOS 6, so is expected to be common.

Right.
Curiously enough, I'm using GCC 4.8 and 4.6 for my tests, and none of them complain about that.
But clang does, and that's why the following lines have been added :
#  ifdef __clang__
#    pragma clang diagnostic ignored "-Wtypedef-redefinition"
#  endif

Maybe something equivalent could be added for gcc too, such as :
#    pragma gcc diagnostic ignored "-Wtypedef-redefinition"

Note that the current situation, which is a consequence of including 'fse.c' rather than 'fse.h', is not satisfying. This is due to the fact than gcc/clang look unable, or very limited, in their ability to inline functions from another module, which is a killer for performance.
I'm therefore extremely likely to change this construction in the future, although final solution is not yet known.
At best, I would like to include a normal .h file, but on condition of preserving performance for gcc/clang. At worst I plan to include all "memory types and routines" within their own module.
Which solution is most appropriate is not yet settled.

Yep, I noted after sending the pull that the Travis build runs a more recent GCC and doesn't have a problem with the existing method. Similarly, I get a tonne of warnings about strict aliasing rules which travis doesn't seem to mention.
../lib/fse.c:737: warning: dereferencing pointer âCTable.35â does break strict-aliasing rules

As far as I'm aware my PR shouldn't hurt performance - it doesn't go away with fse.c being included directly.
If you want to go in the pragma direction I'm happy to test that out.

If you want to go in the pragma direction I'm happy to test that out.

Yes please,
I'm not planning to add to any file at this stage to lib directory at this stage.
It sounds very little, but every file add complexity for project integration, so I'm trying to avoid that.
What may happen later on is that fse.c will simply disappear, being directly integrated into zstd.c. At which stage, the question relative to memory access types and functions will be already settled. As a consequence, at the cost of becoming bigger, zstd.c will remain the sole source file to include, simplifying integration process.
So yes, even the number of files matters.
For the test, adding
#    pragma gcc diagnostic ignored "-Wtypedef-redefinition"
just after the clang one should do the trick.

you said it issued an error

Correct, it is a fatal error and will not build. I have tested the pragma but gcc seems to go out of its way to ignore it:
$ make
make[1]: Entering directory `/home/technion/zstd/programs'
gcc      -I../lib  -O3 -std=c99 -Wall -Wextra -Wundef -Wshadow -Wcast-align -Wstrict-prototypes   -DZSTD_VERSION=\"r0.0.1\"  ../lib/zstd.c xxhash.c bench.c fileio.c zstdcli.c -o zstd
../lib/zstd.c:74: warning: ignoring #pragma gcc diagnostic
../lib/zstd.c:110: error: redefinition of typedef ‘BYTE’
../lib/fse.c:95: note: previous declaration of ‘BYTE’ was here
.....
make[1]: *** [zstd] Error 1
make[1]: Leaving directory `/home/technion/zstd/programs'
make: *** [zstdprograms] Error 2

OK, then there is no other way than to change the code.
I'll look into it, and open an issue to track the progresses.

As stated within #25 :
Fix attempt, downloadable at :
https://github.com/Cyan4973/zstd/tree/streamAPI2


checkpoint required

When using Instagram::setProxy()
it's return An uncaught Exception was encountered
Type: InstagramScraper\Exception\InstagramAuthException
Message: Something went wrong. Please report issue.
Filename: /home/vendor/raiym/instagram-php-scraper/src/InstagramScraper/Instagram.php
checkpoint_required

It's because of IP from the new location.
You can send GET request to challenge URL and after get verification code, send POST request with code. Then you will get session for the user.

please send the code for that

You can refer the code in this thread to avoid checkpoint issue.


better adjustment for potential race condition in test script

Instead of a lazy sleep 10, let's actually validate the output we expect from the kubectl command.

/rerun minikube


Add general toasts/banners

It'd be nice to have a site-wide notification solution, e.g. for:

Livestream info
Further announcements/infos
...

Resolved by #110


added albumentation augmented features

added albumentation augmented features but not completely done with the testing part

Hi @kaldan007, I have completed the changes based on your review. Could you please review the updated code to ensure everything is correct?


ConsulDiscoveryClientConfiguration depends on ServerProperties

I'm currently using ConsulDiscoveryClient with Brixton.M3 and I'm trying to upgrade to Brixton.M5 and I'm hitting a road block which has been introduced with this commit:
https://github.com/spring-cloud/spring-cloud-consul/commit/518958e408c1573984773d90d16bcc44f6a54f2e
Basically, if I'm not mistaken we're saying that the ConsulDiscoveryClient can only be used in the context of a webapp (since ServerPropertiesAutoConfiguration has @ConditionalOnWebApplication). This doesn't seem correct to me since I can have a command line application that should be able to use the consul client to discover services registered with consul.

To avoid the missing ServerProperties bean, I ended up registering one in my own Configuration.

Currently yes. The lifecycle is a holdover from eureka which had to register to use the discovery client. Shouldn't be hard to make a flag to disable registering. Likely in the Camden release we will introduce a Registration interface and maybe annotation since registering with discovery and using discovery are really separate concerns.


Bring back and improve comments clarifying the new bug report template

I've noticed that people can't seem to understand some questions and I wondered why so? For instance: #37942

Minimal reproduction project: I don't exactly understand what this means

That's because the comments clarifying the questions were accidentally removed in c8e92b802c16f8f9d120bf01bc5923c702a3c3ff while migrating to the bug/feature templates workflow.
Note that it might be not necessary to include the "search for duplicates" suggestion because GitHub has a new feature of suggesting duplicate issues built-in now (might be beta feature though). Anyways, I just copied all the previous comments as is.
While we're at it, consider #35724.

We took the courage to modify the previous template. Namely, GLES2/GLES3/Vulkan hints were added to comments, and the minimal reproduction project describes what it actually represents.

Another instance of confusion occurred: https://github.com/godotengine/godot/issues/39011#issuecomment-633284359.

I think many users were confused by the HTML comments and didn't know how to handle them, leading to more situations of users outright deleting the template ("what's this junk code?") instead of filling it.
Yet, it seems the comments were just removed by mistake by @reduz when he added the &>/dev/null proposals template, so we didn't really assess whether it was a good idea.
Let's merge and see what happens, but IMO more than simple one line comments, we need a proper documentation page that tells users how to file issues and why each field is important.

Thanks!


Allow people to go to Gambling Rehab

Seen this on a RSPS & it worked for me and others
Allow for a +rehab or =rehab or some sort of +stopgambling command that once entered you’d get a confirm prompt. And upon saying confirm
Your account would be Locked From Dicing/Dueling or receiving invites for dicing/dueling for X amount of time
On the RSPS it lasts until you do the command again.


[SURE-8426] EKS instance C6 and M6 types are not listed/found in Rancher 2.8.x

Setup

Rancher version: 2.8.2
Rancher UI Extensions:
Browser type & version:

Describe the bug

EKS instance C6/M6 types are not listed/found in Rancher 2.8.2. We suspect the available instance types are hardcoded somewhere in Rancher.
To Reproduce

Rancher 2.8.2
Create EKS and see C6 and M6 instances types are not available (which are available in AWS Console)

Result
Expected Result

Screenshots

Additional context

Internal reference: SURE-8426
@mantis-toboggan-md does this work in 2.9-head? @skanakal I know you referred to https://github.com/rancher/dashboard/pull/10581 elsewhere but I couldn't tell immediately.

@gaktive this does work in 2.9-head. In the vue port we're not hardcoding instance types; the full list is automatically fetched from amazon.

@gaktive FYI - work was done by issue reporter - updated this issue and moved to test for QA to validate

Tested in

v2.8-51052a73adcd6ea40cedcefbedc25e707cdb2f25-head
release-2.8 a38998e

EKS instance C6 and M6 types are available
https://github.com/user-attachments/assets/e30642e3-f8a4-4cea-b2e2-3bc3b47d398b


2022-12-07: File information, especially version number

It would be really nice to have sole file information. Especially a version number.
This is how it looks at the moment:

I've added the details to the installer, but this won't be available until the next release.

Thank you very much!


CMake Error at run_serial_tool.cmake:66 (message): (IDFGH-11196)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[x] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
I am getting the following error when trying to flash the ESP-IDF firmware to my ESP32 device:
CMake Error at run_serial_tool.cmake:66 (message):

Environment:
Operating system: Ubuntu
on virtual machine ESP-IDF version: 5.1.1 ESP32 device: esp32c6-devkitc- v1.2
I have tried the following troubleshooting steps: Made sure that the ESP32 device is connected to my computer and in the correct mode to be flashed. Tried flashing the ESP-IDF firmware using the esptool.py command-line tool. Tried flashing the ESP-IDF firmware using a different IDE or build system. Tried flashing the ESP-IDF firmware to a different ESP32 device. I am still getting the error message. Question: What can I do to fix this error?
@shubhamdp @dhrishi @espressif-bot

on virtual machine

If you are trying to flash a device from a virtual machine then you have to make it available there. When you connect a device to a host machine, it is available there but not in the virtual machines. Usually, the virtual environment software has a setting page when the serial device can be forwarded to the virtual machine.

Related hints with suggestions were added on the esptool side: https://github.com/espressif/esptool/commit/d61da7785e29ab8a926d1f9e24314b6ea9d3f933

I'm closing this for inactivity.

Hello @dobairoland I tried the recommended it is yet not working.
`[1/5] cd /home/ubu/esp/esp-idf/esp32c6-blink-led/build/esp-idf/esptool_py && /home/ubu/.espressif/python_env/idf5.3_py3.11_env/bin/python /home/ubu/esp/esp-idf/components/partition_table/check_sizes.py --offset 0x8000 partition --type app /home/ubu/esp/esp-idf/esp32c6-blink-led/build/partition_table/partition-table.bin /home/ubu/esp/esp-idf/esp32c6-blink-led/build/esp32c6-blink-led.bin
esp32c6-blink-led.bin binary size 0x224d0 bytes. Smallest app partition is 0x100000 bytes. 0xddb30 bytes (87%) free.
[2/5] Performing build step for 'bootloader'
[1/1] cd /home/ubu/esp/esp-idf/esp32c6-blink-led/build/bootloader/esp-idf/esptool_py && /home/ubu/.espressif/python_env/idf5.3_py3.11_env/bin/python /home/ubu/esp/esp-idf/components/partition_table/check_sizes.py --offset 0x8000 bootloader 0x0 /home/ubu/esp/esp-idf/esp32c6-blink-led/build/bootloader/bootloader.bin
Bootloader binary size 0x5410 bytes. 0x2bf0 bytes (34%) free.
[3/5] No install step for 'bootloader'
[4/5] Completed 'bootloader'
[4/5] cd /home/ubu/esp/esp-idf/components/esptool_py && /usr/bin/cmake -D IDF_PATH=/home/ubu/esp/esp-idf -D "SERIAL_TOOL=/home/ubu/.espressif/python_env/idf5.3_py3.11_env/bin/python;;/home/ubu/esp/esp-idf/components/esptool_py/esptool/esptool.py;--chip;esp32c6" -D "SERIAL_TOOL_ARGS=--before=default_reset;--after=hard_reset;write_flash;@flash_args" -D WORKING_DIRECTORY=/home/ubu/esp/esp-idf/esp32c6-blink-led/build -P /home/ubu/esp/esp-idf/components/esptool_py/run_serial_tool.cmake
esptool.py --chip esp32c6 -p Port -b 460800 --before=default_reset --after=hard_reset write_flash --flash_mode dio --flash_freq 80m --flash_size detect 0x0 bootloader/bootloader.bin 0x10000 esp32c6-blink-led.bin 0x8000 partition_table/partition-table.bin
esptool.py v4.7.dev2
Serial port Port
A fatal error occurred: Could not open Port, the port is busy or doesn't exist.
([Errno 2] could not open port Port: [Errno 2] No such file or directory: 'Port')
FAILED: CMakeFiles/flash /home/ubu/esp/esp-idf/esp32c6-blink-led/build/CMakeFiles/flash
cd /home/ubu/esp/esp-idf/components/esptool_py && /usr/bin/cmake -D IDF_PATH=/home/ubu/esp/esp-idf -D "SERIAL_TOOL=/home/ubu/.espressif/python_env/idf5.3_py3.11_env/bin/python;;/home/ubu/esp/esp-idf/components/esptool_py/esptool/esptool.py;--chip;esp32c6" -D "SERIAL_TOOL_ARGS=--before=default_reset;--after=hard_reset;write_flash;@flash_args" -D WORKING_DIRECTORY=/home/ubu/esp/esp-idf/esp32c6-blink-led/build -P /home/ubu/esp/esp-idf/components/esptool_py/run_serial_tool.cmake
ninja: build stopped: subcommand failed.
`

Serial port Port

A fatal error occurred: Could not open Port, the port is busy or doesn't exist.
([Errno 2] could not open port Port: [Errno 2] No such file or directory: 'Port')

Port is not a valid port name. Usually it is either /dev/ttyUSB0 or /dev/ttyACM0 (or some greater number than 0 in the name).

In my case it was the wires I was using to connect USB-to-UART Bridge to my custom esp32 board.


Transparently compress and streaming support

What's the requirement behind this?
I'm not found of spawning a new goroutine for every compressed request, mainly if it's not even necessary.
Maybe we can do this some other way

@jetsanix can you check if this works for you? https://github.com/sschepens/goreq/commit/9f1aaf5b4e3241bb7dc43d1516563482a3103808

@sschepens Thank you for your report, It worked for me!


Not able to download service worker files. the URL is a blank page

[Enter feedback here]
To install service worker, the docs asks to go to this URL and download couple files but the page is blank
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 18b321d1-e8fa-e95a-3345-1407b585cc63
Version Independent ID: def7c23d-f888-3ab6-60a3-95ebb27cc807
Content: Get started with Progressive Web Apps (Chromium) - Microsoft Edge Development
Content Source: microsoft-edge/progressive-web-apps-chromium/get-started.md
Product: microsoft-edge
Technology: pwa
GitHub Login: @MSEdgeTeam
Microsoft Alias: msedgedevrel
https://www.loom.com/share/976bc48d9a93482e8ee2197ce6f926ba

Thanks for the feedback. Indeed the link is broken. I filed an issue about it a few days ago, so I will close this one since it's a duplicate. See #1351.

Thank you! And my bad that i didn't check the existing issue before making one. Thanks again for looking into it.


Feature Request: return errors to stderr instead of stdout

box cli returns all http status errors returned from box (>400) to stdout instead of stderr
This makes it difficult to handle exceptions since I need to parse each JSON response returned to the standard output.
If http errors could be returned to stderr this would make it a lot easier to handle errors within my script.

This issue should be fixed with this current release v1.2.0 @Band-Aid


is there any flutter / dart port?

Needs a flutter port, there are many good local storage packages available for flutter which can be used to do the implementation

I'm not aware of any SymSpell Dart port. Perhaps one could use the dart:js library to access an SymSpell Javascript port.

Do you have a Solution for that already? My try would be to use Dart:FFI, but not sure if i am able to convert the c++ into plain c


Better scheduler.initialise

If there's an error on the init don't try again straight away but back off for 1 second
Respect the context that is passed in.
In schedulerapp.go move the deferred database close to after we've checked for an error, else we'll attempt to close a non existent connection.

Codecov Report
Base: 46.82% // Head: 46.82% // Increases project coverage by +0.00% :tada:

Coverage data is based on head (bf1aec9) compared to base (b99289a).
Patch coverage: 54.54% of modified lines in pull request are covered.

: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
Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2144   +/-   ##
=======================================
  Coverage   46.82%   46.82%
=======================================
  Files         222      222
  Lines       32470    32474    +4
=======================================
+ Hits        15204    15207    +3
- Misses      16056    16059    +3
+ Partials     1210     1208    -2

Flag
Coverage Δ

armada-server
46.82% <54.54%> (+<0.01%)
:arrow_up:

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

Impacted Files
Coverage Δ

internal/scheduler/schedulerapp.go
0.00% <0.00%> (ø)

internal/scheduler/scheduler.go
74.16% <60.00%> (+<0.01%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


ci: running tests on multiple go versions

Closes https://github.com/aquasecurity/libbpfgo/issues/296

Thanks so much for doing this!

@rafaeldtinoco ping~

Sorry been out sick for sometime, returning slowly.

Loved it. Thanks!


codeclimate configs

Codecov Report

Merging #6 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master     #6   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          20     20
  Lines         164    164
  Branches       23     23
=====================================
  Hits          164    164

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 5c74910...b66b1bf. Read the comment docs.


Upgrade the version of governance

Governance version will be upgraded by https://github.com/wso2/product-is/pull/10949. Hence closing this.

Governance version will be upgraded by https://github.com/wso2/product-is/pull/10949. Hence closing this.


Create CODEOWNERS file for SDK

Note: holding off on merge until we get the rest of the @stainless-bot autorelease settings in to minimize the timeframe where signoff and a manual package deployment are needed.


Using a socket to update the plot caption

Heya,
I'm using GLVis + MFEM for my research, and have an application that is continuously pushing new solutions to GLVis through a socket, like you are doing in your MFEM example code.
I found in your GLVis CHANGELOG file a reference to an undocumented socket command plot_caption, which I've been trying to get to work.
My current code is (somewhat like) the one from your examples:
char vishost[] = "localhost";
int  visport   = 19916;
socketstream sol_sock(vishost, visport);
sol_sock.precision(8);
sol_sock << "solution" << endl << *mesh << solution << flush;

How do I push a new plot_caption? I tried a couple of things, like
sol_sock << "solution" << endl << *mesh << solution << flush;
sol_sock << "plot_caption" << endl << "heya" << flush;

sol_sock << "solution" << endl << *mesh << solution << flush;
sol_sock << "plot_caption" << endl << "heya" << endl << flush;

sol_sock << "solution" << endl << *mesh << solution << endl
         << "plot_caption" << endl << "heya" << endl << flush;

but these all mess up GLVis more or less completely, in the sense that it tries to read the next mesh data into the caption.
Is there something I am missing?

Hi @Jannertje,
The plot caption itself has to be quoted -- the first non-whitespace symbol after plot_cation is treated as the quoting symbol. For example, in you code you can single quotes: instead of "heya" use "'heya'". This is done so that you can have almost any combination of symbols in the caption while keeping the parsing code in GLVis simple.
Hope this helps,
Veselin

See also https://github.com/mfem/mfem/blob/master/examples/ex17.cpp#L297-L303


glibc check too intrusive

The check for glibc should not fail, but assume that -lrt is not required.  This will simplify cross-building (where try_run doesn't work) and support alternative libc such as musl.

Thanks for the report!  Should be fixed now.

It works with musl now, but cross-compiling still fails:
CMake Error: TRY_RUN() invoked in cross-compiling mode, please set the following cache variables appropriately:
   GLIBC_DETECTION (advanced)
   GLIBC_DETECTION__TRYRUN_OUTPUT (advanced)
For details see /builddir/ndhc-20150212/build/TryRunResults.cmake
Could not detect glibc version.

Perhaps the whole try_run should be guarded with if(CMAKE_CROSSCOMPILE).

I've disabled the glibc version detection when CMAKE_CROSSCOMPILE is set.  Please let me know if there's still trouble when cross-compiling.

My bad, the flag is actually called CMAKE_CROSSCOMPILING. When I replace this, it works fine.


feat: support TriggerAuthentication properties from ConfigMap

This documents how to use ConfigMaps in TriggerAuthentication as implemented by https://github.com/kedacore/keda/pull/5111
Checklist

[x] Commits are signed with Developer Certificate of Origin (DCO)

see also: https://github.com/kedacore/keda/issues/4830

I will investigate the failure a little bit later but perhaps some @zroubalik or @JorTurFer can point me in the right direction here.
3:36:03 PM:   target does not exist --- docs/2.13/authentication-providers/configmap/index.html --> /docs/2.12/authentication-providers/configmap/
3:36:04 PM: Failed during stage "building site": Build script returned non-zero exit code: 2

does this mean it has to be in 2.13 as well as 2.12? I tried to follow the contrib docs
https://github.com/kedacore/keda-docs#writing-documentation-for-a-new-authentication-provider
$ hugo new --kind provider content/docs/2.13/authentication-providers/configmap.md
WARN  DEPRECATED: Kind "taxonomyterm" used in disableKinds is deprecated, use "taxonomy" instead.
Content "/home/wozy/projects/go/src/github.com/kedacore/keda-docs/content/docs/2.13/authentication-providers/configmap.md" created

I will investigate the failure a little bit later but perhaps @zroubalik or @JorTurFer can point me in the right direction here please?
3:36:03 PM:   target does not exist --- docs/2.13/authentication-providers/configmap/index.html --> /docs/2.12/authentication-providers/configmap/
3:36:04 PM: Failed during stage "building site": Build script returned non-zero exit code: 2

does this mean it has to be in 2.13 as well as 2.12? I tried to follow the contrib docs https://github.com/kedacore/keda-docs#writing-documentation-for-a-new-authentication-provider
$ hugo new --kind provider content/docs/2.13/authentication-providers/configmap.md
WARN  DEPRECATED: Kind "taxonomyterm" used in disableKinds is deprecated, use "taxonomy" instead.
Content "/home/wozy/projects/go/src/github.com/kedacore/keda-docs/content/docs/2.13/authentication-providers/configmap.md" created

The problem is not on your side, we just need to add each new page in https://github.com/kedacore/keda-docs/blob/main/.htmltest.yml like this: https://github.com/kedacore/keda-docs/commit/e03ee096b70db4300216c829c6d12c57cff81025

@wozniakjan could you please add an exception for this file into the .htmltest.yml file?


AWS Lightsail

VPC peering: https://lightsail.aws.amazon.com/ls/docs/how-to/article/lightsail-how-to-set-up-vpc-peering-with-aws-resources

Disk https://lightsail.aws.amazon.com/ls/docs/overview/article/elastic-block-storage-and-ssd-disks-in-amazon-lightsail

Code Example: https://github.com/aws/aws-sdk-go/blob/v1.7.0/service/lightsail/examples_test.go

IAM

I had to manually create an IAM Policy and then associate with IAM role.

{
    "Version": "2012-10-17",
    "Statement": [
        {
            "Sid": "VisualEditor0",
            "Effect": "Allow",
            "Action": "lightsail:*",
            "Resource": "*",
            "Condition": {
                "BoolIfExists": {
                    "aws:MultiFactorAuthPresent": "true"
                }
            }
        }
    ]
}

[ ] Update data files

We have decided to remove support for this.