Dartpad Issue - Unsupported Import

error is as appears:
line 1 - Unsupported import(s):
(package:flutter_test/flutter_test.dart)

+1, seeing the same thing for the layout basics codelab on Chrome for mac version 90.0.4430.93 (Official Build) (x86_64)

Thanks for reporting this.
@srawlins it looks like one of the recent changes for imports may have excluded package:flutter_test/flutter_test.dart, which some of our Flutter codelabs rely on for their exercise completion checks.
Example: https://flutter.dev/docs/codelabs/layout-basics

I'm on it. Apologies for the breakage.

I would think that the culprit of this breakage is https://github.com/dart-lang/dart-services/commit/3cf859547c5276e638ccc8af1357c294c07fabfb but I don't think that change is deployed yet. Possibly https://github.com/dart-lang/dart-services/commit/f5f24c52a12e1cc764634dc9581cd8cc7a94b958 but I don't see how that caused the problem. I've uploaded two PRs which should help but may not necessarily solve. Still investigating.

Actually, it has auto-deployed on the null safety endpoint. Look at https://pantheon.corp.google.com/run/detail/us-central1/dart-service-cloud-run-null-safety/revisions?project=dart-services
The non null-safe version is still running on the prior version: https://pantheon.corp.google.com/run/detail/us-central1/dart-service-cloud-run/revisions?project=dart-services
We can roll back the null-safe end point to dart-service-cloud-run-null-safety-00014-nac to revert

I see, thanks @domesticmouse! I also just landed #1868 .

When I paste that codelab into local DartPad with my fixes, seems to work (though I see a very strange "line 3 • Unused import: 'package:flutter_flutter_test.dart'." analysis error)

I believe that there is a test script added to the end of the code that is submitted for compilation. I guess this code is not added for analysis.
Worth raising a bug.
/cc @RedBrogdon as he probably remembers more of this than I do.

This should now be fixed with https://github.com/dart-lang/dart-services/pull/701 deployed to production.
Please comment if this isn't the case so we can continue figuring out what isn't happy.
Thanks all!

Thank you for the fix!

Thanks for raising the bug @ChillyChaos!


refactor: rename test_data to testdata and some lint fixes

Signed-off-by: ankitm123<EMAIL_ADDRESS>testdata is the idiomatic way to store test related data in go: https://pkg.go.dev/cmd/go/internal/test
The go tool will ignore a directory named "testdata", making it available
to hold ancillary data needed by the tests.

/hold

/hold cancel

/retest lint

/test lint

/cc @msvticket @babadofar @tomhobson

/approve


pass host as command line arguments

Currently, require project/example.com/path/to/file for matching file.
return path.join(baseDir, urlObject.host, urlObject.pathname);

https://github.com/azu/node-cocproxy/blob/ab1461df6add9019c5bca0dca5b28fc427f54e32/lib/local-finder.js#L14

I want to specify example.com by command line argument instead of directory.
UseCase
When Alice  is developing in alice.github.io and Bob is developing in bob.github.io, It it difference host between both.
So, I want to use command line argument instead of directory name for specifing host.

Implementation :

--host : pass host value from command line


Migrate Algolia DocSearch account

Migrate Algolia account on archived Docusaurus v1 site: archive.reactnative.dev
See also https://v1.docusaurus.io/blog/2021/11/05/algolia-migration

@Simek I invited you to a react-native-archive project on the DocSearch dashboard. Let me know if I should invite anyone else at Meta.
v1 site search is still working (readonly), but @shortcuts still recommend to migrate because some day that readonly index might be deleted
Let's wait for the preview to see if search still works (crawl is in progress)

Let me know if I should invite anyone else at Meta.

Probably @lunaleaps + me should be enough 👍

No idea why but Netlify does not add the deploy preview URL...
Anyway, here it is: https://deploy-preview-2990--react-native-archive.netlify.app/

Thank you for the help with the migration Sebastien! 👍

I invited you to a react-native-archive project on the DocSearch dashboard.

Thank you again! I can see both project in the dashboard now.

No idea why but Netlify does not add the deploy preview URL...

I'm not sure if bot has been configured for the archive website deploys correctly, also CI shows the extended workflow, instead of just one deploy step. It looks likes the test workflow is outdated too, but IIRC the archive has not ben changed/edited almost since it creation within the V2 migration.

Anyway, here it is: https://deploy-preview-2990--react-native-archive.netlify.app/

I can confirm that search one the archive website works fine, so this should be safe to merge.


Add a filter provider for TempDataAttribute to RazorPages

Related to #5600
Controllers have a TempDataApplicationModelProvider which configures the SaveTempDataPropertyFilter. RazorPages currently has a workaround but needs a similar filter provider.
Remove TempDataPropertyProvider class as discussed here
cc @rynowak @pranavkm

cc @DamianEdwards

Currently the workaround doesn't work on PageModel instances, so this feature is blocked right now.

@rynowak @Eilon we need to triage and assign this one

@ryanbrandenburg - for now let's try to figure out if the hardcoded approach can be applied to the PageModel as well.
Treat this as a category 2.

OK by me 👍

We still have cleanup to do here, we're just not implementing filters for pages in preview-1.

dfe04bc917f984a928d0975a57ac74ad1437b312


Additional images

I'm trying to replicate this in a slightly different way. Annotated new images, so I figured I'd contribute...

That's merged. Thanks 👍


Fix symbol archiving, blocking VS insertions

This is kind of a strange break but here's the details:

When moving the PTVS build to completely VS 2022, I created a new AzDO environment library at https://devdiv.visualstudio.com/DevDiv/_library?itemType=VariableGroups&view=VariableGroupView&variableGroupId=381&path=PTVS-Dev17. I did this so the old library wouldn't be affected by my changes.
This library contains two variables used by symbol publishing: ArtifactServices.Symbol.AccountName and ArtifactServices.Symbol.PAT. These values are read from the environment by the PublishSymbols@2 task in our build yaml, which only runs on non-PR builds.
When you clone an azdo environment library, secret values don't seem to clone. That means the PAT that was copied was no longer valid. So I made a new AzDO PAT with symbol.read and symbol.write permissions.

When running the build, the Publish symbols step is failing due to authorization issues. See https://devdiv.visualstudio.com/DevDiv/_build/results?buildId=5499226&view=logs&j=12f1170f-54f2-53f3-20dd-22fc7dff55f9&t=8e355ed5-37f3-5c2a-bb8d-5efc727482d6
I reached out to VSEng and someone suggested using $(System.AccessToken) instead of the generated PAT. This did get me past the authorization errors, but the VS insertion fails later on saying that symbols are not archived. Symbol publishing and symbol archiving are two different things.
I spoke with Rick Krause and learned that the reason archiving was not failing before is because when using the microsoft account name and a generated PAT, the symbols are publicly available, which means they are archived automatically.
So we have two options:

Fix the authorization issues when publishing symbols, which will automatically fix the symbol archiving issue (preferred solution). I need help from VSEng on this, as we don't own this service and I have no idea why authorization is failing. I've reached out for help.
Manually archive symbols after we publish them. There are docs on this, and it's definitely doable, but I'd rather just do what we did before.

The issue was the PAT was in the wrong organization. Instead of generating the PAT at https://devdiv.visualstudio.com/_usersSettings/tokens, you have to do it at https://dev.azure.com/microsoft.

Re-opening this because even though VS says symbols are archived, symchk says otherwise. Waiting to hear back from VS PM to see if this is an issue. If it is, I'll need to add the Symbol Archiving task to our yaml. See https://dev.azure.com/devdiv/DevDiv/_apps/hub/ms.vss-build-web.ci-designer-hub?pipelineId=13298&branch=main for an example.

Added the symbol archiving task and kicked off the build at https://devdiv.visualstudio.com/DevDiv/_build/results?buildId=5527922&view=results, but it failed because we don't have space on the fileshare to save symbols.
Put in a ticket for the space to be created for us.
Once that gets completed, I'll re-run the build to make sure the archiving succeeds.
24 hours after that, I'll verify symbols are archived by running symchk.exe /s SRV*http://msdl.microsoft.com/download/symbols <FullPathToBinary>, using any of the binaries in the Symbols build artifact from our build.

I spoke with Rick about permissions issues I was having and it turns out Archiving Symbols can't be run as part of a build pipeline, because build pipelines don't have access to on prem file shares. So I've added this as a step in our release pipeline at https://devdiv.visualstudio.com/DevDiv/_release?_a=releases&view=mine&definitionId=2762.
Ran the release and the Archive symbols task seems to have worked: https://devdiv.visualstudio.com/DevDiv/_releaseProgress?_a=release-environment-logs&releaseId=1225760&environmentId=6253805
I'll verify on Monday to see if the symbols are publicly available.

Looks like it worked.
I downloaded the symbols artifact from https://devdiv.visualstudio.com/DevDiv/_build/results?buildId=5528600&view=artifacts&pathAsName=false&type=publishedArtifacts, copied them to c:\temp, then ran the following command:
symchk.exe c:\temp /v /s SRV*http://msdl.microsoft.com/download/symbols
Got 62 passed and ignored files.


fix: BindHeader see #107

fix #107

Codecov Report

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

@@            Coverage Diff             @@
##           master     #108      +/-   ##
==========================================
- Coverage   93.35%   93.34%   -0.01%
==========================================
  Files          25       25
  Lines        1023     1022       -1
==========================================
- Hits          955      954       -1
  Misses         44       44
  Partials       24       24

Impacted Files
Coverage Δ

decode/decode_core.go
93.37% <100%> (-0.05%)
:arrow_down:

Continue to review full report at Codecov.

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


test PR with Github API

Please pull this in!

Docs Build status updates of commit a4f6c5e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

tutorial-manage-data-disk.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 194cab8:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

tutorial-manage-data-disk.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit df2f27e:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

node-azure-tools.md
:white_check_mark:Succeeded

tutorial-manage-data-disk.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 10b6430:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details
Dependent Files

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/Peachying/testinlineedit, working directory: W:\o1r3-s, target branch: HEAD, target branch commit: 18bad12de12e616210fb91f6a989a82d74d05dcb, source commit: 10b64306ef4532197fc99c83d13bb9de80350ac1.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit b0fd9bd:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details
Dependent Files

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/Peachying/testinlineedit, working directory: W:\0y0b-s, target branch: HEAD, target branch commit: 18bad12de12e616210fb91f6a989a82d74d05dcb, source commit: b0fd9bd1534ef844849a2fa816b81a929fd2d735.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit b88b202:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details
Dependent Files

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/Peachying/testinlineedit, working directory: W:\nsxm-s, target branch: HEAD, target branch commit: 18bad12de12e616210fb91f6a989a82d74d05dcb, source commit: b88b2029fd8d2ab378a018a56f1d7ee9edd790b1.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit 92aeb8c:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details
Dependent Files

:x:Error

Details

[Error] Cannot sync git repo to specified state. Please make sure the pull request is mergeable. Details: Git merge results in conflict, repository remote url: https://github.com/Peachying/testinlineedit, working directory: W:\sfqu-s, target branch: HEAD, target branch commit: 18bad12de12e616210fb91f6a989a82d74d05dcb, source commit: 92aeb8ccb603cb6adedec4611dafb1fe731c0bd7.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

Docs Build status updates of commit f0f95a2:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details
Dependent Files

node-azure-tools.md
:white_check_mark:Succeeded

tutorial-manage-data-disk.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Fede dev

Added a couple functions to core to get the ancestors and descendants of a given structure.
Should be fairly straightforward and it passes tests locally.

I'm again not sure why there's all of these commits and why it marks a couple files as being added...
It might be easier if we just copy paste the functions/tests I've added directly into master and then I trash the FedeDev branch which seems to be all messed up...

I would not mind! Just one thing: test at line 128 I assume you are testing on full allen? I don't remember such regions in the example

aha! I thought the example atlas had metadata about all regions but meshes only for some... I'll check what it looks like on the test atlas and update. Will add directly to master and close this PR.

No, but probably makes sense to have what you described in the next version of the Example, to support more thorough testing

ok, for now I've edited the tests to use what's in the example

I commited directly to master, will close this now.


UAD 3.0 Main Pull Request

As always, all other branches must merge into this one before going into mega. Happy contributing!

Suggestion: either remove jQuery as a dep or move it to a devDep


Fix cmake build error: No such file or directory

Steps to reproduce on OS Linux:

Go to duckdb directory
$ mkdir build
$ cd build
$ cmake ..
$ cmake --build .

duckdb/benchmark/include/sqlite_benchmark.hpp:14:10: fatal error: sqlite3.h: No such file or directory
 #include "sqlite3.h"
          ^~~~~~~~~~~
compilation terminated.

Good catch, indeed this failed on systems without sqlite headers installed (which are rare)


How can I add images, fonts, etc?

I would refer you to rollup-plugin-copy as a possible starting point. This utility generates a generic scaffold quickly. Custom configuration is left up to the user.


Globally specify operation serializer options

We'd like to globally specify OmitNullValues as a default option when serializing, or perhaps even be able to specify endpoint specific options...

What would be the use case for this? Perhaps there is a way we can solve it without adding any more configuration options.

Current use case is as I mentioned, here's my Spine branch changes:
if isNewResource {
    URL = router.URLForResourceType(resource.resourceType)
    method = "POST"
    options = [.IncludeToOne, .IncludeToMany, .OmitNullValues]
} else {
    URL = router.URLForQuery(Query(resource: resource))
    method = "PATCH"
    options = [.IncludeID, .OmitNullValues]
}

Why do you want to omit null values? If it's because of overwriting attributes that weren't fetched, I recommend trying the dirty-checking branch.


Auth with more usernames and improve errors

This commit is an attempt to improve the error message from failed
authentication attempts as well as attempting more usernames. Right now we only
attempt one username, but there are four different possible choices we could
select (including $USER which we weren't previously trying).

Closes #2399

r? @huonw
(rust_highfive has picked a reviewer for you, use r? to override)

r? @brson

@bors r+

:pushpin: Commit c6149e3 has been approved by brson

:hourglass: Testing commit c6149e3 with merge 627855a...

:broken_heart: Test failed - cargo-linux-64

@bors: r=brson f66d716

:hourglass: Testing commit f66d716 with merge e721289...

:sunny: Test successful - cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64


Test reports

I'm suggesting to add some test reports in workflow for better perception of information.

To do this, you need to visualize the trx reports that are generated in the workflow after running unit tests. For visualization, I suggest using test-reporter action. The current workflow is here.


Not found tsconfig.json if open project on parent directory

Hello,
I run into this issue when I try to change from tslint to ESLint in VSCode with an Angular project.
The project directory structure is this:
|- .git/
|- angular/
|  |- src/
|  `- .eslintrc.json
|- laravel/
`- other-files.txt

If I open VSCode like this, I get some errors on every .ts files with this message:
Parsing error: Cannot read file '/home/djw/code/myproject/tsconfig.json'.  eslint
So I need to change the .eslintrc.json file like this:
      "parserOptions": {
        "project": [
          "tsconfig.json",
          "e2e/tsconfig.json"
        ],
        "tsconfigRootDir": "angular/",  // THIS LINE IS ADDED
        "createDefaultProgram": true
      },

Now the VSCode ESLint plugin works fine.
But if I try to run ng lint command, I get an error:
  0:0  error  Parsing error: Cannot read file 'angular/tsconfig.json'
Any idea how can I fix this issue?

@netdjw I am not very familiar with ng lint. What you can try for VS Code is to set the eslint.workingDirectoies setting to angular to ensure the working directory is setting correctly.

@dbaeumer thanks, I added this code in my project folder to this file: .vscode/settings.json
{
    "eslint.workingDirectories": [
        "angular"
    ]
}

Now the error is gone.


[Feature][3.5][Ready] Select_and_order field type

Merged and polished #1626 . Adds a new field type, select_and_order, which shows two columns:

to the left are the items you have selected, and you can reorder them
to the right are the items you can select (by dragging to the left column)

Screenshot

Definition
As mentioned in the original PR, it can be used exactly like select_from_array:
            [
                'name' => 'featured',
                'label' => 'Featured',
                'type' => 'select_and_order',
                'options' => [
                    1 => "Option 1",
                    2 => "Option 2"
                ]
            ]

Or:
            [
                'name' => 'featured',
                'label' => 'Featured',
                'type' => 'select_and_order',
                'options' => Product::get()->pluck('title','id')->toArray()
            ]

The submitted value (in this case featured) should be casted to an array in the model:
            protected $casts = [
                'featured' => 'array'
            ];

And its value will be stored as JSON in the db: ["3","5","7","6"]
Thoughts, anyone? Is it clear enough from the design that you can drag items from the right to the left? Without having labels for each column? I'd hate to insert new translated lines just for this...

Could an arrow in between the columns help?
I’m not into design so take my suggestions with skepticism.

Arrows!! Of course, that’s what was missing :-))
Thank you @joaojoyce - I think you’re right. Even better than between the columns (which would only say “move items between the columns”) I think would be arrows on each element. I expect it to say “move this thing around, or in the other container”.
I’ve polished it a bit more and added arrows, I think we finally have it :-)

Merged into release 3.5 branch. Can't wait to launch it :-)


[Style] Update Slider Background Color in Dark Mode to Match the Theme

What's the style idea?
Description
Currently, the background color of the slider in dark mode is gray, which does not blend well with the overall dark theme. This creates a visual inconsistency and negatively impacts the user experience. The slider background color should be updated to a more appropriate color that aligns with the dark mode theme.
Steps to Reproduce

Enable dark mode in the application.
Navigate to any section where the slider component is used.
Observe that the slider background color is gray, which contrasts poorly with the dark theme.

Expected Behavior
The slider background color in dark mode should be updated to a color that better complements the dark theme, such as a darker shade or a theme-appropriate accent color.
Actual Behavior
The slider background appears gray, which does not match the aesthetic of the dark mode.
Proposed Solution

Locate the CSS or styling rules for the slider in dark mode.
Update the background color to a darker or theme-accented color (e.g., #000080, #444, or a color specified in the design system).

Screenshots

Checklist

[X] I have checked the existing issues.
[X] I follow Contributing Guidelines of this project.
[X] I am a KWOC'24 contributor.

Code of Conduct

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

@Rakesh9100 I would like to work on this issue. Please assign the issue to me.


Google Drive ocamlfuse is very slow

Good morning,
i have install ocmlfuse on ubuntu 14.04 64 bit and is very slow for download and upload.
my config is:
max_cache_size_mb=512
new_revision=true
sqlite3_busy_timeout=500
read_only=false
conflict_resolution=server
verification_code=
drawing_format=png
document_format=odt
form_format=ods
docs_file_extension=true
keep_duplicates=false
presentation_format=pdf
download_docs=true
client_secret=
metadata_cache_time=60
client_id=
spreadsheet_format=ods
debug=false
umask=0o002
There is a method to speed ?
Tanks for help.

Second this.
I know this is old issue, I got only 100-150 kB/s with FUSE. In the other hand, I can upload 2-3 MB/s with browser. Tried changing the metadata_cache_time produces the same result.
My concern (for now) only for uploading though.

You can try setting async_upload=true, or you can try the new beta version (0.6.0). that you can install from a new PPA, if you are on Ubuntu.

Still, the speed very slow. Now it has around 400kB/s. Moreover, I uploaded from SSD, so I don't think the bottleneck is from the disk.

I think the problem is that FUSE uses blocks of 128KB (at maximum), so you have to wait for a (relatively slow) copy to end before the upload begins. If your use case requires frequent uploads of very big files, you should go with https://github.com/odeke-em/drive. You can then use my app to stream the uploaded files (if they contain videos, for example).

rclone also implements FUSE to mount cloud storage, one of them is google drive. I tried that before and mostly can get higher speed. I will look into the details later

OK, but from the source code you linked:

Limitations
This can only write files seqentially, it can only seek when reading.
Rclone mount inherits rclone's directory handling.  In rclone's world
directories don't really exist.  This means that empty directories
will have a tendency to disappear once they fall out of the directory
cache.

I think that rclone has a simple in-memory model, so it's optimized for uploading but has those limitations.

i'm experiencing the same issue with speed of uploading

Maybe the slowness is due to not using the database properly: https://github.com/astrada/google-drive-ocamlfuse/issues/308#issue-230220653. It fully syncs the database state to disk every 128 kilobytes.
Using libeatmydata hack forces unsafe, unsynced database mode and makes uploads fast.

@vi You mention a libeatmydata hack, do you have details? I'm also only getting ~100k/s with ocamlfuse on Ubuntu Disco (i'm doing a ~20,000 photo and video file upload)

@henricook In my case the bottleneck was (SQLite?) database that google-drive-ocamlfuse uses (or used) for internal bookkeeping. Every now and then (a file uploaded or a chunk of larger file uploaded) it commits transaction to that database and waits until it is fully persisted.
This wait is slow, especially with HDD. You typically want it to happen if something important is being save to database, e.g. a bookmark in your browser.
eatmydata on Linux turns off that "wait for database transaction is fully persisted" aspect, making SQLite belive that it persisted while in reality it is still in cache, allowing the application to continue earlier. This trades resilience against sudden system poweroffs for speed.
SQLite has a built-in "unsafe, unflushed" mode that skips it, achieving the same effect without using external modifier libraries. I expect google-drive-ocamlfuse to have an option for it, if not default for it.

Thanks for that explanation @vi - not what i expected. Since my goals were fast one-off upload of a few large folders i've switched to rclone and had a really pleasant time

Since version 0.7.0, I introduced config option metadata_memory_cache=true to cache metadata in memory and flush to sqlite periodically. And since 0.7.1, I introduced write_buffers=true to cache writes in memory (it's off by default because it's still experimental).


No active state for button

After clicking a button it's not changed as active. Also it's not possible to delete the formatting of the selected text by clicking the button again.
Please change this behavior so the buttons would work as the standard button.
Thanks!

Yes, we also have this problem.. How can we fix this?

Can you please provide the code you used to create this issue?

I'm also running into this issue; here's the source I'm using:
    new MediumEditor(document.getElementById('blog_content'),{
      buttonLabels: 'fontawesome',
      toolbar: {
        buttons: ['bold', 'italic', 'underline', 'strike', 'anchor', 'h2', 'h3', 'quote']
      },
      extensions: {
        'strike': new MediumButton({
          label: '',
          start: '<s>',
          end: '</s>'
        })
      }
    })

When I attempt to add and then remove the strikethrough, here's the resulting markup:

  <s>
    <s>
      <s>4567890</s>
    </s>
  </s>

If I keep clicking the button, it keeps adding additional tags, as @boesie mentioned.

I've noticed the problem and am trying to find a solution for it. Unforgenetly I haven't had any success yet.

I had a look at this a couple of days ago.
TL;DR: A more robust solution for returning selected text is necessary.
The problem seems to be that getCurrentSelection() doesn't actually grab the underlying HTML when you select only the part you want to affect.
If I have added my implementation of the bold button from above, the way to remove the styling as this library sits is to select additional text around the styled text. That is, if I have the text:

    Progress isn't made by early risers. It's made by lazy men trying to find easier ways to do something.

Then selecting 'lazy men' and calling getCurrentSelection() won't select lazy men, it only returns the text as rendered.
But if you selected 'by lazy men trying', since the  tags are within the selection, it works. I guess this is why timdown has gone on to create rangy.

That is where I started and actually have almost found a solution for the getCurrentSelection() method. There is only one case left where my concept fails. Unforgenetly further improvements will have to be made as well. I'll update this here in a few days
Concerning Rangy, even though it could work, I'd prefer not to use another library.

I'm still working on the replacement, this is my current approach, it has one mistake left, that the start/end locations sometimes are wrong.
function getCurrentSelection() {
    var sel = window.getSelection()

    if (!sel.rangeCount) return

    var range = sel.getRangeAt(0)
    var ancestor = range.commonAncestorContainer
    var parentValue = ancestor.nodeValue || ancestor.innerHTML

    var selected = parentValue.substring(range.startOffset, range.endOffset)

    if (parentValue.trim() === selected.trim()) {
        return ancestor.parentNode.outerHTML
    } else {

        if (range.startOffset === 0) {
            var outerHTML = ancestor.parentNode.outerHTML
            var withBegin = outerHTML.substring(0, outerHTML.indexOf(selected) + selected.length)
            return withBegin
        }

        if (range.endOffset === ancestor.length) {
            var outerHTML = ancestor.parentNode.outerHTML
            var withEnd = outerHTML.substring(outerHTML.lastIndexOf(selected))
            return withEnd
        }

        var container = document.createElement('div')
        container.appendChild(range.cloneContents())
        var containerHTML = container.innerHTML

	//if(containerHTML.charAt('0') === '<') containerHTML.substring(containerHTML.indexOf('/>'))
        //if(containerHTML.charAt(containerHTML.length) === '>') containerHTML.substring(containerHTML.lastIndexOf('<'))

        return container.innerHTML

    }

}


Storing JS BIOM data more efficiently

The JS BIOM table representation is structured as follows:
{sample1: [feature1, feature2, feature3, ...], sample2: [feature3, feature4, ...], ...}
...where sample1, feature1, etc. are the sample / feature IDs.
This approach inherently results in feature IDs being repeated multiple times, since they'll be listed for every sample they're in (a perfectly dense table with N samples will result in a JS representation where every feature ID is listed N times). If the input dataset isn't that huge and/or if the feature IDs are small strings this isn't a big deal, but when feature IDs are listed as sequences they represent (anywhere from a ~90 to ~500+ character string) this can inflate the empress.html (and the QZV) file sizes, because these long strings are listed multiple times.
A way to get around this problem would be finagling things so that each feature is assigned a unique numeric ID (starting at, say, 1, then going up to number of features), and the actual feature ID is stored only once somewhere in the data passed to Empress' JS code (maybe with the feature metadata, when we add that back in). This way, referencing a feature in the BIOM table won't require nearly as much space in the empress.html file.
Another way to get around this problem (this might be easier) would just be storing the transpose of the table --
{feature1: [sample1, sample3, ...], feature2: [sample1, sample5, sample6. ...], ...}
... since sample IDs are usually much shorter than feature IDs :) This is the approach Qurro currently takes.
Anyway, not urgent or anything, but this would probably help with the issue brought up in #74.
Edit: Per discussion with @wasade, the redbiom data model may be a useful example of how to implement this.

Using numbers instead of sequences would be a good way to cope with this.
Thanks!
Yoshiki
On (Apr-20-20|12:23), Marcus Fedarko wrote:

The JS BIOM table representation is structured as follows:
{sample1: [feature1, feature2, feature3, ...], sample2: [feature3, feature4, ...], ...}
...where sample1, feature1, etc. are the sample / feature IDs.
This approach inherently results in feature IDs being repeated multiple times, since they'll be listed for every sample they're in (a perfectly dense table with N samples will result in a JS representation where every feature ID is listed N times). If the input dataset isn't that huge and/or if the feature IDs are small strings this isn't a big deal, but when feature IDs are listed as sequences they represent (anywhere from a ~90 to ~500+ character string) this can inflate the empress.html (and the QZV) file sizes, because these long strings are listed multiple times.
A way to get around this problem would be finagling things so that each feature is assigned a unique numeric ID (starting at, say, 1, then going up to |number of features|), and the actual feature ID is stored only once somewhere in the data passed to Empress' JS code (maybe with the feature metadata, when we add that back in). This way, referencing a feature in the BIOM table won't require nearly as much space in the empress.html file.
Another way to get around this problem (this might be easier) would just be storing the transpose of the table --
{feature1: [sample1, sample3, ...], feature2: [sample1, sample5, sample6. ...], ...}
... since sample IDs are usually much shorter than feature IDs :) This is the approach Qurro currently takes.
Anyway, not urgent or anything, but this would probably help with the issue brought up in #74.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://urldefense.com/v3/__https://github.com/biocore/empress/issues/158__;!!Mih3wA!R_cNwui6Y8_WFnITcc3EmAULgnSBHbSl3E8lRSkv8oj_XN7xWUzk8INYycraRg4$

Basically something like:
sample_lookup = {'sample_name': <unique_integer>, ...}
sample_lookup_inverse = {<unique_integer>: 'sample_name'}
feature_lookup = {'feature_name': <unique_integer>, ...}
feature_lookup_inverse = {<unique_integer>: 'feature_name'}
sample_counts = {sample_idx: {'counts': [count, count, count, ...], 'features': [feature_idx, feature_idx, ...]}

...where "counts" and "features" are in index order

@wasade Thanks so much! As I'm implementing this, there are two changes that I think might simplify things with this approach. Wanted to post them here to check that I'm not falling into any pitfalls that you may have already dealt with for redbiom.

I think it might be possible to simplify the lookup storage by converting sample_lookup_inverse and feature_lookup_inverse to arrays:

s_ids = ['s1', 's2', 's3', ...]
f_ids = ['f1', 'f2', 'f3', ...]

Each sample or feature's unique ID would just be its 0-based position in the sample or feature array (0 -> s1, etc.). For the sake of things like feature-ID searching, Empress would still need to maintain a non-inverse feature lookup object mapping feature names to their f_ids index -- I guess it's either that or sort the feature IDs in that array in alphabetical order from the start, and then use e.g. binary searching on f_ids to locate a feature's index in the JS. (But that sounds like overkill.)

For the sample_counts internals, do you see a problem with consolidating counts and features into the same object? Something like

sample_counts = {sample_idx: {feature_idx: ct, feature_idx: ct, ...}, sample_idx: {...}, ...}

... Where entries in each sample_idx object where ct is 0 are just omitted. I think this should save some space, but I'm not 100% confident of that.

For the sake of completeness, this is roughly the same approach that's followed in Emperor's Plottable class (see the idx property).

quick update on how this is going -- this branch of my fork compresses the table, sample, and feature metadata mostly as described above before storing them in the HTML.
Although the branch is untested (and broken, since I haven't gotten the JS set up to work with the compressed data yet), the compressed data does save a lot of space: for the HMP dataset Kalen shared with me, the empress.html file from the uncompressed QZV is 47 MB, and the empress.html file from the compressed QZV is 26 MB. There are definitely further ways we could bring this number down, also (e.g. minifying the HTML).

This is great to hear @fedarko. Have you noticed any performance improvements on the UI side of things? Are loading times faster?

Sorry, I'm still working on getting the JS to use the compressed data, but I'll update here when I have that working :)

👍 no worries - exciting times!


Add nix support for missing host function calls.

At a quick glance, the following functions are not yet provided by nix, but would be useful to have in nix for use by wasi-common: mkdirat, unlinkat, renameat, linkat, clock_getres, clock_gettime, posix_fallocate, posix_fadvise.
Currently code in wasi-common and lucet-wasi is calling libc interfaces for these functions directly, which requires unsafe code. Having nix provide these will mean the unsafe code is shifted to nix instead. This may not seem like a win on a theoretical level, but the more we can push unsafe code into dedicated utilities that do simple unsafe things with safe, testable, interfaces, the easier it is to audit everything.
I've marked this as "help wanted" -- this is not an urgent project, but it is a nicely scoped, straightforward, and subdividable project, and it's a good chance to see some of the low-level parts of the software stack that wasi-common is built on. And mentoring is available :-).
Additional resources:

Documentation for these functions in POSIX can be found here: http://pubs.opengroup.org/onlinepubs/9699919799/
Here's an example of implementing a very simple function in nix, though the functions above are more complex and will need testcases :-).

Given that we've effectively decided to abandon using nix in wasi-common, and use our own in-house thin, low-level wrapper crate for used Unix syscalls (aka yanix, see #649), I'm closing this issue.


Use worfklow for publish

See https://github.com/binki/binki-action-publish-create-react-app-to-pages

Fixed by https://github.com/binki/binki-qrcode-labeler/commit/73dc0f88a1a09efc8a6837a3935edaa6878d6bc7


[beta.11] - preload tabs set a black background to tabs

Short description of the problem:
If I add preloadTabs="true" input to the ion-tabs tag the first tab loads fine but the other have a black background. This only happens on iOS
What behavior are you expecting?
Tabs should show have white background
Steps to reproduce:

Run a new project  (ionic start test --v2)
Add this input preloadTabs="true" to the ion-tabs tag
Run the project on iOS

The code below sets a black background to the tabs

Which Ionic Version? 1.x or 2.x
Ionic 2 beta.11
Run ionic info from terminal/cmd prompt: (paste output below)
Cordova CLI: 6.3.1
Gulp version:  CLI version 1.2.2
Gulp local:   Local version 3.9.1
Ionic Framework Version: 2.0.0-beta.11
Ionic CLI Version: 2.0.0
Ionic App Lib Version: 2.0.0
ios-deploy version: 1.8.6
ios-sim version: 5.0.8
OS: Mac OS X Yosemite
Node Version: v5.5.0
Xcode version: Xcode 7.2.1 Build version 7C1002

Hello! Thanks for opening an issue with us! In beta.12 we have decided to remove the preloadTabs options as it is not needed anymore with aot compile and removing it gets rid of complexity. Because of this I will be closing this issue. Thanks for using Ionic!


DOCS Updated PHP compat statement

There's some additional content further down the file, do you mind updating that too, please?

Does SilverStripe 3 work with PHP 7.3+?
SilverStripe 3.7.3 or greater will work with PHP 7.2 and 7.3. Silverstripe 3.6.x is compatible with PHP 7.0 and 7.1, and SilverStripe 3.5 or lower will only work with PHP 5.3 - 5.6.

Notes on the support was included here: https://docs.silverstripe.org/en/3/changelogs/3.7.4/

Thanks for that Bryn, I've actually removed that section. We shouldn't make statements about supporting versions of PHP which have gone EOL yonks ago :)


vSphere deletes PVs when deleting single node

What kind of request is this (question/bug/enhancement/feature request):
Bug Report
Steps to reproduce (least amount of steps as possible):

Create a K8S-Cluster on vSphere (inkl. steps to enable vSphere storage provisioning)
Create a workload with a volume claim
Wait for it to be deployed
Delete the node (in node view) where the pod is running on

Result:

VM gets deleted
All volumes connected to this VM get deleted as well
Rancher recreates the VM according to node template
Rancher/K8S tries to redeploy the workload, but fails, because the volume has been deleted earlier
PVC needs to be deleted in order to proceed to a working state again
data in the original volume is lost

Other details that may be helpful:
From my limited understanding it looks like Rancher is telling Vsphere to "Delete from disk" the VM, which causes vSphere to delete the VM and all VMDKs currently attached to it. But the volumes created by the provisioner should not be deleted.
This is also a user error, as the node should be drained before deleting it, but in the case of a node not responding, this might not be possible any more.
Maybe Rancher should disconnect all volumes before deleting the VM.
Environment information

Rancher version 2.1.6
Installation option : single install

Cluster information

Cluster type: Infrastructure Provider
Machine type: VM and specifications: 7 VMs, 2 CPUs each and 4GB RAM each
Kubernetes version (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"10", GitVersion:"v1.10.5", GitCommit:"32ac1c9073b132b8ba18aa830f46b77dcceb0723", GitTreeState:"clean", BuildDate:"2018-06-21T11:46:00Z", GoVersion:"go1.9.3", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"11", GitVersion:"v1.11.6", GitCommit:"b1d75deca493a24a2f87eb1efde1a569e52fc8d9", GitTreeState:"clean", BuildDate:"2018-12-16T04:30:10Z", GoVersion:"go1.10.3", Compiler:"gc", Platform:"linux/amd64"}

Docker version (use docker version):

17.3.2

gzrancher/rancher#8983
gzrancher/rancher#11020
gzrancher/rancher#12087
gzrancher/rancher#11675
gzrancher/rancher#12210

https://github.com/rancher/rancher/issues/15065
This is still an issue.

This is actually an issue with vSphere and not Rancher/Kubernetes, fully.
If you power off a VM in the vSphere UI then "Delete from disk", it will delete all of the attached VMDK's to it. I am not sure what Kubernetes/Rancher could do to prevent this, other than maybe add logic into Rancher to drain the node/ensure detached VMDK's before deleting it.

https://github.com/kubernetes/kubernetes/issues/75342

well on the side of vsphere this is intended/documented behaviour.
Is rancher deleting the VM or is k8s doing that job?
If rancher is responsible for deleting the VM then it should also make sure that this is a safe operation.

@Oats87
Is this going to be fixed? Has your vSphere team at Rancher never been hit with this problem? It seems like a pretty major one that would deserve a bit more attention given it's ramifications on data loss and been untended to in a year...
If you are going to put a fancy +/- button on the UI, with one click and no confirmation or warning.. that implies that a seamless and safe operation is going to occur and Rancher has the responsibility to make it work correctly. Today this is catastrophic in scale down operations.
This is a Rancher issue more than a Vsphere one. If you read the In-Tree Vsphere Cloud docs here (which is what rancher sets up as documented so I understand): https://vmware.github.io/vsphere-storage-for-kubernetes/documentation/high-availability.html#node-vm-failure. It says that the mode of operation on node failure should expects to transition the disks attached to the failed node in a shutdown state.. and is expecting them to remain attached to the failed node. It can some time for the migration to happen but ranchers vsphere orchestration is deleting the node before this is confirmed or verified as being complete and in turn it is triggering vpshere to delete all volumes still attached to it when deleted..
A few options to address might include:

an option in vsphere clusters to simply power off machines on scale down and power them on on scale up if they already exist only creating a new node when it hasn't been created yet..
for rancher OS provisioned clusters you know how many disks are part of the OS.. you could count the disks attached to the VM through the API and wait to delete it until only the OS disk(s) are left.
an option to disable the +/- button for vsphere clusters when the cloud provisioner is enabled.

Perhaps a scan of the VM for any drives attached matching a regex (not sure its correct) like kubevols/.*pvc.*vmdk could trigger delaying the delete operation up to a certain time and if after that delay it's still not clean then just power down the VM (don't delete) and display a message that we couldn't confirm it was safe to delete the VM due to potentially stuck PV's.

Looks like one of the two could be implemented to address this:

Node should be drained implicitly before deleting the node and VM
Delete node, Poweroff VM, wait for cloud provider to detach the volumes, then delete VM

Also, some relevant changes in the out-of-tree CSI provider on vSphere 6.7u3: https://github.com/vmware/vsphere-storage-for-kubernetes/issues/55#issuecomment-536084195

bump ..

There is a larger body of work that Rancher will be doing to support out of tree providers which will eventually happen so adding supporting for the vsphere out of tree provider is out of scope for this issue. To fix this now we are going to add support for vsphere clusters with the cloud provider to do a proper drain from Rancher while deleting the node which will let the in tree provider do it's work before we call the delete vm in vsphere. this method should work fine when using the - button and the delete from the ... menu.

@luthermonson awesome news!. Any idea when the fix will be released? Will it backport to the 2.4 release stream as well?

@skyscooby it's going into 2.4 first and I will forward port to 2.5+

reproduced in rancher/rancher:v2.48

Create a vsphere cluster w/ cloud provider, with multiple worker nodes
Create a VS storage class
Deploy a workload with a pvc from storage class -> See this scheduled correctly
Get yaml output of one of the node.management.cattle.io objects from the Rancher management plane
Delete the node the workload is running on -> Workload becomes unavailable with error:

Multi-Attach error for volume "pvc-819d0a49-3b22-4cba-b959-498ed955e2d6" Volume is already exclusively attached to one node and can't be attached to another

Upgrade to rancher/rancher:v2.4-3974-head
From Rancher management plane, get the yaml output of one of the node.management.cattle.io objects, and see the finalizer removed, and annotation added:

nodes.management.cattle.io/user-node-remove-cleanup: "true"

Deployed another workload with pvc from storage class to worker 1
Delete worker 1
Worker 1 gets cordoned and drained
Workload gets scheduled to another worker, volume is not deleted in vCenter, and is attached to the new worker node where workload is scheduled

Leaving open for testing in v2.5/master lines

Reopening until merged in v2.5/master

rancher/rancher:master-head 9124f6ed1
rancher/rancher:v2.5-head d87f58a
In each version:

Create vSphere cluster w/ cloud provider, create a storage class
Deployed a workload with pvc from storage class
Delete the worker node where the workload is scheduled
The deleted node gets cordoned and drained
Workload gets scheduled to another worker, volume is not deleted in vCenter, and is attached to the new worker node where workload is scheduled


[IE11/Edge] - ng6 - after toggling nav drawer, the sidenav contents are hidden and cant be restored

Bug Report
In IE11 or Edge , when opening the sidenav and toggle to the navigation drawer, the td-navigation-drawer-menu items are not displayed, and the sidenav's contents can not be restored . It works fine in Chrome
This is in<EMAIL_ADDRESS>and<EMAIL_ADDRESS>Screenshots or link to CodePen/Plunker/JSfiddle
Open normal stackblitz in IE11 or Edge
https://covalent.stackblitz.io/

What is the expected behavior?
Toggling to navigation drawer should display nav-drawer items, and not hide regular sidenav content items
Which version of Angular and Material, and which browser and OS does this issue affect?
Did this work in previous versions of Angular / Material?
Yes, worked in<EMAIL_ADDRESS>and<EMAIL_ADDRESS>Other information
From material changelog 6.0.0-beta.5

sidenav: The MatDrawerToggleResult class has been turned into an type.
The promise returned from open, close and toggle now resolves with the MatDrawerToggleResult type rather than the class.

Believe the change is affecting this line
https://github.com/Teradata/covalent/blob/feature/cli-6/src/platform/core/layout/navigation-drawer/navigation-drawer.component.ts#L178

Hi @emoralesb05  , first, thanks for the great library. Covalent has been a huge benefit for us,  we really appreciate your contributions
I saw this has a needs repro label on it.  Did you have any issue replicating? I'm able to consistently reproduce in both IE11 and Edge on stackblitz using the stock starter on Win 10 OS
https://covalent.stackblitz.io/
If you have any any update it would be much appreciated, this is the last thing holding us back from upgrading to ng6, and 2.0.0-beta.1  . Thank you

@emoralesb05  , I put some work into trying to nail this down. Looks like root cause is actually an with tdCollapse animation in IE11 and Edge . I'm not very familiar with how animations work or I'd attempt a fix  , but hope this helps

Awesome! Thanks for this.

Same bug on Safari!!

Issue is due to https://github.com/angular/angular/issues/23915
You can see in screenshot that element animation doesnt complete , hence display stays 'none'
I'll add comment in angular that issue is also present on IE11 and Edge

I have done using CSS & Its working on all browser, Below are my Code.

div.td-navigation-drawer-menu-content.ng-trigger.ng-trigger-tdCollapse{
display: block !important;
animation: gen_css_kf_101 150ms linear 0ms 1 normal forwards running;
}
div.td-navigation-drawer-content.ng-trigger.ng-trigger-tdCollapse.ng-animating{
display: block !important;
height: auto !important;
animation: gen_css_kf_101 150ms linear 0ms 1 normal forwards running;
}


Build error: 'uint32_t' does not name a type

Godot version
master (bee96aa5cb2e353cbbfc1cb346cf2c27e9748746)
System information
Arch Linux
Issue description
The updated rvo2 seems lacking a header include:
Building Godot (master) ...
scons: Reading SConscript files ...
Automatically detected platform: linuxbsd
Using linker program: mold
Building for platform "linuxbsd", architecture "x86_64", target "editor".
NOTE: Developer build, with debug optimization level and debug symbols (unless overridden).
Scons cache enabled... (path: '/tmp/godot-cache/master')
Checking for C header file mntent.h... (cached) yes
scons: done reading SConscript files.
scons: Building targets ...
[  7%] Linking Static Library tests/libtests.linuxbsd.editor.dev.x86_64.san.a ...
Ranlib Library tests/libtests.linuxbsd.editor.dev.x86_64.san.a ...
[ 24%] Linking Static Library modules/libmodule_multiplayer.linuxbsd.editor.dev.x86_64.san.a ...
Ranlib Library modules/libmodule_multiplayer.linuxbsd.editor.dev.x86_64.san.a ...
[ 24%] Compiling thirdparty/rvo2/rvo2_2d/Agent2d.cpp ...
In file included from thirdparty/rvo2/rvo2_2d/Agent2d.cpp:33:
thirdparty/rvo2/rvo2_2d/Agent2d.h:104:17: error: 'uint32_t' does not name a type
  104 |                 uint32_t avoidance_layers_ = 1;
      |                 ^~~~~~~~
thirdparty/rvo2/rvo2_2d/Agent2d.h:43:1: note: 'uint32_t' is defined in header '<cstdint>'; did you forget to '#include <cstdint>'?
   42 | #include "RVOSimulator2d.h"
  +++ |+#include <cstdint>
   43 |
thirdparty/rvo2/rvo2_2d/Agent2d.h:105:17: error: 'uint32_t' does not name a type
  105 |                 uint32_t avoidance_mask_ = 1;
      |                 ^~~~~~~~
thirdparty/rvo2/rvo2_2d/Agent2d.h:105:17: note: 'uint32_t' is defined in header '<cstdint>'; did you forget to '#include <cstdint>'?
In file included from thirdparty/rvo2/rvo2_2d/Agent2d.cpp:36:
thirdparty/rvo2/rvo2_2d/Obstacle2d.h:62:17: error: 'uint32_t' does not name a type
   62 |                 uint32_t avoidance_layers_ = 1;
      |                 ^~~~~~~~
thirdparty/rvo2/rvo2_2d/Obstacle2d.h:1:1: note: 'uint32_t' is defined in header '<cstdint>'; did you forget to '#include <cstdint>'?
  +++ |+#include <cstdint>
    1 | /*
thirdparty/rvo2/rvo2_2d/Agent2d.cpp: In member function 'void RVO2D::Agent2D::insertAgentNeighbor(const RVO2D::Agent2D*, float&)':
thirdparty/rvo2/rvo2_2d/Agent2d.cpp:370:22: error: 'avoidance_mask_' was not declared in this scope
  370 |                 if ((avoidance_mask_ & agent->avoidance_layers_) == 0) {
      |                      ^~~~~~~~~~~~~~~
thirdparty/rvo2/rvo2_2d/Agent2d.cpp:370:47: error: 'const class RVO2D::Agent2D' has no member named 'avoidance_layers_'; did you mean 'avoidance_priority_'?
  370 |                 if ((avoidance_mask_ & agent->avoidance_layers_) == 0) {
      |                                               ^~~~~~~~~~~~~~~~~
      |                                               avoidance_priority_
thirdparty/rvo2/rvo2_2d/Agent2d.cpp: In member function 'void RVO2D::Agent2D::insertObstacleNeighbor(const RVO2D::Obstacle2D*, float)':
thirdparty/rvo2/rvo2_2d/Agent2d.cpp:409:22: error: 'avoidance_mask_' was not declared in this scope
  409 |                 if ((avoidance_mask_ & nextObstacle->avoidance_layers_) == 0) {
      |                      ^~~~~~~~~~~~~~~
thirdparty/rvo2/rvo2_2d/Agent2d.cpp:409:54: error: 'const class RVO2D::Obstacle2D' has no member named 'avoidance_layers_'
  409 |                 if ((avoidance_mask_ & nextObstacle->avoidance_layers_) == 0) {
      |                                                      ^~~~~~~~~~~~~~~~~
scons: *** [thirdparty/rvo2/rvo2_2d/Agent2d.linuxbsd.editor.dev.x86_64.san.o] Error 1
scons: building terminated because of errors.

CC @smix8
Steps to reproduce
Build godot.
Minimal reproduction project
N/A

Got the same issue, cannot build master too.


Change likely dropped in the last merge

Bring back changes from xla-upstream-1-23

jenkins: retest this please


add vertex prod features

chat vertex async
vertex stream
vertex full generation info
vertex use server-side stopping
model garden async
update docs for all the above

in follow up will add
[] chat vertex full generation info
[] chat vertex retries
[] scheduled tests

cc @lkuligin

@baskaryan Not sure if related to the stop changes, but now getting:
(No issues pre .304 with the same code)
_TextGenerationModel.predict() got an unexpected keyword argument 'stop_sequences'

/langchain/llms/vertexai.py\", line 100, in _completion_with_retry\n    return llm.client.predict(*args, **kwargs)\nTypeError: _TextGenerationModel.predict() got an unexpected keyword argument 'stop_sequences

@ventz which version of google-cloud-aiplatform are you using? please, make sure you've updated to 1.31.1  at least

@lkuligin Thanks -- just came back here to post exactly that :)
Hopefully this helps someone else!


scala 2.13.0 release

Would it be possible to release a scala 2.13.0 compatible jar?
There are releases for many RC and Milestone builds (https://mvnrepository.com/artifact/org.typelevel/macro-compat)

See the discussion on #85 from here: https://github.com/milessabin/macro-compat/pull/85#issuecomment-500201690.


[Quest] Make Login & app id fields single line.

Describe the bug

The current app id and login username & password fields are currently multiline.
They should be a single line that will make pressing enter move to the next field instead of moving to the next line.

To Reproduce
Steps to reproduce the behavior:

launch the FHIR core app
Fill in the application id then press enter. Notice the cursor moves to the next line instead of the next field.

Expected behavior

launch the FHIR core app
Fill in the application id then press enter. The cursor should move to the next field instead of moving to the next line

Screenshots

Additional context
Add any other context about the problem here.

I'd like to try this out


Don't autodoc the PromptDict

It's just reflecting back the built-in methods for the parent class in Results documentation:

done


Question: architectural details for a simple textx editor

Hello @danixeee (also hello @igordejanovic)!
First of all I want to thank you for your excellent work!
I am playing around to provide a minimal editor support for small project files. However, I am not an expert with editor architectures.
I would like to get a hint (a link into your textX-languageserver architecture), how to computational efficiently access information to be able to provide, e.g, parse trees for "half-parsed" models (e.g., when I just type a model and the second half is missing, which makes the model text not parsable completely).
I would also like to have your expertise, if the experiment could possibly yield a usable editor...
https://github.com/goto40/textX-gui-experiment/blob/master/docs/editor.md
I am very unsure about it, is some points is seems not to be the right way...

Hello @goto40.
textX-languageserver implements Language Server Protocol and its basic idea is to separate language support (code completion, diagnostics, ...) from different editors.
By the end of November, we will have python Language Server Framework (LSF) and able to create language servers in just few lines of code (maybe it will be interesting for you).
Also, plan is to rewrite textX-languageserver and create it on top of LSF. You can take a look at current features.
I think you are not able to work with "half-parsed" models, since mm.model_from_file and mm.model_from_str methods will raise exception for non valid models.

Hi!
Thank you for your response. I think I drop the experiment I started. I am looking forward to see how your project evolves...
I would like something simple like metamodel.open_editor()... But maybe this needs some more time.
Best,
Pierre

Hi Pierre,
Our plan is to create language server for textX from scratch using python language server framework (after becoming open-source). That way we will be able to focus on features for textX and not worry about language server protocol implementation.
We will create a new repository with CI/CD and backlog issues and your contribution will be very helpful since you showed interest in this field.
Plan is to integrate viewX into the extension, also. (great project; try it out if you didn't already)
Best,
Daniel

Sounds very good. Maybe you could also have a look on multi file models and
multi Metamodell support...
Igor Dejanović<EMAIL_ADDRESS>schrieb am Di., 16. Okt. 2018,
17:26:

Hi all,
We have a new developer on board (@AirmiX https://github.com/AirmiX)
who will be working on a minimal web-based editor for textX based languages
as a part of his master thesis.
The idea is to produce/generate an editor with syntax
highlighting/folding/outline, and optionally communication with LSP for
textX https://github.com/textX-tools/textX-languageserver if available,
from the textX grammar. The editor should be simple enough to be usable
with little, if any, setup. The goal for this editor is to be used during
grammar development but also to be usable as embeddable editor for desktop
application and, of course, an editor for web apps.
—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/textX-tools/textX-languageserver/issues/5#issuecomment-430282388,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AQGtWl_NvOtyI8p1fpDovl5X-TQV2CNVks5ulfq9gaJpZM4XbReQ
.

Yes, multi-meta-model support is also on the table :)

Sounds great.
@AirmiX Welcome and happy coding! :)

Thank you! :)


KAFKA-10199: Add missing catch for lock exception

The state directory throws a lock exception during intitialisation if a task state directory is still locked by the stream thread that previously owned
the task. When this happens Streams catches the lock exception, ignores the exception, and tries to initialize the task in the next exception.
In the state updater code path we missed to catch the lock exception when Streams recycles a task. That leads to the lock exception thrown to the exception handler which is unexpected and leads to test failures.
More detailed description of your change,
if necessary. The PR title and PR message become
the squashed commit message, so use a separate
comment to ping reviewers.
Summary of testing strategy (including rationale)
for the feature or bug fix. Unit and/or integration
tests are expected for any behaviour change and
system tests should be considered for larger changes.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

No unit test a la shouldRetryInitializationWhenLockExceptionInStateUpdater for this?

Good point! Will add the unit test!

build failures are unrelated


Add feature to test main branch

Currently if we want to introduce a new change on ppocgen.com we test it locally and then deploy it to production. To add a new integration environment we should consider adding a pre-production or staging environment.
I propose having another DNS called dev.ppocgen.com or similar that points to this new environment that always has the changes made in the main branch. In that way we have two production-ready options to compare and see how it behaves before releasing a new version

We've deployed dev.ppocgen.com !!! With this we can test changed before they are production ready


Adding my Spotify Playlist

Hacktoberfest open source contribution.

Update the forked stream first and then make the required changes.


[deviceupdate] Migrate to Samples Workflow V2

This issue is part of the Azure SDK for JavaScript sample quality effort.
The new sample tool (dev-tool samples publish) has been made available to all package maintainers, and we would like to migrate all packages to the new system.
What do I need to do?
This issue is to track migrating all packages within the deviceupdate folder to the new layout:

[ ] iot-device-update

There are a few things to do to resolve this issue:

Attempt to follow the steps in the migration guide for each of the packages. Any problems can be reported to me, and I'll be available to assist in getting everything moved over.
Look at https://github.com/Azure/azure-sdk-for-js/issues/14180, the main tracking issue for this work, and see if your package has any problems listed in the issue description. We would like to fix all of these issues that we are able to fix as we are migrating samples to the new system.

If for any reason you think that this issue should not apply to this service/package, please get in touch with me so that we can discuss the package's needs.
Thank you!

Assigning to @dpokluda who owns this package

This has been resolved


New option -s was added for log minimization

New option -s was addedd for logs minimization.
In case if this option is provided trona will show only critical messages:

In case if user interaction is needed (e.g. promt for degrade script execution)
In case of critical error (e.g. not able to create evolution table)
Resolve #204

Please check #227. This PR should be merged only after it.


[Bug] is_write_index is spelt incorrectly (is_wtite_index)

Describe the bug
The mapping for the is_write_index property is spelt incorrectly.
located:
https://github.com/elastic/terraform-provider-elasticstack/blob/465c336e59950a781a59e9b03b77bf1d6e9a5a81/internal/models/models.go#:~:text=IsWriteIndex  bool                   `json%3A"-,is_wtite_index,-%2Comitempty"`
To Reproduce
resource "elasticstack_elasticsearch_index" "my_index" {
  name     = "my_index"

  alias {
      name           = "my_alias"
      is_hidden      = true
    }
  }
}

Expected behavior
An index is created with an assigned alias
Screenshots

Versions (please complete the following information):

OS: windows
Terraform Version 1.1.16
Provider version v0.0.3
Elasticsearch Version 8.0.0

hey @mikeclearbank thanks for the bug report!
I will address this right away

PR is up #79 and will be merged as soon as CI pass

New version of the provider was released v0.3.1


Upgrade assistant converting HTTP types in function body

Upgrade Assistant should not be changing types for objects that are defined within user code. Today, if you run upgrade assistant with code like the following...
            HttpRequestMessage request = new HttpRequestMessage(HttpMethod.Post, "someurl")
            {
                Content = new StringContent("foo")
            };

... you will have it converted to using HttpRequestData, though it's still attempting to use the HttpRequestMessage constructor, so it gets a build error....
            HttpRequestData request = new HttpRequestData(HttpMethod.Post, "someurl")
            {
                Content = new StringContent("foo")
            };

Upgrade Assistant should not touch that type.
This may be obviated by https://github.com/Azure/azure-functions-dotnet-worker/issues/1838 since with that, we wouldn't be introducing HttpRequestData where it wasn't previously. But there might be a more general pattern here. Even with that, a user-defined HttpRequestMessage should stay as-is.

PR for issue: https://devdiv.visualstudio.com/DefaultCollection/DevDiv/_git/UpgradeAssistant/pullrequest/504090

PR has been merged in


Communicate with peripherals already connected to the OS

Thanks for your hard work @yanshouwang! I'm considering using this plugin for an app I'm developing, but I want to know if the following scenario is supported or planned: As a central, connect to a peripheral that's already connected to the OS. I.e. I don't need to run discovery from the app, I just need to communicate with devices that are already connected.
I perused through the API and unless I missed something it doesn't seem like there's a way of doing it. Is there? Or do you have plans to implement it?

I think I can add a method like retrieveConnectedPeripherals in the next version.

The 6.0.0-dev.0 has released, the retrieveConnectedPeripherals is supported on all platforms except Windows.

Released in 6.0.0


/usr/bin/xvfb-run: line 181: 46303 Segmentation fault

[Environment]
1,OS: centos 8
2,vf_plusglshader has successfully installed.
3,No video card and installed xvfb
[Command]
$ xvfb-run -a --server-args="-screen 0 1280x720x24 -ac -nolisten tcp -dpi 96 +extension RANDR" \ ./ffmpeg -v debug \ -i 0.mp4 \ -vf plusglshader \ -y output.mp4
[Output]

[Question]
Why the transcode failed?

`yum install -y glew glew-devel
yum install -y glfw glfw-devel
yum install -y mesa-dri-drivers
yum install -y libGLEW*
yum install -y mesa
yum install -y mesa-libGLU mesa-libGLU-devel
yum install -y libXfont*`
Some dependency is not installed.
Transcode complete successfully after installing all dependencies!

good~


docs: point users to docs.cilium.io in GH docs

Since users might read the github docs we will provide them a warning
in the documentation, that will not show up in the generated html code
but it shows up in github, to point them into the stable documentation
in http://docs.cilium.io
Signed-off-by: André Martins<EMAIL_ADDRESS>

This change is

test-docs-please

should we backport it?

test-docs-please

@aanm I'm OK with this but needs an addition to undo this when we branch off a release branch from master

@tgraf this was intend to be a message that shows up in github but it does not show up in readthedocs regardless of which version it is. Which message will be equally informative and could stay on all versions that would push the users using readthedocs instead of github?

Which message will be equally informative and could stay on all versions that would push the users using readthedocs instead of github?

OK. The "latest" was misleading to me. I would reword it to:
+    WARNING: You are looking at unreleased Cilium documentation.
+    Please use the official rendered version released here:
       http://docs.cilium.io

test-me-please


arangoexport

my environment running ArangoDB
I'm using the latest ArangoDB of the respective release series:

[ ] 2.8
[ ] 3.0
[ ] 3.1
[* ] 3.2
[ ] 3.3
[ ] self-compiled devel branch

Mode:

[ *] Cluster
[ ] Single-Server

Storage-Engine:

[ ] mmfiles
[ *] rocksdb

On this operating system:

[ ] DCOS on

[ ] AWS
[ ] Azure
[ ] own infrastructure

[* ] Linux

[ ] Debian .deb
[ ] Ubuntu .deb
[ ] SUSE   .rpm
[ *] RedHat .rpm
[ ] Fedora .rpm
[ ] Gentoo
[ ] docker - official docker library
[ ] other:

[ ] Windows, version:
[ ] MacOS, versio

When I use arangoexport to make a mistake:

My vertex collection has 190 million data,and my edge collection has 1 billion data.

This is my collection of vertexs：

and this is my collection of edges:

this is the coordinator log:

this is the dbserver log:

this is a vertex attribute:

and this is a edge attribute:

Is there anything else I can offer?


[WIP] Upgrade AWS SDK to v2 in repository-s3

Description
Upgrade AWS SDK to v2 in repository-s3 plugin
Related Issues

#7391

Check List

[ ] New functionality includes testing.

[ ] All tests pass

[ ] New functionality has been documented.

[ ] New functionality has javadoc added

[ ] Commits are signed per the DCO using --signoff
[ ] Commit changes are listed out in CHANGELOG.md file (See: Changelog)

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Closing as a dup of https://github.com/opensearch-project/OpenSearch/pull/7372, we're doing the upgrade in one go.


Exponentially Decreasing Attention?

Just a thought. Have you considered or tried some sort of variation where the attention still covers the entire image, but at lower granularity the further away it is from the target pixel?
Just thinking that this is how the human attention works, and perhaps it could be the best of both worlds between your current tiled attention approach and full attention

Yes, I have thought about it. Unfortunately we would have to aggregate the tiles somehow, to have a pixel to tile attention out-side its scope. And ToMe has show that aggregation make images bad and LoRAs behave weirdly... And It would require to compute the score matrix and reasembling things before fixing the tiles. It's messy. I tried.
Seems way easier to just restrict global attention altogether. And it works.


Automated cherry pick of #8461: fix(glance): allow public and private operations of guestimage

Cherry pick of #8461 on release/3.5.
#8461: fix(glance): allow public and private operations of guestimage

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/3.5@b033b6c). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff              @@
##             release/3.5   #8462   +/-   ##
=============================================
  Coverage               ?   3.52%
=============================================
  Files                  ?     304
  Lines                  ?   62594
  Branches               ?       0
=============================================
  Hits                   ?    2205
  Misses                 ?   60236
  Partials               ?     153

Flag
Coverage Δ

#aFlag
3.52% <0.00%> (?)

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

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 b033b6c...64ba460. Read the comment docs.

/lgtm
/approve


Add serialisation/deserialisation for components and models

Description
Adds serialisation of models.
TOML was chosen as an output format as it provides human-readable for nested data and JSON failed to be able to deserialize timeseries with NaN's in it without additional effort. serde_json converts the NaN to a null, but then when deserialising the null causes a incorrect type error.
I couldn't see an easy fix for that so I moved on with life.
Checklist
Please confirm that this pull request has done the following:

[x] Tests added
[x] Documentation added (where applicable)
[ ] Changelog item added to changelog/
[x] .pyi files updated

Closes #3

@znicholls FYI This is what it took to get model serialisation working

Yep nice, exactly what we want I think!


Add description field for Job Queue items

When viewing the cron jobs, it so happens that often there are many job records that are very similar but are used for slightly different purposes.
Would be really helpful if the user could add a description to cron jobs / Job Queue

Hi @StephenOTT
For now jobs list on 2.0 version looks like on screenshot. How would you like to make it better? I think the job names speak for themselves.


Update Jinja2 minimum version (jinja2 v3.x compatibility check)

Is your feature request related to a problem?
Flask has increased minimum Jinja2 version to 3.x. To use Flask 2.x a separate deployment environment would be need to me maintained.
Describe the solution you'd like
Update Jinja2 version.

Hi @Jobrowncsm! I don't think bumping the minimum version is appropriate here, but expanding the maximum version to v4. I need to double check none of the 2->3 changes break the way pyinfra is using the library but I don't believe this is the case. I can then bump it up to allow v3.x as well as 2.x.

I've just released the loosened requirement in v1.4.8.


Added more configuration options for sessions

Coverage remained the same at 47.045% when pulling 19e5d1df0b570bfe38a0d70b74cdf2ff744a6ed4 on release/0.13.1 into d84f27c1cebcb28f6f496700fa3539d7ca38bc1d on master.


Naming property Length and using IsNumber isn't valid

If I name a property Length, and then use IsNumber inside AssertThat on it, like this:
https://github.com/fehrm/LengthEA/blob/d9839bdb3e70723c236b9cbadec97680348c4853/LengthEA/Models/TestViewModel.cs#L11-L12
[AssertThat("IsNumber(Length)")]
public string Length { get; set; }

It doesn't validate it correctly in the clientvalidation when validated. If I rename the property to eg. LengthT it works.
I added a sample project to my github, https://github.com/fehrm/LengthEA/ with a very simple testcase for it. It behaves the same if I don't use input type number.
This is using EA 2.9.2, jQuery valid 1.16.0 etc. Here is a link to the packages.config

Hi,
The problem is due to the Length property being overridden by a built-in method of the same name, when model context (within which expression is to be evaluated) is being constructed. Proposed fix here: https://github.com/jwaliszko/ExpressiveAnnotations/commit/f39e43157f56d7283327aef468ce2f4a8a5a046d.
Thanks for raising this defect.

I agree that it is not an optimal name 😄 I think the proposed change is great though, it will enable the developer to be informed and change the name to a more suitable one. As I did for my project.
Thanks for the prompt reply!

The name is as good as any other, it's just EA which failed to handle it. I've just improved the fix: https://github.com/jwaliszko/ExpressiveAnnotations/commit/b2bd4f674660d16029f3d83c5d92afc75e492fca.
This one is able to handle your original naming as expected - without any warnings and changes at your side. Now, only the essential set of methods, actually used in expression, is registered within model context. Previously all available methods were added to the context - hence bigger chance to have naming conflicts.
Finally, if naming conflict with some method actually occurs, e.g. in expression like Length(Length), exception is thrown as client-side compatibility is broken.


Can not install

I could not install plugin on the latest cordova builds.

@fmbuthia I didn't have any problems installing it. Could you provide more details/logs as it's impossible to help you using data you have provided here.

I tried to install using github url like - cordova plugin add https://github.com/ratson/cordova-plugin-admob-free. I was still able to install the other way though - cordova plugin add cordova-plugin-admob-free. So this is not a big deal thanks for the plugin.

I wasn't able to install the plugin using github url also, it have to do something with missing www/admob.js file. I don't really know why was it removed from the repo. @ratson?
cordova plugin add https://github.com/ratson/cordova-plugin-admob-free.git
Sample output:
Failed to install 'cordova-plugin-admob-free':Error: Uh oh!
ENOENT: no such file or directory, open '/Users/kamilrykowski/workspace/party_game/plugins/cordova-plugin-admob-free/www/admob.js'
    at Object.fs.openSync (fs.js:558:18)
    at Object.fs.readFileSync (fs.js:468:33)
    at install (/Users/kamilrykowski/workspace/party_game/platforms/android/cordova/lib/pluginHandlers.js:173:36)
    at ActionStack.process (/Users/kamilrykowski/workspace/party_game/platforms/android/cordova/node_modules/cordova-common/src/ActionStack.js:56:25)
    at PluginManager.doOperation (/Users/kamilrykowski/workspace/party_game/platforms/android/cordova/node_modules/cordova-common/src/PluginManager.js:117:20)
    at PluginManager.addPlugin (/Users/kamilrykowski/workspace/party_game/platforms/android/cordova/node_modules/cordova-common/src/PluginManager.js:147:17)
    at /Users/kamilrykowski/workspace/party_game/platforms/android/cordova/Api.js:254:18
    at _fulfilled (/Users/kamilrykowski/workspace/party_game/platforms/android/cordova/node_modules/q/q.js:834:54)
    at self.promiseDispatch.done (/Users/kamilrykowski/workspace/party_game/platforms/android/cordova/node_modules/q/q.js:863:30)
    at Promise.promise.promiseDispatch (/Users/kamilrykowski/workspace/party_game/platforms/android/cordova/node_modules/q/q.js:796:13)

Error: Uh oh!
ENOENT: no such file or directory, open '/Users/kamilrykowski/workspace/party_game/plugins/cordova-plugin-admob-free/www/admob.js'

It won't work as www/ is generated from src/js/ via babel.


resource/repository_collaborator: revert response handling introduced in #424

Closes #469
Notes

Previously, handling was added to validate the response returned from the Github API when an attempt was made to add a collaborator but this proved to be problematic and breaking

Output of acceptance tests:
--- PASS: TestAccGithubRepositoryCollaborator_basic_personal (13.40s)
--- PASS: TestAccGithubRepositoryCollaborator_basic (26.76s)
--- PASS: TestAccGithubRepositoryCollaborator_caseInsensitive_personal (27.61s)
--- PASS: TestAccGithubRepositoryCollaborator_caseInsensitive (23.75s)

Would you consider backporting this to the 2.x series? It seems like 3.x will take a little while longer and this is a small change. It would help me out as we are getting this exact error.


Analytics

updated version of https://github.com/aerogear/aerogear-unifiedpush-server/pull/540

@corinnekrych @edewit @danielpassos w/ https://github.com/matzew/aerogear-unifiedpush-server/commit/86134ea31f563f662441aabd4cef3107acba6594 we have a PUT endpoint for storing the "app opened due to push" ack.
A little CURL I did is:
curl -u "super:secret" -v -H "Accept: application/json" -H "Content-type: application/json" -X PUT http://<IP_ADDRESS>:8080/ag-push/rest/registry/device/pushMessage/eac8c56c-209e-4f20-95e4-c59404947bf6

I have the endpoint under /registry/device for a few reasons:

it's data sent from the actual device
and no need to add another http basic exception

@secondsun ^
Now that we have a separate endpoint for the metrics - do you think we need a PUT for "updating" the registration metadata? or should the mobile clients be executing against our POST registration API for both: initial registration and continued/updated registration ?

@matzew That kind of depends ;)
Doing a bit of research (IE bothering you in IRC) on Android and SimplePush the device installation id used by the server is provided by the client.  On Android we repeatedly call .register (because of a laundry list of technical reasons).  This invokes a POST request every time.  Because we know the ID and because multiple calls to register always leave the server in the same state we should probably be using PUT for both registration AND updating.
This is mostly a round about way of saying I think we should not create a second method for android setting updates and possibly, in the future, switch to using PUT instead of POST.

@secondsun given the amount of tickets we have for 1.1.0, I'd vote to not do that change for 1.1.0
I see your point, but being pragmatic it does not hurt to do POST for both: initial registration and updating registration data.
I have filed a future JIRA: https://issues.jboss.org/browse/AGPUSH-1399

@matzew  +1 for pragmatism

I am +1 for the separate endpoint for the metric analytic update.
regarding the POST/PUT reg update can be looked later on I guess
:+1:

:+1:  for the separate endpoint


Layout/IndentHeredoc issue on multi-line method calls

When a squiggly heredoc is given on the second (or higher) line of a method argument, Layout/IndentHeredoc wants me to indent the heredoc two spaces more than the last line of method arguments. See this example code:
# frozen_string_literal: true

puts 'A very long argument that forces the 2nd argument onto the next line',
     'Here we go:', <<~TXT
  This is intended
  to be two-space indented.
  Did I find a bug?
TXT

RuboCop is happy (but I am not) if I indent like this:
# frozen_string_literal: true

puts 'A very long argument that forces the 2nd argument onto the next line',
     'Here we go:', <<~TXT
       This was intended
       to be two-space indented.
       Did I find a bug?
TXT

Please note that there are no offenses for the first example if I use << or <<- instead of <<~.

Expected behavior
I would expect to have no offenses for the first example above.
Actual behavior
Offenses:

test.rb:5:1: C: Layout/IndentHeredoc: Use 2 spaces for indentation in a heredoc.
  This was intended ...
^^^^^^^^^^^^^^^^^^^

1 file inspected, 1 offense detected

Steps to reproduce the problem
See the first code example above.
RuboCop version
0.70.0 (using Parser <IP_ADDRESS>, running on ruby 2.6.3 x86_64-darwin18)

I thought found a way to change the cop, but then it broke this untested, but valid indentation:
a_block do
  calling_a_method(
    :very_long_line,
    <<~RUBY
      foo
    RUBY
  )
end

RuboCop is also happy with the following placement of the end marker. Just to give you another option …
# frozen_string_literal: true

puts 'A very long argument that forces the 2nd argument onto the next line',
     'Here we go:', <<~TXT
       This was intended
       to be two-space indented.
       Did I find a bug?
     TXT

The bug still exists on the most recent version
0.74.0 (using Parser <IP_ADDRESS>, running on ruby 2.6.3 x86_64-darwin18)